Perform lease recovery for wasb filesystem

This patch passes a basic test of writing 1 k/v, killing accumulo, and restarting accumulo. I am not sure if it is correctly handling the case where acquireLease throws an exception.

One thing I'm not 100% sure about is the relationship of semantics from AzureBlobStore and HDFS. They both have these things we call "leases", but are their semantics the same?
I can see that for HBase, lease recovery is only done for directories set up for "atomic rename" (aka configured with Page Blobs) https://github.com/apache/hadoop/blob/branch-2.8/hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azure/AzureNativeFileSystemStore.java#L445-L459. However, it seems like this is just an implementation detail for HBase (not that lease recovery requires Page Blobs to be used). I base this finding on https://docs.microsoft.com/en-us/java/api/com.microsoft.azure.storage.blob._cloud_blob.acquirelease?view=azure-java-legacy not saying anything about page/block blobs.
From the presentation I put together the other week, I don't recall Accumulo having any WAL renames that would need to be atomic (in contrast to HBase which moves the WAL once a RS starts working on it).
The above aside: we do want to make sure that fencing the WALs still works for Accumulo, to prevent zombie'd Tservers from causing a ruckus. One thing I haven't been able to figure out is if the following scenario will work as we want it to:

Tserver is in half-dead state, not talking to Master, but is able to keep a lease with ABFS (renew'ed every 40s by default)
Master whacks that ZNode for the tserver to try to kill it (normally, a max of 60s, but maybe longer until we notice if things are really messed up), and starts reassigning things
Tserver doesn't yet observe ZK change, and is still renewing the lease with ABFS
A new TServer calls acquireLease() on this WAL which is still being renewed by the half-dead Tserver.

I can't figure out from docs/code what the expected outcome of this action is. Does it work like HDFS works (the TServer making the acquireLease() call "overriding" the old lease that the zombie tserver is holding)? Maybe you can find someone on the storage team at Azure to ask about the semantics of https://docs.microsoft.com/en-us/java/api/com.microsoft.azure.storage.blob._cloud_blob.acquirelease?view=azure-java-legacy. If not, maybe you can just write a test which simulates the above happening (makes sure that an old client who once held the lease, can no longer append to a file after another clietn called acquireLease).
Assuming the semantics for acquireLease in ABFS are the same as recoverLease in HDFS, I think your change is fine. I'm lamenting the "ugliness" of the if/elseif/else conditional block in LogCloser, but it's not the end of the world.

This change may require the hadoop-azure jar on the runtime classpath even when not using Azure.  The code  ns instanceof NativeAzureFileSystem in HadoopLogCloser may cause that class the be searched for on the classpath.
Maybe the following will avoid it.

Mark dep optional in pom
Create a new class called AzureLogCloser
Document how user can configure Accumulo to use the log closer.

With this approach the class AzureLogCloser will never be loaded unless configured.

Thanks for the reviews, everyone. I'll make these changes and find out if anything else needs to be done for lease recovery in the wasb case.

I did some additional testing and found that I could acquire a lease on a walog manually while the walog was active with the tserver. When the tserver tried to write to the walog while I had the lease, the write failed and the tserver started a new walog. But if I refrained from writing while I held the lease, the tserver was able to write to the walog I had leased after the lease expired.
This seems to mean the tserver is not acquiring a lease when writing to a walog (which might be good, because the wasb FS implementation does not provide a way to break an existing lease). I am wondering if we need to hold the lease longer after it is obtained in the LogCloser, to prevent a half-dead tserver from continuing to write to the walog, or if there would be a better way to handle this situation.

One possible course of actions is that when a tserver gets an error writing to a walog, it does a strong check (not using ZooCache) to Zookeeper to ensure it still holds it lock.  If we could detect lease loss on the tserver, the check could be done then.

I will likely abandon this particular change until one of the file system drivers supports lease operations. However, I am still thinking about how we could simulate a zombie tserver situation to validate that a filesystem (and our use of that filesystem) is behaving as we expect upon lease recovery. I could implement an AzureLogCloser that does nothing when close is called, and this would probably not cause issues for recovery on a system with tservers that die cleanly. Does anyone have ideas for things we could do to test out tserver failure scenarios?

Does anyone have ideas for things we could do to test out tserver failure scenarios?

For testing, it would be nice to be able to make a tserver not halt when it looses its lock in ZK.  The master would thinks its dead, but it would keep running.  I can think of three ways to accomplish this.
First, we could just make it configurable but this feels wrong.
Second we could write some code in accumulo-testing that uses byte buddy to create a java agent which mutates the behavior of the tablet server code.  I did something like this for nanoop.  Maybe there is an easier way to mutate the behavior of compiled java code.  This javaagent could be built in accumulo-testing and configured in accumulo-env.sh for a test.  This option is very complicated, but it allows testing of released binaries w/o having to rebuild.
Third we could maintain a patch somewhere that changes the behavior of Accumulo on lock loss.  For test you build Accumulo with this patch.  I think this option is the most straightforward.  For testing released versions, would need to get the source for that release and rebuild.


"virtual hosts"

Hi, found this through HN. Is there any plan to support "virtual" hosts the way HTTP servers do? This is a shortcoming of SSH that I've run into time after time, because SSH doesn't allow me to indicate what host I'm trying to connect to.
It seems like with the cert setup, the work is already half done, the biggest part of the work I'd imagine is just a way to specify multiple host names with multiple certs.
I'd be interested in contributing if this is something you'd like to see.

It should probably work with reverse proxies like nginx and traefik taking care of vhosts.

ah i didn't think of that. i'll give that a try and report back :)

Please also post here use case in which you want single instance ssh3-server to respond differently on different vhosts. Curiosity is killing me :)

Please also post here use case in which you want single instance ssh3-server to respond differently on different vhosts. Curiosity is killing me :)

just a thought: if H3 is able multiplexing, a simple website and a sshv3 could share the same H3 server, and by hostname it could decide which service is requested. or do i misunderstand the HTTP3 part?

Yes it is a really cool feature that we would love to provide out-of-the-box.
If one wants to mux it in an existing Go app, I am 80% sure that is can already easily be achieved by wrapping the ssh3 handler below in a public API function. Note that currently it is best to dedicate the entire H3/QUIC connection to the current SSH3 conversation as SSH3 currently hijacks every received H3 datagram because quic-go seems to only allow doing that.
Doing that in a more general setup behind a proxy might be achievable as well but it requires one change: currently, privkey auth currently sign TLS exporters that are entirely linked to the current TLS session and we need to change that to have privkey auth going through proxies. With privkey authentication, bearer tokens are not really bearer tokens...
I'll look at that once I publish the protocol specification, it is probably just a matter of time.

Curiosity is killing me :)

What about getting shell into the containers / pods / namespaces?

What about getting shell into the containers / pods / namespaces? Though, vhosts functionality is not the best way to do such things, maybe path or query parameters could be a better match. And vhosts may be better suited for proxing to several different hosts.

If I understand correctly - those will be different instances of ssh3-server in pods and reverse proxy is exactly what decides which vhost goes to which container. I don't see possibility single ssh3-server to be ssh to anywhere except host it is running on.

FYI there is now #89 that has work towards this :)

Also, you can already setup "vhosts" using nginx or haproxy for instance, but it currently has to be done without decrypting the QUIC connection, so it must be done based on the TLS SNI.
My answer to #97 explains how to do that with nginx (version >1.25.3), it will probably be similar with other servers/proxies that parse the QUIC SNI.


It should be possible for a window to get its size from its contents

Currently, it seems that a window determines its size based on the value passed to it by the user and then forces its contents to be laid out within that size.
However, it's often more useful and convenient to do it the other way around: The window should use the usual box layout methods on its content to determine the best size for itself within the usual constraints (which should be user-settable on the window).
Whether to do this, of course, should be an option settable on the window.
Furthermore, there should be separate options to control whether this size can grow and/or shrink in response to future layouts.
This way, it's possible to have a completely dynamically sized window, where showing/hiding UI (for example) inside the window will automatically grow/shrink the window as well.
(I know druid is WIP, but I like it and I didn't see a way to do this nor an existing issue on it, so I've though to open one)

This would be nice to have indeed, but I don't think our current (rather primitive) layout system could do this.
Would be interesting to know how Flutter deals with this.

How should we solve this for widgets that do not have a fixed size, for example the ones that take up any free space left. Should they be considered as having some default width and height (for calculating the window size)?

I think this isn't practical, for both theoretical and technical reasons.
Theoretical: for box constraints to work, you need to have some base constraints; if your initial constraints are (0, 0) and (inf, inf) it will be hard to actually produce a valid layout.
Practical: window managers impose constraints on the size windows can be. On some devices (mobile, e.g.) windows have a fixed size. In addition, when a window is restored on launch from a previous size, we would like to respect that, and use the existing size when computing the initial layout.

I definitely agree, I don't think it's reasonable to ask window size to react nicely as a root node in the layout system used for widgets. But it would be nice to be able to push-pull on this a bit more. Could we grow the ability to set the window size through a Ctx, and the ability to change min/max sizes?
I'm not familiar with the platform restrictions around this to know how feasible it might be.

yes, we could totally expose access to various window properties through contexts or commands. Some of this may already exist as part of #1235.

WindowSizePolicy::Content does this and resizes on every layout. It was made for the popups/dropdowns in the nursery.
I think it fulfills most of this but doesn't currently interact well with resizing.

I have the feeling that the current layout process is working in the wrong direction: it doesn't really make sense for a container to impose a minimum size on its children, and conversely there is no way for a child to convey its minimum size requirement to its parent. Here's a suggestion for a layout system that would allow window sizes to adjust to their contents:

Widgets have a method that returns their constraints. Constraints include at least a minimum size, and possibly a maximum and preferred size.
During the layout phase, constraints propagate bottom-up: containers ask their children for their constraints, add their own requirements and return the aggregated constraints.
During the paint phase, layout is applied top-down: containers are given a size by their parent, choose their children sizes taking into account their constraints and pass the chosen size to each child.
In answer to #1242, the paint box that is given to each widget should always respect the constraints that the widget expressed during the layout phase. However specialized containers may propagate relaxed constraints from what their children requested during layout and take appropriate actions to adjust the size when painting (eg. add scroll bars, add padding, crop or scale).

This way, top-level windows can know the minimum size of their children and adjust automatically to them.


Data corruption  and I/O errors with share on macOS Sequoia

BEFORE SUBMITTING YOUR ISSUE, PLEASE LOOK AT THE PINNED ISSUES AND USE THE SEARCH FUNCTION TO MAKE SURE IT IS NOT ALREADY REPORTED. ALWAYS COMMENT ON AN EXISTING ISSUE INSTEAD OF MAKING A NEW ONE.
Describe the issue
I've been using an Intel Kali VM for some time with UTM with acceptable performance and no issues.  After upgrading to macOS 15, the share between my Mac and the Kali VM began having data corruption issues.  Things work for a short time, then files can't be written to the share from the Kali side.  Occasionally, an I/O error is thrown.
Configuration

