_viewts and _idts timestamp transmission

Hi,
refering to docs Matomo expects as data for the parameters _viewts and _idts  an Unix timestamp.  Matomo forever transfers the timestamp in milliseconds, but if im right Unix timestamps are in seconds. Can you please check this?
Best regards

Hello @benedict-formigas , thanks a lot for reporting this! Indeed, Matomo expects Unix timestamps in seconds. I will correct this as soon as possible.

Version 1.0.2+1 corrects this issue for the following fields:
_viewts
_idts
_ects
cdt

Thanks again @benedict-formigas for reporting this issue!
Can you please close this issue if you confirm this is fixed? Otherwise, I will probably close it myself in a few weeks.
Cheers!


error in generating protein sequence

dear @HeshamElAbd,
thank you very much for developing this very promising tool.
I successfully ran it on my infrastructure, but faced some problem for one case. It seems to complain about difficult cases with small indels. Please find attached the files. Would you be able to run it and check whether there is room for improvement regarding such case?
The program returns me this error Critical error was encountered: for transcript: Solyc02g062560 the following mutations were observed with a list of positions and instructions.
Thanks a lot!
Best regards,
Thomas
test.zip

Hi Thomas,
Thank you very much for your kind words and I am really happy that you found the tool useful.
Could you please tell me which version you are using? I tried to replicate the problem with the latest version:  '0.1.4' and it worked out smoothly.
This version can be downloaded from the bins directory on the cpu_only branch. I would also add it the bins directory on the main branch and update the release of the project.
Thanks a lot for your time and I am truly looking forward to your feedback,
Best Regards,
Hesham

Thank you very much for your answer.
I was indeed using version 0.1.2.
But after re-cloning from the cpu_only branch or even by downloading the bins directly, I was unable to get version 0.1.4. I am probably doing something wrong! I will try to dive deeper into the problem.

Hi Hesham,
I was finally able to work with my example after using the Linux binaries available in the current release.
(It seems however that typing vcf2prot_linux --help still returns vcf2prot 0.1.2.)
Thank you very much for the help!
Thomas

Hi Tomas (@TDDB-limagrain )
Please, it is really my pleasure!!! and I am really glad that everything went smooth. I am really sorry for any inconvenience caused, I will dig deeper into this and will harmonise and tide up the versioning of the tool.
Thanks a lot for using the tool and for your feedback to make it better, please let me know if I there is anything else that needs to be addressed or if there is any feature you would like to have.  Also, if the issue has been solved, please feel free to close the issue.
All the best,
Hesham

Thanks Hesham,
I am closing this issue and re-opening a new one for little problem related to protein identifier format!
Best regards,
Thomas


added gitnux.com

Added a new GitHub copycat gitnux.com which is frequently showing up as the first result in Google searches.

Hi thanks a lot for your contribution!
Are you sure this domain mirrors Github? I can't find any repo/issue/content from Github on Gitnux.

Sorry! I should've inspected more; I formed a habit of distrusting most domains that start with git.


Update kubectl_completion.md

Quote shell commands to improve format

This change is

I signed it!
On Mon, Feb 13, 2017 at 11:26 AM, k8s-ci-robot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. Before we can look at your pull request,
you'll need to sign a Contributor License Agreement (CLA).
📝 Please follow instructions at
https://github.com/kubernetes/kubernetes/wiki/CLA-FAQ
https://github.com/kubernetes/kubernetes/wiki/CLA-FAQ to sign the CLA.
Once you've signed, please reply here (e.g. "I signed it!") and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data and verify that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please sign in with your
organization's credentials at https://identity.
linuxfoundation.org/projects/cncf
https://identity.linuxfoundation.org/projects/cncf to be authorized.

Instructions for interacting with me using PR comments are available here
https://github.com/kubernetes/community/blob/master/contributors/devel/pull-request-commands.md.
If you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://github.com/kubernetes/test-infra/blob/master/prow/commands.md.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/kubernetes.github.io/pull/2508#issuecomment-279442683,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGe7ACRBZvDdFKuQ44MN5tJ8Cufk30Lks5rcIREgaJpZM4L_aet
.


As a ramen developer/tester, I can issue an end-to-end test that builds and deploys ramen from the current local branch, so it can be tested

Currently, ocm-minikube-ramen.sh checks out ramen and ocm-ramen-samples branches containing changes not yet merged into their main branches so that it can complete successfully.  The goal is to remove those undesirable checkouts and instead use the local ramen branch.

I intend to complete this in sprint 2 ending 2021-July-2.

Issue addressed in pull request https://github.com/RamenDR/ramen/pull/82


Flaky MutationEventExtensionsTest test

Will investigate the flaky test https://app.circleci.com/pipelines/github/aws-amplify/amplify-ios/4953/workflows/23e03ca6-e27c-4a90-9f43-7b5aa9c13d93/jobs/42782 testSentModelWithNilVersion_Reconciled

Summarizing some findings on this, so MutationEventExtensionsTest test code tries to set up the state of test using setUpPendingMutationQueue by using a for-loop and saving to storage adapter. A for-loop without waiting for each save result is used to mimic "consecutive saves" scenario. The ordering of saves are performed in order of Save 1, 2, and 3, without waiting for the result of each save. The implementation with the cached formatters could cause the lock to be acquired out of order. ie. Save 1, then 3, then 2. since 2 and 3 are both waiting for the lock while 1 has acquired it. I think this scenario is flaky since acquiring the lock twice causes the out of order scenario to appear more often. By removing the need to acquire the lock twice, the test set-up seems to be consistently successful now- which means the time it takes it iterate over the for loop introduces a delay between saves calls that is greater than the time it takes for calls 2 and 3 to be contending for the lock
Originally posted by @lawmicha in https://github.com/aws-amplify/amplify-ios/issues/1760#issuecomment-1137542608

so odd, the data retrieve during set up has inProcess: false but nothing is modifying that flag (created with inProcess: true)
"MutationEvent(id: "A38979B5-16E6-46F1-9051-679D499ACDDD", modelId: "DF0AB521-019D-4E72-B31F-74DD7D25F88B", modelName: "Post", json: "{\"id\":\"DF0AB521-019D-4E72-B31F-74DD7D25F88B\",\"title\":\"title1\",\"content\":\"content1\",\"comments\":[],\"createdAt\":\"2022-05-26T15:54:44.125Z\"}", mutationType: "update", createdAt: Amplify.Temporal.DateTime(foundationDate: 2022-05-26 15:54:44 +0000), version: Optional(1), inProcess: false, graphQLFilterJSON: nil)"

unless there's another process that's picking it up

root cause is remote sync engine is deqeuing the mutation events and processing them in parallel with the unit test. most probable solution: don't use the same test base as the others and create a simplified one for testing this, since it's only using storage adapter, we only need an in memory storage adapter to perform these unit tests.

Marked this in the main GH issue -https://github.com/aws-amplify/amplify-swift/issues/2340


make torchao test discovery pass in fbcode

Summary:
Running buck test @//mode/dev-nosan fbcode//pytorch/ao/test:torchao_tests used
to work but stopped working sometime in the past ~months due to test
discovery being broken. This diff fixes test discovery and adds a blocklist
to all of the broken tests, so we can at least run the passing tests easily.
Note that for now we have some tests using pytest which is handled in a custom
way. In a future PR it would be great to make torchao use a single testing
framework.
Reviewed By: jerryzh168
Differential Revision: D58422127

ok! Kinda concerning how many tests are broken internally though

this PR enables the tests to at least be discovered and tracked in fbcode, future work can fix individual tests in separate PRs

Oh yes was not implying you fix the failures right now, @vkuzo please feel free to merge this whenever you're ready


Fix example 9 stack overflow on Windows

It uses >2MB of stack memory for the multidimensional arrays, and by default windows only has 1MB

Fixed by new CMake


Cross platform snapshot paths

Output of restic version

restic 0.12.0 compiled with go1.15.8 on windows/amd64

What should restic do differently? Which functionality do you think we should add?
I would need to specify the snapshot path for it matches between Windows and Unix.
What are you trying to do? What problem would this solve?
I am building a open source backup system and I am trying add a backend for restic but I have problems with the snapshot path, because it is inconsistent between operative systems.
Really, the snapshot paths only it is used as tags, and it could be any thing. They are a reference about where it were data, but also it could be a id about what data it is. It is quite likely that data location changes because any reason in the future, but the data context always will be the same.
Use a id for the snapshot path is not possible because it is always converted to absolute and it also depends of the operative system, so there is a inconsistence: https://github.com/restic/restic/blob/fdbd65485e1d8b4f56159f7120a3277979921f4e/internal/restic/snapshot.go#L38
There are some issues opened about this but all are related with to allow relative paths and not with avoid the path normalization based in the operative system.
Possible solutions

Add option to overwrite snapshot paths: https://github.com/restic/restic/pull/3200.

This solution is not completely valid, read motive here: https://github.com/restic/restic/pull/3200#issuecomment-868474255

Add option to save raw paths (--raw-paths): https://github.com/juanrgm/restic/compare/backup-set-path...backup-raw-paths

If to skip completely the path normalization can be a problem, it could add a rules to avoid invalid chars.

Add option to save relative paths (--relative-paths): same proble that the first solution.
Add option to force the use of path conversions in Unix format (--unix-paths).
Add option to force the use of path conversions in Windows format  (--win-paths).

Did restic help you today? Did it make you happy in any way?

Please be more specific - what are you actually suggesting as a solution to your problem?

Thanks for your reply.

Please be more specific - what are you actually suggesting as a solution to your problem?

I edited the issue a bit adding some details, but basically I am proposing a solution to have the control over the snapshots paths to save.

Right, so in summary, you are asking that restic is changed so that it will let you specify the complete paths that are saved for a snapshot.

