Memory leak?

Hi,
I am running dash 18.0.1 (x64) in docker container. The dashd process takes all RAM and swap available.
VM running docker engine has 16GB RAM and 8GB swap space currently (I have been increasing resources from 4GB onwards).
Sync stops now at syncing 2022-02.
2022-09-13T08:50:09Z UpdateTip: new best=00000000000000041420fc06922275fb994daad19f6db0cf99329e734e49f78c height=1623047 version=0x20000000 log2_work=78.754471 tx=40941988 date='2022-02-16T10:41:50Z' progress=0.925776 cache=282.5MiB(1354768txo) evodb_cache=447.3MiB
2022-09-13T08:50:41Z ThreadSocketHandler -- removing node: peer=75 nRefCount=1 fInbound=0 m_masternode_connection=0 m_masternode_iqr_connection=0
2022-09-13T08:52:23Z UpdateTip: new best=0000000000000012debb3c66b970651ed3f0f0d214ee038ee5476162c763d064 height=1623048 version=0x20000000 log2_work=78.754473 tx=40941995 date='2022-02-16T10:42:11Z' progress=0.925776 cache=282.5MiB(1354771txo) evodb_cache=447.3MiB
2022-09-13T08:53:07Z ThreadSocketHandler -- removing node: peer=76 nRefCount=1 fInbound=0 m_masternode_connection=0 m_masternode_iqr_connection=0
Killed

Sync started today at 5:30. RAM consumption was normal. At one point RAM consumption starts to increase dramatically.
Swap usage:

Ram usage:

Config file is empty.
Here are CLI arguments I am using: /usr/local/bin/dashd -addressindex=1  -zapwallettxes -minrelaytxfee=0.00001 -server=1 -listen=1 -maxmempool=800 -rpcallowip=<IP_ADDRESS>  -rpcallowip=<IP_ADDRESS> -rpcport=9998 -rpcuser=user -rpcpassword=password -rpcbind=<IP_ADDRESS>:9998 -onlynet=IPV4 -dbcache=2048 -rpcthreads=4 -rpcworkqueue=32
Same issue happens even if I omit -dbcache=2048 -rpcthreads=4 -rpcworkqueue=32 from CLI.
Is this some known issue?

Not sure, but this may be related to https://github.com/dashpay/dash/pull/5007

Yes, we are trying to figure out why leveldb behaves like this. So far the best solution (imo) is to limit evodb cache i.e. #5007. You can achieve similar results with 18.0.1 by specifying very low -dbcache and -maxmempool params e.g. -dbcache=64 -maxmempool=64. This might slow things down a bit but at least this should you keep mem usage under control and you should be able to sync without waiting for 18.0.2.

This workaround solved the problem, indeed. Thank you very much for help.


schema inspect does not return views?

It looks like support for views was added in 0.13, but when running schema inspect on MySQL I don't see my views. My user can list views in information_schema.tables and can run show create table.
Am I missing something in the documentation, do I need to add a flag to dump views?

❯ /opt/homebrew/bin/atlas version
atlas version v0.14.2
https://github.com/ariga/atlas/releases/tag/v0.14.2

and with go install it's simply
❯ ~/go/bin/atlas version
atlas version - development
https://github.com/ariga/atlas/releases/latest

Using plain go build does not contain this functionality. Installing Atlas as explained in the website should solve the issue: https://atlasgo.io/getting-started#installation

Apologies.
Since I read that it was released in 0.13, I figured 0.14 from homebrew (not your tap, though) would contain it.
Reinstalled with brew install ariga/tap/atlas and it seems to work.
Thanks for all the work you've put into this!


picture merged

After the picture is merged, the picture is compressed. How can I not compress it?

Merged? Are you referring to saveFrame(File file) method in EglSurfaceBase.java? If yes, then it is fairly straightforward, the buf ByteBuffer contains the pixels in RGBA.


Add some type of mpd support for use with Mopidy

@krisajenkins What do you think about supporting this in helm-spotify?

I'd not heard of mopidy before - looks cool. :-)
I think it's probably sufficiently different that it warrant a separate package though. Have you any interest in writing helm-mopidy?

Yeah, I do. I guess it might make sense to query spotify through mopidy if possible in such a plugin, in which case none of this code can really be reused, so you are probably right that it belongs somewhere else.


chore(test): add macro performance tests for combineLatest

Trying to close https://github.com/ReactiveX/RxJS/issues/260.

Merged with f258fc1
thanks, @kwonoj


Update the Train the Trainer Verbiage

Marketing came back to us this week and requested that we make minor changes specifically around the "Train-the-Trainer" phrase in our AI Tour materials. Please complete the following tasks to ensure compliance:

[x]  Change "train-the-trainer" to "session-delivery-resources" in the folder structure
[ ] Add boiler plate text (see below) to the top of the "session-delivery-resources" readme
[x] Change any in-text mentions of "train the trainer" to some version of "deliver this session."

Once all three items have been completed, please mark this issue as complete.
"session-delivery-resources" boiler plate markdown:
The following resources are intended for a presenter to learn and deliver the session.

@cole-g-johnson sorry I a not sure I am getting the 2nd point right "Add boiler plate text (see below) to the top of the "session-delivery-resources" readme". I already have this boiler plate text in my readme: "Along with the video of the presentation, this document will link to all the assets you need to successfully present including PowerPoint slides and demo instructions &
code." Is that enough?


Add test cases

The test suite has now been merged into master, and runs through make as well as being handled by travis.
Right now only three tests are running. Let's add in the rest of the tests from the tests/test_cases folder.

@kapkic it'd be great to get this up running, there's a PR that's mostly finished with this that would be great to get cleaned up and merged.

Also done by @kapkic


Testbench cannot be created when running multipass with qemu

To determine the IP address, it is currently assumed that the ethernet device is called enp5s0. But this is only correct if multipass is configured for lxd. When qemu is used, the interface is called ens3.
The correct way would be to determine the IP address without knowing what the Ethernet device is called in the virtual machine.

To simplify the configuration first, I'll assume multipass is configured for qemu. This is the default setting for multipass after installation in an Ubuntu environment.


Install packages from attributes

Using this recipe, with the following attributes:
          'chocolatey' => {
            'packages' => {
              'sysinternals' =>     { 'action' => 'install' },
              '7zip' =>             { 'action' => 'install' },
              'notepadplusplus' =>  { 'action' => 'install' },
              'GoogleChrome' =>     { 'action' => 'install' },
              'wireshark' =>        { 'action' => 'install', 'version' => '1.12.6' }
            }

will loop over the packages and install them with the options which can be provided to the resource.

I think this is out of scope and should be managed in each user's wrapper cookbook.

Yeah. I think I agree with @chasebolt .

Well, you are free to use the code, or not :wink:
For me the reason for creating this was that (for different (test)envrionments) I kept creating a wrapper cookbook with basically the folowing code (each time):
%w{sysinternals 7zip notepadplusplus GoogleChrome Console2}.each do |pack|
  chocolatey pack
end

And that would be the only thing I would wrap around this cookbook.
On the other hand, also see a lot of people "just" using separate cookbooks, like for 7zip or notepad++ (without specific reason (not to use chocolatey))
Perhaps not part of your goal for this cookbook, but removes the writing (by everyone) of separate resource(/(single)wrapper) for each and every package. I (myself :wink:) find removing/writing less code, very handy.
If your objection is that you feel this cookbook should limit itself to installation and LWRP, I'm glad that for example the authors of haproxy (used yesterday for the first time) don't agree; they provide installation and LWRP, but also read an attribute collection which uses their LWRP to iterate over that collection.
Otherwise everyone would need to write a (wrapper) recipe, which would basically be the same for everyone.
(Totally not saying this removes the need for wrapper cookbook BTW)
So haproxy cookbook allowed me to only include that cookbook, set some attributes, and already be done with it.
(Realize that functionally these are completely different cookbooks, but the ease of use COULD be the same)
So this code only requires (everyone) to specify a simple attribute for each (default installation of an) application, instead of a (everyone) writing separate (wrapper) cookbook for (possibly) each application.
But no earth-shaking feature :smile:, so feel free to use it if you like it, or close it if you don't

closing this especially in light of the fact that using the resource in core chef is the way forward.


CVEs found in roxctl 4.5.3

Hi Team,
We have recently run a vulnerability scan with the latest roxctl version 4.5.3. The following CVEs are being flagged and we would like to check if there's any plans to fix the cves? We previously had ran the vulnerability scan on the older roxctl version 4.5.2 and the same list of CVEs were showing. All the CVEs seems to come from the same library - stdlib:

Appreciate your support and help! :D

Hi @yyuanxin , thanks for your report. master has already been bumped to go 1.22.5, which fixes some of those CVEs (in particular the critical one). I've passed this info on to the release team.


Background shorthand syntax broken in ie8

There are plenty of examples where something like this
div {
  background: url(thing.png) 0 -100px no-repeat;
}

generates the following output...
div{background:url(thing.phg)0 -100px no-repeat}

However, not placing a space between the url(...) and the following values is considered a syntax error in IE8 and doesn't produce the background image.
There are enough cases similar to this represented in the automated tests that I'm curious if this is an explicit browser support decision or a case that simply hasn't come up yet?
Also, if there are cases where syntax similar to that highlighted above does, in fact, work in IE8, I'm curious to learn what they might be.

clean-css is IE9+ compatible by default, but there's IE<9 switch - please see #516 and #497 for more details!

Cool. Thanks! Sorry for the dupe.

No worries :) :+1:


Add additional URL types supported by Reflections

