WIP: Update contrasts to new pybids format

This is WIP but I want to start a discussion on how to go forward. So far I only have some minor syntactic changes.
Previously, pybids returned a single contrast matrix, with each row representing an input from the previous level. E.g. if you had 3 runs for one subject, and one contrasts; at the session level you would have 3 rows. Usually these would be all 1s, as you are trying to take the average of the 3 runs at the session level.
E.g.
contrasts:
   RT-trial_type
0              1
1              1
2              1

index:
   run subject              task
0    1      01  mixedgamblestask
1    2      01  mixedgamblestask
2    3      01  mixedgamblestask

Pybids now returns NamedTuples in order to keep the weights for each contrast separate, in order to accomodate f-tests (which would not fit on this table).
The other difference, is that now the weights are not repeated across input units. In the previous example, you would only have 1 "row".
e.g.
In [34]: contrasts[0].weights
Out[34]:
   RT-trial_type
0              1

This contrast applies to all inputs with the following entities
entities={'subject': '01', 'task': 'mixedgamblestask'})

So basically, I'm trying to figure out if we should emulate the way contrasts were previously represented by expanding out the single row to three... or if there is a more efficient way to do this that makes sense.
Relatedly, we should probably also tack F-tests here, which probably means we shouldn't pass around a single matrix between nipype interfaces (as we wont be able to w/ t- and F tests mixed together.
I think the only difference here is that F tests have multiple rows. I also assume these will mostly be used at the first-level (although not exclusively).
E.g an F-test testing for any effect in RT or Accuracy
RT Accuracy Condition
1       0           0
0       1           0

Just a heads up I pushed a few fixes.

@adelavega Thanks. I'll have a look ASAP.


Observe Resource behavior according to RFC 7641

Hi,
I've read the RFC 7641 (observing resources in CoAP) but i don't fully understand the behavior of the example in examples/observe.
The server with func periodicTransmitter send messages to clients every second but the clients only receive messages every two minutes inside the func observeFunc. How can I allow the client to read all (or most) of the messages sent by the server?
Thanks for your time

Hi. Thx reporting the issue. There was implemented the deduplication of notification some time ago. And the server sends a notification without increasing the observation option.  So it is fixed in commit https://github.com/plgd-dev/go-coap/pull/263/commits/b60e8efc1e3ed4551d65b34a563147eb308dca3f

Thank you for the quick response!


refactor(jmeter): Add config file for jmeter version used by all scripts

Checklist

[X] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[NA] Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
[NA] The description lists all applicable issues this PR seeks to resolve
[NA] The description lists any configuration setting(s) that differ from the default settings
[ ] All tests and CI builds passing

Description
This PR sets a single configuration file jmeter-version.sh for entering the JMeter version, and uses that in all scripts.

Codecov Report

Merging #278 into dev will increase coverage by 1.43%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev     #278      +/-   ##
============================================
+ Coverage     20.14%   21.57%   +1.43%
- Complexity      406      427      +21
============================================
  Files           151      151
  Lines          8196     8366     +170
  Branches       1101     1164      +63
============================================
+ Hits           1651     1805     +154
- Misses         6399     6408       +9
- Partials        146      153       +7

Flag
Coverage Δ
Complexity Δ

#unit_tests
21.57% <ø> (+1.43%)
427 <ø> (+21)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

...com/conveyal/datatools/common/utils/Scheduler.java
46.98% <0%> (-1.21%)
9% <0%> (ø)

...veyal/datatools/manager/jobs/MergeFeedsResult.java
100% <0%> (ø)
1% <0%> (ø)
:arrow_down:

...conveyal/datatools/manager/jobs/MergeFeedsJob.java
78.84% <0%> (+5.45%)
111% <0%> (+21%)
:arrow_up:

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 b3754e5...6619e23. Read the comment docs.

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


Heh, nice. I will probably use this myself.


Support in the url builder for trailing slashes.

Priority
This PR is blocking my next action as I am trying to use the devour client to consume an API that uses trailing slashes for collection URLs.
What Changed & Why
Added a constructor option in object form to optionally append trailing slashes to either/both collection and resource urls, and also an option to be passed to urlFor/buildUrl to accomplish the same thing without the knowledge of url type.
Testing
List step-by-step how to test the changes.

[ ] Run included tests

Bug/Ticket Tracker
Discussed in #57
Legal/Security/Privacy
No known concerns.
Third-Party
None added
People
@Emerson @billxinli @spiderbites

Added constructor option to append trailing slashes to collection and resource urls separately.
Added an option to urlFor to append a trailingSlash.
Added an argument to buildUrl that will append the trailingSlash when true is passed.

Hi @raizyr thanks for putting the work in on this and apologies for taking a while to look at this.  I've made some changes here: https://github.com/twg/devour/commit/5892571a13c8fb0aca99eaf3d6ce53024310c1af
I wanted to remove the separate slash argument for buildUrl and have the slash settings just set in the constructor.  I've also paired down the tests a bit.
If this would satisfy your use case I'll go ahead a close your PR and re-open one from the devour branch.
Cheers

The changes in 5892571a13c8fb0aca99eaf3d6ce53024310c1af seem to restrict slashes to all or nothing.  This would make it difficult to use devour for existing APIs where a combination is used, ie. slashes on collection URIs and no slashes on single resource URIs.

Nope, it keeps that functionality in there.  See an example in the tests here: https://github.com/twg/devour/commit/5892571a13c8fb0aca99eaf3d6ce53024310c1af#diff-3f6a2558d91eece0e6bb598535c20778R229

Yeah, sorry, I scanned right past it the first time I looked.  This should work for my use case as well as any others I can think of at the moment. Thanks!


Simplify CMake dependency management.

Overview
The configuration of dependencies for CMake is getting out of control, we need to simply it.  I am going to propose that we support only two configurations:

external: all the dependencies are compiled from source, obtained using ExternalProject. This would be the preferred build configuration, we would use this most of the time ourselves.  It is the same approach we use for the Bazel builds.
package: all the dependencies are assumed to be installed and discovered using find_package().  This would work for both vcpkg-based builds and builds that install google-cloud-cpp.

Detailed Design
New external dependencies
We would need to introduce external packages for some of our current dependencies:

libcurl is currently supported only as a find_package() module.
OpenSSL (or more likely BoringSSL) is needed by libcurl, gRPC and google/cloud/storage.
zlib is needed by libcurl and protobuf.

Simplify configuration
We should introduce a single configuration option GOOGLE_CLOUD_CPP_DEPENDENCY_PROVIDER that sets the source for all the modules.
Handling Differences
Some dependencies create different targets when compiled using ExternalProject vs. discovered via find_package.  For example, googletest creates GTest::gtest_main when compiled from source, but the CMake module creates GTest::Main.  We will create aliases for such cases and prefer the names created by the ExternalProject.
Sometimes the results of using vcpkg + find_package() vs. find_package() on a Linux installation are slightly different.  In those cases we will normalize the targets created by vcpkg + find_package() to be the same results as ExternalProject.  Again, googletest is the main culprit on this area.
Exceptions
OpenSSL is very hard to manage and an ExternalProject, it does not have a CMake file, and in fact requires Perl to compile.  BoringSSL cannot be installed, so it is not usable as an external project.  We propose to keep using OpenSSL as an external dependency.
Alternatives Considered
Do nothing: we already have too many configurations we do not test, this is not viable.
Only Support one of the configurations: this is not tenable, both package and external are needed because:

Supporting find_package() is good hygiene, it is a necessary configuration to support any package manager, and it is expected that some of our users will require it.
Supporting ExternalProject makes it easier to start using the library (nothing to install before you try it), it makes it easier to develop, and makes the CMake and Bazel build more homogeneous.

What happened to pkg-config: sometimes the OS does not install the CMake configuration files and the only way to discover the dependencies is to use pkg-config(1).  We have "supported" pkg-config builds in the past, but they are not tested and are very brittle.

@houglum @frankyn @mbrukman please take a look and let me know if it sounds crazy.

If it vastly simplifies dependency management, that's a huge plus for maintainability.  From a developer standpoint, this gets +1 from me.
I admittedly don't have enough experience with various "real-world" build systems/scenarios (read: non-Bazel) to determine whether or not this might be annoying for users though -- offhand, I can't think of any reasons users might want to support mixed dependency configurations, but that doesn't mean such reasons don't exist :)

@coryan We'd generally prefer vcpkg + find_package() to give the same results as find_package() (unless find_package() is horrifyingly broken ;)). Could you go into a bit more detail about the gtest issue?

@ras0219-msft  the problem is not with vcpkg is how we use it (or abuse it maybe?).  In any case,
x created #1300 to track this specific issue.  Let's discuss further there if you want to.

Overview
The configuration of dependencies for CMake is getting out of control, we need to simply it.  I am going to propose that we support only two configurations:

external: all the dependencies are compiled from source, obtained using ExternalProject. This would be the preferred build configuration, we would use this most of the time ourselves.  It is the same approach we use for the Bazel builds.
package: all the dependencies are assumed to be installed and discovered using find_package().  This would work for both vcpkg-based builds and builds that install google-cloud-cpp.

Question: For external dependencies, will CMake compile and install these libraries or is the user expected to still install these? My current guess, given it is the same approach we use for the Bazel builds a user will not need to install them.

Simplify configuration
We should introduce a single configuration option GOOGLE_CLOUD_CPP_DEPENDENCY_PROVIDER that sets the source for all the modules.

Will this environment variable act similar to LD_LIBRARY_PATH? If not, could add an example?

Alternatives Considered
Only Support one of the configurations: supporting find_package() is good hygiene, it is a necessary configuration to support any package manager, and it is expected that some of our users will require it.  Supporting ExternalProject makes it easier to start using the library (nothing to install before you try it), it makes it easier to develop, and makes the CMake and Bazel build more homogenous.

w.r.t this alternative, does the following line act as a pro when choosing this option or comparison with the proposed?

Supporting ExternalProject makes it easier to start using the library (nothing to install before you try it), it makes it easier to develop, and makes the CMake and Bazel build more homogenous.

Question: For external dependencies, will CMake compile and install these libraries or is the user expected to still install these?

