Application breaks when run on empty cluster: Noused_capacity field. 🐞

Problem
yunikorn-history-server breaks when trying to run on an empty cluster.
How to reproduce

Install a fresh K8s cluster (I used Kind)
Install ynikorn using the helm chart
run yunikorn-history-server using ./build/event-collector
Also check the response of /ws/v1/partitions. Example: curl -X GET http://localhost:9889/ws/v1/partitions

Details
In a newly created k8s cluster yunikorn REST API does not return the used_capacity field. But the following code depends on used_capacity filed.
https://github.com/G-Research/yunikorn-history-server/blob/684fe1a9d341057d69f0e4250e085a0654ba6b2f/internal/repository/partitions.go#L35-L44
Response from ynikorn API:
yunikorn endpoint: /ws/v1/partitions
[
{
"clusterId": "mycluster",
"name": "default",
"capacity": {
"capacity": {
"ephemeral-storage":<PHONE_NUMBER>84,
"memory":<PHONE_NUMBER>,
"pods": 110,
"vcore": 5000
}
},
"nodeSortingPolicy": {
"type": "fair",
"resourceWeights": {
"memory": 1,
"vcore": 1
}
},
"totalNodes": 1,
"applications": {
"total": 0
},
"state": "Active",
"lastStateTransitionTime":<PHONE_NUMBER>159504400
}
]

Note: If you deploy anything in the cluster, the yunikorn API will return the used_capacity field. Even if you subsequently remove all resources from the cluster, this field will persist. This oversight may explain why this bug has been overlooked.

@sudiptob2 Thank you so much for the reporting! I am going fix this with my upcoming PR.

it should be fixed now, let me know if this has been resolved


Cvat uploader cypress

Resolves https://github.com/Weed-AI/annotation-platform/issues/28

the docker-compose here requires pre-building an image. Rather, we should merge in the submodule branch

That branch is still using original image. All these changes of cvat and cypress are only on dev. Will move to prod when it's working

It's fine to update the commit ref in the submodule branch. OTOH, @spikelynch hasn't updated .github/workflows/frontend.yml to use the submodule, and we should open a PR to test it separately.

I'll update the workflows on the submodule branch.

closed by #612


foobar

Description of the change:
Motivation for the change:
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

foobar


Admin cannot access programs without going through preview

Describe the bug
When I login as a program admin via dev tools I cannot see the full list of programs until I complete the program I landed on.
To Reproduce
Steps to reproduce the behavior:

Load the homepage
Click the dev tools and login as a CiviForm admin
See program page with no way to get to the list of programs

Note: This bug seems inconsistent so you might need to load up a specific program page first (eg https://staging-aws.civiform.dev/programs/test), then go to the home page, then login as an admin via dev tools.
Expected behavior
When logging in as an admin I expect to see the page with the list of programs.
Screenshots
Video: https://github.com/civiform/civiform/assets/24319951/e1cb5e08-98e1-4afe-919a-31c4fc896b29
Desktop (please complete the following information):

OS: iOS
Browser: seeing this in both Chrome and Firefox

This is very odd. Looking into it now. There's no feature flag

I think the issue might be with the REDIRECT_TO_SESSION_KEY value getting set when the program is loaded. programBySlug() sets the redirect value to itself and then it's still set when logging in as an admin and it gets read in the CallbackController which sees that the value is set and redirects to the programBySlug page. I'm not sure why we didn't have this problem before though.

I checked out an old commit (c0212a457ec2c38c660cbe9f8e199fd6f2b2175f from June 22) and confirmed this was a problem before any of the program preview changes. I'll work on a patch.

Generally, REDIRECT_TO_SESSION_KEY is set very optimistically and is only ever cleared if the user goes through the expected path of signing in after loading a program by the slug. It's set even if the program can't be found and isn't cleared if the user goes to the home page or to another program's page by ID instead of by slug. We should revisit this logic.


Result filters get executed in case of model-binding and/or model-validation failures.

Go to Filter types
Under Result filters, the 2nd point: Run only when the action method executes successfully seems incorrect, because result-filters gets executed even in case of model-binding and/or model-validation failures. But action methods doesn't executes in case of model-binding and/or model-validation failures.
Image for reference:

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

ID: a70a955f-dd5a-c46b-dfe8-21521e619320
Version Independent ID: ebb299e0-6c1c-7077-689d-ac711352e418
Content: Filters in ASP.NET Core
Content Source: aspnetcore/mvc/controllers/filters.md
Product: aspnet-core
Technology: aspnetcore-mvc
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Can you provide a link to a public gitHub repo with the simplest possible sample that shows the problem?

Hey @Rick-Anderson! just wanted to know if this is a valid issue? Also, do let me know if I can provide any more info.

In this context, I think "executes successfully" means that there are no exceptions. Model-binding failures, etc, aren't exceptions, and so the Result Filters run.


title fix

There was an error while validating job 495d2e1d-1398-454f-94ed-180f600187c6 Message:
Result status: Failed
There was a problem accessing Github repository 'https://github.com/Microsoft/cordova-docs.git'.
Details:Microsoft.LightweightJobEngine.Core.Language.ContextCommandBase+ContextCommandException:
Result status: Failed
There was a problem accessing Github repository 'https://github.com/Microsoft/cordova-docs.git'.
---> System.AggregateException: One or more errors occurred. ---> System.ArgumentException: Error clonning repository ---> LibGit2Sharp.NotFoundException: Branch 267 not found. Available branches: origin/master
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetBranch(Repository repo, String name)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass30.<>c__DisplayClass34.b__29(KeyValuePair2 branch)    at System.Collections.Generic.List1.ForEach(Action1 action)    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass30.<CloneRepository>b__28()    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.<>c__DisplayClass1.<ExecuteAction>b__0()    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction[TResult](Func1 func)
at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction(Action action)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)    --- End of inner exception stack trace ---    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository()
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetVersionAsync(String path)
at Microsoft.DPS.Commands.IdentifyDocumentsToPublish.CheckExternalServicesCommand.Do(IDictionary2 context)    --- End of inner exception stack trace ---    --- End of inner exception stack trace ---    at Microsoft.LightweightJobEngine.Core.Language.ContextCommandBase.ThrowIfErrorsLogged()    at Microsoft.DPS.Commands.IdentifyDocumentsToPublish.CheckExternalServicesCommand.Do(IDictionary2 context)
at Microsoft.LightweightJobEngine.Core.Language.JobWorkflow1.RunCommands(Guid jobId, IDictionary2 context, CancellationTokenSource cancelSignal)


Add .arduboy upload support

Uploading .arduboy files is significantly more complex than uploading sketch files, so it should be its own command. Perhaps something like ardugotools write package name subprog device, where you must provide the correct subprogram within the package (since there can be many). There should be an "auto" where it will select the first option which correctly matches your device. Consider where to extract the arduboy files temporarily though (can it be done in memory? do we WANT to do it in memory?)

OK, instead, I think we should do this:

Add a command which lets you fuse an fx bin and save together
Add a command which parses the fx cart from a FILE and outputs the results as.... a folder structure?
Add a command which will "recompile" the folder into a flashcart
Add a command which will export a .arduboy file into a folder scheme that is compatible with the export

oh also: https://stackoverflow.com/questions/20357223/easy-way-to-unzip-file


Show the change logs of updated package

I wonder if it's possible to grab the change logs of updated packages after running
pipenv update
Most of packages will have change logs in their repo. e.g

https://github.com/kennethreitz/requests-html/releases
https://github.com/pytest-dev/pytest/blob/master/CHANGELOG.rst
https://github.com/urllib3/urllib3/blob/master/CHANGES.rst

It will be more productive if I could have the latest updates via CLI.

I like the idea a whole lot, although I have to say it's definitely awhile off for implementation -- we have a bunch of work to do on some core functionality before we are on this type of thing
could be something someone might be able to get together or find a library for, not totally sure what's out there

Big c


Language support

Doesn't support Cyrillic, Hindi, Chinese and other languages. Will their support be added?

yes, i need support language too

and Arabic language too

`

`

great！thank you


Blog post on our internal projects

IMPORTANT: blog posts must be approved by two reviewers before they can be published, one of which must be a native English speaker.
This is a basic post running down some of our internal projects and why / how one might choose to have internal projects

@rdormer Hey, please take a look at the comments when you have some time. I can publish this on Wednesday. Thanks!


Add integration tests to save advance configs under x-wso2 production and sandbox endpoints.

Public fix : https://github.com/wso2/carbon-apimgt/pull/12196

Merge the fix first and rerun the git actions before merging this PR.

Public fix was merged.
Retriggered the test builds under github checks.


support array response schemas

closes #8
I don´t really understand why only type, properties and required we´re allowed, please elaborate on that if it´s important.

Anyone there to review this? I would really like to use the official package rather than my local version.

Array support is very much needed!

@SaltyAom sry for ping, but have you considered this? We need array support!

hi @david-plugge @BE-CH !
I would love to review this, there is some conflicts in src/utils.ts would you be able to update the PR and I can review :)
i'm a new maintainer helping out!

i realize this might be a hard conflict to resolve since the new release, let me know if I can help at all!

Awesome, i will resolve the issues as soon as possible :)

@david-plugge Can I finish this pr for you?

@david-plugge Can I finish this pr for you?

Feel free to do so :)


507 create datasets form improvements

Made changes according to the description section in the issue.
I realize we can overwrite the default error prompt so I replaced it with the MUI alert component; also removed the "." in the error message.

please make description a text box / taller box.


fix: get last match from preg_match for domain/branch naming

Consider
$branch = "feature/foo-bar-abz";
$pattern = "/feature/(.*)/";
if (isset($pattern)) {
preg_match($pattern, $branch, $matches);
}
$matches contains
Array
(
[0] => feature/foo-bar-abz
[1] => foo-bar-abz
)
The last match seems to be generally what we’d want.
Tried to write a test for this, but the test/setup work seems a bit more complicated than I expected.

@mehrancodes ping

@mehrancodes I can't think of any right now.  Looks great!  Thanks for the tests!  I couldn't figure out how to do a full setup of the forgeservice to test against, but you just redid this to not require it at all - better option!


Fixed resource_named for Ruby 2.3

In Ruby 2.3, our methods to convert a string to a class name had issues. This fix should also be a lot faster; we're no longer looping through the entire ObjectSpace.
I recommend releasing after this fix.
cc @hdiomede , @tmiotto

:+1:

:+1:


Mid point for an unbounded interval

