/var/lib/containers conflicts with system containers

system containers install under /var/lib/containers/atomic and they need to use the same file system as /ostree/repo so that hard links from the OSTree repository can be used.
Now container-storage-setup is using by default /var/lib/containers for creating the storage and that breaks the assumption behind system containers for sharing the storage with OSTree.
Should the default be /var/lib/containers/storage?
@rhatdan

Where do we use /var/lib/containers? Are you referring to man page where we give an example where a logical volume can be mounted? That's just an example and we can change it.
We will create /var/lib/container-storage-setup though to store metadata of our own.

I was referring to the man page and to the container-storage-setup.conffile:
# CONTAINER_ROOT_LV_MOUNT_PATH="/var/lib/containers"

uncommenting this line will probably be the first choice when someone wants to use it

@giuseppe we are not creating storage under /var/lib/containers. Assumption is that /var/lib/containers is the directory where container runtime will store its metadata. So we can mount something on /var/lib/containers so that metadata is not stored on rootfs but goes to separate disk.
So /var/lib/containers/storage will not fit.
May be we can put /var/lilb/container-runtime/ for now as an example.
@rhatdan WDYT?

Yes this man page was referring more to where ocid is storing it storage.  Container-storage-setup is just taking input from the packagers about where it should create storage.  We can change the example to be /var/lib/containers/storage if you want @giuseppe .

yes thanks.  If it can help, I can prepare a PR

SGTM


v8go: Memory leak on long lived contexts

Environment

Deploy method: jitsucom/jitsu docker
Jitsu version:  1.39.3

Description
Memory grows very fast and is never freed, possibly related to v8go.
Steps to reproduce
Runs the server processing a high volume of events
Expected behavior
Free the memory
Actual behavior
Memory is never freed

I'm sorry that this is happening. Sometimes, Jitsu requires fine-tuning under heavy load to avoid growing memory consumption. Would you mind sharing:

Your Jitsu config
Events volume
What is your destination and source?

And what led you to believe that issue is connected with v8?
Also, if you want we can help you to set up advanced telemetry so we would be able to see you anonymized stats. If you're interested, please join our Slack and ping me directly

@ezradiniz Currently, the main memory consumer on setups with high volume of events – is inmemory queue for Retroactive User Recognition.
We are currently working on redesign of user recognition in https://github.com/jitsucom/jitsu/issues/805
that will stop relying on inmemory queue - that will help to prevent memory leak on high  volume of events.
Advanced telemetry that @vklimontovich  mentioned may help to get certain of what is causing memory leaks.

Sorry for the late response.
I don't have the production configuration at the moment, but below is the Jitsu config that I used for local testing.

NOTE: In the real configuration, we have multiple Redshift destinations (some are in mode batch), also we are using USER_RECOGNITION_ENABLED=false

Jitsu Configuration (eventnative.yaml)
server:
  name: test
api_keys:
  - id: my-api-id-1
    client_secret: my-client-secret
    server_secret: my-server-secret
    origins:
      - '*'
      - 'my-origin'
destinations:
  redshift:
    only_tokens:
      - my-api-id-1
    type: redshift
    mode: stream
    data_layout:
      transform: |-
        function getProperties() {
            return {
                url: $.url,
                ip: $.source_ip,
            };
        }

        const common = {
            JITSU_TABLE_NAME: $.event_type,
            ...getProperties(),
        };

        const events = [
            "my_event",
        ];

        if (events.includes($.event_type)) {
            return {
                ...common,
            };
        } else {
            return null;
        }
      table_name_template: test
      primary_key_fields:
        - eventn_ctx_event_id
    config:
      db: my-db
      host: my-redshift-host
      password: my-password
      username: my-username
      port: 5439
      schema: my-schema
meta:
  storage:
    redis:
      host: my-redis-host
      port: 6379

Steps to reproduce:

Start the Jitsu server with the config (yaml) above
Send a bunch of events to the Jitsu server, for example:

#!/bin/bash

set -ex

while true
do
  # This event will be skipped
  curl -XPOST --data '{"event_type": "bad_event", "url": "https://fake-url.com"}' 'http://localhost:8001/api.gr9ju?p_t8bpu=my-client-secret'
  sleep 0.05
done

Wait a while and watch the memory grow

Now repeat the same test, but with this change:
// ...
// DO NOT process the event
// res, err := ProcessEvent(function, event)
// close(processDone)
// if err != nil {
// 	vte.results <- err
// } else {
// 	vte.results <- res
// }
close(processDone)
vte.results <- nil
// ...

(Reference: https://github.com/jitsucom/jitsu/blob/master/server/templates/template_executor.go#L229)
The memory will be fine and will not grow forever.
When I run the same test in Jitsu v1.38.6 there is no memory problem.
That's why I assume the problem is related to v8go
This week I will try to post more info about it.

Hi @ezradiniz
thank you for investigation!
let me suggest a few corrections:
close(processDone)
vte.results <- nil

this line make jitsu skip event processing. This will drop memory consumption anyway.
can you please try to repeat the test with following lines:
close(processDone)
vte.results <- event

Also in our testing we have noticed that processing of big batches makes jitsu allocate up to 2Gb or RAM.
And it doesnt get back to 200mb after batch is fully processed. But sending another batch doesn't increase memory consumption further. What numbers do you observe on your setup?

Hi @absorbb

can you please try to repeat the test with following lines:

I'll try to repeat the suggested test this week and post the results here.

Hi @ezradiniz !
Thank you for the log. it is very helpful.
we start working on the fix. i hope it will be ready by Tuesday


running error

i get this error when i try to run memex-desktop 0.0.7. i am using win 11. i uninstalled all previous versions and tried clean install but did not help.

Thanks for the note, we're investigating!

we just released v0.3.10 yesterday but I think there was an issue publishing to NPM where all other platforms got published to NPM but windows did not (https://www.npmjs.com/package/vectordb?activeTab=dependencies). We're going to get that up there today. Thanks for the bug report and patience!

Thanks @changhiskhan for chiming in!
@hodanli: Check out the latest 0.0.9 release of Memex Desktop and see if it works now - updated the affected package of Lance

unfortunately. i get same error. using: Memex-Desktop-v0.0.9-win-x64

curious what electron is actually looking for. Here's the actual artifact on npmjs: https://www.npmjs.com/package/@lancedb/vectordb-win32-x64-msvc, you can see that the latest version is up there.

0.0.23
this works without issue.

Thanks for the feedback!


Compile does not throw exception when unknown keyword is used in schema definition that is not referred to in the schema

What version of Ajv are you using? Does the issue happen if you use the latest version?
Latest 8.0.5
Ajv options object
Default
JSON Schema
Snippet embedded in test suite below
Sample data
No data just schema validation.
Your code
Code on RunKit with contents of 3rd test case that should raise error. Entire test suite is below. All three tests should succeed. The first two succeed but the third fails.
// note draft 2019
const Ajv = require('ajv/dist/2019');

describe('Ajv behavior tests', () => {
  it('Schema with definitions should compile', () => {
    const ajv = newAjv();
    const schema = {
      $id: 'https://example.com/test/schema/1.0.0/',
      $schema: 'https://json-schema.org/draft/2019-09/schema#',
      type: 'object',
      definitions: {
        A: {
          type: 'integer',
        },
      },
    }

    expect(ajv.compile(schema)).not.toBeUndefined();
  });

  it('Schema with definitions but unknown "foo" keyword should fail', () => {
    const ajv = new Ajv();
    const schema = {
      $id: 'https://example.com/test/schema/1.0.0/',
      $schema: 'https://json-schema.org/draft/2019-09/schema#',
      type: 'object',
      foo: 'bar',
      definitions: {
        A: {
          type: 'integer',
        },
      },
    }

    expect(() => {
      ajv.compile(schema);
    }).toThrow(new Error('strict mode: unknown keyword: "foo"'));
  });

  it('Schema with definitions but unknown "foo" keyword in definition A should fail', () => {
    const ajv = new Ajv();
    const schema = {
      $id: 'https://example.com/test/schema/1.0.0/',
      $schema: 'https://json-schema.org/draft/2019-09/schema#',
      type: 'object',
      definitions: {
        A: {
          type: 'integer',
          foo: 'bar',
        },
      },
    }

    expect(() => {
      ajv.compile(schema);
    }).toThrow(new Error('strict mode: unknown keyword: "foo"'));
  });
});

What results did you expect?
I expected the 3rd test case to fail saying that it did not recognize keyword foo.

The error is not raised because this definition is not used. It will be raised when the definition is used by this or some other schema. It is not practical to pre-compile all definitions, but if you need it to validate the schema definitions that might be used elsewhere you can do it with a loop:
ajv.addSchema(schema)
for (ref in schema.definitions) ajv.getSchema(`{schema.$id}#/definitions/${ref}`)

It will throw if any of the schemas in the definitions does not compile.
Doing it as part of compile could be wrong for some use cases.

Thanks for the quick feedback. I understand the impracticality of compiling all schema definitions, especially those without predecessors. This could result in some odd situations for Ajv users where invalid schema are presumed to be valid. A configuration parameter to toggle compiling all would have been wonderful here. It would serve the schema only validation use cases well.

for (ref in schema.definitions) ajv.getSchema(`{schema.$id}#/definitions/${ref}`)

Unfortunately, ^^^^ is not raising the error either. Is there a way to explicitly trigger validation?

for (ref in schema.definitions) ajv.getSchema(`{schema.$id}#/definitions/${ref}`)

Unfortunately, ^^^^ is not raising the error either. Is there a way to explicitly trigger validation?

Doh, I found out why this did not work as-is. There was no dollar sign in front of ${schema.$id}#/def...


validateProps

Closes #24. This is an integration of PropTypes in Roact.
TODO:

[x] Gate type checks behind a configuration flag
[x] Document this

This depends on #93.

Coverage increased (+0.4%) to 90.171% when pulling 61614d4d7dab81029a4e5d582824363aac2de805 on AmaranthineCodices:propTypes into 93358cabd77f780e1305553b571d40d56e7c93e3 on Roblox:master.

I think the prop types implementation, no matter which one we use, should just have a functional interface to maximize the ability to support multiple implementations and custom validators. We probably don't need an apply or validate method -- just functions would work well!
I imagine the signature for a prop validator function would be (props, propKey, component) -> success, result.
Usage would still be:
Component.propTypes = {
    foo = PropTypes.number, -- or any function of the form above
}

Some further notes about this:

Hard dependency on #93 in its current form, where the stack trace method is named rootElementTraceback.
Another stack trace is collected when the component is created; this is used to provide information about what component to look at when propTypes is not a function.
Compatible with any validation function that has the signature value -> success, reason.
Default traceback message has been moved from the reconciler to Core under the name _defaultSource. It may need a better name; I'm not sure!

This should be ready to go now!

An interesting thing that popped into my mind: does gating this behind a global flag make unit testing harder?
Naturally, you want to run all tests with typechecking enabled and also develop with typechecking enabled, but how do you best represent that with the config system only (rightfully?) letting you set configuration once?

I'm not entirely sure it blocks unit testing, to be honest, though that may be an artifact of how I run my unit tests. My tests don't run at the same time as the main entry point (they have their own entry point), so the configuration can be set once per entry point without issue.

Alright! I've gotten this back to mergeable as of now; I think this should be pretty easy to port to the new reconciler!

Closing in favor of #174 targeted at the new reconciler.


Update according to IsoBibItem.ics cardinality change

IsoBibItem.ics is Ics[1..*] instead of Ics[1]
https://github.com/riboseinc/isodoc-models/commit/639b52cf11a73771739d74fe4ac2a14b224d4e76
Related to https://github.com/riboseinc/isodoc-models/issues/33

asciidoctor-iso currently makes no use of ics, so there is nothing to change.
asciidoctor-gb does; I will note that they should be comma delimited in the readme.

@opoudjis interesting. ISO does assign ICS codes (often multiple) to every standard, but it seems that they’re not reflected in the content. However due to the fact they’re assigned, they are represented in the IsoBib model. Maybe we should also support it in this gem to make a normal IsoBib identical to the metadata IsoBib in an IsoDoc.

Will add to the gem, the same way it is added to asciidoctor-gb, but it is only going to show up in the XML.

Sounds great. Thanks!


merge leftovers from clean-up

All my fix and tests were actually never in the origin pull request of clean-up... I think I pushed to origin instead of upstream 🤦.
There it is then. This implements

adding some test for integrals.jl
fixing type stability for _pforce and _vforce
add check to avoid integrating forces on bodies that are not 2D curves.

This is ready, all the test run as well.


fork that doest not have meaningful keyword

Maybe don't show this kind of fork. Or presenting the commit message.

I think it's okay that we couldn't extract keywords on this kind of fork.


GEOPY-678

**GEOPY-678 - **

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@cdc6f2c). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop       #8   +/-   ##
==========================================
  Coverage           ?   84.45%
==========================================
  Files              ?        3
  Lines              ?      148
  Branches           ?       22
==========================================
  Hits               ?      125
  Misses             ?       17
  Partials           ?        6

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


Selecting text after an image selects the image too

Selecting text using the keyboard (eg using the Shift key + arrow keys, or Shift + End) will select an image that precedes the text, if the initial caret position is at the start of a text line. This happens even if there are multiple linebreaks between the image and the line of text.
So let's say you add an image, a few linebreaks, then some text, and you later decide to remove that line of text, so you place the caret at the beginning of the line and press Shift+End. The selection expands backwards before the caret, and selects the image too, so when you press Del the image gets deleted.
This happens in both Firefox and Chrome.

I will not fix that in v2.
Since a new v3 is on the road, I hope this will not be buggy in it.


Once datadog agent crashed and redeployed the metrics are not shown in Datadog dashboard

I'm using the java dogstatsd client for sending metrics. The issue started when the Datadog agent crashed. Once it's redeployed again the metrics are stopped to be shown in the Datadog dashboard. I see my log trace after calling by statsd client but the metrics are not shown in the datadog dashboard. Is there a reconnection issue between java dogstatsd client and datadog agent preventing the metric publishing?
The Java application uses: Gradle 6.3 springboot': '2.3.0.RELEASE java-dogstatsd-client 3.0.0

Hi @dmitriymermelstein,
Sorry to hear you are missing metrics. It does seem like a problem with reconnection. Could you please provide some details about your environment:

What operating system are you using?
Are you using UDP or unix sockets (UDS) to communicate with the agent?
Is the agent or application deployed in the container? If yes, what kind of containers?
Do you use an error handler with the statsd client? If yes, did you get any errors?
Can you see other metrics, such as host metrics, after the agent restart in the app?

I don't know if this applies to your situation, but one possible reason for this issue is when using containers and the dogstatsd unix socket volume uses full path to the file. Due to how container volumes work, client does not see the new socket after the agent restart, and  can not communicate with the new agent process. The solution in this case is to use a directory volume instead of a full path (e.g. path:/var/run/datadog instead of path:/var/run/datadog/dsd.socket).

Hi @vickenty,
Thank you for your response.
Please see below the answers to your questions.

OS - Linux.
Used UDP by default.
Amazon EKS.
Yes, should be printed an error. The errors are not printed.
No, I also do not see any datadog.dogstatsd.client.* metrics once the agent crashed and redeployed.

How can I initially make sure that metrics are sent by dogstatsd client?
This my code:
StatsDClient statsd = new NonBlockingStatsDClientBuilder()                 .prefix("statsd")                 .hostname(statsdHost)                 .port(8125)                 .errorHandler(e -> System.out.println("StatsDClient error: " + e.getMessage()))                 .build(); statsd.count("app.datadog.metrics.test", 1);

Thank you for providing this extra information.
Just in case, what is the value of statsdHost in your code? If the agent pod is restarted, it's IP address will change, so the traffic may end up going to the wrong destination (although it shouldn't matter if the dogstatsd traffic goes via the host port).
But the fact that you don't see not only statsd metrics, but also host metrics after the agent restarts, suggests that the issue may be in the connection between the agent and Datadog. If it is the case, please check out our agent troubleshooting page, and feel free to contact our support if you require further assistance.

