How do I assert on cassette.all_played conditionally based on wheter I record or replay?

The following test fails when recording:
import vcr
import requests

def test_all_played():
    with vcr.use_cassette(path="tmp.yaml") as cassette:
        requests.get("https://google.com")
        assert cassette.all_played  # cassette.all_played is False during the recording

and works fine when replayed, which is a bit of a surprise for the people I work with. Is there any idomatic way of introspecting whether a cassette is being recorded vs. being played back? This way I could do something like:
import vcr
import requests

def test_all_played():
    with vcr.use_cassette(path="tmp.yaml") as cassette:
        requests.get("https://google.com")
        if cassete.is_being_played_back:
            assert cassette.all_played

Right now I ended up with:
import vcr
import os
import requests

RECORD_MODE = vcr.mode(os.environ.get("TEST_RECORD_MODE", "none"))  # defaults to vcr.mode.NONE

def test_all_played():
    with vcr.use_cassette(path="tmp.yaml") as cassette:
        requests.get("https://google.com")
        if RECORD_MODE == vcr.mode.NONE:
            assert cassette.all_played

Then if I want to record the tests, I run them with env TEST_RECORD_MODE=once python -m pytest ....
Is there maybe another way of doing what I want, i.e. asserting everything has been played back or not trying to assert at all?
It would even be easier for me if this setting would be part of the use_cassette. For example, the use_cassette could raise NotAllPlayedError when exiting context manager in the playback mode, but do nothing in the replay. If this could be configurable from the use_cassette, it would allow me to use use_cassette as a function decorator (I would not even need the cassette instance):
def raise_(e):
    raise e

@vcr.use_cassette(on_not_all_played=lambda: raise_(NotAllPlayedError)):
def test_all_played():
    import requests
    requests.get("https://google.com")

I ended up using the cassette's write_protected method as a sign that I am in the recording mode:
import vcr
import requests

def test_all_played():
    with vcr.use_cassette(path="tmp.yaml") as cassette:
        requests.get("https://google.com")
        if cassete.write_protected:
            assert cassette.all_played

So far it seems to work.
Here's my pytest based setup:
@pytest.fixture
def myvcr() -> vcr.VCR:
    # The default "none" value of RECORD_MODE means the test will fail if there is no record for it
    # found--this is useful to prevent test from running in the CI if a recording has not been
    # checked in.
    record_mode = vcr.mode(os.environ.get("TEST_RECORD_MODE", "none"))
    return vcr.VCR(
        record_mode=record_mode,
        path_transformer=vcr.VCR.ensure_suffix(".yaml"),
        decode_compressed_response=True,
        filter_headers=["Authorization"],  # Parts of the request that will not be recorded.
        # TODO Switch from raw_body to body, when https://github.com/kevin1024/vcrpy/issues/749 is released.
        match_on=["method", "scheme", "host", "port", "path", "query", "raw_body"],
        record_on_exception=False,  # If exception is not caught within `use_cassette` context manager, the recording would fail.
    )

@pytest.fixture
def cassette_path(request: pytest.FixtureRequest) -> pathlib.Path:
    test_name = request.node.name
    return pathlib.Path(__file__).parent / "resources" / "cassettes" / (test_name + ".yaml")

@pytest.fixture
def cassette(cassette_path: pathlib.Path, myvcr: vcr.VCR) -> Iterator[vcr.cassette.Cassette]:
    with myvcr.use_cassette(path=str(cassette_path)) as cassette:
        yield cassette

def test_all_played(cassette):
    requests.get("https://google.com")
    if cassette.write_protected:
        assert cassette.all_played


Get control over Telemetry construction

The code in ApplicationInsightsSink.ForwardLogEventPropertiesToTelemetryProperties always forwards all the properties (both built-in, like LogLevel or RenderedMessage, and custom) to the AI object, which leads to duplication. All examples below are valid for TraceTelemetry.

LogLevel and RenderedMessage. I, frankly, see no point in having these custom properties at all, because LogLevel is also forwarded to TraceTelemetry.SeverityLevel (I understand that this mapping is not 1-to-1), and RenderedMessage goes to the TraceTelemetry.Message.

Enricher properties. For example, I have HttpRequestIdEnricher to help me collate logs in non-AI case (such as Seq). But AI has its own OperationCorrelationTelemetryInitializer, which provides data for the same purpose, so I don't want the HttpRequestId property showing up in AI logs (and I don't want to turn off OperationCorrelationTelemetryInitializer because (a) there's some AI info that's collected by AI directly and (b) AI provides UI support for collation on Operations only). And the same applies to, for example, user ID.

So in general I'd like to be able to:

Totally skip some properties when mapping them to Telemetry custom properties.
Be able to map some properties not to custom properties, but to some part of TelemetryContext

And if we generalize this a bit more, we can come to the ability of custom transformation from LogEvent to Telemetry so that I can, for example, transform log messages coming from Serilog Timings/Metrics to DependencyTelemetry and MetricTelemetry.
What do you think of it?

Another option would be using / taking a Func<ITelemetry, ..> and check what it returns (Trace-/Event-/Exception/Dependency- or MetricTelemetry) and send that over to AI..
Hrm - that might even be a tad better.

I was thinking more along the lines of Func<LogEvent,string,ITelemetry> for event creation, then some kind of property filter, returning bool to indicate whether the property should be forwarded, and for advanced cases - Action<LogEvent,string,ITelemetry> to allow any possible mapping scenario. We also have to keep in mind the exception transformation and property flattening.
So, all in all it comes down to:

Func<LogEvent,string,ITelemetry> telemetryCreator, which allows to create ITelemetry based on event and formatted message
bool trackExceptionsSeparately, which, when set to false, puts events with exceptions to "normal" pipeline (allowing them to be created by the telemetryCreator)
bool populateTelemetry, which indicates whether the ITelemetry from telemetryCreator was already populated, or we have to put it through pipeline
Action<LogEvent,string,ITelemetry> propertyForwarder, which allows to populate passed in ITelemetry from event and formatted message (maybe we should also pass in property flattener)
Func<string,object,ITelemetry> propertyFilter, which allows to check whether passed in property (name + value) should be forwarded to ITelemetry

Obviously, the last two options are mutually exclusive, we either use our own forwarder, or just filter the forwarding in normal forwarder.

Already working on an updated Action<LogEvent,IFormatProvider,ITelemetry> sample.. will take just a couple more minutes :)
I'd rather not make the interface too noisy with too many parameters and options of filtering etc but provide a raw constructor hook so the few who do want and need to customize the telemetry can do so, and everyone else can keep using the existing default ones via Trace- or Event Telemetries.

I like idea for controlling telemetry construction. :+1:
Now when https://github.com/serilog/serilog/pull/772 is merged we can implement 2nd optional behavior - property json serlialzation, and allow users to select from flatern, json serialization and user implementation.

Alright.. updated #16 - now you can specify a Func<LogEvent, IFormatProvider, ITelemetry> logEventToTelemetryConverter when adding the ApplicationInsights sink, i.e. like this:
var log = new LoggerConfiguration()
    .WriteTo
	.ApplicationInsights("<MyApplicationInsightsInstrumentationKey>", LogEventsToMetricTelemetryConverter)
    .CreateLogger();

// ....

private static ITelemetry LogEventsToMetricTelemetryConverter(LogEvent serilogLogEvent, IFormatProvider formatProvider)
{
    var metricTelemetry = new MetricTelemetry(/* ...*/);
    // forward properties from logEvent or ignore them altogether...
    return metricTelemetry;
}

.. which gives you full control over what ITelemetry you want to create and more importantly.. how :+1:
There's one caveat though, it must be one of the TelemetryClient.Track* -able types, but I think that's managable.
What do you think.. :+1: or :-1: ?

The problem with this approach is that if I simply want to exclude one or two properties from the telemetry, I'll have to copy the whole code from the original implementation.

There's one caveat though, it must be one of the TelemetryClient.Track* -able types, but I think that's managable.

Why don't you simply use Track method? I understand that it is marked as "for internal use", but it's still public and I don't see any problem with it.

