Extract and Parse JSON in Callbacks at Client Side

I am facing an issue that when I try to extract and parse JSON received from the server at the client side using on(method: callback:) I am unable to parse the payload.
I am using getArgument<T: Decodable>(type: T.Type) method in ArgumentExtractor but I am getting nil values even though I am receiving the payload. I am parsing the payload in the following way:
self.chatHubConnection!.on(method: "getNotification", callback: { (payload: ArgumentExtractor?) in

            let response = try! payload?.getArgument(type: SocketResponse?.self)
            //here response is nil

        })

Following is the screenshot of the above code with debug logs for better understanding.
https://drive.google.com/file/d/1E4GCguhff6U6MW1lAUrvsqt4vFQHwZL1/view?usp=sharing

This seems like a mismatch between the Json payload and SocketResponse type definition. How does your SocketResponse type looks like?
Unrelated note: it is much easier and cleaner to use the helper method for handling callbacks instead of using the overload that takes ArgumentExtractor:
self.chatHubConnection!.on(method: "getNotification", callback: { (response: SocketResponse) in
            // code using response
        })

@moozzyk  My response coming from web API. So we have used to for that.
Response is like
[{“restaurantId":159,"menuId":50,"isMultipleOrder":false}]
So how can I get the values?

I  am a bit confused about the web API. You need to have a SignalR Server that the client receives response from. As requested when opening the issue - please attach logs to let me see what the client receives. Otherwise I won't be able to help.

As you can see in the image, I want projectId, projectTaskId from arguments but as this is Dictionary coming from web so
I am unable to parse it to JSON format. So my question is how can I get these values.

You need to create a class/struct that conforms to the Decodable protocol and matches your json. This is pretty much the same deal as on the server side (or in C# client). In this particular case you would need to create something like this (in this case the struct is simple and automatically conforms to the Codable protocol because all the members have types that conform to the Codable protocol, the name of the type should not matter but the name of the members and their types need to match the payload):
struct ProjectDetails {
    let projectTaskId: Int
    let projectId: Int
    let isTimerStopped: Bool
}

Then you should be able to receive your data as:
self.chatHubConnection!.on(method: "SignalRRefreshTimeTracking", callback: { (projectDetails: ProjectDetails) in
            // code using response
        })

You can find more on this in the F.A.Q..

@moozzyk  Okay Thanks


Update README

Update cluster info in README:
screenshot:

@eobbad not essential right?

too many changes since then worth it to resolve?


Travis does not run on Node v6 at all

@tryghost/html-to-mobiledoc has a dependency to jsdom (13.2.0).
jsdom dropped support for Node v6.
The problem is that no tests are executed for Node v6, because the installation fails. Is that fixable? Have not looked into it.

There's no hard dependency on the latest version of jsdom, we could pin the last version with node 6 support until the 6 LTS is dropped?

Does this also impact Ghost seeing as we use it for html input parsing?

Does this also impact Ghost seeing as we use it for html input parsing?

Ghost has @tryghost/html-to-mobiledoc as optional dependency :)

Given that node v6 support ends in ~6weeks, we should find the quickest and simplest solution, i.e exclude the v6 tests. I have no expectation that html-to-mobiledoc ever works on node 6.

closed by 6eed3eb4ef0b93e1be0ac750fe984dd0bda8847e


Consider dropping command-line configs for tools that have config file support

For nearly all of the tools we run on the user's behalf we provide a way to specify command-line arguments for the user. The problem with this is if we switch to how we run these tools to use e.g. library support, then continuing to support the command-line arguments becomes brittle as we then have to translate from CLI to Python calls.
It might be best long-term to promote that users use configuration files instead and then stop trying to support command-line arguments as well (e.g. use pylintrc instead of python.linting.pylintArgs). This also would help simplify error reports as users regularly file issues about the command-line arguments not working and it turns out they didn't break everything up as appropriate in the array (e.g. writing ["--flag arg"] instead of ["flag", "arg"]).

As we are moving towards separate tools extensions, this is out of date.


Removed GetAwaiter on Response

Prerequisites

[X] I have written a descriptive pull-request title
[X] I have verified that there are no overlapping pull-requests open
[X] I have verified that I am following the Nancy code style guidelines
[X] I have provided test coverage for my change (where applicable)

Description
Removed the GetAwaiter method on Response. It was accidentally included when we added the new route syntax.
Closes #2518

@khellang pushed


Training process goes well, but evaluation results are either 0% or 100 % success rates.

Hi, thanks for your awesome work. I follow the steps in README and successfully train the policy network. The loss decreases to a very low level. However, as shown in figure, the evaluation results show that max success rates are either 0% or 100% on benchmarks like kitchen_light_on-v3, kitchen_ldoor_open-v3 and kitchen_sdoor_open-v3. Is this reasonable? Or do you have any suggestions?

Hello,
Yeah this is not expected, while there can be variability between checkpoints only 0% or 100% is not what I observed.
First, could you share the exact command you are running? How many demos/what representation/etc.
Also, you followed the installation setup instructions in the ReadMe here, correct? Did you try the "Verifying Correct Installation" commands, and observe the expected numbers listed there (~60% for R3M and ~30% for CLIP)? If you are not getting those numbers then something likely is wrong in installation. For the Franka Kitchen did you make sure to add the line FIXED_ENTRY_POINT = RANDOM_DESK_ENTRY_POINT  in mj_envs to use the Randomized Desk?

Yes, this looks like an environment version mismatch issue. Please follow the Readme instructions exactly like Suraj mentioned to get the correct version of environments for evaluation.

Hi Suraj, thanks for your quick reply. I used the exact same commands as you suggested in the README for verifying correct installation. I follow your suggestions and add the line FIXED_ENTRY_POINT = RANDOM_DESK_ENTRY_POINT in mj_envs, and now that evaluation results look normal. Is it similar to the results of your experiment?

Yup that looks right to me!


update Dockerfile's nginx version to 1.20.1+

A security issue in nginx resolver was identified, which might allow an attacker who is able to forge UDP packets from the DNS server to cause 1-byte memory overwrite, resulting in worker process crash or potential other impact. (CVE-2021-23017)
The nginx:1.19.5 version in the Dockerfile is vulnerable to CVE-2021-23017 which is marked as critical in Amazon Linux Security Center. It is reasonable to also update the sample in order to prevent vulnerable versions from being used again.
Reference:

NGINX security advisory (CVE-2021-23017)
NVD - CVE-2021-23017
Amazon Linux Security Center - ALAS-2021-1507

I agree with this. It's better to avoid using vulnerable versions.


Feature/unauthorized user can view the contacts relationships

Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata

**lurch:attach W-024269

Tracking W-024269


Build push a docker image to dockerhub on release

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.90%. Comparing base (f507b4a) to head (5166f12).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##             main     #336    +/-   ##
========================================
  Coverage   97.90%   97.90%
========================================
  Files          54       54
  Lines        2389     2389
  Branches      487      296   -191
========================================
  Hits         2339     2339
  Misses         34       34
  Partials       16       16

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Awesome. Did you test this workflow somewhere @charalamm? (No is fine, I'd just like to know)

@dionhaefner I tested it only when pushing on a branch and without the commit tag. Now that I am thinking about it I could test it in a throwaway repo. I will do that and then we can merge I guess.

We could also merge this and do a post-release to test the deployment. Not quite as elegant but if it saves you work I'd be up for it.

Hello @dionhaefner I have tested it in separate repo and it works and uses the release tag as the docker image tag. It also updates the latest tag image. I am merging if that is ok
Tests are failing for irrelevant reasons as far as I can see

Sounds great, thanks @charalamm !


Disable 1ds codeql

From Azure sdk for js repo >

All the other languages have disabled codeql and we are now hitting blocking issues when running codeql on macos (https://dev.azure.com/twcdot/Data/_workitems/edit/138580) and even on the other OS's the configuration causes the builds to run for a long time.

https://github.com/Azure/azure-sdk-for-js/pull/30277

No changes needing a change description found.

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/1131/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/1131/


ConnectionTimeout ignored at certain times

At times HikariCP ignores the connectionTimeout setting. Not sure what the reason is or how to diagnose this.  I'm using version 2.4.3 and have connectionTimeout set to 2000ms. This is what happens from time to time:
Caused by:
 java.sql.SQLTransientConnectionException: UserShardPool:jdbc:mysql://dbuser.mydomain.com:3306/UserShard - Connection is not available, request timed out after 38104ms.
at com.zaxxer.hikari.pool.HikariPool.getConnection(HikariPool.java:195)
at com.zaxxer.hikari.pool.HikariPool.getConnection(HikariPool.java:147)
at com.zaxxer.hikari.HikariDataSource.getConnection(HikariDataSource.java:83)

Any guidelines on how to diagnose this?

Many possibilities for this, gathered from issued reported and resolved over time. in no particular older :)

it can be stupid mistake and instantiate the data source multiple times.
properties are not passed 'correctly' to hikaricp, usually by framework layer.
3  older driver and or hikaricp, use latest stable release.
apparent connection leak.
driver does not correctly implement or support Connection.setNetworkTimeout()
check value of wait_timeout on the mysql server.
socketTimeout
enable tcpKeepAlive
driver does not 'correctly' implement Connection.isValid()
use efficient connectionTestQuery

Hope this hels

Thanks for feedback Nitin. The problem we faced proved to be more subtle.
When thread starvation occurs HikariCP 2.4.3 will report "Unusual clock change" and evict all connections from a pool. It will then start re-creating those connections. It can take a while. Since the system is under heavy load there is a time window when all connections have been removed and none have been added yet. If a getConnection() comes in that window you can get "Connection is not available, request timed out" exception. Version 2.4.4-SNAPSHOT seem to improve handling of this scenario significantly.
I'm still going through 2.4.4-SNAPSHOT's code to see if this scenario (emptying pool and filling it up again) can occur in any other execution path.
For future reference, here's a setup that we used (in a form of answer to Nitin's questions):
1 + 3. We used latest versions of Spring Boot (1.3.4), HikariCP (2.4.3), Mysql driver (5.1.38) and Hibernate. Connection pool is instantiated once and held as a singleton inside DI container.
2. Double checked that - HikariCP reports settings as they should be (found one bizarre setting value that turned out to be deprecated field [see Issue #587])
4. Switched leak detection on - no issues here
5. Used latest mysql driver with latest version of mysql 5.5 branch - since this is a large scale system (range sharding implemented over 100 db clusters [one  connection pool per cluster]) we have quite efficient monitoring in place.My team validated that this was not an effect of networking issues nor db-side problems.
6. Latest mysql driver seems to use work well.
7. No issues here so far (timeouts work as expected [connectionTimeout aside] and there's always a large amount of spare connections in the pool at any time).

HikariCP 2.4.4 has been released.  It contains a fix explicitly for the issue of thread starvation triggering the clock shift detection, and subsequent eviction. Let us know if you observe any additional unusual behavior.


Labeling outlines as a semantic segmentation class

Hi!
I have a question about how I can label edges of cells in my pictures.
Are there any functions in cellpose for it?
And if not, is it possible to use outlines from .npy file for it?

Yes, you can export an image that has only the cell outlines. There is information on how to do this on the docs here.
You could use the .npy file to do it if you've already ran CP on your images; the same information is in the file


Bugfix/placeholder

Added fixes for placeholder.

Added view constraint to avoid number input text field to become too small when no placeholder was set.
Forwarded attributedPlaceholder attribute properly.

:+1:


Trouble retrieving English names for South African provinces

For some reason I have yet to figure out, the English names for South African provinces aren't being consistently retrieved:
https://github.com/wooorm/iso-3166/blob/7aacab6bae950d5b9811c79961b1d4928a2a5b56/2.js#L25208-L25252
For reference it should be:
  {
    code: 'ZA-EC',
    name: 'Eastern Cape',
    parent: 'ZA'
  },
  {
    code: 'ZA-FS',
    name: 'Free State',
    parent: 'ZA'
  },
  {
    code: 'ZA-GP',
    name: 'Gauteng',
    parent: 'ZA'
  },
  {
    code: 'ZA-KZN',
    name: 'Kwazulu-Natal',
    parent: 'ZA'
  },
  {
    code: 'ZA-LP',
    name: 'Limpopo',
    parent: 'ZA'
  },
  {
    code: 'ZA-MP',
    name: 'Mpumalanga',
    parent: 'ZA'
  },
  {
    code: 'ZA-NC',
    name: 'Northern Cape',
    parent: 'ZA'
  },
  {
    code: 'ZA-NW',
    name: 'North-West',
    parent: 'ZA'
  },
  {
    code: 'ZA-WC',
    name: 'Western Cape',
    parent: 'ZA'
  },

Hey we're also seeing this. The data actually differs from what's shown in Wikipedia. Any ideas on why/can we get it corrected?

Then presumably either Wikipedia changed, or there is a bug in the code here. Crawling tons of user based data from Wikipedia is error prone. Solving things not trivial. But it is possible: you could do a PR to solve things here (try running the scripts and if they do not work, fix them). Or on Wikipedia. Or, raise an issue with ISO to get them to open source data. It’s ridiculous that this isn’t open.


Troubles connecting to local Ganache

Hello,
first of all, thank you for the amazing work!
I am using a truffle Project to deploy my migrations to a local ganache instance.
Isn't it possible to connect useDapp to my ganache local network running on localhost:7545?
Regards

It is a bit tricky, but you can do it

you need to setup your Ganache to have existing chainId
deploy multicall and set the address in config
configure MetaMask to use ganache

It is on the roadmap, but a bit further ahead (in configuration milestone)
Will accept PR. :)

with version 0.3.3 you should be able to connect with MetamMask as long as chainId is 1337

Hey mate. Awesome thank you, I will try it out! I was using useWallet for the connections but will now definetly switch to useDApp. I guess there are some more features to come? Please stay as lean as possible, I would prefer having less functions working properly over massive features :).

I am currrently seeing this issue after the newest update: provider.js?28e5:30 Missing multicall address for chain id 1337

with version 0.3.3 you should be able to connect with MetamMask as long as chainId is 1337

Version 0.3.3 ?  I see the latest version as 0.3.15. Is version 0.3.3 available via npm/yarn yet?

There was a lot of small releases, but if you update to 0.3.15 it will have changes from 0.3.3 as well.
But if you really want to install some specific version you can always do yarn add<EMAIL_ADDRESS>

Hi - useDapp looks really cool and very easy to get started with, but I am also seeing the error "Missing multicall address for chain id 1337".
I am using @usedapp/core 0.3.15, my local network is http://<IP_ADDRESS>:8545 (with the local network provided by Hardhat, chainId 1337).
Having some documentation around connecting to localhost will be really helpful, I imagine most devs will be testing locally before using e.g. Infura.
Thanks!

Implemented in #245


fix(topic): require lowerCamel topic names instead of lower_snake

Breaking change, fixes #2328
Original topic name PR is #1958

Maybe we can just plan this with the customer repo. It might be a good time to get some of this in and we can manually update the db(s) as needed to get them running again. I'm happy to help with this and pair with us/them on getting it deployed.

@mistermoe has said it's okay to wipe the databases (except the secrets) table
@gak also has a breaking change he wants to get in, so let's merge these changes but not deploy until we have both in

@mistermoe has said it's okay to wipe the databases (except the secrets) table
@gak also has a breaking change he wants to get in, so let's merge these changes but not deploy until we have both in

@alecthomas I was going to help deploy the topic names PR today, but sounds like we'll need to wait for @gak's change as well, so we can do a deploy this evening or tomorrow instead 👍


Not working with lolcat.

Fastfetch is not working with lolcat. I just updated fastfetch from version 1.11.0 to 1.12.2.
Command (fastfetch|lolcat) should work fine and produce output like this

But, it is not showing any logo after update to version 1.12.2.

In 1.12, we enable --pipe automatically when we detect fastfetch is being piped. Specify --pipe false if you don't want the behavior.

Thanks.


Update regular expressions to get signature

Need to make some more changes.

Updating existing tests

Updating existing tests

Thank you so much, it is working.

Is anyone else experiencing no longer getting back 720p (itag 22) on music videos? For example: https://www.youtube.com/watch?v=yWfsla_Uh80

@CWftw that video does have an 720p version but it does not contain audio (itag 136)

Thanks @SoneeJohn for the fix—are you planning to address the failed checks?

@CWftw I encountered the same issue. I can't get 720p on all music videos since yesterday.

Thank you @SoneeJohn — @0xced what are prospects for a merge? (Would love to see this fix added to a release so the CocoaPods distro can be updated.) Thanks!


Add custom tags

Fixes #311
Specify tags for a model:
{{
  config(tags=['nightly'])
}}

-- OR:

{{
  config(tags='nightly')
}}

select ..

or:
models:
  snowplow:
    # Set tags as a string
    tags: bi

  my_project:
    clickstream:
      # Set tags as a list
      tags: ['nightly', 'crm']

    advertising:
      tags: ['nightly']

Select models with a tag:
$ dbt run --model tag:nightly

TODO:

[x] add test for setting tags in dbt_project.yml
[x] good error message when a bad selector is provided
[x] support setting tags as a string

Error message when a bad selector is provided
$ dbt run --model tags:10
Found 1 operations, 179 macros, 1 analyses, 1 seed files, 3 tests, 3 models, 1 archives
The 'tags' selector specified in tags:10 is invalid. Must be one of [tag, fqn]
* Spec='tags:10' does not identify any models and was ignored

WARNING: Nothing to do. Try checking your model configs and model specification args

nice! very excited about this :)


add a new doc 'common issues and solutions'

What this PR does / why we need it:
This document was created to systematically compile and manage potential issues users might encounter, along with their resolutions. By having a structured approach, we can ensure that everyone contributing to the documentation follows a consistent template, making it easier to maintain, update, and troubleshoot. This will help prevent disorganization and make the document more manageable as it grows, ensuring that problems can be quickly identified, resolved, or removed when no longer relevant.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Type of changes
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Feature/Issue validation/testing:
Please describe the tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[ ] Test A

[ ] Test B

Logs

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

Re-running failed tests

/rerun-all - rerun all failed workflows.
/rerun-workflow <workflow name> - rerun a specific failed workflow. Only one workflow name can be specified. Multiple /rerun-workflow commands are allowed per comment.

@Jooho Isn't it better to add it in the kserve website ?

What about renaming this to be FAQs?

I am ok with the renaming the title.

Do we actually need the author and date information?

I was thinking of maintaining this doc like the following options.

every 3 months after the issue is created, we can send an email to the authors check if the issue they created is still valid or not.
we can move issues that were written after 2 years to an old FAQ or something.

I  don't have preference about this so I am open to accepting any idea about it.

@Jooho Isn't it better to add it in the kserve website ?

I think the website is suitable for showing more official features. So we need to pay more attention to PR review. However, I think it is important to share with other users about their experiences and solutions without limiting specific categories, such as FAQs. If a user writes according to the specific template and writes enough to understand the problem content and solution, we can easily merge PRs, so I think this repository is more suitable than the website.
But I am open for this question as well :)

As it is more related to dev, might be ok to keep it here.
But having a section in the official docs with common issues that users trying to run KServe hits, would be a great addition as well.

But having a section in the official docs with common issues that users trying to run KServe hits, would be a great addition as well.

good idea. I will follow up this after this pr is merged.

What about renaming this to be FAQs?
I am ok with the renaming the title.
@terrytangyuan I’ve reconsidered the suggestion to rename the document to "FAQs" and now believe that the original title might be more appropriate. The content primarily focuses on issues and provides solutions, giving it more of a troubleshooting guide feel. What do you think?

@yuzisun do you have any other questions?

/lgtm
/approve

@yuzisun do you have any other questions?

no, thanks for the docs!!


$parent Property Not Being Set

Describe the Bug

The $parent property isn't set on elements created when applying an element template. Furthermore, we don't have any tests verifying the $parent property being set. I've added a failing test case to showcase this: https://github.com/bpmn-io/bpmn-js-element-templates/actions/runs/6308838449
Steps to Reproduce

Apply an element template that creates a zeebe:Input
Inspect the created element
$parent property is undefined

Expected Behavior
The $parent property is set.

Environment

Library version: 1.4.0

Was trying to test my changes in bpmn-js-element-templates with local linking but the linking doesn't work work.

Although this happens, this is not what's causing https://github.com/camunda/web-modeler/pull/6230#issuecomment-1733742589 or any other known issues.
For this reason, I think we can deprioritise and move back to backlog.


CI update, Python 3.8 support

Run travis and appveyor tests on Python 3.8 and fix some linting warnings via https://config.travis-ci.com/explore
Status: Currently waiting for binary wheels of dependencies

scikit-image: their build depends on numpy, but they removed their pyproject.toml and with it their declaration of build deps, meaning pip doesn't know to install numpy before building skimage. Also, using tox, I didn't find out yet if it is possible at all to install numpy before the other packages. Work-around: build in separate build environment before running tox.
llvmlite: needs special attention for building (custom llvm build etc.)
h5py, psutil, scikit-learn: are built from source

Plan: merge this with allowing failures. Disable the work around so we see always the real status.
See also: https://github.com/scikit-image/scikit-image/issues/4261
Contributor Checklist:

[x] I have added myself to the creators.json file
[x] I have added a changelog entry for my contribution
[ ] I have added/updated documentation for all user-facing changes - may want to update installation docs to include Python 3.8 once the tests are running there.

I just re-ran the CI pipelines and, while the linux build succeeds, the Windows build fails with a NotImplementedError somewhere deep in tornado... so, we'll need to wait a bit longer for things to stabilize to fully support Python 3.8 in LiberTEM.

We'll need something like this in our dependencies once distributed 2.11 is released:
distributed >= 2.2.0;python_version<'3.8'
distributed >= 2.11.0;python_version>='3.8'

Distributed 2.11 is now released, let's see if this works

Retried this PR to check for #695. No travis-related checks appearing in this PR. Somehow the webhooks seem to be broken?

@sk1p What is the status and future plan here? That's a lot of CI failures... 😅 How does it relate to #565? Regarding the 0.5 release, do you think we can include experimental support for Python 3.8, or do we postpone that?

Regarding the 0.5 release, do you think we can include experimental support for Python 3.8, or do we postpone that?

I think we can advertise experimental Python 3.8 support - I'm now using it in development and it works fine. Not sure how the situation is on Windows, though.
Regarding this PR, I don't think we'll have to merge it - for 0.6 or 0.7 I'd like to finish the move to Azure Pipelines, which mostly means adjusting the release script and no longer using travis-sphinx.

Hm, we have "distributed>=2.2.0,<2.9" because of the weird errors that newer versions of distributed caused. I've just retested with the brand new distributed 2.14, and it still locks up. Technically we can still work with the local context. I am testing on 3.8 now to see if the error exists there, too.

Confirmed: Distributed 2.14 locks up with prototypes/reproducers/test_dask_remote.ipynb on Python 3.8.

Regarding this PR, I don't think we'll have to merge it

So should we close it, then? :-)