See:
https://github.com/ronmamo/reflections/blob/master/src/main/java/org/reflections/vfs/Vfs.java

Merging into another issue.


[Suggestion] Introduce Int8 MIDL type

Since this project is not aiming to preserve binary compatibility with Windows Runtime, I would like to suggest introducing signed 8-byte Int8 primitive type into Xlang type system.
Current MIDL 3.0 doesn’t support Int8 type, and only has unsigned 8-bit integer type. But introducing the signed one would make a lot of sence if you expect to support UTF-8 strings in Xlang ABI.
This would require changes to MIDL 3.0 definition language of course.

A signed 8-bit integer was not considered useful by the original WinRT architects. We may revisit that since the underlying ECMA-335 spec certainly supports it. Thanks for the suggestion!

@BrentRe Basically I currently use them to return strings (sbyte* arrays) allocated in a specific memory pool (not with .NET Marshal) back to the C# caller. They are easily converted to managed strings using just a String constructor.
That might not be a viable case for Xlang, but I think other users could also give some insights into how they could use them.

The lack of actual signed 8-bit quantities in WinRT was annoying for implementing WinML tensors of that data type (see the INT8 data here for the ONNX Machine Learning format, https://github.com/onnx/onnx/blob/master/onnx/onnx.proto). Tensors are multidimensional arrays of data types, and we shouldn't need to reinterpret_cast int8_t to an array of unsigned char (which semantically doesn't really make sense) when it should be possible to just pass the array_view<int8_t> directly to WinRT API's. The data type is actually used, as quantized tensors greatly reduce multimegabyte model files. I wouldn't want to see XLang have the same limitation, especially given it's intended to have even broader interop.

UTF-8 encodes all 1,112,064 valid code points in Unicode
Yeah, I see no use for signed bytes as code points and would discourage that (Unicode should always be unsigned), but I see other numeric processing uses outside machine learning too (e.g. signed 8-bit types used in older audio formats, like 8-bit PCM and telephony from ITU-T Recommendation G.711).

They are a basic data type in:

Python numpy int8 - https://docs.scipy.org/doc/numpy/user/basics.types.html
C++ int8_t - https://en.cppreference.com/w/cpp/types/integer
Java byte (though arguably awful that Java made 'byte' signed by default, rather than a distinct sbyte type).
C# sbyte - https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/sbyte

Given this data, I'm inclined to support Int8 in the xlang type system. Thanks.
We'll need to figure out the xlang <--> Windows Runtime interop story for types using Int8 but we should be able to do something reasonable.


Adaptions due to Expo SDK changes

Updated initialising of analytics
Compatibility for bare workflow apps
Added properties from expo-application

Any updates on this?


fix: set a default value for appConfig

Codecov Report

Merging #921 (353a95b) into release-next (7768c16) will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           release-next     #921   +/-   ##
=============================================
  Coverage         31.60%   31.60%
=============================================
  Files                46       46
  Lines              1174     1174
  Branches            280      280
=============================================
  Hits                371      371
  Misses              803      803


Sources: Change 'url' and 'generator' to 'ref'

There are three values in the DataSource table for referencing the object that produces rows: url, ref and generator. These three should be combined into 'ref'.
Then, the 'urltype'  should be changed 'reftype'

What are the choices for ref field?
What are the choices for reftype field?

Ref can be anything. It is  a URL for files with a URL, a partition name or partition id for partitions,  a Google Doc ID for google docs. For generator objects, it is the generator class name.
The reftype value is not strongly defined, but there are some values for it in orm.source.DataSourceBase#is_downloadable, which is used to determine if the file should be downloaded during ingest. Then, bundle.bundle.Bundle#_ingest has a special check for a reftype of 'generator'.
The values in orm.source.DataSourceBase#is_downloadable should be moved to a class constant. Currently they are hard coded in the method.
However, I thought I had assigned this one to myself. I've already done part of it, so I'll take you off and finish it up.


fix(build): bump python core version to 3.11.3

PR summary
This PR bumps up our python core version to 3.11.3 to pull in the authenticator fix.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

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


Fixes response patching of POST request

Original pull request: #78

GitHub

Fixes #77

Hi, @skipdavis1117. Sorry for closing your original pull request (#78), I've got stuck with rebasing your remote branch and couldn't push the new state of the commits tree. If you don't mind, I've opened this follow-up with your commits preserved. Thanks for understanding.

Solution
I've looked into this issue and spotted that on the Service Worker level right before dispatching a bypassed request it operated on the GET request, despite the original request being of method POST. This was related to the destructuring of the request Object. Apparently, if you destructure a POST request it produces a GET request.
https://github.com/open-draft/msw/blob/16e536a22de1be266cd038c2811dae2d7fb71ecd/src/mockServiceWorker.js#L102-L105
When the request object is passed to the Request constructor directly, it inherits the method and other properties accordingly.
I've also used the requestClone in the bypassed request handling to prevent "request already sent" and similar errors. I think it's generally a good practice in terms of intermittent request handling.

@skipdavis1117, could you please review this change so we could merge it? Thanks.

No worries, you could just ask me to do it.  I'd be happy to instead of
making more work for you.  I am more interested in working product than my
feelings. I have not had much time to troubleshoot this week, but I think
you were right that the problem is likely in the Request creation and not
in the fetch.  I'll have a little more time this weekend.
Thanks
On Fri, Mar 27, 2020, 5:39 AM Artem Zakharchenko<EMAIL_ADDRESS>wrote:

Hi, @skipdavis1117 https://github.com/skipdavis1117. Sorry for closing
your original pull request (#78
https://github.com/open-draft/msw/pull/78), I've got stuck with
rebasing your remote branch and couldn't push the new state of the commits
tree. If you don't mind, I've opened this follow-up with your commits
preserved. Thanks for understanding.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/open-draft/msw/pull/84#issuecomment-604906109, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AO2GAK7HGB2HBWZ3WZ5OMQTRJRX4HANCNFSM4LU3Y3JQ
.

@skipdavis1117, hey, absolutely no worries! You add no extra work for me. I've discovered a fix while debugging, so I decided to push it and not trouble you. The tests you've contributed are great, I'm extremely thankful!
There are a few open issues, if you have time take a look at any of them and voice your opinion. Ones I would recommend to look are #21, #55 (discussion mainly).

Great!  I'll look at those
On Sat, Mar 28, 2020, 6:58 AM Artem Zakharchenko<EMAIL_ADDRESS>wrote:

@skipdavis1117 https://github.com/skipdavis1117, hey, absolutely no
worries! You add no extra work for me. I've discovered a fix while
debugging, so I decided to push it and not trouble you. The tests you've
contributed are great, I'm extremely thankful!
There are a few open issues, if you have time take a look at any of them
and voice your opinion. Ones I would recommend to look are #21
https://github.com/open-draft/msw/issues/21, #55
https://github.com/open-draft/msw/issues/55 (discussion mainly).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/open-draft/msw/pull/84#issuecomment-605431063, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AO2GAKYSTU7VSX6RRVBHHJTRJXJ5JANCNFSM4LU3Y3JQ
.


Release 2.3.1

This is a draft of changelog for the 2.3.1 release.
Here you can find all related translations changes: 2.3.0...master.

⚠ Do not merge this pull request manually ⚠
You should approve this pull request instead. After approving, I will automatically update it (release date needs to be adjusted), merge, and tag a new release. You can read more about the process on forum.

Show release notes preview
2.3.1

General changes:

Updated validation translations.

All changes: 2.3.0...2.3.1.

Success! Now you can announce new release on forum:
## Version [`2.3.1`](https://github.com/flarum-lang/hebrew/releases/tag/2.3.1)

**General changes**:

* Updated validation translations.

All changes: [2.3.0...2.3.1](https://github.com/flarum-lang/hebrew/compare/2.3.0...2.3.1).

In order to update run these commands:

```console
composer update flarum-lang/hebrew
php flarum cache:clear
```


EinsteinTensor.from_metric(metric) fail index out of range

Fail with
C:\ANA\lib\site-packages\sympy\tensor\array\ndim_array.py in _parse_index(self, index)
65         if isinstance(index, (int, Integer)):
66             if index >= self._loop_size:
---> 67                 raise ValueError("index out of range")
68             return index
69
when executed
einst = EinsteinTensor.from_metric(metric)
from your Einstein Tensor calculations using Symbolic module example.
🐞 Problem

Trackback from my Jupiter lab session on fail:
ValueError                                Traceback (most recent call last)
 in ()
----> 1 einst = EinsteinTensor.from_metric(metric)
2 #einst.tensor()
C:\ANA\lib\site-packages\einsteinpy\symbolic\einstein.py in from_metric(cls, metric)
47     def from_metric(cls, metric):
48         t_ricci = RicciTensor.from_metric(metric)
---> 49         r_scalar = RicciScalar.from_riccitensor(t_ricci, t_ricci.parent_metric)
50         einstein_tensor = (
51             t_ricci.tensor() - (1 / 2) * metric.lower_config().tensor() * r_scalar.expr
C:\ANA\lib\site-packages\einsteinpy\symbolic\ricci.py in from_riccitensor(cls, riccitensor, parent_metric)
203             parent_metric = riccitensor.parent_metric
204         ricci_scalar = tensorcontraction(riccitensor.tensor(), (0, 1))
--> 205         return cls(ricci_scalar, riccitensor.syms, parent_metric=parent_metric)
206
207     @classmethod
C:\ANA\lib\site-packages\einsteinpy\symbolic\ricci.py in init(self, arr, syms, parent_metric)
172         )
173         self._order = 0
--> 174         self._expr = self.arr[0]
175
176     @property
C:\ANA\lib\site-packages\sympy\tensor\array\dense_ndim_array.py in getitem(self, index)
65                 return self._array[index]
66             else:
---> 67                 index = self._parse_index(index)
68                 return self._array[index]
69
C:\ANA\lib\site-packages\sympy\tensor\array\ndim_array.py in _parse_index(self, index)
65         if isinstance(index, (int, Integer)):
66             if index >= self._loop_size:
---> 67                 raise ValueError("index out of range")
68             return index
69
ValueError: index out of range

🖥 Please paste the output of following commands

I am operating out of an Anaconda environment, but did not use conda to obtain and install your code, einsteinpy-0.2.1-py_1.tar.bz2 (I just decompressed it and placed the relevant folder in:
C:\ANA\Lib\site-packages\einsteinpy
Don't use pip.
I'm using SymPy 1.1.1 so perhaps you are expecting later version (couldn't find any version testing in your software though).
Notes I made on the failure:
n \sympy\tensor\array\dense_ndim_array.py getitem(self, index) method of class DenseNDimArray(NDimArray) is invoked with intent of retrieving items from an array. If checks for symbolic index (is not). It chen checks if the index is an instance of tuple and if so if any of its elements is a slice. If not it returns, invoking ndim_array.py method parse_index, where a test for index GTE loop_size fails with "Index ouf of range".
I sucessfuly ran some of your code:
sch = SchwarzschildMetric()
sch.tensor()
and
csfm = ChristoffelSymbols.from_metric(sch_inv)
csfm.tensor()
and
new_csfm = csfm.change_config('lll')
new_csfm.tensor()
Just ran into trouble in the symbolic module test.
I am hoping this problem will look familiar to you.

@AncientZygote thanks for reporting this :)
I would highly encourage you to paste the code which caused the problem. Other than that, please ensure that you are using the latest developmental code base on github and not from pip as there was some API breaking changes in sympy.
Steps to install the latest code-base are provided here :
https://docs.einsteinpy.org/en/stable/getting_started.html#latest-versions

@shreyasbapat A lot of the troubles could have been avoided if EPY 0.2.2 had been released :(

I cannot replicate this issue on recent API (einsteinpy>=0.3). Closing this.

I cannot replicate this issue on recent API (einsteinpy>=0.3). Closing this.


Run tests with a range of values for inliner_aggressiveness to test for divergences in behaviour.

@aakoshh noticed in #6340 that compiling the trait_eq::inequality test fails with inliner_aggressiveness = 0 but passes with inliner_aggressiveness =<PHONE_NUMBER>854775807. We would expect that the resultant brillig bytecode should be equivalent but this is not the case.
We should update the test suite to run at a number of different levels of inliner aggressiveness to check that we don't get different behaviour presented.
cc @sirasistant as non-inlining behaviour is resulting in correctness issues.

Investigating...

I see the issue, it's related with array_set_mut:
unconstrained fn main() {
    let mut bounded_vec1: BoundedVec<Field, 3> = BoundedVec::new();
    let mut bounded_vec2: BoundedVec<Field, 3> = BoundedVec::new();
    bounded_vec1.push(1);
    bounded_vec2.push(2);

    std::println(bounded_vec1.storage());
    std::println(bounded_vec2.storage());
}

After Array Set Optimizations:
brillig(inline) fn main f0 {
  b0():
    inc_rc [Field 0, Field 0, Field 0]
    v2 = allocate
    store [Field 0, Field 0, Field 0] at v2
    v3 = allocate
    store u32 0 at v3
    inc_rc [Field 0, Field 0, Field 0]
    v5 = allocate
    store [Field 0, Field 0, Field 0] at v5
    v6 = allocate
    store u32 0 at v6
    call f1(v2, v3, Field 1)
    call f1(v5, v6, Field 2)
    v11 = load v2
    call f2(v11)
    v13 = load v5
    call f2(v13)
    return
}
brillig(inline) fn push f1 {
  b0(v0: &mut [Field; 3], v1: &mut u32, v2: Field):
    v3 = load v1
    v5 = lt v3, u32 3
    constrain v5 == u1 1 '"push out of bounds"'
    v7 = load v0
    v8 = load v1
    v9 = load v1
    v10 = array_set mut v7, index v9, value v2
    store v10 at v0
    store v8 at v1
    v11 = load v0
    v12 = load v1
    v14 = add v12, u32 1
    store v11 at v0
    store v14 at v1
    return
}
brillig(inline) fn println f2 {
  b0(v0: [Field; 3]):
    call v1(u1 1, v0, [u8 123, u8 34, u8 107, u8 105, u8 110, u8 100, u8 34, u8 58, u8 34, u8 97, u8 114, u8 114, u8 97, u8 121, u8 34, u8 44, u8 34, u8 108, u8 101, u8 110, u8 103, u8 116, u8 104, u8 34, u8 58, u8 51, u8 44, u8 34, u8 116, u8 121, u8 2⁴×7, u8 101, u8 34, u8 58, u8 123, u8 34, u8 107, u8 105, u8 110, u8 100, u8 34, u8 58, u8 34, u8 102, u8 105, u8 101, u8 108, u8 100, u8 34, u8 125, u8 125], u1 0)
    return
}

As you can see in fn push, it's using array_set mut, which ignores reference counters, so it's mutating [Field 0, Field 0, Field 0] which is shared in two variables (if it wasn't mut, it would work correctly, since its reference counter is correct, 2)

cc @vezenovm

Good spot! I eyeballed the whole SSA printout top to bottom but missed the mut at the last step.
I'll open a PR with the test changes to call with/without brillig and various inlining aggressiveness.

Probably 2 or 3 levels of aggressiveness should be enough (MIN_VALUE, 0, MAX_VALUE). Or MIN_VALUE and MAX_VALUE only if it's too costly

I'm looking at a fix to the array mut issue. I'm actually surprised to that v7 is being marked mutable as the pass bans arrays loaded from references to be marked mutable, but we must be missing something.

Looking at [https://github.com/noir-lang/noir/blob/d8e549aad6bae0f96621c57b58a301693463f732/compiler/noirc_evaluator/src/ssa/opt/array_set.rs#L116-L119] it looks like the return block overrules the load 👀


arbitrary color for accentColor

When I try setting accentColor to a hex code I see this warning:
Value is not accepted. Valid values: "rosewater", "flamingo", "pink", "mauve", "red", "maroon", "peach", "yellow", "green", "teal", "sky", "sapphire", "blue", "lavender".(1)

Is it possible to use an arbitrary color for this property?

Hey,
I think you might be able to overwrite the value in the settings.json after setting one color.
Not exactly sure, but from my understanding that should work.
For instructions read the README.md

It's intended to be overwritten like so:
  "catppuccin.accentColor": "pink",
  "catppuccin.colorOverrides": {
    "mocha": {
      "pink": "#ff69b4"
    }
  },

that works, thanks


What is the v0.100.0 tag?

Describe the bug
The latest tag is really v0.25.33 but a tag exists at v0.100.0. Why does this tag exist?

https://github.com/chanzuckerberg/terraform-provider-snowflake/issues/539


Usage with Nuxt 3

Hi, I'm trying to get your package to work with Nuxt 3.
I tried importing it in my nuxt.config.ts like this:
import { defineNuxtConfig } from 'nuxt3'

export default defineNuxtConfig({
    buildModules: [
        ['vue3-openlayers', '@pinia/nuxt', '@intlify/nuxt3', '@vueuse/core/nuxt']
    ],
    intlify: {
       // ...
},
    build: {
        // ...
    }
})

Which gives me this error on app startup
ERROR  Cannot start nuxt:  Cannot read property 'webpackJsonpvue3_openlayers' of undefined

  at module.exports.00ee (node_modules/vue3-openlayers/dist/vue3-openlayers.common.js:191:148)
  at node_modules/vue3-openlayers/dist/vue3-openlayers.common.js:203:9
  at g (node_modules/jiti/dist/jiti.js:1:55111)
  at requireModule (node_modules/@nuxt/kit/dist/index.mjs:86:26)
  at installModule (node_modules/@nuxt/kit/dist/index.mjs:1241:50)
  at initNuxt (node_modules/nuxt3/dist/index.mjs:905:11)
  at async load (node_modules/nuxi/dist/chunks/dev.mjs:6713:9)
  at async Object.invoke (node_modules/nuxi/dist/chunks/dev.mjs:6752:5)
  at async _main (node_modules/nuxi/dist/chunks/index.mjs:386:7)

When I place the import at the end of the buildModules array, the error disappears, but upon opening my testpage I receive the following errors:
[Vue warn]: Failed to resolve component: ol-map
If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.
[Vue warn]: Failed to resolve component: ol-view
If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.
[Vue warn]: Failed to resolve component: ol-tile-layer
If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.
[Vue warn]: Failed to resolve component: ol-source-osm
If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.
Invalid value used as weak map key
  at WeakMap.set (<anonymous>)
  at normalizePropsOptions (./node_modules/vue/node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:2689:11)
  at createComponentInstance (./node_modules/vue/node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:6115:23)
  at renderComponentVNode (./node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:197:22)
  at Module.ssrRenderComponent (./node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:623:12)
  at _sfc_ssrRender (file://./.nuxt/dist/server/server.mjs:4813:31)
  at renderComponentSubTree (./node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:263:13)
  at renderComponentVNode (./node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:214:16)
  at renderVNode (./node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:304:22)
  at renderVNode (./node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:310:17)

Not sure if this is the right place to ask, but I'm stuck.
Thanks in advance!

Hello,
you don't need to add vue3-openlayers to buildmodules. You can use as a plugin.
Nuxt will automatically read the files in your plugins directory and load them. You can use .server or .client suffix in the file name to load a plugin only on the server or client side.
All plugins in your plugins/ directory are auto-registered, so you should not add them to your nuxt.config separately.
Example usage of nuxt3 : https://codesandbox.io/s/vue3-openlayers-nuxt-forked-e3rfy?file=/plugins/vue3-openlayers.client.js


New version: ppy.osu version 2024.1009.1

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue? - Replace #191694

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

ReleaseNote and Scope can't be added because:

This should be right now.


[Python] Converting from NumPy to large_string or large_binary returns not implemented

Describe the bug, including details regarding any error messages, version, and platform.

arr = np.array(["a", "b"])
pa.array(arr, type=pa.large_string())

This returns
    pa.array(arr, type=pa.large_string())
  File "pyarrow/array.pxi", line 316, in pyarrow.lib.array
  File "pyarrow/array.pxi", line 83, in pyarrow.lib._ndarray_to_array
  File "pyarrow/error.pxi", line 121, in pyarrow.lib.check_status
pyarrow.lib.ArrowNotImplementedError: NumPyConverter doesn't implement <large_string> conversion.

I think this should work?
Component(s)
Python

Yes, there is indeed no specific reason for this to not work (apart from someone implementing it).
Looking at the code, this is default fallback because the NumPyConverter only is implemented for the non-large StringType. We have this:
https://github.com/apache/arrow/blob/c38c7788915652f5bdfaa07945809efc1188980c/python/pyarrow/src/arrow/python/numpy_to_arrow.cc#L661-L666
But no equivalent Visit(const LargeStringType& type). The implementation for StringType is based on ChunkedStringBuilder, which is a chunked version of StringBuilder. We already have LargeStringBuilder, so it should certainly be possible to add a ChunkedLargeStringBuilder as well, so we can template the NumPyConverter to work with both builders.


Instagram Live

What solution do you have for downloading live?

Hi @Amirhf1, so for the late response.
I didn't try this but I guess you can easily download them using your session id and stream URL.

Thanks @beNitinhere for my help working on your project.

@Amirhf1 You're welcome :)

@beNitinhere
In this app, I didn't notice the API
Exactly how it connects to Instagram even without API

@Amirhf1 Actually Instagram does not provide APIs for stories, posts, and others so we use web-scraping using user-auth session id.


[Feature Request] Adding USPS Mail Delivered status into USPS Mail workflow

Is your feature request related to a problem? Please describe.
Currently, the USPS mail workflow does not include the final status "USPS Mail is Delivered." This creates ambiguity for users tracking mail, as they cannot confirm when delivery has been completed.
Describe the solution you'd like
Adding a "USPS Mail is Delivered" status to the USPS mail workflow. USPS has started sending notifications via email when the mail has been delivered to the mailbox. The email is sent from<EMAIL_ADDRESS>This status should be triggered once the email is received to the recipient's email address. It should be visible in tracking updates to provide a clear end point in the mail delivery process.
Describe alternatives you've considered
Reviewing the email sent by USPS regarding mail delivered.
Additional context
Adding this status aligns with the package-tracking workflow provided by this integration. It would enhance the user experience by providing complete visibility into the mail delivery lifecycle.

Thanks!

There's already a binary_sensor for this.

Thank you!


Add --list-flavors and --list-accents to Whiskers

These flags would output a list (in various formats maybe, with -o) of the flavors and accents in Catppuccin. Would simplify something like https://github.com/r58Playz/ctp-devtools/blob/74bccc4f5922a9070013ab951e9d9cbfc678d532/generate.sh#L3-L4.

These flags would output a list (in various formats maybe, with -o) of the flavors and accents in Catppuccin. Would simplify something like<EMAIL_ADDRESS>
would using a .tera file not fix this?

No, Tera matrices are already used in that script linked above. The for loop is needed to symlink other files for every flavor and accent.


Add support to use external redis-sentinel as celery broker

This PR adds support to utilize an external redis-sentinel instance as the celery broker.
https://docs.celeryq.dev/en/stable/getting-started/backends-and-brokers/redis.html#configuration
https://docs.celeryq.dev/en/stable/userguide/configuration.html#broker-transport-options
Changes:

adds CELERY_BROKER_TRANSPORT_OPTIONS configuration to settings.dist.py and helm values
adds redis sentinel scheme to helpers.tpl and helm values
adds documentation

Tested via building changes and deploying defect-dojo to Kubernetes and connecting to a redis-sentinel instance also running on Kubernetes.
[12/Oct/2022 21:35:05] INFO [celery.worker.consumer.connection:22] Connected to sentinel://:**@redis-sentinel-redis-sentinel:26379//
[12/Oct/2022 21:35:05] INFO [celery.worker.consumer.mingle:40] mingle: searching for neighbors
[12/Oct/2022 21:35:06] INFO [celery.worker.consumer.mingle:49] mingle: all alone
[12/Oct/2022 21:35:06] INFO [celery.apps.worker:176] celery@defect-dojo-defectdojo-celery-worker-77569d5564-g5j5l ready.

I am new to the project and Django overall, so if there is a better way to add support for this, please let me know.

@tks98 Pr needs to be against dev branch.

@tks98 you have some linter alerts.

@damiencarol I addressed the linter alerts, please let me know if anything else is required on my end

@tks98 some of the tests are failing. Please take a look.

@alles-klar I added a check to see if the transport options env var is set before trying to load it via json. This fixed the failing unit test, and should fix the remaining workflows once they are run.

@tks98 To me the PR looks good but it is in draft state. If it is ready and you want get it merged please remove the draft state.

@tks98 To me the PR looks good but it is in draft state. If it is ready and you want get it merged please remove the draft state.

@alles-klar I removed the draft state, thank you.


Updated package.json

Dependencies added in package.json to solve error Cannot find module 'stylus' and 'stylus-loader'

stylus is already included in devDependencies
stylus-loader added in 8d3afb3


Update chinese documentation for readability

change
MyBatis You can use simple XML or annotations for configuration and native Map
into
MyBatis You can use simple XML or annotations to configure and map native information

@center1024 Thanks for your contribution, However we need to write a issue description/commit comment using English. Please re-work for this PR.

@kazuki43zoo google translator.  Seems right to me ;)

@center1024
Thank you for the PR!
This should be included in #1012 according to @lihengming .


Proposals: Lifecycle events as local events and broadcastLocal drop

I love the simplicity of the moleculer services and their structures.  I really appreciated the enhancements for events handling on 0.11.x . I think there's still some small room for improvements here, specifically about the local events handling, which have 2 broadcast methods: broadcast('$eventName') and broadcastLocal('eventName'). What is the actual use cases for these? What do you think to deprecate broadcastLocal altogether and just use brodacast('$eventName') for local events too?
Also, service lifecycle events are all local by definition. So, why there are those specialized methods here? What about three local events like $service.created, $service.started , $service.stopped'?

The broadcastLocal is important for internal events because it won't be transferred to remote nodes. Like $services.changed, $node.connected...etc.
In a normal project recommended to use the broadcast and emit.
The lifecycle events is different as other, because it is only specified to a service instance, like events is not. It's also come from Vue

Yep, I see the lifecycle events coming from Vue, but what so you think if those events are also emitted locally to the same service? Here is the approach I'm using to fake this approach:
"use strict";

module.exports = {
	name: "test",
    events: {

	    "**" (payload, sender, event) {
            console.log(`TEST: Event '${event}' received from ${sender} node:`, payload);
        }
	},

 	started() {
        // emit a local event
        this.broker.emit("$service.started",null, [this.name]);

    },

    created() {
        // emit a local event
		this.broker.emit("$service.created",null, [this.name]);
    },

    stopped() {
        // emit a local event
		this.broker.emit("$service.stopped",null, [this.name]);
    }
}

You can do it in your services if you want. But the emit will transfer these events to other nodes too where maybe it is not relevant.

mmh, but from the doc: Every local events must start with $ (dollar sign). E.g.: $node.connected. If you call these events with emit or broadcast, they will send to only local services.

Hmm, it's true, sorry.

Same for the broadcastLocal proposal. If broadcast('$event') is always local, could it better to just use this and drop brodacastLocal?

Maybe, but in case of backward compatibility I won't remove it. But you can use broadcast instead of broadcastLocal. It does same thing if the event name starts with '$'.

If I may put my two cents in... I'd vote for:

Prefixing moleculer's internal events (e.g. $node.connected) with $
Not prepending $ to event names just because they're emitted/broadcast locally; using broadcastLocal / emitLocal instead.

My reasoning is the event name should not change based on consumer.

I think it is better to have a simple, clean and general purpose API which could be used as the foundation for higher level capabilities and I would rather  a clear and unique way to emit local vs remote events, at least at this development stage. Infact, @deividasjackus with '$' you are still changing the event handling logic based on consumer, as they are internal events only (local events within a node).
By the way, I like the actual behavior, I just think it is better to get rid of duplications and have consistent API and events (i.e. the service lifecycle events proposal).

@pibi I'm all for a simple, clean and general purpose API. Be that as it may, I am also for clean semantics.
An event name serves a clear purpose and should not have anything to do with routing logic (while it can specify the event name scope, e.g. $node).

@deividasjackus I agree with you about the clean semantics. Infact I think that calling actions locally are quite a corner-case in a microservice platform, isn't it?  I think they should be used for new moleculer modules or to develop middlewares than to high level usage.
Also, it would be interesting to know the reasons of the drop of the emitLocal() method, as I think it is quite similar to the broadcastLocal() scenario.
I know that other frameworks (mainly senecajs) uses pattern matching for routing, which is essentially a generalization of our "name.of.event" approach (it is a matter of serialization). My gut feeling is that the explicit *Local() methods don't help to avoid misunderstandings, unless we are not dropping the local $ for non-internal local events.
You could think this a bike shedding, but I had recently worked on one of the largest senecajs development, with more than 50 microservices, developed by a full remote team spanning 12 timezones. In this scenario we had quite a tough time with senecaJS and its declarative approach to microservices. Here I see moleculer more as the opinionated (as in configuration over code) framework to be used on large development scenarios (I really love the broker.loadService('fileName'), you can see the same approach on our senecajs reference architecture here: https://github.com/pibi/senecatest/blob/master/microservices/user/plugin.js#L17 )

@pibi,

Calling local actions: I agree partially. If you subscribe to the concept of "microsystems" rather than "microservices" (as proposed by Jonas Bonér) local calls/events seem quite reasonable to me in certain situations, but it indeed is far from a common pattern (for now?).

Explicit methods: I agree. broadcast and emit should route globally, broadcastLocal and emitLocal should route locally. Neither should not have event name related logic, expectations or restrictions. Disclosure: I'm still not as familiarized w/ v.10 changes as well as I'd like to be; was not aware emitLocal was dropped, let alone of the reason for it.

Seneca: it's a shame you had to go through that, but great in the sense that you can now share and help Moleculer avoid the same mistakes 😃 Not bikeshedding at all IMO. A consistent core built on learnings from other projects' biggest annoyances isn't all that bad 😛

The emitLocal is not dropped, just renamed.
The emit logic in v0.11 is changed to use balanced event logic to support event-driven arch & event sourcing. The old emit logic was a simple broadcasting (to all services), so I renamed the old methods to broadcast* from emit*
The emitLocal in v0.10 is renamed to broadcastLocal in v0.11,
The emit in v0.10 is renamed to broadcast in v0.11.
I think the current logic is straightforward. All internal events got the $ prefix. And the $ prefix is reserved for internal core module events, like $node.* or $services.changed. So they don't conflict with the user custom events.
The emit sends event with balancing & grouping which is essential for event drive & event sourcing. The event will be sent to only one instance per services.
The broadcast sends event to all services on all nodes (include local node & local services)
The broadcastLocal sends event to all local services only. It is used by core modules to emit the internal $ events.
However in emit and broadcast have a protection, that they don't transfer events which is prefixed with $.
In a nutshell:
In your own services, don't use the $ prefix in event names when emit events. Use normal event names like user.created, payment.charged...etc and send them with emit. If you want to notify all services (e.g. cache.clear, or config.changed), use the broadcast method.


Parsing error: Cannot read file '/monorepo/tsconfig.json' eslint error in VS Code

I think it might be worth adding a note regarding VS Code eslint extension workingDirectories setting. More info here https://github.com/microsoft/vscode-eslint#mono-repository-setup
Adding this to the VS Code settings resolves the error.
    "eslint.workingDirectories": [
        {
            "pattern": "./apps/*/"
        },
        {
            "pattern": "./packages/*/"
        }
    ],

Hey @oste, thanks a lot.
I'm actually on Jetbrains product line, would you mind adding a P/R to the repo ?
Would be awesome

Certainly. Let me know how this looks https://github.com/belgattitude/nextjs-monorepo-example/pull/737

I merged https://github.com/belgattitude/nextjs-monorepo-example/pull/737, so I'll close.
Thank you

Sorry for bumping—posting this for others who might be searching this issue.
I had similar issue; adding VSCode settings did not work; solved by updating each package's eslint config, eg. /apps/blog-app/.eslintrc.js.
module.exports = {
+  parserOptions: {
+    tsconfigRootDir: __dirname,
+    project: ['./apps/*/tsconfig.json', './packages/*/tsconfig.json'],
+  },
}

Copy-pasted from https://github.com/typescript-eslint/typescript-eslint/blob/master/docs/getting-started/linting/MONOREPO.md#one-tsconfigjson-per-package-and-an-optional-one-in-the-root
yarn lint is not affected by all these, seems to be a VSCode + eslint extension issue.


Fix/656

Closes #656

Codecov Report
Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 64.01%. Comparing base (8fcc58a) to head (f9e8d33).

Files
Patch %
Lines

lib/src/commands/doctor_command.dart
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #657      +/-   ##
==========================================
- Coverage   64.59%   64.01%   -0.59%
==========================================
  Files          49       49
  Lines        1895     1895
==========================================
- Hits         1224     1213      -11
- Misses        671      682      +11

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Generate entire catalogue client

This PR tweaks openapi-generator to generate a Catalogue class that serves the generated API classes and uses retrofit and our own authentication.
The generated client is fully functional.
You can try the generation yourself by running ./catalogue/bin/generate.sh
This is only a small step away from dynamically generating TIDAL-compatible Android clients for any submitted OpenApi-compatible spec file.

You can try the generation yourself by running ./catalogue/bin/generate.sh

I think we should run this as part of the build process prior to compiling, rather than relying on people/CI manually calling a script when appropriate.

You can try the generation yourself by running ./catalogue/bin/generate.sh

I think we should run this as part of the build process prior to compiling, rather than relying on people/CI manually calling a script when appropriate.

Of course, but we're at the beginning of this process. There will be a number of intermediate steps that work but not like desired in a final product.


Module 1.1 Assignment

Given what you have learned about Git, why would you want to use Git in your classroom?

👋 Hi @LeandroDCI  can you answer the question as well 😄

👋 Hi @ericdrosado, sure sorry! 😄
We use git in the classroom because makes collaboration easier, allowing changes by multiple people to all be merged into one source.


fix: Close keyframe block on getStyles

Fixes #200

@cometkim Thanks for this <3
While we're still improving our CI workflow and PRs arent running tests. Can you confirm that you can do these succesfully:

yarn test
yarn build

Otherwise this PR looks good to me!

@peduarte tests are all green (but build?)

@cometkim one last thing, I implemetned CodeSandbox CI in #198
But it's not working here because your branch doesn't have the latest code from canary
Can you update this branch, and push, so I can validate everything is working on CodeSandbox?
Sorry, gotta be extra cautious while we don't have a decent CI in place :)

forgot to rebase :sweat_smile:

@cometkim ok looks like CodeSandbox is trying to build the packages, let's see. Seems to be stuck 😅
It worked here #198

Haha no worries, I can wait. slow CIs give me a break XD

looks like some issue with codesandbox ci

lol

I'm on my phone now so can't do much. Is it an issue with codesandbox or with our integration?
Can we try doing another commit?

No, I think the codesandbox (at least the CI service) is down now. no response from service.
I gonna sleep today. I will check maybe 9 hours later :)

@peduarte

@cometkim great work, thank you


Not honoring default['ssh_known_hosts']['file'] attribute in 6.x?

Cookbook version
6.1.0
Chef-client version
2.4.17
Platform Details
Ubuntu 14.04
Scenario:
Our test-kitchen builds are failing due to the custom known_hosts file we have specified not existing. It appears the cookbook is now ignoring the default['ssh_known_hosts']['file'] attribute and just writing to the default file unless explicitly overridden using file_location in the recipe.
Steps to Reproduce:
Run the tests from the recipe using an attributes file instead of hardcoded file_location and see what happens. https://github.com/chef-cookbooks/ssh_known_hosts/raw/master/test/fixtures/cookbooks/ssh_known_hosts_test/recipes/resource.rb
Expected Result:
Previously (in 5.2.1 and earlier) the custom attribute specified file was created and populated.
Actual Result:
Now the entries are simply added to the host wide known_hosts and not the user/application specific known hosts defined in attributes.

6.X moved this from the attribute to the resource itself. This is part of getting this resource ready to be moved directly into Chef. Chef 15 will ship with this resource and eliminate the need to depend on this cookbook. Since this cookbook will be going away the attributes for the cookbook also need to go away. This is breaking change, but that was part of the reason for the major version bump.


GUID conflicts when using this plugin together with DracoUnity

When using both your gltf extension plugins like this
    "com.atteneder.draco": "https://gitlab.com/atteneder/DracoUnity.git",
    "com.atteneder.ktx": "https://gitlab.com/atteneder/ktxunity.git",

Unity throws a few errors like this:
GUID [315b0c33fa9ea48f4bf45d3f6a2646c0] for asset 'Packages/com.atteneder.ktx/Runtime/Plugins' conflicts with:
  'Packages/com.atteneder.draco/Runtime/Plugins' (current owner)
We can't assign a new GUID because the asset is in an immutable folder. The asset will be ignored.

Unity cannot generate new meta files because the packages in the PackageCache are immutable.

thanks for the report. I noticed it as well (without major issues though), but I have to look into it.
Do you, by any chance, know whether *.meta files can be omitted/ignored by GIT for packages?

I noticed it as well (without major issues though)

It doesn't create issues in the Editor, however creating a build (e.g for Android) fails.

Do you, by any chance, know whether *.meta files can be omitted/ignored by GIT for packages?

As far as I know they can't be omitted since Unity can't create them because they are in an immutable folder.

Those should be fixed now!


Fix race between ExternalPluginManager and PluginManager during profile change.

EPM and PM race to do their profile change work, because they do not use subscriber priority and do their work on different threads. This pull request makes sure that EPM runs before PM so that external plugins that are not installed in the new profile cannot be turned on by PM. If they are turned on by PM, it can cause plugins that do not shut down cleanly to cause issues like leftover infoboxes or overlays, and makes the profile change take longer.

With the exception of startup, the profile changes actually always already happen on the executor thread. So, this can probably be fixed just by prioritizing the EPM to run before PM.

added in 89bfa828c90a0f5b1fbf8d5948bdd8d3c7139adc


buttons don't work

I used to use joystick.js  I used nipple.js both have the same bug even on the demo you can just understand what I mean just open the demo on A touch phone buttons inputs wont wont work just the just the joystick will work.

I couldn't reproduce the issue with the devices I have.
Please give me more details about what device/code you are using and re-open the issue if needed.


update helpernode commit

contains fixes for haproxy failover and ppc64le grub failure
Signed-off-by: Prajyot Parab<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Prajyot-Parab
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Prajyot-Parab]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bpradipt, Prajyot-Parab
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Prajyot-Parab,bpradipt]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bpradipt, Prajyot-Parab, yussufsh
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Prajyot-Parab,bpradipt,yussufsh]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[wdm] Improve

[wdm] Delete ansible temp file after execution

[wdm] remove hardcoded path from ansible-playbook

[wdm] flush stdout/err before calling subprocess.run

/test gpu-operator-e2e

merging


Make sure it also works with tilted screens

If one turns on the other screen orientations for the device and run the example, this calendar is not rendering properly anymore. Make sure that those device orientations are supported.

@ruurd what device / simulator / demo are you using? I'm able to repro if I rotate during on the "scroll to day with animation" demo, but only while the animation is running. Are you able to repro for any other demos?

I'll make an STR for you later.

STR:

open HorizondatCalendarExample workspace
select HorizondatCalendarExample project then target
check all device orientations possible (all four)
build and run
select any example available and wait until rendering is complete
rotate the device

EXP: calendar is rendering properly when device is being turned 90 degree CW and CCW
ACT: calendar is only partly visible
Device: iPhone11 in simulator

Ah I understand the bug! This is just a sample project layout issue. I can update the constraints so that the example project looks decent in all orientations.

Maybe that's a good thing because this gives the wrong impression and might put people off...

@ruurd I just put up a PR to fix this. Thanks again for the detailed bug report

Good. I'll keep an eye on that. And the bug report: noblesse oblige, right?

This is fixed now @ruurd !


configure: Add git-describe information in version

Necessary information for people that are using versions
that are not a major release.
Signed-off-by: Patrick José Pereira<EMAIL_ADDRESS>

ping @lucasdemarchi


all routes should return the full url of the page and its title

With help of allroutes code, we need to get the entire list of routes with its full url and name or title of the page.
If we pass additional parameter title to the Route:: add and get it working with name of the link, which is fine to organize them.

You cannot get the "url" or "link" of all routes because it just does not exist inside the data. For example the route /user/([0-9]*)/edit can have the url /user/7/edit and also the url /user/4/edit. You cannot convert this pattern to a link.
Instead you should have an array or a database table holding your navigation entries. You can use this for generating menus. But routes and URLs are two different things. A router is not a link or navigation generator.

Not all routes dude, some the main routes like few main pages might we have, that's only we can save it with a page title. and re call them when we prepare the menu and current page highlight options.

!??! I still don't get it. Please try to create complete sentences.

THere is no use for that since you always route part of your output ready (ex: /user/([0-9]*)/edit). but if you still want to know the complete url, a basic php code like this should work:
$url = $_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI'];


fix(experimental): refactor to persistQueryClient and persistors

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fixed error with time variable when writing netcdf file

There was a problem in seaexplorer.py because of how
xarray handles the encoding for the calendar in attrs on
variable 'time'. To fix it, I replicated the code used to fix
the same issue with the encoding for the units in attrs on
variable 'time'. There may be other places this fix is needed.

That makes sense.  However, it would be nice if the yaml could somehow indicate the units in a way that xarray was happy with?

I do wonder if both these pops merit a warning that we are ignoring what is in the yaml....


Slight tweaks to suggestion systemd service

Now the default configuration for environment and arguments matches what the bot expects when running under systemd.
Additionally there's now a suggestion (and assumption in the service) that you run it under an unprivileged "seance" user, which is a general best practice. (More tweaks in the future to further lock down the service are probably worth considering since it needs only network access at the moment).

The suggested useradd command creates a regular user with the home directory as /home/seance; it might be better to change it to something like useradd --home-dir=/srv/seance, and if you're going to explicitly put running as a special user in the service file, then it should probably also mention that if Séance is installed with pip3 install --user, that that command will need to be run as the Unix user running Séance.

We suggest using no home directory since it's not actually required and installing globally.
If that approach sounds okay we'll rework the readme in the PR.

Installing globally on Linux tends to lead to some messiness, since it can interfere with the system package manager. On the other hand, it's the preferred approach on macOS. That's why I had plain pip3 install in the readme — with recent versions of pip, it'll suggest --user if you run it without sudo.
You could have it be agnostic in the readme, something like:

Such a user can be created usually with sudo useradd seance, possibly with something like --home-dir=/srv/seance if you want to install Séance with pip3 install --user.

Installing globally on Linux tends to lead to some messiness, since it can interfere with the system package manager.

Was afraid you would say this. It's a fair point. I forgot how pip handles global installs (not well), and think you're probably right. It seems a bit silly to have a homedir just so a user install works, but it's the easiest answer here outside of system packaging for Séance.
How does this change sound to you:

Such a user can be created with sudo useradd seance. To avoid installing Séance globally, which might be ill-advised, you can create a home directory for the user, something like sudo useradd --create-home --home-dir /srv/seance seance will create a home directory for the user in /srv/seance. To install seance and sdnotify for this user use sudo -u seance pip3 install --user sdnotify git+https://github.com/Qyriad/Seance.

This guides someone through the whole process, especially the last bit which can be confusing for a user which isn't setup with login by virtue of having no password set.

That works~

Alas, if only global pip installs installed to /usr/local by default; then we wouldn't have any of these problems in the first place.

Alas, if only global pip installs installed to /usr/local by default; then we wouldn't have any of these problems in the first place.

Yes indeed. I'll make the changes.


Unable to find stanford-postagger.jar on CI server

The CI server reports that NLTK cannot find the Stanford POS tagger:
https://nltk.ci.cloudbees.com/job/nltk/lastCompletedBuild/TOXENV=py27-jenkins,jdk=jdk8latestOnlineInstall/testReport/nltk.tag/stanford/StanfordPOSTagger/
Cf. http://stackoverflow.com/questions/34726200/nltk-was-unable-to-find-stanford-postagger-jar-set-the-classpath-environment-va

This requires the same hack from https://gist.github.com/alvations/e1df0ba227e542955a8a. I think a more principled way to handle this is to complete #1249 and then wrap the new stanford API in StanfordPOSTagger, StanfordNETagger and StanfordTokenizer.

Thanks @alvations. But I noticed a problem in jenkins.sh,
and hope this resolves the problem (https://github.com/nltk/nltk/commit/f3a40c5ee4f7a7d7f455aa758fba99a44949332e).

Problem solved except for Python 2.7.

Mysteriously resolved for Python 2.7 as well.


Problem with Transaction to tensorflow1.0

After upgrading my tensorflow version to 1.0, this code will have Error even if I have transformed this code to 1.0 version.
The error is
ValueError: Variable d_h0_conv/w/Adam/ does not exist, or was not created with tf.get_variable(). Did you mean to set reuse=None in VarScope?

Ok, this problem is solved

Can you please tell me how was it solved? I have the same issue but can't figure it out.
Thanks,

@Fuckmi, how did you solve the issue? I also get the same error (tensorflow 1.0.0). Thanks!

sorry for my late reply.
Just follow the changes in
https://github.com/yenchenlin/pix2pix-tensorflow/pull/14/files
good luck


Cleanup: delete and ignore temporary OS files (.DS_Store)

A .DS_Store file is specific temporary local file, generated by Mac OSX, containing specific settings for a local setup. It is common practice to prevent them from pushing into the VCS.
To be on the safe side, I added some typical os generated files into the existing gitignore

Thanks!


Container feature is not supported when runner is already running inside container.

Getting this error. and runners fail.
Container feature is not supported when runner is already running inside container.

This is an issue github/actions/runner's. See

https://github.com/actions/runner/issues/406
https://github.com/actions/runner/issues/367


Permission denied! Please run script with sudo privileges!

I gave sudo permission but the script doesn't work. I got the same error on different operating systems. What should I do ?
MACOS

Do you want to install Elasticsearch? [Y/n]: n
Do you want to install Kibana? [Y/n]: n
Permission denied! Please run script with sudo privileges!

There is no data on 2021-11-08 13:53:57.659693+03:00 in voip
Do you want to import all cti json feeds to elasticsearch? [Y/n]: Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/urllib3/connection.py", line 159, in _new_conn
    conn = connection.create_connection(
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/urllib3/util/connection.py", line 84, in create_connection
    raise err
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/urllib3/util/connection.py", line 74, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 61] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/elasticsearch/connection/http_urllib3.py", line 245, in perform_request
    response = self.pool.urlopen(
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/urllib3/connectionpool.py", line 726, in urlopen
    retries = retries.increment(
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/urllib3/util/retry.py", line 386, in increment
    raise six.reraise(type(error), error, _stacktrace)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/urllib3/packages/six.py", line 735, in reraise
    raise value
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/urllib3/connectionpool.py", line 670, in urlopen
    httplib_response = self._make_request(
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/urllib3/connectionpool.py", line 392, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/http/client.py", line 1279, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/http/client.py", line 1325, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/http/client.py", line 1274, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/http/client.py", line 1034, in _send_output
    self.send(msg)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/http/client.py", line 974, in send
    self.connect()
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/urllib3/connection.py", line 187, in connect
    conn = self._new_conn()
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/urllib3/connection.py", line 171, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x1060decd0>: Failed to establish a new connection: [Errno 61] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/abdullahcicekli/CTI-Feed-Collector/cti_collector.py", line 319, in <module>
    postElastic()
  File "/Users/abdullahcicekli/CTI-Feed-Collector/cti_collector.py", line 82, in postElastic
    es.indices.create(index=elasticIndex.lower(), ignore=400)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/elasticsearch/client/utils.py", line 152, in _wrapped
    return func(*args, params=params, headers=headers, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/elasticsearch/client/indices.py", line 119, in create
    return self.transport.perform_request(
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/elasticsearch/transport.py", line 390, in perform_request
    raise e
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/elasticsearch/transport.py", line 358, in perform_request
    status, headers_response, data = connection.perform_request(
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/elasticsearch/connection/http_urllib3.py", line 258, in perform_request
    raise ConnectionError("N/A", str(e), e)
elasticsearch.exceptions.ConnectionError: ConnectionError(<urllib3.connection.HTTPConnection object at 0x1060decd0>: Failed to establish a new connection: [Errno 61] Connection refused) caused by: NewConnectionError(<urllib3.connection.HTTPConnection object at 0x1060decd0>: Failed to establish a new connection: [Errno 61] Connection refused)

LINUX

Permission denied! Please run script with sudo privileges!
Traceback (most recent call last):
  File "cti_collector.py", line 316, in <module>
    feedService()
  File "cti_collector.py", line 299, in feedService
    blocklistDE()
  File "cti_collector.py", line 274, in blocklistDE
    data = urlopen(req).read().decode('utf-8')
  File "/usr/lib/python3.8/urllib/request.py", line 222, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/lib/python3.8/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/usr/lib/python3.8/urllib/request.py", line 640, in http_response
    response = self.parent.error(
  File "/usr/lib/python3.8/urllib/request.py", line 569, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.8/urllib/request.py", line 502, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.8/urllib/request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 503: Service Unavailable

Script was configured for ubuntu. ELK stack download link has changed. You can check and edit. You can see the error when you read it.

I can't see any change in the code. Are you sure you pushed it ?

I said you can change. I didnt fix my code. ELK download link changes frequently

I am sorry. got it a little late. Ok. I will try.

I understand the problem. I got stuck with the try-except block in configureServices() function because I selected "no" in elastic and kibana setup options. For this, when we return the results of installKibana() and installElasticsearch() to the configureServices() function, the error disappears. So if the option is no, the function should not work. The same should be added to the postElastic() function. Could it be better this way? What about?
FOR configureServices() :

For postElastic() :

p1 = click.confirm('Do you want to install Elasticsearch?', default=True)
p2 = click.confirm('Do you want to install Kibana?', default=True)

postElastic() function is used for import all cti feeds to elasticsearch database. In the last error blacklistDE returned connection refused. Errors are not from ELK services. Please debug your script see the actual failed area.


Idea: llm logs conversations

A command for listing just conversations - their IDs, titles, number of messages and start date.

Not sure how best to design this one.
I got to this:
@cli.command()
def conversations():
    "List conversations stored in the logs"
    db = sqlite_utils.Database(logs_db_path())
    migrate(db)
    for row in db["conversations"].rows_where(order_by="id"):
        click.echo(
            "{id} {model}\t{name}".format(
                id=row["id"], model=row["model"], name=' '.join((row["name"] or "").split())
            )
        )

I don't like it much though:
01h88dkcfxtkn1hpp5mk8er1q9 openrouter/anthropic/claude-2	import llm from llm.default_plu…
01h88frbzhhz3jwyxarq9m9wkj mlc-chat-Llama-2-7b-chat-hf-q4f16_1	What was the beef between the t…
01h88fxfmdenpksh27ewrn3766 mlc-chat-Llama-2-13b-chat-hf-q4f16_1	What was the beef between the t…
01h88p72dmmw46bnw96mz7we3g gpt-3.5-turbo	from typing import List, Dict d…
01h88qmbkg4tnqzgzvdj58hrtc gpt-3.5-turbo	ubuntu install build essentials…
01h88qve060a3r93hahhn7bg33 gpt-3.5-turbo	ubuntu: tclsh not fonud
01h88r1q9x6sa9tw57g3fajtj1 gpt-3.5-turbo	What do you know about OpenAI F…

The design challenge here is how much information to display and how to display it.
I think I want:

Conversation ID - required, needed to paste into llm logs --cid X to see the actual conversation.
Model ID - but these can be pretty long. I could try and find the shortest alias instead?
Model name - or some textual indication of what the conversation was about. Eventually these will be LLM generated and better - right now they are the first 20 or so characters of the initial prompt
Conversation start date (and maybe most recent date? Gets even messier then though)
How many messages there are in the conversation

Really not sure how best to lay those out in a concise terminal format though.

Docker might be good inspiration

In particular:

Shortening the id to a unique prefix or suffix for display
Truncated lon strings like the last message
Generating or using friendly names

I think shell_gpt's solution is pretty good. (It's the sole reason I still use sgpt.)
shell_gpt allows you to name your chats. It's very handy for retrieving them, and providing instant topical context when I have a new question. I have chats for databases, nix, parsing, etc.
If people are trying to resume a conversation, a name beats a cid hands down.

just came here to say the same thing - i'd like to name my convos then continue them like llm chat --cname thesis-assistant


Includes within includes?

Is it possible to have a template.html like:

    @@include("./partials/@@variable.html")

and then have something outside like:
    @@include('./partials/template.html', {
        "variable": "some_other_template"
    })

You may do this

    @@include("./partials/" + variable + ".html")

It's just a variable :-)

@TrySound
It doesn't seem to work for me.
gulp-file-include v 0.9.0
  @@include("../demo/block-module.html",{
    "content" : "block-module-one"
  })

    @@include("../demo/" + content + ".html")

the output html displays:
@@include('../demo/' + content + '.html')

@coderhaoxin

Not supported for now.

Add to todo list of v1.0.0, shame on such a long delay for v1.0.0 :cry:

Reference on #62.
This issue could be closed.

Is there any workaround?


Corrupted configuration cache is persisted, resulting in guaranteed failure (until explicitly invalidated)

In some scenarios, when the configuration cache is corrupted, this corrupted configuration is persisted to the cache instead of being cleared.
Expected Behavior

This corrupted configuration should be cleared (or, if not possible to do partial evictions, the cache should be invalidated). The cache should be rebuilt on the next use, such that any errors due to configuration cache corruption are no longer observed (assuming a well-formed build script).
Current Behavior

The corrupted configuration cache will be loaded upon even invocation of ./gradlew build, resulting in failures in all futures attempts (until the cache is manually invalidated).
Context

It took quite a bit of time to understand what the issue was, due to only a minimal amount of information being returned, combined with a much much faster failure than I had ever observed prior for this project.
Steps to Reproduce

Your Environment

Build scan URL: (cannot link, will provided all relevant information below)
Gradle 7.6
Local Console Output
Starting a Gradle Daemon (subsequent builds will be faster)
Configuration cache is an incubating feature.
Reusing configuration cache.

FAILURE: Build failed with an exception.

* What went wrong:
org.gradle.internal.operations.BuildOperationInvocationException (no error message)
> java.io.EOFException (no error message)

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.

* Get more help at https://help.gradle.org

CONFIGURE FAILED in 5s

Publishing build scan...
https://gradle.***.com/s/***

Configuration cache entry reused.

Gradle Scan "Failure" tab output
Failure 1 of 1 (No message provided)

(No message provided)
> Buffer underflow.

Exception
org.gradle.internal.operations.BuildOperationInvocationException: (No message provided)
  > at org.gradle.internal.operations.DefaultBuildOperationRunner.throwAsBuildOperationInvocationException(DefaultBuildOperationRunner.java:192)...
    Caused by: java.io.EOFException: (No message provided)
    > at org.gradle.internal.serialize.kryo.KryoBackedDecoder.maybeEndOfStream(KryoBackedDecoder.java:80)...
      Caused by: com.esotericsoftware.kryo.KryoException: Buffer underflow.
      > at com.esotericsoftware.kryo.io.Input.require(Input.java:181)	...

Notes

I believe this corruption only existed in the ./gradle build cache, as I was able to run a module-specific task without any issues.
Once the cache was corrupted for ./gradlew build, I observed failures 100% of the time, until I manually invalidated the cache via rm -rf .gradle/configuration-cache.
For any failing commit, I was able to execute ./gradlew build on my CICD pipeline, with zero failures observed.

Sorry that you're having trouble with Gradle!
Your issue lacks information about how to reproduce the problem you're having. A minimal reproducible example can really help us track down and fix your problem. We may also be able to suggest workarounds or ways to avoid the problem if we can reproduce it.
The ideal way to provide a reproducer is to leverage our reproducer template.
This issue will be closed after 7 days, unless you can provide more information.

Did you have org.gradle.unsafe.configuration-cache.warn=true set?

Did you have org.gradle.unsafe.configuration-cache.warn=true set?

@eskatos I had the same issue and org.gradle.unsafe.configuration-cache.warn=true was set.
Removing it fixes the issue...


AdminLTE Custom Tabs - First tab deactive problem

Hello!
When using "AdminLTE Custom Tabs", the tab's title move to the right by 1px.
To reproduce the problem:
1: Find the "AdminLTE Custom Tabs" in "UI Elements > General"
2: Click on "Tab 2" (Tab 1 was in active mode)
3: "Tab 2" and "Dropdown" title will move to the right by 1px.
This problem appears in stable and 2.1-beta release.
This problem appears even without tab content (just UL and LI is enough).
This problem does not appears in default bootstrap nav-tab.
Using Chrome 42.
Best regards!

Confirmed.
Thank you for reporting this! Will get to it soon.

Fixed in master. Thanks again!

test

What are you testing @persoomar? :)

Lol
Sorry it was a mistake :)
Good night Abdallah .
Le mar. 13 déc. 2016 à 03:26, Abdullah Almsaeed<EMAIL_ADDRESS>a écrit :

What are you testing @persoomar https://github.com/persoomar? :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/almasaeed2010/AdminLTE/issues/378#issuecomment-266617274,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWLj-oj5Sk2aHjq5pvvXhrZZbNCATfIkks5rHgJggaJpZM4EMH_y
.

where i can get the version with the customer tabs,  i need the change the right panel without refresh all page


Oto test coverage

Headless testing of such a low level library might be impossible, or take a lot of work to get started. I'd like to explore getting some test coverage for oto.
The biggest roadblock is testing the drivers without any actual sound cards available. There's a couple ways we could go about doing this. In the test, we could replace theContext's driverWriter with something like the following
// stepPlayer is used for headless mode audio, such as for tests
// you can control exactly how many steps it takes which allows for verification
// of the PCM data written to it.
type stepPlayer struct {
	ThrowWriteError bool
	stepStart       chan []byte
	stepDone        chan struct{}
}

func (l *stepPlayer) Write(b []byte) (int, error) {
	if l.ThrowWriteError {
		return 0, errors.New("write error")
	}
	l.stepStart <- b
	<-l.stepDone

	return len(b), nil
}

func (l *stepPlayer) Close() error {
	return nil
}

func (l *stepPlayer) Bytes() []byte {
	return <-l.stepStart
}

func (l *stepPlayer) Step() {
	if len(l.stepDone) > 0 {
		return
	}
	l.stepDone <- struct{}{}
}

(this is what I replaced oto.Player with in engo to test it. All it does is take in the PCM data sent to the player and spit it back out.) This lets the tests check that data is actually being passed, and the tests can compare that data to what it expects. The upside is it should work, with build tags. The downside is that it wouldn't actually test any of the actual drivers, only the mock test driver would actually ever be tested. So coverage would be less than 100%.
Another alternative would be to create/use dummy drivers for testing. For example, ALSA has a loopback driver here. If it could be setup on Travis, then this driver would take in data and spit it out, similar to above. This would have the added benefit of actually testing the driver code, so coverage could get to 100% (or at least pretty good!) There's also a node.js package here. Not sure about windows/ios/mobile testing. I'll look around and see.

Dummy drivers for testing sound the way to go so far.


Support to set local relay port through environment variable

Relay local port variable through environment variable.
If not set will default to 7000

@diazemiliano thanks for your contribution, please for next contributions sign your commits


Changed Abode cache file path, add cache path to config flow

Description:
Fixes a permission denied issue by passing in a cache path in the config flow. This commit also removes the CWD path ./ in the defined cache/pickle file location to match how several other components define their pickle file locations: feedreader logi_circle ring mopar
Related issue (if applicable): fixes #28242
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the [development checklist][dev-checklist]

Still working to fix the issue...

This fixes the issue...thanks @MisterWil


Add ability to configure the item comparer function.

We need the ability to configure the item comparer using something like Lodash.isEqual. With this, we can do the following...
import _isEqual from 'lodash/isEqual';
...
...
let choices = new Choices(...., {
  itemComparer: _isEqual
});

Nice @travist - thanks 💯

Thanks for merging this @jshjohnson. What is the possibility to get this back-ported into the 3.x branch? We will make a pull request against master if that makes it easier for you.
Right now we are duck-punching a Choices method within formio.js to allow us to have deep equality checks. Having this in the 3.x choices would save us from that hack.

@travist  Sure, feel free!

Here it is thanks. https://github.com/jshjohnson/Choices/pull/334


Run Handesk with existing Laravel project

It is possible to run the Handesk with a existing Laravel project? As if it was a common package.
I'm asking this because the installation process aims a clean installation.

Actually it is meant to run as standalone app, so you don't merge two different things,
But I find that a really easy way to go is to have it on the same server with virtual hosts so you can have
handesk.yourdomain.com and yourdomain.com at the same server
if you need comunication between both apps, you can use the api (https://github.com/BadChoice/handesk-php)

Do users then have to have 2 registrations/logins?  One to the main site and one to the Handesk?

Right now, as handesk goes appart, they need to, but we can try to work on
something to make it use just one, we can try to create some kind of auth
driver so you can configure it to use your main site’s users
Jordi Puigdellívol
El 5 de marzo de 2018 a las 11:15:42, Brian Merritt (
<EMAIL_ADDRESS>escribió:
Do users then have to have 2 registrations/logins? One to the main site and
one to the Handesk?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/BadChoice/handesk/issues/101#issuecomment-370372431,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFuzGRiBafLYhsJZvQjKmdxruvlLqcjks5tbRBMgaJpZM4SVy-N
.

It makes sense to run it as a separated thing. But in fact, this thing about having two login systems is not optimal.
But if we had some kind of integration, would be a blast!

yeah, we would need to have a laravelApp auth driver o something like this, but the problem is that we need to auth it there, so the other app needs some kind of endpoint, or handesk needs to know about the other app key an auth database connection

If running on the same server, you could have a background process that adds the users from one site to handesk (assuming both use the same password encryption algorithms).
Alternatively, they could use the same login details for both - less secure, but easier for users.


auto-patchelf: don't use grep -q, as it causes Broken pipe

This rare sitation was caught when building zoom-us package:
automatically fixing dependencies for ELF files
/nix/store/71d65fplq44y9yn2fvkpn2d3hrszracd-auto-patchelf-hook/nix-support/setup-hook: line 213: echo: write error: Broken pipe
/nix/store/71d65fplq44y9yn2fvkpn2d3hrszracd-auto-patchelf-hook/nix-support/setup-hook: line 210: echo: write error: Broken pipe

The worst is that derivation continued and resulted into broken package:
https://github.com/NixOS/nixpkgs/pull/55566#issuecomment-470065690
I hope, replacing grep -q with grep will remove this race condition.
cc @dasJ @aszlig as I see there was previously some attempt to fix Broken pipe in other places (https://github.com/NixOS/nixpkgs/pull/40598)

:thinking:  I see it doesn't cause mass rebuilds. Ok.

@aszlig I'll merge this in a week if you don't object. I can't find shorter solution, which would work with pip list example above.
It requires rebuilding Qt, so I'll leave this for staging job.

It requires rebuilding Qt, so I'll leave this for staging job.

Since when does this need to rebuild Qt? Looking at the changed paths reported by ofborg, I can't see anything related to that, am I blind?

@aszlig you are correct, it was rebuilding Qt because of staging branch. All is good on master.


WIP: Add end to end integration tests running against emulator for MySql source database

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@nevinheintze - Added you as a reviewer to provide early feedback on the decision to add mysql container in the same .yaml file.
I am working on adding integration_test.go under mysql/ which will contain end-to-end integration tests for mysql.

Hi @nevinheintze - this is ready for another pass. The tests are very basic in nature, though they cover both mysqldump and direct connect. Once this skeleton is in, we can think about what exact tests should we have - not just for mysql, but for postgresql and dynamodb as well.


Merged update from the main repo.

Context

Changes proposed in this pull request

Guidance to review

Relevant links
Things to check

[ ] I have added any new ENV vars in all deployed environments
[ ] I have tested any code added or changed
[ ] I have run integration tests

Merged from main repo


fix: service name

Hello @dpschen
Thanks for you contribution!
I'm really sorry for the delay between your PR creation and the review 😞
Can you just signoff your commit (equivalent agreeing Docker DCO) and we'll be ready to merge it!
Thanks 🙏


Can't import the package

Expected Behavior
When selecting the .unitypackage in the Finder it should add the package to my Unity Project.
Actual Behavior

Steps to Reproduce
The exact first two steps from the "Getting Started" section.
I reproduces it with multiple project including a blank one.
I don't have any issue with other packages.
Specifications

Unity Version: 2020.3.9f1
Platform (Windows or Mac): M1 MacBook Pro / macOS Big Sur 11.3.1

Hi,
Does the assets/plugins folder exist? If not try to create before import. Never seen this error before.
The only thing I could think of is the libraries not be able to import, but it should as they are tagged as any OS and mono compatible.
Could you try to import this package? I exported directly from Unity Editor instead of the generated by CI.
UnityStreamDeckExportedByEditor.zip

Hi and thanks for your answer.
The "ProjectRepo/Assets/Plugins" didn't exist so I created it but the end result was the same.
I downloaded "UnityStreamDeckExportedByEditor.zip" but it doesn't contain any .unitypackage file. Is it another way to import packages ?

Inside the zip file, there's a .unitypackage file that you can import.
Extract the zip anywhere, go to the Unity Editor, right-click on the Assets folder, import package and select that file.

If you keep the version I exported manually over the version from GitHub releases you can delete the test.txt file that is inside: Assets/Plugins/StreamDeck/Editor/

For some reason, the OS was identifying the .unitypackage file as a folder.

I was able to import the package in my Unity project that way. Thanks a lot for your time, I'd star your repo if it was not already done :)

Thank you, I hope it helps.