The default LogEvent > Event- or TraceTelemetry are public extension methods to LogEvent instances (see Line #80 and following or if you only want to use the default Property forwarding, it's similarly an extension method) - so in your own code you can use those and post-process / remove the properties you don't want as AI properties and i.e place some in the TelemetryContext.
The handful of developers who do / might want to customize the Telemetries can do it with this rather raw but simple approach, if we added more overloads that take predicates/filters and selectors it would add extra code to maintain.
If more users would ask for a simpler approach over time I'd revisit it in the future, but for now and because you opened the ticket (and were the first one to miss telemetry customization), can/could you do everything you wanted with the approach in #16?
Re: (not) calling .Track(..):
Someone of t he AI team took the time to explicitly state 'Do not call' in the documentation for the .Track() method, and while it may do little harm, I therefore decided against it.. I think while the current Track* implementation is not elegant per se, it will do for now and when AI gets new Telemetry types, I'll add them.

If more users would ask for a simpler approach over time I'd revisit it in the future, but for now and because you opened the ticket (and were the first one to miss telemetry customization), can/could you do everything you wanted with the approach in #16?

Let's try it on simple thing that brought me here. Can you provide an example of code doing the following: I want everything done "as before" (i.e., LogEvent maps to TraceTelemetry, and so on, and so forth), but properties corresponding to log level and formatted message are not forwarded, the property "HttpRequestId" is not forwarded, and property "UserId" is forwarded to telemetry.Context.User.Id?

Someone of t he AI team took the time to explicitly state 'Do not call' in the documentation for the .Track() method, and while it may do little harm, I therefore decided against it..

The same code states

This method needs to be public so that we can build and ship new telemetry types without having to ship core.

So it seems just like something we need. May be we can go and ask AI developers if it is appropriate to use this generic approach?

Sure, here's the example converter and usage assuming you want them as TraceTelemetry instances:
public static void Main()
        {
            var log = new LoggerConfiguration()
                .WriteTo
                .ApplicationInsights("<MyApplicationInsightsInstrumentationKey>", ConvertLogEventsToCustomTraceTelemetry)
                .CreateLogger();
        }

        private static ITelemetry ConvertLogEventsToCustomTraceTelemetry(LogEvent logEvent, IFormatProvider formatProvider)
        {
            // first create a (sink) default Trace Telemetry but without the log level, and (rendered) message (template)
            var telemetry = logEvent.ToDefaultTraceTelemetry(
                formatProvider,
                includeLogLevelAsProperty: false,
                includeRenderedMessageAsProperty: false,
                includeMessageTemplateAsProperty: false);

            // then go ahead and post-process the telemetry's context to contain the user id as desired
            if (logEvent.Properties.ContainsKey("UserId"))
            {
                telemetry.Context.User.Id = logEvent.Properties["UserId"].ToString();
            }

            // and remove UserId and HttpRequestId from the telemetry properties
            if (telemetry.Properties.ContainsKey("UserId"))
            {
                telemetry.Properties.Remove("UserId");
            }

            if (telemetry.Properties.ContainsKey("HttpRequestId"))
            {
                telemetry.Properties.Remove("HttpRequestId");
            }

            return telemetry;
        }

(The default implementations check whether there's a LogEvent.Exception and if so, return / build an ExceptionTelemetry instance.. but you could decide whether you'd want that, too).
Regarding usage of TelemetryClient.Track(..) I've opened an issue with the AI folks, see above.

I admit, this look nice.

Alright, thanks for the idea & feedback :+1: Build is running and an updated package will be pushed to nuget shortly thereafter.
Cheers!

There seems to be a problem.
Here's the config:
                .WriteTo.ApplicationInsights(TelemetryConfiguration.Active,
                    (logEvent, formatProvider) => logEvent
                        .ToDefaultTraceTelemetry(formatProvider, includeLogLevelAsProperty: false)
                        .Do(telemetry =>
                        {
                            telemetry.Properties.Remove(HttpRequestIdEnricher.HttpRequestIdPropertyName);
                        }))

And here's the exception in SelfLog
2016-06-23T14:11:34 Caught exception System.ArgumentNullException: Value cannot be null.
Parameter name: formatProvider
   at Serilog.ExtensionMethods.LogEventExtensions.ForwardPropertiesToTelemetryProperties(LogEvent logEvent, ISupportProperties telemetryProperties, IFormatProvider formatProvider, Boolean includeLogLevel, Boolean includeRenderedMessage, Boolean includeMessageTemplate)
   at Serilog.ExtensionMethods.LogEventExtensions.ToDefaultTraceTelemetry(LogEvent logEvent, IFormatProvider formatProvider, Boolean includeLogLevelAsProperty, Boolean includeRenderedMessageAsProperty, Boolean includeMessageTemplateAsProperty)

Yep that is/was a regression - thanks for catching it! I unfortunately only tested it with an actual formatProvider instance :frowning: Fixed build is running / package coming shortly thereafter

Its seems to work now, thank you!


Some sites generate a SIGTRAP

Similar to the attached...
63c05b4d-007e_08-DOOBLE-exceptq.txt

This is very odd. Something implanted an "int 3" into the code (the error location actually points behind the "int 3" instruction and as you can see, the following 2 bytes seem to be the leftover of a complete instruction whose first byte was overwritten with the "int3" instruction).
That would either happen if

it is explicitely coded

if the trace facility dynamically implants an "int 3" breakpoint (if, for example, you do a "trace on jfs", then jfs.ifs will get int3´s implanted at various places in the loaded binary so that it can spit out trace info at those points)

if a debugger dynamically implants an "int 3" breakpoint (for about the same reason as 2), it would for example be able to display the register contents at that point)

