indexUpdate: remove index from index bucket if len(indexValue) == 0

I think empty indexValues should be deleted to keep the index clean.
I have for example an index that maps categories to articles and if i delete all articles that belong to a category i get:
bolter -f kb.db -b _index:Article:Category
Bucket: _index:Article:Category
+----------------------------------------+-----------------------------------------------------------------------------------------------------------------------+
|                  KEY                   |                                                         VALUE                                                         |
+----------------------------------------+-----------------------------------------------------------------------------------------------------------------------+
|                                       |                                                                                |
| 01B870NPNFZHNM83BA8VGB0B15            |                                                                                |
| 01B8M37D74MCTYYBG954B7TN5R            |                                                                                |
| 01B8MMGYM2X7KZ7N4AGEG5ZEE0            |                                                                                |
| 01B98M4SMZRT8BN0NHESRABHMP            |                                                                                |
| 01B9BNN74QHNSX8TC82HHPH5Q7            |                                                                                |
| 01B9BW4NW4D8RC69KBNDVJWQHW            | 01B9BW6ZTEJ7BGE73KEXJFQMZB01B9C0N9M24FW5XQ5CK6QQRX6701B9K46JXEHPP3D0D5DPS3449R01B9K7FHXN3G2NHF3NPSQFPJ99 |
| 01B9GMWT1SGWC0SQPQF2W2AXK6            | 01B9GPPHG49CH8XW3QBW73AHN3                                                                                |
| 01B9K2KQHCWGHB8VFVSPSP1YD3            | 01B9K24BBK0EFWHMA63EW02HMP01B9K2EKY2PN17BY8NEE454EX5                                                           |
| 01B9KE5CKESACHMDP5PYA2ZJJR            | 01B9KC8RN6EP4P9DXY0M4Q0PJZ01B9KCGPS010JW8E4NDSSCC20H01B9KD56K23S38TEAX8X4VXN5N                              |
| 01B9KE90SWSWBQ0PXJBEWTKR9J            | 01B9KE0WKB048RWS6127RHNGZR01B9KSPWSKMCPV2R3T4JWQA2VJ01B9KTVP2BYXMQD2VDXCD2S2AZ                              |
| cfeb78d5-2ff2-43b0-91d0-389eb1a7b1eb |                                                                                |
+----------------------------------------+-----------------------------------------------------------------------------------------------------------------------+

Yep, I agree.  That was an oversight, and with large data-sets it could definitely become an issue.

Also, it looks like all test pass, I need to get travis working with coveralls for prs.


[RFC] something I tried that actually makes it slower

Hi,
I took a look at your buffer-optmz branch.  I know it's not finished, but I tried to experiment with it and build on top of it.  Eventually, though, I realized that what I was trying to do was actually sort of an alternative to what you're doing, so I switched back to master.
Whereas your branch changes Connection to conditionally flush after writing depending on a flag, I change it to never flush after writing.  Instead, the event loop flushes pending writes when it's about to block on reading.
This approach works better with the other part of my change, which is to allow parallel calls from multiple fibers.  I add a helper method run_par which takes a list of callbacks and runs each of them on a new fiber, resuming the original fiber when all have finished.
As a first use, I change the code that re-requests the current buffer and window after every legacy Vim API call to do so in parallel:
  @__client.run_par [
     lambda { resp = @__client.public_send(method, *args, &block) },
     method(:refresh_curbuf),
     method(:refresh_curwin),
  ]

I'm think (but am not sure) it's valid to send these requests 'in parallel' with the caller's actual request, which might change the current buffer/window.  The use of fibers rather than real threads guarantees that the requests will be sent out in order, and on neovim's end, I think it serializes request processing, so it won't handle the later requests until it's finished the first one?
Anyway, I confirmed this results in fewer write() calls / context switches.  Unfortunately, it also makes LustyJuggler much slower, rather than faster!  I think the creation of fibers and other objects for every request is adding GC pressure...  So for now this is kind of useless.  I'm submitting it anyway in case you had any thoughts on it – in particular, even if run_par is a bust, I think this approach to flushing is better.  I'll also try to see if I can get it running faster.

Did you see any improvements with just the flush-before-read change?

Gonna close this, feel free to reopen if you want to pick it back up.


Bump up axios to make npm audit quieter

Current status:
axios  <=0.21.1
Severity: high
Incorrect Comparison in axios - https://github.com/advisories/GHSA-cph5-m8f7-6c5x
Server-Side Request Forgery in Axios - https://github.com/advisories/GHSA-4w2v-q235-vp99
Depends on vulnerable versions of follow-redirects
No fix available
node_modules/axios
  spamwatch  *
  Depends on vulnerable versions of axios
  node_modules/spamwatch

axios has had a few updates since the last update of this library - https://www.npmjs.com/package/axios - would be nice if it could be bumped up

Fixed in<PHONE_NUMBER>08434d25385b7db391d45e3210304f.


Exception when is handling events in android mobile WebViewFragments

In line https://github.com/silviomoreto/bootstrap-select/blob/master/js/bootstrap-select.js#L136
the Event constructor is throwing exceptions because the protection in line
https://github.com/silviomoreto/bootstrap-select/blob/master/js/bootstrap-select.js#L134
is not enough when the constructor Event(name, options) is not defined and only is defined the  constructor Event().
This is happening to us when we change from version 1.4.3 to 1.10.0.
You are including the Event(...)  from 1.7.3
And we are happy only until 1.7.2  :-)
Maybe some like this could be a solution to the issue:
      if (typeof Event === 'function') {
        // For modern browsers
        try {
          event = new Event(eventName, {
            bubbles: true
          });
        } catch (error) {
          event = document.createEvent('Event');
          event.initEvent(eventName, true, false);
        }

What version of Android mobile are you using? When using that same version, does it say your browser supports DOM event constructors here: http://rauschma.github.io/event_constructors_check/?

We are using it inside WebView, by example, the issue happens using it with these two user agent:
User-Agent=Mozilla/5.0 (Linux; U; Android 4.1.2; es-es; GT-N8000 Build/JZO54K) AppleWebKit/534.30 (KHTML, like Gecko) Version/4.0 Safari/534.30
User-Agent=Mozilla/5.0 (Linux; Android 4.4.2; GT-I9515L Build/KOT49H) AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/<IP_ADDRESS> Mobile Safari/537.36

Does it say your browser supports DOM event constructors here: http://rauschma.github.io/event_constructors_check/?

Ok,
we use the   http://rauschma.github.io/event_constructors_check/  in a webview and obtain this:

and we have now submitted to you the results by pushing in the  "Submit to Browserscope" button.
We hope you have found this information useful.

Hrmmm...seems like the JS didn't run - it should either say "YES" or "NO" after Your browser supports DOM event constructors (background):. Is JavaScript enabled on the browser?

Closing until you can provide more info.


[FLINK-25053][docs] Document how to use the usrlib to load code in the user code class loader

What is the purpose of the change
Document how to use the usrlib to load code in the user code class loader
Brief change log
272b759c65511b88aab0cc322b38033e89f9f945 Document how to use the usrlib to load code in the user code class loader
Verifying this change
docs, no tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Thanks for review @wangyang0918. I've addressed all your comments. Please help to review it again.

@wangyang0918 The FLINK-24897 has been merged, and according to your last comments,  I have update the "Fix comments" commit. Please help to review it again at your convenience

Merging this PR now.


ENH: Sparse splu for non-square matrices using SuperLU

Is your feature request related to a problem? Please describe.
I tried to LU factorize sparse matrices using scipy.sparse.linalg.splu (uses SuperLU). But it raised an error saying only supported for square sparse matrices.
It can be seen in official documentations that SuperLU routines do support non-square matrix LU decomposition.
Official site line 3 : https://portal.nersc.gov/project/sparse/superlu/
User Manuel page 5 line 4 (section 1.1) : https://portal.nersc.gov/project/sparse/superlu/superlu_ug.pdf
Describe the solution you'd like.
Currently it is restricted for input matrix to be square. We can remove that and modify gstrf function call like gstrf(M, N, .....) to allow non square matrices.
Few changes will be required too in C implementation of interface between SuperLU and Python.
Describe alternatives you've considered.
No response
Additional context (e.g. screenshots, GIFs)
No response

I changed few files and managed to add support for non square matrices
link to commit - link
I tested for 3x2 and 4x2 matrices - It worked
I tested for 2x3 and 2x4 matrices - It didn't work
I attached script and output.
out.txt
script.txt

It turns out there is SCIPY_FIX in SuperLU/SRC/bdsnode_bmod.c  => It inhibits factorisation of matrices like 2x4.
If we comment out that FIX  => SuperLU/SRC/dpivotL.c returns non zero value (It should return 0 for successful pivoting, non-zero value means some diagonal element of U is 0 or U is singular). ----- It also uses its own SCIPY_FIX.
I need help on these issues.

Thanks @Shay2Shay for the deep dive into SuperLU. I am not a domain expert but opening a PR would probably be the best way to progress here.


[Stress tester XFails] Update XFails

Remove a few timeout XFails that are no longer occurring.
Resolves rdar://90030545.

Ah, looks like these aren't consistently passing now. We might need to change the timeout instead. Going to mark as draft.

We have a mechanism in place that marks test cases as soft-xfail if they take more than 1/2 of the timeout. soft-xfails don’t cause an XFail failure but stop an XFail from causing an XPass. So removing the timeout xfails once they cause an XPass for the first time is the correct way to go.


Is there a way to control the attack of the playback?

Would love to be able to ramp the volume of playback to reduce clicks of starting/stopping mid waveform.

Did you use wavesurfer.setVolume()? I do not understand why you must start/stop?

If I understood correctly what you meant, and if you're using the WebAudio or MediaElementWebaudio backend, maybe you can use the AudioParam.linearRampToValueAtTime(value, endTime) method, applying it to the GainNode.


UQS smart back navigation (EXPOSUREAPP-9325)

Changes
Navigation back from negative and pending test results.
ToDo
⚠  Important ⚠  : verify back navigation behaviour on final screen (should back stack be wiped after navigation from test results?)
Story
EXPOSUREAPP-8903
Subtasks
EXPOSUREAPP-9325
How to test
For pending and negative test results:

Scan them from all scanning start points (Submission Dispatcher fragment, CheckIns tab, Certificates tab)
Proceed with submission flow
Check that troll bar and device button leads to the point where you start

⚠  Note for testers ⚠  : as for acceptance criteria, starting with submission dispatcher fragment would lead to main fragment at the end of the flow and not submission dispatcher.

Please have a look at https://github.com/corona-warn-app/cwa-app-android/pull/4089

Please have a look at #4089

Thanks, this is better approach.


Add sensitive marker to logs

Description
Add sensitive marker to logs to mask s3 object key
Issues Resolved
N/A
Check List

[ ] New functionality includes testing.
[ ] New functionality has a documentation issue. Please link to it in this PR.

[ ] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks for providing the change. Could there be more places, where this kind of change would make sense?

Thanks for the review! Yeah, there could be. We've been trying to catch those from security reviews (like this one) and being conscious about potential sensitive data in logs during development.


SyntaxError: invalid syntax

Hello there,
I have installed the script on my Manjaro Linux and when trying to run it using this command
hacktorian
i got the following error

Hello, as I see you use Parrot OS as I do. On Parrot, Python3 is used by default. The "hacktronian" program, however, uses Python version 2.7
Therefore, if you want to start the program, you have to enter the command "sudo python2 hacktronian" in order to start the program. I hope I could help you.

Hello, as I see you use Parrot OS as I do. On Parrot, Python3 is used by default. The "hacktronian" program, however, uses Python version 2.7
Therefore, if you want to start the program, you have to enter the command "sudo python2 hacktronian.py" in order to start the program. I hope I could help you .


Setting SelectField multiple prop

How can I set the multiple prop of the SelectField component to true (as per material-ui docs)?
I tried putting multiple on the Field component, and it does get passed down to SelectField in the DOM as multiple=true, but the prop seems to have no effect on the behavior of the component. In other words, it behaves as if multiple were false.
Example code:
<Field name={this.getDynamicName()} component={SelectField} floatingLabelText={this.getDynamicLabel()} hintText="Select..." multiple>
    {this.renderMenuItems()}
</Field>

Interesting.  I am able to get the multiple functionality to start triggering the expected behavior.  The problem I have after is manipulating the state in the redux store while using seamless-immutable.  I noticed that SelectField uses DropDownMenu which calls a method handleMenuItemTouchTap.  handleMenuItemTouchTap uses array.push and array.splice to manipulate the change of value.  Not sure if these two are related but it could be why you are not seeing any items add to your state.

Ah, I'm using redux-form/immutable with react-boilerplate, so that's probably the issue.

It appears Material-Ui is going to make changes so you can access the properties of the component  not sure the intended release for it though but you can see it here

Hello @kdavidmoore -- I see that this issue was closed, but I'm not sure I understand the solution? I'm running into a similar (if not the same) issue trying to create a "multiselect" with this library.
Are you able to illustrate how you solved this?

I have the same problem as @takealuke .
Even though I have multiple={true} on SelectField, I can see the index in
handleChange is set.
According the the documentation, index should be undefined if multiple={true}


Update PyO3 Bindings

The PyO3 version used in this repo is not compatible with the latest. This pr updates the dependency to PyO3 and removes a now unnecessary import. I also added 2 little conversion tests, which are not executed by the ci (this would require additional ci config for the Python interpreter).

It might also be a good idea to keep the version of pyo3-tch in sync with pyo3, not tch


ravamped printing of PowerSystem type

converted NamedTuple for generators to a dedicated structure, "GenClasses"
smart printing of PowerSystem type
overload of summary command (in addition to show) for PowerSystem (and the new GenClasses) -- helpful with output of varinfo()

I have in mind to create some print tests before merging this, but I wanted to make this preview available now.
First, please let me know if my changed type for PowerSystem.generators is OK. Scanning to see how it is later used, I think my changes are OK. The existing tests in PowerSystems passed, but I haven't yet tried running tests in PowerSimulations with this change.

@jd-lara I had that exact concern. In my initial search for uses of "generators" in PowerSimulations, I did not find where iteration was used, but I see it now in device_retreval.jl. Do you have an objection to use the struct type if I implement an iterator for it? See latest commit efaaf22906346b4e990ef44dbca4798d85d9548c . I tested this so far with:
    if source isa Nothing
        println("nothing")
    else
        println(source)
    end
end

I recognize this almost a case of the tail wagging the dog, but I think there are advantages (in addition to printing) for having PowerSystem.generators be a custom type rather than a generic NamedTuple. It cleans up the PowerSystem constructor code, and is consistent with all the other subcomponents being custom types. If you are amenable, I will gladly run PowerSimulations tests to make sure nothing breaks.

Codecov Report

Merging #124 into master will decrease coverage by 0.48%.
The diff coverage is 13.63%.

@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
- Coverage   50.73%   50.25%   -0.49%
==========================================
  Files          36       36
  Lines        2919     2949      +30
==========================================
+ Hits         1481     1482       +1
- Misses       1438     1467      +29

Impacted Files
Coverage Δ

src/utils/print.jl
0% <0%> (ø)
:arrow_up:

src/base.jl
92.3% <100%> (ø)
:arrow_up:

src/models/generation.jl
64.7% <28.57%> (-26.21%)
:arrow_down:

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 d1e3119...efaaf22. Read the comment docs.

@kdheepak wrote some tests for printing a few months ago. It looks kind off like this
@assert "$PowerSystem" == "PowerSystems.PowerSystem(buses=5, generators=7, branches=6)"

Basically compares the output with a string that resembles what you expect to see when printing.

Please check whether the last two commits here look OK to you:
https://github.com/NREL/PowerSystems.jl/pull/124/commits/dcced39462edaaf18e200d1838751f3662d2f0ff I overwrote the printing of TimeArray from TimeSeries because their implementation does not differentiate between REPL (text/plain) output and print() output.
https://github.com/NREL/PowerSystems.jl/pull/124/commits/00c31e72339772846cfb8ead287d969927b3c795 Adds the printing tests. I think I did the right thing here, but let me know if I missed something important or if I did something naive. If I missed a test for the printing of your favorite subcomponent, I would prefer to add it later.

This looks good to me. I think you can merge


Why pass the RowMetadata argument as an additional parameter?

Hello! I am writing an R2DBC implementation for YDB.
Interface Result has a map method where two parameters are expected BiFunction(Row, RowMetadata), but why pass RowMetadata here if there is a getMetadata() method for the Row interface.

https://stackoverflow.com/questions/77472326/why-pass-the-rowmetadata-argument-as-an-additional-parameter


Gravity Spy subject selection service

Describe your change here.
Review checklist

[ ] First, the most important one: is this PR small enough that you can actually review it? Feel free to just reject a branch if the changes are hard to review due to the length of the diff.
[ ] If there are any migrations, will they the previous version of the app work correctly after they've been run (e.g. the don't remove columns still known about by ActiveRecord).
[ ] If anything changed with regards to the public API, are those changes also documented in the apiary.apib file?
[ ] Are all the changes covered by tests? Think about any possible edge cases that might be left untested.