I'm closing this issue for now, feel free to re-open if you have further questions!


Non-allocating fast path for compiling code

I'm showing you this sketch to give you a change to influence my solution to ensure that it is upstreamable. The goal is to get rid of as many managed allocations as possible when compiling and running JavaScript code. My particular use case involves retrieving a native char array from Unity (the game engine) and piping it straight into ClearScript.
Being able to call ClearScript from bursted C# (no reference types allowed) is a stretch goal.

Hi @AnsisMalins,
Thanks for posting your PR. If we understand it correctly, your change avoids the allocation of a managed string to hold the code to be compiled.
If that's correct, do you have an estimate of the performance impact of this change relative to the cost of script compilation? It's difficult to believe that it would be significant.
BTW, we're currently working on ClearScript 7.5, which we're hoping will greatly reduce managed heap traffic by eliminating heap-allocated lambdas and scopes, among other things.

Being able to call ClearScript from bursted C# (no reference types allowed) is a stretch goal.

ClearScript includes many public classes that the client must instantiate. Isn't that a problem for burst-compiled C#? Or were you thinking of bypassing the managed layers completely and interfacing directly with ClearScript's native assemblies?
Thanks!

Oops, I was digging in the wrong direction. The bigger problem is ScriptObject.Invoke. I'll do more research and come back to your with a better change.

How far out is ClearScript 7.5? If more than a month, can I get at it early?

Hi @AnsisMalins,

How far out is ClearScript 7.5?

In addition to extensive changes intended to reduce managed heap pressure, ClearScript 7.5 will include a major new API for exposing host objects to V8 for fast access. Early results indicate a roughly 70% reduction in method call overhead and allocation rate for objects exposed via the new API. Unfortunately, we're still at least a couple of months away.

If more than a month, can I get at it early?

Once most of the changes are in place and the new API is feature-complete, we could post the pre-release code in a branch, if that would help. We just aren't quite there at the moment.
Thanks!

Hi @m3taphysics,

It would help massively to reduce the overhead we face if you could share the pre-release code for evaluation as soon as it is available!

Sure, we'll post the code changes to a branch as soon as they're in a release candidate state. If there's sufficient interest, we'll put up pre-release NuGet packages as well.
However, please understand that the full benefits will apply only to host resources exposed via a new API. For better or worse, the most straightforward and "natural" .NET code does a lot of allocation, so the new API goes pretty far off the beaten track.
Nevertheless, existing ClearScript hosts should see a modest uplift from the reduction or elimination of things like heap-allocated scopes, capturing lambdas, and parameter arrays.
Cheers!


puppet/catalog-diff: use proper subquery verb

It became select_resources on newer versions. Reuse the same logic as
with the other string.

@raphink unfortunately but I changed my workplace so I'm not sure if I will ever be able to finish it :( I will probably be only able to come around this again in the range of months so if anyone wants then feel free to pick this up!


Seemingly incorrect "FormatException" when using stream.transform(json.decoder)

Hello!
Consider:
import 'dart:convert';

void main() {
  Stream.fromIterable([
    '{"type":"list_lints","lints":["my-lint-rule: This is a custom lint rule."]}',
    '{"type":"lint_done"}',
  ]).transform(json.decoder).listen((event) {
    print(event);
  });
}

This throws with:
FormatException: Unexpected character (at character 1)
{"type":"lint_done"}
^

#0      _ChunkedJsonParser.fail (dart:convert-patch/convert_patch.dart:1465:5)
#1      _ChunkedJsonParser.parse (dart:convert-patch/convert_patch.dart:866:48)
#2      _JsonStringDecoderSink.addSlice (dart:convert-patch/convert_patch.dart:1551:13)
#3      _JsonStringDecoderSink.add (dart:convert-patch/convert_patch.dart:1556:5)
#4      _ConverterStreamEventSink.add (dart:convert/chunked_conversion.dart:69:18)
#5      _SinkTransformerStreamSubscription._handleData (dart:async/stream_transformers.dart:111:24)
#6      _RootZone.runUnaryGuarded (dart:async/zone.dart:1594:10)
#7      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:365:11)
#8      _BufferingStreamSubscription._add (dart:async/stream_impl.dart:297:7)
#9      _MultiStreamController.addSync (dart:async/stream_impl.dart:1127:36)
#10     new Stream.fromIterable.<anonymous closure>.next (dart:async/stream.dart:375:26)
#11     _microtaskLoop (dart:async/schedule_microtask.dart:40:21)
#12     _startMicrotaskLoop (dart:async/schedule_microtask.dart:49:5)
#13     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#14     _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:185:5)

This seems unexpected. As changing .transform(json.decoder) to .map(jsonDecode) removes the exception.
Dart:
Dart SDK version: 3.5.0-323.0.dev (dev) (Tue Jul 2 06:25:57 2024 -0700) on "macos_arm64"

json.decoder's doc mentions:

It may be stateful and should not be reused.

I assume that's the gist of the issue?
But then, why does json.decoder implement StreamTransformer if we shouldn't use it for stream.transform?
I'm probably missing something :)

