MS Teams using "/" not "/teams/teams" for mention in channel

Hiiiiiii,  I've had some really great success in the past with BotKit and Slack.  I'm just trying to integrate with MS Teams, a little more tricky but not impossible.  I've reached a point where a 1:1 conversation with the bot is work ing find and my handlers kick in when you say the right stuff, great!
However, if I do an @ bot, instead of using the defined "/teams/receive" endpoint, it switches to using "/", now I've monkey patched a handler in for "/" to do the same as the teams/receive endpoint but It will just send a 200 back and hang for a while before saying there was a problem in teams.
Is this known/expected behaviour? Again really great job, it didn't take me too long to get this up and running the documentation will need an update soon because the registration method for a bot will currently expire in 3 months, I'll try and help with that if I get to it before anyone else.

There is a strong change this is a nuance with setting up a teams bot, it looks like when you direct add a bot to teams ( the zero pain method ), you end up restricting it to just 1:1 conversations, the pain and manifest version is what get's you team conversations, will report back shortly with learnings.

Two confirm, this was because the bot needed to be sideloaded, without sideloading you're restricted to 1:1 conversations and mentioning a bot in channel will yield the / root call I was seeing. RTFM ;)

@auberryberry thanks for figuring that out, we'll think about ways to make that more explicit in the docs.

No worries @peterswimm and thank you, maybe I’m a bit of a pioneer when it comes to teams and BotKit, I don’t suppose you saw my message in Slack?

@auberryberry I did not but feel free to @ mention me in a thread based on it if you need anything further!


Enhance AddressBook: Add CRUD support for company database

Added CRUD support for company database in addressbook.json.
Wrote and modified existing tests to support and test new functionalities effectively.
Implemented low-level functionality and methods (e.g. in JsonAdaptedCompany and JsonSerializableAddressBook) to support reading and writing of company data to/from Json format.
Updated the UI to be able to display list of companies.

For issue #90 .

NOTE:
Codecov checks did not pass due to lack of UI testing -> UI testing not required for tp.


Not available in Maven Central

Apparently, the library is not available on Maven Central. Neither under the coordinates
<dependency>
  <groupId>org.openize</groupId>
  <artifactId>drako</artifactId>
  <version>1.4.0</version>
</dependency>

that are mentioned in https://products.fileformat.dev/drako/java/ , nor under the coordinates
<dependency>
  <groupId>org.fileformat</groupId>
  <artifactId>drako</artifactId>
  <version>1.0</version>
</dependency>

that are shown here in the README.
Also, in order to be used by others, the role of the com/aspose/csporter/helpers has to be clarified. Both in terms of licensing, as well as in availability of the source code.

I have deployed to maven central repository today, and also fixed the dependency issue.


The MongoDB PECL extension has not been installed or enabled

I have the following error. My php version is 5.6, a x64 bit system and wamp. I downloaded the pecl driver "5.6 Thread Safe (TS) x64" and pasted inside ext folder in php and also edited the php.ini file.Restarted the local server. but still the error is coming.Anyone have any solution the error is "The MongoDB PECL extension has not been installed or enabled."

If you run phpinfo() from the same PHP process that reports the extension as missing, you should be able to confirm the INI file(s) used for that environment. Note that there may be different INI files for web and CLI environments. The PHP.net docs on configuration file and installing a PHP extension on Windows may have more information available to assist with troubleshooting.
If you're only seeing an error about the extension missing, it's likely that the appropriate INI file wasn't modified, or that extension= directive was parsed by the target file was not found. If PHP actually did load a DLL file and there was some incompatibility (e.g. using a TS DLL on a non-TS build of PHP) I believe you'd see a very different error message.

Hi, I am having the same issue. Getting "The MongoDB PECL extension has not been installed or enabled" error. I have corrected my php.ini, and can see the MongoBD version and details with phpinfo(). My php version is 5.6.24 and MongoDB dll version is 1.4.3. Can you please help me to resolve the issue.

I faced the same issue, and I placed the DLL name in php.ini,
php.ini-development, phpforapache.ini. in all these three files
On Mon, Apr 30, 2018 at 7:35 PM, Sajin Sabu<EMAIL_ADDRESS>wrote:

Hi, I am having the same issue. Getting "The MongoDB PECL extension has
not been installed or enabled" error. I have corrected my php.ini, and can
see the MongoBD version and details with phpinfo(). My php version is
5.6.24 and MongoDB dll version is 1.4.3. Can you please help me to resolve
the issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mongodb/mongo-php-library/issues/494#issuecomment-385436497,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AgWUC9eUZVHHpiLWj4trvdyzTTTWdBqzks5tty9WgaJpZM4SA6Ql
.


Picovoice Documentation Issue (/docs/api/porcupine-angular/)

Hi,
Can you include the process of conversion of .ppn file to Base64 and use in the angular application.
This can help us while creating a new custom keyword.
Thanks.

We recently added support for shipping Base64 directly out of the Picovoice console. Keyword models trained for WASM from now on will include a .txt file containing the Base64 string of the model, that can be then used in your application. We are currently in the process of updating the documentation and tutorials to reflect this feature.

We recently added support for shipping Base64 directly out of the Picovoice console. Keyword models trained for WASM from now on will include a .txt file containing the Base64 string of the model, that can be then used in your application. We are currently in the process of updating the documentation and tutorials to reflect this feature.

Thanks for the update @ErisMik .
So, if I am using using Windows(x86_64)  how to convert is my doubt. Sorry, I was not clear in before.

If your model's .zip includes the *_b64.txt file already, skip to step 2.

There are a couple ways to convert to base64 on Windows but the easiest is just an online tool like this. Train your model on the Picovoice console, download the .zip, extract it and use the online tool to convert the .ppn file into a base64 string.

Copy your base64 string from the website or the .txt file and follow the online docs here to use your base64 string in a PorcupineKeywordCustom object. Then use that object when calling init() on porcupineService.


Refactor HoodieCommitArchiveLog

This should archive all the instants into a archive timeline log (not just commits)

Remove HoodieCommitArchiveLog use the HoodieLogFormat to archive commits
Introduce Avro schemas for archived commits (not used in the active timeline)
Archive all instants - not just commits to the archivelog


Get AuthenticationState in view model

Question
Hi,
How can I get AuthenticationState in ViewModel? I'm trying with:
[CascadingParameter] Task AuthenticationState { get; set; }
but it is always null.
Thanks.
Code sample
No response
Version
6.0.6
Are you using Blazor WASM or Blazor Server?
Blazor WASM

Cascading parameters are supported. Could you create a minimal reproduction to show the exact issue? Keep in mind that you need to declare such parameters in both the component and the view model in order to make this work. Have a look at the Readme for some examples regarding parameters.

Hi,
Here is my component code:
`@attribute [Authorize]
@layout MainLayout
@using System.Collections.ObjectModel
@inherits MvvmComponentBase

    @if (@BindingContext.DataSource == null)
    {

    }
    else
    {

    }

@code{
[CascadingParameter] Task authenticationState { get; set; }
[Parameter] public ObservableCollection DataSource { get; set; }
}`
and here is my ViewModel code:
`public class TeamViewModel : ViewModelBase
{
private readonly ITeamService _teamService;
private readonly IJSRuntime _jsRuntime;
[CascadingParameter] Task<AuthenticationState> authenticationState { get; set; }
[Parameter] public ObservableCollection<User> DataSource { get; set; }
public MetaData Paging { get; set; }

public TeamViewModel(ITeamService teamService, IErrorHelper errorHelper, IJSRuntime jsRuntime)
{
    _teamService = teamService;
    _jsRuntime = jsRuntime;
}

public override async Task OnInitializedAsync()
{
    var a = authenticationState.Result;
    //var response = await _teamService.GetTeamAsync(new QueryParameters() { AccountId = 2 });
    //DataSource = new ObservableCollection<User>(response.Items);
    //Paging = response.MetaData;
}

}
`

Can you confirm that the property has a value in the component itself?

I have added initialization in my component, like this:

@attribute [Authorize]
@layout MainLayout
@using System.Collections.ObjectModel
@inherits MvvmComponentBase<TeamViewModel>

    @if (@BindingContext.DataSource == null)
    {
        <Placeholder />
    }
    else
    {
        <TeamMembers<EMAIL_ADDRESS>    }

@code{
    [CascadingParameter] Task<AuthenticationState> authenticationState { get; set; }
    [Parameter] public ObservableCollection<User> DataSource { get; set; }

    protected override void OnInitialized()
    {
        var auth = authenticationState.Result;
    }

}

However, now my BindingContext is null. I;m guessing that it is because I have OnInitialize in both component and ViewModel,. Do you have any idea how to resolve this.

If you override OnInitialized() in your component you need to call base.OnInitialized(). Otherwise the binding will be lost.

@bojankostic81 try changing the visibility to public (on both the component and view model), then I believe it should work.
[CascadingParameter] public Task<AuthenticationState> authenticationState { get; set; }

@hrannzo good catch, does parameter injection into components work without a public setter? If so, such a use case should be supported by this library.

@klemmchr It seems that a property can be private but it must have a setter defined when receiving a cascading value. The setter can be private and init. If the setter is missing then an exception is thrown from the belly of the Blazor.
CascadingComponent.razor
Name1=@Name1

Name2=@Name2

Name3=@Name3

Name4=@Name4

Name5=@Name5

@code {

    [CascadingParameter(Name = nameof(Name1))]
    public string? Name1 { get; init; }

    [CascadingParameter(Name = nameof(Name2))]
    public string? Name2 { get; private set; }

    [CascadingParameter(Name = nameof(Name3))]
    private string? Name3 { get; init; }

    [CascadingParameter(Name = nameof(Name4))]
    private string? Name4 { get; set; }

    // Attribute commented out since it would crash the app
    // Runtime error: Cannot provide a value for property 'Name' on type '...' because the property has no setter.
    //[CascadingParameter(Name = nameof(Name5))]
    public string? Name5 { get; }

    // Does not compile: BL0004 - Component parameter should be public
    // [Parameter] private string Name6 { get; set; }
}

WrapperComponent.razor
<CascadingValue Value="@("Hello")" Name="Name1">
    <CascadingValue Value="@("World")" Name="Name2">
        <CascadingValue Value="@("Foo")" Name="Name3">
            <CascadingValue Value="@("Bar")" Name="Name4">
                <CascadingValue Value="@("Crash")" Name="Name5">
                    <CascadingComponent></CascadingComponent>
                </CascadingValue>
            </CascadingValue>
        </CascadingValue>
    </CascadingValue>
</CascadingValue>

Rendered output:

@hrannzo thanks for the investigation, then this is an actual bug


Footer appears behind the tutorial

