Incorrect example

Your example page https://sorbet.org/docs/flow-sensitive is about
x = maybe_int.nil? && (2 * maybe_int)

and I'm pretty sure the author actually meant
x = !maybe_int.nil? && (2 * maybe_int)

with a not (!nil?).  Alternatively || would do.
Cheers!

Thanks, it looks like this was fixed in #4414.


[CustomLoss] Update example

[CustomLoss] Update example

**Changes proposed in this PR:**
- Update example using LayerClient
- Add LayerClient example to AppTest

**Self evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

resolves #1256, #954

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1320. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.

:octocat: cibot: @zhoonit, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1320-202106212050380.59588003158569-f0b193c822cf49f7dc580ca0a2e95d12f52ce406/.

:octocat: cibot: @zhoonit, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1320-202106241807560.46425700187683-0a3159fb7b59fef5891fff5235de6481a232227a/.


Version 2.0.0

This PR implements major changes which increases the software to version number v2.0.0

The following issues from the original repo are solved in here

https://github.com/SiLab-Bonn/irrad_control/issues/44
https://github.com/SiLab-Bonn/irrad_control/issues/41

It is finally done.


Update danger

Description

Update danger

Code coverage
Total coverage: 95.13%

Powered by Slather

  Generated by :no_entry_sign: Danger


Add -fine-grained-timers flag

When this is off, we skip collecting per-request timers, which eliminates most of the overhead of using -stats-output-dir.

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please test Windows


Feature/event categories

Add the option to attach a category to event - both for create and edit - Backend support.

Awesome job! Merged on your first shot :) Very nice!


Ios 8 date picker not woring

Its not working for ios 8,
Plz help

anyone else having these problems mentioned here: https://github.com/VitaliiBlagodir/cordova-plugin-datepicker/issues/112 ?

Hi @hafl1016 ,
I have this same problem as discussed in the ticket #12

@fpmweb
I think you wanted to reference to  ticket #112?


Uni.join().first(unis).toTerminate() must not support concurrency limit

It does not make sense to join on the first Uni to signal (item or failure) with a concurrency limit.
The currently unreleased API must throw an exception when a concurrency limit has been set, or be reworked to avoid expressing such an intent.

See #825 #829


PDF cuts off text

I like the Download Template as PDF option. But it seems to be cutting off some of the months, or partial months.

This should be fixed now. There was a bug where the PDF got cut off depending on your current vertical scroll position within the page. Thanks for reporting and for using this tool :)


Can't upload a specific length of MultipartFormData.BodyPart.bodyContentLength from an InputStream

When I upload a large file, the entire file is uploaded in a single shot, which is not what I expect since I've set MultipartFormData.BodyPart.bodyContentLength.
I'm trying to upload a slice of length body.bodyContentLength but I don't see any boundary constraining the MultipartFormData.writeBodyStream method.
Is my assumption valid? Should the upload be limited to body.bodyContentLength? Is this a bug? Is this a feature request?

Question asked on StackOverflow: https://stackoverflow.com/questions/76210072/alamofire-doesnt-limit-multipartformdatas-inputstream-upload-length-to-multipa
This proof of concept properly limited the request to bodyPart.bodyContentLength as expected:
private func writeBodyStream(for bodyPart: BodyPart, to outputStream: OutputStream) throws {
    let inputStream = bodyPart.bodyStream

    inputStream.open()
    defer { inputStream.close() }

    var totalBytesRead = 0
    var bytesLeftToRead: UInt64 = bodyPart.bodyContentLength
    var bufferSize: Int = streamBufferSize
    while inputStream.hasBytesAvailable && bytesLeftToRead > 0 {
        if bytesLeftToRead < streamBufferSize {
            // TODO: Review if it's ok to do this cast
            bufferSize = Int(bytesLeftToRead)
        }
        var buffer = [UInt8](repeating: 0, count: bufferSize)
        let bytesRead = inputStream.read(&buffer, maxLength: bufferSize)

        if let streamError = inputStream.streamError {
            throw AFError.multipartEncodingFailed(reason: .inputStreamReadFailed(error: streamError))
        }

        if bytesRead > 0 {
            if buffer.count != bytesRead {
                buffer = Array(buffer[0..<bytesRead])
            }

            try write(&buffer, to: outputStream)
            bytesLeftToRead -= UInt64(bytesRead)
        } else {
            break
        }
    }
}

For reference I'm trying to upload to Kaltura video service provider. They provide a chunked upload approach, so we can have multiple requests each uploading a part of the stream.
High level pseudocode would look like this:
let inputStream = InputStream(URL: videoURL)
for chunk in chunks {
    inputStream.skip(bytes: chunk.resumeAt) // extension implemented over InputStream to skip a number of bytes.
    multipartFormData.append(inputStream,
        withLength: chunk.size,  // <<<<<<< THIS is set to `MultipartFormData.BodyPart.bodyContentLength` thus it should limit `MultipartFormData.writeBodyStream` while loop
        name: uploadKey,
        fileName: fileName,
        mimeType: mimeType
    )
    alamofireSession.upload(multipartFormData: multipartData, ...)
}

I don't have time to work on this now, but feel free to experiment with an implementation that gets you what you want. If you get something working put up a PR and I'll take a look.

I've remarked it as a bug but ultimately it doesn't matter.

@jshier I put up a PR:
https://github.com/Alamofire/Alamofire/pull/3730

The fix for this has been released in 5.8.0.


it is helpful to actually respond with information about the error th…

…at just happened. This gets saved into the history, which we may or may not want

not sure if this will break tests or not, but we should consider responding with details when an error happens. this allows us to actually give errors to the user when we are streaming rather than just closing the connection (which is not super helpful)

Closing this, the error is already recorded with state change


Update README.md

Hi @ezbercih,
I've approved your changes, but unfortunately I can't merge this PR unless you sign the CLA above.


Fix that map-clone-special endpoint doesn't clone node-tags atm [or find alternative, eg. tag-transfer system]

I hope to contribute to this project, but I don't know which issue to solve.
My email address is bitmargin at hot mail dot com.
If you want to discuss about collaboration, please ping me.


[ref] Go router

default mutex from net/http is relatively week
https://github.com/julienschmidt/go-http-routing-benchmark by author of httprouter
https://github.com/julienschmidt/httprouter use a radix tree and seems to be the fastest
I was mainly using https://github.com/gorilla/mux which has a god sub router feature

In fact

for simple routes ... a hard coded router might also work .... if strings.StartWith

so far using default mux is enough ... and we didn't enforce what router user should use, router itself is  just
a http handler


Property today does not exist -  type error in mapDays

using typescript, "today" in mapDays argument is not defined:
Property today does not exist on type {     date: DateObject;     selectedDate: DateObject | DateObject[];     currentMonth: object;     isSameDate(arg1: DateObject, arg2: DateObject): boolean; }

Hi.
Thank you for reporting this bug.
It should be fixed in v4.5.1


add travis CI badge - issue #66

adds TravisCI badge for issue #66

also updates the waffle.io badge to match travis badge height, based on the Waffle.io recommendation here: https://github.com/waffleio/waffle.io/issues/2932

👏 awesome!  Thanks!


Set cxxstd=20 on benchmarks

and upgrade benchmark servers to Ubuntu 22.04

Upgraded operating system, compiler version, and cxxstd.
$ gcc --version
gcc (Ubuntu 11.3.0-1ubuntu1~22.04) 11.3.0

$ clang --version
Ubuntu clang version 14.0.0-1ubuntu1

See attached image.

2023-03-11 through 2023-03-22 was the earlier configuration.
2023-03-24 through 2023-03-27 after the upgrade the frequency lock was temporarily disabled. On these server I'm locking the frequency of the CPU to a low value to improve the stability and repeatability of results.
2023-03-28 through 2023-03-31 after the upgrade, and the frequency lock is back in place. Results are 10% better than historical numbers.
2023-04-02 Set cxxstd=20. another 1% better.

Explore all the results at https://benchmark.cppalliance.org/ Some benchmarks improved after the system upgrade and others are slightly worse. It depends on the benchmark test.


Import statement inside function

Hi, Miguel. As PEP8 said,

Imports are always put at the top of the file, just after any module comments and doc strings, and before module globals and constants.

However, there are many import statements were inside the function, such as here. Is it better to move them to the top of the file?
Besides, the import statement below was redundant with this. Can it be removed?

Yes, those redundant imports should be removed, good catch.
Using imports inside functions is sometimes justified, in my opinion. Consider this example. If I move the profiler's import to the top, then it will be imported for every use of the application, even though there is only one specific use case that requires the profiler. The application will be bigger because of this import than in most cases isn't needed.
However, I think the example you indicated does not really need the imports to be inside the function, for those you don't really gain much by keeping them inside, and obviously I missed that the models eventually were added at the top as well, so they are now duplicated. I will remove those imports from the function.

Thanks for the detail explanation.


Add method to load all schemas

Hi! I propose a new method to load all schemas from the ledger.

Thanks for the contribution. Two things:
First. Please be aware that all commits need to be signed off, otherwise we can not merge your branch - https://github.com/apps/dco#usage As you just have a single commit you can easily fix that - https://stackoverflow.com/questions/13043357/git-sign-off-previous-commits
Second. As the api also allows to to filter the response by query params I would also make this available in the client like:
package org.hyperledger.aries.api.schema;

import lombok.Builder;
import lombok.Data;
import org.hyperledger.aries.api.AcaPyRequestFilter;

@Data
@Builder
public class SchemasCreatedFilter implements AcaPyRequestFilter {
    private String schemaId;
    private String schemaIssuerDid;
    private String schemaName;
    private String schemaVersion;
}

and in the client:
    public Optional<List<String>> schemasCreated(@Nullable SchemasCreatedFilter filter) throws IOException {
        HttpUrl.Builder b = Objects.requireNonNull(HttpUrl
                .parse(url + "/schemas/created")).newBuilder();
        if (filter != null) {
            filter.buildParams(b);
        }
        Request req = buildGet(b.toString());
        return getWrapped(raw(req), "schema_ids", List.class);
    }


bug(forge): overlapping nested remappings are not resolved

Component
Forge
Have you ensured that all of these are up to date?

[x] Foundry
[x] Foundryup

What version of Foundry are you on?
forge 0.2.0 (02e430c 2023-07-21T00:24:08.481175000Z)
What command(s) is the bug in?
forge build
Operating System
macOS Silicon
Describe the bug
Problem
#5397 added support for remapping contexts (closing #1855), however, the implementation does not handle all edge cases yet.
Suppose you have a project structure like this:
project/
└── lib/
    ├── foo/
    │   └── lib/
    │       └── openzeppelin-contracts
    └── bar/
        └── lib/
            └── openzeppelin-contracts