The JSON decode can only decode one JSON value, because the concatenation of two JSON values is not valid JSON. (It might be valid something else, like "JSON-lines", but that's not what is decoded.)
You can use a streaming JSON decoder to decode a single large JSON input that arrives in multiple packages, without having to accumulate and concatenate all the strings before decoding.
If your input is a Stream<Uint8List>, like a network connection, then something like var result = await input.transform(utf8.decoder).transform(json.decoder).first; should work.


Fix regression introduced by #688

Remember to filter out deleted metadata a.k.a tombstones.
Fix for #703
Didn't add a changelog item as it's a bugfix to a change which wasn't released yet.

https://media.giphy.com/media/jSB2l4zJ82Rvq/giphy.gif


Update Log Config (Days)

Allow days to keep logs to be defined via env.php, keep 3 as the default.

Hi, there are an example to edit log days in env.php? I have the previous env.php and can't figure how to edit it.
Or continue to do so in logging.php
Thanks!

Hi, there are an example to edit log days in env.php? I have the previous env.php and can't figure how to edit it. Or continue to do so in logging.php Thanks!

It is nothing different from other env.php items and displayed/highlighted in changed files section of this pr.
Use LOG_DAYS in your env.php


The use of &T{} for composite literals is strange, and in particular inconsistent with the way you write a type. Consider the following code:
foo := struct{
  m map[int]string
  a [10]int
  sl []string
  st struct{}
  p *struct{}
}{
  m: map[int]string{},
  a: [10]int{},
  sl: []string{},
  st: struct{},
  p: &struct{},
}

p is the only member for which the type declared does not match the type used in the composite literal. The logical fix is to make a composite literal of type *T mean a composite literal to a pointer to a new value of type T initialized from the literal, in place of &T.
This is probably incompatible with #20446.

foo := struct{
        p *int
}{
        p: &i,
}

&T have two meaning.

create value T from literal
and get address of the value

If want to create new int value and get pointer of the value, we can't do it without new builtin. (ex: new(int)) If you mean a := *int or a := *int{} should have meaning to get pointer of new value, compiler will have to look backward literals like C. Then compiler will be slow.

But then what's the dereference operator? I think the most logical thing would be to use & in the type signature of pointers, so that if you have a var i int then &i's type is &int and the type signature and literal expression also match. And there is a historical precedent (in C++) for &T being a pointer to T.
But I don't think it's worth it.

If you are to replace &T{} with *T{}, I would be confused. I am used to seeing the pointer/deref (*) and the address of (&) operators. I think Go's usage of them has been consistent with other C-family programming languages. If you are to replace those, you will create unnecessary confusion to many people, considering the popularity of C/C++ and the people's familiarity with them.
I don't agree with this proposal.

Note that in your example, the lines
  st: struct{},
  p: &struct{},

must today be written as
  st: struct{}{},
  p: &struct{}{},

(two pairs of braces each).
I suspect that, to avoid a parsing ambiguity, the latter case would need to be written:
  p: (*struct{}){},

but today that syntax is rejected by the compiler (https://play.golang.org/p/6GxjWrCfgN).

Also note that under https://github.com/golang/go/issues/12854 they could be written even more uniformly:
foo := struct{
  m map[int]string
  a [10]int
  sl []string
  st struct{}
  p *struct{}
}{
  m:  {},
  a:  {},
  sl: {},
  st: {},
  p:  {},
}

This idea was discussed during the development of Go 1 (that is, as a change from the earlier versions of Go to the standarized Go 1), but was rejected at that time.

It is important to understand that & and * aren't interchangeable tokens, but rather opposite and complementary. & means "take the address of value", and * means "get the value at address". From this, it makes no logical sense to say *T{}, because you would wind up reading this as "take the value at the address T{}", and then you realize T{} isn't an address and so the statement makes no sense.
Perhaps memorizing this would help you stop confusing &T{} and *T{} as much (as you said in the proposal for #20446).
This also explains @cznic's comment: what if you want to point to a variable that already exists in a composite literal? (The line that is missing from their comment is var i int.)

@andlabs The * operator is distinct from the * type-constructor. The & operator is dual to the * operator, but the * type-constructor has no dual.
&T{} is the expression "address of a literal of type T", but *T is the type "pointer to T". The * operator is not involved in this proposal in any visible way.

@bcmills Yes, but is &T{} itself not an expression? Given what you said, another way to say what I was trying to say is that this proposal conflates the * type-constructor with the * operator, if that makes more sense.

The way I read this proposal, it is that &T{} should be read as "the address of a literal of type T", and *T{} should be read as "a literal of type *T". (The latter is currently rejected as a syntax error.)

Ah, yes, that is different. If we do expand composite literals to all types (a more general version of your reading of the proposal), by current rules it would have to be (*T){...}.
Isn't the current situation due to ambiguity between the binary * operator? A potential solution would be to drop * for indirection, but that seems to be beyond the scope of the current proposal phase for Go 2?

There doesn't appear to be strong support for this, and you say yourself it would be incompatible with #20446 / #9097.  It was also discussed years ago and rejected.  Closing.


⚠ AlphaRatio has degraded performance

In 26a5be5, AlphaRatio ($AR) experienced degraded performance:

HTTP code: 200
Response time: 8859 ms

Resolved: AlphaRatio performance has improved in 29ae29e.


Improve Fastlane app description

I copied description from io.naox.InnerBreeze.metainfo.xml and added the instruction to make it easier to understand for users.
Added new locale, please see #43 to make it easier to fix typos in translation.
Thank you for the app!

merged the language changes, will be available for v 1.3.2


Microsoft Authenticator import wrong

The TOTP for the Microsoft Account in the Microsoft Authenticator app is 8 numbers long. The code the tool generates tells andOTP to only generate 6 tokens which is not accepted at Microsofts end.
Simply changing to 8 digits does not work either.

I just tried scanning an 8 digit code into Microsoft Authenticator and the digits portion of the otpauth:// URI seems to be ignored. There doesn't seem to be a way to manually add a code and specify the number of digits either.
You can dump your database by running this script? You should be able to see what the format is:
import pprint
import sqlite3

from android_otp_extractor.adb import guess_adb_interface
from android_otp_extractor.contrib import open_remote_sqlite_database

adb = guess_adb_interface('/data/data/')

with open_remote_sqlite_database(adb, '/data/data/com.azure.authenticator/databases/PhoneFactor') as connection:
    connection.row_factory = sqlite3.Row

    cursor = connection.cursor()
    cursor.execute('SELECT * FROM accounts;')

    for result in cursor.fetchall():
        pprint.pprint(dict(result))
        print()

For comparison, here is what I see with a non-Microsoft account:
{'_id': 1,
 'aad_authority': '',
 'aad_ngc_totp_enabled': 0,
 'aad_permission': 0,
 'aad_tenant_id': '',
 'aad_user_id': '',
 'account_capability': 1,
 'account_type': 0,
 'cached_pin': '',
 'cid': '',
 'encrypted_oath_secret_key': '',
 'group_key': '00000000000000000000000000000000',
 'identity_provider': '',
 'is_totp_code_shown': 1,
 'mfa_pin_encryption_key_alias': '',
 'name': 'uihiuh',
 'ngc_ski': '',
 'oath_enabled': 0,
 'oath_secret_key': '6tfrwfx2ignyo74d4tnsqbtdxbbgjekn4leewvojw4gnfivaoakqx4kr',
 'paws_url': '',
 'restore_capability': 0,
 'update_flags': 0,
 'username': 'uihiuh',
 'ux_position': 0}

I get the following (replaced secret values):
{'_id': 1,
 'aad_authority': '',
 'aad_ngc_totp_enabled': 0,
 'aad_permission': 0,
 'aad_tenant_id': '',
 'aad_user_id': '',
 'account_capability': 7,
 'account_type': 1,
 'cached_pin': '',
 'cid': '1234567890abcdef',
 'encrypted_oath_secret_key': 'xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx=',
 'group_key': '00000000000000000000000000000000',
 'identity_provider': '',
 'is_totp_code_shown': 1,
 'mfa_pin_encryption_key_alias': '',
 'name': 'Microsoft',
 'ngc_ski': 'jk6_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx-48',
 'oath_enabled': 1,
 'oath_secret_key': 'xxxxxxxxxxxxxx==',
 'paws_url': '',
 'restore_capability': 0,
 'username'<EMAIL_ADDRESS> 'ux_position': 0}

I don't handle encrypted_oath_secret_key at all. I'll check out how the app uses it.

Thanks for looking into that. If you need some more information, testing or anything else I could help with feel free to ping me.

Authenticator uses the Android KeyStore which means it's not possible to decrypt the secret easily.
However, with root access it should be possible to just pretend to be the app and interface with the KeyStore to decrypt the data (since we're not actually interested in the cryptographic material handled by the TPU, just the decrypted key, as intended).
I believe the only thing required is a simple KeyStore Java "proxy" that can be run with su as the appropriate app user. I don't have much experience with Java but I'll give it a go.

I've got a prototype working that can decrypt the andOTP database using this method but it requires setenforce 0 and does IPC in a strange way. Once I get the code cleaned up and tighten up security, android-otp-extractor should be able to export secrets from any app.

@SuperSandro2000 I may be over complicating this. Can you run that script again and take the oath_secret_key and fix its encoding with this code?
>>> import base64
>>> base64.b32encode(base64.b64decode('8UzXgg9agj7MNq=='))
b'6FGNPAQPLKBD5TBW'

See if that secret works with 8 digits and a 30 second period. I can't get Microsoft Authenticator running on my primary phone but this works on a much older Android phone.

Yeah that works. :)


Lifecycle for beforeGet or beforeFetch

This could be cool to control when a model change some property so you can manage the behavior for example: mixin beforeGet and afterUpdate.

Sorry this have to be in waterline repo. Actually they have this in mind for 0.11v.
Best.


Jerryscript not build for NRF52840 target

I have tried to build the Jerryscript mbedOS for NRF52840 platform by using the command "gulp --target=NRF52840_DK". But i have have faced the "core_cm4.h" header file not visible to "nrf52840.h" file.
So i have copied the cmsis path header files into "./build/jerryscript/targets/mbedos5/mbed-os/targets/TARGET_NORDIC/TARGET_NRF5x/TARGET_SDK_14_2/device/" path and then the "core_cm4.h" file issue get resolved.
Now i have faced below error,
/repel/mbed-js-repl-example$ gulp --target=NRF52840_DK
[17:14:11] Using gulpfile ~/repel/mbed-js-repl-example/Gulpfile.js
[17:14:11] Starting 'make-build-dir'...
[17:14:11] Finished 'make-build-dir' after 135 μs
[17:14:11] Starting 'makefile'...
[17:14:11] Starting 'bundle'...
[17:14:11] Starting 'ignorefile'...
[17:14:11] Finished 'ignorefile' after 25 ms
[17:14:11] Finished 'makefile' after 44 ms
[17:14:11] Starting 'get-jerryscript'...
[17:14:11] Finished 'get-jerryscript' after 116 μs
[17:14:11] Starting 'getlibs'...
[17:14:11] Finished 'getlibs' after 12 ms
$ make getlibs
make: Nothing to be done for 'getlibs'.
[17:14:11] Finished 'bundle' after 99 ms
[17:14:11] Starting 'cppify'...
[17:14:11] Finished 'cppify' after 25 ms
[17:14:11] Starting 'build'...
[17:14:12] Found native packages: mbed-js-repl
$ make BOARD=NRF52840_DK EXTRAS=/home/pc/repel/mbed-js-repl-example/node_modules/mbed-js-repl/.
[17:14:13] build/make
[17:14:13] Finished 'build' after 1.72 s
[17:14:13] Starting 'default'...
[17:14:13] Finished 'default' after 29 μs
cd jerryscript/targets/mbedos5 && pip install -r tools/requirements.txt && rm -f source/js_encoded.cpp && rm -f source/pins.cpp && rm -f source/main.cpp && make BOARD=NRF52840_DK EXTRA_SRC="/home/pc/repel/mbed-js-repl-example/node_modules/mbed-js-repl/.:../../../source" EXTERN_BUILD_DIR=../../../out/NRF52840_DK NO_JS=1
DEPRECATION: Python 2.7 will reach the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 won't be maintained after that date. A future version of pip will drop support for Python 2.7.
Requirement already satisfied: pycparser<=2.17 in /usr/local/lib/python2.7/dist-packages (from -r tools/requirements.txt (line 1)) (2.17)
Requirement already satisfied: simpleeval in /usr/local/lib/python2.7/dist-packages (from -r tools/requirements.txt (line 2)) (0.9.8)
Requirement already satisfied: pycparserext in /usr/local/lib/python2.7/dist-packages (from -r tools/requirements.txt (line 3)) (2016.2)
Requirement already satisfied: ply>=3.4 in /usr/local/lib/python2.7/dist-packages (from pycparserext->-r tools/requirements.txt (line 3)) (3.11)
make[1]: Entering directory '/home/pc/repel/mbed-js-repl-example/build/jerryscript/targets/mbedos5'
python tools/generate_pins.py NRF52840_DK
Traceback (most recent call last):
File "tools/generate_pins.py", line 247, in
main()
File "tools/generate_pins.py", line 237, in main
pins = enumerate_pins(pins_file, ['./tools'] + list(includes), defines)
File "tools/generate_pins.py", line 154, in enumerate_pins
parser=GnuCParser())
File "/usr/local/lib/python2.7/dist-packages/pycparser/init.py", line 93, in parse_file
return parser.parse(text, filename)
File "/usr/local/lib/python2.7/dist-packages/pycparserext/ext_c_parser.py", line 64, in parse
return self.cparser.parse(text, lexer=self.clex, debug=debuglevel)
File "/usr/local/lib/python2.7/dist-packages/pycparser/ply/yacc.py", line 331, in parse
return self.parseopt_notrack(input, lexer, debug, tracking, tokenfunc)
File "/usr/local/lib/python2.7/dist-packages/pycparser/ply/yacc.py", line 1106, in parseopt_notrack
p.callable(pslice)
File "/usr/local/lib/python2.7/dist-packages/pycparser/c_parser.py", line 654, in p_decl_body
typedef_namespace=True)
File "/usr/local/lib/python2.7/dist-packages/pycparser/c_parser.py", line 394, in _build_declarations
self._parse_error('Invalid declaration', coord)
File "/usr/local/lib/python2.7/dist-packages/pycparser/plyparser.py", line 55, in _parse_error
raise ParseError("%s: %s" % (coord, msg))
pycparser.plyparser.ParseError: /home/pc/gcc-arm-none-eabi-6_2-2016q4/arm-none-eabi/include/sys/_stdint.h:48: Invalid declaration
Makefile:72: recipe for target 'source/pins.cpp' failed
make[1]: Leaving directory '/home/pc/repel/mbed-js-repl-example/build/jerryscript/targets/mbedos5'
make[1]: *** [source/pins.cpp] Error 1
Makefile:14: recipe for target 'all' failed
make: *** [all] Error 2
events.js:183
throw er; // Unhandled 'error' event
^
Error: Command make BOARD=NRF52840_DK EXTRAS=/home/pc/repel/mbed-js-repl-example/node_modules/mbed-js-repl/. exited with code 2
at ChildProcess.handleSubShellExit (/home/pc/repel/mbed-js-repl-example/node_modules/gulp-run/command.js:166:13)
at Object.onceWrapper (events.js:317:30)
at emitTwo (events.js:126:13)
at ChildProcess.emit (events.js:214:7)
at Process.ChildProcess._handle.onexit (internal/child_process.js:198:12)
i have attached "_stdint.h" for reference_stdint.h.zip
From the file i understood the __uint32_t is invalid. Could you help to resolve this issue.
Please note the same source successfully compiled for NRF52832 by using the "NRF52_DK" command.
Environment details:
Host OS: Ubuntu 16.04 LTS
Target : nRF52840
CC Version: arm-none-eabi-gcc (GNU Tools for ARM Embedded Processors) 6.2.1 20161205
Jerryscript Source Repository: https://github.com/ARMmbed/mbed-js-repl-example

Can anybody explain why this error come in _stdint.h file even it was defined in "#include <machine/_default_types.h>" header file?

Can anybody help to figure out this issue?

CC: @fbmrk @thegecko

I have done the jerryscript porting for NRF52840 chipset by modified the PinNames.h manually (i have Only configured port 0 pins in PinNames.h file. if add the port 1 pins in PinNames.h file then faced some compile time issues. So i have skipped the port 1 pins configuration in PinNames.h file) and did some patch work in definitions.json file.
After long struggle then the hex/bin file generated & it booted successfully in NRF52840 platform. By using the main.js file i can able control the LEDs. But i faced the issues in JerryScript console.
Once after the JerryScript console get opened then press any keys in keyboard (except the "Enter" key) then i faced below error.

Cc @janjongboom

@ARM93 The ISR issue is fixed in mbed-js-v2 like this: https://github.com/janjongboom/mbed-js-repl/commit/28b996d9c902223fbae4d0e67f2749c8d8117293 Should be able to land this cleanly on mbed-js-repl.
Unfortunately pin name generation still breaks on the NRF52840_DK target.

@ARM93 The ISR issue is fixed in mbed-js-v2 like this: janjongboom/mbed-js-repl@28b996d Should be able to land this cleanly on mbed-js-repl.
Unfortunately pin name generation still breaks on the NRF52840_DK target.

@janjongboom Thanks for your update. Now console is working fine. :-)
Now i start to working on port 1 adding into PinNames.h header file issue.