According to the standard, the mid point of unbounded intervals, as well as for empty intervals, should return no value at level 1; cf. table 10.2, in page 39 of the standard. Yet, at level 2, a value can be fixed, for instance for entire intervals.
The question is why (in the ITF1788) test suite mid(Interval(0,Inf) the largest finite floating point?
See https://github.com/JuliaIntervals/IntervalArithmetic.jl/issues/107
cc @dpsanders

It is defined in Section 12.12.8 of the standard.
Also in the simplified version IEEE 1788.1, you can find the same definition in Section 6.7.6 (http://grouper.ieee.org/groups/1788/email/msg09048.html).


Managed Client E2E test parity with JS/HTML client

Need to bring test coverege of Managed client E2E test to parity JS/HTML client..

Part of this issue was fixed in merge
https://github.com/Azure/azure-mobile-apps-net-client/pull/13


Retract package-operator.run/package-operator

Summary

Change Type

Check List Before Merging

[ ] This PR passes all pre-commit hook validations.
[ ] This PR is fully tested and regression tests are included.
[ ] Relevant documentation has been updated.

Additional Information

we need to double check if that is actually working before merging :D

You can retract versions, you can't retract the whole module :(


Duchies can enter a state where they add an unbounded number of log entries

Split off from #840
https://rally1.rallydev.com/#/?detail=/defect/685530265769&fdp=true

Kingdom DB query to identify Measurements that have an abnormal number of log entries:
WITH Counts AS (
  SELECT COUNT(*) cnt, MeasurementConsumerId, MeasurementId
  FROM MeasurementLogEntries
  GROUP BY MeasurementConsumerId, MeasurementId
  HAVING cnt > 100
)
SELECT Counts.*, ExternalComputationId, State, CreateTime
FROM Measurements JOIN Counts USING (MeasurementConsumerId, MeasurementId)
ORDER BY cnt DESC;

It looks like all the instances we're seeing are for Measurement that are in a terminal state. The system should not allow MeasurementLogEntries to be added for Measurements that are in a terminal state.
Additionally, any changes to child resources of Measurement (e.g. Requisition, ComputationParticipant, MeasurementLogEntry) should probably update the UpdateTime of the Measurement.


Addes back to home

I added a link on 404 page which redirects user to the home page of the website which may help people to back to home and explore the website!

I think it may help the team! Thank you

Is this project still active @github

Thanks for merging!

@all-contributors add @mjvbz to contributors

@all-contributors Please add @mjvbz for code


Bug: weird shifting of the ai pr description button

Describe the bug

I have to click thrice, to catch up and finally generate a description, because of the hidden H button.
Steps to reproduce
GIF attached.
Affected services
browser-extension
Platforms
No response
Browsers
No response
Environment
No response
Additional context
No response
Code of Conduct

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

Contributing Docs

[X] I agree to follow this project's Contribution Docs

@diivi, can you tell me how to reproduce this? I am unable to. How can I get the H button to stay hidden at first and appear later. Like in the GIF. For me, it works just fine.

@diivi, can you tell me how to reproduce this? I am unable to.

I didn't really do anything other that what you show in the GIF 😅, I'll try to reproduce this in incognito mode and let you know, maybe there's another extension interfering here or something.

@diivi, can we close this as we haven't been able to reproduce the issue?


Is there anything for which contribution is required?

Is there something through which I can contribute to this project?

If you have a question about a proposed change, create an issue, and I'll comment. If you want to merge code, make a PR, and I'll review it.


Speed up checksum calculation for items

I noticed that roughly 80% of my site's compilation time is due to checksum calculation. While #789 eliminates some of that time, it still weighs on the speed of nanoc.
I think that (some) items can be smarter about their checksum calculation. For instance, an item from the filesystem could generate its checksum based on the SHA1 hash of the file contents, rather than having to parse the entire file and calculating a SHA1 hash of all fields, as is done now. Other data sources could calculate checksums even faster. For instance, the BibTeX datasource could generate one checksum per .bib file, instead of having to generate a checksum for each of the individual entries in that file.
I would like to implement this idea and submit a pull request, but I'd appreciate some guidance on how to implement this. In particular, I'm wondering about the following:

Currently, Nanoc::Int::Checksummer is responsible for its checksumming all other classes. However, this makes that component rather omniscient, in contrast with the principles of object-oriented design. Wouldn't it be good that each entity is responsible for checksumming itself?

If so, this would mean we need to extend basic classes like String with a checksum method. This might or might not be desired.
An alternative is to implement Checksummer such that it first checks whether the object supports a calc_checksum method and if so, calls it.

I see two ways in which a datasource could work:

Either it would generate a field such as :checksum, which the checksum method then uses if it exists.
Otherwise, it would directly implement the calc_checksum method on the item it returns, either by monkey patching or subclassing. However, I wonder to what extent subclassing is desired, given that Item has moved to the Nanoc internals.

On the theme of subclassing and speed: having subclasses of Item would also allow us to have dedicated methods to access properties, which would allow lazy loading. Alternatively, the new_item method could accept a block as parameter, which is only executed when the properties are really needed.

What does a calc_checksum look like? Does it take a digest as argument, which it then updates? Or does it take no arguments?

Implemented in #793.


SNOW-902662: DF.to_pandas_batches() batch size parameter

Current behaviour
When using DataFrame.to_pandas_batches() it returns a Pandas Dataframe Iterator that generates Pandas Dataframes with a "random" number of rows.
Desired behaviour
I would like to insert a parameter in the to_pandas_batches() method, where I fix the number of rows for each Pandas Dataframe generated.
How would this improve snowflake-snowpark-python?
This would be beneficial since the user would be able to control the chunk sizes to process and be sure that his processes don't get overloaded by the snowflake back-end's calculation of the number of rows.

I am also interested in this feature


Proposal of struct size getter

It would be convenient to have a way to get the total size of a struct parser at compile time
something like:
struct(myparser):
  u1: flag1
  u2: flag2
  u4: field1
  u4: field2
  u13: field3

const packetsize = myparser.encodedsize # 24 bits

after rewiring my code I now see how a future implementation of SocketBitStream would solve this problem the right way, as encoded struct size is only used when alternating binarylang with manual handling of socket recv


[Feature Request] Want add the global configure

Existing Component
是
Component Name
Many like Dialog
Description
I want config some component's parameters globally to make most of the component customized, but now can't attained.

You can wrap Dialog up into your own custom component, register it globally, and use that component instead.

But that is too fussy. And this would be used by many developers.


All Images Preloaded When mmenu Plugin is Used on the Site

We're attempting to implement Lazysizes on our site and are seeing an issue that we've been able to isolate as only occurring when the code for the popular mmenu plugin is also being used on the site.
What is happening is shown in this video.  The class of "lazyload" and "lazyloaded" updates as you scroll as expected when looking at the source code but the images themselves are already showing regardless of the status of that class. Additionally, data-src and other data-elements are showing as expected in the source as well.
It seems that something about the way mmenu is manipulating the DOM is causing the images not to lazyload, even though the source code seems to be updating as expected. That plugin is a very popular one for mobile menus and one that we would like to continue using so am hoping that there's a fix here that would enable us to continue using that plugin, along with Lazysizes.

Turns out mmenu wasn't actually the issue with lazy loading on our site.  Once I fixed that, it fixed the issue.
For some reason, removing the mmobile plugin also caused lazy loading to work again, even when that data-src code was in there. Not sure why that was the case. Anyway, mmenu and lazysizes are playing nicely now that the other data-src code has been fixed.


Tenant Level Customer Key Encryption

[Enter feedback here]
Please confirm if encryption is applied to public folder mailboxes and shared mailboxes in Exchange online as well.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 18803bfa-118a-09ee-947b-abf1ee996b7a
Version Independent ID: 87179b6d-d917-1aa9-259a-6f2de072c049
Content: Customer Key for Microsoft 365 at the tenant level (public preview) - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/customer-key-tenant-level.md
Service: o365-seccomp
GitHub Login: @KCCross
Microsoft Alias: krowley

@sinnamingrrl Thank you for your feedback. Yes, Encryption is applied to all contents in Exchange Online including the Public folders, Shared mailboxes, e-mail body content, calendar entries, and the content within email attachments.
Hope this helps!
Thanks
Sri

@sinnamingrrl Thank you for your feedback. Yes, Encryption is applied to all contents in Exchange Online including the Public folders, Shared mailboxes, e-mail body content, calendar entries, and the content within email attachments.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @sinnamingrrl Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@msbemba Thank you very much for the contribution and sharing this explanation. @sinnamingrrl Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


How to get insert ID?

I want to get the id of the last insert. How can I?

I don't have a project set up with this right now.
Have you tried
$db->statement('INSERT INTO ...', [...data...])
    ->then(function(\mysqli_result $result)
    {
        $id = $result->insert_id;
    });

http://php.net/manual/en/class.mysqli-result.php
I can't see there is a insert_id property in mysqli_result.

I see what you mean. I was looking in the wrong place.
I found: http://php.net/manual/en/mysqli.insert-id.php
But, this likely isn't going to work with the async nature of the code.
I'm sorry, I don't know how to get the id of the last insert.

Can the class return the mysqli object so we can get the id of the last insert?

https://github.com/M2Mobi/lunr/blob/master/src/Lunr/Gravity/Database/MySQL/MySQLAsyncQueryResult.php

https://github.com/KhristenkoYura/react-mysql/blob/master/src/Client.php
I can see some people just get the insert id from the connection when the result is back.
But I am not sure what it will happen if multiple inserts are running and returning at the same time.

Ah, I see what you mean.
So then here : https://github.com/dustingraham/react-mysql/blob/master/src/Database.php#L106
Where the line $deferred->resolve($result); is, you would need to pass in $read to the promise resolve callback. I have not worked on this code in over a year, so I don't have the setup currently to test and implement this. I'm not sure whether the resolve call would need an array, or if it can add another parameter.
You might just need to add the parameter so $deferred->resolve($result, $read); and then add the mysqli object to  ->then(function(\mysqli_result $result, $mysqli) but I can't say for sure. That does look promising based on the link you provided.

Ah, yes, what you just pointed out seems good as well.
I think it would work because you're calling the insert id before releasing the mysqli that is being reaped.

Thanks. Let me try.


Admins can activate archived channels - resolves #157.

Not sure if the RouteServiceProvider approach is really the cleanest, but I think I prefer it to doing withoutGlobalScopes() in two places.
Cheers!

Just putting this here so it doesn't get lost, can then replace withoutGlobalScopes() with withArchived() if/when #142 is merged


Windows 7 'vagrant up' does nothing.

Issue Type

Bug Report / Support Request

Your Environment

vagrant --version && vboxmanage --version
Vagrant 1.9.7
'vboxmanage' is not recognized as an internal  or external command

Your OS

Windows ( 7 ) 64 bit

Full console output

Command ran
vagrant up

No console output

Summary

vagrant up does nothing .
I installed Virtualbox, vagrant just like on the quick guides. I extracted the drupal vm to Desktop and changed the default.config.yml to config.yml.
I tried uninstalling and installing Virtualbox and Vagrant but still after typing vagrant up on cmd or gitbash, it does not throw any error. I am not sure if it is running on the background but, it is just stuck  with blinking cursor.

Seems to be an upstream issue https://github.com/mitchellh/vagrant/issues/8783. Try with downgrading to Vagrant 1.9.6

Yeah, definitely a Vagrant bug. For now, install Vagrant 1.9.6 and don't upgrade until the linked issue is fixed.

Upgrading powershell should resolve the issue. The latest release (1.9.8) includes this PR #8839 that checks the powershell version and raises an error if the minimum is not satisfied. Cheers!

You can find instructions on how to update PowerShell at https://github.com/hashicorp/vagrant/issues/8783#issuecomment-328858234.
I'll close this issue


no_std support

Hiya!
From a quick cursory glance over the API of this crate it looks like it would be possible to support no_std without too much trouble, mostly just cfging all std uses with either core or alloc. Is this something you would be interested in? I could of course do so myself but thought I would ask first.
Supporting no_std would allow this crate to be more easily used on more under-powered systems and make it a tad-bit smaller when compiling as well, which would be quite handy in my use-case.

It would be great for this crate to support no_std, but as Ip{v4, v6}Addr requires std we would need to decide what to exclude when std is gone. Any ideas?

There are a few different options depending on how much of the std::net functionality you want to retain.
I think just using no-std-net as a simple drop-in replacement could suffice, but that repo seems to be a bit stale. It does however implement all the stuff you would need. There seems to have been some discussion on if the net stuff should be used in core that just petered off.
smoltcp's solution to this was to implement their own quite thorough IP types:

IpAddress Enum
v4 definition
v6 definition

You could also go the nuclear option like r-efi and have a very small IP Address implementation, but that might be too limiting.
I think from a perspective of minimizing the amount of dependencies something like what smoltcp has done would be preferable. Since the code is 0BSD licensed and actively maintained one could just copy that code into this project and replace everything related to std::net with that.


[arm.network] swagger update for grammar, spelling, punctuation

@DeepakRajendranMsft please review. This update contains description updates to the network.json swagger in api version 2016-09-01
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] Validation errors from the Linter extension for VS Code have all been fixed for this spec. (Note: for large, previously checked in specs, there will likely be many errors shown. Please contact our team so we can set a timeframe for fixing these errors if your PR is not going to address them).
[ ] The spec follows the patterns described in the Swagger good patterns document unless the service API makes this impossible.

This change is

Hi @georgewallace, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (gwallace). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

we have reviewed this on codeflow. LGTM

@markcowl @amarzavery  can you please merge this? we dont want any more merge conflicts, so want to push this in soon

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

No modification for NodeJS

 https://github.com/Azure/azure-sdk-for-python/pull/856

 https://github.com/Azure/azure-sdk-for-ruby/pull/548


[TC-VALCC-4.4] Python script has to updated by updating the variable name as per test plan for Step 11 and 12

Test Case
TC-VALCC-4.4
Description

In step 11 and 12, the variable name has to be updated from utcTime to utcTime2 as per the test plan.

Script Reference:

Test Plan Reference:

Platform
raspi

@KishokG, @andy31415 asks for clarification on this change request before approving the related PR, can you please provide more detail?

@raul-marquez-csa and @andy31415 I raised this ticket, bcoz the variable name is updated in test plan from utcTime to utcTime2(PR: https://github.com/CHIP-Specifications/chip-test-plans/pull/4008)


:wrench: Slight refactoring to improve page load times.

I reduced the production bundle size, by optimizing some imports.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 7 (0.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 20.058%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

frontend/src/components/Sidebar/DistrictMap.tsx
0
1
0.0%

frontend/src/components/ScenarioCard.tsx
0
2
0.0%

frontend/src/components/IconBar.tsx
0
4
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
231

Relevant Lines:
916

💛  - Coveralls


Mirror Ledger onto CashCtrl

Objective
Extend the CashCtrlLedger class to synchronize a general ledger entries in pyledger format onto a remote CashCtrl instance.
Detailed Description
Create a method named mirror_ledger() within the CashCtrlLedger class to synchronize a set of target ledger entries, specified as a DataFrame in the pyledger.ledger() column schema, onto a remote CashCtrl instance. Notice that ledger entries are named 'journal entries' in CashCtrl. This method will handle the addition of new ledger entries, updates to existing entries, and optionally, the deletion of entries not present in the target DataFrame.
class CashCtrlLedger(LedgerEngine):

    def mirror_ledger(self, target: pd.DataFrame, delete: bool = True):
        """
        Synchronizes ledger entries on a remote CashCtrl account with the target state provided as a DataFrame.
        Args:
            target (pd.DataFrame): DataFrame with desired target ledger entries in pyledger format.
            delete (bool, optional): If True, deletes ledger entries on the remote account that are not present in the target DataFrame.
        """

Field mapping between pyledger (left) and CashCtrl (right) includes:

'date' (datetime.Date) -> 'dateAdded'
'account' -> Look up account['number'] via ('creditId' if amount > 0 else 'debitId')
'counter_account' -> Look up account['number'] via ('debitId' if amount > 0 else 'creditId')
'currency' (str) -> 'currencyCode'
'text' (str) -> 'title'
'vat_code' (str | pd.NA) -> Look up tax['name'] via 'taxId'
'document' (str) -> 'reference'

Ledger entries do not have unique identifiers. The logic to identify entries to add, modify or delete is more complex than for the account chart or for vat codes.
Tasks

Accessor and Mutator Methods

Implement a method to retrieve the general ledger (named 'journal' in CashCtrl) in pyledger DataFrame format.def ledger(self) -> pd.DataFrame:
    """Retrieves the account remote ledger from CashCtrl instance formatted to the pyledger schema."""

Develop methods for adding, updating, and deleting accounts based on the pyledger data.def add_ledger_entry(self, ...):
    """Adds a new ledger entry to the remote CashCtrl instance."""

def modify_ledger_entry(self, ...):
    """Updates an existing ledger entry on the remote CashCtrl instance."""

def delete_ledger_entry(self, account: str):
    """Deletes a ledger entry from the remote CashCtrl instance."""

Write unit tests for these methods.

Implementation of mirror_ledger()

Ensure remote and target data are represented by data frames with identical columns and column types. Duplicate entries are allowed.

Ensure the target account chart conforms to the expected schema: `target = StandaloneLedger.standardize_ledger(target).
Retrieve current accounts from CashCtrl. `remote = self.ledger()

The algorithm needs to identify elements to keep, delete, add, modify, and perform operations using mutator methods.

This is conceptually an optimization task with the number of required changes as a cost function. However, we are free to take a more pragmatic approach; we do not need to guarantee optimality.
Our sole hard requirement is to ensure the resulting data matches (identical values, possibly arranged in a different sequence).
Soft requirement: limit the number of additions and deletions by modifying instead of deleting and re-creating similar but not identical entries.
Probably best solved by:

Discarding equal elements on both sides from both data frames (caution: possible duplicates are allowed)
Identifying elements that are close, modifying on the target side, discarding from both data frames.
Deleting remaining elements on the target data frame.
Adding remaining elements on the source data frame.

Maybe difflib.get_close_matches helps to identify similar, but not identical entries.

Extend unit tests to cover the mirror_ledger() method.

include scenarios where the target DataFrame is empty, contains NA vat_code and counter_account.
include scenarios where  vat_code is not defined
include scenarios where  account, or counter_accounts are not defined in the account chart.

Submit Pull Request

Submit the changes for review, assigning the PR to Lukas.

Test Data

Generate some initial test data in the CashCtrl GUI.
Lukas can provide a more challenging set of test data as work progresses.

Data Structure
Note that ledger data frames in pyledger can mix both single and collective postings.

Single postings are financial transactions where an amount is debited from a single account and credited to a single account. These transactions occupy a single row in the ledger data frame. A minimal definition includes the 'date', 'currency', 'amount', a credit 'account', a 'counter account' to debit, and a 'text' explanation. Optional fields include 'vat_code' and 'document'.

Collective postings are financial transactions where one or more accounts are debited and credited in a single transaction on the same date. The amounts credited minus the amounts debited must sum to zero in the base currency of the accounting system. These transactions occupy multiple rows in the ledger data frame. To denote multiple lines forming a single transaction, the date is written only on the first row, with the subsequent rows of the same transaction having NA in the date column.

In summary: subsequent rows with a missing (NA) date form a single transaction together with the last row that has a non-missing date.

Assigning ledger data to single transactions is implemented in LedgerEngine. standardize_ledger.
df['id'] = df['date'].notna().cumsum().astype(pd.StringDtype())

See pyledger/ledger_engine.py#L621

Reopened for the mirror function


Fix migration source name in rollback all

This looks like it's an oversight, probably not caught because of low custom MigrationSource usage. Until this is merged, the temporary workaround is to make sure your custom migration objects have a file property that matches their name.

@caseywebdev Could you add a CLI test for this to make sure we don't break it again?

Added

Thank you!

Released in 0.21.3


allow (sanitized) html tags for body

Is it possible to allow a subset of html tags for the message body? Currently tags are just displayed as text

According to the official docs, it seems that the body attribute is simply a string that does not resolve any HTML tags. Unfortunately, there is nothing Push itself can do to change this, as its just a robust shim wrapper around the native API calls.

ok that explains it.


As a user, I would like to copy the search parameters of my saved query

@lilimelgar commented on Mon Sep 24 2018
Issue #440 was about showing the parameters of a saved query. This shows in a pop-up on mouse over. This issue is about facilitating the exporting/copying of those parameters as text, so the user can paste them in their reports/papers.
So, add a button either in the pop-up for copying, or an extra button in the query menu (now it has Delete and Open) for the same purpose.

@lilimelgar @jblom
Issue done in labo-components branch "task/#157-copy-query-params-to-clipboard"
What has been done?
The query icon  has been changed into a button so now alongside with showing the current query on hover once the button is clicked the query itself is saved to the clipboard as shown in the following screencast
https://drive.google.com/open?id=1bqgZADoBySHENLDTESAd8oe7FRNd1NVR
The functionality has been tested in Chrome, Safari and Firefox latest version (No IE testing done). In all those cases it worked as expected.
@lilimelgar : I assigned the issue to you too for testing and once commented/approved please let me know to create a pull request to merge it into master.

@avlisodraude As we commented, it would be useful to add a text that says: "Copy to clipboard". Please let me know when done.


Squash all "getting started" pages into a single one

Description
Clearly and concisely explain the problem.
You can mention unclear or missing information, typos, grammatical mistakes, etc.
Any contributions and suggestions are welcome!
Squash all "getting started" pages together, as they are all pretty short except UMD one.
This may have bad impact on SEO for these pages, but those pages are actually used just a couple times per developer, not really that problematic IMO
Website Section URL
Website live preview, paste the page URL or section name (eg. Docs, if it happens on all docs page) below.
https://quasar.beget.it/start/pick-quasar-flavour

Move "UMD/Standalone" option to be the last too

Remove https://quasar.beget.it/start/quasar-cli and directly link to https://quasar.beget.it/quasar-cli/installation
We probably need to integrate some of the docs from the first to the second one
Push more on Quasar CLI and suggest people to use it to start, instead of leaving the user to choose without a suggestion

Let's not squash all pages to just one, for now


Canvas fallback

If someone doesn't have canvas enabled there is JavaScript errors which can cause other working scripts not to work.  With the rise of canvas blocking it would be nice if chart Js had some fallback or even a way to alert the user canvas is disabled.  The reason canvas blocking is increasing is because it can be fingerprinted by browsers and is a privacy issue for some users.  Thanks for the great library and hope you can consider.

I agree that we should gracefully handle this condition. I am happy to merge a PR that prevents a crash when the canvas does not exist.

@yonatoshi old post, but can you give more details about the JS errors and the way to disable canvas? what kind of fallback / alert are you thinking of?

Fixed in #3474


Search users by list of phone number (users/_search)

As a developer I want to search users by a list of phone number similar to inUserIdsQuery
Acceptance Criteria

[ ] I am able to search users by a list of phone numbers
[ ] Query is implemented in the management API
[ ] Query is implemented in the user service
[ ] Console is extended

@eliobischof @stebenz Can you please estimate this issue?

+1 from inbound

@hifabienne what exactly should be extended in the console?
ATM, the user list doesn't even show the phone number:

0.5 for the rest

@hifabienne what exactly should be extended in the console? ATM, the user list doesn't even show the phone number:
0.5 for the rest

I think we should have the phone number in the list and also be able to filter for it

Is this currently being worked on? If not, I'd like to work on it.

Is this currently being worked on? If not, I'd like to work on it.

It's not and would be great if you can take this over

@almaraz333 are you still working on this?

I am not. Unassigned myself a few weeks ago so if someone else wants to pick it up feel free!


dagger.io/dagger tests should follow same dir structure as universe.dagger.io

From discord:
Q. I'm not seeing any tests (.cue, .bats)  in dagger.io/dagger or dagger.io/dagger/core like we have in universe.dagger.io packages. I often look to those for example usage. Was looking just now to answer a question from user about core.#Export
A.dagger.io/dagger and dagger.io/dagger/core etc tests are currently in the [dagger/dagger/tests](https://github.com/dagger/dagger/tree/main/tests) folder.
From @aluzzardi:
Improvement: We should probably follow the same structure in both ... core predates universe, we improved the organization of tests but never backported old tests

I am going to close this for now, but we can reopen when we focus on universe


Add HKR actors

Maybe you want to pull something over but I've changed the API and core pars a lot because I don't like the idea of copying the FritzBox instance into all actor instances. Because of that I've moved all "actor-methods" into the FritzBox class to be called with the actor as argument - like:
from fritzhome.fritz import FritzBox
fb = FritzBox('<IP_ADDRESS>', 'user', 'password')
fb.login()
actors = fb.get_hkr_actors()
value = fb.get_hkr_tsoll(actors[0])

Sorry, but this PR cannot be merged:

You're breaking API compatibility without a technical reason
This also breaks the CLI  (__main__.py), which a lot of people use
You replaced the .gitignore, which had nothing to do with HKR

If you want to contribute to this project, please split your Pull Requests into smaller chunks and check for breaking changes.

No problem, just wanted to let you know there is some HKR code if you'd like to copy that parts. Did not intend to get all that merged actually.


google-generativeai 0.8.3 :snowflake:

google-generativeai 0.8.3 :snowflake:
Destination channel: Snowflake

PKG-6047
source code
PyPi
conda-forge

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_fc2ljy_0nl/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:1: patch_must_be_in_build: patch must be in build when source/patches is set.
===== Final Report: =====
1 Error and 0 Warnings were found


Forced module order to be fixed based on config input

Description
This pull request changes select_models to use a list rather than a set to find the non-duplicated model names. This means that the inputted order of models is preserved, so that models are always called in the order the user specifies them.
This doesn't fix the broader issue of how we should handle model ordering, where the information should be stored, etc. It does fix the issue of vr_run sometimes randomly stalling when a valid model order is entered.
Type of change

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[x] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Further checks

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works

Codecov Report

Merging #245 (5b63306) into develop (1fe8e44) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #245      +/-   ##
===========================================
+ Coverage    95.37%   95.39%   +0.01%
===========================================
  Files           42       42
  Lines         1643     1650       +7
===========================================
+ Hits          1567     1574       +7
  Misses          76       76

Impacted Files
Coverage Δ

virtual_rainforest/main.py
84.09% <100.00%> (+1.37%)
:arrow_up:

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


Help: networkConnections in debian / ubuntu returns empty []

The latest release, in fedora and osx works!
It returns my open local ports.
In ubuntu, also as root, returns []
Help?

@cekvenich seems that Debian (and Ubuntu) now switched to ss instead of netstat. I will add code to handle both alternatives.

@cekvenich should be fixed. Version 4.21.2 just published. Can you test it on your side?

Give me 24 hours, I will.

@sebhildebrandt  I tested new build on Ubuntu and I can confirm that it is fixed.
Thank you for rapid response. Cheers, Vic


oauth2 authentication vs swagger ui

Description
Connexion oauth2 security scheme expects a Bearer token be passed in by the client/user. This token is passed to the method defined by the x-tokenInfoFunc parameter defined under Components: securitySchemes: in the openapi.yaml.
The ability to return a list of scopes from the x-tokenInfoFunc and use those scopes to determine access to API endpoints as defined by the security parameter in the openapi spec file is incredibly useful and highly functional.
However, the swagger UI interprets this oauth2 configuration as a more traditional oauth flow and prompts the user to enter client_id and client_secret in the "Authorize" UI widget. What is the expected way to get swagger to prompt for a Bearer token to tap into the connexion oauth process instead so that users can make use of the swagger UI?
I've tried adding an additional BearerAuth: section, and while this does cause Swagger to offer an additional Bearer token input, it also causes Connexion to throw many many errors about a missing x-bearerInfoFunc during startup.
Example openapi.yaml
security:
  - BearerAuth: []
  - okta: [read]
components:
  securitySchemes:
    BearerAuth:
      type: http
      scheme: bearer
    okta:
      type: oauth2
      flows:
        authorizationCode:
          authorizationUrl: none
          tokenUrl: none
          scopes:
            read: grants access to read-only endpoints
            write: grants access to write endpoints
            admin: grants full access to all endpoints
      x-tokenInfoFunc: auth.verify_token

@Ruwann could you take a look at this?


app crashes with OnlineTileSourceBase.getBaseUrl causing a IllegalArgumentException: bound must be positive

Issue Type
[x ] Question
[x ] Bug
[ ] Improvement
[ ] Build system related
[ ] Performance
[ ] Documentation
Description and/or steps/code to reproduce the problem
I'm using offline maps (from 4uMaps) of an area encompassing the UK at zoom levels 3 - 13.
        map.setTileSource(new XYTileSource("4uMaps", 0, 18, 256, ".jpg", new String[]{}));

When I move outside of an area covered by an existing tile I get the following logcat errors:-
   java.lang.IllegalArgumentException: bound must be positive         at java.util.Random.nextInt(Random.java:388)         at org.osmdroid.tileprovider.tilesource.OnlineTileSourceBase.getBaseUrl(OnlineTileSourceBase.java:67)         at org.osmdroid.tileprovider.tilesource.XYTileSource.getTileURLString(XYTileSource.java:42)         at org.osmdroid.tileprovider.modules.MapTileDownloader$TileLoader.loadTile(MapTileDownloader.java:223)         at org.osmdroid.tileprovider.MapTilePreCache.search(MapTilePreCache.java:125)         at org.osmdroid.tileprovider.MapTilePreCache.access$100(MapTilePreCache.java:30)         at org.osmdroid.tileprovider.MapTilePreCache$1.run(MapTilePreCache.java:41)         at org.osmdroid.util.GarbageCollector$1.run(GarbageCollector.java:31)         at java.lang.Thread.run(Thread.java:764)
How can I resolve this (other than by not going beyond existing tile coverage.  Or is that they only way)?
Environment
If it's a bug, version(s) of android this affects:
Android 8 - sdk 28
Version of osmdroid the issue relates to:
6.1.0

change new String[]{} to new String[]{""}

Amazingly quick response!! - Many thanks, that has done the trick.  Can I suggest that you change the relevant code in https://github.com/osmdroid/osmdroid/wiki/Offline-Map-Tiles to show this.

no problem. right place right time. wiki updated


[BUG] The holtWintersAberration() function fails with an IndexError exception

Describe the bug
I'm trying to use the holtWintersAberration() function to detect anomalies, but it fails with a python exception. Here is the output I get from Grafana:
Traceback (most recent call last): File "/opt/graphite/local/lib/python2.7/site-packages/django/core/handlers/base.py", line 132, in get_response response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/opt/graphite/webapp/graphite/errors.py", line 101, in new_f return f(*args, **kwargs)
File "/opt/graphite/webapp/graphite/render/views.py", line 130, in renderView data.extend(evaluateTarget(requestContext, targets))
File "/opt/graphite/webapp/graphite/render/evaluator.py", line 34, in evaluateTarget result = evaluateTokens(requestContext, target) File "/opt/graphite/webapp/graphite/render/evaluator.py", line 67, in evaluateTokens return evaluateTokens(requestContext, tokens.expression, replacements)
File "/opt/graphite/webapp/graphite/render/evaluator.py", line 102, in evaluateTokens args = [evaluateTokens(requestContext, arg, replacements) for arg in rawArgs]
File "/opt/graphite/webapp/graphite/render/evaluator.py", line 67, in evaluateTokens return evaluateTokens(requestContext, tokens.expression, replacements)
File "/opt/graphite/webapp/graphite/render/evaluator.py", line 128, in evaluateTokens return func(requestContext, *args, **kwargs) File "/opt/graphite/webapp/graphite/render/functions.py", line 4151, in holtWintersAberration elif upperBand[i] is not None and series[i] > upperBand[i]: IndexError: list index out of range

To Reproduce
This has happened for every metric I tried this with, but I'm expecting that other people would see different values.
Expected behavior
I expected the graph to be plotted.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS flavor: Ubuntu 16.04
Graphite-web version 1.2.0-dev
Django 1.8.19 / Python 2.7.12
Setup from OS packages

For some reason this works with bootstrapInterval='3d'.

Also, this doesn't happen on the holtWintersConfidenceBands() function with the exact same parameters.


if we have more than one component

az containerapp env dapr-component set
--name $CONTAINERAPPS_ENVIRONMENT --resource-group $RESOURCE_GROUP
--dapr-component-name statestore
--yaml statestore.yaml
In the dapr-component-set creation command given above, how would we define if we have more than one component, which is usually the case. One after the other with different component names and  its yaml file just below it? . Is there a way of defining them via json arrays perhaps?

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

ID: afddc810-761b-10bf-38af-ca113b9a9237
Version Independent ID: a44380a6-fc16-0f56-17cf-013775391329
Content: Tutorial: Deploy a Dapr application to Azure Container Apps using the Azure CLI
Content Source: articles/container-apps/microservices-dapr.md
Service: container-apps
GitHub Login: @asw101
Microsoft Alias: aawislan

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Hi @ffrankozz, thank you very much for your patience. I've confirmed with the Dapr team that if you need to add multiple components, run the az containerapp env dapr-component set command multiple times to add each component.
I have a PR in to update this doc accordingly. Please feel free to let us know if there's anything else we can help with!

@asw101 Thanks for the reply.  That is what I  had done   anyways. I was hoping  there was probably  a shorter way. Nevertheless, thank you.

I updated the Dapr overview doc and added a link to the document to the tutorial.
@ffrankozz while you have to run the CLI command multiple times, you can use a Bicep or ARM template to define multiple Dapr components instead.  Hope this helps.

#please-close


Ask about the use of the repository

Hello, I am a deeplabcut use of novice, for the use of the repository, there are some questions to understand, hope the author can give answers, thanks! Does the repository use deeplabcut to train and tag animals and A, B, and C in the Y Maze, and then import the tagged video into the library to analyze the detailed data required?

Hello, I am a deeplabcut use of novice, for the use of the repository, there are some questions to understand, hope the author can give answers, thanks! Does the repository use deeplabcut to train and tag animals and A, B, and C in the Y Maze, and then import the tagged video into the library to analyze the detailed data required?
Thank you for your question.
You are right, this repository is used for analyzing and plotting the coordinates of animals with in the Y-maze arena. To adapt this repository, you have to follow the DeepLabCut protocol to obtain the coordinates of animals.

Thank you for your reply. Does the repository not provide sample data?Can you provide sample data to learn about the repository? Thank you very much for your help.

Unfortunately, I could not directly provide you the data for personal use.
Please check our publish paper, and contact the corresponding author via e-mail to request the usage of the data. I will close this issue.

Hello, I have contacted the corresponding author by Email, as follows:
Dear Professor Hayashi,
We are highly interested in the behavioral data analysis section of your published article. Would it be possible for you to share the raw data used in analyzing the behavioral data through DeepLabCut as presented in your paper? This would greatly assist us in learning from the DeepLabCut scripts employed in your research.
We are eagerly awaiting your reply.
Thank you,


Add additional time to the timer while it's running

Lines 66-81 allows a the timer directive to add time to the timer while it's running. Adds
seconds, minutes, hours and days.

@danielrey28  - Can you look at the build error please? And also, some tests for this change?

Will do. I'll need some time though, this is my first time contributing.
Thanks!
On Fri, Jan 9, 2015 at 10:01 AM, Siddique Hameed<EMAIL_ADDRESS>wrote:

@danielrey28 https://github.com/danielrey28 - Can you look at the build
error please? And also, some tests for this change?
—
Reply to this email directly or view it on GitHub
https://github.com/siddii/angular-timer/pull/145#issuecomment-69344847.


Soduku (Gesture Detection)

Hey @TusharAMD pls assign me this issue for gssoc 22

@TusharAMD plss close this issue as pr is done and merged


Pattern for sending to multiple threads

Your checklist for this issue

[X] Jenkins version 2.249.2

[X] Plugin version 2.43

[X] OS Cloudbees Jenkins docker hub container

Description
The slackResult from SlackSendStep contains a .threadId , but follow-on messages to that thread only go to one channel, if the original message was sent to multiple channels.
Is there a pattern for sending to threads in multiple channels?

Only the last one is returned, if you want to publish to multiple rooms and threads then you need to do multiple slackSend calls

Thanks, Tim


YAMLS COLORS PLEASE

As I run eksdemo dry-run it prints yamls sections
Is difficult to parse, it would be ideal to have the yaml in the terminal color coded

This is a great idea and thanks for the suggestion.  Do you have any experience with Go libraries that could provide some color coding?

Found: https://github.com/alecthomas/chroma. Will see if I can get a branch for you to test @csantanapr


nxtrim:  command not found

Hello,
I am probably missing something obvious here.  My NxTrim-0.4.1 install appears to go fine, and I have re-installed it in different locations as a sanity check, but I cannot seem to run what I think is a nxtrim standard run command.  For example, if I run:
nxtrim -1  /media/rob/4TB_HD/rawTM_Pcrispum_3.5kb.ACAGTG.L005_1P.fastq.gz -2  /media/rob/4TB_HD/rawTM_Pcrispum_3.5kb.ACAGTG.L005_2P.fastq.gz -O /media/rob/4TB_HD/nxtrim --norc --separate --preservemp-mp
Then the program immediately throws an error:  nxtrim:  command not found.  I am in the NxTrim-0.4.1 directory when I run the command and I can see the nxtrim program (112.5 kB Program file) and the mergeReads program (59.5 kB Program file) in the NxTrim-0.4.1 directory too.
I also get the same error when I try something simple with the example data:
nxtrim -1 /example/MP_R1.fastq.gz -2 /example/MP_R2.fastq.gz -O /home/rob/.linuxbrew/Cellar/NxTrim-0.4.1/example
Again, the program immediately throws an error:  nxtrim:  command not found.
I have reloaded all my BOOST libraries, so do not think it is a problem with BOOST (as mentioned in a previous "issue."
Any suggestions?
Rob

Try including the path to the Nxtrim. My NxTrim was downloaded and compiled in this directory:
$pwd
/Users/odoland/software/Nxtrim/
I think yours is in /home/rob/.linuxbrew/Cellar/NxTrim-0.4.1/
Then to run it, I would try =/home/rob/.linuxbrew/Cellar/NxTrim-0.4.1/Nxtrim -1 /example/MP_R1.fastq.gz -2 /example/MP_R2.fastq.gz -O /home/rob/.linuxbrew/Cellar/NxTrim-0.4.1/example
There's a way to run it with just typing Nxtrim. I forget how, but I think it may be using an export PATH command or $cp /home/rob/.linuxbrew/Cellar/NxTrim-0.4.1/Nxtrim /usr/local/bin.

Thanks odoland.  That seemed to help move things forward, but now it throws
another error:
Problem reading
/media/rob/4TB_HD/rawTM_Pcrispum_5-7kb-GCCAAT-L005_1P.fastq.gz.
I have tried to run it with the .fastq file set as well, and it throws the
same error.
Are the dashes ("-") or the underscores ("_") in my file name the issue?
Suggestions?
Rob
On Tue, Jul 19, 2016 at 11:09 AM, odoland<EMAIL_ADDRESS>wrote:

Try including the path to the Nxtrim. My NxTrim was downloaded and
compiled in this directory:
$pwd
/Users/odoland/software/Nxtrim/
I think yours is in /home/rob/.linuxbrew/Cellar/NxTrim-0.4.1/
Then to run it, I would try
=/home/rob/.linuxbrew/Cellar/NxTrim-0.4.1/Nxtrim -1 /example/MP_R1.fastq.gz
-2 /example/MP_R2.fastq.gz -O
/home/rob/.linuxbrew/Cellar/NxTrim-0.4.1/example
There's a way to run it with just typing Nxtrim. I forget how, but I think
it may be using an export PATH command or $cp
/home/rob/.linuxbrew/Cellar/NxTrim-0.4.1/Nxtrim /usr/local/bin.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sequencing/NxTrim/issues/26#issuecomment-233683359,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJr1XbvIB5A9w6bw4XdN9UHkhpD58wNKks5qXPbBgaJpZM4JP4E3
.

I don't think the "-" or "_" are the issues. Have you tried NxTrim on the example data set?
I made two bogus files: blahblah.R1.fastq.gz and blahblah.R2.fastq.gz
The contents of each file are just:
"Hello World!"
and then Nxtrim -1 blahblah.R1.fastq.gz -2 blahblah.R2.fastq.gz  was still able to work (although nothing was trimmed).
So I think it could be a permission error with your file? Maybe not allowed access?
I tried the command:  (which changes permissions)
$chmod 000 on blahblah.R1.fastq.gz
and I was able to replicate your problem.
"Problem reading: blahblah.R1.fastq.gz"
So maybe try to change permissions:
https://en.wikipedia.org/wiki/Chmod
Try this command: The 777 allows read,write and execute
$chmod 777 /media/rob/4TB_HD/rawTM_Pcrispum_5-7kb-GCCAAT-L005_1P.fastq.gz.
Hope that works!


Update beancount_query_language.md

Fix beanquery source location

Hello! Please edit the original document at Google Docs.


Refactor Appkit to use sigma sdk

In this PR:

[x] update sigma dependency to use v5.x-scala-js branch
[x] update code and make sure the tests pass
[ ] move ReducedTransaction implementation to Sigma
[ ] TBD

@aslesarenko visit the official ERGO page to start a chat with an official agent on a live chat support to report this for review and for further assistance via LIVE CHAT

subsumed in #225


Sign messages

Zap should provide a UI for users to sign messages. This can be used to authenticate their node's public key when users use their node as an online identity

Closed via #2696


Migrate Zygote to ChainRulesCore

Codecov Report
Base: 52.84% // Head: 53.19% // Increases project coverage by +0.35% :tada:

Coverage data is based on head (739ed55) compared to base (ad13aac).
Patch coverage: 56.25% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
+ Coverage   52.84%   53.19%   +0.35%
==========================================
  Files           4        5       +1
  Lines         176      188      +12
==========================================
+ Hits           93      100       +7
- Misses         83       88       +5

Impacted Files
Coverage Δ

src/chainrules.jl
46.15% <46.15%> (ø)

src/derivative.jl
100.00% <100.00%> (ø)

src/scalar.jl
34.14% <100.00%> (-1.41%)
:arrow_down:

src/codegen.jl
42.85% <0.00%> (+15.58%)
:arrow_up:

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

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


Charttool issues if the directory already exists

Version:
λ tk --version
tk version v0.13.0

Repro:

Init charts:

tk tool charts init

Try adding a chart from a repo that's already added to helm:

λ tk tool charts add<EMAIL_ADDRESS>                                                                                Adding 1 Charts ...
 OK<EMAIL_ADDRESS>Added 1 Charts to helmfile.yaml. Vendoring ...
Syncing Repositories ...
Pulling Charts ...
reading chart manifest: open .../k8s/charts/longhorn/Chart.yaml: no such file or directory

Try vendoring separately:

λ tk tool charts vendor
Syncing Repositories ...
Pulling Charts ...
reading chart manifest: open .../k8s/charts/longhorn/Chart.yaml: no such file or directory

Realise I need to add longhorn repo to chartfile.yaml:

directory: charts
repositories:
- name: stable
  url: https://charts.helm.sh/stable
- name: longhorn
  url: https://charts.longhorn.io
requires:
- chart: longhorn/longhorn
  version: 1.1.0
version: 1

Try vendoring again:

EXPECTED RESULT:

Chart is downloaded into currently empty charts/longhorn directory.

ACTUAL RESULT:
λ tk tool charts vendor
Syncing Repositories ...
Pulling Charts ...
reading chart manifest: open .../k8s/charts/longhorn/Chart.yaml: no such file or directory

WORKAROUND:

Delete the .../k8s/charts/longhorn directory and invoke tk tool charts vendor again.

Might be related to #420 🤔

Happy holidays and thanks for working on this lovely tool ❤

Hey, thanks for filing this!
I'm having troubles reproducing this however. It seems to me like steps above are incomplete, given there are no tk tool charts add commands.
Could you post a series of commands to reproduce this in a fresh directory, just after running tk init?

Dang, I'll be in my shamecube. Corrected.
Though now I cannot reproduce it it seems, it just gives me this:
 λ tk tool charts add<EMAIL_ADDRESS>Adding 1 Charts ...
 OK<EMAIL_ADDRESS>Added 1 Charts to helmfile.yaml. Vendoring ...
Syncing Repositories ...
Pulling Charts ...
Error: repo longhorn not found
exit status 1

, and doesn't create the directory.
Though if I create an empty directory in the charts folder, I can repro:
λ tk tool charts vendor
Syncing Repositories ...
Pulling Charts ...
reading chart manifest: open .../charts/longhorn/Chart.yaml: no such file or directory

Is there a workflow that could've led to the directory being created?

@Temikus I'm not aware of any workflow that could possibly lead to your scenario. Let's close this as we can't really do anything at the moment. Feel free to re-open if this occurs again :)


TGP-2821: Add correlation ID to response when a request for data is made

Checklist for developers before reviewing

[x] I have merged in main
[x] I have assigned the PR to me
[x] I have run precheck.sh to run all tests, format and check the code coverage
[x] I have written unit test and integration tests to cover any code that I wrote, where necessary
[x] I have linked tickets to any TODOs I have written
[x] I have run the microservices with the latest SM2 config
[x] I have checked all the Acceptance Criteria pass
[x] I have run the local trader-goods-profile-api-tests
[x] I have spoken to the QAs about any changes that may need to be made to the api tests OR no changes to the api tests are necessary
[x] I have moved the ticket to Review in Jira
[x] I have left a message in tgp-prs on slack to alert other Devs that this ticket is ready for review
[x] I have checked my PR is not in draft mode

Checklist for reviewer 1

[ ] I have added an emoji to the relevant message in tgp-prs to show I am reviewing
[ ] I have checked out the branch
[ ] I have run the microservices with the latest SM2 config
[ ] I have checked all the Acceptance Criteria pass
[ ] I reviewed the code and left comments where necessary
[ ] I have left a message on the relevant tgp-prs message to show that I have finished reviewing

Checklist for reviewer 2

[ ] I have added an emoji to the relevant message in tgp-prs to show I am reviewing
[ ] I have checked out the branch
[ ] I have run the microservices with the latest SM2 config
[ ] I have checked all the Acceptance Criteria pass
[ ] I reviewed the code and left comments where necessary
[ ] I have left a message on the relevant tgp-prs message to show that I have finished reviewing

Checklist for developers before merging

[ ] The PR has been approved by 2 other developers
[ ] I have merged in main
[ ] I have run precheck.sh to run all tests, check formatting and code coverage
[ ] I have run the microservices with the latest SM2 config
[ ] I have checked all the Acceptance Criteria pass
[ ] I have checked that the QAs have merged in any changes to the api tests
[ ] I have run the local trader-goods-profile-api-tests

Reminders for developers after merging

Wait for pipeline to pass
Move ticket to QA column in Jira
Leave message in tgp-qa on slack to alert QAs that this ticket is ready for testing


Mismatch in number of sequences

Hi,
The page for the sequences of family RF00014 says there are 40 sequences for that family. However, when downloading the file with the aligned sequences (FASTA), I can only count 38 distinct sequences. Am I missing something?
Best,
Emanuele

Hi Emanuele,
Thank you for reporting this.  The number of significant sequences is 40, but there are only 39 sequences in the FASTA file. I will look into it and try to get it fixed the soonest possible.
Kind regards,
Ioanna

Another quick question. I noticed that the RNA sequences in the database contain the 'T' base as opposed to 'U'. Is this just for simplicity or is there some reason?
Thanks,
Emanuele

Dear Emanuele,
Our sequence database is composed of a collection of non-redundant complete genomes that we annotate with non-coding RNAs using our covariance models. We extract the sequences in the fasta files directly from those genome assemblies, hence the Ts. You can get the exact ncRNA sequence by replacing the Ts with Us. To cross check this you can use RNAcentral sequence search and search for one of the sequences in RF00014.
I would also like to inform you that the fasta files have now been fixed and are available on the FTP
Please feel free to let me know if you have any other questions.
Kind regards,
Ioanna

Closing this issue - please feel free to reopen if the problem persists.


Add/verify support for saving/loading hybrid pytorch models

What is the expected enhancement?
Verify (or add) an ability of saving/loading hybrid PyTorch models with quantum layers to a file for further re-use and inference.

Closed in #377


Format for galaxy tool metadata extractor imports

The data currently imported from the Galaxy tool metadata extractor come as a json file where many of the keys include whitespaces. Can we revise this to have no whitespaces in the keys? This is a very minor point, but sometimes whitespaces in JSON keys can cause problems (although there usually is a workaround).
Would it be possible to change these keys, e.g., Maybe replace whitespaces with underscores?
Another point is whether there is somewhere a documentation that describes the format of the CSV source file, detailing the meaning and source of each field?
Thoughts @arash77 @bebatut @paulzierep ?

Fixed in #14


print warning if actual triton cache dir is on NFS, not just for default

move the logic that prints a warning when triton cache dir is on NFS to act on the actual calculated cache_dir rather than on the default.
this means that:

when the default directory (in the user's home directory) is on NFS but TRITON_CACHE_DIR is set to a non-NFS directory, no warning will be printed whereas prior to this change a spurious and confusing warning was printed
when the user's home directory is not on NFS but TRITON_CACHE_DIR is set to an NFS directory, a warning will be printed whereas prior to this change no warning would be printed

fixes #6486

@jrandall Thank you for the contribution!


Unit testing

Extremely ugly and non-efficient code, but looks like it works.

Uh oh, something went wrong with the build. Need to check on that

Hooray Jenkins reported success with all tests good!

Thanks for the updates :-) Checkstyle says all clear!


aria-label & aria-expanded added to hamburger

First thing, PLEASE READ THIS: ReactPlay Code Review Checklist

Description
aria-label and aria-expanded added to hamburger menu.
Fixes # (issue)
#1041
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Screenshots or example output

@atapas I didn't see that, now it is fixed. You can check now.

Hey @koustov Please review it.

Thanks @atapas for reviewing and merging it.


Licensed under MIT license?

Hi!

I was wondering if this was licensed under MIT license to match Godot Engine's license https://github.com/godotengine/godot/blob/master/LICENSE.txt
Thank you for releasing this amazing work.

Yes; I just added MIT license to the repo.


Create a test suite

Main thing for a test suite is to test various input patterns against the regex to ensure email addresses are correctly extracted. Per the readme, the regex will never be perfect but the test suite should at least provide working examples of data that meets the expected pattern and equally, is excluded from the expected pattern.

This is already addressed in PR #3


add mock data script

(re)Added a script to load mock data into the DB.

Can you please delete the deploy\db directory which contains the records.js file.  Once I see this, I can go ahead and merge it and we can do another PR to tweak the data if needed/desired.


[labs/context] Context decorators error on private or protected fields

Discussed in https://github.com/lit/lit/discussions/3875

Originally posted by NealRame May  4, 2023
In my component I want to add a context properties which is also private
@provide({ context: someContext })
@state()
private _someData = 0

I read in the documentation that it is possible however when doing so, I get that typescript compiler error :
error TS2345: Argument of type 'MyComponent' is not assignable to parameter of type 'ReactiveElement & Record<"_someData", number>'.
  Type 'MyComponent' is not assignable to type 'Record<"_someData", number>'.
    Property '_someData' is private in type 'MyComponent' but not in type 'Record<"_someData", number>'.

89     @provide({ context: someContext })

Could someone explain me if I am missing something please?
I am using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

Well, this is difficult to address. It seems that either we have type-checking on public fields to ensure the context type and field type are compatible, or we allow decorating private fields, but not both.
The constraint comes from the fact that a property decorator only receives arguments and type parameters for the public class interface and the property name, and unfortunately in the case of private and protected fields we get the name but that field isn't present in the public interface.
The new standard decorators fix this because the decorators get the value type as a type parameter. This could theoretically be added to old decorators, but the TypeScript team has said they're not going to invest in the old decorators anymore.
I have figured out a way to let you override the types that the decorator infers so that you can prevent a decorator from type-checking on a private field. It's not pretty, but it's a workaround:
  @provide<string, any>({context: stringContext})
  @state()
  private _aString = 'a';

The first parameter must match the context type, and the second parameter must be any to override the class's type.
So... we're faced with a decision here between:

field type-checking, but no decorating private fields
decorating private fields with no DX hit, but no field type-checking
field type-checking and decorating private fields with some boilerplate

It seems like option 3 is good, but an argument for dropping field/context type-checking is that many, if not most, context fields should be private or protected so we shouldn't negatively impact private fields.
cc @benjamind

I feel like option 2 is better. Upgrading to standard decorators that wouldn't require this workaround would be easier with that too.

Fixed by #3733


Add portable .zip to torrent

Summary
I respect bandwidth costs and like using P2P whenever possible. Can you please add the portable .zip to the torrent or make a separate torrent for it?
Thanks!

I'm not sure I follow. To which torrent? The portable zip is added to each release both here on GitHub and on FOSSHub

:facepalm: This issue was meant for OBS. Sorry, @vktr. Great job with picotorrent.


ReferenceError: Cannot access 'isServer' before initialization

Describe the bug
Hi kyd1:
swc is a very awesome tool. Last year I forked Next.js v11 to modify it. used swc version is 1.2.205, I add some  custom code like below in some file
export const isServer = typeof window === 'undefined'
export const isClient = typeof window !== 'undefined'

Recently when I work in my monorepo project which include the forked next.js v11,  packages which compiled with swc
had a symlink of dependency problem, so find the CHANGELOG from swc github, upgrade swc to 1.3.11, the symlink problem resolved. My another project use modify next.js as meta framework, the project run next dev works well, finally
crash at next build, error looks like:
ReferenceError: Cannot access 'isServer' before initialization

full example code at https://github.com/zackshen/swc-build-error-repro
Input code
src/index.js
export * from "./error";
export * from './mod_a';
export const isServer = typeof window === 'undefined'
export const isClient = typeof window !== 'undefined'

src/mod_a.js
import { mod_b } from "./mod_b";

export function mod_a() {
    mod_b()
}

src/mod_b.js
import { isServer } from "./index";

export function mod_b() {
    if (isServer) {
        console.log('blablabla')
    }
}

function new_instance() {
    if (isServer) {
        return { server: true }
    }
    return { server: false }
}

export const instance = new_instance()

src/error.js
export const Error = () => {}

Config
{
  "jsc": {
    "target": "es2016",
     "parser": {
       "syntax": "typescript",
       "dynamicImport": true
    }
  },
  "module": {
    "type": "commonjs"
  }
}

final compiled file dist/index.js
if _export statement moved to the file end , yarn test will pass.

"use strict";
Object.defineProperty(exports, "__esModule", {
    value: true
});
function _export(target, all) {
    for (var name in all) Object.defineProperty(target, name, {
        enumerable: true,
        get: all[name]
    });
}

_export(exports, {
    isServer: () => isServer,
    isClient: () => isClient
});
_exportStar(require("./error"), exports);
// ======= Error ============
// ** Call chain ---> mod_a ---> mod_b ---> index
// ======= Error ============
_exportStar(require("./mod_a"), exports);
function _exportStar(from, to) {
    Object.keys(from).forEach(function (k) {
        if (k !== "default" && !Object.prototype.hasOwnProperty.call(to, k)) Object.defineProperty(to, k, {
            enumerable: true,
            get: function () {
                return from[k];
            }
        });
    });
    return from;
}

const isServer = typeof window === 'undefined';
const isClient = typeof window !== 'undefined';

Playground link
https://github.com/zackshen/swc-build-error-repro
Expected behavior
yarn test pass successfully.
Actual behavior
Run yarn test cause error:
ReferenceError: Cannot access 'isServer' before initialization

Version
1.3.11
Additional context
No response

Your code is wrong

I did "type": "module" and patched imports/exports and did node src/index.js

@kdy1 thanks，It's wrong of my code.


 Will update, rebase, and remake pull request.

Hi @adamyala, just following up on this. "429 Too Many Requests" has been implemented by another contributor, but we are still missing the others. Thanks!

P.S. - You may wish to open a new PR to score some some https://hacktoberfest.digitalocean.com points. 😄

@adamyala Are you going to continue working on this PR? I had a similar idea as well - making sure all errors have their classes and tests.

@kgriffs I could take over this PR or open a new one that does the same thing, as a follow up to #834

@wssbck all yours. I unfortunately having had the time to revisit this one.

Hi @wssbck, did you still want to take this and get it over the finish line? It would be good to get it done in the next couple of weeks if you have the bandwidth. Just let me know. Thanks!

@kgriffs I should be able to finish it over the next two weeks.

@kgriffs @adamyala as #973 is merged, this PR can be closed I think.


Provide a way to manage the vert.x stack

Provide a way to add and remove artifact into the vert.x stack installed locally (or even create a new one).
A list of available artifacts should be provided as a reference (like a catalog), but it should be possible to add custom artifacts.
Removing artifact need to remove not-used dependencies.
It is important that version conflicts are managed.

The stack manager has been integrated.
NPM and Docker build updated.


Populate req.locale with requested locale

Would be great if you could take a peek at req and see which locale i18next had assigned.

you could access req.locale for currentLng; and req.i18n for full object -> use req.i18n.t([key]) for translation

alright...just let me know if you need more stuff.


intro json patch library

proof pr at https://github.com/openshift/origin/pull/29163

/assign @deads2k @bertinatto

/lgtm
/approve


Android: isInstalled fails for package entries with double quotes

The .indexOf pattern was wrong: the colon was outside the quotes

Hey!
What about the second check that does have quotes?
 ~content.indexOf(':${name}'`

Before:
(content) => ~content.indexOf(`:"${name}"`) || ~content.indexOf(`':${name}'`),
                               ^  outside quotes

After:
(content) => ~content.indexOf(`":${name}"`) || ~content.indexOf(`':${name}'`),
                                ^ inside quotes

The second check is fine because the colon is inside the (single) quotes. The first check is looking for double quotes.

Ah, true, entirely missed that!
@Kureev do we need quotes in the second check? Can't it simply be :${name} ?

Actually, we always need to check for this pattern:
https://github.com/rnpm/rnpm-plugin-link/blob/master/src/android/patches/makeBuildPatch.js#L9

Simply put, we need to check for :${name}

Anyone wants to make a PR?

I'm planning to round off my side activities before the weekend and resume working on rnpm.
That would be a first thingy I'll tackle if nobody submit a PR before.

Closing in favour of #49 that slightly unifies these checks since " and ' are irrelevant! Thanks for bringing this up to the table - it's been a valuable hint! I hope the fact we are merging another PR does not discourage you from further contributing to this project! The more Android heros here the better for all of us!


Security claims on BT Mainline DHT

(serious, please correct me if there is something I overlooked)

BitTorrent's DHT is probably one of the most resilient and censorship-resistant networks on the internet.

What do you base that claim on? As far as I know, there is an attack vector on the Mainline DHT that has been published for years[0] and that is being exploited in the wild[1]. As far as I can tell, the only reason the Mainline DHT still works is that nobody wanted that badly to damage it.
[0]  Wang, Kangasharju: Real-world sybil attacks in BitTorrent mainline DHT; DOI:10.1109/GLOCOM.2012.6503215
[1] Wang, Kangasharju: Measuring Large-Scale Distributed Systems: Case of BitTorrent Mainline DHT; DOI:10.1109/P2P.2013.6688697

My claim was not that it doesn't have any vulnerabilities, but that in spite of the vulnerabilities it is still one of the largest distributed networks on the internet.
Personally I think that nobody is damaging it because (i) it would still require some economical effort to accomplish and (ii) there wouldn't be much incentive in actually carrying out such attacks. (it's not like Bitcoin where there's money on the line).

I guess it's fine if you think these attacks don't hurt this system. I even think this system is a really interesting case study despite any attacks on the underlying DHT. However I'd still suggest you update the first line of that README, as this line is, strictly speaking, false and might give someone without a background in distributed systems a false impression of the kind of guarantees any such system can provide given the current state of technology.
If you want I can send you a PR on that line, but I don't know whether I'd manage to make it have the right ring to it. Maybe something like "most widely used distributed system".

@jaseg it says "probably" and also the BT mainline DHT is the largest DHT in existence - and the larger such network is the more resilient and censorship-protected it is basically.
I do agree that the message can be re-worded, but it's not completely wrong

I agree that there is an issue in presenting it this way.
This issue can be mitigated in various ways:

Reword the intro text to make it less objectionable
Keep the wording the same but add a but of a caveat and a link to a section that talks about DoS (censorship) and privacy related attacks.

Regardless, you should ceate a new research section to explore some of the Sybil-resistant DHTs out there. Don't marry yourself to one. Use IPFS, as I believe I has support for arbitrary DHTs.


CompositeController goes into hot loop

Hi,
I am new to metacontroller and github in general, so please let me know if I forgot something. Thanks for writing/contributing to metacontroller, it appears to be the ideal solution for my first operator.
I am running the HelloWord example (https://metacontroller.app/guide/create/) on a microk8s cluster. When I create a HelloWorld resource the controller picks it up, the child pod is created, but it appears that metacontroller continually calls the sync webhook, even nothing has changed, quickly generating many generations:
metacontroller.log
I0527 14:01:39.489674       1 controller.go:406] sync HelloWorld hello/your-name
I0527 14:01:39.506012       1 controller.go:406] sync HelloWorld hello/your-name
I0527 14:01:39.513006       1 controller.go:406] sync HelloWorld hello/your-name
I0527 14:01:39.623230       1 request.go:485] Throttling request took 106.161551ms, request: PUT:https://<IP_ADDRESS>:443/apis/example.com/v1/namespaces/hello/helloworlds/your-name
I0527 14:01:39.630174       1 controller.go:406] sync HelloWorld hello/your-name
I0527 14:01:39.823080       1 request.go:485] Throttling request took 183.409633ms, request: GET:https://<IP_ADDRESS>:443/apis/example.com/v1/namespaces/hello/helloworlds/your-name
I0527 14:01:40.023058       1 request.go:485] Throttling request took 196.123082ms, request: PUT:https://<IP_ADDRESS>:443/apis/example.com/v1/namespaces/hello/helloworlds/your-name
I0527 14:01:40.026549       1 controller.go:406] sync HelloWorld hello/your-name
I0527 14:01:40.223061       1 request.go:485] Throttling request took 192.155825ms, request: GET:https://<IP_ADDRESS>:443/apis/example.com/v1/namespaces/hello/helloworlds/your-name
I0527 14:01:40.423835       1 request.go:485] Throttling request took 198.906384ms, request: PUT:https://<IP_ADDRESS>:443/apis/example.com/v1/namespaces/hello/helloworlds/your-name

I've enabled auditing with
apiVersion: audit.k8s.io/v1
kind: Policy
rules:
- level: RequestResponse
  resources:
    - group: example.com
      resources: ["helloworlds"]

which produces
audit.log

I notice that metacontroller sets status.observedGeneration in addition to what the webhook returns. Naively speaking it looks like an update triggers a sync, which updates status.observedGeneration, which triggers a sync, etc. ?

I am having the same issue. It seems all my children are ok and don't regenerate, but the parent runs on an infinite loop, continuously going up forever. Your thoughts are mine as well, there is a feedback loop with some field that is triggering a gen but shouldn't be, like observedGeneration. Still poking around.

5,9c5
<     "annotations": {
<       "kubectl.kubernetes.io/last-applied-configuration": "{\"apiVersion\":\"foo.com/v1\",\"kind\":\"MyCrd\",\"metadata\":{\"annotations\":{},\"name\":\"thing\",\"namespace\":\"project-bar\"},\"spec\":{\"param\":\"stuff\"}}\n"
<     },
<     "creationTimestamp": "2019-06-11T13:18:04Z",
<     "generation": 2617,
---
>     "annotations": {},
11,14c7
<     "namespace": "project-bar",
<     "resourceVersion": "3084449",
<     "selfLink": "/apis/foo.com/v1/namespaces/project-bar/mycrd/thing",
<     "uid": "5835c903-8c4b-11e9-ac2e-42010a8e00f7"
---
>     "namespace": "project-bar"
18,21d10
<   },
<   "status": {
<     "foo": "bar",
<     "observedGeneration": 2616

Running a sorted json diff on last-applied-configuration and current config yeilds nothing that should be running up the gens

seems like a regression, downgrading to metacontroller/metacontroller:v0.3.1 from metacontroller/metacontroller:v0.4.0 seems to make it stop

The last time this came up, I think the problem was that newer versions of Kubernetes started setting metadata.generation on CRD objects, even if the CRD's status subresource is disabled. That broke a brittle assumption in Metacontroller that metadata.generation being set meant that it ought to pass that through to status.observedGeneration.
The workaround we found is to make sure the status subresource is enabled for your CRD. This is also recommended in general for any CRD that you use with Metacontroller.

just to confirm, adding
  subresources:
    status: {}

to my CRD fixed the issue
more information here:
https://kubernetes.io/docs/tasks/access-kubernetes-api/custom-resources/custom-resource-definitions/#status-subresource

Adding the below has the unfortunate side effect that the status returned from the webhook is no longer being stored with the instance of the CRD. Are there any alternative solutions?
  subresources:
    status: {}

Adding the above from @jordan-da fixed this for me too.. I was very confused by the hot-loop using the Go example.


Add API method addPrompts to Config SDK

Comment from https://github.com/lastmile-ai/aiconfig/pull/658#discussion_r1437910213 (also check out the summary for how to test that well in typescript)
Would be good to have a method called add_prompts(prompts: List[Prompt]) (python) or addPrompts(prompts: Prompt[]) (typescript) that adds a list of prompts to an aiconfig. We currently have add_prompt(prompt: Prompt) (python) addPrompt(prompt: Prompt) (typescript)
We should have 2 separate push requests for this:

Python --> You can also see how addOutputs was added to Python and automated testing was added in https://github.com/lastmile-ai/aiconfig/pull/689 (to run the automated tests, go to aiconfig/python dir and run pytest
Typescript --> It should be similar to what you did for Python except you now add the method to the Config.ts file and add automated testing in the testProgramaticallyCreateConfig.ts file. You can test by going to the aiconfig/typescript dir and running yarn test

You can also check out that pull request here: https://github.com/lastmile-ai/aiconfig/pull/632. Also see for where addPrompt (typescript) and add_prompt (Python) are defined)

Going to assign this to Sandra


WIP hungarian

Sorry for the big diff, poedit seems to have reformatted lines.

I'm travelling today, so i might be able to do a bit more, but if you don't hear by midafternoon my time, just merge.
On November 18, 2018 4:10:30 AM EST, Carlos Cordoba<EMAIL_ADDRESS>wrote:

@martindurant, could you finish this one by tomorrow? Or should I merge
it as it is?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/spyder-ide/spyder/pull/8250#issuecomment-439678138

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Thanks Martin!


fix minor build issues

GCC 10.0.1 barked about sqlite3.c
'make swish' continued despite ^C:

...
compiling swish/http.ss
^Cbreak>
compiling swish/websocket.ss
compiling swish.library
^Cbreak>
^Cbreak>
...

Thanks for the quick review, Bob!


[improve][test] Add integration tests for TLS

Signed-off-by: Zixuan Liu<EMAIL_ADDRESS>Motivation
Add integration tests for TLS.
Modifications

Add integration tests for TLS
Add TLS config for broker and proxy
Refactor how to get broker URL and web URL
Fix tls conf of pulsar compact-topic command

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

/pulsarbot rerun-failure-checks

@nodece I think you forget to add the test to CI.

Any update on this PR?

@lin-zhao This PR blocked by https://github.com/apache/pulsar/pull/17426.

I just reviewed and merged #17426.

Codecov Report

Merging #17141 (246b085) into master (6c65ca0) will increase coverage by 35.83%.
The diff coverage is 50.00%.

@@              Coverage Diff              @@
##             master   #17141       +/-   ##
=============================================
+ Coverage     34.91%   70.74%   +35.83%
+ Complexity     5707      437     -5270
=============================================
  Files           607       26      -581
  Lines         53396     2246    -51150
  Branches       5712      245     -5467
=============================================
- Hits          18644     1589    -17055
+ Misses        32119      484    -31635
+ Partials       2633      173     -2460

Flag
Coverage Δ

unittests
70.74% <50.00%> (+35.83%)
:arrow_up:

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

Impacted Files
Coverage Δ

...apache/pulsar/proxy/server/DirectProxyHandler.java
63.63% <50.00%> (ø)

...n/java/org/apache/pulsar/broker/PulsarService.java

...pulsar/broker/admin/impl/PersistentTopicsBase.java

.../org/apache/pulsar/broker/admin/v2/Namespaces.java

...broker/delayed/InMemoryDelayedDeliveryTracker.java

...rg/apache/pulsar/broker/service/BrokerService.java

.../service/SystemTopicBasedTopicPoliciesService.java

.../pulsar/broker/stats/BrokerOperabilityMetrics.java

...ion/pendingack/impl/MLPendingAckStoreProvider.java

...g/apache/pulsar/compaction/CompactedTopicImpl.java

... and 572 more

/pulsarbot rerun-failure-checks

This PR is blocked by Pulsar CI / CI - System - Sql (pull_request) .


Tag closing issue on apexcharts parent component

Hi there,
I encountered a strange problem. When I put props in the ACharts component which is parent of apexchart, the first tag of the AChart component encountered a problem. I had to declare constants and put the props data in them to avoid the problem. I do not know why. someone had the same problem ?
At the bottom right of the component side you see in the comments the way in which I subsequently declared constants to put the data there and that solved the problem

yes, its about the clean code.
and i agree the second picture of your code. is the way to place the data and bind into your Achart
rather than hardcode into your html.
i though it can be closed issue


Support For ZDMA?

Hi,
I currently have a ZDMA card. It differs from other normal DMA cards since it utilises a Thunderbolt interface and 2x100T chips:
https://github.com/ufrisk/pcileech-fpga/tree/master/ZDMA
I was following the below tutorial to get things working:
https://www.youtube.com/watch?v=0JwMng7khxU
It appears that ZDMA is not support since I get DMA initialisation failed. I think the root cause is that the Windows Driver for ZDMA requires leechcore_driver.dll:

ZDMA requires a signed Windows kernel driver. Download it here. Right click on the .inf file and select "install" in the popup menu that appears to install the driver. Additionally ZDMA requires the file leechcore_driver.dll. This file should be included in the latest downloads of PCILeech and MemProcFS.

Let me know if I'm doing anything wrong or if this is due to the project not supporting ZDMA. Thanks!

Also I have run DMA tests to confirm my ZDMA card is working correctly when I was trying to get this all working.

If your card is supported by MemProcFS& pcileech, the plugin works. Plugin just a bridge between memprocfs and cheatengine.

If you're using ZDMA make sure you actually have leechcore_driver.dll inside the same folder as plugin.dll aswell.


Command copy is not friendly

On page https://kubevela.io/docs/quick-start#step-2-deploy-your-first-application, when copying the command and paste it in cli, it will show as below.
$ kubectl apply -f https://raw.githubusercontent.com/oam-dev/kubevela/master/docs/examples/vela-app.yaml
application.core.oam.dev/first-vela-app created

This is not as expected. Only the command should be copied.

这个需要在 Markdown 文件去掉 $ 符号，copy 这个按钮只会复制   内的内容

https://github.com/oam-dev/kubevela/pull/1661 will fix it

oam-dev/kubevela#1661 will fix it

Great.


Plan out auton path - 2 Hatch Rocket

Create a path that will score 2 disks high,
incorporate splines, intake/extake, arm movement, and limelight

switching focus on autons


story is not a function [Vue]

Issue:
Which version are you using?
0.3.0
Are you using storybook-router with a react based project or a vue one?
Vue
Please describe the problem:
Imposible to invoke StoryRouter as on README
Please explain how to reproduce the issue or (better) provide an example to do it.
Simple do.
storiesOf('NavBar', module)
  .addDecorator(StoryRouter())
  .add('default', () => NavBar);

will return into a story is not a function, doing .addDecorator(StoryRouter) fix the issue.
i can work on a PR for this, i just want to be sure i'm not crazy =p

NVM, this was because i was importing StoryRouter as default