So should we close it, then? :-)

If we commit to switching to Azure pipelines at the beginning of 0.6, I'd say go ahead and close, otherwise it is useful to have for ad-hoc py38 testing.

If we commit to switching to Azure pipelines at the beginning of 0.6, I'd say go ahead and close, otherwise it is useful to have for ad-hoc py38 testing.

Ok, let's leave it open. I've removed milestone.

Closed in favor of #565


Migrate incremental data via parsing and filtering data

Report https://github.com/apache/incubator-kvrocks/runs/7034653085?check_suite_focus=true#step:10:687

This test case spent almost 30s on my local machine and it should be even slower in CI environment. So we can increase the wait time from 50s => 100s if has no idea. Do guys
have any thoughts about this case? @ChrisZMF @ShooterIT @caipengbo

It's a valid workaround. However, it's better to make the waiting determinately - is it possible?

It's a valid workaround. However, it's better to make the waiting determinately - is it possible?

Yes, can check the migrate finished status which we're doing at line: slotmigrate.tcl#L513, we can remove the time limit.

@tisonkun removed the time limit, can help to take a look again.

Is there any way we can reduce the waiting time?  I think this test case is taking too long.

Maybe we can test whether increase the pipeline size can help or not


Url or api for just after hotword detection.

I d like to use a dash button to simulate the hotword detection. Is it possible to run a specifique http commande to start the after hotword process  ?