@ARM93 Pin generation is fixed now in https://github.com/ARMmbed/mbed-js-v2/commit/a41fce512ea8eb9692ef3374548db1ef4ffa8bb4. I'm inclined not to upstream this to this repository as we won't maintain the target here in the future, but rather support mbed-js-v2.

@janjongboom Thanks for your support.  This  ARMmbed/mbed-js-v2@a41fce5 update not resolved my issue. I just added the port 1 pins as port 0 pin in the PinName enum then my issue getting resolved.
Find the attached file for your reference PinNames.h.zip


[Other] A necessary step to activate Clean Up (Part of Apple Intelligence on MacOS 15.1 Beta 3)

Additional Information
Manual Execution of [Method 1]
When executing the commands of [eligibility_util], additionally execute [./eligibility_util forceDomainAnswer --domain-name OS_ELIGIBILITY_DOMAIN_STRONTIUM --answer 4].
It works when SIP is activated.

What is the behavior of "Strontium"? I didn't find any detailed information about it on the Internet. [https://theapplewiki.com/wiki/Eligibility](The Apple Wiki) site doesn't have anything about Strontium.

It is possibly the domain to unlock Clean Up as part of Apple Intelligence in the latest beta according to the OP.
And I'll upgrade my Mac and confirm it later.

What is the behavior of "Strontium"? I didn't find any detailed information about it on the Internet. The Apple Wiki site doesn't have anything about Strontium.

There is no such domain “STRONTIUM” in MacOS 15.0 beta 8.
It was possibly only added in the latest MacOS 15.1 Beta 3.

请问会有Interl版本的eligibility_util_sip 文件吗

请问会有Interl版本的eligibility_util_sip 文件吗

No plan yet.
You can follow the script to build it yourself on your intel Mac.
https://github.com/Kyle-Ye/eligibility/blob/2fe29d014b38a1c2b55556e41720d5091e5da440/.github/workflows/publish_release.yml#L21-L34


exit code 100

Bug description
failed to solve: executor failed running [/bin/sh -c apt-get install -y python3-pip]: exit code: 100
execute command "docker compose up -d"
Steps to reproduce
install docker
install dependences
execute the commands for the instructions
Environment Information
Docker version 20.10.17, build 100c70180f
Screenshots

Relevant log output
Ubuntu

Confirmations

[X] I'm running the latest version of the main branch.
[X] I checked existing issues to see if this has already been described.

@Hant7 Is still relevant?
Can you try with the new instructions:
docker run -d -v weights:/usr/src/app/weights -v datadb:/data/db/ -p 8008:8008 ghcr.io/nsarrazin/serge:latest


Enable null safety for package:flutter & sky_engine

Tracking issue for enabling null safety (https://github.com/dart-lang/language/issues/110) in package:flutter

Is sky_engine using non-nullable feature?
I've tried to run flutter pub run build_runner watch and I've got an error like flutter/bin/cache/pkg/sky_engine/lib/ui/channel_buffers.dart:152:17 - This requires the 'non-nullable' language feature to be enabled.
and due to https://github.com/flutter/flutter/issues/40490 I couldn't add this feature.
What should I do to run build_runner?

Is sky_engine using non-nullable feature?
I've tried to run flutter pub run build_runner watch and I've got an error like flutter/bin/cache/pkg/sky_engine/lib/ui/channel_buffers.dart:152:17 - This requires the 'non-nullable' language feature to be enabled.
and due to #40490 I couldn't add this feature.
What should I do to run build_runner?

It's been resolved by adding
dependency_overrides:
  analyzer: 0.39.14

in pubspec.yaml

@mit-mit Do we still need this item for TP2?

We need the framework, but it looks like that might be tracked in https://github.com/dart-lang/sdk/issues/40146 and not here. So perhaps this bug can be closed as a duplicate, @leafpetersen ?

@mit-mit  Please leave this issue open until the referred issue is closed. Most people won't be looking for information on this part of flutter in the dart repo, and in fact "Flutter framework null safety migration" sounds like it should be in the flutter repo altogether.
I do kind of wish issues relating to flutter would stop being put in dart repo.

@mit-mit @leafpetersen  I think we can close this, agree?

I think so; but note that we're now blocked on #59879

closing


Forgot password page:prompt  over page.both english and chinese version

In GitLab by @tmon546596046 on Jul 31, 2018, 16:14
fix issue #1612

In GitLab by @luotian-github on Nov 8, 2019, 07:57
please fix it in 1.5

In GitLab by @liyanq528 on Nov 8, 2019, 07:57
fixed by pull request #1619

In GitLab by @sokril on Nov 8, 2019, 08:26
closed

In GitLab by @sokril on Nov 8, 2019, 08:26
assigned to @guyingyan

In GitLab by @liyanq528 on Nov 12, 2019, 02:51
mentioned in merge request !1049

In GitLab by @liyanq528 on Nov 13, 2019, 05:02
mentioned in merge request !843

In GitLab by @liyanq528 on Nov 13, 2019, 09:18
mentioned in merge request !1292

In GitLab by @liyanq528 on Nov 13, 2019, 11:18
mentioned in merge request !988


Does this blaze optimization only support parquet files?

Does this blaze optimization only support parquet files? does it support text and orc files ？

related to #498


Lack of defaultTo - porting proposal

I couldn't find the defaultTo function present in the lodash JS
I could create a pull request for the same
The defafultTo function signature would be as follows:
_::defaultTo(object|associative array, string|array, defaultValue[, defaultToOnEmpty])

Thank you @punit-kulal, you are welcome to send a pull request. Let me know if you need any assistance

done


Add probot move config file

@steved0x : Thanks for your contribution! The author, @, has been notified to review your proposed change.


Boto3 conversion

Hi there:
This a new PR to convert aws-ec2-assign-elastic-ip to use boto3, instead of boto.
Because boto3 still doesn't have get_instance_metadata() equivalent, it adds ec2-metadata as a requirement.
The only functional change in this PR is that AllowReassociation is always False, whereas in the current boto implementation, whether it is True or False depends on the date that your EC2 environment was created.  See https://github.com/skymill/aws-ec2-assign-elastic-ip/issues/24 for more details.
I've tested the use cases that broke in https://github.com/skymill/aws-ec2-assign-elastic-ip/issues/28 or https://github.com/skymill/aws-ec2-assign-elastic-ip/issues/29 and they both work correctly in this branch.

Just a quick bump on this -- if you'd prefer to stick with boto2, given the problems from the last release, I understand, but we will want to continue work in our fork to add features.

@sebdah It appears that we also ran into reassociation bug. Because the default has changed, it's impossible to send valid request with old boto. In order to fix this, a client change to boto3 is required.
Would you be willing to look into this?

I'm really swamped with dealing with Covid-19 stuff at work, if someone could review the code and give this branch a go I'd be happy to merge, package and release.

Done.

Awesome, thanks guys. I'll get to this a little later today.

Thanks for the help here @pmkane and @przemyslaw-elias. I have released version 0.10.0 now.

Guys, it looks like the merge to master didn't go as planned. Before that it's been working properly, but now it's giving an error:
Traceback (most recent call last):
  File "/usr/local/bin/aws-ec2-assign-elastic-ip", line 31, in <module>
    aws_ec2_assign_elastic_ip.main()
  File "/usr/local/lib/python2.7/site-packages/aws_ec2_assign_elastic_ip/__init__.py", line 59, in main
    address = _get_unassociated_address()
  File "/usr/local/lib/python2.7/site-packages/aws_ec2_assign_elastic_ip/__init__.py", line 115, in _get_unassociated_address
    all_addresses = connection.get_all_addresses()
  File "/usr/local/lib/python2.7/site-packages/botocore/client.py", line 566, in __getattr__
    self.__class__.__name__, item)

