Implements a reverse_map() method for JordanWignerMapper().

Summary
This implements a reverse_map() method in the JordanWignerMapper(). It allows users to go back from qubit operators into fermionic operators. Completes issue #255.
Also included a unit test.
Details and comments
Instead of using the implementation linked in issue #255 I modified it to use the FermionicOp object for the mapping. Seemed like it would be easier for anyone to understand the code. Both implementations are equivalent.

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

I see there are failures which are not of this change but due to a newly released pylint version - I created #1346 as this needs fixing outside of this PR to get CI running again and passing on the code that is already in main here.

Sure, I'll fix the mypy error by pushing the complex inside FermionicOp.

Included the pylint fixes in this PR. Hope that's'okay.

I'll let this run so we can check things anyway if you like. (Once you have a PR that has been merged here it will not longer need a maintainers approval to run - its just on the first PR)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

38 of 40 (95.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 86.785%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_nature/second_q/mappers/jordan_wigner_mapper.py
38
40
95.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
8820

Relevant Lines:
10163

💛  - Coveralls

Sorry my bad in the suggestion, if you did not correct it was

:class:``JordanWignerMapper`

should have been

:class:`.JordanWignerMapper` where it was a . not a second `

@woodsp-ibm Let me know if you think the note is okay when you can.

@mrossinek Can I get you to take a look at this too.

Hey, just wanted an update here. Is there anything I should do about this still?

The build errors with mypy are fixed by #1362, which is still open. As far as progressing this hopefully @mrossinek can take a look.


Add disableGlobalErrorsHandling option

Feature for not initializing global errors handling
This options still allow you send events manually

Describe the reason of this feature

@neSpecc you may need to use this "catcher" for sending events but not for catching errors.


Wip slayer plugin kill count fix

Attempts to accurately count slayer kills for the slayer plugin by rather than assuming 1 xp drop = 1 kill, it instead takes the slayer xp drops possible by each monster killed on the tick the xp dropped and then determines which of those monsters most likely died to produce that total xp drop.
Reasons for WIP:

Need death animation ids for the following npcs [DUSK_7888, DUSK_7889, GIANT_ROCKSLUG, SMALL_LIZARD, DESERT_LIZARD, ANCIENT_ZYGOMITE]
Need to add more tests of the kill estimation
Need to do in game test of death animation check for the npcs that die at hp > 0
like to have - make partial xp drops factored in

Attemps to for the most part fix (https://github.com/runelite/runelite/issues/1046).
Will likely also fix cannoning on slayer tasks.
Both need to be tested. I cannot personally test either due to a lack of osrs levels/stuff.

All data was grabbed using (https://github.com/daviscook477/osrs-data/blob/scrape-monster-data/src/main/java/net/runelite/data/dump/wiki/SlayerXpDumper.java)

-1 indicates no Slayer xp is earned from that creature. Format of the list is slayerxp, combat level so those monsters you highlighted are indicated to give no Slayer xp.
I could run the list through a quick bit of code to remove all monsters for which the Slayer xp is -1. Is that what you're recommending?

cleaned up proposed code for this plugin by:

refactoring functionality (namely knapsack solver and determining xp drop for an npc) outside of the slayerplugin class into own classes
adding some null checks and additional comments
tracking monster death animations by name rather than id so it's less brittle

so this bug fix isn't exactly working. I've tried testing this a few times and continually run into the problem of not realistically being able to determine what tick an npc died on. Basically in order for this to work I have to be able to figure out what xp drops are obtained on each tick and what npcs die on that tick too. If that information could be determined then the idea of using a knapsack solver on the xp drops that each npc could give works pretty well (as seen with the unit tests I added for this). Unfortunately in testing, I can't correlate the xp drops and npc deaths accurately. For the most part a slayer xp drop is given the same tick that the npc starts its death animation. Using this gives fairly accurate results but every few minutes of slayer some death animation/xp drop doesn't line up and results in the kill tracker getting confused.
If anyone has a different idea on how to determine which npcs have died when an xp drop comes I'd really appreciate the ideas since I do want to get this pr working.
My current idea on how to make this work is to cut the idea of tracking npc deaths completely from this pr. Instead I would integrate (https://github.com/runelite/runelite/pull/6746) into this pr and then any time an xp drop is given rather than trying to compute the kills from the actual npcs that died I could compute kills by just looking at all the npcs that could be killed on task. So rather than solving bounded knapsack problem on the actual npcs that died, you just do unbounded knapsack problem on the npcs that could have died. This would definitely be less accurate than a working version of my current solution but it wouldn't require having to time the xp drops to the npc deaths. The only main issue I see with this is that without tracking npc deaths filtering out xp drops from xp lamps and stuff like that will really mess up the counter. Since the current counter would just treat those drops as 1 kill it wouldn't be too messed up but with this one trying to say figure out how many blue dragons make up a 670xp lamp drop, it would miss by maybe 5 kills or something. This could mostly be avoided by checking to see if the player killed any of the slayer npcs recently but again this gets into the problem of trying to track kills.
Another potential solution would simply be to track all the npc deaths within some rolling window of time and rather than trying to actually pair those deaths with the xp drops simply using those npcs (that are on the list of actually killable npcs for some task) to determine the number of kills.  Then we can just solve unbounded knapsack problem using that fire giant npc. This would mostly avoid quest/lamp xp drops because we could see that no npcs had been killed prior to an xp drop coming in. The only main issue I see with this result is that it might be slightly less accurate than actually knowing which npcs died on which tick (which I'm starting to think is impossible) and that for the first kill of a task there wouldn't be any npcs in this death log so we couldn't figure out what npcs might have given the xp drop. This could probably be avoided by simply delaying processing each xp drop for like a few ticks (maybe 6 ticks that's like 3 seconds) so that the despawn log could be filled up with some npcs.
So the basic proposal would be this:

keep track of which npcs that have despawned that are on task (by using https://github.com/runelite/runelite/pull/6746) in a window of ~20 ticks.
when a slayer xp drop comes in wait 6 ticks and then solve unbounded knapsack problem on the npc types that are in the despawn log (the 20 tick window basically giving like 14 ticks before the xp drop and 6 ticks after the xp drop)
whatever the result of that is is the actual amount of kills that caused that xp drop.
So if we see a 222xp drop on a fire giants task and the player in fact has been killing fire giants this will say 2 kills were made - similar idea for if they were bursting dust devils

The only trouble with this is I already tried a rolling window of kills and it did not work sufficiently accurately enough. Sometimes kills just wouldn't get registered in time. One way to avoid this would be to use a hybrid of kills and currently loaded npcs. Basically this means we would try to see what xp drops out of those from killed monsters and currently loaded npcs that could make up the slayer xp drop. We can't rely solely on the loaded npcs since the player realistically could kill all the npcs of one type in a multicombat zone with bursting. The only problematic thing for this type of solution is something like desert lizards. The 3 different cb levels of lizards give 15, 25 and 40 xp respectively. A 40 xp drop has no way to differentiate between being a 40 lizard or a 15 and a 25 lizard when we aren't tracking the kills.


Issue with Aggregator in Ocelot: Requires At Least Two Routes to Execute

Expected Behavior / New Feature
Aggregators in Ocelot should execute correctly when configured with a single route, allowing modifications to the response as per the aggregator's logic.
Actual Behavior / Motivation for New Feature
The aggregator does not execute when it is configured with only one route. Instead, the request seems to be handled by another route with a wildcard, and the response from the microservice is returned without any modifications by the aggregator.
Steps to Reproduce the Problem

Configure a route with a wildcard in ocelot.json.
Configure an aggregator with only one route in ocelot.json.
Make a request to the aggregator's route.
Observe that the aggregator does not execute.

Specifications

Version: Ocelot 22.0.1
Platform: .NET 6
Subsystem: N/A

{
  "Routes": [
    {
      "SwaggerKey": "administrador",
      "UpstreamPathTemplate": "/api/administrador/v1/{everything}",
      "UpstreamHttpMethod": [
        "GET",
        "POST",
        "PUT",
        "DELETE"
      ],
      "DownstreamHostAndPorts": [
        {
          "Host": "",
          "Port": 0
        }
      ],
      "DownstreamPathTemplate": "/api/{everything}"
    }
  ]
}

{
  "Aggregates": [
    {
      "UpstreamPathTemplate": "/api/mix/v1/IniciarSesion",
      "RouteKeys": [
        "loginJWT"
      ],
      "Aggregator": "LoginAggregator",
      "Description": "Crea el JWT luego de la respuesta de login de administrador."
    }
  ],
  "Routes": [
    {
      "UpstreamPathTemplate": "/api/administrador/v1/Usuario/login",
      "UpstreamHttpMethod": [
        "GET",
        "POST",
        "PUT",
        "DELETE"
      ],
      "DownstreamHostAndPorts": [
        {
          "Host": "",
          "Port": 0
        }
      ],
      "DownstreamPathTemplate": "/api/Usuario/login",
      "Key": "loginJWT",
      "SwaggerKey": "loginJWT"
    }
  ]
}

Hi!
I needn't to develop aggregator classes.
Use Delegating Handlers feature!
But current restriction to have at least 2 routes is quite natural thing when combining 2 sources of data.
But, make sense to remove this restriction.
When developing aggregator you work with DownstreamResponse object.
When developing delegating handler you work with  HttpRequestMessage and HttpResponseMessage objects.
But currently, you need to develop custom Http handler which allows you to transform content both of request and response, HttpRequestMessage and HttpResponseMessage objects.
In your user scenario, you want to convert data of the response, thus you have to work with HttpResponseMessage object!
Hope it helps!

@ggnaegi  Oh, no!
We forgot to check this user scenario before the release 😏

@ggnaegi  Oh, no!
We forgot to check this user scenario before the release 😏

Ouch...

@RaynaldM @ggnaegi If not joking... Do we accept it?
His feature request is processing single route by Aggregation (multiplexer) middleware.
So, he wants to process single route by IDefinedAggregator class to convert/analyze data of the response and return converted result to upstream finally.
Current restriction by validators (I guess) is having at least 2 routes with aggregation keys to define an aggregator in Aggregates section.
To be fair, as I recommended, such data conversions can be performed by custom delegating handler.
Is this sufficient argument to start development for multiplexer?
And, seems the author doesn't care about his issue anymore; it could be closed after our decision.

@camilocalderont Dear author,
Where are you?

if @camilocalderont wants to do it himself, why not?

Hi, @RaynaldM, @raman-m, I greatly appreciate your attention to my case. I'm a big fan of Ocelot and use it daily as an orchestrator service and even as an entry point.
Your alternative is very good, but in my case, I use a class to organize and dynamically change the host and port. Therefore, it was better for me to use Aggregators instead of Controllers, as it would duplicate an endpoint that is already in another .Net project being orchestrated. Thus, I had to use an Aggregator and define 2 routes, but only use the first one.
It would be nice to not have that restriction.

@camilocalderont

Thus, I had to use an Aggregator and define 2 routes, but only use the first one.

Sounds like a life hack! 🤣 You are true Ocelot fan! 😉
Imagine we will disable 2 routes validator for Aggregation. What's the goal? Just remove fake route from JSON?
My vote weight is 0.5. I neither refuse nor accept at this time.
@ggnaegi  Your opinion?
Voting

@RaynaldM  👎 ➖
@raman-m   ❔ ➕➖
@ggnaegi     ❔

@raman-m @camilocalderont @RaynaldM  By design, if only one route, then when don't make any further checks and process the downstream call. It's a bit straight forward, but that's the way it is at the moment. The problem is probably that the Aggregator has been placed at the wrong spot in the multiplexer and we are mixing concerns. From my side I would prefer moving the Aggregator somewhere else in the pipeline, renaming it, and calling it ResponseTransform instead.

@ggnaegi Your vote is negative, right? ➖
Regarding middleware name... It doesn't matter for me, and I don't see a benefit in renaming.
Regarding a position in the pipeline... It makes sense to find appropriate and right position 👍

@camilocalderont
Dear Camilo!
Sorry, but we didn't accept the issue for development!
There are 2 reasons for that

In version 23.1 we enhanced Multiplexer (see release notes). And with one route Aggregation logic simply skips Aggregation main block and behaves as standard route. This change was introduced because of performance. So, your IDefinedAggregator will not work! It will not be called!
I guess your current life hack with 2nd fake route is acceptable solution. Fake route should return empty data or small data block which you will ignore for sure when parsing JSON data.

We will be glad to review your open PRs in future!
Stay with Ocelot! 🐯


Pianoroll from parts bug

fixes bug in compute_pianorolls where the ordering of durations was different from the ordering of onsets.

Codecov Report

Merging #112 (4c07137) into develop (7ca1894) will increase coverage by 0.28%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #112      +/-   ##
===========================================
+ Coverage    77.95%   78.24%   +0.28%
===========================================
  Files           50       52       +2
  Lines         8583     8700     +117
===========================================
+ Hits          6691     6807     +116
- Misses        1892     1893       +1

Impacted Files
Coverage Δ

tests/__init__.py
100.00% <ø> (ø)

partitura/utils/music.py
61.14% <100.00%> (+3.34%)
:arrow_up:

tests/test_pianoroll.py
100.00% <100.00%> (ø)

partitura/__init__.py
100.00% <0.00%> (ø)

partitura/io/exportparangonada.py
74.35% <0.00%> (ø)

tests/test_parangonada.py
96.15% <0.00%> (ø)

Continue to review full report at Codecov.

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

Nice catch! All seems to work for me. I'm merging


Could this sdk be used on Windows?

The docker runs very well on Linux. Thanks a lot~~
Could it run on Windows?
I installed docker software on Windows and pulled down the sgx-rust image.
I run into trouble when trying to run the image, as I don't know how to map the sgx device into the docker.
After I enabled SGX in BIOS and installed the SGX PSW, I can see the device in 'device manager' as follows.

Would you please give some advice?
Any advice is always welcome~~

The device shown in the figure is TPM, not Intel SGX.
I don't have a plan for Windows support -- but I believe it's do-able.

Thanks.


Css mirroring monam

Added feature to implement rtl mirroring.

Note to any other reviewer: please only have a look at files with mirroring in their path (most of the others are merge / rebase artefacts and will be dropped shortly)

Please also fix your email to calm the CLA bot (rebase + edit commit + maybe set again your user.email)

Thanks Monam!


xoffset works in .opts but not as kwarg.

I'm using hvplot.labels. The options for this seem quite finicky and under documented.
ALL software version info
hvplot '0.7.0'
holoviews '1.14.1'
Description of expected behavior and the observed behavior
This does not work:
df.hvplot.labels(x='Month', y='value', text='label', text_font_size='6pt', text_color='blue', xoffset=20)

This works:
df.hvplot.labels(x='Month', y='value', text='label', text_font_size='6pt', text_color='blue').opts(xoffset=20)

Also, in terms of changing the size and color of the labels, no amount of kwargs or opts seemed to work until I found this page and started passing the bokeh option names directly https://docs.bokeh.org/en/latest/docs/user_guide/styling.html#creating-custom-themes
Also, why does hvplot change the bokeh x_offset to xoffset? Wouldn't there be higher discoverability if the option names remained the same across the ecosystem? Maybe this is a silly question.
I guess my main point is that his page is under-documented: https://hvplot.holoviz.org/reference/pandas/labels.html

I can reproduce the original issue.

Also, why does hvplot change the bokeh x_offset to xoffset? Wouldn't there be higher discoverability if the option names remained the same across the ecosystem? Maybe this is a silly question.

I think actually HoloViews/xoffset and Bokeh/x_offset are expressed in different units (coordinates vs. screen units) so it's probably better they have different names!


Freeze the habit

As a Logged User I want to pause the habit so I can frezze that.
Acceptance criteria

User can freeze the habit on 5, 10, 15 or 20 days.
If user freeze the habit the counter decrease in one every 5 days.
After  unfreeze the habit user should be able to continue track the habit.

Not relevant


Implementation of spot checks [DO NOT MERGE]

This PR will stay open until the spot checks epic is ready to be merged to master.

 The first lines are short enough, but then it complains for the longer description, which I guess shouldn't be limited.

OK, I get it, it's the wrapping... I'll try to be more diligent next time, sorry.


graphics: Use an internal enlarged image for a too-small image

A too-small image like 1x1 tends to require a big enlargement factor when scaling, but such scaling cannot be enlarged as intended:
https://user-images.githubusercontent.com/182612/61988369-4c4ae500-b05b-11e9-8813-35c4f1a3a311.png
For example, on the above image, the gray shadow part cannot cover the entire screen and the edges are exposed.
From my experience, the threshold sounds like x256 or so. I can check this by unit tests.
I think this can be solved by making a bigger image internally and use it with an adjusted scale.

This can be implemented as a negative level of mipmap, maybe?

See #611


docs(guide/Filters): describe your change...

No comma necessary or helpful before a conjunction.

Will merge if you sign the CLA


Upgrade the references Microsoft.Rest.ClientRuntime.Azure.TestFramework

Need update ActiveDirectory here:
https://github.com/Azure/azure-sdk-for-net/blob/f4c8a9e67d3fd323fcc72270b92871128ae5c288/sdk/mgmtcommon/TestFramework/ClientRuntime.Azure.TestFramework/Microsoft.Rest.ClientRuntime.Azure.TestFramework.csproj#L43
https://github.com/Azure/azure-sdk-for-net/issues/6458 depends on this fix. Thanks.

it is similar to https://github.com/Azure/azure-sdk-for-net/issues/7006.

Given the age of this issue and the deprecation of most of the track one libraries, closing this as stale.  Please feel free to reopen if this is still needed.


fix a typo

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?
[x] Have you enabled GitHub Pages Previews by pressing Enable GitHub Pages on the right sidebar to trigger a pages build for this PR?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

A typo “只是" now is been fixed as "知识".

Thanks so much for your contribution!  You rock, @codinggirl!


[DV] Common coverage holes

Description
I'm working on addressing these 2 common coverage holes, which exist in almost all blocks. Please ignore them for now.

[x] toggle coverage in prim_secdec_*

[ ] condition coverage at this line in *reg_top.u_chk
assign err_o = tl_i.a_valid & (|err | (|data_err));

[ ] condition coverage at this line in *reg_top.u_reg_if
assign err_internal = addr_align_err | malformed_meta_err | tl_err | instr_error | intg_error;

@jdonjdon @cindychip @matutem @sriyerg

Moving V3 tasks to next iteration of earlgrey.


Signing with unknown attribute crashes app

To reproduce:
Construct a contract with an attribute which the user does not have
Scan the QR
The app crashes without error.
Details:
OS: iOS
App version: 5.5.3 (35)

Sorry for the late response.
Which attribute did you use to test this? I can't seem to reproduce this on both the version you mention, nor on the current version of the app.

To be honest, I don't really know anymore. If you've tested it with unknown attributes in a signing session like you said, than it is probably fixed, or my bug report is invalid.
If you like you can close this one.

What do you mean with unknown? I have tested it with attributes that the app was aware of of existing (i.e. attributes that were in the apps scheme), but just didn't have an instance of. Do you mean you changed something to the scheme only on the irma server producing the request? So that the attribute did not occur in the scheme as known by the app?

The problem was, I think, that before condiscon was released:

the irma server did not properly check if the requested attributes exist in the scheme, it just passed them on to the app;
the app would crash on such nonexisting attributes.

Both of them now check existence of the attributes in the scheme before doing anything, so this should be fixed now.


fix(warp): remove warp-contracts and all implementations

BREAKING CHANGE: this removes all smartweave implementations using warp-sdk. The result is an only AO compatibile ANT and IO network contracts. Some utitlies are preserved due to their usefulness

:tada: This PR is included in version 2.0.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix Camera2D drag margin property is affected by zoom property #18912

Fixes https://github.com/godotengine/godot/issues/18912

PR #18916 was merged, could you rebase to remove commit a2a2ba1?

@akien-mga commit a2a2ba1 was reverted :)

Thanks!


Split elemental-operator and elemental-register at OBS level

This splits the elemental-operator OBS package in two different OBS src packages. One for the server side and another one for the client side.
This is a PoC around rancher/elemental-obs#26

Build example https://build.opensuse.org/project/show/home:dcassany:branches:isv:Rancher:Elemental:Dev


Issue/5816 inmanta module release output release tag

Description
When the inmanta module release command is executed, it doesn't produce any output. That way you have to execute a git log to figure out which Git tag was created. This information is required to push the tag later on using the git push origin  command. Now the release command will output the tag it created.
closes #5816
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Processing this pull request

Merged into branches master in 0002151f40d47c3fb50f93e8f6342f76ae78882a


[improve][doc] ConsumerBuilder.java comments

Motivation
Improve the comments in ConsumerBuilder.java for clarity and cleanliness.
These can be the first documentation devs see in an IDE, so they should inspire more confidence in users.
Modifications
Cleaned up the comments in the ConsumerBuilder.java file.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] Anything that affects deployment

Documentation

[x] doc
[ ] doc-required
[ ] doc-not-needed
[ ] doc-complete

[INFO] There are 3 errors reported by Checkstyle 8.37 with /home/runner/work/pulsar/pulsar/buildtools/src/main/resources/pulsar/checkstyle.xml ruleset.
Error:  src/main/java/org/apache/pulsar/client/api/ConsumerBuilder.java:[358] (sizes) LineLength: Line is longer than 120 characters (found 123).
Error:  src/main/java/org/apache/pulsar/client/api/ConsumerBuilder.java:[744] (sizes) LineLength: Line is longer than 120 characters (found 131).
Error:  src/main/java/org/apache/pulsar/client/api/ConsumerBuilder.java:[755] (sizes) LineLength: Line is longer than 120 characters (found 131).

Please fix CI failure @mendonk.

Codecov Report

Merging #18706 (25ed28b) into master (bbc7a25) will decrease coverage by 3.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #18706      +/-   ##
============================================
- Coverage     48.34%   45.27%   -3.08%
- Complexity     9368     9817     +449
============================================
  Files           618      677      +59
  Lines         58586    66562    +7976
  Branches       6097     7130    +1033
============================================
+ Hits          28326    30134    +1808
- Misses        27078    32912    +5834
- Partials       3182     3516     +334

Flag
Coverage Δ

unittests
45.27% <ø> (-3.08%)
:arrow_down:

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

Impacted Files
Coverage Δ

...g/apache/pulsar/broker/service/StreamingStats.java
0.00% <0.00%> (-83.79%)
:arrow_down:

...sar/broker/stats/metrics/ManagedLedgerMetrics.java
23.88% <0.00%> (-76.12%)
:arrow_down:

...lsar/broker/service/StickyKeyConsumerSelector.java
50.00% <0.00%> (-50.00%)
:arrow_down:

...ersistentStickyKeyDispatcherMultipleConsumers.java
0.00% <0.00%> (-50.00%)
:arrow_down:

...lsar/broker/service/InMemoryRedeliveryTracker.java
58.82% <0.00%> (-29.42%)
:arrow_down:

...lsar/broker/loadbalance/impl/ThresholdShedder.java
3.27% <0.00%> (-27.87%)
:arrow_down:

...ersistentStickyKeyDispatcherMultipleConsumers.java
18.31% <0.00%> (-24.76%)
:arrow_down:

...ker/loadbalance/impl/LeastLongTermMessageRate.java
73.33% <0.00%> (-20.00%)
:arrow_down:

.../org/apache/pulsar/broker/lookup/LookupResult.java
56.66% <0.00%> (-20.00%)
:arrow_down:

...e/HashRangeAutoSplitStickyKeyConsumerSelector.java
0.00% <0.00%> (-19.36%)
:arrow_down:

... and 185 more

Merging...


Rscript using system version I think

@noamross @dirkschumacher I'm excited about the potential for shiny/electron. I was working on the setup and it seems that running the Rscript uses the system version of Rscript version rather than the version that was downloaded in. Does this make sense? This can be an issue if it then tries to install package versions associated with the system Rscript (I have 4.0) versus the version being downloaded in get-r-mac.sh

I think this is part of #29? I'm finding some weirdness in attempting to install the packages needed. I found that I was able to install packages with
R_HOME_DIR=r-mac ./r-mac/R -e 'install.packages("automagic", lib = "r-mac/library")'

However, the library is not explicitly set and will use packages from your system library instead of the one in the app.


Shortcuts don't work on new brackets windows.

All shortcuts do not work on any further brackets instances I open.

Most likely related to this: #11826
I have occasionally ran into that too, but haven't found a way to repeat it successfully.

To replicate: Try to go to Debug > New Brackets Window. And in that window, no shortcuts are working. I'm not sure why you think that it's related to the issue you posted. For it doesn't mention multiple instances of brackets.

@SilverSilence I mention being able to trigger the issue by opening new Brackets window(s) in the bottom of that post. Sometimes I lose the undo ability, sometimes the shortcuts. But most of the times opening them works out just fine, just like when I tried to test this again just now. If it always does it to you, then great (sort of, great as in can you post your Health Report from Help -> Health report.

Oh and do you get any errors to console. Have you tried it without extensions?

Okay, this ios really weird. I just tried it again now and the shortcuts work?? I didn't change anything, I had the same applications running... And it's the FIRST time that they work :/
As for your questions: Yes, I had the issue with and without the extensions. I'll test it again tomorrow. Should it not work again, I'll let you know.

@SilverSilence If you have still problems feel free to reopen or open a new issue.

Am facing this problem, I tried loading brackets without extensions and still the same, even after brackets reinstall too.


Blurry Text Using CPU Plugin

Describe the bug
When activated, the GPU plugin causes my in-game text (chatbox, dialogue, etc.) to become blurred and less opaque.
To Reproduce
Steps to reproduce the behavior:

Simply active the GPU plugin

Expected behavior
The text would retain its sharpness and clarity.
Screenshots
Before:

After:

Environment (please complete the following information):

OS: Windows 10 64-bit
GPU: GeForce GTX 1060
RuneLite version: 1.5.0
Launcher version: 1.6.0

If Anti aliasing is enabled in Nvidia control panel it will cause this sort of behaviour.
You can try disabling anti aliasing globally or customise Runelites profile within the control panel.
After you have made the changes, restart the client.

You should add the solution to the wiki

If Anti aliasing is enabled in Nvidia control panel it will cause this sort of behaviour.
You can try disabling anti aliasing globally or customise Runelites profile within the control panel.
After you have made the changes, restart the client.

This did the trick! Thanks a million!


Map is not refreshed when coming back from Editing a waypoint or cache

If you click on a cache or waypoint on the map and change it (adding/removing/changing waypoints) the map is not updated when the cache/waypoint is closed.
A manual refresh is also not available (?)

Anyone working on this one? This bug really bothers me as it applies to everything (waypoints, offline log, updating live map after viewing details of an unknown, etc).
If no one else is already working on, I can try taking a look, but no guarantees how far I will get ;)

I planned to work on this, but did not yet start, so if you have time just go ahead :)
If you wish you can start outlining ideas here first, so perhaps I can at least fill in with my knowledge of the code base.

I added me as a stand-in.


Update CopyObject and CopyPart requests not to remove leading slash from object keys

Description
Fixes #1720 (also internal issue DOTNET-4410).
Motivation and Context
This PR updates the CopyObject and CopyPart marshallers not to remove leading slashes from the specified object keys.
Testing

Dry-run: DRY_RUN-48cd51a1-d13d-4917-a705-49004234b630
Added integration tests
Also tested the example from the original issue with the built .nupkg and confirmed it works (using either a bucket and an access point for SourceBucket):

var client = new AmazonS3Client(RegionEndpoint.USEast1);

var response = client.CopyObject(new CopyObjectRequest
{
    SourceKey = "/",
    DestinationKey = "/",

    SourceBucket = "bucket1",
    DestinationBucket = "bucket2",
});

// Outputs: OK
Console.WriteLine(response.HttpStatusCode);

Types of changes

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

Checklist

[x] My code follows the code style of this project
[x] I have read the README document
[x] I have added tests to cover my changes
[x] All new and existing tests passed

License

[x] I confirm that this pull request can be released under the Apache 2 license

Can you do a double check and run this scenario using .NET Core? I want to make sure the difference in HttpClient and changes to Uri class don't produce any different behavior.

Yes, I can. I tried the Copy an object from one Amazon S3 bucket to another and Get an object from an Amazon S3 bucket examples from the docs (using /testfile.txt instead of testfile.txt as the object keys) and my applications in (net6.0 and net48) produced the same results.

We may have to change PowerShell to do the same.
https://github.com/aws/aws-tools-for-powershell/blob/master/modules/AWSPowerShell/Cmdlets/S3/Advanced/Copy-S3Object-Cmdlet.cs#L582
https://github.com/aws/aws-tools-for-powershell/blob/master/modules/AWSPowerShell/Cmdlets/S3/Advanced/AmazonS3Helper.cs#L39
https://github.com/aws/aws-tools-for-powershell/blob/master/modules/AWSPowerShell/Cmdlets/S3/Advanced/Copy-S3Object-Cmdlet.cs#L675
https://github.com/aws/aws-tools-for-powershell/blob/master/modules/AWSPowerShell/Cmdlets/S3/Advanced/Copy-S3Object-Cmdlet.cs#L691

We may have to change PowerShell to do the same.

Thanks for pointing that out, I'll create a PR in the PowerShell repository to address those changes.


Is current admin-bro able to add custom page at side bar?

I would like to add custom page that can be accessed through side bar.
But as far as I can find, only way to create item in the left side navigation bar is to add resource.
How can I customize left navigation bar so that I can create a custom page?

Duplicated issue.

@amityaffliction You should link the other issue ;)

you can use https://adminbro.com/AdminBroOptions.html#pages

@wojtek-krysiak The link is not working anymore, it redirect me to the main page.
Was it still possible to do it?

@wojtek-krysiakСсылка уже не работает, перенаправляет на главную. Было ли еще возможно это сделать?

https://docs.adminjs.co/AdminJSOptions.html#pages

https://docs.adminjs.co/AdminJSOptions.html#pages

Thanks!

Hi How to achieve this? the link is says page does't exists.


Scheduled print won't turn on the printer.

Hi,
I'm using PSU control to power on the printer with a GPIO connected to a relay in the PSU of the printer.
I put /home/pi/oprint/bin/octoprint plugins psucontrol:on in the Print Scheduler settings as a system command to run before the prints but it doesn't work. I tried running the line via SSH bridge, it works just fine there.
When I start the printer, the scheduled print will start normally if it was planned.
I had some problems with the old plugin Automatic Shutdown (even if it was deactivated), so I just removed it just in case but it didn't change a thing unfortunately.

Duplicate of #12 and already resolved in version 0.0.4rc1 I believe if you want to try it out.


Language Support: Kotlin

PR
Champion

#

Description

[ ] Kotlin as a target language (collaboration with JetBrains)
[ ] Kotlin as a source language (requires that javascript modules will use the kernel, which will actually improve our coverage)

Progress

[x] Tracking Issue Created
[ ] RFC PR Created
[ ] Core Team Member Assigned
[ ] Initial Approval / Final Comment Period
[ ] Ready For Implementation

[ ] implementation issue 1

[ ] Resolved

any progress?

2019 till today (time passed)... an entire global shutdown, pandemic, economic crises, record high inflation, tech layoffs, world war and no Kotlin support.


Flutter problem ??

When i run my app the android studio blocks and i cant do anything anymore even the cursor i cant deplace it. I should shut down the computer with the button and i restart it and the same problem still even i tried in visual studio same thing

Perhaps this is an issue with your hardware. Check your Operating System's System monitor (Task Manager if on Windows) and check your hardware usage (mainly memory) and see if you are running out of memory.

Perhaps this is an issue with your hardware. Check your Operating System's System monitor (Task Manager if on Windows) and check your hardware usage (mainly memory) and see if you are running out of memory.

Can u explain more

The Github issue tracker is for bugs and features requests for the Flutter SDK. Help and support questions like these are better suited for Q&A sites like StackOverflow or on one of the flutter community sites at https://flutter.dev/community .


Cargo error: serde conflict with time v0.3.26 crate

Using the latest serde and the latest time crates, produce a cargo error.
Cargo.toml
serde = { version = "1.0.177", features = ["derive"] }
time = "0.3.26"

Cargo check
error: failed to select a version for `serde`.
    ... required by package `time v0.3.26`
    ... which satisfies dependency `time = "^0.3.26"` of package `hello-world v0.1.0 (/tmp/cargo-outdatedRi1D4I)`
versions that meet the requirements `>=1.0.126, <=1.0.171` are: 1.0.171, 1.0.170, 1.0.169, 1.0.168, 1.0.167, 1.0.166, 1.0.165, 1.0.164, 1.0.163, 1.0.162, 1.0.161, 1.0.160, 1.0.159, 1.0.158, 1.0.157, 1.0.156, 1.0.155, 1.0.154, 1.0.153, 1.0.152, 1.0.151, 1.0.150, 1.0.149, 1.0.148, 1.0.147, 1.0.146, 1.0.145, 1.0.144, 1.0.143, 1.0.142, 1.0.141, 1.0.140, 1.0.139, 1.0.138, 1.0.137, 1.0.136, 1.0.135, 1.0.134, 1.0.133, 1.0.132, 1.0.131, 1.0.130, 1.0.129, 1.0.128, 1.0.127, 1.0.126

all possible versions conflict with previously selected packages.

  previously selected package `serde v1.0.183`
    ... which satisfies dependency `serde = "^1.0.183"` of package `hello-world v0.1.0 (/tmp/cargo-outdatedRi1D4I)`

Temporary Fix
It seems to work fine when downgrading to time v0.3.25

This is a deliberate change. You can read more about it here: https://github.com/time-rs/time/discussions/611

#611

So the resolution at the moment is either keep the time crate out-dated or serde out-dated

Yes. See the linked discussion/vote for details.


[Feature Request]: DICOM IO delete

What would you like to happen?
The DICOM IO (Java SDK) currently supports retrieving metadata. We should extend the functionality to allow it to delete a study using projects.locations.datasets.dicomStores.studies.delete, called on the Healthcare Client, under the hood.
Issue Priority
Priority: 3 (nice-to-have improvement)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[X] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue


Double closing curly braces ( }} ) seem to break the detection