Describe the bug
Footer appears behind the tutorial
Steps to reproduce the behavior
1.Navigate to https://arcade.makecode.com/beta#
2.Tutorials----->Game Maker Guide
Actual behavior
Footer appears behind the tutorial.

Additional context
1.OS: Windows(rs6)
2.arcade version: 1.3.25
3.Microsoft MakeCode version: 6.8.11

should be fixed with https://github.com/microsoft/pxt/pull/7742, planning to do a patch release with the telemetry updates and this should also be visible then!


Cannot commit lepiter changes in case the lepiter database is in a nested directory

From discord:
a12l — Today at 11:17 AM
The problems seems to occur when the Lepiter notebook is located two subdirs below the git repo's root.
To reproduce:

$ mkdir git-bug-demo
$ cd git-bug-demo
$ git init
$ mkdir -p src foo/bar

Import the git repo git-bug-demo and select the src/ directory when creating the repo metadata.

Create an initial commit with the metadata. Close the git view.

Create a Lepiter DB inside foo/bar using the Lepiter view.

Open git view again and click on the git-bug-demo repository to view its changes.

Try to commit the Lepiter DB metadata. Note that a new commit is created, but the Lepiter DB metadata isn't committed.

Seems related to my report on Discord:

Although the next problem seems to be that changes in multiple DBs are not all committed via the Git Tool. I also tried committing a second time but the commit was empty. Anyone else experiencing this?


⚠ Bofrost.nl has degraded performance

In 346629f, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1084 ms

Resolved: Bofrost.nl performance has improved in a0eda15.


Add a dnn_mmod_ex_test file to easilty test a dnn

Thanks for another PR.
However, there is already an example program for this: http://dlib.net/dnn_mmod_face_detection_ex.cpp.html.  Also, this seems like more of a tool than an example.  Everything in the examples folder should be thought of as essays that introduce or explain some aspect of dlib useage.  So their primary goals need to be about being educational, easy to read, and so forth.  That is somewhat at odds with making useful programs in their own right, which in general belong in some other location like the tools subfolder, or more often in a different repository dedicated to a specific application rather than in the main dlib repo.


What it has been rendered on DAG UI seems different from the execution result.

Sumally
Is it supposed to have grp-2 > grp-2-main > grp-2-status on DAG UI as well after grp-1-main-status went successful?
since it’s an auto job workflow that grp-1 runs followed by grp-2 execution depends on the status.
Diagnosis

Argo Workflow Version: v3.1.5

When run the attached yaml (wf-for-issues.yml), 'grp-2' node did not appear in the workflow DAG UI.
grp-1 has two tasks: success and echo status.
grp-2 has two tasks: failure and echo status. *grp-2 has dependency with grp-1.

main.grp-1.grp-1-main was processed successfully.
main.grp-1.grp-1-main-status was skipped, as grp-1-main’s status was not Failed or Errored.
main.grp-2 should’ve been run automatically since it has dependency with the status of main.grp-1.
'grp-2' node did not appear in the workflow DAG UI. (Even though We get the echo "grp-2-main is Failed".)

wf-for-issues.yml
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: test-sample-

spec:

  entrypoint: main

  templates:

    # ----------------------------------------------------------------
    # containers

    # success
    - name: exit0
      container:
        image: alpine:latest
        imagePullPolicy: Always
        command: [sh, -c]
        args: ["echo 0; exit 0"]

    # error
    - name: exit1
      container:
        image: alpine:latest
        imagePullPolicy: Always
        command: [sh, -c]
        args: ["echo 1; exit 1"]

     # echo
    - name: echo
      inputs:
        parameters:
          - name: taskname
          - name: exitstatus
      container:
        image: alpine:latest
        imagePullPolicy: Always
        command: [sh, -c]
        args: ["echo {{inputs.parameters.taskname}} is {{inputs.parameters.exitstatus}}"]

    # ----------------------------------------------------------------
    # dag

    - name: main
      dag:
        tasks:

          - name: grp-1
            template: grp-1

          - name: grp-2
            template: grp-2
            depends: >-
              grp-1

    - name: grp-1
      dag:
        tasks:

          - name: grp-1-main
            template: exit0

          - name: grp-1-main-status
            template: echo
            arguments:
              parameters:
                - name: taskname
                  value: 'grp-1-main'
                - name: exitstatus
                  value: "{{tasks.grp-1-main.status}}"
            depends: >-
              grp-1-main.Failed || grp-1-main.Errored

    - name: grp-2
      dag:
        tasks:

          - name: grp-2-main
            template: exit1

          - name: grp-2-main-status
            template: echo
            arguments:
              parameters:
                - name: taskname
                  value: 'grp-2-main'
                - name: exitstatus
                  value: "{{tasks.grp-2-main.status}}"
            depends: >-
              grp-2-main.Failed || grp-2-main.Errored

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

@sarabala1979 do you think you could help this person?


test: remove difficulty from UbiquiStick tests