UTM Version:  4.5.4
macOS Version:  15.0.1
Mac Chip (Intel, M1, ...): M3 Max

Crash log
If the app crashed, you need a crash log. To get your crash log, open Console.app, go to Crash Reports, and find the latest entry for either UTM, QEMU, QEMUHelper, or qemu-*. Right click and choose Reveal in Finder. Attach the report here.
Debug log
For all issues related to running a VM, including crashes, you should attach a debug log. (This is unavailable for macOS-on-macOS VMs. Attach an excerpt of your system log instead.)
To get the Debug log: open UTM, and open the settings for the VM you wish to launch. Near the top of the QEMU page is Debug Log. Turn it on and save the VM. After you experience the issue, open the VM settings again and select Export Log... and attach it here.
Upload VM
If your issue is related to a specific VM, please upload the config.plist from your VM's .utm directory. To get this, right-click the VM in UTM and choose "Show in Finder". Then right-click the selected file in Finder and choose "Show Package Contents". The config.plist file is now visible. Right-click it and choose "Compress". Attach the resulting config.plist.zip file here.
You can upload the entire .utm if needed, but note this includes your VM's drive image and may contain personal data. Since Github has an attachment size limit, you may want to upload to another service such as Google Drive. Link it here.

#me too!

Please upgrade to 15.1 and try again. There was a known bug in macOS which broke I/O.


Android - Attachment - Persisting loading indicator on sending two attachments continuously

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open any chat
Add an attachment
without waiting for it to finish uploading add another attachment.
(try with different file size if unable to reproduce.)

Expected Result:
Loading indicator should not be shown when upload is finished
Actual Result:
Loading indicator is shown on first attachment.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: n/a
Reproducible in production?: n/a
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/27a13f77-c566-4cda-b368-f0740db24191
Expensify/Expensify Issue URL:
Issue reported by: @kishanknows
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685379914808459
View all open jobs on GitHub

I couldn't reproduce this but its likely down to me using Browsterstack to test the native android app. Checking if anyone with an android phone can assist.

I attempted to on a Pixel 6 Pro and am unable to reproduce this behavior either.

Requested better reproduction steps in the slack thread, not able to reproduce as of yet.


Cannot set minimap background

monaco-editor version: 0.20.0
Browser: any browser (tested with FF, Chrome + Safari)
OS: macOS
Playground code that reproduces the issue:
Go to the playground page: https://microsoft.github.io/monaco-editor/playground.html#customizing-the-appearence-exposed-colors.
Change the theme definition code to:
monaco.editor.defineTheme('myTheme', {
    base: 'vs',
    inherit: true,
    rules: [{ background: 'EDF9FA' }],
    colors: {
        'editor.foreground': '#000000',
        'editor.background': '#EDF9FA',
        'editorCursor.foreground': '#8B0000',
        'editor.lineHighlightBackground': '#0000FF20',
        'editorLineNumber.foreground': '#008800',
        'editor.selectionBackground': '#88000030',
        'editor.inactiveSelectionBackground': '#88000015',
        "minimap.background": "#FF0000",
        "minimap.selectionHighlight": "#00FF00",
    }
});

As you can see I added two minimap settings. The selection highlight works as expected, while the minimap background color does not change. The same setting works in vscode, so I assume something went wrong when extracting Monaco from there:

I just found this patch from @alexdima https://github.com/microsoft/vscode/commit/3e500bb935ff3f5254a40aeca511ba0f71fd6b01, which adds support for this background to vscode. Inspecting the sources of Monaco shows that this has not been taken over, even though this patch was added 6 months ago.

There hasn't been a new version since February (see #2074). So I guess you could try this again whenever a new version is released that includes that patch. I don't know about any workarounds in the meantime unfortunately, but I also never used the themeing support myself 😅

Let's track via https://github.com/microsoft/monaco-editor/issues/2074


In enhancer.nalr.build, make parameter cfs as None by default

Matlab version of NALR has 3 parameters, HL, nfir, fs
function [nalr,delay]=eb_NALR(HL,nfir,fsamp)

In Clarity, NALR is implemented as a Class in enhancer.nalr.
class NALR:
    def __init__(self, nfir, fs):
        [...]
    def hl_interp(self, hl, cfs):
        [...]
    def build(self, HL, cfs):
        HL = self.hl_interp(np.array(HL), np.array(cfs))
        [...]

For the cases when cfs is not available and to match MATLAB's use of HAAQI and HASQI, it is needed to make cfs=None and add the if conditional
def build(self, HL, cfs=None):
    if not cfs is None:
         HL = self.hl_interp(np.array(HL), np.array(cfs))
    [...]

Instead of
def build(self, HL, cfs=None):
    if not cfs is None:
         HL = self.hl_interp(np.array(HL), np.array(cfs))
    [...]

it will be changed as
def build(self, HL, cfs=None):
    if cfs is None:
         cfs = [250, 500, 1000, 2000, 4000, 6000]
    [...]

@TuZehai Does NALR CLass need to use the custom firwin2?
At this point, shouldn't be better to adopt to Scipy implementation?

I can not quite remember the detail now... I imagine the difference should be very minor?

I will PR this minor change first as I need it to continue migrating HASQI.
Then we can come back and check the fir2 and firwin2


enhance(pods): configure export pod v2  to create new service and pod configs.

This PR adds an option to create a new service connection and export pod v2 config. Apart from this, it adds a few fixes that were found relevant while testing export pod v2 in general.

madge report:

master: 65 circular dependencies
this branch: 65 circular dependencies

Testing

[x] Write Tests
[x] Confirm existing tests pass
[x] Confirm manual testing
[ ] If your tests changes an existing snapshot, make sure that snapshots are updated
[ ] If you are adding a new language feature (graphically visible in VS Code/preview/publishing), make sure that it is included in the test workspace

Docs

[ ] if your change reflects documentation changes, also submit a PR to dendron-site and mention the doc PR link in your current PR

Analytics

[ ] if you are adding analytics related changes, make sure the Telemetry docs are updated

are there docs for this change?

are there docs for this change?

Part of https://github.com/dendronhq/dendron-site/pull/363/files?show-viewed-files=true&file-filters[]=


Fixed bug where field was not rerendering when sync error changed

Fixes #1024.
Moved getSyncError() from ConnectedField to Field.

One of these days I will learn to run eslint before I create the PR. 😧

Published in v6.0.0-rc.1.


add scale and center to model perturb examples

Earth-2 MIP Pull Request
Description
This PR adds a file that was not committed in PR77 and closes issue #84 by applying source to both dimensional and non dimensional state vectors
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The CHANGELOG.md is up to date with these changes.
[x] An issue is linked to this pull request.

Dependencies

\blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