Yep.
I would like to know if core maintainers are agree with any solutions (or maybe other) to continue with the PR or not.
I know that there are pending pull requests and I would not want to increase the queue in vain.

I don't understand the use case. Paths on Unix/Windows are platform-specific, thus it sounds wrong to completely ignore the difference. And then --set-path should be sufficient?

but also it could be a id about what data it is

This sounds to me like you're trying to "tag" a snapshot via its backup path. Wouldn't it be possible to use tags instead?

I don't understand the use case. Paths on Unix/Windows are platform-specific, thus it sounds wrong to completely ignore the difference. And then --set-path should be sufficient?

--set-path keeps converting the path into absolute (ex: --set-path /data -> D:\data).

This sounds to me like you're trying to "tag" a snapshot via its backup path. Wouldn't it be possible to use tags instead?

That is the only alternative, but if the path is different between snapshots, restic will re-scan all the directory again, and what happens with the deduplicy? Does it will work in snapshots with differents paths? --parent should to fix these problems, right?

@juanrgm Deduplication will still be 100% effective regardless of the paths and parent snapshot used, and restic will still only upload what actually changed since the last backup. Using a parent snapshot that contains the files you are backing up again only affects the scanning of local files - when restic knows a parent snapshot that contains the same files, it can determine by metadata which of them hasn't changed since the last backup, and then doesn't have to scan them again to find data blocks that has possibly changed. But neither of this affects deduplication or re-uploading. This is purely about the scanning of the source data to back up. In simple terms, without a good parent snapshot, scanning takes a bit more time, that's it.

I don't understand the use case. Paths on Unix/Windows are platform-specific, thus it sounds wrong to completely ignore the difference. And then --set-path should be sufficient?

--set-path keeps converting the path into absolute (ex: --set-path /data -> D:\data).

But when is it necessary to refer to files using the same path, when the files were backed up on different systems? I understand that file paths are converted, but I still don't know enough about your use case to understand when that could be a problem.

The main purpose was to avoid the OS dependency when an app is migrated between systems and also abstract the data path in case it was changed in the future (due to system change or simply path change).
Using tags and specifying always --parent (for speed up) I can fix that problems.

Restic cannot abstract away differences between operating systems. In particular, file metadata (in particular ACLs) can only be transferred in a lossy way between OSes. Judging from the description here, what's actually wanted is more like #4026.


Inject Providers for dependencies which require db access

Description
Please explain the changes you made here.
Release notes
The title of the pull request will be used as the default release notes description. If more detail is needed to communicate to partners the scope of the PR's changes, use this release notes section.
Checklist
General
Read the full guidelines for PRs here

[ ] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release | database >
[ ] Created unit and/or browser tests which fail without the change (if possible)
[ ] Performed manual testing (Chrome and Firefox if it includes front-end changes)
[ ] Extended the README / documentation, if necessary

User visible changes

[ ] Followed steps to internationalize new strings
[ ] Wrote browser tests using the validateAccessibility method
[ ] Tested on mobile view. See mobile device mode
[ ] Manually tested at 200% size
[ ] Manually evaluated tab order

New Features

[ ] Add new FeatureFlag env vars to server/conf/helper/feature-flags.conf
[ ] Conditioned new functionality on a FeatureFlag
[ ] Wrote browser tests with the feature flag off and on, etc.

Instructions for manual testing
If instructions are needed for manual testing by reviewers, include them here.
Issue(s) this completes
Fixes #<issue_number>; Fixes #<issue_number>...

Current dependencies on/for this PR:

main

PR #5771   👈

PR #5695

PR #5730

PR #5731

This comment was auto-generated by Graphite.


Bad behaviour of captured vkEnumerateDeviceExtensionProperties

In every extension layer in this repository captured vkEnumerateDeviceExtensionProperties advertise their extensions if pLayerName == NULL, but the spec states:

When pLayerName parameter is NULL, only extensions provided by the Vulkan implementation or by implicitly enabled layers are returned. When pLayerName is the name of a layer, the device extensions provided by that layer are returned.

@charles-lunarg ISTR we discussed this at some point in the last few months and decided that these layers were doing what they needed. But I cannot find where. Do you remember?

This issue was more of "is it intended behaviour on not" rather then "found an error", so if it is by-design that's fine with me.

The other discussion can be found here: https://github.com/KhronosGroup/Vulkan-Docs/issues/2033


Add choices and documentcloud_pages to FilerStatusTypesCd.status_desc (in other.py)

Overview
Add choices and documentcloud_pages to the status_desc field on FilerStatusTypesCd the database model.
Basically, we think status_desc should be defined as a "choice field" (in Django parlance), which just means it's a field with a defined list of valid values and an intelligible definition for each value.
Our goal is for every potential choice field to include references to locations in the official documentation where the field's valid values and their meanings are described AND to have those choices included in the field's definition.
And we need your help!
What to do
Step 1: Claim this ticket by adding yourself as an Assignee (to the right)
Step 2: Find where in the official documentation status_desc's list of valid values is defined.
The best place to look first is in the .CAL layout documents: .CAL Format version 2.01 and .CAL Format version 1.05.02. These are documents that describe the layout of the .CAL file format, which is the required electronic format of any filings ingested by the CAL-ACCESS system (more on that here if you are interested).
You can start by opening the document and just searching for "status_desc". For example, if you were assigned to tackle the stmt_type field on the CvrCampaignDisclosureCd model, you would be looking for something like this::

But note that the same column might appear on multiple tables, and the list of valid values may vary depending on the table. We want to be sure to find the list of valid values for the status_desc on the FilerStatusTypesCd model.
Another good place to look is in the MapCalFormat2Fields document, which describes how .CAL format fields are mapped to CAL-ACCESS database table columns. Note the Valid_V column on the far right.
Step 3: Copy the DocumentCloud ID and the page number(s) where status_desc's list of valid values is defined.
The DocumentCloud ID can be found the in the URL that points to the document. For example, the url to the .CAL Format version 2.01 doc is http://www.documentcloud.org/documents/2712034-Cal-Format-201.html#document/p1, and the id is 2712034-Cal-Format-201.
And the start page we want to grab is the DocumentCloud page number, not the page number printed on the .PDF. If the list of valid values spans multiple pages, then grab the page where the list ends as well.
Step 4: Find where status_desc is defined in other.py in the /calaccess_raw/models/other.py file underneath the FilerStatusTypesCd class.
Since the model files are rather long, take care to be sure you find the where status_desc is defined for the FilerStatusTypesCd model. It should look something like this:
    class FilerStatusTypesCd(CalAccessBaseModel):
        # ...
        # several lines of code later...
        # ...
        status_desc = fields.CharField(
            max_length=11,
            db_column='STATUS_DESC',
            blank=False,
            verbose_name='status description',
            help_text='This field is undocumented',
        )

Step 5: Add the document references
Say you found the list of valid values listed in three places. You would list them in the documentcloud_pages attribute like this:
    class FilerStatusTypesCd(CalAccessBaseModel):
        # ...
        # several lines of code later...
        # ...
        status_desc = fields.CharField(
            max_length=11,
            db_column='STATUS_DESC',
            blank=False,
            verbose_name='status description',
            help_text='This field is undocumented',
            documentcloud_pages=[
                DocumentCloud(id='2711616-MapCalFormat2Fields', start_page=10),
                DocumentCloud(id='2712033-Cal-Format-1-05-02', start_page=19),
                DocumentCloud(id='2712034-Cal-Format-201', start_page=24, end_page=25),
            ]
        )

Note: You only need to specify the end_page if the list spans multiple pages.
Step 6: Update the status_desc's help_text (if necessary)
It might be missing or it might be an empty string or it might not be consistent with what you see in the documentation. Note: The entire help_text string needs to be enclosed in single- or double-quotes like this:
    help_text='This is the help text',

If the help text is really long (80+ characters), then it needs to span multiple lines like this:
    help_text='This is the help text...blah...blah...blah...blah...blah...blah...\
...blah...blah...blah',

Step 7: Define the choices
The normal Django pattern is to define choices within the model class as a tuple which contains a bunch of two item tuples. The first item of each inner tuple is the valid database column value, and the second item is the definition. It should look something like this:
    STATUS_DESC_CHOICES = (
        ('AAA', 'Some definition of AAA'),
        ('BBB', 'Some definition of BBB'),
        ('CCC', 'Some definition of CCC'),
        # ...and more..
    )

Then we assign this tuple of tuples to 's choice argument with a single line: choices=STATUS_DESC_CHOICES.
When finished, it should all look something like this:
    class FilerStatusTypesCd(CalAccessBaseModel):
        # ...
        # several lines of code later...
        # ...
        STATUS_DESC_CHOICES = (
            ('AAA', 'Some definition of AAA'),
            ('BBB', 'Some definition of BBB'),
            ('CCC', 'Some definition of CCC'),
            # ...and more..
        )
        status_desc = fields.CharField(
            choices=STATUS_DESC_CHOICES,
            max_length=11,
            db_column='STATUS_DESC',
            blank=False,
            verbose_name='status description',
            help_text='This field is undocumented',
            documentcloud_pages=[
                DocumentCloud(id='2711616-MapCalFormat2Fields', start_page=10),
                DocumentCloud(id='2712033-Cal-Format-1-05-02', start_page=19),
                DocumentCloud(id='2712034-Cal-Format-201', start_page=24, end_page=25),
            ]
        )

So some of these choice fields show up on multiple tables (the various office_cd fields are a good example), and of course it's rather redundant to repeat these values and their definitions over and over. In some these cases, we've consolidated these valid values into one place: calaccess_raw/annotations/choices.py.
So then instead of listing out all the choices in the tuple of tuples, we can do this:
    OFFICE_CD_CHOICES = get_sorted_choices(choices.OFFICE_CODES)
    office_cd = fields.CharField(
        choices=OFFICE_CD_CHOICES,
        # ...other stuff...
    )