Right now tests in the development branch are failing with error:
Failing tests:
Encountered 3 failing tests in test/ubiquistick/UbiquiStick.t.sol:UbiquiStickTest
[FAIL. Reason: Difficulty is not supported after the Paris hard fork. Please use vm.prevrandao instead. For more information, please see https://eips.ethereum.org/EIPS/eip-4399] testRandom_ShouldReturnRandomValue() (gas: 3471)
[FAIL. Reason: Difficulty is not supported after the Paris hard fork. Please use vm.prevrandao instead. For more information, please see https://eips.ethereum.org/EIPS/eip-4399] testSafeMint_ShouldMintGoldToken() (gas: 9448422)
[FAIL. Reason: Difficulty is not supported after the Paris hard fork. Please use vm.prevrandao instead. For more information, please see https://eips.ethereum.org/EIPS/eip-4399] testTokenURI_ShouldReturnTokenURIForTokenTypeGold() (gas: 9472735)
Encountered a total of 3 failing tests, 219 tests succeeded

This PR removes deprecated vm.difficulty used by foundry

It appears that more lines were changed outside of the ones directly related to vm.difficulty but I trust that they were for good reasons.

Yes, when I removed it completely the default "new random value" (prevrandao) became equal to 0 so I had to refactor a few things in the tests like the number of a generated golden NFT or a generated random number.

@zgorizzo69 pls check this PR


feat: add interface showMark & marks

Coverage increased (+0.01%) to 69.438% when pulling f99b6a457f913e080b9614976a4b4b01a5041dbd on sanheng:dev into 10b5f56835e07497b02cfd65df9e5e4880b150d5 on ZhonganTechENG:dev.


test: bump coredns version to 1.7.0

coredns < 1.7.0 has a bug that makes the services resolution to become
out-of-sync with the last state from Kubernetes in case coredns
suffers from a disconnection with kube-apiserver [1]. This bug is fixed on
all versions equal and above 1.7.0. [2]
In our CI this affects all Kubernetes jobs 1.18 and below and can result
in flaky tests that have the result in the following similar logs:
service IP retrieved from DNS (<IP_ADDRESS>) does not match the IP for the service stored in Kubernetes (<IP_ADDRESS>)

[1] https://github.com/coredns/coredns/issues/3587
[2] https://github.com/coredns/coredns/pull/3924
Signed-off-by: André Martins<EMAIL_ADDRESS>Fixes https://github.com/cilium/cilium/issues/17401
Fixes https://github.com/cilium/cilium/issues/14598

/test

While we are here, why do we have coredns_deployment.yaml instead of letting kubeadm to choose the version and to install it? AFAIU such issues could have been avoided if the latter approach was chosen.

While we are here, why do we have coredns_deployment.yaml instead of letting kubeadm to choose the version and to install it? AFAIU such issues could have been avoided if the latter approach was chosen.

@brb We have our own deployment so that we can control the options used for our CI. We would have had the same issue given that coredns versions picked by kubeadm are the same ones we install [1]. Well, not anymore given that now we are bumping the coredns versions to fix flakes described in the PR description.
[1] https://github.com/coredns/deployment/blob/master/kubernetes/CoreDNS-k8s_version.md

test net next got a real failure, fixed and re-trigering CI

/test

Required tests are passing (except CI 3.0 which are not impacted by this change). The team review request is covered. Marking ready to merge.


Update MobiFlight Connector solution to .NET4.8

Is your feature request related to a problem? Please describe.
Update MobiFlight Connector solution to .NET4.8 to ensure support by Microsoft. The current .NET versions used by MobiFlight are not supported anymore.
Describe the solution you'd like
All projects in solution are .NET 4.8
Describe alternatives you've considered
Upgrade to .NET Core, but this is a long-term goal.
Additional context
https://devblogs.microsoft.com/dotnet/net-framework-4-5-2-4-6-4-6-1-will-reach-end-of-support-on-april-26-2022/

Also partially updated in #826


[Impeller] Investigate backdrop filter performance.

https://flutter-flutter-perf.skia.org/e/?begin=1678400491&end=1678463408&keys=X12c0192934b0ecc79bb42a39d31d4821&selected=commit%3D33736%26name%3D%252Carch%253Dintel%252Cbranch%253Dmaster%252Cconfig%253Ddefault%252Cdevice_type%253DiPhone_11%252Cdevice_version%253Dnone%252Chost_type%253Dmac%252Csub_result%253D90th_percentile_frame_rasterizer_time_millis%252Ctest%253Dbackdrop_filter_perf_ios__timeline_summary%252C&xbaroffset=33736

I think backdrop filter performance is explained by existing issues. This app creates a large number of backdrop filters so:

Inefficiency of subpasses due to blocking on GPU work:  https://github.com/flutter/flutter/issues/120399
Inefficiency of full screen texture fills due to lack of blits.
Potentially bad allocation patterns for offscreen MSAA: https://github.com/flutter/flutter/issues/120398

We could also be smarter about cases where there are multiple backdrop filters in the same scene, but I don' necessarily think that is going to be a very general optimization

A couple of other issue to add to the mix here:

https://github.com/flutter/flutter/issues/122657
https://github.com/flutter/flutter/issues/122658

This isn't a preview-exit blocker and all the more granular issues with this are understood and cross-referenced. Closing this.


Translation of the mod Harvest Festival 0.5.25 (Brazilian Portuguese)

Translation of the mod Harvest Festival 0.5.25 (Brazilian Portuguese)

@GirafiStudios already did this for ya


Update settings.yml

Description

Release required?

[x] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

When merging, use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author. The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).
Merge by using "Squash and merge".

Dup of https://github.com/terraform-ibm-modules/terraform-ibm-ocp-all-inclusive/pull/149


ARROW-14139: [IR] [C++] Table flatbuffer object fails to compile on older GCCs

Rename Table to Source
Sort RelationImpl variants

@cpcloud do we have a github actions workflow which validates the flatbuffer files and perhaps try to compile the generated source code?

@cpcloud do we have a github actions workflow which validates the flatbuffers files and perhaps tries to compile the generated source code?

@kszucs I do not think we have a separate workflow that does this to my knowledge.

It would be great to have one to at least compile the .fbs files so we can ensure that a PR like this is safe to merge.

Yep. Made a JIRA to track it: https://issues.apache.org/jira/browse/ARROW-14144


Fix PKCS12_create() so that a fetch error is not added to the error stack

Fixes #15392
PBE algorithms such as NID_pbe_WithSHA1And3_Key_TripleDES_CBC will
currently always fail to the EVP_CIPHER_fetch() call, so the fallback to
a legacy algorithm always happens. In this case the error stack should
ignore the fetch error.

Checklist

[ ] documentation is added or updated
[x] tests are added or updated

The check-ansi test failure is relevant.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Changed the pem string to der data for the test.

Merged to master as this was re-approved 24h ago already. Thank you.


Measures update

Description

In order to win even more time for starting the cubes, we could add the folder and the formatting of the measure within the map of Measures.update().
Context

Other information (if relevant)

Hi Alexis, this would indeed be nice but currently measure folders and formatters can only be changed on instances of the Measure class which you can only get once the measure has already been published to the cube. What you suggest requires keeping track of the folders and formatters on the Python side before measures are published. This would add some complexity so the speed up gained from doing this would have to be worth it. Do you have some perf measurements to share?

I just did a use case where I change the folder of 300 measures, the cube took 2min30 to do that.

Hello, what is the status on this issue ? It could be critical for some projects that needs to start as fast as possible and it is a huge waste of starting time.


Pull main into develop for timer hotfix

As per usual for Git Flow, pull main into develop to absorb hotfix.

Well nuts. Need to do this by hand again. 😦


Added Review Us Page. [Issue #18]

Added customer review page against issue no. #18 made the changes as requested by you in PR https://github.com/subhadeeproy3902/BloxAI/pull/57
Kindly check and merge this PR.
P.S would really appreciate if you can raise the issue level from 1 to 2/3.

Updates

[ ] Update the Carousal Speed (not readable at this speed)
[ ] Styling issues with the Review Form

Still have the scroll height
The form does not fit the screen
Suggestions:

It would be better if we could improve the design of the review page. In addition, we can also make customer testimonials in other designs
where the review form is not highlighted according to the UX. If we can make the form more appealing

@subhadeeproy3902 Can you please raise the issue level from 2 to 3 as I have added testimonials also in the contact page and you have assigned level 3 for that same issue in PR #114.

but #114 one UI is too good compared to this one !

but that one is based totally on my structure and in the PR 80 you have assigned level 3 for a form. Referring to that i have also testimonials in the review page with the  auto slider effect. @subhadeeproy3902


Add a direction to take_focus

Currently, views just "take" the focus, without any direction information.
This means a linear view will always start with the first element in focus, even if the selection comes from an element to the left/bottom (like a dialog button).
Adding a direction parameter to take_focus would allow views to change their behavior appropriately.

Done in 91fdf96066f211b9d275793de5419341c57e1ef8.


Fix header width

Header overflowed page at certain width. Changed css to make the header more responsive.

I tested this change, and it is an improvement.  Thanks!


I did some stuff

So uh I made CMP, BEV, and BOD cause why not. I did a little testing and they appear to work.

Must use the development branch for pull requests, not master.


No package code in sdist

Hi, there must be something that have gone awry as none of the package code are in the PyPI sdist:

Can we address this?

Which version of ipyslides you are using?
It will help me to check things on my side easily.

This is from the most recent version available on PyPI, 3.0.2: https://files.pythonhosted.org/packages/b8/37/61974068c2b2ed3b3b92a26615d296b09a0850fed32b92f87820a9b12f42/ipyslides-3.0.2.tar.gz

You are right, it somehow is not picking up code in tar.gz, see size of two files in screenshot below.
This needs to overwrite all releases with empty code and republish on PyPi. You will be able to get version 3.0.0 corrected today. I will leave a comment after resetting. This will still be ahead of 3.0.2, so no worries about losing things.

I just update version 3.0.3rc1, because unfortunately, we can no longer overwrite a PyPi package.

Thanks for pointing it out. Since I am using development install, I pushed many versions without noticing it.

Thanks for pointing it out. Since I am using development install, I pushed many versions without noticing it.

This is from the most recent version available on PyPI, 3.0.2: https://files.pythonhosted.org/packages/b8/37/61974068c2b2ed3b3b92a26615d296b09a0850fed32b92f87820a9b12f42/ipyslides-3.0.2.tar.gz

You can now safely use most recent and feature complete version 3.0.4.


Analytics for cosmoshub-4

Setup default API node
Setup indexer
Setup views for user behavior tracking

Add specific points for tracking:

Atom Transfer volume (in Atoms, not $)
Atom Number of transactions
Bonded Ratio (or justthe bonded balance and total supply)
Number of Active Accounts (wallets)
Number of delegators
Centralization: Top 10 validators have 40% voting power, e.g.

Setup text api with those metrics, ideally setup test api with support of query by "time-ago" parameter into request


Added support for using langchain serve endpoints

New LangChainServeLLMGenerator Class: Implements HTTP POST request handling to interact with LangChain Serve LLM APIs, allowing integration with Garak probes.

I have read the DCO Document and I hereby sign the DCO

recheck

Thanks for this! We'll get it reviewed

Implemented URI validation to ensure the API endpoint is correctly formatted before attempting to connect.
Replaced model_name usage with an environment variable (LANGCHAIN_SERVE_URI) for setting the API endpoint, let me know if other usage is preferred!
Error handling:

Logging for different types of HTTP errors (4xx and 5xx).
Changed return values from empty strings to None.

Removed unused config params i.e. temperature and penalties that are managed server-side.
Added tests covering URI validation, initialization, mocked generation, and error handling scenarios using requests_mock.


Select not working (Android)

Handling of 'touchstart' input event was delayed for<PHONE_NUMBER> ms due to main thread being busy. Consider marking event handler as 'passive' to make the page more responive.  Ionic.bundle.js:7507
When the select is touched, the above warning is given and the select never opens up with the options.  The main browser I used is Chrome.
What behavior are you expecting?
Touch the select and the options should appear to be selected.
Steps to reproduce:
To reproduce, open the chrome developer tools and toggle the device toolbar. Clicking on the select box will now not expand the options.
**Please see http://stackoverflow.com/questions/39642248/how-can-i-get-ionic-select-box-working-on-android-again/39774861#39774861
Which Ionic Version? 1.3.1
Plunker that shows an example of your issue
For Ionic 1 issues - https://plnkr.co/edit/8L4ERxWROkflF1aPAcxk?p=preview

Hello, thanks for opening an issue with us! I am going to be closing this one as a duplicate of https://github.com/driftyco/ionic/issues/8181. Thanks for using Ionic!


Remove deprecated deps, and unused imports, & deps

Description
Removes unused dependencies and imports and migrates deprecated apis of logger's verbose to trace and logger's date time format from printTime. Also updates the repository url to the current github url.
Testing (ignore for documentation update)
Not applicable for this change
Type of change

[x] Bug fix
[ ] New feature
[ ] Breaking change
[x] Documentation update

Checklist:

[x] I have read and agree to the Code of Conduct.
[x] I have signed the Contributor License Agreement (CLA).
[x] I have considered the Security Policy.
[x] I have self-reviewed and tested my code.
[x] I have updated documentation as needed.
[x] I agree to the project's custom license.

Additional Notes:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


personalizations / substitutions not working with simple text or html content on v3 api

Issue Summary
Hello, we would like to perform personalizations / substitutions using v3 api but without using a template, as described here
https://sendgrid.com/docs/for-developers/sending-email/personalizations/#sending-a-single-email-to-a-single-recipient-with-substitutions
technically it should be straightforward, because you can just allow replacement of the substitutions keys when they are found in the text or html email content, but currently that does not seem to work.
When sending through the v3 api an object like
{
  text: ' hello {{first_name}} ... ',
  "personalizations": [
    {
      "to": [
        {
          "email"<EMAIL_ADDRESS>        }
      ],
      "substitutions": {
        "{{first_name}}": "Johann"
      },
      "subject": "YOUR SUBJECT LINE GOES HERE"
    }
  ]
}

substitutions don't work when you provide the email content directly without using a template

@thomas-topway-it This is an issue in the Go version as well (trying to override subjects). It must be further up the chain in the SendGrid API.

Hi @thomas-topway-it are you using the sendgrid-nodejs library to make the request?

@JenniferMah
sure, we are using sendgrid-nodejs

You'll need to drop the wrapper characters in the substitutions keys. The library adds the handlebars automatically.
"substitutions": {
  "first_name": "Johann"
},

it works removing the brackets, thanks


INT E2E refactor round 2

Adding back the e2e refactor to support int.  this time run rp and e2e is still one stage with bash if to skip unneeded steps in int/prod cases. This is needed because the rp process was being killed after it's set ended so it wasn't running by the time the e2es were running.

/azp run e2e

/azp run e2e

/azp run e2e

/azp run e2e

Okay I've verified pr e2e and int e2es will work with this now. @julienstroheker @nilsanderselde can you take another look?

@jim-minter  Can I get a review here?


Add multi AND helper method

@rdblue updated the PR (forced pushed since the change is minor and not sure if you need to see the difference with previous commit).

Merged. Thanks for contributing this!


Add Java binding to get skins in FilamentAsset

Fixes #5290

@prideout Ok no problem. However before doing it, I want to clear what I have to change.
From:
public int getJointCountAt(int skinIndex);

To:
public int[] getJointCount;

Is this change what you want?

If you are going to return an array you need to accept an array as input. If the input is null, return a new array, otherwise use the one passed as a parameter. You also need to check the array's length and throw if too small.

The existing precedent in gltfio is to create arrays from scratch, under the assumption that queries are infrequent.  This unfortunately allows users to kill performance, so we should revisit it in the future.
For now, I would prefer that this class be self-consistent.
Further complicating matters is the fact that skinning joints are conceptually arrays-of-arrays, which I'd rather not deal with.
I propose the following, just two methods:
public @NonNull String[] getSkinNames() { ... }
public @NonNull @Entity int[] getJointsAt(int skinIndex) { ... }

While not ideal, it's at least similar to existing gltfio Java API's.

@prideout Thanks for your guidance. It's updated and please check it.

@prideout No worries and I fixed my huge mistake. Thanks for pointing out.


Support for decoding/verifying token from string

Are there any plans to implement a decode/verify function to get the claims of a JWT string?

Not at present. Decoding and verifying brings with it considerably more security concerns so it would need to be carefully implemented. It isn't something I have need of so I have not worked on this.


Don't run CI on PRs that don't need them

Travis runs our PRs serially which is part of the reason it takes so long to run them during the day. There are many PRs that don't need to run the full suite of integration changes. We should quit early when those occur. In the Travis hooks we'll need to detect the no-op change and short-circuit the rest of the execution.
We need to figure out the cases where the tests don't need to be run. One very possible candidate is docs/

... or we buy more Travis capacity.  I think @craigmcl is driving that.


GitHub throttling errors when repositories lists are large

This is not the issue with the terraform-github-team module.
This is the limitation of the module in regards to the current throttling GitHub policies.
The current rate limit documentation here.
If you want to terraform a small/medium size organization you have ~50 teams and ~500 related repositories.
Terraform allows you to register one PAT token per provider. If you want to avoid rate limits, you have to register multiple PATs.
Since the rate limit applies per user, not per PAT, those PATs have to be from different users.
If you know other ways to address the GitHub rate limit problem, I would appreciate it.

Hey Pavel,
Thanks for reaching out. We are usually splitting up the state in order to avoid long terraform runtimes as well as the mentioned rate limits. You may create several alias provider that you assign for different modules and resources - it will solve your rate limit problem but the execution time will be long no matter what. Does that make sense?


Open Etching

[ ] currently do not have open etching with fixed cap on Rune

Should we implement it

It isn't possible to implement this and have SPV proofs, because in order to know if a mint transaction is valid, you would have to download and check all transactions since the mint started, to know for certain that it hadn't already been minted out.


Correction in TSQL Code

Key name used in EncryptByKey is wrong in 1st TSQL code. It should be key_ReplDataShare instead of key_DataShare

Updated Code -
-- Execute at the Publisher on the publication database.
USE AdventureWorks2022;
GO
-- Create the database master key if it doesn't exist.
IF NOT EXISTS (SELECT * FROM sys.symmetric_keys
WHERE [name] LIKE '%DatabaseMasterKey%')
CREATE MASTER KEY ENCRYPTION BY PASSWORD = 'Pub$p@55w0Rd';
-- Create the cert_keyProtection certificate if it doesn't exist.
IF NOT EXISTS (SELECT * FROM sys.certificates
WHERE [name] = 'cert_keyPublisher')
CREATE CERTIFICATE [cert_keyPublisher]
WITH SUBJECT = 'Publisher Key Protection';
-- Create the key_ReplDataShare symmetric key if it doesn't exist.
IF NOT EXISTS (SELECT * FROM sys.symmetric_keys
WHERE [name] = 'key_ReplDataShare')
CREATE SYMMETRIC KEY [key_ReplDataShare] WITH
KEY_SOURCE = 'My key generation bits. This is a shared secret!',
ALGORITHM = AES_256,
IDENTITY_VALUE = 'Key Identity generation bits. Also a shared secret'
ENCRYPTION BY CERTIFICATE [cert_keyPublisher];
GO
-- Open the encryption key.
OPEN SYMMETRIC KEY [key_ReplDataShare]
DECRYPTION BY CERTIFICATE [cert_keyPublisher];
GO
-- Create a new CreditCardApprovalCode column in the SalesOrderHeader table.
ALTER TABLE Sales.SalesOrderHeader
ADD EncryptedCreditCardApprovalCode VARBINARY(256) NULL;
GO
-- Insert encrypted data from the CreditCardApprovalCode column.
 It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 44af891d-6aa8-9422-5048-80bed7d36035
Version Independent ID: 7898b8ef-2b39-4ac4-828c-48bf242a66a9
Content: Replicate encrypted columns (SSMS) - SQL Server
Content Source: docs/relational-databases/replication/security/replicate-data-in-encrypted-columns-sql-server-management-studio.md
Service: sql
Sub-service: replication
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

Thanks @Santro2511, we have an internal PR that will correct this issue, which will publish in the next day or so.


CLI Install: Provide snap package

Describe the enhancement you'd like to see
We should provide a way for users using snapd/snap to install kusk via the Snap Store.
Additional context
Easier for users to install kusk and potentially update it.

I believe that snap is not a relevant packaging as it has heavily declined in popularity due to its dependance on Canonical and the "unwelcome" way of installation.

we decided we wouldn't do snap packages.


Feature/substudy website consent

https://jira.movember.com/browse/TN-2732

Hello @achen2401! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file portal/models/coredata.py:

Line 447:1: E302 expected 2 blank lines, found 1

In the file portal/models/tou.py:

Line 10:1: E265 block comment should start with '# '
Line 10:80: E501 line too long (81 > 79 characters)

In the file portal/views/portal.py:

Line 602:13: E128 continuation line under-indented for visual indent


Submit queue is wildly optimistic

Right now it claims Estimated Merging 38238.829 PRs per day. If only!

It was launched in dry-run mode.


#185: Use discriminator fields only in codecs

Moves implicit discriminator mapping to Coproduct creation, instead of Codec creation. To avoid duplication.
Removes all discriminator params from model creation.
Sets discriminator param and value in codecs.

We have chosen not to make this functionality configurable, to avoid complexity.
This change should be released as a new major version, as it contains breaking changes.

Thank you, this is great work!

We have chosen not to make this functionality configurable, to avoid complexity.

:+1:

This change should be released as a new major version, as it contains breaking changes.

This project follows semantic versioning. According to the rules versions prior to 1.0.0 does not have to guarantee any compatibility (https://developerexperience.io/practices/semantic-versioning). As this is indeed a breaking change I would like to release it as 0.5.0, if that is ok for you. I would like to not to go with 1.0.0 before making the parser more robust.

I have no idea why the mill integration test is failing but it seems that this is some weird gh quirk. I tested it locally and it passes. It also passes on the CI, just on another branch:https://github.com/ghostbuster91/sttp-openapi-generator/commits/test-ci

I have no idea why the mill integration test is failing but it seems that this is some weird gh quirk. I tested it locally and it passes. It also passes on the CI, just on another branch:https://github.com/ghostbuster91/sttp-openapi-generator/commits/test-ci

It passed locally for me as well, and it is passing now, after my commit.

This change should be released as a new major version, as it contains breaking changes.

This project follows semantic versioning. According to the rules versions prior to 1.0.0 do not have to guarantee any compatibility (https://developerexperience.io/practices/semantic-versioning). As this is indeed a breaking change I would like to release it as 0.5.0, if that is ok for you. I would like to not to go with 1.0.0 before making the parser more robust.

That's great. I didn't know this about Semantic version.

I made a commit to your PR. After some small adjustments it seems that this case is also covered :) Unless, I misunderstood you.

The test for the allOf discriminator is exactly the case I had in mind. It's such a surprise to me that it works without changes, I hadn't even considered to just make the test and see if it runs.


Add select acc screen when user in All Account mode to show address

Is your feature request related to a problem? Please describe.
Add select acc screen when the user is in 'All Account' mode
Describe:

Step 1: When the user uses All Account mode click on the receive button
Step 2: Display the account selection screen: Design: https://www.figma.com/proto/4cYWHubrmdayui6VybR3BA/SubWallet-font-2?node-id=4051%3A13455&scaling=min-zoom&page-id=4%3A14
Step 3: After the user has finished selecting the account, the network selection screen is displayed
Step 4: After the user has selected the network, the screen of receiving will be displayed: QR code, Address: https://www.figma.com/proto/4cYWHubrmdayui6VybR3BA/SubWallet-font-2?node-id=158%3A1709&scaling=min-zoom&page-id=4%3A14

[ ] Update UI follow design:

[ ] Add tooltip when hover to ? icon

[ ] Bug UI

Search box size is changed when entering search keywords (see file below)

Improve UI:

[ ] Add the x icon button to clear text for the search box (both Network Selection & Account Selection popup)

[ ] Update height-weight for popup (both Network Selection & Account Selection popup)

@S2kael Update UI

Validation passed!


issues with “status:ready-for-qa” label

rc-test-template.md currently refers to “status:ready-for-QA” label in multiple places.  The label is actually “status:ready-for-qa” (lowercase).   Since we have multiple “QA:*” labels, perhaps the label should be changed, so that we're using uppercase everywhere. If not, then the template should be changed to refer to “status:ready-for-qa”.

This is inconsistency is not a big deal for GitHub searches, because they are apparently case insensitive.  E.g. "is:issue label:status:ready-for-QA" and "is:issue label:status:ready-for-qa" will find the same issues.

Since it is already lowercase, it should be fine to stick with that.  I'll adjust the templates/book.  Thanks for pointing out.

Done in above commit.  Closing


guerrillamail duplicates a dot character when the dot appears at index 76

Example:
Send email from anywhere to guerrillamail with the following text:
123456789012345678901234567890123456789012345678901234567890123456789012345.com
Result in Show original:
123456789012345678901234567890123456789012345678901234567890123456789012345=
..com
guerrillamail duplicates the dot.
This causes links to be broken.
Good luck :)

Thanks. Fixed the code now, will deploy on Guerrilla Mail soon.


Enable running unit-tests in GitHub workflow

Trying to enable running unit-tests in GitHub workflow.

Windows is hitting #1872 issue.
@halx99 Can you help me in setting up running on macOS? I can't figure out how to insert arch for the command line here: https://github.com/smilediver/axmol/blob/ade1eba39c25f3d1a3909019db4fb9cdaf3067ba/.github/workflows/build.yml#L124
And I also have no idea why Linux fails. I don't know if the path to exe is wrong or something else...

sure

The linux will crash due to initialize FileUtilsLinux too early before entering main entrypoint

does the doctest support run tests in main function?

Thanks for getting this to work!


optimize covid_datasette

Optimize the covid_datasette example to have better "mechanical sympathy" with volumefs.

Unpack report files in temp dir
Preload report files
Update db in temp dir

Type of Change

[ ] New example
[x] Example updates (Bug fixes, new features, etc.)
[ ] Other (changes to the codebase, but not to examples)

Checklist

[x] Example is testable in synthetic monitoring system, or lambda-test: false is added to example frontmatter (---)

[x] Example is tested by executing with modal run or an alternative cmd is provided in the example frontmatter (e.g. cmd: ["modal", "deploy"])

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-1556d00.modal.run

nice! looks like it now runs in <5 minutes, rather than over 10!


[Backport 1.x] Update log4jVersion to v2.20.0

Backport 8f884a435c7f0518e8337edca3d7586c63d5d3d3 from #478.

Codecov Report

Merging #480 (e8c26e9) into 1.x (592da7a) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##                1.x     #480   +/-   ##
=========================================
  Coverage     65.92%   65.92%
  Complexity      186      186
=========================================
  Files            35       35
  Lines           804      804
  Branches         24       24
=========================================
  Hits            530      530
  Misses          263      263
  Partials         11       11

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


get access to TemplateResolver

Hi Remi,
I think this might be out of date now, as I have merged a similar PR. Can you check?

ok


Crash when loading into world - Only Hexcraft and dependencies installed

Hello. I am working on a large modpack and Hexcraft loaded just fine. I was trying to create a resourcepack to prevent the book from spawning on player load and to avoid waiting for my modpack each time to load I decided to just load Hexcraft and it's dependencies. I just hit install on Hexcraft on Curseforge and it installed everything for me. I tried loading a world and it crashed.
Here is the crashlog:
crash-2022-07-18_10.49.38-client.txt
crash-2022-07-18_10.49.35-server.txt
Mods and versions:
Cardinal Components v4.20
Fabric API 0.57.0
Fabric Language Kotlin v1.7.0
Hexcasting v0.9.1
Patchouli v71.1
PAUCAL v0.4.7
The only thing I can think of for the reasoning it works in my modpack and not alone is another mod in my modpack has a embedded Cardinal Components that is overwriting this one. However I am not good with mods or coding so I have no clue. Any help is appreciated. Thank you.

In the short term, Cardinal Components 4.1.4 works for most things, excluding Edify Sapling.  Cardinal Components 4.2.0 doesn't seem to have any relevant breaking changes, but the current Fabric/gradle.properties doesn't support it.

Am able to reproduce
crash-2022-07-30_07.27.59-client.txt


Windows/Resource.rc: use user-provided version as "nice" fileversion

There are two instances of FileVersion (resp. ProductVersion): a numerical one
and a literal one. If the user forced a version string, use it as-is as the
literal one.

[ ]   General Comment
 @tcahuzax @krocard please review
 ## [Current coverage][1] is 78.80%

Merging #348 into master will not affect coverage as of [decb454][3]

@@            master   #348   diff @@
=====================================
Files          210    210
Stmts         7159   7159
Branches         0      0
Methods          0      0
=====================================
Hit           5642   5642
Partial          0      0
Missed        1517   1517

Review entire [Coverage Diff][4] as of [decb454][3]
[1]: https://codecov.io/github/01org/parameter-framework?ref=decb454e1358232d70197203d01f14ede4f40577
[2]: https://codecov.io/github/01org/parameter-framework/features/suggestions?ref=decb454e1358232d70197203d01f14ede4f40577
[3]: https://codecov.io/github/01org/parameter-framework/commit/decb454e1358232d70197203d01f14ede4f40577
[4]: https://codecov.io/github/01org/parameter-framework/compare/8ca3cad1af2c24d79ca8de22a15274c85341fc27...decb454e1358232d70197203d01f14ede4f40577
Powered by Codecov. Updated on successful CI builds.

 :+1:
 :-1:
 @krocard please re-review
[ ]   File: SetVersion.cmake:L33-40
 Why in the cache ?
[ ]   File: SetVersion.cmake:L72-80
 idem on the next line, set the cache variable
 Maybe you want to move this test in the if(NOT DEFINED PF_VERSION) block.
[x]   File: SetVersion.cmake:L56-65
 Set the CACHE variable, do not create a new one with the same name !
 see https://cmake.org/cmake/help/v3.0/command/set.html

This pull request is now managed with CodeReviewHub. Refresh.

@tcahuzax @krocard please review

Current coverage is 78.80%

Merging #348 into master will not affect coverage as of decb454

@@            master   #348   diff @@
=====================================
  Files          210    210
  Stmts         7159   7159
  Branches         0      0
  Methods          0      0
=====================================
  Hit           5642   5642
  Partial          0      0
  Missed        1517   1517

Review entire Coverage Diff as of decb454

Powered by Codecov. Updated on successful CI builds.

:+1:

:-1:

@krocard please re-review

:shipit:


Fix Git Pinned Version

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind documentation

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area crawler

/area ci

/area utils

What this PR does / why we need it:
Simply removing the pinned git version, which no longer exists in the base image as python's slim image updated to Debian bookworm.
Which issue(s) this PR fixes:

Fixes # N/A
Special notes for your reviewer:

LGTM label has been added.  Git tree hash: 9fc173c45dcecb9c0ef5ddad9f10012bdf5c57ca

New changes are detected. LGTM label has been removed.

LGTM label has been added.  Git tree hash: 2349ad415feeea848a5842915296f3a25ac3d825


ipcache: Plumb daemon context through IPCache

Plumb the primary daemon context into the label injector logic and the
async prefix releaser in order to reduce the likelihood of issues where
this logic continues to execute beyond the lifetime of the agent.
Found during unit testing, where the identity allocator would be closed
while the ipcache's garbage collection of identities was still running.
Technically this does not fully close the gap since it only initiates
the shutdown but does not confirm that it has completed. Currently, only
test code relies on this working but in future as we modularize the code
we may introduce more formal ordering for this in order to fully ensure
the correct ordering.
Suggested-by: Jussi Maki<EMAIL_ADDRESS>

/test

The arm64 test failure in Travis CI is fixed in https://github.com/cilium/cilium/pull/21681, so can be ignored here.

/test

/test

EKS cluster creation failed, causing ConformanceEKS and Conformance AWS-CNI workflows to fail.

/ci-eks

/ci-awscni


Use normalised counts as input

Hi Pascal,
The readme mentions that the input requires a table of "integer UMI counts".
I'm wondering whether normalised UMI counts, e.g. from Sanity, could be used as well. Is this supported? Or can I transform normalised values back to integers for your method to work?
Best,
Dominik

Hi Dominik,
The statistical model behind cellstates is based on the expected noise in UMI counts. If you use a non-integer input, it will simply round the values to the nearest integer. But since the statistics of the normalised data is different, it will not give the right results.
If you want to use cellstates together with sanity, I would suggest to run cellstates first on the unnormalised data, sum up the UMI counts for each cluster and then run sanity on this. I think Jérémie has done this before, so maybe check with him.
Cheers,
Pascal

Hm okay, thank you! I will ask him.


Language Options Not Accessible on Header

https://github.com/user-attachments/assets/5b44d1f7-64f8-4d2e-a99e-8e0a512c2834
The website's header currently displays three language options, but the site content is only showing in English. The language selection feature is not functional.
Proposed Solution: I can implement the necessary functionality to make the language options accessible and ensure that the page content can be viewed in all available languages. This will enhance the user experience and provide a seamless multilingual interface.
Please assign this issue to me under the Hacktoberfest topic so I can work on it. Thank you!

Thanks for creating the issue in WildGuard! The issue has been assigned to you.
Before you start working on your PR, please make sure to:

⭐ Star the repository if you haven't already.
Pull the latest changes to avoid any merge conflicts.
Attach before & after screenshots in your PR for clarity.
Include the issue number in your PR description for better tracking.

Don't forget to follow @anuragverma108 – Project Admin & @PriyaGhosal -Project Collaborator  – for more updates!
We're excited to see your contribution as part of Hacktoberfest 2024! 🎉
Happy open-source contributing!


Negations in pseudocodes

How to negate a condition in pseudocode? By using plain text "not", as in Multipop? Or rather comparing it to \const{false} as in the solution to 10.4-3? Once decided, apply consistently in other places.

The negations in the textbook seem to use the "not" word. Included this rule in CONVENTIONS.md.


feat: the upstreamDecl linter

This linter is intended to work much like minImports in helping with splitting files. It places a warning on each set of declarations that could be moved to an earlier file. It takes into account dependencies on declarations in the current file, so that a whole block of declarations can be moved even if there are dependencies between the declarations in the block.
Since definitions are often deliberately placed in their own file, this linter does not place a warning on any declaration depending on a definition in the current file.

maintainer delegate

I have done some cleanup and I think this is good to go!

One more comment: the linter is called upstreamableDecl, but the PR title uses upstreamDecl.  I may actually have a slight preference for upstreamDecl as the name of the linter as well.

@Vierkantor, could you address @adomani's remaining suggestions, and then merge?

bors d+

I think that Kim meant
bors delegate=Vierkantor
😊

bors merge


[Request] Regex + widcard

This plugin is amazing and definitely gives more flexibility than menu entry swapper. I would love to see it have the ability to do regex in the swaps though,
For example to match several different jewellery items at once like the ring of dueling and wealth
rub,^.*(wealth|duel).*$
Replacing two entries of
rub,*wealth*
and
rub,*duel*

I don't find this example compelling. It looks harder to write, and most people don't know regex anyways.
I would use regular MES to swap rub on that item. it's easier to do imo (just shift-right-click).


Improve envtool diagnostics

Closes #342.

Where is a part with logging git verison, go version, docker version?

I've added go version in the first line but forgot about others. Current changes fixing it.
Right now diagnostic output would look like:
Looks like something went wrong..
Please file an issue with all that information below:

        OS: windows
        Arch: amd64
        Version: v0.1.0-21-g4b5d84c-dirty
        Commit: 4b5d84c03e805130b8b75f04086d735d57f202bf
        Branch: issue-342-improve-envtool-diagnostics

        Go version: go1.18
        git version 2.32.0.windows.2
        Docker Compose version v2.3.3
        git version 2.32.0.windows.2

        Error: failed to connect to <IP_ADDRESS>:5432


Create Maximize the sum of selected numbers from an array to make it …

…empty
Given an array of N numbers, we need to maximize the sum of selected numbers. At each step, you need to select a number Ai, delete one occurrence of it and delete all occurrences of Ai-1 and Ai+1 (if they exist) in the array. Repeat these steps until the array gets empty. The problem is to maximize the sum of the selected numbers.
Note: We have to delete all the occurrences of Ai+1 and Ai-1 elements if they are present in the array and not Ai+1 and Ai-1.

Hi Dear @deepak7336
Great code
Thanks for your contribution......
Show your support on my youtube channel web Pixar(The institute of coding)
like , share , subscribe
youtube - https://youtu.be/5qnQjgdpFPg
telegram-channel - https://t.me/WebPixar
join our telegram channel for more goodies , swag packs


Question: Integrating a custom framework

I run a keyword based automation framework that uses selenium. The framework executable runs json scripts that translate keywords into selenium commands. The framework is run as an executable. I would like to add the framework to my docker images. Two questions: 1)  can Selenoid launch my executable when Selenoid runs, and 2) is there a way my framework can communicate with Selenoid to inform it when a test starts and stops (e.g. API) so the Selenoid UI can display running tests? Programmatically setting capabilities as well?

@offdesk Selenoid is just an API to launch browsers. It is not even aware you are launching tests. So everything outside of manipulating browsers is out of Selenoid scope. What you are describing is test automation report \ CI responsibility.


Update BIN 413426

This BIN changed and it is now a DEBIT card.
Fixes #14

Coverage remained the same at 98.684% when pulling 49893a107b80b89cd5a836e3cb7b69c655f98471 on luizkowalski:luizkowalski-patch-bin-mbank into 69d6db73407a04dc89a78cf9fa0d8a87d2aae161 on hugolantaume:master.


Greenkeeper/npm check updates 3.0.5

Pull Request Test Coverage Report for Build 240

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

Totals

Change from base Build 239:
0.0%

Covered Lines:
68

Relevant Lines:
68

💛  - Coveralls


Vector -> Sequence

I think the build won't work as this needs hs-geojson as a peer ... not sure how to do this in circleCI.


Type resolution fails when importing iti with "moduleResolution": "NodeNext"

Problem Description
When using iti in a TypeScript project with "moduleResolution": "NodeNext" in your tsconfig.json, type information for iti cannot be located.
This line:
import { createContainer } from 'iti'
Produces the error:
Could not find a declaration file for module 'iti'. '<project path>/node_modules/iti/dist/iti.modern.js'
implicitly has an 'any' type.
  Try `npm i --save-dev @types/iti` if it exists or add a new declaration (.d.ts) file
  containing `declare module 'iti';`

The project will still compile and run with tsc; you just don't have type information for iti for static checking, linting, etc.
Cause
There are two causes:

The exports map in iti's package.json contains an exports map, but the types key is not defined for each export. See this TypeScript issue for a complete explanation. When exports is defined, top-level types or typings keys are ignored.

The export ... from statements in iti/src/index.ts need to use an explicit .js extension. Without this, import { createContainer } from 'iti' will nominally succeed, but the type of createContainer will just be import, and the static analysis gets very confused.

Proposed fix

The first issue is simple: just add "types": "./dist/src/index.d.ts" to the exports map in package.json.

For the second issue, I'll be honest: I'm fairly new to JS/TS. My understanding is that explicit .js extensions are required for import/export statements when using ESM ("type": "module" in package.json). So I'm not sure how iti is getting away with "bare" imports in the first place. But in any case, adding extensions only to the export statements in src/index.ts is enough to satisfy the consumer's linter and should be backwards-compatible with CJS. With that said, two of the iti tests (dispose....) try to import { createContainer } from '../src', which needs to be ../src/iti after this change. The other tests already import this way.

I have a fork with these proposed changes made to iti and iti-react. I can make a PR if you'd like.

Oh great! Thank you for opening this up! Will check it and merge into the upstream 🙇♂

Also, this is some great description, I've learned something new :)

@molszanski thoughts on getting this fix added?

@danielmahon oh!
Yeah, forgot about it! Will release a minor with better cjs/mjs compatibility!

Should work with 0.7.0 version


Disconnecting the user

Hello! Tell me please, when the user is disconnected, is the message sent to the script? Or you need to constantly listen to the processes?

Script gets it's STDIN closed and  after some time (100ms + closetime) SIGINT will be sent... If that is caught by script it has 160ms + closetime to terminate or websocketd would follow up with SIGTERM... if that did not help after additional 500 ms + closetime before it would be killed.
SIGINT and SIGTERM is likely noop in Windows. But i'm not sure. Kill uses cross-platform function so it should be working properly.
closetime set by --closems flag in milliseconds
You can catch SIGINT in your script to perform some cleanup tasks. Or, if you passing input to some other process it'll be possible to catch process completion due to lack of STDIN to start cleanup

Thank you


[BEAM-9364] Refactor KafkaIO to use DeserializerProviders

This builds on your previous PR with some API improvements. Since this adjusts the public API that will be out as part of 2.20.0 is probably better if we include it as part of 2.20.0 too.
R: @aromanenko-dev

Run Java KafkaIO Performance Test

Weird it did not fail, but it is referrencing and incorrect (older) run. See here for the actual run
https://builds.apache.org/job/beam_PerformanceTests_Kafka_IO/452/

Run Java KafkaIO Performance Test

Fixes done PTAL @aromanenko-dev

Run Java PreCommit

Run Java KafkaIO Performance Test

Thanks !


fix: sidekick no longer opening in dev

Fix #193

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


readonly

Hi Nashio,
Just to let you know, I had to change a line of your code in the 'addListeners:' function from 'if( this.settings.readOnly ){ return; }' to 'if( this.settings.readonly ){ return; }'.  readonly with lower cap o is defined in your default definition.
I appreciate you taking care of this.  I like your software.

hi @binskeep , thanks for pointing that out, my typo was actually in the settings section.


Add option to apply discount codes on tickets

Describe the feature you'd like
For paid events, we can a show list of discount codes listed by the organizer and from there he/she can then apply the discount accordingly
Would you like to work on the issue?

[x] Yes

@iamareebjamal should i go ahead?

That's not the point of discount codes AFAIK. And I'm also not completely sure about that

I've checked it on the website and backend it is meant for that only purpose

The point being, if I show all the discount coupon, everyone will obviously use them. Sometimes they are hidden and awarded

Coupons number can be limited and also many app does this these days zomato,paytm etc everyone does this

duplicate


Not available on atom

FYI, https://atom.io/packages/language-suricata is a 404.
It would be nice if this was available for Atom
Thanks!

I moved on to vscode before I could finish this package.
Feel free to fork and finish :)