CMake will download and compile the libraries. They are not "installed", at least not in the sense that they are on a well-known location (such as /usr/local).

Will this environment variable act similar to LD_LIBRARY_PATH? If not, could add an example?

It is a configuration option for CMake, not an environment variable.  You would use it like any other configuration option:
cmake -H. -Bbuild-output -DGOOGLE_CLOUD_CPP_DEPENDENCY_PROVIDER=package

w.r.t this alternative, does the following line act as a pro when choosing this option or comparison with the proposed?

Supporting ExternalProject makes it easier to start using the library (nothing to install before you try it), it makes it easier to develop, and makes the CMake and Bazel build more homogenous.

Yes, that is a pro.

Thanks for clarifying @coryan.
I like alternative Only Support one of the configurations, but it reads more as a follow-up to having package and external which can be used to eventually unify the two.
Either sounds good to me.

Ah, okay! Then the proposed solution of using package and external sgtm! Thanks for further clarifying!

@coryan Would a vcpkg integration style that acts like ExternalProject be useful to you?

@ras0219-msft Maybe?
I prefer to minimize the number of tools my customers have to install before they try out the library.
With the current proposal they can (in the vast majority of cases) just say cmake -H. -Bfoo && cmake --build foo and everything gets downloaded and compiled for them.
Do you think we could have a mode where the user just types cmake -H. -Bfoo && cmake --build foo and that magically downloads vcpkg, compiles vcpkg, compiles all the other dependencies using vcpkg, and then compiles the libraries?  That would be truly awesome because I would have to maintain less code.

Yes, that second case is what I meant :)

@ras0219-msft that level of magic sounds really cool, yes we would definitely be interested in it.


Can esp not show teammates? Show only enemies

Can esp not show teammates? Show only enemies

It is able to show team mates, simply untuck "Exclude Team" in the Misc section.

My question is, I'm not careful enough


Incorrect response in API on error

API endpoints that fail with external errors seem to convert them to 500 Internal Error instead.

Fixed by #154


(MAINT) Use the upgrade install option for rpm packages

Images used in beaker tests might have all kinds of preinstalled
packages on them, some of which may be out of date. In our case, we
were preinstalling the puppetlabs el 7.10 release package on images,
but then running beaker's install_puppet function as a provisional step
in our tests. Since a new release package was added, this started
failing[1]. We were able to fix this easily by fixing our images, but
users depending on externally hosted images, such as vagrant boxes, may
not be so lucky.
This patch changes the -i flag of the rpm install command to -U so that
the package will be either installed or, if already present but out of
date, upgraded. This will not change functionality for users with clean
images who do not need to upgrade anything. The --replacepkgs flag that
was already present in the rpm command is not sufficient to remove
conflicting files from older versions of packages.
[1] http://logs.openstack.org/77/308277/2/check/gate-openstackci-beaker-centos-7/0736982/console.html#_2016-04-20_14_14_12_690

+1

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2401/
Test PASSed.

Thanks for pushing this up to us @cmurphy!
Once that's done, we'll do one last run through CI, but that should be just a formality, and we look forward to merging this soon!

@kevpl done!

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2405/
Test PASSed.


Permission denied (publickey)

Gitea version (or commit ref): 1.2.0+rc2
Git version: 1.9.1
Operating system: Ubuntu 14LTS
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Description
Hello,
I'm not sure whether this is an issue or just a support request. I have a problem accessing my Gitea sources using SSH.
On my Mac, I have my own user with my SSH key pair. My public key is set on my Gitea account.
From my Mac, I may git clone, no problem at all. But when I login via SSH to a distant server (to my Vagrant, to any of my distant dedicated servers, ...), I cannot git pull any longer.
However the SSH identity forwarding is enabled for all these servers and it's working, I have checked doing a: echo "$SSH_AUTH_SOCK".
The problem is that it used to work prior to upgrading Gitea to 1.2.0+rc2. I really don't understand why this should have any impact on an SSH authentication mechanism but I thought it'd be best to talk to you guys about this. The point is that I find this confusing that it occurs on all distant servers (not only one). I don't think this is a server related configuration issue as I didn't touch anything there.
I could not find anything interesting in the logs. Actually, I even did a multitail *.log on Gitea, I tried to git pull but no line, not even one, was appended to the output.
I know what you will answer to me, you will ask me to downgrade but I'm not very confident with downgrading on a PROD application.
If you have any idea...
Thanks
Screenshots

FYI, I don't use Gitea's builtin SSH:
; Whether use builtin SSH server or not.
START_SSH_SERVER       = false

Also, in /var/log/auth.log, here is what I get when I git pull:
Sep 20 10:23:11 new sshd[18985]: Connection closed by 194.3.XXX.XXX [preauth]
I tried to regenerate the .ssh/authorized_keys from Gitea but that didn't fix.

Hello,
Still stuck on this one. Has anyone got the chance to look at this?
Thanks

ssh -vvv<EMAIL_ADDRESS>maybe you can debug the ssh

Closing. Please re-open if needed.

I have the same problem when I change my data to a new server. when I pull the repo via ssh, what i get are always the Connection closed by authenticating user git x.x.x.x port 61164 [preauth], i tried many ways to solve this like update /etc/ssh/sshd_config file, nothing worked.
Finally i think this is may be the wrong .ssh dir permission, i change it to 700 and it works! help this can help to someone:-)


Add list preferred deps for all TPLs to release notes

Describe what needs to be documented.
Add to release notes HTML, and the template
Is the documentation missing?
Helps people know how to avoid the combinatorial jungle.

We could probably have build_visit generate the html for this section of the relnotes. That might be better than expecting developers to keep yet another two files in sync.

resolved in #19444


Still maintain?

Hi there! Do you still maintain this gem?

No, I don't actively develop this gem. You're welcome to make pull requests.


[Model] American Psycho

this textures seem to contain the correct textures for American Psycho Patrick Batemen but the name of the package is wrong and uses the name of a Starwars character Obiwan. this will cause problems.

URL: https://unrealarchive.org/models/unreal-tournament/A/american-psycho_01b8fa3a.html
Hash: 01b8fa3a763b95f6b09d7d8745fe43b99c9781cc
Current name: American Psycho

Linked technical issue.
Thanks :tada:


Three missing features.

1.- I do not know exactly how this works, but in the video the Kyetho team wins by default, I imagine that event coordinators did something https://www.youtube.com/watch?v=4QsITCYvqc0&t=381
2.- In this video the leader type /ready and starts the countdown https://www.youtube.com/watch?v=UiNR1n80H1Q&t=498
3.- On this video he type /subbed and the player which has been replaced has been kicked https://www.youtube.com/watch?v=UiNR1n80H1Q&t=564

The sub command is pretty old and is not really needed. /ready is already implemented

1 is completely irrelevant in Cardinal, 2 is implemented, 3 just use /kick