minor comments LGTM

@camallen quick catch-up review on that last commit?


set packageserver replicas to 1 for single node

The single-node topology should allow only a single instance of the package server. The 1 replica default was to avoid fatal discovery errors for the packageserver api, which have been updated to non-fatal errors as of https://github.com/kubernetes/kubernetes/pull/73035.
Reverting the default packageserver replicas to 1

/approve

/retest

/retest

/lgtm

/retest


"Extra" ratings granted date

In gitlab by @CalumTowers on Aug 1, 2016, 17:54
Extra ratings such as an I1 will show exactly when it is granted instead of a length of time and then displaying an exact time in the tooltip.

In gitlab by @NFarrington on Aug 6, 2016, 13:45
Fixed by 93c2729e9524e2f0af7c55ddc5e0b6a2b447289c


Cannot create or rename entries in Explorer on a large repo

VS Code Version: 1.55.2
OS Version: macOS Big Sur 11.2

Steps to Reproduce:

Open a git repo with a large number of files ( > 500,000)
In the Explorer, create a new file
Wait up to 10 seconds

Expected: The file name entry should continue to exist until file name is entered
Observed: The file name entry is randomly terminated, resulting in either no file (if file entry was empty) or a partially-named file (if mid-typing)
This might be a duplicate of https://github.com/microsoft/vscode/issues/114573, but as pointed out on that issue, I am still seeing this problem even though the previous issue is closed.

Does this issue occur when all extensions are disabled?: Yes

Dup of https://github.com/microsoft/vscode/issues/112293

I also updated to my version of VS Code today and now am seeing this issue. I don't know the cut-off, but this is only happening on a very large repo, and not in any of the smaller microservice repos I contribute to.


⚠ Bofrost.lu has degraded performance

In 38242f3, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2447 ms

Resolved: Bofrost.lu performance has improved in c6c2516 after 41 minutes.


'go vet' as part of Go starter workflow

Why:
What is 'go vet'?[1]
'go vet' should be part of most Golang projects when building. Because of this, adding the command to the 'Go starter workflow' hoping more projects will include it in their pipeline.
[1] "https://pkg.go.dev/cmd/vet" Vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string. Vet uses heuristics that do not guarantee all reports are genuine problems, but it can find errors not caught by the compilers.
What's being changed

'go vet' command added as part of Go starter workflow

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@SchulteMarkus Thanks so much for submitting a PR! I'll get this triaged for review :zap:

``` Apteka


Add Proxy Support to Enhance Accessibility in Restricted Networks

Description Currently, users in some regions, particularly in China, are facing issues with IP blocks that prevent them from accessing certain services directly. This has impacted the usability of the software, as direct connections to services like speech.platform.bing.com are not possible without bypassing these restrictions.

Proposed Feature I propose adding a proxy configuration option to the application settings. This would allow users to specify an HTTP or HTTPS proxy through which all traffic will be routed. This feature would be particularly beneficial for users attempting to connect from regions with restricted internet access.

Benefits

Accessibility: Users in restricted regions could utilize the software without interruptions.
Flexibility: It provides more options for users who might need to route traffic for reasons other than censorship, like security policies within corporate networks.
User Experience: Enhancing connectivity options can lead to a broader user base and improve satisfaction for current users who are facing connectivity issues.
Additional Context Many users in China rely on proxies or VPNs to access various internet services due to local regulations and restrictions. Incorporating built-in support for proxy configurations would significantly enhance the functionality and appeal of the software among this user base.

Thank you for considering this feature request. I believe it would make a significant improvement in the accessibility and usability of the software for many users facing network restrictions.

I'm sorry that tungstenite-rs doesn't support http or https proxy. https://github.com/snapview/tungstenite-rs/issues/177
I think it may be a little hard for me to implement one by myself. I'll have a try, but don't expect too much from me.

Thank you so much for your project—it has brought me a lot of enjoyment! I also just tried it myself and realized that indeed, wss doesn't work with an http proxy. I've been using Clash in tun mode, which works somewhat, but it seems implementing this fully would be quite challenging. No worries, though—regardless of the outcome, I really appreciate all your hard work and the amazing code!

speech.platform.bing.com is target to an IP in North America, but it seems not be restricted in the region.

wss could worked with http proxy. It is just not implemented by tungstenite-rs

Oh, I see! Looks like we'll have to wait for a solution from the source. Thanks so much for letting me know, and I really appreciate all your efforts!


ARC linker script: application overflows the ICCM

Hi,
I'm in the progress of migrating our code base from Zephyr v2.7.4 to v3.3.0. We are using the GNU toolchain from SDK 0.15.
I noticed that when I build our app, the map file shows that the 'datas' section exceeds the size of the ICCM. This did not happen when I built the same app in 2.7.4.
I see in the map file that this happens because the linker creates a fill area at the end of the rodata section, which causes  __rodata_region_end to point to the end of the ICCM.
The LOADADDR of the datas section is right after this, leaving no space to the datas section in the ICCM. This leads me to lose a lot of valuable ICCM area and it is critical for us as it is our ICCM ROM.
Was there any padding or alignment that was added between these versions that causes this issue?
How can I overcome this?
I think the offending line is include/zephyr/arch/arc/v2/linker.ld:140:
           MPU_ALIGN(ABSOLUTE(.) - __rom_region_start);
Once I remove it the build passes.
What do you guys think?

This was added in #46775 to support XIP with MPU protections. And with power-of-two size/alignment restrictions this indeed may be wasting some memory. If MPU protection is not relevant, or if you can e.g. merge more regions with compatible attributes, the alignment is probably not relevant to you.

Hi @jacoba-at,
may I ask you to provide build log of v2.7.4 and v3.3.0?
Most interesting part is memory region statistics (which is shown at the end of the build log) and the error message you see with  v3.3.0.
It also would be good to have .config file to check Zephyr features / configuration.
Thanks!

build_2.7.4.txt
config_2.7.4.txt
zephyr_2.7.4.map.txt
[build_v3.3.0.txt](https:
zephyr_pre0_3.3.0.map.txt
//github.com/zephyrproject-rtos/zephyr/files/10873567/build_v3.3.0.txt)
config_3.3.0.txt

@evgeniy-paltsev Our ICCM ROM starts at 0x60050000 and ends at 0x60060000.
Note in the 3.3.0 map that __rodata_region_end is set to be the same as __rom_region_end, with a fill of 0x6680 bytes, while in the 2.7.4 map, the __rodata_region_end  is before  __rom_region_end.


outIn eases, similar to the ones found in the exschwasion template

For the first half of the ease it is out, and for the second half of the ease it is in. Can be pretty good for moving things on and off screen smoothly.

haha whoops did not mean to press that

Done! They'll be in the next version


Diagrams.assert calls Future two times

One line assert from org.scalatest.diagrams.Diagrams calls Future two times:
import org.scalatest.diagrams.Diagrams
import org.scalatest.funsuite.AnyFunSuite
import scala.concurrent.ExecutionContext.Implicits.global
import scala.concurrent.duration.{Duration, SECONDS}
import scala.concurrent.{Await, Future}

class ReproduceTest extends AnyFunSuite with Diagrams {
  test("reproduce") {
    assert(Await.result(Future { println("in future"); 42 }, Duration(1, SECONDS)) == 42)
  }
}

result:
in future
in future

Two line assert is ok:
    assert(Await.result(Future { println("in future"); 42 },
      Duration(1, SECONDS)) == 42)

result:
in future

Checked versions:

scalatest 3.2.12, scala 2.13.7
scalatest 3.2.14, scala 3.2.1

@antarcticer Fyi I submitted the following PR for this:
https://github.com/scalatest/scalatest/pull/2219
If it is approved by @bvenners , we can include this in the upcoming 3.2.15 release.


Speedup CI and reduce test run time

What is the problem the feature request solves?
It looks like that CIs are quite slow, the most time consuming parts varies from 60m to 100+m.

It would be great that we could speed them up and hopefully keep them under ~30m.
Describe the potential solution
No response
Additional context
No response

We can probably split the shuffle related tests into separate CI jobs

... or reduce the amount of data in shuffle, right now it is 10000 in many tests.

@advancedxy @viirya what do you think if we move all the MacOS CI pipelines to post-commit, and only keep the ubuntu-latest pipelines as pre-commit? It seems the former are pretty slow especially with Java 8, while the latter are usually faster (~40min).
I think having ubuntu-latest should be sufficient to catch any potential issue in the PRs.

Okay for me. If we find any thing wrong withe Mac OS, we can take action after commit to revert it or fix it.

OK, let's keep ubuntu-latest and MacOS Apple Silicon in the pre-commit then, and move MacOS X86 to post-commit. I'm also making changes in #109 to improve the test time in shuffle & aggregation. Will ping you once it is ready.


Keras implementation help

Versions

TensorFlow version = 1.15
Python version = 3.7
Operating system = archlinux

Issue

Hi i am trying to implement  this with keras but i am stuck with LSTM and ctc layers
here is my model summary please have a look at it

what are the things do i need to keep in keras model.
is the above model structure will work on this

@shekarneo Sorry I haven't seen your edit. The above model structure will probably work. You can overfit it on a batch and see

what is the sequence length you have used, can i have your tg id for discussion

I'm not sure what tg id mean :P

sorry, telegram id or any

Sorry I don't use telegram, you can hit me up on Facebook instead :P

sorry sir, i don't use facebook can we met on instagram, whatsap or gmail/g+

:3 Google hangout at<EMAIL_ADDRESS>will OK

please accept my invitation in hangouts<EMAIL_ADDRESS>


Mithril IE9 m.route(url...) using #

In the docs i saw that IE9(& lower) m.route(...) doesn't work because of pushstate support.
My app uses # for urls, and it seems you can just create a quick work-around for getting this to work in IE9 without refreshing. (don't know about other IE's)
Maybe you could add this work-around in mithril itself, or something similar. I don't really need it though, just asking your opinion on this, if it's bad idea or not.
So here it is, i used jquery for adding the html + setting the Attr. I haven't tested this version.
document.body.innerHTML += '';
var mRedirect = function(url){
    document.getElementById('mithrilVirtualLink').setAttribute('href', '#'+url);
    document.getElementById('mithrilVirtualLink').click();
}

I have to end up doing this as a (hopefully) temp hack for IE9 not rendering the view on m.route.
// Called only if IE<=9
function redirect(url) {
  window.location.href = "#" + url;
  setTimeout(m.redraw, 0);
}

The big problem I'm noticing that it may be in IE9 is that hash changes are not being picked up by Mithril's routing. For instance, in this fiddle I have to set the hash in advance before bootstrapping the routes so it shows something. Without that the page does not render at all in IE9. Chrome/Firefox/Safari/IE11 works as expected. Even with setting the hash in IE9 clicking on the links do change the hash but does not cause page updates.
This is also apparent in https://gist.github.com/lhorie/681e23f3f0014b16aa6d. I tried this and in IE9 the m.module part renders fine whereas the m.route part does not render. No errors are thrown when debugging.
I tried with 0.1.30, 0.1.22, and 0.1.24 and neither worked for me.

This is fixed and will be released in the next version


Why DropdownMenu.ItemIcon is not showing when used in a custom component?

Hi, thanks for this great lib!
I have a question :)
import * as DropdownMenu from "zeego/dropdown-menu";
import { DropdownMenuLabel } from "../DropdownMenu";
import { dropdownStyles } from "../DropdownMenu.styles";
import { View } from "react-native";
import { PressableFeatherIcon } from "../PressableFeatherIcon";
import { ComponentProps } from "react";

const FrontMenuItem = DropdownMenu.create(
  ({
    onSelect,
    androidIcon,
    iosIcon,
    key,
    children,
  }: {
    androidIcon: string;
    iosIcon: ComponentProps<typeof DropdownMenu.ItemIcon>["ios"];
    key: string;
    children: string;
    onSelect: ComponentProps<typeof DropdownMenu.Item>["onSelect"];
  }) => {
    return (
      <DropdownMenu.Item
        key={key}
        onSelect={onSelect}
        style={dropdownStyles.item}
      >
        <DropdownMenu.ItemTitle>{children}</DropdownMenu.ItemTitle>
        <DropdownMenu.ItemIcon
          androidIconName={androidIcon}
          ios={iosIcon}
        />
      </DropdownMenu.Item>
    );
  },
  "Item",
);

export function FrontMenu() {
  return (
    <DropdownMenu.Root style={{}}>
      <DropdownMenu.Trigger>
        <View>
          <PressableFeatherIcon
            color="blue"
            featherIconName="plus-circle"
            iconSize={32}
            pressableStyle={({ pressed }) => ({ opacity: pressed ? 0.5 : 1 })}
            onPress={() => {}}
          />
        </View>
      </DropdownMenu.Trigger>

      <DropdownMenu.Content style={dropdownStyles.content}>
        <DropdownMenuLabel>Help</DropdownMenuLabel>

        <FrontMenuItem
          key="mine"
          androidIcon="ic_menu_gallery"
          iosIcon={{ hierarchicalColor: "#00FF00", name: "photo" }}
          onSelect={() => {}}
        >
          See more 1
        </FrontMenuItem>

        <DropdownMenu.Item
          onSelect={() => {}}
          key="zeego"
        >
          <DropdownMenu.ItemIcon
            androidIconName="ic_menu_gallery"
            ios={{ hierarchicalColor: "#00FF00", name: "photo" }}
          />

          <DropdownMenu.ItemTitle>See more 2</DropdownMenu.ItemTitle>
        </DropdownMenu.Item>
      </DropdownMenu.Content>
    </DropdownMenu.Root>
  );
}

the outcome is this:

so you can see from the image that when I use DropdownMenu.Item with DropdownMenu.ItemIcon directly, it works. When I wrapped it inside a custom component like <FrontMenuItem>, it stops working.
What am I missing here, thanks :)

Please see the docs on creating custom components.
Zeego is doing magic here and will not work on native if you wrap components without explicitly passing props in render. Zeego is extracting the props from your components and passing them to the native side through some tricks, and that’s why it works the way it does. As a result, this is the one limitation…
which is to say, this is expected, but there’s always a way to make it work somehow.


Base64UrlEncoder performance improvements

Changed code paths to encode with bytes always. Temporaly breaking paths to EncodeString TBD if we can remove it.
Encode(byte, int, int) is optimized, as well as current UnsafeEncode (part of the EncodeString code path)

Are there any performance tests for comparison before and after with improvements?

@bgavrilMS

Thanks for doing this work @cesarpayan. It looks like Encoding operations are now ~2x faster and take 50% less memory. Pretty good win.
MSAL PR + perf tests including memory: https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/pull/2879/


Upgrade Docker Base to GoLang Image

Description:  Upgrade Docker Base to GoLang Image
Changes Proposed in PR:
modified Dockerfiles for debian, ubi, debug mode to a single multistage golang based dockerfiles
modified makefile targets to suit above and hence removed unused files in build-tools
Added new target in makefile for docker tag and push
Fixes: Internal
General Checklist

[ ] Updated Added functionality/ bug fix in release notes
[ ] Added examples for new feature
[ ] Updated the documentation

CRD Checklist

[ ] Updated required CR schema

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run


Test

travis test should fail

@RWOverdijk test failed but travis passed it?!

@doktordirk That is interesting. It might be this line: https://github.com/SpoonX/aurelia-orm/blob/master/build/tasks/test.js#L17 which was needed back when there was a bug that caused karma to take forever with returning anything.

tja, wasn't that

Woa... Then the exit code isn't 1.

ah, i saw at some time a karma issue mentioned. let me try that

fixed


Add cookie and privacy policies

Starting point for cookie and privacy policies

Coverage remained the same when pulling fb706e80a714ac06f4e879ac2596e93ba9e86163 on feature-policy-pages into 7d18676ace92f49f9bf639a8cd7bd184abea210d on master.

Coverage decreased (-0.95%) to 79.78% when pulling a22531d0a20c888095f3d9c7ebb5d36e6051c8f4 on feature-policy-pages into 7d18676ace92f49f9bf639a8cd7bd184abea210d on master.


Problems with debug start

Every time, when I try to start the debug, my screen just flash the other views, but close immediately.
This is my .sublime-project:
{
	"folders":
	[
		{
			"folder_exclude_patterns":
			[
				"node_modules",
				".pic"
			],
			"path": "/home/rafael/Documents/Projects/KRS/Clientes/Terra Azul/physical_test-frontend"
		},
		{
			"path": "/home/rafael/go/src/physical_test-api"
		}
	],
	"settings":
	{
		"delve_enable": true,
        "godebug_executables":
        {
            "debug":
            {
                "mode": "debug"
            },
            "test":
            {
                "mode": "test"
            }
        },
        "godebug_port": 7890,
        "mode": "debug",
        "host": "localhost",
        "port": "3456"

	}
}

Hi Rafael
If actual, please tell me the version of the operating system, sublime 3, delve, golang

Thanks. :smile:
OS:
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 18.04.1 LTS
Release:	18.04
Codename:	bionic

Sublime Text 3:
Version: 3.1.1, Build 3176

Delve:
Delve Debugger
Version: 1.1.0
Build: $Id: 1990ba12450cab9425a2ae62e6ab988725023d5c $

Go:
go version go1.9.2 linux/amd64

The test command ("mode": "test" in your .sublime-project) allows you to begin a new debug session in the context of your unit tests, see dlv test for details
If there are questions, I'm ready to answer them


Update README to make FRSCA clearer

This is the first of multiple upcoming changes to the FRSCA docs
to make the purpose of FRSCA clearer, how to use FRSCA today and
how to start developing on FRSCA itself.
Signed-off-by: Michael Lieberman<EMAIL_ADDRESS>

Looking for feedback on how I can do the three primary goals of updating these docs:

Make the intent of FRSCA clearer. What is it's purpose?
Make it easier for folks to learn how to start using FRSCA
Make it easier for new contributors to get started with developing FRSCA itself.


[Feature] Variables selector

In this review:

I added feature of selecting hosts by its' variables. There are variables and *no_variables
Variables should  be present in inventory for host and no_variables should not. (passed as var:val)
Also it's possible to specify just variable that should be defined no matter what value it has (just var
passed ).

I moved argument parsing into separate Parser class, because it took too much space in main.py in
my opinion.

I added unittests for variables feature, but also for hostnames selector and moved hostname parsing into Parser.

Also, please rebase the branch to include new CI test

Thanks a lot! ;)

Thank you my friend :)


Added Robinhood broker class

In order to make the Robinhood class I modified main.py to not need the broker-uri when Robinhood is selected. I added the 'rh' directory in brokers. This includes the modified Robinhood API and the modified pyEX API so they can be installed with zipline. I also added the google bundle downloader which is still a little buggy.
I've done a good amount of testing to see if all the functionality works, but I could use some help.

Thank your for your contribution. Unfortunately additional work needs to be done before we can merge it. I still haven't got through completely, will need at least one-more pass to better understand what's going on. Meanwhile: could you please address my comments, rebase your branch and fix the flake8 & testcase failures? Thank you!

Coverage decreased (-0.06%) to 86.378% when pulling 2ae0308891ce2cbb2b5b805f11d39f1494198298 on rigoorozco:master into 016fa8f47dd39be29df43071d330c9cb23299dc7 on zipline-live:master.


Stabilize support for Jupyter Notebooks

Summary
This PR stabilizes the support for Jupyter Notebooks and adds them to Ruff's default inclusion set.
Closes: #12456
Closes: https://github.com/astral-sh/ruff-vscode/issues/546
Test Plan

[x] Make sure all tests pass, updated snapshots are valid
[ ] Ecosystem checks?
[x] Run Ruff in a repository containing both Python files and Jupyter notebooks

There are couple more references in the docs that require updating which I'm doing right now.

Can you rebase/re-run the ecosystem checks. The augmented assignment adds a lot of noise.

There are quiet a large number of violations and some violations seem less appropriate for notebooks (e.g. denying the use of print).

Can you rebase/re-run the ecosystem checks. The augmented assignment adds a lot of noise.

It it up-to date with ruff-0.6 branch.

Let's see if re-triggering the ecosystem checks help

Let's see if re-triggering the ecosystem checks help

Oh, I see what the problem is. The ruff-0.6 branch needs to be rebased on main to remove the merge conflict. The CI is paused in that branch which means it never ran after the reverted commit.

@MichaReiser I've made some edits as per your suggestions. Thanks.
I've also updated some rule documentation to reflect the different behavior for Jupyter Notebooks.

@MichaReiser, I pushed a few small updates. Would you mind having another quick look?


No "Share" (i. e. in gallery) on Android

Description
On the Galery I can "share" the shown image with some applications (like HipChat, Slack, Threema etc.). It would be great if images (or whatever) can also be "shared" with Rocket.Chat.
Devices and Versions
Your Rocket.Chat.Android version: 3.1.1
Your Rocket.Chat Server version: 0.71.1
Moto G4, Android 7.0

Also links or text from other apps like Firefox should have Rocketchat in their sharing options.

duplicate of #98


Add rickroll by rick astley

Song Name
rickroll
Artist Name
rick astley
Source
Youtube
Youtube Link
dQw4w9WgXcQ
Direct File Link
https://www.youtube.com/watch?v=dQw4w9WgXcQ
Song ID
99807
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 44062068
Downloaded: No

Auto nong

Accept
El sáb., 2 nov. 2024 13:17, auto-nong[bot] @.***>
escribió:

Thanks for contributing!
Someone will soon review your request.
Here is the JSON result for the reviewer:
{
"name": "rickroll",
"artist": "rick astley",
"source": "youtube",
"startOffset": 0,
"yt-id": "dQw4w9WgXcQ",
"songs": [
99807
]
}
To accept the request, a reviewer needs to write "accept".
—
Reply to this email directly, view it on GitHub
https://github.com/FlafyDev/auto-nong-indexes/issues/5767#issuecomment-2452972525,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMTQXDJ4Y3TNXQZL7YMJTZLZ6S7GHAVCNFSM6AAAAABRBVWKDWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJSHE3TENJSGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Aceept

Okkkkk?


Why MainActivity_ ?

Hello Gicheon Kang,
why in your app\src\main\AndroidManifest.xml have
<activity android:name=".MainActivity_"
and not
<activity android:name=".MainActivity" ?

I suppose it's using androidannotations,which requests replacing Activity with Activity_.


Changing [value] for multiple select only adds items to selection (does not remove any)

When I change the [value] of a select2 with the "multiple" attribute turned on, options in the new value that were not previously selected become selected, but all options that were previously selected also remain selected -- so my selection is the union of the old values and the new values, rather than just being the new values.
This also means that if I pass an empty array to [value], nothing changes, whereas I would expect all options to be un-selected.
I believe the issue is in setElementValue(), the innermost if needs an else to turn the selected property off when the option is not contained in the new value, like so:
if (newValue.indexOf(option.value) > -1) {
  this.renderer.setElementProperty(option, 'selected', 'true');
} else {
  this.renderer.setElementProperty(option, 'selected', null);
}

Or I suppose the whole block could just collapse to:
this.renderer.setElementProperty(option, 'selected', newValue.indexOf(option.value) > -1 ? 'true' : null);

yes actually adding that fix solves the problem, hope @NejcZdovc could update this plugin


sendable Weak Reference

Hi,
it seems there is no sendable Weak reference?

There isn't (they weren't needed for the use case this crate was developed for), but I believe one could be added in a straightforward way.

When we have a cyclic data structure, which T has a field with weak reference to itself, then we need a SendWeak to make SendRc sendable


configure.ac contains non-POSIX syntax

In GitLab by @perarnau on Jan 6, 2021, 11:41
[[ ]] and == used for tests in configure.ac are not POSIX compliant. On top of that, having a mix of test, [[]], = and == tests is confusing for future maintainers as to whether there's a difference between them in behavior.

In GitLab by @NicolasDenoyelle on Mar 15, 2021, 13:00
mentioned in merge request !174


Update C++ standard to 17

check #970 for details

I compiled successfully on Macos! Thank you very much for your contribution.❤

Can I upgrade rocksdb to 7.x at the same time?

Can I upgrade rocksdb to 7.x at the same time?

@xiaobiaozhao I think we can wait and see because rocksdb7.x is a lot different than rocksdb6.x. I'm not sure what services currently use rocksdb7.x for production (maybe we can ask in the rocksdb issue), and there's no immediate requirement for kvrocks to use the new rocksdb7.x features.

Thanks all. Merging...


🐛透视表分页功能，以树状结构展示行维度，切换分页，在点击角头行图标，表格数据渲染异常

🏷 Version

Package
Version

@antv/s2
1.54.5

@antv/s2-react

@antv/s2-vue

Sheet Type

[v ] PivotSheet
[ ] TableSheet
[ ] GridAnalysisSheet
[ ] StrategySheet
[ ] EditableSheet

🖋 Description

透视表分页功能，以树状结构展示行维度，切换分页，在点击角头行图标，表格数据渲染异常
⌨ Code Snapshots

🔗 Reproduce Link
https://s2-v1.antv.antgroup.com/zh/examples/react-component/pagination/#pivot

🤔 Steps to Reproduce

options添加配置  hierarchyType: 'tree',

切换分页至【2】或者【3】页

点击【省份/城市】角头行的折叠图标两次

再次展开时，表格数据渲染异常

😊 Expected Behavior
预期结果表格数据渲染正常

😅 Current Behavior

💻 System information

Environment
Info

System

Browser

:tada: This issue has been resolved in version @antv/s2-v2.0.0-next.15 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version @antv/s2-v1.54.7 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version @antv/s2-react-v1.46.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version @antv/s2-v2.0.0-next.19 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Access to DataTransfer object on drag events

Would it be possible to add support to the API of the dock panel tab and group for setting the DataTransfer? I have a use case where the tab represents a file, and the group is a directory, and dragging the tab or group onto an input, I need to be able to specify the value of the DataTransfer object on the drag handler.
I've tried my own stab at it, but I have difficulty figuring out how the Tab objects are created in relation to the tab component.

Originally posted by @techbech in https://github.com/mathuo/dockview/issues/298#issuecomment-1634048922

@techbech Just initial thoughts, would something like this work?
interface DragPanelEvent {
    panel: IDockviewPanel;
    nativeEvent: DragEvent;
}

interface DragPanelEvent {
    group: IDockviewGroupPanel;
    nativeEvent: DragEvent;
}

api.onWillDragPanel((event: DragPanelEvent) => { /** */ })

api.onWillDragGroup((event: DragPanelEvent) => { /** */ })

Thank you!
I noted that dataTransfer is read-only, but I guess it will still work.
Just expanding on yours. Is the following how I would use it?
interface DragPanelEvent {
    panel: IDockviewPanel;
    nativeEvent: DragEvent;
}

interface DragGroupEvent {
    group: IDockviewGroupPanel;
    nativeEvent: DragEvent;
}

event.api.onWillDragPanel(({panel, nativeEvent}: DragPanelEvent) => {
    if (panel.params.filePath) {
        nativeEvent.setData("text/plain", panel.params.filePath);
    }
})
event.api.onWillDragGroup(({group, nativeEvent}: DragGroupEvent) => {
    if (group.params.directoryPath) {
        nativeEvent.setData("text/plain", "my-directory/");
    }
})

That would be awesome! And it will also work with serialize/deserialize, right?

Awesome!
I just played around and found the updateParameters(...) method on the group. But it's fine in my case to get the data from elsewhere.
Let me know if you could use any help or could use someone to discuss things with.


Fix float32 rounding issue

Float32 supports 7 significant digits (integer+fractional).
This commit rounds 32-bit floats to 7 significant-digits. Since the float32 values of the PowerTags never exceed 7 digits this returns the original measured values.
For example:
233.1 Volt represented in Float32 becomes<PHONE_NUMBER>03516 Volt.
When rounding to 7 significant-digits the original value of 233.1 Volt is recovered.

Thanks a lot! :)
Will release this after I fix the SmartLink issue as well.


Refactor backend code into routes, middleware, and controllers for each module

I want to refactor the backend code into routes middleware and controller for each module this will increase the modules readability more.

@Luson045 can i work on this issue?

you need to wait it's better not to change the structure now

@Luson045 okayy do you have any other backend work currently? I want to contribute to backend too

@17arindam not yet, i'll open issue as soon i find something

@17arindam okay thanks


i am getting this continously Place the chessboard on the plane where the object needs to be detected.. and i want measure object dimensions using one depth sense camera(D435) and reference plane is white

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ R200 / F200 / SR300 / ZR300 / D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
{ legacy / 2.. }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
<Describe your issue / question / feature request / etc..>

Hi.
Will you be needing further help with this? If so can you please provide more information than a title. If we don’t hear from you in 7 days, this issue will be closed.
Thanks


operation: clean ups to remove nops, flatten and rename

Now that we have flattened the categories, we have several nops that should just be reduced to Control::True, so remove them.
We are also removing object levels from the configuration by making Lookup and FlatMap unit variants, and while at it Lookup has been renamed to Indexes.

bors r+


Another side dock

We have a dock on the side where we show filed, search, ... That used to be ok till some time ago when I started to use VS Code on all my projects. The thing I miss most is to be able to dock another navigator to a new location. For example, I want OUTLINE to be on the right full height always show the structure of my file. I have a lot of files in a project and when I share height between project navigator and outline, I cannot use well any of those.
The ability to drug any navigator to the right and have docks on both sides is ideal.  But at least to get OUTLINE navigator as separate column would be nice too.

See #49923 for this being discussed for Outline.


[AutoDiff] Add differentiation benchmarks

Revives #31108 by @marcrasi.

@swift-ci benchmark

@swift-ci benchmark

@swift-ci benchmark

Interesting... I'm unable to reproduce this on macOS.

@swift-ci benchmark

@swift-ci benchmark

@edymtt any concerns with https://github.com/apple/swift/pull/34901/commits/3e86dc9396a8177eadeed7d2bc8ebdf6f0dc94ea?

@swift-ci please test

Performance: -O

Improvement
OLD
NEW
DELTA
RATIO

CharacterLiteralsLarge
97
87
-10.3%
1.11x (?)

UTF8Decode_InitFromBytes_ascii_as_ascii
452
407
-10.0%
1.11x (?)

UTF8Decode_InitFromData_ascii_as_ascii
644
581
-9.8%
1.11x (?)

CharacterLiteralsSmall
305
284
-6.9%
1.07x (?)

Added
MIN
MAX
MEAN
MAX_RSS

DifferentiationArraySum
1799
2178
1927
—

DifferentiationIdentity
304
306
305
—

DifferentiationSquare
590
591
591
—

Code size: -O

Regression
OLD
NEW
DELTA
RATIO

TestsUtils.o
28270
28810
+1.9%
0.98x

Performance: -Osize

Regression
OLD
NEW
DELTA
RATIO

Breadcrumbs.MutatedIdxToUTF16.ASCII
3
4
+33.3%
0.75x

NSStringConversion.UTF8
652
770
+18.1%
0.85x (?)

Array2D
6272
6752
+7.7%
0.93x (?)

ObjectiveCBridgeFromNSArrayAnyObjectForced
4220
4540
+7.6%
0.93x (?)

Improvement
OLD
NEW
DELTA
RATIO

ProtocolDispatch
333
307
-7.8%
1.08x (?)

Added
MIN
MAX
MEAN
MAX_RSS

DifferentiationArraySum
1681
1862
1743
—

DifferentiationIdentity
311
312
312
—

DifferentiationSquare
600
600
600
—

Code size: -Osize

Regression
OLD
NEW
DELTA
RATIO

TestsUtils.o
22409
23197
+3.5%
0.97x

Performance: -Onone

Added
MIN
MAX
MEAN
MAX_RSS

DifferentiationArraySum
1673
1887
1745
—

DifferentiationIdentity
380
380
380
—

DifferentiationSquare
703
704
703
—

Code size: -swiftlibs

✅
Benchmark Check Report

⚠⏱
DifferentiationArraySum execution took at least 1653 μs.  Decrease the workload of DifferentiationArraySum by a factor of 2 (10), to be less than 1000 μs.

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 8-Core Intel Xeon E5
  Processor Speed: 3 GHz
  Number of Processors: 1
  Total Number of Cores: 8
  L2 Cache (per Core): 256 KB
  L3 Cache: 25 MB
  Memory: 64 GB


test: adds api-extractor check

Codecov Report

Merging #33 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #33   +/-   ##
=======================================
  Coverage   99.71%   99.71%
=======================================
  Files          12       12
  Lines         356      356
  Branches       36       36
=======================================
  Hits          355      355
  Partials        1        1

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 41bc969...52b78f9. Read the comment docs.

@gigobyte thanks for the suggestion in the other issue. I've removed the export and it works now!


MembersResponse lists old pods

Describe the bug
Working with the MembersRequest right now. This is the response I get. I only have 3 meta pods running in my K8s/Kind setup. I assume that the other records are old meta pods that no longer exist.
2023-02-21T10:12:13Z	INFO	member: address:{host:"<IP_ADDRESS>" port:5690} is_leader:true
2023-02-21T10:12:13Z	INFO	member: address:{host:"<IP_ADDRESS>" port:5690}
2023-02-21T10:12:13Z	INFO	member: address:{host:"<IP_ADDRESS>" port:5690}
2023-02-21T10:12:13Z	INFO	member: address:{host:"<IP_ADDRESS>" port:5690}
2023-02-21T10:12:13Z	INFO	member: address:{host:"<IP_ADDRESS>" port:5690}

To Reproduce

Setup your cluster on kind
Create multiple meta pods all managed by a StatefulSet
Delete a couple of these pods using kubectl delete pod <pod-name> --force

Expected behavior

MembersResponse should only ever return the exact number of meta members

Additional context
I am not sure if this is really a bug. For later requests I get 3 meta members, just as expected. So maybe this is just a temporary thing during pod failover?

@shanicky clarified that this is indeed expected behaviour: If a meta node gets deleted, their ETCD lease will not be renewed and the keys will disappear if the lease times-out. The MemebersResponse is thus eventually consistent with the state of the cluster.


Log errors during sync

Coverage increased (+0.04%) to 91.049% when pulling 80fab5f3ddc80ebcb8ed051ed9e88724c57f4a56 on log_sync_summary into 248375cb3f36edfadfabbced8a98a7d041b85ce8 on master.


RifCommsNode base

Description
The objective of this PR is to introduce a first approach to implement the RifCommsNode class, the implementation of TransportNode that uses RIF Communications pub sub node.
On this PR, the abstract methods of TransportNode are implemented and must be iterated on following PRs.
Changelog
Implemented:

__init__ that starts a rif comms client and define the instance attributes of the class:

_config
_raiden_service
_client
_address_to_message_queue
_stop_event
_greenlets

start with some pending TODOs
_run with some pending TODOs
stop with some pending TODOs
send_message (pending _send_with_retry and _send_raw)
no-op start_health_check and whitelist
link_exception and join

TODO's

Global worker and prioritze global msgs?
Join_global_room at start, maybe we can use a bootstrap node for rif comms here?
Log class variable?
what about _run if we dont handle global msgs?
what are self.greenlets used for? Does we really need it? we only need one for leave topics?
does stop need to invoke the end comms and unsubscribe from grpc channel?
self._client.start_listener_thread() ? we need a thread for listen messages

closed in favor of #157


Question

Hi. I'm borrowing some code from this mod, specifically the boats and signs, and I have a question about the signs.
So, I got my signs working and everything but if a add it to any tags, for instance the #signs tag, or the mineable/axe tag it doesn't add it and instead removes the tag from everything else, including vanilla items. And also the model file that assigns it the breaking particles also doesn't work.
Do you know how to fix this or am I just screwed until I figure it out myself?

I'd have to see your code to work that out, can you show me the repo you are working from with this issue?

sorry i didn't see you respond. i wasn't notified in any way plus i accidentally yeeted our modem so i had no internet for a week.
my repo is private. is it possible to give you access without making it public?
i'll try to remember to respond this time, lol.


Conversion of decoder with past_key_values to float16.

Hi @Ki6an. With the converted ONNX model generated. I was trying to convert the decoder_init and decoder to float16. I did the quantization with onnxruntime's transformer optimizer. I was able to convert the decoder_init to fp16 but while converting the decoder with past_key_values I am getting following issue:
AssertionError                            Traceback (most recent call last)
 in ()
25     ,
26     )
---> 27 optimized_model.convert_float_to_float16()  # FP32 -> FP16
28 optimized_model.save_model_to_file('/content/optimized_models/t5-base-qa-qg-hl-decoder.onnx')
29
6 frames
/usr/local/lib/python3.7/dist-packages/onnxruntime/transformers/../tools/symbolic_shape_infer.py in add_suggested_merge(self, symbols, apply)
216
217     def add_suggested_merge(self, symbols, apply=False):
--> 218         assert all([(type(s) == str and s in self.symbolic_dims) or is_literal(s) for s in symbols])
219         symbols = set(symbols)
220         for k, v in self.suggested_merge.items():
AssertionError:
Code used:
from onnxruntime.transformers import optimizer
#Decoder.
optimized_model =optimizer.optimize_model(
input='/content/models/t5-base-qa-qg-hl-decoder.onnx',
use_gpu=True,
opt_level=1, only_onnxruntime= True
,
)
optimized_model.convert_float_to_float16()  # FP32 -> FP16
optimized_model.save_model_to_file('/content/optimized_models/t5-base-qa-qg-hl-decoder.onnx')

@Ki6an
Sorry to revive this issue again. But when I am doing prediction using the above decoder I am getting this issue:
RuntimeException: [ONNXRuntimeError] : 6 : RUNTIME_EXCEPTION : Non-zero status code returned while running Cast node. Name:'graph_input_cast5' Status Message: /onnxruntime_src/onnxruntime/core/framework/op_kernel.cc:78 OrtValue* onnxruntime::OpKernelContext::OutputMLValue(int, const onnxruntime::TensorShape&) status.IsOK() was false. Shape mismatch attempting to re-use buffer. {4,12,1,64} != {4,12,163,64}. Validate usage of dim_value (values should be > 0) and dim_param (all values with the same string should equate to the same size) in shapes in the model.

@Ki6an
The same issue was coming in the fp16 version of decoder_init . But setting use_symbolic_shape_infer=False did solved this issue in decoder init. But I am already doing the same for decoder and I am getting this issue. Could you please suggest any fix?

@Ki6an I am using the fastT5 model for exporting the ONNX does it do it?


opt-out

This is an opt-out request for: all accounts on vulpine.club

Sure! Can you DM<EMAIL_ADDRESS>from a vulpine.club admin to confirm?

vulpine.club doesn't even have any A/AAAA DNS records...

@rtucker alternatively you're welcome to email<EMAIL_ADDRESS>from a vulpine.club address, and I can use DKIM to confirm. Or any other way you want.


Customize class name and Tailwind theme key

Hey!
Sometimes it is useful to change default spinner class to something different. This allows to resolve conflicts between styles or apply spinner to already existing designs. I implemented this functionality, new test and minor README fixes. In addition I added option for changing Tailwind theme key too.

Looks good to me, only request I have is make it so that you don't have to pass any config,

But you don't have to. It should work fine if you use my version of plugin the way you did before
https://github.com/aniftyco/tailwindcss-spinner/tree/caf7433a4ff382396670df9789c7957af4a6f8ce#usage
I use empty object as default spinner config here:
https://github.com/aniftyco/tailwindcss-spinner/pull/10/commits/b76fcc4c08b6d162291eaf6544ba402c4e135c16#diff-168726dbe96b3ce427e7fedce31bb0bcR19
and default class name and theme key here
https://github.com/aniftyco/tailwindcss-spinner/pull/10/commits/b76fcc4c08b6d162291eaf6544ba402c4e135c16#diff-168726dbe96b3ce427e7fedce31bb0bcR93
As I mentioned in README - configuration is optional:
https://github.com/aniftyco/tailwindcss-spinner/pull/10/commits/9c4a3a3d61d07e717c651b798748d40b5bcd70da#diff-0730bb7c2e8f9ea2438b52e419dd86c9R46

you can do the following and get all default stuff.
tailwind.config.js:
{
  plugins: [
    require('tailwindcss-spinner'),
  ]
}

I don't think it is possible to do this without breaking backwards compatibility. Spinner plugin module exports factory function now, not the plugin itself.
https://github.com/aniftyco/tailwindcss-spinner/blob/caf7433a4ff382396670df9789c7957af4a6f8ce/index.js#L19
Did you mean?

tailwind.config.js:
{
  plugins: [
    require('tailwindcss-spinner')(),
  ]
}

I don't think it is possible to do this without breaking backwards compatibility. Spinner plugin module exports factory function now, not the plugin itself.

I think this is a feature of tailwind if you pass it like that, as long as our plugin has default config will work.
Have to do some testing to make sure.

I don't think it is possible to do this without breaking backwards compatibility. Spinner plugin module exports factory function now, not the plugin itself.

I think this is a feature of tailwind if you pass it like that, as long as our plugin has default config will work.

The feature of tailwind is to take a function (as plugin) and call it while initializing. In case of tailwindcss-spinner it must be this function
https://github.com/aniftyco/tailwindcss-spinner/blob/caf7433a4ff382396670df9789c7957af4a6f8ce/index.js#L20
not the one exported from index.js on line 19. The one on L19 is NOT the plugin. It is the plugin factory. This is the reason why you have to call the result of require() now even without my changes (as current README says).
https://github.com/aniftyco/tailwindcss-spinner/tree/caf7433a4ff382396670df9789c7957af4a6f8ce#usage
I can't change this behaviour without breaking backwards compatibility of tailwindcss-spinner. And I don't think I should. In any case this is not the point of my specific Pull Request. If we need this to work like you described, we should start another one.

Have to do some testing to make sure.

https://github.com/aniftyco/tailwindcss-spinner/pull/10/commits/643088341bf25fda875e63efd90db910d7cb6038#diff-1dd241c4cd3fd1dd89c570cee98b79ddR29
It doesn't pass.

Sorry this didn't get merged in @KACAH!

Sorry this didn't get merged in @KACAH!

Np ) Btw, waaay after our conversation about clean require and configuration parameters I actually found the way to make it work as you wanted.

Looks good to me, only request I have is make it so that you don't have to pass any config, you can do the following and get all default stuff.
tailwind.config.js:
{
  plugins: [
    require('tailwindcss-spinner'),
  ]
}

There is a new pull request.


Reference issue

Latest version seems to have a reference bug in or app now. Happened over night.
We’re getting “process not defined”. But haven’t changed our code at all

2.1.0-beta.0 had broken build that leaked the process.env checks to the final bundle. it was fixed in 2.1.0-beta.1


Latest eks-d versions

Issue #, if available:
Description of changes:
I'll merge this tonight to limit the vmware test image churn.
/hold
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/unhold
/approve


docs(scully): fix documentation snippet on router plugin type page

The second code snippet on
https://scully.io/docs/Reference/plugins/types/router/ page is incorrect
for two reasons. Firstly, the snippet is marked as 'Javascript' but it's
strongly typed (uses Typescript) and secondly the 'HandledRoute' is
missing. Change the marked language and update the imports
accordingly.
BREAKING CHANGE:  None
PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/scullyio/scully/blob/main/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?
The second code snippet on
https://scully.io/docs/Reference/plugins/types/router/ page is incorrect
for two reasons. Firstly, the snippet is marked as 'Javascript' but it's
strongly typed (uses Typescript) and secondly the 'HandledRoute' is
missing.
Issue Number: N/A
The marked language is now set to 'Typescript and the import section has been updated accordingly.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
None

 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.


[e2e]: nodejs create main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v39.0.121
Run: https://github.com/slsa-framework/example-package/actions/runs/6231916691
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.nodejs.create.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.nodejs.create.main.default.slsa3.yml
Trigger: create
Branch: v39.0.121
Date: Tue Sep 19 06:25:52 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/v39.0.122
Run: https://github.com/slsa-framework/example-package/actions/runs/6244794580
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.nodejs.create.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.nodejs.create.main.default.slsa3.yml
Trigger: create
Branch: v39.0.122
Date: Wed Sep 20 06:28:40 UTC 2023
Tests are passing now. Closing this issue.


Share code across Uint64List and Int64List.

The implementation of Uint64List and Int64List are identical when it comes to all operations: length, [] []=, ... as Dart has wraparound int64 and cannot distinguish unsigned from signed 64-bit integers.
Due to our usage of copying mixins, we may be able to share code between Uint64List and Int64List.

(Have we considered just doing typedef Uint64List = Int64List; and see if anyone notices? 😉 )


Feature request - add put/get command to store data in NVM

Could you add AT+NVM=a,0xVV and AT+NVM?a where a is an address (0 to 16 or 32) and VV is a 8b hex value.
This command will allow to store and read data in the murata NVM space. One of the use is to store a modem setting state and some extra information.
It is usefull when the nvm capabilities of a board is limited (like mkr1310 - even if it is supposed to have an onboarded spi flash)

I would be happy to consider a pull request if you want to submit one. The command should use the AT$ prefix (AT+ is reserved for Murata-compatible commands).

Unfortunately have so many projects pending :(
I will see if I found some time for doing it

Please feel free to start a new discussion in the Discussions tab if you start working on it and need advice.

Having some train today and setup the environment, will see if I can quickly made an addition for this

I'm close to a working version.
the current limit is the EEPROM available size, I can have a maximum of 4 byte of user data.
The code is only using 3KB on the 6KB available and I assume most of the partitions are over-estimated.
do you think @janakj we can reduce some of the partitions to extend the User NVM space ?

merge request sent

I'm close to a working version. the current limit is the EEPROM available size, I can have a maximum of 4 byte of user data. The code is only using 3KB on the 6KB available and I assume most of the partitions are over-estimated. do you think @janakj we can reduce some of the partitions to extend the User NVM space ?

Existing partitions are left slightly larger than necessary to accommodate for future additions. Also, the size of some of LoRaMac-node data structures stored in NVM changes depending on what features and regions are enabled at compile time.  I sized the existing partitions to almost entirely fill one EEPROM bank (half total EEPROM size). But we can definitely carve out more space for the user data partition. There should be plenty of space.
I will see what can be resized when I get around to reviewing your pull request.


Bump react peerDependency to ^15.0.0

Would you be so kind to adjust it?

Is this still beeing considered?

Yes, please update peerDependencies to use the latest React. Thank you!

@malte-wessel do you have any plans on updating the dependencies. Thanks!


Fix panic with weighted token state scaling factor

U256::exp10 panics if result would overflow.
Test Plan
I do have a test that constructs a full WeightedPool to trigger this but don't think it is useful to include since the code change is obvious enough.

Good point. I wasn't sure if that a bound was enforced anyway. Followed to the code a bit but couldn't find it. One follow up could be to use a wrapper type to enforce the 18 maximum so we can unwrap. Or we could store the 10**scaling_factor value directly as U256 instead of as a u8 scaling_factor which also avoids the problem. But that only works if the raw scaling _factor isn't used.

I wasn't sure if that a bound was enforced anywhere. Followed to the code a bit but couldn't find it.

Its enforced by the SC: https://github.com/balancer-labs/balancer-v2-monorepo/blob/c18ff2686c61a8cbad72cdcfc65e9b11476fdbc3/pkg/pool-utils/contracts/BasePool.sol#L487
I do like checking here, the alternative would be to have a separate pub type ScalingExponent(u8) type that guarantees that its wrapped value belongs to [0, 18], but that is a lot of overhead for very little benefit.


Add asm signature systems endpoint

as above

this will be added as a separate PR as it slipped through the cracks when i was coding rest of the signature cases. Should be done by Friday this week


use redis 3.2 template for redis 4.x

this would allow usage of protected_mode in 4.x, too

I've merged some serious refactor so this is fixed in another way.


[Event Hubs] Allow users to configure log level for send requests

The options bag for send requests as described in https://github.com/Azure/azure-sdk-for-js/issues/2661#issuecomment-491565934 should be used to pass the log level as described in https://github.com/Azure/azure-sdk-for-js/issues/2662#issuecomment-491648021
This work is blocked by #2845

It has been concluded that controlling log level at operation level is not very beneficial. Until we get user feedback asking for this feature, we will stick to setting the log level at the client level.


Constantize the class name to remove an unneeded case statement.

Vm and MiqTemplate both set @explorer = true so they're nearly
identical test cases.  Use Vm and Host since Host doesn't set @explorer.
Improves: Cyclomatic complexity for custom_buttons is too high. [10/6]
https://www.codacy.com/app/ManageIQ/manageiq/file/2225639467/issues/source?fileBranchId=3174192#l255

@Fryguy @martinpovolny Please review


Can't recognize certain Russian Cyrillic variants

Sorry I can't provide a free test case. Some early Soviet-era Russian Cyrillic texts make Л and Д look like Λ and Δ. I don't know if I have "fast" or "best" rus.traineddata, but the version I have reads these letters as А.
Environment

Tesseract Version: 3.05.02
Commit Number:
Platform: Darwin [name]MacBook.fios-router.home 16.7.0 Darwin Kernel Version 16.7.0: Thu Jun 21 20:07:39 PDT 2018; root:xnu-3789.73.14~1/RELEASE_X86_64 x86_64

Current Behavior: -l rus interprets these forms of Л and Д as А.
Expected Behavior: should interpret these forms of Л and Д as Л or Д.
Suggested Fix: perhaps a wider range of samples for training?

You are not willing to share input images than you are alone with you problems.
You can try to use current code with the tessdata_best or tessdata_fast.

And how am I supposed to switch from one tessdata to the other?

This has the alternative L but not the alternative D:
https://archive.org/details/B-001-017-874/page/n11

First of all need to install tesseract 4. Then you can have several tessdata folders and you will change TESSDATA_PREFIX environment variable to right place .e.g.:
export TESSDATA_PREFIX=/opt/tessdata
tesseract i1912.jpg i1912_tessdata -l rus

export TESSDATA_PREFIX=/opt/tessdata_best
tesseract i1912.jpg i1912_tessdata_best -l rus

export TESSDATA_PREFIX=/opt/tessdata_fast
tesseract i1912.jpg i1912_tessdata_fast -l rus


Support for both public and private listeners on AGIC

Currently with the AGIC 0.7.1, is it's not possible to have an HTTPS listener (443) on public and an http listener (80) on the private IP. When enabling usePrivateIP, it attaches both the ssl listener and the port 80 listener to the private IP which leaves the public with no listeners.
This is the setup we need the AGIC to replicate:

I would appreciate it if someone could look into adding this feature.
Thanks!

Thanks for the request. This should be part of the next release.

Here is the release 0.8.0 supporting the private ip annotation.
https://github.com/Azure/application-gateway-kubernetes-ingress/releases/tag/0.8.0
Closing this issue.


Feature request: Optional feature for rayon support

Currently, code like the following don't work:
use bitvec::prelude::*;
use rayon::prelude::*;
use std::sync::atomic::AtomicUsize;

fn main() {
    let mut list = bitvec![Lsb0, AtomicUsize; 0; 100];
    list.par_iter_mut().step_by(2).for_each(|mut bit| {
        *bit = true;
    });
    println!("{:?}", list);
}

But the above works if vec![false; 100] is used instead of bitvec![Lsb0, AtomicUsize; 0; 100] (or bitvec![0; 100]), or if .par_iter_mut() is replaced with .iter_mut().
It would be great if bitvec had support for rayon to allow its data structures to be iterated through in parallel in a convenient manner (i.e., with rayon's par_iter() or par_iter_mut() methods). Afaiu, this would require rayon's ParallelIterator and/or IndexedParallelIterator traits be implemented for bitvec's types.
Fwiw, here is part of a message from @myrrlyn about one way support might be added:

rayon methods require that i directly implement its extension traits. they don't have a blanket impl Iterator -> impl ParallelIterator transform. you can fake it with a much uglier boilerplate by using rayon::scope to drive a serial iterator, then parallelize it by making the loop body spawn a thread containing the rest of the work onto the rayon scope

Sample program that currently works:
use bitvec::prelude::*;

#[test]
fn parallelism() {
  let bits = bits![mut Lsb0, usize; 0; 800];
  rayon::scope(|s| {
    for (idx, chunk) in bits.chunks_mut(20).enumerate() {
      s.spawn(move |_| {
        chunk.store(!0u32);
        println!("{:02}", idx);
      });
    }
  });
  assert!(bits.all());
}

Implementation notes from doing a quick sketch:

Rayon is a good way to test that I have correctly managed impl {Send,Sync} in bitvec; should've used that much sooner
Rayon defines new slice par-iter adapters. I also will need to do this, because implementing Rayon's iteration traits directly on my seq-iter adapters causes method resolution failures. Bless up for macro_rules!.
My implementation should pretty much just copy directly from rayon/src/iter/plumbing/mod.rs and rayon/src/slice/mod.rs. Bridging to the thread-pool is Rayon's problem, not mine, and they've solved it. The only real logic is in <MyType as Producer>::split_at; everything else appears to be boilerplate for gluing types together.
This appears to be fairly straightforward, but it's going to be a lot of tedious work putting everything together. Certainly doable, equally certainly not something I can afford to front-load unless requested.

Questions:

Rayon only defines these parallel slice iterators. Should we define par-iters for everything in bitvec/src/slice/iter.rs or just the Rayon ones? Reasons against: it's extra work that they didn't do, so by definition, [bool].rchunks(W).par_iter() doesn't work and therefore nobody can expect BitSlice.rchunks(W).par_iter() to work. Reasons for: bitvec is already a superset of core, so there's precedent for going above and beyond the source.

I probably won't get back to this until, oh, February, so this should serve to restore everything I learned today when I do pick it back up.

It seems rayon has added implementations for RChunks, although there hasn't yet been a version published on crates.io containing these changes.

I found it easiest to just use bits.as_raw_mut_slice().par_iter_mut() and manually set bits of the integer storage.

Is there any progress now? Can we implement traits such as FromParallelIterator<bool> to facilitate more convenient collaboration with Rayon?
let mask = [10.; 100];
let m = mask.par_iter().map(|x| *x > 100.).collect::<BitVec>();

Is there any progress now? Can we implement traits such as FromParallelIterator<bool> to facilitate more convenient collaboration with Rayon?
let mask = [10.; 100];
let m = mask.par_iter().map(|x| *x > 100.).collect::<BitVec>();

This one has been bugging me, I can iterate in parallel but collecting I'm not sure. Maybe collecting into the underlying T and then call BitVec::from_vec?


add EfficientNet for torchvision

🚀 Feature
Add EfficientNet models into torchvision module.
Motivation
I very love this projects.
I noticed that in 2019 it was said that efficientnet models would be added. #980
But up to now, there is no efficientnet models.
I'm looking forward to adding more new models to torchvision.
Additional context
I refer to the official efficientnet model of tensorflow implemented with keras module link and the mobilenetV3 model in torchvision link. Then I built efficientnet models Test9_efficientNet and the conversion weight script trans_weights_to_pytorch.py.
If you think it's useful a little bit, I hope you can add efficientnet models to torchvision as soon as possible.

Hey @WZMIAOMIAO. I've skimmed #980 and as far as I understood the problem is not the implementation, but rather the training. If you have more insights or ressources, I suggest you get in touch with the other participants in the discussion.
Anyway, since there already is an issue for this, lets keep the discussion there.


add hplayground to stack

Hi,
I cannot find it currently in stack:
https://www.stackage.org/package/hplayground
any plans to put it there?
thanks,
Alex.

Yes. I'm integrating it with transient and putting it in stack. Currently
it is in an alpha state under GHCJS. look at the package :
https://github.com/agocorona/ghcjs-hplay
After that we will do the Haste integration
2016-02-28 13:26 GMT+01:00 Alex Greif<EMAIL_ADDRESS>
Hi,
I cannot find it currently in stack:
https://www.stackage.org/package/hplayground
any plans to put it there?
thanks,
Alex.
—
Reply to this email directly or view it on GitHub
https://github.com/agocorona/hplayground/issues/10.

--
Alberto.

Hi, thanks for the quick answer.
I saw that even haste is not in stack, so maybe this will be a blocker.
Alex.


SCT-1221 Make invalid token error more readable

Pull Request Test Coverage Report for Build 123

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.671%

Totals

Change from base Build 122:
0.0%

Covered Lines:
533

Relevant Lines:
563

💛  - Coveralls


added reductions_test file

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[ ] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[ ] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Related issues

Related issue #
Closes #

Checklist

[ ] Code follows style guide (ruff)
[ ] Tests added
[ ] Documented the changes

If you have comments or can explain your changes, please do so below.

hi @brentomagic - looks like you added an empty file?

my bad, let me correct that.

hey @brentomagic - looks like you're missing several imports, you should see if you can run this locally

thanks @brentomagic ! the test runs, but now we get
E   AttributeError: 'ArrowNamespace' object has no attribute 'min'

Could you add that please? Take a look at PandasLikeNamespace, see how min is implemented there, and try to add it

looking good!!!
looks like you're just missing the implementation for max?
FAILED tests/expr/reductions_test.py::test_expr_min_max[table] - AttributeError: 'ArrowExpr' object has no attribute 'max'


Responsive example

Hello
I just wanted to ask for a responsive example. I think the framework isn't mobile-first, so it's been hard for me to achieve a responsive result.
Would be great to have better support for that.
Thanks

@jahirfiquitiva the docs themselves are responsive. Material design itself is modeled to be mobile first and mobile friendly. Have you checked the material.io for design guidance?

are the docs open source too? mind linking me to their source?
and btw, I wasn't looking for design guidance, but rather implementation guidance.
Like a section where you explain how to "transform" toolbar items into a drawer when in mobile devices, or something like that?
Maybe even components that do that... transform their contents/look based on breakpoints

They're hacked together, but you can find some examples of mobile implementation on there. Honestly I think the only difference is the main navigation.
https://github.com/jamesmfriedman/rmwc/tree/master/src/rmwc/docs

thanks for sharing

No problem


adding bonito data to cudaPOA

A new function generate_bonito_long_reads is added to run long-read sample in cudapoa on the data extracted from bonito.
in this PR, the whole data set extracted from bonito is added in a text file sample-bonito.txt
this text file consists of 1000 windows (each consists of 6 sequences). The sample_cudapoa uses only first few windows, this file can be truncated to a much smaller size.

the previous branch got corrupted since by mistake changes for benchmarking were pushed to it, therefore I resent this PR.


Change nuimos list into dictionary,  in nuimo_app backend

handling based on mac_address

nuimo_app changes missing, to be completed


Traversal fixes

Fixed

$.fn.siblings and $.fn.has now return a Collection instead of an Array.
$.fn.closest returns an empty collection instead of the current collection if no selector provided
$.fn.closest correctly checks all items in the collection, not just the first.
$.fn.filter returns the collection as-is if no selector provided.

Added

$.fn.is, $.fn.not, $.fn.has, $.fn.find, $.fn.filter and $.fn.closest now support an element as the argument instead of a string based selector. ( #132 )

@simeydotme I've changed a test or two to match the new functionality, but could use some help building tests that will test the fixed and added functionality. Think you can spare some time to add the extra tests?

hey @shshaw :)
yeh, sorry I've been so lax last few weeks, I want to get back into the unit tests full swing on the other branch... gonna have to look at how to rebase with all the recent updates. I don't really want to put effort into QUnit as it hasn't got coverage reports or anything and it's already missing a ton of stuff.

@simeydotme what about using karma with jasmine?

yeh @vivekimsit that's what I'm doing on another branch :)
moving all tests to a CLI/NPM based testing framework which can be run from Travis CI,
You can view it, but it's still work in progress :)
https://github.com/kenwheeler/cash/tree/npm-scripts/test
I would suggest to maybe add some base-line tests in the current qUnit, @shshaw just to make sure there's some sanity checking going on and it's not breaking anything... it'll take too long to wait for the npm-scripts branch to be up to snuff :) :bow:
apologies


Add Paradise High School, CA, US

https://phs.pusdk12.org/

@AlexanderKharitonov First stage of verification passed, official domain is pusdk12.org as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@AlexanderKharitonov Pull request merged. Thank you.


hassio_supervisor armhf 118 fails to start

Yesterday, I successfully deployed Hassio on my Odroid XU-4 running Ubuntu LTS 16.04 but after reinstalling it today the hassio_supervisor contained is now failing to start.
I run the following command to deploy hassio on my Odroid's Docker CE:
curl -sL https://raw.githubusercontent.com/home-assistant/hassio-build/master/install/hassio_install | bash -s -- -m raspberrypi3
The hassio_supervisor container fails with the following error:
Traceback (most recent call last):,
OverflowError: Python int too large to convert to C long,
"main", mod_spec),
File "/usr/local/lib/python3.7/runpy.py", line 193, in _run_module_as_main,
File "/usr/local/lib/python3.7/runpy.py", line 85, in _run_code,
exec(code, run_globals),
File "/usr/local/lib/python3.7/site-packages/hassio/main.py", line 24, in ,
attempt_use_uvloop(),
File "/usr/local/lib/python3.7/site-packages/hassio/main.py", line 15, in attempt_use_uvloop,
import uvloop,
from .loop import Loop as __BaseLoop  # NOQA,
File "/usr/local/lib/python3.7/site-packages/uvloop/init.py", line 7, in ,
File "uvloop/includes/stdlib.pxi", line 128, in init uvloop.loop
All the necessary packages and requirements seem to be in place.
Any tips about how to troubleshoot this error?

remove -- -m raspberrypi3 ,and try again

Thanks for helping me with this issue.
Removing -m will be a problem since hassio_install script has the following statement:
case $ARCH in
    "i386" | "i686")
        MACHINE=${MACHINE:=qemux86}
        HOMEASSISTANT_DOCKER="$DOCKER_REPO/$MACHINE-homeassistant"
        HASSIO_DOCKER="$DOCKER_REPO/i386-hassio-supervisor"
    ;;
    "x86_64")
        MACHINE=${MACHINE:=qemux86-64}
        HOMEASSISTANT_DOCKER="$DOCKER_REPO/$MACHINE-homeassistant"
        HASSIO_DOCKER="$DOCKER_REPO/amd64-hassio-supervisor"
    ;;
    "arm" | "armv7l" | "armv6l")
        if [ -z $MACHINE ]; then
            echo "[ERROR] Please set machine for $ARCH"
            exit 1
        fi
        HOMEASSISTANT_DOCKER="$DOCKER_REPO/$MACHINE-homeassistant"
        HASSIO_DOCKER="$DOCKER_REPO/armhf-hassio-supervisor"
    ;;
    "aarch64")
        if [ -z $MACHINE ]; then
            echo "[ERROR] Please set machine for $ARCH"
            exit 1
        fi
        HOMEASSISTANT_DOCKER="$DOCKER_REPO/$MACHINE-homeassistant"
        HASSIO_DOCKER="$DOCKER_REPO/aarch64-hassio-supervisor"
    ;;
    *)
        echo "[Error] $ARCH unknown!"
        exit 1
    ;;
esac

So in my case, being armv7l, I need to enter one the available machine types (either armhf or raspberry3).
It shouldn't impact getting the hassio-superviser container (where this issue seems to be) as there's only one but it instruct hassio_install to download the right homeassistant container.
So, under the light of the above statements, will the need to use -m clause impact your fix at all?

That was a uvloop bug that is fixed with 0.11.0 and that is installed with >= 119


HALON-578:  Update Resource graph structure.

Created by: qnikst
Quite a massive PR but passes all tests.

Closing as an obsolete


Failure on sample multifast5 curl cake data for NanoTail

Cannot find an output folder either
[f8/d9b858] process > check_reference            [100%] 1 of 1 ✔
[f3/e922a5] process > tailfindr                  [100%] 1 of 1 ✔
[de/6f19f0] process > filter_bam                 [100%] 1 of 1 ✔
[60/1bded1] process > tail_nanopolish            [100%] 1 of 1 ✔
executor >  local (6)
[f8/d9b858] process > check_reference            [100%] 1 of 1 ✔
[f3/e922a5] process > tailfindr                  [100%] 1 of 1 ✔
[de/6f19f0] process > filter_bam                 [100%] 1 of 1 ✔
[60/1bded1] process > tail_nanopolish            [100%] 1 of 1 ✔
[-        ] process > collect_nanopolish_results -
[33/46846f] process > collect_tailfindr_results  [100%] 1 of 1, failed: 1 ✘
[-        ] process > join_results               -
WARN: Killing pending tasks (1)
Error executing process > 'collect_tailfindr_results (output)'

Caused by:
  Process `collect_tailfindr_results (output)` terminated with an error exit status (1)

Command executed:

  head -n 1 tailfin_1 >> output_findr.csv
  grep --no-filename  -v "tail_start" tailfin_* >> output_findr.csv
  awk -F"," '{if ($5!="" && $1!="read_id") print $1"	"$5}' output_findr.csv > output.findr.len

Command exit status:
  1

Command output:
  (empty)

Command error:
  WARNING: Your kernel does not support swap limit capabilities or the cgroup is not mounted. Memory limited without swap.
  head: cannot open ‘tailfin_1’ for reading: No such file or directory

Work dir:
  /home/minion/master_of_pores/NanoTail/work/33/46846f612f6f3068435c0a98f4d955

Tip: when you have fixed the problem you can continue the execution adding the option `-resume` to the run command line
Failed to invoke `workflow.onComplete` event handler

 -- Check script 'nanotail.nf' at line: 305 or see '.nextflow.log' file for more details

can you check inside the folder
/home/minion/master_of_pores/NanoTail/work/33/46846f612f6f3068435c0a98f4d955 ?

the directory is populated with symlink to the result of the tailfinder container but then file is empty. This was my param.cfg . I change the directory to the output from NanoPreprocess
params.cfg.txt
From error message it cannot open tailfin_1 but the symlink is called tailfin_ from what I can see
.command.log/run
WARNING: Your kernel does not support swap limit capabilities or the cgroup is not mounted. Memory limited without swap.
head: cannot open ‘tailfin_1’ for reading: No such file or directory
work/33/46846f612f6f3068435c0a98f4d955/.command.log (END)

drwxrwxr-x 2 minion minion 4.0K  4月 22 02:23 .
drwxrwxr-x 3 minion minion 4.0K  4月 22 02:23 ..
-rw-rw-r-- 1 minion minion    0  4月 22 02:23 .command.begin
-rw-rw-r-- 1 minion minion  194  4月 22 02:23 .command.err
-rw-rw-r-- 1 minion minion  194  4月 22 02:23 .command.log
-rw-rw-r-- 1 minion minion    0  4月 22 02:23 .command.out
-rw-rw-r-- 1 minion minion 3.2K  4月 22 02:23 .command.run
-rw-rw-r-- 1 minion minion  215  4月 22 02:23 .command.sh
-rw-rw-r-- 1 minion minion    1  4月 22 02:23 .exitcode
-rw-r--r-- 1 minion minion    0  4月 22 02:23 output_findr.csv
lrwxrwxrwx 1 minion minion  106  4月 22 02:23 tailfin_ -> /home/minion/master_of_pores/NanoTail/work/f3/e922a517abc3b9cbadf31428d188cf/output-multifast5_1_findr.csv

This is the result of tailfinder in f3/e922a5
oqutput-multifast5_1_findr.csv
read_id,tail_start,tail_end,samples_per_nt,tail_length,file_path
0d752c10-05c6-4a72-a0f3-6c65b56f776f,10014,12814,27.22,102.87,.//multifast5_1.fast5
00666bd7-121a-4194-9e6c-a6556286138c,13201,15476,39.19,58.05,.//multifast5_1.fast5
0a1d4b39-e15a-4b3e-90f7-2de7ecbd2638,7527,11727,40.65,103.31,.//multifast5_1.fast5
094f1315-ed28-4659-9aa9-f1a0b05735d8,7458,10258,28.98,96.62,.//multifast5_1.fast5
0e46b21e-c95f-4253-b03c-ee90b658e99a,11008,14558,29.76,119.27,.//multifast5_1.fast5
194e05c0-3dc5-4b08-b97b-77e7406386bd,6406,8281,29.19,64.24,.//multifast5_1.fast5
1c5f8f50-83f1-41ab-9d0f-77fed47f05b3,5793,6343,35.42,15.53,.//multifast5_1.fast5
1f59fd65-e396-4516-91a8-4e9b94f0ceac,6454,8979,290,8.71,.//multifast5_1.fast5
10f54f83-5c8f-4d8e-a493-26653ea4ea50,9925,12100,27.02,80.49,.//multifast5_1.fast5
089a34bb-8c02-4fed-8214-790868062e9f,16323,20248,52.7,74.48,.//multifast5_1.fast5
124b76d8-ebc0-4069-a027-1d01240c840e,4426,7626,34.27,93.37,.//multifast5_1.fast5
2f421677-bba2-4045-ad33-a9ab71bd8253,4917,8392,100,34.75,.//multifast5_1.fast5
1198182b-e7b9-498e-a54f-be8b78385deb,20312,21387,43.23,24.87,.//multifast5_1.fast5
2435bec6-374d-4528-8913-9d6516e616a0,8322,9972,32.08,51.44,.//multifast5_1.fast5
22c1ca2a-18f4-4206-942f-9fe93ac5444e,10173,12748,36.17,71.2,.//multifast5_1.fast5
362e6666-56de-4ea8-9f3f-8e2ff0135513,7772,9372,31.31,51.1,.//multifast5_1.fast5
2ca6c068-6dff-48be-bed3-63a6c0ffd4bb,5992,7567,26.93,58.49,.//multifast5_1.fast5
1db10d73-2486-45b9-a26b-ee192d6e6f40,7087,9737,28.65,92.49,.//multifast5_1.fast5

Hi,
I found a problem when you run a single fast5 file. Now I fixed it. Please remember that the pipeline needs that you run the preprocessing with the option:
 counter             = "YES"

Best
Luca


Display WebPack's warnings

If I try to bundle the following code:
const importLazy = require("import-lazy")(require);
const lodash = importLazy("lodash");
console.log ( lodash );

With webpack (v4.4.1) directly I get a warning saying:
WARNING in ./index.js
1:42-49 Critical dependency: require function is used in a way in which dependencies cannot be statically extracted
 @ ./index.js

Basically the way lodash is being imported is too dynamic for WebPack and this file can't be bundled properly.
If I try to bundle it with ncc I get 0 warnings, so I may think the thing worked while in fact it produced a broken bundle.
All warnings and errors from WebPack should be displayed.

The Webpack warning wouldn't actually be accurate here in ncc due to the fact that we rewrite this case to be a dynamic require to lodash in the build.
That is, you will get a bundle that attempts to require lodash itself, throwing an error only after it is not found then.
This is how ncc works as of the latest beta to more closely match the way files execute in Node itself.

Should perhaps then ncc transform the code properly to account for import-lazy and similar libraries?


fix: description not required field

Does not require a github repository to have a description to post a build. Additionally does not require OS or Ref. Necessary for the yargs repository to be used for testing.

Is there a test to update?

Added test