there is now https://github.com/dgenzer/suricata-highlight-atom


Support passing version number to Version VSIX task

The Version VSIX task currently expects a build number from which it parses the version. This might be useful in some cases, but in my pipe I determine the version through GitVersion and have the version number already available in a variable.
Therefore it would be nice if the version, as it will be written to the VSIX manifest, can be passed to the task.

Not sure of that is a different task for could be handled by allowing an override version  number passed in  from a build variable?

Not sure if I understand correctly. You suggest a different task which allows to pass in the version number instead of build number?
IMHO it should be not a problem to add a second input which for the version, additional to the current input of the build number. User could then use either or.

This was fixed with https://github.com/rfennell/vNextBuild/commit/1ce0150e3b599af0a161d2f94a2c7983fdae22ff


Documentation

Describe the bug
The documentation is not working.
To Reproduce

Goto: https://lifter-api-wrapper.readthedocs.io/en/latest/index.html#

Expected behavior
Docs to work.

This has been moved to https://weightliftingnz/github.io/lifter-api-wrapper

Note that mkdocs is used along with mkdocstrings to autogenerate docs from the docstrings of the python function.


fix: account name checks FE-724

Closes https://github.com/FuelLabs/fuels-wallet/issues/1263

Fixes:

User not being able to change the account name to case-sensitive variations of the same word.

