[FEAT][FB123] Add missing line to main method in GithubCiCdApplication

Type
enhancement

Description

Improved readability in the GithubCiCdApplication class by adding a missing line break in the main method.

Changes walkthrough
Relevant filesEnhancement

GithubCiCdApplication.javaImproved Readability in `main` Method

src/main/java/fr/benseddik/planning/GithubCiCdApplication.java
Added a missing line break in the main method for better readability.

+1/-0

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

PR Description updated to latest commit (https://github.com/Benseddik-Fethi/Github-CI-CD/commit/8a920c99ad4ac53186c24c7a4267833e3a8102c6)

PR Review

      PR feedback                       ⏱ Estimated effort to review [1-5]
1, because the PR consists of a single, straightforward change: adding a line break for improved readability in the main method of the GithubCiCdApplication.java file. The simplicity of the change and the context make it easy to review.

 🧪 Relevant tests
No

 🔍 Possible issues
No

 🔒 Security concerns
No

 Code feedback:

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review. The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.
When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:
/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:
[pr_reviewer]
some_config1=...
some_config2=...

  Utilizing extra instructions
The review tool can be configured with extra instructions, which can be used to guide the model to a feedback tailored to the needs of your project.
Be specific, clear, and concise in the instructions. With extra instructions, you are the prompter. Specify the relevant sub-tool, and the relevant aspects of the PR that you want to emphasize.
Examples for extra instructions:
[pr_reviewer] # /review #
extra_instructions="""
In the 'possible issues' section, emphasize the following:
- Does the code logic cover relevant edge cases?
- Is the code logic clear and easy to understand?
- Is the code logic efficient?
...
"""

Use triple quotes to write multi-line instructions. Use bullet points to make the instructions more readable.

  How to enable\disable automation

When you first install PR-Agent app, the default mode for the review tool is:

pr_commands = ["/review", ...]

meaning the review tool will run automatically on every PR, with the default configuration.
Edit this field to enable/disable the tool, or to change the used configurations

  Auto-labels
The review tool can auto-generate two specific types of labels for a PR:

a possible security issue label, that detects possible security issues (enable_review_labels_security flag)
a Review effort [1-5]: x label, where x is the estimated effort to review the PR (enable_review_labels_effort flag)

  Extra sub-tools
The review tool provides a collection of possible feedbacks about a PR.
It is recommended to review the possible options, and choose the ones relevant for your use case.
Some of the feature that are disabled by default are quite useful, and should be considered for enabling. For example:
require_score_review, require_soc2_ticket, and more.

  Auto-approve PRs
By invoking:
/review auto_approve

The tool will automatically approve the PR, and add a comment with the approval.
To ensure safety, the auto-approval feature is disabled by default. To enable auto-approval, you need to actively set in a pre-defined configuration file the following:
[pr_reviewer]
enable_auto_approval = true

(this specific flag cannot be set with a command line argument, only in the configuration file, committed to the repository)
You can also enable auto-approval only if the PR meets certain requirements, such as that the estimated_review_effort is equal or below a certain threshold, by adjusting the flag:
[pr_reviewer]
maximal_review_effort = 5

  More PR-Agent commands

To invoke the PR-Agent, add a comment using one of the following commands:

/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs 💎: Generate docstring for new components introduced in the PR.
/generate_labels 💎: Generate labels for the PR based on the PR's contents.
/analyze 💎: Automatically analyzes the PR, and presents changes walkthrough for each component.

See the tools guide for more details.
To list the possible configuration parameters, add a /config comment.

See the review usage page for a comprehensive guide on using this tool.

Generating PR code suggestions
Work in progress ...


Feature - EKS - Separate ASGs per subnet (for AZ aware auto-scaling)

What kind of request is this (question/bug/enhancement/feature request):
Feature - EKS - Separate ASGs per subnet (for AZ aware auto-scaling)
Steps to reproduce (least amount of steps as possible):
Create an EKS cluster using multiple worker subnets
Result:
SUCCESS: Cluster is created and the CloudFormation stack uses separate ASGs (one per subnet) - this allows you to use the auto-scaling feature and be aware of multiple AZs when scaling (the previous behaviour is one single ASG which means you have no control over the AZ that's expanded)
FAIL: Cluster does not work
Other details that may be helpful:
https://github.com/rancher/kontainer-engine/pull/200/files#diff-c18744328cdc32c74aa39b295238b3fcR517

@dstockton Can you confirm that this is optional and doesn't change the default behavior today?
Should there be a related terraform provider change along with this?

Yes and yes there is https://github.com/terraform-providers/terraform-provider-rancher2/pull/185#discussion_r349024936


thrift: bidirectional and encoder filter config validation

Signed-off-by: kuochunghsu<EMAIL_ADDRESS>Commit Message:
More test coverage for bidirectional and encoder filter config.
Additional Description:
Risk Level: low
Testing: unit
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/assign-from envoyproxy/first-pass-reviewers

/assign @zuercher
as maintainer/codeowner


Issue with .exists? for ruby 3.2

Ruby 3.2 no longer supports .exists?
Please add me as a contributor and I will submit a pull request to fix this. thanks!

I was able to run yadr on Fedora 39 by installing ruby 3.1 with rvm.
I wanted to write here in case anyone gets an error and can't figure out how to fix it

Having the same issue, how can I help?


Ajaxfileupload control error raising upload complete event and start new upload

HI, I'm facing issue with AjaxFileupload control in VS2013 when hosted in IIS(local) server. But it is working fine while I am running with IIS Express. Please help me out.
Screen shot for reference.

Do you run the site in Medium trust environment or on a web farm?

Hi Thanks for your reply. I am running this application in  my local IIS (My system) with the trust level as "Full". still i am not able to use the AjaxFileupload Control. Please Suggest.

What error do you see in a browser's console (F12)?
Are server-side events (UploadStart, UploadComplete and UploadCompleteAll) raised correctly?

Hi, I have written UploadComplete event in server side.but it is not executing or pointing there to complete the upload process. Seeking your assistance

Would you please comment out this line in the Layout.master file
<asp:ScriptReference Path="~/Scripts/AjaxControlToolkit/Bundle" />
and upload an error screenshot in the browser's console, like this one?

I have commented the line in layout.master and tried to upload again., but it is showing same error.In browser console it is showing that."Error raising upload complete event and start new upload".  Please find the screen shots for your reference.

Yes, I am struck at the same point in the above post. could you please help how to overcome that challenge? Thanks in advance.

Would you please show network data in the browser's developer tools (see the screenshot)?

Hi Please find the attached..

The 404 error indicates that client code cannot find the AjaxFileUploadHandler.axd upload handler.
It seems that the Sample site is not located in the root of the web site.
You can try to place it in the root of the web site or change the path to AjaxFileUploadHandler.axd in Web.config.
Please see this comment for more details: https://github.com/DevExpress/AjaxControlToolkit/issues/43#issuecomment-203889967

Hi Thanks for your swift help... Its working fine.


Add health check make command

I accidentally force pushed to #30 and broke the diff so I've duplicated the code here.
I'll sort out the tests because I know the previous MR was having trouble so we can get this merged this week.

Closes #28

I've tested this on PHP 7.4 in a Laravel 5.5 app and it all works as expected.

@miff2000 I'll ask someone to review this in the morning and then I'll merge it in.

Tagged at v1.10.0
https://github.com/ukfast/laravel-health-check/releases/tag/v1.10.0


Allow usage of filter blocks for Scope resource

This may not be currently possible, but I could not figure out how to create more advanced queries/filters for scopes. If this is possible, could someone help with an example?
e.g. Creating the following query for a scope resource in Terraform.
✻ resource_type = workspace or OS Version contains MSWindows10

@frazhamid Did you manage to fix this issue? So, far I can only use the following.
short_query_type    = "eq"
short_query_field   = "ip"
short_query_value   = "<IP_ADDRESS>"


Timothygwebb patch 2

@fordnox I have decided to refactor these at a later date I have had an enlightenment.


[WIP] hypre runtime compute policy

todo:

[ ] guard more with #if MFEM_HYPRE_VERSION >= 22600

@tomstitt, do you think this will be ready in time for mfem-4.6?

@tomstitt, do you think this will be ready in time for mfem-4.6?

Should be, I'm almost done testing locally. Next I need to cleanup what I have a little and fix backward compatibility and go through the examples/tests/miniapps and handle the HYPRE_USING_GPU blocks they have

@tomstitt, is this ready for review?

@tomstitt, is this ready for review?

No.. not yet. It's taking me longer to get full backwards compatibility than expected. We also need https://github.com/hypre-space/hypre/tree/gpu_init, which isn't merged yet but I've reached out to try to get an ETA.

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

Update: This functionality is now in a Hypre release version (2.31, after https://github.com/hypre-space/hypre/pull/1068).

Now that PR https://github.com/mfem/mfem/pull/3922 has been merged in master, please update this PR by:

merging master in here and resolving conflicts
replacing use of double with real_t
guarding double/single-only sections with MFEM_USE_DOUBLE and MFEM_USE_SINGLE
making sure tests run cleanly with default and MFEM_PRECISION=single configurations

See https://github.com/orgs/mfem/discussions/4207 for more details and/or ask @dylan-copeland for help.

@v-dobrev and @sebastiangrimberg -- This has been updated so the examples do not require any changes. Can you take another look?
@tomstitt -- Can you add an item about this in CHANGELOG?

@v-dobrev and @sebastiangrimberg -- This has been updated so the examples do not require any changes. Can you take another look?

I think this approach looks great. I haven't tested it on a GPU build, but can next week if needed. The changes look good to me to merge as is.

@tomstitt -- Can you add an item about this in CHANGELOG?

Done!

Thanks @tomstitt !

Sorry for the delay here, I just got around to building and testing with CUDA support. I think this looks good, I just have one suggested style change to make the unit test consistent with the examples in their initialization of initialize Hypre, followed by the mfem::Device (this doesn't change functionality by the design):
diff --git a/tests/unit/pcunit_test_main.cpp b/tests/unit/pcunit_test_main.cpp
index e45f495d8..0ba491bf9 100644
--- a/tests/unit/pcunit_test_main.cpp
+++ b/tests/unit/pcunit_test_main.cpp
@@ -26,11 +26,11 @@ int main(int argc, char *argv[])
    return MFEM_SKIP_RETURN_VALUE;
 #endif

-   mfem::Device device("cuda");
 #ifdef MFEM_USE_MPI
    mfem::Mpi::Init();
    mfem::Hypre::Init();
 #endif
+   mfem::Device device("cuda");

    // Include only tests that are labeled with both CUDA and Parallel.
    return RunCatchSession(argc, argv, {"[CUDA]","[Parallel]"}, Root());

and
diff --git a/tests/unit/cunit_test_main.cpp b/tests/unit/cunit_test_main.cpp
index 2c5829d19..9e31f1b34 100644
--- a/tests/unit/cunit_test_main.cpp
+++ b/tests/unit/cunit_test_main.cpp
@@ -22,6 +22,7 @@ int main(int argc, char *argv[])
 #endif

    mfem::Device device("cuda");
+
    // Include only tests labeled with CUDA. Exclude parallel tests.
    return RunCatchSession(argc, argv, {"[CUDA]", "~[Parallel]"});
 }

I am also getting the inverse of the behavior I'd expect for the unit tests, but correct me if I'm wrong.

Running pcunit_tests with CUDA builds of MFEM and Hypre produces the same results on both master and this PR (though I am getting a test failure for "Parallel H1 Full Assembly" where the results are not equal, not sure why this is. Maybe something to do with the version of Hypre I am using or something like that?)
Running punit_tests works fine on master with GPU builds, but on this PR I get the following error:

Verification failed: (!alias) is false:
 --> Cannot register an alias!
 ... in function: static void* mfem::MemoryManager::Register_(void*, void*, size_t, mfem::MemoryType, bool, bool, unsigned int&)
 ... in file: /software/palace/build-cuda-x86_64/extern/mfem/general/mem_manager.cpp:826

@tomstitt, can you reproduce this? I would have expected the punit_tests test to fail on master since MFEM would be using CPU and Hypre expecting GPU, but this PR should support that.

Here's the diff to fix running unit_tests and punit_tests for GPU builds of Hypre:
diff --git a/tests/unit/punit_test_main.cpp b/tests/unit/punit_test_main.cpp
index ecc45fcb7..cda8cd2ed 100644
--- a/tests/unit/punit_test_main.cpp
+++ b/tests/unit/punit_test_main.cpp
@@ -30,6 +30,7 @@ int main(int argc, char *argv[])
    mfem::Mpi::Init();
    mfem::Hypre::Init();
 #endif
+   mfem::Device device("cpu");

    // Only run tests that are labeled with Parallel.
    return RunCatchSession(argc, argv, {"[Parallel]"}, Root());
diff --git a/tests/unit/unit_test_main.cpp b/tests/unit/unit_test_main.cpp
index af6f6bb98..f8a550dd2 100644
--- a/tests/unit/unit_test_main.cpp
+++ b/tests/unit/unit_test_main.cpp
@@ -21,6 +21,8 @@ int main(int argc, char *argv[])
    return MFEM_SKIP_RETURN_VALUE;
 #endif

+   mfem::Device device("cpu");
+
    // Exclude parallel tests.
    return RunCatchSession(argc, argv, {"~[Parallel]"});
 }

This ensures Hypre::InitDevice gets called which correctly tells Hypre to not use the GPU based on the mfem::Device configuration (otherwise we get the error above about aliasing).
I also found a few issues with unit tests for GPU builds when not using the device. Those can be found in 1fbeee2.

thanks @sebastiangrimberg !

Thank you @sebastiangrimberg for fixing the unit tests! I completely neglected those =|

I am also getting the inverse of the behavior I'd expect for the unit tests, but correct me if I'm wrong.

Running pcunit_tests with CUDA builds of MFEM and Hypre produces the same results on both master and this PR (though I am getting a test failure for "Parallel H1 Full Assembly" where the results are not equal, not sure why this is. Maybe something to do with the version of Hypre I am using or something like that?)
Running punit_tests works fine on master with GPU builds, but on this PR I get the following error:

Verification failed: (!alias) is false:
 --> Cannot register an alias!
 ... in function: static void* mfem::MemoryManager::Register_(void*, void*, size_t, mfem::MemoryType, bool, bool, unsigned int&)
 ... in file: /software/palace/build-cuda-x86_64/extern/mfem/general/mem_manager.cpp:826

@tomstitt, can you reproduce this? I would have expected the punit_tests test to fail on master since MFEM would be using CPU and Hypre expecting GPU, but this PR should support that.

I see both errors and they are unexpected to me, let me take a look and try out your fixes !

⚠ There are some changes in master that will probably need to be updated for the runtime Hypre device configuration.
One example is this: https://github.com/mfem/mfem/blob/756fd52c2b078acd4c64e235b5fb24093fb30d83/linalg/hypre.cpp#L3576-L3603

I think @sebastiangrimberg's suggestions for the unit tests make sense and I added them, except the one for unit_test_main.cpp where adding
   mfem::Device device("cpu");

as suggested, will not make a difference, I think, since this test will not use hypre even when using a parallel MFEM build.

The CI test failure is the result of the sedov tests reusing the main function from punit_test_main.cpp in their binary and thus configuring the Device twice: First in punit_test_main.cpp and then again in test_sedov.cpp (in the TEST_CASE("Sedov") block). The first solution that comes to mind is to move the Sedov device configuration into the respective main functions so no unit tests are configuring the Device inside of a TEST_CASE.

I have pushed changes to fix the miniapps unit tests to a branch sjg/hypre-runtime-compute-policy. Rather than the above suggestion, it was easiest to just have test_sedov.cpp and test_tmop_pa.cpp just define their own main functions, as is done in test_debug_device.cpp.

I have pushed changes to fix the miniapps unit tests to a branch sjg/hypre-runtime-compute-policy. Rather than the above suggestion, it was easiest to just have test_sedov.cpp and test_tmop_pa.cpp just define their own main functions, as is done in test_debug_device.cpp.
If these look good, feel free to merge them here @v-dobrev. They should fix both CMake and GNU Make builds.

Thanks @sebastiangrimberg! I pushed a small fix and some tweaks to your branch and then merged it here.

Merged in next for testing...

Re-merged in next for testing...

Re-merged in next for testing...


Changed date picker mode to select year only, it is crashing. Can you check it again for me? #yearPicker

Will check what is going on

@trandinhhuong Here is the example and everything works fine.  Are you changing the mode programmatically? If you happened to change the code while the menu was still open, it may not work until the page reload, after that you should be good


Beam backend - IO utility + One op.

This is the first step of the proposed beam backend. This PR introduce the following:

A beam module "temporian.beam" containing the beam inference and IO utilities.
The implementation of three operators (moving_sum, select and add_index).
Testing utilities to ensure equality of the beam and "numpy" results.
Proposal for the internal implementation of the Beam backend.

Some of the next steps after this PR include:

Large scale tests.
Implementation of all the operators.
Utility to reduce the boiler plate code to implement horizontal operators.
(Possibly) refinement of the API.

Went through the ops. Select and window look super clean, but I had a lot of trouble understanding how the add_index works, maybe just need to get used to the data structure inside the beam evsets. Some improvements in naming of the data structures and in docstrings might help. Amazing work as always, thank you!

Thanks, and that is a good point.
I am adding examples in the internal function documentation to improve understanding.


Download page, and other stuff

Hi!

You can for example use github Releases, itch.io or handmade network for that.
You could also add to the todo list:

embed textures and fonts in executable, so you can just download an executable (not a zip), and you don't have to care about directory data being in one directory higher then executable.
add icon
make window title bar be something easier on the eyes then "(null)((null))"

I really like your project. I also couldn't find any decent mp3 player for windows.
I thought about writing my own but now it turns out I don't have to; so thanks and good luck.

thanks for the suggestions, I appreciate it!
I was on a bit of a hiatus from this project, but the suggestion of adding a Release is great and - already done - now.
And your other points are definitely things I will add to my to-do list and may tackle soon.


:boat: Port the codebase to typescript

This makes me think that we should port the codebase to pure TS at one point in the (near) future 🤔
CC/ @eslint-community/eslint-plugin-node

Originally posted by @MichaelDeBoey in https://github.com/eslint-community/eslint-plugin-n/pull/367#pullrequestreview-2404893785

This makes me think that we should port the codebase to pure TS at one point in the (near) future 🤔

I am currently in the jsdoc > typescript camp for a few reasons:

Any transform cause more problems in the future than they solve now
I prefer adding documentation lines to functions, meaning I have to make a jsdoc block anyway
The allowJs flag means that we can use all of typescript's typing benefits without the transpilation step.

Its not that I am not against using typescript, I just find that I prefer jsdoc :thinking:

_Originally posted by @scagood in https://github.com/eslint-community/eslint-plugin-n/pull/367#issuecomment-2447204689

I'm fully with @scagood on this – I see no reason to swap JSDoc for TS syntax – they are both first class syntaxes supported by the TypeScript type checker

I wanted to add some pros for typescript here so things are a little balanced:

Making advanced types in JSDoc, while possible is sometimes cumbersome.
Integrations with IDEs when using typescript directly is a little smoother (eg autocompletion of types in JSDoc blocks)
typescript + JSDoc feels like a second class citizen in a few cases. For example:

the @import tag does not work if there is not already at least one import.
#320 Almost certainly would not have happened in full typescript

@aladdin-add @MichaelDeBoey
What do you think either way?

I like typescript + jsdoc: easier to write some advanced types, and no build needed just like before. 😄

https://github.com/eslint-community/eslint-plugin-n/issues/320 Almost certainly would not have happened in full typescript

It's exactly for these kind of things that I think having this build step makes more sense

Bugs can happen anywhere in the TS type declaration generation.
If we want to guard against it then we should either manually create the type declarations or add tests for the generated ones.

We can actually remove the type declaration generation from this plugin as the only types that gets uploaded and used are the 5-10 lines of types/index.d.ts
The internal type checking is the more important and useful one.

@voxpelli - it's just i did in https://github.com/eslint-community/eslint-plugin-n/pull/366.
it split the two things:

committed a user defined index.d.ts - for users.
make tsc just do type-checking - for devs.

I am not sure I agree that because that problem was simply mitigated that was should dismiss it as not a problem :thinking:

I do however think that we should test the index.js interface, especially as that will cause the exported types to be tested too.

🤔 I think I am going to close this as 'not doing'


Adding code to suppress default warning when unused method result in …

…Swift 3

I think I might instead mark the bind and functions with @discardableResult so that the warnings don't appear

Sounds good to me 👍🏻
—
Nick Farrant
07720635147

On 10 Oct 2016, at 09:19, Hamilton Chapman<EMAIL_ADDRESS>wrote:
I think I might instead mark the bind and functions with @discardableResult so that the warnings don't appear
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/pusher/pusher-websocket-swift/pull/98#issuecomment-252558465, or mute the thread https://github.com/notifications/unsubscribe-auth/AF9hfSzbuA-xeN69C3oIPiQEcuYBNmWDks5qyfUEgaJpZM4KRFZW.

Added in 0030fa43


Updated Deployment description for clarity.

Rephrases part of the description of a Deployment in the "Deploy an App" section of the "Kubernetes Basics" tutorial to be more clear.
Fixes https://github.com/kubernetes/website/issues/19312

signed

/assign @makoscafee

signed

@brentcklein any update on this?

/lgtm
/approve


Subclass primitive types as jitclass

Feature request
In some applications, one uses a primitive type to represent data.
It would be great to be able to subclass this type to indicate it is used in a certain manner and to factor the code.
Example: right now I'm trying to code a SAT solver in numba (for fun and experimentation, not performance). It produces code like that:
Var = numba.int32
Lit = numba.int32

@njit(Lit(Var))
def lit_new(var):
    return var

@njit(Lit(Lit))
def lit_neg(lit):
    return -lit

@njit(numba.boolean(Lit))
def lit_sign(lit):
    return lit < 0

@njit(Var(Lit))
def lit_var(lit):
    return abs(lit)

@njit(numba.uint32(Lit))
def lit_index(lit):
    if lit_sign(lit):
        return 2 * (lit_var(lit) - 1)
    else:
        return 2 * lit_var(lit) - 1

Thanks for the query, does this documentation about extending Numba help numba.pydata.org/numba-doc/dev/extending/interval-example.html ?

Thank you for the answer.
I have read most of the documentation.
My query would more look like inheritance for jitclass from primitive data types.
The code I wrote does what I want. The only problem is that I would have to drop some performance (I guess) by using a jitclass with only one attribute. In the example I'm using jit methods on a primitive data type (which in return produces code that is less readable).
But I'm not sure this is even a desirable feature for most users.
Globally, I find it hard to define custom compiled data types in numba (without reflection), as compared to directly writing C++ or Cython (sorry about this little rant).
I think there is already a feature request about that, but type inference in the constructor based on the spec would be great too! (eg for lists)

Are you trying to achieve something like pattern-matching?
Do you want to overload methods on the subclass?

I'm not sure about what you mean with pattern matching.
I don't need to overload methods on the subclass.
I would like to access all the methods of the parent class (here an int) and add other methods.
Currently there are two alternatives : using functions like I did in the example (but there is no benefit from the type system and the code quality is low without objects) or creating a new jitclass with just one attribute and "binding" all the methods.
I would really like the ability to inherit jitclasses, from native types or even other jitclasses. Other types that are supported like namedtuples would be great.
The ability to override methods would be an advantage too.


Withdraw unbounded possible before slash is applied

Staking pallet applies slashes up to but not including active_era - slashing_defer_duration (27)

But withdraw unbonded is allowed starting from current_era = unbonded_era + bondig_duration (28).

Taking into account that current_era is set before active_era i think it is possible to unbond in the era when unapplied slash is created and perform withraw unbonded when corresponding current era is set before active_era changes (so before slash is applied).

The current_era and active_era are always the same except for 1 session, since the session pallet has 1 session delay. Only when we elect a new validator set, the current era is active_era + 1 for 1 session, and then they become the same again.
First glance, this does not seem like an issue for me.
Withdrawing happens on the basis of current_era, and you can only withdraw if current_era has been incremented at least 28 times after you called unbond.
Slashing happens on the basis of active_era, and your slash will be applied after active_era has been incremented 27 times.
While current_era and active_era can be phase-shifted, the progress at the same pace, so the slash should always happen before withdrawing is possible.

That's right. But what I noticed is that the slash is not applied for the active_era - 27 in the active_era but one can withdraw unbonded chunk from current_era - 28 when current_era is set

but one can withdraw unbonded chunk from current_era - 28 when current_era is set

Hmm yeah this seems to be the case: https://github.com/paritytech/substrate/blob/master/frame/staking/src/lib.rs#L447
The BondingDuration is not inclusive, but SlashDeferDuration is inclusive. This could cause issues, have to look into it more.

(I haven't had time to confirm this yet, but added it to our backlog so eventually we get to it. )

This was fixed as a part of https://github.com/paritytech/substrate/pull/11823


Allow to visualize component information about an instance

That would include exported variables, imported ones, etc.
And show imports too.

Ergonomy will have to be improved, but at least, there is now a basis for this feature.


UnitOfMeasure should not be an object within the model

Discussion on 24 August committee meeting:  Since we require the ADAPT unit of measure system, should we have anything more than Unit of Measure code on any object in the model?

Agreed.

Agreed.

Confirmed at September 7, 2022 meeting

Unit of Measure and UnitOfMeasure Dimension are replaced by the respective codes from the ADAPT Unit System.


Fix G-Cloud search button

Brings in a bugfix for the search button on G-Cloud.
>> Bug on Tracker

--> be
      fore

--> now

This requires https://github.com/alphagov/digitalmarketplace-buyer-frontend/pull/309

👍


More explicit explanation regarding view updates caused by array mutation

In the list rendering docs, I find this mutation methods being a bit unclear:

Mutation Methods

Vue wraps an observed array's mutation methods so they will also trigger view updates. The wrapped methods are:

push()
pop()
shift()
unshift()
splice()
sort()
reverse()

You can open the console and play with the previous examples' items array by calling their mutation methods. For example: example1.items.push({ message: 'Baz' }).

Personally, I find this to be a bit oversimplified in comparison to what it actually means.
I have provided a Codepen example where you can see what this means in practice.
For me, this was somewhat unexpected. I did not until now know that by changing an array which is referenced in dom, Vue will recompute the whole component.*) (* Not necessarily sub-components *)
When I first spotted this in my own codebase, I did not understand how by changing an in data, the whole thing re-computes, for me, I thought logically only the array affected was the only thing to change, but it does make sense now after I have investigated it deeper. However, now that I came back to these docs to see for an explanation, this line is the only thing that is offered:

Vue wraps an observed array's mutation methods so they will also trigger view updates. The wrapped methods are:

Before my observation, reading the docs, I would understood the line as the following:

This is only applied to for-example an v-for: If you update the array used in a v-for, the page(view) will get re-rendered. This seems logical.

Now however, I know that any referenced array that gets mutated by array prototypes will make the page re-render.
My code-pen example above seems pretty obvious to me now, but if you instead of simply rendering the siblingArray, you use it as a prop to a child-component(dom reference), this stuff gets really confusing.

Hey @BlitZz96!
This has nothing to do with the mutation methods of an array. When you mutate any data property that's bound in the template, the view re-renders. You can check this fork of your example which mutates a foo (a number), causing the exact same thing.
The fact that methods are eager is explained in the docs:

However, the difference is that computed properties are cached based on their reactive dependencies. A computed property will only re-evaluate when some of its reactive dependencies have changed. This means as long as message has not changed, multiple access to the reversedMessage computed property will immediately return the previously computed result without having to run the function again. [...] In comparison, a method invocation will always run the function whenever a re-render happens.

The important part is that Vue uses a virtual DOM to keep track of the changes it needs to make to the real DOM. The parts that don't change aren't updated, as you can see in the following GIF.

Does this help?

Thank you for creating this example,, this makes sense to me now! 💯


🔥 Improved footer

🫤 Description

Improving the footer by incorporating a more appropriate sentence.

📍 Use Case

Give more details about the license, authors, etc.

📦 Additional Information

Add any other relevant information here.

✋🏻 Upvotes

If you support this enhancement, add a 👍🏻.

🤔 To-Do

If you have specific tasks or action items related to this enhancement, list them here.

🌱 Contribute

[] I want to contribute

🎉 Hacktoberfest

[] I want to participate

📌 Related Issue

#1

hii @divyanshudhruv
So Want i want to do is that i want to enhance the footer
from this
Template Developed With By <> Divyanshu Dhruv
To
@2023 All Rights Reserved | Template Developed With By <> Divyanshu Dhruv
More over the year "2023" will be updated dynamically with the help of JS, so that we dont need to manually come to  the page and update it every year.
This will  be  the maximum enhancement which can be done to the footer for now.

Ok, good luck. Assigning the issue. You may open pr in development branch during hacktoberfest

Thanks for assigning the task to me.
I will make the pull request on 1st of October

Nice. But you can either increase the width of the footer or put the Template develpoed by <> Divyanahu Dhruv on a new line to avoid any congestion.

Cool. The desktop one looks nice. Also you may give space after copyright symbol to make it more clear

Ok i will leave space after the copyright's symbol.
Can you tell me which words would you like to be made bold.

Cool. You may bold the copyright icon and the year and also put space after |

Roger sir

Can you also send the screenshot after you did the issue? You may open pr into main branch

Here you go,
I have made all the changes as you ask me to do, hope this satisfices your needs

Cool. I think remove the bold and if possible decrease the line height, else it's OK! Good job @Vishal-1004

I Guess the changes are clearly implemented and seen in this screen shot. Let me know  if can make a pull request with this and that to in branch i am suppose to make the pull request

Cool you can open pr. Good job

Congrats @Vishal-1004


Consolidate device diagnostics and create a tool to regenerate them

Combines the logic from HA Core, this library, and https://github.com/dmulcahey/zha-device-exporter/ into a single method: Device.get_diagnostics_json().

Everything works but with a small problem: clusters that modify the state (e.g. double battery percentage) modify it globally, so running the tool keeps doubling them after re-load. We'll need to restore entity state after loading device info, it seems.


Bump Slint to version 1.7.0 and switch for winit-wayland to Linuxkms …

…backend
This removes the need to run a weston container.

LGTM


Verify bulk_insert works correctly in Rails 5

@shaktik1989 is investigating using https://github.com/jamis/bulk_insert to optimize bulk operations.  We will want to make sure it works correctly in Rails 5 once this change is merged in.

Cool, will make sure I check after #238 is merged.


operationstatusresult api cli

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

General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

please resolve the code conflict.

@yanzhudd  @wangzelin007 , Can you help review again?


Typos in Volume Performance Rules

Love the MP, however there are a few typos in the volume performance rules;
Weites Operations per Second
Total Bandwidth has a '$' after PureVolume
Reads Operations per Second possibly should be Read Operations per Second

Thanks for this. We will address them.

Reported to engineering.


Setup app and lib deployment actions

Details

[x] setup skynet deployment pipeline for alpha apps on the develop branch
[x] setup npm publishing for libs on the master branch
[x] setup subdomains

Need to research HTTPS and skynet

https://docs.netlify.com/domains-https/https-ssl/#custom-certificates


add functionality for --force-parent-span-id

similar to --force-trace-id and --force-span-id, this PR adds the ability to specify --force-parent-span-id or OTEL_CLI_FORCE_PARENT_SPAN_ID to override a parent span id.
cc/ https://github.com/equinix-labs/otel-cli/issues/240

hey @tobert, thanks for the feedback. I pushed that change from string to hex, so hopefully it should pass now. 🤞


About caption_mask.

Hi,
I don't understand the part of caption_masks, especially energies[~masks] = -float('inf'). Can you explain?
Thanks.
caption_generator.py >> def forward():

 if captions is None:
            _, captions = outputs.max(dim=2)
        caption_masks = (captions != self.vocab.word2idx['<PAD>']) * (captions != self.vocab.word2idx['<EOS>'])
        caption_masks = caption_masks.cuda()

local_reconstructor.py >> def forward():

      attention_masks = caption_masks.transpose(0, 1)
        decoder_hidden, attn_weights = self.attention(last_hidden, decoder_hiddens, attention_masks)

temporal_attention.py >> def forward(self, hidden, feats, masks=None):

        if masks is not None:
            energies = energies.squeeze(2)
            energies[~masks] = -float('inf')
            energies = energies.unsqueeze(2)
        weights = F.softmax(energies, dim=1)

The mask is defined in the line as follows:
caption_generator.py >> def forward():

caption_masks = (captions != self.vocab.word2idx['<PAD>']) * (captions != self.vocab.word2idx['<EOS>'])

which makes only the position where the words before the end of the caption to have a value of 1 and the other positions (<EOS> or <PAD>) to have a value of 0.
Then, ~mask will make the position that contains <EOS> or <PAD> to have a value of 1, and the other positions to have a value of 0. Therefore,
temporal_attention.py >> def forward(self, hidden, feats, masks=None):

        if masks is not None:
            energies = energies.squeeze(2)
            energies[~masks] = -float('inf')
            energies = energies.unsqueeze(2)
        weights = F.softmax(energies, dim=1)

the above lines will assign -float('inf') to the attention coefficients (energies) at the positions that contains <EOS> or <PAD>. Then, the attention weights (weights) at the positions will be 0 after applying softmax. Finally, we can ignore the hidden states of <EOS> and <PAD> through this masking.


(WIP) Adding translation support

This PR adds translation support to the game. It is pretty easy and very straightforward, I'm not doing anything fancy here. The translations are stored inside a .csv file (utf-8) and loaded using the Godot Locale support.
This PR will add also the Brazilian Portuguese translation to the game, so... yeah, later other translations can be added by other contributors.

Excited to see this land, thanks!

@RoddieKieley @ernesgonzalez33 what do you think?

@rluders great idea! I will test it when I have the time and maybe add Spanish too!! You can merge it if you see it works. Or we can wait for @RoddieKieley's approval

@ernesgonzalez33 actually I was testing it right now, and I think that I found a bug related to the OptionButton. But, yeah, go for it... Add the Spanish too! :+1:

Now it is working as it should be. 100% sure. I'll merge it.


List() parameter - NullReference

Hello,
I've tried to define filter with Contains() operator just like in your example.
This is what I have defined:
var values = new List<int> { 1, 2, 3, 4, 5 };
var values2 = new List<int?> { null, 1, 2, 3, 4, 5 };
modelBuilder.Filter(DataFilters.MustHavePlace, (IMustHavePlace p, List<int> placeList) => placeList.Contains(p.PlaceId), () => values);
modelBuilder.Filter(DataFilters.MayHavePlace, (IMayHavePlace p, List<int?> placeList) => placeList.Contains(p.PlaceId), () => values2);

If I'll call the query for the specified entity it will always return the following exception
System.NullReferenceException: Object reference not set to an instance of an object.
at EntityFramework.DynamicFilters.DynamicFilterExtensions.SetSqlParameters(DbContext context, DbCommand command)
at EntityFramework.DynamicFilters.DynamicFilterCommandInterceptor.SetDynamicFilterParameterValues(DbCommand command, DbContext context)
at EntityFramework.DynamicFilters.DynamicFilterCommandInterceptor.ReaderExecuting(DbCommand command, DbCommandInterceptionContext`1 interceptionContext)
at System.Data.Entity.Infrastructure.Interception.DbCommandDispatcher.<Reader>b__d(IDbCommandInterceptor i, DbCommand t, DbCommandInterceptionContext`1 c)
at System.Data.Entity.Infrastructure.Interception.InternalDispatcher`1.Dispatch[TTarget,TInterceptionContext,TResult](TTarget target, Func`3 operation, TInterceptionContext interceptionContext, Action`3 executing, Action`3 executed)
at System.Data.Entity.Infrastructure.Interception.DbCommandDispatcher.Reader(DbCommand command, DbCommandInterceptionContext interceptionContext)
at System.Data.Entity.Internal.InterceptableDbCommand.ExecuteDbDataReader(CommandBehavior behavior)
at System.Data.Entity.Core.EntityClient.Internal.EntityCommandDefinition.ExecuteStoreCommands(EntityCommand entityCommand, CommandBehavior behavior)
at System.Data.Entity.Core.Objects.Internal.ObjectQueryExecutionPlan.Execute[TResultType](ObjectContext context, ObjectParameterCollection parameterValues)
at System.Data.Entity.Core.Objects.ObjectQuery`1.<>c__DisplayClass7.<GetResults>b__6()
at System.Data.Entity.Core.Objects.ObjectContext.ExecuteInTransaction[T](Func`1 func, IDbExecutionStrategy executionStrategy, Boolean startLocalTransaction, Boolean releaseConnectionOnSuccess)
at System.Data.Entity.Core.Objects.ObjectQuery`1.<>c__DisplayClass7.<GetResults>b__5()
at System.Data.Entity.SqlServer.DefaultSqlExecutionStrategy.Execute[TResult](Func`1 operation)
at System.Data.Entity.Core.Objects.ObjectQuery`1.GetResults(Nullable`1 forMergeOption)
at System.Data.Entity.Core.Objects.ObjectQuery`1.<System.Collections.Generic.IEnumerable<T>.GetEnumerator>b__0()
at System.Data.Entity.Internal.LazyEnumerator`1.MoveNext()
at System.Linq.Enumerable.Single[TSource](IEnumerable`1 source)
at System.Data.Entity.Core.Objects.ELinq.ObjectQueryProvider.<GetElementFunction>b__3[TResult](IEnumerable`1 sequence)
at System.Data.Entity.Core.Objects.ELinq.ObjectQueryProvider.ExecuteSingle[TResult](IEnumerable`1 query, Expression queryRoot)
at System.Data.Entity.Core.Objects.ELinq.ObjectQueryProvider.System.Linq.IQueryProvider.Execute[TResult](Expression expression)
at System.Data.Entity.Internal.Linq.DbQueryProvider.Execute[TResult](Expression expression)

I don't see anything wrong with your filter.  Can you provide some more information:

Version of EF and of the DynamicFilter library
Database & version you are using
The 2 models & interfaces you are referencing (just the relevant parts - enough so I can try to reproduce the problem).

Here are additional information:

EF v6.1.3, I've also tried v6.1.2 and v6.1.1, but without success. DynamicFilters v1.4.1.
DB is MS SQL 2014 on localhost and MS SQL 2012 on production machine.

IMustHavePlace interface
public interface IMustHavePlace
{
    int PlaceId { get; set; }
}

IMayHavePlace interface
public interface IMayHavePlace
{
    int? PlaceId { get; set; }
}

Place model
public class Place
{
    public virtual int Id { get; set; }

    public virtual string Name { get; set; }

    public virtual string Description { get; set; }

    public virtual string Street { get; set; }

    public virtual string City { get; set; }

    public virtual string Zip { get; set; }

    public virtual string CountryCode { get; set; }

    public virtual ICollection<Machine> Machines { get; set; }

    public virtual ICollection<Article> Articles { get; set; }
}

Machine model implements IMayHavePlace interface
public class Machine : IMayHavePlace
{
    public virtual int Id { get; set; }

    public virtual string MachineNumber { get; set; }

    public virtual string Name { get; set; }

    public virtual string Description { get; set; }

    [ForeignKey("PlaceId")]
    public virtual Place Place { get; set; }

    public virtual int? PlaceId { get; set; }
}

Article model implements IMustHavePlace interface
public class Article : IMustHavePlace
{
    public virtual int Id { get; set; }

    public virtual string Name { get; set; }

    public virtual string Description { get; set; }

    public virtual string Ean { get; set; }

    [ForeignKey("PlaceId")]
    public virtual Place Place { get; set; }

    public virtual int PlaceId { get; set; }
}

I used your models and filters to try to reproduce the problem.  I am not getting the same error as you, but there is a problem with this filter:
var values2 = new List<int?> { null, 1, 2, 3, 4, 5 };
modelBuilder.Filter(DataFilters.MayHavePlace, (IMayHavePlace p, List<int?> placeList) => placeList.Contains(p.PlaceId), () => values2);

The problem is with the handling of a nullable list and how the null check is being done for the null value.  I actually have a unit test for this same type of filter and I have the test condition wrong so I didn't notice that it wasn't working.  This is fixable but it may take a bit of work so it might be a few days.
In the mean time, you can create your filter like this and it will accomplish the same thing:
var values2 = new List<int> { 1, 2, 3, 4, 5 };
modelBuilder.Filter(DataFilters.MayHavePlace, (IMayHavePlace p, List<int> placeList) => !p.PlaceId.HasValue || placeList.Contains(p.PlaceId.Value), () => values2);

Can you try that and see if it clears up your errors?  If not, I may need a full test project that reproduces the problem.

I tried change the filter as you suggested, but without success, still the same error. Problem is both IMustHavePlace and IMayHavePlace are producing this error.
I'm using ABP Framework (http://aspnetboilerplate.com/) in my project and I'm waiting to confirm that there is no problem with DynamicFilters. See http://forum.aspnetboilerplate.com/viewtopic.php?f=2&t=484.
You can try to generate a new template project from their web site and you can test filters on that project.
Maybe it will be helpful.

Sorry for the late response.  I think I just tracked this down to a context.Database.Log call.  That's only done when using a List/Contains() and I was not checking to make sure there was actually a logger attached!  I just checked in a fix and pushed out an update to NuGet.  Let me know if that fixes it for you.

I can confirm it is fully working. We can close this issue.
Thank you very much.


Enable -Xfatal-warnings, Remove remaining warnings

Huh. Those changes don't fail -Ywarn-value-discard? I could've sworn they would/do...
Anyways, Travis disagrees with me. :+1:

I guess I'm getting confused with NonUnitStatements which I often also have enabled.


Javascript bot MSTeams mobile apps not rendering action iconUrl

JavaScript
node-js bot sending adaptive card
botframework v4
Details
I have an adaptive card with Action.ShowCard and Action.ToggleVisibility actions with iconUrl for the button.  The images render just fine on MSTeams mac app and MSTeams website, but on both iOS and Android MSTeams apps, the images do not show.
Here you can see it working on the desktop app:

Here you can see a mobile screen shot where the buttons do not render:

Attached is also the card json:
{"type":"AdaptiveCard","version":"1.2","body":[{"type":"TextBlock","text":"Here is some relevant information...","wrap":true},{"type":"Container","separator":false,"items":[{"type":"TextBlock","text":"Pryon requires employees to book air travel in coach or economy classes.  **You must have manager approval**  to book first or business class travel.","wrap":true,"maxLines":16},{"type":"FactSet","facts":[{"title":"Source","value":"[All Files/dev/Travel Policy, Benefits, and Employee Handbook/Pryon_Employee_Travel__8.19.docx](https://app.box.com/file/589339734969)"}],"isVisible":true,"id":"mainFactSet_0"},{"type":"FactSet","facts":[{"title":"Confidence","value":"51%"},{"title":"Name","value":"All Files/dev/Travel Policy, Benefits, and Employee Handbook/Pryon_Employee_Travel__8.19.docx"},{"title":"Level","value":"weak_accept"}],"isVisible":false,"id":"details_id_0"}],"style":"default","isVisible":true,"actions":[]},{"type":"Container","separator":true,"items":[{"type":"TextBlock","text":"All business travel must be booked 14 days prior to your departure date.  **If that is not possible, obtain approval from your manager. As always, use common sense when booking your travel to balance costs with convenience.**   ","wrap":true,"maxLines":16},{"type":"FactSet","facts":[{"title":"Source","value":"[All Files/dev/Travel Policy, Benefits, and Employee Handbook/Pryon_Employee_Travel__8.19.docx](https://app.box.com/file/589339734969)"}],"isVisible":true,"id":"mainFactSet_1"},{"type":"FactSet","facts":[{"title":"Confidence","value":"44%"},{"title":"Name","value":"All Files/dev/Travel Policy, Benefits, and Employee Handbook/Pryon_Employee_Travel__8.19.docx"},{"title":"Level","value":"reject"}],"isVisible":false,"id":"details_id_1"}],"style":"emphasis","isVisible":false,"actions":[],"id":"answer_containter_1"},{"type":"Container","separator":true,"items":[{"type":"TextBlock","text":"•Travel not booked in  **economy class** ","wrap":true,"maxLines":16},{"type":"FactSet","facts":[{"title":"Source","value":"[All Files/dev/Travel Policy, Benefits, and Employee Handbook/Pryon_Employee_Travel__8.19.docx](https://app.box.com/file/589339734969)"}],"isVisible":true,"id":"mainFactSet_2"},{"type":"FactSet","facts":[{"title":"Confidence","value":"44%"},{"title":"Name","value":"All Files/dev/Travel Policy, Benefits, and Employee Handbook/Pryon_Employee_Travel__8.19.docx"},{"title":"Level","value":"reject"}],"isVisible":false,"id":"details_id_2"}],"style":"default","isVisible":false,"actions":[],"id":"answer_containter_2"}],"$schema":"http://adaptivecards.io/schemas/adaptive-card.json","actions":[{"type":"Action.ToggleVisibility","title":" ","iconUrl":"https://cdn2.iconfinder.com/data/icons/social-icons-33/128/Trello-128.png","isVisible":true,"targetElements":["answer_containter_1","answer_containter_2","details_id_0","details_id_1","details_id_2"]},{"type":"Action.ShowCard","style":"default","title":"Related","card":{"type":"AdaptiveCard","version":"1.2","body":[{"type":"Input.ChoiceSet","id":"related_choice","isMultiSelect":false,"style":"expanded","spacing":"none","height":"stretch","choices":[{"title":"Do i need to fly commercial?","value":"Do i need to fly commercial?"},{"title":"Can I fly on whatever airline I want?","value":"Can I fly on whatever airline I want?"}],"value":"Do i need to fly commercial?"}],"actions":[{"type":"Action.Submit","title":"Ask"}]}},{"type":"Action.ShowCard","version":"1.2","title":" ","iconUrl":"https://storage.googleapis.com/pryon-bot-assets/fa-comments-blue.png","card":{"type":"AdaptiveCard","body":[{"type":"TextBlock","text":"Please give us your feedback on the answer."},{"type":"Input.ChoiceSet","id":"feedback_rating","isMultiSelect":false,"style":"expanded","choices":[{"title":"Answer was on topic and helpful","value":"5"},{"title":"Answer was on topic, but not helpful","value":"4"},{"title":"Answer was not on topic","value":"3"},{"title":"Did not receive an answer","value":"2"}]}],"actions":[{"type":"Action.Submit","title":"Submit Feedback","data":{"feedback_exchange_id":"97f261fc-408d-4406-8ada-5bfd8bb3b70b"}}]}}]}

@vikrantsingh01 is this something you can help get us a first pass on to see if this is really an AdaptiveCards issue? Thanks a ton!

This can be verified in the latest Teams Alpha.

That means verified the repro or the fix?

That means verified the repro or the fix?

@mattcobb
this most likely means that once teams' mobile app finishes updating their app with later version of adaptive cards sdk, this issue will be addressed.
the screenshot from @VikrantSingh01 shows images from action icon url, and as they mentioned, this image was taken from their alpha build.

@vikrantsingh01 thanks for helping.. Could you confirm if you see the repro still with Teams Mobile alpha drop or if you've verified it fixed?  Thanks!


improve Prometheus metrics removal

What does this PR do?
Improve Prometheus metrics removal.
Motivation
Formerly metric removal was done by tracking configuration generations. This was an easy first solution but is actually not correct and for setups where Traefik does not receive a lot of configuration updates, old metrics can hang around for quite some time.
The new approach is to keep track of Traefik's current configuration and to remove metrics that "belong" to an outdated configuration, once they were scraped. This way we can be sure that we don't keep old metrics too long and we can ensure that metrics for short-lived configuration still appear when the /metrics is scraped, as we only remove them after the scrape happened.
More

[x] Added/updated tests

FTR: I missed a local optimization and will update the PR soon again.


Missing semicolons

Description
No semicolons in lines 8, 9 and 11 of remove_duplicates.cc
Steps to reproduce

Always a bug because no semicolon

Expected behavior
compiling
Actual Behavior:
Not compiling
Additional info:
Windows
Chrome

Added semicolons to lines 8, 9, and 11. Bug = fixed.


Uncaught ReferenceError: html2canvas is not defined.

Hi , I am using html2canvas to convert html to jpg. When I am using

https://cdnjs.cloudflare.com/ajax/libs/html2canvas/0.4.1/html2canvas.js

this 0.4.1 cdn its working fine. But On

https://cdnjs.cloudflare.com/ajax/libs/html2canvas/0.5.0-beta4/html2canvas.js

or larger then 0.4.1 its giving error
Uncaught ReferenceError: html2canvas is not defined.

I am using on Magento 2

Latest version is 1.0.0


Unclear how I can have a central Makefile.toml for a workspace.

Consider this directory structure:
├── Cargo.toml
├── Makefile.toml
├── bar
│   └── Cargo.toml
└── foo
    └── Cargo.toml

I want to be able to be inside foo directory and Makefile.toml of the workspace to be loaded automatically. The only way I found is to create a new makefile within foo and use extend = "../Makefile.toml".
Is there a way without creating new makefile in foo?

can you check this one out?
https://github.com/sagiegurari/cargo-make#usage-workspace-extend
this however relevant if you are running from the workspace root.
when running directly in the member crate, the extend you did is the solution

Ok thanks. I will this solution.


Placement of vector inside of a bracketed fraction leaves large white space

Description
The equation
$ K_(i j) = sigma^2 exp(- (norm(vec(a_i - a_j, b_i - b_j))_2^2)/(2 l^2)) $

Produces

Which has considerably more white space in the denominator than I expected. I'm not sure if this is expected behaviour, so I'm opening this just to let y'all know. I didn't see any other issues related to this, so I apologize if I missed something that's already open.
Reproduction URL
No response
Operating system
Linux
Typst version

[X] I am using the latest version of Typst

$$
K_{ij} = \sigma^{2}\exp\left( - \frac{\left| \begin{pmatrix}
a_{i} - a_{j} \
b_{i} - b_{j}
\end{pmatrix} \right|_{2}^{2}}{2l^{2}}\right)
$$

This is just an idea, but you could probably use a contextual function and a show rule on frac to measure fractions and check if their size is too lopsided, then use place(dy: _) to shift them down. Then a show rule on lr could check for this (check if the lr contains state??) and resize to match.

You may not like it, but it's not a bug. It's an inevitable consequence of typesetting convention.
I recommend writing 1/(2l) in front instead so your fraction isn't so unbalanced.


bug(pnpm): require() of ES Module

With pnpm after pnpm build and then pnpm preview crashes with strange errors. With yarn everything works fine.

Operating System: Linux
Node Version:     v18.1.0
Nuxt Version:     3.0.0-rc.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      runtimeConfig, css, modules, vueuse, schemaOrg, unocss, experimental
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

[nuxt] [request error] require() of ES Module /home/ririen/Projekty/Project/Website/.output/server/node_modules/node-fetch/src/index.js from /home/ririen/Projekty/Project/Website/.output/server/node_modules/cross-fetch/dist/node-ponyfill.js not supported.
Instead change the require of index.js in /home/ririen/Project/Project/Website/.output/server/node_modules/cross-fetch/dist/node-ponyfill.js to a dynamic import() which is available in all CommonJS modules.
  at Object.<anonymous> (./server/node_modules/cross-fetch/dist/node-ponyfill.js:1:19)
  at Object.<anonymous> (./server/node_modules/graphql-request/dist/index.js:84:34)
  at async Promise.all (index 0)
  at async ./server/chunks/handlers/renderer.mjs:11064:24
  at async ./server/chunks/handlers/renderer.mjs:11120:64
  at async ./server/node_modules/h3/dist/index.mjs:417:19
  at async Server.nodeHandler (./server/node_modules/h3/dist/index.mjs:367:7)

i get this error with yarn after yarn preview or node .output/server/index.mjs
(node:95748) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
(Use `node --trace-warnings ...` to show where the warning was created)
/Users/denisnikonov/projects/nuxt-text/.output/server/node_modules/node-fetch/src/index.js:9
import http from 'node:http';
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at Object.compileFunction (node:vm:352:18)
    at wrapSafe (node:internal/modules/cjs/loader:1033:15)
    at Module._compile (node:internal/modules/cjs/loader:1069:27)
    at Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Module._load (node:internal/modules/cjs/loader:827:12)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:170:29)
    at ModuleJob.run (node:internal/modules/esm/module_job:198:25)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:409:24)

Node.js v18.1.0

@ririen @DenisNikonov This was addressed upstream and should now be working without issue for you.
be sure to delete your lockfile (yarn.lock or package-lock.json) and ensure that your new lockfile uses unenv >= 0.5.2, this can be verified for yarn by running yarn why unenv

@Diizzayy It works! Thank you for your work


drivers: nrf_qspi: use DT_INST_PROP_OR over binding default

Switch driver to use DT_INST_PROP_OR for sck-delay default instead of
specifying in binding.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

This is not consistent with the agreed upon plan at #25164 (comment) which allows defaults from bindings where they are appropriate.
What do we need to do to keep that plan moving forward? Should I submit a change to the tooling to put the default properties into the generated dts file? It's not my wheelhouse, but it's the only remaining step I can contribute to.

I'm less concerned about that functionality, but we need to probably come up with some guidelines about when default is acceptable.  Its not totally clear to me when that is.  Especially with the addition of DT_PROP_*_OR seems need of default should be even less.


Feature request: option to show seconds

With an option to display time in HH:MM:SS format, this app would be perfect for the use case of taking a picture of the clock with a camera for the purpose of later synchronizing the times of photos with the times in a GPX file recorded by the phone.

I will check that.

You can check beta version via:
https://play.google.com/apps/testing/de.quaddyservices.dynamicnightlight

In a few minutes the normal version has option to show seconds, too.

Works great, thank you!


Use dump endpoint for turso db shell $DB .dump

This is a quick fix to enable folks to get dumps of arbitrary size.
We should patch the libsql-shell-go to use the endpoint instead of its own implementation.

Tested with:
> turso db shell flying-circuit
Connected to flying-circuit at https://flying-circuit-athoscouto.turso.io

Welcome to Turso SQL shell!

Type ".quit" to exit the shell and ".help" to list all available commands.

→  select count(*) from fly;
COUNT (*)
33554432
→  .quit
> turso db shell flying-circuit .dump > fly.dump
> wc -l fly.dump
 33554437 fly.dump
> head fly.dump
PRAGMA foreign_keys=OFF;
BEGIN TRANSACTION;
CREATE TABLE IF NOT EXISTS libsql_wasm_func_table (name text PRIMARY KEY, body text) WITHOUT ROWID;
CREATE TABLE IF NOT EXISTS fly(a);
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
> tail fly.dump
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
INSERT INTO fly VALUES(1);
COMMIT;

I'm not sure whether this works correct.
╰──> turso db shell dumptest ".dump"
PRAGMA foreign_keys=OFF;
BEGIN TRANSACTION;
CREATE TABLE IF NOT EXISTS libsql_wasm_func_table (name text PRIMARY KEY, body text) WITHOUT ROWID;
COMMIT;

We probably don't want the dump of libsql_wasm_func_table, do we?

@haaawk I don't know. Can users register functions? If so it makes sense to have it on the dump. It may be work to create a issue ok sqld repo about it.


big.Int has a problem in exp function?

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

1.6.5

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
Windows 10, amd64
go env Output
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\.....\AppData\Local\go-build
set GOENV=C:\Users\.....\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=......
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=......
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=c:\go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=.....\windows_amd64
set GOVCS=
set GOVERSION=go1.16
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=NUL
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config

What did you do?
Look at https://play.golang.org/p/KpvLzGHWtou
expected result is 1 not 1111
In other words,
I think there is something wrong in big.Int Exp method.
This code:
func TestExp(t *testing.T) {
p := big.NewInt(5147)
ab = big.NewInt(5146 + 5147)
t.Logf("ab=%d", ab)
ab.Mod(ab, p)
t.Logf("ab=%d", ab)
c := big.NewInt(1111)
c_pow_ab := new(big.Int).Exp(c, ab, p)
t.Logf("c_pow_ab=%d", c_pow_ab)
}
and this code
func TestExp(t *testing.T) {
p := big.NewInt(5147)
ab = big.NewInt(5146 + 5147)
t.Logf("ab=%d", ab)
// ab.Mod(ab, p)
t.Logf("ab=%d", ab)
c := big.NewInt(1111)
c_pow_ab := new(big.Int).Exp(c, ab, p)
t.Logf("c_pow_ab=%d", c_pow_ab)
}
should both return give "1" as result of the exponentiation.
The second snippet gives "1111".
In general, not "moduling" the exponent before entering EXP seems to return "one more power" of the base for each "order" addition, that should instead be a don't care.
The base of the exponentiation seems not to suffer the problem.
What did you expect to see?
1
What did you see instead?
1111

Had to check better. Possibly I'm doing confusion.


: Image for service serge was built because it did not already exist. To rebuild this image you must use docker-compose build

Bug description
stalling collected packages: sentencepiece, rfc3986, asyncio, websockets, uvloop, urllib3, ujson, typing_extensions, tqdm, toml, sniffio, PyYAML, python-multipart, python-dotenv, packaging, orjson, MarkupSafe, itsdangerous, idna, httptools, h11, filelock, dnspython, click, charset-normalizer, certifi, uvicorn, requests, pymongo, pydantic, Jinja2, email-validator, anyio, watchfiles, starlette, motor, lazy-model, huggingface-hub, httpcore, sse-starlette, httpx, fastapi, beanie
Successfully installed Jinja2-3.1.2 MarkupSafe-2.1.2 PyYAML-6.0 anyio-3.6.2 asyncio-3.4.3 beanie-1.17.0 certifi-2022.12.7 charset-normalizer-3.1.0 click-8.1.3 dnspython-2.3.0 email-validator-1.3.1 fastapi-0.95.0 filelock-3.10.2 h11-0.14.0 httpcore-0.16.3 httptools-0.5.0 httpx-0.23.3 huggingface-hub-0.13.3 idna-3.4 itsdangerous-2.1.2 lazy-model-0.0.5 motor-3.1.1 orjson-3.8.8 packaging-23.0 pydantic-1.10.7 pymongo-4.3.3 python-dotenv-1.0.0 python-multipart-0.0.6 requests-2.28.2 rfc3986-1.5.0 sentencepiece-0.1.97 sniffio-1.3.0 sse-starlette-1.3.3 starlette-0.26.1 toml-0.10.2 tqdm-4.65.0 typing_extensions-4.5.0 ujson-5.7.0 urllib3-1.26.15 uvicorn-0.21.1 uvloop-0.17.0 watchfiles-0.18.1 websockets-10.4
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
Removing intermediate container 6a4772c9868d
---> 7d52cdb4300b
Step 20/28 : COPY --from=llama_builder /tmp/llama.cpp/llama /usr/bin/llama
---> 2d1bc67ebd29
Step 21/28 : COPY ./web/package.json web/
---> 5ad251e958d3
Step 22/28 : COPY ./web/package-lock.json web/
---> ac7bf07e7416
Step 23/28 : RUN cd web && npm install
---> Running in 3ce4a67b33f7
added 247 packages, and audited 248 packages in 49s
51 packages are looking for funding
run npm fund for details
found 0 vulnerabilities
npm notice
npm notice New minor version of npm available! 9.5.1 -> 9.6.2
npm notice Changelog: https://github.com/npm/cli/releases/tag/v9.6.2
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice
Removing intermediate container 3ce4a67b33f7
---> df7025bd9c0c
Step 24/28 : COPY ./web /usr/src/app/web/
---> cbb26bc088c0
Step 25/28 : COPY ./api /usr/src/app/api
---> 7f09f6fb61cb
Step 26/28 : COPY deploy.sh /usr/src/app/deploy.sh
---> 1f1322009a38
Step 27/28 : RUN chmod +x /usr/src/app/deploy.sh
---> Running in 4b0b0cea3080
Removing intermediate container 4b0b0cea3080
---> 6544683f68f8
Step 28/28 : CMD /usr/src/app/deploy.sh
---> Running in a56fd3d98787
Removing intermediate container a56fd3d98787
---> 598872c7f3b2
Successfully built 598872c7f3b2
Successfully tagged serge_serge:latest
WARNING: Image for service serge was built because it did not already exist. To rebuild this image you must use docker-compose build or docker-compose up --build.
Creating serge_serge_1 ... done
Attaching to serge_serge_1
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_serge_1 exited with code 132
Steps to reproduce
git clone https://github.com/nsarrazin/serge.git
cd serge
docker compose up -d
Environment Information
docker: 20.10.23
OS: Debian Testing x64
Screenshots
No response
Relevant log output
No response
Confirmations

[X] I'm running the latest version of the main branch.
[X] I checked existing issues to see if this has already been described.

git clone
docker compose build
docker compose up -d

@gaby It still crashes
 ---> Using cache                                                                                [25/1467]
 ---> ac7bf07e7416
Step 23/28 : RUN cd web && npm install
 ---> Using cache
 ---> df7025bd9c0c
Step 24/28 : COPY ./web /usr/src/app/web/
 ---> Using cache
 ---> cbb26bc088c0
Step 25/28 : COPY ./api /usr/src/app/api
 ---> Using cache
 ---> 7f09f6fb61cb
Step 26/28 : COPY deploy.sh /usr/src/app/deploy.sh
 ---> Using cache
 ---> 1f1322009a38
Step 27/28 : RUN chmod +x /usr/src/app/deploy.sh
 ---> Using cache
 ---> 6544683f68f8
Step 28/28 : CMD /usr/src/app/deploy.sh
 ---> Using cache
 ---> 598872c7f3b2
Successfully built 598872c7f3b2
Successfully tagged serge_serge:latest
PC:/opt/serge$ docker-compose up
serge_serge_1 is up-to-date
Attaching to serge_serge_1
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     7 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod
serge_1  | /usr/src/app/deploy.sh: line 12:     8 Illegal instruction     (core dumped) mongod

Are you using an older cpu? Mongo 6.x requires AVX instructions set

Architecture:            x86_64
  CPU op-mode(s):        32-bit, 64-bit
  Address sizes:         36 bits physical, 48 bits virtual
  Byte Order:            Little Endian
CPU(s):                  8
  On-line CPU(s) list:   0-7
Vendor ID:               GenuineIntel
  Model name:            Intel(R) Core(TM) i7 CPU         920  @ 2.67GHz
    CPU family:          6
    Model:               26
    Thread(s) per core:  2
    Core(s) per socket:  4
    Socket(s):           1
    Stepping:            5
    CPU max MHz:         2667.0000
    CPU min MHz:         1600.0000
    BogoMIPS:            7000.29
    Flags:               fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ht tm pbe syscall nx rdtscp lm constant_tsc arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpui
                         d aperfmperf pni dtes64 monitor ds_cpl vmx est tm2 ssse3 cx16 xtpr pdcm sse4_1 sse4_2 popcnt lahf_lm pti ssbd ibrs ibpb stibp tpr_shadow vnmi flexpriority ept vpid dtherm flush_l1d

The i7 920 doesnt have support for AVX, which is required for Mongo and llama.cpp

@gaby thanks, is there a way to run this another way on this machine?


removed wrong comments

In this function there are no parameters a and b

Could we replace it with the correct comments ?

I'll create a new one


Include the impl project in root aggregation

I forgot to aggregate the new impl project in the root project, which meant some tests weren't being run.

Stealing this.


Add wheel publishing to publish.sh

Hi, lately installing fastavro is taking quite a bit time, because it is compiling using Cython.
To faster up entire package installation you could build wheels (https://packaging.python.org/tutorials/distributing-packages/#wheels).

I made some modifications to the travis script to incorporate the docker building of the linux wheels. I tested this with a dummy tag (that I've since deleted) and it published the wheels to github which I'll plan to grab before uploading to pypi (like what is done with the windows wheels). The next time a release happens there should be linux wheels with it.
I'm going to close this now, but thanks for all the help.

Great work!
Thanks


Parameter field is not case sensitive for github command

When the command is executed, it brings me to the github under point 4 and not 3. Maybe making it all case sensitive will make it easier for the user.

Hmm would it be better if we do not allow inputs for the same name regardless of the case? If they do not have the same name, there would be less confusion for the user.

This the same for some other commands such as view. I think its best if we standarise throughout the code if NAME should be case sensitive or not.

Yeah, I agree with @DesSnowy. It would be easier if we solve the root cause so that everything else is standardised

I believe having name field to be case insensitive would be a good idea since it is better to remember two different person with the same name by some other means like adding the surname rather than by the case.

Hmm why wasn't this closed


Use "NewImplFull" in injection-gen

Expected Behavior
I jump from a tekton issue: https://github.com/tektoncd/pipeline/issues/3804
I guess we should use NewImplFull in injection generation:
seems the NewImpl and NewImplWithStats is already deprecated.
with this change we can use custom Rate Limiter if the DefaultControllerRateLimiter can not satisfy us.
Actual Behavior
Steps to Reproduce the Problem

Additional Info

@lionelvillard


code health gmdifftime test

[x] tests are added or updated

Update gmdifftime to use the test infrastructure

Updates || positioning

Still looks good to me. My plus one reconfirmed. @levitte?

Pushed. Thanks.


Input in Chinese not tokenized correctly

Tokens are separated by space, punctuation, etc. For Chinese input (and maybe many other languages as well), an entire sentence will be considered a token. This affects most analyses.
We could select the correct tokenizer according to the language the user has chosen.
Also, as suggested by @neubig in today's meeting, we could change the input textboxes Input Lang and Output Lang to dropdown selection Input Format and Output Format to allow other formats such as images in the future.
See the comment here.

Sorry, @PaulCCCCCCH
maybe you have already shared the system output files with me previously, but could you share them with me again?

Hi @pfliu-nlp
Sorry for my late reply. I sent the output on slack. Is this what you are looking for, or do I need to pull out some other logs?

Thanks, @PaulCCCCCCH
Actually, could you share the two files you have uploaded to the explainaboard web so that I can test it locally from my side
(1) one is the test samples with true labels
(2) the other is the prediction files output by the model

Thanks, @PaulCCCCCCH Actually, could you share the two files you have uploaded to the explainaboard web so that I can test it locally from my side (1) one is the test samples with true labels (2) the other is the prediction files output by the model

I've sent the files on slack. Thanks!

@PaulCCCCCCH this is one simple solution (in case you want to use it locally)
https://github.com/neulab/ExplainaBoard/pull/483

Sure. I think I'll just turn it on locally for my current analysis task.
I think we can discuss more on how to support different tokenizers later. Thanks @pfliu-nlp

@pfliu-nlp : I think this issue is fixed by #483, right? If so could you close it?


Constructor for simple distributions with bijection applied

Frequently, we want to sample from a distribution that is a simple invertible transformation of an existing probability distribution, but because the distribution is not available, we either have to implement it (which is not conceptually complicated, but is verbose), or we have to reparametrize the random choices in our model.
For example a user may be forced to write:
k = @trace(geometric(0.5), :k_minus_one) + 1

whereas the user wanted to express (but could not):
k = @trace(geometric(0.5) + 1, :k)

Similarly for continuous random variables, a user is forced to write:
x = exp(@trace(normal(0, 1), :log_x))

when what they wanted was:
x = @trace(exp(normal(0, 1), :x)

The mismatch between the ideal semantic names (k, x in the examples above) and the addresses (the parametrization of the model) is not ideal. A concise syntax that handles the above cases, and other cases that are also easy to handle (statically) would be nice.
The same idea holds for generative functions more generally, but more research is required to capture the general case (and the simple case for distributions has already popped up).

@alex-lew When you get a chance, can you write your latest understanding of what we've been talking about regarding distribution combinators, and add it to this thread? I think there is a restricted version of it (independent of our ongoing research involving involutions and trace types) that could be implemented quickly, and would prove users with a lot of value. But the first step is writing up a more concrete technical plan.
It is also good to add this here, so we can discuss possible syntax changes that might be implied.

This was addressed by https://github.com/probcomp/Gen/pull/103


[crossbar.io w/ py3] builtins.TypeError: 'IdGenerator' object is not an iterator

whilst trying to persuade crossbar to run under py3, I encountered this issue:
  File "/usr/lib/python3.4/site-packages/crossbar-0.11.0-py3.4.egg/crossbar/router/dealer.py", line 391, in on_authorize_success
    invocation_request_id = next(self._request_id_gen)
builtins.TypeError: 'IdGenerator' object is not an iterator

this is fixable by adding the following to autobahn/util.py in class IdGenerator():
    def __next__(self):
        return self.next()

migrated to: https://github.com/crossbario/crossbar/issues/433


Search article

Description
Add search feature in article page and setup base CI/CD
Linked Issues
Close https://github.com/yehezkielgunawan/yehezgun-v3/issues/122
Close https://github.com/yehezkielgunawan/yehezgun-v3/issues/96
Preview or Screenshot

Use this https://www.npmjs.com/package/changelogithub
Just setup the Github Action for release purpose, because it's just for personal use.


fix: new offer submission QA fixes

During QA, we found the following issues:

If the user clicked "continue" without selecting an option, no error would be displayed.
Amounts with a decimal point would cause the mutation to fail.
The message for when the offer is too low should have been a variant="warning".
The whole ☝ message should be clickable.

Fixes are as follow:

If that edge case occurs, the "different amount" option will be automatically selected, displaying an error.
Amounts are now rounded to avoid decimals.
Corrected to use  variant="warning".
The message is now wrapped in a Clickable.

https://user-images.githubusercontent.com/7117670/151883048-a624299b-7697-4107-9f08-ccaf8f10f9ba.mov
cc @artsy/negotiate-devs

Umm the linter is correcting my tests in making them fail 🤔


Fix bad logging in DeadAgentReaper

Actually log the agent being reaped.

@davidxia @rohansingh @mattnworb @negz

:+1:

:+1:

Current coverage is 45.91%

Merging #837 into master will increase coverage by +5.12% as of 38378b1

@@            master    #837   diff @@
======================================
  Files          114     263    +149
  Stmts         4604   11157   +6553
  Branches       850    1556    +706
  Methods          0       0
======================================
+ Hit           1878    5123   +3245
- Partial        205     446    +241
- Missed        2521    5588   +3067

Review entire Coverage Diff as of 38378b1

Powered by Codecov. Updated on successful CI builds.


Nacos 注册中心服务列表显示异常！

[ ] I have searched the issues of this repository and believe that this is not a duplicate.
[ ] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.7.3
Operating System version:  mac
Java version: 1.8

Steps to reproduce this issue

服务提供者项目为A【里面接一个接口】，2个项目消费A，分别为 项目B 和项目C。
服务注册到nacos 将项目名没有放在服务名上，而是放在了元数据中。导致3个项目启动  只有2个服务。分别一个提供者、一个消费者【应该是2个消费者】。
2个消费者调用没有影响。列表bug

Pls. provide [GitHub address] to reproduce this issue.
Expected Result
What do you expected from the above steps？
Actual Result
What actually happens?
If there is an exception, please attach the exception trace:
Just put your stack trace here!

duplicate of #5426


etcd has various error messages when two out of Five hosts are powered off with a scale policy of max = 5

Rancher Version:: 1.1.0-rc5
OS and where are the hosts located? (cloud, bare metal, etc):: Ubuntu, Cloud
Environment Type: (Cattle/Kubernetes/Swarm/Mesos): Kubernetes
Steps to Reproduce:

Create a Kubernetes set up with 5 hosts (scale policy: max=5)
Create pods/rc
Power off two hosts
Kuberntes is active and functional. All pods are accessible.

The following error messages are seen in the etcd logs:
etcd2 logs:
/30/2016 3:10:31 PM2016-06-30 22:10:31.980058 W | etcdserver: cannot get the version of member 8c7228a6cac3369f (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: i/o timeout)
6/30/2016 3:10:36 PM2016-06-30 22:10:36.768536 W | etcdserver: failed to reach the peerURL(http://<IP_ADDRESS>:2380) of member a5d68226916cf3a (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: getsockopt: no route to host)
6/30/2016 3:10:36 PM2016-06-30 22:10:36.772923 W | etcdserver: cannot get the version of member a5d68226916cf3a (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: getsockopt: no route to host)
6/30/2016 3:10:37 PM2016-06-30 22:10:37.979192 W | etcdserver: failed to reach the peerURL(http://<IP_ADDRESS>:2380) of member 8c7228a6cac3369f (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: i/o timeout)
6/30/2016 3:10:37 PM2016-06-30 22:10:37.979289 W | etcdserver: cannot get the version of member 8c7228a6cac3369f (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: i/o timeout)
6/30/2016 3:10:42 PM2016-06-30 22:10:42.767946 W | etcdserver: failed to reach the peerURL(http://<IP_ADDRESS>:2380) of member a5d68226916cf3a (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: getsockopt: no route to host)
6/30/2016 3:10:42 PM2016-06-30 22:10:42.768483 W | etcdserver: cannot get the version of member a5d68226916cf3a (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: getsockopt: no route to host)
6/30/2016 3:10:43 PM2016-06-30 22:10:43.975029 W | etcdserver: failed to reach the peerURL(http://<IP_ADDRESS>:2380) of member 8c7228a6cac3369f (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: i/o timeout)
6/30/2016 3:10:43 PM2016-06-30 22:10:43.975138 W | etcdserver: cannot get the version of member 8c7228a6cac3369f (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: i/o timeout)
6/30/2016 3:10:48 PM2016-06-30 22:10:48.768482 W | etcdserver: failed to reach the peerURL(http://<IP_ADDRESS>:2380) of member a5d68226916cf3a (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: getsockopt: no route to host)
6/30/2016 3:10:48 PM2016-06-30 22:10:48.768602 W | etcdserver: cannot get the version of member a5d68226916cf3a (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: getsockopt: no route to host)
6/30/2016 3:10:49 PM2016-06-30 22:10:49.977031 W | etcdserver: failed to reach the peerURL(http://<IP_ADDRESS>:2380) of member 8c7228a6cac3369f (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: i/o timeout)
6/30/2016 3:10:49 PM2016-06-30 22:10:49.978175 W | etcdserver: cannot get the version of member 8c7228a6cac3369f (Get http://<IP_ADDRESS>:2380/version: dial tcp <IP_ADDRESS>:2380: i/o timeout)
6/30/2016 3:10:50 PM2016-06-30 22:10:50.383507 W | rafthttp: the connection to peer a5d68226916cf3a is unhealthy

etcd4 logs
/30/2016 3:09:01 PM2016-06-30 22:09:01.921491 W | rafthttp: the connection to peer 8c7228a6cac3369f is unhealthy
6/30/2016 3:09:14 PM2016-06-30 22:09:14.542657 W | rafthttp: the connection to peer a5d68226916cf3a is unhealthy
6/30/2016 3:09:31 PM2016-06-30 22:09:31.921692 W | rafthttp: the connection to peer 8c7228a6cac3369f is unhealthy
6/30/2016 3:09:44 PM2016-06-30 22:09:44.542883 W | rafthttp: the connection to peer a5d68226916cf3a is unhealthy
6/30/2016 3:10:01 PM2016-06-30 22:10:01.921938 W | rafthttp: the connection to peer 8c7228a6cac3369f is unhealthy
6/30/2016 3:10:14 PM2016-06-30 22:10:14.543314 W | rafthttp: the connection to peer a5d68226916cf3a is unhealthy
6/30/2016 3:10:31 PM2016-06-30 22:10:31.922315 W | rafthttp: the connection to peer 8c7228a6cac3369f is unhealthy
6/30/2016 3:10:44 PM2016-06-30 22:10:44.543559 W | rafthttp: the connection to peer a5d68226916cf3a is unhealthy
6/30/2016 3:11:01 PM2016-06-30 22:11:01.922543 W | rafthttp: the connection to peer 8c7228a6cac3369f is unhealthy
6/30/2016 3:11:14 PM2016-06-30 22:11:14.543944 W | rafthttp: the connection to peer a5d68226916cf3a is unhealthy
6/30/2016 3:11:31 PM2016-06-30 22:11:31.922759 W | rafthttp: the connection to peer 8c7228a6cac3369f is unhealthy

These logs indicate the 2 nodes that failed (were on the hosts you shut off) have failed. It is expected functionality of etcd.


[Menu] - text menus receive @attachedBorder

Steps to Reproduce
Create a text menu that is top attached.
Expected
The menu should collapse its bottom margin to appear closely attached to the next element.
Result
The Menu does this, but also adds borders.
Testcase
http://jsfiddle.net/w5nuycoe/1/

Fix
The tabular menu is excluded from @attachedBorders, the text menu should also be excluded:
https://github.com/Semantic-Org/Semantic-UI/blob/master/src/definitions/collections/menu.less#L1885

Bump.
@jlukic any chance this could go out next release, that'd be superb.  It is putting a small damper on the SUIR Datetime component.

Fixed in the community fork Fomantic-UI by https://github.com/fomantic/Fomantic-UI/pull/2084


Copy function doesn't seem to work

I need to be able to drag-copy from a listbox and drop into another listbox and the code seems to imply that this should be possible but it either doesn't work or I'm misunderstanding how it should work.
Using the "Bound ListBoxes" example, the lower ListBox indicates that holding the Control key should result in a drag-copy operation but it behaves exactly the same regardless of the state of the Control key - always moving from one list to another.
It looks like the moveData boolean expression in DefaultDropHandler.Drop is just wrong. It also appears that the drag-copy behavior is incompletely implemented in general. I made some changes that made the drag-copy operation behave a little more like I would expect but, when I dragged the copied item back to the source listbox, it duplicated it (I suppose that's a possible interpretation of what should happen but not at all intuitive).

@MrBigDog2U I'll fix this in the default drop handler by doing this
        var copyData = (dropInfo.DragInfo.DragDropCopyKeyState != default(DragDropKeyStates)) && dropInfo.KeyStates.HasFlag(dropInfo.DragInfo.DragDropCopyKeyState)
                       //&& (dropInfo.DragInfo.VisualSource != dropInfo.VisualTarget)
                       && !(dropInfo.DragInfo.SourceItem is HeaderedContentControl)
                       && !(dropInfo.DragInfo.SourceItem is HeaderedItemsControl)
                       && !(dropInfo.DragInfo.SourceItem is ListBoxItem);

for now, I don't know if we should prevent the copy action if target = source

I don't think that's right either. From this condition, if the SourceItem
is a ListBoxItem (or any of the other specified controls), then copyData
will be false regardless of the KeyStates. This means that you can never
copy a ListBoxItem.
My own expectations of what to do if (target == source) is that it would
never be a copy although I just checked Windows Explorer and, if you
ctrl-drag an item within a list in the right hand pane, it DOES do a copy.
So I don't know if preventing the copy on that condition is right either.
On Wed, Feb 3, 2016 at 8:01 AM, Jan Karger<EMAIL_ADDRESS>wrote:

@MrBigDog2U https://github.com/MrBigDog2U I'll fix this in the default
drop handler by doing this
    var copyData = (dropInfo.DragInfo.DragDropCopyKeyState != default(DragDropKeyStates)) && dropInfo.KeyStates.HasFlag(dropInfo.DragInfo.DragDropCopyKeyState)
                   //&& (dropInfo.DragInfo.VisualSource != dropInfo.VisualTarget)
                   && !(dropInfo.DragInfo.SourceItem is HeaderedContentControl)
                   && !(dropInfo.DragInfo.SourceItem is HeaderedItemsControl)
                   && !(dropInfo.DragInfo.SourceItem is ListBoxItem);

for now, I don't know if we should prevent the copy action if target =
source
—
Reply to this email directly or view it on GitHub
https://github.com/punker76/gong-wpf-dragdrop/issues/162#issuecomment-179311975
.

the SourceItem is different to VisualSourceItem. this is directly ListBoxItem or TreeVieItem etc if you have an ItemsControl without ItemsSource binding. so copy this is limited for now, cause it's not an easy way to do this.
Von meinem iPhone gesendet

Am 03.02.2016 um 17:17 schrieb MrBigDog2U<EMAIL_ADDRESS>I don't think that's right either. From this condition, if the SourceItem
is a ListBoxItem (or any of the other specified controls), then copyData
will be false regardless of the KeyStates. This means that you can never
copy a ListBoxItem.
My own expectations of what to do if (target == source) is that it would
never be a copy although I just checked Windows Explorer and, if you
ctrl-drag an item within a list in the right hand pane, it DOES do a copy.
So I don't know if preventing the copy on that condition is right either.
On Wed, Feb 3, 2016 at 8:01 AM, Jan Karger<EMAIL_ADDRESS>wrote:

@MrBigDog2U https://github.com/MrBigDog2U I'll fix this in the default
drop handler by doing this
var copyData = (dropInfo.DragInfo.DragDropCopyKeyState != default(DragDropKeyStates)) && dropInfo.KeyStates.HasFlag(dropInfo.DragInfo.DragDropCopyKeyState)
//&& (dropInfo.DragInfo.VisualSource != dropInfo.VisualTarget)
&& !(dropInfo.DragInfo.SourceItem is HeaderedContentControl)
&& !(dropInfo.DragInfo.SourceItem is HeaderedItemsControl)
&& !(dropInfo.DragInfo.SourceItem is ListBoxItem);
for now, I don't know if we should prevent the copy action if target =
source
—
Reply to this email directly or view it on GitHub
https://github.com/punker76/gong-wpf-dragdrop/issues/162#issuecomment-179311975
.

—
Reply to this email directly or view it on GitHub.


ValueError: could not broadcast input array from shape (118,128,3) into shape (128,128,3)

Hello, Wei, when I am training the agent in the V-rep simulator, there will always occur a ValueError of "inconsistent shape (x, 128, 3) with (128, 128, 3)". Do you have any idea with this issue? Part of the log is as follows:
-- Maximum Location at [210 227]
-- Metric for current frame: 0.734154
With peak_dis: 0.173634, flatten: 0.820850, max_value: 0.924716
13478it [15:28,  9.80s/it]
-- Push from [152. 198.] to [106.74516600406096,<PHONE_NUMBER>9593904]
-- Maximum Location at [180 340]
-- Metric for current frame: 0.769855
With peak_dis: 0.264738, flatten: 0.859865, max_value: 0.852448
13479it [15:38,  9.88s/it]
-- Push from [193. 311.] to [257.0, 311.0]
-- Maximum Location at [175 349]
-- Metric for current frame: 0.880095
With peak_dis: 1.000000, flatten: 0.890577, max_value: 0.785750
[!!] Nearest distance: 0.084278
[*] Update the scene 16 with 3 object removed
-- Maximum Location at [175 349]
-- Metric for current frame: 0.880095
With peak_dis: 1.000000, flatten: 0.890577, max_value: 0.785750
13480it [15:57, 12.57s/it]
-- Push from [146. 398.] to [210.0, 398.0]
-- Maximum Location at [370 335]
Traceback (most recent call last):
File "main.py", line 88, in
tf.app.run()
File "/home/ys/anaconda3/envs/active-perception/lib/python3.6/site-packages/tensorflow/python/platform/app.py", line 48, in run
_sys.exit(main(_sys.argv[:1] + flags_passthrough))
File "main.py", line 72, in main
agent.train()
File "/home/ys/Active-Perception/DQN/dqn/agent_18.py", line 66, in train
screen, reward, terminal = self.env.act(action, if_train=True)
File "/home/ys/Active-Perception/DQN/simulation/environment.py", line 517, in act
self.screen, self.local_afford_new, self.location_2d = self.camera.local_patch(self.index, (self.screen_height, self.screen_width))
File "/home/ys/Active-Perception/DQN/simulation/environment.py", line 222, in local_patch
return get_patch(location_2d, self.cur_color, self.cur_depth, post_afford, patch_size)
File "/home/ys/Active-Perception/DQN/util/utils.py", line 44, in get_patch
patch_rgbd[:,:,0:3] = patch_color
ValueError: could not broadcast input array from shape (118,128,3) into shape (128,128,3)

The problem is caused by this:

First, the input of our model is a cropped image (128 X 128) around highest point in the Affordance Map, which has size of 424 X 512.
So if the point is near the boundary of the Affordance map, it will cause the cropped image fail to fit 128 X 128.
In your case, the maximium point is [370, 335], so in one side, there's only 424-370=54 left, which is added with other side 64 pixel to be 118.

Solution
Our solution for this situation is to add boundary around the Affordance Map, i.e. ref to this line, it is the boundary of the collecting box in the simulation. And I postprocess the Affordance Map to set point value out of the boundary to ** be zero **, so the highest point won't be too close to the boundary. You can shrink it to a safer size, like [64, 424-64, 64, 512-64]
I know it's a naive solution, but I don't think the way to pad it with 1 or 0 will be better.

Thanks for your help. I have followed your suggestion and the issue has been resolved.


Terminal text select to be included

Can Terminal text select to be included as part of the functionality please ? thanks

Functionality is already part of VSCode


viewing an enum instance in the variable explorer

Description
What steps will reproduce the problem?

I defined a class, which inherits from the Enum class in package enum, with only class objects defined, as one would for an enum.
During debugging of my code I wanted to view an object of this class.
The type was listed as 'None' in the variable explorer, though the value indicated it was on object of the enum class.
On double-clicking, this error was produced.
Traceback
  File "d:\program files\python3\lib\site-packages\spyder\widgets\variableexplorer\collectionseditor.py", line 544, in createEditor
    readonly=readonly)
  File "d:\program files\python3\lib\site-packages\spyder\widgets\variableexplorer\collectionseditor.py", line 1356, in setup
    remote=remote)
  File "d:\program files\python3\lib\site-packages\spyder\widgets\variableexplorer\collectionseditor.py", line 1302, in __init__
    title)
  File "d:\program files\python3\lib\site-packages\spyder\widgets\variableexplorer\collectionseditor.py", line 1179, in __init__
    minmax=minmax)
  File "d:\program files\python3\lib\site-packages\spyder\widgets\variableexplorer\collectionseditor.py", line 135, in __init__
    self.set_data(data)
  File "d:\program files\python3\lib\site-packages\spyder\widgets\variableexplorer\collectionseditor.py", line 175, in set_data
    self.title += data_type
TypeError: Can't convert 'NoneType' object to str implicitly

Versions

Spyder version: 3.3.1
Python version: 3.5.1
Qt version: 5.9.1
PyQt5 version: 5.9
Operating System: Windows 10

Dependencies
pyflakes >=0.6.0  :  1.6.0 (OK)
pycodestyle >=2.3 :  2.3.1 (OK)
pygments >=2.0    :  2.2.0 (OK)
sphinx >=0.6.6    :  1.5.3 (OK)
rope >=0.9.4      :  0.10.7 (OK)
jedi >=0.9.0      :  0.11.0 (OK)
nbconvert >=4.0   :  5.1.1 (OK)
pandas >=0.13.1   :  0.19.2 (OK)
numpy >=1.7       :  1.14.5 (OK)
sympy >=0.7.3     :  None (NOK)
cython >=0.21     :  None (NOK)
qtconsole >=4.2.0 :  4.3.1 (OK)
IPython >=4.0     :  6.2.1 (OK)
matplotlib >=2.0.0:  1.5.1 (NOK)
pylint >=0.25     :  1.8.2 (OK)

Thanks for reporting. We'll take a look at this in a future release.

Closing as a duplicate of #8169.