Consider this string
Here is a link https://domain.com/d/foo?email={{foo.bar}}&name={{foo.name}}

Even though curly braces are not common in URL, one would expect for https://domain.com/d/foo?email={{foo.bar}}&name={{foo.name}} to be matched to an actual link.
However linkify-it seems to stop matching after the 2nd } character in the URL, resulting in these matches:
[ Match {
    schema: 'https:',
    index: 3,
    lastIndex: 44,
    raw: 'https://domain.com/d/foo?email={{foo.bar}',
    text: 'https://domain.com/d/foo?email={{foo.bar}',
    url: 'https://domain.com/d/foo?email={{foo.bar}' },
  Match {
    schema: '',
    index: 53,
    lastIndex: 61,
    raw: 'foo.name',
    text: 'foo.name',
    url: 'http://foo.name' } ]

Notice how foo.name is being matched as another link because of that 😅
Once again, I know that { and } are not valid characters per the RFC, but Slack and other softwares correctly match this case so I guess there's something that can be done to fix that in linkify-it 😊
I'm happy to provide more details/examples.

Looks like dupe of https://github.com/markdown-it/linkify-it/issues/23. IMHO worth to RIP one more example there :)

I thinks this does not needs fix in linkify-it, but you can add kludge to your app: replace {{ and }} with something unique and non-breaking-link, and replace back after linkify run. That's dirty, but may be ok for your specific case.