@pmkane in your PR you changes connection.get_all_addresses into connection.describe_addresses, but the same code slipped through merge to master.
Moreover, I don't know whats wrong, but I had to manually run pip install ec2_metadata because it's been missing.

Moreover, I don't know whats wrong, but I had to manually run pip install ec2_metadata because it's been missing.

Hmm, yes, there is a new dependency added. It must be added to the list of dependencies. Also it seems like the boto dependency should be bumped.
I'll prepare a bugfix PR shortly.

Ugh, sorry -- I clearly rushed the merge.  I am happy to submit a PR to clean up my own mess.  Will do so, unless @sebdah has already gotten there.

Looks like you did, thank you @sebdah.


Separate sidekiq integration into separate files

What
Separates the sidekiq integration into separate files
Why
I'd like to configure Raven without sidekiq, but use a custom error handler that inherits from Raven::SidekiqErrorHandler. Currently  the Raven::SidekiqErrorHandler class cannot be required separately from the Sidekiq.configure_server call.

Closing this due to inactivity. Feel free to reopen it if you feel like continuing it though 😄


doesn`t show mask when field focused

mask shown only if i already enter first symbol.
but it neccesery to display mask when field focused
for example, i need enter phone number
              <TextInputMask                 placeholder="Your phone"                 onChangeText={(formatted, extracted) => this.handleMaskedField(formatted, extracted)}                 mask={"+7 ([000]) [000] [00] [00]"}                 keyboardType="numeric"               />
in some countries writtings +7 and 8 are equalent, and some users enter first symbol 8,
then they see the mask erase this number, and start to enter again.
that`s not ux-friendly,
Please add property, which gives a control to show mask on focus or not

@AtNovember thank you for the issue. I understand that is not UX friendly, but I just integrated the third part library https://github.com/RedMadRobot/input-mask-ios into RN. Maybe I will rewrite native code to my own in the future, but for now please refer this issue to input-mask-ios library.
Thank you for understanding. Спасибо за понимание 😉


mineração

This issue tracker is only for technical issues related to Qtum.
Describe the issue
Can you reliably reproduce the issue?
If so, please list the steps to reproduce below:
1.como minerar sua moeda?
2.
3.
Expected behaviour
Tell us what should happen
Actual behaviour
Tell us what happens instead
Screenshots.
If the issue is related to the GUI, screenshots can be added to this issue via drag & drop.
What version of Qtum are you using?
List the version number/commit ID
Machine specs:

OS:
CPU:
RAM:
Disk size:

Any extra information that might be useful in the debugging process.
This is normally the contents of a debug.log or config.log file. Raw text or a link to a pastebin type site are preferred.

Qtum is proof of stake, there is no proof of work mining.
Qtum é prova de participação, não há provas de mineração de trabalho.


Fixed Texture disposing

Changed GL.DeleteProgram(Handle) to GL.DeleteTexture(Handle)

This is what happens when you copy-paste without paying attention...
Good catch, thanks!


Error on install

Netbox version: 3.4.3
I get error when executing upgrade.sh. What could be a problem?
Collecting netbox_data_flows@ https://github.com/Alef-Burzmali/netbox-data-flows/releases/download/v0.4.0/netbox-data-flows-0.4.0.tar.gz
Using cached netbox_data_flows-0.4.0-py3-none-any.whl
Collecting netbox-secretstore
Using cached netbox_secretstore-1.4.1-py3-none-any.whl (66 kB)
Collecting netbox_attachments
Using cached netbox_attachments-1.1.2-py3-none-any.whl (17 kB)
Collecting importlib
Using cached importlib-1.0.4-py3-none-any.whl
Collecting pycryptodome
Using cached pycryptodome-3.17-cp35-abi3-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (2.1 MB)
Installing collected packages: netbox_data_flows, importlib, pycryptodome, netbox_attachments, netbox-secretstore
Successfully installed importlib-1.0.4 netbox-secretstore-1.4.1 netbox_attachments-1.1.2 netbox_data_flows-0.4.0 pycryptodome-3.17
Applying database migrations (python3 netbox/manage.py migrate)...
Traceback (most recent call last):
File "/opt/netbox/venv/lib/python3.9/site-packages/django/db/backends/utils.py", line 89, in _execute
return self.cursor.execute(sql, params)
psycopg2.errors.UndefinedTable: relation "netbox_data_flows_dataflowgroup" does not exist
LINE 1: ... "netbox_data_flows_dataflowgroup"."tree_id" FROM "netbox_da...

Please use the latest version v0.4.1, it is already fixed.
Thanks

Working fine with v0.4.2.
Than you Alef-Burzmali


Cmdlets can't be loaded on a network share

If the cmdlets are installed on a network share (and share not mounted as a drive), the cmdlets error with "Unable to find setting configuration".
The file name is missing the share name (full UNC path), such as \Users\name\modules\….
Fix is in PowerBISettings.cs, there may be other areas where network paths are not supported.

Should be fixed in https://www.powershellgallery.com/packages/MicrosoftPowerBIMgmt/1.0.342


Issue #1702

Tests for issue #1702 and first fix to use the option again.
Still missing correct charset content-type headers for the preview server. (Maybe also for txt files and similar).

Thank you


Standardize formats

We are using several standard formats but also custom ones.
For standard formats see:

https://json-schema.org/understanding-json-schema/reference/string.html#format
https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.2.md#data-types

I believe it's beneficial to have a complete list of all formats that we support, their semantic and relation to standards, e.g. (not complete!):
type: string

bcp47 language tag (custom)
byte
date
date-time
email
gtin-13 article number (custom)
hostname
ipv4
ipv6
iso-3166-1-alpha-2 country code (custom)
iso-4217 currency code (custom)
iso-639 language code (custom)
json-pointer
password
regex
time
uri
uri-template (RFC6570)
urn (custom)
uuid

type: integer

int32
int64
bigint (custom)

type: number

float
double
decimal (custom)

urn is actually only used once to define the x-api-id property and I believe it's incorrect since it doesn't have a urn: namespace.
https://tools.ietf.org/html/rfc8141

@tkrop Do you remember why we used urn?


AlterNAT at scale Questions

Our Dev/Staging environments quite frankly suck, low traffic at best and in testing we encountered no issues at all (this is good)
When we tried to simulate a production environment, AlterNAT we noticed the instance would start to drop traffic when we reached higher Lambda Execution volumes.    We hit the instance's PPS limit and packets were dropped.
Increasing the NAT instance class seemed to help.
Our Production environment is a different beast.
The vast majority of our NAT traffic is from Lambda executions.  (Occasionally bursting past 300,000 executions per minute)
I'm concerned about hitting a PPS Limit and having drops in production.
Since you stated you send several PB of traffic, I'm going to guess your traffic is a lot more than ours (it would make sense)
Our short lived lambdas (for SSO, Dynamodb lookups, Small API Requests) are all quick in and out, but our long running lambdas can run upwards of 6 minutes (Data continues to flow to/from the browser during this so the pps do not stop)
Without going into specifics:

Do you notice any dropped packets / throttling with a c6gn.8xlarge?
I'm going to guess that your NAT Instances are in every private subnet and thus you can spread out the load across multiple subnets.

All of our lambda's use a single subnet with a NAT Gateway in that subnet.  I unfortunately cannot do that as re-engineering the architecture is not feasible until winter 2025.
(This has been transferred from an email with @bwhaley for public visibility and comments)

Thanks for the question.
We have observed non-zero values in the ENA metrics, namely bw_{in,out}_allowance_exceeded and pps_allowance_exceeded. The values have been so low relative to the total bandwidth/pps that we've just ignored it and counted on TCP to sort it out. Some number of them are probably queued/delayed, but some number are dropped and will result in a retransmit.
These most likely happen because of microbursts in traffic. It's pretty hard to avoid. You can keep upsizing instances until you get to the max to see if that resolves it. It will definitely help, as you observed and as AWS states, such as in this article. If it doesn't, fixing microbursts can be challenging. This article mentions some advanced strategies for mitigating bursts if you cannot scale horizontally (e.g. in your case since you have the constraint of a single subnet/route table), but those approaches are not going to work with Lambdas.
If you haven't already seen it, this article discusses how you can measure pps limits if you want to test different instance types. You may also be able to set up packet captures and look for retransmits to see how widespread the problem actually is.
I opened #107 to make it easier to expose these metrics with Alternat.

@thedoomcrewinc Does this help at all? Are you going to try some larger instances or anything as a next step?

@bwhaley Apologies for the delay in update.
We're in a push for our Back to School effort, and I won't be able to test this until after Aug 1st.  I'll update shortly there after.

Follow up as promised:
After testing various instances classes and size combos up to a c7gn.16xlarge, we determined that the sweet spot was a c6gn.8xlarge instance.
We too observed non-zero values, however like you observed and commented on, we can safely say we are not worrying about the impact.
Microbursts do occur but in general we don't worry about it.
I'll report back in September after the vast majority of schools are back in session and our traffic levels have stabilized at the new "normal"

Thanks, I appreciate that you're following up here!


Loosen immutables version constraint

Fixes #8.

@1st1 can you merge it, please?

Merged & released v2.4


Missing password for user, and running in non-interactive shell

New Issue Checklist

[ x ] Updated fastlane to the latest version
[ x] I read the Contribution Guidelines
[x ] I read docs.fastlane.tools
[x ] I searched for existing GitHub issues

Issue Description