fix: improve the color contrast of pagination links (resolves #397)

[X] This pull request has been linted by running npm run lint without errors
[X] This pull request has been tested by running npm run start and reviewing affected routes
[X] This pull request has been built by running npm run build without errors
[X] This isn't a duplicate of an existing pull request

Description
Adjust the color of pagination links according to the figma design.
Steps to test

Check the report of "lighthouse CI - Mobile" generated by the lighthouse CI pull request;
Accessibility score of a few pagination links is dropped from 100 to 98.
Run lighthouse - Mobile report with pages that have problems with the fix.

Expected behavior:
Accessibility scores should be back to 100.

@cindyli this is interesting I wonder why only those two pages failed

@cindyli this is interesting I wonder why only those two pages failed

In the list of URLs tested by the lighthouse CI, only these 2 pages have pagination.


⚠ Slack has degraded performance

In c4f3ed8, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 45 ms

Resolved: Slack performance has improved in aa09869 after 7 hours, 22 minutes.


Support hysteria v1.2.1

Support hysteria v1.2.1

I am not particularly familiar with GoLang, so please confirm whether the part about CA certificate is correct.

After I added this code, the option to configure CA certificate or use AC certificate string is no problem.
I don't know what the fingerprint option is and haven't tested it.

what is hysteria and why this PR was not merged?

@hadifarnoud The changelog of hysteria 1.2.1 does not contain any protocol updates


cumul_add -- Decryption Failure

Alberto (@ibarrond), I have merged the previous implementation with your suggested optimal solution. After checking intermediate results, I noticed one huge difference between encrypted parameter and original one:
enc_col_sum_msr = np.array([HE.cumul_add(c, in_new_ctxt=True) for c in enc_square_residue]) VS
col_sum_msr = np.sum(square_residue, axis=0)
where
enc_residue = ciphertext - row_mean - col_mean + data_mean (the same way as you recommend, encrypting one column per ciphertext)
enc_square_residue = enc_residue ** 2
for i in range(len(ciphertext)):
   HE.rescale_to_next(cipher_square_residue[i])
   HE.relinearize(cipher_square_residue[i])
size = 2/3 & scale = 30

I believe the HE.cumul_add has to be fixed but as it does not raise an error, it's now difficult for me to understand what I did wrongly. (add that I didnt find any similar issue on Pyfhel, if you want I can open the discussion there).

Add that, I've just made a mistake in getting mean for columns and then finding a residue. Btw, it needs an integration test to see how encryption are being done.

HE.cumul_add performs as many rotations and additions to accumulate all the slots inside a ciphertext. Previously you were using a single ciphertext per data point, whereas now you encrypt an entire vector inside each ciphertext, hence there is no alternative. How different are the results?

ckks_params = {
    'scheme': 'CKKS',
    'n': 2 ** 15,
    'scale': 2 ** 30,
    'qi_sizes': [60] + 15 * [30] + [60]
}

n_elements = n_rows*n_cols
np.random.seed(42) # Fixed seed for reproducibility
data = np.random.randint(0, 800, size=(n_rows, n_cols))

#NON-ENCRYPTED
msr =        np.sum((data * n_elements
                    - data.sum(axis=0) * n_cols\
                    - np.expand_dims(data.sum(axis=1), axis=1) * n_rows
                    + data.sum())**2) / (n_elements **3)

row_msr = np.sum((data * n_elements
                    - data.sum(axis=0) * n_cols\
                    - np.expand_dims(data.sum(axis=1), axis=1) * n_rows
                    + data.sum())**2, axis=1) / ((n_elements **2) * n_cols)

col_msr = np.sum((data * n_elements
                    - data.sum(axis=0) * n_cols\
                    - np.expand_dims(data.sum(axis=1), axis=1) * n_rows
                    + data.sum())**2, axis=0) / ((n_elements **2) * n_rows)

#ENCRYPTED
# encrypt data --> One column per ciphertext
c_data = np.array([HE.encrypt(data[:,i]) for i in range(n_cols)])

# sum for each row --> sum all ciphertexts
c_row_sum = sum(c_data)

# sum for each column --> sum all elements on each ciphertext
c_col_sum = np.array([HE.cumul_add(c, in_new_ctxt=True) for c in c_data])

# sum for all elements --> sum all row/col sums
c_all_sum = c_col_sum.sum()

# square residues
sq_res = (c_data * n_elements - c_row_sum * n_rows - c_col_sum * n_cols + c_all_sum)**2
for i in range(len(sq_res)):
    HE.rescale_to_next(sq_res[i])
    HE.relinearize(sq_res[i])

# row_msr
c_msr_row_p1 = sum(sq_res) / n_cols
c_msr_row_p2 = c_msr_row_p1 / n_elements
c_msr_row = c_msr_row_p2 / n_elements
HE.rescale_to_next(c_msr_row)
dec_row_msr = decr_round(c_msr_row)[:n_rows]

# col_msr
c_msr_sum = np.array([HE.cumul_add(c, in_new_ctxt=True) for c in sq_res])
c_msr_col_p1 = np.array([HE.cumul_add(c, in_new_ctxt=True) for c in sq_res]) / [n_rows]
c_msr_col_p2 = c_msr_col_p1 / [n_elements]
c_msr_col = c_msr_col_p2 / [n_elements]
for i in range(len(c_msr_col)):
    HE.rescale_to_next(c_msr_col[i])
dec_col_msr = (decr_cols(c_msr_col)[0][:n_cols]).T

# msr
c_msr_p1 = c_msr_sum.sum() / (n_elements)
c_msr_p2 = c_msr_p1 / (n_elements)
c_msr = c_msr_p2 / (n_elements)
HE.rescale_to_next(c_msr)
dec_msr = decr_round(c_msr)[0]

n_rows, n_cols = 50, 5

np.allclose(dec_row_msr,row_msr, tol): True - dec_row_msr: [ 21017.14  67883.6   12920.7  ...  47110.41  42149.32  17044.25] - row_msr: [20880.18 67441.25 12836.5   ...  46803.42 41874.66 16933.17]
np.allclose(dec_col_msr,col_msr, tol): False - **dec_col_msr**: [431133.11 347502.12 904697.82  54864.63 135273.35] - col_msr: [47427.35 43211.43 37276.83 50020.02 35328.45]
np.allclose(dec_msr,msr, tol): False - **dec_msr**: 374694.19 - msr: 42652.81

n_rows, n_cols = 2884, 17

np.allclose(dec_row_msr,row_msr, tol): True dec_row_msr:[46861.3  42879.17 54115.96 ... 59788.33 59586.04 49103.53] - row_msr: [46554.16 42598.13 53761.27 ... 59396.47 59195.5  48781.69]
np.allclose(dec_col_msr,col_msr, tol): False - dec_col_msr: [50514.51 48903.68 51049.25 ... 51865.84 49498.85 52001.38] - col_msr: [50181.61 48582.06 50535.79 ... 51392.22 49138.81 51587.52]
np.allclose(dec_msr,msr, tol): True - dec_msr: 50529.9 - msr: 50137.27

I went through different test scenarios, and what I suppose is happening could be related to the size of the data and the number of ciphertext slots (i.e. the smaller matrix the lower accuracy). Once I thought it is doing correctly but the accuracy reaches its minimum (0.03 out of 1) as our large data set is significantly changing its size during the process.


Smelting items too large for essentia smeltery

Not necessarily a bug, but since documentation is slim on TC6, not sure where to look.
The issue I have is the tank size for the essentia smeltery.  By design, if the tank doesn't have enough room, the next item won't be smelted.  I'm not sure the size of the tank, but Cyclic mod's 'Automatic Torch' has a total of 411 essentia assigned to it (201 Potentia, 201 Ignis and 11 Metallum), which is presumably well over what the tank can hold (I'd guess maybe 200 or so)
Is there an alternative to smelting this down so I can get something useful from it?  Or is it just trash for me?

You could make a huge batch of something in the crucible.
Just make sure to math out all the ingredients and have a bunch of quartz slivers on hand.

Modded items factor in essentia based on their ingredients unless they have been defined by Thaumcraft. That's probably where the ridiculous amount of essentia points is coming from (I assume it's crafted with a lot of coal? Maybe more than just a lot?). I'd say avoid it if it's too much. If its essentia is derived from the recipe, then you should be able to just use what you used to craft it to obtain what you need. You could use it to intentionally make flux in crucibles, though- if you want to get primordial pearls from rifts and whatnot or learn how to craft some flux controlling devices.

Yeah unfortunately you won't be able to smelt down items with that much essentia. The limit in the smeltery exists for a reason.


Fix Nginx websocket config: proxy_pass is not inherited

Whoops! Hit "create pull request" a bit too fast. The proxy_pass directive is not inherited in child location blocks and should be specified again.

nginx config seems tricky :) It seems like we can remove the need for reverse proxies in Plausible if we enable TLS and cert management there.

Thank you!

nginx config seems tricky :) It also seems like we can remove the need for reverse proxies in Plausible if we enable TLS and cert management there.