This is still an issue. >:[

@eramdam hi after many years :). I'd like to clarify status of this issue.

is it critical or can be rejected?
should we consider it as part of #23 or as special case?

Close as dupe of #23


the while stop condition in DPCNN seems wrong?

https://github.com/649453932/Chinese-Text-Classification-Pytorch/blob/6cb26819af7b646275aff8a4693676f2849e67f6/models/DPCNN.py#L69
Should it be  >= 2?

I think we can use the dpcnn pytorch code of Tencent/NeuralNLP-NeuralClassifier
dpcnn_pytorch_code_of_tencent

Ok, I will see it


add k8s connector merge patch

unchecked-in part in the previous pr for update JSON format
https://github.com/open-cluster-management/grc-ui-api/pull/122

@ycao56
last two questions for create/update data flow:

when create policy automation, system will create a random resourceVersion in metadata. Will we use the same resourceVersion in the update, or + 1 on the original resourceVersion. resourceVersion is required for updating.

for 'manual' with annotations JSON, what should the mode value, 'disabled' or 'once' or it doesn't matter?

A third thing is when updating policy automation, all other things can be saved/changed except metadata.annotations, metadata. annotations is still the previous value after updating, here I used the same resourceVersion in previous metadata, should I used sth like previous resourceVersion + 1?
update JSON
{
  "json": {
    "kind": "PolicyAutomation",
    "apiVersion": "policy.open-cluster-management.io/v1beta1",
    "metadata": {
      "name": "policy-grc-666-policy-automation",
      "namespace": "default",
      "annotations": {
        "policy.open-cluster-management.io/rerun": "true"
      },
      "resourceVersion": "3592168"
    },
    "spec": {
      "policyRef": "policy-grc-666",
      "mode": "disabled",
      "automationDef": {
        "type": "AnsibleJob",
        "name": "Demo Job Template",
        "secret": "grc-testing"
      }
    }
  }
}