is not likely as (see above) it seems that a 3-byte instruction had its first byte dynamically overwritten with an "int 3" instruction.
My gut feeling would be that under some circumstances, EXCEPTQ implants dynamic tracepoints by using the "DosDynamicTrace" API (see point 2)) but I don`t know when it would do that.

I got a different SIGTRAP today in some pdfium code - when trying to save a page as PDF
63e8992c-0040_0e-DOOBLE-exceptq.txt
Agree it's a little strange..

A similar SIGTRAP printing to PDF was fixed in b400f57a8f31f7d176b4b0792a19ba4bf1eb0e84 will create a new ticket including a site URL/steps to reproduce if more are found.

Some more info about browsers using TRAP 3,
https://github.com/bitwiseworks/mozilla-os2/issues/269, note that this
one was compiler flag influenced.
Seems some were due to OOM's (Out of Memory) conditions too.


Block or Transaction Validation Failed

I'm getting the following error while sending NEO using NEON-js
{"message":"Error sending NEO","code":500,"details":"Error sending NEO","error":"http://seed1.ngd.network:10332: Block or transaction validation failed.","errorStack":"Error: http://seed1.ngd.network:10332: Block or transaction validation failed.\n    at Query_Query. (/app/node_modules/@cityofzion/neon-js/dist/index.js:7791:23)\n    at Generator.next ()\n    at fulfilled (/app/node_modules/@cityofzion/neon-js/dist/index.js:7474:58)\n    at process._tickCallback (internal/process/next_tick.js:68:7)"}
  let sendNEOPromise = await new Promise(async (resolve, reject) => {
    try {
      const { default: Neon, api, wallet } = require('@cityofzion/neon-js');
      const { rpc } = require('@cityofzion/neon-js');
      const { NEO_PUBLIC } = require('../../../v1/spectrum/util/constants');

      const privateKey = PROD ? process.env.NEO_PRIVATE_KEY : process.env.NEO_PRIVATE_KEY_TEST;
      const myAccount = new wallet.Account(privateKey);

      const Neoscan = new api.neoscan.instance(PROD ? 'MainNet' : 'TestNet');
      let balance = await Neoscan.getBalance(NEO_PUBLIC);

      let tx = Neon.create.contractTx();
      tx.addIntent('NEO', amount, receiver)
        .addRemark('something') // Add an remark
        .calculate(balance) // Now we add in the balance we retrieve from an external API and calculate the required inputs.
        .sign(myAccount.privateKey); // Sign with the private key of the balance

      const hash = tx.hash;
      const serializedTx = tx.serialize();

      await rpc.Query.sendRawTransaction(serializedTx).execute(
        PROD ? 'http://seed8.ngd.network:10332' : 'https://test3.cityofzion.io:443'
      );

      return resolve(['success', hash]);
    } catch (err) {
      console.log(chalk.red('error: ', err));
      postSlackError(`Error sending NEO: ${err}`);
      return reject(errorRespHelper(res, 'Error sending NEO', 500, true, err));
    }
  });
  return sendNEOPromise;
}

The code works locally when I call this function but fails when I run it on the server.
We've tried different RPC Clients from http://monitor.cityofzion.io/ but that hasn't worked either.

One thing i note is the difference in your keys. You are using a static NEO_PUBLIC to retrieve the balance but your private key for signing depends on your env. This wont work for one case unless your private keys are identical.


parse-params public

Hello,
I'd like to use parse-params in my code but it's private. This change makes it public, as it can be useful as generic query strring parser.
(for example I'd like to use it to capture query string parameters in hrefs when scraping web pages with enlive).
Maybe parse-params is private for a reason, in that case sorry for the noise.
Cheers,
Giacomo

It's probably a good idea to make this functionality public, but the ring.middleware.params namespace is not the right place for it, as parse-params is not middleware.
It would probably make more sense for it to be in a ring.util.* namespace, because it's a utility function. I'm thinking perhaps ring.util.codec. We could then have two functions, ring.util.codec/param-decode and ring.util.codec/param-encode. Or perhaps form-decode and form-encode.

It would sound strange to me, as an user, to have a query string parsing function mixed with url encoding functions.
I tried moving assoc-params and parse-params to ring.util.params instead.
How does it look?

Why would it seem odd to have functions for parsing form-urlencoded parameters in the same place as urlencoded strings?

Having them in the same place looks good, but wouldn't be misleading for someone looking for a parser to have to find it inside a file named codec?

Hello,
I implemented form-encode and form-decode plus a simple test.
Cheers,
Giacomo

Sorry for taking a while to look at this.
The assoc-param function doesn't quite seem appropriate for the codec namespace. Let me think a little about this before I merge.
Usually I'm not this particular about moving functions around, but these functions seem really hard to place in an appropriate namespace. Please bear with me :)


UI Inconsistencies

Notice the arrow tip? Any idea why this happens?
I'm on MacVim on OS X 10.8.4

I had to patch the font (literally resized special chars) for making it look good. It didn't took long, so maybe somebody will find it an acceptable way to work around this issue too.

@aandrieiev can you explain how did you manage to do this, and/or upload your font file somewhere? I'm trying to patch/find a good version of Consolas, but the separators are always misaligned..

@wladston sure, you can find the patched version of Luculent here. I used Glyphs Mini (it has a trial period) to open the font file and manually change the height of separator characters. You can do it to Consolas as well.

@aandrieiev your airline is looking great :) Well, in the end I decided not to go with the patched fonts after all ... I found some nice unicode symbols that don't look so bad…

Closing this as it is not an issue with airline, but rather the font that gets used instead


Removes article on how to install deb package with Traffic Director

This content has drifted is likely broken due to bintray links. In any case, it has nothing to do with the CLI which is the only thing we'll end up when builds move upstream. I setup a redirect until the referer does the same as the upstream content will eventually change when they host their own deb packages.

verified the redirect works


enable  port forwarding within PowerShell

The very cool features on linux is about the port forwarding when we ssh to a remote server.
I hope PowserShell come with these cool things.

PowerShell doen't implement a transport but uses ssh as transport. So the feature you request is ssh feature and you can use it directly from ssh.

@tlsalex Put another way, you can run ssh.exe just fine in a powershell console. Port forwarding is an SSH feature, not a "linux" feature, and you can do this on Windows with Putty (Plink) or Win32_OpenSSH, or Mac with OpenSSH, and you can run both commands from inside a powershell console, so what you want already exists! 👍


Remove duplicated code in crd plugin

Removing duplicated code in setting up informers and moving non controller related code into crd_utils.go
Also closes: #137
Signed-off-by: Serguei Bezverkhi<EMAIL_ADDRESS>

update was removed because controller translates update into add and delete, so the update in crd_queue.go case would never be reached.

@mestery Please review when you have a chance, it is important fix for several issues.


How to rename username in login screen?

I am new here, and I want to use this theme. How can I rename the Username in the login screen?

In src/configs/user.js:
https://github.com/Renovamen/playground-macos/blob/0b0892f5ecb0a2e0716829e12c22e5d696589268/src/configs/user.js#L2

thanks :smile:


Fail to expand the attached storage accounts using key and name for Azure China accounts

OS: Windows
Build Version: 20200314.1
Regression: Not a Regression
Repro Steps:

Sign in with Azure China account --> Copy the primary key of one storage account --> Attach this storage account using key and name in Azure Storage Explorer.
Expand the attached storage account.

Expect:
Succeed to expand the attached storage account.
Actual:
Fail to expand the attached storage accounts using key and name.
More Info:
There is no "EndpointSuffix" in the primary key.

The primary key is the credential for connecting to the storage account.  It shouldn't include the endpoint suffix.  This is by design.

Got it. Thanks for your share.
Meanwhile could you help to share more info about launching the attaching storage accounts using primary key and name in Azure Storage Explorer for Azure China and Azure Germany accounts?


Version 1.20.2 fails to run on Windows 10 v1809

Bug description
Version 1.20.2 fails to start on Windows 10 v1809.
Error message returned from:
docker : Error response from daemon: container 5fb0aa8da94f5862adc267727babc0a09386286928fea8b708cb6d03415ff353 encountered an error during Start: failure in a Windows system call: The virtual machine or container exited unexpectedly. (0xc0370106)
At line:1 char:1
+ docker start portainer
+ ~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (Error response ...y. (0xc0370106):String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

Error: failed to start containers: portainer

Expected behavior
Version 1.20.1 runs without problems. I expect version 1.20.2 to do the same. ;-)
Steps to reproduce the issue:
docker pull portainer/portainer
docker run -d --restart always --name portainer --isolation process -h portainer -p 9000:9000 -v C:\ProgramData\Portainer:C:\Data -v //./pipe/docker_engine://./pipe/docker_engine portainer/portainer

Technical details:

Portainer version: 1.20.2
Docker version (managed by Portainer): 18.09.3
Platform (windows/linux): Windows 10 Pro v1809
Command used to start Portainer: docker run -d --restart always --name portainer --isolation process -h portainer -p 9000:9000 -v C:\ProgramData\Portainer:C:\Data -v //./pipe/docker_engine://./pipe/docker_engine portainer/portainer
Browser: Chrome

Additional context
Version 1.20.1 works without any problem.

@ssbkang can you reproduce this one?

@deviantony Yeah, I can reproduce this
ocker run -d --restart always --name portainer --isolation process -h portainer -p 9000:9000 -v C:\ProgramData\Portainer:C:\Data -v //./pipe/docker_engine://./pipe/docker_engine portainer/portainer:1.20.2
834f7cc4ebf3e6fd7a564eb77ebb2f85794c99e65d60b886f3eab2e79e4267bf
C:\Program Files\Docker\Docker\Resources\bin\docker.exe: Error response from daemon: container 834f7cc4ebf3e6fd7a564eb77ebb2f85794c99e65d60b886f3eab2e79e4267bf encountered an error during Start: failure in a Windows system call: The virtual machine or container exited unexpectedly. (0xc0370106).

Yes I can reproceduce it too.

docker run portainer/portainer:1.20.2 (windows)
with --isolation process,
error message is encountered an error during Start: failure in a Windows system call: The virtual machine or container exited unexpectedly.
without  --isolation process,
The container is created, but docker start stucks.
No problem if I compile 1.20.2 from source and run portainer like this
docker run -d -p 9000:9000 -v %CD%\dist:C:\app -v %TMP%\portainer:C:\data -v \\.\pipe\docker_engine:\\.\pipe\docker_engine --name portainer mcr.microsoft.com/windows/nanoserver:1809 C:\app\portainer.exe --no-analytics --template-file C:\app\templates.json

@ajkauffmann @NAVspecialist @linquize would you please give another go with the latest image?
PS C:\Users\skan0> docker image pull portainer/portainer:latest
latest: Pulling from portainer/portainer
Digest: sha256:a75450f44e2f5bcece74a273afcd0627029e423f6ea1ba20b3af62a5556bbb06
Status: Downloaded newer image for portainer/portainer:latest
PS C:\Users\skan0>
PS C:\Users\skan0> docker run -d -p 9000:9000 --name portainer --restart always -v C:\ProgramData\Portainer:C:\Data -v \\.\pipe\docker_engine:\\.\pipe\docker_engine portainer/portainer:latest
20d93158f13f4a0c1a64f8a77238872aa22e270395d5a49495dc860b418dfc8d
PS C:\Users\skan0> docker logs -f 20d
2019/03/08 14:59:06 Starting Portainer 1.20.2 on :9000

Please try without --isolation process for now, there seems to be an issue with this.

Hello there
I have tried without the --isolation process and then I can start Portainer without an error. But but when I try to connect to an endpoint (Local) then I get the error:
"Failure -Endpoint is unreachable and there is no snapshot available for offline browsing."
I tried to change the \.\pipe\docker_engine:\.\pipe\docker_engine (backslash to forwardslash) but that made no difference.
In 1.20.1 - //./pipe/docker_engine://./pipe/docker_engine works quite fine also with --isolation process

I'm running by default in isolation process, so removing the option does do much I guess.
Pulled the latest image (created 2019-03-08 02:20:41) and created the container. No error message during container creation, but it won't work after all.
Docker logs gives me this error log:
docker : 2019/03/08 16:24:09 open C:\data/portainer.db: Access is denied.
At line:1 char:1
+ docker logs portainer
+ ~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (2019/03/08 16:2...cess is denied.:String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

2019/03/08 16:24:10 open C:\data/portainer.db: Access is denied.
2019/03/08 16:24:11 open C:\data/portainer.db: Access is denied.
2019/03/08 16:24:12 open C:\data/portainer.db: Access is denied.

This was with volume -v C:\ProgramData\Portainer:C:\Data
When create the container without this volume, I get this output from the logs:
docker : 2019/03/08 16:18:08 mkdir C:\data/tls: Access is denied.
At line:1 char:1
+ docker logs portainer
+ ~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (2019/03/08 16:1...cess is denied.:String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

2019/03/08 16:18:09 mkdir C:\data/tls: Access is denied.
2019/03/08 16:18:10 mkdir C:\data/tls: Access is denied.
2019/03/08 16:18:11 mkdir C:\data/tls: Access is denied.
2019/03/08 16:18:13 mkdir C:\data/tls: Access is denied.

Reverting back to version 1.20.1 and everything is normal again.

@ajkauffmann Thanks for testing. We've noticed that there is an issue with our current build - please use 1.20.1 for now, and we will get back to you soon as the build is fixed.

@ajkauffmann The latest image will be good now, please try again.

@ssbkang Tested and confirmed, it works with the latest image. Thanks for great work, much appreciated!

Awesome, thanks a lot for raising this issue - closing!


Responsive Design Fixes

Before:
https://github.com/midudev/minimalist-portfolio-json/assets/85462420/adc339d4-a260-4699-adb7-5c213e1151b6
After:
https://github.com/midudev/minimalist-portfolio-json/assets/85462420/7eb642f6-4f97-43b3-9ad5-584aea9b6c90

It's my first PR contributing to open source 🥹

👏


Component installation does not fail when loses connection (WindowsOS)

There is no any notification telling "no connection" or "no internet" for a user when the network breaks down.

Installation still does not fail when connection is lost.

Deleted The Subutai Boxes
Started installation process
and in the middle of I disabled the ethernet connection.
CC gave me a message that connection is lost.
Fixed in prod


Fix audience documentation

Fix the documentation for Signet::OAuth2::Client#audience. It looks like it was copied from #issuer.

@blowmage Can you rebase this branch for the rubocop fix that we just merged?

Rebased.


flexbox + ui grid + angular-resizable?

@Reklino - could flexbox + angular-resizable + ui grid produce a resizing ui grid that can shrink?

I believe so... But from my experience with UI-Grid, you may have to trigger the resize manually. I'd experiment with using the resizing event in angular-resizable to trigger the resize.

ok. so here's the starting point without using angular-resizable:
the running sample shows the ui grid in a flexbox. it starts out with scrollbars. if you grow the window, it resizes fine. if you then shrink the window, the grid maintains its largest size and scrollbars appear.
here's the repo: https://github.com/cmichaelgraham/ng-ui-grid-flexbox
ui-grid directive is here
app module def is here
i'll work on updating the sample to use angular-resizable next :smile:

ok, got the above working without angular-resizable - will circle back to implement a resizing version at some point...
i wrapped the ui grid div in a div that uses flex to grow and made it position relative, then I positioned the ui grid to be absolution with top, left, bottom, and right to 0px (well, right and bottom are 2px, to be honest :wink:)
Repo: https://github.com/cmichaelgraham/ng-ui-grid-flexbox
Running sample: http://cmichaelgraham.github.io/ng-ui-grid-flexbox/index.html

@cmichaelgraham - Did you ever get it working with the angular-resizable? This is exactly what I want to do...


Optimize index

What is the purpose of this change? What does it change?
Optimize creation of new index so that already existing index files are used.
Creation of new index is used at the moment in prune and rebuild-index.
With this pull request the index creation is much faster, especially for remote repositories.
Was the change discussed in an issue or in the forum before?
The change is similar to one part of PR #2340 and the problem is discussed in many issues, e.g. #2162
Checklist

[x] I have read the Contribution Guidelines
[ ] I have added tests for all changes in this PR
[ ] I have added documentation for the changes (in the manual)
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[x] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[ ] I'm done, this Pull Request is ready for review

Seems like the original PR broke some tests with packs in more than one index. This should be fixed now.

While still being confident that this PR can improve the current situation for some users, I now believe that on the long run a refactoring of rebuild-index and prune (and thus removing internal/index) should be favored.

Hi @aawsome,
I used this PR in conjunction with #2513 during my prune of a large (12M object / ~55TB) AWS S3 backed repository. I don't know how to tell if it was faster or slower than normal, since this was my first prune of this very large repository. As far as I can tell, it worked great. Backups have continued normally without any issue since the repository was pruned (almost 2 weeks ago).
It isn't much feedback, but it's the best I can do. I wanted to give something back, since I had used and tested this PR. Thanks!

@irasnyd

I used this PR in conjunction with #2513 during my prune of a large (12M object / ~55TB) AWS S3 backed repository.

Just to clarify: As far as I understood you used cleanup-index and cleanup-packs from #2513, right?
In this case this PR was not used at all. These two commands do not use internal/index as there is already an index data structure in repository/index using both data structure multiplies the memory usage and is one of the reasons why prune has a memory issue (besides the performance issue due to reading all packs)

@irasnyd

I used this PR in conjunction with #2513 during my prune of a large (12M object / ~55TB) AWS S3 backed repository.

Just to clarify: As far as I understood you used cleanup-index and cleanup-packs from #2513 instead of prune, right?

Yes, that is correct. I only used the cleanup-index and cleanup-packs commands from #2513. I did not use prune.

In this case this PR was not used at all. These two commands do not use internal/index as there is already an index data structure in repository/index. Using both data structure multiplies the memory usage and is one of the reasons why prune has a memory issue (besides the performance issue due to reading all packs which can be improved by using this PR)

Apologies for my confusion. I understand better what is going on now. Thanks!

I close this PR for two reasons:

with #2513 there is a functionality to replace prune which I assume to be even faster than prune with this functionality. Moreover, this PR is not needed for #2513
in #2523 I already proposed a new index implementation which IMO should be enhanced such that it can also be used within #2513 and then 'prune' can be replaced.


after updating pod its showing some errors

Hi @amrit42087
Care to provide which errors are you getting? Steps to reproduce the issue would be also welcome.

It works with no errors for me. If you are getting errors, you probably have not updated your main project to Swift 4.2 and it's trying to compile SwiftSpinner with Swift 4. By default, pods will inherit the Swift language version of the main project.


Main window always on top bug

Describe the bug
The main window stays always on top even after the setting is unticked.
To Reproduce
Steps to reproduce the behavior:

Go to Settings
Untick 'Main window always on top' under Miscellaneous.
Main windows still stays on top of other windows no matter what.

Expected behavior
The main window should go under other windows.
Screenshot

The main window is still on top even though the browser is the one in focus and 'Main window always on top' is unticked
Desktop (please complete the following information):
Edition	Windows 10 Pro
Version	20H2
Installed on	9/12/2020
OS build	19042.685
Experience	Windows Feature Experience Pack 120.2212.551.0

@samuelmacdg You are probably rendering OnlyM media. From the wiki:
"Miscellaneous => Main window always on top - If checked, the OnlyM window remains on top of others. Note that OnlyM automatically remains on top when media is being displayed."

Thank you for the response. No media is being played at the time but it might be rendering the media window which I've to always be displayed. Thanks again for the great work. We highly appreciate it.

Yes, if the media window is open then that will enforce the "on-top" behaviour. I'll close the issue for now.
Regards, Antony


chore: bdd-spring-1573151208 to 0.0.1

chore: Promote bdd-spring-1573151208 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


Support the Elasticsearch 5.x

Overview

Crowi can not work Elasticsearch 5.x

Include this pull-request

Upgrade dependency

11.x to 12.x

Modified old-style mappings

"type" : "multi_field" is deprecated.

https://www.elastic.co/guide/en/elasticsearch/reference/1.4/_multi_fields.html

use string instead of multi_field

http://stackoverflow.com/questions/40301061/elasticsearch-5-mapperparserexception-with-multi-field

Modified query field name

use stored_fields in Elasticsearch 5.x

When ES 2.x, use fields. Because ES 2.x does not support stored_fields

I got error messages like this:

{
  "error": {
    "root_cause": [
      {
        "type": "parsing_exception",
        "reason": "The field [fields] is no longer supported, please use [stored_fields] to retrieve stored fields or _source filtering if the field is not stored",
        "line": 1,
        "col": 11
      }
    ],
    "type": "parsing_exception",
    "reason": "The field [fields] is no longer supported, please use [stored_fields] to retrieve stored fields or _source filtering if the field is not stored",
    "line": 1,
    "col": 11
  },
  "status": 400
}

To consider this problem, we have to decide the policy which version of es we'll support.
The maintenance const increases if we'll support miltiple versions of es, so IMO, how about the idea below:

Crowi 1.5.x supports ES 2.x (and depends on 11.x),
Crowi 1.6.x supports ES 5.x (and depends on 12.x).

It seems like a good idea.
I plan to modify this p-r below:

Crowi 1.6.x

Check ES version setupSearcher

https://github.com/crowi/crowi/blob/6ef92d6b1e2c567be3b2fd87e7c6500a5579c55e/lib/crowi/index.js#L227

When detect ES 2.x

Show warning to console.log()
Set null to self.searcher

Remove appendFields method

https://github.com/crowi/crowi/pull/150/files#diff-3f4aa453cefa49f6431f1bba3bb53a8eR356

It would be great!

Log
ES 5.x

localhost:9200 is ES 5.x

% MONGO_URI=mongodb://localhost/crowi \
  ELASTICSEARCH_URI=localhost:9200 \
  REDIS_URL=localhost \
  FILE_UPLOAD=local \
  DEBUG=crowi:crowi node app.js
[development] Express server listening on port 3000

ES 2.x

localhost:9201 is ES 2.4.2

% MONGO_URI=mongodb://localhost/crowi \
  ELASTICSEARCH_URI=localhost:9201 \
  REDIS_URL=localhost \
  FILE_UPLOAD=local \
  DEBUG=crowi:crowi node app.js
  crowi:crowi Critical error occured. +0ms
[Error: Detect unsupported Elasticsearch version: 2.4.2]
Error: Detect unsupported Elasticsearch version: 2.4.2
    at /Users/suzuki/work/crowi-develop/crowi/lib/crowi/index.js:244:22
    at process._tickCallback (node.js:368:9)
From previous event:
    at Crowi.setupSearcher (/Users/suzuki/work/crowi-develop/crowi/lib/crowi/index.js:234:10)
    at /Users/suzuki/work/crowi-develop/crowi/lib/crowi/index.js:88:17
    at processImmediate [as _immediateCallback] (timers.js:383:17)
From previous event:
    at Crowi.init (/Users/suzuki/work/crowi-develop/crowi/lib/crowi/index.js:87:6)
    at Object.<anonymous> (/Users/suzuki/work/crowi-develop/crowi/app.js:10:7)
    at Module._compile (module.js:410:26)
    at Object.Module._extensions..js (module.js:417:10)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Function.Module.runMain (module.js:442:10)
    at startup (node.js:136:18)
    at node.js:966:3

in #181, support ES 2 and 5, and merged to master by #175
So close this PR thank you


IllegalArgumentException in org.apache code on Android 9

Recently, I upgraded to Android 9 and my app stopped working. I have tested the exact same code on a different phone with a clean install of the app on each of them and the app running on Android 9 gives this stack trace:
W/System.err: java.lang.IllegalArgumentException: Credentials may not be null W/System.err:     at org.apache.http.util.Args.notNull(Args.java:54)         at org.apache.http.impl.auth.BasicScheme.authenticate(BasicScheme.java:159)         at org.apache.http.client.protocol.RequestAuthenticationBase.authenticate(RequestAuthenticationBase.java:120)         at org.apache.http.client.protocol.RequestAuthenticationBase.process(RequestAuthenticationBase.java:99)         at org.apache.http.client.protocol.RequestTargetAuthentication.process(RequestTargetAuthentication.java:82)         at org.apache.http.protocol.ImmutableHttpProcessor.process(ImmutableHttpProcessor.java:133)         at org.apache.http.protocol.HttpRequestExecutor.preProcess(HttpRequestExecutor.java:167)         at org.apache.http.impl.client.DefaultRequestDirector.execute(DefaultRequestDirector.java:484)         at org.apache.http.impl.client.AbstractHttpClient.doExecute(AbstractHttpClient.java:835)         at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:72)         at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:56)         at org.lightcouch.CouchDbClientBase.executeRequest(CouchDbClientBase.java:480)         at org.lightcouch.CouchDbClientBase.get(CouchDbClientBase.java:531)         at org.lightcouch.CouchDbClientBase.get(CouchDbClientBase.java:542)         at org.lightcouch.CouchDbContext.serverVersion(CouchDbContext.java:122)         at org.lightcouch.CouchDbContext.<init>(CouchDbContext.java:57)         at org.lightcouch.CouchDbClientBase.<init>(CouchDbClientBase.java:102)         at org.lightcouch.CouchDbClientAndroid.<init>(CouchDbClientAndroid.java:111)         at me.retrodaredevil.solarthing.android.request.CouchDbDataRequester.requestData(CouchDbDataRequester.kt:43)
The stack trace goes on below with a few more unrelated calls. I'm thinking that this isn't something I'm doing wrong because it goes through 9 org.apache calls and 7 org.lightcouch calls. If I am doing something wrong, the code should really be changed so whatever it is that's happening throws an exception sooner.
Any help would be appreciated.

Hi there,
which version of apache httpclient used by your app?

I am not sure. I don't think I ever explicitly imported it. Here are my dependencies. I think the only relevant thing in here is the version of LightCouch, but maybe this will help.
dependencies {
    implementation fileTree(include: ['*.jar'], dir: 'libs')
    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
    implementation 'com.android.support:appcompat-v7:28.0.0'
    implementation 'com.android.support.constraint:constraint-layout:1.1.3'
    testImplementation 'junit:junit:4.12'
    androidTestImplementation 'com.android.support.test:runner:1.0.2'
    androidTestImplementation 'com.android.support.test.espresso:espresso-core:3.0.2'
    implementation 'com.github.eidolon1138:solarthing:v1.1.1'
    // https://mvnrepository.com/artifact/org.lightcouch/lightcouch
    implementation 'org.lightcouch:lightcouch:0.2.0'
}

Should I try explicitly importing another version of apache httpclient? Will that conflict with whatever Android uses?

It may be try to use the 'legacy' version of httpclient that shipped with android.

You're correct! I came across this post on stackoverflow and it fixed it with one simple line. https://stackoverflow.com/questions/50446148/how-to-use-apache-http-on-android-p
Line added inside  in AndroidManifest.xml:
<uses-library android:name="org.apache.http.legacy" android:required="false"/>


CommonPrefixes not parsed correctly if it contains multiple Prefix tags

<CommonPrefixes>  is not parsed correctly if it contains multiple <Prefix> tags.
S3 itself does not behave this way but S3Mock does (https://github.com/adobe/S3Mock/issues/215)
This may be an incorrect response but if it is botocore should raise an error not just return raw xml Element objects
tested with botocore 1.20.24
import botocore.session
session = botocore.session.get_session()
operation_model = session.get_service_model("s3").operation_model('ListMultipartUploads')
protocol = operation_model.service_model.protocol
parser_cls = botocore.parsers.PROTOCOL_PARSERS[protocol]
parser = parser_cls(timestamp_parser=lambda x: x)
parser
body = """
<?xml version="1.0" encoding="UTF-8"?>
<ListMultipartUploadsResult xmlns="http://s3.amazonaws.com/doc/2006-03-01/">
  <Bucket>botocoretest1374528673-218</Bucket>
  <NextUploadIdMarker></NextUploadIdMarker>
  <Delimiter>/</Delimiter>
  <Prefix>foo</Prefix>
  <MaxUploads>1000</MaxUploads>
  <IsTruncated>false</IsTruncated>
  <CommonPrefixes>
	<Prefix>foo/</Prefix>
	<Prefix>foobar/</Prefix>
</CommonPrefixes>
</ListMultipartUploadsResult>
""".strip()
parsed = parser.parse({'body':body, 'headers':{}, 'status_code':200}, operation_model.output_shape)
parsed

{'ResponseMetadata': {'HTTPStatusCode': 200, 'HTTPHeaders': {}},
 'Bucket': 'botocoretest1374528673-218',
 'Prefix': 'foo',
 'Delimiter': '/',
 'NextUploadIdMarker': '',
 'MaxUploads': 1000,
 'IsTruncated': False,
 'CommonPrefixes': [{'Prefix': [<Element '{http://s3.amazonaws.com/doc/2006-03-01/}Prefix' at 0x1098830e0>,
    <Element '{http://s3.amazonaws.com/doc/2006-03-01/}Prefix' at 0x109883130>]}]}

Hi @smoofra,
Have you seen this behavior in the wild with an actual request?

Yes, I encountered it when I was writing some unit tests for my application  using s3mock.

Hi @smoofra,
Thanks. I meant have you seen it when running a request against an actual AWS S3 endpoint?
I can reproduce your example, but I'm not familiar enough with using delimiters to know immediately if the syntax is not correct. I also tried it with a real response for a ListObjectsV2 request where there are multiple prefixes, and it parsed it as expected.

Checking in as there hasn't been any update here in a while. Is this still an issue in recent versions of boto3/botocore and if so can you share a snippet for reproducing it?


Please submit builds in the releases tab

Before you start...

[X] Have you checked if similar issue have already been reported?
[ ] Have you updated the FluentHub? You might be using an old version.

Description

Alternative solutions

Relevant Assets

We're taking a look to it because we are having some issues. We'll provide the bundle as fast as possible in next releases. By the moment, the only way to open/install the app is by compiling it by yourself. Sorry for the inconveniences.

I'm making sideload pipeline for GitHub release
https://github.com/fluenthub-community/FluentHub/blob/sideload-azure-pipelines/builds/azure-pipelines-sideload.yml

Uploaded. can you install? @Advik-B


Number of records mismatch with GBIF

The latest publication of Watervogels (3.3) has 601,495 records, while GBIF indicates 605,228 (3.733 more). This might be due to an old problem where GBIF records where not deleted, see: http://dev.gbif.org/issues/browse/POR-2343 (which discusses Florabank).

Suggested auto-delete: http://dev.gbif.org/issues/browse/PF-2279

Thank you for the feature request.  I have fixed the 3 smallest ones, but the larger one seems to have discrepancies.  I am forcing a recrawl now.
If there is a web developer who cares to try, it would be relatively easy to fork this project [1] to use the IPT inventory [2] to automate this check.
[1] http://timrobertson100.github.io/vertnet-gbif-diagnostic/target/index.html
[2] http://data.inbo.be/ipt/inventory/dataset

All 4 datasets are now correctly up to date in GBIF

Thanks @timrobertson100


Move locked account functional test scenario from legacy to master

There are more to go before we can completely delete the "Legacy" tests here: https://github.com/alphagov/digitalmarketplace-functional-tests/blob/jenkins/features/supplier/supplier_journey.feature
But... it's the end of the mission so I'm going to PR these on their own now.  Slowly slowly we will whittle it away...
I also adjusted the password reset Scenario to perform the journey from the homepage, rather than just starting directly at the page itself.
This feels like a useful thing for functional tests to do - we want reassurance that links between different frontend apps are all working OK, and that users can do the thing without needing to know a particular URL.

WFM :+1:


disconected

I was running a dedicated server, and I joined I was trying to move corves good feather to lunar island I used command prompt "c_gonext("lunar sapling") and it worked then I turn on Free Crafting and precraft the telolecaterfocus then I click and  I get disconnected so I go on VoxLauncher and get the message NoSuchProcess: .  your servers will be saved and shut down However, the apps UI may not show it properly

did I mention in charter select maxwells head didn't show up Hehe


Cross field validation

I need to validate a password confirmation but on
public boolean validate(Annotation annotation, CharSequence input) { ...

I only have access to the value of the field itself and there is no way to access the password field.
I hacked the library in this nasty way:
@Custom(value = CustomValidatorInline.class, messageId = R.string.validation_password_no_coincidence)
private EditText editTextPasswordConfirmacion;

CustomValidatorInline.password = editTextPassword.getText().toString();
if(FormValidator.validate(this, new SimpleErrorPopupCallback(this))) { ...

public static class CustomValidatorInline extends BaseValidator<CharSequence> {

	private static String password;

	@Override
	public boolean validate(Annotation annotation, CharSequence input) {
		if (input == null) {
			return false;
		}
		else {
			String confirmation = input.toString();
			return confirmation.equals(password);
		}
	}

}

It's been a year and you haven't posted update. I think it's about time to do that. :)

+1


Refactor app-ecs-services into a module and per-environment projects

Why I am making this change
Refactoring the Terraform projects will enable us to run terraform commands directly, without wrapper scripts and source environment_{environment}.sh to set .tfvars.
What approach I took
Much the same as #218 etc.
Paired with @matthewcullum-gds.

Should I delete deploy.sh?

@idavidmcdonald No, we don't differentiate any more between enclave and non-enclave. Bye bye Makefile and hello terraform ....

There are references to deploy.sh in https://github.com/alphagov/prometheus-aws-configuration-beta/tree/terraform_module_app-ecs-services/terraform/modules/enclave still I think

I think we can also delete the stacks/ directory and the references to it in .gitignore 🚮


Fix some tests and docs.

Fix some tests and docs. Merge to the master.
Target branch: buffer-set.

@parisk review please.

@AndrienkoAleksandr can you please rebase with the updated buffer-set branch?

ok, I will rebase.

done, @parisk.

We need to fix one more test escape-sequences-test.js ...


Coverage

Added some tests to achieve 100% statement coverage.
Added handling of unsupported inputs in the expression constructor.
Optimized code in Expression.toString and Expression.toTex

Hah, thank you.


DBZ-6672 Improve NotificationIT stability

closes: https://issues.redhat.com/browse/DBZ-6672

@mfvitale Applied, thanks


iTunesRichPresence for MAC!?!?!

Hello! I was wondering if you could make a version for Mac?!?!

Duplicate of #68, #37, #23
As noted in the readme, there are currently no plans for a macOS version of the app. There is an existing issue (#68) open tracking this feature request.

Okay, thank you! I gave you the new logo here,
https://github.com/nint8835/iTunesRichPresence/issues/79
On Tue, Mar 2, 2021 at 11:14 AM Riley Flynn<EMAIL_ADDRESS>wrote:

Duplicate of #68
https://github.com/nint8835/iTunesRichPresence/issues/68, #37
https://github.com/nint8835/iTunesRichPresence/issues/37, #23
https://github.com/nint8835/iTunesRichPresence/issues/23
As noted in the readme, there are currently no plans for a macOS version
of the app. There is an existing issue (#68
https://github.com/nint8835/iTunesRichPresence/issues/68) open tracking
this feature request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nint8835/iTunesRichPresence/issues/81#issuecomment-789024557,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS6233KDZBODWN6XQ4THLHTTBUFIFANCNFSM4YPIOWGA
.


Feature Request : Operations menu

The Operations menu is messy, and long, and only about half of it is really needed.
Suggestions:

Add a "Collapse All" option to shrink the list for moving operations around easier
Start with files and elements collapsed
Keep things collapsed after they are moved
Add a "group" option that will AUTOMATICALLY group things by functions, put all of the Raster stuff together, then the engraving, and finally the cut operations.
Option to have the menu start collapsed

The tree stuff is being worked on currently, since it suffers a series of defects.  The work in 0.7.0 is mostly focused on making it dynamic and allowing grouping. So in theory if you load a file it will just modify the tree rather than needing to rebuild the whole thing, and allow it to build the icons dynamically in their own threads because that stuff is slow and causes serious optimization issues.

You can collapse that, you just can't tell. Click on it and hit the left arrow on your keyboard. Though any change to the tree will make it expand all again.
Files is going to be deprecated. They will be a custom group around elements. So elements -> file -> paths and you could easily access the sub elements.
Collapsing isn't something the current tree does well, collapsing things is damned near pointless since they will automatically expand again the next thing you tweak.
Grouping stuff is a primary need and part of the reason for the rewrite. There's also functional needs for grouping within the cutplanning. Grouping is also needed to move objects as groups. So if the object is a single thing it shouldn't all need selecting. It should select correctly.
Staying collapsed works, starting collapsed might not be useful except for some options like templates (groups of operations that should be switched on fly for different materials etc). It might be worthwhile to save whether the particular operations should be collapsed, but not as part of the startup, but rather as part of the saved file.

All good stuff...
#3, not staying collapsed... a bother, as I was trying to group things so
as to not have a cut in the middle, and it was a pain every time i did
something it insisted on expanding the {unrepeatable} list...  if a user
collapses it, it should STAY collapsed  until they expand it.
my $.02 anyway...
On Tue, Feb 9, 2021 at 5:49 PM tatarize<EMAIL_ADDRESS>wrote:

The tree stuff is being worked on currently, since it suffers a series of
defects.  The work in 0.7.0 is
mostly focused on making it dynamic and allowing grouping. So in theory if
you load a file it will just modify the tree rather than needing to rebuild
the whole thing, and allow it to build the icons dynamically in their own
threads because that stuff is slow and causes serious optimization issues.

You can collapse that, you just can't tell. Click on it and hit the
left arrow on your keyboard. Though any change to the tree will make it
expand all again.
Files is going to be deprecated. They will be a custom group around
elements. So elements -> file -> paths and you could easily access the sub
elements.
Collapsing isn't something the current tree does well, collapsing
things is damned near pointless since they will automatically expand again
the next thing you tweak.
Grouping stuff is a primary need and part of the reason for the
rewrite. There's also functional needs for grouping within the cutplanning.
Grouping is also needed to move objects as groups. So if the object is a
single thing it shouldn't all need selecting. It should select correctly.
Staying collapsed works, starting collapsed might not be useful
except for some options like templates (groups of operations that should be
switched on fly for different materials etc). It might be worthwhile to
save whether the particular operations should be collapsed, but not as part
of the startup, but rather as part of the saved file.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/meerk40t/meerk40t/issues/354#issuecomment-776296599,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJ2GY4FFQNL2IJI6CI4CFTS6G3WVANCNFSM4XLZOYEA
.

--
Need a quick answer? Call our booking line at<PHONE_NUMBER>, or book your
inspection on our website 24/7.

Bob Sisson, ACI, BVI
Inspections by Bob, LLC
www.inspectionsbybob.com
<EMAIL_ADDRESS>MD Lic#29666; ASHI Member #212016

Tree stuff got finished.


Basic Integration with BED and Salesforce API.

Description of change
It's now possible to interact with BED and Salesforce using the BED salesforce API.
Checklist

[X] If this is a releasable change, has a news fragment been added?

Explanation
A news fragment is required for any releasable change (i.e. code that runs in or affects production) so that a corresponding changelog entry is added when releasing.
Check changelog/README.md for instructions.

[X] Has this branch been rebased on top of the current develop branch?

Explanation
The branch should not be stale or have conflicts at the time reviews are requested.

[x] Is the CircleCI build passing?

General points

Other things to check

Make sure fixtures/test_data.yaml is maintained when updating models
Consider the admin site when making changes to models
Use select-/prefetch-related field lists in views and search apps, and update them when fields are added
Make sure the README is updated e.g. when adding new environment variables

See docs/CONTRIBUTING.md for more guidelines.

Break the PR's up make smaller


commits leading up to v3.0

Azure pipelines passed in previous run (all Python versions, all platforms). Updates to github actions script for conda install run into issue with a wrong version of libgcc (called by scipy fft). Will ignore this (and hope for the best).


a beautiful color logger is wanted.

I think there should be a improvement for logger: beautiful log message format can make us happy.
tools:

coloredlogs
loguru

Pr is welcome ~

In related works, I use loguru loguru a lot.  So, I suggest this tool for python-wechaty

Using an existing logger module/package would be great!
The TS/JS Wechaty is using brolog which is written by myself and the reason is very simple: I want to use a logger which can compatible with the Node.js & Browser environment together.
If we do not consider the Browser, like Python Wechaty (other Polyglot as well), we can consider using a more mature solution other than writing by ourselves.

This is my consideration: beautiful & mature.


Ephemeral Pull

Overview
All ephemeral pull consumers to be created during subscription.
The behavior is documented in ADR-X.
Clients and Tools

[x] Schemas @ripienaar
[ ] Go @wallyqs
[x] Java @scottf
[ ] JavaScript @aricart
[x] .Net @scottf
[ ] C @kozlovic
[ ] Python @wallyqs
[ ] Ruby @wallyqs
[ ] Rust @derekcollison @Jarema @caspervonb

Other Tasks

[ ] docs.nats.io updated
[ ] Update ADR to Implemented
[ ] Update client features spreadsheet

Client authors please update with your progress. If you open issues in your own repositories as a result of this request, please link them to this one by pasting the issue URL in a comment or main issue description.

You can remove me from the Rust client, Tomasz and Casper driving 100% these days.


set command in which-key not work

I can use this plugin normally by manually running command.
But when I set cmd on which-key plugin, it only can convert the current buffer all indent, can not use for range and still in visual mode.
Kooha-2023-08-11-01-31-21.webm
wk.register({
...
...
	i = {
		name = "indent",
		t = { "<cmd>TabsVsSpacesConvert spaces_to_tabs<cr>"	, "space to tab"},
		s = { "<cmd>TabsVsSpacesConvert tabs_to_spaces<cr>"	, "tab to space"},
	},
}, {
	prefix = "<leader>",
	mode = {"n", "v"}
})

but follow command in which key can work:
wk.register({
...
...
	i = {
		name = "indent",
		t = { "<cmd>'<,'>TabsVsSpacesConvert spaces_to_tabs<cr>"	, "space to tab"},
		s = { "<cmd>'<,'>TabsVsSpacesConvert tabs_to_spaces<cr>"	, "tab to space"},
}, {
	prefix = "<leader>",
	mode = {"n", "v"}
})

it should be worked in the first method (I think it should be).

Thanks for the report. Sorry about taking forever to reply.
This is as documented:

This is intended behavior, as the visual marks are what you automatically get when you start entering a command : with a range selected. Without the visual selection marks, it should target the entire buffer.


Use IP multicast for faster broadcast and multicast on netlrts

Original issue: https://charm.cs.illinois.edu/redmine/issues/1876

https://en.wikipedia.org/wiki/IP_multicast

Original date: 2018-04-19 06:47:10

I think verbs has similar support

Original date: 2018-05-09 15:57:02

I think we probably want to implement this with verbs first, rather than for netlrts, since we care more about verbs performance. But we'd want an LRTS API for this so that we can then implement support for it on whatever machine layers support it.

Original date: 2018-05-14 23:30:13

It looks like the network protocol aspect of IP multicast is fairly simple. Sending packets is the same as with unicast, only with a group address instead of an individual address. Listeners do the following initialization once:
struct ip_mreq request;
request.imr_multiaddr.s_addr = /* multicast group address, such as <IP_ADDRESS> */;
request.imr_interface.s_addr = htonl(INADDR_ANY);
int retval = setsockopt(/* socket */, IPPROTO_IP, IP_ADD_MEMBERSHIP, (char *)&request, sizeof(request));
if (retval != SOCKET_ERROR) { /* success... */ }

The IP multicast API does not help with choosing a group address. Our implementation must pick one. (Querying techniques require raw sockets, which need root privileges at some stage for access to be granted to a program.) Two concerns arise from this:

Performance: Network hardware must filter unwanted packets received from a multicast group address.
Correctness: I'm not certain this is relevant, but if Converse receives packets from a different Charm++ instance on the same cluster, will it know to reject them?

This is my estimation of what IP multicast support needs:

Constructing the LRTS and machine layer infrastructure for hardware-level broadcasts
Testing at init time that all nodes are reachable with IP multicast, falling back to unicast if not usable
Handling any correctness issues that may arise from suboptimal network hardware implementations
Add user-facing configuration of group address


Added Tab to select a user

When pressing Tab, it will now mention the user

this also needs to be done in kibbeh


[Question] Project status

Hi! Thanks for this awesome gem!
I noticed that PRs are not being merged, so I added the project to my list of gems needing contributors.
If the project is currently being maintained or just on a hiatus, please, do respond and I'll remove it from the list.
Thanks again! :smile:

I also noticed that the build matrix only tests Rails 5.2, not 6.0 or 6.1 -- but, then, CI isn't actually working anyway, becuase it's set up to run on travis-ci.org, which has been disabled by it's owner.
I am left thinking lograge is kind of maintainer-abandoned, and not tested under Rails 6+.  It is still recommended by many places on the internet though (including heroku docs).


[Improvement] Test all combinations of Backend Integration Test at once

What would you like to be improved?
Currently, we run backend integration tests with these (testMode, backend) combinations:

embedded, jdbcBackend
embedded, kvBackend
deploy, jdbcBackend
deploy, kvBackend

But this will cause CI pipeline to run 12 times because we also have to test 3 Java versions (8, 11, 17),
which is a big waste of time.
And even more, at local test with IDE like Intellij, we can only run tests with default config: embedded, kvBackend.
For those who doesn't familiar with Integration test configurations, they will find their code failed at CI with config like deploy, jdbcBackend and can't reproduce locally.
How should we improve?
Continue from the discussion at here in #2813 .
we could modify startIntegrationTest method to:
@ParameterizedTest
    @CsvSource({
        "embedded, jdbcBackend",
        "embedded, kvBackend",
        "deploy, jdbcBackend",
        "deploy, kvBackend"
    })
    void startIntegrationTest(String testMode, String backend) {

But I found that @BeforeAll seems to be executed only one time.
So my question is: Is it a way to parameterize beforeAll ?
After I googled it and found this comments, but I don't know it works or not.
https://github.com/junit-team/junit5/issues/3157#issuecomment-1435009129
c.c. @yuqi1129, @mchades

I will close this issue since the kvBackend is deprecated now.  You can open a new issue if necessary


TMDB provider links

Summary
Needs like 2 more backup links not just one it only has vidsc.to needs more servers than that
Please confirm the following

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.

Duplicate of #98.

I can't play some movies and series I get this error in the tmdb provider in the streamflix app
no extractors  found

Duplicate of #98


Add Climate Data Operators

Insert URLs to the project here: https://code.mpimet.mpg.de/projects/cdo

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as Good First Issue of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.
All new projects listed will be posted on our Mastodon channel.

https://ecosyste.ms/ doesn't currently have support for redmine but probably should be added.


Create battery-information.py

The more "abstract" functions that battery information can do. More can be added based on battery type (1s, 2s, etc) and at request.

@WestonZavadil LMK when this is ready for a final review and if the script I created worked for allowing it to run

@WestonZavadil Any updates on this? Or is it waiting for a review?


TestWorkflow_SearchArtifacts failing on master

Caused by #8557.
https://github.com/argoproj/argo-workflows/actions/workflows/ci-build.yaml?query=branch%3Amaster
@rohankmr414 could you please take a look?

error: error pruning namespaced object /v1, Kind=PersistentVolumeClaim: rpc error: code = Unavailable desc = transport is closing
make: *** [Makefile:434: install] Error 1

@alexec I guess there is some issue with the installation. Not sure if #8557 caused this.

Not sure what is going on here. This is failing in unrelated PRs.

https://github.com/argoproj/argo-workflows/runs/6242825979?check_suite_focus=true

An example of a build that is failing, yet is on a branch which does not contain releated changes:
https://github.com/argoproj/argo-workflows/runs/6242825979?check_suite_focus=true

@alexec looks like I messed up the unit tests for ArtifactSearchResults #8557. Looks like the fix https://github.com/argoproj/argo-workflows/pull/8554/commits/70c6a9d6bbab740a288fe7447cda574aa8739d3f by @wujayway in #8554 should fix the issue

I think the fix should come in its own PR, otherwise all PR are blocked until that PR is merged,. Could I please ask you to submit that?

I think the fix should come in its own PR, otherwise all PR are blocked until that PR is merged,. Could I please ask you to submit that?

yeah sure


Training for voc2012

In your codes,i want to know if it just for voc2007 or i need to add some extra codes when use the voc2012 dataset?Because when i use the voc2012  to train,a error occur:
FileNotFoundError: [Errno 2] No such file or directory: '/home/dl/VSST/zwm/DCNet-main/VOCdevkit/VOC2012/Annotations/000005.xml'
as we know,the annotation of voc2012 datasets use the 'year_id.xml' format.Thank you for your answer.

No need for extra code for voc2012. But if you still have this problem, please feel free to reopen this issue.


Moving _check_endings call to make_future

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[x] Did you update the CHANGELOG?

Type of Change

[ ] Examples / docs / tutorials / contributors update
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Improvement (non-breaking change which improves an existing feature)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Proposed Changes

Related Issue

Closing issues

closes #399

Codecov Report

Merging #413 (2f88adc) into master (7e43eb9) will decrease coverage by 33.59%.
The diff coverage is 42.85%.

@@             Coverage Diff             @@
##           master     #413       +/-   ##
===========================================
- Coverage   87.09%   53.49%   -33.60%
===========================================
  Files          98       99        +1
  Lines        4945     5002       +57
===========================================
- Hits         4307     2676     -1631
- Misses        638     2326     +1688

Impacted Files
Coverage Δ

etna/datasets/tsdataset.py
71.03% <42.85%> (-21.74%)
:arrow_down:

etna/commands/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

etna/commands/backtest_command.py
0.00% <0.00%> (-96.43%)
:arrow_down:

etna/commands/forecast_command.py
0.00% <0.00%> (-92.00%)
:arrow_down:

etna/commands/__main__.py
0.00% <0.00%> (-87.50%)
:arrow_down:

etna/commands/resolvers.py
0.00% <0.00%> (-80.00%)
:arrow_down:

etna/analysis/outliers/density_outliers.py
22.44% <0.00%> (-75.52%)
:arrow_down:

etna/transforms/fourier.py
27.08% <0.00%> (-72.92%)
:arrow_down:

etna/transforms/filter.py
28.57% <0.00%> (-71.43%)
:arrow_down:

etna/analysis/outliers/median_outliers.py
25.00% <0.00%> (-70.84%)
:arrow_down:

... and 61 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 7e43eb9...2f88adc. Read the comment docs.


Support for vite 4

Currently the peer dependencies restrict vite to<EMAIL_ADDRESS>|| ^3.0.0" but vite 4 is already available.
Is this plugin compatible?

I went through migration guide of Vite 4, and I don't see any breaking changes that would affect this plugin. I will bump peer dependencies to ^4.0.0 in the near future. Let me know if you find any bugs!

Updaten in version v1.1.3


chore(package.json): Update karma bin path

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Reintroduces bin to path for npm test similar to https://github.com/AngularClass/angular2-webpack-starter/commit/216fa0db6c7e4092cfdf22b25aafb9b0bdfaba95

What is the current behavior? (You can also link to an open issue here)

Error: Cannot find module '/Users/user/angular2-webpack-starter/karma'
    at Function.Module._resolveFilename (module.js:339:15)
    at Function.Module._load (module.js:290:25)
    at Function.Module.runMain (module.js:447:10)
    at startup (node.js:139:18)
    at node.js:999:3
npm ERR! Test failed.  See above for more details.

What is the new behavior (if this is a feature change)?
Tests run.

Other information:

thanks!


will dockerlint restrict user from installing ssh and telnet ?

Hi @daashayani, this repo is also a Docker authorization plugin. And it's based on Casbin, a new authorization library which supports flexible access control models like ACL, RBAC, ABAC.
If you are saying that if the Docker user can be restricted from installing a software inside a Docker container, I'm afraid not.

@hackerchai

@hackerchai


Support for Python 3.9

Supersedes #7166
Depends on https://github.com/dask/distributed/pull/4460

Contains a temporary hack needed until the distributed PR is merged.
Note: numba support for Python 3.9 is already available in numba-0.53rc2 on the numba channel; however it contains a regression that causes our tests to fail: https://github.com/numba/numba/issues/6773

I'm getting issues on parallel pytest on MacOS.
@jrbourbeau or anybody else with a macbook at hand: could you please run this and report the output?
conda create -y -n demo_hash_py39 -c conda-forge python=3.9
conda activate demo_hash_py39

python --version
python -c 'print(hash("Hello world"))'
python -c 'print(hash("Hello world"))'
PYTHONHASHSEED=42 python -c 'print(hash("Hello world"))'
PYTHONHASHSEED=42 python -c 'print(hash("Hello world"))'

conda deactivate
conda env remove -n demo_hash_py39

The current situation is that Python 3.9 on Windows is unstable. I had

a C-level stack overflow on an unspecified test
an interpreter crash on a trivial test that should trigger a RecursionError

the two things are likely related. I'm going to try and reproduce on my local windows machine.
Everything else is good to go.

The issue in Windows was caused by ipython, which causes the interpreter to crash instead of raising RecursionError: https://github.com/ipython/ipython/issues/12197

@jrbourbeau @mrocklin Both PRs are ready for review and merge.
The latest round of tests will fail as I just reverted to distributed@master.
Good tests visible as of the previous commit https://github.com/dask/dask/runs/2013590328


Silent error on undefined

I'd like to use conditional tasks. It would be nice to add this feature.
run(env.clean ? 'clean' : undefined, function () {});

Maybe better pass this error in callback?

Resolved with promises
new Promise(function (resolve) {
	if(env.clean) {
		run('clean', resolve);
	} else {
		resolve();
	}
}).then(function () {
	run(['script', 'style', 'markup', 'sprite', 'image', 'font'], function () {
		done();
	});
});

You are making this overly complicated.  Just put an if block around your run-sequence.
if(env.clean) {
    run('clean', done);
} else {
   done();
}

I will not be removing the error on invalid task names (or empty task names).

@OverZealous My decision is about processing assets after optional clean.

@TrySound That doesn't change my answer.  You can also use apply and an array to call run-sequence with an unknown number of parameters.


Interval selection to single value

If I have for example a double selection in this interval [1, 5], can I select only one value, for example only the value 3? (the min and the max to overlap on the same value)

At the moment it doesn't, but I guess it would be useful to allow min and max to be the same.


Crash on attempting to empty/delete recycle bin content

Crash on attempting to empty/delete recycle bin content
To Reproduce
Steps to reproduce the behavior:

Go to 'Aves Gallery'
Delete a bunch of pics
Go to Recycle Bin
Attempt to empty the recycle bin/delete a selection of images
See error

Expected behavior
Just delete the stuff and show empty recycle bin kinda message
System information:
Aves version: 1.6.0-izzy (Build 1066)
Flutter version: 2.10.1 (Channel stable)
Android version: 11 (SDK 30)
Device: Xiaomi MI A2
Google Play services: ready
System locales: en_US
Aves locale: en

Thanks for the report! I can't reproduce this with my own test devices, but I just saw similar Crashlytics reports (from an LG device with Android 7.0). I'll take a look.

releasing v1.6.1 with the fix


Add Text Blast to Example App Showcase

Text Blast is an open source MMS text blasting app written in React Native (source) and also includes an analogous Ionic implementation (source) which should be particularly useful to Ionic developers considering React Native and vice-versa.

@facebook-github-bot shipit

Thanks Brent!

@facebook-github-bot shipit

merged


Weird error: "Resolving ... for build dependencies doesn't lead to expected result .... Resolving dependencies are ignored for this path."

This probably isn't an issue with the duplicate package checker plugin, but this warning is surfaced by it. Does anyone know of potential solutions to solve it?
<w> [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] Resolving 'color-name/package.json' in /path/to/project/node_modules/duplicate-package-checker-webpack-plugin/lib for build dependencies doesn't lead to expected result '/path/to/project/node_modules/color-name/package.json', but to '/path/to/project/node_modules/duplicate-package-checker-webpack-plugin/node_modules/color-name/package.json' instead. Resolving dependencies are ignored for this path.
<w>  at unknown 4 color-name/package.json
<w>  at file dependencies /path/to/project/node_modules/duplicate-package-checker-webpack-plugin/lib/index.js
<w>  at file /path/to/project/node_modules/duplicate-package-checker-webpack-plugin/lib/index.js
<w>  at file dependencies /path/to/project/sub/path/webpack.config.js
<w>  at file /path/to/project/sub/path/webpack.config.js
<w>  at resolve commonjs /path/to/project/sub/path/webpack.config.js
<w> [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] Resolving 'color-convert/package.json' in /path/to/project/node_modules/duplicate-package-checker-webpack-plugin/lib for build dependencies doesn't lead to expected result '/path/to/project/node_modules/color-convert/package.json', but to '/path/to/project/node_modules/duplicate-package-checker-webpack-plugin/node_modules/color-convert/package.json' instead. Resolving dependencies are ignored for this path.
<w>  at unknown 4 color-convert/package.json
<w>  at file dependencies /path/to/project/node_modules/duplicate-package-checker-webpack-plugin/lib/index.js
<w>  at file /path/to/project/node_modules/duplicate-package-checker-webpack-plugin/lib/index.js
<w>  at file dependencies /path/to/project/sub/path/webpack.config.js
<w>  at file /path/to/project/sub/path/webpack.config.js
<w>  at resolve commonjs /path/to/project/sub/path/webpack.config.js

I was able to resolve the issues by removing the duplicate packages from my project using yarn resolutions:
  "resolutions": {
    "color-convert": "2.0.1",
    "color-name": "1.1.4",
...


Outer whitespace -- such as "4pm " -- causes the timer to not start

Ah okay, automagically added it here. Perfect. Getting use to github a bit. :D


[Search v1] Implement HoldMoneyRequestOnSearch command

We've unblocked Hold as an action on the Search page, so we need to go ahead and implement this section and of the doc for the Hold command only
Issue OwnerCurrent Issue Owner: @luacmartins

Just assigned so I can keep track of the issue, but we're looking for internal volunteers here.

Thanks @luacmartins - waiting for volunteers

not overdue

waiting for internal volunteers

I have a bit of bandwidth so I'll work on this one since we haven't gotten volunteers yet.

Got a couple of PRs up. Gonna finish the Auth one on Monday.

PRs in review

This is done!


cant fill empty bucket by dragging

Description

You cannot fill a bucket, mop bucket or trolley by dragging it if it is empty.
First, you need to fill the bucket once with a blue bucket, and then you will be able to fill it to the top by dragging.
Reproduction

Drag empty mop bucket on water tank
Dont work

Screenshots

Additional context

Chemistry got hit by this bug/rework introduced 8-9 days ago too, as you no longer can jugmix by dragging either.


Default profiles are on by default

This avoids annoying issues with IDE integration and building where you have to remember incantation to run correct combination. This entails following changes

Hadoop default changed from 2.7.4 to 2.9.1
yarn, kubernetes, hadoop-cloud and kinesis modules are on by default

SparkR is left out since it requries a bit more invasive changes to enable by default

@lwwmanning

👍


Fix error printing

See https://github.com/microsoft/vscode/pull/86405 for details

@bpasero So, it seems that we're repeating what was in the previous PR :-)
Even without message it looks decent. I.e.:
  ERR : Error:
       at stacktrace
       at stacktrace

So, I think, it makes no sense to add a message. It shows a type and empty message after colon.
What do you think?
P.S. I can't reuse the code from spdlogService.ts as @sandy081 suggested, since we use the fact that console allow printing objects in JS representation, and thus more readable than what's available in the file logger.

@bpasero Also, the error isn't unknown, it's Error, it just doesn't have a message.

Thanks, I pushed one commit on top. The reason why I think we always want an error message is that these errors might be shown to users and it is not very helpful to have no information provided in that case.

@bpasero That sounds great! Will check whether the error reported correctly.


Customer support ladder?

👋  Hi. Dave Rupert here. Long time Drasnerd. I love the rubric you've developed here and have even started implementing it in my own company and startup.
Have you had any thoughts around ladders for customer support (aka "customer success") roles? Is that similar to a DX role? I'm imagining a less public-facing role, a person who fields emails, stack overflows, or social media support DMs.
Would love to hear your thoughts. Not time sensitive in any way.

Hey Dave!
What you're describing is what I would call a Support Engineering role, or at least at Netlify that's how we defined it and it was in another part of the company, so it's not included here. But I agree it's an incredibly important role!
If someone else has one for that type of role, I'm happy to link it up in the readme so that people can find it as a resource.
Hope you're doing well :)


Add workaround for MSVC mutex constructor issue

This issue is a MSVC compiler bug affecting certain versions of Visual Studio 2022. When using std::mutex  a null dereference may occur, leading to a silent crash in Release mode, as illustrated in the image below.

Adding the compiler option /D"_DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR" serves as a workaround for this problem.
Reference:
https://hydrogenaud.io/index.php/topic,126070.0.html
https://github.com/microsoft/STL/wiki/Changelog#vs-2022-1710

@ilya-lavrenov, could you please kindly review it, thanks!

build_jenkins


Allow formStore.setError error message to be a ReactElement

Motivation
https://github.com/ariakit/ariakit/blob/70b384c5f29bf8d2de81b5260c19b896d27e8069/packages/ariakit-core/src/form/form-store.ts#L30
Currently, the error message is restricted to only be a string. This is quite limiting. Allow ReactElement to be used as well, so we can have actionable buttons in the error.
Usage example
formStore.setError(
  formStore.names.email,
  <>
    There is no account associated with this email address. Consider
    <button>signing up</button>?
  </>,
);

Requirements
Typescript type check should pass when a ReactElement is used as the error message.
Workaround
 // @ts-expect-error This works, the type is just too strict.

Currently, I can get the desired behaviour by suppressing Typescript.
Possible implementations
No response

We could potentially type this as ReactNode as it already encompasses string and other types. However, I believe this might cause a breaking change because some users might have been using this value in contexts that only support strings.


Fixed some broken image links in docs

The four images in The Python Server API by Example (docs) weren't working because their path had changed. I fixed that.

[Current coverage][cc-pull] is 88.14%

Merging [#372][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #372   diff @@
==========================================
  Files            18         18
  Lines          1240       1240
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1093       1093
  Misses          147        147
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [9451348...ccb7f79][cc-compare]
[cc-base-branch]: https://codecov.io/gh/bigchaindb/bigchaindb/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/bigchaindb/bigchaindb/compare/9451348d34cd1b2b711d6347b08c4d8ac9617aae...ccb7f791304d0bd9ff4938eb1d5b7bc3f7d7f857
[cc-pull]: https://codecov.io/gh/bigchaindb/bigchaindb/pull/372?src=pr


feat: do not use default tide features

Remove use of Tide default crate features.. Without this I am forced to use Tide's default logger as well as a couple other features.

Thanks!

Published to crates.io as v0.5.0!

thank you @chrisdickinson


Fix widget ordering In Dashboard not working

Fixes widget ordering in Dashboard not working

Hi @cgjgh,
Good catch!  I had already provided that fix on another node, but forgot to implement it here.
Thanks!!
Bart


Tweak getter function made throwable

Description

The getter function of Tweak is made throwable

Motivation and Context

With the addition of these changes #63 we now have multiple reasons where Tweak can be a nil value.
To better capture the error the getter is made throwable.

How Has This Been Tested?

Tested the example app and updated the Unit tests.

Screenshots (if appropriate):
N/A
Types of changes

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

Checklist:

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

Thanks @albertodebortoli. I have addressed your comments.

Thanks for the suggestions @dchakarov 👍


Allow address to be overridden on cli

Summary by Sourcery
Add a CLI option to allow overriding the default address and port for the webhook server, with validation for the address format.
New Features:

Introduce a CLI option to override the default address and port for the webhook server.

Enhancements:

Add validation for the address format, including host and port, before starting the webhook server.

The Go version in go.mod (1.23.0) appears to be incorrect as this version does not exist. Please update to a current stable version like 1.21.x

The current latest go version is 1.23.4


JuliaRegistrator

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/278
After the above pull request is merged, it is recommended that you create
a tag on this repository for the registered package version:
git tag -a v0.1.0 -m "<description of version>" 58392efe669adfd7be65183cac6e325e76c24118
git push origin v0.1.0


Options to abbreviate type names

Rather than this: Class <subcommand>(<command>:command-line-parser:command-line-parser)
I would prefer this: Class <subcommand>(<command>)
In fact, I'm not sure what the current rules are. For example why does it not display Class <subcommand>:command-line-parser:command-line-parser(<command>:command-line-parser:command-line-parser)? I didn't import <subcommand> and <command> into the current module differently.
What I'm sure of is that frequently the :module:library part just makes it harder to read long parameter lists. As long as textDocument/declaration (i.e. M-.) works I can find the module and library if I need to.

At a minimum, we could remove :dylan:dylan from names exported from the dylan:dylan module.  <boolean>:dylan:dylan becomes just <boolean>.
Also, perhaps some standardized abbreviation for when the module name and library name are the same. For example, :command-line-parser:command-line-parser could be :command-line-parser. That is, if there is no library part then it's the same as the module part.
These could be additional command-line options that are configurable in the client.


Disable default Keyboard Events

As the title suggests,
Is there a way to completely disable keyboard based events from dnd kit?
I really do not require ARIA based features for my particular desktop application and I have other key bindings attached to space bar.
Currently space will make my list of elements scroll down and enter will set a highlight around my element. I'd prefer to completely disable the keyboard interactions on dnd kit.
I'm trying to disable the defaults by changing these values but still the elements will highlight on enter

const keyboardSensor = useSensor(KeyboardSensor, {
    keyboardCodes: {
        start: [],
        cancel: [],
        end: []
    },
});

Many thanks in advance

You can completely opt out of the KeyboardSensor yes.
import {PointerSensor, useSensor, useSensors} from '@dnd-kit/core';

const sensors = useSensors(
  useSensor(PointerSensor),
);

return (
  <DndContext sensors={sensors}>
    ...


Desktop Notifier

Make a desktop notifier that will notify for a specific task on our laptop/computer

I would like to work on this issue


Roll engine 4444c9cb6c9f..ccb5034d5436 (2 commits)

https://github.com/flutter/engine/compare/4444c9cb6c9f...ccb5034d5436
git log 4444c9cb6c9f1859c1807cff415cd716bb9612ff..ccb5034d5436097a1ba71be18876b9a9e2f6f635 --no-merges --oneline
ccb5034d5 Generate weak pointers only in the platform thread (flutter/engine#9431)
da29c0473 Roll src/third_party/skia 6ce447a4ec20..91032f8cd7ea (5 commits) (flutter/engine#9455)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (amirha@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/6169247332433920

Commit queue failed; closing this roll.


Added command for installing libtool

When running ./autogen.sh the following error will be output: configure.ac:70: error: possibly undefined macro: AC_PROG_LIBTOOL unless libtool is already installed. To relieve the users from the troubleshooting it should be added to the walkthrough.

@felfa01 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@steveesp are we good to merge?

#sign-off


feature/dataframe

This PR:

adds Dataframe code (lifted from g3t)

Note: Still a work in progress - currently only a specimen data frame is created

$ fq dataframe --help
Usage: fq dataframe [OPTIONS]

  Create dataframes

Options:
  --db-path TEXT      path to sqlite db
  --output-path TEXT  path output tsv
  --help              Show this message and exit.

Remaining tasks:
[ ] modify GraphDefinition to retrieve ResearchStudy and ResearchSubject
[ ] add additional dataframes
[ ] create  pypi release
[ ] other ?


fix CameraRoll.saveToCameraRollOverrideType

Prompted by #459

I thing we should totally avoid this empty string.

I am not sure about that. Avoiding the empty string can cause other mistakes when you copy a binding and forget to change the string. And it means more work...

The opposite is valid: you can copy a binding & forget to update the non visible implicit string.
I thing it far more easy to forget about the empty string.
I think Chenglou & other got a discussion about this and stopped to recommend empty string

In our bindings, in >99%, the "" is correct because we use the same name on the Reason side as on the JS side.
The remaining cases are those where we provide two or more bindings for the same function, like xxx and xxxWithConfig. In those cases you need to pay close attention on implementation and review. For the remaining 99.9% it is overkill to specify the function name again IMHO.
And by using the naming convention xxxWithYyy, it is easy to search for these cases in the code. I used this convention in all cases in my code. I think saveToCameraRollOverrideType is the only case where we don't have it, maybe we should change it to saveToCameraRollWithType.

Fair enough.
saveToCameraRollWithType make sense then?

Fair enough.
saveToCameraRollWithType make sense then.


One Server, Multiple Clients

What's attracting me to GraphQL for a suite of Meteor apps is the one GraphQL server with multiple clients, as per this diagram on learngraphql.com

I'm not sure how this is meant to be achieved with Meteor though. Do I have a GraphQL server and have to expose HTTP end points manually? Do I connect via DDP? What's the suggested solution here.
Presumably the GraphQL Server in this case wouldn't even need to be a Meteor application, I could just have Meteor clients that are making calls to it, right?

Really want to progress based on this diagram from learngraphql.com. Any thoughts on how the different client apps would authenticate against the GraphQL Server?


Update release.yaml

This fixes the bug introduced in https://github.com/cognitedata/function-action-oidc/pull/50 (GITHUB_OUTPUT lacks an $env: in front (as this is PowerShell)


Doc: DKIT: add control profiling online guide (#12389)

DKIT add control profiling online guide

which was cherry-picked from master.