[09:19:54]: Driving the lane 'ad_hoc' 🚀
[09:19:54]: ---------------------
[09:19:54]: --- Step: produce ---
[09:19:54]: ---------------------
+----------------+--------------------------------------+
|              Summary for produce 2.85.0               |
+----------------+--------------------------------------+
| username       |             |
| app_identifier |.00000000.beta  |
|                | .CustomForm.e1628dba.Testing         |
| app_name       | Test UID                             |
| app_version    | 1.00                                 |
| sku            |<PHONE_NUMBER>                           |
| language       | English                              |
| team_id        | XXXXXXXX                           |
| skip_itc       | true                                 |
| platform       | ios                                  |
| skip_devcenter | false                                |
+----------------+--------------------------------------+
-------------------------------------------------------------------------------------
Please provide your Apple Developer Program account credentials
The login information you enter will be stored in your macOS Keychain
You can also pass the password using the FASTLANE_PASSWORD environment variable
See more information about it on GitHub: https://github.com/fastlane/fastlane/tree/master/credentials_manager
-------------------------------------------------------------------------------------
+---------------+--------+
|      Lane Context      |
+---------------+--------+
| PLATFORM_NAME |        |
| LANE_NAME     | ad_hoc |
+---------------+--------+
[09:19:54]: Missing password for user, and running in non-interactive shell
+------+---------+-------------+
|       fastlane summary       |
+------+---------+-------------+
| Step | Action  | Time (in s) |
+------+---------+-------------+
| 💥   | produce | 0           |
+------+---------+-------------+

Environment

✅ fastlane environment ✅
Stack

Key
Value

OS
10.13.3

Ruby
2.3.3

Bundler?
false

Git
git version 2.14.3 (Apple Git-98)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.13.3 (17D47)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib

OpenSSL Version
LibreSSL 2.2.7

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
9.2

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.85.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.0.0

slack-notifier
2.3.2

atomos
0.1.2

CFPropertyList
2.3.6

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.3

xcodeproj
1.5.6

rouge
2.0.7

xcpretty
0.2.8

terminal-notifier
1.8.0

unicode-display_width
1.3.0

terminal-table
1.8.0

plist
3.4.0

public_suffix
2.0.5

addressable
2.5.2

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.6.4

tty-cursor
0.5.0

tty-spinner
0.8.0

babosa
1.0.2

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

excon
0.60.0

faraday
0.14.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.1

gh_inspector
1.1.3

mini_magick
4.5.1

multi_json
1.13.1

multi_xml
0.6.0

rubyzip
1.2.1

security
0.1.3

xcpretty-travis-formatter
1.0.0

dotenv
2.2.1

bundler
1.16.1

faraday_middleware
0.12.2

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.1

mime-types-data
3.2016.0521

mime-types
3.1

little-plugger
1.1.4

logging
2.2.2

jwt
2.1.0

memoist
0.16.0

os
0.9.6

signet
0.8.1

googleauth
0.6.2

httpclient
2.8.3

google-api-client
0.13.6

io-console
0.4.5

libxml-ruby
2.9.0

json
2.1.0

generated on: 2018-03-09

I already have the password stored in the keychain it works if I run the code manually

@klinkert0728 Does your password by any chance have a \ in it? It maybe related to this issue - https://github.com/fastlane/fastlane/issues/10746

@joshdholtz  hey thanks for your reply. No the password does not contain \ . however i was able to fix the issue by moving the password to the system keychain along with the certificate, and then i move it back to the login keychain. and it works perfect.


Manually selectively hide or show cell outputs

Since the notebook will be rendered as doc page. For clarity, we need to manually choose which cell outputs to show or hide.
For example, this is a good cell output that should be left visible. This cell output adds context and necessary information

from FAISS_with_HF_datasets_and_CLIP.ipynb
On the contrary, we should hide this cell output (warnings, install processes, etc)

By hide/show, I mean: make a PR to this repo making necessary changes to the notebooks. You can hide the cell outputs in jupyter notebook. And if necessary, you can change python logging to suppress warnings & only output errors
cc: @MKhalusova @aymeric-roucher

I'll take care of this, thanks!


Add license

Recommend https://opensource.org/licenses/MIT
Edit: looks like package.json already has it mentioned but this could be [license.md] file or a mention at the readme as well to make it clear.
Thanks for the package btw!

Thx for Issue and for your interest in my work


bump test: Point Onyx to a forked branch

Details
This PR is intended to test the Onyx changes from https://github.com/Expensify/react-native-onyx/pull/490

OK, I'm generating the adhoc build for it right now and it should post once it's done. I'm reaching out to the QA team to ask them about bandwidth to test it this week. Are there any specific areas/flows/platforms you would like to have them test specifically? If so, please list them here so they can refer to them.

OK, thanks! I have reached out to QA to have testing scheduled for this. More than likely, it will happen next week as there are some other ad-hoc builds currently being tested.

Regression is completed on Web and Android. No build specific issues found

@paultsimura I only requested for QA to test on web and 1 native platform to speed up testing. I think any regressions would have been caught with only those two platforms. They aren't planning to test any others.
Since their testing didn't reveal any issues, I think it's OK to move forward with merging your change in Onyx and then bumping the Onyx version in App to officially roll it out.


Update PAF and SOW UI

Refs #3477

[x] "External Reviewer" to "Advisory council" PAF detail page Review section.
[x]  Update font size of PAF detail page as per the design
[x] Update approval page ui as per design.

Thanks @sandeepsajan0 Can you please look at the spacing here(link) for the Review section?

@abhiXII I have updated the spacing for the review section.
@frjo Its latest version is also ready for testing.

Deploying latests to test now.


Install of workbox-webpack-plugin@next is failing with a rollup error

Library Affected:
workbox-webpack-plugin
Browser & Platform:
webpack / node (8.9.1) / npm (6.9.0)
Issue or Feature Request Description:
Attempting to install workbox-webpack-plugin@next or<EMAIL_ADDRESS>is failing with the following error. After first release this worked at one point but is now failing for both me locally, on a AWS machine and a co-worker's machine.
Edit: Verified the failure on CI builds as well
npm ERR! path /mnt/airlab/repos/hyperloop/node_modules/rollup/dist/bin/rollup.js
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall chmod
npm ERR! enoent ENOENT: no such file or directory, chmod '/mnt/airlab/repos/hyperloop/node_modules/rollup/dist/bin/rollup.js'
npm ERR! enoent This is related to npm not being able to find a file.

https://github.com/rollup/rollup/issues/3029

Thanks for flagging this—looks like it's fixed on the Rollup side of things now.

Yep, confirmed CI and all is resolved now.


Update .env.example

@LULdev Please be careful not to commit API credentials to source control. I've gone ahead and revoked that API token so that folks can't use it to make calls at your expense.


Suggestion: Add a "default/no override" option for control nodes' default cursor

As per https://github.com/godotengine/godot/issues/18043 / https://github.com/godotengine/godot/pull/18093 we now have a method for setting the default cursor shape; the problem that arises still is that the shape is overridden by any Control Nodes' default cursor setting - which is good, and exactly as intended - except there could be times where you don't want that to happen. E.g. background nodes with stuff happening and changing the cursor on the foreground, or purely hierarchical nodes like a "fullscreen" parent node for UI trees.
I reckon it'd be a good thing if, in the list of cursor shape overrides for the control node, you had a "default shape" and/or "don't do anything" setting; if the cursor moves over a UI element that has this cursor setting, it simply won't change its shape.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so tha
t proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Typo fixes

Overview

Fix typo in internal correct_locale_prefix_effect function
Fix several common typos across the docs
Fix English ordinal examples
Consistently capitalize language names: French and English
A few minor grammar tweaks

Thanks a lot for those typos! I appreciate that a lot, English isn't my mother tongue and with all the work I have to do to push v0.4 as fast as possible I make mistake writing doc.
The doc still has a lot of incoming changes for v0.4, I may overwrite some of your modifications...

Thanks !


Detect if this is a merge or rebase in rebase_migration command

Resolve https://github.com/adamchainz/django-linear-migrations/issues/80. Adds logic to check if there is a merge request in progress. If so, handle it by swapping the migration files.

I don't think this is the right approach. It would be better to fix django-linear-migrations to handle both cases. It's possible to detect if the current directory is a Git repository and whether there's a rebase or merge in progress with a few Git commands.

Agree, it's not the fix for the command, it's just the less risky change that allows us to modify the command. I thought it would be simpler to add changes that don't change the logic. But I can make the existing command work with both rebase and merge. I think we can check the existence of the MERGE_HEAD file in the .git directory (in the same way as the git client does it.
The question for me is how far the library should go to find out the path to the .git directory?

We could execute something like git rev-parse --git-dir directly in the script to find out the path to the .git directory. But in this case we rely on the existence of git command.
We could ask the user to do it via some setting. Because I'm not sure that setting.BASE_DIR is reliable enough to use it.
And in this case, there is also a question about what we should do if there is no Git repository?

Option 2: use git rev-parse --git-dir. Ensure it does not crash if git is not available. Don't error if there is no Git repository, carry on with the existing presume-rebase behaviour.

It's ready for review when you have time @adamchainz . I tested it, works as expected for both rebase and merge

Great, thanks 👍. Can you update the docs to mention the automatic detection , and add a changelog note? I’ll properly review a bit later, if you can do those tasks it would help.

I think your pull request did not allow me to push changes, so I have had to create a new one in #261.
For future ref, please ensure the box is checked: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork


Activate TextSelection on all interesting values/results

Ctrl+C is already supported in the app, but:

not Tablet/Device without keyboard-friendly
It's not really easy to guess that you can Ctrl+C without selecting the result.
You can't copy an operation in the history
You can't copy a part of a result (to ignore the fractional part of a number for example)

If we allow users to select texts, the application will feel more like a "Desktop app", will be easier to use on a tablet and users will be more prompt to use Ctrl+C to use the result in another app.

Note:
I didn't enabled TextSelection on CalculationResult when the control isn't "active" (text in bold), if we do,  the textblock can't catch the Tap events on the textblock, needed to allow users to activate the control.

Rudy, again, you're rocking these changes.  Please do keep them coming.  As I've been commenting on your other PR's just now, our team is standardizing on a flow where a behavior change needs to have an associated Issue which will provide the opportunity for discussion and approval before we'll consider merging in the corresponding code change.
We are in the process of updating CONTRIBUTING.md and our PR templates to clarify this.  In the meantime, please open an Issue to track this so that we can start the discussion.  Our team members will be able to join in on the discussion on Monday.
Again, please keep the contributions coming.  I'm really looking forward to getting in all of these changes once we've had a bit more time to talk them through.

Closing this PR until it's associated with an approved issue. Locking the PR so that conversation on the problem being addressed is discussed in the Issue, so that future PR discussion can be limited to the implementation of the fix.

Oh! I was sure this comment was about the another textblock, I totally
agree, this label should not be selectable, it was a mistake

or not selectable in both.
Second to that, I'm arguing that the 'Date' label should not be
selectable. The only action the user can take by selecting the label is
to copy the label alone, meaning they can't grab Date: Wednesday, May 19,
2021 as a single run. What is special about the label then? What is
helped by allowing it to be selectable where it wasn't before? Why does
this label deserve to be selectable but other labels like 'From' and 'To'
are not selectable?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/calculator/pull/213#discussion_r265849137,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABK3KrdKq08LIkVZOjK5rIqcAur5t8W0ks5vWyZlgaJpZM4bmcGC
.