True and true. However for the second point: the fact that putting services behind reverse-proxies like Nginx instead of exposing whatever other web server directly is pretty much industry standard is exactly because they are known to have a sane and well established TLS stack implementation, standard well-known architecture/configuration, good performance, etc. It also makes cert management easier (i.e., you update all your certs on the reverse proxy and don't have to worry about updating each cert for each service you host in a separately). Additionally, most people are already using reverse proxies regardless if they host multiple things on the same machine.
Not sure what you are going to use internally for that, but definitely take the above into consideration.
Just my two cents, I am definitely not an expert!
Keep up the good work :smile:


Splay Sort Algorithm

Description of Change

#2872 resolved
Checklist

[✓] Added description of change
[✓] Added file name matches File name guidelines
[✓] Added tests and example, test must pass
[✓] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[✓] Relevant documentation/comments is changed or added
[✓] PR title follows semantic commit guidelines
[✓] Search previous suggestions before making a new one, as yours may be a duplicate.
[✓] I acknowledge that all my contributions will be made under the project's license.

Notes:
I have added Splay Sort Algorithm in cpp please merge it.

Please merger this pr and give me hacktoberfest tag


leaderboard hanging link

The leaderboard doesn't work.
It just keeps on loading.
There is no link to store the leaderboard entries. I am a beginner in js, so don't know if it's being handled there.

still an issue

maybe someone could make another leaderboard, comment it, then the ones who want to implement it could use it and we would be happy. (i think i cant do one, im very new to js, but i will try) :D


Onboard OpenJS projects to LFX Security Dashboard

I believe @lukeschantz is going to work on this.

What does https://lfx.linuxfoundation.org/tools/security/ adds to Dependabot and similar tools? Will this require any action from the maintainers?


allow gzip for API calls

The documentation recommends using gzip for API calls, but it doesn't look like the php-client is doing this. I thought maybe the Google_Service or Google_Service_Resource classes would be able to customize the headers of the Google_Http_Request object they ultimately call, but there didn't seem to be a way to do that.
Thanks for listening!

Hi, i just found this issue and have been taking a closer look at the packages.
It seems the functionality to add the GZIP header has been removed.
Is there still a way to add it manualy?

You should be able to enable gzip encoding by providing a custom HTTP client instance:
use GuzzleHttp\Client;

$google = new Google_Client;

$google->setHttpClient(new Client([
  'headers' => ['Accept-Encoding' => 'deflate, gzip']
]));

In my tests, this will result in gzipped responses.

Thanks!!


Update Issues

Update pop up causes MMW to crash without internet.

Should be fixed with commit b5cce52.


[DNM] Display Fat & Protein on Apple Watch

Added Nutrients screen where a user can log Carbs, Protein, and Fat from Apple Watch.
Notes

The new screen is disabled by default
Picker step can be toggled between 5 and 1 via double tap for more precise numbers.

iPhone
Apple Watch

There already was a WCSession for sending carbs to phone. Why not make another one for fat and protein also?

There already was a WCSession for sending carbs to phone. Why not make another one for fat and protein also? I think that would be the easiest way forward.

The handlers watchStateModel.addCarbs and the WatchManager's message["carbs"] session listener have their own logic after submitting and receiving carbs. In order to keep them unchanges I just created a separate flow.
Otherwise, we couldn't just send two messages because of apsManager.determineBasal() at the end of each. Same for error handling when encoding / decoding the message.

using it since 2 weeks, works fine - great job

Closing in favor of the current implementation which is in the main branch already.


update NFD operator OLM catalog

Signed-off-by: Carlos Eduardo Arango Gutierrez<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

Pleas change directory name to be same as operator version in csv (without 'v'). For example 4.6 to 4.6.0

Pleas change directory name to be same as operator version in csv (without 'v'). For example 4.6 to 4.6.0

@mvalarh @J0zi requested changes addressed
Ready for a second review

@mvalarh @J0zi requested changes addressed
Ready for a second review

@ArangoGutierrez from lemon test you can see that you don't have replases in v4.6.0 which should point to v4.5.0 or put as another channel in package file for v4.5.0

@ArangoGutierrez from lemon test you can see that you don't have replases in v4.6.0 which should point to v4.5.0 or put as another channel in package file for v4.5.0

/ok-to-test

/ok-to-test

@ArangoGutierrez from lemon test you can see that you don't have replases in v4.6.0 which should point to v4.5.0 or put as another channel in package file for v4.5.0

Test are happy now, thanks for your tips

@ArangoGutierrez from lemon test you can see that you don't have replases in v4.6.0 which should point to v4.5.0 or put as another channel in package file for v4.5.0

Test are happy now, thanks for your tips

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/retest

/retest

/retest

/retest

/retest

/retest

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/retest

/retest

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

/retest

/retest

@mvalarh  ci is failing with Evicted The node was low on resource: memory I don't know how to address that

@mvalarh  ci is failing with Evicted The node was low on resource: memory I don't know how to address that

@J0zi Can you please have a look?

@J0zi Can you please have a look?

/retest

/retest

PING
/retest

PING
/retest

PING @J0zi
/retest

PING @J0zi
/retest

PING @J0zi
/retest

PING @J0zi
/retest

@ArangoGutierrez, please squash your commits. It will help.

@ArangoGutierrez, please squash your commits. It will help.

@ArangoGutierrez, please squash your commits. It will help.

Rebased and Squashed! thanks @J0zi

@ArangoGutierrez, please squash your commits. It will help.

Rebased and Squashed! thanks @J0zi

ha!  fixed!!!!
@J0zi  @mvalahtv  ready for review

ha!  fixed!!!!
@J0zi  @mvalahtv  ready for review


Error opening .vsd files with diagram.io

Attempting to open older MS 2k10 vsd files and receive the following error.
Error loading file
Unknown error
VAs zu P 06.zip

found by accident that renaming the extension from .VSD to .vsd fixed the issue.


Added support for SMAP-E-OPL data

LVT reader for SMAP-E-OPL
This PR adds support to read new SMAP retrieval product (SMAP-E-OPL).
Resolves #1327
Testcase
/discover/nobackup/projects/usaf_lis/mnavari/AF/SMAP_E_OPL/analysis_mn/SMAP_E_OPL_vs_NRT_PR_1328

@LIS-navari I was able to compile your updates with intel and gnu compilers.  I was also able to run your testcase and replicate your output.  I will perform a final code review.
Before accepting, the following items will need attention:

Copywrite notices at the top of your new SMAP datastream files need to be updated.
The new SMAP retrieval product needs to be added as an "analysis data source" in the lvt.config.adoc file.

Thanks!

Hi @cmclaug2
Could you please elaborate on the following
"Copywrite notices at the top of your new SMAP datastream files need to be updated."
Thanks

@LIS-navari,
Please update the copywrite year in ./readSMAPEOPL_SMObs.F90 and ./SMAPEOPL_SM_obsMod.F90 to 2022.

@LIS-navari,
I made a minor edit to the config file and fixed my own typo. This PR is completed and ready to merge.  Thank you.


fix: merge qna settings from generated files in js runtime

Description
Merge settings from generated settings files (e.g. QnA, Luis settings) in js runtime.
Task Item
#minor
Screenshots

Coverage remained the same at 56.747% when pulling e6c98e7284398d4ef19a37cc497c77d77c2fb19c on zim/merge-settings into 8cb80f989ca84d8caccde257bb3651030c6ece43 on main.


Can I change id key mapping?

Hi, I am new to type-graphql-dataloader and I am trying to build a POC to evaluate using it with Apollo and MongoDB.
As you might now MongoDB uses  identity keys named _id, while Dataloader expects identities stored in properties named id.
Is there any way to change this setting on the Dataloader, so to expect identities stored in properties called _id?
I imagine that there are other approaches for solving this problem, but if such a setting exist it would seem clean to me.
Thank you for your work!

Though I am not sure what you are trying to do, dataloader is ignorant to properties of cached objects, therefore having id nor _id should not matter. @Loader will work with whatever DB you are using (e.g. https://github.com/slaypni/type-graphql-dataloader#with-custom-dataloader).

Thanks for explaining, I am looking exactly at that example:
  @FieldResolver()
  @Loader<number, Photo[]>(async (ids, { context }) => {  // batchLoadFn
    const photos = await getRepository(Photo).find({
      where: { user: { id: In([...ids]) } },
    });
    const photosById = groupBy(photos, "userId");
    return ids.map((id) => photosById[id] ?? []);
  })
  photos(@Root() root: User) {
    return (dataloader: DataLoader<number, Photo[]>) =>
      dataloader.load(root.id);
  }

How does dataloader.load(root.id) know how to pick the right object from the loader which is returning an array of Photo objects? Is it just because of the order in the array?

Watching this video may solve your question: https://www.youtube.com/watch?v=ld2_AS4l19g
Would you take a look it?

Thanks for pointing out that video. It answered exactly my question!


Improve P-value map and check linear modeling normality

There is a simply way you can approach that try to make a small rectangle to mark our significant sub-area in red
Here is link should be useful
http://matplotlib.org/1.5.0/examples/shapes_and_collections/artist_reference.html

Trying to do a basic normality test like QQ-plot and we see how the result looks like

Thanks for the link!

The biggest idea is producing out the result which is simply and clear because the big chunk grade is everything is worked out!
Thanks for your work

https://piazza.com/class/id3duvdb9wt1ge?cid=156

It is done so far!


feat: direct link route report updates

PR summary

Update to include new fields in DL Route Report
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] 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)

PR Type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

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


Update my packages

Motivation for this change
This updates a list of packages I maintain.
I test-build all of them on my machine.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@pSub I guess this can be merged now, right?
@GrahamcOfBorg build pipes cli-visualizer wordgrinder fltrdr


lib.updateName: use a different mechanism

I'm merging in a few days if there are no objections.
I found it slightly confusing that the paths of packages like bashInteractive didn't reflect the name seen by nix-env.  I really prefer to keep these names equal.
Now appendToName changes also the name-part in $out and drv paths. Name changes will cause a rebuild now, but it's a custom to use it for differently configured variants, i.e. the rebuild would happen anyway.

