adding graph relabeling and sublattice mappings.

I'm making two significant additions in this PR, and I don't expect it to be merged as-is.  Happy to make them separate issues (also todo; documentation is incomplete, there are no tests, ~I haven't even checked for obvious typos/omitted imports~).  Both are additions to the pegasus_coordinates class.  My plan is to repeat this work for Chimera and Zephyr, but I'm pausing here to get some feedback on the interface, and what I'm calling things.  Yes please bikeshed this
Graph Converters: I've added three functions to convert graphs from one labeling to another.  For example, if you've made a pegasus_graph p with integer labels, but you'd prefer to work with coordinate labels, this saves on boilerplate.  What was previously
p = pegasus_graph(m, ...)
coords = pegasus_coordinates(m)
nodes = coords.iter_linear_to_pegasus(p.nodes())
edges = coords.iter_linear_to_pegasus_pairs(p.edges())
p_coord = pegasus_graph(m, node_list = nodes, edge_list = edges)

is now
p = pegasus_graph(m, ...)
coords = pegasus_coordinates(m)
p_coord = coords.graph_to_pegasus(p)

Sublattice Mappings: This adds functionality to make homomorphisms from small Pegasus graphs into large Pegasus graphs.  This is broken into two steps: (step 1) generating "offsets" which are used to produce (step 2) mappings from a pegasus_graph(m_small) to pegasus_graph(m_large).  We show a barely-contrived use case below; searching for fully-yielded P3 subgraphs:
host_graph = pegasus_graph(m, ...)
host_coords = pegasus_coordinates(m)
mask_graph = pegasus_graph(3)
for offset in host_coords.sublattice_offsets(3):
    mapping = host_coords.sublattice_linear_mapping(3, offset)
    subgraph = host_graph.subgraph(mapping(q) for q in mask_graph)
    if subgraph.number_of_edges() == mask_graph.number_of_edges():
       print("found a fully-yielded P3 graph with offset", offset)

@pau557 @hhtong @jackraymond I think you'll interact with the sublattice mappings the most; @arcondello @randomir I expect you'll have opinions about that and the graph relabeling interface.

Codecov Report

Merging #211 (ba317c4) into main (e988c71) will decrease coverage by 2.02%.
The diff coverage is 18.57%.

@@            Coverage Diff             @@
##             main     #211      +/-   ##
==========================================
- Coverage   71.15%   69.13%   -2.03%
==========================================
  Files          29       29
  Lines        1751     1821      +70
==========================================
+ Hits         1246     1259      +13
- Misses        505      562      +57

Impacted Files
Coverage Δ

dwave_networkx/generators/pegasus.py
72.75% <18.57%> (-14.37%)
:arrow_down:

Continue to review full report at Codecov.

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

I've been asked to do nice_coordinates for Zephyr, and since I already had this branch going and my head was already deep in coordinate spaces, added that to the scope of this PR and finished the whole thing off.  I'm still open to bikeshedding, as long as it's roughly search-and-replace-able.

In your commit comment you mention graph_to_pegasus, but that function seems to be absent.

A couple of superficial problems with nice coordinates, might be worth thinking about. How nice do we want nice to be?
Having negative coordinate values in the nice scheme is a little bit awkward to me. This could be addressed by a modification of the cell definition (i.e. having a few x, y values of m+1, rather than a few x,y values of -1 - pushing awkward boundary conditions to high integers rather than near the origin).
Some documentation describes nice_coordinates x and y as limited to the range [0,m), which is inaccurate without greater context.
The nice coordinate scheme indices don't play nice with the visualization tools (draw_zephyr). Variables ordered in x (or y) are not ordered vertically and horizontally in standard visualization.
Lots of other nice features that could be ruined in trying to workaround these two issues of course..
Z(m=3,t=1) shows features.
m = 2
t = 1
graph = dnx.zephyr_graph(m=m,t=t)
graph = coords.graph_to_nice(graph)
dnx.draw_zephyr(graph,with_labels=True)

@jackraymond and I had a sidebar; the outcome is to drop the current proposal for nice_coordinates and use a 4-coordinate Chimera addressing scheme as depicted below (a C_2n embeds directly in to a Z_n, mapping external couplers to odd couplers).  I'll take a crack at implementing the interface tomorrow.

After time to digest, I came up with a much simpler interface for sublattice mappings.  On a D-Wave internal slack channel, this proved to be a popular option.
def pegasus_sublattice_mappings(source, target):
    """yields mappings from Chimera or Pegasus graphs into Pegasus graphs"""

def chimera_sublattice_mappings(source, target):
    """yields mappings from Chimera graphs into Chimera graphs"""

def zephyr_sublattice_mappings(source, target):
    """yields mappings from Chimera or Zephyr graphs into Zephyr graphs"""

Unable to decide between my original definition of nice_coordinates and the one(s) proposed by Jack, I decided that neither deserves the name "nice".  Instead, I've made it easy, through the interface sketched above, to simply generate all sublattice mappings from chimera_graph(m, m, 8) or  chimera_graph(2*m, 2*m, 4) to zephyr_graph(m, 8) through the same unified interface:
import dwave_networkx as dnx
from matplotlib import pyplot as plt
c2 = dnx.chimera_graph(2)
z2 = dnx.zephyr_graph(2)
for f in dnx.zephyr_sublattice_mappings(c2, z2):
    plt.figure(figsize=(10,10))
    dnx.draw_zephyr_embedding(z2, {0:[f(v) for v in c2]})
    plt.savefig(f"single{''.join(map(str,f.offset))}.png")
    plt.close()
c28 = dnx.chimera_graph(2, t=8)
z4 = dnx.zephyr_graph(4)
for f in dnx.zephyr_sublattice_mappings(c28, z4):
    plt.figure(figsize=(10,10))
    dnx.draw_zephyr_embedding(z4, {0:[f(v) for v in c28]})
    plt.savefig(f"double{''.join(map(str,f.offset))}.png")
    plt.close()

@arcondello I've rebased & rearranged the deltas to be more cogent

@boothby , can you rebase one more time to fix the CI issues?

@jackraymond , are you finished reviewing?


error in nucleoatac run

Hi Alicia,
I am exciting to try NucleoAtac,
I run   nucleoatac run and get the following error
nucleoatac version 0.3.1
start run at: 2016-01-21 13:38
/usr/local/lib/python2.7/dist-packages/matplotlib-1.5.1-py2.7-linux-x86_64.egg/matplotlib/font_manager.py:273: UserWarning: Matplotlib is building the font cache using fc-list. This may take a moment.
warnings.warn('Matplotlib is building the font cache using fc-list. This may take a moment.')
Traceback (most recent call last):
File "/usr/local/bin/nucleoatac", line 291, in
main()
File "/usr/local/bin/nucleoatac", line 69, in main
from nucleoatac.run_occ import run_occ
File "/usr/local/lib/python2.7/dist-packages/nucleoatac/run_occ.py", line 19, in
from nucleoatac.Occupancy import FragmentMixDistribution, OccupancyParameters, OccChunk
File "/usr/local/lib/python2.7/dist-packages/nucleoatac/Occupancy.py", line 12, in
from pyatac.fragmentsizes import FragmentSizes
File "/usr/local/lib/python2.7/dist-packages/pyatac/fragmentsizes.py", line 10, in
from pyatac.fragments import getAllFragmentSizes, getFragmentSizesFromChunkList
File "/usr/local/lib/python2.7/dist-packages/Cython-0.22-py2.7-linux-x86_64.egg/pyximport/pyximport.py", line 431, in load_module
language_level=self.language_level)
File "/usr/local/lib/python2.7/dist-packages/Cython-0.22-py2.7-linux-x86_64.egg/pyximport/pyximport.py", line 210, in load_module
mod = imp.load_dynamic(name, so_path)
File "pysam/cfaidx.pxd", line 21, in init pyatac.fragments (/home/qu/.pyxbld/temp.linux-x86_64-2.7/pyrex/pyatac/fragments.c:7547)
ImportError: Building module pyatac.fragments failed: ['ValueError: pysam.cfaidx.FastqProxy has the wrong size, try recompiling\n']
python 2.7.6
numpy  1.10.4
scipy 0.16.1
cython 0.22
pysam 0.8.4
on Ubuntun 14.04
Thanks!

Not sure what is going on here, some issue with loading a file from pysam.  I would suggest trying to reinstall pysam, especially as error message says "try recompiling" in reference to the error with the file from pysam.

Thanks,
I change to pysam 0.8.2, it works properly now.
by recompiling 0.8.3 or install pysam 0.8.1, the error keeps.
I don't know the reason.


dx12: Implement push constants support