update response from the backend
{
  "apiVersion": "policy.open-cluster-management.io/v1beta1",
  "kind": "PolicyAutomation",
  "metadata": {
    "annotations": {
      "policy.open-cluster-management.io/rerun": "true"
    },
    "creationTimestamp": "2021-05-17T02:26:56Z",
    "generation": 3,
    "managedFields": [
      {
        "apiVersion": "policy.open-cluster-management.io/v1beta1",
        "fieldsType": "FieldsV1",
        "fieldsV1": {
          "f:status": {}
        },
        "manager": "governance-policy-propagator",
        "operation": "Update",
        "time": "2021-05-17T03:05:53Z"
      },
      {
        "apiVersion": "policy.open-cluster-management.io/v1beta1",
        "fieldsType": "FieldsV1",
        "fieldsV1": {
          "f:metadata": {
            "f:annotations": {
              ".": {},
              "f:policy.open-cluster-management.io/rerun": {}
            }
          },
          "f:spec": {
            ".": {},
            "f:automationDef": {
              ".": {},
              "f:name": {},
              "f:secret": {},
              "f:type": {}
            },
            "f:mode": {},
            "f:policyRef": {}
          }
        },
        "manager": "unknown",
        "operation": "Update",
        "time": "2021-05-17T03:11:30Z"
      }
    ],
    "name": "policy-grc-666-policy-automation",
    "namespace": "default",
    "resourceVersion": "3597557",
    "selfLink": "/apis/policy.open-cluster-management.io/v1beta1/namespaces/default/policyautomations/policy-grc-666-policy-automation",
    "uid": "d8998a97-7b18-4ab5-94d2-0b4939706afa"
  },
  "spec": {
    "automationDef": {
      "name": "Demo Job Template",
      "secret": "grc-testing",
      "type": "AnsibleJob"
    },
    "mode": "disabled",
    "policyRef": "policy-grc-666"
  }
}

when creating policy automation, the system will create a random resourceVersion in metadata. Will we use the same resourceVersion in the update, or + 1 on the original resourceVersion. resourceVersion is required for updating.
Don't change resourceVersion. This is system generated.

for 'manual' with annotations JSON, what should the mode value, 'disabled' or 'once', or doesn't matter?
For edit, don't change the value. For create, set it to disabled.

A third thing is when updating policy automation, all other things can be saved/changed except metadata.annotations, metadata. annotations is still the previous value after updating, here I used the same resourceVersion in previous metadata, should I used sth like previous resourceVersion + 1?
Never change resourceVersion as I said in question No 1. You can run following command with -v99 to see how kubectl sends the request

oc annotate configmap -n default create-service-now-ticket --overwrite policy.open-cluster-management.io/rerun=true -v99

You will find in the output it was actually doing a PATCH request
I0517 08:35:10.588927   39524 request.go:1097] Request Body: {"metadata":{"annotations":{"policy.open-cluster-management.io/rerun":"true"}}}
I0517 08:35:10.588997   39524 round_trippers.go:423] curl -k -v -XPATCH  -H "Content-Type: application/merge-patch+json" -H "User-Agent: oc/openshift (darwin/amd64) kubernetes/02c1100" -H "Authorization: Bearer sha256~WT2bDcgOI07mrztDhUhNhJEtm6aXaZLSq8KpwlZX5jI" -H "Accept: application/json" 'https://api.policy-grc-cp-autoclaims-lxqj7.dev08.red-chesterfield.com:6443/apis/policy.open-cluster-management.io/v1beta1/namespaces/default/policyautomations/create-service-now-ticket?fieldManager=kubectl-annotate'
I0517 08:35:10.613739   39524 round_trippers.go:443] PATCH https://api.policy-grc-cp-autoclaims-lxqj7.dev08.red-chesterfield.com:6443/apis/policy.open-cluster-management.io/v1beta1/namespaces/default/policyautomations/create-service-now-ticket?fieldManager=kubectl-annotate 200 OK in 24 milliseconds

when creating policy automation, the system will create a random resourceVersion in metadata. Will we use the same resourceVersion in the update, or + 1 on the original resourceVersion. resourceVersion is required for updating.

Never change resourceVersion. This is system generated.

for 'manual' with annotations JSON, what should the mode value, 'disabled' or 'once', or doesn't matter?

For edit, don't change the value. For create, set it to disabled.