If you think the field your looking at is a candidate for this sort of consolidation, let's have a conversation about it. Just post a comment below.
If you can, check the status_desc's defined choices against the actual values in status_desc's database column.
This will require you to install the app and run the updatecalaccessrawdata command. Then you can get the distinct column values with a GROUP BY query like this:
    SELECT STATUS_DESC, COUNT(*)
    FROM FILER_STATUS_TYPES_CD
    GROUP BY 1
    ORDER BY 1;

It's likely that you'll find undocumented values in database column, some of which are variants of the valid values. You can note these by repeating the same definition for each variant like this:
    OFFICE_CD_CHOICES = (
        ('GOV', 'Governor'),
        ('gov', 'Governor'),
        ('GUV', 'Governor'),
        # ...
    ),

If you're dealing with a set of choices that's been consolidated into calaccess_raw/annotations/choices.py, you can map the different variants to the same definition using the key of the choices dict like this:
    OFFICE_CD_CHOICES = (
        ('gov', look_ups.OFFICE_CODES['GOV']),
        ('GUV', look_ups.OFFICE_CODES['GOV']),
        # ...
    ),

If you're not able to check the database column values for whatever reason, don't sweat it. We'll deal with it in a later issue.
Wrap up: Review your changes and create a pull request

That's it!
PS: If any of this feels confusing, or if it doesn't seem to match up with what you are seeing, or if you have any suggestions about another approach, don't hesitate to speak up. You can post a comment below, or hit up the #california-civic-data channel on the News Nerdery Slack.

FilerStatusTypesCd is actually a look-up table.


users

it works for only one victim or can I send to this app to multiple victims and still receive sms from them?
can you tell me for how many victims it works

For now, I coded it only for one. In future updates I will try to


IndexRedirect not passing query

In an example like the one below, the IndexRedirect is ignoring the "query" prop:
<Router history={history}>
   <Route path="/" component={App}>
     <Route path="events" component={EventsPage} />
     <Route path="masterdata" component={MasterDataPage} />
     <IndexRedirect to={'/events'} query={{ player: 23 }} />
   </Route>
</Router>

Moreover, it would be maybe more consistent to use for both Redirect and IndexRedirect the same way to pass the new path to the Link components (using Location describer: {
pathname: '/events',
query: { player: 23 }
})

Using a pattern like <IndexRedirect to="/about" query={{ the: 'query' }} /> works just fine for me. Possibly something wrong with your history setup? (Try this w/e.g. the active-links example.)
We can't really use location descriptors in to for <Redirect> because of the specific semantics there around merging location.query/&c. with route.query when specified. In the more general redirect case, I think it's generally desired to forward those; if needed, you can always write a custom redirect component that suits your specific needs.

The strange thing is that the Link component works completely fine: it adds the query params in the URL after the '?'.
With IndexRedirect nothing happens.
I'll try to find out why :)

Maybe I found out why it doesn't work: I'm using redux-simple-router and it seems that it's not passing query params at all.
Thank you anyway :)

This was the same as https://github.com/rackt/react-router/issues/2917, BTW. Fixed on master of history.


Move CreatePodInfraContainer to DockerManager.

This will allow us to start making it an implementation detail of the
Docker runtime.

/cc @dchen1107 @yifan-gu

LGTM

LGTM. Will merge once it is green.

Build is green!


Collateral Manager V2

Uses thirdweb's token bundling contracts to simplify collateral management and reduce gas costs for submit bid by ~50%

Need to fix tests and write new ones


[deliver][download_metadata] give precedence to generate metadata from edit version instead of live version

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
Resolves #17568
There's a good context into the problem here: https://github.com/fastlane/fastlane/issues/17568#issuecomment-744423393
Description
Given the fact that deliver has a use_live_version option and another edit_live option and both default to false, seems like looking up the live version and then the edit version was wrong. So I inverted this condition and, if anyone run into issues, they can use the use_live_version to force using the live version instead.
Testing Steps
To test this branch, modify your Gemfile as:
gem "fastlane", :git => "https://github.com/fastlane/fastlane.git", :branch => "rogerluan-download-metadata-from-edit-version"

😥 once again, tests are failing in the CI but they're all green locally. I don't have visibility into what's failing in the CI 😞


Chat says that snapcast played on the pi but no music plays on the client

I have followed the information from the tutorial pretty thoroughly but asking nabu to play on a particular client results in the assistant confirming something is playing but the music player isn’t playing and no audio is going through to the client. When selecting the song and client manually, the song plays

I would start by checking to make sure the satellite is actually exposed to Assist, then make sure you have aliases to cover the various ways you could refer to it when speaking.


make entities use PyMunkPhysicsEngine

making all entities use PyMunkPhysicsEngine (rather than no engine for enemies and SimplePhysicsEngine for players)

Maybe we need to take a look at impulse forces for when entities are hit by weapons? https://api.arcade.academy/en/latest/_modules/arcade/pymunk_physics_engine.html#PymunkPhysicsEngine.apply_impulse

If enemies should not collide with each other, I think you could make a collision_handler. Maybe also for weapon/entity collisions? https://api.arcade.academy/en/latest/api/physics_engines.html#arcade.PymunkPhysicsEngine.add_collision_handler


Link to official CRA

There is an official release of create-react-app with TypeScript support that will be available very shortly.
How do you feel about updating the docs to reference create-react-app directly instead of the wmonk/jpavon forks when the new version lands?
Also for reference: https://github.com/facebook/create-react-app/issues/5581

i am fine with that. just been busy. will take a PR!

awesome stuff. so happy CRA supports typescript. thanks for the help and review everyone


First implementation of get lazy modules entries.

Awesome work. Looks great.

I need the loadChildren feature with webpack and can't wait until RC6 is released therefore I used the nightly builds but it does not work :(

ReferenceError: System is not defined

So I realized that I now need systemjs (in other words... 2 module loaders?)?
Which steps are needed to make the Angular 2 webpack example (https://angular.io/docs/ts/latest/guide/webpack.html) work with loadChildren?
Thank you so much!


remove todo, add gitSHA into version information

I1214 14:57:52.480730   20446 main.go:83] No controller_config_file provided; using empty config.
I1214 14:57:52.481311   20446 main.go:113] tf_operator Version: 0.3.0+git
I1214 14:57:52.481332   20446 main.go:114] Git SHA: ba0bdfb
I1214 14:57:52.481338   20446 main.go:115] Go Version: go1.9.2
I1214 14:57:52.481343   20446 main.go:116] Go OS/Arch: darwin/amd64

This change is

LGTM

@jlewi thanks you very much, will do it

@zjj2wry Let me know when this is ready for another look.

Coverage remained the same at 37.782% when pulling 97fc9bf37163a764c3da043b9b9b9dc5fca0aeaa on zjj2wry:makefile into 60b8f7a20e99519f6cc1a1c9fe2bdae6444b1eed on tensorflow:master.

@jlewi sorry for delay, this ready, ptal


Fix native_posix build when C++ exception handling is enabled

This series fixes the broken native_posix* build when C++ exception handling is enabled (CONFIG_EXCEPTIONS=y).
The actual fix is limited to the last commit, but I've found a few other things that require fixing while investigating this issue, so I'm including them here as well.
Fixes #36250.

@stephanosio Will this fix be backported to the v2.6 branch?

@stephanosio Will this fix be backported to the v2.6 branch?

Added backport tag.

Added backport tag.

Thanks!


[BUTTON IDEA]: Striped Button

1. Name of button: Striped Button
2. Description: A button that has a striped background. For each button color on the website, it should use the color and a darker shade of it for the stripes
3. Button Type (Animated, Special, etc...): Button Style
4. Will you work on it?: No, please comment on this to be assigned to it.

I would like to work on this issue

I would like to work on this please!

@Ovillena you may go ahead


Error: Python version not supported

When I tried to deploy this I got this error. I did not change anything while deploying

so am i


⚠ Bofrost.at has degraded performance

In f0cfb25, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1179 ms

Resolved: Bofrost.at performance has improved in 3e4c9c8.


GraphQL should not be enabled by default

We will not implement this since we would need to be able to retrieve this setting from the Frontend, and refactor search and list queries to use URL query params when it is disabled.


Update dockertoolbox to 1.10.2b

Also, check out cask-repair, it makes Cask updates like this quite easy (details in CONTRIBUTING)

Will merge as soon as Travis turns green

Thank you for the contribution. It was merged directly as 3f77a181214f424a413d8fef8b0f058f05bf0e27 to keep commit history cleaner. Your contribution is still credited to you.


use case: data fingerprinting/auditing

Report
Consider how we can better explain how DVC can be useful for data fingerprinting and auditing, especially for highly regulated industries like healthcare and banking.
cc @efiop @shcheklein

cc @jurv11  - I think it should be on the dvc.org, but also a blog post, + some updates to Iterative.ai

@dberenbaum @jurv11 any action items here?

@shcheklein Any thoughts on whether this should be a priority?

Yep, at least seems to be part of the effort that we've been discussing - business / high level value of the ecosystem.


⚠ Samsung has degraded performance

In 316a9b4, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2597 ms

Resolved: Samsung performance has improved in 9795309.


RELEASE.md: update with gcr promotion guidelines

Ref: https://github.com/kubernetes-sigs/prometheus-adapter/issues/414

/lgtm


pluralize --include-placeholders flag