Example: Changing a case typo, like Fuel wAllet -> Fuel Wallet was not possible

User not being able to change or create accounts that are substring of eachother.

Example: Changing Fuel Wallet -> Wallet was not possible

Considerations:

The filterByName function sound like it could be used for a search filter, in which it should be case insensitive, but that is not the case at the moment, it is only used for account name editing and creation.
Users can now create multiple accounts with similar names, which can cause confusion, changing only a letter case, so:

Account 1 and account 1 can now coexist as account names

it's missing to add a changeset @nelitow
pnpm changeset


Implement an internal filter list

There are several types of concepts we might want to filter:

Inappropriate concepts, such as:

"Suicide": https://github.com/NCATSTranslator/Feedback/issues/289
"sadomasochism": https://github.com/NCATSTranslator/Feedback/issues/292
"fetishism": https://github.com/NCATSTranslator/Feedback/issues/293
"rape": https://github.com/NCATSTranslator/Feedback/issues/306
"nymphomania": https://github.com/NCATSTranslator/Feedback/issues/314

Overly general concepts, such as:

"Adult disease": https://github.com/NCATSTranslator/Feedback/issues/290

Incorrectly modeled concepts, such as:

"Completed suicide" (yes, it is a disease, but it's not curable): https://github.com/NCATSTranslator/Feedback/issues/291

There are also several places we could implement filtering:

In Babel: UMLS:C0277562 should not be included in any cliques in Node Normalization or Name Resolution. This would prevent "adult disease" from showing up in autocomplete or in Translator results.
In Name Resolution: UMLS:C0277562 could be included in NodeNorm, but should not be included in NameRes' Solr index. "Adult disease" could still show up in Translator results, but won't show up in autocomplete.
In UI: calling NameRes will still return UMLS:C0277562, but the UI will filter it out when displaying autocomplete results.

This has now been implemented at https://github.com/NCATSTranslator/Blocklists and in translator-devops (PR https://github.com/helxplatform/translator-devops/pull/709). Closing.


Feature Request: add observed- keyword to groupby

Hi, dask developers! Thank you for this very nice package, I have been using extensively for a while.
I am missing Pandas keyword observed for groupby with categorical variables. Without it, the memory used by my processes is huge, as groupby by default creates all possible tuples of categories, no matter whether they are found in the dataset.
Minimal example of the expected behavior:
import dask.dataframe as dd
import pandas as pd

df = pd.DataFrame({
    'c1': pd.Categorical(['v11', 'v12', 'v11'], ['v11', 'v12', 'v13']),
    'c2': pd.Categorical(['v21', 'v22', 'v23']),
    'c3': [1, 4, 7]})

ddf = dd.from_pandas(df, npartitions=1)
ddf.groupby(['c1', 'c2'], observed=True).sum().compute()

Desired output:

c1
c2
c3

v11
v21
1

v11
v23
7

v12
v22
4

I recently tried to implement observed support in my Dask branch here: https://github.com/OliverHofkens/dask/tree/feature/groupby-observed-categoricals
This wasn't too much work, just storing observed and passing it as argument to the GroupBy._aca_agg functions got a lot of it working. However, some test cases in this parametrized test case failed: https://github.com/OliverHofkens/dask/blob/b9feeb254e18c88ca9d5ab34143049d4a53e6d85/dask/dataframe/tests/test_groupby.py#L2291-L2323
Not because of Dask, but because of a wide variety of categorical/groupby related bugs in the underlying Pandas code. Mainly:

Cases where Pandas coerces categoricals to object when setting it as an index.
Cases where Pandas drops categorical columns when aggregating with observed=False. (Here, Dask actually does the right thing, but the test fails because we compare with Pandas behavior)

Once those issues are fixed in Pandas, I will gladly submit a PR with observed support.

Any news on the development of observed?

Doesn't look like it. All the discussion happens here on github.
Are you interested in working on this @MatteoSchiavinato?

Perhaps @OliverHofkens can share if any of the underlying issues in pandas have been fixed.

@TomAugspurger I don't think I have the necessary knowledge! I solved my error with a workaround so for now I don't need it.

Hey, thanks for reminding me about this! I fast-forwarded my branch and the test still fail, but on something different than before.
Once I find some time I'll see if we can finish this feature!

Bumping also, thank you!

Bumping as well, this would be an extremely helpful feature.

Just an FYI, the pandas behavior is potentially changing, which would make this more important to implement: https://github.com/pandas-dev/pandas/pull/35967
In the future pandas will require this keyword when grouping by one or more categoricals.

I can take on wrapping up the work that @OliverHofkens started. I'll open a PR shortly.


[ENH] Array graph compatibility with causal-learn

Fixes #12 for causal-learn
Changes proposed in this pull request:

Before submitting

[ ] I've read and followed all steps in the Making a pull request
section of the CONTRIBUTING docs.
[ ] I've updated or added any relevant docstrings following the syntax described in the
Writing docstrings section of the CONTRIBUTING docs.
[ ] If this PR fixes a bug, I've added a test that will fail without my fix.
[ ] If this PR adds a new feature, I've added tests that sufficiently cover my new functionality.

After submitting

[ ] All GitHub Actions jobs for my pull request have passed.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@1c0e864). Click here to learn what that means.
The diff coverage is 63.95%.