Terminal "xterm-cursor-layer" Broken In Firefox 105.0.3

Describe the bug
The terminal has some overlay effect obscuring the output in firefox.  If you copy paste from the terminal the text is readable.
Link to the blitz that caused the error
https://stackblitz.com/edit/sveltejs-kit-template-default-dcvphm?file=src%2Froutes%2F%2Bpage.svelte,src%2Froutes%2Fabout%2F%2Bpage.js&terminal=dev
Steps to reproduce
Once the terminal loads the error will occur without further interaction.
Expected behavior
Seeing text in the terminal.
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform
Browser name  = Firefox
Full version  = 105.0
Major version = 105
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:105.0) Gecko/20100101 Firefox/105.0
performance.memory = undefined
Hash = 3e4f172269cb19a250be0dfdff1ab657cb850d0e

Additional context
No response

Due to bug in firefox


Dropping the alarm on live resources

standalone, signals only work in the main thread.  this seemed to work in non-main threads, but since i don't know why i'm taking it out

Coverage increased (+0.05%) to 86.255% when pulling 77c694a36a91b4a1595431d34f36acf344635935 on hotfix/disable-alarm into a2cd2fbc40ec49e062ad5d0159fdf7c15bd7414a on master.


ghcid sometimes fails to parse valid source file

On saving a valid source file from VSCode, sometimes ghcid will fail to interpret the code. The most common error I have seen is the following:
<interactive>:68:53-56: error:
    • Variable not in scope: main :: IO a0
    • Perhaps you meant ‘min’ (imported from Prelude)

...done

This suggests that the source file is temporarily empty or something. I've not seen this happen much (if at all) on macOS. I wonder if there is some kind of race condition in the file watcher employed by ghcid. Alternatively, this could be some strange behaviour exclusive to VSCode and/or an interaction between the Haskero plugin and VSCode. I'll report back with more information as I get it.
Environment:

Ubuntu Desktop 18.04
VSCode 1.28.2 (with Haskero plugin)
Haskell Stack 1.9.1

Repro:

Create new, empty project using Haskell Stack:

stack new ghcid-demo simple
cd ghcid-demo
mv \{\{name\}\}.cabal ghcid-demo.cabal
echo resolver: lts-12.16 >> stack.yaml
echo packages: [.] >> stack.yaml

From newly created project directory, start VSCode with code .
Open integrated terminal in VSCode
Inside integrated terminal, start ghcid with stack exec ghcid
Make edits to src/Main.hs and save a few times

Approximately every other time I save, ghcid will fail to typecheck the source file with the error above regardless of whether the source file is valid Haskell or not.
Update (1): I experimented using Vim instead of VSCode. I ran tmux to split my terminal with Vim in one split and stack exec ghcid in the other. I cannot reproduce the failure with this configuration.
Update (2): Back in VSCode, I tried repeating the scenario with Haskero both enabled and disabled. This seems to make no difference: I still see invalid parses from ghcid from time to time.
This suggests that there might be something fishy about how VSCode saves files.

Weird. I use vs code on Mac and Windows and it's rock solid there. Some editors (Vim) do weird delete, rename, recreate tricks and ghcid works around them. Any idea why it says line 68 of interactive? That strikes me as odd. The --verbose output of ghcid might help shed some light, it's pretty explicit about all the steps it takes.