In both foo and bar, openzeppelin-contracts is remapped to the following:
@openzeppelin/contracts/=lib/openzeppelin-contracts/contracts/

The bug is that only one of the two remappings gets resolved, not both.
Error Log
Sneak peek:

Unable to resolve imports<EMAIL_ADDRESS>
See the full log here.
The issue is that only the v2-core remapping appears in the auto-detected remappings:
lib/v2-core:@openzeppelin/contracts/=/Users/prb/Sablier/examples/lib/v2-core/lib/openzeppelin-contracts/contracts/

Reproduce

Clone sablier-labs/examples at commit eae781
Remove the permit2 remapping in remappings.txt
Run forge build

Yep I'm on this today actually @PaulRBerg — will rollback the remappings context PR first to unblock you & others, and we'll get a fixed PR later!

Btw @PaulRBerg — any chance you can provide a more minimal repro with a recent foundry version? this way it'll be much easier

I'm afraid that I don't have any repro smaller than the Sablier examples repo. However, what should matter is the IERC20 import. You should be able to focus on that.
It is also difficult to provide a small repro due to the nature of this bug - it gets activated when there are nested deps with shared deps between them.
Re Foundry version - I ran foundryup now and I observed the same behavior when running forge build.

Still having issues with overlapping remappings. I have updated my foundry to the latest version.
This is the relevant part of my remappings.txt:
@openzeppelin/contracts/utils/math/SafeMath.sol/=src/SafeMath.sol
@openzeppelin/=lib/openzeppelin-contracts/
With the remappings arranged like above, all imports including the SafeMath one crash whereas when arranged the other way around, all imports work except for the SafeMath one.

+1
"@ensdomains/=lib/evmgateway/"
"@ensdomains/ens-contracts/=lib/ens-contracts/"
"@ensdomains/buffer/=lib/buffer/"
the first one works but the following ones don't

Just switch to node modules guys - check out my Foundry template.
https://twitter.com/PaulRBerg/status/1736694426578137247

this apps full of bugs haha.

any progress on this issue? Still having it a year later.


set runtimeClassName on pod spec

What does this PR do?
This PR introduces the ability to set a runtimeClassName attribute for pod spec.
It fixes https://github.com/traefik/traefik-helm-chart/issues/991
Motivation
I want to fix https://github.com/traefik/traefik-helm-chart/issues/991
More

[x] Yes, I updated the tests accordingly
[x] Yes, I ran make test and all the tests passed

Is it expected this PR is still a draft ? Also cf. my comment

I wanted to wait for a response in the original issue if this is the intended fix which seems to be. I push the PR forward now.


#1: add support for horses in database initialization

Adds horse as a type in DB

/paulo - can you please review this pull request?

/paulo - can you check now?


macOS wallet fail

The wallet fails on Monteray, just splash screen opens for a second then terminal returns "Posix Signal: Bus Error: 10"
I gave it full disk access still fails.
Saved Apple's Problem Report to a gist all specs are there

Are you using the release candidate? <IP_ADDRESS> Is broken I've just removed it from releases.
The Candidate build here should work in the meanwhile https://github.com/Raptor3um/raptoreum/releases/tag/<IP_ADDRESS>

We appreciate the debug log by the way it confirms the earlier issue!


[20566] Update fastrtps migrated headers

This PR updates fastrtps migrated headers:
fixed_size_string.hpp
md5.h
SerializedPayload.h
Related PR:

https://github.com/eProsima/Fast-DDS/pull/4518

Merge after

https://github.com/eProsima/Fast-DDS-Gen/pull/317

CI will fail until https://github.com/eProsima/Fast-DDS-python/pull/109 is merged

Manual CI with correct branches here


Microsoft tools: change relative imports to top-level absolute imports

Change the module imports from relative imports to top-level absolute imports for the Microsoft tools.
Moving any of the Microsoft tools that used relative imports to the scons site tools folder would fail on import (i.e., the relative import paths become invalid when the tools are moved).
Internal changes: no test and/or documentation changes necessary.
The first in a sequence of smaller PRs from #4409.
Contributor Checklist:

[x] I have created a new test or updated the unit tests to cover the new/changed functionality.
[x] I have updated CHANGES.txt (and read the README.rst)
[x] I have updated the appropriate documentation

Hmm, as to the rationale, not sure why things which are part of the SCons distribution need to support being moved out of the distribution.
That said, for the individual tool modules, it does seem okay to import SCons.Tool.MSCommon as this proposes, doing this a relative import is not necessary.

Hmm, as to the rationale, not sure why things which are part of the SCons distribution need to support being moved out of the distribution.

To (easily) customize an existing tool outside of the scons source tree (e.g., additional functionality, bug fixes, debugging, etc),  The tools can be copied to the site tools folder and modified as desired.
For example, depending on how older msvc versions are excised from the code base with regard to the backend tools, there may be utility for a limited set of users in using ms tools from a previous release of scons (modified if necessary) and bypassing the front-end msvc selection for legacy builds.

An example of a locally customized tool is shown below based on a recent mailing list discussion for the microsoft assembler and the masm tool (quick and dirty, just for fun, and likely bug-riddled).
32-bit targets use 'ml' and 64-bit targets use 'ml64'.  However, in recent msvc releases, a 32-bit build will have both 'ml' and 'ml64' on the path.  One would want the first eligible program found in the path.
The example below assumes that the MSVC tool has already been configured (i.e., msvc is loaded before masm).  Otherwise, more work would need to be done.
A custom masm.py tool that attempts to find the correct binary could be something like the following:
...
import SCons.Defaults
import SCons.Tool
import SCons.Util

ASSuffixes = ['.s', '.asm', '.ASM']
ASPPSuffixes = ['.spp', '.SPP', '.sx']
if SCons.Util.case_sensitive_suffixes('.s', '.S'):
    ASPPSuffixes.extend(['.S'])
else:
    ASSuffixes.extend(['.S'])

ml_progs = ['ml64', 'ml']  # prefer ml64 if none found in generate (exists ignored)

def detect_ml(env, progs):
    program = None
    found = []
    for prog in progs:
        p = env.WhereIs(prog)
        if p:
            found.append((prog, p))
    if found:
        if len(found) == 1:
            program = found[0][0]
        else:
            found_paths = {
                os.path.normcase(os.path.split(t[-1])[0]): t[0]
                for t in found
            }
            env_path = env['ENV']['PATH']
            if SCons.Util.is_String(env_path):
                env_path = env_path.split(os.pathsep)
            for p in env_path:
                p = os.path.normcase(p)
                found_prog = found_paths.get(p, None)
                if not found_prog:
                    continue
                program = found_prog
                break
    return program

def generate(env) -> None:
    """Add Builders and construction variables for masm to an Environment."""
    static_obj, shared_obj = SCons.Tool.createObjBuilders(env)

    for suffix in ASSuffixes:
        static_obj.add_action(suffix, SCons.Defaults.ASAction)
        shared_obj.add_action(suffix, SCons.Defaults.ASAction)
        static_obj.add_emitter(suffix, SCons.Defaults.StaticObjectEmitter)
        shared_obj.add_emitter(suffix, SCons.Defaults.SharedObjectEmitter)

    for suffix in ASPPSuffixes:
        static_obj.add_action(suffix, SCons.Defaults.ASPPAction)
        shared_obj.add_action(suffix, SCons.Defaults.ASPPAction)
        static_obj.add_emitter(suffix, SCons.Defaults.StaticObjectEmitter)
        shared_obj.add_emitter(suffix, SCons.Defaults.SharedObjectEmitter)

    ml = detect_ml(env, ml_progs)

    env['AS']        = ml if ml else ml_progs[0]
    env['ASFLAGS']   = SCons.Util.CLVar('/nologo')
    env['ASPPFLAGS'] = '$ASFLAGS'
    env['ASCOM']     = '$AS $ASFLAGS /c /Fo$TARGET $SOURCES'
    env['ASPPCOM']   = '$CC $ASPPFLAGS $CPPFLAGS $_CPPDEFFLAGS $_CPPINCFLAGS /c /Fo$TARGET $SOURCES'
    env['STATIC_AND_SHARED_OBJECTS_ARE_THE_SAME'] = 1

def exists(env):
    return detect_ml(env, ml_progs)
...

DISCLAIMER: The masm.py tool was not actually tested.  The _detect_ml function had limited testing.

I don't see any harm and certainly some utility in this change so merging.


relative_mse can only handle 2 arguments