Also gives this error:
Traceback (most recent call last):
  File "/home/nbrenowitz/workspace/earth2mip/examples/workflows/run_ensemble_model_noise.py", line 93, in <module>
    main()
  File "/home/nbrenowitz/workspace/earth2mip/examples/workflows/run_ensemble_model_noise.py", line 89, in main
    run_inference(model, config, perturb, group)
  File "/home/nbrenowitz/workspace/earth2mip/earth2mip/inference_ensemble.py", line 382, in run_inference
    run_ensembles(
  File "/home/nbrenowitz/workspace/earth2mip/earth2mip/inference_ensemble.py", line 131, in run_ensembles
    for k, (time, data, _) in enumerate(iterator):
  File "/usr/local/lib/python3.10/dist-packages/tqdm/std.py", line 1182, in __iter__
    for obj in iterable:
  File "/home/nbrenowitz/workspace/earth2mip/earth2mip/networks/__init__.py", line 210, in __call__
    yield from self._iterate(x=x, time=time)
  File "/home/nbrenowitz/workspace/earth2mip/earth2mip/networks/__init__.py", line 238, in _iterate
    x += self.source(x_with_units, time) / self.scale * dt
TypeError: generate_model_noise_correlated() got an unexpected keyword argument 'model'
nbrenowitz@fb7510c-lcedt:~/workspace/earth2mip$

/blossom-ci


Use plan instead of dry-run.

Terraform users will be familiar with the terraform syntax and terminology, so we should use plan instead of dry-run.  https://www.terraform.io/docs/commands/plan.html
If there are any other cases where the terraform terminology makes sense to use it, we should do that.

It's a breaking change, but the plugin is documented only since yesterday.
LGTM

LGTM


Fix Parser of FESTWERTEBLOCK to support multiline WERT

Hello,
I have a DCM file where in one of the 1D "FESTWERTBLOCK"s there are several "WERT" lines:
FESTWERTEBLOCK DATA_XXXXXX 7
   LANGNAME "Value to select ....."
   DISPLAYNAME XXXXXX
   FUNKTION XXXXX
   EINHEIT_W "-"
   WERT   0.0000000000000000   0.0000000000000000   0.0000000000000000   0.0000000000000000   0.0000000000000000   0.0000000000000000
   WERT   0.0000000000000000
END

I have modified the parser to support such values.

I think it is already supported but it seems that your DCM is not according the specification. If you have a multiple rows I think the FESTWERTEBLOCK has to be defined as a matrix...

Thank you for your feedback.
The file I‘m using has been generated with the latest version of INCA. I don’t think that the file is in a wrong format.


Fix for SPR 9096

Hey @rstoyanchev, I believe this fixes SPR-9096. Please merge if it works for you.

I've made a couple of updates and committed the results in 3.1.x. Please, remove your SPR-9096 branch. I rebase the changes on top of the latest in 3.1.x which had a few commits.


ENH: sparse: Generalize coo_array to n dimensions

This PR rewrites scipy.sparse.coo_array in a way that theoretically could support arbitrary n-dimensional shapes. In practice, coo_array will only fully support 1d and 2d shapes. The coo_matrix subclass will preserve its existing 2d-only semantics.
The main transformation is replacing the individual .row and .col attribute arrays with a single .indices attribute that holds a tuple of index arrays. We maintain the invariant that len(x.shape) == len(x.indices) == x.ndim.
Support checklist:

[x] constructors (shape, dense, sparse, tuple)
[x] reshape
[x] nnz
[x] transpose (including axes=(...) keyword)
[x] resize
[x] tocsc / tocsr / todia

these probably only make sense for the 2d case, so we raise an exception
as of now todok also raises, but we could easily add n-d support there too

[x] tocoo
[x] diagonal and _setdiag

raises an error for non-2d cases

[x] _with_data
[x] sum_duplicates
[x] eliminate_zeros
[x] toarray (1d and 2d only)
[x] _add_dense (1d and 2d only)
[x] _mul_vector (1d and 2d only)
[x] _mul_multivector (1d and 2d only)

Note: the last 4 items on the checklist are routing 1d array data through the 2d native code paths, which requires the creation of a dummy col array containing all zeros. If we want to squeeze the last bit of performance out of these code paths, we'll want to implement specialized C++ overloads. That's out of scope for this PR, though.

Interesting. Are you discussion the scope of changes or some sort of roadmap/plan in Seattle @perimosocordiae? I'd expect general n-D support for scipy.sparse to be infeasible, and pure Python changes for COO only to not be competitive with PyData Sparse (which already has an n-D COO class with a better, numpy-compatible API than scipy.sparse has).

We're planning to put together a report of the various scipy.sparse work we're doing here in Seattle (including a roadmap for future work) hopefully by the end of day tomorrow.
This PR is a bit exploratory, to see how feasible basic n-D support would be. My initial goal is to get 1-d COO arrays working reasonably well, and then take a look at other formats. With the ongoing change to use array semantics, users will start asking for sparse 1-d arrays (as the result of foo[:, 3] for example), and I don't want to tell them that they have to stop using scipy.sparse entirely.

It's not exactly a roadmap, but the final report from Seattle is available here: https://hackmd.io/1Q2832LDR_2Uv_-cV-wnYg

Thanks for sharing - looks like that was a productive sprint!
My main points of feedback on the "what remains to be done" are:

It seems to me like the key goal of the introduction of sparse array classes is to get rid of the need for np.matrix, so it can be removed from numpy and we get rid of * working like @. That is likely to involve deprecation and removal of *_matrix classes - but that goal and the plan for deprecation is left completely implicit. It'd be great to make that explicit.
n-D support for scipy.sparse does not seem feasible (given the state of the C++ code) nor the right plan (accelerating PyData Sparse development seems much preferred).

@rgommers I completely agree on (1). Would you like to see the roadmap updated here? https://docs.scipy.org/doc/scipy/dev/roadmap-detailed.html#sparse

I'd suggest opening a new RFC issue with the plan. This may be the most impactful deprecation we've ever done, and it's unclear if we should go ahead with that and on what timescale. It requires discussion with scikit-learn and other major users. Scikit-learn is particularly conservative, and their minimum SciPy version right now is 1.5.0: https://github.com/scikit-learn/scikit-learn/blob/main/sklearn/_min_dependencies.py#L17. So it'd be great to work out if and when *_matrix classes can be deprecated, how long they should stick around after that, and then also what can be done with np.matrix.

@dschult @stefanv @rossbar @ivirshup @jjerphan This should now be ready for review!

As discussed in the meeting, we should come up with a list of things that need implementing and figure out which are being pushed to follow up PRs. Features include:

[ ] more testing
[ ] toarray() that doesn't go through the current native code path
[ ] indexing

[ ] For coo arrays
[ ] So other classes (e.g. CSC, CSR) return 1d coo

[ ] broadcasting for math ops

I've got a rudimentary version of TestCommon1d. Meaning it does things like "add two sparse arrays" or "multiply by scalars". The tests show that coo 1d arrays are not yet able to do some arithmetic operations, e.g. the _spbase class handles _add_sparse and uses .tocsr() which raises for 1D arrays.
I have thought about this occasionally over the past two weeks. Rather than overriding _add_sparse in the _coobase class, or special casing 1D in tocsr, I would prefer we merge this PR knowing that it doesn't provide a full suite of 1D functionality.  The next steps toward more functionality might be to get _base.py and _csr.py working with 1d. Luckily, in 1D, the csr and coo formats are very similar. indptr in 1D becomes indptr == [0 self.nnz] and the indices have to be (un)wrapped in a tuple in the conversion. So, I think it should be fairly straightforward to move from the coo format to the csr format for 1D.
This PR provides a nice coo format with 1d support including matmul but not addition and a good foundation for an eventual nd.  Following this we will have a separate PR that allows 1D in csr, csc classes (and base class support). Thus at each PR's merge, the main repo supports all existing 2d operations and incrementally adds 1D capabilities.
Thoughts?

I've made a PR to this PR's branch with the following:
Some misc fixes of remaining errors for coo-1d.

Mypy error on dtype not being defined in _spbase.
We could set it to something expecting it to be overwritten by the subclass.
But self.dtype.type appeared before this PR. Turns out adding -> str typing in the function signature line caused more strict checking by mypy. I removed  the -> str thinking we can fix the mypy error on dtype in a different PR. If you prefer we can define it to be a vaccuous value that will be overwritten. I think with typing you can also set the type without assigning a value, but that choice is yours to make. :)

Mypy error on max_size = np.prod(shape) because the result might not be an integer. I changed it to manually multiplying the values (which is 100 times faster anyway, but is very fast for both).

den.resize(arg) was raising due to references of this array not allowing resize. So I added refcheck=False which was suggested by the test message. Also the reference is what we are creating: res and we know it isn't going to cause problems.  [This occurs in 3 tests of resize within test_coo.py.]

test error complaining about intc not equaling int32 when we used type(res) == type(exp). That idiom is used in a few other tests to check that they are both ndarrays. But in the 1d_mul_vector case, the result is a scalar. So the type becomes the dtype of the original arrays, and that can be intc for scipy and int32 for numpy.  I don't think we need to test the type of the results. We should check it is a scalar and then check that the values are equal. Checking for scalar is suggested to be done (in the np.isscalar documentation) by testing that res.ndim == 0. So I replace the existing test with that test. The next line checks that res equals exp` so they both must be scalars.

errors due to bounds exceeded within np.revel_multi_index inside _coo_base.reshape(). This had a TODO comment, that took me way too long to notice, pointing to a PR where the code to handle reshaping accounts for this limitation on revel_multi_index. The 1d case was actually easy to handle (index is already flat). So I added code to mimic what was there before (with indices instead of row and col). It is passing tests and hopefully OK.

I removed white space on two empty lines in test_coo.py.

I think this could fix the failed tests in Linux Meson, Windows and Mypy. That should give us a green check if all goes well. Finger's crossed.

Great!  We've got the green check!
Is there anything else needed for this 1d coo_array PR? (maybe change the title to state 1d instead of nd)
The questions about doc_strings being completed by other software should probably be settled elsewhere. (I'd prefer it to be in the doc_string itself, so people reading the code can see it without going to a browser, but I think that's not relevant here.)
There are still 1d-array features (like indexing, arithmetic operations, etc) that are not implemented, but probably need a csr version of 1d (which is in the works...)

Is there anything else needed for this 1d coo_array PR? (maybe change the title to state 1d instead of nd)

I think this PR should be merged after rewording nd for 1d.
What do you think? 🙂

Ok, all the nd wording is now replaced with 1d, and the code itself checks that shapes are either 1d or 2d.
Assuming that CI is green, I'd like to get this merged soon to maximize the time in which we can shake out any unforeseen issues before the 1.13 release.
Many many thanks to @dschult, @stefanv, @jjerphan, @ivirshup, and all the other contributors who have been so instrumental in making this PR happen!

FYI, the introduction of coo_array created a regression in scikit-learn.
See https://github.com/scikit-learn/scikit-learn/pull/28047/.

To give a bit more context, the change of behaviour happens when creating sparse arrays from a 1d array-like object. This was possible before this PR and would create a sparse array with a single row. T
With this PR the following code
from scipy import sparse
sparse.csr_array([1., 2., 3.])

now yields an error:
ValueError: Cannot convert a 1d sparse array to csr format

We were able to adapt the scikit-learn tests without too much work, but this does feel like a breaking change for sparse arrays. I'll leave it to scipy maintainers to reflect about the trade-off between this breaking change and the functionality allowed by this PR.
Here is a summary  the things that broke in scikit-learn tests:

sparse arrays created from a 1d array-like used to be fine and create a 2d sparse container with a single row, e.g. csr_array([1., 2., 3.]). After this PR you get an error. To be honest, there did not seem to be a particularly good reason why we were creating sparse containers from 1d array-like, other than it works and creates the data we want ...
on the same topic coo_array([1., 2., 3.]) is 1d after this PR and used to be 2d, this could affect some downstream code, although scikit-learn tests were not affected by this (we probably don't use coo format that much I guess)
the last change is a bit more edge-casy and is probably less of an issue. In our tests we create sparse arrays with 64 bit indices (mostly to check that they are supported in some scikit-learn code), we were doing things like:array = coo_arrray([1., 2.])
array.row = array.row.astype('int64')
array.col = array.col.astype('int64')

After this PR the source of truth seems to be .indices and we now do:array.indices = tuple(each.astype('int64') for each in array.indices)

Creating a sparse array from a 1d array-like will now try to create a 1d array.
At the moment, coo is the only allowed format, though we should soon have dok and probably csr formats be possible.  Before the next release this creation statement will construct a 1d-array instead of raising an error.
But... it is definitely going to mix up existing code that 1d array-like input creates 2d sparse matrix but a 1d sparse array. We will need to make that clear in any transition document because it is a desired change.  I suppose we could raise an error for that code with sparse matrices (no creating 2d matrix with 1d array-like input). But that seems extreme.
The other important feature this issue brings to light is that A.row for a 1d array returns np.zeros_like(self.col) so it has the dtype of A.col.  Thus code that used to convert the types of the index arrays by converting like this:
A.row.astype(new_dtype)
A.col.astype(new_dtype)

won't change the type of the rows -- because the A.row gets created as an array of zeros with dtype of A.col. By the time we change the type of A.col it is too late to change the type of the already created A.row.  It would work to reverse the order:
A.col.astype(new_dtype)
A.row.astype(new_dtype)

This works!!  because the col gets changed before row gets created.
I think it is probably better to reduce surprises by raising on A.row when A is 1d.  There are no row indices -- and while we can construct some to make it "like" a 2d array with rows and columns, it really isn't. Returning an array of zeros constructed just for the row property does not allow interactive adaptation of the value of the row indices.
Proposal:

make A.row raise an exception when A is 1d.
make a lot of noise for the transition about making sure that array-like input is 2d when you want 2d output.

The second could be facilitated by changing spmatrix to raise on 1d array-like input. But I don't propose that.

OK thanks for the detailed answer, this helps a lot to understand the general direction.
When support to 1d csr/csc is added in scipy development version, we will probably want to test thoroughly what happens in scikit-learn. I would guess that some of our code assumes issparse implies 2d and that feeding a 1d sparse may create some not super  user-friendly error.
I agree that making spmatrix raise an error when constructed from a 1d array, does not seem desirable.


Sqitch: Updated prod to state of dev

Added the new database migration updates in dev to prod
with sqitch.
Renamed blogpost table to blog.
Change-Id: I2bbb00004da1ff424524e33188f9eda1cb4c0d31

Code Review comments can be found at https://gerrithub.io/q/I2bbb00004da1ff424524e33188f9eda1cb4c0d31


Installation from pip fails - no module widgets

I have a python3 virtual environment in which I first installed PyQt5  and then pyFlightAnalysis - see env below:
(mypy3env) ubuntu@ubuntu:~/github/pyFlightAnalysis$ pip list
numpy (1.14.3)
pip (8.1.1)
pkg-resources (0.0.0)
pyFlightAnalysis (1.0.1b1)
PyOpenGL (3.1.0)
PyQt5 (5.10.1)
pyqtgraph (0.10.0)
pyulog (0.6.0)
setuptools (20.7.0)
sip (4.19.8)

When I try run analysis I get error:
ubuntu@ubuntu:~/github/pyFlightAnalysis$ analysis
Traceback (most recent call last):
  File "/home/ubuntu/github/pyFlightAnalysis/mypy3env/bin/analysis", line 7, in <module>
    from pyflightanalysis.analysis import main
  File "/home/ubuntu/github/pyFlightAnalysis/mypy3env/lib/python3.5/site-packages/pyflightanalysis/analysis.py", line 16, in <module>
    from widgets import QuadrotorWin
ImportError: No module named 'widgets'

Similarly, building from source fails - as you would expect because dependency is missing. See #5

run command in your virtual environment

ls mypy3env/lib/python3.5/site-packages/pyflightanalysis/

check whether  widgets.py  exists along with analysis.py

Yes it is present:
(mypy3env) ubuntu@ubuntu:~/github/pyFlightAnalysis$ ls mypy3env/lib/python3.5/site-packages/pyflightanalysis/
analysis.py  icons  __init__.py  models  objloader.py  __pycache__  widgets.py
(mypy3env) ubuntu@ubuntu:~/github/pyFlightAnalysis$

I fixed the problem. It's due the module name will change when run a module as a file. Details see here.
Reinstalling it from pypi will fix this issue.


Augeas didn't load /etc/rsyslog.conf with Rsyslog.lns from /var/lib/puppet/lib/augeas/lenses: Get did not match entire input (line:40, character:0)

I am getting errors on ubuntu 14.04 LTS trying to edit rsyslog...
augtool> ls /augeas/files/etc/rsyslog.conf/error/
pos = 966
line = 40
char = 0
lens = /usr/share/augeas/lenses/dist/rsyslog.aug:45.10-.57:
message = Get did not match entire input

The line that it's choking on is this:
*.* @@<IP_ADDRESS>:1514;RSYSLOG_FileFormat

With context:
# Filter duplicated messages
$RepeatedMsgReduction on
*.* @@<IP_ADDRESS>:1514;RSYSLOG_FileFormat

Any ideas? To my knowledge, this is a package default file.
EDIT: This line is not stock, it's added for according to Alertlogic's documentation, which lists exactly that line.

Thank you for the report. I can confirm that the current Augeas lens cannot parse this line.
It can however parse:
*.* @@<IP_ADDRESS>:1514

so the problem seems to be the ;RSYSLOG_FileFormat bit.

Please open an issue against the Augeas project for this. I personnaly don't know rsyslog templates, so I'm not sure what this ;RSYSLOG_FileFormat bit does and how it should be parsed.


The UI could be improved

The following things could be improved:

Color scheme
Font
Layout
Can I work on these?

which ui that you want to improve like of index page?

@jayk-gupta currently the framework for CSS is Bootstrap. You can refer to Bootstrap documentation for reference and if you have any questions related to it you can ask in our discord's 'ask-doubts' channel.
Let us know if you are still working on the issue so we can keep it open otherwise we'll close it. @akshitagupta15june


docplex: Error while placing 'if' condition inside the constraint

Hi,
I require a help.
I am getting an error when trying to place an if condition inside this constraint:
mdl.add_constraints(mdl.sum(x[i,j] for i in m if i,j in A)>=1 for j in n)

Error:
 File "<ipython-input-53-94c3e0ea9065>", line 5
    mdl.add_constraints(mdl.sum(x[i,j] for i in m if i,j in A)>=1 for j in n )
                               ^
SyntaxError: Generator expression must be parenthesized

Tried several times but couldn't resolve this.
For this model, the variables were defined as:
m=[i for i in range(1,8)]
n=[i for i in range(1,11)]
A=[(1, 1),(1, 3),(2, 1),(2, 6),(2, 8),(3, 2),(3, 5),(3, 7),(4, 1),(4, 8),
   (4, 9),(5, 2),(5, 4),(5, 7),(6, 1),(6, 3),(6, 5),(6, 10),(7, 4),(7, 5),(7, 6),(7, 9)]

and the decision variables are:
x=mdl.binary_var_dict(A,name='x')

It would be of great help if you could provide a solution to appropriately construct this constraint.
Many THANKS.

This is a pure python issue. In your generator expression, if i,j in A is malformed.
A is a list of tuples, so you want i,j to be a tuple. The right expression is: if (i,j) in A
Note: if all you do on 'm' and 'n' is iterating over the list, your code will be more efficient if you just write:
m= range(1,8)
n=range(1,11))

Likewise, unless you need A as alist, if all you do is check that tuples are in A, use a set.

This is a pure python issue. In your generator expression, if i,j in A is malformed.
A is a list of tuples, so you want i,j to be a tuple. The right expression is: if (i,j) in A
Note: if all you do on 'm' and 'n' is iterating over the list, your code will be more efficient if you just write:
m= range(1,8)
n=range(1,11))

Likewise, unless you need A as a list, if all you do is check that tuples are in A, use a set.

Thanks for the help.


dp.change not firing after a dp.error (2nd and subsequent times) when you correct the invalid date using keyboard

I am using version  4.17.42.
Follow this process to see the bug.
Using the picker with keepInvalid = true, enter a valid date, say '2016-05-26'.
Now do the following 2 things.

Using the keyboard insert an 'a' character somewhere in the date text to make it invalid. A dp.error event is raised. Anything can be used here.
Now use the keyboard to remove this invalid character and make date valid again. A dp.change event is raised.

But now repeat steps 1. and 2., the 'dp.error' event will happen in step 1, but no 'dp.change' event will happen in step 2. If you were to correct the date in step 2 using the picker, the 'dp.change' event would be raised.
A fiddle (my first one... hope I have this set up right :) to show this is here:  http://jsfiddle.net/DavidHanlon/jd80j4hc/
I am just logging the dp.change and dp.error events to the Chrome console.
I am using the picker with knockout and knockout validation. Following the steps above leaves my model value out of sync with the text box value, and the knockout validation error does not clear after making the correction because the binding handler is not aware of the updated value in the text box.

Same issue here!
Did you managed to work around it someway?

We have a workaround, but it isn't very satisfying.
We set keepInvalid=false so that the picker will automatically correct an invalid typed in date. We updated the knockout binding handler to detect when this happens and set an observable subproperty when it does.  Using this subproperty we can display a warning message to the end user to alert them that the value they had just entered was invalid and adjusted. Pretty lame but the best we can do.


Consistently label spaces in filenames

Summary

Here's an example tree from one of the tests, reduced to just include the basics:
sub-PNC/
    anat/
        sub-PNC_desc-preproc_T1w.nii.gz
        sub-PNC_from-T1wNative_to-T1wACPC_mode-image_xfm.mat
        sub-PNC_from-T1wACPC_to-T1wNative_mode-image_xfm.mat
        sub-PNC_from-T1w_to-MNI152NLin2009cAsym_mode-image_xfm.h5
        sub-PNC_from-MNI152NLin2009cAsym_to-T1w_mode-image_xfm.h5
    dwi/
        sub-PNC_acq-realistic_space-T1w_desc-preproc_dwi.nii.gz
        sub-PNC_acq-realistic_space-T1w_dwiref.nii.gz

We have transforms from T1wNative to T1wACPC, but no images in either space. Instead, space-T1w is used as a standin for T1wACPC in the DWI data, and there's no space in the anatomical data. I think we should consistently think of having no space entity as being in space-orig or space-native. For example, if you have a T1w and a T2w, the sub-X_desc-preproc_T1w.nii.gz and sub-X_desc-preproc_T2w.nii.gz would be assumed not to be in the same space.
Here's what the updated tree might look like:
sub-PNC/
    anat/
        sub-PNC_desc-preproc_T1w.nii.gz  # T1w in the native/original T1w space
        sub-PNC_desc-preproc_T2w.nii.gz  # T2w in the native/original T2w space. NOT T1w space!
        sub-PNC_space-T1wACPC_desc-preproc_T1w.nii.gz
        sub-PNC_space-T1wACPC_desc-preproc_T2w.nii.gz
        sub-PNC_from-T1w_to-T1wACPC_mode-image_xfm.mat
        sub-PNC_from-T1wACPC_to-T1w_mode-image_xfm.mat
        sub-PNC_from-T1wACPC_to-MNI152NLin2009cAsym_mode-image_xfm.h5
        sub-PNC_from-MNI152NLin2009cAsym_to-T1wACPC_mode-image_xfm.h5
    dwi/
        sub-PNC_acq-realistic_space-T1wACPC_desc-preproc_dwi.nii.gz
        sub-PNC_acq-realistic_space-T1wACPC_dwiref.nii.gz

I think we should consistently think of having no space entity as being in space-orig or space-native.