There is a flag on Jarvis to do just that:
$> ./jarvis.sh -h
    Usage: jarvis.sh [-bc:ihklnp:s:ux:z]
[...]
    -l  directly listen for one command (ex: launch from physical button)
[...]

So your button should run the following shell command:
$> path/to/jarvis.sh -l
You: #listens for a command

Is it ok or do you really need to trigger it from the network?

When I run ssh pi@mypi /home/pi/jarvis/jarvis.sh -l if he understand the command no problems. but if it didn't understand 3 times Jarvis stay active and I can't restart a hotwork.

For me a API should be better.

Indeed I just tested, if Jarvis is launched with -l flag, it 3 failed recognitions, it goes back to hotword detection mode instead of exiting.
I'll fix that first before evaluating if API could be used to do this.

@JZacharie fixed, please update and retest.

I just have tested.
With remote ssh the audio go thru my pulse audio : The input stay on my Jarvis box mic. but the output try to go to my remote ssh shell box .
If I use the rest/API to start the after hotword detection, it  will bypass the audio mecanism.
I'll try implement it.
Regards

So @JZacharie you need to trigger the same as ./jarvis.sh -l from the API right?
I can do this if you really need it.

For me it will be very helpfull if all commands could be run from REST/API. (Like execute and say.)
But if you implement it. Could you enable it for all commande line possibility : [-bc:ihjklmnp:qs:uvwx:z]
And is it possible to provide the only need data from the APi repond ? Not all DATA as the console do.

Actually it's not that simple because you can't launch multiple sessions of Jarvis in parallel (concurrent use of mic / speakers).
For execute & say, it's not a problem as Jarvis exists at the end.
For -l (directly listen for acommand) it should be ok too, as it will exit at the end.
However, for other commands that just "start Jarvis", you will run into problems.
If your project is to use a Dash button and trigger via RestAPI the hotword recognition (directly listen for a command), the -l flag on the API could be just what you need.
What do you think?

Any update on this?

No answer of the issue creator since 10 Jan 2017 => Issue closed.
Don't hesitate to reopen the issue if there is some evolution.


The pattern for value 102 of Code128 is wrong

Code128 value 102 pattern is defined as 0x7a2, but it is actually 0x7ae.
As a result, an incorrect barcode is output when FNC1 is output or when the check digit result is 102.

Fixed


Unable to connect to IPython kernel