@@           Coverage Diff           @@
##             main      #16   +/-   ##
=======================================
  Coverage        ?   67.28%
=======================================
  Files           ?       13
  Lines           ?      752
  Branches        ?      195
=======================================
  Hits            ?      506
  Misses          ?      202
  Partials        ?       44

Impacted Files
Coverage Δ

pywhy_graphs/classes/admg.py
82.50% <ø> (ø)

pywhy_graphs/array/export.py
61.64% <61.64%> (ø)

pywhy_graphs/classes/functions.py
66.66% <66.66%> (ø)

pywhy_graphs/config.py
88.00% <90.90%> (ø)

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


Logger class is not serializable

Description
Since 2.21.0 serializing a Logger raises java.io.NotSerializableException.
Maybe this is caused by this commit (see my comment on commit): https://github.com/apache/logging-log4j2/commit/e86d28bdb927f8047b924eb459f4c63bd2e43176
Configuration
Version: 2.21.0
Operating system: Windows-10
JDK: OpenJDK 1.8.0_382
Logs
Exception in thread "main" java.io.NotSerializableException: java.lang.ThreadLocal
	at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1184)
	at java.io.ObjectOutputStream.defaultWriteFields(ObjectOutputStream.java:1548)
	at java.io.ObjectOutputStream.writeSerialData(ObjectOutputStream.java:1509)
	at java.io.ObjectOutputStream.writeOrdinaryObject(ObjectOutputStream.java:1432)
	at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1178)
	at java.io.ObjectOutputStream.defaultWriteFields(ObjectOutputStream.java:1548)
	at java.io.ObjectOutputStream.writeSerialData(ObjectOutputStream.java:1509)
	at java.io.ObjectOutputStream.writeOrdinaryObject(ObjectOutputStream.java:1432)
	at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1178)
	at java.io.ObjectOutputStream.writeObject(ObjectOutputStream.java:348)
	at test.TestSerializeLogger.main(TestSerializeLogger.java:14)