(BIDS spec agrees)[https://bids-specification.readthedocs.io/en/stable/appendices/coordinate-systems.html]:

The scanner coordinate system is implicit and assumed by default if the derivative filename does not define any space-. Please note that space-scanner SHOULD NOT be used, it is mentioned in this specification to make its existence explicit.

I agree that no space is inherently space-orig.


Replace all COMPILER_MSVC to _MSC_VER and _WIN32 accordingly

-D compile flag should be avoided as much as possible. Many Google projects like Chromium follow  (which uses CMake or GN) only define macro in header file like below and always include the header file to get the macro (IWYU).
#ifdef _MSC_VER
#define COMPILER_MSVC
#endif

However, such rule is not easy to implement for Tensorflow as Bazel require strict header dependency. It is easier to just use _MSC_VER and _WIN32 directly.
Other stuff:

Remove ARRAYSIZE macro in tensorflow/stream_executor/platform/port.h (only used in tensorflow/stream_executor/cuda/cuda_driver.cc, which is now replaced with TF_ARRAYSIZE from tensorflow/stream_executor/platform/macros.h).

@mrry I fixed one more ARRAYSIZE -> TF_ARRAYSIZE. PTAL and re-trigger CI test.

@tensorflow-jenkins test this please

@meteorcloudy For some reason, I got the email notification that you pinged @tensorflow-jenkins for CI test, but your comment is missing in Github web and the CI test is not triggered.

That's right.. I deleted the comment because I just realized TF is not using Jenkins to do presubmit anymore, I don't know how to trigger a retest. @mrry Can you do that?

@meteorcloudy Sure: for future reference, you now have to apply the "kokoro:force-run" label to the PR to kick off the tests.

@mrry Got it, thanks!

Last commit fixes the remaining failing CI tests ("GPU CC", "GPU Python3" and "XLA"), caused by including non-existing header file. Sorry for the spam.


feat: remove debug log otlp events

When ingesting OTLP events we are logging a message at debug level with the whole event.
On local benchmarks this is using ~50% cpu time and ~8% memory (space), besides making the logs harder to read (the message is quite big).
This was here since the first commit so I'm not sure if I'm missing something but I don't think we should log the whole event.

On local benchmarks this is using ~50% cpu time and ~8% memory (space), besides making the logs harder to read (the message is quite big).

This is only the case with debug-level logging enabled, right? Why are benchmarks being run with that configuration?

This was here since the first commit so I'm not sure if I'm missing something but I don't think we should log the whole event.

Certainly not by default (i.e. at info level), but why not at debug level?
This logging has been useful in the past to diagnose OTLP translation issues.

This is only the case with debug-level logging enabled, right? Why are benchmarks being run with that configuration?

Yes, that's correct. I'm running the benchmarks against apm-server tilt setup which has debug logging enabled.

Certainly not by default (i.e. at info level), but why not at debug level?

I guess I'm comparing this to the intake/rum ingestion process. AFAIK we are only doing this for otel and it's surprising behaviour.
I do have a few concerns regarding potential leaks/side effects, for example wouldn't this make any processor redacting PII or securty keys, headers, etc. pointless because the data is logged as soon as it is received by the otlp processor ?

This is only the case with debug-level logging enabled, right? Why are benchmarks being run with that configuration?

Yes, that's correct. I'm running the benchmarks against apm-server tilt setup which has debug logging enabled.

OK, but we also have the option of disabling that, so the points in the PR description are moot.

Certainly not by default (i.e. at info level), but why not at debug level?

I guess I'm comparing this to the intake/rum ingestion process. AFAIK we are only doing this for otel and it's surprising behaviour.

Fair point.

I do have a few concerns regarding potential leaks/side effects, for example wouldn't this make any processor redacting PII or securty keys, headers, etc. pointless because the data is logged as soon as it is received by the otlp processor ?

I wouldn't say it's pointless, because again, this is only for debug-level. This requires action from the user.
Nevertheless, I agree that it would be safer to not give the option, and instead direct users to direct users to OTel SDK logging and/or using opentelemetry-collector in between with a logging exporter. Before removing the code, would you please identify suitable alternatives to point users at? Maybe for now we just add some links somewhere in dev_docs - later on we should probably update the APM guide's troubleshooting section.


Various gradle headaches

Sorry this is a bit vague, but it's intentionally so....
It seems like every month or so, gradle starts throwing off some new and complicated looking error. Often seems to be related to the version of flutter, android SDK, gradle, google services, kotlin etc etc, and/or the mismatch between versions.
It just feels like an under-polished area of the flutter experience.

Can you post your error message here?
Also please add the output of flutter doctor

I got it working after 4 hours or so of piecing together tips from various google searches. This is a general comment, not a specific issue - simply to say that the gradle build process seems fragile when upgrading various element of the app/framework/libraries etc.
I would say over the last year, I've had around 10 different error messages related to this overall issue. One suggestion would be to include "build tips" with each flutter release, so people can make sure all their versions and settings are up-to-date.

Could you please still provide us with more information so that we can improve it?
What did you change?
What problems did you have.
As a general Tipp if sich problems occur after an update, create an empty project and copy your files into it.

Sounds good. I will make a list of everything I can remember.

Similar class of issue: https://github.com/flutter/flutter/issues/37699

@mwk24 but #37699 is an iOS bug.

Yes, I know. But it is a "build headache". I apologize, perhaps Issues is not the correct forum for what is a more general product suggestion. Either way, I would like to add that flutter is my favorite dev/design experience of all time.

Thanks
@csells

As is, this isn't really an actionable bug. If there are problems you are having with Gradle, it would be better to file specific issues (or better yet, find the existing ones and upvote them).


fix(errors): improve error message in tcp.c and udp.c

The argerror doesn't seem to be providing much, can we just switch to luaL_error which has printf functionality built in?

sure. I'll change the PR


[PT Run] Windows Terminal Plugin

Summary of the Pull Request
What is this about:
Open in PT Run Windows Terminal profiles

TO-DO

[ ] Setup
[ ] Enable localization
[ ] Plugin icon
[ ] Plugin results icon
[ ] Test on Windows 11
[ ] Add tests

What is include in the PR:

Support both Windows Terminal and Windows Terminal Preview
The plugin read the profiles from the settings JSON file
Option for open the profiles in a new tab

How does someone test / validate:

Validate that the new plugin is working as expected

Quality Checklist

[x] Linked issue: #4194
[x] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Does it make sense to show the results when searching for wt/Windows Terminal (Preview) without using activation command (global results).

@davidegiacometti I have created a .light and .dark icon that might be useful for this plugin :):