[pivotal-cf/om#235]

Merging this. Changing p-automator references. It will run through our pipeline


inflate_fast_avx2 causes corrupted -- crc32 mismatch with pigz

Opening as a follow-up for https://github.com/madler/pigz/issues/123 (it contains more details)
Originally reported here: https://github.com/containers/podman/issues/23822
pigz fails with a  CRC32 mismatch error when decompressing a blob file. The same file passed the integrity check when tested with gzip.
Steps to reproduce:
Pull the container image with skopeo:
$ skopeo copy docker://docker.io/ollama/ollama:0.3.8-rocm oci:/var/tmp/ollama

Attempt to decompress the layer 1db71b1d7c67607172266cd839e3e429bf523aab5b0df4761fc0d05ec55dc727:
$ pigz -d < /var/tmp/ollama/blobs/sha256/1db71b1d7c67607172266cd839e3e429bf523aab5b0df4761fc0d05ec55dc727 > /dev/null
pigz: skipping: <stdin>: corrupted -- crc32 mismatch

The same file passes the integrity test with gzip:
$ gzip --test /var/tmp/ollama/blobs/sha256/1db71b1d7c67607172266cd839e3e429bf523aab5b0df4761fc0d05ec55dc727 && echo ok
ok

If I apply the following patch to disable using inflate_fast_avx2, the issue does not happen, --without-unaligned at configure time does not seem to make any difference:
diff --git a/functable.c b/functable.c
index dd8f7731..10abb358 100644
--- a/functable.c
+++ b/functable.c
@@ -115,7 +115,7 @@ static void init_functable(void) {
         ft.adler32_fold_copy = &adler32_fold_copy_avx2;
         ft.chunkmemset_safe = &chunkmemset_safe_avx2;
         ft.chunksize = &chunksize_avx2;
-        ft.inflate_fast = &inflate_fast_avx2;
+        //ft.inflate_fast = &inflate_fast_avx2;
         ft.slide_hash = &slide_hash_avx2;
 #  ifdef HAVE_BUILTIN_CTZ
         ft.compare256 = &compare256_avx2;

What Fedora version is this on, and what specific rpm versions of pigz and zlib-ng?
And for the self-compiled zlib-ng, what release(s)/commit(s) is this tested with?
I see that Fedora 40 and Rawhide are still based on 2.1.7, but I am unsure what you were using.

What Fedora version is this on, and what specific rpm versions of pigz and zlib-ng? And for the self-compiled zlib-ng, what release(s)/commit(s) is this tested with?

the error happens on Fedora 40.
The self-compiled zlib-ng is based on develop:
$ git log -1
commit e4fb3803afb8b46131abdeaaf927575d06621b0a
Author: Adeel Mujahid<EMAIL_ADDRESS>Date:   Thu Aug 29 14:27:43 2024 +0300

    Address CR feedback

I see that Fedora 40 and Rawhide are still based on 2.1.7, but I am unsure what you were using.

Doing a ton of time travel with our repo and pigz' and I don't have a smoking gun yet.  I do, however, find it strange that the file just happens to be above the 4GB mark and many of the zlib functions expect 32 bit arguments.  I'm starting to think that's a pretty important piece of the puzzle.

I mirrored the 4GB file here, for easier download without needing Docker
http://mirror.circlestorm.org/1db71b1d7c67607172266cd839e3e429bf523aab5b0df4761fc0d05ec55dc727

@giuseppe are you sure your patch fixed the issue?  I'm not seeing any difference disabling those functions in the function table.  I was hoping that was a helpful clue but it's coming up empty.

Oh, "with_native" evidently overrides things in the function table these days.  I'll see if I can sniff it out.  Thanks for the report.


Feature freeze for Stan 2.26

The feature freeze is now in effect for the Stan/Cmdstan 2.26 release and only bug-fixes should be merged until this issue is closed.
There is an outstanding question whether we should bump the major version of Math to 4.0 due to the changes in the LDLT_factor interface (https://github.com/stan-dev/math/pull/2315#issuecomment-761581647) - changes in this PR: https://github.com/stan-dev/math/pull/2266
I dont mind bumping the major version for Math so if those more familiar with the changes suggest a major bump I think we should do it. We have a week to decide.
Any thoughts @syclik @bbbales2 @wds15 @t4c1 @SteveBronder @yizhang-yiz et al?
cc: @serban-nicusor-toptal
Thanks!

Bump bump!

Bump bump!

+1 for the bump! (Now that versions have drifted, we could follow semantic
versioning without having to worry about special cases.)
On Tue, Jan 19, 2021 at 7:40 AM Ben Bales<EMAIL_ADDRESS>wrote:

Bump bump!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/issues/2327#issuecomment-762815922, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADH6F5KTQCGQZZYC642URLS2V4TNANCNFSM4WIO6HVA
.

+1 for the bump! (Now that versions have drifted, we could follow semantic
versioning without having to worry about special cases.)
On Tue, Jan 19, 2021 at 7:40 AM Ben Bales<EMAIL_ADDRESS>wrote:

Bump bump!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/issues/2327#issuecomment-762815922, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADH6F5KTQCGQZZYC642URLS2V4TNANCNFSM4WIO6HVA
.

Ok cool. If no one comes forward with some valid reasoning why not to bump, the release version should be 4.0 for Stan Math.
Stan/CmdStan proceed as normal (2.26).

Ok cool. If no one comes forward with some valid reasoning why not to bump, the release version should be 4.0 for Stan Math.
Stan/CmdStan proceed as normal (2.26).


feat(network): limit connections from same ip

What type of PR is this?
feat
What this PR does / why we need it:
Restrict connections from same ip. Add same_ip_conn_limit to network configuration, default value is 1.
We'll reject new connections from ip which reach connection limit.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/test chaos

/test chaos

/lgtm

/approve


Add timja to modern status

Description

I wish to deliver a dark-theme fix for https://github.com/jenkinsci/modernstatus-plugin/pull/5
The original maintainer appears to have deleted their github account, the ghost can be seen here: https://github.com/jenkinsci/modernstatus-plugin/pull/1
No one had access to the repo on GitHub, I added myself to check
any objections, / + or - 1?
cc @jsoref
Submitter checklist for adding or changing permissions

Always

[ ] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[ ] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[ ] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[ ] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

+1

+1


Little formatting problem and suggestion

Hello,
I just noticed a little formatting problem about some phone numbers. We can find these two format : +33123456789, and +33 1 23 45 67 89.
An example :
→ https://bit.ly/2sYY5AS (first format)
→ https://bit.ly/2MIi8wm (second format)
Moreover, I have a suggestion. When we are selecting a business or service, it would be nice to keep its name or icon displayed when we are zooming out on the map to keep in mind where we are going to if we have several destinations (for future itineraries).
Thank you !

thanks for your feedbacks and suggestions :heart:
You will be able to add an address to your favorites soon!
About the phone number, you are right, we need to improve the formatting!
If you or anyone else want to contribute by working on this enhancement, please create an issue in Idunn repository so we can provide guidance: we already have a few ideas about how to handle it.
PS : @JulesLassara if you have other suggestions or bugs, please create a new issue for each topic, it will be easier for us to provide you explanations or updates about them ;)


dialyzable, gastrulate

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[d,g]

/trunk merge


@alloy => Show a not for sale label when a work is considered not for sale

This is close to done.
I've added the logic for showing "Not for Sale". I made a few changes that specifically increase the LOC at the expense of making it as close as possible to our shared logic doc.

Fixed https://github.com/artsy/eigen/issues/865

Had to make a test pending, as it was testing behaviour that doesn't match with the doc.

I think the CHANGELOG is not up-to-date, but I’ll check that after merging.


meta: 2.18.2 not marked as "Release"

In https://github.com/johndyer/mediaelement/releases, it is listed as a tag, but not as an officially blessed "Release" (Latest Release button point on 2.18.1).
Is it conscious or 2.18.2 is normal release as the others?

@johndyer Any thoughts on this?

@canadaduane I updated the tag to the latest one; from now on they will be tagged. Thanks for reporting this.


Cleanup XML of action SQL #1994

I have change the Const.toBoolean() method to supports the value 'T' as TRUE for backwards compatibility with SQL Action serialization when switching to HopMetadateProperty.

@hansva remember to apply to these PRs the same changes I've applied in the transform's Dialog classes contained in my PR #2942 to fix issue #2939. Let me know if you need help.

The same logic is not applied to the workflow AbstractMeta class so there is no problem


googleearth-pro: fix xkb keyboard and use nixpkgs libs

Motivation for this change
Fixes googleearth-pro keyboard not working.
Using Nixpkgs latest Qt libs makes it work way better.
Security Issues
Among others, googleearth-pro uses :

openssl 1.0.0
libav_12
icu54

libav_12 still exists on nixpkgs but the two left aren't even there anymore so I had to use the bundled dlls.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Pinging Maintainers
@friedelino

Result of nixpkgs-review pr 120449 at db069817 run on x86_64-linux 1

1 package built successfully:

googleearth-pro

2 suggestions:

warning: fixup-phase
fixupPhase should not be overridden, use postFixup instead.
Near pkgs/applications/misc/googleearth-pro/default.nix:101:3:
    |
101 |   fixupPhase = ''
    |   ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/misc/googleearth-pro/default.nix:62:3:
   |
62 |   installPhase =''
   |   ^

Why did you close this PR ?

Why did you close this PR ?

Closed it by mistake. Sorry for that.

Why did you close this PR ?

Closed it by mistake. Sorry for that.

No problem :+1:


unable to resolve object permissions on objects with integer primary keys using Postgres

The issue flagged in this ancient django-guardian issue is rearing its head with the permissions checking with this package. The HasObjPerm directive is generating SQL code which cannot be executed against Postgres because it is using the IN operator without a typecast.
The django-guardian shortcuts library has handled this here, but it looks like this package is usingObjectPermissionChecker directly.
exception
django.db.utils.ProgrammingError: operator does not exist: bigint = character varying
LINE 1: ...odename" = 'view_offer') LIMIT 1) OR "offer"."id" IN ((SELEC...

generated SQL
SELECT "offer"."id",
       "organization"."id"
  FROM "offer"
  LEFT OUTER JOIN "organization"
    ON ("offer"."organization_id" = "organization"."id")
 WHERE (EXISTS(SELECT (1) AS "a" FROM "auth_permission" U0 INNER JOIN "auth_user_user_permissions" U1 ON (U0."id" = U1."permission_id") WHERE (U1."user_id" = 6 AND U0."content_type_id" = 26 AND U0."codename" = 'view_offer') LIMIT 1) OR EXISTS(SELECT (1) AS "a" FROM "auth_group" U0 INNER JOIN "auth_user_groups" U1 ON (U0."id" = U1."group_id") INNER JOIN "auth_group_permissions" U3 ON (U0."id" = U3."group_id") INNER JOIN "auth_permission" U4 ON (U3."permission_id" = U4."id") WHERE (U1."user_id" = 6 AND U4."content_type_id" = 26 AND U4."codename" = 'view_offer') LIMIT 1) OR "offer"."id" IN ((SELECT DISTINCT "guardian_userobjectpermission"."object_pk" FROM "guardian_userobjectpermission" INNER JOIN "auth_permission" ON ("guardian_userobjectpermission"."permission_id" = "auth_permission"."id") WHERE ("guardian_userobjectpermission"."user_id" = 6 AND "auth_permission"."content_type_id" = 26 AND "auth_permission"."codename" = 'view_offer' AND "guardian_userobjectpermission"."content_type_id" = ("auth_permission"."content_type_id"))) UNION (SELECT DISTINCT "guardian_groupobjectpermission"."object_pk" FROM "guardian_groupobjectpermission" INNER JOIN "auth_group" ON ("guardian_groupobjectpermission"."group_id" = "auth_group"."id") INNER JOIN "auth_user_groups" ON ("auth_group"."id" = "auth_user_groups"."group_id") INNER JOIN "auth_permission" ON ("guardian_groupobjectpermission"."permission_id" = "auth_permission"."id") WHERE ("auth_user_groups"."user_id" = 6 AND "auth_permission"."content_type_id" = 26 AND "auth_permission"."codename" = 'view_offer' AND "guardian_groupobjectpermission"."content_type_id" = ("auth_permission"."content_type_id")))))

models
class Organization(models.Model):
    id = models.AutoField(primary_key=True)

class Offer(models.Model):
    id = models.AutoField(primary_key=True)
   organization = models.ForeignKey(Organization)

schema:
@gql.django.type(Organization)
class OrganizationType:
    id: gql.auto

@gql.django.type(Offer, filters=OfferFilter, directives=[HasObjPerm("app.view_offer")])
class OfferType:
   id: gql.auto
   organization: "OrganizationType"

class Query:
  offers: List[Optional[OfferType]] = gql.django.field(
        directives=[IsAuthenticated()],
        filters=OfferFilter,
    )

query:
{
  offers { id organization { id } }
}

This pull request works around this issue and hints at a more general fix. Don't know when I'll get the time to make a contribution back. https://github.com/zacatac/strawberry-django-plus/pull/1

Hey @zacatac ,
The fix seems fine, the only issue I see with it is that the object_pk might not be an integer. For example, some people use UUIDField for pks
Probably it would be nice to introspect the model at the time of doing that query and check which kind of field the pk is, and cast object_pk to that kind. What do you think?

hey @bellini666. Totally agree that introspecting on object_pk. The django-guardian lib handles that introspection here.


Pass more uniforms to screen shaders

It would be great if things like the time and maybe even cursor position were available from within the shader.

no, because it's impossible to track damage when a shader has access to dynamic uniforms


Meteor buildpack fails with tar error

This meteor buildpack doesn't seem to work on Deis.
$ deis config
=== rocket Config
BUILDPACK_URL        https://github.com/olalonde/heroku-buildpack-meteor.git
HEROKU_APP_NAME      rocket

$ git push deis -f
Counting objects: 3, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (3/3), done.
Writing objects: 100% (3/3), 422 bytes | 0 bytes/s, done.
Total 3 (delta 2), reused 0 (delta 0)
-----> Fetching custom buildpack
-----> Node.js app detected
+ echo -----> Listing HEROKU_ environment variables
+ echo HEROKU_APP_ID:
+ echo HEROKU_APP_NAME:         rocket
+ echo HEROKU_DYNO_ID:
+ echo HEROKU_RELEASE_VERSION:
+ echo HEROKU_SLUG_COMMIT:
+ echo HEROKU_SLUG_DESCRIPTION:
+ export_env_dir
+ env_dir=
+ whitelist_regex=
+ blacklist_regex=^(PATH|GIT_DIR|CPATH|CPPATH|LD_PRELOAD|LIBRARY_PATH)$
+ [ -d  ]
+ dirname -- /tmp/buildpacks/custom/bin/compile
-----> Listing HEROKU_ environment variables
+ cd -P -- /tmp/buildpacks/custom/bin
+ cd ..
+ pwd -P
+ BUILDPACK_DIR=/tmp/buildpacks/custom
+ APP_CHECKOUT_DIR=/tmp/build
+ APP_SOURCE_DIR=/tmp/build
+ [ ! -d /tmp/build/.meteor ]
+ mktemp -d /tmp/buildpacks/custom/meteor-XXXX
+ METEOR_DIR=/tmp/buildpacks/custom/meteor-GImr
+ COMPILE_DIR_SUFFIX=.meteor/heroku_build
+ COMPILE_DIR=/tmp/build/.meteor/heroku_build
+ mkdir -p /tmp/build /tmp/buildpacks/custom/meteor-GImr /tmp/build/.meteor/heroku_build
       HEROKU_APP_ID:
+ PATH=/tmp/buildpacks/custom/meteor-GImr/.meteor/:/tmp/build/.meteor/heroku_build/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
+ echo -----> Installing node
+ curl -sS --get https://semver.io/node/resolve/0.10.x
       HEROKU_APP_NAME:         rocket
       HEROKU_DYNO_ID:
       HEROKU_RELEASE_VERSION:
       HEROKU_SLUG_COMMIT:
       HEROKU_SLUG_DESCRIPTION:
-----> Installing node
+ NODE_VERSION=0.10.41
+ NODE_URL=http://s3pository.heroku.com/node/v0.10.41/node-v0.10.41-linux-x64.tar.gz
+ tar -zxf - -C /tmp/build/.meteor/heroku_build --strip 1
+ curl -sS http://s3pository.heroku.com/node/v0.10.41/node-v0.10.41-linux-x64.tar.gz -o -
+ export npm_config_prefix=/tmp/build/.meteor/heroku_build
+ export CPATH=/tmp/build/.meteor/heroku_build/include
+ export CPPPATH=/tmp/build/.meteor/heroku_build/include
+ echo -----> Installing meteor in /tmp/buildpacks/custom/meteor-GImr
+ HOME=/tmp/buildpacks/custom/meteor-GImr /bin/sh
+ curl -sS https://install.meteor.com
-----> Installing meteor in /tmp/buildpacks/custom/meteor-GImr
Downloading Meteor distribution
remote: ######################################################################## 100.0%
tar: .meteor/packages/babel-compiler/.5.8.24_1.127zxv6++os+web.browser+web.cordova/npm/node_modules/meteor-babel/node_modules/defs/node_modules/.bin: Directory renamed before its status could be extracted
tar: .meteor/packages/babel-compiler/.5.8.24_1.127zxv6++os+web.browser+web.cordova/npm/node_modules/meteor-babel/node_modules/defs/node_modules: Directory renamed before its status could be extracted
tar: .meteor/packages/babel-compiler/.5.8.24_1.127zxv6++os+web.browser+web.cordova/npm/node_modules/meteor-babel/node_modules/defs: Directory renamed before its status could be extracted
tar: .meteor/packages/babel-compiler/.5.8.24_1.127zxv6++os+web.browser+web.cordova/npm/node_modules/meteor-babel/node_modules/.bin: Directory renamed before its status could be extracted
tar: Exiting with failure status due to previous errors
Installation failed.
To ssh://git@deis.local3.deisapp.com:2222/rocket.git
 + 70bdf57...cd70ab4 deis -> deis (forced update)

Related: https://github.com/meteorhacks/meteord/issues/64 https://github.com/meteor/meteor/issues/5762
I'm not sure if this is something that can be fixed by Deis but posting here just in case.

As you've probably seen in my issues (meteorhacks/meteord#64 and meteor/meteor#5762) the problem can not be fixed by Deis as it is linked to OverlayFS which is the default storage driver for Docker on CoreOS.
OverlayFS doesn't like the fact that a lot of files are created while decompressing this specific tar... An issue should be opened on docker's repo I think.

Ok, will do.


feature - search ui

Github Issue URL/Tags (required)
What's this PR do? Please give a technical summary (required)

[x] create search UI

Any references (stackoverflow …) (optional)

:+1: There are no linter violations.


[SMALLFIX] Replace anon type with lambda in DFSM.Metrics#registerGauges

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5878/
Test FAILed.

Part of https://github.com/Alluxio/new-contributor-tasks/issues/562

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5879/
Test PASSed.

alluxio-bot, merge this please.


[WIP] Backend storage: Support offline snapshot

What this PR does
This PR aims to add offline snapshot and restore support of VMs with backend storage.
Online snapshot and clone will be supported in follow-up PRs.

Fixes # https://issues.redhat.com/browse/CNV-51793
Release note

Bugfix: Support offline snapshot of VMs with backend storage

/test pull-kubevirt-e2e-k8s-1.31-sig-storage

One thing to consider here is what happens if there are writes to backend storage during the snapshot? Even if the guest agent is running I'm pretty sure the freeze hook will not pause i/o to the backend volumes. @jean-edouard what do you think? For example, could a encryption key be updated/added during the snapshot potentially making some data inaccessible after the restore?

Totally unrelated and probably dismissable - is snapshotting these 100% necessary? can we maybe get away somehow or does a libvirt user do the same for their snapshot?

I'm working on a big update on this PR to actually support and test this behavior, currently just trying to fix the case when we are restoring TPM to a different VM. I'll hopefully post it later today.

not sure about this. could be breaking existing user flows. Fact is that tpm state rarely changes so snapshot+restore in same namespace will usually be no problem

not sure about this. could be breaking existing user flows. Fact is that tpm state rarely changes so snapshot+restore in same namespace will usually be no problem

Removed webhooks and simply ignored the backend volume (+ triggering an event) during online snapshot/clone.

Removed webhooks and simply ignored the backend volume (+ triggered an event) during online snapshot/clone.

Comment WRT clone flow. For security reasons, a clone should not have the same backend state as the source

Removed webhooks and simply ignored the backend volume (+ triggered an event) during online snapshot/clone.

Comment WRT clone flow. For security reasons, a clone should not have the same backend state as the source

Already replied offline but I agree we might need to reconsider if clone is necessary or even possible to support. However this PR just skips the backend volume when doing a clone so we shouldn't worry about the security implications here.

/test pull-kubevirt-fuzz

/retest

/test pull-kubevirt-e2e-k8s-1.31-sig-compute-migrations

Something's failing after rebase, will address it tomorrow.

/test pull-kubevirt-code-lint
/test pull-kubevirt-code-lint

/test pull-kubevirt-e2e-k8s-1.31-sig-operator
/test pull-kubevirt-manifests

/test pull-kubevirt-e2e-k8s-1.31-sig-operator


Ignore hidden inputs

Currently, all inputs will be registered, even if they are hidden. This is a pretty big problem. Not strictly speaking a bug, but a problem.

Fixed by #17

Doesn't ignore them if they have the "hidden" attribute, just if their type is hidden.

Also ought to handle disabled. Perhaps still attach listeners to disabled and hidden inputs, but don't validate them while they are in that state?

Also ought to handle disabled. Perhaps still attach listeners to disabled and hidden inputs, but don't validate them while they are in that state?


randomize instrumentLimitEvery

If we run in fork-and-exec mode, staticCnt will starts
from zero for each run. So for example __sanitizer_cov_trace_cmp4 will
only be executed at the 4096th time...
If the target is small to have 4096 cmp during execution,
__sanitizer_cov_trace_cmp4 trick will never work.
Even if the target is big, instrumentLimitEvery is may still biased....
So I just make it random..

Cool, thank you!


Question about re-implementations

I have a question/doubt regarding the plug-in implementation.
Why does it reimplement steps such as changelog generation and release creation? AFAIK release-it plugins allow one to implement only some of the steps, instead of the whole thing.
I'm asking so because there is no GItLab release support. Instead of porting this feature, this plugin probably should not be opinionated about releases, only about changing the CLI command to run it with pnpm.

How about disable the npm publish
tried adding both package.json and release-it.json it doesnt work
 "npm": {
    "publish": false
  },


Is DefaultAzureIdentity supported in .NET Core applications?

Query/Question
How can we help?
Is DefaultAzureIdentity supported in .NET Core applications? If not, is there another encapsulation object that should be used?
Environment:

Name and version of the Library package used: Azure.Identity 1.2.2
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects):

Windows 2016/2019, .NET Core 3.1.x

Host (useful for support):
Version: 3.1.3
Commit:  4a9f85e9f8
.NET Core SDKs installed:
No SDKs were found.
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All
Microsoft.AspNetCore.All 2.1.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All
Microsoft.AspNetCore.All 2.1.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All
Microsoft.AspNetCore.All 2.1.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All
Microsoft.AspNetCore.All 2.1.9 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All
Microsoft.AspNetCore.All 2.1.10 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.Al
Microsoft.AspNetCore.All 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All
Microsoft.AspNetCore.App 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App
Microsoft.AspNetCore.App 2.1.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App
Microsoft.AspNetCore.App 2.1.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App
Microsoft.AspNetCore.App 2.1.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App
Microsoft.AspNetCore.App 2.1.9 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App
Microsoft.AspNetCore.App 2.1.10 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.Ap
Microsoft.AspNetCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App
Microsoft.AspNetCore.App 3.1.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App
Microsoft.AspNetCore.App 3.1.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App
Microsoft.NETCore.App 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.8 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.10 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

IDE and version : [e.g. Visual Studio 16.3]
Microsoft Visual Studio Professional 2019
Version 16.6.5

Is DefaultAzureIdentity supported in .NET Core applications?

Yes it is. What exactly the problem you have?

No problem. We are using it for .NET Framework, and we were not sure about .NET Core. We’ll try a test implementation and report any problems/confusion in the process.
From: Alexander Sher<EMAIL_ADDRESS>Sent: Tuesday, August 25, 2020 1:49 PM
To: Azure/azure-sdk-for-net<EMAIL_ADDRESS>Cc: James Bristow<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [Azure/azure-sdk-for-net] Is DefaultAzureIdentity supported in .NET Core applications? (#14555)
[CAUTION: This email originated from outside of the ASH organization]
Is DefaultAzureIdentity supported in .NET Core applications?
Yes it is. What exactly the problem you have?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-net%2Fissues%2F14555%23issuecomment-680261880&data=02|01|jamesbr%40ashn.com|7cedb1f4fd054ca4a0bc08d84938399f|da7b84dc67484ba59b021f3a0bdc2b50|0|1|637339853115184400&sdata=V3ZsFld26msE4O3L1B9809DZMuL%2B5Yvwittt0EmcFK8%3D&reserved=0, or unsubscribehttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANLJUNLGGQ443DHPOODJZYTSCQPR5ANCNFSM4QLCGQ6A&data=02|01|jamesbr%40ashn.com|7cedb1f4fd054ca4a0bc08d84938399f|da7b84dc67484ba59b021f3a0bdc2b50|0|1|637339853115194359&sdata=O3mAWF1mELRJ%2BHUQ8PqI3Pdn0Q8r8ONBDMG2wzyT1rw%3D&reserved=0.
CONFIDENTIALITY NOTICE: This e-mail, and any attachments, are intended solely for the use of the intended recipient and may contain information that is confidential, proprietary, copyrighted and/or legally privileged. Any unauthorized disclosure, distribution, or use of this information is prohibited. If you are not the intended recipient, and have received this message in error, please notify the sender immediately by reply e-mail and destroy all copies of the original message and any attachments. Your compliance is appreciated.


Support SSLKEYLOGFILE env var

WRT my question on the mailing list, looks like this should allow logging TLS session keys. The code compiles and tests pass, but as something of a Go newbie I didn't know how to test this with gemget.
No need to merge this PR if it is badly written, but I'd love to see gemget support this.

Could you explain the purpose of this for gemget? I don't really get what the point would be. It also feels a bit strange to have this in the library, but it's not a big deal.
It'd be nice to change the flag to GEMINI_TLSKEYLOGFILE or something like that, to reduce conflict issues.
Go code things:

You always need to be closing the file, so right under  config.KeyLogWriter = w in both files, you should add defer w.Close()
os.O_WRONLY|os.O_CREATE should become os.O_WRONLY|os.O_CREATE|os.O_TRUNC to truncate the file each time. Or does it make sense to append to this kind of file?

Could you explain the purpose of this for gemget? I don't really get what the point would be. It also feels a bit strange to have this in the library, but it's not a big deal.

This can be used for debugging, ie. decrypting TLS traffic with Wireshark. The workflow is:

You have to control either the server or the client (they negotiate session keys, and they both know them).
Make the software log sessino keys
Make wireshark log all traffic to/from the software
Do something
Give the saved session keys to Wireshark, and you'll see the decrypted traffic.

At least that's the theory.
It would be nice to have this functionality in gemget as it's the pretty much the nearest thing to curl in Gemini. Whatever connections I'd like to debug, I could ask gemget to log the keys and record the traffic with wireshark.
Adding the functionality to the library has the benefit that it's available "for free" also in any client or server using go-gemini.

It'd be nice to change the flag to GEMINI_TLSKEYLOGFILE or something like that, to reduce conflict issues.

Recent versions of curl, firefox and chrome already have SSLKEYLOGFILE so I think this is something of a standard.

You always need to be closing the file, so right under  config.KeyLogWriter = w in both files, you should add defer w.Close()

Good to know!

os.O_WRONLY|os.O_CREATE should become os.O_WRONLY|os.O_CREATE|os.O_TRUNC to truncate the file each time. Or does it make sense to append to this kind of file?

I think it's better to append, especially since every Gemini request renegotiates the TLS connection AFAIK. But I'm new to this so I might be wrong.

Here's some documentation about the feature in libcurl: https://ec.haxx.se/usingcurl/usingcurl-tls/tls-sslkeylogfile
Notably the curl that ships with macOS doesn't support the flag. Neither does the curl installed from Homebrew. I don't understand why. But on Debian it works.

That all, makes sense to me, thanks!

I think it's better to append, especially since every Gemini request renegotiates the TLS connection AFAIK. But I'm new to this so I might be wrong.

Yes, every request will renegotiate. But if you want to append, you should OR in os.O_APPEND. Then if you test your changes that'd be great! Make sure you're actually able to walk through that process of using the keys in Wireguard.
After that I'll be happy to merge this in, thanks.

Took me a bit to figure out how to build gemget with this (go mod vendor, then copy the code over). But yes, it works!

Thanks! Merged.

Just wanted to add that this feature is in v1.3.0 of Amfora, I forgot to mention that. It will be in the next release of gemget (latest is v1.4.0), but I don't think I need to update here when that happens.


CMake configuration fails with MSBuild as the generator because of empty 'CMAKE_BUILD_TYPE'

I actually wasn't able to get the repo building anymore using the instructions in the README... CMAKE_BUILD_TYPE seems to evaluate to empty string even though I call init_all / init.cmd with the right params. Perhaps something is out of sync in the readme?
Originally posted by @asklar in https://github.com/microsoft/wil/pull/317#discussion_r1161873794

Can you share what commands you ran and the outputs you got?

F:\wil> scripts\init.cmd -c msvc -g msbuild
The system cannot find the drive specified.
The system cannot find the drive specified.
Using compiler....... msvc
Using architecture... x64
Using build type..... debug
Using build root..... F:\wil\build\msvc64debug

-- Building for: Visual Studio 17 2022
-- Selecting Windows SDK version 10.0.25300.0 to target Windows 10.0.
-- The C compiler identification is MSVC 19.35.32215.0
-- The CXX compiler identification is MSVC 19.35.32215.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: F:/Program Files/Microsoft Visual Studio/2022/Enterprise/VC/Tools/MSVC/14.35.32215/bin/Hostx64/x64/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: F:/Program Files/Microsoft Visual Studio/2022/Enterprise/VC/Tools/MSVC/14.35.32215/bin/Hostx64/x64/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
Feeds used:
  C:\Users\asklar\.nuget\packages\
  https://api.nuget.org/v3/index.json
  C:\Program Files (x86)\Microsoft SDKs\NuGetPackages\
  C:\temp

Attempting to gather dependency information for package 'Microsoft.Windows.CppWinRT.2.0.221121.5' with respect to project 'F:\wil\build\msvc64debug\packages', targeting 'Any,Version=v0.0'
Gathering dependency information took 258 ms
Attempting to resolve dependencies for package 'Microsoft.Windows.CppWinRT.2.0.221121.5' with DependencyBehavior 'Lowest'
Resolving dependency information took 0 ms
Resolving actions to install package 'Microsoft.Windows.CppWinRT.2.0.221121.5'
Resolved actions to install package 'Microsoft.Windows.CppWinRT.2.0.221121.5'
Retrieving package 'Microsoft.Windows.CppWinRT 2.0.221121.5' from 'C:\Users\asklar\.nuget\packages\'.
Adding package 'Microsoft.Windows.CppWinRT.2.0.221121.5' to folder 'F:\wil\build\msvc64debug\packages'
Added package 'Microsoft.Windows.CppWinRT.2.0.221121.5' to folder 'F:\wil\build\msvc64debug\packages'
Successfully installed 'Microsoft.Windows.CppWinRT 2.0.221121.5' to F:\wil\build\msvc64debug\packages
Executing nuget actions took 453 ms
CMake Error at tests/CMakeLists.txt:44 (if):
  if given arguments:

    "STREQUAL" "RelWithDebInfo"

  Unknown arguments specified

-- Configuring incomplete, errors occurred!
See also "F:/wil/build/msvc64debug/CMakeFiles/CMakeOutput.log".

yes, init-all fails for me because I don't have/use clang :)

Gotcha. So, in that case, it sounds like this should just be scoped down to running init.cmd with MSBuild as the generator

I have a mitigation for this---
Wrap the if conditions that check CMAKE_BUILD_TYPE (on ~line 45 and ~75) in checks for Clang:
if (CMAKE_CXX_COMPILER_ID MATCHES "Clang")
    # For some unknown reason, 'RelWithDebInfo' compiles with '/Ob1' as opposed to '/Ob2' which prevents inlining of
    # functions not marked 'inline'. The reason we prefer 'RelWithDebInfo' over 'Release' is to get debug info, so manually
    # revert to the desired (and default) inlining behavior as that exercises more interesting code paths
    if (${CMAKE_BUILD_TYPE} STREQUAL "RelWithDebInfo")
        # TODO: This is currently blocked by an apparent Clang bug: https://github.com/llvm/llvm-project/issues/59690
        # replace_cxx_flag("/Ob1" "/Ob2")
    endif()
endif()

if (CMAKE_CXX_COMPILER_ID MATCHES "Clang")
    if (${CMAKE_BUILD_TYPE} STREQUAL "Debug")
        set(DEBUG_BUILD TRUE)
        set(HAS_DEBUG_INFO TRUE)
    elseif(${CMAKE_BUILD_TYPE} STREQUAL "RelWithDebInfo")
        set(HAS_DEBUG_INFO TRUE)
    endif()
endif()

This is probably not the correct approach (CMAKE_BUILD_TYPE should probably have a value during msbuild), but it unblocked me.

This is probably not the correct approach

Right. The problem is not with the target compiler, but with the target generator. MSVC+Ninja works correctly; it's only when using MSBUILD as the generator that this issue presents itself


update packages

:white_check_mark: Build TheIdServer 7.0.3-fix-dependencies1-0038 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/530a65ff73 by @)


🚀 - Provide taiga-ui-theme.scss

Which @taiga-ui/* package(s) are relevant/releated to the feature request?
No response
Description
Current State
There exists only a Less version of taiga-ui-theme : @taiga-ui/core/styles/taiga-ui-theme.less.
Requested feature
It would be nice to also have a Sass version of taiga-ui-theme: @taiga-ui/core/styles/taiga-ui-theme.scss.

Hi!
You can use taiga-ui-theme.less with your pure CSS, SCSS and other projects. Angular resolves this less files automatically.
See https://taiga-ui.dev/getting-started#styles

You can use taiga-ui-theme.less with your pure CSS, SCSS and other projects. Angular resolves this less files automatically.

Currently we are using this option but it would be nice if it would be possible using @import to avoid implicit importing.

@jekru
Unfortunately, it is too difficult to support both versions (.less + .scss) of the this file.
taiga-ui-theme.less contains only 4 imports (at the first glance, looks simple):
https://github.com/Tinkoff/taiga-ui/blob/e3bcca0239217c507cb2736b079b2e70132fe47e/projects/core/styles/taiga-ui-theme.less#L1-L4
But all of this imports contain huge amount of other imports.
For example, see one of them:
https://github.com/Tinkoff/taiga-ui/blob/e3bcca0239217c507cb2736b079b2e70132fe47e/projects/core/styles/theme/wrapper.less#L1-L12
To create taiga-ui-theme.scss-version, we should duplicate enormous amount of .less-files.
It is strictly violates DRY-principle without much need.
I understand your desire to have possibility to import this file explicitly.
But if angular-way works, let's keep only this approach.
However, if you face any bugs with this approach, let's know.
We are ready to fix any bugs!


query for resources on ksm collection activity

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix

Needs QA check

@kodadot/qa-guild please review

Context

[x] Closes #8034

Problem fixed
collection activity query was missing the resources field, which is how we get the images on ksm
tested on other chains as well, works as it should
Link
https://deploy-preview-8035--koda-canary.netlify.app/ksm/collection/dac5c7f54029d0e73c-XCHIMPZ/activity
Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[x] My fix has changed UI

Copilot Summary

🤖 Generated by Copilot at 88d0e56
Refactored queryPrefix variable in useCollectionActivity composable to use a map object. Added collectionActivityEvents query for Kusama network to fetch and display NFT collection events.

🤖 Generated by Copilot at 88d0e56

queryPrefix maps
blockchain networks to queries
autumn of refactor

Thanks!
pay 30 usd

😍 Perfect, I’ve sent the payout
💵 $30 @ 5.53 USD/DOT ~ 5.425 $DOT
🧗 1cAsKZYNRb8dkSCpn4eVkEn6ycNZTGoDo5dGDgB8J1UUWK8
🔗 0x5260deb13c4129b5a62fd37e732915549eeddd54525e590d5c20ec269282fa3c
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Bash completion for ros and Docker

The following worked for me in the interim in terms of getting docker command completion:
sudo curl -L -o /etc/bash_completion.d/docker https://github.com/docker/cli/raw/master/contrib/completion/bash/docker

Then log out and back in again, or just source  /etc/bash_completion.d/docker.

Depending on your console the distro might prodive docker completion packages.
Here are the packages for Alpine:
https://pkgs.alpinelinux.org/packages?name=docker-*-completion&branch=v3.8

Add it to milestone v1.5.1.
We don't want the default console to be too complicated.
For other consoles, I think we can add it.

It seems that integrating bash completion for Docker is very complicated for ROS, because ROS can switch user-docker and each docker has its own completion files.
Maybe we can only tell the user how to do it in the documentation.
Do you have any ideas? @olljanat

Supported consoles:

alpine
centos
debian
fedora
ubuntu

Supported engines:

docker-18.06.0
docker-18.06.0_arm
docker-18.06.1
docker-18.06.1_arm

For the default console, it's a bit complicated to enable bash-completion. We want the default console to keep simple, therefore, the default console is not currently supported.

@olljanat We have supported Docker Bash completion in other consoles in the rancheros v1.5.1-rc1 version.


After "Install from File" addon is not appearing in addons window

Thanks a lot for this addon! Guys like you make things happen and enable others to make their mods!
Hats off for this complex tool sir!!!
Im having strange difficulties instaling it though.  Installed it  with "Install from file" button. When i push this button it does not show  installed addon. It shows huge list of blender addons. Wierd stuff. In the stats field it says that addon installed. I even can find it in the addons_contrib folder. Its a Blender_ioEDM-0.2.3.a folder that has multiple files in it(4 folders and 5 files) But when i try to type in edm  in the addons browser search field nothing shows up. I use fresh install of 2.78 on C drive. Am i doing something wrong?
Thanks!

Did you ever fix it?


Apply animation to navigateUp()

Describe the solution you'd like
I hope apply scroll animation when call navController().navigateUp()
Describe alternatives you've considered
I found accompanist-navigation-animation:0.16.1, which does not have gestures, but does this issue: https://medium.com/androiddevelopers/animations-in-navigation-compose-36d48870776b
Is it possible to combine Fragula with it?

Hi @neslxzhen , I think it's possible, but I need some time to understand how it works internally. I didn't migrate Fragula on the latest version of NavComponent yet

Fixed in v2.6


Update definition and documentation for policy callback

Closes #8

I'll put this out in 1.0.0.rc.1, which it looks like I'll need to get back to soonish. Probably 1.0 in August.

👍 awesome


rasterio should be optional

Currently, rasterio is required at least to run the tests:
$ py.test datashader --verbose
================================ test session starts ================================
platform darwin -- Python 3.6.0, pytest-3.0.5, py-1.4.32, pluggy-0.4.0 -- /Users/cball/anaconda3/bin/python
cachedir: .cache
rootdir: /Users/cball/code/datashader, inifile:
collected 68 items / 1 errors

====================================== ERRORS =======================================
_________________ ERROR collecting datashader/tests/test_raster.py __________________
ImportError while importing test module '/Users/cball/code/datashader/datashader/tests/test_raster.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
datashader/tests/test_raster.py:12: in <module>
   with xr.open_rasterio(TEST_RASTER_PATH) as src:
../../anaconda3/lib/python3.6/site-packages/xarray/backends/rasterio_.py:117: in open_rasterio
   import rasterio
E   ModuleNotFoundError: No module named 'rasterio'
!!!!!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!!!
============================== 1 error in 2.96 seconds ==============================

I'll make rasterio truly optional, but I don't seem to be able to assign issues to myself.
Note: I couldn't install rasterio on OS X using conda - I had to use pip (https://github.com/ContinuumIO/anaconda-issues/issues/2428).

I couldn't install rasterio on OS X using conda

(Although the conda-forge version of rasterio worked.)


Last line includes CR LF (\r\n) if file ends in empty line

I am analyzing many CSV parsers in .NET and I have noticed that fastCSV includes a \r\n sequence in the last column of the last row if the file ends in an empty line. Many tools include the trailing line ending (both Windows and Linux) so this behavior may be unexpected.
For example,
a,b,c
d,e,f

The f field will contain a \r\n after the f.
All of these other CSV readers do not include the \r\n in the last field.

Repro: CsvRepro.zip

I can't reproduce the problem, do you have an example?

Here's a small repro:
using System;
using System.Collections.Generic;
using System.IO;
using Newtonsoft.Json;

namespace CsvReproSimple
{
    class Program
    {
        static void Main(string[] args)
        {
            var data = "a,b\r\n";

            using (var reader = new StringReader(data))
            {
                var nreco = new NReco.Csv.CsvReader(reader);
                var result = new List<List<string>>();
                while (nreco.Read())
                {
                    var record = new List<string>();
                    for (var i = 0; i < 2; i++)
                    {
                        record.Add(nreco[i]);
                    }
                    result.Add(record);
                }

                Console.WriteLine("NReco:");
                Console.WriteLine(JsonConvert.SerializeObject(result, Formatting.Indented));
            }

            Console.WriteLine();

            using (var reader = new StringReader(data))
            {
                var result = fastCSV.ReadStream<List<string>>(reader, 2, ',', (o, cols) =>
                {
                    for (var i = 0; i < 2; i++)
                    {
                        o.Add(cols[i]);
                    }
                    return true;
                });

                Console.WriteLine("fastCSV:");
                Console.WriteLine(JsonConvert.SerializeObject(result, Formatting.Indented));
            }
        }
    }
}

The output looks like this:

NReco:
[
  [
    "a",
    "b"
  ]
]

fastCSV:
[
  [
    "a",
    "b\r\n"
  ]
]

Should be fixed in v2.0.8

Awesome! I should be able to give it a try later today. I'm working on a blog post to compare performance of various CSV readers


Fix test stability

Make sure set the target branch to develop
What is the purpose of the change
Improve unit test stability
Brief changelog

org.apache.rocketmq.client.trace.DefaultMQConsumerWithOpenTracingTest#testPullMessage_WithTrace_Success has a race condition that when countDownLatch in callback executed, countDownLatch.await will pass, but finishSpan is not surely executed because it executed after callback, so tracer.finishedSpans will not have data if finishSpan is executed a little late after countDownLatch.await, and test will fail. Since there is no sutable assert point to make sure that finishSpan is executed, i add a waitAtMost to retry this assert for one second.
The init methods of DefaultMQPushConsumerTest & DefaultMQConsumerWithOpenTracingTest mock something after consumer.start, but consumer.start creates and starts some thread pool to execute, and the ThreadLocal mechanism of mockito will mess with them which creates a race condition, so mock method won't be effective sometimes. I refactor those codes to make mock before consumer.start to avoid those race conditions.

Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.2%) to 54.06% when pulling 0aaaeb60c377ac513860dc0789420e1d1628aedd on ayanamist:fix-test into 432236e983ede0d40b13b0dd0128932afd09df01 on apache:develop.

Codecov Report

Merging #2917 (0aaaeb6) into develop (432236e) will decrease coverage by 0.57%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #2917      +/-   ##
=============================================
- Coverage      48.32%   47.75%   -0.58%
- Complexity      3174     4522    +1348
=============================================
  Files            418      552     +134
  Lines          26391    36508   +10117
  Branches        3507     4830    +1323
=============================================
+ Hits           12753    17433    +4680
- Misses         11982    16851    +4869
- Partials        1656     2224     +568

Impacted Files
Coverage Δ
Complexity Δ

.../util/data/collect/impl/ListDataCollectorImpl.java
42.42% <ø> (ø)
6.00 <0.00> (?)

...org/apache/rocketmq/common/stats/StatsItemSet.java
43.28% <0.00%> (-7.47%)
17.00% <0.00%> (-2.00%)

...org/apache/rocketmq/store/ha/WaitNotifyObject.java
78.84% <0.00%> (-1.93%)
11.00% <0.00%> (-1.00%)

...ava/org/apache/rocketmq/filter/util/BitsArray.java
59.82% <0.00%> (ø)
30.00% <0.00%> (ø%)

...a/org/apache/rocketmq/store/StoreStatsService.java
29.50% <0.00%> (ø)
26.00% <0.00%> (ø%)

.../rocketmq/client/impl/ClientRemotingProcessor.java
2.91% <0.00%> (ø)
1.00% <0.00%> (?%)

...a/org/apache/rocketmq/client/impl/MQAdminImpl.java
5.14% <0.00%> (ø)
3.00% <0.00%> (?%)

...pl/consumer/ConsumeMessageConcurrentlyService.java
47.57% <0.00%> (ø)
12.00% <0.00%> (?%)

...pache/rocketmq/test/client/rmq/RMQSqlConsumer.java
80.00% <0.00%> (ø)
3.00% <0.00%> (?%)

...est/listener/rmq/concurrent/RMQNormalListener.java
96.15% <0.00%> (ø)
6.00% <0.00%> (?%)

... and 140 more

Continue to review full report at Codecov.

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


GetLinearTickersAsync() results in an error

Deserialize JsonSerializationException: Error converting value {null} to type 'System.Decimal'. Path 'result.list[0].predictedDeliveryPrice', line 1, position 492. [Data only available in Trace LogLevel]
error occurs when calling the GetLinearTickersAsync() function.

Checking..

Fixed. Plesase check https://github.com/burakoner/Bybit.Api/commit/86600864ece1012bba322e3a1b1e468998410614 and https://www.nuget.org/packages/Bybit.Api/1.1.1

{"symbol":"10000NFTUSDT","lastPrice":"0.003690","indexPrice":"0.003692","markPrice":"0.003692","prevPrice24h":"0.003665","price24hPcnt":"0.006821","highPrice24h":"0.003700","lowPrice24h":"0.003635","prevPrice1h":"0.003685","openInterest":"83657260","openInterestValue":"308862.60","turnover24h":"116621.9975","volume24h":"31803260.0000","fundingRate":"-0.000603","nextFundingTime":"1682784000000","predictedDeliveryPrice":"","basisRate":"","deliveryFeeRate":"","deliveryTime":"0","ask1Size":"1103300","bid1Price":"0.003685","ask1Price":"0.003695","bid1Size":"876700","basis":""}
same error occurs on 'basisRate'.
it looks like it needs to be revised 'basisRate', 'deliveryFeeRate', 'basis' section that has null value

Hi
I always do a deep inspection for all issues. When I look at Api docs for predictedDeliveryPrice, I released that documentation says "Predicated delivery price. It has value when 30 min before delivery". I missed this explanation while I was building this data model. But there is no extra information for other nulled properties.
The main problem is Bybit keep this values in string. So they can be empty as you wrote. I'm fixing them now. Despite I and other users tested too much, there will be issues like this.
Thank you for your feedback. Please check new version in 10 minutes.

@burakoner
Hi, I've tried it, but it seem there's still an error.
Deserialize JsonSerializationException: Error converting value {null} to type 'System.Decimal'. Path 'result.list[46].fundingRate', line 1, position 26081. [Data only available in Trace LogLevel]
You'd better try 'Market.GetLinearTickersAsync()' function on the entire symbol without parameters and check if it's work.

I've fixed it in my codes. There will be some extra updates according to bybit api updates. I'll release new version today. Thank you for feedback

Please check these out
https://github.com/burakoner/Bybit.Api/commit/f6a08048dfb69ff3f74b9b9592367fb7efb272dc
https://www.nuget.org/packages/Bybit.Api/1.1.3

Yes. Because it uses shared Order response model with Rest Api