Reproduction
package test;

import java.io.ByteArrayOutputStream;
import java.io.IOException;
import java.io.ObjectOutputStream;
import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;

@SuppressWarnings("javadoc")
public class TestSerializeLogger {
	public static void main(final String... args) throws IOException {
		try (ObjectOutputStream objOut = new ObjectOutputStream(new ByteArrayOutputStream())) {
			Logger logger = LogManager.getLogger("Test");
			objOut.writeObject(logger);
		}
	}
}

Alright, I've converted that test into a JUnit one here:
    @Test
    void canSerializeRoundTrip() throws IOException {
        final ReusableMessageFactory factory = new ReusableMessageFactory();
        final ByteArrayOutputStream bytes = new ByteArrayOutputStream();
        try (final ObjectOutputStream out = new ObjectOutputStream(bytes)) {
            out.writeObject(factory);
        }
        try (final ObjectInputStream in = new ObjectInputStream(new ByteArrayInputStream(bytes.toByteArray()))) {
            assertDoesNotThrow(in::readObject);
        }
    }

Should be simple enough to fix!

@makkimesser, could you share some details about your use case, please? Why do you serialize Loggers?

@makkimesser, could you share some details about your use case, please? Why do you serialize Loggers?

I stumbled over the issue while running our test-suite, when testing V2.21.0 and I got the mentioned NotSerializableException. There are some tests with classes containing loggers, if the logger-field is static or transient, there is no problem, but if the logger-field is non-static and non-transient the serialization method failed in V2.21.0 while it did not fail with V2.20.0.
We have such non-static and non-transient logger-fields in abstract parent classes to avoid to declare the logger in each of the child-classes. I'm not sure about the real reason of this behaviour, the code base is created before I joined my current job.

Thanks so much for sharing these details, much appreciated.

... non-transient logger-fields in abstract parent classes to avoid to declare the logger in each of the child-classes.

non-transient fields – be it of type Logger or something similar – unrelated with DTO they are situated in are an accident waiting to happen. I would strongly advise you to refactor your code rather than relying on the fact that Log4j types are serializable, which won't hold anymore in Log4j 3. Is such a refactoring possible in your case? If not, would you mind sharing details?

@makkimesser,
The usual reason for using instance Logger fields on abstract classes, instead of static fields is that each subclass wants to use its own name in log messages (using the %c pattern, which means "logger name").
This was usually an ersatz for what developers really want: the %C pattern, which provides the class name that generated a log statement. Traditionally the %C pattern is computationally expensive, but using static weaving provided by the Log4j Transform Maven plugin you can remove the performance penalty. The plugin might be rough around the edges, but some committers already use in production.

The Apache Wicket Framework does serialize Pages constantly, if these Pages contain non-static Loggers (which happens sometimes) the application throws an Exception since reusablemessagefactory violates the serializable contract.

@renoth,
Can you check the 2.22.0-SNAPSHOT we published on https://repository.apache.org/snapshots? Does it solve your problem?


[Internal] Query: Adds Query Advice Request and Response capabilities

Pull Request Template
Description
This PR adds the capability to request and receive Query Advice in QueryRequests. The response from BE is an URL-encoded JSON string that will get decoded to UTF8 text.
To request query advice, use the new QueryRequestOpton.PopulateQueryAdvice
Once this is set to true, see the advice via FeedResponse.QueryAdvice property
Note: the test is marked Ignore because the emulator used in pipeline does not support Query Advisor yet. Furthermore, this request option is internal and not yet public to wait for the deployment
Type of change
Please delete options that are not relevant.

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

Closing issues
To automatically close an issue: closes #IssueNumber

    /// PopulateIndexMetrics is used to obtain the index metrics to understand how the query engine used existing indexes

...
 ...

Refers to: Microsoft.Azure.Cosmos/src/RequestOptions/QueryRequestOptions.cs:109 in 9b053c7. [](commit_id = 9b053c71128cccbe1a2025e9082b6b4795db51bc, deletion_comment = False)


No longer able to work with VARIANT

Summary
The 0.53.0 update provided some "improved" handling for VARIANT structs.
This change removed the ability to inspect the details of the VARIANT, making some of the use-cases I am using impossible.
I realize that you want a rust-safe way to deal with things in this crate, but I need way more control that what I am given. I need to be able to create variants of arbitrary type, from their underlying data representations.
I used to be able to do this.
While I think it's great that you guys are working on improving the rust binding support...  I don't think that the C bindings should be hidden.
Instead of using VARIANT everywhere in the rust crate... why don't you make a RustVariant type instead that the new apis use, so that it's still possible to do lower-level stuff.
Crate manifest
No response
Crate code
No response

One example of the fallout here is: #2877.

I think all that you would need to do is make a
inner(self) -> imp::VARIANT;

impl for VARIANT so that I could continue my work.
Or implement From<VARIANT> for imp::VARIANT and the reverse.

If you go this route, I would need to understand what to do for making sure memory is safe.
I was looking through imp::VARIANT* and it looks like the ManuallyDrops are gone?

Let's keep the conversation in one place. Closing as duplicate of #2877.

@kennykerr I would ask that you re-open this. Supporting more types is not really the problem this issue is addressing.  Unless you want to add imp::VARIANT conversion to VARIANT in the other issue.

I'd rather just discuss all issues/requests for VARIANT related support in one place.


ARROW-11026 [Rust]: Run tests without requiring environment variables