A third thing is when updating policy automation, all other things can be saved/changed except metadata.annotations, metadata. annotations is still the previous value after updating, here I used the same resourceVersion in previous metadata, should I used sth like previous resourceVersion + 1?

Never change resourceVersion as I said in question No 1. You can run following command with -v99 to see how kubectl sends the request
oc annotate configmap -n default create-service-now-ticket --overwrite policy.open-cluster-management.io/rerun=true -v99

You will find in the output it was actually doing a PATCH request
I0517 08:35:10.588927   39524 request.go:1097] Request Body: {"metadata":{"annotations":{"policy.open-cluster-management.io/rerun":"true"}}}
I0517 08:35:10.588997   39524 round_trippers.go:423] curl -k -v -XPATCH  -H "Content-Type: application/merge-patch+json" -H "User-Agent: oc/openshift (darwin/amd64) kubernetes/02c1100" -H "Authorization: Bearer sha256~WT2bDcgOI07mrztDhUhNhJEtm6aXaZLSq8KpwlZX5jI" -H "Accept: application/json" 'https://api.policy-grc-cp-autoclaims-lxqj7.dev08.red-chesterfield.com:6443/apis/policy.open-cluster-management.io/v1beta1/namespaces/default/policyautomations/create-service-now-ticket?fieldManager=kubectl-annotate'
I0517 08:35:10.613739   39524 round_trippers.go:443] PATCH https://api.policy-grc-cp-autoclaims-lxqj7.dev08.red-chesterfield.com:6443/apis/policy.open-cluster-management.io/v1beta1/namespaces/default/policyautomations/create-service-now-ticket?fieldManager=kubectl-annotate 200 OK in 24 milliseconds

Got them

/hold

@ycao56
FYI, meet the same bug when patch nested JSON,
add k8s connector merge patch resolved it
https://stackoverflow.com/questions/63228652/oc-patch-replace-image-name-error-json-cannot-unmarshal-object-into-go-value

/unhold

/lgtm


What if something doesn't have a website?

Hello! I am just wondering what would happen if a package does not have a website. Does it go under github.com?

Yeah you can see we’ve had to do a few like this:
https://github.com/teaxyz/pantry.core/tree/main/projects/github.com
We expect there will need to be an aliasing system at some point.


Make examples of clients

It would be nice for me, as a potential user of the pythogen to see real clients.
For example PetStoreClient - https://petstore.swagger.io/
The sync and async versions.

https://github.com/artsmolin/pythogen/pull/17


Set default input values for AMI build configuration

Setting default opinionated values for AMI build configuration so customers can set only values for parameters they really want to control while letting the tool default to these values for other fields.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/lgtm

/cherrypick release-0.12


Add a license

This is especially useful if you want to allow people to use your work :-)
I've used the Apache License since it's used in the Facebook PHP Graph SDK repository too.

@khawkins98 ping 😄

Had to jog my memory a bit on this script, am always surprised how it's still of use so many years on!
All seems agreeable. Committing. Thanks for the PR.


Release KubeOne 1.4.0-rc.1

Scheduled to happen: Friday, 2022-02-11
Before Release