I tried to run an example via python3 full_size_with_crops.py
I get the following error:
Traceback (most recent call last):
  File "/home/vertexwahn/dev/figure-gen/examples/full_size_with_crops.py", line 6, in <module>
    figure = FullSizeWithCrops(
  File "/home/vertexwahn/.local/lib/python3.10/site-packages/figuregen/util/templates.py", line 148, in __init__
    self._errors = [
  File "/home/vertexwahn/.local/lib/python3.10/site-packages/figuregen/util/templates.py", line 149, in <listcomp>
    self.compute_error(reference_image, m)
  File "/home/vertexwahn/.local/lib/python3.10/site-packages/figuregen/util/templates.py", line 238, in compute_error
    return image.relative_mse(method_image, reference_image, 0.01)
TypeError: relative_mse() takes 2 positional arguments but 3 were given

It seems that templates.py make use of relative_mse which can only handle 2 arguments.
    def compute_error(self, reference_image, method_image) -> Tuple[str, List[float]]:
        return image.relative_mse(method_image, reference_image, 0.01)

Any hints on how to get around this?

Should be fixed with a8e48188cabdd3a3e6abebfacf57a3b1ac158af5

After the fix I get
ERROR: pdflatex reported an error. The .pdf file was copied anyway but likely contains errors.
Traceback (most recent call last):
  File "/home/vertexwahn/dev/figure-gen/examples/full_size_with_crops.py", line 21, in <module>
    figuregen.figure(figure, width_cm=17.7, filename="full_size_with_crops.pdf")
  File "/home/vertexwahn/.local/lib/python3.10/site-packages/figuregen/figuregen.py", line 420, in figure
    backend.generate(grids, width_cm * 10, filename)
  File "/home/vertexwahn/.local/lib/python3.10/site-packages/figuregen/backend.py", line 89, in generate
    self.write_to_file(result, filename)
  File "/home/vertexwahn/.local/lib/python3.10/site-packages/figuregen/pdflatex.py", line 80, in write_to_file
    shutil.copy(pdf_filename, filename)
  File "/usr/lib/python3.10/shutil.py", line 417, in copy
    copyfile(src, dst, follow_symlinks=follow_symlinks)
  File "/usr/lib/python3.10/shutil.py", line 254, in copyfile
    with open(src, 'rb') as fsrc:
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/tmpcxe2rfix/figure.pdf'

Solved my problem. The issue can be closed. ;)


AWS/EC2 giving only 2 datasets, while 3 datasets were expected

Using the dafault values for Interval (15 minutes) and Period (5 minutes).
AWS/RDS is providing 3 datasets for the above Interval, while AWS/EC2 is only giving 2 datasets.

I'm having the same issue with AWS/EC2 metrics.  Put it on 5min/5min and 75% of my instances never return metrics, even though they're there.  I have verified time is properly set.

@dennis-bsi I too was suspecting the slowness of Cloudwatch to return the metrics. But for RDS, I was receiving all. After moving back the interval vindow to 5 minutes, Im getting all the datasets, Thanks.

I'm experiencing exact same problem, and I will have to modify code as above. Is it possible to release new version with 'delay' parameter ?


Serializations errors with default configuration

Upon initial setup of pycascading i could not get the examples to run, instead the following exception would be spit out:
WARN mapred.LocalJobRunner: job_local_0001
java.lang.NullPointerException
at org.apache.hadoop.io.serializer.SerializationFactory.getDeserializer(SerializationFactory.java:77)
at org.apache.hadoop.mapred.MapTask.getSplitDetails(MapTask.java:392)
at org.apache.hadoop.mapred.MapTask.runOldMapper(MapTask.java:411)
at org.apache.hadoop.mapred.MapTask.run(MapTask.java:371)
at org.apache.hadoop.mapred.LocalJobRunner$Job.run(LocalJobRunner.java:210)
The class it is attempting to deserialize is cascading.tap.hadoop.MultiInputSplit, which should be handled by org.apache.hadoop.io.serializer.WritableSerialization but that is not available to the SerializationFactory for some reason.
The serializations available(as per jdb) are com.twitter.pycascading.bigintegerserialization.BigIntegerSerialization and cascading.tuple.hadoop.TupleSerialization
Adjusting pycascading/java/src/com/twitter/pycascading/Util.java:79 where the io.serializations property is set to include WritableSerialization, or commenting the line out completely, allows the examples to run to completion.  I'm not entirely sure why WritableSerialization would be included when that line is commented out, but not when its used.
Software versions:
cascading-1.2.4-hadoop-0.19.2+
hadoop-<IP_ADDRESS>
jython-2.5.2

The BigIntegerSerializer class does convert BigIntegers (which is how Jython represents Python integers) to Writables. WriteableSerialization should be provided by Hadoop by default, so it seems Cascading is not picking it up for some reason with your setup, or when we provide the serializers in io.serializations the defaults get overwritten.
We do need the BigIntegerSerializer though, otherwise we have to cast every Python integer to Java Longs in the Python UDFs before returning them to the output collector. I'm a bit surprised the examples run when you comment this out though. For now I'll add the WritableSerialization to Util.java, and will look for the real reason.
Which example is it where you have this problem? Is it the word count? If so, could you try to cast the len(...) expression to a Long() in line 40, and see if the error persists?
Thanks for the catch and the explanations.

Sorry by word count I meant reduce.py above...

I have no idea why later Hadoop releases are dropping WritableSerialization.
Cascading 2.0 wip-183 or later should magically include it if omitted.


Move/rename file as a command

I like to use files using the VS Code file explorer, as this is the only way to move/rename a file and then have TypeScript prompt you if you would like to update imports.
However, in a large project, it gets extremely difficult to move files using drag and drop. Folders tend to get very large, with multiple levels of nesting.
Is there an alternative way to move a file? Perhaps this could be done as a command? I'm aware some extensions provide this, however they use the file system directly, which means TypeScript won't prompt to update imports for the moved/renamed file.

The new version adds support for "Cut" to the context menu, which is a big help: https://code.visualstudio.com/updates/v1_31#_cut-command-in-explorer-context-menu.

To rename a file I use that extension : https://github.com/sleistner/vscode-fileutils
It can be useful.

Does that trigger TS's "update imports"?

I try with es6 imports, no, it just rename file.


{CI} Disable persist credentials

persistCredentials: true will persist authentication information (such as the GitHub token) into Git’s configuration, which may lead to security risks.
Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.


Upgrade to Bevy 0.13

This was an easy one and largely mechanical. But I am not super familiar with EntityMapper, so it would be great if someone could help test that part.

Can we have a release? Thanks.


query/show user account data [Feature-request]

https://element-hq.github.io/synapse/latest/admin_api/user_admin_api.html#account-data
allows to query for user account data, this information could be helpful in some cases for troubleshooting / etc.

Make it something similar to rooms' state events


Delete current-limitations-for-apps-using-apis-from-the-shell.md

Basically all of these features are supported now - this page has outlived its usefulness.

:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:warning:Warning
View
Details

mixed-reality-docs/building-2d-apps.md
:warning:Warning
View
Details

mixed-reality-docs/development.md
:warning:Warning
View
Details

mixed-reality-docs/app-model.md
:warning:Warning
View
Details

mixed-reality-docs/current-limitations-for-apps-using-apis-from-the-shell.md
:white_check_mark:Succeeded
View

mixed-reality-docs/TOC.md

[Warning] Invalid file link:(~/current-limitations-for-apps-using-apis-from-the-shell.md).

mixed-reality-docs/building-2d-apps.md

Line 96: [Warning] Invalid file link:(~/current-limitations-for-apps-using-apis-from-the-shell.md#universal-apis-not-supported).
Line 120: [Warning] Invalid file link:(~/current-limitations-for-apps-using-apis-from-the-shell.md#universal-apis-not-supported).
Line 147: [Warning] Invalid file link:(~/current-limitations-for-apps-using-apis-from-the-shell.md).

mixed-reality-docs/development.md

Line 0: [Warning] Invalid file link:(~/current-limitations-for-apps-using-apis-from-the-shell.md).

mixed-reality-docs/app-model.md

Line 55: [Warning] Invalid file link:(~/current-limitations-for-apps-using-apis-from-the-shell.md).
Line 129: [Warning] Invalid file link:(~/current-limitations-for-apps-using-apis-from-the-shell.md).
Line 160: [Warning] Invalid file link:(~/current-limitations-for-apps-using-apis-from-the-shell.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix dependency condition is always true when it's checking a value that doesn't exist

Signed-off-by: Chao<EMAIL_ADDRESS>
What this PR does / why we need it:
For each dependency in Chart.yml, the condition will be considered as true if it is checking a value that doesn't exist.
For this example:
...

dependencies:
  - name: subchart
    repository: http://localhost:10191
    version: 0.1.0
    condition: subchart.enabled

...

If subchart.enabled is missing in values.yml, the subchart will be enabled by default which is considered as an unexpected behavior.
This PR fixes the issue, closes #10296
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[x] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

Seems that some unit tests failed in CircleCI, but those are running fine at my local. Does anyone know the reason?


[Gallery - Photos] They still look too low res

Photos don't look high res.

Do you still have this issue @spacedino?
Or do we need @3x sized assets?

make the 2x images bigger and use a png optimizer, i usually try to have 2x somewhere around 1024 x 768

if the default images are too small make the current 2x the default. Using a png optimizer should help you bring down the file sizes.


Cant download a Youtube video - getting <urlopen error EOF occurred in violation of protocol (_ssl.c:590)>

Am Using the latest version and running youtube-dl --version gives me 2018.08.22.
What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)

Description of your issue, suggested solution and other information
I am running the below command for a specific Youtube video that is playable normally in the browser
youtube-dl https://www.youtube.com/watch?v=lEkEi4dp1bU&list=PLFaW_8zE4amOp3pOVCrGO-Ox--q4WShsQ&index=4

And got this error after multiple try
[youtube] lEkEi4dp1bU: Downloading webpage
[youtube] lEkEi4dp1bU: Downloading video info webpage
[youtube] lEkEi4dp1bU: Downloading MPD manifest
[youtube] lEkEi4dp1bU: Downloading MPD manifest
[dashsegments] Total fragments: 948
[download] Destination: Ep4 - React Component Communication - Create File sharing web app with Express & ReactJs from Scratch-lEkEi4dp1bU.f136.mp4
[download] 100% of 120.78MiB in 13:45
[dashsegments] Total fragments: 1012
[download] Destination: Ep4 - React Component Communication - Create File sharing web app with Express & ReactJs from Scratch-lEkEi4dp1bU.f140.m4a
[download]  27.2% of ~76.41MiB at  2.81MiB/s ETA 01:48ERROR: unable to download video data: <urlopen error EOF occurred in violation of protocol (_ssl.c:590)>

The output of curl -v https://www.youtube.com/watch?v=lEkEi4dp1bU&list=PLFaW_8zE4amOp3pOVCrGO-Ox--q4WShsQ&index=4 is below
curl-output.txt

Most likely temporary network issue on your side.


Update string ref to use createRef

I'm not sure if I need to commit the dist/ directory, let me know if I need to remove that.
Closes #13

Thanks! Just one small fix.
I was committing the dist directory so the demo works.

Thanks!

Glad to help! Could we get a release for this?

v1.2.0 is released!


Request updating code

Thank you very much for open sourcing your FSM.
Would you mind updating code in github according to your talk in ACCU 2019? fsm.h in ACCU 2019 is more updated than fsm.h in github
https://youtu.be/JGYxOieiZnY
Thank you

Could you please describe in more detail which part do you miss? During my talk. I provided a few implementations alternative and the implementation in this repo is only one of the possibilities here.

Sorry, I misunderstood implementations alternatives as the more updated code.


Add E2E tests using Cypress.io

Hi everyone,
What do you plan to do for E2E testing? Can I propose using https://www.cypress.io/ that is going to be open sourced soon and thus is a good match for this open source project?
I took liberty of writing a few tests against https://swimlane.github.io/ngx-ui/ and you can see the tests in https://github.com/bahmutov/e2e-ngx-ui repo.

The testing tool itself is a dev dependency, works on Mac and CI (Windows is coming) and you can just keep it running as you work with the code. A typical test for a form for example looks a lot like Mocha/Jasmine unit test
https://github.com/bahmutov/e2e-ngx-ui/blob/df7ede09e8af8d2e402fa5fdf89d9540f134e295/cypress/integration/basic-spec.js#L6-L14
  before(() => {
    cy.visit('/')
    cy.get('.page-loader')
      .should('not.be.visible', {timeout: 20000})
  })
  it("loads", () => {
    cy.contains("Angular Style and Component Library")
  })

There is huge number of commands and assertions included, plus the docs at https://docs.cypress.io are really good.
I setup CI on Circle, but any CI would work. We also have free dashboard for OSS projects which can store screenshots / test results / video recording of runs. For this project it is at https://dashboard.cypress.io/#/projects/psie99/runs

Videos of runs is really useful to debug failing CI tests

Let me know if this is something the project needs. The tests could be part of the swimlane/ngx-ui repo or some separate repo like swimlane/ngx-ui-e2e for example, whatever makes sense for you. We at Cypress would be happy to promote our free (and soon OSS) tool by helping people start E2E testing.

Hello Gleb,
Thank you for the information.  I am working on getting Cypress up an running across several of our projects (both open and internal), this obviously takes some time.  ngx-ui-e2e is very helpful and I will try to get it into ngx-ui ASAP.

@Hypercubed of course! and I will be very happy to answer any questions on https://gitter.im/cypress-io/cypress and the docs are a good resource: just search there, a good answer usually shows up :) https://docs.cypress.io


Normalize MultiPolygon output in h3SetToLinkedGeo

Fixes #53
Adds an internal function normalizeMultiPolygon to align output of h3SetToMultiPolygon with the GeoJSON spec, ensuring that each polygon has only one outer loop, followed by holes. Note that this is more correct than the implementation currently in geojson2h3, as it should properly handle all topologies, including nested donuts.
I will update with a benchmark, but in most simple cases (no holes) this should be close to master performance due to early exits/skipped loops.

Coverage remained the same at 100.0% when pulling 4f6d90683f2ca3a6f452458ad0f75f934161aa86 on nrabinowitz:normalize-multipolygon into 469a50bcb34c612a2bee7f0142036237f3888a27 on uber:master.

@isaacbrodsky - all comments addressed, thanks for the review!


Avoid readlink -f on macOS

As noted by https://github.com/tensorflow/custom-op/issues/7#issuecomment-459502616, readlink -f fails on macOS. After this change, I can build the pip package on macOS without problems.

@googlebot I signed it!

On 6 Mar 2020, at 2:50 pm, googlebot<EMAIL_ADDRESS>wrote:
@googlebot I signed it!

Thanks for the PR @prouast! Just one nit.


Computed Watchers Not Firing

Version
2.5.16
Reproduction link
https://vuejs.org/v2/guide/computed.html#Watchers
Steps to reproduce
Use the demo for entering a question on the Docs page using a Amazon Kindle 7'' or Android Tablet (I used a Lenovo Tab 3 7'' version)
https://vuejs.org/v2/guide/computed.html#Watchers
What is expected?
When typing in the field the value is evaluated after each letter is tapped when using a browser on a tablet, as per the demo instructions.
What is actually happening?
Nothing is evaluated until the space bar is tapped or a question mark / exclamation mark is selected from the keyboard.

I use this to create a drop down list of options when a user starts typing into a text field using Android tablet specifically Amazon Fire.
The user starts typing and after letters are entered the list is shown..
After updating then the list is only shown if the user taps the space bar or the enter key on the on screen keyboard.
This works as expected when using a browser on a computer.
I have tested my code on Mac using Safari, Chrome and Firefox and no errors are in the console.
On Amazon Fire and it works as expected when on version 2.5.13
Updating my VueJS to version 2.5.16 and this stops working on the tablet and requires the space key, exclamation mark, question mark or enter key to be tapped for the evaluation to fire/run when using the tablet device.

hi, may I try to fix this as my first bug?

@zaggy Sure! 😀

Just a guess but would it be this issue that was fixed that causes it? https://github.com/vuejs/vue/pull/7121
I'm not sure how I would exclude that from a version for testing.

What's the status of this?

@zaggy are you working on it? if not I would like to pick this up! 🙂

is this still an issue?

Yes it is. I'm using the standard help page / demo page on a Kindle Fire and you have to tap the enter key when typing letters before the check is made. A question mark or an exclaimation mark after a word does fire the check the same as tapping the enter key.

Just a note this may have to do with v-model preventing temporary updates during a composition (which is necessary for some input methods e.g. Chinese) - and some devices may be firing composition events even for normal English keyboards. That said I don't have a Kindle Fire so it's hard for me to reproduce.

I could replicate also on a Lenovo Tab 3. On both I have spell check / suggestions disabled in off chance that would help but it doesn't.

It okay on an old Android 5 device I tested with but the Kindle Fires have the issue. Even with all the spelling options disabled there is an underline on the text you type in the field. Once you press space bar or enter that line goes and then the event is fired and in my case the dropdown shown.

Is this reproducible with Android emulation? If so, would be a fun bug to work on. Had a similar issue with an application for work.

Hope you've fix the bug


[CI] test codegen for all Move sdks

Motivation
Close a gap in our CI testing regarding the transaction-builder-generator.
Test Plan
CI

I think we should keep these tests ignored (but ensure they're run in CI) or look at the approach taken by the move-prover tests so that we don't start forcing devs to install go and java in order to get tests to start working.

@bmwill How about I land this PR without removing the #[ignored] and without triggering the new -s option anywhere to begin with?

Make sure your dependencies are install in CI (so i suppose ensure that your -s flag is used in CI) and then add a step to the unit-test workflow in .github/workflows/ci-test.yml to run cargo test -p transaction-builder-generator --ignored

Make sure your dependencies are install in CI (so i suppose ensure that your -s flag is used in CI) and then add a step to the unit-test workflow in .github/workflows/ci-test.yml to run cargo test -p transaction-builder-generator --ignored

Tentatively done. Not sure if it's going to be run every time or not.

@bmwill @rexhoffman I followed @bmwill's suggestion and added an action step to run the non-Rust unit tests (and not force all engineers to install Go etc). Is this PR good to land?

I wonder why non-rust-lint was cancelled here. (It's the second time I see that)

Yikes, so the lint failure is due to the dev-setup.sh script changes.   This job intentionally didn't run on the docker image (the big beefy images from gha have most of the tools we would need, and a range of versions, but not all, not the pinned nightlies, rust tools, etc).  But we had to quickly switch to vms on circle ci when our machines got nerfed.   This job was my way of making sure we could always go to the vm if we had to.
https://github.com/diem/diem/pull/7635/checks?check_run_id=1955251525

I think you just need to up the time out, 5 minutes is all it used to need, but since it's installing a bunch of python and maybe java now, maybe bump to 15-20?  It's also a much smaller machine, we may need to make it an xl now like all the other jobs.
https://github.com/diem/diem/blob/master/.github/workflows/ci-test.yml#L84

gumble it is a container build now.... when did I do that....
anyway:
    runs-on: ubuntu-latest
    timeout-minutes: 5

to
    runs-on: ubuntu-latest-xl
    timeout-minutes: 10

should be enough.

/land

:exclamation: Land has been canceled due to this PR being updated with new commits. Please issue another Land command if you want to requeue this PR.

/land

Mmm tests::shared_mempool_test::test_broadcast_dependencies is failing but that's unrelated.


[release-1.6] disable enforcing success rate based outlier detection

Manual cherry pick of #25534
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@jacob-delgado can you please take a look at this manual backport for 1.6?


cordova.plugins is undefined

Hello,
I'm new to working with Cordova plugins, please forgive if it's a newbie question.
For some reason after installing the plugin, and trying to use it(in a device ready state), i get an error saying can't read "permissions" of undefined. Which of course refers to cordova.plugins. I searched far and wide, but can't seem to find a solution. Any kind of advice would be appreciated.
My setup is Nodejs with a custom compiler, within Cordova. When deployed on an android emulator, debugged with Chrome, under the sources tab, i can see the permissions in the plugins folder, but it still not able to access it.
Please reply if you can.
regards, Ferenc

try
cordova plugin prepare

@NeoLSN Thanks, I already tried that, unfortunately it does not help.

@NeoLSN Thank you, I managed to find the problem. For some reason, the function that was supposed to be called after the device was ready, somehow got called before. It was a bug in our custom compiler.


Use default region if no region is found in PVKey.Features

Fixes #928

Thanks so much for this @isra17 ! assigning to @nealormsbee for a review.

Performed additional testing by:

Removing region labels from one of our AWS clusters
Confirming the price mismatch
Deploying an image with these changes
Seeing that the price is correct with or without the PV label
Other prices in the response were unaffected

Merging this. Thanks for the contribution, @isra17


Multiple tables by prefix

A commonly used technique of LevelDB is to segment data by key prefix:
!users!a1234
!users!foobar
!posts#active!blabla

db.createReadStream({ gt: '!users!', lt: '!users!\xff' })

The mimics the "table" functionality of SQL database.
However most SQL databases use Btree while LevelDB uses LSM storage structure.
The current implementation of SQLdown store key-values into one big table, which is not an ideal scenario for SQL.
In theory it should be possible to split SQLdown storage into multiple tables such as
_leveldown_!users!
_leveldown_!users#meta!
_leveldown_!posts!

get(), put(), del(), will select table according to the prefix it matches. While createReadStream() may join multiple tables accordingly, something like
select key, value from "_leveldown_!users!", "_leveldown_!posts!" order by key

The implementation is definitely not trivial, just some ideas aligning SQLdown to SQL table storage structure.

One of the things about the Sublevel method of of prefixing tables is that
it allows you to still access them like a single table so it would be very
non trivial to implement.
That being said a btree is probably just as performant here as well are
only using it for ranges and key lookup. One place we might be able to
improve things could be a combined index on the key and sequence number
On Wed, Aug 5, 2015, 11:58 PM Adrian C Shum<EMAIL_ADDRESS>wrote:

A commonly used technique of LevelDB is to segment data by key prefix:
!users!a1234!users!foobar!posts#active!blabla
db.createReadStream({ gt: '!users!', lt: '!users!\xff' })
The mimics the "table" functionality of SQL database.
However most SQL databases use Btree while LevelDB uses LSM storage
structure.
The current implementation of SQLdown store key-values into one big table,
which is not an ideal scenario for SQL.
In theory it should be possible to split SQLdown storage into multiple
tables such as
leveldown!users!
leveldown!users#meta!
leveldown!posts!
get(), put(), del(), will select table according to the prefix it
matches. While createReadStream() may join multiple tables accordingly,
something like
select key, value from "leveldown!users!", "leveldown!posts!" order by key
The implementation is definitely not trivial, just some ideas aligning
SQLdown to SQL table storage structure.
—
Reply to this email directly or view it on GitHub
https://github.com/calvinmetcalf/SQLdown/issues/28.


[8.x] Dynamically load relationship counts

This PR enables relationship counts to be loaded dynamically.
This is useful if you're checking counts in multiple places and you don't want to run multiple queries.
For instance, let's say you have an Event model with a people_per_order attribute and a HasMany::Order relationship. Now, if you need to display both the number of orders the event has and how many people are going, you could do something like this, but you'll run a count query twice.
{{ $event->orders()->count }}
{{ $event->orders()->count() * $event->people_per_order }}

In this case, sure, you could just load orders in-memory and run the count on the collection, but in more extreme cases that may become a hassle due to the load on the DB.
Instead, with this PR, you could do:
{{ $event->orders_count}}
{{ $event->orders_count * $event->people_per_order }}

I'm very sorry but I wasn't completely sure where to put the test, and I also wasn't sure wether to throw an exception if someone tries to access the count of a relation that does not exist.
I'm also setting the target branch as master because this is a breaking change IMO.
Thanks for the patience! :)

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Add support for an optinal type guard for the query functions

For example for query something like:
export async function query<T extends object = unknown>(
  client: Pool | PoolClient,
  sql: SqlQuery,
  guard: (value: unknown): value is T = (value) => true
): Promise<T[]> {

So by default you'd get unknown as T, but you could change that by giving a guard function. You get these guards for free when using runtypes or io-ts, so it shouldn't be too much of a burden.
If performance is an issue, the use of the guard func to validate the query result could be dependent on NODE_ENV for example

I was thinking of something similar today, although I think I'd rather make them separate functions.
Maybe we could have query(client, sql) = queryWith(identity, client, sql), queryOne(parser, client, sql) === queryOneWith(identity, client, sql) and so on.

That could work too. My main concern though is to have the library not let me shoot myself in the foot by allowing unchecked type coercion or any types coming out of these kinds of functions.
So IMO the return type should be unkown (unknown[] for the ones returning multiple elements) and only allow changing the type by somehow validating what is returned

@googol I tried to tackle this in https://github.com/sluukkonen/possu/pull/17
Let me know what you think.

Fixed by #17


Deprecate use_std cargo feature: use std instead .

Related to #657 .

@gnzlbg: no appropriate reviewer found, use r? to override

cc @dtolnay this essentially fixes #657 , once we release the next breaking version of libc, we can remove the use_std feature, completing the fix.

@bors: r+

:pushpin: Commit 4ac26afa83e83bd1da49b637af814f56515c8184 has been approved by gnzlbg

:hourglass: Testing commit 4ac26afa83e83bd1da49b637af814f56515c8184 with merge bb41337b63f92bb3a9e66afb12e95acffbe5c785...

:broken_heart: Test failed - checks-travis

@bors: retry

:hourglass: Testing commit 4ac26afa83e83bd1da49b637af814f56515c8184 with merge b722d8c691fd5012253578c0a60fe3d98f100e64...

:sunny: Test successful - checks-cirrus, checks-travis, status-appveyor
Approved by: gnzlbg
Pushing b722d8c691fd5012253578c0a60fe3d98f100e64 to master...


Crash on world load containing sorting machine

Hi,
I've had this issue where every time I would place a sorting machine into my world and reload that world, the game would crash. I would be able to load back into the world, but my sorting machines would reset and upon quitting the world and reloading, the crash would occur again. I have my sorting machines set up in this manner: https://i.imgur.com/SCMo1Gw.png. This doesn't occur when the machines are set up so one machine into several different colours, but i would like a large variety of items in one chest at once.
I am playing on Minecraft 1.12.2 and using Teckle verion 1.12-5.394. Here is an example of a crash log due to this bug: https://gist.github.com/Wupp/5a6b49962b28e35483b135bb3218ca1f

I need a bit more info if I'm going to go reproduce this. Show the inventory of each sorting machine so I can copy to the best of my abilities. Not sure what those posts are by the way, if it's a redstone clock they're not needed as the sorting machine has one built in.

Here's a video where I set up the system causing the crash: https://www.youtube.com/watch?v=I8GrhzrjxEw&feature=youtu.be . The machine at the beginning was there to demonstrate that in that particular setup, the machine would not crash, and reload to prove this. I build the machine, and let it function for a short period of time, then reload the world, which crashes the game. When I relaunch, the world loads, but the sorting machines inventory has been cleared. I then attempt to relaunch without further touching the sorting machine, which crashes the game again. After relaunching for the last time, i show that the first machine is not affected, as its inventory is not cleared.
BTW thanks for the tip about the automated timer in the machine, I had no clue!

I just went ahead and remade the structure you have in the video but couldn't get the same result.
Replace your local teckle jar with the one hosted here and give me the full log file once you experience the crash. It has a small debug message that I can use to try and figure this out.

I tried replacing the jar and the mod wasn't identified by the game. When I tried opening it with WinRAR, it wouldn't open and suggested "the archive is either in unknown format or damaged". For reference the release version opened fine.

Yeah apparently Firefox decided to pause the download without telling me and just made an empty file, because "people don't usually download this file".
Try this one: https://unascribed.com/f/e9d57b9c_Teckle-1.12-5.409-dirty.jar

Here you go: https://gist.github.com/Wupp/72be141a4ecef2f0cad9b7d9b3e4a40a

We'll need the full log, not just the crash report.

Oh, sorry about that, I should've read more thoroughly. I believe this is what you're looking for though: https://gist.github.com/Wupp/0afa8cbcdeb8864f1cdc69622c28de4e

Yep, that contains the debug log darkevilmac was talking about. :+1:

This bug is really weird, the world isnt null and there's no way the filter id or buffer id is null. Whipping up a new build with even more debug data.

@Wupp use this.

Interestingly enough, I've replicated the same setup from the previous versions to produce the bug, and it hasn't crashed. I'll continue to use this version to test, so if it starts crashing again and I can figure out the cause, I'll post back here. Otherwise, I have no clue as to how to reproduce the bug in this debug version.

This is just weird as that build is identical short of some log calls.
On Oct 27, 2017, 8:26 PM, at 8:26 PM, Wupp<EMAIL_ADDRESS>wrote:

Interestingly enough, I've replicated the same setup from the previous
versions to produce the bug, and it hasn't crashed. I'll continue to
use this version to test, so if it starts crashing again and I can
figure out the cause, I'll post back here. Otherwise, I have no clue as
to how to reproduce the bug in this debug version.
--
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub:
https://github.com/elytra/Teckle/issues/26#issuecomment-340135961

Somehow I got the same crash (using the version on jenkins -.-)

Is this issue still happening or has it been magically fixed?


Bond Bar out of margin

UI Version: 0.9.3
Description:
Disable or hide bond bar if the user is revoked.
See picture below.

 Remove the scrubber disk. Solves a whole class of bugs.


[Icon Request] Support for APL

[x] I'm sure this issue is not a duplicate.

[x] I'm submitting an icon request:

Type: languageId
Icon Name: apl
Sample original Icon: N/A
Language ids: apl

Language Extensions: https://marketplace.visualstudio.com/items?itemName=OptimaSystems.vscode-apl-language

More info: https://en.wikipedia.org/wiki/APL_(programming_language)

It might help to differentiate between "traditional" APL and Dyalog APL. The latter is heavily modernised and supports many concepts which never existed in the original language: classes, control-flow (:If, :Else), and object-oriented programming.
For ordinary APL, no "official" logo exists, but the "grade up" symbol (⍋) is probably one of the more distinctive shapes of the language's alphabet:

(Also, the author of that language extension ported my work. Good to see it's going places. 😀)

@Alhadis It's well known, you are a pioneer, Sir. 😄

APL now has a new logo and dyalog is also a version of APL massively used.
Information about new apl logo: https://aplwiki.com/wiki/APL_logo
Is it possible to reopen this issue?


fix: 修复_UniFilePickerOnDeleteEvent类型缺少index字段

Description 描述
在官网组件文档里没有写明删除事件里有index字段,但是在插件市场组件库里更新说明里写了,实际返回参数里也带了index字段

Linked Issues 关联的 Issues
Additional context 额外上下文

Summary by CodeRabbit

New Features

Added the ability to track the index of a deleted file in the file picker component.

Thank you!


Cannot update metadata of already uploaded files

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Windows
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[X] 💥 Microsoft Edge
[X] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

Browser version: Latest Edge, latest Chrome
SPFx version: 1.15, but the issue is also present when serving our dev build with 1.18
Node.js version: Various

Describe the bug / error
Unsure if this is actually a problem in SPFx or something Sharepoint-internal, but we're experiencing the issue in the context of our SPFx solutions.
The gist of the issue is that we seemingly cannot update metadata of already uploaded files in Sharepoint. When uploading a fresh file it is possible to set various column values, but edits after it has been uploaded are seemingly rejected. Inspecting the requests made we see that there are POSTs to a ValidateUpdateListItem() method, and this returns a 200, but nothing happens. Additional versions are created in the version history of the file, but they contain no changes.
We began noticing the problem yesterday across multiple tenants, and we have made no recent code changes or updates to our own .sppkgs.
Further testing has revealed that the issue is also present when editing items directly from Sharepoints own list view, see below.
Steps to reproduce

Upload a file somewhere (setting metadata during upload still works):

Attempt to update a value, as you can see the UI (optimistically?) updates to the new value initially:

Shortly afterward the UI reverts to the original value:

A new entry is created in the version history, but with no changes:

Here's the request payload for ValidateUpdateListItem:

And here's the response object:

Expected behavior
We expect the document/file to save and persist any metadata changes, as has been the case until this week. 😊

Whatever caused this seems to have been fixed or rolled back during the night. Everything is working again.


Is there a way to apply flip before preventOverflow?

CodePen demo

https://codepen.io/Justineo/pen/QaypBM
Steps to reproduce the problem

with placement set to bottom-start
when the reference is on the right side of the container
the popper is shifted (#pop)

What is the expected behavior?
Provide a way to apply flip when the popper is placed outside the container rather than shift it. (Like #pop2 in the CodePen demo.)

What went wrong?
Sometimes we want to flip the popper when it's outside the container, not when it overlaps with the reference element.
Any other comments?
I've looked into how popper handles this now, but not sure how to handle this.

I'm facing this same problem when trying to implement a navigation menu.
I've tried setting the flip behavior but it doesn't work as expected when the popper element overflows on the right side:
{
  placement: 'bottom-start',
  modifiers: {
    flip: {
      behavior: ['bottom-start', 'bottom-end']
    }
  }
}

In this shot the "Dropdown" in the top bar is the reference, and the expanded drop down menu is the popper element.

Similar issue (another example):
{
    placement: 'bottom-start',
    modifiers: {
        flip: {
            behavior: ['bottom-start', 'bottom-end']
         }
     }
 }

@arechsteiner setting the boundariesElement to viewport will make sure the whole popper is shown
{
  placement: 'bottom-start',
  modifiers: {
    preventOverflow: {
      boundariesElement: 'viewport'
    }
  }
}

setting the boundariesElement to viewport will make sure the whole popper is shown

It's not the desired behavior. This will shift the popper element to the left but we want its right border to align with the right border of the reference element (flip instead of shift).

@Justineo Yep I understand it's not the desirable behaviour but @arechsteiner it will atleast show all the content.

@FezVrasta any news about this feature?


The Asp.Net Web Api get request cannot return a result larger than 20K

Describe the bug
The Asp.Net Web Api get request cannot return a result larger than 20K.
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core '3.0.0-preview3-19153-02' with VS 2019 v 16.0.0 Preview 4.1
Run this cmd dotnet new blazorhosted
With these code changes
Change the Range from 5 to 200 in the following line.

return Enumerable.Range(1, 5).Select(index => new WeatherForecast

The line should look like this
return Enumerable.Range(1, 200).Select(index => new WeatherForecast

Run the App
Click on the "Fetch Data" nav menu item.

See error
No data is displayed.

Expected behavior

The page should display 200 lines of WeatherForecast Data
I would expect to see an exception.

Screenshots
N/A
Additional context
This worked in asp.net core preview 2
Some investigation showed that the call to the WebApi
forecasts = await Http.GetJsonAsync<WeatherForecast[]>("api/SampleData/WeatherForecasts");

was returning garbage.
Output of dotnet --info
.NET Core SDK (reflecting any global.json):
 Version:   3.0.100-preview3-010431
 Commit:    d72abce213

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.17134
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\3.0.100-preview3-010431\

Host (useful for support):
  Version: 3.0.0-preview3-27503-5
  Commit:  3844df9537

.NET Core SDKs installed:
  2.1.503 [C:\Program Files\dotnet\sdk]
  2.1.504 [C:\Program Files\dotnet\sdk]
  2.1.600-preview-009472 [C:\Program Files\dotnet\sdk]
  2.1.600-preview-009497 [C:\Program Files\dotnet\sdk]
  2.1.600 [C:\Program Files\dotnet\sdk]
  2.2.103 [C:\Program Files\dotnet\sdk]
  3.0.100-preview3-010431 [C:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.0.0-preview3-19153-02 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.1.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.8 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.1 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.0.0-preview3-27503-5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.0.0-preview3-27504-2 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

Damm Github ! Lost all my issue details when navigating to "other possible duplicate issues"

Thanks for contacting us, @AceTheWiz.
@javiercn, can you please look into this? Thanks!

Thanks for contacting us, @AceTheWiz. This is a dupe of https://github.com/aspnet/AspNetCore/issues/9322 and have is now fixed.


Improve performance

Hello! Thanks again for this library, it has been very useful.
Over in the Exercism test runner we've hit a problem where large inputs would cause gap to take longer to complete than we have available. I'd really like to continue to use this package, so we'll have to make some performance improvements.
I've spent the morning trying to understand how we might improve the performance of the algorithm used, but it is beyond my understanding.
I looked into what Elixir does and they implement an algorithm from the paper "An O(ND) Difference Algorithm and Its Variations" by E. Myers. I benchmarked it with a 360 character string with a few randomly placed edits and it ran in 1/1950 the time with 1/570 the memory usage, even with a performance hit from not consolidating Elixir protocols.
Looking at the implementation it seems like it could be adapted to Gleam straightforwardly, perhaps we can steal follow their lead and use the same algorithm.
Cheers,
Louis
https://github.com/elixir-lang/elixir/blob/74bfab8ee271e53d24cb0012b5db1e2a931e0470/lib/elixir/lib/list.ex#L1047-L1174

Thanks for opening an issue and really nice to see that gap is useful.
I also went back to look at the implementation to see what could be done in order to improve the performance and I really wish that I had written some more comments on the algorithm.
From your benchmarking it seems like a good idea to try to implement the algorithm from Elixirs to improve the performance of gap. I will give that a shot.
/
John

@lpil Thanks for sharing the benchmarking code 🙏
I'm still trying to figure out the Myers diff algorithm in order to implement it in gleam. Meanwhile I did some adjustments where any leading/trailing matches are omitted when running the diff. It's not close to as efficient as the Myers algorithm but gave an improvement of 1/7 in time and 1/6 in memory usage compared to the old implementation on the dataset from your benchmarks. Worst case is of course same as the for the old implementation but should help a bit on long string with similar starts or ends.

So version 0.6.0 didn't live very long ...
Now added Myers diff algorithm (adapted from Elixir). I used the version in Elixir 1.4 as a base (the newer version handled nested diffs which gap anyway doesn't support).
Benchmarks seemed promising (about 1/50 time and 1/20 memory compared to the optimised LCS version) so hopefully this can solve the timeout issues when running on Exercism.

Forgot to close this after the new algorithm was implemented.


Annoying popup

Description
I would get this annoying popup every time I would open neovim on a non git repo folder
I know it has to check if it's a git repo but it's annoying to have it show

Neovim version
NVIM v0.9.1
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Operating system and version
Windows 10
Steps to reproduce
Opening neovim in a non git directory
Expected behavior
To not have 4 notification on at once saying i am not in a git directory
Actual behavior
4 annoying notifications
Minimal config
local setup, neogit = pcall(require, "neogit")
if not setup then
    return
end

neogit.setup {
    auto_refresh = true,
    remember_settings = true,
    disable_insert_on_commit = false,
    commit_popup = {
        kind = "split",
    },
    preview_buffer = {
        kind = "split",
    },
    popup = {
        kind = "split",
    },
    signs = {
        section = { "", "" },
        item = { "", "" },
        hunk = { "", "" },
    },
    integrations = {
        diffview = true
    },
    sections = {
        untracked = {
            folded = false
        },
        unstaged = {
            folded = false
        },
        staged = {
            folded = false
        },
        stashes = {
            folded = true
        },
        unpulled = {
            folded = true
        },
        unmerged = {
            folded = false
        },
        recent = {
            folded = true
        },
    },
    mappings = {
        status = {
            ["p"] = "PushPopup",
            ["P"] = "PullPopup"
        }
    }
}

What does the console say?
Also, which remote are you using for the plugin itself?

This just happens on Windows, and it suppresses the new remote notification. Even after changing it over to NeogitOrg it still pops up. Also, this seems to happen for me every time I open Neogit, not just in a git repo.
The console just says  1 git --no-pager -c color.ui=always --no-optional-locks config --get remote.origin.url (470.000 ms) [stderr 1].
Haven't used Windows since 2de09534a25993550f3aa2abf4c05d2496e55b94 was put out so I'm thinking it happened somewhere in adding the new notification. I'll dig a bit deeper sometime today.

Ok. Thanks for the insights.
Managed to reproduce on Mac by remote.orign.url (mispelled). Perhaps it is just a git config key difference then

6c83d8f should silence it
I'll remove it all together on friday

https://github.com/NeogitOrg/neogit/commit/6c83d8f6ba26a1313199225994857b516ab921de should silence it

That did the trick on my end.

ill check that later when i can

It's ok now and I don't get those anymore
Tho the console still pops up


Timestamps on a query returned with seconds precision

On simple queries (select * from measurement) I get a precision of 1 second on timestamps. The timestamps in my db are nanoseconds.
This is in VS2019 with influxdb-cxx installed via vcpkg.
Seems a bug, but perhaps I'm not using the library correctly?

I guess the time string used for parsing is second only and doesn't respect finer resolution. Furthermore std::get_time() seems not to support milli- or nanoseconds.
Just to be sure, can you verify the actual query returns proper nanosecond timestamps?

Yes it does return nanosecond. mTimestamp actually contains the nanosecond value, but it's casted down to seconds by the return type.

std::chrono::time_point<std::chrono::system_clock> Point::getTimestamp() const
{
  return mTimestamp;
}

Perhaps  use std::chrono::time_point_caststd::chrono::nanoseconds ?

Wait … the received point contains a correct nanosecond timestamp after parsing? Eg. after this line?

I'm using curl and the http protocol.
I compiled boost support in, but I'm instantiating an object via influxdb factory with the following connesion string:
'http://user:pass@influxhost:8086?db=testdb&precision=ms'

I do see the nanosecond precision inside the mTimestamp class attribute. And I see that it's the http protocol that is being used.

Cancel my previous comment. My bad. The timestamp from the query is actually seconds.

No problem :wink:. The current implementation supports only seconds resolution on received points unfortunately.
But it's a good feature, maybe it can be combined with #86.

Here the responseString buffer contains the proper timestamp, with millisecond precision, as requested by the connection url.
namespace influxdb::internal
{
    std::vector<Point> queryImpl(Transport* transport, const std::string& query)
    {
        const auto response = transport->query(query);
        std::stringstream responseString;
        responseString << response;

However
timeString >> std::get_time(&tm, "%Y-%m-%dT%H:%M:%SZ");

casts it to a second precision.
Since you're actually constructing a chrono object, it might be wise to actually convert the string directly without passing trough a tm structure.

No problem 😉. The current implementation supports only seconds resolution on received points unfortunately.
But it's a good feature, maybe it can be combined with #86.

Yes, I need it. For instance, I need to log events with a sub-millisecond precision. They arrive in batches of 10-20 every few seconds, and I must preserve their order. So, nanoseconds are important in this case (I write with another writer).

Since you're actually constructing a chrono object, it might be wise to actually convert the string directly without passing trough a tm structure.

That's true, but the only way to do this (at least I know) requires C++20.

I'm not sure if you prefer to not depend on external libraries. https://howardhinnant.github.io/date/date.html Is an example of a single header file library that does a lot of things, including conversion.
It seems well written and requires C++11 or C++14 for best performance. But, in this simple case, parsing the final part of the timestamp returned by influx is sufficient. Timestring looks like 2021-05-04T06:00:01.219037Z
and a simple search for the dot towards the Z shall be easy.
What concerns me is that all this is a performance nightmare. Those timestamps are already in long-ints inside influx. They're converted to UTC then passed to the client only to be converted back.
Maybe something along these lines is better?:
char *p = timeString.c_str() ;
int res = 0, d ;
while ( *p++ != '.');
while ( (d = *p++) != 'Z')
   res = res * 10 + d - '0' ;
mTimestamp += res ;

Thanks for the suggestion, I'll have a look at it 👍. Especially if it's the pre-C++20 implementation, so we can switch to the STL version later easily.

What concerns me is that all this is a performance nightmare. Those timestamps are already in long-ints inside influx. They're converted to UTC then passed to the client only to be converted back.

Indeed. A way to query the timestamp in an int type would be nice.

Maybe something along these lines is better?:

No C code please.

The date library looks promising. I have ported the time parsing on the date_library branch, which should support more precise timestamps now.
Could you test your use case with that version?

Ok, could not resist ;)
I can confirm that the conversion works on nanoseconds and on microseconds. Good job!
Thank you and thanks to Howard Hinnant.

Perfect, I'll merge to master and create a new release.


Video explaining this codebase

Hi folks!
I created a YouTube video where I do a deep dive/walk-through of this repo (+ the paper).
Maybe someone finds it useful: https://www.youtube.com/watch?v=y7J6sSO1k50
Hopefully it's ok to share it here in the form of an issue, do let me know!

Thank you for the video.
Could you share how to get the model to train a random dataset, say CelebA, along with the commands.


Windows builds on Travis

Maybe we can get rid of the whole appveyor setup.
https://blog.travis-ci.com/2018-10-11-windows-early-release

Would require something like this for now.

Ah, shows how much I know about Travis :)
Close for now and open a new issue once it's feasible?

Sounds good to me.


update to release 2020.1.1

now the we got the PyPI source tarball working, we can distribute from pypi instead

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Feature Request: Multi-Level YAML Tags

Hi,
first of all, i do appreciate your great work with this tracker.
I was just trying to implement a tracker based on my YAML tags. However, my tags are on two levels

So, would it be possible to plot a chart based on one of the tags within the Habits parent tag?
i.e. for Steps, or of Meditation?
thank you!

@diaovidiu, use a dot to refer to a nested value, here is an example: https://github.com/pyrochlore/obsidian-tracker/blob/master/examples/TestFrontmatter.md#deep-values

Thank you @demig00d for your reply.
For i don't know what reason, this is NOT working for me.
Above the structure I have in the daily notes and below, the code i'm using and the error i get.

I was just trying to implement a tracker based on my YAML tags. However, my tags are on two levels

Oh, I missed that you are trying to use booleans as values, there is a known limitation on this https://github.com/pyrochlore/obsidian-tracker/issues/79#issuecomment-948460235, try replacing them with 1 and 0 in your frontmatter.
If the error still occurs, post the frontmatter and query as text here inside the code blocks to make it easier to reproduce.

The same problem is happening to me.

in the end, it worked for me with the dot notation. But, be aware, the same error you might get for different other reasons: i.e. not finding the folder you mentioned, or not finding any data in the folder mentioned.
mind the lowe/capital case as well.
So, check if indeed the folder you are having the daily notes is "Periodic Notes/Daily" or "Periodic/Daily" or "periodic/daily" etc

in the end, it worked for me with the dot notation. But, be aware, the same error you might get for different other reasons: i.e. not finding the folder you mentioned, or not finding any data in the folder mentioned. mind the lowe/capital case as well.
So, check if indeed the folder you are having the daily notes is "Periodic Notes/Daily" or "Periodic/Daily" or "periodic/daily" etc

Interesting. I confirmed that the folder is correct. Moreover, when I change to using unnested frontmatter in the periodic file and in the tracker code block, the calendar appears as expected. So it is indeed the frontmatter nesting that is the issue for me.

Closing as original poster stated problem was resolved for them. If someone else is still having an issue, please try updating to the latest release (just released earlier today 1.11.0) and see if the issue still persists. If so, please open a new issue providing a dataview search and example files.

in the end, it worked for me with the dot notation

@diaovidiu How did you get it to work? I'm trying to use dot notation but not having any luck.
For example, I have frontmatter
---
sleep:
	quality: 3
---

in a note in my Periodic/Daily folder. And I have a tracker
searchType: frontmatter
searchTarget: sleep.quality
folder: Periodic/Daily
ignoreZeroValue: true
dateFormat: YYYY-MM-DD
startDate: -1w
line:
	xAxisLabel: Day
	xAxisTickLabelFormat: dd
	yAxisLabel: Sleep Quality
	yAxisUnit: 1-5
	yAxisTickLabelFormat: i
	yAxisTickInterval: 1
	yMax: 5.5
	yMin: 0
	fillGap: true

But I get a "no valid Y value" error.
I've confirmed that the same tracker works if the frontmatter is formatted as
---
sleep-quality: 3
---

and the tracker as
searchType: frontmatter
searchTarget: sleep-quality
folder: Periodic/Daily
ignoreZeroValue: true
dateFormat: YYYY-MM-DD
startDate: -1w
line:
	xAxisLabel: Day
	xAxisTickLabelFormat: dd
	yAxisLabel: Sleep Quality
	yAxisUnit: 1-5
	yAxisTickLabelFormat: i
	yAxisTickInterval: 1
	yMax: 5.5
	yMin: 0
	fillGap: true

So it seems that I can't reference nested frontmatter. But you said you got it to work?

+1, facing the exact same issue, the frontmatter and tracker don't work as expected when frontmatter is nested YAML:
Frontmatter:
---
habit.waterIntake: 2
---

Tracker (doesn't work):
searchType: tag
searchTarget: habit.waterIntake
line:
  title: Water Intake

if I change the tracker type to text, it works perfectly:
searchType: text
searchTarget: habit.waterIntake
line:
  title: Water Intake

Looks like an issue with parsing nested YAML.


C++ Generation via the node infrastructure

Generating code via the node infrastructure. First iteration - reflection working and other functionality up-to-par with previous code generation.
TODO: Code formatting

Closing this pull request. Find new one at https://github.com/dotnet/corert/pull/1611


Workaround for Whatsapp Web UI lag

This is a workaround for Whatsapp Web UI constant lagging due to Whatsapp abusing history.pushState().
This MAY introduce other problems, but we have few conclusive reports on that so far.

@all-contributors please add @fernandofig for code


Upgrade to 1.9.0 wrong order of links in Kibana navigation

Hi,
I have just upgraded the OpenDistro to 1.9.0
The order of menu items in Kibana looks wrong.
I expect the Discover to be on top, what can cause it?

I am using Helm Chart with the latest Docker images.

This should be related to latest changes in kibana plugins. @joshuali925 pls help to take a look for sql workbench firstly.


Fixing bare variable warnings

Fixing deprecated support of bare variables in with_items statements

Added a 2nd commit that allows for setting User_Alias defaults.  For Example:
sudoer_defaults:

:MONITOR_USERS     !logfile

This would produce the line:
Defaults:MONITOR_USERS     !logfile

There's a bug on your comparison of existing vs authorized sudoer specs.  See commit 70bc091


Friday 22-09-2023 roll call

Roll Call!
Leave a comment with these two things:

A summary of your study using only emojis
Something you'd like to share, anything goes! (within respect)

emoji cheat sheet

good morning :)

hi

Good morning

Good morning

Good Morning

Good morning

present

present

Present

present

Present!

🖖🏼


Extensibility with Amplify CLI [Auth + Storage + Custom-resources + Export to CDK]

Feature 1: Use Amplify maintained, easy to use types to override Amplify-generated  resources
How to use?

“amplify  init”  to create a new Amplify project
“amplify  add auth”  or “amplify add storage” to add your Cognito, S3, or DDB resources3.
“amplify  override auth”,  “amplify override storage”, or “amplify override project” to override  the auth or storage resources / root stack
“amplify  push”  to deploy!

Feature 2: Add custom AWS resources via “amplify  add custom”

“amplify  add custom”  to use either CDK or CFN to add new custom resources
Edit the CDK or CFN  file  to your heart’s content! 🎉  (hint: there’s  a package.json in your resource’s folder, where you can add any additional  node modules you’d like!)
“amplify  push”  to deploy!

Feature 3: Export the Amplify stack as CDK to  use in your own CDK-based deployment pipeline

“cdk init app  language=typescript” to create your CDK app
Go to your Amplify  project folder
“amplify export --cdk  --out ” with your desired export directory
Go back to your CDK app  folder
Install the Amplify  Exported Backend construct: “npm i @aws-amplify/cdk-exported-backend”
Add “ import {  AmplifyExportedBackend } from '@aws-amplify/cdk-exported-backend' ” to  your CDK app
Use Amplify resources within your CDK app “const amplifyStack = new AmplifyExportedBackend(app, "amplify-stack", { path: "./amplify-export-extensibilitytest", stage: "dev" })”

Codecov Report

Merging #8776 (a87038e) into master (81cc461) will decrease coverage by 0.57%.
The diff coverage is 52.45%.

@@            Coverage Diff             @@
##           master    #8776      +/-   ##
==========================================
- Coverage   56.92%   56.34%   -0.58%
==========================================
  Files         756      835      +79
  Lines       42246    47268    +5022
  Branches     8659     9476     +817
==========================================
+ Hits        24047    26632    +2585
- Misses      17378    19818    +2440
+ Partials      821      818       -3

Impacted Files
Coverage Δ

...utils/awscloudformation/assets/cognito-defaults.js
62.50% <ø> (-1.50%)
:arrow_down:

...vice-walkthrough-types/cognito-user-input-types.ts
100.00% <ø> (ø)

...s/awscloudformation/utils/trigger-file-uploader.js
16.66% <0.00%> (ø)

...wscloudformation/utils/verification-bucket-name.ts
41.66% <ø> (ø)

...gory-auth/src/provider-utils/supported-services.ts
100.00% <ø> (ø)

...ges/amplify-category-custom/src/commands/custom.ts
0.00% <0.00%> (ø)

packages/amplify-category-custom/src/index.ts
0.00% <0.00%> (ø)

...r-utils/awscloudformation/utils/layerCloudState.ts
15.94% <0.00%> (-0.24%)
:arrow_down:

...rage/src/provider-utils/awscloudformation/index.ts
14.10% <ø> (ø)

...ation/service-walkthroughs/dynamoDb-walkthrough.ts
49.06% <ø> (ø)

... and 151 more

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 81cc461...a87038e. Read the comment docs.


Handle Multiple Node Migration [Rough Draft]

User Story

As a user with multiple current nodes, when 2.3 is released I want to be able to migrate all of them without losing any data or losing the ability to use any of my nodes.
After my nodes have migrated, I still want the option to switch between them like I currently do.

Problem Statement
When a node is migrated, by default all the data will go to a new data directory. If a second node was migrated, it would also have its data written to that directory, wiping out the first node's data. If a user has multiple nodes, we don't want any loss of data, and we want them to be able to continue to access all their nodes. To do this we need to let the client tell the server which directory to write to when migrating, so no node's data is overwritten.
Functional Requirements

[ ] When a 2.3 client starts a node, instead of passing the coin it should pass a --dataDir (or -d) flag to the server to tell it what data directory to use.
[ ] The data directory should be different from the 2.2.x directory, so old data is preserved just in case.

Something like "OpenBazaar2.3" plus the old coin name?
This should cause the server to migrate that old node to the new data directory. From then on the client should use the new data directory when starting that node.

OPTIONAL: Should we also give the user the ability to specify a different path? This might be nice if users want to run multiple stores from the bundled app, they could name the folders something that is easy to remember, like "ShoesStore" and "HatsStore".

Looking at:

This should cause the server to migrate that old node to the new data directory.

@jjeffryes Assume a user has an existing bitcoin installation at the default install location and wants to install their setup at /abc. What does the 2.3 UI pass into the server on the first startup (which would initiate the migration) so it knows which directory to import from and to export the migrated data to?

Assume a user has an existing bitcoin installation at the default install location and wants to install their setup at /abc. What does the 2.3 UI pass into the server on the first startup (which would initiate the migration) so it knows which directory to import from and to export the migrated data to?

@placer14  Thanks, that was an oversight in my description. We need the -d flag to tell the server what the from directory is, so we'd need some new flag for the to directory, right?
So if you start with -d=bitcoin and -n=abc (I'm using n as the new directory flag, it can be whatever), it would read the data from /bitcoin and write the migrated data to /abc?

We need the -d flag to tell the server what the from directory is, so we'd need some new flag for the to directory, right?

I think it's way easier to just migrate the existing directory if the -d flag is passed rather than specifying another directory.

Why don't we do this: Since we already agreed on using the "connection manager" approach in the bundled setup, why don't we just convert BTC, BCH and ZEC into separate connections in the manager with their associated datadir as the default paths. This would make their install location clear to the user and allow them to manage them as the separate identities they are. And instead of starting the server using --bitcoincash the UI would just pass the -d. Then the server would just migrate the store as it normally would (into the new multiwallet) using that store's id.


Use go test framework to run the tests

Replace current format where all tests for basically functions in a single file.

Checked-in


EL-1308 refactor - change back link to be decided by controller

https://dsdmoj.atlassian.net/browse/EL-1308
What changed and why
Refactor to make previous link decided by controller rather than view, so that a different decision can be made when Early eligibility is introduced
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing
Branch is generally up to date with main Github - definitely no conflicts
No unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
PR description says what changed and why, with a link to the JIRA story.
Diff has been checked for unexpected changes being included.
Commit messages say why the change was made.

this looks great 👍

happy to approve this for UAT, I have tested a number of pages and they all behave as per main does at present.


[MAINT] Support python 3.12

Closes #664
To be able to support Python 3.12 I had to bump some of the dependencies. Most importantly:

Numpy from 1.X to 2.X
Pandas from 1.X to 2.X

Alright ! Tests are passing with Python 3.12.
@thibaultdvx let me know if you see anything that should be changed before merging.

Let me know if I can merge this @thibaultdvx


Requires asciidoctor

Requires asciidoctor hence makepkg fails.

Right. I made a mistake.
Thanks for the report.


Google Font syntax

The syntax for loading Google fonts described in the README doesn't seem consistent with what Google Fonts wants. This doesn't work:
WebFont.load({
  google: {
    families: ['Open Sans:n4,n7']
  }
});

But this does work:
WebFont.load({
  google: {
    families: ['Open Sans:400,600']
  }
});

In fact, when you go to the Google Fonts site and prepare to download your selection, it recommends something like the following: Open+Sans:400,600:latin (using a + instead of a space, and adding the script at the end after another colon).
Spent a while stumbling around trying to figure out why I couldn't get bold working, and I imagine others have done the same, so I think this would be helpful to include in the README.

+1!

+1
C'mon Google.

This has been fixed now. Thanks for reporting.


fix(ColorBy): Use workaround for array names with single colons

Reference bug: https://discourse.vtk.org/t/paraview-glance-no-color-mapping-when-data-array-name-contains-colon/2022
@jourdain

LGTM maybe you could have created a const SEPERATOR at the top of the file instead. ;-)

I considered using const SEPARATOR but I decided to do this so I can minimize any weird merge conflicts with my branches. It'll look nicer soon. :)

Then go for it, merge when you want...

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


encode error with python 2.7.9

root@Achurch:~# python dp_crypto.py -k https://apps.morlock.intra/Telerik.Web.UI.DialogHandler.aspx 48 hex 9
dp_crypto by Paul Taylor / Foregenix Ltd
CVE-2017-9248 - Telerik.Web.UI.dll Cryptographic compromise
Attacking https://apps.morlock.intra/Telerik.Web.UI.DialogHandler.aspx
to find key of length [48] with accuracy threshold [9]
using key charset [01234567890ABCDEF]
Key position 01: (0) [A]Traceback (most recent call last):
File "dp_crypto.py", line 354, in
mode_brutekey()
File "dp_crypto.py", line 204, in mode_brutekey
found = get_key(session)
File "dp_crypto.py", line 168, in get_key
keychar = test_keypos(key_charset, unprintable, found, session)
File "dp_crypto.py", line 122, in test_keypos
if test_keychar(keychar, found, session, pad_chars[i] * 3):
File "dp_crypto.py", line 67, in test_keychar
found + keychar, session, pad_chars
File "dp_crypto.py", line 34, in get_result
(encrypt(plaintext, key) + base).encode()
UnicodeDecodeError: 'ascii' codec can't decode byte 0x80 in position 1: ordinal not in range(128)

Hi @LeonardoNve the exploit has been written for python3. Can you try
python3 dp_crypto.py -k https://apps.morlock.intra/Telerik.Web.UI.DialogHandler.aspx 48 hex 9
let me know if that works.
Thanks,
Paul

No response, so closing, but feel free to let me know if that didn't resolve it for you and I will re-open.

hello, can anyone help me ? i ran the command: ./dp_crypto.py -k https://thanhtra.com.vn/DesktopModules/TNComments/Telerik.Web.UI.DialogHandler.aspx 48 hex 9 and kept getting the error like below:
dp_crypto by Paul Taylor / @bao7uo
CVE-2017-9248 - Telerik.Web.UI.dll Cryptographic compromise
usage: dp_crypto.py [-h] {d,e,k,b,p} ...
dp_crypto.py: error: invalid choice: 'https://thanhtra.com.vn/DesktopModules/TNComments/Telerik.Web.UI.DialogHandler.aspx' (choose from 'd', 'e', 'k', 'b', 'p')
What should i do now?


filtering file entries

@drujensen I think that the version you wrote is reasonable, but please let me reject it this time. Thank you for telling your use case.


“API rate limit exceeded for rauchg” on download link

‣ curl https://hyper-updates.now.sh/download/osx
{"message":"API rate limit exceeded for rauchg.","documentation_url":"https://developer.github.com/v3/#rate-limiting"}

Can't reproduce 🤔
❯ wget  https://hyper-updates.now.sh/download/osx
--2016-10-14 17:49:17--  https://hyper-updates.now.sh/download/osx
Resolving hyper-updates.now.sh... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to hyper-updates.now.sh|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 39487739 (38M) [application/octet-stream]
Saving to: 'osx'
[...]

I'm still seeing this. ¯_(ツ)_/¯
Here are some relevant headers:
x-ratelimit-limit: 5000
x-ratelimit-remaining: 0
x-ratelimit-reset:<PHONE_NUMBER>

It looks like the limit resets every hour, so it might be coming and going.
More info here:

https://developer.github.com/v3/#rate-limiting
https://developer.github.com/v3/rate_limit/

i got it

its here https://github.com/zeit/hyper/blob/master/app/auto-updater.js#L22-L28

i adressed related problem in here https://github.com/zeit/hyper/pull/913

might fix this issue

#928  might fix this issue


Gulp issue

options was not passed properly..
But still couldn't figure out why the comments are removed for the previous case though..

rebase screw up fixing it..


Cpu fan not working

Describe the bug
After installing BigSur I found that cpu fan never runs which causes laptop to restart frequently.
To Reproduce
Follow the guide, install bigsur and use the efi from this repo without any modifications.
Expected behavior
CPU fan should run when temperatures rises.
macOS Info:

OS: Big Sur
Version: 11.0.4
Laptop Model: T450s

Does the fan spin when you booting the laptop or in Windows?

yes. It was working fine before on ubuntu.

Does it work now in Ubuntu? Or when you are powering ON the laptop?

Now I don't have ubuntu so I can't say. But when I power ON the laptop it runs.

Check if TempToFans.SSDT is enabled in Config.plist and also install the Thinkpad Assistant app as described by the README

TempToFans.SSDT is enabled and also yogaSMCNC and ThinkPad Assistant both are installed and running on login. When I try to enable fan control in yoga SMCNC a popup comes with Ec is unavailable.

I don't know how but I installed smcFancontrol app and after modifying and reverting settings now fans are running after cpu reaching 55°C.

And I see max cpu temp on running 4k video on YouTube is 78°C. Is it good or bad?

That probably means your fan isn't working as it should. I suggest you test it with another OS. Like just a Linux distro from a USB drive to see if the fans are working as they should.

Ok. Have you or anyone else saw restarting unexpectedly sometime?

Ok. Have you or anyone else saw restarting unexpectedly sometime?

Maybe after a very long sleep. Like if I leave the laptop for about 2 days

Not on sleep. On zoom call I experienced this

That doesn't happen on mine. However T450/T450s have an issue with the second RAM slot becoming loose. This means on of the RAM jiggles inside the slot and this causes the laptop to freeze and shutdown/reboot. Check this by removing 1 RAM at a time to determine if you have loose RAM slot

That can be issue because I am using second RAM slot.


Support entity name for permissions management

@LucidOcean thanks for merging, could you please update nuget package as well?

Package <IP_ADDRESS> uploaded.

thanks


Add .node-version

For use with nodenv

We're using NVM for the Readers Web repos. Do you know how nodenv compares? In the metric of Git stars, NVM is significantly more popular but I don't know that nodenv was even considered.

I compared rbenv and rvm several years ago and preferred rbenv's approach. From there I picked nodenv just by association with rbenv, so I can't give a good comparison to nvm. Nodenv does have a similar self-promotional page about how they compare.
I don't think we should necessarily force everyone to switch over, but it would be nice to support both. It looks like nvm can use a .nvmrc file and we could symlink to that from .node-version

Closing for now - will re-investigate after https://github.com/wikimedia/wikimedia-page-library/pull/175


Add a code migration guide from v3 to v4

Thank you again for this great effort!
I am trying to migrate from v3 to v4 but I am having quite some problems. As I could not find a migration guide I am asking here.
How are Indexes handled now? In the changelog it only states,

Indexes have been simplified

Which does not really help me understand how I can migrate my old code.
For example, index types seem to not be supported anymore. This leads me to the question of how I would migrate code like this @Index(type: IndexType.value).
Also, case sensitivity seems to not be provided anymore like in @Index(type: IndexType.value, caseSensitive: false).
It would also help me immensely if you could explain to me how to migrate where clauses. Because this statement

Where clauses have been removed in favor of automatic index handling

tells me there are none but I used them a lot to fine-tune the performance of my application. This is crucial as I am searching a text database with more than 300.000 entries.
Maybe we could put a migration guide together?

To be clear, I am not asking for automatic migration of an existing database. I would like a guide to migrate my v3 code to v4.

I checked and thankfully I only use @Index() without typing. But there are no where clauses? I cannot comprehend that. How do you supposed to where then?


Waypoint: Concatenate Strings with concat

That Waypoint is actually about concatenating Arrays, not Strings.

I made a pull request with the suggested fix. https://github.com/FreeCodeCamp/FreeCodeCamp/pull/4410

Closed by #4410