I don't think this is a good idea. overrideDerivation is an ugly, inefficient hack that we shouldn't rely on in Nixpkgs itself. (It's fine for ad-hoc modifications in ~/.nixpkgs/config.nix.)
The clean solution is to compute the correct name in the Nix expression for the package, e.g.
mkDerivation {
  name = "bash-" + optionalString interactive "interactive-" + ...;
}

Also, I don't think it's that confusing. The goal is to disambiguate names for nix-env, which it does.

From programming point of view, the following pattern seems better to me, and it is used in nixpkgs on quite a few places:
foo-var2 = appendToName "-var2" (foo.override { /* some feature specs */ });

It also has more "functional style" feeling for me. Currently I see no suitable implementation, though, if overrideDerivation is better avoided.

(triage) can be closed?

We should rather use a fixed point to do such modifications instead of overriding derivation attributes once they've been computed.

@vcunat any objections closing this?

I don't understand what you mean about "fixed point", but in any case I see no use to keep the PR open.


The MediaPlaylist segment's url is relative path, not full uri

flutter doctor:
[✓] Flutter (Channel stable, 3.0.5, on Microsoft Windows [Version 10.0.19044.1766], locale zh-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[✗] Chrome - develop for the web (Cannot find Chrome executable at .\Google\Chrome\Application\chrome.exe)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.
[✓] Visual Studio - develop for Windows (Visual Studio Community 2022 17.1.1)
[✓] Android Studio (version 2021.2)
[✓] IntelliJ IDEA Ultimate Edition (version 2022.1)
[✓] VS Code (version 1.66.2)
[!] Proxy Configuration
    ! NO_PROXY is not set
[✓] Connected device (3 available)
[✓] HTTP Host Availability

flutter_hls_parser: "2.0.0"
The MasterPlaylist from https://boba.ettoday.net/video/307/910/334742
And the content is:
#EXTM3U
#EXT-X-VERSION:3
#EXT-X-STREAM-INF:BANDWIDTH=2500000,RESOLUTION=1920x1080
chunklist_b2500000.m3u8?token=D0aMMSFEGGcH7vd2X4QLSQ&expires=1657986292
#EXT-X-STREAM-INF:BANDWIDTH=1500000,RESOLUTION=1280x720
chunklist_b1500000.m3u8?token=D0aMMSFEGGcH7vd2X4QLSQ&expires=1657986292
#EXT-X-STREAM-INF:BANDWIDTH=800000,RESOLUTION=854x480
chunklist_b800000.m3u8?token=D0aMMSFEGGcH7vd2X4QLSQ&expires=1657986292

The MediaPlaylist content:
#EXTM3U
#EXT-X-VERSION:3
#EXT-X-TARGETDURATION:12
#EXT-X-MEDIA-SEQUENCE:0
#EXTINF:10.0,
media_b2500000_0.ts
#EXTINF:10.0,
media_b2500000_1.ts
#EXTINF:10.0,
media_b2500000_2.ts
...
...
...
#EXTINF:10.0,
media_b2500000_415.ts
#EXTINF:6.662,
media_b2500000_416.ts
#EXT-X-ENDLIST

My code:
  Future<void> parseHls(Uri url, [HlsMasterPlaylist? masterPlaylist]) async {
    HlsPlaylist playlist;
    print('parseHls $url\n'
        '${masterPlaylist == null ? 'Not provided' : 'Provided'} master playlist');
    final data = await dio.getUri(url).then((res) => res.data);
    playlist = await HlsPlaylistParser.create(masterPlaylist: masterPlaylist)
        .parseString(url, data);
    if (playlist is HlsMasterPlaylist) {
      // master m3u8 file
      playlist.variants
        ..sort((a, b) => b.format.bitrate! - a.format.bitrate!)
        ..forEach((v) {
          log('Variant  resolution: ${v.format.width} x ${v.format.height}, bitrate: ${v.format.bitrate}\n${v.url}');
        });
        return parseHls(playlist.variants.first.url, playlist);
    } else if (playlist is HlsMediaPlaylist) {
      // media m3u8 file
      print(playlist.segments.first.url);
      // output media_b2500000_0.ts
    }
  }

thank you for your feedback.
This is expected behavior.
All parsed url variant follows original one.
If original is absolute path, the parsed one is absolute.
The same is true for relative one.


Support loading TIFF subIFD's

What are you trying to achieve?
I'm trying to load a DNG, and DNG's are saved in parts in TIFF subIFD's. Regardless, subIFD is part of the TIFF format, and there doesn't seem to be a way to specify a subIFD when loading a file/buffer, nor is there a way to know how many subIFD's a TIFF has.
Libvips appears to support this.
Also please note that subIFD is not the same as pages. The file I'm trying to read has 1 page, so the pages feature doesn't resolve this.
Also, I know very well that DNG isn't supported, but the issue is more general than that - subIFD is a TIFF standard, and DNG just happens to take advantage of it. Supporting this TIFF feature might be a step towards being able to read DNG with sharp - or at least decode the bits I need at a fairly awesome speed.
Have you searched for similar feature requests?
The word subIFD doesn't occur anywhere in this entire repo (well, it does now, but I mean before creating this issue).
What would you expect the API to look like?
Could be a property subIFDs: Array<Buffer> in the metadata() object, each of which can be passed back into a sharp() constructor. This would be in line with other metadata, which are also returned as binary blobs.
Another option would be to simply add subIFDs: number just to know how many there are. Then you could pass a subIFD index to the constructor like sharp('foo.dng', { subIFD: 2 }). This would be somewhat in line with the libvips API.
What alternatives have you considered?
A custom TIFF reader, but it's very slow and clunky. I'm sure lipvips can do much better.
Is there a sample image that helps explain?
Any ol' DNG will do, but I'll be happy to supply one, if need be.

Hello, yes, libvips supports this feature so adding it to sharp should be fairly straightforward.
We'll need to expose the subifd property as an option on sharp's constructor and ensure it is set when loading TIFF images, in very much the same way that level is exposed for OpenSlide approximately here:
https://github.com/lovell/sharp/blob/0dc325daa4a5b15bdbd1d6ff2375e0d18b4380c7/src/common.cc#L317
Happy to accept a PR for this, if you're able.

It's just like the n-pages property, so you should be able to copy-paste the code for that:
$ vips copy k2.jpg x.tif[pyramid,subifd]
$ vipsheader -a x.tif
x.tif: 1450x2048 uchar, 3 bands, srgb, tiffload
width: 1450
height: 2048
bands: 3
format: uchar
coding: none
interpretation: srgb
xoffset: 0
yoffset: 0
xres: 2.835
yres: 2.835
filename: x.tif
vips-loader: tiffload
n-subifds: 4
n-pages: 1
resolution-unit: in
orientation: 1

ie. five layers in this pyramid, with the main image as layer 0 and four subifds.

Thanks John. For anyone looking to expose this subifds metadata in sharp, a good place to start is to follow the trail of existing code for the pages property returned by metadata, approximately here:
https://github.com/lovell/sharp/blob/0dc325daa4a5b15bdbd1d6ff2375e0d18b4380c7/src/metadata.cc#L62

Commit https://github.com/lovell/sharp/commit/43a085d1ae126016bd7c9be74881983554214394 adds subifds to the metadata response and accepts an optional subifd via the constructor.
https://sharp.pixelplumbing.com/api-input#metadata
https://sharp.pixelplumbing.com/api-constructor#parameters
This is available in v0.28.1.


Publishing to Ruby gems.

Any idea when you might be publishing to ruby gems.
We're going to be developing and depending on this i believe, so in the mean time we've forked to make sure we can anchor for releases and we'll just pull from the upstream.
Having a formal release will make this easier and we're less likely to need our fork.

@j-mcnally Did you end up finding a work-around? I'm having issues and I'm too new to this to figure out where it is.
Initialization keeps failing due to #<NoMethodError: undefined method configure' for Clover:Module>`
I wonder if forking it might just do the fix...

We just rolled our own entire integration. Sorry.


Drop rafter reconciler from control-plane configuration

Description
Changes proposed in this pull request:

drop rafter reconciler from configuration
drop grafana dashboard for rafter reconciler
etc

Related issue(s)
https://github.com/kyma-project/control-plane/issues/1798

/test all

/test pre-main-control-plane-reconciler-upgrade-kyma2-latest-to-main-gardener

LGTM


Provide a form on the UI to get peer review on vulnerability advisory quality

hi , i want to work on this issue. could you please tell how to proceed?


[9.x] Fix setUp and tearDown method visibility and casing.

This PR fix setUp and tearDown method visibility and casing.

Thanks!


Remove legacy support and default to *.hlx.page

The sidekick extension no longer has to deal with Helix 2 configs.

:tada: This issue has been resolved in version 5.3.0 :tada:
The release is available on:

GitHub release
v5.3.0

Your semantic-release bot :package::rocket:


Ethereum API

I there an API we can use like the web3 object provided by Metamask or Mist ?
I would love to use my Nano S with my apps.
Thanks !

We'll have a JS API soon - for the time being you can directly use the application commands documented in https://github.com/LedgerHQ/blue-app-eth/tree/master/doc - there are a few Python samples at https://github.com/LedgerHQ/blue-app-eth
You can subscribe to our developer Slack at http://slack.ledger.co to be notified when it's out and discuss the API features.


Consider changing build systems

Ok...
I still remember an issue where you complained that CMake's language never works like you'd expect. I hate having to download a 20-dozen-file program and building it just to use AsmJit.
It also seems like the CMake script has LOTS of boilerplate (for instance, the Asmjit_Add_* macros).
So...I was wondering if you'd look into using another build system such as fbuild, Boost.Build, Waf, Gyp, or maybe even shudder SCons shudder.
Thoughts? I have experience using all of the above except SCons (and somewhat Gyp), so I could help with whatever you need.
As a point of interest, Boost.Build has built-in support for selecting between a static and a shared library.

Well, I have designed asmjit in a way that no build system is really needed and people that don't like cmake can just include files in their project(s) and it will magically work.
The reason I'm using cmake is that it is somehow able to generate project files for all platforms that I use and it's still possible to use it in a way I do; for example I can embed asmjit in another project without even building a static library - I don't think that this is supported in other build systems as there is always a "product".
I would not use waf, gyp or scons as I have always experienced problems with these, and I don't really know the others you mentioned. I mean cmake just seems to be a lesser evil at the moment, but if there is a new build system, that is not using python, can generate project files, and is really cross-platform and solves all things in a much more elegant way cmake does, I would be really interested ;)

Boost.Build does NOT generate project files, but it doesn't use Python and is VERY much cross-platform, and it's really flexible (albeit poorly documented). It's primarily used to build Boost, which maked it obvious that it has strong cross-platform support.
Fbuild uses Python, but it's also truly cross-platform and is easily my favorite build tool.
Tundra doesn't generate project files, either, but it doesn't use Python. I'm not quite sure too much else about this tool, but it seems very nice.
Premake looks like it might be your best bet. It generates project files for lots of IDEs and is very elegant.

Well, I think that cmake is simply the best from these, it's really widely supported and devs usually have it installed anyway. Premake was here before cmake and didn't make it, and I have never heard of Tundra for example.
I mean unless there is a really good reason I would not switch. I have several projects using cmake and although I don't like its syntax at all I found it to be the tool that actually works and if there is a problem I'm probably not the first one facing it.
I think the main problem is that I'm not really a cmake expert and I'm making some things more complicated than they are, so the script itself is probably much more bloated than it should be :)

I guess that makes sense...
It just bugs me that CMake is huge, takes forever to build, and generates the longest, slowest makefiles on the planet (4000+ lines for a 1-source, 1-executable program) which for some stupid reason call back into CMake, along with the fact that the CodeBlocks generator just reverts back to make and doesn't set up executables correctly. The only non-suckish part is the Ninja generator.
That is also the longest sentence I have ever written...
I was using CMake for a project that uses AsmJit. Then it irritated me, so I hopped between 20 build systems before using fbuild. Since I didn't want Python to be a dependency, I added a Boost.Build script as an option. Both were very short and elegant and made me much, much happier. ;)

Well, I didn't really study the output, but yeah, that's a bit crazy.
I would even consider adding more build systems so people can pick what they consider the best, but I'm just afraid that I will be fixing build scripts instead of coding in such case :)

BTW I would also like to have a build system that works on arm, so maybe I will switch into something else in the future if cmake won't support it

What exactly do you mean by "natively"? Nothing really runs on Android itself except ports of GNU make.
I know that Boost.Build works on Android with very minor tweaks that require nothing other than the NDK itself (the example in the link looks daunting, but it's really easier than it looks).
There are also CMake scripts for Android, but it is an extra dependency.
Of course, if you wanted real Android support, you could torture yourself AND AsmJit's users by using Gradle. :)

@kobalicek So...I was toying around and experimenting...and guess what? A fully-functional Boost.Build build system for is just 56 lines long! Gave me no trouble whatsoever. It's here. The only cheat was that I used globs to shorten the file. Even without the globs, it's over 200 lines shorter than the CMake build script and it actually a bit more portable.
It's also a tad easier to read. ;)

I meant to compile asmjit on the Android device itself, this is currently not supported by cmake and I'm not sure how about the other build systems, but not in a rust with this as there are odroids that I can use for ARM development ;)
And yeah, the build looks cool, but I'm still not convinced to switch, maybe I can add it as an alternative, but still I don't really feel like maintaining two build scripts if one already works :)

@kobalicek
Boost.Build theoretically should be able to be built to run on Android, assuming someone patches the Boehm configure script to make it pick up the NDK correctly (stupid autotools).
But...I would've thought that CMake would run, considering Android now actually has the C++ standard library.

Well, if more asmjit users actually come here and convince me I will add this build script as an alternative to cmake, +boost is sufficient comment for me to know there is a need for it.
But anyway, I really think that asmjit is so simple project to build, there are basically no dependencies and configuration is controlled through compile-time macros, so in the end I think that everybody is just embedding anyway :)

There are a vast number of C++ capable build systems for which the same could be said (i.e. prettier, fewer lines of code, faster): Google build, Scons, Meson, premake, Maiken, ... One could probably go on for quite a bit.
Still, my vote would be for the keeping the existing cmake build system. It's not pretty but it is the de-facto standard for many C++ projects, and it facilitates interoperability with other CMake-based projects that recursively include asmjit using the "add_subdirectory" or "ExternalProject_add" mechanisms.

CMake isn't great but it's pretty much won the cross platform C++ build wars for now. With CMake you fit "natively" into the highest number of projects build systems. That is: I can include asmjit as a submodule and add it to my own projects CMake with add_subdirectory and link to it.
Non CMake people (or just the lazy) can just include the cpp/h files in the project themselves which is hardly any effort.
If I see a python based build system for a project and I'm working on Windows I usually sigh/laugh and move on.
tl;dr: +1 for CMake from me.