I would also like to request a version bump when this gets merged (:

@msiglreith Thanks! Could you check the build error?
11 |     ast.set_compiler_options(&hlsl::CompilerOptions {
   |                               ^^^^^^^^^^^^^^^^^^^^^ missing `root_constants_layout`
error: aborting due to previous error

Oh, I missed the tests and examples! Should be fixed now.

Great, I'll bump the version after the merge.
bors r+


Fix warnings in Xcode 7.3

Issues fixed:

The workspace reference to the R.swift.Library project was an absolute path instead of a relative path.
The ResourceAppTests target still had leftover Framework search paths from Xcode 6
Swift 2.2 deprecated ++/-- operators
Swift 2.2 deprecated function currying syntax

Let me know if you want me to bump the podspec version.

Thanks for the fixes!
No need to bump the podspec/release a new version this is only code internal to the project. So users won't see these warnings, they will ship with the next release. :)


Remove bootstrap npm script

Signed-off-by: Yulia Nedyalkova<EMAIL_ADDRESS>

Coverage remained the same at 89.151% when pulling 2a16c0f5d3571665af148692273e91deff740afb on SAP:remove_bootstrap into e000144b0e8726b4eb07ceb46557a45278ff95a6 on cloudfoundry-incubator:master.


Correct StreamPosition in text node inside script element

Related to #175.
Probably it's a good idea to implement other indexes - Line and LinePosition.

Hello @arabcewicz ,
Thank you for your contribution, we will look at it this week.
Best Regards,
Jonathan

Hello @arabcewicz ,
Thank you for your contribution,
Your pull has been merge & released in v1.8.1
Best Regards,
Jonathan


Use opensim-core master build for github Actions of this repo

Steps to reproduce
Expected result
Actual result
(include screen grabs where appropriate)
Environment and GUI version

As of now we use a cached version of opensim-core (on myosin) dating back to 1/2020, need to update opensim-core action to upload the latest build of master instead.

@chrisdembia Do you know what's involved on the opensim-core side? Is there a better way to do it without using myosin to upload? Thanks

I suggest that this repo does what Moco does in GitHub Actions, which is build opensim-core without using Sourceforge/Myosin, and cache the opensim-core build using GitHub Actions.
https://github.com/opensim-org/opensim-moco/blob/4f25baf0096fee0c1288747e8f17abe866f8ef05/.github/workflows/continuous_integration.yml#L39-L67

close via #1211


Multiple dots in tld being ignored

Trying to use multiple dots in the tld (e.g. "tld": "foo.bar")
Everything appears to work fine as long as you navigate directly to the correct app url (e.g. http://biz.foo.bar).
However, the frontend strips anything after the first dot in the tld.
https://github.com/typicode/hotel/blame/master/src/front/App.vue#L204
Is there a reason to strip out multiple dots?

Hi @CaryLandholt,
I must say I haven't thought about using dots in tld actually.
And not sure if a tld can have dots.
But you can set the tld to bar, name your app foo and access it using *.foo.bar (wildcard subdomains are supported).
Or you can directly name your app biz.foo.
hotel add <cmd> --name foo


Add initial implementation of datastore-backed sessions

Though a much more robust implementation is available in the 1.9.66 SDK, it would require too much effort to backport it to the current SDK right now. This particular implementation is inefficient,
but I think anything is an improvement over the in-memory implementation that it replaces.
Resolves #3014.

http://ocd.appscale.net:8080/job/Daily Build/6326/

Demo
https://github.com/AppScale/hawkeye/pull/162 with master: http://ocd.appscale.net:8080/job/Hawkeye/18594/
https://github.com/AppScale/hawkeye/pull/162 with this branch: http://ocd.appscale.net:8080/job/Hawkeye/18593/


Add Kubernetes docs

Codecov Report

Merging #67 (56849cc) into main (5715af5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #67   +/-   ##
=======================================
  Coverage   82.15%   82.15%
=======================================
  Files          14       14
  Lines         695      695
=======================================
  Hits          571      571
  Misses         88       88
  Partials       36       36

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 5715af5...56849cc. Read the comment docs.


Add HTTP.cat extension for visual HTTP status codes with fancy cats 🐈⬛🐈

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Works as expected, maybe you could add a way to copy the image to the clipboard too? You can see how it's done in the GIF extension 🙂

Works as expected, maybe you could add a way to copy the image to the clipboard too? You can see how it's done in the GIF extension 🙂

@pernielsentikaer that's an awesome idea! Will do!

@pernielsentikaer there you go! 😊


ARROW-3818: [R] Table$from_batches

https://issues.apache.org/jira/browse/ARROW-3818?filter=12344983

library(arrow)
#>
#> Attaching package: 'arrow'
#> The following object is masked from 'package:utils':
#>
#>     timestamp
#> The following objects are masked from 'package:base':
#>
#>     array, table

b <- record_batch(tibble::tibble(int = 1:2, dbl = rnorm(2)))
tab <- table_from_batches(b,b,b)
tab
#> arrow::Table
as_tibble(tab)
#> # A tibble: 6 x 2
#>     int     dbl
#>   <int>   <dbl>
#> 1     1 -0.267
#> 2     2 -0.0270
#> 3     1 -0.267
#> 4     2 -0.0270
#> 5     1 -0.267
#> 6     2 -0.0270

# this supports !!! splicing so
batches <- rep(list(b), 20L)
tab <- table_from_batches(!!!batches)
tab
#> arrow::Table
as_tibble(tab)
#> # A tibble: 40 x 2
#>      int     dbl
#>    <int>   <dbl>
#>  1     1 -0.267
#>  2     2 -0.0270
#>  3     1 -0.267
#>  4     2 -0.0270
#>  5     1 -0.267
#>  6     2 -0.0270
#>  7     1 -0.267
#>  8     2 -0.0270
#>  9     1 -0.267
#> 10     2 -0.0270
#> # … with 30 more rows

Created on 2019-02-05 by the reprex package (v<IP_ADDRESS>00)

I'm just not sure about the naming. In python the py.Table is exposed but in the R package, the classes are not really meant to be used directly, so defining a static method does not really work, as of now the class is arrow:::`arrow::Table`  ...
We could expose certain classes a first class citizens of the package if we really want the syntax Table$from_batches()

Actually #3565 is better because it does not need a separate table_from_arrays() function, as the table() itself can handle list of record batches

Rebased.


Compiling not truly necessary

I've included uftpd.py under lib/ and imported and run it without problems on an ESP32.  The warning on the README might scare people away who could use it without rolling their own firmware with frozen in byte code!

Thanks for the comment. As far as I remember, that results from the first versions of the ESP32 code, which left very little space in RAM. And there were no SPIRAM variants available at that time.

I have updated the othtr variants too and modified the comment in README.


Unrecognized argument toml config

Hi,
After install your module with poetry I can't use my custom path config,
I get flake8: error: unrecognized arguments: --toml-config
when I launch poetry run flake8 --toml-config src/pyproject.toml
The poetry show show me that all is installed
flake8              6.1.0
pyproject-flake8    6.1.0

Thanks

Oups you're right... Sorry


Determine and test games filled behavior with new dispatch assignment

Currently the 03_Games_Filled test fails because it expects the same behavior as standard assignment - that is, when all slots are filled, the player sees a "sorry" message. However, this doesn't happen with the new dispatch method, as there is no explicit "cap" on participants. What do we want to happen? Can we update the test to match?

Removed test.


What replaces RelationalQueryModelVisitor in .net Core 3?

What replaced RelationalQueryModelVisitor in the latest version of .net Core?

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

ID: b1d91292-59de-23dc-a696-6a2e02941174
Version Independent ID: d724555c-2100-be32-63dc-aebf3e2ae47a
Content: RelationalQueryModelVisitor Class (Microsoft.EntityFrameworkCore.Query)
Content Source: dotnet/xml/Microsoft.EntityFrameworkCore.Query/RelationalQueryModelVisitor.xml
Product: entity-framework-core
GitHub Login: @dotnet-bot
Microsoft Alias: divega

EF Core 3.0 does not use QueryModel coming from Remotion.Linq hence it is altogether removed without any replacement.
State exact scenario you are trying to achieve by using this class.
P.S. https://en.wikipedia.org/wiki/XY_problem

Thank you for the information, I suggest to update the documentation pointing the reader to the new documentation for newer versions of the framework.

I believe https://docs.microsoft.com/en-us/dotnet/api/microsoft.entityframeworkcore.query.relationalquerymodelvisitor?view=efcore-2.2#applies-to section should cover that it does not apply to latest version.
Let us know if we can improve further in any way.


(website playground) panicked at 'capacity overflow' with using range

Hi!
I recently discovered uiua through a code_report video. It looked very interesting so I wanted to try it out. So far it is very fun!
While playing with it, I encountered what I think is a bug:
Program
In the website playground, I have typed the following program:
test ← +⇡
test 15 10

r ← ⇡
r 3

Expected output
[10 11 12 13 14 15 16 17 18 19 20 21 22 23 24]
[0 1 2]

Actual output
None. Instead, "Running" is being displayed forever.
Console
The following errors are printed:
site-11b5530f73ff5e21.js:384 panicked at 'capacity overflow', library/alloc/src/raw_vec.rs:524:5

Uncaught RuntimeError: unreachable

Full traceback: www.uiua.org-1696081573340.txt
Specs
Browser: Google Chrome 118.0.5993.32 (Official Build) beta (64-bit)

Hopefully, this report gives enough information to be useful. Thank you in advance for your response!

For some reason, the traceback doesn't display in my browser, and gets downloaded instead. In case it also happens to you, I'm writing it down here:
Error with Permissions-Policy header: Origin trial controlled feature not enabled: 'attribution-reporting'.
Error with Permissions-Policy header: Origin trial controlled feature not enabled: 'run-ad-auction'.
Error with Permissions-Policy header: Origin trial controlled feature not enabled: 'join-ad-interest-group'.

Multiplication overflowed and crashed the interpreter.
c1c974c88f814 makes it catch the overflow and return an error.
I'm going to leave this open for now, because it's true that this behavior is very unintuitive, and the way that bindings work probably needs to be reworked to avoid this.

I have improved the way that bindings work to make them independent of what is already on the stack.
As of ac86db4024309, your example
test ← +⇡
test 15 10

r ← ⇡
r 3

now produces the outputs you expected.

Thank you very much!


Ray storage

What do these changes do?
Implement ray storage backend.

Related issue number
Fixes #1991

Hi, @fyrestone , test cases failed if ray is not installed.


[dv,pmp] Timeouts because of infinite loops in the generated program

@marnovandermaas and I talked about this internally so I'm assigning him because of that. Please feel free to unassign yourself if you feel like it :)
In the case of a random PMP configuration in which we don't allow execution at the address of exception handler, we end up going inside an infinite loop (which is totally legal). We need to flag those cases as passing because we want their coverage data in our regression results as they cover the cases where pmpcfg0 is configured to random values. This issue tracks that task.

If I remember correctly, we talked about the case when the instruction fault trap handler is set to non-executable. We can catch this case by checking whether two RVFI packets in a row are exactly the same with the trap flag set. With the trap flag not set, it could be a spinning loop, which we might want to allow in certain cases.

We discussed this in the meeting and Greg mentioned that we could check the signal Ibex produces on a nested double trap (a trap raised within the trap handler). In UVM, we should be able to check whether Spike also sees a nested double trap. When we see a certain number (say 1,024) nested double traps in a test, we can decide to terminate the test. For the full random PMP test, we should have a flag that says that this termination of the test should return success while by default it would end up as a failure.


Increase max_message_size gives unknown error

I have an email with about 12MB in attachments that gave error:
Expected response code 250 but got code "552", with message "552 Message too large (maximum size 14MB)
Setting max_message_size to 15:
Expected response code 250 but got code "552", with message "552 Message too large (maximum size **15MB**)
Setting it higher gives:
Expected response code 250 but got code "", with message ""
The data columns in the raw-yyyy-mm-dd tables are type LONGBLOB
Is there another setting I need to change?
(The max_message_size was implemented in ticket #268)

If you're very close to the limit, it might be that you're over without realising it as there can be extra overhead or byte size calculated differently depending on what you're looking at.
Are you restarting Postal when changing the configuration? The empty response suggests something else went wrong, are you able to see anything relevant in the log files?

Right, the log files...
[smtp.1:31311] [2021-03-31T14:06:30.194] ERROR -- : [JYHMXV] Mysql2::Error: The size of BLOB/TEXT data inserted in one transaction is greater than 10% of redo log size. Increase the redo log size using innodb_log_file_size.
Increasing innodb_log_file_size fixed it.
Thanks

ah good to know, I'll write that down


ARM support

Any plans for building an ARM version of the extension?

Hi!
What do you mean exactly?
As far as I know, the extension does not care on the architecture the compiler is targeting. As long as the compiler is able to provide the data needed to compilescore to know what happened it should work.
Please feel free to provide more details, as I think I did not fully understand the request.

Oh I see! Sorry for the misunderstanding.
Sadly I don't have access to a Windows based ARM machine at the moment to try this out. I will have a look if I can setup a github action to validate and create a win-arm64 version. If that is not possible I might have to ask for your help to see where the roadblocks are when building the extension on your end.

I updated the vsix and the manifest in order to work on ARM64 windows machines. As said before I can't try it out on the real thing, but I managed to virtualize a win11 ARM over an apple silicon laptop, and it seems to work.
I would really appreciate if you could give it a shot installing the VS2022 extension artifact generated by the github runners in this Arm64 manifest update change and let me know if you see any issues.

Thanks for the update! At least with my setup the updated version works 👍

Thanks for validating!! I will wrap up a new version and upload it to the market place when I have some spare time.


FlowAutoColumnCount error on iOS

If I set the FlowAutoColumnCount property to true on iOS, the app crashes immediately when the first element is added to the item source.
The FlowListView is inited at the app start, and the app crashes even if I implement the example (SimpleExamplePage) with the autocolumncount.
This is the error description in the HockeyApp crashlog:
" *** Terminating app due to uncaught exception 'CALayerInvalidGeometry', reason: 'CALayer position contains NaN: [nan 21.5]' "

The error also occurs, if the FlowAutoColumnCount property is set to false, and there's one FlowColumnSimpleTemplateSelector element in the FlowColumnsTemplates property's list.

It seems like that the source of the error is connected with the column with.
If I set the FlowAutoColumnCount to true, and doesn't set a FlowColumnDefaultMinimumWidth (or set it lesser or equal than the half of the page), and add 2 FlowColumnSimpleTemplateSelector to the FlowColumnsTemplates list, the error doesn't happen.
If the FlowColumnDefaultMinimumWidth is bigger than the half of the page, and there's two columns, the error happens independently from the FlowAutoColumnCount property.
If I set only 1 FlowColumnSimpleTemplateSelector, the error happens, no matter how I set the other properties.

@tamasszadvari , sorry for answering so late, I've been quite busy. Did you sort it out? I'll try to investigate it as I'm planning to release a new version of FlowListView. Thanks.

No problem, I compiled the library for myself with the PR modifications, and used that dll. :) I've not done a complete investigation, because the fix in the PR solved my problem. But regarding to the notes, that I've made here, and the fix that I've made, the problem has to be gone. (Every case that I've written here makes one column - if I'm correct -, and that caused an error.)

@tamasszadvari Thanks for the info! I'll close it and release a fix release then.


Support lock.

As a user I should be able to lock via a screen icon (or shortcut) encrypting the datastore again.

Need endpoint for locking the software


login to cluster and configure cli

Currently CLI configuration is left to the user or whatever script is calling tests. This is an essential part of running the test and is therefore now captures in test setup

I think this is redundant?


Update release branch for 0.26.7

Description
Cherry-picked: https://github.com/radius-project/radius/commit/bf4e6bb1a508902f6d0068cf2a2df251d6ca152e
Cherry-picked: https://github.com/radius-project/radius/pull/6553/commits/24192aa3bdb9f1e9c72f3afabf0bbbe2951de31b
Type of change

This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional).

Auto-generated summary

🤖 Generated by Copilot at 1f3633a
Summary
📝🚀🐛

This pull request fixes a bug in the Radius Kubernetes controller that affected the encoding of secret values for the Radius agent. It updates the deployment_reconciler.go and recipe_reconciler.go files and their corresponding unit tests. It also adds the release notes and updates the versions.yaml file for Radius v0.26.7.

Sing, O Muse, of the valiant deeds of Radius, the cloud-native agent
That swiftly fixes bugs and errors in its Kubernetes deployment
And of the skilled developers who craft the code with care and wisdom
And document the changes in the notes and files for all to see

Walkthrough

Add release notes for Radius v0.26.7 (link)
Fix double-encoding of Kubernetes secret values in deployment_reconciler.go and recipe_reconciler.go (link, link, link, link)
Remove unused import of encoding/base64 package from deployment_reconciler.go, deployment_reconciler_test.go, recipe_reconciler.go, and recipe_reconciler_test.go (link, link, link, link)
Update unit tests to match the plain string values of secret data in deployment_reconciler_test.go and recipe_reconciler_test.go (link, link, link)
Update versions.yaml to reflect the latest supported version of Radius (link)

/ok-to-test


document pi firmware setup

Add software section to SETUP.md.

First pass here:
https://github.com/sharedstreets/curb-wheel/pull/25

This shipped. For additional changes, let's open new issues.


Define legend styles

Placement
Entry order
Styles for blocks or lines
Style for continuous scales
Style when bin sizes are not equal widths

See #17 for some other thoughts

Also tagging @amfeldman to this issue since I can't properly assign it to her for some reason


"Open Recent" not containing saved files

Issue Type: Bug
"File > Open Recent" should also contain files which were recently created and saved - not just recently opened files.
VS Code version: Code 1.68.1 (30d9c6cd9483b2cc586687151bcbcd635f373630, 2022-06-14T12:52:13.188Z)
OS version: Darwin x64 21.5.0
Restricted Mode: No

Steps to Reproduce:

open VS Code
in the blank new file, enter some text
save it (I did "test.txt" on the Desktop)
"File > Open Recent" ==> no "test.txt" in there
Ctrl+R ==> also no "test.txt"

Needs 20 upvotes - please vote above with the 👍 - thanks!

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Additional info from @chrisdias (in #153376):
File | Open Recent doesn't show new files created in session
Version: 1.69.0-insider (user setup)
Commit: e71b6105ebb4fbcfbf38a1b62bcbafdb048d3468
Date: 2022-06-27T07:59:47.622Z
Electron: 18.3.5
Chromium: 100.0.4896.160
Node.js: 16.13.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22000

Steps to Reproduce:

start vscode fresh
create a new file, add some text
save it locally
File | Open Recent

Result: the file is not in the list
Expected, the file shows up in the recent list, sort of like how this works:

go to a terminal
run code-insiders <some new file.txt>
make some changes to the file, save it
File | Open Recent

Result: the file is in the list

Thanks, @chrisdias! 👍

Additional info from @chrisdias (in #153376):
File | Open Recent doesn't show new files created in session
Version: 1.69.0-insider (user setup)
Commit: e71b6105ebb4fbcfbf38a1b62bcbafdb048d3468
Date: 2022-06-27T07:59:47.622Z
Electron: 18.3.5
Chromium: 100.0.4896.160
Node.js: 16.13.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22000

Steps to Reproduce:

start vscode fresh
create a new file, add some text
save it locally
File | Open Recent

Result: the file is not in the list
Expected, the file shows up in the recent list, sort of like how this works:

go to a terminal
run code-insiders <some new file.txt>
make some changes to the file, save it
File | Open Recent

Result: the file is in the list

Thanks, @chrisdias! 👍

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

I went looking into the code that manages the entries in the recently opened list and agree that not having the ones from the save dialog is a bit inconsistent.
To clarify, the original idea of the recently opened list was to fill it with entries from:

interactions with file dialogs (previously only opening, but with this change also saving)
passing file paths from the command line or when the OS opens VSCode with a path
opening editors via drag and drop

The idea was never to just add entries from any editor you open to this list, because that could become messy quickly. For opening recently opened files of the workspace, we have our own picker control for this purpose:

I can still today not really find good UX advice what the driving factor should be for filling the File | Open Recent... menu. If someone has more pointers, please let me know.


Make comparison operators more standard for FlatSet & SmallSet

std::set comparison operators are templated and should accept other types of sets than only Self.

std::set comparison operators are templated and should accept other types of sets than only Self.

Misread the definition, operand sets should be the same. Closing the issue.


The interface stock_profit_forecast() still doesn't work,please update.

Describe the bug
As I reported in ,this bug still exists in version AKShare 1.8.78.
When sortTypes and sortColumns are set,the return data is duplicated,the result is so many companies are missing.
another issue:
ak.stock_profit_forecast() returns many duplicated data,there are about 2731 records,and 782 are duplicated records,you can test it by print stock_profit_forecast()["名称"].to_list()
This issue can be fixed by removing these two parameters:
params = {
'reportName': 'RPT_WEB_RESPREDICT',
'columns': 'WEB_RESPREDICT',
'pageNumber': '1',
'pageSize': '500',
'sortTypes': '-1',
'sortColumns': 'RATING_ORG_NUM',
'p': '1',
'pageNo': '1',
'pageNum': '1',
'filter': '',
'_': '1640241417037',
}

stock_profit_forecast

接口设计原则是与数据源网站统一；建议在获取数据到本地后进行去重处理

This is a bug and has nothing to do with design  principles. @jindaxiang
From the website,we can see 2731 different companies,ak.stock_profit_forecast() returns 2731 companies,but about 782 are duplicated records,that's to say about 782 company data are missing.We can get only 1949 company data with stock_profit_forecast().
For example,the company '天海防务' are in website:

But we can't get the data from stock_profit_forecast(),it is missing.
Am I clear?

This is a bug and has nothing to do with design principles. @jindaxiang From the website,we can see 2731 different companies,ak.stock_profit_forecast() returns 2731 companies,but about 782 are duplicated records,that's to say about 782 company data are missing.We can get only 1949 company data with stock_profit_forecast(). For example,the company '天海防务' are in website:  But we can't get the data from stock_profit_forecast(),it is missing. Am I clear?

已修复该问题；具体问题为东财网页端的数据返回就有重复值，目前已经按你描述的方法剔除相关请求参数，并新增 symbol 接口参数，升级到 AKShare 1.8.80 版本使用

Thank you for your hard work! @jindaxiang


Fix various messages and documentation bugs

Closes #146

Edit message will show edited ... no matter if any changes are made or not.
Fix here was to check equality of Person objects and throw message if not changed

Closes #150

Adding a professor would result in duplicate person message shown if there are duplicates
This is incorrect as it does not show the user the status of the operation
Lets update this to show how many profs added and how many duplicates

Closes #192

Trivial documentation bug

Looks good! But we may need to update the UG to reflect the new image shown

On second thoughts, this might be a feature addition as we are showing how many profs are going to be added


SignIn with LinkedIn dosen't work (Deprecated)

! Explanation: If you create linkedIn app now, you can't choose product "Sign In with LinkedIn" with scopes ['r_liteprofile', 'r_emailaddress']. Also you don't have credentials for access /me endpoint.
https://learn.microsoft.com/en-us/linkedin/consumer/integrations/self-serve/sign-in-with-linkedin  (Deprecated)
New approch  is here
https://learn.microsoft.com/en-us/linkedin/consumer/integrations/self-serve/sign-in-with-linkedin-v2
SignIn with OpenId was changed:

scopes,
endpoint for retrive user info,
response object for user info

@kiahreading oh, and you need provide new scope fields [openid, email, profile] in your /connect/linkedin action

Hi @AlexGlushko, thank you very much for getting back.
Sorry for the questions, it's just that I have got other forms of login working with PHP League's libraries (eg. Google), but haven't had to make any code changes etc. That is a bit beyond my knowledge.
To make sure I understand you correctly, did you use this Library and make patches to the phpleague/oauth2-linkedin library itself or you wrote something completely separately without this library?

@kiahreading  i still using this lib, but just modified my controller which calls on /connect/linkedin-check action (mentioned in docs for this lib) for call new endpoint provided by LinkedIn (/oauth/v2/userinfo)

I've created my own class that is extending from the Provider and modified it according to this documentation
https://learn.microsoft.com/en-us/linkedin/consumer/integrations/self-serve/sign-in-with-linkedin-v2
`<?php
namespace Cornatul\Social\Social;
use GuzzleHttp\RequestOptions;
use Illuminate\Support\Arr;
use Laravel\Socialite\Two\User;
class CustomLinkedInProvider extends \Laravel\Socialite\Two\LinkedInProvider
{
public $scopes = ["profile","w_member_social", "openid", "email"];
protected function getUserByToken($token)
{
    return $this->getBasicProfile($token);
}

protected function getBasicProfile($token)
{
    $response = $this->getHttpClient()->get('https://api.linkedin.com/v2/userinfo', [
        RequestOptions::HEADERS => [
            'Authorization' => 'Bearer '.$token,
            'X-RestLi-Protocol-Version' => '2.0.0',
        ],
        RequestOptions::QUERY => [
            'projection' => '(sub,name,picture,given_name,family_name,email,locale,email_verified)',
        ],
    ]);

    return (array) json_decode($response->getBody(), true);
}

/**
 * {@inheritdoc}
 */
protected function mapUserToObject(array $user)
{
    return (new User)->setRaw($user)->map([
        'id' => $user['sub'],
        'nickname' => null,
        'name' => $user['name'],
        'first_name' => $user['name'],
        'last_name' => $user['given_name'],
        'email' => $user['email'],
        'avatar' => $user['picture'],
        'avatar_original' => "",
    ]);
}

}
`

it's simple: add new defaultScopes after create provider

        $provider = new LinkedIn([
            'clientId' => $this->getParameter('app.client_id'),
            'clientSecret' => $this->getParameter('app.client_secret'),
            'redirectUri' => $this->getParameter('app.redirect_uri'),
        ]);
        $provider->defaultScopes = ['w_member_social', 'email', 'profile', 'openid'];

        $authorizationUrl = $provider->getAuthorizationUrl();

        return new RedirectResponse($authorizationUrl);

I guess this can be closed

I guess this can be closed

I'd suggest not to close it but to adapt the code or release another plugin like oauth2-linkedin-oidc.
For new apps, the current implementation won't work, and even if we can get access token using the suggested solutions above, methods like \League\OAuth2\Client\Provider\LinkedIn::getResourceOwnerDetailsUrl won't work.


Create .gitattributes

Coverage remained the same at 100.0% when pulling 92336eebe726297eb462d6915cadd0425ff4642d on reedy:reedy-patch-1 into 27a93dc8057ae0b315e90d8e79cb499d27044552 on WhichBrowser:master.

Thanks!
Just noticed a small potential problem...
https://github.com/WhichBrowser/Parser-PHP/blob/27f37af/composer.json#L23
Tests are autoloaded into the same namespace, so with the code as is, composer is presumably going to try and load the test files too whatever...
Needs a bit of testing (before the next release)...
If it does break things, mind if I add a PR to move tests to `WhichBrowser\Tests`` or something and update composer.json accordingly?
Or maybe try https://getcomposer.org/doc/04-schema.md#exclude-files-from-classmaps

Looks like it works fine as is, so no further changes needed


Turbo is not communicating with custom remote cache server

What version of Turborepo are you using?
1.1.2 (tried also 1.1.3 canary versions)
What package manager are you using / does the bug impact?
Yarn v1
What operating system are you using?
Linux
Describe the Bug
Hey!
When I add --api param with custom cache remote server it does not send any data to it. I've tried almost everything, multiple providers and then I switched to request bin to check if turbo sends any data or request and it seems like it does not send anything.
I've tried git clean -dfX on repo to clean everything, then yarn to install deps and then the command below:
yarn turbo run build-turbo --concurrency=4 --api="https://ency9q7ukjtd3zy.m.pipedream.net" --token="my-token"

The workflow has zero requests:

It would be great, if we could somehow pass a param to command that will force or log what is going on, if it tries to fetch remote cache, upload it, etc.
The config of turbo.json is:
{
  "baseBranch": "origin/main",
  "teamId": "superteam",
  "pipeline": {
    "build-turbo": {
      "outputs": ["dist/**", "apps/cms/public/modules/**", "src/generated/**", ".next/**", "!.next/cache/**"]
    }
  }
}

When we used it before vercel, this setting worked.
Thanks

You will need to define pass —token as a non-empty string.

I did. See the command above. Thx

@homoky You also need to pass --team - just ran into this myself.
https://github.com/vercel/turborepo/blob/0c24f5a620c4429809714616aec358f8bf560af7/cli/internal/cache/cache.go#L37-L40

Thanks @dcherman, it works now like a charm. I would recommend to not close the issue until it is somehow reported when --api or apiUrl in config are present.


noParse: /node_modules/ leads to Cannot find module

With the following config, I'm unable to require modules from my app directory without getting a Cannot find module error (at nodejs run time).
var config = {
  context: path.join(__dirname, '..'),
  module: {
    loaders: [
      { test: /\.jsx?$/, loader: 'babel', query: { presets: ['react', 'es2015'] } },
      { test: /\.s?css$/, loader: ExtractTextPlugin.extract('style', 'css', 'sass') }
    ],
    noParse: /node_modules/
  },
  resolve: {
    extensions: ['', '.js', '.jsx'],
    modulesDirectories: ['node_modules', 'app']
  }
};

What I'm trying to do: I'd like to get rid of the externals hack for my server side bundle.

noParse: /node_modules/

That makes no sense. Use externals if you don't want these modules inside the bundle.

Why does that not make sense?

@Jormundir  Because webpack needs to process all of the modules in the node_modules folder

Take a look at https://github.com/liady/webpack-node-externals

as I understand the reason why it cannot find modules when you set noParse: /node_modules/ is that noParse is tested against absolute file path that also includes a loading absolute path, e.g. /Users/foo/bar/baz/qux/node_modules/babel-loader/index.js!/Users/foo/bar/baz/quux/your/project/file.js, so it matches when you specify /node_modules/ and this is why everything is ignored
not sure my explanation is clear enough though :)


Allow to use new version of tenacity

Currently opal-common pins tenacity to version 6.3.1.
When we build the Docker image of our own fetcher based on opal-fetcher-postgres, it will show this error if we use new version

8.0.1 is installed but tenacity==6.3.1 is required by {'opal-common'}

It would be great to allow us to use new version in future. Thanks! 😃

I do need to upgrade the versions of tenacity and other libraries required by opal.
However it's a bit difficult because of inter-dependency constraints they have.
For example, as it stands, there is a coupling between opal dependencies and its lower level libraries dependencies - fastapi_websocket_pubsub and fastapi_websocket_rpc.
I can probably try to do it when we release the next version :)

I think we can move to pin to a minimum (non major) version instead of an exact version; for stable packages like tenacity and FastApi.
Though in this case of Tenacity it wouldn't have helped since it changed via two major versions [6 to 8]  (with possibly breaking changes)


xdr: Regenerate XDR files using Protocol 14.

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
services/friendbot, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.
[ ] I've updated any docs (developer docs, .md
files, etc... affected by this change). Take a look in the docs folder for a given service,
like this one.

Release planning

[ ] I've updated the relevant CHANGELOG (here for Horizon) if
needed with deprecations, added features, breaking changes, and DB schema changes.
[ ] I've decided if this PR requires a new major/minor version according to
semver, or if it's mainly a patch change. The PR is targeted at the next
release branch if it's not a patch change.

What
Update .x files and auto-generated XDR code.
Why
We need to support Protocol 14.
This is using the following branch https://github.com/jonjove/stellar-core/tree/cap-0033
Known limitations

We can move forward with this, however, we need to revisit this https://github.com/stellar/go/issues/2868


[stable/wordpress] Update ingress api version for k8s 1.16. Signed-of…

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Make wordpress compatible with k8s 1.16
Which issue this PR fixes

fixes #17727

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @javsalgar

/ok-to-test

Please, can you take a look at this comment in the attached issue?

I think this PR #17758 encompasses the different cases

Hi @circa10a, I believe this can be closed since #17731 was merged.


…Move physicalcheck into validate_fx_channels

Move physicalcheck into validate_fx_channels as it requires hasPan and hasTilt
Fixes the problem discussed in  #1357 by @mcallegari
Sorry, that happened in the refactoring to make smaller functions due to Codacy complexity warnings. I should have seen that in my review.

Fixed by e5e37ac3c4ea0a865293591e7838d45f896159a2


⚠ Processors has degraded performance

In c075d91, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 45 ms

Resolved: Processors performance has improved in 8b50b81.


FAQ page

may I able to use my own image?
may I able to use my own k8s?
may I able to use my own SUT?
different with running caliper with monitoring?

Hello @SamYuan1990 . I want to work on this issue but can you please help me with the answers so that I would be able to create a FAQ page?

Hello @SamYuan1990 . I want to work on this issue but can you please help me with the answers so that I would be able to create a FAQ page?

thanks, @sheetalj2205 , but I will create it myself with existing questions and answers.

https://github.com/hyperledger-labs/PerformanceSandBox/wiki/FAQ


Update to 7.2pr2

This was something! I think I've got everything working...
Some issues (not sure if I caused these or if they are just work in progress):

Wind lamps seem a bit underpowered, and seem to be unobtainable
Coconuts don't have textures
Shields and painted glass seem to be unobtainable

half of the mod was WIP

looks all good.


Duplicate column headers

My import source has a duplicate column header: "amount" appears twice. If I select either "amount" column in the converter, it only displays the values from the second "amount" column.
I know, this is a violation of whatever spec we could imagine for CSV. Still, it happens. Perhaps in parsing we could append a _1 to the end of duplicate column titles or something.

Good catch. I have the same situation, but the second "amount" is the EUR value, so it is what I want. I will look at how papaparse parses it with duplicate headers and see if they can be differentiated.


Refactored tests with PHPUnit assert methods

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
yes

I've refactored several tests with assertFalse, assertTrue and assertNull methods.

Thanks!


mavo call in welcome message, and popover visible

Hello.
I knew a little bit of grapesJS and discovered Mavo today.
I don't understand why this great project (Create) is stalled. Mavo also deserves more traction and seems to go on.
Being able to prototype your frontend and data structure at the same time without any coding is great for beginners! (like me, or I'm just fed up with coding language).
In this PR, I simply wanted to encourage users and you Francesca and Lea.
I'm not really a coder, but at this level, anybody could be able to contribute :)

Not sure what's the best way to modify my PR

You just push to the branch used to create the PR. I see that you've created this from master, so anything you push to your repo, will be added to this PR.
Fine to use the CDN in a next PR!

Also, why popover was hidden by default in the first place?

I have no idea. Maybe @francesca-cicileo remembers?

I tried checking out the rollback from my fork and it didn't update this PR, so I manually edited the "files changed" in this PR and it seems to work. Hope it's fine.
I'm not a scientist, nor a coder, so your article about your friend dealing with the simple JS/TS algorithm speaks to me, loudly :smile:  I commented it recently.

Seems ok now, merged, thanks!


Migrate from ini to conf

Migrate from ini to conf to hand off the path of the config file to a library instead of managing it ourselves. This should still allow user configuration (although slightly less user friendly) because the path can be read to the user with the .path function.
This is necessary for #22 because the manual paths stuff seems to be messing up real bad and mocking them properly is becoming a huge pain. If I find another way around this without switching to conf, this issue will be closed without a PR.

This might not be necessary as it seems to have been an issue with the compilation process changing the output directory slightly when tests were added into the mix.


[New Feature]: Inform PR authors when they are using an invalid schema

Description of the new feature/enhancement
When a PR is made for a package, the schema version should be a valid version.
A label "Manifest-Version-Error" should be added to the PR informing authors when they are using an invalid version.
PR authors should be informed which version of the schema is the latest supported by validation, and when a PR is created with an invalid version, a label should be added to inform them.
Proposed technical implementation details (optional)
No response

Related to #80572
Related to #83014


[Opera 12.17] I can't access the extension preferences

Hi
I just installed the O12 extension (developer version) but it seems there's no way to access the preferences... For what I know there should be a cog in the top right coner on YT but nothing appears
Is it broken?

this plugin broke a little for opera 12.17 in latest builds, sometimes it doesnt install properly, you need to uninstall it, install older one (for example from official opera store), then  again install the newest build on top, that should fix it :/


Change to use ctrl.ctrl->page instead of ctrl.ns.lba_data_size

In line 773 of include/page_cache.h, we changed to use ctrl.ctrl->page instead of ctrl.ns.lba_data_size to calculate the maximum transfer size of a command

Bug: https://github.com/ZaidQureshi/gpudirect-nvme/issues/5
I do not see a problem with this. Merging.


github: Clarify machine access removal process

Add reference to assist users to know that they should not close the issue when access is required as per the described process.
Checklist

[x] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[x] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[ ] VPC/QPC not applicable for this PR
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

/thaw


search.useIgnoreFiles ignored when working with new search

VSCode Version: 1.29.0
OS Version: Mac OS 10.14

Steps to Reproduce:

have "search.useIgnoreFiles: false" in your workspace settings
add folder to .gitignore
search for content inside that folder with "Use Exclude Settings" enabled in search window
still can't find content inside that folder even though useIgnoreFiles is disabled
enable "search.useLegacySearch" in the workspace settings and it starts working as it should

@roblourens this code ignores the value in settings if the value in options is set. Hope that's any help
disregardIgnoreFiles: typeof options.disregardIgnoreFiles === 'boolean' ? options.disregardIgnoreFiles : !folderConfig.search.useIgnoreFiles

Awesome ❤


Breaks resolve bonus for kills

After getting a melee kill and suspending support weapons, kills no longer give resolve.

Resolved by current version.


Provide build instructions to rebuild the app from Google Play

I tried to rebuild the apk from Google Play but did not get too far. There are no clear build instructions and I couldn't guess how to compile it neither.
My full review can be found here.

Also have had issues compiling!
I believe sencha cmd has been deprecated and integrated into another software suite which (I believe) renders the current code useless? Really Im clueless.
I have no history with sencha cmd, but I believe that is what is used to compile this codebase after digging around a little.

| I believe sencha cmd has been deprecated and integrated into another software suite which (I believe) renders the current code useless? Really Im clueless.
You are correct, Sencha touch has been merged into the ExtJS toolkit (sencha.com) . The build process was using Phonegap to provide native functionality and using Phonegap build services to handle building/signing the builds.
The build process was simply zipping up the github repo codebase into a .zip file and uploading it to the phonegap build service (https://build.phonegap.com/) which also has been depreciated.
Below is the build.sh script I would use to create builds.
I plan on working on a CP compatible mobile and desktop wallet in 2021, so I probably won't be spending the time to port the app from Sencha Touch to ExtJS toolkits.
#!/bin/sh
#
# Script to handle building the native app
#
# Written By  : Jeremy Johnson<EMAIL_ADDRESS>#####################################################
curr_dir=`pwd`
base_dir=/Users/jjohnson/Sites/freewallet-mobile/
cfg_file=$base_dir/phonegap/config.xml
app_file=$base_dir/app/controller/Main.js
tmp_file=$app_file.tmp

# Extract the current version from phonegap/config.xml file
version=`cat $cfg_file | grep "version=" | grep -v xml | awk -F "=" '{print $2}' | tr -d '"'`;

# Get the line # where FW.APP_VERSION is specified in app
linenum=`cat $app_file | grep -n "FW.APP_VERSION=" | awk -F ":" '{print $1}' `

echo "Detected version $version...";

# Swap in the new version number
cat $app_file | sed "$linenum s/.*/        FW.APP_VERSION=\"$version\"/" >  $tmp_file
mv $tmp_file $app_file

# Check if user wants to create web build
while true; do
    read -p "Create production build (web)?" yn
    case $yn in
        [Yy]* ) cd $base_dir
                sencha app build production
                break;;
        [Nn]* ) break;;
        * ) echo "Please answer yes or no.";;
    esac
done

# Check if user wants to create native build
while true; do
    read -p "Create native builds (iOS/Android)?" yn
    case $yn in
        [Yy]* ) cd $base_dir
                phonegap remote logout
                sencha app build native
                break;;
        [Nn]* ) break;;
        * ) echo "Please answer yes or no.";;
    esac
done

# Return user to the current directory
cd $curr_dir

Thanks for the quick reply!
Thanks for the bit of guidance, Ive been able to build freewallet-desktop just fine but struggling for longer than Id like to admit trying to build this repo.. lol
Im looking into porting this codebase to ExtJS toolkits.
I want to build this wallet into a mobile wallet for Aspire ( aspirecrypto.com ) which is based on counterparty and our own blockchain ( not on bitcoin, saving space on chain for bitcoin :) ) Starting from this base I figured would be much quicker than building a mobile wallet from scratch.

@ScriptProdigy there are hundreds of apps out there. I would suggest to not build on one that uses deprecated libraries and build pipelines.
@jdogresorg please let me know when you have some complete build instructions including dependencies up.

@ScriptProdigy there are hundreds of apps out there. I would suggest to not build on one that uses deprecated libraries and build pipelines.

hundreds of open source counterparty mobile wallets? Please link me

@ScriptProdigy I just recently ported some other apps from Sencha Touch to ExtJS7 and it wasn't really that difficult.... Also, not sure if your aware of it, but Tokenly put out a fork of copay which worked with Counterparty for a time.... I believe https://github.com/tokenly/pockets is the repo

hundreds of open source counterparty mobile wallets? Please link me

Nah, not counterparty. I track 240 apps at WalletScrutiny but those are for Bitcoin.


Pooling allocator / memfd: do CoW mapping from file when using a .cwasm

Right now, the memfd-based copy-on-write mechanism for Wasm memories builds the CoW base image on the fly, when the module is first loaded/compiled. This "memfd" then lives just within the process as an anonymous file held open by an fd.
However, some applications of wasmtime separate out the compilation and runtime phases, such that the compilation emits a .cwasm file on disk that the runtime phase later loads. This raises the question: a .cwasm file is also a file, which can be held open by an fd and mmap'd. Why not just mmap the images for the memory/ies straight from the file?
This would mean that we not only would lazily populate the mapping after each repeated instantiate, but we would lazily page the data in from disk on the first instantiate. The virtual-memory subsystem could also throw away the pages (the original file pages would always remain clean in the page cache) under heavy memory pressure, since they have a disk backing. In contrast, a memfd would need to be swapped out to vacate the space.
Thus, doing this would both have advantages to load-to-first-instantiate latency, and to memory pressure on systems with many modules loaded.
As a final advantage, though memfd_create is a Linux-specific syscall, opening a file for an fd is something every Unix can do. If we also replace our use of madvise(MADV_DONTNEED) with a fresh-mmap-on-every-instantiate, we could use this scheme on macOS too (and maybe Windows, but I know less about the semantics of holding files open there).

The main functional change(s) this would require would be to:

create a ready-to-use image of each heap memory with initializers in the .cwasm. This is basically like an ELF binary's image of .data / .rodata. Right now I think we have this in a condensed form (list of initializers with offsets) instead;
possibly tweak the API to pass an open File into a new constructor for Module.


maint: adding group_timepoints to init

this PR adds group_timepoints for direct import - for consistency, since this was already added for engraftment.

This PR is also blocked (due to test failures) by this PR. Once it's merged, I'll re-run the tests and they should work as expected.


Beginner with FastAPI getting error on Uvicorn

First Check

[x] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from fastapi import FastAPI
app = FastAPI()

@app.get("/")
async def root():
    return {'message': "Hello World"}

Description
I am getting this error.
←[32mINFO←[0m:     Started server process [←[36m1200←[0m]
←[32mINFO←[0m:     Waiting for application startup.
←[32mINFO←[0m:     Application startup complete.
←[31mERROR←[0m:    [Errno 10048] error while attempting to bind on address ('<IP_ADDRESS>', 8000): only one usage of each socket address (protocol/network address/port) is normally permitted
←[32mINFO←[0m:     Waiting for application shutdown.
←[32mINFO←[0m:     Application shutdown complete.

Operating System
Windows
Operating System Details
Windows 10 Pro 64 bit
FastAPI Version
0.70.1
Python Version
3.10.0
Additional Context
No response

Actually, I restarted the command prompt and changed the directory. The command uvicorn main:app  worked like a charm.


Correctly chunk sse calc in importance block sizes

A previous patch tried to do this, but missed the 4x4 case on subsampled
frames. Roughly 2% speedup on the default speed level.
https://beta.arewecompressedyet.com/?job=ref_c1432f0%402020-06-13T00%3A43%3A41.614Z&job=sse4x4%402020-06-13T00%3A42%3A15.960Z

Coverage decreased (-0.02%) to 82.012% when pulling 3fa94f06ae962d25f38d87a5d768fa9b5e861d3d on KyleSiefring:sse_4x4_for_real_review into 132be5cf3812a2233e14492c273c512e49ee278a on xiph:master.


static fragment data passed to handlebars context

Static fragment data passed to the handlebars.
Description

During FragmentContext initialization static data (prefixed with data-const- is now passed stored into the context. Prefix used for these properties is _const. Therefore data attribute data-const-foo can be referenced from handlebars as {{_const.foo}]
Motivation and Context

Better code organization - and solution for some CMS systems that are not able to process the tags inside the fragment.

Implementation of https://github.com/Cognifide/knotx/issues/391
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I hereby agree to the terms of the Knot.x Contributor License Agreement.

@Skejven changelog and upgrade notes updated :)

This PR is outdated. Please check Knot.x 2.0 implementation to validate this idea.


Endpoints no longer exist or error

The endpoints no longer work as expected. They were working until end of last week (5th April 2019).
https://graph.microsoft.com/testcpimtf/trustFrameworkPolicies
{
  "error": {
    "code": "UnknownError",
    "message": "",
    "innerError": {
      "request-id": "66099be5-064a-49d1-8bb7-cb8df84e7230",
      "date": "2019-04-09T11:05:10"
    }
  }
}

Uploading a policy
https://graph.microsoft.com/testcpimtf/trustFrameworkPolicies/{id}/$value
{
  "error": {
    "code": "UnknownError",
    "message": "<Error><Message>No HTTP resource was found that matches the request URI 'https://cpim.windows.net/api/trustFrameworkPolicies('[REDACTED_ID]')/$value'.</Message><MessageDetail>No type was found that matches the controller named 'trustFrameworkPolicies('[REDACTED_ID]')'.</MessageDetail><StackTrace /></Error>",
    "innerError": {
      "request-id": "a8a6e45e-1c2b-439f-86d1-0546ab0aaf53",
      "date": "2019-04-09T11:13:32"
    }
  }
}

What have they changed to or have they been removed completely?

Please try replacing /testcpimf/ with /beta/

Thanks, error is now:
{
  "error": {
    "code": "",
    "message": "No HTTP resource was found that matches the request URI 'https://cpim.windows.net/graph/trustFrameworkPolicies'.",
    "innerError": {
      "message": "No routing convention was found to select an action for the OData path with template '~/entityset'.",
      "request-id": "1aca82bf-1471-41e6-a131-f4bd69ea595c",
      "date": "2019-04-09T12:42:52"
    }
  }
}

This is for a request to:
https://graph.microsoft.com/beta/trustFrameworkPolicies

Thanks for reaching out @tjpeel.
The correct api is https://graph.microsoft.com/beta/trustFramework/policies
Apologies for the confusion. We are in the process of updating our samples and documentation.

The sample is now updated


DataprocClusterJobFlowInstanceProfile Needs S3 permissions

EMR machines need to be able to pull jars and files from S3 and need permissions to do so

raster-foundry-deployment/dataproc/cfb/dataproc-cluster.yml attaches AmazonElasticMapReduceforEC2Role (which has Full S3 Access permissions) to the instance profile. Is there a specific bucket that EMR machines haven't been able to access?

Confirmed. I removed the policy and successfully executed an ingest with new imagery.


tracing: update LightStep tracer

Description:
Updates to the a newer version of LightStep's tracer. Much of the tracer was rewritten to provide better performance.
In our benchmarking, the speedup is around 2x-5x
Risk Level: Medium
Testing:
Ran an example and verified that it reports traces.

Seems like you need to merge master?

Does this fix https://github.com/envoyproxy/envoy/issues/7363 ?

Note: LightStep's tracer doesn't do anything with the gRPC response message. Only the response codes are relevant for determining if a report request succeeded.
Since the response isn't used, I changed the code to ignore it and dropped the invalid response tests.

@lizan @moderation - could you guys take a look?

From a patch removal and dependency management POV it LGTM. Will need to defer to @lizan for the code changes

@rnburn I just built successfully for aarch64 but I get the following error when trying to run. Any tips on how to debug further and get more detailed output appreciated.
[2020-03-11 18:38:24.851][16423][info][tracing] [source/common/tracing/http_tracer_manager_impl.cc:25] instantiating tracing driver: envoy.tracers.lightstep
fish: '/home/username/Library/envoy/…' terminated by signal SIGILL (Illegal instruction)

Does it have anything more in the stack trace?

I'm compiling with the following in my .bazelrc file. I'm guessing I'll need to change to get more info. I pinged you on Slack in case that might be faster:
build --define disable_object_dump_on_signal_trace=disabled
build --define signal_trace=disabled


Documentation for sea-orm

Adds documentation for all public types for this crate.

@billy1624 know why the build failed?

@tyt2y3 I am investigating a way to solve the issue. What I have discovered on my end is that the Github workflow runs cargo test --workspace which tests even the documentation.
Somehow the test is not detecting crate imports in the documentation. There are errors like
  |
3 | use sea_orm::entity::prelude::*;
  |     ^^^^^^^ use of undeclared crate or module `sea_orm`

even though use sea_orm::entity::prelude::*;  is imported.

@billy1624 know why the build failed?

Which error? There are a lot of errors loll

@billy1624 @tyt2y3 Can someone help me with this issue?
Running cargo test --workspace seems to be the issue. It is not detecting sea_orm_macros in documentation examples.
But when i run cargo test --doc or cargo test --all-targets everything works fine, the macros and modules are detected. Does anyone have suggestions on how to import macros on documentation tests ?

Click on the line number on the left and copy & paste the URL

@billy1624 @tyt2y3 Can someone help me with this issue?
Running cargo test --workspace seems to be the issue. It is not detecting sea_orm_macros in documentation examples.
But when i run cargo test --doc or cargo test --all-targets everything works fine, the macros and modules are detected. Does anyone have suggestions on how to import macros on documentation tests ?

Is this related? #249

Hey @charleschege, is it possible for you to grant write access to us? I want to add the follow code snippet to .github/workflows/rust.yml.
  test-temp:
    name: Unit Test (Temp)
    runs-on: ubuntu-20.04
    strategy:
      matrix:
        arg: ["--workspace", "--doc", "--all-targets"]
    steps:
      - uses: actions/checkout@v2

      - uses: actions-rs/toolchain@v1
        with:
          profile: minimal
          toolchain: stable
          override: true

      - uses: actions-rs/cargo@v1
        with:
          command: test
          args: ${{ matrix.args }}

https://docs.github.com/en/github/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork

I have added access to the repo

Hey @charleschege, can you point out which error should I look into? E.g. https://github.com/SeaQL/sea-orm/runs/4054619075?check_suite_focus=true#step:4:589 this error. Thanks!

My issues stem from this test run:
https://github.com/SeaQL/sea-orm/runs/4054619075?check_suite_focus=true

Hey @charleschege, it’s already 11pm (HKT) for me. I’m about to sleep. Will talk to you tmr.

wow now it builds! a big thanks to @billy1624


Sort on multiple keys with datasets.Dataset.sort()

Feature request
From discussion on forum: https://discuss.huggingface.co/t/datasets-dataset-sort-does-not-preserve-ordering/29065/1
sort() does not preserve ordering, and it does not support sorting on multiple columns, nor a key function.
The suggested solution:

... having something similar to pandas and be able to specify multiple columns for sorting. We’re already using pandas under the hood to do the sorting in datasets.

The suggested workaround:

convert your dataset to pandas and use df.sort_values()

Motivation
Preserved ordering when sorting is very handy when one needs to sort on multiple columns, A and B, so that e.g. whenever A is equal for two or more rows, B is kept sorted.
Having a parameter to do this in 🤗datasets would be cleaner than going through pandas and back, and it wouldn't add much complexity to the library.
Alternatives:

the possibility to specify multiple keys to sort by with decreasing priority (suggested solution),
the ability to provide a key function for sorting, so that one can manually specify the sorting criteria.

Your contribution
I'll be happy to contribute by submitting a PR. Will get documented on CONTRIBUTING.MD.
Would love to get thoughts on this, if anyone has anything to add.

@mariosasko If I understand the code right, using pyarrow.compute.sort_indices would also require changes to the select method if it is meant to sort multiple keys. That's because select only accepts 1D input for indices, not an iterable or similar which would be required for multiple keys unless you want some looping over selects. Doesn't seem that straight-forward but I might be missing something here...

@MichlF No, it doesn't require modifying select because sorting on multiple keys also returns a 1D array.
It's easier to understand with an example:
>>> import pyarrow as pa
>>> import pyarrow.compute as pc
>>> table = pa.table({
...     "name": ["John", "Eve", "Peter", "John"],
...     "surname": ["Johnson", "Smith", "Smith", "Doe"],
...     "age": [20, 40, 30, 50],
... })
>>> indices = pc.sort_indices(table, sort_keys=[("name", "ascending"), ("surname", "ascending")])
>>> print(indices)
[
  1,
  3,
  0,
  2
]

Thanks for clarifying.
I can prepare a PR to address this issue. This would be my first PR here so I have a few maybe silly questions but:

What is the preferred input type of sort_keys for the sort method? A sequence with name, order tuples like pyarrow's sort_indices requires?
What about backwards compatability: is it supposed to also accept the old way of calling sort() or should both column and kind be deprecated?
If sort_keys is provided in the same format as for pyarrow's sort_indices, reverse doesn't make much sense either and should be deprecated as well I assume.

I think we can have the following signature:
def sort(
        self,
        column_names: Union[str, Sequence[str]],
        reverse: Union[bool, Sequence[bool]] = False,
        kind="deprecated",
        null_placement: str = "last",
        keep_in_memory: bool = False,
        load_from_cache_file: bool = True,
        indices_cache_file_name: Optional[str] = None,
        writer_batch_size: Optional[int] = 1000,
        new_fingerprint: Optional[str] = None,
    ) -> "Dataset":

So we should:

renamecolumn to column_names. column is a positional argument, so it's OK to rename it (not marked as positional-only with "/", but still should be fine)
deprecate kind
keep reverse instead of introducing sort_keys, but we should allow passing a list of booleans that defines the sort order of each column from column_names to it (reverse = False would be equal to [False] * len(column_names) and reverse = True to [True] * len(column_names))

I am pretty much done with the PR. Just one clarification: Sequence in arrow_dataset.py is a custom dataclass from features.py instead of the type.hinting class Sequence from Python. Do you suggest using that custom Sequence class somehow ? Otherwise signature currently reads instead:
    def sort(
        self,
        column_names: Union[str, List[str]],
        reverse: Union[bool, List[bool]] = False,
        kind = "deprecated",
        null_placement: str = "last",
        keep_in_memory: bool = False,
        load_from_cache_file: bool = True,
        indices_cache_file_name: Optional[str] = None,
        writer_batch_size: Optional[int] = 1000,
        new_fingerprint: Optional[str] = None,
    )

Also, to maintain backwards compatibility, I added conditionals for null_placement, because pyarrow's null_placement only accepts at_start and at_end, and not last and first.
If that is all good, I think I can open the PR.

I meant typing.Sequence (datasets.Sequence is a feature type).
Regarding null_placement,  I think we can support both at_start and at_end, and  last and first (for backward compatibility; convert internally to at_end and at_start respectively).

I meant typing.Sequence (datasets.Sequence is a feature type).

Sorry, I actually meant typing.Sequence and not type.hinting. However, the issue is still that dataset.Sequence is imported in arrow_dataset.py so I cannot import and use typing.Sequence for the sort's signature without overwriting the dataset.Sequence import. The latter is used in the align_labels_with_mapping method so it's necessary inside arrow_dataset.py.
To import typing.Sequence as something else than Sequence to avoid overwriting may only be confusing and doesn't seem good practice!? Please advise,
Thanks for all the clarifications!

You can avoid the name collision by renaming typing.Sequence to Sequence_ when importing:
from typing import Sequence as Sequence_

Resolved via #5502


findContours() error since recent change

Since the recent changes (mentioning newer opencv which raspbian doesn't have?), speed-cam.py fails with:
Traceback (most recent call last):
  File "speed-cam.py", line 521, in <module>
    speed_camera()
  File "speed-cam.py", line 395, in speed_camera
    thresholdimage, contours, hierarchy = cv2.findContours( thresholdimage,cv2.RETR_EXTERNAL,cv2.CHAIN_APPROX_SIMPLE )
ValueError: need more than 2 values to unpack

Rolling back the changes in this area, made in https://github.com/pageauc/rpi-speed-camera/commit/cfa677e9a56592c82f8da5a03d55671b7897976f , works around the problem.

I have tested under latest opencv2 and 3 using python 2 and 3 without issue.  I previously had code to detect python2 and 3 since opencv2 and 3 had slightly different syntax.  The new versions now have the same syntax so I took the detection out.  This means you will need the latest Jessie and opencv updates.
It looks like you are running under python2 but not sure what version of Raspbian you are using.  My suggesting would be to run latest Jessie and opencv versions.  If you rerun the install this should update Rasbian but if you are not running Jessie you will have problems.
This is type of thing is due to changes being made to the python libraries.
Let me know if this is still a problem ....
Regards Claude ...

I have updated speed-cam.py and added a try except for the findcontours.  It should run the alternate syntax and recover with the new syntax.  If that fails it will exit with a message.  I do not have older build so if you can test this and it runs OK let me know.
Thanks Claude ...

Did some testing and opencv error was not a value error.  Just replaced with a try except and this seems to work OK.  Download ver 3.4 and this should work.  Let me know if there are still problems.  I could not just check for opencv3 vs 2 since both now have same syntax except for older versions of python 2.  This should be a good solution.


Update _index.md

Update setup/learning-environment/_index.md
<

Two suggestions:

Please ensure your Chinese localization PRs have a [zh] prefix in their titles. This will be very helpful for people watching the website repo.
Please always ensure you have single commit per PR, unless you have a compelling reason to split a PR into several commits.

/label tide/merge-method-squash

/retitle [zh] update learning environment index

/approve

/lgtm


Improve conversion code

This issue/improvement has been resolved in v0.3.0

This issue/improvement has been resolved in v0.3.0


Performance: Improve input speed for widgets

Really cool to see this fork! It looks promising so far and I just want to pull over a commonly reported perf issue from NCMS as it was still an issue at the time of the fork... I'm happy to implement the change as well since I already did it over there. lol
Problem
Currently, typing in input fields works fine so long as a collection doesn't contain a lot of inputs and/or lists.
At a certain point when you have enough lists and inputs, the input delay basically ruins the experience. Having opened lists while typing multiplies the performance issue.
Cause
I believe the issue is that for each keystroke to any widget receiving text input, the redux store is recalculating its state and populating it to every single component. This change can take up to 300ms or more per key stroke for large data sets (think: kitchen sink with multiples of each widget, including multiple lists with widgets inside).
Possible fix
Debounce props.onChange in widgets that receive text input.
Affected widgets:

Number
String
Text
Markdown
Code
DateTime
Color

Technical change should be straightforward although a bit tedious. Instead of always calling props.onChange, instead use a piece of internal state to quickly update the input value, and debounce the props.onChange handler (onChange is probably not a good name for this prop anymore, should be something like handleCollectionUpdate maybe?).
Tangent optimizations
Another thing I noticed is that collapsed List or Object widgets are still rendering their DOM, even though they're visually hidden. Completely unrendering those portions of the component would at least improve on reconciliation speed, although I'm sure it's negligible.
Alternatives / thoughts
I'm not sure if it will be possible to instead debounce the actual redux store update (but still continuously calculate the value so the draft state is still correct), but that would be the "smarter" approach. I still think regardless of that change, the inputs should maintain their state internally to ensure state is as fast as possible a la native html.
Another consideration could be to rearchitect the app so the collection data and UI state are more loosely coupled. As an example, any changes to the UI (via input or widget inclusion) trigger an update to the collection data, but the UI itself is only driven by the data on initial page load.

I would inclined to take a combined approach of debouncing the value to redux (maybe something like every 100ms) AND making the inputs uncontrolled (manage their own values instead of relying on redux to tell them).
I am working on a massive revision of the app right now (typescript conversion, removing immutable js, updating several dependencies, switching to a material UI). My goal is to have it far enough long to merge it into the main branch and start pushing to the @next tag on npm by the end of next week (right now the changes are in this branch: https://github.com/StaticJsCMS/static-cms/tree/feature/typescript-conversion). Once that is merged in, we can start looking into this. Feel free to open some PRs for it.

I think I will actually take care of the decoupling of the widgets' ui from redux in the rewrite. I am already changing a lot of the internal struggle of them to be more uniform, so its a good chance to make this change.

Sounds good. :shipit:


Get-PSSessionConfiguration returns an error

Steps to reproduce
PS C:\> Get-PSSessionConfiguration | select name

Expected behavior
Get names of the registered session configurations.
Actual behavior
Name
----
microsoft.powershell
Exception calling "Load" with "1" argument(s): "Could not load file or assembly
'Microsoft.Powershell.Workflow.ServiceCore, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. The
system cannot find the file specified."
At line:33 char:9
+         $serviceCore = [Reflection.Assembly]::Load("Microsoft.Powersh ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : FileNotFoundException

microsoft.powersh...
microsoft.powersh...

Environment data
> $PSVersionTable
Name                           Value
----                           -----
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSVersion                      6.0.0-alpha
SerializationVersion           <IP_ADDRESS>
GitCommitId                    v6.0.0-alpha.8
PSRemotingProtocolVersion      2.3
WSManStackVersion              3.0
BuildVersion                   <IP_ADDRESS>
PSEdition                      Core
CLRVersion

Looks like an old call to Workflow. We should fix this.

There does not appear to be an Enable-PSRemoting command.
I found and ran Install-PowerShellRemoting.ps1 as Administrator. That ended with:
Get-PSSessionConfiguration powershell.6.0.0-beta.1
Get-ChildItem : The client cannot connect to the destination specified in the request. Verify that the service on the
destination is running and is accepting requests. Consult the logs and documentation for the WS-Management service
running on the destination, most commonly IIS or WinRM. If the destination is the WinRM service, run the following
command on the destination to analyze and configure the WinRM service: "winrm quickconfig".
At line:98 char:3
+   Get-ChildItem 'WSMan:\localhost\Plugin\' -Force:$force | ? { $_.nam ...
+   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Get-ChildItem], InvalidOperationException
    + FullyQualifiedErrorId : System.InvalidOperationException,Microsoft.PowerShell.Commands.GetChildItemCommand

Ran winrm quickconfig.
PS C:\Program Files\PowerShell\6.0.0-beta.1> winrm quickconfig
WinRM service is already running on this machine.
WinRM is already set up for remote management on this computer.

Running Get-PSSessionConfiguration yielded:
PS C:\Program Files\PowerShell\6.0.0-beta.1> Get-PSSessionConfiguration

Name          : microsoft.powershell
PSVersion     : 5.1
StartupScript :
RunAsUser     :
Permission    : NT AUTHORITY\INTERACTIVE AccessAllowed, BUILTIN\Administrators AccessAllowed, BUILTIN\Remote
                Management Users AccessAllowed

Exception calling "Load" with "1" argument(s): "Could not load file or assembly
'Microsoft.Powershell.Workflow.ServiceCore, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. The
system cannot find the file specified."
At line:33 char:9
+         $serviceCore = [Reflection.Assembly]::Load("Microsoft.Powersh ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : FileNotFoundException

Name          : microsoft.powershell.workflow
PSVersion     : 5.1
StartupScript :
RunAsUser     :
Permission    : BUILTIN\Administrators AccessAllowed, BUILTIN\Remote Management Users AccessAllowed

Name          : microsoft.powershell32
PSVersion     : 5.1
StartupScript :
RunAsUser     :
Permission    : NT AUTHORITY\INTERACTIVE AccessAllowed, BUILTIN\Administrators AccessAllowed, BUILTIN\Remote
                Management Users AccessAllowed

Name          : microsoft.windows.servermanagerworkflows
PSVersion     : 3.0
StartupScript :
RunAsUser     :
Permission    : NT AUTHORITY\INTERACTIVE AccessAllowed, BUILTIN\Administrators AccessAllowed

Name          : powershell.6.0.0-beta.1
PSVersion     : 5.0
StartupScript :
RunAsUser     :
Permission    : BUILTIN\Administrators AccessAllowed

PS C:\Program Files\PowerShell\6.0.0-beta.1> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.0.0-beta
PSEdition                      Core
BuildVersion                   <IP_ADDRESS>
CLRVersion
GitCommitId                    v6.0.0-beta.1
OS                             Microsoft Windows 10.0.15063
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

@Liturgist It looks like, despite all the errors that you got, you registered a 6.0.0-beta.1 WinRM endpoint. I'd say we should clean up all these errors, but it's still a lower priority considering the functionality appears to be working.

Resolved via #2671


Fixes line width at table bottom

Damn, I just did the exact same piece of work, should have clicked the Pivotal ticket sooner :/
:+1:
Although "No description provided" in the PR message is a no-no.  Ideally we'd have a link to the Pivotal ticket plus a before and after screenshot in there.

e.g.
Fixes this bug: https://www.pivotaltracker.com/story/show/144847791
The final part of the final row underline was missing. This fixes it.
BEFORE

AFTER

Thanks @TheDoubleK forgot about the screenshots.


Add Prometheus and Grafana

As a Zeebe developer, I would like to use Zeebe's built-in Grafana dashboard with docker-compose.

Happy to take a PR for this.

We discontinue this repo - so I close this


Bad codegen for resources with witx_bindgen_wasmtime::import!

Using this witx:
resource foo {
    bar: function() -> string
}

for the macro invocation of witx_bindgen_wasmtime::import!("foo.witx") generates the following code:
mod foo {
    #[allow(unused_imports)]
    use witx_bindgen_wasmtime::{anyhow, wasmtime};
    pub trait Foo: Sized {
        type Foo: std::fmt::Debug;
        fn foo_bar(&mut self, self: () /* BUG: function already has a self parameter */) -> String;
        fn drop_foo(&mut self, state: Self::Foo) {
            drop(state);
        }
    }
    pub struct FooTables<T: Foo> {
        foo_table: witx_bindgen_wasmtime::Table<T::Foo>,
    }
    impl<T: Foo> Default for FooTables<T> {
        fn default() -> Self {
            Self {
                foo_table: Default::default(),
            }
        }
    }
    pub fn add_foo_to_linker<T, U>(
        linker: &mut wasmtime::Linker<T>,
        get: impl Fn(&mut T) -> (&mut U, &mut FooTables<U>) + Send + Sync + Copy + 'static,
    ) -> anyhow::Result<()>
    where
        U: Foo,
    {
        use witx_bindgen_wasmtime::rt::get_func;
        use witx_bindgen_wasmtime::rt::get_memory;
        linker.func_wrap(
            "foo",
            "foo::bar",
            move |mut caller: wasmtime::Caller<'_, T>, arg0: i32, arg1: i32| {
                let func = get_func(&mut caller, "canonical_abi_realloc")?;
                let func_canonical_abi_realloc =
                    func.typed::<(i32, i32, i32, i32), i32, _>(&caller)?;
                let memory = &get_memory(&mut caller, "memory")?;
                let host = get(caller.data_mut());
                let (host, _tables) = host;
                let param0 = _tables
                    .foo_table
                    .get((arg0) as u32)
                    .ok_or_else(|| wasmtime::Trap::new("invalid handle index"))?;
                () // BUG: return type is Result<()>
            },
        )?;
        linker.func_wrap(
            "canonical_abi",
            "resource_drop_foo",
            move |mut caller: wasmtime::Caller<'_, T>, handle: u32| {
                let (host, tables) = get(caller.data_mut());
                let handle = tables.foo_table.remove(handle).map_err(|e| {
                    wasmtime::Trap::new({
                        let res = ::alloc::fmt::format(::core::fmt::Arguments::new_v1(
                            &["failed to remove handle: "],
                            &match (&e,) {
                                (arg0,) => [::core::fmt::ArgumentV1::new(
                                    arg0,
                                    ::core::fmt::Display::fmt,
                                )],
                            },
                        ));
                        res
                    })
                })?;
                host.drop_foo(handle);
                Ok(())
            },
        )?;
        Ok(())
    }
    use witx_bindgen_wasmtime::rt::RawMem;
}

There are two bugs here:

The trait implementation for foo::bar already has a self parameter, but the code is generating another self for what should be the resource being passed in (which appears to have type () and not &Self::Foo? that also seems incorrect).
The generated host function for foo::bar returns () when the closure implicitly returns Result<_>.

On an aside, it seems a little backwards to me that I would use witx_bindgen_wasmtime::import! to export these functions from a host to a Wasm module (which might use witx_bindgen_rust::import! itself to import these functions).
I initially expected to use witx_bindgen_wasmtime::export!, to be honest.

Heh I was actually about to fix this originally when I realized that resources barely worked at all, so I ended up getting distracted getting exported resources working entirely first.
My goal was to soon move on to nested resources like this and fix up all the issues here. I think there's probably quite a few issues lurking here, but I'm hoping it's not too too hard to fix.
I'd like to use method patterns where possible as well (for both imports and exports), so that will likely take some finagling to get that working.

On an aside, it seems a little backwards to me ...

Yeah I'm by no means good at naming things. The intention right now is that import! binds the imports of the wasm module, and export! binds the exports. I think your confusion is pretty reasonable though.
Overally I don't like the import/export terminology. In general I don't like the idea of splitting things, from the perspective of users, into import/export. Let me actually open an issue on this...

There's no rush on this.
I figured I'd update wasmtime-functions to latest Wasmtime and replace the wiggle-based host impl with witx-bindgen and the new witx syntax (especially as using resources to describe requests and responses is sooo much nicer than what it uses now) so that there would be more examples of using witx-bindgen out there.


Try to fix multi-gpu training issue in tumor-detection and vista3d

Fixes #685
Status
Ready/Work in progress/Hold
Please ensure all the checkboxes:

[x] Codeformat tests passed locally by running ./runtests.sh --codeformat.
[ ] In-line docstrings updated.
[ ] Update version and changelog in metadata.json if changing an existing bundle.
[ ] Please ensure the naming rules in config files meet our requirements (please refer to: CONTRIBUTING.md).
[ ] Ensure versions of packages such as monai, pytorch and numpy are correct in metadata.json.
[ ] Descriptions should be consistent with the content, such as eval_metrics of the provided weights and TorchScript modules.
[ ] Files larger than 25MB are excluded and replaced by providing download links in large_file.yml.
[ ] Avoid using path that contains personal information within config files (such as use /home/your_name/ for "bundle_root").

/build

/build

/build

/build


Add tizen_builder_test.dart and plugins_test.dart

Create a class PackageBuilder to enable testing TizenBuilder.buildTpk.
Add tests for tizen_builder.dart and build_targets/plugins.dart.

The total test coverage as of this PR is 39.8% (612 of 1538 lines).

It has nothing to do with this PR but .. I'm just curious...
Do you need a coverage measurement? :smile:

@bwikbs No requirement has been set regarding the test coverage. I just wanted to measure the current status before adding more tests.


Empty string is undefined

Hi,
in my application I set an initial data variable as empty string in my component. My next step is to get the final value from an async call.
Within this time from initialization of the data variable and the overwrite from the ajax call the value is undefined. I have tested it without the ajax call and it is independent of it. If I have defined my variable in the data function of my component as an empty string, it got undefined.
My current dirty solution is to use an string with an empty line in it, because I don´t want an 'undefined' text in my page title.
This very simple example produce an undefined page title:
module.exports = {
    data: function () {
        return {
            title: ''
        }
    }

    metaInfo: function () {
        return {
            title: this.title,
            titleTemplate: '%s | MySite'
        }
    }
}

I think this is the same as #32.

Should be fixed with #81

@Atinux  I'm sorry, but it still have this problem.
And I found the reason, here's the code.
   // backup the title chunk in case user wants access to it
    if (info.title) {
      info.titleChunk = info.title;
    }

    // replace title with populated template
    if (info.titleTemplate) {
      info.title = applyTemplate(component)(info.titleTemplate)(info.titleChunk);
    }
    ...
    var applyTemplate = function (component) { return function (template) { return function (chunk) { return typeof template === 'function' ? template.call(component, chunk) : template.replace(/%s/g, chunk); }; }; };

At line 514, info.title === '' is true, so info.titleChunk is undefined, but info.titleTemplate === '%s | MySite' still  true, so when template.replace(/%s/g, chunk), it will return undefined  | MySite.
And I create a PR, please fixed it or merge the PR, thanks for your work.

It would be awesome if the supplied PR can be merged, this issue is still present as described by IssaTan.


Use LLDB.framework install dir provided by cmake cache

Allow the lldb build to find the correct install dir for LLDB.framework
in a cmake cache. (The path currently hardcoded into build-script is not
correct for embedded builds.)
To do this, build-script needs to tell lldb where to install each
arch-specific LLDB.framework, so it can lipo all these together for a
fat build. build-script does already do this with CMAKE_INSTALL_PREFIX,
but LLDB.framework is special, because it doesn't always share an
install prefix with other lldb products.
Depends on: https://reviews.llvm.org/D72286

Sorry, this is not right. build-script does not include the arch-specific destdir in the framework install prefix to begin with.


still work?

does it still works?
I am still not getting it work :/
i get stuck here:
Message: API has being updated. If you see version 0.87.0, you are using the latest API. Else please do an update.  There are noted issues with the current API and subsequent bans and warnings may very well be issued and you take the risk into your own hands if you follow through.
Error occured in social handler _inventory was not initialized _inventory was not initialized
Traceback (most recent call last):   File "pokecli.py", line 979, in <module>     main()

I ran into the smae thing

try disabling killswitch in your config.json @ondrejlohnisky . That worked for me


AMBARI-23791. Agent is not notified about cluster delete in runtime.

What changes were proposed in this pull request?
Now server notify agents about updates in agent cached info (topology, host configs, metadata, host-level params, alert definitions) after cluster removal.
How was this patch tested?
Manual testing.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2283/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2286/
Test PASSed.


Added Margin & Padding & Font Size Stylus Classes

All the above said helpers can be changed by calling a stylus function underneath.

Actually this class is cleaner and more robust than the one I said in the issue but if you want that one I can make a PR for that one, too.

Thank you very much for the contribution

Your welcome! Do you want me to make a documentation for that?

It would be great and so we are adding more without problem

Amazing! I make a box in grid for these.

It would be great but better if it integrates with the grid!