[x] Write and publish the changelog (#1835, #1839)

Release

[x] Create and push a new tag
[x] Ensure that the release job has succeeded
(watch https://public-prow.loodse.com/?job=post-kubeone-release)

After Release

[x] Update the release's description on the GitHub Releases page to replace
the automatically generated description with the changelog
[x] Download the binaries from GitHub and make sure that:

[x] Checksums are matching
[x] kubeone version returns the expected version

[x] If the release introduces support for a new Kubernetes version, submit
conformance results to https://github.com/cncf/k8s-conformance/ (https://github.com/cncf/k8s-conformance/pull/1807)

Action Items

/assign

The conformance tests are submitted, so we can close this issue.


⚠ sg1grpc.serverssh.us has degraded performance

In dc7542f, sg1grpc.serverssh.us (http://sg1grpc.serverssh.us/) experienced degraded performance:

HTTP code: 400
Response time: 5968 ms

Resolved: sg1grpc.serverssh.us performance has improved in f7cca06.


test(commerce-queries): clean up tests by disabling APQ

Why are these changes introduced?
Addresses ENG-8659

What is this pull request doing?

cleans up the tests to run with APQ disabled. This makes it easier to inspect the args passed to the fetch function without needing to send an error response or parse the url.

How to Test

Checkout this branch
Run npm run coverage all tests should pass with 100% coverage

Checklist

[x] This Pull Request aligns with nacelle-js' Code of Conduct.
[x] You can follow your own "How to Test" instructions and get the expected result.
[x] Screenshots, videos, etc. in the PR description are free of brand names and sensitive details.
[x] Created a changeset (if the change should appear in a changelog).
[x] Updated the README.md with documentation changes (if applicable).

Hi @mdarrik! Are you all set to merge this?


HDInsight left out in documentation

If Databricks is listed here, HDInsight should be represented as well

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

ID: d0bdc71b-0a54-4cc4-1b79-f60e274e9a51
Version Independent ID: 1783570c-3545-71c8-72d3-d5657f77c070
Content: Choosing a machine learning technology
Content Source: docs/data-guide/technology-choices/data-science-and-machine-learning.md
Service: architecture-center
Sub-service: cloud-fundamentals
GitHub Login: @MikeWasson
Microsoft Alias: pnp

Hi @isamllr Thank you for your feedback! We will review and update as appropriate.

@isamllr HDInsights appears to have been mentioned in this section and part of the same recommendation; as such, we've made a minor update in that sentence to highlight that ML Services is an available option when spinning up HDInsight clusters in Azure.
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Upgrade to the latest version of pyee

Updates the pyee module to the latest version. A change is prepared for mycroft-core updating this as well.
The code changes the now deprecated pyee.EventEmitter to the equivalent pyee.BaseEventEmitter

Ping @clusterfudge

Ping @clusterfudge ?


Create How-to-Build-a-Flask-ReactJS-Application-and-Deploy-to-Okteto.md

Draft

Thanks @khabdrick! I'll send my review after the community call. You should join too - https://lu.ma/qgm1qff3

Okay no problem
On Thu, 22 Jul 2021, 12:16 pm Abdulazeez Abdulazeez Adeshina, <
@.***> wrote:

Thanks @khabdrick https://github.com/khabdrick! I'll send my review
after the community call. You should join too - https://lu.ma/qgm1qff3
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/okteto/blog-proposals/pull/40#issuecomment-884834391,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANZMFASD4DL2AH6J5YUY2TLTY74XHANCNFSM47QPWGAA
.

@khabdrick, you're supposed to provide an image not delete the instance in the draft.

https://github.com/okteto/blog-proposals/pull/40#discussion_r675422498

@khabdrick, I don't mind :)

@Youngestdev I am done with the changes :)


identify user metrics for dug

Baseline user metrics need to be determined for each deployment of Dug. What we want to know:

how many people are using Dug at each instance
number of unique users
number of searches completed by users
average time spent using Dug
do users preform multiple searches in 1 session? or 1 search their entire session
ratio of users who "dig deeper" into concepts and KGs : users who preform 1-2 clicks then end session

Tasks for this work inlude:

[ ] make sure each prod deployment of Dug is connected to MixPanel @frostyfan109
[ ] set up dashboards to capture above analytics @hhiles
[ ] generate initial reports, share with leadership @hhiles
[ ] tweak above questions and generate a second wave of reports after 1-2 months @hhiles

conversation w. griffin and hannah

currently cannot tell what data coming into mixpanel is coming from which deployments
observed UI issue - searches are often fired twice. dug API is being called 2x; perhaps a react problem. needs to be fixed so our data isn't incorrect.
google analytics is sunsetting a lot of their previous analytics tools and updating their toolsuite to be competitive against mixpanel, and should offer some things we can't currently do in mixpanel

new tasks added to main body


Replace placeholders

added cool styled svg graphics
replaced copy (only Forge left)
changed metadata
added CONP logo
general style updates

Can we set the same width for hero's buttons maybe? Because it looks like they have different margins.


Make sure all buttons have correct border / outline.

According to jop all buttons with borders need to be darkGrey on hover. Right now outline is the same but maybe we need to differentiate from hover to comply with UI standards.

Borders of buttons or input fields like the token search bar, transaction list/adress, settings wheel, and the menu on the left all have borders #1A1E25. All those 'ghost button's will have #6E7686 as hover color. For the focus/active state they should become blue.
I hope that makes sense, and if there are cases which this doesn't work we should look into how to adjust. But this should work. I'll screenshot some examples.

Inactive

Hover

Active/focus

Yeah, focus is when you tab. This styling can also be used for active.


fixed PricingCard styling for Premium + Founder users

Bug I fixed: I'm an annual subscriber that's also a founder. The PricingCard and PricingTables were highlighting and signalling that I am a Pro subscriber but not a founder. I was expecting that the founder card would be highlighted.
Fix: I simply corrected the logic regarding a premium user and whether or not they are a founder.

Thank you!


java-WordCount

String filtering by individual words.
Sorting by number of instances word is present, largest to smallest.

We need to talk about commits!!
The commits are a sign of not having a good gameplan going in. If you have a good gameplan, you will commit like a superstar. Committing well is not really a major issue. When you are working on a live production codebase, there will be rules and requirements and to follow. However, I think that developing the habit of making a game plan in English before writing in code is really important. And that will show up in your commits.


Max color component value

I merged this over manually in 3d523ee8b0118aef9594e08aafb7fb1d75383b0f so that the two main changes in the branch could be separate commits


one line return

Just a formated code, converting an arrow funtion with return in an implicit return for simple statements.

Hey, thanks for your first commit! 🚀
I noticed that you might not be familiar with our style guidelines for commits.
I recommend taking a look at the CONTRIBUTING.md file to see how we like to format our commit messages.
This will help you follow the same style that we use, which will make it easier for everyone to understand the changes that are being made.
Let me know if you have any questions about this!
https://midu.dev/buenas-practicas-escribir-commits-git/

Just a question:
For clarity or much simpler reading:

1ª case: app.get('/leaderboard', (ctx) => ctx.json(leaderboard))  OR
2ª case: app.get('/leaderboard', ctx => ctx.json(leaderboard))

If 2ª is the case: omit parens when possible. Can we change prettierrc.js --> arrowParens: "avoid"? In this case all project wil be afected in the format.
What's your piece of advice, your recomendation in these cases?

In general, it's a good practice to omit parentheses when possible to improve readability and make the code more concise. However, whether or not to use "avoid" for the arrowParens setting in prettierrc.js ultimately depends on your team's coding style guidelines and preferences ( in this case in @midudev).
If you decide to use "avoid", it will affect the formatting of all the code in the project, so it's important to make sure that this is something that your team ( @midudev ) is comfortable with.
🍻


Mouse doubleclick stops continous pressed events.

I can reproduce in the Buttons scenario:

Hold the mouse perfectly still and press-release then press-hold again. Increments once, but then does  not repeat:

https://i.imgur.com/eE3BSgr.gif
Using the Mouse scenario the issue has to do with the doubleclick logic.
Here, I just pressed-hold-released:

Here, I pressed-released-pressed-hold:

Originally posted by @tig in https://github.com/gui-cs/Terminal.Gui/issues/3323#issuecomment-2053691080

Double clicking on the second image is controlled by the Win32 API which sends the double click flag and there is no way to avoid this, even if the button is pressed.

Any progress on this issue: I can easily repro : if you click a 2nd time without moving the mouse, you get no 2nd click event.
This is where I got stuck on the road to processing double clicks.  Happy to provide more testing and feedback.


Pin objects that are defined in Ruby.

DEBUGGER__ and FrameInfo are defined in Ruby, so they are not
automatically pinned.  rb_mDebugger and rb_cFrameInfo are static
global variables that hold references to these objects.  If compaction
happens, these static references can go bad.  To fix this, we register
the two references as "mark objects" so that the GC will automatically
pin the references.
Fixes: #272

wow now Class/Module objects are not pinned...

@ko1 Class and module are pinned if they are defined via C api.  rb_define_class will automatically pin the class, but class Foo; end will not.  IOW cGlobal = rb_define_class(...) is safe, but cGlobal = rb_const_get(...) is not safe.

Yeah thank you for explanation. I misunderstand the specification.


Laravel is throwing an arror after adding 'Contentful\Laravel\ContentfulServiceProvider' to the providers array

After running:
php composer.phar require contentful/contentful-bundle:@beta

And adding 'Contentful\Laravel\ContentfulServiceProvider' to the providers array in config/app.php, I get the following error:
FatalThrowableError in ProviderRepository.php line 146:
Class 'Contentful\Laravel\ContentfulServiceProvider' not found
PHP 7.0.8
Laravel Framework version 5.3.6

Sill happening.

Install contentful/laravel instead of contentful/contentful-bundle
php composer require contentful/laravel
You'll also need to set in your composer.json file (see  #2):
{
    "prefer-stable": true,
    "minimum-stability": "dev"
}

Fixed in #2 thanks to @jevets.


Don't record BG values auto-populated by CGM as manual SMBG

See attached Trello card for details.

 Tandem X2 wizard bolus with BG auto-populated from CGM shows up as manual SMBG reading


⚠ Avatar API Endpoint has degraded performance

In fb6d43f, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 3922 ms

Track here:
https://devforum.roblox.com/t/avatar-service-failing/985288
Roblox is currently unaware.

Track here:
https://devforum.roblox.com/t/avatar-service-failing/985288
Roblox is currently unaware.

Resolved: Avatar API Endpoint performance has improved in d8c97be.

Resolved: Avatar API Endpoint performance has improved in d8c97be.


Negative Box Selection

This PR implements -

When using box selection, Ctrl+Shift to remove from the selection.
Show the corresponding hints while dragging.
discord

!build


Don't include completed items in task count(?)

Did you check if a similar feature request has already been reported?
✔
Is your feature request related to a problem? Please describe.
for me the badge count represents items that need attention, which completed items do not.

Hi @freder. I disagree on this one, as the filter will apply also to the completed todos. If you don't want to have the completed todos to be counted, you can simply hide the completed todos by using the shortcut (Ctrl Or CMD + H) or by using the view drawer in the navigation.


More search filtering

Can you add filters to the Browse page to search by contributor, and by type?
Contributor: I'd like to be able to search for anime by publisher/studio (ghilbi, NHK, etc) and for mangas by a certain author. Ideally this information should also be displayed in the main page of the detail view as well (#494).
Type: It should be possible to further narrow down searches beyond manga/anime to OVA/TV/Movie/etc fir anime and Manga/Manwha/Novel/etc for print.
Also, the publishing status should either have a n/a entry (search regardless of publication status) or the ability to select more than one option from it. I keep going through search results lately, and not finding anything because the publication status is set incorrectly.

Can you add filters to the Browse page to search by contributor, and by type?
We are using https://myanimelist.net/anime.php for this part and does not contain that info I think.
What do you mean with type?
I'd like to be able to search for anime by publisher/studio
That is possible but I was way too lazy to add all those names manually. I will do it soon.
Type: It should be possible to further narrow down searches beyond manga/anime to OVA/TV/Movie/etc fir anime and Manga/Manwha/Novel/etc for print.
That does already exist.
Also, the publishing status should either have a n/a entry
I was working on it.

Rather than add manually, we could see about an API endpoint to fetch the latest Publisher list, as it's likely to change.

Can you add filters to the Browse page to search by contributor, and by type?
We are using https://myanimelist.net/anime.php for this part and does not contain that info I think.

How about a way to search people.php instead? MAL is a bit odd/lacking in this; I can search most other book websites for a title+author combination (public library, amazon, goodreads, etc)

Maybe. This is an API-level issue, not an application one, though. That means it may take a bit longer to get implemented, if at all.
Basically, in order to use MAL in a somewhat sane way, we have a separate piece of code that runs on a server that acts like a web browser to MAL and returns information in a format that the application can use. Basically, we move the intensive part of interpreting ever-changing HTML pages to a central point that can be updated quickly when needed. MAL is very quirky, so having this helps standardize how the application works and takes away a lot of code and complexity that would drain mobile batteries.
You can put in a request for this extra search feature over at the API site, or I can do so if you would rather not.

Done


x?.y works even when x is not an optional

Because of the way ?. is implemented, things like this happen:
ergo$ return {a: 1}?.a;
Response. none
  :  {emit: Unit[], response: Nothing?, state: Unit}?

(This should be a type error, I think, because people will surely try to do this and get confused.)

I agree. I seem to recall this is the semantics in Swift which I started from, but I don't like it.

Can be compiled to (let $opt : Any? = <operand> ; match …), where the let just forces the operand to be of an optional type.

Looking at the example more carefully, this is definitely not the semantic in Swift. It's certainly a bug.


Update gradle wrapper for Kotlin language/api 1.5

getting rid of a branch snapshot and cleaning up verification metadata

@bot-gradle test and merge

OK, I've already triggered a build for you.

(getting notified)

Pre-tested commit build failed.

@bot-gradle test and merge

OK, I've already triggered a build for you.


Language Server testing example

A while I ago I created a language server for Ember.js projects (https://github.com/emberwatch/ember-language-server). We initially forked the (old) example language server (https://github.com/Microsoft/vscode-languageserver-node-example) and built our LS on top of it. While we gave our best to test the language server, we were always missing basic integration tests. I tried creating integration tests a couple of times but somewhere on the way I always get stuck. I can imagine that this is a major problem for a lot of people that start working with the language server example.
So I think a basic integration test example could improve the language server example in this repository a lot.
What I would like to see in an example test would basically do:

Start the server and connect to it
Send a specified request
Check the response if it is correct

This is resolved. See https://github.com/Microsoft/vscode-docs/blob/vnext/docs/extensions/example-language-server.md#testing-the-language-server and the updated sample code: https://github.com/Microsoft/vscode-extension-samples/tree/master/lsp-sample.
The doc will become public next week, but you can view it now on vnext branch of vscode-docs.


Generate separate documentation for Kubernetes and OpenShift

Move content from master.adoc to assemblies
Create master-kubernetes.adoc and master-openshift.adoc and build separate documentation for each
Consolidate/remove duplicated assemblies no longer needed

@jenmalloy @ncbaratta This removes all the context hacking and i think makes the guide a lot easier to read. I will update the upstream website to link to 2 separate docs once this is merged.

@ncbaratta Addressed the review comments

@jenmalloy updated


Support passing through switches for exec/run

Prior to this implementation something like the following was not possible:
nib run --service-ports web puma

These changes add support for accepting some options (switches) for the run and exec commands specifically.
Resolves #89

👀 @blankenshipz

LGTM! 👍


Autoreload not working

I've started a new project running the yo generator and started the server, first of all when I replaced the route I had to delete the __clientTemp folder, then when I change the getElements content I've to restart the server, i that supposed to work that way?

Hot reloading only works for changes to components right now. Modifications to routes.js or the page classes still require manual reload. But pm2 can solve that nicely:
pm2 start npm --watch --ignore-watch="__clientTemp" -- start

Ok gotcha, thank you @CreepGin
Just seen that also hot reloading is not enabled by default in development but I have to add --hot in the npm start command

To same late comer's time, I met same problem and managed to find it is the hot field in .reactserverrc controlling hot-reload. Just set it to be true and Hot-Module-Reload is enabled for npm start.
Adding --hot in the npm start comand doesn't work. I suppose there is some feature change after @alex88 closed this issue.


ERROR CODE 218 - invalid argument in method keybase.1.config.appendGUILogs

Whenever I try to quit Keybase via "Quit Keybase" from the main windows, I get this error. The app stays open after this.
In order to quit Keybase, I have to go to the systray menu and select "Quit Keybase" from the hamburger menu.
my log id: e24ac8ecf45316d03d81ab1c

I'm having this same error. Keybase also showed the same error message when I tried to use the builtin "Send feedback" screen to report this error.
I was trying to quit Keybase because I wondered if that might fix the ongoing issue I've been having where the whole window sometimes turns solid white for no reason, and doesn't stop being white until I close and reopen the Keybase window one or a few times. (On Windows 10)


STY: More PEP8 amendments

Few more PEP8 amendments.

benchmarks/benchmarks/optimize.py
benchmarks/benchmarks/signal.py
benchmarks/benchmarks/sparse.py

Thank you.

LGTM, merged. Thanks @Newman101


Why is the AuthenticationEvents trigger not supported in the .NET Isolated Worker Process?

Library name and version
Microsoft.Azure.WebJobs.Extensions.AuthenticationEvents 1.0.0-beta.8
Query/Question
Creating an Azure Function in isolated worker process is often preferred to have more control over .NET versions and other dependencies. What's the reason for this library not supporting it?
Environment
No response

This issue has been transferred to the repository where the Azure Functions team maintains the worker packages needed for extensions to be used with the isolated model.   see: #2449.


Not compatible with babel-plugin-react-element-info

I use babel-plugin-react-element-info plugin in my project.
But seems like Tippy doesn't like it.

Error: [tippy]: data-qa-node is not a valid option

What does this plugin do? Seems like it's passing that prop to <Tippy> but it won't be accepted

I think if you create a wrapper component that strips those props and forward the valid ones to <Tippy />, it will work.
import Tippy from<EMAIL_ADDRESS>
function TippyProxy(props) {
  const validProps = { ...props }
  delete validProps['data-qa-node']
  delete validProps['data-qa-file']
  return <Tippy {...validProps} />
}

export default TippyProxy

If those props should be forwarded to the reference DOM node, you can clone the children prop and add them inside this proxy I guess

I'll try, thanks!


Failed to download 'PIAWireguard'

Summary
Steps to reproduce
Pod install can not install PIAWireguard

I have the same problem with "PIAWireguard", anybody can help me ?

Hi!
Please, check if you are using this specific Go version: 1.13.4
It's a requirement from the WireGuard library to compile using that version.
Regards,
Jose

Dear ueshiba,
Tks,I will try it.

I have used this specific Go version: 1.13.4 in my pic. help me!!! Dear usshiba.

@woniupapa did you try with a pod deintegrate before running the command again?

@ueshiba yes, I did it, and the same problem.


Removed redundant button_tag from link_to

Resolves #1880

@dclarizio Please review

tried several places in UI, seems to work fine. Merging.

@skateman can you rebase and repush.

@h-kataria done!


Case Sensitive SQL Server databases fail ingestion

https://github.com/datahub-project/datahub/blob/b4bf1d4b1d22d8c808e992f62a1899227e303308/metadata-ingestion/src/datahub/ingestion/source/sql/mssql.py#L115
When attempting to use a SQL Server database that is case sensitive as a source, the ingestion fails due to the 'SYS.TABLES' object not existing.

PR has been merged!


Need help with Tabstrip

I tried to add the code to Tabstrip tab-position.js file that would cause the same effect as this script section does for the original Telerik demo set the position of tabs to LEFT, RIGHT and BOTTOM
While I understand how this works in Telerik's world - it seems that I do not understand how it should work in our plugin. While the action of setting the tabs position ought to take place in the JS file, I do not know how to get the access to the three instances of the Tabstrip object that is declared in the Tabstrip html file.

In analogy with the code for button api sample I tried something like
export class TabPosition {

    this.tabPosition = "left"
}

This is clearly not what I want (access the Tabstrip object from JS and set its tabPosition bindable to have the value "left")
What is the right syntax I should have used?

I think you need to bind to the tabPosition property.
aurelia-kendoui-tabstrip="tab-position.bind=tabPosition"

There are two ways we can handle changes to bindable properties in our wrappers. Firstly we can use the api provided by kendo such as $(button).enable(true) in the enabledChanged function of our wrapper.  This is the preferred approach.  But kendo hasn't made these api  functions for every possible property of their controls,  which means that for some property changes we observe, we have to destroy and reinitialize the control. This is a more inperformant way than using the api provided by kendo.
I just noticed that in the sample of the button,  we don't bind to the enabled property, so it doesn't even use the enableChanged function

Also, I don't want to be demotivating, but the tabstrip is also a non-input type control, which means it should be wrapped in a custom element?


[QUESTION] just a question

just a question , how did you make the QS squared instead of rounded?

I've been working on a third project lately that I plan to publish soon, it's an LSPosed module too, but i'm thinking about turning it to an RRO overlay


Change seeds so they work the first time they are passed in

Bug Report
Currently the way randomColor works, a seed will work the very first time it's passed in, but if the seed changes, it uses the last seed that was passed in to generate the color(s), and then it sets the new seed, causing the wrong output.
Here's an example:
randomColor({seed: 1, count: 10})
// Output:
['#8bd343', '#a31f44', '#f23aa5', '#25bfd1', '#b9dd5d', '#b2ffd3', '#017984', '#c4214f', '#fcd928', '#55dbd2']

randomColor({seed: 2, count: 10}) // Note the seed has changed from 1 to 2
// Notice that this is the same array as when it was seeded with "1"
['#8bd343', '#a31f44', '#f23aa5', '#25bfd1', '#b9dd5d', '#b2ffd3', '#017984', '#c4214f', '#fcd928', '#55dbd2']

randomColor({seed: 2, count: 10}) // Note the seed is still 2
// This is now the correct array of colors
['#7ed65e', '#d8d82f', '#46e2a1', '#77eaea', '#d062fc', '#8e6af2', '#604ce0', '#e5d177', '#9df2e2', '#efcb8d']

As you can see, when you run the generator with one seed (first run), change the seed, run it again (second run), it generates the wrong array of colors. Then when you run it for the third time (with the
same seed as the second run), it generates the correct array of colors.
Solution
This pull request modifies the behavior of the seed option so that the seed is reset at the end of each run.
I've also written a test that fails if you use the original version of randomColor (before my changes), so that you can run it (via npm run test) and easily see the bug.
Misc
I've also bumped the version number in package.json, as well as added a version number to the bower.json file. I also created a new tag for the 0.4.1 release (not sure if this will move over when my PR is merged in though…)

Hi Matthew! Thank you for your work in fixing this bug. I'd be happy to merge this if you remove the tests & associated metadata files from this pull request.

@davidmerfield why do you not want tests for the project?

Whoops! So sorry! Somehow I missed your comment.
I am also wondering why you would not want tests for the project. Do you mind explaining why?

@kimmobrunfeldt @daviesgeek It's not that I don't want to use tests – I don't want to use Karma/Jasmine. I'd rather move the question of how to add tests to this project to a separate issue/pull request. It is much easier to manage single-problem pull requests. Again, thank you for your work fixing this bug and apologies for my delayed response!

Okay I've update the PR to remove the unit tests!

Perfect, thank you again!


missing model_name param in ChatAnthropic

System Info
Python 3.8.16
langchain==0.0.268
Who can help?
@hwchase17 @agola11
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
ChatAnthropic doesn't have model_name attribute. instead, it has a model attribute that specifies the name. Other chat models such as ChatOpenAI, ChatVertexAI has model_name attribute, this breaks the interface while integrating with multiple LLMs.
Expected behavior
llm = ChatAnthropic(model_name='claude-2')
print(llm.model_name)

one of the main points of this pkg is to provide a unified interface for LLM no? if so, I can't see any reason to implement it like this. Also they max_retries and n param are missing

hi @joshuasundance-swca and @baskaryan I also found max_retries missing does it will also be included in the merge?


versioneer show

It would be awesome to have the versioneer command line support a sub command to show the current version:
$ versioneer show
2.5.0

or even better, a default behavior when install is already done:
$ versioneer
Usage: versioneer install

$ versioneer install
...

$ versioneer
1.0.0

It would run this line I end up writing a lot of times:
python -c "import versioneer; print(versioneer.get_version())"
(I might be missing something though, since this sounds like a very commun case.)

A lot of version information is available to any project using versioneer already:
$ python setup.py version
running version
keywords are unexpanded, not using
got version from VCS {'error': None, 'date': '2020-02-27T10:28:13-0500', 'version': '0+untagged.429.g739a148', 'dirty': False, 'full-revisionid': '739a148ce320f0f5e6b50eeac40d7b19b0880a12'}
Version: 0+untagged.429.g739a148
 full-revisionid: 739a148ce320f0f5e6b50eeac40d7b19b0880a12
 dirty: False
 date: 2020-02-27T10:28:13-0500

I'm not sure if this ticket is a request for a command that only prints out the version item in that version dictionary without the extra noise or if this command satisfies the use-case.  The latter seems likely to me since this command does show the version as well as a bunch of other useful information that went into computing that version.

This command totally satisfies the use case, thanks !
Maybe mention it in the doc ?
I didn't know about this commad even though I had the impression I read the
doc carefully...
Thanks again :)
Le jeu. 27 févr. 2020 18:07, Jean-Paul Calderone<EMAIL_ADDRESS>a écrit :

A lot of version information is available to any project using versioneer
already:
$ python setup.py version
running version
keywords are unexpanded, not using
got version from VCS {'error': None, 'date': '2020-02-27T10:28:13-0500', 'version': '0+untagged.429.g739a148', 'dirty': False, 'full-revisionid': '739a148ce320f0f5e6b50eeac40d7b19b0880a12'}
Version: 0+untagged.429.g739a148
full-revisionid: 739a148ce320f0f5e6b50eeac40d7b19b0880a12
dirty: False
date: 2020-02-27T10:28:13-0500
I'm not sure if this ticket is a request for a command that only prints
out the version item in that version dictionary without the extra noise
or if this command satisfies the use-case. The latter seems likely to me
since this command does show the version as well as a bunch of other useful
information that went into computing that version.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/warner/python-versioneer/issues/204?email_source=notifications&email_token=AAQUZ2MJOMF3NXJGUS63PNDRE7XMFA5CNFSM4KSPMOPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENFEPQY#issuecomment-592070595,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQUZ2NZKBVVLPJN3S76H6LRE7XMFANCNFSM4KSPMOPA
.


Product page: the website card is not well integrated

Hi everyone,
On a product page, and if this product has a website, the card at the bottom has several issues :

The Ripple is drawn below the card
The link is in blue, whereas in the app, it's brown
We have arrows for openable items, for this one, we should have the icon that we use on the Settings

@g123k I can see the link in the blue color. Could you please specify which icon you meant? Is it the right arrow?

I mean this icon:


[BUG]:  defaultNow()  syntax error

What version of drizzle-orm are you using?
0.28.5
What version of drizzle-kit are you using?
0.19.13
Describe the Bug
Syntax error while executing drizzle-kit push:mysql with schema below.
export const users = mysqlTable('users', {
    id: serial('id').primaryKey(),
    joined_at: timestamp('joined_at', { mode: 'date', fsp: 0 }).notNull().defaultNow()
});

Here is full sql error:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '(now()),

This syntax error occurs when using defaultNow() in schema. It generates incorrect sql (now()) that causes the error.
Expected behavior
deafultNow() in schema should have generated now() instead of (now()).
Environment & setup

Name
Version

Mysql
5.7.37-log MySQL Community Server

OS
Windows 10

I'll close this one in favor of #921.
It will be fixed by #1114


Switching context via --context is not working

Still using the current-context in the kubeconfig

If you do not use the wrong context name, everything is working as expected


ActiveModel::MassAssignmentSecurity::Error: Can't mass-assign protected attributes for Template: migrated, dmptemplate_id

Expected behaviour:
When running rake db:seed, should seed the database without error.
Actual behaviour:
Running rake db:seed RAILS_ENV=development gives
rake aborted!
ActiveModel::MassAssignmentSecurity::Error: Can't mass-assign protected attributes for Template: migrated, dmptemplate_id

Steps to reproduce:
Follow the instruction on https://github.com/DMPRoadmap/roadmap

rake db:seed RAILS_ENV=development works after I remove the following lines from seeds.rb:

migrated: false,
dmptemplate_id: 1,
migrated: false,
dmptemplate_id: 2,
migrated: false,
dmptemplate_id: 3,
because from the code (db/migrate folder and schema.rb file) I don't see such columns been created. But I'm not sure why this happen.

thanks for reporting this issue @Yu-Chieh-Henry-Yang. From version 1.1.0 the template model is as follows:
Template(id: integer, title: string, description: text, published: boolean, org_id: integer, locale: string, is_default: boolean, created_at: datetime, updated_at: datetime, version: integer, visibility: integer, customization_of: integer, family_id: integer, archived: boolean, links: text)`

and therefore I suspect that's the reason the seeds raises that Exception. Thanks again for providing as how did you fix it. We will have to remove old attributes from db/seeds.rb

When we look into this ticket. I propose that we drop the seeds.rb entirely and just create a rake task to add only the data necessary (e.g. language, question formats, etc.). That would require an update of the test_helper.rb as well though

Cool, thanks for the replies @jollopre @briri


Can't use system environment variables in tasks.json -> options -> env

VSCode Version:
1.34.0
OS Version:
windows 7

Steps to Reproduce:
my tasks.json like :
{
    "options": {
        "env": {
            "sdk-root": "${env:SDK_HOME}"
        }
    },
    "tasks": [
        {
            "type": "shell",
            "label": "run sdk",
            "command": "${env:sdk-root}\\make.exe",
            "args": [
            ]
        }
    ],
    "version": "2.0.0"
}

SDK_HOME is an environment variable that has already been added in the system. But when I run this task,  sdk-root is always empty.
It works well if I replace ${env:sdk-root} with ${env:SDK_HOME} .

The documentation describes this as the intended behavior https://code.visualstudio.com/Docs/editor/tasks#_custom-tasks


watch SMTP connections in browser

watch/client: display transactions in new rows
watch/server: get results via get_all
instead of checking based on list of plugins
watch: prune connections > 100
watch/server: don't sent data if same as last sent

Just about to review the code - but what immediately jumps out is that there's no documentation in this commit??

Ok - this looks OK to me, there's obviously a lot of hardcoding in this at the moment and plenty of room for improvement in the future.
Just had a quick look at plugins.js and I think this might be a useful change that could potentially reduce quite a bit of hardcoding that you have in this plugin:
--- a/plugins.js
+++ b/plugins.js
@@ -12,6 +12,8 @@ var utils       = require('./utils');
 var util        = require('util');
 var states      = require('./connection').states;

+exports.registered_hooks = {};
+
 function Plugin(name) {
     this.name = name;
     this.base = {};
@@ -26,6 +28,13 @@ function Plugin(name) {
 }

 Plugin.prototype.register_hook = function(hook_name, method_name) {
+    if (!Array.isArray(exports.registered_hooks[hook_name])) {
+        exports.registered_hooks[hook_name] = [];
+    }
+    exports.registered_hooks[hook_name].push({
+        plugin: this.name,
+        method: method_name,
+    });
     this.hooks[hook_name] = this.hooks[hook_name] || [];
     this.hooks[hook_name].push(method_name);

This would allow the plugins to see this variable as this.registered_hooks and would also allow me to remove some hardcoding to the plugin order option in bin/haraka too.

there's no documentation

True, that will come, and it's going to be pretty short:

Enable http in Haraka (http.ini)
Add 'watch/server' to config/plugins
Point your web browser at http://my.haraka.com/watch/

I think this might be a useful change that could potentially reduce quite a bit of hardcoding

Sorry to not understand, but which hard coding would that remove? Beside perhaps the SPF plugin which looks at the scope to determine HELO vs MAIL FROM, I'm not seeing how having the plugin hook names would be a benefit.

update_seen

I can see a much better way to handle this. I'll put it in the main comments.

One way to learn which plugins exist would be another AJAX call to watch/server.js that reads config/plugins and returns the list. The problem with doing that, versus having an assortment of plugin names hard coded and then dynamically redrawing the display table is making whatever comes back look good in the web interface. If they're being learned dynamically, there'd be an obvious desire to move the hard coded plugin names from client.js into watch.ini, so folks can choose the display ordering. Or make it deterministic based on order in config/plugins. Or...

Ok - let me be a bit more clearer on all points.
Docs: all core plugins require them to be accepted for inclusion.  We don't have any plugins shipped in core without docs.   I'd have already committed this after I reviewed it if this were the case.  After all - how is anyone not already with Haraka and this plugin going to know what this is, why they would want it and how to set it up or indeed even know of its existence as it won't appear on the web site.
Regarding the code I supplied: I noticed for example that you're splitting the plugin name by dot to work out where it runs e.g. mail_from.is_resolvable which is redundant if you know which plugins run on which hook.
There also seemed to be other places in the code that having knowledge of this would also be useful as this isn't something you can work out by reading config/plugins.  This would give you the exact run order as determined by plugins.js at start-up which I thought would eliminate a whole bunch of guess work.
It was merely a suggestion - nothing more.  I'm going to add that code anyway as it reduces the amount of hardcoding of a function I added to bin/haraka and will automatically pick up new hooks without me having to remember to add them.
It's also just occurred to me that watch could run it's own hook to have plugins report their status to it.  That would also allow it to work out which plugins can report to it and in what order and would provide a way for other plugins to integrate with it without having to modify configuration files to do it.

all core plugins require them

No surprises there.

I'd have already committed this

That surprises me, and that's partly why there's no docs yet. There's almost zero options, and I don't expect everyone will have the same goals and expectations of the plugin that I do. I expected at least a few constructive comments that would shape the plugin more, and I'll write the docs when there's at least one other person who thinks the code is "good enough."  I'm happy to hear you think so.

I noticed for example that you're splitting the plugin name by dot to work out where it runs e.g. mail_from.is_resolvable which is redundant if you know which plugins run on which hook.

Thank you for clarifying. Having another sharp mind thinking about it is helpful and I wasn't sure what you meant.
More specifically, I'm parsing the plugin names to work out where I should display them. That is only is only loosely correlated with where a plugin runs. For example, if I were to look at plugins that hook mail, I'd get this list:
access
bounce
data.headers
data.uribl
karma
mail_from.is_resolvable
queue/smtp_proxy
rcpt_to.host_list_base
rcpt_to.qmail_deliverable
record_envelope_addresses
spf

Of those, the only ones that have any particular affinity for mail are the mf.is_resolvable and spf. I thought about having overrides for plugins that "hook the world," like data.uribl, access, and karma, but that still doesn't whittle down the field enough. Nor does it control the aesthetics of placement and ordering in the display. And I surely don't want to add to the reasons for having plugin ordering "matter" in config/plugins.

It's also just occurred to me that watch could run it's own hook to have plugins report their status to it.

Complicate every plugin for the convenience of watch? I thought avoiding that was the entire reason we went down the road to result_store (to complicate every plugin for the benefit of several other plugins (like elasticsearch)).

That would also allow it to work out which plugins can report to it and in what order and would provide a way for other plugins to integrate with it without having to modify configuration files.

The difficulty I see in being able to benefit from such an arrangement is deciding how best to display the data. Should access, uribl and karma have a box in every phase because they report a result there?  Or should they be aggregated into one box as they are now with a tooltip for more info? Which phase? There's a slew of similar issues like that boil down to opinions about how best to present the data. I expect some evolution of watch in this area.
Thanks again for you feedback. I'll some docs written up soon.

@smfreegard, docs added.

That surprises me, and that's partly why there's no docs yet. There's almost zero options, and I don't expect everyone will have the same goals and expectations of the plugin that I do. I expected at least a few constructive comments that would shape the plugin more, and I'll write the docs when there's at least one other person who thinks the code is "good enough." I'm happy to hear you think so.

You should have labelled this as 'Wants Review' and 'Work in Progress' is you didn't want it committed after review then...

More specifically, I'm parsing the plugin names to work out where I should display them. That is only loosely correlated with where a plugin runs

Ok - but what a hacky way to determine this and why I was dead against (re-)naming all the plugins in this way.

Complicate every plugin for the convenience of watch? I thought avoiding that was the entire reason we went down the road to result_store

We've already gone down this road, it's already a partially trodden path - the plugin naming being one step, result_store being another - and we went that route because trying to put that behaviour into *.notes would have caused all sorts of issues (outbound being the main issue, where txn.notes gets copied into hmail.notes).
Adding some redundant code into each plugin to provide optional metadata to another core plugin - I'd be fine with, as we wouldn't be adding the hook into core; it would be implemented inside your watch plugin.  It would be a far less pervasive change than results_store turned out to be (that particular patch-bomb was so big that I haven't bothered to merge my own plugins with core since, so improvements I've made locally haven't been contributed because I haven't had time to merge it and shake out all the bugs).  It also wouldn't be modifying the actual plugin code, just adding some hook_init_watch and hook_watch_result functions, that without the watch plugin being loaded, will never get called.

The difficulty I see in being able to benefit from such an arrangement is deciding how best to display the data. Should access, uribl and karma have a box in every phase because they report a result there? Or should they be aggregated into one box as they are now with a tooltip for more info? Which phase? There's a slew of similar issues like that boil down to opinions about how best to present the data. I expect some evolution of watch in this area.

That's why I'm suggesting that you do it with metadata supplied by the actual plugin.  Surely you can come up with some sort of meta API to do that?  A init_watch hook called by watch at init_master would seem to be an obvious choice for that.   It could identify each plugin and which hooks they want to report on and where the data to report is stored etc.

Thanks again for you feedback.

No problem - you're welcome.


How can I crop a feature map through given coordinates like croping an image？

Hi, everyone. I'm trying to crop a feature map from given coordinates, like ROIAlign operation in resent paper Mask RCNN. Is there any way to do such thing?

ROIPooling?

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!


Feature Request: Expanded class like flutter

Reference: https://api.flutter.dev/flutter/widgets/Expanded-class.html

Every flet control has a prop named expand. It internally implements the flutter's Expanded widget.

here in flet... flex is not available, whereas in flutter it has flex param

In Flet you do expand=2, for example: 2 here is actually the flex.

In Flet you do expand=2, for example: 2 here is actually the flex. Flet's API tries to keep things simple...

Thanks a lot for informing