Well, it seems like I've lost this battle...

Not lost, but atm cmake fits the most people although I don't really like it as well :)


ModuleNotFoundError: No module named 'glow'

Meet an error when I tried to load waveglow checkpoint downloaded from NGC.
Error information is:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/kk/anaconda3/envs/py3_torch/lib/python3.7/site-packages/torch/serialization.py", line 426, in load
    return _load(f, map_location, pickle_module, **pickle_load_args)
  File "/home/kk/anaconda3/envs/py3_torch/lib/python3.7/site-packages/torch/serialization.py", line 613, in _load
    result = unpickler.load()
ModuleNotFoundError: No module named 'glow'

@un-knight @gprachi28 I am facing the same problem, too. Did you solve it?

The thing is that a new WaveGlow checkpoint is generated by nvidia/waveglow repo. So you can copy glow.py from there, and use this code to upload the model.

Please use the checkpoint provided here

Please use the checkpoint provided here

I downloaded the model from here but still getting the error.
ModuleNotFoundError                       Traceback (most recent call last)
/tmp/ipykernel_25490/3452766092.py in
1 path = "waveglow/waveglow_ljspeech_2/waveglow_256channels_universal_v5.pt"
2
----> 3 waveglow = torch.load(path)['model'].state_dict()
4 waveglow = waveglow.remove_weightnorm(waveglow)
5 waveglow.cuda().eval()
~/python_envs/tacotron/lib/python3.7/site-packages/torch/serialization.py in load(f, map_location, pickle_module, **pickle_load_args)
606                     return torch.jit.load(opened_file)
607                 return _load(opened_zipfile, map_location, pickle_module, **pickle_load_args)
--> 608         return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
609
610
~/python_envs/tacotron/lib/python3.7/site-packages/torch/serialization.py in _legacy_load(f, map_location, pickle_module, **pickle_load_args)
785     unpickler = pickle_module.Unpickler(f, **pickle_load_args)
786     unpickler.persistent_load = persistent_load
--> 787     result = unpickler.load()
788
789     deserialized_storage_keys = pickle_module.load(f, **pickle_load_args)
ModuleNotFoundError: No module named 'glow'

Can anyone got to solve the problem? I am facing the same problem.

Same problem here

same problem here!

with append path to main folder waveglow I've success to load the model
import sys
sys.path.append(path_base+'/waveglow') #where path_base is the path untill main folder

def load_waveglow(waveglow_path):
    waveglow = torch.load(waveglow_path)['model']
    waveglow = waveglow.remove_weightnorm(waveglow)
    waveglow.cuda().eval()

@finardi
Thank you. It worked out the way you did.
Before, sys.path.appeng(relative path) will report an error
After modification, sys.path.appeng(absolute path) has no errors


Dynamo Query API: Support KeyConditionExpression

What this PR does:

Add the skeleton for Query API
Support KeyConditionExpression (the most important parameter in Query API)

What this PR does not address:

Exactly the same error message as DynamoDB
Legacy parameters
FilterExpression
ProjectionExpression
Limit
Pagination
ConsistentRead

Which issue(s) this PR fixes:
Fixes #
Checklist

[ ] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[ ] CLA Signed: DataStax CLA

Right now, we are using Regex to match KeyConditionExpression. It is probably the fastest way (needs benchmarking) but less extensible, flexible, and readable compared to an Antlr (parser) approach. Later, we could revisit this part and refactor using Antlr if needed.


Sensors unknown when the device is turned on

System Health details
I switched from ZHA to Z2M and tried to rebuild the Powercalc sensors.
Some devices: No problem.
Some others: Strange things happen
For example: My e14 Ikea bulbs are working well. e27 bulbs not.
I can see the sensor states if the bulb is turned out. Like Power 0,23W and Energy 1,2kWh (this state is from ZHA times and I like it, that it is still here and does not start from zero).
If I turn the bulb on, both sensors become unavailable. Turn out again: The old states are back there.
Why is that? Help please!
Checklist