@davidegiacometti I have created a .light and .dark icon that might be useful for this plugin :) (and already in the style for #13385)

@niels9001 I like these icons, but in my opinion it would be better to just use the terminal icon (or at least in the results):

@niels9001 icon must be used for the settings.
For results I would like to use the profile icons from Terminal but the icon could be an image file or a resource inside Terminal package.
Need to look in to this. Otherwise let's consider plugin or Terminal icon.

I have created a .light and .dark icon

Why not stick to MDL2?

I have created a .light and .dark icon

Why not stick to MDL2?

Icons for plugins aren't MDL2 but custom

Post about result's icon: https://github.com/microsoft/PowerToys/issues/4194#issuecomment-927296307

Post about result's icon: https://github.com/microsoft/PowerToys/issues/4194#issuecomment-927296307

I'd go for the colored Terminal icon in Run itself, but the Fluent Icons-version for the plugin manager list in Settings.

The PR is ready to review.
I have updated the screenshots in the first post.
Somebody please can build and test the plugin on Windows 11?

Works nicely on Windows 11.


Use RASCIL for SKA-MID

Preliminary workflow with RASCIL. It's very different to how OSKAR operates (different data structures). Uses a different sky model.

[ ] Design a good 95%-OSKAR-compatible-API for RASCIL


Auto link all urls in a single note

How do you auto-link all the URLs in one note? Currently using cmd+shift+e on each link so is there some shortcut available?

+1 for this one, just came here to ask this and found it on the top of the list.
Please add a option to fetch the urls in a the selected text or doing it in all of a specific note.
Thank you.

This older issue is also related: https://github.com/zolrath/obsidian-auto-link-title/issues/9


ui: add select box to select capture type

This has to check the capture type for a given interface type

https://github.com/skydive-project/skydive/pull/567

closed: #567


Include constant type properties in navtree

TypeScript code: Intl. DisplayNames.supportedLocalesOf ();
SupportedLocalesOf Go to Definitions and view the outline:
One is vscode and the other is WebStorm.

VScode does not have this outline item, WebStorm does

Feels like a duplicate of #52557 - the point of outline view is to be shorter/more-condensed than the file itself, so we necessarily need to elide things for that to happen.

Looking at the example here, the screen is just full of JSDoc comments. If the navtree isn't meant for this, is there a feature that gives an easy way to get a sense of all the members a type declares?


Add library drag and drop sorting

Describe your suggested feature
Add library drag and drop sorting
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 0.16.1.
[X] I will fill out all of the requested information in this form.

Care to elaborate?

Care to elaborate?

They probably mean a custom sort order where you can freedly drag and drop your entries from one position to another and it stays in that position.

It's what I mean

Before, in the settings it was possible to do this, it was much more practical.

I believe it should be possible to use some implementation of https://github.com/aclassen/ComposeReorderable for the drag/drop. Then in the library tab when the user is in selection mode, an option in the topbar for custom order could be added. This will allow the user to drag/drop instead of select more items and the order idx will be stored in the library item model. Finally a custom sort option could be added in the settings dialog.

Before, in the settings it was possible to do this, it was much more practical.

Yes, this was omitted when the screen in question was migrated to the new Compose UI framework. My personal guess is that it would have been a nightmare to properly implement.

I believe it should be possible to use some implementation of https://github.com/aclassen/ComposeReorderable for the drag/drop. Then in the library tab when the user is in selection mode, an option in the topbar for custom order could be added. This will allow the user to drag/drop instead of select more items and the order idx will be stored in the library item model. Finally a custom sort option could be added in the settings dialog.

That looks promising but as with every feature for Mihon, someone will have to take this up and PR it in order for it to happen 👍 Thanks for the research and recommendation


Fix SA1509 getting reported for consecutive blocks

Fixes #689

What should happen with the code below? I think it will report SA1509, but I think it should not report any diagnostic.
class Foo
{
    void Bar()
    {
        /* test comment */

        {
        }

        {
        }
    }
}";

I still claim it should raise SA1509 for first nested block in both cases:
class Foo
{
    void Bar()
    {
        /* test comment */

        {
        }

        {
        }
    }
}";

and
class Foo
{
    void Bar()
    {
        // test comment

        {
        }

        {
        }
    }
}";

And also for:
class Foo
{
    void Bar()
    {
        #region Test region

        {
        }

        {
        }
        #endregion
    }
}";

@vweijsters The current code reports SA1509 in that case, and this pull request does not change that. Since that line is not between blocks, it's also outside the scope of this particular issue; you can file a new issue if you'd like to see the behavior in that case changed.
@Przemyslaw-W I added unit tests to verify that the behavior of this pull request adheres to your expectations.

Current coverage is 79.49%

Merging #702 into master will decrease coverage by -0.01% as of f2d3461

@@            master    #702   diff @@
======================================
  Files          555     555
  Stmts        33508   33548    +40
  Branches      9243    9261    +18
  Methods
======================================
+ Hit          26640   26669    +29
- Partial       5369    5380    +11
  Missed        1499    1499

Review entire Coverage Diff as of f2d3461

Powered by Codecov. Updated on successful CI builds.

The docs should reflect the behavior.

:+1:


InheritedProtocolConformance: Return the conforming type's DeclContext

Not the superclass's. This can cause mapping types out of context to not
resolve archetypes, because the DeclContexts won't match, e.g:
internal class _IteratorBox<Base: IteratorProtocol> :
_AnyIteratorBoxBase<Base.Element> { internal override func next() ->
Base.Element?  }
Here, _IteratorBox's associated type Element for IteratorProtocol is
Base.Element, but the conformance comes from the superclass,
_AnyIteratorBoxBase. This meant that Base.Element couldn't be mapped to
an interface type, which should be:
(dependent-member
(generic-type-parameter depth=0 index=0) member=Element)
The old behavior also broke inherited conformances across serialization boundaries, because subclasses would fail to inherit conformances a deserialized class inherited itself from a base class (SR-1158).

@DougGregor Does this look reasonable for 2.2? There's something more subtle going on that I don't fully understand, and wouldn't want to mess with in the scope of 2.2. @bitjammer made this change to master as b0d6341420f4018ee80523fdab27721ad8afb959 for other reasons, and it also gets deserialization working in the case of SR-1158.

@swift-ci Please test

@tkremenek Can we take this for 2.2 (contingent on tests passing and @DougGregor's approval)?

@jckarter Yes we can take it, assuming review and testing passes.

Test fails:
Testing: 0 .. 10.. 20
FAIL: Swift :: SILOptimizer/specialize_inherited.sil (1876 of 7874)
******************** TEST 'Swift :: SILOptimizer/specialize_inherited.sil' FAILED ********************
Script:
--
/home/buildnode/jenkins/workspace/swift-PR-Linux@2/buildbot_linux/swift-linux-x86_64/bin/sil-opt -target x86_64-unknown-linux-gnu  -module-cache-path '/tmp/swift-testsuite-clang-module-cacheH7m5_2' -enable-sil-verify-all<EMAIL_ADDRESS>-generic-specializer -module-name inherit | FileCheck<EMAIL_ADDRESS>--
Exit Code: 1

Command Output (stderr):
--
/home/buildnode/jenkins/workspace/swift-PR-Linux@2/swift/test/SILOptimizer/specialize_inherited.sil:39:17: error: expected string not found in input
// CHECK-LABEL: @_TTSg5C7inherit8MMStringCS_8MMObjects8HashableS__GSQCS_6MMFont___callee : $@convention(method) (@in MMString, Int, @owned MMStorage<MMString, ImplicitlyUnwrappedOptional<MMFont>>) -> Bool {
                ^
<stdin>:31:1: note: scanning from here
// %0 // user: %4
^
<stdin>:47:23: note: possible intended match here
sil shared [noinline] @_TTSg5C7inherit8MMStringS0_s8HashableS__GSQCS_6MMFont___callee : $@convention(method) (@in MMString, Int, @owned MMStorage<MMString, ImplicitlyUnwrappedOptional<MMFont>>) -> Bool {
                      ^

--

@jckarter Is this just a test case that needs to be updated, or something more?

@tkremenek Yeah, the test case can just be updated. I'll take care of it and start the tests again.

FYI, @DougGregor signed off offline on this change.

@swift-ci Please test


Fails to load under a Linux ROCm environment

All packages from requirements.txt are installed and up to date.
Traceback (most recent call last):
  File "/sab_files/AI_Projects/ComfyUI/nodes.py", line 1887, in load_custom_node
    module_spec.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/sab_files/AI_Projects/ComfyUI/custom_nodes/ComfyUI_VLM_nodes/__init__.py", line 29, in <module>
    system_info = get_system_info()
  File "/sab_files/AI_Projects/ComfyUI/custom_nodes/ComfyUI_VLM_nodes/install_init.py", line 48, in get_system_info
    system_info['cuda_version'] = "cu" + torch.version.cuda.replace(".", "").strip()
AttributeError: 'NoneType' object has no attribute 'replace'

AMD cards not supported for now

The mac branch works. Thanks for the updates!


Themis doesn't build on debian 9 with openssl 1.1.0f

As a temporary solution: to build themis on debian 9, install libssl1.0-dev instead libssl-dev

guys, let's move to 0.9.6 :)

agreed :( but let's not delay this for too long.

#258


Fix bug in documentation of slant_inclined_path_equivalent_height in …

…ITU 676
Use P instead of p to refer to the atmospheric pressure.
Fix #64 .

Codecov Report

Merging #65 (b8e7430) into master (ab8e82d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #65   +/-   ##
=======================================
  Coverage   96.60%   96.60%
=======================================
  Files          25       25
  Lines        6688     6689    +1
=======================================
+ Hits         6461     6462    +1
  Misses        227      227

Impacted Files
Coverage Δ

itur/models/itu676.py
98.45% <100.00%> (+<0.01%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Improve the UI of Website

can i try it

can i try it

Good Luck 😃

Hey, I am a participant of GSSOC'22. I want to work on this issue .Can you please aassign me?

hello , I am a participant of GSSOC'22 , can you assign me this project to work .

Hey, I want to work on this issue can you please aassign me?

Hello, can you please assign me this issue?

@1ananya3 Any updates?

Yeah.The light theme is simple . I am adding some background effects so
it'll look cool and the template color is not changing while changing the
theme mode. Also I want to add an effect of font change So anyone can keep
the font in their own way.                            If you want to add
something in it please let me know ,I'll definitely keep that in mind.
On Tue, 1 Mar 2022 at 12:46, Swapnil Srivastava @.***>
wrote:

@1ananya3 https://github.com/1ananya3 Any updates?
—
Reply to this email directly, view it on GitHub
https://github.com/swapnilsparsh/30DaysOfJavaScript/issues/129#issuecomment-1055101841,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWQOODMPZ3GIU6T5KHVDXY3U5W76PANCNFSM5HMC7T4A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

I am a participant of GSSOC'22. I want to work on this issue .Can you please aassign me?

Yeah.The light theme is simple . I am adding some background effects so it'll look cool and the template color is not changing while changing the theme mode. Also I want to add an effect of font change So anyone can keep the font in their own way. If you want to add something in it please let me know ,I'll definitely keep that in mind.
…
On Tue, 1 Mar 2022 at 12:46, Swapnil Srivastava @.> wrote: @1ananya3 https://github.com/1ananya3 Any updates? — Reply to this email directly, view it on GitHub <#129 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AWQOODMPZ3GIU6T5KHVDXY3U5W76PANCNFSM5HMC7T4A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.Message ID: @.>

Go ahead everything seems fine 👍🏻

@1ananya3 Update?


Google Play Music not working?

I updated the app and now it doesn't recognize Google Play Music anymore. What do I do?

Chiming in to confirm that this issue still exists with latest OS, GPM, and SLS.

Fixes with #413

Still got the Problem on S8+ (Android 8) :/

I'm experiencing a similar problem with Android 8.1 on the Nokia 7 Plus - neither this app nor the official Last.fm app can detect music being played and send my scrobbles to Last.fm

hello everyone,
I new comment format will be used for issue discussion.
Please, if you have the time and this is still an issue for you.
Use the following format
Preferred Feature format
Request for Feature:
Reason:
Preferred issue forma

Phone Model:
Android Version:
SLS Version: (5.3)
Music App(s) In Use: (ex. YT Music, Spotify)
Detected Music Apps(s): (List what apps SLS detected)
Enabled Options: (List what options you have enabled)
Issue: (Description of What The Issue Is)
Parameters: (ex. When, Where, Why & How The Issue Occurs)

#479  <- please try the newest signed debuggable android apk. Thanks.


Support casting db types

Casting db types not supported first-class. This can only be accomplished with FromSql.
sys.index_columns.key_ordinal is BIT which maps to byte.
ColumnModel.PrimaryKeyOrdinal is int. Materializing throws an InvalidCastException.
Workaround:

.FromSql() with CAST (ic.key_ordinal AS INT)

Could overlap with a solution for #3932

Triage: This scenario is a dupe of https://github.com/aspnet/EntityFramework/issues/242. That feature would probably support this by reading it back as BIT but then allowing you to map it to an int property (rather than doing the conversion inline in the query).


Revert "Remove unused is_pair_like hack."

Reverts apache/beam#6956
This breaks dataflow.

run python postcommmit

Sure, Also running an internal test to make sure this works.

Run Python PostCommit

Post commit is failing because of unrelated hdfs issue.
Committing the change.


Reload Map and multiple maps in the same page

I'm creating a dashboard with draggable maps. The fact is that when i drag a new map, the dashboard refreshes and the old map dissapears, i just can see the markers in the first map, and the recently created map doesn't loads anything. The first map works when i refresh the page (F5) but the second doesn't appears... and something that i noticed is that when I zoom in the second map, the zoom affects the first map... Do you have any Idea on what's going on?? how can I use multiple maps in the same page? How can I reload the coordinates from options or my entire map??
I'm uploading some pictures to clarify

In this picture, you can see my map working correctly

This is what happens when I add another map, both maps seems not to be present

When I refresh the entire page, the first map loads, please check the map markers, they are correctly positioned in this picture

I zoomed a bit in the second map and guess what? The first map got zoom and my markers changed their location...

Do you have your component and template code that you could share?
One thing that can cause this behavior is if you reuse the same Layers between multiple maps.
Generally with Leaflet, a layer can only be added to one map (including baselayers). This just means you need to recreate your baselayers for each map.

Well, maybe that's the problem... i'm creating new maps with the same tileLayer... I obtain my map options like this
options: {
      layers: [
        L.tileLayer('https://cartodb-basemaps-{s}.global.ssl.fastly.net/light_all/{z}/{x}/{y}.png',
          {
            maxZoom: 15,
            subdomains: 'abcd'
          })
      ],
      zoom: 5,
      center: L.latLng([this.mapCenters['Center'].mapCenterLatitude, this.mapCenters['Center'].mapCenterLongitude])
    }

For each new map, options are the same... any idea on how can i make a small variation or how can i fix this?

A lot of times what I do in this situation is create an array or map of "map specs" in my component and each map spec is an object that contains everything the map needs like options, layers, etc. That way, when I want to add a new map, I add a new map spec to the array or map.

@reblace can you please give me an example? The fact is that i need to use the same map, maybe with another latitude and longitude, and different markers that i will obtain from a service, and well the tile I'm using is the one that i need...

@reblace nevermind, i just make it work! Thanks for the support, great job with the library!

@Oscarsgc I am facing this same issue, how did you make it work?

@Oscarsgc @reblace @philglen I'm having the same issue. I have two components, that are each showing a map, but with different overlays. they both want to use the same baselayer, but for some reason it is not possible to have the radio button of the base layer selected on both the maps, when I select the baselayer on the second leaflet map, it gets unselected on the first one. my code looks like this on both the components:
leaflet-test.html
<div
        style="height: 600px;width: 100%"
        leaflet
        [leafletOptions]="options"
        [leafletLayersControl]="layersControl"
      ></div>
    </div>

lealfet-test.ts
 baselayer = tileLayer('https://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png', {
    maxZoom: 20
  });
  public options = {
    layers: [this.baselayer],
    zoom: 14,
    scrollWheelZoom: false
  };

  layersControl = {
    baseLayers: {
      Map: this.baselayer
    }
  };

@sebastienpattyn93 The issue is almost always that you're reusing the same layer object for both maps. You have to create unique layers for each map and keep them separate.
Layers are mutated by the map to which they are added, so they are tied to that map. You get these weird cross-map interactions when you try to reuse them.

@reblace: Let's say two Leaflet maps should use the same layer (and are synced).
When only one map can be used per layer, two of these layers have to be used.
When the same tile provider is used, wouldn't this result in two HTTP requests, one being unnecessary?


Add shields to project review tickets to promote use.

Closes https://github.com/gratipay/gratipay.com/issues/4634.

https://github.com/gratipay/gratipay.com/blob/master/tests/py/test_project_review_process.py

Next commit will have a test, but I need to fix the trailing slash before /svg in both code and test.

:tada:

Now onto /about/me/widgets...

Now onto /about/me/widgets...

This should actually go on the project, not the owner.

Rebased (was 9676a908efd555faf8d00f5d5a1a7093a774e22f), and then I did a little formatting of both code and resulting content in abdad0722c8f17e10d447b9bb536051b2fafe4e9.
Issue example: https://github.com/whit537/test/issues/4.
Widget page:

I'm ready for this to be merged if you're good with my commit, @mattbk.

I think it looks good.

Y u no like, Travis?

Rebased and Travis should be fixed, was 6b25f2c34174241880242e55c3ea787fff950d4a.


Add modifiable source_type support to sas emitter

Signed-off-by: Tatsuhiro Chiba<EMAIL_ADDRESS>Tihs PR is related to #329.
In order to change source_type value for registry crawler, I prepared an additional sas emitter config in crawler.conf. SAS emitter reads the value from specified file and uses it for sending data to sas api.

@tatsuhirochiba: Travis failing

Codecov Report

Merging #330 into master will decrease coverage by 0.07%.
The diff coverage is 37.5%.

@@            Coverage Diff            @@
##           master    #330      +/-   ##
=========================================
- Coverage   88.27%   88.2%   -0.08%
=========================================
  Files         116     116
  Lines        4650    4655       +5
=========================================
+ Hits         4105    4106       +1
- Misses        545     549       +4

Impacted Files
Coverage Δ

crawler/plugins/emitters/sas_emitter.py
73.86% <37.5%> (-3.25%)
: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 f265ac7...9cdb3d3. Read the comment docs.

@sahilsuneja1 Thanks. I just fixed test code and travis build was passed.

Thanks @tatsuhirochiba
I had some doubts:

How is source_type a secret?
Who would change it between 'container' and 'image' ?
Can it simply be source_type = container or source_type = image ?
Can it automatically be inferred from --crawlmode?

@nadgowdas

@tatsuhirochiba
This change seem to be required for cfc to distinguish between regcrawler crawling image and live/metric crawler for live containers. And both these crawlers run with "OUTCONTAINER" mode.
Lets consider few options:

for regcrawler, set the --namespace as "image: image-path" since, they in emitter they can be identified as source_type=image. For live containers, do not set any namespace, so k8s-namespace will be set by plugin.
Your solution : putting source_type as a secret and reading it from there. This would require multiple secrets to be stored in k8s. One for live container and other for regcrawler. ? Or you can  mount source_type secret just for regcrawler. And instead of assertion of this secret, check for its existence and use if available
Use environment variable: Add source_type as env variable and in emitter check and set it accordingly. Just don;t force use of it. If env variable is available then use it from there, if its not present, don;t fail and use default derived from metadata.

I personally would recommend option(3) as its seems cleaner with minimum

@sahilsuneja1 @nadgowdas Thanks for your comments!
I adopted option(3); retrieving source_type from env variable and setting similar value of system_type as a default if source_type env is not available.

Thanks!
@tatsuhirochiba: can you please squash the commits into one?
@nadgowdas: can you please approve?

@sahilsuneja1 I merged them into a single commit.

@tatsuhirochiba can you pls update the PR. We can merge this change then.

@nadgowdas just update PR comment.

@tatsuhirochiba It still shows branch is out-of-date.?

oh, just merged untracked PR..

Thanks @tatsuhirochiba. PR merged


SAKIII-4694 - Embed content links not working correctly across a 1.1 upg...

https://jira.sakaiproject.org/browse/SAKIII-4694

verified and merged.


Antlr4 upgrade

@MomoDeve Use the latest version of libraries next time, thx
@WhiteBlackGoose Make sure the project compiles successfully next time, thx

Thanks for fixing code. Actually it was my first try creating grammar with Antlr, so I am sorry for bad parts of my code 😅

Ngl this is my first experience with Antlr too

Nice


Makes Eternia init + misc cleanup

Misc Rider-suggested code cleanup
sealed some things
Adds coping with null _current
Adds tolerance for color = "" which BYOND allows in modified types in DMMs.

Do we really want to be putting readonly on fields? I understand it's a micro-opt, but it might be worth profiling times for.
https://codeblog.jonskeet.uk/2014/07/16/micro-optimization-the-surprising-inefficiency-of-readonly-fields/

@ZeWaka https://devblogs.microsoft.com/premier-developer/avoiding-struct-and-readonly-reference-performance-pitfalls-with-errorprone-net/

and https://devblogs.microsoft.com/premier-developer/the-in-modifier-and-the-readonly-structs-in-c/


ensure build task does not exit during start task

Also fixes build task params. They were two booleans, but start task called them with a single object with two booleans.


Running sample fails due to 502, remove jcenter?

Describe the bug
I'm trying to run the sample, but in current stable AS I get an issue
Error resolving plugin [id: 'org.jetbrains.dokka', version: '1.4.32']
> Could not resolve all dependencies for configuration 'detachedConfiguration2'.
   > Could not determine artifacts for org.jetbrains.dokka:org.jetbrains.dokka.gradle.plugin:1.4.32
      > Could not get resource 'https://plugins.gradle.org/m2/org/jetbrains/dokka/org.jetbrains.dokka.gradle.plugin/1.4.32/org.jetbrains.dokka.gradle.plugin-1.4.32.jar'.
         > Could not HEAD 'https://jcenter.bintray.com/org/jetbrains/dokka/org.jetbrains.dokka.gradle.plugin/1.4.32/org.jetbrains.dokka.gradle.plugin-1.4.32.jar'. Received status code 502 from server: Bad Gateway

I understand we can't do anything about the 502, and it might be temporary, but maybe we can try to remove jcenter/bintray repo?

Well apparently everything is seemingly down. So probably not the fault of this repo... it would be nice to have a samples.apk for this sort of thing. 😄

I remove gradlePluginPortal and also added the sample apk to the release. Process is still manual now but I should figure out how to automate that at some point

Love it. Thank you soooo much!


Add https://cors-anywhere.now.sh to "Live examples" in README

Hello. I have deployed your CORS proxy to Now.
If you would like, feel free to merge this to add it to the listing in the README!
Have a great day 🍻  !

Coverage remained the same at 100.0% when pulling 662eafc0a208d7a7371a5dc21d3654931721d839 on TooTallNate:patch-1 into 97c6e53020da622b006842ef112bc721cac0c47d on Rob--W:master.

Coverage remained the same at 100.0% when pulling 662eafc0a208d7a7371a5dc21d3654931721d839 on TooTallNate:patch-1 into 97c6e53020da622b006842ef112bc721cac0c47d on Rob--W:master.

Thanks for hosting another public demo. I'm not merging this PR though, because I cannot guarantee that the instance that you're hosting stays up and running.


Having problems moving large files

Moving large folder errors out: not enough free space (I have over 500 GB).

Might be a netatalk issue? Anyone have any ideas?

Here's how the remaining space looks. The share partition should be 1.83 TB, my linux system / partition is 28.7 gb - I guess it's getting the wrong free space info.

Maybe you have configured disk quota or something of the likes.
Could you provide the result of the following command docker exec -ti <netatalk container id> df -h?
Might be that the docker you’re using does not support such large volumes or something like that. I myself have it successfully running with multiple 3TB volume shares.

On 7 Sep 2017, at 05.42, Tony Ko<EMAIL_ADDRESS>wrote:
Moving 5gb+ folder errors out: not enough free space (I have over 50 GB).
Might be a netatalk issue? Anyone have any ideas?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

In case others search answer for the issue, for me it was solved by using separate share with volume pointing to mount point of host of drive.
So in docker run command i have now:
--volume /media/my-volume:/shares/my-volume
instead of  /media:/shares
in afp.conf:
[elements-hfsplus] path = /shares/elements-hfsplus
instead of:
[shares] path = /shares/


library to parse transaction info into readable strings

Please refer to various cookbook to get started
Note that near-api-js power Web Wallet
One interesting cookbook is
https://github.com/near/near-api-js/blob/master/examples/cookbook/transactions/get-tx-detail.js
above will help understand the relationship between a transaction, chunk, and receipts. And also Actions as well.

Few good read:
https://nomicon.io/ChainSpec/Transactions.html
https://nomicon.io/RuntimeSpec/Receipts.html
https://nomicon.io/RuntimeSpec/Actions.html
With this task we want to achieve is below:
"give me an account name and a transaction hash related to that account and we can tell you "what happened" or "how it went"

discussion with mike, we possibly need a separate app for this
see example below
near state does-not-exist.demo.testnet
above a more friendly response
near create-account ztg.xword.demo.testnet --masterAccount xword.demo.testnet --initialBalance 660
but above give a stack trace
what we want is a simple nodejs module which when installed with near-api-js into a new app should always provide friendly response

https://github.com/ZTGProjects/near-api-js/tree/parse-tx-readable-string


Documentation indicates OpenShiftSDN is a (preferred) supported default network CNI Provider

Describe the bug
Current OKD latest document indicates two default supported network CNI Providers, OpenShiftSDN and OVNKubernetes, in that order.  In the OKD working group discussion, it seems only OVNKubernetes is regularly tested.  The document seems to imply that OpenShiftSDN is the prefered case and talks about "migrating from" and "rolling back to" between this case and OVNKubernetes.
In fact, if the install-config.yaml does not specify which to use, OVNKubernetes is selected.  This should be clarified and perhaps OpenShiftSDN should be deemphasized.  If there is a preference between these two, this should be clearly indicated.
See https://docs.okd.io/latest/networking/openshift_sdn/about-openshift-sdn.html and https://docs.okd.io/latest/networking/ovn_kubernetes_network_provider/about-ovn-kubernetes.html
Version
OKD Latest documentation: https://docs.okd.io/latest/welcome/index.html
Becomes an issue with 4.6 issue  #430
How reproducible
Documentation issue
Log bundle
Not applicable

/assign

/assign

@vrutkovs We can hide the OpenShiftSDN topics if needed.

OpenshiftSDN is a valid option in OKD, its just not being default.
I think its best to keep the examples, but its worth adding a note that OKD uses OVN by default (not sure where it would fit best, probably install instructions?)

/close


ASP.NET Core 1.1 web application does not start when running with IIS Express in Visual Studio 2017

I've already raised this issue with StackOverflow. This issue may already be addressed elsewhere and apologies if so, but I can't seem to find anything specific, and the suggestions on StackOverflow don't seem to work either:
Problem
I've recently installed Visual Studio 2017 Professional and have just created the boilerplate ASP.NET Core Web Application
This is what happens when I hit Run (With IIS Express):

And this is what Visual Studio 2017's Output window says:

The program '[8260] dotnet.exe' has exited with code -2147450749 (0x80008083).
The program '[6476] iisexpress.exe' has exited with code 0 (0x0).

Assuming that this is a known issue, are there any steps to fix this, or any available updates to Visual Studio that will fix it?

I experienced same issue on windows 10, I want to attach IIS Express Trace log files but github does not allow .xml extensions. I have also raised this issue on stackoverflow  but no response.

@DureSameen does running Visual Studio in admin mode yield any better results?

no, there is no change in admin mode.

@DureSameen I get the same thing when I run a .NET Core console application.

The program '[17372] dotnet.exe' has exited with code -2147450749 (0x80008083).

@DureSameen try installing the x86 SDK, see if that helps...
https://www.microsoft.com/net/download/core

A coworker and myself are seeing the same problem on Win7 and Win10 respectively.
I don't think the dotnet SDK is at fault here considering launching the project manually with dotnet run works just fine. It's only when launching from Visual Studio with IIS Express 10. Also, this problem doesn't occur when running a docker-compose project around the same aspnetcore binary.
I read on a different StackOverflow post that someone had success installing IIS8 to get an ASP.NET Core app running, so I'll be trying that here now.

I found the issue.
When running an ASP.NET Core app in VS on IIS Express, you have to configure the web host to use IIS Integrations to handle IIS-specific runtime settings (like capturing IIS exceptions, etc.).
All I had to do was add the following to the WebHostBuilder in a fresh ASP.NET Core web application and it worked fine.

I'm not sure what effect this could have on other dependent projects (like running in docker) but I'll keep investigating as I go along with a new project we're starting.

Considering a default dotnetcore project shouldn't have IIS configs in the template, I can see why it was left out. But it would be nice to contact the correct party to make sure something like that is setup when generating a project in VS, since IIS is the default server option in the project currently.

This won't matter as much after https://github.com/aspnet/Templates/issues/837, we won't need UseIISIntegration in any templates anymore.
Also note that UseIISIntegration is a no-op if you launch your site outside of IIS/Express.

I found the workaround.
The main issue was My machine already had Visual Studio 2015 Community Edition with updates side by side of Visual Studio 2017.
I removed it and then I tried to remove all individual program beginning with word Core (SDK and run time etc).
Then I remove .NET Core Command-Line Interface (CLI) Tools after that Visual Studio 2017 worked like a charm.

This issue still persists after months of reporting it and has not been fixed.  Yesterday, I downloaded the latest VS2017 15.1 updates and no fixes have been put in place.

One of my colleagues is having the same issue. Dot net Run works fine, however we need IIS-Express for debugging. Ironically, only MVC project is having not running but Web Api project runs fine.
He also has VS2015 installed previously, and that needs to be installed as he has to work on a legacy project which was built on VS2015.
Any advice will be appreciated. Thanks.

@series0ne As this bug been resolved for you?
@the-super-coder Have you tried the suggested fix in this page? If it doesn't work, please open a new bug with reproduction step to your problem.

My Colleague Umer was having this issue, I suggested the fixes on this issue but that did not work. He needs VS2015 and VS2017 both installed. So he has created a new issue here:
https://github.com/aspnet/Home/issues/2041

Hey gents, hope all is well. I was struggling with this as well and it seems like I might have found the solution. What works for me is closing the solution, navigating to where you keep the solution file and deleting .vs folder. Then open again and it works. It's obviously a workaround but at least a quick one. Also it seems it happens when I pull a lot of new projects via git. So that might be causing it somehow. Good luck!

Hi @PeterKottas,
Thanks for contributing.
@umerwaheed,
Could you please try if this temporary workaround could get us working?

@PeterKottas  @the-super-coder  still not working :(.

I had this same issue and after digging around for a while I discovered that I had dotnet preview v1.0.0-preview2 installed under Programs Files (x86) and a non-preview version in x64.   I think VS is launching the x86 (preview) version but expecting to see the full version.   To fix this, I did the following.

from programs and feature uninstall every visible dotnet core.  (note: this did not remove the x86 preview)
go to https://www.microsoft.com/net/download/core#/sdk and install BOTH x86 and x64 SDK packages
open command line and from the root directory check run:  dotnet --version   (at the time of writting it was 1.0.4
fire up .net core project in VS 2017 and run.

When I did all of the above, I was able to start up the site in IIS Express from VS.
For the record:  you dont need to run VS in administrator mode (unless your targeting full-blown IIS)

@philblackburn solution worked for me

So... I had this issue using IIS Express as well, I switched to self-hosting all of my microservices... I have a project with about 10 ASP.NET Core web-api projects.
Unfortunately that didn't fully solve the issue....   So frustrating when you waste hours a week just trying to get a stable debug session going...I don't know if there is a separate issue reported for that somewhere or if it falls under the same issue.

what a problem in my environment
Running Ubuntu 16.04TLS / nginx /.NetCore 2.0
Error when using systemctl status error and service didnot start
Working on Widows and IIS Environment

sudo systemctl status n2nplatform.service
n2nplatform.service - N2NPlatform
Loaded: loaded (/etc/systemd/system/n2nplatform.service; enabled; vendor preset: enabled)
Active: activating (auto-restart) (Result: exit-code) since Wed 2017-10-18 09:15:36 ICT; 1s ago
Process: 42579 ExecStart=/usr/bin/dotnet /usr/share/nginx/n2nplatform.com/N2N.dll (code=exited, status=1/FAILURE)
Main PID: 42579 (code=exited, status=1/FAILURE)
Oct 18 09:15:36 ubuntu16 systemd[1]: n2nplatform.service: Main process exited, code=exited, status=1/FAILURE
Oct 18 09:15:36 ubuntu16 systemd[1]: n2nplatform.service: Unit entered failed state.
Oct 18 09:15:36 ubuntu16 systemd[1]: n2nplatform.service: Failed with result 'exit-code'.

I have the same issue with Core 2 it suddenly stops working. "The site can't be reached" is the only message.

Change the localhost port
Properties of Project --> Web --> URL Project localhost:15115 -->Create virtual directory
SAVE and RUN
Good luck

Closing this because there have been several releases of .NET Core and Visual Studio since this issue was opened, and many bugs have been fixed that might have addressed this. If anyone is still running into this, please log a new issue with details. Thanks!

FYI This was still happening with Visual Studio 15.7.4 and .NET Core 2.0.3, our team encounters this countless times every day.   We have not yet upgraded to 2.1.x.

To report VS issues, it's best to use the Report a Problem tool.
If you encounter a problem with Visual Studio, we want to know about it so that we can diagnose and fix it. By using the Report a Problem tool, you can collect detailed information about the problem, and send it to Microsoft with just a few button clicks. See here for more details.

Hey gents, hope all is well. I was struggling with this as well and it seems like I might have found the solution. What works for me is closing the solution, navigating to where you keep the solution file and deleting .vs folder. Then open again and it works. It's obviously a workaround but at least a quick one. Also it seems it happens when I pull a lot of new projects via git. So that might be causing it somehow. Good luck!

This solved my issue. Thanks a bunch!

Uncheck EnableSSL, shows up the website with broken CSS, atleast something better then nothing. I will go from here.


Handling list of dictionaries differs between python versions

Python 2.7 supports dictionaries comparison, so sorting of dict list is supported by default.
On python 3 __lt__ is not defined for dictionaries, so it makes harder to migrate projects that uses jinja2 to python 3.
For example for template
"{{ items|sort }}"

tmpl.render(items=[{'b': 4}, {'a': 3}, {a': 2}, {'b': 1}])

leads to TypeError: '<' not supported between instances of 'dict' and 'dict'"
Providing even basic support for dictionaries would make migration process a lot easier.

duplicate #355

Same as #1287, I don't want to change how sorting works in Python. If you need to sort this type of data, either sort it ahead of time in Python, where you can make the key function as complex as necessary, or override the sort filter to work differently.

Sorry for closing these after you put in the work of PRs. I really do appreciate that you put the time in to implement them even though they were closed, it can be useful for anyone looking for this type of sorting.


JBRULES-3153 ChangeSet loading will take 5 minutes without an internet connection

I upgraded the KnowledgeAgent to use Options for configuration in addition to strings

Applied to master. @sotty can you please close the related JIRAs? Thank you.


Dualstack overlay cni

This PR links to this: https://github.com/Azure/azure-container-networking/pull/1866
Reason for Change:
This PR is to provide dualStack CNI changes to create a NC with both ipv4 and ipv6 address in dualStackOverlay mode on both Windows&Linux OS
The PR includes following UTs:
(1) RequestIPs() => request ipv4 & ipv6 ip addresses and make sure dual-stack NC creation
(2) RequestIPAddress() => request only ipv4 and make sure single-stack NC is working fine
(3) ReleaseIPs() => release ipv4 & ipv6 ip addresses and make sure they are released
(4) ReleaseIPAddress() => release ipv4 address only and make sure this ipv4 address can be released in single-tenancy
Tests we did manually(on both Linux and Windows):
(1) Network(ipv4+ipv6):
pod -> pod (one node and different nodes)
pod -> host
host -> pod
pod -> internet
host -> internet
(2) Launched dual stack overlay cluster in different regions: eastus, eastuseuap, centraluseuap
(3) Test Different Linux OSs (including Ubuntu 19, Ubunt22 and CLB-Mariner)
(4) Test different Linux processors (ARM/AMDH&I)
(5) NMAgent tests
(6) HNS tests: https://microsoftapc-my.sharepoint.com/personal/ppereira_microsoft_com/_layouts/15/onedrive.aspx?id=%2Fpersonal%2Fppereira_microsoft_com%2FDocuments%2FDocuments%2FProjects%2FAKS-IPV6-Support&ct=1680642656649&or=Teams-HL&ga=1

this is a continuation of https://github.com/Azure/azure-container-networking/pull/1866 as Paul seems to have run into some account issue.

/azp run

/ap run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run


[NavigatorIOS] How to modified the title/leftButton/rightButton on the navigation bar after the scene is shown?

You can use setState to change the property and let React rerun render function

@Prometheuser's answer is directionally right. If you need more clarification can you please use Stack Overflow to ask questions and tag them with react-native? http://stackoverflow.com/questions/tagged/react-native
Many people from the community hang out on Stack Overflow and will be able to see your question. Using Stack Overflow for questions also helps us use Github issues to keep track of bugs that need to be fixed.

thank you both! I'll try~


[proofing] Add moderator dashboard

This is a simple starting point for a more feature-rich dashboard that
will let moderators manage the proofing effort as a whole.
Test plan: unit tests and tried it on dev.

Over 24h, landing


Checking coding-style with pep8

There are lowerCamelCase variable names, class name.
There are tabs indentation

Thanks. Edited codes with autopep8, and then uploaded.