This PR is based on @mqy 's great work in: https://github.com/apache/arrow/pull/8967.
(If we want to take this PR, we can either merge it in to https://github.com/apache/arrow/pull/8967/files# directly or I can make a new independent PR when that is merged).
Rationale
The outcome is that developers can now simply run  cargo test in a typical checkout without having to mess with environment variables. I think this will lower the barrier to entry for people to contribute.
Changes

Code from https://github.com/apache/arrow/pull/8967 to encode heuristics of where to check for test data
Remove all references to ARROW_TEST_DATA and PARQUET_TEST_DATA and uses the test_util methods instead
Update the comments / error messages in test_util

Example Error Handling
Error handling: here is what happens with a fresh checkout and no git modules checked out  and no environment variables set:
cargo test -p arrow
---- ipc::reader::tests::read_decimal_be_file_should_panic stdout ----
thread 'ipc::reader::tests::read_decimal_be_file_should_panic' panicked at 'failed to get arrow data dir: env `ARROW_TEST_DATA` is undefined or has empty value, and the pre-defined data dir `/private/tmp/arrow/rust/arrow/../../testing/data` not found
HINT: try running `git submodule update --init`', arrow/src/util/test_util.rs:81:21

Here is an example of what happens when ARROW_TEST_DATA is pointing somewhere non existent
ARROW_TEST_DATA=blargh cargo test -p arrow
...
--- ipc::reader::tests::read_decimal_be_file_should_panic stdout ----
thread 'ipc::reader::tests::read_decimal_be_file_should_panic' panicked at 'failed to get arrow data dir: the data dir `blargh` defined by env ARROW_TEST_DATA not found', arrow/src/util/test_util.rs:81:21

Codecov Report

Merging #8996 (195e743) into master (1ecef42) will increase coverage by 0.00%.
The diff coverage is 96.15%.

@@           Coverage Diff           @@
##           master    #8996   +/-   ##
=======================================
  Coverage   82.65%   82.66%
=======================================
  Files         200      200
  Lines       49795    49790    -5
=======================================
  Hits        41159    41159
+ Misses       8636     8631    -5

Impacted Files
Coverage Δ

rust/arrow/src/util/test_util.rs
90.90% <ø> (ø)

rust/datafusion/examples/flight_server.rs
0.00% <0.00%> (ø)

rust/datafusion/src/datasource/csv.rs
81.25% <ø> (ø)

rust/arrow/src/ipc/reader.rs
82.86% <100.00%> (ø)

rust/arrow/src/ipc/writer.rs
83.25% <100.00%> (ø)

rust/datafusion/src/datasource/parquet.rs
96.92% <100.00%> (ø)

rust/datafusion/src/execution/dataframe_impl.rs
96.26% <100.00%> (ø)

rust/datafusion/src/physical_plan/csv.rs
82.78% <100.00%> (ø)

rust/datafusion/src/physical_plan/parquet.rs
80.46% <100.00%> (+0.15%)
:arrow_up:

rust/datafusion/src/physical_plan/planner.rs
80.45% <100.00%> (ø)

... and 4 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 ca685a0...195e743. Read the comment docs.

@alamb those two envs also stay in rust/arrow/README.md
How about remove them from rust/arrow/README.md, or link user to to rust/README.md ?

How about remove them from rust/arrow/README.md, or link user to to rust/README.md ?

Good idea @mqy  -- done.

@nevi-me  -- I have not been able to find a satisfactory way to avoid using the common parquet_test_data function from the arrow crate in the parquet crate. Since they are only used in tests, I think this is ok -- however, if you would like a different organization please let me know and I can do it as a follow on PR

Hey @alamb, it's fine, we can merge the changes as they are. Thanks for trying to move the function.


Humanoids do not have clients therefore they cannot "view" inventories or "interact" with things

The reason for this change is for clarity in the API. To say that a humanoid "views" an inventory or an entity can "interact" with a container is misleading: these are properties of a client connection. I am not saying that Humanoids or other entities should not have inventories, I have no qualms about that.
Pinging @Mumfrey
Signed-off-by: Walker Crouse<EMAIL_ADDRESS>

@windy1 No, we can use viewing inventory as a simulation.

@liach There is literally no use cases for that if you really think about it. What is "viewing" an inventory really? It's opening a GUI and maybe rearranging some items. Well non-player entities certainly can't do that on their own accord so if you want to "simulate" it then you just access their inventory directly. Humans can hold inventories, yes, but they cannot "view" them in the same sense as players which is why they should not share this method.

Somehow I suggest that Humanoids could occupy an inventory in some cases, such as trading with villagers which only allows one player to trade with at one time. At least you should retain Interactable's generic, what if other entities could interact with inventories like Villagers?

By the way how doesn't travis ci build this pull request?

@liach I had not yet opened my common PR. Look, all of which you described is achievable without an abstract notion of an open inventory for humanoids. Non-player human entities are no different from any other entity in MC so by your logic we should just add these open inventory methods to the base Entity class.

There's several places in the API that seem to refer to humanoids and players interchangeably yet they are distinctly different, mainly in the inventory API.
For example, HumanInventory should really be player inventory as currently humanoids only have the default 5 entity inventory slots (armor and hand), and a CraftingInventory doesn't make sense for a humanoid.

I'm going to go ahead and merge this now. @Mumfrey let me know if you have any objections to this.

In my opinion, you should let an owner to merge such pull requests in the future, such as @Zidane or @gabizou.

@liach
I don't see an issue with windy merging this PR here and we have no rules in place that state a core dev cannot do so. He made it a PR to generate discussion in-which was done but ultimately his changes are exactly what needed to happen.
You and the others above are confusing Inventory and Container from the official codebase. Mods don't interact with an inventory via Container, they literally just set its contents (normally without doing any sort of verification checking). Players are the only ones who go through the Container logic and that is represented correctly here.

I might add that there's a clear difference between interacting with an inventory where packets are sent down the wire to interact with the inventory (i.e. clients interacting with the inventory while connected to the server), and interacting with the inventory directly on the server side (which is what all mods/plugins should be doing regardless).


What's the position of Metadata.CKB_RELATED_INFO_KEY?

Question
I want to get the proof of Metadata.CKB_RELATED_INFO_KEY through eth_getProof.
What's the position (or storage keys) of it?
https://github.com/axonweb3/axon/blob/3dbfe5a515c8c46eec434d4584acec13b8d69649/core/executor/src/system_contract/metadata/store.rs#L24
https://github.com/axonweb3/axon/blob/835cb164fae28f3d200fc2a0eb790bfedd426049/core/api/README.md?plain=1#L1081-L1087
https://github.com/axonweb3/axon/blob/835cb164fae28f3d200fc2a0eb790bfedd426049/core/api/README.md?plain=1#L1910-L1917
In Further
If I'm incorrect, please point out.
I have only read some documents from the internet.
In my understand, the positions are defined as following:
contract Demo {
    uint     a = 1;
    uint128  b = 2;
    uint32   c = 3;

    /* ... omit ... */
}

storage(position = 0) is 0x0000000000000000000000000000000000000000000000000000000000000001
                                                                                ^
storage(position = 1) is 0x0000000000000000000000000000000300000000000000000000000000000002
                                                          ^                               ^

Also, the position will be the key in the storage.
Since Axon system contracts use self-defined keys for all state variables, how could users to get those keys?
If one data has more than 256 bits, it will have more than 1 position.
If data type is array or map, it's more complex.
But each data for Axon system contracts has only one key, how to process these situations?
References

#1561
#1457

Duplicated with #1561


Listeners on Video not being removed in time for next Scene

When next Scene loads, historical time data is still present in the system while the Video component is being unmounted

Fixed in 0.6.6


[Question] How to get output embeddings or last hidden states?

Question
Could someone explain how to integrate a simple linear layer on top of the LLaVA model for a classification task, similar to what's implemented in LlamaForSequenceClassification? Specifically, is there a method to access the last hidden states or output embeddings that can be used as input for a classifier?
I don’t want to fine-tune the LLaVA model because my dataset is quite small. Instead, I have a classification task, and it would be more effective to take the LLaVA model inference, feed the embeddings or last hidden states into a linear layer, and train it. However, I’m not sure how to implement this. Could someone please guide me?

I have the same issue. Have you solve this problem?


[Monster of the Week] "The Divine" playbook - The divine armor is missing in the translation json entries.

The "divine-gear6" json entry is missing from the json files.

I have fixed it in my fork but will wait until next week before making a pull request as I'm also adding the playbooks from Tome of Mysteries.


RPC response error -32002: Transaction simulation failed: Error processing Instruction 0: account data too small for instruction

I'm just starting out with Solana and Anchor development.
Currently following the tutorial at this address and I'm hitting a snag.
I'm using the latest beta solana 1.18.5 and anchor 0.30.0.
Steps to reproduce:

anchor init test
cd test
anchor build -> "package solana-program v1.18.11 cannot be built"
cargo update -p<EMAIL_ADDRESS>--precise 1.18.5 (annoying, but ok, only needs to be done once...)
anchor build
anchor keys sync
Start solana test validator in another shell solana-test-validator
anchor test --skip-local-validator -> Successful
Modify lib.rs to add a parameter to the initialize function, and mirror the change in the test file
anchor test --skip-local-validator -> Error below

Error: Deploying program failed: RPC response error -32002: Transaction simulation failed: Error processing Instruction 0: account data too small for instruction [3 log messages]
There was a problem deploying: Output { status: ExitStatus(unix_wait_status(256)), stdout: "", stderr: "" }.

Steps I tried to do to resolve (right after the last command)

anchor keys sync (NOTE: this does NOT change the program id in the code)
anchor test --skip-local-validator -> Same error
Stop and restart solana-test-validator
anchor test --skip-local-validator -> Same error

What gives? I'm super confused and it's a really frustrating experience.
I noticed that doing cargo clean then building again makes the error go away, but that's such bad DX I can't see myself doing that every time I make a change to my program!
NOTE: after cargo clean, the next call to anchor keys sync reports: "Found incorrect program id declaration in Anchor.toml for the program test"
This would indicate a cache invalidation problem with anchor keys sync.

package `solana-program v1.18.11` cannot be built

This error is related to your Solana version. With Anchor 0.30.0, it's recommended to use 1.18.8 as noted in release notes:
solana-install init 1.18.8

The first anchor keys sync step is redundant, as anchor init command already initializes programs with correct program ids. It should only output:
$ anchor keys sync
All program id declarations are synced.

Error: Deploying program failed: RPC response error -32002: Transaction simulation failed: Error processing Instruction 0: account data too small for instruction [3 log messages]
There was a problem deploying: Output { status: ExitStatus(unix_wait_status(256)), stdout: "", stderr: "" }.

This error happens because in Solana 1.18, Solana's deploy command doesn't allocate enough space for future program upgrades. When you added that extra parameter to the initialize function, program size increases, which makes deploying it to the same program account impossible without resizing. This should ideally be done automatically, but apperently the technology is not that advanced yet. For now, you can manually run solana program extend command.
Running anchor test (without --skip-local-validator flag) would not run into this problem, as each deployment is a new one. You can also run anchor test --detach if you'd like to keep the test-validator running after the tests.

Steps I tried to do to resolve (right after the last command)

anchor keys sync (NOTE: this does NOT change the program id in the code)
anchor test --skip-local-validator -> Same error
Stop and restart solana-test-validator
anchor test --skip-local-validator -> Same error

What gives? I'm super confused and it's a really frustrating experience.
I noticed that doing cargo clean then building again makes the error go away, but that's such bad DX I can't see myself doing that every time I make a change to my program! NOTE: after cargo clean, the next call to anchor keys sync reports: "Found incorrect program id declaration in Anchor.toml for the program test"
This would indicate a cache invalidation problem with anchor keys sync.

All this is irrelevant to the actual problem. You can run anchor clean instead of cargo clean if you'd like to keep the program keypair file, which would result in consistent program ids.

Thanks for the detailed answer. I'm surprised that changing the size of the program by modifying the code does not trigger an update of the program ID via anchor keys sync, if the existing program account is not able to hold the program data. Is there a reason this could not be done?
Another option, like you mentioned, could be to automatically call solana program extend on behalf of the user. I believe this would massively improve the DX when using anchor.

Another option, like you mentioned, could be to automatically call solana program extend on behalf of the user. I believe this would massively improve the DX when using anchor.

This was added to Solana CLI in https://github.com/anza-xyz/agave/pull/791. It will also work for Anchor since we use Solana CLI internally to deploy.

Are there any downsides to removing --skip-local-validator when running tests? The tutorial I was following suggested it in their day1 course.

It depends on what you mean by downside. Including that option means you'll keep the existing state and only redeploy the program. This can be interpreted as both a downside and an upside. On the other hand, starting fresh each time is a more common pattern.


chore: deprecate Python 3.6 support

Python 3.6 has reached end-of-life at the end of December 2021, and new
pip releases no longer support it.
Check for python versions at load of the jsii runtime for Python, and
emit a warning if Python < 3.7 is detected. Additionally, change the
pip build-tool requirement to be pip ~= 22.0 if Python >= 3.7 is
used.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Should we not be using docs.python.org/3/library/warnings.html ?

I did not know about that :)