Issue Type: Bug
Expected:
Start Visual Studio Code. Open an ipynb file, a jupyter kernel is started and I can execute my code.
Actual:
Start Visual Studio Code. Open an ipynb file. A promt appears 'Connecting to IPhython kernel' (Source: IPhyton extension). After a while I get the error message 'Timed out waiting to get a heartbeat from the kernel process'.
Python output in Terminal:
~\anaconda3\envs\MLGPU\python.exe "c:\Users\user.vscode\extensions\ms-python.python-2020.9.111407\pythonFiles\pyvsc-run-isolated.py" -c "import ipykernel"
~\anaconda3\envs\MLGPU\python.exe "c:/Users/user/.vscode/extensions/ms-python.python-2020.9.111407/pythonFiles/pyvsc-run-isolated.py" vscode_datascience_helpers.daemon --daemon-module=vscode_datascience_helpers.kernel_launcher_daemon -v
Error 2020-09-28 21:03:09: Failed to start the Daemon, StdErr:  C:\Users\user\anaconda3\envs\MLGPU\python.exe: can't open file '"c:/Users/user/.vscode/extensions/ms-python.python-2020.9.111407/pythonFiles/pyvsc-run-isolated.py"': [Errno 22] Invalid argument
Error 2020-09-28 21:03:09: Failed to start the Daemon, ProcEndEx [Error: Timeout waiting for daemon to start
at Timeout._onTimeout (c:\Users\user.vscode\extensions\ms-python.python-2020.9.111407\out\client\extension.js:24:225838)
at listOnTimeout (internal/timers.js:531:17)
at processTimers (internal/timers.js:475:7)]
Error 2020-09-28 21:03:09: Failed  to start the Daemon, Ex [Error: Timeout waiting for daemon to start
at Timeout._onTimeout (c:\Users\user.vscode\extensions\ms-python.python-2020.9.111407\out\client\extension.js:24:225838)
at listOnTimeout (internal/timers.js:531:17)
at processTimers (internal/timers.js:475:7)]
Error 2020-09-28 21:03:09: Pinging Daemon Failed, Class name = h, completed in 5001ms, has a falsy return value, Arg 1: {}, Return Value: undefined [Error: Timeout waiting for daemon to start
at Timeout._onTimeout (c:\Users\user.vscode\extensions\ms-python.python-2020.9.111407\out\client\extension.js:24:225838)
at listOnTimeout (internal/timers.js:531:17)
at processTimers (internal/timers.js:475:7)]
Error 2020-09-28 21:03:09: Failed to create daemon, Class name = h, completed in 5003ms, has a falsy return value, , Return Value: undefined [Error: Timeout waiting for daemon to start
at Timeout._onTimeout (c:\Users\user.vscode\extensions\ms-python.python-2020.9.111407\out\client\extension.js:24:225838)
at listOnTimeout (internal/timers.js:531:17)
at processTimers (internal/timers.js:475:7)]
Error 2020-09-28 21:03:09: Failed to create the daemon service, defaulting to activated environment [Error: Timeout waiting for daemon to start
at Timeout._onTimeout (c:\Users\user.vscode\extensions\ms-python.python-2020.9.111407\out\client\extension.js:24:225838)
at listOnTimeout (internal/timers.js:531:17)
at processTimers (internal/timers.js:475:7)]
Extension version: 2020.9.111407
VS Code version: Code 1.49.2 (e5e9e69aed6e1984f7499b7af85b3d05f9a6883a, 2020-09-24T16:29:41.983Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
AMD Ryzen 7 3700X 8-Core Processor              (16 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.95GB (18.64GB free)

Process Argv
--crash-reporter-id 42b52e0f-c079-4f64-ab53-9b5417d1b5f4

Screen Reader
no

VM
0%

Hi @randomstructures, thanks for reporting a bug and sorry for the trouble. Just to check, does your username on Windows contain a space? If so, I believe you are running into the same problem as was reported in https://github.com/microsoft/vscode-python/issues/14118, and we are rolling out a fix for that problem in the near future. Please let me know and otherwise we can dig in further 😊

Hello @joyceerhl
Yes, my windows username contains a space. I think that your diagnosis is correct and #14118 already describes the situation that I am experiencing.
Thank you very much for looking into this!

Issue Type: Bug
Expected:
Start Visual Studio Code. Open an ipynb file, a jupyter kernel is started and I can execute my code.
Actual:
Start Visual Studio Code. Open an ipynb file. A promt appears 'Connecting to IPhython kernel' (Source: IPhyton extension). After a while I get the error message 'Timed out waiting to get a heartbeat from the kernel process'.
Python output in Terminal:
~\anaconda3\envs\MLGPU\python.exe "c:\Users\user.vscode\extensions\ms-python.python-2020.9.111407\pythonFiles\pyvsc-run-isolated.py" -c "import ipykernel"
~\anaconda3\envs\MLGPU\python.exe "c:/Users/user/.vscode/extensions/ms-python.python-2020.9.111407/pythonFiles/pyvsc-run-isolated.py" vscode_datascience_helpers.daemon --daemon-module=vscode_datascience_helpers.kernel_launcher_daemon -v
Error 2020-09-28 21:03:09: Failed to start the Daemon, StdErr: C:\Users\user\anaconda3\envs\MLGPU\python.exe: can't open file '"c:/Users/user/.vscode/extensions/ms-python.python-2020.9.111407/pythonFiles/pyvsc-run-isolated.py"': [Errno 22] Invalid argument
Error 2020-09-28 21:03:09: Failed to start the Daemon, ProcEndEx [Error: Timeout waiting for daemon to start
at Timeout._onTimeout (c:\Users\user.vscode\extensions\ms-python.python-2020.9.111407\out\client\extension.js:24:225838)
at listOnTimeout (internal/timers.js:531:17)
at processTimers (internal/timers.js:475:7)]
Error 2020-09-28 21:03:09: Failed to start the Daemon, Ex [Error: Timeout waiting for daemon to start
at Timeout._onTimeout (c:\Users\user.vscode\extensions\ms-python.python-2020.9.111407\out\client\extension.js:24:225838)
at listOnTimeout (internal/timers.js:531:17)
at processTimers (internal/timers.js:475:7)]
Error 2020-09-28 21:03:09: Pinging Daemon Failed, Class name = h, completed in 5001ms, has a falsy return value, Arg 1: {}, Return Value: undefined [Error: Timeout waiting for daemon to start
at Timeout._onTimeout (c:\Users\user.vscode\extensions\ms-python.python-2020.9.111407\out\client\extension.js:24:225838)
at listOnTimeout (internal/timers.js:531:17)
at processTimers (internal/timers.js:475:7)]
Error 2020-09-28 21:03:09: Failed to create daemon, Class name = h, completed in 5003ms, has a falsy return value, , Return Value: undefined [Error: Timeout waiting for daemon to start
at Timeout._onTimeout (c:\Users\user.vscode\extensions\ms-python.python-2020.9.111407\out\client\extension.js:24:225838)
at listOnTimeout (internal/timers.js:531:17)
at processTimers (internal/timers.js:475:7)]
Error 2020-09-28 21:03:09: Failed to create the daemon service, defaulting to activated environment [Error: Timeout waiting for daemon to start
at Timeout._onTimeout (c:\Users\user.vscode\extensions\ms-python.python-2020.9.111407\out\client\extension.js:24:225838)
at listOnTimeout (internal/timers.js:531:17)
at processTimers (internal/timers.js:475:7)]
Extension version: 2020.9.111407
VS Code version: Code 1.49.2 (e5e9e69aed6e1984f7499b7af85b3d05f9a6883a, 2020-09-24T16:29:41.983Z)
OS version: Windows_NT x64 10.0.18363
System Info

I was also having the issue
if the kernel issue is just in vscode and not in the jupyter notebook ("which you might be opening with anaconda prompt").
you should install the previous version of python, check this link!! and this link also.


Potential symlink/directory traversal issue in user ID extraction

Commit b1bb73e01c9bf0b1b75e50a2d1947b14a8174eee (“ Teach "Run" to dig user IDs out of containers”) added the fopenContainerFile function. It does not correctly deal with symbolic links, which could point to something outside of the container root.

Are you suggesting something like https://github.com/nalind/buildah-1/commit/97ca24a7b4cc2739edf8b6b12f1c87ab103a0caa?  What did we not catch with the (possibly over-restrictive) lstat-after-opening method?

 (8) [Buildah] Add "dockah build-using-dockerfile" aka "buildah bud"

 (8) [Buildah] Add "dockah build-using-dockerfile" aka "buildah bud"

@fweimer The patch with the fix I'm looking at had a conflict in it, so I've rebased it to https://github.com/nalind/buildah-1/commit/2c387528f2af7655d231a4989f67b21c90e1c385.  I'd appreciate any guidance you can offer.

@nalind, when this code is run, is the file system directory tree dead and inaccessible to unprivileged processes, or has concurrent modification of the tree to be expected?

The filesystem tree is mounted on the host, so modification is possible for users with the necessary privileges.

@nalind, in this case, the code still has a time-of-check-time-of-use race condition because after symbolic link resolution, the file system tree could change and contain symbolic links again.
You either have to use chroot to restrict pathname lookups (no longer needs root in current Fedora), or reimplement kernel pathname lookup using openat and O_PATH.

 [buildah] Fix potential symlink traversal problem when looking up numeric IDs for USER settings

 [buildah] Fix potential symlink traversal problem when looking up numeric IDs for USER settings

@nalind have you done anything else for this issue?

@nalind any update on this?

Alright, I'm beginning to think there's no way around having to use chroot().  https://github.com/projectatomic/buildah/compare/master...nalind:user2 contains what I'm thinking should prevent the directory traversal problem.


API Changes: IIO

Task Description
Some changes are required to match Soletta's API design conventions

[ ] Rename sol_iio_device_trigger_now() to sol_iio_device_trigger()
[ ] Change return type of functions that may fail to return error code instead of boolean false. Example: sol_iio_read_channel_value()

We can close this :D

certainly !!! Thanks


Edit error handling

Description

Currently, if an edit request is posted, regardless of success or failure, the edit drawer will close.
We need better error handling on this.
Expected Behavior

If request fails: display error message + don't close the drawer
If request passes: close drawer, preserve scrolling
Actual Behavior

Regardless of success/failure, edit drawer will close
Steps to Reproduce

Import a character (Grod for example)
Edit Grod's entry
Modify notes
Request fails, drawer closes

Context

Possible Fix

Better handling on edit for backend redirecting or fix it in front-end somehow (Promise object)
This issue happens because errors prop gets updated after the put request, therefore the form thinks its successful since errors is an empty object ({})
Other
Description

Additional Notes

Hey team! Please add your planning poker estimate with ZenHub @AnnieTran13 @luigibesani @pascwhale


Code cleanup

@akanto plz review. Enthused on equals issue what i found today, i did some code cleanup, found many errors next to style issues.

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2793/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/284/

Last 4 are news

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2800/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/291/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2801/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/292/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2802/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/293/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2803/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/294/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2804/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/295/

Ready to merge from my side.

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2808/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/299/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/312/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2820/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2821/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/313/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2822/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/314/

Raise a JIRA for this issue and prefix every commit message with JIRA id. Rebase it and I will merge it immediately after that

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2828/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/320/


feat: Infer artist series when creating alert form artwork screen

This PR resolves ONYX-566

Related Force PR: https://github.com/artsy/force/pull/13244

Description
This change adds artist series as an initial criterion when creating the alert from the artwork screen.

PR Checklist

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

To the reviewers 👀

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

Changelog updates
Changelog updates

Cross-platform user-facing changes

Infer artist series when creating alert form artwork screen - ole

iOS user-facing changes

Android user-facing changes

Dev changes

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

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against dcc3003737d7ba6cd676544cd9e9d4cc7a4a9827

Closing this PR because we only want to suggest artist series (Slack Thread)


Added more examples and test to Meronym and Holonym

Added

More examples
Test and
Conditions

Hi,
I would rather have a single pull request with the documentation close to complete, rather than adding things bit by bit.    Or make it clear what is done and what not done.
Currently, this is still missing:

checking of project specific links

https://metacpan.org/pod/WordNet::QueryData
plWordnet
PWN I think is ok, ..

Examples and tests for each relation.
links to references

Linking examples to OMW can maybe be left until the macro issue is resolved

If I get you correctly, documentation for all relations should be complete before I raise a PR? And what should I do about this particular PR after adding those extras you pointed out?

You can either redo this PR, or withdraw it and make a new one.
 And what should I do about this particular PR after
adding those extras you pointed out?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/globalwordnet/gwadoc/pull/28#issuecomment-693197266,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIPZRQMQP5MYWZPWJRLFJLSGBJYZANCNFSM4RNCUHJQ
.

--
Francis Bond http://www3.ntu.edu.sg/home/fcbond/
Division of Linguistics and Multilingual Studies
Nanyang Technological University

Let me interject. This is not the main branch but the gsod one. Personally I think small merges are fine, maybe even preferable. I'd rather not have to review a PR that does too many things at once.
@gconnect Before withdrawing the PR, let me talk this over with Francis and get back to you so we agree on the proper procedure.

Apparently you don't have to withdraw, just add commits and it will update the pull request.

@gconnect Hi, I spoke with Francis and I think we agree. Let me try to summarize. Small pull requests are fine, but it's best if they can be "complete" in some sense. This means that if the pull request is for adding information about 1 or 2 relations, all the relevant information is included. Furthermore, the pull request should not bundle unrelated changes (e.g., a UI or settings change). The purpose of these guidelines is to make reviewing the pull request easier (it's easiest when we can clearly see all the relevant changes without other noise) and to keep the Git history clean (so later we can point to where something changed).
Francis didn't realize that you can keep adding commits to the same pull request, which is why he suggested closing it and opening a new one. Since that is possible, there's no need to close this one.
I hope that clarifies things!

Hi,
I would rather have a single pull request with the documentation close to complete, rather than adding things bit by bit. Or make it clear what is done and what not done.
Currently, this is still missing:

checking of project specific links

https://metacpan.org/pod/WordNet::QueryData
plWordnet
PWN I think is ok, ..

Examples and tests for each relation.

links to references

Linking examples to OMW can maybe be left until the macro issue is resolved

I am not sure what to do with the information provided here  https://metacpan.org/pod/WordNet::QueryData

It is hard to review this because the merged commits are showing up as diffs for this PR as well. It would be better if the new commits were rebased on the the others so only the new ones are included here.

How do I go about it?

How do I go about it?

Actually that's a good question. I don't want to make things too difficult, so maybe we should just proceed with the duplicated commits for now, and I will cherry-pick the relevant commits when I merge the gsod branch into the master branch. In other words, don't worry about it for now.

I resolved the conflicts locally then merged. After the merge I noticed that the examples for meronym and holonym are backwards and/or need to be double-checked. I'll take a look at that tomorrow.

I pushed fixes for the examples in 1619432b35113f5aaace56bd54b499ded677d7a9


button to export warped 3D models

Let's have a button to export the warped 3D model. This would be the reference sample warped to mean shape if the sliders are at (0,0), and then subsequent applied PC values. This button will sit just above the 'create animation' section.
Please make the default export option to the disk (not into the scene), and in PLY format.

The transient models (on the 2nd 3D viewer) is now unhidden. The user can save this model, though the ExportAs Dialog box as needed.


arangodump fails to authenticate with '@@' in the password

My Environment

ArangoDB Version:  3.11.8
Deployment Mode:  Leader/Follower
Deployment Strategy: Kubernetes
Configuration:
Infrastructure:
Operating System:
Total RAM in your machine:
Disks in use:
Used Package:

Component, Query & Data
Affected feature: arangodump (and maybe other CLI tools?)

AQL query (if applicable):

AQL explain and/or profile (if applicable):

Dataset:

Size of your Dataset on disk:

Replication Factor & Number of Shards (Cluster only):

Steps to reproduce

Create a user with a password that includes the string '@@'
Try to execute an arangodump passing the value in the --server.password argument

Problem:
It appears that when you pass a password with '@@' in it into the arangodump cli via the `--server-password argument, it's escaping that somehow, even if passed using single quotes to remove interpretation from the shell.
Workarounds that do work:

leave the option off and let it prompt (this doesn't work in scripts)
add an additional '@' for each one in the string. i.e. if the password is 'foo@@' then use 'foo@@@@'

Related:
If a password has an '@', I can add two and the password still validates. i.e. these two are interchangeable:
arangodump ... --server.password 'foo@'
arangodump ... --server.password 'foo@@'
Expected result:

Related PR to improve the startup option description: https://github.com/arangodb/arangodb/pull/20876

Thank you both! That makes sense, I had missed that bit.

You were not the first by far - hence the cause of action ;)


How to install these missing nodes?

Hello, I am a beginner, how to install these missing nodes?
Thanks you!
missing nodes:
DownloadAndLoadMochiModel
MochiTextEncode
MochiSampler
MochiDecode

Go to custom_nodes directory in ComfyUI
git clone git@github.com:kijai/ComfyUI-MochiWrapper.git
cd ComfyUI-MochiWrapper
pip install -r requirements.txt
pip install SageAttention

转到 ComfyUI 中的目录custom_nodes
git clone https://github.com/kijai/ComfyUI-MochiWrapper.git
cd ComfyUI-MochiWrapper
pip install -r requirements.txt
pip install SageAttention

The problem has been solved, thank you very much!

Hello, have done this but still cant see the nodes, still red. Any other ideas?

Hello, have done this but still cant see the nodes, still red. Any other ideas?

Update torch to 2.5.0

Hello, have done this but still cant see the nodes, still red. Any other ideas?

In my experience, many comfyui nodes have conflicts, if upgrading torch still does not solve the problem, please create a new comfyui directory (for mochi only), may solve the problem

Thank you for your reply. Sorry for the silly question, but how do I install torch 2.5?

Thank you for your reply. Sorry for the silly question, but how do I install torch 2.5?

Try any of the following command lines:
pip3 install torch==2.5.0 torchvision --index-url https://download.pytorch.org/whl/test/cu118
pip3 install torch==2.5.0 torchvision --index-url https://download.pytorch.org/whl/test/cu121
pip3 install torch==2.5.0 torchvision --index-url https://download.pytorch.org/whl/test/cu124

Thank you. I would use any of those commands on a cmd window in the mochi folder or the comfy one?

Yes, it seems mochi has no strict requirements for the cuda version. In my own case, I use cu118

sorry Im a bit new all this. I get this error. What im I doing wrong?

Thank you. I would use any of those commands on a cmd window in the mochi folder or the comfy one?

Yes, it seems mochi has no strict requirements for the cuda version. In my own case, I use cu118

Get this window when I try to do it in the /python_embeded folder

Thank you. I would use any of those commands on a cmd window in the mochi folder or the comfy one?

Yes, it seems mochi has no strict requirements for the cuda version. In my own case, I use cu118

Get this window when I try to do it in the /python_embeded folder

Run In the ComfyUI_windows_portable\python_embeded folder:
python.exe -m pip install -U torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cu124

Still getting the same window...frustrating.

Still getting the same window...frustrating.

It feels like there are many possibilities, maybe there is a problem with the version of python you installed? Try to install a version that is compatible with your system. Use the following two lines to see the version?
python --version
python -c "import platform;  print(platform.architecture())"
Or if phthon hasn't been upgraded to a new version, try:
python -m pip install --upgrade pip

Thank you for your reply. I ended up having problems even starting comfy with the same message so I reinstalled comfy and nodes and it’s working now. Thanks again 👍

Go to custom_nodes directory in ComfyUI
git clone https://github.com/kijai/ComfyUI-MochiWrapper.git
cd ComfyUI-MochiWrapper
pip install -r requirements.txt
pip install SageAttention

Top man. Thanks for the advice. Worked a treat.


Disable feed on dev environment

Is there an option to disable jekyll-feed on development environement? Maybe by using jekyll.environemnt?

As far as I know, there is no way to disable feed generation once you have added jekyll-feed to your plugins.

@parkr @sharu725 I've added a pull-request in https://github.com/jekyll/jekyll-feed/pull/346 to do this, adding a new skip_development: true config option, because I would find this very useful myself as my site is getting slower and slower, so skipping this plugin without having to remove it from the plugins list is really helpful to me =)

Fixed by #370!


[Proposal] ST Connector Document Open Instruction Manual

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
This guidebook is designed to provide guidance and advice for Seatunnel connector doc development, helping you write high-quality technical docs.
Doc Structure and Organization
A connector doc needs to provide users with the following four core elements:
Dependencies, how to create connectors, connector parameters, attributes, data type mappings

Dependency
How to create a connector
Error ❌: The demo lists a bunch of optional configuration items for the user, causing the user to be unable to run the common example
Correct ✅: We want to provide users with a minimal demo of the minimum configuration items at the beginning of the doc, and users can immediately experience the use of the connector by copying the demo
Connector parameters
Connector parameters are a list and overview of all full entries, describing the granularity to basic information.
We can understand this element as a "dictionary", which is mainly used for users to quickly find parameters during use.
Features
The feature section requires us to explain in detail the nested parameters and core function parameters. This section requires that examples must be provided
Data type mapping
The data type of the data source db and the data type of the seatunnel are displayed on a table.
Can let the user very clearly figure out, synchronize the current data, my data synchronization conf file schema and how to write to map the structure of the data source
Example
https://github.com/apache/incubator-seatunnel/pull/4389/files#diff-5384eeb077736ed8c196058d29007ba814deeb01e098960edbed793db900996c

Apache MongoDB connector

[x] batch
[x] exactly-once
[x] column projection
[x] parallelism
[x] support user-defined split

The MongoDB Connector provides the ability to read and write data from and to MongoDB.
This document describes how to set up the MongoDB connector to run data reads against MongoDB.
Dependencies
In order to use the Mongodb connector, the following dependencies are required.
They can be downloaded via install-plugin.sh or from the Maven central repository.

MongoDB version
dependency

universal
Download

How to create a MongoDB Data synchronization jobs
The example below shows how to create a MongoDB data synchronization jobs:
# Set the basic configuration of the task to be performed
env {
  execution.parallelism = 1
  job.mode = "BATCH"
}

# Create a source to connect to Mongodb
source {
  MongodbV2 {
    connection = "mongodb://user:password@<IP_ADDRESS>:27017"
    database = "test_db"
    collection = "source_table"
    schema = {
      fields {
        c_map = "map<string, string>"
        c_array = "array<int>"
        c_string = string
        c_boolean = boolean
        c_tinyint = tinyint
        c_smallint = smallint
        c_int = int
        c_bigint = bigint
        c_float = float
        c_double = double
        c_bytes = bytes
        c_date = date
        c_decimal = "decimal(38, 18)"
        c_timestamp = timestamp
        c_row = {
          c_map = "map<string, string>"
          c_array = "array<int>"
          c_string = string
          c_boolean = boolean
          c_tinyint = tinyint
          c_smallint = smallint
          c_int = int
          c_bigint = bigint
          c_float = float
          c_double = double
          c_bytes = bytes
          c_date = date
          c_decimal = "decimal(38, 18)"
          c_timestamp = timestamp
        }
      }
    }
  }
}

# Console printing of the read Mongodb data
sink {
  Console {
  }
}

Connector Options

Option
Required
Default
Type
Description

connector
required
(none)
String
The MongoDB connection uri.

database
required
(none)
String
The name of MongoDB database to read or write.

collection
required
(none)
String
The name of MongoDB collection to read or write.

schema
required
(none)
String
MongoDB's BSON and seatunnel data structure mapping

match-query
optional
(none)
String
In MongoDB, $match is one of the aggregation pipeline operators, used to filter documents

projection
optional
(none)
String
In MongoDB, Projection is used to control the fields contained in the query results

partition.strategy
optional
default
String
Specifies the partition strategy. Available strategies are single, sample, split-vector, sharded and default. See the following Partitioned Scan section for more details.

partition.size
optional
64mb
MemorySize
Specifies the partition memory size.

partition.samples
optional
10
Integer
Specifies the samples count per partition. It only takes effect when the partition strategy is sample. The sample partitioner samples the collection, projects and sorts by the partition fields. Then uses every scan.partition.samples as the value to use to calculate the partition boundaries. The total number of samples taken is calculated as: samples per partition * (count of documents / number of documents per partition).

no-timeout
optional
true
Boolean
MongoDB server normally times out idle cursors after an inactivity period (10 minutes) to prevent excess memory use. Set this option to true to prevent that. However, if the application takes longer than 30 minutes to process the current batch of documents, the session is marked as expired and closed.

Features
MatchQuery Scan
In MongoDB, $match is one of the aggregation pipeline operators used to filter documents. Its position in the pipeline determines when documents are filtered.
$match uses MongoDB's standard query operators to filter data. Basically, it can be thought of as the "WHERE" clause in the aggregation pipeline.
Here's a simple $match example, assuming we have a collection called orders and want to filter out documents that meet the status field value of "A":
db.orders.aggregate([
  {
    $match: {
      status: "A"
    }
  }
]);

In data synchronization scenarios, the matchQuery approach needs to be used early to reduce the number of documents that need to be processed by subsequent operators, thus improving performance.
Here is a simple example of a seatunnel using $match
source {
  MongoDB {
    uri = "mongodb://user:password@<IP_ADDRESS>:27017"
    database = "test_db"
    collection = "orders"
    matchQuery = "{
      status: "A"
    }"
    schema = {
      fields {
        id = bigint
        status = string
      }
    }
  }
}

Projection Scan
In MongoDB, Projection is used to control which fields are included in the query results. This can be accomplished by specifying which fields need to be returned and which fields do not.
In the find() method, a projection object can be passed as a second argument. The key of the projection object indicates the fields to include or exclude, and a value of 1 indicates inclusion and 0 indicates exclusion.
Here is a simple example, assuming we have a collection named users:
// Returns only the name and email fields
db.users.find({}, { name: 1, email: 1 });

In data synchronization scenarios, projection needs to be used early to reduce the number of documents that need to be processed by subsequent operators, thus improving performance.
Here is a simple example of a seatunnel using projection:
source {
  MongoDB {
    uri = "mongodb://user:password@<IP_ADDRESS>:27017"
    database = "test_db"
    collection = "users"
    matchQuery = "{ name: 1, email: 1 }"
    schema = {
      fields {
        id = bigint
        status = string
      }
    }
  }
}

Partitioned Scan
To speed up reading data in parallel source task instances, seatunnel provides a partitioned scan feature for MongoDB collections. The following partitioning strategies are provided.

single: treats the entire collection as a single partition.
sample: samples the collection and generate partitions which is fast but possibly uneven.
split-vector: uses the splitVector command to generate partitions for non-sharded collections which is fast and even. The splitVector permission is required.
sharded: reads config.chunks (MongoDB splits a sharded collection into chunks, and the range of the chunks are stored within the collection) as the partitions directly. The sharded strategy only used for sharded collection which is fast and even. Read permission of config database is required.
default: uses sharded strategy for sharded collections otherwise using split vector strategy.

source {
  MongoDB {
    uri = "mongodb://user:password@<IP_ADDRESS>:27017"
    database = "test_db"
    collection = "users"
    partition.strategy = single
    partition.samples = 100
    schema = {
      fields {
        id = bigint
        status = string
      }
    }
  }
}

Data Type Mapping
The following table lists the field data type mapping from MongoDB BSON type to Seatunnel data type.

MongoDB BSON type
Seatunnel type

ObjectId
STRING

String
STRING

Boolean
BOOLEAN

Binary
BINARY

Int32
INTEGER

Int64
BIGINT

Double
DOUBLE

Decimal128
DECIMAL

DateTime
TIMESTAMP_LTZ(3)

Timestamp
TIMESTAMP_LTZ(0)

Object
ROW

Array
ARRAY

Format and layout

Title

Dependencies
------------

Sub-headings

**Partitioned Scan**

Table display

MongoDB BSON type
Seatunnel type

ObjectId
STRING

String
STRING

4.Use of Note #
It is convenient for users to paste the official website example directly before using it.
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

What does this Forwarded meaning ?

How do I write schema field type map of a object list type in mongoDB ?
the mongo data like
{
 "groupName": "mygroup",
 "memberList": [
   {
      "name":"alex",
	  "age":33
   },
   {
      "name":"bob",
	  "age":25
   }
 ]
}

the SeaTunnel config
source{
   schema{
       fields {
        groupName = string
        memberList = "array<object>"
     }
   }
}

It failed !!
The schema of memberList is not right.  What's the correct schema of memberList ?
Thanks a lot .
Aames


Not working on Win10 v20H2 with iTunes v12.10?

Hi,
is this application still working?
Didn't get the itunes status to show up in discord.
In the iTunes Rich Presence window the currently playing is showing up.
Windows 10 20H2
iTunes <IP_ADDRESS>
Discord Stable 71615 (583a822)

I am having the same issue.
Windows 10 Pro 20H2
iTunes <IP_ADDRESS>
Discord Stable 72071 (6cb1462)


bracketPairColorizer default setting

BracketPairColorizer is nice plugin.
However default setting is very heavy.
Please change default setting of bracketPairColorizer as follow:
"bracketPairColorizer.scopeLineRelativePosition": false,
"bracketPairColorizer.showHorizontalScopeLine": false,
"bracketPairColorizer.showVerticalScopeLine": false,
"bracketPairColorizer.timeOut": 100,

I think this setting is new Microsoft conspiracy. LOL

I think that it is better to change it so as not to drop Microsoft's reputation.

I'm not too sure what you're about but Bracket Pair Colorizer is not an extension made by Microsoft so you should open an issue here.

Please file an issue against the Bracket Pair Colorizer extension


VPP fix - enahanced NAT session timeouts

Coverage decreased (-0.02%) to 76.271% when pulling f0db5a2773d9bc2da52807cdcce79d0e45a663ea on rastislavszabo:master into 2c70b4446ab26468d43b1cc7f774ec63f71d0638 on contiv:master.


[Suggestion] Launch process is slow  (Initialize VDUs in parallel)

Code is from AllocateresourceTask.java
VDUs should be started in parallel (Unless they depend on each other). Each VDU should have independent initialization cycle. Failure of one VDU should not effect (abort) the initialization of other VDUs.

    for (VirtualDeploymentUnit vdu : virtualNetworkFunctionRecord.getVdu()) {
      List<Future<List<String>>> ids = new ArrayList<>();
      VimInstance vimInstance = vims.get(vdu.getId());
      if (vimInstance == null) {
        throw new NullPointerException(
            "Our algorithms are too complex, even for us, this is what abnormal IQ means :" + "(");
      }
      vimInstance = vimRepository.findFirstById(vimInstance.getId());
      log.debug(
          "Allocating VDU: "
              + vdu.getName()
              + " to vim instance: "
              + vimInstance.getName()
              + " - id: "
              + vimInstance.getId());
      for (NFVImage image : vimInstance.getImages()) {
        log.trace("Available image name: " + image.getName());
      }
      ids.add(
          resourceManagement.allocate(
              vdu, virtualNetworkFunctionRecord, vimInstance, userData, keys));

      for (Future<List<String>> id : ids) {
        id.get();
      }
    }

Hi,
I would like to change the content of this issue by splitting in two different.

allocate vdu in parallel
put vnfr to error if one / all vdu fail

so i will close this issue with regards to point 1 and i will open another one for what concerns point 2
if this ok?
@mpauls ?

sounds fair

opened new one, @Rizwan-Qamar please follow the new one (#101). I will close this.

The fix was not working, need to reopen the enhancement


[loc][Query] Context and better support for plurals

For string below community member asked to provide context for placeholder, and also support for plurals in different languages. As it is probably difficult to translate in current state.
Original comment
BG ISSUE - What are the variables? Also, needs support for plurals.
by lyubomirv, 5 months ago
PL ISSUE - This needs better support for plurals to be translated nicely.
by wojtekmaj, 5 months ago
Context
src/vs/editor/browser/widget/diffReview:English (en):plaintext:header
Resource
vs_editor
String Hash
6434f4fde2ca6a67bdc0778b3406a765
String text
Difference {0} of {1}: original {2}, {3} lines, modified {4}, {5} lines

I've extracted the plural to separate nls calls and I've added explanations:
header.setAttribute('aria-label', nls.localize({
	key: 'header',
	comment: [
		'This is the ARIA label for a git diff header.',
		'A git diff header looks like this: @@ -154,12 +159,39 @@.',
		'That encodes that at original line 154 (which is now line 159), 12 lines were removed/changed with 39 lines.',
		'Variables 0 and 1 refer to the diff index out of total number of diffs.',
		'Variables 2 and 4 will be numbers (a line number).',
		'Variables 3 and 4 will be "no lines", "1 line" or "X lines", localized separately.'
	]
}, "Difference {0} of {1}: original {2}, {3}, modified {4}, {5}", (diffIndex + 1), this._diffs.length, minOriginalLine, originalChangedLinesCntAria, minModifiedLine, modifiedChangedLinesCntAria));

@alexandrudima Thanks! We just got one additional comment where this change to your comment was suggested:
Current: Variables 3 and 4 will be "no lines", "1 line" or "X lines", localized separately.
Fix: Variables 3 and 5 will be "no lines", "1 line" or "X lines", localized separately.
Can you please confirm this? Thanks!


Change image picker intent to ACTION_OPEN_DOCUMENT and save image locally

I'm suggesting this patch to fix the issue #51 regarding non persistent custom icons.
According to https://developer.android.com/guide/topics/providers/document-provider.html "Use ACTION_OPEN_DOCUMENT if you want your app to have long term, persistent access to documents owned by a document provider. An example would be a photo-editing app that lets users edit images stored in a document provider."
I've tested this successfully with my Google Nexus 7 Tablet and Android 6.0.1.

Would be great if you could merge this.
The non-persistent icons are a tiny bit annoying.

I further improved this PR to copy selected images to the applications local storage.

#87

@npmccallum are there any issues with this PR, that prevent it from being merged? This would improve the usability greatly.

Great Contribution, as Intent.ACTION_PICK is not the best method to open images in this case.
Just one more thing: I'm not sure what the policy of this repository is, but it would make sense to squash your three commits into one, I think


Improve exception text when failed to parse TimeSpan value

Related to #49

For whatever reason, looks like Azure DevOps failed to display the PR results even though it technically ran.


Fix backport branch deletion on merge

Description

Fixes backport branch deletion

Related Issues
Resolves #2128

Check List

[ ] ~New functionality includes testing.~
[ ] ~New functionality has been documented.~
[ ] ~API changes companion pull request created.~
[x] Commits are signed per the DCO using --signoff.
[ ] ~Public documentation issue/PR created.~

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

@kotwanikunal can you provide the testing details?

@kotwanikunal can you provide the testing details?

Sample Run: https://github.com/kotwanikunal/k-NN/actions/runs/11718170042/job/32639030059
GHA delete confirmation: https://github.com/kotwanikunal/k-NN/pull/3


add rebus

Associated Issue: #34
Please get familiar with the contributing guide before submitting a PR.
Summary of Changes

Add rebus for Hot Air Balloon 🔥 🌬 🎈

Pull Request Test Coverage Report for Build 87

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

Totals

Change from base Build 86:
0.0%

Covered Lines:
87

Relevant Lines:
99

💛  - Coveralls

Nice, thank you! 🙂


feat: support vertical text align for bound containers

⚠ WIP ⚠
for https://github.com/excalidraw/excalidraw/issues/4586
https://user-images.githubusercontent.com/11256141/155977569-61945bbf-6ce7-44dc-a2ab-9f576cb78eda.mp4
TODOS

[x] Support vertical-align for bounded text containers

[ ] Finalize the icon for vertical alignment, currently, I am using the align icons

Updated the icons to ☝🏻

I'd rename verticalAlign to textVerticalAlign, since the former is too generic.
A few bugs:

the text align icons don't have active state when wysiwyg active

the text align is reset to top after you edit

bottom vertical align acts incorrectly in specific cases, such as this:

create a sticky note of single line of text, and exit wysiwyg
set vertical align to bottom, and horizontal align to left or right
edit again, and add multiple new lines

I'd rename verticalAlign to textVerticalAlign, since the former is too generic.
A few bugs:

the text align icons don't have active state when wysiwyg active

the text align is reset to top after you edit

bottom vertical align acts incorrectly in specific cases, such as this:

create a sticky note of single line of text, and exit wysiwyg
set vertical align to bottom, and horizontal align to left or right
edit again, and add multiple new lines

I have pushed fix for the above issues, can you try now ?

Updated the icons to ☝🏻

one weird thing though, the vertical-align icons for right and center look different in your video @dwelle, any idea why ?


Add a way to dynamically choose singular/plural noun form depending on a specified number

My current idea for this is to introduce a plural-sensitive subtype specifier using two periods instead of one.
My name is <name> and I am [n:1;100] <timenoun..s-unit> old.

Rant will keep track of the last printed number and check its value when this specific subtype specified is encountered on a query. If the number is one, the subtype falls back to index 0 (or if the user included a regular subtype as well, falls back to that instead). If the number is not one, the term specified by the plural-sensitive subtype is chosen.
The desired behavior is shown by the following outputs:
My name is John and I am 43 years old.
My name is Kevin and I am 1 year old.
My name is Jimbob and I am 100 years old.

This seems seriously useful, but the syntax is a little weird. I really can't think of a better alternative though, so pending one I'd like to see this as proposed.


Add explanation when .translate is at the end of Transform

In the examples on https://observablehq.com/collection/@d3/d3-zoom I see that you are adding .translate twice and to the end:
d3.zoomIdentity.translate(width / 2, height / 2).scale(40).translate(-x, -y)
In the docs it says that the order matters. It would be nice to add an explanation to the docs.
Thanks for your amazing work!

I read "order of transformations matters" in the context of applying a transform as a css style or attribute: transform="translate(x,y) scale(k)".
In the notebook you're mentioning the explanation for the double call to translate is that, when we call transform.translate(x,y), the actual elements t.x, t.y of the transform t will be incremented by t.k×x, t.k×y, like so:
d3.zoomIdentity
  // translate
  .translate(300, 200) // { k:1, x: 300, y: 200 }
  // scale (and keep the translated point unchanged)
  .scale(40) // { k: 40, x: 300, y: 200 }
  // translate (taking into account the scale)
  .translate(-1, 1) // {k: 40, x: 260, y: 240}

The relationship between these numbers is described in the matrix https://github.com/d3/d3-zoom#zoom-transforms ; I'm not sure what to add to make this more explicit? Maybe in https://github.com/d3/d3-zoom#transform_translate?

So maybe  https://github.com/d3/d3-zoom#transform_translate should read:

a transform whose translation tx1 and ty1 is equal to tx0 + tk x and ty0 + tk y, where tx0 and ty0 is this transform’s translation.

instead of

a transform whose translation tx1 and ty1 is equal to tx0 + x and ty0 + y, where tx0 and ty0 is this transform’s translation.

?


Mennekes Amtron Xtra/Premium wallbox support

Is your feature request related to a problem? Please describe.
Latest firmware update of Mennekes Amtron Xtra/Premium (with HCC3 controller) enables modbus support, I can't find any application offering support for this yet.
Describe the solution you'd like
Please add support for this wallbox charger.
Describe alternatives you've considered
Complete modbus library description for the wallbox is found here: https://update.mennekes.de/hcc3/1.13/Description Modbus_AMTRON HCC3_v01_2021-06-25_en.pdf
Linked from this site:
https://www.chargeupyourday.de/services/software-updates/
Also added the PDF-file here
Description Modbus_AMTRON HCC3_v01_2021-06-25_en.pdf
.

Gute Idee, können wir gerne mit aufnehmen!

@stiajoe can you contact me at<EMAIL_ADDRESS>for remote access?

Note: phase switching is not supported over Modbus. Only reading the configured phases is documented.

@stiajoe any update?

I’m afraid not. Going to pick up where I left during next week. Hopefully people are back to work and on site after the Christmas vacation so they can do some testing.
Happy New Years - Stian

des. 2021 kl. 16:13 skrev andig @.***>:

@stiajoe any update?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.

There is also a variant of the Mennekes boxes called Amtron Professional, which uses a different set of modbus registers (cf. PDF).
I have started an implementation here: https://github.com/matspi/evcc/blob/amtron-professional/charger/amtron_prof.go
Would it make sense to incorporate these changes into this module/PR or would you rather create a separate charger implementation, i.e. should I open a new issue?

Registers look completely different- I'd suggest separate implementation.


japonicus geometry scaling

abolished geometry scaling
Email from Snezhka for PMID:30664646 (check the paper for def, placement, etc. - maybe has_output abnormal cell morphology?)

abolished geometry scaling FYPO:0007783


Add ability to only record audio, with start/stop being activated by one hotkey

I'd like to suggest a feature that allows one to capture just audio. An example scenario would be whenever I want to record the audio of some video online in a different language to add to an Anki deck of that language, it would be nice to be able to start and stop a recording of that audio simply with two presses of a single hotkey. I don't necessarily want to capture the screen, I would just like to be able to record the audio. I have yet to find an application that fulfills these exact parameters that I want.

Kap (and Aperture) currently records sound from your microphone or other input devices. I don't think we have an option to record system sound, so we'd have to look into that as a possibility before something like this would be feasible.

Well actually, using apps like Loopback, I am able to use system sound as a custom input device, so that's not a problem for me. I would just like an option to only record the audio from the input device with no screen capture. And as said before, it would be nice to be able to start and stop the recording with one hotkey.

Curious, @oudajosefu are you using Anki + MIA Dictionary add on?
@karaggeorge this app won't need to worry about the input device, as this is solved by BlackHole.
I would also like an audio option only, and the ability to start/stop recording using a global keyboard shortcut.

We've decided not to pursue this feature for now. We want to remain focused on improving our current feature set, and from out point of view this adds overhead that would require resources we'd rather invest in making recording and sharing screen captures even faster, and to continue improving our plugin ecosystem.
A dedicated audio recording tool paired with video editing software that allows you to manipulate the audio channels would serve your example use case better.

I can understand. Sadly there is no audio recording tool that starts/stops using a keyboard shortcut. They all require a convoluted interface and aren't made for "fast" recording workflows.
However, this app is perfect for this kind of workflow.

I decided to build this tonight as I've been looking for a small project to finish in an evening.
Recordia - build 1.zip

It records at 48000 hz to either M4A (AAC, compressed) or WAV (PCM, uncompressed). You can set a custom keyboard shortcut to toggle recording.
Some things missing (but planned):

Launch at login preference.
Set default output directory.
Pause/resume recording (also with keyboard shortcut).

Feedback welcome if you notice anything else missing or things that could be improved.
I'm quite busy for the next month, but I plan to finish it and put it in the App Store in October (free).

I've spent two days trying to make a mac automator workflow using nodejs, and you've built a functional app in an evening.
:sob:

I decided to finish it, so it's now in the App Store: https://apps.apple.com/app/id1529006487

@evolutionxbox Yes, this issue was originally opened because I was trying to find a way to record audio like Windows users do in ShareX for use in Anki and the MIA Dictionary add-on. I am really glad to see that it was finally completed. At one point, I also tried to make a dedicated app to accomplish this function, but unfortunately couldn't get over the learning curve with everything happening at the time. @sindresorhus Appreciate your help, I'll definitely be using it from now on!


Use correct config value for the page size for queueable jobs

See concrete/config/concrete.php

Thanks!


[Sb} Fix the Scope In SB Management Client

Use the right scope when using TokenCredential and enabling forwarding on subscriptions. This fixes a bug where an error would be raised when using token credential to create a subscription with forwarding.

/azp run python - servicebus - tests

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

/azp run python - servicebus - tests


[Question]: Is there something like an Owned Graph?

Thank you very much for the crate! I just started using it and wanted to ask for something.
Request
An owned version of tabbycat::Graph.
Details
I have a data structure and want to generate a tabbycat::Graph to represent it.
The labels, ids, and attributes will be created during the representation, so they are not a reference to anything that my data structure has (I have fields that implement Display, but I do not want to store their representation beforehand). Therefore, it would be better to pass the owned data to tabbycat, but I can't.

+1 to this, it seems like by far the most common use case, and I don't see a way for tabbycat as currently designed to handle it. This seems to have been designed with premature memory optimization -- is it really likely that an application creating (not even rendering) dot graphs is going to be memory limited?

Hi.
I see the conversations here and I am seeking for maintainers for this project (or a new owner if anyone is interested). I am happy to accept PRs. Sorry for my slow responses. I am also sorry that currently I still cannot attach a high priority for this project. Hopefully I will sort everything out in several weeks.
As for the ownership issue, I agree with the idea that purely borrowed strs are indeed too restrictive for the usage and owned values should be a better choice. I would like to do a version bump after the PR. The maintenance is long overdue.
Feel free to make any suggestions.
Enjoy.
Sent from ProtonMail mobile
-------- Original Message --------
On Apr 14, 2022, 12:43 AM, Josh Burkart < @.***> wrote:

@.***SchrodingerZhu Would you accept a PR changing everything to be owned?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Great thank you @SchrodingerZhu; I'm excited to help out. Could you please give me permission to push branches other than master, so I can make PRs for you to review?

Great thank you @SchrodingerZhu; I'm excited to help out. Could you please give me permission to push branches other than master, so I can make PRs for you to review?

Forks should also work. IIRC, private repos do not have such fine-grained access control.


Deprecate cupy.round_

numpy.round_ will be deprecated in NumPy 1.25.

/test mini,head

/test mini,head

CI failures are not related.

I guess the test part needs to be updated (as NumPy 1.25 emits DeprecationWarning) in v12 branch as well.


[TEST] Make IE11 fixes also target Edge

This'll probably need a script to add a style if IE or Edge are detected, unless there's a hack to detect both with CSS.

:+1:


Test configuration can have too much impact on unrelated tests

Describe the bug
This is basically an extension/write-out of the fix that was done in this commit
A whole set of tests have impact on each other when config is changed.
If a certain test requires configuration to be altered (For example, a new spring config is added or an 'older' one is removed), this could have impact on other tests that do explicit checks on the number of returned beans.
In the past this was 'quick-fixed' by simply altering the indexes used in the actual test, but that is not a sustainable solution.
Expected behavior
Altering configuration with relation to a test should not have any effect on other tests.
The configuration should be clearly outlined to be specific to a subset of tests, so the 'indexes' of the objects don't need to be altered all the time
Proposed fix
We want to keep the configuration of certain tests completely separate from other ones, so that the usage of indexes / checks of nr of objects can be shielded.
A solution for this is to use more specific 'mock/bogus' spring beans that are specific to a certain test suite
For example:

For the tests in 'ExternalSourcesRestControllerIT', we can create external sources that support entity type: journalExternalSourcesRestControllerIT or something like that, so other tests will NEVER use that specific bean

If we can do this is a predefined convention and start fixing this for all tests that sin against these conventions, we can stop new ITs falling back into the same problems as before
Related work

There are a bunch of different commits that alter the actual indexes etc just to fix some tests, an example is the one mentioned above

@jonas-atmire : Thanks, and I do agree that there's more that can be done to make our ITs more "standalone".   I'll pull this over to the 7.5 board in case anyone wants to start work on this.
That said, I do want to make sure that this is either estimated or better scoped though before it can be "paid work". The scope of this ticket is still quite vague, it simply describes a common problem and one example.  So, while volunteers are always welcome to chip away at this problem (by fixing problematic tests they run across), I think I'd need a clearer estimate before we can pay for it out of the DSpace budget.

@tdonohue
Thanks for the notes, I agree that this issue is indeed still a bit vague and needs to be more scoped out so it doesn't become a tangle of random changes and 'possible' issues.
I'll be expanding it with more and more examples as we come across them or if we find out issues that could have impact on other tests.
That way we can make a distinction in:

Configuration that could have impact on other tests
Tests themselves that have impact on other tests
Other possible issues that can break unrelated tests

I'll strive to get a streamlined explanation of a suite of test-problems, and then create an estimate for this
Extension
Another one of those examples for now is the OrcidQueueConsumerIT that has a hardcoded 'non bogus' handle, that can have an impact on other tests.
This does NOT fail on github for some reason, but does for us locally due to a certain order of test-processing where 12456789/200 is already taken up by another test it seems
I see that in the past this was handled by creating tests with a provided handle like '123456789/testHdlResolver' that we know will never be created prior to that particular test.
We need to make sure to double-check all newly created tests that all of the 'handles' are unique etc

#8597 fixes three issues related to this ticket, all of which were discovered when the ordering in which our tests execute in GitHub CI changed recently. (Occasionally, when GitHub CI updates its images, the ordering in which tests run does change. This can result in flakey tests failing consistently.)
Issues were fixed in both OrcidQueueConsumerIT and in ExternalSourcesRestControllerIT (each of these were mentioned by @jonas-atmire above).  Another flakey test issue was also found & fixed in CreateMissingIdentifiersIT

Another example of where this happens often:

Add a new searchFilter bean to discovery.xml
add said bean to the sideBarFacets of the defaultConfiguration

-> This ends up in multiple ( something around 25 currently) tests failing in DiscoveryRestControllerIT
This happens because a lot of these tests are hard-coded with the default DSpace filters.
I usually solve this issue by overwriting the discovery.xml in src/test/data/dspaceFolder/config/spring/api of the dspace-server-webapp module, but I am unsure if that's a good general fix to apply.

@jensvannerum I would suggest that the best thing to do would be for the test suite to create its own Discovery configuration and inject that into the code under test, ignoring the one that Spring builds.  However, the code under test may not (currently) be designed to permit that.
We have a lot of code which is unintentionally structured to defeat good testing practice.  If we are going to be test-driven, we need to do a lot of refactoring so that certain collaborators (such as configuration objects) are provided rather than linked by the classloader.


make it work with MAMP

Hi there, just wanted to know if this also works in combination with MAMP?

@hirasso Never tried it. But this should be doable. So would you like to use it for all your mamp prjects, or only selected ones?

Did some googleing: https://github.com/jozefizso/mamp/blob/master/apache/conf/extra/httpd-autoindex.conf
So autoindex is enabled on mamp. You could simply place it in your project folder, or edit your gobal file.

Yes I want to have it for all my MAMP projects... So, how would I go about it? Would I just have to overwrite this file with your file that is configured like you say under "Global Installation" ? Sorry, I really don't have any idea about server configs ;(

Me neither ;) Yeah that what I would have tried.
PS: if you are a OSX user I don't see why you even need MAMP though.

I use MAMP for its mysql skills. What I have found so far: you have to uncomment line 563 of MAMP/conf/apache/httpd.conf, so that MAMP actually uses it.
Include /Applications/MAMP/conf/apache/extra/httpd-autoindex.conf

downloading mamp now ;)

actually, it seems to not even need the mod_autoindex file. I just put the .htaccess file .theme folder you provided to my root folder and removed the /($folder) entries inside the .htaccess. Works now! :)

super cool! Would you like to edit the readme to add instructions for mamp ?

Would you mind testing the same thing on your machine first? Not that I have missed something...

I did it works.


any better loading?

can we use require path of the file to load the asset?

Sorry, I'm not sure how you could require and svg file. You wouldn't be able to use a "real" svg file anyway, as it would have to be transformed to remove the outer "cruft" markup.
A build process could take a folder of SVGs, transform them and create the svgs object.
Out of interest, what don't you like about the current method?

I think OP wants something like this:
// ./icons/Email.js
import React, {Component} from 'react'
import SvgIcon from 'react-native-svg-icon'
import { Path } from 'react-native-svg'

class IconEmail extends Component {
  constructor(props) {
    super(props)
    this.svg = {
      email: {
        svg: <Path fill="#000000" d="M4,4H20A2,2 0 0,1 22,6V18A2,2 0 0,1 20,20H4C2.89,20 2,19.1 2,18V6C2,4.89 2.89,4 4,4M12,11L20,6H4L12,11M4,18H20V8.37L12,13.36L4,8.37V18Z" />,
        viewBox: '0 0 24 24'
      }
    }
  }

  /*
   * https://github.com/stowball/react-native-svg-icon/issues/11
  */
  setNativeProps = (nativeProps) => {
    // do nothing here, and that seems to work
  }

  render = () => {
    return (
      <SvgIcon name={'email'} {...this.props} svgs={this.svg} />
    )
  }
}

export default IconEmail

Usage:
import React, {Component} from 'react'
import { View, Text } from 'react-native'
import IconEmail from './icons/Email'

export default class MyForm {
  constructor(props) {
    super(props)
    this.state = {}
  }
  render = () => {
    return (
      <View>
        <IconEmail fill='#ccc' />
        <Text>I am a form. lol.</Text>
      </View>
    )
  }
}

I guess this way of loading 'assets' is quiet common, but whatever, those are just my 2 centz.