add better URL validation

This protects against dropping the path in baseURI. Before this patch baseURI would lose it's path and only retain everything before the first slash. This also protects against multiple slashes.
For example http://localhost:8081/redappples/ was turning into http://localhost:8081/ yet I needed the base path and was expecting this. For people that are expecting the old behaviour this may be an issue though so we might have to add a test and do a major bump.

published as v1.6.2 to npm @OmgImAlexis


Update anyhow to version 1.0.58

The incompatibility was fixed for intel-mkl-tool https://github.com/rust-math/intel-mkl-src/issues/68
The current version of linfa is not compatible with ndarray-linalg/intel-mkl-static.

I think you can remove the anyhow dependency altogether

Yes, much better. In order to maintain the commit history clean and get rid of this last commit, do you have any preference? Or is it ok if I add a new commit on top of it?

New commit is fine. Also can you update the MSRV in CI to 1.59 to avoid the CI errors?

It seems that ndarray-linalg needs to be updated to version 0.15 (https://github.com/rust-ndarray/ndarray-linalg/pull/320), to support intel-mkl-tool version 0.3.0,  which fixes use anyhow::*; (https://github.com/rust-math/intel-mkl-src/pull/70).
I will update it and see if this does not break anything else.


Fork with HD and 4K class extensions

Hi @johno and Co.
I've been super satisfied using Tachyons on every single project for a while now.
Long story short, I make a lot of apps which need to support HD and 4K touchscreens, often with different layouts, so have forked the repo to add -hd and -4k class extensions toall the files that use media queries.
It adds ~3kB to the gzip (16.97KB) and it's not universally useful, so not sure if you want a PR? Also let me know if you have any preferences re: attribution, name etc?
https://github.com/simon-hayden/tachyons-4k
Thank you for Tachyons and all the other projects you share with us. Always lovin' your work!

Thanks but not interested in adding this in the main build. There are a number of cases to support that are of equal weight (orientation for one) and would cause a larger spike than 3kb when all is said and done. We handle things like this as opt in add on packages that aren't a part of the main build. A small mention of the prior art involved in the fork is always appreciated. Thanks for checking - appreciate it.


Quantization Aware Training with tf.GradientTape gives Error in TensorFlow2.0

Hey Guys, I am using TensorFlow-2.2, tensorflow_model_optimization and Python 3.8. I am trying to quantize and train a LeNet-300-100 Dense neural network which contains sparsity of 91.3375%. This means that 91.3375% of the weights are zero. I was following the Quantization TF tutorial and I wanted to train such a sparse network which has been quantized using tf.GradientTape rather than q_aware_model.fit().
If you look into the example code, the relevant code snippets are:
quantize_model = tfmot.quantization.keras.quantize_model

# q_aware stands for for quantization aware.
q_aware_model = quantize_model(model)

# 'quantize_model' requires recompilation-
q_aware_model.compile(
    optimizer = tf.keras.optimizers.Adam(lr = 0.0012),
    loss=tf.keras.losses.categorical_crossentropy,
    metrics=['accuracy']
)

# Define 'train_one_step()' and 'test_step()' functions here-
@tf.function
def train_one_step(model, mask_model, optimizer, x, y):
    '''
    Function to compute one step of gradient descent optimization
    '''
    with tf.GradientTape() as tape:
        # Make predictions using defined model-
        y_pred = model(x)

        # Compute loss-
        loss = loss_fn(y, y_pred)

    # Compute gradients wrt defined loss and weights and biases-
    grads = tape.gradient(loss, model.trainable_variables)

    # type(grads)
    # list

    # List to hold element-wise multiplication between-
    # computed gradient and masks-
    grad_mask_mul = []

    # Perform element-wise multiplication between computed gradients and masks-
    for grad_layer, mask in zip(grads, mask_model.trainable_weights):
        grad_mask_mul.append(tf.math.multiply(grad_layer, mask))

    # Apply computed gradients to model's weights and biases-
    optimizer.apply_gradients(zip(grad_mask_mul, model.trainable_variables))

    # Compute accuracy-
    train_loss(loss)
    train_accuracy(y, y_pred)

    return None

@tf.function
def test_step(model, optimizer, data, labels):
    """
    Function to test model performance
    on testing dataset
    """

    predictions = model(data)
    t_loss = loss_fn(labels, predictions)

    test_loss(t_loss)
    test_accuracy(labels, predictions)

    return None

# Train model using 'GradientTape'-

# Initialize parameters for Early Stopping manual implementation-
# best_val_loss = 100
# loc_patience = 0

for epoch in range(num_epochs):

    if loc_patience >= patience:
        print("\n'EarlyStopping' called!\n")
        break

    # Reset the metrics at the start of the next epoch
    train_loss.reset_states()
    train_accuracy.reset_states()
    test_loss.reset_states()
    test_accuracy.reset_states()

    for x, y in train_dataset:
        train_one_step(q_aware_model, mask_model, optimizer, x, y)

    for x_t, y_t in test_dataset:
        test_step(q_aware_model, optimizer, x_t, y_t)

    template = 'Epoch {0}, Loss: {1:.4f}, Accuracy: {2:.4f}, Test Loss: {3:.4f}, Test Accuracy: {4:4f}'

    '''
    # 'i' is the index for number of pruning rounds-
    history_main[i]['accuracy'][epoch] = train_accuracy.result() * 100
    history_main[i]['loss'][epoch] = train_loss.result()
    history_main[i]['val_loss'][epoch] = test_loss.result()
    history_main[i]['val_accuracy'][epoch] = test_accuracy.result() * 100
    '''

    print(template.format(
        epoch + 1, train_loss.result(),
        train_accuracy.result()*100, test_loss.result(),
        test_accuracy.result()*100)
         )

    # Count number of non-zero parameters in each layer and in total-
    # print("layer-wise manner model, number of nonzero parameters in each layer are: \n")
    model_sum_params = 0

    for layer in winning_ticket_model.trainable_weights:
        # print(tf.math.count_nonzero(layer, axis = None).numpy())
        model_sum_params += tf.math.count_nonzero(layer, axis = None).numpy()

    print("Total number of trainable parameters = {0}\n".format(model_sum_params))

    # Code for manual Early Stopping:
    if np.abs(test_loss.result() < best_val_loss) >= minimum_delta:
        # update 'best_val_loss' variable to lowest loss encountered so far-
        best_val_loss = test_loss.result()

        # reset 'loc_patience' variable-
        loc_patience = 0

    else:  # there is no improvement in monitored metric 'val_loss'
        loc_patience += 1  # number of epochs without any improvement

Gives the following error:

--------------------------------------------------------------------------- InvalidArgumentError                      Traceback (most recent call
last)  in
19
20     for x, y in train_dataset:
---> 21         train_one_step(q_aware_model, mask_model, optimizer, x, y)
22
23
~/.local/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py
in call(self, *args, **kwds)
578         xla_context.Exit()
579     else:
--> 580       result = self._call(*args, **kwds)
581
582     if tracing_count == self._get_tracing_count():
~/.local/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py
in _call(self, *args, **kwds)
642         # Lifting succeeded, so variables are initialized and we can run the
643         # stateless function.
--> 644         return self._stateless_fn(*args, **kwds)
645     else:
646       canon_args, canon_kwds = \
~/.local/lib/python3.8/site-packages/tensorflow/python/eager/function.py
in call(self, *args, **kwargs)    2418     with self._lock:
2419       graph_function, args, kwargs =
self._maybe_define_function(args, kwargs)
-> 2420     return graph_function._filtered_call(args, kwargs)  # pylint: disable=protected-access    2421     2422   @property
~/.local/lib/python3.8/site-packages/tensorflow/python/eager/function.py
in _filtered_call(self, args, kwargs)    1659       args and
kwargs.    1660     """
-> 1661     return self._call_flat(    1662         (t for t in nest.flatten((args, kwargs), expand_composites=True)    1663
if isinstance(t, (ops.Tensor,
~/.local/lib/python3.8/site-packages/tensorflow/python/eager/function.py
in _call_flat(self, args, captured_inputs, cancellation_manager)
1743         and executing_eagerly):    1744       # No tape is
watching; skip to running the function.
-> 1745       return self._build_call_outputs(self._inference_function.call(    1746
ctx, args, cancellation_manager=cancellation_manager))    1747
forward_backward = self._select_forward_and_backward_functions(
~/.local/lib/python3.8/site-packages/tensorflow/python/eager/function.py
in call(self, ctx, args, cancellation_manager)
591       with _InterpolateFunctionError(self):
592         if cancellation_manager is None:
--> 593           outputs = execute.execute(
594               str(self.signature.name),
595               num_outputs=self._num_outputs,
~/.local/lib/python3.8/site-packages/tensorflow/python/eager/execute.py
in quick_execute(op_name, num_outputs, inputs, attrs, ctx, name)
57   try:
58     ctx.ensure_initialized()
---> 59     tensors = pywrap_tfe.TFE_Py_Execute(ctx._handle, device_name, op_name,
60                                         inputs, attrs, num_outputs)
61   except core._NotOkStatusException as e:
InvalidArgumentError:  var and grad do not have the same shape[10]
[100,10] 	 [[node Adam/Adam/update_4/ResourceApplyAdam (defined at
:29) ]]
[Op:__inference_train_one_step_20360]
Errors may have originated from an input operation. Input Source
operations connected to node Adam/Adam/update_4/ResourceApplyAdam:
Mul_4 (defined at :26)
sequential/quant_dense_2/BiasAdd/ReadVariableOp/resource (defined at
/home/arjun/.local/lib/python3.8/site-packages/tensorflow_model_optimization/python/core/quantization/keras/quantize_wrapper.py:162)
Function call stack: train_one_step

Is there a way to combine TF model Quantization along with tf.GradientTape?
Thanks!

@arjun-majumdar
Request you to provide colab link or simple standalone code with supporting files to reproduce the issue.(I believe the saved model file is missing in our environment.It helps us in localizing the issue faster.Thanks!

@ravikyram added the .h5 saved model file to sub-model
Please let me know if I can help with anything else?

@arjun-majumdar
I am not able to open example code given by you.Please, help.Thanks!

@ravikyram here is the URL to Google Colab Jupyter Notebook
Let me know if I can help in anything else. Looking forward to your reply.

@arjun-majumdar
Please, grant me access for Colab notebook.Thanks!

@ravikyram Granted! Hope it works now.

I have tried in colab with TF version 2.2 ,nightly version and was able to reproduce the issue.Please, find the gist here.Thanks!

@arjun-majumdar Sorry for the late response. Can you please check with recent TFversion 2.4rc3 and let us know whether the issue persists or not. Thanks!

@jvishnuvardhan
I installed TF 2.4rc3 as asked and the error persists. If you navigate to the last cell in the notebook, you can find the error:

InvalidArgumentError                      Traceback (most recent call last)
 in ()
19
20     for x, y in train_dataset:
---> 21         train_one_step(q_aware_model, mask_model, optimizer, x, y)
22
23
4 frames
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/execute.py in quick_execute(op_name, num_outputs, inputs, attrs, ctx, name)
58     ctx.ensure_initialized()
59     tensors = pywrap_tfe.TFE_Py_Execute(ctx._handle, device_name, op_name,
---> 60                                         inputs, attrs, num_outputs)
61   except core._NotOkStatusException as e:
62     if name is not None:
InvalidArgumentError:  var and grad do not have the same shape[10] [100,10]
[[node Adam/Adam/update_4/ResourceApplyAdam (defined at :29) ]] [Op:__inference_train_one_step_25816]
Errors may have originated from an input operation.
Input Source operations connected to node Adam/Adam/update_4/ResourceApplyAdam:
sequential/quant_dense_2/BiasAdd/ReadVariableOp (defined at /usr/local/lib/python3.6/dist-packages/tensorflow_model_optimization/python/core/quantization/keras/quantize_wrapper.py:167)
Mul_4 (defined at :26)
Function call stack:
train_one_step

Is there any resolution to this problem?
Thanks!

@arjun-majumdar Sorry for the late response.  Is this still an issue for you?
It would be great If you could provide a simple standalone. The current code is too long for us to debug and resolve the issue. We generally suggest to post this kind of questions in Stackoverflow where there is a large community to support this kind of questions.
GitHub is mainly for bugs and performance related issues. Thanks!

@jvishnuvardhan Yes, the issue still persists.
A "simple" standalone code is as follows:
# Create a quantized model-
quantize_model = tfmot.quantization.keras.quantize_model

# q_aware stands for for quantization aware.
q_aware_model = quantize_model(winning_ticket_model)

# 'quantize_model' requires recompilation-
q_aware_model.compile(
    optimizer = tf.keras.optimizers.Adam(lr = 0.0012),
    loss=tf.keras.losses.categorical_crossentropy,
    metrics=['accuracy']
)

@tf.function
def train_one_step(model, mask_model, optimizer, x, y):
    '''
    Function to compute one step of gradient descent optimization
    '''
    with tf.GradientTape() as tape:
        # Make predictions using defined model-
        y_pred = model(x)

        # Compute loss-
        loss = loss_fn(y, y_pred)

    # Compute gradients wrt defined loss and weights and biases-
    grads = tape.gradient(loss, model.trainable_variables)

    # type(grads)
    # list

    # List to hold element-wise multiplication between-
    # computed gradient and masks-
    grad_mask_mul = []

    # Perform element-wise multiplication between computed gradients and masks-
    for grad_layer, mask in zip(grads, mask_model.trainable_weights):
        grad_mask_mul.append(tf.math.multiply(grad_layer, mask))

    # Apply computed gradients to model's weights and biases-
    optimizer.apply_gradients(zip(grad_mask_mul, model.trainable_variables))

    # Compute accuracy-
    train_loss(loss)
    train_accuracy(y, y_pred)

    return None

# Get one batch of training data and labels-
x, y = next(iter(train_dataset))
print(f"x.shape = {x.shape} & y.shape = {y.shape}")

# Perform one training iteration/step-
train_one_step(q_aware_model, mask_model, optimizer, x, y)

You can refer to the complete Google Colab jupyter-notebook here
Let me know if you need anything else.

Team, any update on this

@arjun-majumdar Could you please try on the TF v2.6.0 and let us know if it helps? Thank you!


expand clipping region to include one little county-was causing proje…

…ction error
noticed that changes in #101 cut one little county off in PR. Just expanded so that we don't get a projection error and can keep on moving!

Could you also edit the other line that I made Marty edit? Line 81: https://github.com/USGS-VIZLAB/water-use-15/pull/103/files#diff-1b045ffa0e22d4eccda9408d9c5d9777R81

oops - completely meant to do that


Non-blocking Schema Initialization for ASP.net core middleware

Is your feature request related to a problem? Please describe.
I'm just started looking into this tool so perhaps I'm missing something, but it would be nice if there were the option to load the schema in a non-blocking when using the ASP.net core middleware.
Right now I'm having to do something like this to initialize the schema in Startup.cs:
public void ConfigureServices(IServiceCollection services) {   ISchema schema = Schema.Create(File.ReadAllText(($"{env.ContentRootPath}/GraphQL/schema.graphql")));   services.AddGraphQL(schema); }
Describe the solution you'd like
It would be nice if there were an alternative async lazy initialization. I imagine there could be performance downsides if requests come in before the application loads; however, this could be mitigated by using something like this (https://github.com/thomaslevesque/AspNetCore.AsyncInitialization) for those concerned about performance.
Describe alternatives you've considered

Not using the middleware and implementing async schema loading/the endpoint
Maybe this is possible and I've just overlooked something key

Is your concern here for the ReadAllText?
What would be the gain here?

Yep, my primary concern is needing to do blocking file IO associated with ReadAllText, although it could extend to the schema binding as well depending on performance. The primary benefit would be a faster startup time in applications that do a lot of work starting up.
On a side note, I really appreciated what you're doing with this tool, it reminds me a bit of graphql java tools. I was happy to find there seems to be support for async graphql resolvers, which is far more important when it comes to performance.

The schema binding itself does no IO work. So, it basically is only the ReadAllText. I think it becomes problematic with dependency injection since everything there is sync.
So, we could introduce some kind of Async alternative to create an assembly but if you let dependency injection take care of creating the schema instance than you are basically blocking again.
However, we will do a lot of work for startup performance in general with version 9.1. There will be options to lazy compile resolvers and other stuff.

Thanks @michaelstaib , I'm looking forward to the changes in 9.1! I'm a big proponent of schema-first development and it sounds like the upgrade will involve some major improvements there. I've found this pattern to be a nice clean way to get started with schema-first development:
            var schema = Schema.Create(schema, c =>
            {
                c.BindResolver<MutationResolver>().To("Mutation");
                c.BindResolver<QueryResolver>().To("Query");
            });

    public class MutationResolver
    {
        ...
        [GraphQLName("updateVersion")]
        public Settings UpdateVersion(string version)
        {
            ...
            return settingsService.get();
        }
    }

If you are on version 9 preview try the new schema builder:
SchemaBuilder.New()
    .AddDocumentFromString("type Query { ....")
    .BindResolver<MutationResolver>().To("Mutation")
    .BindResolver<QueryResolver>().To("Query")
    .Create();

I haven't upgraded yet, but I'll be sure to use that when I do. Thanks for the tip!

If you are on version 9 preview try the new schema builder:
SchemaBuilder.New()
    .AddDocumentFromString("type Query { ....")
    .BindResolver<MutationResolver>().To("Mutation")
    .BindResolver<QueryResolver>().To("Query")
    .Create();

@michaelstaib I've upgraded to version 9.0.1 but it doesn't look like it supports the "To("..")" in your schema builder example. Is it done another way now?

This one is now implemented with the new configuration API on version 11.


Remove the GetSafeNormal call for the path centroid calculation

Why couldn't this happen before 0.44 was packaged :D


little grammar error

add asking what os they are on too?

@Coolfeather2 No need, due to the logs mentioning what OS they are on.

@OxLemonxO If your gonna do that, don't do the "the". If you want to get into specifics, "the" makes it sound as if CubeCoders is apart of the name of the application. You can fix my mistake, however, by either leaving it as is, or doing CubeCoders'


A lot of projects are outdated

Which sample are you reporting a bug in?
DWriteColorGlyph
Describe the bug
Similarly to #1320, it said I was missing ImageContentTask.props. I tried defining VSINSTALLDIR, but then I realized that the project configuration was completely empty: there's no output directory, no nothing. Configuration warnings and errors pop up.
To Reproduce
Open this project in the latest Visual Studio Community, I guess
Expected behavior
A fully ready sample project just like most of the other samples here
Screenshots

Configuration
Visual Studio version: Community 17.9.5
Additional context
Seems to happen to other samples

How did you manage to fix it?


Uncaught Error: Error from Instagram: The access_token provided is invalid.

I've got 3 production sties that were using instafeed with out any issues. Until a couple of weeks ago when I now get 'Uncaught Error: Error from Instagram: The access_token provided is invalid.'
I haven't changed anything and my instagram clients are the same.
http://willpowerfitness.co.uk/
http://botanyblend.com/

I think Instagram changed their API on June 1. So until instafeed.js gets updated it probably won't work.

Yep same thing.. got this running on a few live sites. Any word on an update?

This is nuts. There was a fix for this in the issues, and even that seems to be resetting at the moment. Really could do with this getting fixed.

Follow the instructions from to create access token using the correct client id and it should fix the issue
https://www.instagram.com/developer/authentication/

This will not work when my app is still in Sandbox mode right? How long does this usually take? (I'm am working on a project from another developer who is not working at my firm anymore.. don't know his app_id en client_id and stuff.. So i had to create a new one)

^ @amandeepsinghvirdi 's tip works but you'll need to update the client settings so that https://elfsight.com/service/generate-instagram-access-token/  is included in as one of the redirect URLs.

Does anyone know if any requests still allow the client_id to authenticate? I have yet to find one that didn't auth error without an auth token.

I can't authenticate using client ID only either. We're only using this library to display a feed of tagged images without any login required, so it's not really possible for me to store an access token in a secure way.. I don't believe it's considered safe to just generate a token with my own user and paste this in the js-file..?

That's precisely what I did.

On Jul 6, 2016, at 12:29 AM, Martin<EMAIL_ADDRESS>wrote:
I can't authenticate using client ID only either. We're only using this library to display a feed of tagged images without any login required, so it's not really possible for me to store an access token in a secure way.. I don't believe it's considered safe to just generate a token with my own user and paste this in the js-file..?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes, I tried with the sandbox version with auth token as well, but I needed all pictures from a tag, which of course wouldn't work.
I'm not sure how the verification process works around this, but maybe it would be possible to create and verify a common user/client for use in instafeed.is? So that we all could use the same access token, but there's no content/images on the account?

We’d slam into rate limits pretty fast if any substantive group of people started using it. If memory serves, we now get 500 requests/hour, which is fine for modest sites, but easily reachable with “crowd sourcing”.

On Jul 6, 2016, at 11:29 AM, Martin<EMAIL_ADDRESS>wrote:
Yes, I tried with the sandbox version with auth token as well, but I needed all pictures from a tag, which of course wouldn't work.
I'm not sure how the verification process works around this, but maybe it would be possible to create and verify a common user/client for use in instafeed.is? So that we all could use the same access token, but there's no content/images on the account?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/stevenschobert/instafeed.js/issues/424#issuecomment-230862557, or mute the thread https://github.com/notifications/unsubscribe/ABpeiyaadcebdW3mboYpZZ99xeUIMBW5ks5qS_QYgaJpZM4I8jeF.

Ah, I see, that's too bad. Seems to me like instagram don't want us to show image feeds on other sites anymore.. Which also seems like a "shoot yourself in the foot" kind of thing to do...

That’s 100% their intention. This move was entirely calculated with the goal of quashing “unapproved” glimpses of the Ig user experience.

On Jul 6, 2016, at 11:38 AM, Martin<EMAIL_ADDRESS>wrote:
Ah, I see, that's too bad. Seems to me like instagram don't want us to show image feeds on other sites anymore.. Which also seems like a "shoot yourself in the foot" kind of thing to do...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/stevenschobert/instafeed.js/issues/424#issuecomment-230864876, or mute the thread https://github.com/notifications/unsubscribe/ABpeixXbw0AhFYAjQpGIbbKcJqdx7vDpks5qS_YcgaJpZM4I8jeF.


Less invasive versioning \ diffbase \ SyncDAO/VersionNoDAO/StoreAndForwadDAO

SyncDAO/VersionNoDAO to use generated "Versioned" classes
This change generates "versioned" classes with properties version_ and deleted_ for interal use with SyncDAO and VersionNoDAO. Currently, VersionNoDAO expects a versioned of (and does not automatically create one, but maybe it should?).
The value of this change is that client code outside of the SyncDAO can operate on classes that do not contain versioning information; this becomes an implementation detail hidden inside: (Sync DAO ==> ... VersionNoDAO => ... ==> ).

@jacksonic PTAL. I am not as happy with this as I thought I would be for two reasons:

Look at the stress test code. It feels like enough layers need to know about versioning that it is hardly worth keeping it out of the one layer that doesn't. Also, the developer now needs to keep track of which layers do and don't know about versioning. Maybe I just feel that way because what's being tested is the interface between versioned and unversioned objects; WDYT?
I have a use case that uses two layers of SyncDAO. I'm not sure whether this change is compatible with that use case. (Or at least, SyncDAO may need another change to detect that of is already versioned, and don't bother adapting client objects).

LMK what you think.

Manually merged to master: https://github.com/foam-framework/foam2/commit/5d61f913e0f9720ebb9f726d8633f47f3781a231


Themed Icon for Android 13+

Google introduced a new icon technique starting with Android 13 and higher called Themed Icons.

Hi @CrazyMarvin,
I would like to work this tak. Will add more details here on how I plan proceed soon.

I want to do this task ,please assign it to me @CrazyMarvin

@CrazyMarvin Please review the changes https://github.com/Crazy-Marvin/Morse/pull/117

Thank you very much for your help, @snaik20! 😘
@sachin2709, this is a similar issue: https://github.com/Crazy-Marvin/ToDont/issues/139. Would you like to work on this?


feature request: ZLint command line -version flag

I see that v3 was released recently. I just did an update from what I believe is v2 to v3. Is there an easy way to tell the update was successful and which version I am running?  I think a -version switch would be really useful.

:wave: Hi @weddi-eddy

I see that v3 was released recently.

Not quite yet, just a release candidate. We should have a final v3.0.0 release uploaded in the next few days.

I think a -version switch would be really useful.

This seems like a reasonable feature request for the zlint cli 👍

Not quite yet, just a release candidate. We should have a final v3.0.0 release uploaded in the next few days.

This has been published now: https://github.com/zmap/zlint/releases/tag/v3.0.0

Is there an easy way to tell the update was successful and which version I am running?

@weddi-eddy Until there's a -version flag the easiest way is probably to download the binary release for your platform from the release page's assets. E.g. https://github.com/zmap/zlint/releases/download/v3.0.0/zlint_3.0.0_Linux_x86_64.tar.gz for 64bit Linux. Running the zlint binary from the release archive will guarantee you're using the right version :-)

There's a new -version flag for both zlint and zlint-gtld-update as of c47eab4 that will appear in the next release.
In the meantime I noticed that the --help output from existing release binaries includes the release version. You can use that as a work-around for previous releases (I tested with both 3.0.0 and 3.1.0 but I believe this was implemented from the start of publishing binaries).


How to use

I want to use weave for merging client and server jar but there is no command? How to use. Help please, thanks.

Weave is now dead. Use stitch
https://maven.fabricmc.net/net/fabricmc/stitch/<IP_ADDRESS>/stitch-<IP_ADDRESS>-all.jar
And in command line, run
java -jar stitch-<IP_ADDRESS>-all.jar mergeJar <client-jar> <server-jar> <output>

see https://github.com/FabricMC/stitch/blob/57b0e7fe30e4d7768b26f610385434771e90aad3/src/main/java/net/fabricmc/stitch/commands/CommandMergeJar.java#L36

Or you can just clone https://github.com/FabricMC/yarn switch to the version you'd like and run ./gradlew mergeJars

Thank you!


Footer

Dev Summary
I redesigned footer according to the newer moc.

Test Plan

repro steps:

localhost:3000

##Moc
https://www.figma.com/file/KE2Sw6ob07agN25RmeNuFg/GiveCycle-wireframes?node-id=4076%3A12228
Resources

-MUI divider
https://mui.com/material-ui/api/divider/
-MUI grid
https://v3.mui.com/layout/grid/
https://www.youtube.com/watch?v=HCsFwwolXZw (MUI grid example)

@janechodance oh also, in order to test locally i needed to merge main and then run npm ci in both server and client. lmk if you need help with any of it


Can not Replace Enter or New Line

VSCode Version:1.0
OS Version:windows 10 (1511)

Steps to Reproduce:

open New File.

2.Ctrl+H

USE \n

4.can not find ENTER OR newline

I have noticed a number of similar regex issues, like not being able to select .* and others. I didn't open issues but it seems like the regex search feature needs some work.

Duplicate of https://github.com/Microsoft/vscode/issues/313


Add rack server test suit

Adding rack based application to the /test dir. Now there are two applications:

puma-test-app
rack-test-app

Both of them are tested upon the running the test suit.
Also I've refactored the build.sh and test.sh scripts so the TRAP instruction will work properly.
@mfojtik @soltysh PTAL

lgtm.


refactor: all front page

Codecov Report

Merging #38 into master will decrease coverage by 0.15%.
The diff coverage is 52%.

@@            Coverage Diff             @@
##           master      #38      +/-   ##
==========================================
- Coverage   39.91%   39.75%   -0.16%
==========================================
  Files           8        8
  Lines         228      244      +16
  Branches       38       41       +3
==========================================
+ Hits           91       97       +6
- Misses        137      145       +8
- Partials        0        2       +2


[MRG]: allow n_dimensional search lights

This allows using n-dimensional arrays with the search light, rather than being constrained to 3D array.
This is the first step to before being able to have a rolling window with more than 1 time point.

Current coverage is 86.80% (diff: 100%)

Merging #3481 into master will decrease coverage by <.01%

@@             master      #3481   diff @@
==========================================
  Files           339        339
  Lines         59052      59065    +13
  Methods           0          0
  Messages          0          0
  Branches       9008       9008
==========================================
+ Hits          51263      51274    +11
- Misses         5070       5071     +1
- Partials       2719       2720     +1

Powered by Codecov. Last update 5d2a649...b7dc41b

ready for review

besides LGTM

thx @kingjr

thanks for the reviews!


Event handler lifetime shouldn't be static

https://github.com/1c3t3a/rust-socketio/blob/095db3a0e98cee5254ef32609222e036efb217d0/socketio/src/client/builder.rs#L169-L172
Currently, when building client with event handler, the lifetime of the event handler is required to be static, but I think it should be as long as the client instance.
I want to pass some objects, queue for example, to handlers without transferring ownership. It seems impossible at the moment.
This is the code that I want to use
{
    let (tx, rx) = channel::<String>();
    let callback = |payload: Payload, socket: RawClient| {
        tx.send("...");
    };
    let callback2 = |payload: Payload, socket: RawClient| {
        tx.send("...");
    };
    let socket = ClientBuilder::new(url)
        .on("evt1", callback)
        .on("evt2", callback2)
        .connect()
        .expect("Connection failed");
}

I just found that crossbeam_channel works for me since the sender can be cloned. But I still think the handler should have shorter lifetime.

Fair point! Didn't think about this in the implementation. I am very happy to accept a PR for this!

@doraeric In your example, you are missing move for tx. Once you do that it should work.
{
    let (tx, rx) = channel::<String>();
    let tx_clone = tx.clone();
    let callback = move |payload: Payload, socket: RawClient| {
        tx_clone.send("...");
    };
    let callback2 = move |payload: Payload, socket: RawClient| {
        tx.send("...");
    };
    let socket = ClientBuilder::new(url)
        .on("evt1", callback)
        .on("evt2", callback2)
        .connect()
        .expect("Connection failed");
}


errorrr

Data directory not writable by web server: data/

Data directory not writable by web server: data/

Configure config/Database.ini file according to your requirements,

The following PHP extensions are required: pdo_sqlite
Only keeping mysql. Deleted from config/Database.ini but still getting this.

Leave config/Database.ini as is,  create a folder named data.  That's all.


feat: Update_product.json-1503

Related Issue
Closes #1503

Description

Screenshots

Checklist

[X ] My code adheres to the established style guidelines of the project.
[X ] I have included comments in areas that may be difficult to understand.
[X ] My changes have not introduced any new warnings.
[X ] I have conducted a self-review of my code.

@lakshmishreea122003 the title can have at max 2 words

I have named all my PRs in the same manner. Then why is this PR title wrong.


Suport clang version 3.8

Description
Check CLANG version, and according to the version,
set the correct parameters. fix for #1072
Status
READY
Requires Backporting
I don't think so, since I don't think new linux versions will use legacy branches
NO
Migrations
NO
Additional comments
Any additional information that could be of interest
Todos

[ ] Tests
[ ] Documentation
[ ] Changelog updated
[ ] Backported

Steps to test or reproduce
Outline the steps to test or reproduce the PR here.

Hi @mazimkhan I fixed according to your comments

Another thing I noticed is that we do not actually have any Asan tests with Clang in all.sh. Might be worth adding some as part of this PR.

@andresag01 do you mean (1) as build-only test (just to make sure our CMakeLists.txt has the correct options) or also actually running the test suites, and if so, would it be (2) in addition or (3) in replacement to gcc-asan? I think we really want at least (1) to make sure we avoid regressions on that PR. I don't think we want (3) because all.sh already takes a long time to run and I don't think it would add enough value to justify making it even longer. I'm not sure about (2) as I'm not clear on whether Clang still gets the new sanitizing goodies before GCC or if they're fully at the same point now.

@mpg: Thanks for the comments. I think ideally, we should do build and run tests. However, given your comments on how long all.sh already takes, then I think we should just include a build test to ensure that we at least are able to compile with it.

Hi all can we fix and merge this PR asap. It is required for CI. buildbot CI uses clang 3.4 and gcc 4.8 for ASAN tests. For migration to Oulu latest versions of debian and Freebsd are used and getting old versions of clang and gcc are just too hard. With this fix latest versions of clang and gcc work fine in Oulu CI.
Hence a quick resolution on this would greatly help.

Clang versions dates

08 Mar 2016
3.8.0
download
release notes
docs

05 Jan 2016
3.7.1
download
release notes
docs

01 Sep 2015
3.7.0
download
release notes
docs

16 Jul 2015
3.6.2
download
release notes
docs

26 May 2015
3.6.1
download
release notes
docs

27 Feb 2015
3.6.0
download
release notes
docs

02 Apr 2015
3.5.2
download
release notes
docs

20 Jan 2015
3.5.1
download
release notes
docs

03 Sep 2014
3.5.0
download
release notes
docs

Clang 3.6 that supports the new cmd flag version was launched in 27 Feb 2015. We can make a call to drop support for older versions and just change the flag to new format -fno-sanitize-recover=all.

I fully agree with Azim's suggestion. IMO these build options are only for developers, it's perfectly acceptable to have stronger requirements for them than for user options. Also, the few (if any) people who need it can always use Make and pass the exact CFLAGS they need on the command line.

Unfortunately Andres uses old version of clang that does not support this new option format.
--> clang --version
Apple LLVM version 7.0.0 (clang-700.0.72)
Target: x86_64-apple-darwin14.4.0
Thread model: posix

@andresag01 do you run asan build on your mac. I know you have an Ubuntu m/c as well.

@mazimkhan: Most of my development work is on Ubuntu 16.04, but sometimes (very rarely) I run Asan builds.

By the way on Mac, you can easily install gcc from homebrew, which supports Asan (or at least that was the case 2 years ago, but hopefully it didn't regress). (Installing clang with Asan from homebrew used to be harder, you had to compile it and sometimes it failed - I have no idea how it is now.)
(This probably doesn't matter since you're doing most of your work on Ubuntu and rarely use Asan, but if you happen to need it on your Mac some time...)

I modified the file to support only new Clang versions

Since it does seem to be desirable to still support older versions (for people running OSX or CentOS), why not do the check dynamically, but using clang --help?
$ clang --version | head -n1; clang --help |grep fno-sanitize-recover
Debian clang version 3.5.0-10 (tags/RELEASE_350/final) (based on LLVM 3.5.0)
  -fno-sanitize-recover   Disable sanitizer check recovery

vs
clang --version | head -n1; clang --help |grep fno-sanitize-recover
clang version 3.8.0-2ubuntu4 (tags/RELEASE_380/final)
  -fno-sanitize-recover=<value>

I'm okay with approving this for 'development', and '2.7'. I also don't see the need to do the clang --help workaround.
@mazimkhan - do we need this fix for 2.1 for the sake of the CI? If we do, I guess we do need the clang --help workaround.
@andresag01 / @hanno-arm - can you both please approve this PR, if you feel you can?

@mazimkhan - do we need this fix for 2.1 for the sake of the CI? If we do, I guess we do need the clang --help workaround.

@sbutcher-arm We do need this fix for 2.1 as well. But I don't understand why we need clang --help workaround for that.

@mazimkhan - do we need this fix for 2.1 for the sake of the CI? If we do, I guess we do need the clang --help workaround.

@sbutcher-arm  I guess you mean to say we can't remove support for Clang < 3.6 in 2.1. In that case yes we would need the clang --help workaround. If we do it in 2.1 then it may well be done for 2.7. It is up to you to decide.

Marking this as 'needs work' as we need back ports with the workaround.

@mazimkhan - Your review is stale. Can you please re-review, and approve again?


Limited viewers of a story/archived story but no pagination data

Please follow the guide below

Issues submitted without this template format will be ignored.
You will be asked some questions and requested to provide some information, please read them carefully and answer completely.
Do not post screenshots of error messages or code.
Put an x into all the boxes [ ] relevant to your issue (like so [x] no spaces).
Use the Preview tab to see how your issue will actually look like.
Issues about reverse engineering is out of scope and will be closed without response.

Before submitting an issue, make sure you have:

[x] Updated to the lastest version v1.4.0
[x] Read the README and docs
[x] Searched the bugtracker for similar issues including closed ones
[x] Reviewed the sample code in tests and examples

Which client are you using?

[x] app (instagram_private_api/)
[ ] web (instagram_web_api/)

Purpose of your issue?

[ ] Bug report (encountered problems/errors)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

The following sections requests more details for particular types of issues, you can remove any section (the contents between the triple ---) not applicable to your issue.

Describe your issue
I'm trying to retrieve some data about stories or archived ones. The number of viewers of a story  is correctly displayed but the number of users in the "viewers" list seems to be limited at 5. Furthermore, i can't find any data about pagination (max_ids or similar) except for "viewer_cursor": "5".
What is this value? How can i get the full viewers list?
Thank you

Please provide a code snippet. I can't even tell which function you're referring to.
At this point, there are well over 100 endpoints, and I cannot be expected to remember enough details for each and every one of them to know easily what you're talking about.

Sorry! @ping
I tried different methods like:
stories = api.user_story_feed(user_id)
or
archived_index = api.stories_archive()
if archived_index["items"]:
item_ids = [a['id'] for a in archived_index['items']]
archived_stories = api.reels_media(item_ids)
or
the reels_tray(**kwargs) method
The returned json contains only 5 viewers.
Furthermore, i noticed that when using feed_only_me(**kwargs), the key "likers" is missing even tough the "like_count" says "18" (right number of likes). When the number of likes is small, eg "2", the key is there. This happens when i retrieve not-archived medias too.
Thank you

I just added the story_viewers() endpoint in 3a2befe8906dedd34cf1ef145d4b076dfe1765b9.
This library by default doesn't modify the response from IG. If something is missing, it simply wasn't returned by IG.

thank you very much

thank you very much. you're doing an amazing work on this API.

@ping please see this issue discussing the inability of story_viewers() to paginate the voters list with the max_id


 I wonder if it is possible to somehow retrieve all fields of my form that are dirty?
For example if this is my form:
const form = ref<Form>({
        name: 'abc', <----- dirty
        email: '',
        skills: [
            {
                skillName: '',
                level: 0
            },
            {
                skillName: 'SuperPower', <----- dirty
                level: 0
            }
        ],
    })

I have 2 fields that are dirty here:

name
skills[1].skillName

It would be nice if regle had some function like getDirtyFields that would return me only those fields that are dirty like this:
{
    name: 'abc',
    skills: [
        null,   <--- this is null because skill[0] does not have ANY dirty fields

        {
            skillName: 'SuperPower'
        }
    ]
}

Yeah that would be totally possible to add!

+1

@martinszeltins @kyong9793 , should it extract only dirty fields that are valid ? Should it perform a validate before getting the values?

It's available in 0.2.3 under r$.$extractDirtyFields.
There is one parameter filterNullishValues that defaults to true. That will only return defined values.
Use r$.$extractDirtyFields(false) if you want don't want any filtering.
I need to add doc for it

To note also, this method is available at every nested level of your state.
In your exemple you can do r$.$fields.skills.$extractDirtyFields()

https://regle.vercel.app/core-concepts/validation-properties.html#extractdirtyfields


🐛 BUG: publish fails when attempting to publish a Worker to routes that already exist

What version of Wrangler are you using?
2.1.3
What operating system are you using?
Mac
Describe the Bug
When attempting to publish a Worker to routes that already exist, possibly on a different worker, or have been modified in the dashboard, the publish fails with:
✘ [ERROR] A request to the Cloudflare API (/accounts//workers/scripts//routes) failed.
workers.api.error.unknown [code: 10013]
I can understand that there may be no one right answer to resolve this issue, but the error message could be improved and could include steps for resolution of common scenarios, e.g: manually remove the routes in the dashboard (or via API I suppose) that are needed for the publish to succeed.

Hey @jsvsco - any chance you could send over a minimal reproduction of this?

Create a new worker in the Cloudflare Workers dashboard.
Assign it a route on a custom domain.
Initialize a new worker locally in Wrangler with a different name from the worker in #1.
Add the same route as #2 to its wrangler.toml.
Attempt to publish the worker with Wrangler and you should receive the error above.

Ah custom domains get dealt with separately than other routes.
So perhaps we need to implement something similar to https://github.com/cloudflare/wrangler2/commit/b44cc26546e4b625870ba88b292da548b6a340c0#diff-d79823fe1e90cdc27158bbeaab5f58954e81375f84f4403ca124318c7bccd54fR673-R686 for custom domains.

I didn't think about it, but it makes sense that custom domain is important. I updated the original report.


[ MB-16350 ] Adding DOSFSC and DDSFSC test data

Jira ticket
Summary

[x] Adding DOSFSC to the MTO Service Item Factory
[x] Adding DDSFSC to the MTO Service Item Factory

Verification Steps for Reviewers
These are to be checked by a reviewer.
Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

More instructions incoming...

Messages

:book:
:link: MB-16350

  Generated by :no_entry_sign: dangerJS against a59c48cbabec8065bba57c3083a82c5ccc832382

@michaelvaldes-truss This should be ready to review and is tied to both our tickets MB-16350 and MB-16395 as a setup for the new service items.
Let's review this separately from the followup work I'll be doing to complete my ticket.


Errors on Android RN40

RN40.
When following instructions and usingreact-native link react-native-facebook-keyframes, it is adding:
import com.underscope.react.keyframes.RNFacebookKeyframesPackage; rather than:
import com.underscope.keyframes.RNFacebookKeyframesPackage; to MainApplication.java.
However, even when I fix that manually I am still getting errors, please find attached.

Good catch! I'm trying to figure out this one #8 then I'll publish a new npm version.

Hey @eranbes I've just published 1.0.1 including the fix. Thanks for reporting this!


Store coordinate spaces

Store coordinate spaces in skeleton files in the same manner as https://github.com/io7m/smf/issues/2.

The mathematics in the package assumes a right-handed coordinate system.
It's possible that the specification could require explicit storage of coordinate space information, but the implementation will only support right-handed -Z forward.

This is something the Blender plugin will need to be able to produce, and the skeleton compiler able to evaluate.

Additionally: What does a program do if it loads a compiled skeleton and finds out that the coordinate system doesn't match what it requires?


Restore system-test fixes lost in merge.

See: https://travis-ci.org/GoogleCloudPlatform/gcloud-python/builds/70509954

@dhermes I intend to merge this one ASAP, as we have broken tests on master.

LGTM


⚠ Bofrost.lu has degraded performance

In ed2fff1, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1578 ms

Resolved: Bofrost.lu performance has improved in f1dd6fe after 13 minutes.


Add argument check for aggregation functions with single argument

Fix #11503
This can help prevent unexpected results when calling single argument aggregation with multiple arguments

Codecov Report

Merging #11556 (bea6abe) into master (9c3a6e7) will decrease coverage by 48.64%.
Report is 6 commits behind head on master.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #11556       +/-   ##
=============================================
- Coverage     63.11%   14.47%   -48.64%
+ Complexity     1108      201      -907
=============================================
  Files          2325     2325
  Lines        124749   124829       +80
  Branches      19048    19050        +2
=============================================
- Hits          78733    18075    -60658
- Misses        40410   105224    +64814
+ Partials       5606     1530     -4076

Flag
Coverage Δ

integration
?

integration1
?

integration2
?

java-11
14.47% <0.00%> (-48.56%)
:arrow_down:

java-17
?

java-20
?

temurin
14.47% <0.00%> (-48.64%)
:arrow_down:

unittests
14.47% <0.00%> (-48.63%)
:arrow_down:

unittests1
?

unittests2
14.47% <0.00%> (-0.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

...gregation/function/AggregationFunctionFactory.java
0.00% <0.00%> (-81.36%)
:arrow_down:

...y/aggregation/function/AvgAggregationFunction.java
0.00% <0.00%> (-80.46%)
:arrow_down:

...aggregation/function/AvgMVAggregationFunction.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...gation/function/BooleanAndAggregationFunction.java
0.00% <0.00%> (-60.00%)
:arrow_down:

...egation/function/BooleanOrAggregationFunction.java
0.00% <0.00%> (-60.00%)
:arrow_down:

...aggregation/function/CountAggregationFunction.java
0.00% <0.00%> (-68.75%)
:arrow_down:

...gregation/function/CountMVAggregationFunction.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ation/function/DistinctAvgAggregationFunction.java
0.00% <0.00%> (-86.67%)
:arrow_down:

...ion/function/DistinctAvgMVAggregationFunction.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ion/function/DistinctCountAggregationFunction.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 18 more

... and 1474 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


whitespace issues in helptext

Global Flags do not have trailing whitespace but Flags always does, meaning "Error: xxx" will have an empty line between it and the help text if the command has no persistent flags in its upstream, but no empty line if it does.

fixed in #19


Adjust the sleep time before tearing down e2e clusters and add a rough formula

…for determining how long the sleep should be.

LGTM


how to create []enums

emmmm

In detail please

what i mean is collectionFormat in Struct.
Default value is csv. But when i try to use multi and also in query or formdata, it still not workk.

In detail please


Update scalapb-json4s to 0.11.0

Updates com.thesamet.scalapb:scalapb-json4s from 0.10.3 to 0.11.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.thesamet.scalapb", artifactId = "scalapb-json4s" } ]

labels: library-update, semver-minor

This depends on scalapb 0.11


Add lightweight generics to foundation objects

[ ] anagram
[ ] bob
[ ] etl
[ ] grade-school
[ ] hamming
[ ] hello-world
[ ] leap
[ ] nucleotide-count
[ ] perfect-numbers
[ ] phone-number
[ ] robot-name
[ ] space-age
[ ] word-count

Xcode 7 added a way to add type information to collection classes like NSArray, NSSet and NSDictionary. ObjectiveC is historically a non typed collection language but with the addition of lightweight generics a test case can specify the return to be typed so instead of just requiring the return type to be NSDictionary, now we are able to ask for a NSDictionary<NSString *, NSNumber *> where the key is a NSString and the value is NSNumber.
https://developer.apple.com/library/ios/documentation/Swift/Conceptual/BuildingCocoaApps/InteractingWithObjective-CAPIs.html#//apple_ref/doc/uid/TP40014216-CH4-ID173

Would you also describe exactly what you mean by this? Is it something that definitely has to be added? If not, how would you know whether or not it needs it? How do you do it?
That would let other people jump in and fix it as well.

Good call. Perhaps this is something that we don't do to the first couple of problems since it does add some complexity. I updated the description above.

Excellent. If/when you fix one of them, would you also link to the change so people can go see an example?


Add default header patterns to SimpleKafkaHeaderMapper

Fixes https://github.com/spring-projects/spring-kafka/issues/1310

@artembilan Fixed problems reported by checkstyle... can you please explain what you mean with "Please, add your to affected classes." I checked the CONTRIBUTING.adoc but found nothing to clarify. Thank you

@garyrussell , WDYT?

I agree.
@jdarc he meant "add your name" (to the @author list).

Since we all agree, it looks like we need to start new 2.4 version and merge all the related changes over there already.
The Dec 6th is not around the corner though...

Yeah - I just moved the date - the 2.4.0 kafka-clients has slipped to Nov' 30.
Unfortunately, they don't currently publish snapshots so we can't add a CI build, but I will push my 2.4 branch to a PR against a temporary 2.4 branch for you to review.
I have pushed a 24WIP branch and will merge this there.

Merged as 5091fdc0ecad21dba3788bbea8000a8e3e1a2958 on 24WIP.
Thanks for the contribution!!


ng-model on datepicker does not work

Using the ng-model attribute on the datepicker directive instead of the surrounding tag causes the datepicker to not bind to the model
I have created a plunkr where the the first datepicker has the ng-model attribute on the datepicker and the second where it has it on the surrounding div
http://plnkr.co/edit/EFYBUuDt23U3wHvsD2rG
Note that this issue does not occur in version 0.4.0

I am using bootstrap timepicker in my code and setting value on page load using ng-model

And loading some html on runtime. Its working fine for me when i set value on page
But when i try to update the value in ng-model it will not reflecting on view side but value is updated in ng-model
Can you help in this.

It seems this issue has been resolved, i've updated the plunker from @garethdjames.
http://plnkr.co/edit/2hLMd4KLbAMWxp2EtHXI

in a span class ng-model working or not .


Add release workflow

Adding a simple GitHub workflow which does create a GitHub release (with auto-generated changelog) and publish the NPM package on push of version tags.
With this workflow the process would then look as follows:

Run npm version ... locally
Push to GitHub, e.g. git push && git push --tags
Let the workflow do the rest (and maybe check / adjust the release notes)

I've added safe-publish-latest as a safety measure, which makes sure the publish process will only run if the version is a valid "latest" version.
@gustavohenke This would allow me to create releases as well. Feel free to close this pull request in case you prefer to retain the sole person to do so. Otherwise, can you please create a "NPM Automation Access Token" and store it under a secret called NPM_TOKEN in this repo. Thanks!

Thanks, the idea is good!
...I have write access to a couple of other popular packages though, so I'd like to have this first: github/roadmap#557

Oh, of course! I'm surprised that this is still not solved. It's scheduled for Q4 so let's hope we'll have it by the end of the year... 🎉

Granular access tokens are now available (beta): https://github.blog/changelog/2022-12-06-limit-scope-of-npm-tokens-with-the-new-granular-access-tokens/
Unfortunately, the max expiration date is limited to 365 days which seems to be a bit of a blocker for me when using it in workflows...
Maybe this will change once released under general availability: https://github.com/github/roadmap/issues/556

I've gotten access to the granular tokens and added one under NPM_TOKEN.

That's a great "End of Year" gift 🎉  Thank you!

We can get started with this limit 🙂 one year is enough time to see a lot of changes.

Hah, true 😃 Let's see how things evolve!

Didn't quite work...
https://github.com/open-cli-tools/concurrently/actions/runs/4548335823/jobs/8019259304
https://github.com/open-cli-tools/concurrently/actions/runs/4548335823/jobs/8019259458

Didn't quite work... https://github.com/open-cli-tools/concurrently/actions/runs/4548335823/jobs/8019259304 https://github.com/open-cli-tools/concurrently/actions/runs/4548335823/jobs/8019259458

Hopefully fixed with #408 🤞

Didn't quite work... https://github.com/open-cli-tools/concurrently/actions/runs/4548335823/jobs/8019259304 https://github.com/open-cli-tools/concurrently/actions/runs/4548335823/jobs/8019259458

Hopefully fixed with #408 🤞

Works now 🎉 https://github.com/open-cli-tools/concurrently/actions/runs/4549032653


[Fix-14503][Worker] Fix the problem of subprocess cannot be killed

Purpose of the pull request

close: #14503

Brief change log

doKill() must be running before cancelApplication() so that pstree could get all the sub process id and kill them

Verify this pull request

Hi, @Radeity could you please help review this

This is strategy, when we kill the shell task, we will use process.destroy(); to cancel the process.

Codecov Report

Merging #14504 (afabd3a) into dev (c3c2dda) will increase coverage by 0.00%.
The diff coverage is 58.00%.

:exclamation: Current head afabd3a differs from pull request most recent head 5acc38e. Consider uploading reports for the commit 5acc38e to get more accurate results

@@            Coverage Diff            @@
##                dev   #14504   +/-   ##
=========================================
  Coverage     38.72%   38.73%
+ Complexity     4577     4574    -3
=========================================
  Files          1237     1237
  Lines         43496    43487    -9
  Branches       4817     4790   -27
=========================================
- Hits          16845    16843    -2
+ Misses        24781    24776    -5
+ Partials       1870     1868    -2

Impacted Files
Coverage Δ

...scheduler/api/controller/DataSourceController.java
36.36% <0.00%> (-2.35%)
:arrow_down:

...e/dolphinscheduler/common/constants/Constants.java
75.00% <ø> (ø)

...er/plugin/task/api/utils/DataQualityConstants.java
0.00% <ø> (ø)

...nscheduler/plugin/task/api/utils/ProcessUtils.java
0.00% <0.00%> (ø)

...rver/worker/processor/WorkerTaskKillProcessor.java
0.00% <0.00%> (ø)

...eduler/api/service/impl/DataSourceServiceImpl.java
50.83% <58.33%> (+2.36%)
:arrow_up:

.../org/apache/dolphinscheduler/api/enums/Status.java
100.00% <100.00%> (ø)

...ler/data/quality/flow/batch/reader/JdbcReader.java
95.00% <100.00%> (ø)

...ler/data/quality/flow/batch/writer/JdbcWriter.java
85.00% <100.00%> (ø)

...cheduler/plugin/task/dq/utils/RuleParserUtils.java
87.03% <100.00%> (+0.04%)
:arrow_up:

... and 37 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I cannot reproduce this on my mac.

How to reproduce

create a shell task with sleep 300
kill the task
sleep 300 is still running

Hi, @zhongjiajie @Radeity Could you guys please take a time to reproduce this bug ?
On my mac, the task is killed at 14:48

but sleep 300 is still running at 14:49

I cannot reproduce this on my mac.

How to reproduce

create a shell task with sleep 300
kill the task
sleep 300 is still running

Hi, @zhongjiajie @Radeity @SbloodyS Could you guys please take a time to see if this bug can be reproduced?
On my mac, the task is killed at 14:48
but sleep 300 is still running at 14:49

Hi, @rickchengx , I can reproduce this bug on my linux server.

@ruanwenjun I think we could fix this bug first through this PR. And we could improve the kill logic to use the cancelApplication() in each task plugin. It looks like a lot of work. WDYT


RowKey not populated in Instances table per documentation

The section about the Instances table states the following:

The Instances table is another Azure Storage table that contains the statuses of all orchestration and entity instances within a task hub. As instances are created, new rows are added to this table. The partition key of this table is the orchestration instance ID or entity key and the row key is a fixed constant.

However, when a Durable Functions orchestration populates the Instances table, I am seeing the RowKey as empty/null and not a "fixed constant" as the documentation describes. It seems that either the documentation is incorrect, or a bug exists within Durable Functions that is not populating this RowKey value as intended.

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

ID: afd094d0-fb49-e206-2f99-c3fec27bb20f
Version Independent ID: 42fd1479-2478-3137-b43e-da13a4c10a48
Content: Performance and scale in Durable Functions - Azure
Content Source: articles/azure-functions/durable/durable-functions-perf-and-scale.md
Service: azure-functions
GitHub Login: @cgillum
Microsoft Alias: azfuncdf

@carfarmer We are checking this and we will get back to you.

@carfarmer Indeed, this wording is misleading. The RowKey is always expected to be empty. We will fix this.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_9d2c385b-c660-4040-97ea-4fc492dd63eb/pull/1 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target_9d2c385b-c660-4040-97ea-4fc492dd63eb.git

Docs Build status updates of commit dcc04f3:
:clock10: Pending: waiting for processors (5 builds ahead of you)
:warning: Docs Build is busy, currently there are 5 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit dcc04f3:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-02-04-19-56-29 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-02-04-19-56-29

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


Add pre-flight checks for framework status

In order for the tests to run the frameworks must have particular
statuses. This change adds pre-flight checks to verify them and tell the
user which ones need to be changed.

Works like a charm :+1:


Benchmark, security, comparison

Are there any benchmarks which compare redbird with apache and nginx for example by serving static files. People use nginx with express because it serves static files much faster. According to them nginx prevents DDOS attacks as well, and increases security. Ofc. I never saw any concrete examples. I think they just use it because everyone else do so.
off:
In my case I cannot find a real purpose to use a reverse proxy with nodejs. I have a home server with some nodejs services. Some of them run express and have HTTP interface. These are mostly for personal usage on the local network, but I think I'll add a dyndns and publish a few, for example my blog. It does not really need performance by static files, etc. Why do I need a reverse proxy to do this? Wouldn't express + fail2ban + port forwarding on the router be enough?

Running nodejs on port 80 requires root which can be a security concern.
Additionally handling the complexity of SSL may be something to hand off to a reverse proxy.

+1 for performance comparison to nginx. Please Please!

Redbird is similar to Nginx in its purpose, though Nginx has a much fuller feature set and is much faster (it's written in C, what do you expect the benchmark to show?).
One thing I like about Redbird is that routes can be modified on the fly with no downtime. Nginx servers must be restarted to modify routes. It's also in NodeJS, and I know NodeJS well and Nginx config language not very well, so for me it's highly configurable.
My specific usecase is that I have a lot of different websites running on a single EC2 server. Redbird is very very lightweight (t2.micro doesn't have much space, and I guess nginx would be alright, but shrug), and I can do cool things with it in NodeJS.

I think that for the sake of comparison it would be nice to provide some benchmarks. At least we would be able to see how much performance we are loosing by not using nginx and helping the user to understand if redbird is a suitable alternative or not. I do not expect it to be terribly un-performance though, since at the end is using nodejs libuv which is supposed to be quite efficient and becomes more efficient in every release. Then you have the v8 overhead which of course will introduce latencies that are more difficult to optimize. I will try to make some benchmarks when I have some spare time.

@manast Thanks!

I love node so proxying with it would be bomb! But it would help me make the switch more comfortable if I knew what loses I would be taking.
@manast thansk

I also wanted to know what performance hit I'd take when using redbird, and did some benchmarks myself using httperf.
First I've started a simple hello world server, and clustered it using pm2 start ./test.js -i max to use all cores.
Then I've benchmarked it using httperf --server myserver.com --port myport --num-conn 1000 and it was able to do around 2800 requests per second.
Then I've done the same through nginx, and it was able to process around 1800 req/s.
Then I've done the same through redbird, and it was only able to process around 800 req/s.
So.. the performance hit is pretty big compared to nginx. But I'm no expert in bencharking, so take those results not for serious. Other benchmarks might result in something completely different.
Anyways, I won't use redbird for anything that is under heavy load. But for my private stuff, it's really cool and the easy configuration is absolutely worth it.

@VanCoding run the same test but setting the global http agent to keep alive. There is more info regarding performance in this thread: https://github.com/nodejitsu/node-http-proxy/issues/1058
Redbird should perform almost the same as http-proxy-

@manast I'm not sure if I understood that correctly. I've removed the comments here: https://github.com/OptimalBits/redbird/blob/master/lib/proxy.js#L81
This improved the requests per second to 1200 from 800.

and are you using cluster? since nginx is threaded in order to compare in the same terms you need to enable cluster.

oh well, no I didn't. I thought it was on by default.
Turning it on makes it only slower, though. I've tried with redbirds cluster option and also tried running the proxy with pm2's -i max, but in both cases, it had only half the requests per second as before (~600). Running a single process thoug gave me 1200 req/s again.
But reading the readme, I've found out I can also disable loggin. This brought me another 300 req/s. So the distance isn't that big to nginx anymore.
But it's really strange that clustering doesn't speed it up. The hello world server is clustered from the beginning and it gives me 4x the requests per second compared to it running in a single process.
Maybe something's wrong with my VM, I don't know. I really wouldn't give much about my tests...
I'm looking forward to your benchmarks.

@VanCoding  I think you should use less cores. The benchmark and the os requires at least a single core I think. I am not sure about what pm2 does by max cores, so maybe it uses n-1 by default. Try with the exact numbers, maybe it changes the speeds.

Running nodejs on port 80 requires root which can be a security concern.
Additionally handling the complexity of SSL may be something to hand off to a reverse proxy.

Or it simply requires configuring the OS properly which still requires the dev to have root but node doesn't need to run as root.

Would be nice to have a graph, but thanks @VanCoding for the results, those would do for me for now.

@manast when you get the chance, would love to see how v1.0.2 compares to NGINX without logging turned on.


implementation, test case, doc for --check-new option

I'm using crypt on an underlying Amazon Cloud drive, wanting to keep most-recent backup copies of some local directory trees.  Experimenting with the options I saw that a second or later copy operation only picks local files to copy if they are new (never before copied up) or the size is different. I understand this is because ACD does not allow setting the modification time and so it is not checked, and crypt does not support comparing the file hashes.
I wrote a powershell script to extract file names from the output of cryptcheck and for example on a local tree with about 110,000 files it found 300 more files than the copy operation with existing options.  I extended this script to generate a copy command for each file, and it did work but it was slow and made poor use of the available bandwidth due to the overheads of running all those separate commands.
I starting looking at the rclone code for ways that I might improve on this.  I decided that the simplest way to pick out the files I want re-copied is to compare the local timestamp with the remote timestamp: when the local timestamp is more recent it means the local file was altered after its last "backup".
I implemented this as a new option, --check-newer.  The impact on the code is pretty small.  I did update the doc and, while my most extensive testing is using a powershell script, I also was able to work up a test case that exercises the changed logic.
I suppose there may be "better" ways to deal with the underlying concern, such as maybe hooking in the re-hashing used in cryptcheck to the equal function used to decide if a file needs to be copied from the src to the dest.  This being my first-ever exposure to golang and my second-ever attemtp to make a contribution on github I decided to stick with the simplest change that I thought would serve.

This appears at first glance very similar to the -u, --update flag.  Does that flag work for your use case?
If not can you describe how the semantics of --check-new and --update differ?
Thanks
Nick

D'OH!  On re-testing but focusing on --update, I see that it does seem to produce the same result as what I just did with --check-newer.

No worries, rclone has a lot of options!


changing data output format to array of dictioaries as requested

Description
Changed output to array of dictionaries(json objects) as requested by web-backend.
for raw.py sites.py and villages.py
[line 13] output = names_codes.to_json(orient="index") >>> output = names_codes.to_json(orient="records")
No other changes

[ ] This is a bugfix.
[ ] This is a new feature.
[x] This is part of a feature.

Organization

[ ] Have you linked this pull request to a Trello card?

Review

[x] Have you added two reviewers to this pull request?
[ ] Is this a work in process?
[x] Is this ready to be reviewed?

No big issues. Thank you for your great job!


export: Set UserMessage.flags.is_private properly in build_usermessage

I think this isn't an active bug, since we don't have private messages to import yet, but we should set the upcoming UserMessage.flags.is_private (#9753) to True in build_usermessage whenever importing a private message.  So I think the near-term task is probably to tweak build_usermessage to have an explicit keyword argument is_private_message=False, and pass the argument where we call it.
Otherwise, folks who import from Slack will have the "is:private" narrow missing some messages that belong there.
@rheaparekh FYI.

This was fixed in 78f6e3ac7dca241860d83ca77461dac5a2dd588d.


Robot State Machine - Add file reception state

Add specific state to do file reception in order to avoid some of the bugs encountered

[ ] file reception must be in the global state machine
[ ] it must be accessible only when charging
[ ] it must be accessible only when battery is > xx%
[ ] it should show a file reception image indicating what is happening
[ ] it should show a file received "notification" when transfer is over
[ ] if the above images are not available, it must display a neutral face

Related PR:

#1067


ihaskell install fails on macOS

Issue description
ihaskell build fails completely on macOS Sierra.
It failed on ghc-parser-<IP_ADDRESS> specifically.
setupCompileFlags: -package-db=/private/var/folders/by/mw_ml3rx2hg25s45hc_j5_fm0000gn/T/nix-build-ghc-parser-<IP_ADDRESS>.drv-0/package.conf.d -j4 -threaded
[1 of 1] Compiling Main             ( Setup.hs, /private/var/folders/by/mw_ml3rx2hg25s45hc_j5_fm0000gn/T/nix-build-ghc-parser-<IP_ADDRESS>.drv-0/Main.o )

Setup.hs:2:1: warning: [-Wdeprecations]
    Module ‘System.Cmd’ is deprecated: Use "System.Process" instead
Linking Setup ...
configuring
configureFlags: --verbose --prefix=/nix/store/69q02xrmzfrcxqxvjxqkhhy5fgn7asxr-ghc-parser-<IP_ADDRESS> --libdir=$prefix/lib/$compiler --libsubdir=$pkgid --with-gcc=clang --package-db=/private/var/folders/by/mw_ml3rx2hg25s45hc_j5_fm0000gn/T/nix-build-ghc-parser-<IP_ADDRESS>.drv-0/package.conf.d --ghc-option=-optl=-Wl,-headerpad_max_install_names --disable-split-objs --disable-library-profiling --disable-profiling --enable-shared --enable-library-vanilla --enable-executable-dynamic --enable-tests
unused rules: 20
shift/reduce conflicts:  42
reduce/reduce conflicts: 1
unused rules: 18
shift/reduce conflicts:  46
reduce/reduce conflicts: 1
unused rules: 18
shift/reduce conflicts:  46
reduce/reduce conflicts: 1
Configuring ghc-parser-<IP_ADDRESS>...
Setup: Encountered missing dependencies:
base >=4.6 && <4.9, ghc >=7.6 && <7.11
builder for ‘/nix/store/f4xvwx1hf6aibf0iq0mk2nnv8ikfip5x-ghc-parser-<IP_ADDRESS>.drv’ failed with exit code 1
cannot build derivation ‘/nix/store/papmmm0nb2mamyz71lzq54l701vh452g-ghc-8.0.1-with-packages.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/mkkr6vb5l58xqi18s4qic5p9i76zbzx6-ihaskell-<IP_ADDRESS>.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/32bzi39iqcjgrkkqvcwmw6bq68wvd60q-ihaskell-with-packages.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/32bzi39iqcjgrkkqvcwmw6bq68wvd60q-ihaskell-with-packages.drv’ failed

Steps to reproduce
nix-env -iA nixpkgs.ihaskell
Technical details

System: macOS Sierra 10.12.1
Nix version: 1.11.4
Nixpkgs version: 17.03pre96825.497e6d2

Looks like this is an upstream issue: https://github.com/gibiansky/IHaskell/issues/704


[ci] Fix end of line markers at end of file per git standards (ie emp…

…ty last line)

Coverage: 87.537% (+0.009%) from 87.528% when pulling 0183bad4e1bbf48d526e0feb4870756b1704da44 on hazendaz:copyright into 3505cc1a12c7dfb366eba5e29c92e32e8cc4c723 on mybatis:master.


Draft: Sprint 1: Community and Features Pages

This PR focuses on building out the community page while enhancing components and testing them with features page content. For this PR, the Feature page will get it's static content and any components that it shares on the community or home page.

Hello, I was wondering why jsx is used instead of tsx as typescript is part of the dependencies of the project ?

I ran into some issues with the last section of the community page (screenshot below).
This problem is related to being able to access the nested data effectively. It's possible to do it by just calling the variables one at a time, however these would get pretty long and unwieldy. It will also happen in the future with other data structures of a similar complexity.
Solution
The solution i'm working on uses this sript
const extractData = (arr) => {
  return arr.map((item) => {
    return structuredClone(item);
  });
};
which makes an immutable copy of a given object. The next step is to use this with destructured assignment to be able to pull the information out of an object and make it more sensible to access data.

[This article](https://www.freecodecamp.org/news/destructure-object-properties-using-array-map-in-react) shows the next step where data is pulled out of the array using map and assigned with destructuring.

Next Steps
I don't want to hold this PR up anymore with this issue. So I'm going to focus on cleaning up the code and getting the rest of the content ready to merge first. Then this can be addressed next week.

FWIW, WFM

Hello, I was wondering why jsx is used instead of tsx as typescript is part of the dependencies of the project ?

Is it possible to get an answer ?

Hello, I was wondering why jsx is used instead of tsx as typescript is part of the dependencies of the project ?

I'd agree with @benoitf too that it would be beneficial for everything to be typescript for future changes to be easily implemented.
The podman desktop team uses typescript intensively and we'll be contributing to future documentation / changes to the site as well.
It'd be an easy change as it's just changing the extension of the jsx files to tsx. So there's not much impact.


Way for users to delete requested tag

It has been suggested that the user who requested a tag or search can trigger the tag to be deleted. The suggestion was to react to the message with X emoji. I was also thinking an .oops command could also undo a mod command.
This would require remembering a user to last requested message mapping.

.oops deleting the message above would be very useful, especially for unglobal tags. Also, delete the .oops command after 2 seconds or something

I like the idea of an .oops command, it can definitely be made.
The easiest way to do this is just save the command output in a short lived cache, and allow up to 30 seconds for example to use it. However, nowadays discord also support buttons. Im not sure a privately seen button can be added to messages that are viewable by everyone, but it is worth looking into


Update Caddy to v2.3.0

Updated Caddy to v2
Fixes #206
Continuation of #207

Thanks for this @bhumijgupta, I don't see any point in keeping two separate variables as whoever updates will resolve their merge conflicts with need to update their hash.
Let's drop the ADMIN_HASH_PASSWORD variable and just move forward with ADMIN_PASSWORD.

The only reason I went ahead with ADMIN_HASH_PASSWORD was that while logging in to the Grafana UI user can then enter their plaintext password rather than having to remember the hashed password

Actually that's a really good point that completely slipped my mind.
Can I confirm are you one of the users that is/was affected by the Caddy versioning issue?
If so i'm curious/wondering whether or not just trying to pull the official image for 1.0.5 resolves the issue or not (just means we don't need to worry about updating the password in two places).

In my previous workplace, I faced this issue and internally rewrote the config for Caddy v2 to solve the issue. I did not test with pulling the official image for 1.0.5 . But I suggest it wil sooner or later get deprecated. It's better to make a breaking change now and provide support for longer time (with newer Caddy features).

Gentle ping @nightah

Just FYI, i tried running this instead of the main branch, but caddy crashes with the same index/slice error others have been reporting on v1.x

@DigiDr Can you please tell which docker version you are using?

Docker version 20.10.4, build d3cb89e @bhumijgupta

@DigiDr I could not reproduce the error you mentioned in v20.10.5 . My docker version is specified below.
Client: Docker Engine - Community
 Version:           20.10.5
 API version:       1.41
 Go version:        go1.13.15
 Git commit:        55c4c88
 Built:             Tue Mar  2 20:18:20 2021
 OS/Arch:           linux/amd64
 Context:           default
 Experimental:      true

Server: Docker Engine - Community
 Engine:
  Version:          20.10.5
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.13.15
  Git commit:       363e9a8
  Built:            Tue Mar  2 20:16:15 2021
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.4.3
  GitCommit:        269548fa27e0089a8b8278fc4fc781d7f65a939b
 runc:
  Version:          1.0.0-rc92
  GitCommit:        ff819c7e9184c13b7c2607fe6c30ae19403a7aff
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

It still isn't very clear the root cause that this issue is looking to resolve and whether this actually is a genuine fix.
My concern above all is that this will be a breaking change for all existing installations (without user intervention) and it's not clear whether this looks to resolve the issue some users appear to have with the Caddy container.

Updated docker to match yours @bhumijgupta without any improvement, so we can safely rule that out.

@bhumijgupta @nightah
Excuse me for confusing matters, but i was pulling the wrong commit. After figuring out the original bug i went back and looked at this again.
I can confirm that @bhumijgupta fix with an update to v2 does work.


Set max_validators to 125 by default

https://github.com/palomachain/paloma/pull/268
Background
Normal default is 100, we would like more people to be able to be validators on the testnet.
Testing completed

[x] manual review

@Vizualni Do you want to do the same thing for this line
jq-i 'walk(if type == "string" and .. == "stake" then "grain" else . end)' genesis.json

(which probably needs to use 000000ugrain anyway)


corepop distribution building ... docker maybe?

So corepop is a bootstrap binary described here: https://github.com/truedat101/Seed/blob/main/corepops/README.md
A bit of a mystery who builds these and where.  But they've been passed down by generations?  🤣
So what is needed:

either a docker container to easily build these for different linux distros
a repository that has scripts to cross compile
looks like linux ARM64 is missing
mac OS X M series is missing

Are these written in assembly originally?  Hopefully not.
Indeed, assembly, porting guide is here: https://fricas.org/~hebisch/poplog/port.txt

So ... rabbit hole continues...
The bootstrap is only on the web server: https://fricas.org/~hebisch/poplog/poplog-v16.03.tar.bz2
I know this should be in git.  I feel like a solo webserver with source that only exists in this one place is a failure point of lost history, like a boat that is lost at sea with the treasure.

Here's the recipe for going from .ll to .s : https://www.devever.net/~hl/recompile
Really good approach.  I'll look at this possibly further if my basic porting effort is slow.

In theory, you can compile the .ll files back together with clang.  Super interesting!


F #2451 Run vnm hooks

I discarded run-parts, it would only pass stdin to the 1st executable file.
Fixes #2451

I added a bunch of .readme files since git doesn't allow to track empty directories


Rename Contacts#email_address to Contacts#email

Ref https://github.com/dnsimple/dnsimple-developer/pull/75

👍


The linker command, for windows, is exceeding the maximum character length

I don't know if it's the final version, but if it is, it will be necessary to correct the names of the C files generated for windows. At the moment the command that calls the linker has almost 9000 characters, which exceeds the Windows limit of 8191. This is due to the stdlib file names being like this: @m..@s..@s.choosenim@stoolchains<EMAIL_ADDRESS>For the other destinations the names are also like this.

Hello, it is not a final version and will be used for the devel version only https://github.com/nim-lang/Nim/pull/21108, is there some potential solution? I simply ran koch csource -d:danger.

Now it should be better https://github.com/nim-lang/csources_v2/tree/master/c_code/2_5


leading spaces removed from ``` sections

Hi! The k8s gateway-api project uses this tool to generate its reference docs. Thank you!
One small problem is that yaml samples (which are included in ``` sections) have their leading spaces stripped somewhere in the process so the output is no longer proper yaml.
Here's the input: https://github.com/kubernetes-sigs/gateway-api/blob/e89bbb7042311703d9e076435ef75f15cf8665c2/apis/v1alpha2/httproute_types.go#L120
Here's the output: https://gateway-api.sigs.k8s.io/v1alpha2/references/spec/#gateway.networking.k8s.io%2Fv1alpha2.HTTPRouteRule
Is there a way to tell gen-crd-api-reference-docs to leave the formatting alone in ``` sections?
Thank you!

You can check the source code.

OK, will do.


Cannot download packages on Debian 9

Problem encountered on https://dotnet.microsoft.com/download/linux-package-manager/debian9/sdk-current
Operating System: Linux Debian 9 - x64
Got this:
Get:4 https://packages.microsoft.com/debian/9/prod stretch InRelease [4,023 B]
Err:4 https://packages.microsoft.com/debian/9/prod stretch InRelease
  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)

Same as https://github.com/dotnet/core/issues/3598

Had the same issue with Windows Subsystem for Linux using Debian to install .NET Core on it.
I'll wait for 15 minutes and see if it resolves itself as per this comment.

Had the same issue with Windows Subsystem for Linux using Debian to install .NET Core on it.
I'll wait for 15 minutes and see if it resolves itself as per this comment on https://github.com/Microsoft/mssql-docker/issues/409.

Duplicate of https://github.com/dotnet/core/issues/3598


Panel operating mode defect

I have few comments, please check

can you please paste the test section in commit message @SunnySrivastava1984

can you please paste the test section in commit message @SunnySrivastava1984

Done.


i18n support

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
Thanks for this helpful library.
It would be great if it was possible to translate the standard error messages for validation rules (required and for email field type). Currently it can be done only with passing own validation config object to all component instances or by overriding all components code.
Examples 🌈
New property for FormContainer which could receive default error messages or other way to declare them would be appreciated.

Guys, how about creating a context API with an initial state containing the English translation. Anyone who needs to use i18n in their project can add the provider at the top level of the main project component and dispatch the translation when it changes

Error provider is introduced in 5.16.x onwards, simple setup is shown as
https://react-hook-form-material-ui.vercel.app/?path=/story/textfieldelement--with-form-error-provider


https://github.com/firebase/quickstart-cpp/issues/86/AndroidManifest.xml

There doesn't appear to be anything actionable in this issue.


Remote assets?

Hi,
is there any reason, why the favicons are not in the docker container, but instead are served directly from your server? this way you can get referrer to my installation.
Could you please put the favicon in the images and locally load them?
https://github.com/haishanh/cherry/blob/10fa73817869acb7536534f5374d462cc3c7e430/src/app.html#L6

Ah...those remote favicons are rather temporary, just forgot to pull and serve them locally. Thanks for reporting.

Addressed with https://github.com/haishanh/cherry/commit/b55840b19164bcf30e1712edf86cd9b4822cce65. It's in latest version (v0.0.6) now.


Use commit instead of updating state value directly

Ref: https://github.com/penguin-statistics/frontend-v2/blob/a7a37c1efafbf5201b27ebc81575968134b5421e/src/store/modules/options.js#L34
(Otherwise will cause the state value to not be committed to persisted storage, which causes the time of existed user ID history not to be updated when needed)

Implemented in v3.4.0. Closing


Use Local data as watcher in live gekkos

Hi
Is it posible to use local data to live gekkos?
I mean if I'll starting the bot i would like to not wait to collect data, but for warm up use data already collected.

You can already do this when using the CLI interface. I'll be improving the flow for the UI gekkos while working on the new UI. But until than this is not possible unfortunately :(


refactor: Extract the Python handler into its own repository

That was a bit of work :stuck_out_tongue:
It needs #355 to support the install cache feature of PDM (resulting in multi-location namespace packages: handlers). Merged

Any idea why CI keeps expecting Python 3.6 results?

Probably something in https://github.com/mkdocstrings/mkdocstrings/settings/branches but I'm just guessing because I don't have access to that

Yeah I think I saw settings related to CI some time ago but I can't manage to find them...

Well have you tried the exact link I just sent??

I did but I missed the edit button :rofl:
It's there indeed, thanks

You're now an owner of the org by the way

Thanks much :grin:

@pawamoy I don't intend to review, you can merge. Maybe I'll look at everything just before release

Actually hmm tomorrow is the day that I have time

Take your time, I'll probably merge tomorrow instead :slightly_smiling_face:

@pawamoy OK you can consider everything approved, other than the comments I left :)


Fix image pull after a failure

Fixes #1893
When resuming from a failed pull the local content store Writer.Truncate() was not seeking to the proper position in the file. This caused writes to happen after the previously written content, instead of at the start of the file. Ref https://golang.org/pkg/os/#File.Truncate ( It does not change the I/O offset)
Truncate() is called from https://github.com/containerd/containerd/blob/v1.0.0/services/content/service.go#L395-L399.

appveyor timed out, doesn't seem related to the changes in this PR

Wow, some of the tests are taking a minute to ran, hence the timeout. Might be due to a recent change

Restart the build, hopefully it was just an issue on appveyor side

Codecov Report

Merging #1916 into master will increase coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1916      +/-   ##
==========================================
+ Coverage   47.29%   47.32%   +0.02%
==========================================
  Files          89       89
  Lines        8830     8833       +3
==========================================
+ Hits         4176     4180       +4
+ Misses       3968     3965       -3
- Partials      686      688       +2

Flag
Coverage Δ

#linux
50.7% <0%> (+0.04%)
:arrow_up:

#windows
42.28% <0%> (+0.03%)
:arrow_up:

Impacted Files
Coverage Δ

content/local/writer.go
52.63% <0%> (+2.63%)
:arrow_up:

content/helpers.go
32.87% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Should we cherry pick this? This seems pretty bad to me.

LGTM


Release python-http and python-jython processors

As a developer, I'd like to release python-http and python-jython as OOTB processors.
Acceptance:

New apps are generated for supported binders
Static file includes the new apps
Docs reflect in the aggregated ref. guide
Rest of the python apps are excluded from release (exclude sub-modules from build perhaps?)

This is complete.


thanks !

Hey, just wanted to let you know this came in handy for me today :)
So thank you for publishing it!

Thanks!


release of .6

I'm hoping to integrate this api into a library which is used to get census data into Pandas dataframes.  https://github.com/synthicity/popgen  The current version of this library on pypi seems to have a few issues, but the version in github seems to be fine.  Is there any way you guys could upload the latest version to pypi and call it a release?  Thanks!

Apologies for the terribly long delay. I'm working through PRs and open issues and should have a 0.8 release out this week. Thanks!


Is the intention of this different from NASA-UTM 'put-operation'?

https://app.swaggerhub.com/apis/utm/uss/v4#/A.%20Required%20Endpoints/put_enhanced_operations__gufi_
Is this redundant? Or are the two trying to solve a different problem? Apologies for maybe coming in late into this discussion.

Also I guess the flight declaration protocol was developed internally at GUTMA and this repository takes that work further. I am happy to discuss with you offline as well to bridge this gap between the two.


Prepare for volsync block device test

Fix issues in volsync test, making it easier to debub
Allow unprivileged pods to access block devices

Not adding yet the block device tests since it always fail. But the preparations can be merge now.
Based on #1536 for testing, I can rebase on master before merge.

Rebased on main so we can merge only the volsync part now.


how do i stop forks

thats how they are interfering but it doesnt let me click on item

am or can i stop forks .,but they are not letting me eliminate other users who are on my screen


Method to use internal RTC

With time sync over downlink message

approach: (just a note for myself)

device sends its local time within the data package
uplink payload formatter calculates the deviation between the device and the server time and passes the package to the webhook
the cloud function gets the data package and checks if the deviation is in the acceptable range (+ stores the data)
if the deviation exceeds the threshold then a downlink message will be enqueued with the needed adjustment parameters

To do: (note for myself)

[x] remove external rtc code
[ ] implement internal rtc
[ ] add timestamp to data package
[ ] process timestamp on server and enqueue downlink message
[ ] extract and apply the deviation information from the downlink message
[ ] Change the time handler from DateTime to (int)seconds and remove all dependencies of the RTClib.h

all finished and tested


My car sped up with stationary lead car in front

I am not sure what happened. There was a stationary lead car in front of me (traffic lights) but my car just continuously drove towards it. I had to brake because it shows no indication of braking before it was too late.
How to reproduce or log data
Unsure.
Expected behavior
My car was supposed to slow down way earlier.
Device/Version information (please complete the following information):

Device: comma two
Dongle ID: 7bc774bc8e17fd8a
Route: 7bc774bc8e17fd8a|2020-11-21--04-03-47
Timestamp: Nov 21@12:32:58
Version: 0.7.10
Car make/model: Toyota Corolla 2019

The lead car was detected just after you disengaged, at a distance of about 100 meters. At first the model thought the car was moving, when it was 90 meters out it fused with the radar point and correctly detected that it was stationary.
The situation with a stopped car between other car is a tricky situation, which should improve when we move more towards end-to-end longitudinal control. A human would slow down when approaching a traffic situation like this, irregardless if it already sees a stopped car in their lane or not.


Validation issue with +353 2fg0 917 4350

Please read the "guidelines for contributing" (linked above) and fill in the template below.
Country/region affected (e.g., "US"):
Example number(s) affected ("+1<PHONE_NUMBER>"): +353 2fg0 917 4350
The phone number range(s) to which the issue applies ("+1 555 555-XXXX"):
The type of the number(s) ("fixed-line", "mobile", "short code", etc.):
The cost, if applicable ("toll-free", "premium rate", "shared cost"):
Supporting evidence (for example, national numbering plan, announcement from mobile carrier, news article): IMPORTANT - anything posted here is made public. Read the guidelines first!
link to demo

@haseeb-heaven
Thank you for taking the time to improve libphonenumber!
Please let us know what is the issue here. See our contribution guidelines.

+353 20 917 4350 (without the "fg"?) is a drama code in the Republic of Ireland (source https://www.comreg.ie/csv/downloads/ComReg0804.pdf )

@dracos,
Thanks for sharing the official doc.
In this doc, it is mentioned that these are for "Drama use" and no further details are provided. We are not sure to which category these belong to. Ex: toll-free, fixed-line..etc.
And we can't connect to phone number that you have shared (Response: "The number is disconnected and no longer in service"). Please share any working number and also details about the category along with supporting evidence. See our guidelines.

I know it's not a working number, it's for drama use as you say, ie. a telephone number to be used in television/film/etc. So I was providing evidence it should not be a number matched by libphonenumber (unless you wish to start recording drama use numbers, which I would find useful but I understand if you wouldn't). You don't match UK landline drama use numbers, for example.

Thanks for your inputs here. :) We do not support such numbers.
Based on OFCOM's numbering plan, 121 is Birmingham's area code in GB.
https://www.ofcom.org.uk/__data/assets/pdf_file/0013/102613/national-numbering-plan-june-2017.pdf (Page 27). So we support<PHONE_NUMBER> as fixed-line number. I also see some valid number online. Ex: http://www.colliers.com/en-gb/birmingham/about/contact-us
Do you think that some part of them (subranges) are for "Drama use".
Please provide any valid official evidence for same. See our "provide supporting evidence guidelines"
Note: You can take a look at our FAQ page as well.

Would you like a new ticket for that? You currently support US drama numbers when there's an area code (#1393), so I assume the UK is the same issue as that. The official list of UK drama numbers are given at https://www.ofcom.org.uk/phones-telecoms-and-internet/information-for-industry/numbering/numbers-for-drama

Thanks for sharing all the range details specific to drama numbers in GB. Yes, please create a separate issue specific to GB.
As mentioned by Lara in other issue, fixing GB issue will take some time to do. Our priority is to making sure valid numbers are marked as such.
These do not fit into any of our present categories and it is not worth maintaining a separate one as the use case is very limited.

We don't require that a number to be supported by the library has a human at the other end since we already accept premium rate services and they might go to an automated system instead. But to date we only accept ranges that a human might call or send an SMS to.

See https://github.com/googlei18n/libphonenumber/blob/master/FAQ.md#what-about-m2m-machine-to-machine-numbers
Closing this as working as intended as there is no validation issue with origina IE number reported.

Related issue for the GB drama discussion: #341

Opened #1948 for GB numbers passing, thanks.

@giggsey and @dracos,
Thanks for sharing all the details. :)