[X] I have read the FAQ to see if there is a known solution to my problem.
[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
I switched from ZHA to Z2M and tried to rebuild the Powercalc sensors.
Some devices: No problem.
Some others: Strange things happen
For example: My e14 Ikea bulbs are working well. e27 bulbs not.
I can see the sensor states if the bulb is turned out. Like Power 0,23W and Energy 1,2kWh (this state is from ZHA times and I like it, that it is still here and does not start from zero).
If I turn the bulb on, both sensors become unavailable. Turn out again: The old states are back there.
Why is that? Help please!
Reproduction steps

...
Debug logs
nothing :(

Diagnostics dump or YAML config
No response

ogger: homeassistant.components.mqtt.light.schema_json
Source: components/mqtt/light/schema_json.py:316
Integration: MQTT (documentation, issues)
First occurred: 03:15:47 (2 occurrences)
Last logged: 03:16:31
Invalid or incomplete color value '{'color_mode': 'xy', 'color_options': None, 'linkquality': 81, 'power_on_behavior': 'previous', 'state': 'ON', 'update': {'installed_version': 268572245, 'latest_version': 268572245, 'state': 'idle'}, 'update_available': None}' received for entity light.statuslicht_eg
Invalid or incomplete color value '{'brightness': 254, 'color_mode': 'xy', 'color_options': None, 'linkquality': 183, 'power_on_behavior': 'previous', 'state': 'ON', 'update': {'installed_version': 268572245, 'latest_version': 268572245, 'state': 'idle'}, 'update_available': None}' received for entity light.statuslicht_og

So the error seem to be caused by upstream component/integration. In this case MQTT
I suggest you create an issue report for that in HA core tracker.
You can maybe find some more details in the logs why powercalc "flips out" on it, might be wisefull to temporarily enable debug logging for powercalc.
That should provide insides about why the powercalc sensor goes to unavailable. But this is probably causes by some erronous color returned by the mqtt light.

Hi. Same thing with xiaomi aqara (ZNLDP12LM) bulbs. when it in off state powercalc sensors become unavailable.
Logger: custom_components.powercalc.strategy.lut
Source: custom_components/powercalc/strategy/lut.py:116
Integration: Powercalc (documentation, issues)
First occurred: 07:05:42 (37 occurrences)
Last logged: 11:52:29
light.0x00158d000396a57f_light: Could not calculate power. no brightness set
light.0x158d0002d40e17_light: Could not calculate power. no brightness set

Hm...   its probably XiaomiGateway3 integration bug.  https://github.com/AlexxIT/XiaomiGateway3/issues/1254

These both are issues with other integrations, so really should be reported in their respective issue trackers.
Powercalc needs the brightness and color information to lookup power for the light.
Each light MUST have this information according to HA specifications. When it doesn't powercalc sensor will report errors, but also HA lights cards etc will misbehave.
Closing this issue as it's not something I can fix.


[System.ArgumentNullException] System.ArgumentNullException: Value cannot be null. Parameter name: key

Hi,
could you help where can i start to find the following error:
[I][12:35:03] Connecting to Jira...
[I][12:35:03] Retrieving Jira fields...
[I][12:35:04] Retrieving Jira link types...
[D][12:35:05]    Get item count using query: 'project=SONDEV ORDER BY Rank ASC'
[I][12:35:06] Export started. Exporting 11248 items.
[W][12:35:07] Epic link field missing for config field 'customfield_10002'.
[W][12:35:07] Sprint link field missing for config field 'customfield_10001'.
[W][12:35:07] User mapping file 'C:\Temp\JiraExport' not found, ignoring mapping user identities.
[I][12:35:07] Initializing Jira field mapping...
[W][12:35:07] No target type 'Feature' is set, field status cannot be mapped.
[D][12:35:07]    Enumerate remote issues
[I][12:35:08] Processing 1/11248 - 'SONDEV-11105'.
[D][12:35:09]    Downloaded item.
[E][12:35:09] Unexpected migration error.
[System.ArgumentNullException] System.ArgumentNullException: Value cannot be null.
Parameter name: key
at System.Collections.Generic.Dictionary2.Insert(TKey key, TValue value, Boolean add)    at JiraExport.JiraItem.ExtractFields(String key, JObject remoteIssue, IJiraProvider jira)    at JiraExport.JiraItem.BuildRevisions(JiraItem jiraItem, IJiraProvider jiraProvider)    at JiraExport.JiraItem.CreateFromRest(String issueKey, IJiraProvider jiraProvider)    at JiraExport.JiraProvider.<EnumerateIssues>d__26.MoveNext()    at JiraExport.JiraCommandLine.ExecuteMigration(CommandOption user, CommandOption password, CommandOption url, CommandOption configFile, Boolean forceFresh, CommandOption continueOnCritical):    at System.Collections.Generic.Dictionary2.Insert(TKey key, TValue value, Boolean add)
at JiraExport.JiraItem.ExtractFields(String key, JObject remoteIssue, IJiraProvider jira)
at JiraExport.JiraItem.BuildRevisions(JiraItem jiraItem, IJiraProvider jiraProvider)
at JiraExport.JiraItem.CreateFromRest(String issueKey, IJiraProvider jiraProvider)
at JiraExport.JiraProvider.d__26.MoveNext()
at JiraExport.JiraCommandLine.ExecuteMigration(CommandOption user, CommandOption password, CommandOption url, CommandOption configFile, Boolean forceFresh, CommandOption continueOnCritical)
[I][12:35:09] Export complete. Exported 11248 items (1 errors, 4 warnings) in 00:00:06.

Hello @pdonat01 :)
Hard to say from just this information, but there is definitely a bug. A dictionary insert fails somewhere inside JiraIterm.ExtractFields(). It probably has something to do with misconfiguration at your end, but we should implement a more helpful log message.
Could you provide us with your config file and the full export log, for future reference?
We need additional information from the debugger to proceed. We can do this in one of two ways:

You clone the source code and debug the tool locally in Visual Studio (e.g. community edition), and provide us with the information we need from the VS debugger.
You provide us with a Jira API token over a private channel, and I can run some tests against your Jira organization to try and reproduce the error. You can revoke the token as soon as we are done.

If security is an issue, you can always set up a sandbox project in Jira with only a sample of the problematic issues and obfuscate sensitive data. As long as the issue is reproduceable.

Let me know which method would suit you best!
// Alexander

Relabeled as support issue due to lack of reply


Slides disappearing on nested virtual sliders

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/p/sandbox/swiper-virtual-slides-forked-zgw8xd
Bug description
Slides get removed in nested swiper of swiper with virtual slides if you change swiper.virtual.slides array.
Nested slider doesn't have to be initialized for this bug.
It's a very old issue: #2377
Expected Behavior
Nested slides stay in place
Actual Behavior
Nested slides disappear from DOM

Swiper version
10.0.4
Platform/Target and Browser Versions
Windows, Firefox 116.0.1
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

v

https://codesandbox.io/p/sandbox/swiper-virtual-slides-forked-j2n8fn?file=%2Findex.html%3A90%2C43

OK, disabling cache helps... But it's not obvious and still feels like a bug. At least should be noted in docs.

Changing nested slides class helps as a workaround.


Data Science with Python : DBSCAN Algorithm

Welcome to 'DSWP' Team, good to see you here
With this issue, readers will get introduced with core information to DBSCAN Algorithm along with sample code completely in a effective way.
To get assigned to this issue, add your serial numbers mentioned in the spreadsheet of "Data Science with Python", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
If you had referred any resources, add them up in "DS Resources". Similarly if you had used datasets, include them in "DS Datasets".
Domain : Machine Learning
Mentors Assigned : Kareem Negm / Aishani Singh
Points to Note :
The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Changes should be made inside the Datascience_With_Python/ directory & Datascience_With_Python branch.
Follow Contributing Guidelines & Code of Conduct before start Contributing.
This issue is only for 'GWOC' contributors of 'DSWP' domain.
All the best. Enjoy your open source journey ahead. 😎

Name-Uttara Bahad
Serial No-328
Type: Documentation
Approach-Introduction, Sample example code with explanation, Advantages/ Disadvantages, Use.

Issue assigned to @uttarabahad

Mam is it up for audio?.

Hello @prathimacode-hub , please assigned me the issue

@prathimacode-hub , Mam
I am M. Ashish Reddy. I would like to do video on this issue.
Batch number: 5
serial number:174.
Approach: A detailed explanation with examples and real life uses

Issue assigned to @ashish-reddy-20-08 for video contribution


Error EE: 1C (Outdoor unit computer communication error)

Hello
First of all, thank you for this work, it is something that I have been looking for for a long time to be able to integrate one aircon in a home automation installation.
I have followed the steps to do it on an esp32, but I have found a problem that will surely be related to the connection of the slave.
Could you explain in detail how you have connected the system to the air conditioning?
Following the instructions, the aircon shows an EE: 1C error (Outdoor unit computer communication error) and the main control is disconnected by turning off the aircon.
Is there a connection / power-up sequence for both the aircon and the esp32 so that it is detected as a slave without problems?
Thanks for the help

Sorry I didnt see this before now. I'll attempt to explain the boot up sequence to the best of my memory, I havent looked at this for a while :)
When the AC powers up it continuously sends out frames on the 3-wire bus looking for a primary controller. When a primary controller is finished booting it will wait (a few minutes from my testing, but i dont know the exact time) for frames from the indoor unit.
Once it gets a frame, it announces itself to the indoor unit by sending a login frame, and the indoor unit will respond by sending status frames.
When it receives its first status frame, the primary controller will search for a secondary controller by sending a single frame to the secondary controller address. If the secondary controller successfully forwards that frame back to the unit the primary controller will continue to send frames to the secondary controller. Otherwise it considers itself the only controller and never attempts to send frames to the secondary again.
This means that to add a secondary controller, you need to power cycle the entire system (and also set the primary controller to forget its settings by toggling the dip switch on the back)

Thanks for the answer, unfortunately I have left this parked due to lack of time, as soon as I can get back to it I will try it and comment on the result.
Thanks again

Hello @unreality,
Thank you for your code. I'm too getting mad trying to get it working as a secondary controller.
My master shows a couple of 9C during boot and then nothing. Just the master (wall mounted controller) works and the esp sits there without being able to 'announce' itself or interact with the bus.
I've done a lot of testing and I can see the master querying out for the secondary (around 4 sec after boot) on the bus. The secondary (my esp) then sends out its status frame but the primary looks like it is not recognizing it and stops broadcasting frames to the slave.
I've tried to modify the response delay in code guessing it was a timing problem on the bus but nothing.
In a previous post you suggest to reset the primary controller settings by a dip switch but I can't find any source of information on this. I see the DIP1-2 is used to set the controller unit as master or slave but I can't really understand the meaning of the other switches (fujitsu docs mainly say they're factory set and to not touch)

@krahabb  I believe its the same DIP switch - PIN 1 sets the primary/secondary, and pin 2 turns on/off remembering information (such as the clock time) when the system loses power.


Using WAG/WAF and regional VNet integration at the same time

I have a web app which needs outbound connectivity to an on-premises database (tcp1433) over an ExpressRoute. The inbound connections come from the Internet but not directly - I want to place a WAG/WAF in front of the web app to give me layer 7 protection.
Do I have to use an ASE for the web app or can I use 'regional vnet integration' for the database connection at the same time as using the WAG/WAF for the inbound connections? Thanks

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

ID: ae743313-9367-b465-e31a-5beae33018d1
Version Independent ID: c0cd348d-764a-572b-d900-ff0cffe12a31
Content: Networking features - Azure App Service
Content Source: articles/app-service/networking-features.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@ChrisSommers Thanks for the feedback! We are currently investigating and will update you shortly.

@ChrisSommers Yes, a web app should be able to integrate with a WAF and regional VNET with expressroute. The AppGateway WAF for example uses the FQDN of the web app, so there shouldn't be any configuration issues with that. You should not need an ASE for this configuration.
With that being said, this is a more complex setup so feel free to reach out to us if there are any issues with setting it up. You can reach us for troubleshooting questions on Microsoft Q&A.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Update README.md

With new update to 0.5.2, the readme still showed 0.5.1. Instead of changing it, it is better to stick to "0.5" (as "0.5" means "^0.5.0" it shouldn't be a problem). The only issue is you have breaking changes from 0.5.x to the 0.5.(x+1) then the users have to specify the entire version.

Thanks for the PR, it's perfect! Merging right away 👍


Getting a user ID based on via nickname / on-screen name?

I hate posting my question here but i'm having trouble finding any information on something like this. I would like to get a users ID based from a string of their username.

You can LINQ through the Users collection within Guilds.

you can always join https://discord.gg/discord-api and join the channel #dotnet_discord-net to ask questions ;)

Seems like this has been resolved, closing for now.


Change some operators to fp32 calculation implicitly

Change some operators to fp32 calculation implicitly instead, for preventing certain bug (e.g, to_float in Conv3d and GroupNorm)

These changes need to be verified for training in the future. e.g. how is the performance degraded after fp32 conversion,  does it benefit the training outcome?


Add button for flushing mncache

Specially for windows user, we often need to delete mncache for wallet to display correct info.
This will require to restart the wallet and start staking from scratch.
It would be good if theres a buttton that we could flush the cache file so that we dont restart the wallet

We can add this in 1.0.8

Fixed, will be released in 1.0.9


home link is obsolete

The link http://mlafeldt.github.com/tomdoc.sh (set as the repo's home URL) is not valid.

Thank you for pointing that out. I have removed the link.


Improve UI when credentials are missing in proof request (v2) | Dev

Resolved by: https://github.com/hyperledger/aries-mobile-agent-react-native/pull/1008


Proficiency/Proficiency Die scaling is broken/linearly scaled for custom max level

Originally in GitLab by @CrackJackFlood
If you have a custom maxlevel set the proficiency scaling is linear so a level 30 character gets 1d18 proficiency die for example.
Preferably max level and proficiency should be easily customizable in the settings.
I have tried to find the hooks and references to change the proficiency scaling myself in the worldscript but have been unable to find it, pointer in the right direction would be much appreciated.

You can cap the proficiency for an individual character using an active effect targeting system.attributes.prof. Since there is not rules for proficiency dice above a certain level, our current solution seems as good as any.


ESLPopup: optimize position calculations + test

:tada: This PR is included in version 5.0.0-beta.42 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Ship Detail Page

Changes:

Needs to use the light weights for the font
Sigil is missing
Background color for the Toggle should also be Green ( See Figma)
Loom Size Slider needs to be styled like the design
Start Button should be a button and calendar should be a button as well
Bottom Buttons should be 24px
Production

Design
https://www.figma.com/file/Gwf2q6521ybcvHrff1lA0I/Groundseg?type=design&node-id=607%3A3175&mode=dev

This has been updated


WhatsNew.Text from NSAttributedString Availability Issue

WhatsNewKit Environment

WhatsNewKit version: 2.1
Xcode version: 13.2.1
Swift version: 5
macOS version running Xcode: 11.6.2
Dependency manager (SPM, Manually): SPM

What did you do?
Try to create an instance of WhatsNew.Text from a NSAttributedString with iOS deployment target 14.0:
WhatsNew.Text(NSAttributedString())

What did you expect to happen?
No build error. NSAttributedString should be available on iOS 3.2+.
What happened instead?

'(init(_:)' is only available in iOS 15.0 or newer

Hi @iosdeveloper,
WhatsNewKit is currently using the Text.init(_ attributedContent: AttributedString) initializer which is only available >= iOS 15.0
A fallback for < iOS 15.0 is currently not implemented when initializing a WhatsNew.Text instance with a AttributedString or NSAttributedString.

Ah, that makes sense. I'm trying to use this from UIKit and have to be backwards compatible one major iOS version. I will just make it plain text only as a fallback then.

@SvenTiigi  Unfortunately I'm seeing a crash when running my iOS app on Mac due to the #available(iOS 15.0, *) checks failing. The issue was acknowledged by an Apple engineer in the forums: https://developer.apple.com/forums/thread/691559?answerId=690192022#690192022
The solution is also mentioned there. Can we update the check in Text+WhatsNewText.swift accordingly?

This problem should be fixed with Xcode 13.2
https://developer.apple.com/documentation/xcode-release-notes/xcode-13_2-release-notes
Fixed an issue that caused availability checks in iPhone and iPad apps running on a Mac with Apple silicon to always return true, which caused iOS apps running in macOS Big Sur to see iOS 15 APIs as available, resulting in crashes. These availability checks now return the correct result for apps compiled with Xcode 13.2. (83378814)

That works, thanks. I found another issue for "iOS app on Mac". The Continue button doesn't dismiss the WhatsNewViewController. Manually calling dismiss(animated:) in onDismiss for the primary action works. Not necessarily your fault though, as it works on iOS.


Improve readme

Update readme, to make users understand more on how to use git magic.

Thank you for opening the issue. Soon I will improve that


Domain activation not working!

Hi, I have also a bit confusing Problem.
You can see on this Screenshot, that I have added all CAA on my domain:

Next I go to the AWS-Console, and start the Verfification Process.
And here I get the following Problem:

So what should i do now?
Originally posted by @dersven98 in https://github.com/aws-amplify/amplify-console/issues/2470#issuecomment-1000262926

hi, thanks, i am using cloudflare, but i will test your instructions.
Am 8. Feb. 2022, 22:17 +0100 schrieb hloriana @.***>:

Hi @dersven98 which DNS provider are you using? If you're using GoDaddy, can you make sure to follow step 5 in these instructions to create a second CNAME record to point to the AWS Certificate Manager (ACM) validation server?
Add a custom domain managed by GoDaddy
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Interactive mode: subdirectories are not sized in Windows 10

When executing dua i inside a directory in Windows 10 all subdirectories are reported as having a size of 0.0 Bytes.

Thanks for reporting. That's an odd one, as directories are computed by looking at the sizes of all files contained within, recursively.
If files have a size, the directories should also have a size no matter which OS dua runs on.
Since I cannot try it on windows myself, a fix would have to be contributed.

I've found the problem at https://github.com/Byron/dua-cli/blob/master/src/traverse.rs#L108.
data.name only contains the file name, not the full path, so data.name.size_on_disk_fast(m)... always ends up returning a 0 file size.
I'll post a PR with a fix tomorrow, if you'd like; I need to test the change on *nix machine (my WSL instance doesn't format correctly).

It's interesting, as the apparently working counterpart for the non-interactive version indeed uses the whole path. This might already be the fix here as well.
Ok, I had to try it out - and it really does want to have the filename in the entry, as it will build the path dynamically via the tree - this saves memory.
Sorry for not waiting for the PR, I was just too intrigued to finally see this issue go, but I would be happy if you could give it another round of testing on windows to verify this issue is fixed for good.
Thanks so much for your investigation and all your help!

It's interesting, as the apparently working counterpart for the non-interactive version indeed uses the whole path. This might already be the fix here as well.

Yep, that was the "working baseline" that I used to trace down the bug and figure out the correction.

Ok, I had to try it out - and it really does want to have the filename in the entry, as it will build the path dynamically via the tree - this saves memory.
Sorry for not waiting for the PR, I was just too intrigued to finally see this issue go, but I would be happy if you could give it another round of testing on windows to verify this issue is fixed for good.

No worries. I was fixing it in a different manner... your fix obviously better matches your coding style. 😄
And, it's working on my windows machine! 🚀

Thanks so much for your investigation and all your help!

Happy to help.
BTW, I fixed the disabled tests (see PR #61).

Awesome! So I will be closing this issue, finally :)!
Besides, I wouldn't mind seeing an alternative solution in case you want to share your approach/fix. In any case, if you feel it might be more maintainable or easier to understand, another PR is definitely welcome 👍.
And now, back to TUI… I think the core capability of it, to me, is supporting various backends and using a buffer-diffing approach to manipulate the terminal state efficiently. Maybe one day that will become its own crate so people can build on top of it with alternative implementations - TUI's way of forcing certain ways of usage with traits is usually in my way 😅. Anyway, off-topic, rambling, done :D.

Besides, I wouldn't mind seeing an alternative solution in case you want to share your approach/fix. In any case, if you feel it might be more maintainable or easier to understand, another PR is definitely welcome 👍.

Nah, it was just a draft and I was still hacking/iterating on it. No grand refactoring is lost with it deleted. 😄
Your solution is quite understandable.

And now, back to TUI… I think the core capability of it, to me, is supporting various backends and using a buffer-diffing approach to manipulate the terminal state efficiently. Maybe one day that will become its own crate so people can build on top of it with alternative implementations - TUI's way of forcing certain ways of usage with traits is usually in my way 😅. Anyway, off-topic, rambling, done :D.

I'm somewhat new at rust. I look forward to seeing your progress.
Thanks for the tool and your recent work on the windows/cross-platform story!

Thank you so much for fixing this issue!


Add CI job to check go mod tidied

Reference issue
Fixes #36

@at-wat sorry for the delay! I am on paternity leave for 4 more weeks, will try my best to keep up ;)


Added conversion from BGR to RGB

cv2.imread() returns images in BGR format (https://docs.opencv.org/4.5.2/d4/da8/group__imgcodecs.html#ga288b8b3da0892bd651fce07b3bbd3a56). Added function to convert to RGB.

I think we also need to reconvert when we show the webcam image though.

Good point...hmmm.... Does that mean the cropped image could be RGB or grayscale coming out of the get_features_from_image() function? If it's RGB, it would need to be converted back to BGR prior to calling imwrite(), yes?

Fixed and released in v1.0.5


Addresses issue #629 - matching Figma for /home page, mobile views and did some large screen tweaks as well.

Tweaked /home component to match Figma for sm, md, large screens.

@kevinashworth I've taken out withWidth / width and taken out the fractions for sizing (I don't remember why I chose to do that, haha). And I also noticed that on mobile the yellow heading text was looking weird and the call to actions weren't centering, so I fixed those.

I pushed a couple updates to the add-org.spec tests that were causing a build failure.  Hopefully that won't be a problem from now on.


Swift APIView: refactoring and missing declaration support

This PR adds complete support for missing declarations and improves support for existing declarations like enums and protocols.
It also contains refactoring to improve the rendering consistency of the code.

Latest edits:
https://apiviewstaging.azurewebsites.net/Assemblies/Review/29dc0e0aec33421ea7a6728eb05178e5

/azp run tools - notification-configuration,tools - test-proxy


BZ-1125077 - upgrading code to default behavior in 7.x

@mbiarnes Could you review this when you get a chance?

was this really meant for @mbiarnes?
anyway, +1 from me as the build is green here

@mswiderski Please merge when you get a chance. Thanks!


Create common base template for html emails.

We should create a base template for our html emails that includes the html scaffolding and CSS. By html scaffolding I mean the stuff like the following.
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">

    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
    <title>Zulip</title>

Step 2 is then to change all our emails to extend the template.
You can read about extending templates at http://jinja.pocoo.org/docs/2.9/templates/ (search the page for "extends"), or grep for "extends" in templates/zerver to see examples in our codebase.
All our email templates are in templates/zerver/emails, and all our html emails end in .html. For this issue, don't worry about making the emails look better, or improving the text.

@zulipbot claim

@rishig @vaidap is this now resolved with #5258 done?

ah oops, forgot I had opened an issue for this. Closing.


Allow updates to kubectl-cheatsheet.md

@johndmulhausen @eparis : In support of unblocking this PR - https://github.com/kubernetes/kubernetes/pull/26315

I believe the bot is correct and this just shouldn't live in the main repo, but will wait for other comments from @johndmulhausen and @david-mcmahon

@eparis thanks. i've adjusted my origin PR (and removed the update to kubectl-cheatsheet.md)


Edit Yaml Comments Nesting Level

Proposed changes
When we embed comments into the yaml on Cluster Edit if the comment is embedded in a multi-line string the yaml editor breaks and users have to remove the comments (or nest them correctly) to save the cluster. AFAICT without implementing some pretty complex logic it would be pretty hard to detect if we are in a mulit-line string while looping over the yaml lines to build the content for the editor.
I am proposing the following solution:
While looping, keep track of the lines whitespace. If we embed comments, prepend tracked whitespace to the comment line.
Before dumping the yaml to an object, so it can be parsed over the cluster config (clusters dont take a yaml file), strip all comment lines. Normally we dont have to do this because js-yaml will drop comments when converting yaml to object, but in the case where we embedded comments into ML strings, we must also strip manually. I dont think its useful to send our comments with the string so this made sense to me.

Types of changes
Bugfix

Linked Issues
rancher/rancher#28532

Further comments
@vincent99 @deniseschannon Would this be acceptable? We hit the particular issue mentioned because of the addons: |- multi-line string param in the customers cluster yaml. Because it contained the word ingress which matched our comments translation map we push the ingress comments. But that also means that we have the ingress comments at the normal ingress level. Not the most desirable result. The other option would be disabling embedding of the comments on edit. Thats even easier to do but thought it may raise more confusion.

Isn't the problem that it's trying to insert the comment there in the first place instead of where it was intended?  It doesn't seem like this is really addressing the problem, it's still matching a word in a different random key.  And could be causing new ones (a multiline string can contain a line starting with # that shouldn't be dropped because it's not a comment)


NPM keeps exiting with status:unhealthy and code 137 - out of memory issues?

Version: Latest (2.6.2)
Hardware Raspberry Pi 3B (1GB RAM)
So far this happened 3 times. NPM works perfectly for a few hours and then suddenly dies  while all other containers keep running fine. SS from portainer:

The error I see is Exited with 137
Inspect:

Logs (don't show any errors - crash happened at 6:29AM, nothing at that moment):

I suspect that my RPi might be running OOM? But last I checked htop the RAM was at 600MB/1GB so quite far from full. Any ideas?
PS: I have put Restart Policy as Always in order to alleviate this...

Same here !

Probably the same 2.7.0 bug, raised in #728 and fix is being tested now.

Ignore my previous (deleted) comment, I jumped the gun there.
I've been running NPM on a few different servers including some older architecture ones for a while and this hasn't happened. I'd say the always restart policy is probably the best temporary solution for now without being able to reproduce this myself.