Sample program Main.hs (the content doesn't matter - as long as it's valid Haskell):
{-# LANGUAGE DataKinds #-}
{-# LANGUAGE GADTs #-}
{-# LANGUAGE KindSignatures #-}
{-# LANGUAGE TypeFamilies #-}
{-# LANGUAGE UndecidableInstances #-}

module Main (main) where

data Nat = Zero | Succ Nat

data Vector (n :: Nat) (a :: *) where
    VNil :: Vector 'Zero a
    VCons :: a -> Vector n a -> Vector ('Succ n) a

instance Show a => Show (Vector n a) where
    show VNil = "VNil"
    show (VCons a as) = "VCons " ++ show a ++ " (" ++ show as ++ ")"

add :: Nat -> Nat -> Nat
add Zero n = n
add (Succ n) m = add n (Succ m)

type family Add n m where
    Add 'Zero n = n
    Add ('Succ n) m = Add n ('Succ m)

main :: IO ()
main = do
    putStrLn "Hello"

Here's the output of running stack exec ghcid -- --verbose 2>&1 | tee ../out.txt in response to typing extra spaces at the end of the file and saving from VSCode repeatedly:
%STDIN: :reload
%STDIN: INTERNAL_GHCID.putStrLn ['#','~','G','H','C','I','D','-','F','I','N','I','S','H','-','1','7','~','#']
INTERNAL_GHCID.hPutStrLn INTERNAL_GHCID.stderr ['#','~','G','H','C','I','D','-','F','I','N','I','S','H','-','1','7','~','#']
%NOTIFY: Modified "/home/rcook/src/tlph/src/Main.hs" 2018-11-23 17:31:01.557799827 UTC False
%STDOUT: #~GHCID-START~##~GHCID-START~#[1 of 1] Compiling Main             ( /home/rcook/src/tlph/src/Main.hs, nothing )
%STDOUT: Ok, one module loaded.
%STDOUT: #~GHCID-START~##~GHCID-FINISH-17~#
%STDERR: #~GHCID-FINISH-17~#
%STDIN: :show modules
%STDIN: INTERNAL_GHCID.putStrLn ['#','~','G','H','C','I','D','-','F','I','N','I','S','H','-','1','8','~','#']
INTERNAL_GHCID.hPutStrLn INTERNAL_GHCID.stderr ['#','~','G','H','C','I','D','-','F','I','N','I','S','H','-','1','8','~','#']
%STDOUT: #~GHCID-START~##~GHCID-START~#Main             ( /home/rcook/src/tlph/src/Main.hs, nothing )
%STDOUT: #~GHCID-START~##~GHCID-FINISH-18~#
%STDERR: #~GHCID-FINISH-18~#
%MESSAGES: [Loading {loadModule = "Main", loadFile = "/home/rcook/src/tlph/src/Main.hs"}]
%LOADED: ["/home/rcook/src/tlph/src/Main.hs"]
]0;All good, at 09:31:01 - tlph
All good (1 module, at 09:31:01)

%WAITING: /home/rcook/src/tlph/.ghcid ./stack.yaml /home/rcook/src/tlph/tlph.cabal /home/rcook/.ghc/ghci.conf /tmp/haskell-stack-ghci/57f1a991/ghci-script /home/rcook/src/tlph/src/Main.hs
%WAITING: /home/rcook/.ghc /home/rcook/src/tlph /home/rcook/src/tlph/src /tmp/haskell-stack-ghci/57f1a991
%NOTIFY: Modified "/home/rcook/src/tlph/src/Main.hs" 2018-11-23 17:31:29.740270498 UTC False
%WAITING: Notify signaled
%RELOADING: /home/rcook/src/tlph/src/Main.hs

Reloading...
  /home/rcook/src/tlph/src/Main.hs

%STDIN: :reload
%STDIN: INTERNAL_GHCID.putStrLn ['#','~','G','H','C','I','D','-','F','I','N','I','S','H','-','1','9','~','#']
INTERNAL_GHCID.hPutStrLn INTERNAL_GHCID.stderr ['#','~','G','H','C','I','D','-','F','I','N','I','S','H','-','1','9','~','#']
%NOTIFY: Modified "/home/rcook/src/tlph/src/Main.hs" 2018-11-23 17:31:29.745815983 UTC False
%STDOUT: #~GHCID-START~##~GHCID-START~#[1 of 1] Compiling Main             ( /home/rcook/src/tlph/src/Main.hs, nothing )
%STDERR:
%STDERR: /home/rcook/src/tlph/src/Main.hs:1:1: error:
%STDERR:     The IO action ‘main’ is not defined in module ‘Main’
%STDERR:   |
%STDERR: 1 | {-# LANGUAGE DataKinds #-}
%STDERR:   | ^
%STDOUT: Failed, no modules loaded.
%STDOUT: #~GHCID-START~##~GHCID-FINISH-19~#
%STDERR: #~GHCID-FINISH-19~#
%STDIN: :show modules
%STDIN: INTERNAL_GHCID.putStrLn ['#','~','G','H','C','I','D','-','F','I','N','I','S','H','-','2','0','~','#']
INTERNAL_GHCID.hPutStrLn INTERNAL_GHCID.stderr ['#','~','G','H','C','I','D','-','F','I','N','I','S','H','-','2','0','~','#']
%STDOUT: #~GHCID-START~##~GHCID-START~##~GHCID-START~##~GHCID-FINISH-20~#
%STDERR: #~GHCID-FINISH-20~#
%MESSAGES: [Loading {loadModule = "Main", loadFile = "/home/rcook/src/tlph/src/Main.hs"},Message {loadSeverity = Error, loadFile = "/home/rcook/src/tlph/src/Main.hs", loadFilePos = (1,1), loadFilePosEnd = (1,1), loadMessage = ["\ESC[;1m/home/rcook/src/tlph/src/Main.hs:1:1: \ESC[;1m\ESC[31merror:\ESC[0m\ESC[0m\ESC[;1m\ESC[0m\ESC[0m\ESC[;1m","    The IO action \8216main\8217 is not defined in module \8216Main\8217\ESC[0m\ESC[0m","\ESC[;1m\ESC[34m  |\ESC[0m\ESC[0m","\ESC[;1m\ESC[34m1 |\ESC[0m\ESC[0m \ESC[;1m\ESC[31m{\ESC[0m\ESC[0m-# LANGUAGE DataKinds #-}","\ESC[;1m\ESC[34m  |\ESC[0m\ESC[0m\ESC[;1m\ESC[31m ^\ESC[0m\ESC[0m"]}]
%LOADED: ["/home/rcook/src/tlph/src/Main.hs"]
]0;1 error - tlph
/home/rcook/src/tlph/src/Main.hs:1:1: error:
    The IO action ‘main’ is not defined in module ‘Main’
  |
1 | {-# LANGUAGE DataKinds #-}
  | ^

%WAITING: /home/rcook/src/tlph/.ghcid ./stack.yaml /home/rcook/src/tlph/tlph.cabal /home/rcook/.ghc/ghci.conf /tmp/haskell-stack-ghci/57f1a991/ghci-script /home/rcook/src/tlph/src/Main.hs
%WAITING: /home/rcook/.ghc /home/rcook/src/tlph /home/rcook/src/tlph/src /tmp/haskell-stack-ghci/57f1a991

That's not something I've seen before - it looks like ghci has seen the file half written which an editor should never do. Are you using an up to date vs code and ghcid? Is there some kind of overwrite in place option enabled?

$ uname -a
Linux mymachine 4.15.0-38-generic #41-Ubuntu SMP Wed Oct 10 10:59:38 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
$ code --version
1.29.1
bc24f98b5f70467bc689abf41cc5550ca637088e
x64
$ stack --version
Version 1.9.1, Git revision f9d0042c141660e1d38f797e1d426be4a99b2a3c (6168 commits) x86_64 hpack-0.31.0
$ stack exec ghcid -- --version
Auto reloading GHCi daemon v0.7.1

I've disabled all Haskell-related extensions in VSCode and reset all settings to default (by deleting settings.json etc.) and the problem persists.

I just did some experimentation with inotifywait:
#!/bin/bash
while inotifywait -e modify src/Main.hs; do
  stat src/Main.hs
done

This will occasionally report a zero-length file size. I think it's safe to say that this behaviour is unrelated to ghcid. It looks like some VSCode weirdness.
I'll close this issue. Thanks for your time!

I created a VSCode issue: https://github.com/Microsoft/vscode/issues/63720. I think that this issue might be an interaction between Git, VSCode and inotify.

Thanks! That does seem like a VS Code bug, and one I've never seen on my machines.

I think I do a very brief sleep after I detect a change, so would be weird that writing doesn't complete in that window. I should take a look.

I have the same issue with (g)vim on Arch. About once in every 10 saves, ghcid sees a partial file.

Does ghcid then quickly soon after spot the full file? As in you only see the partial file for a fraction of a second? It's meant to immediately rerun if it detects a file changed while it was loading.
Does adding a sleep before https://github.com/ndmitchell/ghcid/blob/master/src/Wait.hs#L108 help?

I've been seeing this error about 9 times out of every 10 saves when using VSCode and the remote extensions (the remote end is an Ubuntu 19.04 box).
Adding sleep 0.05 as suggested in Wait.hs seems to fix the problem.

If this is an issue that keeps cropping up then perhaps adding the sleep always isn't a bad idea. 0.05s is small enough that it's probably not perceptibly slower (although always sad to think of those 0.05s adding up 😢). Anyone want to whip up a PR, given that I can't test it as it works robustly for me.


Suggest tasks for the Evaluation Harness

Creating an Evaluation Harness for code LLMs
We are working on an Evaluation Harness that covers a wide array of coding tasks and programming languages. We'd appreciate your input!
Existing list
Please take a look at the existing sheet of evaluation benchmarks here.
Contribute
Please use the following template to suggest new tasks for the Evaluation Harness.

Name
Link
Number of samples
Languages
Available on the HF Hub

HumanEval
https://github.com/openai/human-eval
164
Python
Yes

Here's the Markdown snippet that you can copy/paste:
|Name|Link|Number of samples| Languages |Available on the HF Hub|
|:-|:-|:-|:-|:-
| | | | | | | |

Name
Link
Number of samples
Languages
Available on the HF Hub

DS-1000 Data Science Code Generation
https://ds1000-code-gen.github.io/
1000
Python
No

Name
Link
Number of samples
Languages
Available on the HF Hub

CodeXGlue
https://github.com/microsoft/CodeXGLUE
14 datasets for 10 diversified code intelligence tasks (including code-code, code-test, text-code, text-text), Train/Val/Test splits for each dataset in the range of 1k-100k samples
Java, C++, Python, Javascript, C, PHP, Ruby, Go, C#
Not sure


🎨 update progress component to cap values at max and 0

in conclusion we said that it was up to the developer to do it on their side when they handle their states, example:
<Button type={"dark"} onClick={() => setValue(value - 25)}>
  -
</Button>

could be:
<Button type={"dark"} onClick={() => setValue((value - 25) > 0 ? 0 : (value - 25))}>
  -
</Button>


[BUG] Add missing social media icons

Description
2 social media icons are missing on the "About Us" page.
I want to Fixed this issue properly.
PA please assign me this task.

Screenshots
No response
Any additional information?
No response
What browser are you seeing the problem on?
No response
Checklist

[X] I have checked the existing issues.
[X] I have read the Code Of Conduct
[X] The changes don't break the code
[X] Please attach the screenshots and video of the development work once it is done

@swaraj-das @AnitSarkar123 please assign me


Cypress test for issue 1841

Cypress test for issue 1841
Motivation and context

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2020 Intel Corporation
#
# SPDX-License-Identifier: MIT

Pull Request Test Coverage Report for Build 7010

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.004%) to 69.726%

Files with Coverage Reduction
New Missed Lines
%

cvat/apps/engine/media_extractors.py
1
77.29%

Totals

Change from base Build 6998:
0.004%

Covered Lines:
11726

Relevant Lines:
16363

💛  - Coveralls


No package manager post-install instructions to enable key bindings

Testing with debian 10, apt install fzf -y but ctrl-r is replaced afterwards. This is good IMO and it should be up to each user to enable it. I would expect to find instructions to do so in the readme here, or in fzf --help relating to how to set up key bindings. Or have I just missed it?

Currently, we're stating that some additional steps may be required.
https://github.com/junegunn/fzf#using-linux-package-managers

Shell extensions (key bindings and fuzzy auto-completion) and Vim/Neovim plugin may or may not be enabled by default depending on the package manager. Refer to the package documentation for more information.

We're not the maintainer of the packages, so we can't provide complete documentation for all of those packages here. That would make our documentation unnecessarily long.

For Ubuntu 20.04.1 LTS, I noticed the file README.Debian while going through the list of files provided by the package manager, and it included post-install instructions.
Bash
====

Append this line to ~/.bashrc to enable fzf keybindings for Bash:

   source /usr/share/doc/fzf/examples/key-bindings.bash

Append this line to ~/.bashrc to enable fuzzy auto-completion for Bash:

   source /usr/share/doc/fzf/examples/completion.bash

Zsh
===

Append this line to ~/.zshrc to enable fzf keybindings for Zsh:

   source /usr/share/doc/fzf/examples/key-bindings.zsh

Append this line to ~/.zshrc to enable fuzzy auto-completion for Zsh:

   source /usr/share/doc/fzf/examples/completion.zsh

Fish
====

Issue the following commands to enable fzf keybindings for Fish:

   mkdir -p ~/.config/fish/functions/
   echo fzf_key_bindings > ~/.config/fish/functions/fish_user_key_bindings.fish

Vim
===

The straightforward way to use fzf.vim is appending this line to your vimrc:

   source /usr/share/doc/fzf/examples/fzf.vim


HTML Escaping broken

It seems that if your input to the _.escape function has anything that looks like it's already escaped, then the escape method is failing to encode it correctly.
e.g.
_.escape("martyn&stuff") // produces "martyn&stuff"
_.escape("martyn&stuff") // produces "martyn&stuff"

The latter should really produce:
 "martyn&amp;stuff"

Is there any reason you can't just do something like:
    _.escape = function(t) {
        var d = document.createElement('div');
        d.appendChild(document.createTextNode(t));
        return d.innerHTML;
    }

My recommendation:
return input
.replace(/&/g, '&')
.replace(/</g, '<')
.replace(/>/g, '>')
.replace(/"/g, '"')
.replace(/'/g, ''');

It looks oversimplified, but in my experience, it's all that's necessary and produces consistently accurate results. Even the last line (single quote) isn't really necessary unless you're using single quotes to escape attributes.
It's important to put the & replacement first so you don't erroneously double-escape other entities.
@martynsmith only reason would be performance. It's a heck of an overhead for a basic substitution function.
I'll submit a pull request for this one if no one picks it up. Cheers folks :)

Without actually testing this, are you sure a handful of search and replace regexes are actually faster? ;-)
I might go write a few tests, I'm quite curious now ...

Your proposal does not work on Server Side JS.

Yup, and my brief testing of the performance says it's fairly dismal on that front too. What hx suggested looks pretty good, and it won't break on my example :-)

@martynsmith it depends on the JS engine; most are smart enough to recognize literal expressions and perform similarly to a standard search/replace. Unfortunately if you don't use a regex, it only replaces the first occurrence.
I haven't yet tried something along these lines:
while((pos = input.indexOf('&')) !== -1)
    input = input.substr(0, pos) + input.substr(pos + 1);

There'd need to be a last-character check in there too, do it once for each of the 5 characters... it avoids using a regex but I doubt it'd be faster. I've seen all sorts of variations converting the string to character arrays etc... kudos to the man who has the time to benchmark them all and publish the results.
Someone posted an elegant solution involving a callback function here: http://stackoverflow.com/questions/5499078/fastest-method-to-escape-html-tags-as-html-entities But I doubt the overhead of an extra function will do us any favors either.

So, the behavior that you're describing here is actually a feature, and not a bug .... but I can see how some folks wouldn't want it.
The current implementation that we're using is:
// Escape a string for HTML interpolation.
_.escape = function(string) {
  return (''+string)
    .replace(/&(?!\w+;|#\d+;|#x[\da-f]+;)/gi, '&')
    .replace(/</g, '<')
    .replace(/>/g, '>')
    .replace(/"/g, '"')
    .replace(/'/g, ''')
    .replace(/\//g,'/');
};

... which purposefully avoids double-encoding things that are already encoded as HTML entities. This makes is possible to mix user-entered input with already-correct HTML without errors.
On the other hand, what you're proposing makes sense if you want to escape content that you already know is HTML escaped. Why would you ever want to do that?

The problem is that the method doesn't actually HTML encode the input.
The point here is that I'm taking user input, then I want to HTML encode it. I don't care what the user types in, I just want that displayed back to them. What you have here is not doing that, it's encoding it "if it feels it should", but why should it decide what does and doesn't need encoding, that's for me (the developer) to decide.

@jashkenas a real-world example would be a site like this, which displays code, or a site that documents code or markup. If GitHub were to use _.escape() to display my ramblings on the subject of the <br /> tag, I'd end up with a lot of unwanted line breaks in my post.
The principle is that _.escape() attempts to auto-detect the state of its input, but we know from all our toils with HTML entities, UTF-x, percent encoding and so forth that it's better to discipline your code so that you always know what state your data is in, and just apply encoding transformations where necessary.
On a brief side note, I've come across a quirk in Backbone's routing system that does a similar things with % signs in URIs which leads to headaches when, for example, someone wants to search for the literal string "%25", but I'll raise that at a later date.
Ultimately I think the solution is to have two methods; _.escape() as a blunt-instrument that encodes everything, and something like _.autoEscape() akin to what's currently in the library.
From the point of view of consistency, similar methods like escape(), encodeURIComponent() and atob() will also double-encode if you want them to. Real-world uses are rare but they do come up, mainly in shunting information around in reusable blobs.

This change is great, but a version with the old feature should be provided, in my opinion.

Can't be done with a nice API, I'm afraid. We're not going to add another version of <%- ... %> to Underscore templates. We should just pick the better of the two options here.

Mh, ok I agree. Sorry

@yuchi If you need the old behaviour, you can simply un-escape your data first, or use <% print(_.escape(myUnescapeMethod(prop))) %>. Not exactly pretty, I know, but them's the breaks.
@jashkenas I don't think Underscore templates need another interpolation syntax, but the old behaviour of _.escape() does have some value. PHP's html_entities() method has the $double_encode argument to solve exactly this problem (see http://php.net/html_entities) – perhaps mimicking it with _.escape(mixedEncodingString, false) would be the way to go.

That PHP's function was exactly what I was thinking about, and it's quite pretty, and the signature you're talking about is what could do the job.
BTW my complaint was only a theorical one, on my application I personally use Dust.js templates.