403 Forbidden role reminders

This issue is present in the get role reminders but it also occurs to me it could pop up in other areas too.
Because it's not possible to check if a user has blocked the bot, or disabled DMs, we should have a unified way to handle the error of not being able to send a message to a user

a user has blocked the bot, or disabled DMs

Is that the leading theory on why the forbidden error is occurring?

a user has blocked the bot, or disabled DMs

Is that the leading theory on why the forbidden error is occurring?

I can't say for certain but that's the only thing I can think of - everything else that it could be is being checked for in one way or another already so that's my guess

Sure, a good theory as any. A try-except would then be suitable for this situation as there is no way to check for these conditions (See this StackOverflow post).
However, I think it is essential that we only except for a ForbiddenError and perform an additional check to attempt to guess why that error was raised. If it was raised because of one of the mentioned conditions then a log message should be sent and the loop should continue to the next member, otherwise the error should continue to be raised if the reasoning for it was not known. This will help with future debugging if issues occur.


Response to #88

Hello @ferventcoder .  Thank you for your response to issue #88 .  I would like to continue our discussion on this issue, if you do not mind.  I want to make sure I understand what you are telling me is the solution to this issue. You are telling me that I need to add an operating system specific windows update as a universal dependency in the nuspec file for the chocolatey package in question. I understand where you are coming from, but, even if it were clear in the first place how to designate a windows update msu as a dependency (I understand in regards to an exe, but I don't think that msu files/Windows updates have versions) in the nuspec file, sounds like (feel free to correct me if I am wrong) what you are saying is the solution would break the chocolatey package for all but server 2012 R2/ windows 8.1 devices, which I think is much worse than just updating the test environment. Or, were you suggesting manually performing a dependency validation/install within the chocolateyinstall script, which, while the scripting is not difficult, means I have to research for the KB with the same updates for other operating systems as well, and then it spirals into a needlessly complicated mess. I think this is just one of those things where, if users want the product to work correctly, then, like for most things, they need to make sure that they have the appropriate updates installed.

Closing this issue as it is a duplicate of the comment that is in #88. If needed we can open this again and revise the title.

@pauby Obviously trying to have a discussion here, so, yes, I would very much like to reopen the issue.  Not sure how you expect me to communicate this, as I assume that you do not monitor closed issues.  So, if you see this, then yes, I would like to reopen #88 .  At least gives me some assurance that you all can see my responses and are monitoring the issue

@demdante The issue you are discussing here is the same issue that is being discussed over at #88. So the issue here is not new and is a duplicate.


Missing variables in plants_eia860 table from pudl-v0.6.0-2022-03-12 release

iso_rto_code and other variables missing from plants_eia860 table in pudl-v0.6.0-2022-03-12 release
Below, the list of variables in the plants_eia860 table from the most recent pudl-v0.6.0-2022-03-12 release. This is not the beta version on Zenodo Archives, but it does appear to be at least a recent release. I downloaded this instance in April 2022. The problem? It does not match the list of variables in the SQLite plants_eia860 table?

Medium-Low: This component is not crucial to my analysis, but without it makes the analysis more complicated. Specifically, the variable I need the most is the iso_rto_code and it's unfortunately not there. I can do a workaround by pulling that from the SQLite table, but it's not very elegant.

To Reproduce
I'm using Docker and JupyterHub, so the code in python is
A = """ SELECT * FROM   plants_eia860 AS plants; """ a = pd.read_sql(A,pudl_engine) a

The bug appears to be related to the software / database? I've attached the settings.yml file I'm using to specify which data to load, and make a note of where in the ETL process the error is happening. See here:
settings.zip
Have I found an error or inconsistency in the data that PUDL brings together? Seems to be a difference in the tables on SQLite and in the database downloaded from Zenodo.
Does the problem happen when I'm starting up the database (importing data) or does it happen later when I'm trying to use the database? When I'm trying to use the database.

Expected behavior
I'd like to use iso_rto_code from the plants_eia860 table. This variable is not available in the distribution I downloaded from Zenodo archives.
Software Environment?

Operating System: MacOS 10.15
Python version and distribution: 3.9.10 | packaged by conda-forge | (main, Feb  1 2022, 21:24:11)
[GCC 9.4.0]
How did you install PUDL?

I downloaded from Zenodo archives and then ran docker-compose up. This is the first table I've run into issues with -- it does not appear to be a problem with other tables, e.g., generators_eia860 or utilities_eia860.

Hi there! Glad you've found PUDL useful, and sorry this discrepancy is causing you difficulty.
Since the release in March we've moved some of the attributes around. It turned out that some of the columns we'd previously inferred to be static and unchanging (like iso_rto_code) actually do change very occasionally, and so they were moved from plants_entity_eia (which has no report_date) to plants_eia860 (which has annual resolution).
We also added in columns like respondent_frequency which had previously been excluded incorrectly, and added the data_maturity column when we integrated the early release data from EIA for 2021.
The data published on https://data.catalyst.coop is built using the dev branch of the repository, which gets updated every few days (whenever the nightly builds succeed after integration of new data or code). There's metadata describing it published alongside (all the column descriptions there), and also the PUDL data dictionary on Read The Docs.
Nightly build outputs are also deposited in the public (requester-pays) storage bucket: gs://intake.catalyst.coop/dev if you want to download them directly.  You can also download the whole database behind the Datasette deployment. There's a small link at the bottom of the PUDL DB page: https://data.catalyst.coop/pudl

Great! Thank you for clarifying--and for your hard work on PUDL! I've pulled the updated DB and have it working now.


Update license attribute

specifying the type and URL is deprecated:
https://docs.npmjs.com/files/package.json#license

yea, know that, thanks!

@jonschlinkert need republish

yea, know that, thanks!

still, it's good to have that info in the PR, for reference and for others who don't know.

@pdehaan thanks! bumped and published


[francetv] add support for www.ludo.fr as playlist

FranceTVVideosIE is added to handle per video download
Fixes #13087

Before submitting a pull request make sure you have:

[X] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[X] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[X] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[X] New extractor
[ ] New feature

Description of your pull request and other information
Add support for http://www.ludo.fr/heros/ playlist (LudoFrPlaylistIE) with reusable common FranceTVVideosIE extractor to process URL generated by playlist.

Build is failing because test cases are geo-restricted...


More info at: https://travis-ci.com/Qiskit/qiskit-terra/jobs/251511129

API timeout


Request: Merchant without GM on the scene

Hi!
Would it be possible to have a Merchant sheet working without a GM on the same scene/offline?

My implementation relies on DnD5 LootSheet. I don't think that it's possible because some actions are actually sending requests to the GM and are executed by the GM. If the GM is not on the scene or offline, these actions won't work. It has to be implemented that way because users don't have permissions on the LootSheet and FVTT won't let them execute actions.


[test] Implement rom_e2e_shutdown_exception_asm

Fixes #14273
Signed-off-by: Dan McArdle<EMAIL_ADDRESS>

Re-add the ready to merge tag if you meant for these tests not to run in CI, or if you don't want to add the cw310_rom tag so they match the filters.

From "Experimental CW310 ROM Tests":
//sw/device/silicon_creator/rom/e2e:shutdown_exception_asm_otp_rma       PASSED in 18.0s


Add context.Context as first parameters to client/provider calls

Apologies if this has already been discussed.
I may be missing something, but is it possible to add Go's context.Context as a first parameter to all client/provider methods (interface definitions)?
Ex: https://github.com/open-feature/go-sdk/blob/d1d0b5e95eaf8f7e8fff5d0b0a59e8af12c75dc2/pkg/openfeature/client.go#L16
would become something like:
BooleanValue(ctx context.Context, flag string, defaultValue bool, evalCtx EvaluationContext, options EvaluationOptions) (bool, error)

It seems as a user of this SDK that I would want to be able to use context to pass request timeouts upstream to the provider (ie: in case request to provider is taking too long to return, error which would return the 'default' value).
Is this meant to be done via a Hook per request? If so would it be possible to document this usage?
Although it still seems that passing context.Context as a first class (first) parameter would be ideal IMO since it is a Go convention.

I agree that this would be beneficial and is definitely the idiomatic approach, I'm all for making this change.
As this would be another breaking change, we ought to decide and commit sooner rather than later.
cc @james-milligan @beeme1mr @toddbaert

i agree with @skyerus, context would be a great addition to the sdk

Awesome. I'd be happy to create a PR, if nothing else to at least show what it would look like

Amazing, thanks for the quick turn around!

Hey @markphelps, thanks again for your suggestion. It will be included in an upcoming release.


Fix PARAM_EXT_SET

This adds the basic ability to have camera controls set by QGC.

Despite the errors on build, which are not happening on my machine, the ack answer isn't right.

Both fixed.


CODEOWNERS + lint, build, and push workflows

Added CODEOWNERS
Added workflows to lint, build, and push to GHCR


Fixed #24072 -- Added FileResponse

https://code.djangoproject.com/ticket/24072

Nice patch!

Ok, I just pushed a new commit which should address all of the current comments.

merged in 3d2cae0896ee8026d1c2c5d31e4c4c8f74f2fef4, thanks.


Issue 416 - Help subcommand accepts other subcommands as arguments

Merge after #449
Closes #416

@homu r+

:pushpin: Commit babfd26 has been approved by kbknapp

:hourglass: Testing commit babfd26 with merge cc127f9...

:sunny: Test successful - status

Thank you! I :heart: love :heart:  this feature!
One tiny complaint is that the help message looks a bit awkward.

Shouldn't help messages aim to fit nicely in 80 columns without wrapping?

@crumblingstatue It should wrap nicely at term width (which is not necessarily 80) unless running it in Windows. However, I also think it should be shorter.
how about:
    help    prints this message or help message of given subcommand

@sru yeah we can shorten it if needed I'm fine with that since I think it's
a little long too.
@crumblingstatue if you're not disabling the default cargo features or
running Windows it should wrap at the term width...although I think adding
a setting to wrap at a specified width regardless would be great too. I'll
add a new issue to track that once I get home if someone hasn't already
done it.
On Tue, Mar 15, 2016, 7:57 PM Sung Rim Huh<EMAIL_ADDRESS>wrote:

@crumblingstatue https://github.com/crumblingstatue It should wrap
nicely at term width (which is not necessarily 80) unless running it in
Windows. However, I also think it should be shorter.
how about:
help    prints this message or help message of given subcommand

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/kbknapp/clap-rs/pull/450#issuecomment-197072825

@kbknapp I opened #452 and #453 about this.


Add logger in the MobileCore

Related issues:

AGDROID-727

@wtrocki I only added a test to check if the default implementation and whatever new implementation are going to be correctly returned by MobileCore. I have no idea how to test the real implementation :confused:


Create MinLeverage control

Similar to MaxLeverage create a MinLeverage control.
MinLeverage takes a float as well as a deadline date.  MinLeverage validations will only happen after the algorithm has reached the deadline date.  This allows algorithms enough time to reach their min leverage because it may take time for them to have a non zero leverage.

Coverage decreased (-0.003%) to 87.559% when pulling 360670ed8d62e269609b2f42a58bbb133e51e537 on min-leverage into 7c6920cd65c4aed88430c27221fa6e3366dc5961 on master.

Coverage decreased (-0.07%) to 87.494% when pulling 360670ed8d62e269609b2f42a58bbb133e51e537 on min-leverage into 7c6920cd65c4aed88430c27221fa6e3366dc5961 on master.

Coverage decreased (-0.003%) to 87.56% when pulling 070027d8358fe0792f85307a3b9a491d03983215 on min-leverage into 7c6920cd65c4aed88430c27221fa6e3366dc5961 on master.

Coverage increased (+0.01%) to 87.576% when pulling e4955de05821fab12490e046e18a30b58f3aaa78 on min-leverage into 7c6920cd65c4aed88430c27221fa6e3366dc5961 on master.

Coverage increased (+0.01%) to 87.576% when pulling e4955de05821fab12490e046e18a30b58f3aaa78 on min-leverage into 7c6920cd65c4aed88430c27221fa6e3366dc5961 on master.

Coverage increased (+0.01%) to 87.576% when pulling 4ba07c700fc220b0f8c11f0b0277a1238bb0ac57 on min-leverage into 7c6920cd65c4aed88430c27221fa6e3366dc5961 on master.

Coverage increased (+0.01%) to 87.577% when pulling fa711f87ffa5626dd3c7dce1f39a8e3ec198e621 on min-leverage into 7c6920cd65c4aed88430c27221fa6e3366dc5961 on master.

LGTM


Docs: Add ko-KR translation

Added Korean translation

@all-contributors please add @sierra41 for translation

Thank you very much, I appreaciate it ❤


gradient returns None

Issue type
Performance
Have you reproduced the bug with TensorFlow Nightly?
No
Source
source
TensorFlow version
2.7.1
Custom code
Yes
OS platform and distribution
CentOS Linux 7 (Core)
Mobile device
No response
Python version
3.8.12
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
Running on CPU only
Current behavior?
In the code below I don't understand why test_tape which is a tf.GradientTape() returns an empty (None) gradient. I have made sure to call the models so weights are initialized, checked that losses are actual tensors, redefined inputs as tf.Variable() and alternatively attempted using tape.watch(). My intuition is that the first call to train_optimizer.apply_gradients() somehow breaks the computational graph which is why test_tape does not track variables associated with model_copy but I am not sure how to fix the issue.
Standalone code to reproduce the issue
import tensorflow as tf
import tensorflow.keras as keras
import tensorflow.keras.backend as keras_backend
import numpy as np

def sineGenerator(amplitude=None, phase=None):
    if amplitude is None:
        amplitude = tf.random.uniform(shape=[], minval=0.1, maxval=5.0)
    if phase is None:
        phase     = tf.random.uniform(shape=[], minval=0.0, maxval=np.pi)
    def _gen(x):
        return amplitude*tf.math.sin(x+phase)
    return _gen

def genX(sample, minval=-5.0, maxval=5.0):
    return tf.expand_dims(tf.random.uniform(shape=[sample], minval=minval, maxval=maxval), 1)

class sineModel(keras.Model):
    def __init__(self):
        super().__init__()
        self.hidden1 = keras.layers.Dense(40, input_shape=(1,))
        self.hidden2 = keras.layers.Dense(40)
        self.out     = keras.layers.Dense(1)

    def call(self, x):
        x = keras.activations.relu(self.hidden1(x))
        x = keras.activations.relu(self.hidden2(x))
        x = self.out(x)
        return x

def copyModel(model, x):
    model_copy = sineModel()
    output = model_copy(x)
    output = model(x) # should not be necessary since model(train_x) is called before model_copy = copyModel(model,x) in maml_train()
    model_copy.set_weights(model.get_weights())
    return model_copy

def maml_train(model, total_iterations=10, meta_train_steps=10, meta_test_steps=100):
    log_step = total_iterations //10 if total_iterations > 10 else 1

    test_optimizer  = keras.optimizers.Adam(learning_rate=0.001)
    train_optimizer = keras.optimizers.Adam(learning_rate=0.001)

    losses, total_loss = [], 0.

    for step in range(total_iterations):
        sineGen = sineGenerator()

        test_x     = genX(meta_test_steps)
        test_y     = sineGen(test_x)

        train_x    = genX(meta_train_steps)
        train_y    = sineGen(train_x)

#         test_x, test_y, train_x, train_y = tf.Variable(test_x), tf.Variable(test_y), tf.Variable(train_x), tf.Variable(train_y)
#         test_x, test_y, train_x, train_y = tf.convert_to_tensor(test_x), tf.convert_to_tensor(test_y), tf.convert_to_tensor(train_x), tf.convert_to_tensor(train_y)

        model(train_x)
        model_copy = copyModel(model, train_x)

        with tf.GradientTape() as test_tape:
#             test_tape.watch([test_x, test_y, train_x, train_y])
            with tf.GradientTape() as train_tape:
#                 train_tape.watch([test_x, test_y, train_x, train_y])
                train_loss = tf.reduce_mean(keras.losses.mean_squared_error(train_y, model(train_x)))

#             print('train_loss type', type(train_loss))
            gradients = train_tape.gradient(train_loss, model.trainable_variables)
            train_optimizer.apply_gradients(zip(gradients, model_copy.trainable_variables))

            test_loss = tf.reduce_mean(keras.losses.mean_squared_error(test_y, model_copy(test_x)))

#         print('test_loss type', type(test_loss))
        gradients = test_tape.gradient(test_loss, model.trainable_variables)
        print(gradients)
        test_optimizer.apply_gradients(zip(gradients, model.trainable_variables))

        total_loss += test_loss
        losses += [total_loss/(step+1)]

        if step % log_step == 0:
             print('Training loss (total) at step %d: \t %.4f' % (step, total_loss/(step+1)))
    return losses

sModel_maml = sineModel()
maml_train(sModel_maml)

Relevant log output
[None, None, None, None, None, None]

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-402-4ed8f16df6b0> in <module>
     76
     77 sModel_maml = sineModel2()
---> 78 maml_train(sModel_maml)

<ipython-input-402-4ed8f16df6b0> in maml_train(model, total_iterations, meta_train_steps, meta_test_steps)
     65         gradients = test_tape.gradient(test_loss, model.trainable_variables)
     66         print(gradients)
---> 67         test_optimizer.apply_gradients(zip(gradients, model.trainable_variables))
     68
     69         total_loss += test_loss

~/pyenvs/tensorflow2/lib/python3.8/site-packages/keras/optimizer_v2/optimizer_v2.py in apply_gradients(self, grads_and_vars, name, experimental_aggregate_gradients)
    631       RuntimeError: If called in a cross-replica context.
    632     """
--> 633     grads_and_vars = optimizer_utils.filter_empty_gradients(grads_and_vars)
    634     var_list = [v for (_, v) in grads_and_vars]
    635

~/pyenvs/tensorflow2/lib/python3.8/site-packages/keras/optimizer_v2/utils.py in filter_empty_gradients(grads_and_vars)
     71   if not filtered:
     72     variable = ([v.name for _, v in grads_and_vars],)
---> 73     raise ValueError(f"No gradients provided for any variable: {variable}. "
     74                      f"Provided `grads_and_vars` is {grads_and_vars}.")
     75   if vars_with_empty_grads:

ValueError: No gradients provided for any variable: (['sine_model2_59/dense_555/kernel:0', 'sine_model2_59/dense_555/bias:0', 'sine_model2_59/dense_556/kernel:0', 'sine_model2_59/dense_556/bias:0', 'sine_model2_59/dense_557/kernel:0', 'sine_model2_59/dense_557/bias:0'],). Provided `grads_and_vars` is ((None, <tf.Variable 'sine_model2_59/dense_555/kernel:0' shape=(1, 40) dtype=float32, numpy=....

@yavorkovachev I tried to replicate the issue on colab, and faced a different error as shown in the gist. Could you please let me know if anything is missed here to reproduce the issue. Please make sure to share all the dependencies as well. Thank you!

@sushreebarsa Apologies. I forgot to include a function which is used inside the custom training function ( maml_train() ). I just tested on Colab and can reproduce the original error after copy and pasting the updated code.

@yavorkovachev Thank you for your kind response!
The gradient can return none when the target is connected to the source. Please have a look at this link to know more on the ways to disconnect the gradient. Please let us know if it helps?
Thank you!

@sushreebarsa Thanks for your reply. I understand the gradient tape is detached and I have checked out the information in the link you provided. In fact, from the commented out lines you can see that I tried to ensure that I avoid the common pitfalls that lead to detachment (I also followed this post: http://blog.ai.ovgu.de/posts/jens/2019/001_tf20_pitfalls/index.html). I think
train_optimizer.apply_gradients(zip(gradients, model_copy.trainable_variables))

somehow breaks the computational graph. I have found a way to fix the code without using the train_optimizer.apply_gradients() step but I don't exactly understand what is wrong with the code above and to me it seems like it should work correctly.


Cannot start the latest interlock with --swarm-url

I'm consistently getting an error about the --swarm-url parameter to interlock.
I'm using the latest docker, docker-machine, docker-swarm and interlock (interlock:1.0.0 or interlock:ng)
docker run ehazlett/interlock:ng --swarm-url $DOCKER_HOST --swarm-tls-ca-cert=/etc/docker/ca.pem --swarm-tls-cert=/etc/docker/server.pem --swarm-tls-key=/etc/docker/server-key.pem --plugin nginx start

results in an incorrect usage followed by this
time="2016-02-28T11:02:48Z" level=fatal msg="flag provided but not defined: -swarm-url"

Note the single dash before swarm-url.

Yes as noted in the readme this is a major refactor.  ehazlett/interlock:latest still points at the old version for this reason.  If you want to use 1.0+ I strongly suggest looking at the docs (https://github.com/ehazlett/interlock/tree/master/docs) as there are quite a few differences.  First, there is now a configuration file which is why you are getting the error for the flag.  I'm planning to add a migration guide soon but the docs should help in the meantime.  See https://github.com/ehazlett/interlock/blob/master/docs/getting_started.md for a quick start.
Thanks!

Oh, I actually use the config file. I forgot to include the volume in the sample above since I'm normally use a docker compose file. I did double checked and the config file is in /bin/config.toml on the container. I use the same config.tom that you list in the nginx example.
So are the --swarm-xxxx parameters not passed by parameters to the command anymore? If not, I can't see which new config file entries correspond to them
Also, are the new config parameters case sensitive? They are different between the example in the doc you list and the examples/nginx.


Discard duplicate jobs

Summary
The concurrency controls that currently exist allow blocking execution of "duplicate" jobs, or jobs with the same arguments. These jobs wait, then execute after the currently running job finishes. There's no way to discard all duplicate jobs and prevent them from running completely.
Proposal
Add a way to discard jobs that get blocked by the concurrency controls.

@matteeyah, I've got this on my mind as well! In the end, I'm not sure there's a version of https://github.com/basecamp/solid_queue/issues/105 that I like or that it's useful for what we need, so I thought, in the meantime, to add another parameter to limits_concurrency to discard instead of blocking. Something like:
on_conflict: :discard

By default, it'd be:
on_conflict: :block

which would be the current behaviour.

That's exactly what I had in mind!

For reference - We're currently using https://github.com/veeqo/activejob-uniqueness. It serves a slightly different purpose, but it also checks for duplicate jobs.
It also has an on_conflict option - it supports logging a conflict, raising an error on a conflict and passing a proc to handle the conflict manually.

Another vote for following the activejob-uniqueness conventions.
I would note that of the strategies offered by the gem, until_and_while_executing is the one we have found most-useful.

Is there any more word on this? Or any openness to others trying to contribute?

Hey @nhorton, sorry for the delay! I'm back at this now and would like to get this feature ready for version 1.0.

of the strategies offered by the gem, until_and_while_executing is the one we have found most-useful.

I think that would be the only strategy supported because of the way concurrency controls work. Right now jobs don't unblock other jobs or release the semaphore until they complete, and for simplicity, it'd be much easier to keep it that way.

Or any openness to others trying to contribute?

Definitely! 🙌

@rosa Is there an ETA for 1.0 ?

@bilby91, my hope is by the end of August but you know, there are always unexpected things 😅 😅

@rosa Awesome! I look forward to testing this feature.

In the end I won't be able to get this one in for v1.0, but I hope to get it there shortly after. Sorry!


Fix HeavySelect2MultipleWidget

Fixed a "selector not found" error similar to the one fixed in 2bf42c5bdabaa22b90878d9cd91a5912c6a6720a.

Coverage increased (+0.38%) to 58.45% when pulling fb2ba7c0dc6ddc57abd9160bd3f35abd45796908 on DMOJ:patch-1 into 5f9477b6640fc01f61936d53282be0e27b3e6583 on applegrew:master.

Coverage increased (+0.38%) to 58.45% when pulling fb2ba7c0dc6ddc57abd9160bd3f35abd45796908 on DMOJ:patch-1 into 5f9477b6640fc01f61936d53282be0e27b3e6583 on applegrew:master.

@quantum5 can you add a test for the error you described?

+1, after upgrading to the 4.3.1 version od django_select2 I also got this "Uncaught ReferenceError: selector is not defined" error and applying patch from this pull request seems to fix it.
My use case is quite complicated since I use django_select2 together with patched version of django-inplaceedit, but the above mentioned error is happening wherever I use HeavySelect2MultipleWidget.

OK, I was able to create a failing test that this commit fixes. I'm quite new to github tho and unsure how to proceed but I guess I'll just create another pull request with that test and this fix, but of course credit goes to @quantum5

Please see #175

Can we get this fix into a minor release 4.3.2?  I want to use django_select2 in another package and I don't want to have to use an editable dependency in requirements (if that is even possible?)

Editable dependency in requirements.txt is possible.

What about setup.py install_requires?

@applegrew I think we should create a new release from master.

Ok.
Regards,
Nirupam
Cink http://cink.applegrew.com | Home http://www.applegrew.com |  Git
https://github.com/applegrew/ | Blog http://blog.applegrew.com |  MaapIn
http://www.maap.in
On Fri, Sep 18, 2015 at 1:59 PM, Johannes Hoppe<EMAIL_ADDRESS>wrote:

@applegrew https://github.com/applegrew I think we should create a new
release from master.
—
Reply to this email directly or view it on GitHub
https://github.com/applegrew/django-select2/pull/174#issuecomment-141380652
.

I am just about to publish version 5.0.0 based on master code. @codingjoe Please update the Changelog.md with important changes that have been made. Currently I have put a very generic statement.

@applegrew why a new major version? There were just minor fixes. I've been on a version 5 branch since forever. That will be very miss leading. Can you please pull the version, tho this isn't really nice.
Just create a new fix release. There haven't been any features added. I'll write a Changelog.

BTW, it would be cool if you could add me as a colaborator on pypi. That we I can release myself and we can avoid this confusion in the future.

Oops. Removed v5 from PyPi.

Ok master is released in 4.3.2


RDP Connection Sequece (Part 1)

Implemented RDP connection sequence up to capability sets exchange
Ran cargo fix, cargo clippy and cargo fmt on code in Jet so Travis pipeline passes
Added rustdoc for the previous iteration of RDP

The file rdp-proto/src/rdp/tests.rs is not referenced anymore, the tests fail to compile if I add them.

Oops, I divided the old rdp.rs file into several and did the same with the test. But apparently I forgot to remove this file after that. I will remove it in the next PR.


add app icon to fastlane structure

Fixes the most pressing thing from: https://github.com/k9mail/k-9/issues/4562
(This should make K-9 have an icon again after the next release in the f-droid listing)

Thanks :+1:


[SPARK-21990][SQL] QueryPlanConstraints misses some constraints that can be recursively inferred

What changes were proposed in this pull request?
When I inspected the latest change of SPARK-21979, I found we could miss few constraints that can be recursively inferred.
E.g. consider constraints a === b && b === c && a === 1 && b === 5. Currently we only infer a === c && b === 1 && c === 5, but c === 1 can be also inferred from newly inferred constraint a === c.
How was this patch tested?
Added test.

Test build #81711 has started for PR 19216 at commit d76f8f0.

Test build #81711 has finished for PR 19216 at commit d76f8f0.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81711/
Test FAILed.

Merged build finished. Test FAILed.

Test build #81720 has started for PR 19216 at commit 4e85e5f.

Test build #81720 has finished for PR 19216 at commit 4e85e5f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81720/
Test PASSed.

Hi @viirya ,
could you find a better example?
The one in the description/test case should be optimized to False.

@gengliangwang Sorry, what the meaning it should be optimized to False?

 a === b && a === 1 && b === 5

The result is always False.

@gengliangwang Thanks. I'm modifying the test case.

@gengliangwang How about the modified one in the description?

Test build #81749 has started for PR 19216 at commit e4cffda.

Thanks.
I run the test case with the rule "InferFiltersFromConstraints", the result has c > 5.
The rule will be executed multiple times.

@gengliangwang Yeah, I see. InferFiltersFromConstraints will construct new Filter from inferred constraints and c > 1 will be inferred in the new Filter. I'm trying to see if there can be an exception. If no, I think it's safe and I can close this one.

@gengliangwang I think it should be safe. Let me close this now. Thanks.

Test build #81749 has finished for PR 19216 at commit e4cffda.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81749/
Test PASSed.

I'm thinking more about this.
Actually I think we shouldn't rely on an optimization rule to produce correct constraints. If there is a logical plan other than Filter that can get its material of constraints from outside (maybe data source or others) in the future, we won't infer the correct constraints.
But currently looks like it doesn't cause any trouble.

Yes, I agree. But constraints is used in filter inference only, and it works now. The code changes in this PR make the logic complex.
I think we should make the logic simpler, if we decide to do this.

Yeah, as it doesn't cause any trouble now, I don't bother to re-open it.
Btw, another bad to reply on the optimization rule to produce correct constraints is the extra cost introduced by re-constructing the whole query plan tree, and the re-inferring of constraints in Filter. Instead of this, inferring the correct constraints in one pass should be more efficient.


logfire auth

Question
I was trying logfire on a FastAPI app hosted on render.com, I got "You're not authenticate, run logfire auth" on the render log.
So, is a way I can pass the auth credential a environment variable?

Hi @kenmoh,
The idea is to create a write token, and set the LOGFIRE_TOKEN environment variable there.
You can read more about it here.

Oh great! Thank you @Kludex .


Community Invitation to the OpenJS Collaborator Summit + Info

Hi all! In an effort to make sure we're fully getting the word out to OpenJS Foundation project communities about the collaborator summit, I'm posting the following message on main or admin repos:
This year, the Node Collaborator summit is opening up to include and welcome all former JS Foundation projects. The Collaborator summit is a space where projects and working groups within projects have the dedicated time and space to get together and get work done. We will have project-specific meetings, cross-project meetings on issues like security and standards, and of course meetings to discuss new programs and patterns for the new CPC.
The event will be in Berlin on May 30-31st at the Courtyard Marriott City Center. It is free to register with the password 'collabsummit.' We are also inviting all projects/attendees to suggest meetings and sessions via our CFP - these are not conference talks, but rather collaborative conversations or meetings you would like to hold with your peers. We ask that you register in advance so that we can allocate meeting room space and resources, and so that we can publish where conversations are scheduled in the day or two leading up to the event.
Links:

Session Proposals
Registration (password is collabsummit)
Information on how to apply for the Travel Fund
Summit Planning Repo

Event was already held, closing


Question about Monthly

For Monthly you have this listed:

monthly_day_order_number
Yes
Number of the collection_day each month. E.g., if collection_day is "sat", 1 will mean 1st Sunday each month, 2 for 2nd Sunday each month etc. (integer 1-4)Default: 1(relevant for monthly_collection)

Is Sunday a typo?  Should it be Saturday?
I'm trying to figure out how to set a collection for the 2nd Tuesday of each month.
Would I set collection_day as tue and monthly_day_order_number as 2 ?

Yes, this is a typo, thanks for finding it, will fix it in next release.
Yes, you got it right.

Sweet! Thanks for this tool!
Want me to close issue or leave open?

You can leave it, I will close it with the release. Thanks.

Fixed in the release 1.6


Display gird-wrapper as table and insert treegrid right after the original tree.

Hey @legendetm thanks for the PR. The way it currently works is that the container is the original tree element, gridparent is container's parent. We put the gridWrapper (just a
) as a child of the gridparent, and then put a child to the gridWrapper as midWrapper (just another
). midWrapper gets one
 column for each column of the grid, and the tree column (normally 0) gets the original element reparented in there.
Kind of go from:
gridparent
 |-- container

To:
gridparent
 |-- gridWrapper
      |-- midWrapper
           |-- col0
                |-- container
                |-- col1 dataCell
                |-- col2 dataCell
                     ....
                |-- coln dataCell

What you are proposing, I think, looks like this:
gridparent
   |-- container
   |-- gridWrapper
      |-- midWrapper
            |-- col0 (I think?)
            |-- col1 dataCell
            |-- col2 dataCell
                 ....
            |-- coln dataCell

My questions are:

What do we gain from it?
Does it break anything else that maps out in preparing the headers and the grid, especially the headers?

I just changed the location where the gridparent is inserted.
For example, I have the following setup:

  <jstree />

then current code appends the gridparent to the end of the jstree parent:

  <gridparent />

and I changed it to be inserted after the jstree, so the end result will be:

  <gridparent />

I just changed the location where the gridwrapper is inserted.
For example, I have the following setup:

  <jstree />

then current code appends the gridwrapper to the end of the jstree parent:

  <gridwrapper />

and I changed it to be inserted after the jstree, so the end result will be:

  <gridwrapper />

If you look at one of the test cases odd-named-id-test.html included in the tests folder, you see the following html:

First Tree

Tree grid is loading.

With the current version, it looks like this:

After the changes I made, it looks like this:

Oh, I get it! The current design implicitly assumes that container is the only child of parent. If it isn't, strange things result. The change ensures that the placement is consistent?

Yes, the change 74cf871 always inserts the gridwrapper right after the jstree.

This breaks some cases. Try this for example https://github.com/deitch/jstree-grid/blob/master/tests/widths-test.html

Ok, I see that it brakes the test Fixed Width, All Columns Defined, Columns > Width, but in the same time it fixes some other tests where the size is fixed, for example Fixed Width, All Columns Defined, Columns < Width, Fixed Width, Columns Percentages, Fixed Width, All Columns Defined, One auto Column and some others.
In the test file widths-test.html you have two values Expected and Actuals where the latter shows the wrapper width not the actual width.

But after the updates the tables have the correct width

I see that some more work needs be done to support tables where columns' width > table width, but I'm not sure which way to go.
The simplest way is to add another div wrapper with overflow-x: scroll, if the table width is present.

My mind is swimming trying to get my head around this. I remember significant effort to get all of the alignments right the first time around (then the second, then the third... this has had a lot of iterations).
The question is, why does this break those cases?

The braking change is .jstree-grid-wrapper {display: table}.
The current layout with the css display overloading is

So, now when the first div has fixed width and the tris wider, it automatically adds the scrolling to the div. But, because tr doesn't have table as parent, its width is not correctly shown.
See more http://stackoverflow.com/questions/3480060/css-displaytable-row-does-not-expand-when-width-is-set-to-100
The layout after my changes is

Now, the table and all the cells have correct with, but the second example Fixed Width, All Columns Defined, Columns > Width fails, because the table will sizes to its content.
In conclusion, to fix both problems, the layout should be following

These have been incorporated separately; take a look at the code.
Thanks for the PR!


Undefined: WIN32, __builtin_assume, __builtin_addressof

Bug description
I'm trying to use Pinocchio in Simulink on Windows with Simulink's custom code functionality.
The compiler is configured to be MinGW64 and header and lib path are configured the same as previously on Linux, which I have compilation working and the executable was tested and running.
The Pinocchio is installed using conda.
Expected behavior
When compile the code, the following error shows up. (undefined WIN32, __builtin_assume, __builtin_addressof)

Error while parsing custom code for model 'pino_simulink_test'. Fix the errors in your custom code or disable 'Import custom code' if your custom code has incompatibilities. If your custom code uses C++ language syntax, ensure that 'Language' is set to 'C++' in Model Configuration Parameters > Simulation Target.
Caused by:
In line 150 of [c:\users\haowenl\appdata\local\anaconda3\library\include\pinocchio\macros.hpp](matlab: opentoline('c:%5cusers%5chaowenl%5cappdata%5clocal%5canaconda3%5clibrary%5cinclude%5cpinocchio%5cmacros.hpp', 150)),
expected an expression:
|  #if WIN32
|           ^
In line 67 of [c:\users\haowenl\appdata\local\anaconda3\library\include\boost\smart_ptr\detail\local_counted_base.hpp](matlab: opentoline('c:%5cusers%5chaowenl%5cappdata%5clocal%5canaconda3%5clibrary%5cinclude%5cboost%5csmart_ptr%5cdetail%5clocal_counted_base.hpp', 67)),
identifier "__builtin_assume" is undefined:
|          _builtin_assume( local_use_count >= 1 );
|          ^
In line 40 of [c:\users\haowenl\appdata\local\anaconda3\library\include\boost\core\addressof.hpp](matlab: opentoline('c:%5cusers%5chaowenl%5cappdata%5clocal%5canaconda3%5clibrary%5cinclude%5cboost%5ccore%5caddressof.hpp', 40)),
identifier "builtin_addressof" is undefined:
|      return builtin_addressof(o);
|             ^
|          detected during:
|            instantiation of "T *boost::addressof(T &) [with T=const pinocchio::GeometryModel::CollisionPairVector]" at line 649 of "c:\users\haowenl\appdata\local\anaconda3\library\include\boost\foreach.hpp"
|            instantiation of "boost::foreach_detail::auto_any<T *> boost::foreach_detail::contain(T &, mpl::false *) [with T=const pinocchio::GeometryModel::CollisionPairVector]" at line 268 of

Reproduction steps
Steps to reproduce the behavior:

compile the code in Simulink
or use "validate custom code" button when setting up the Simulink environment.

Simulink Environment parameters:
Include Headers:
#include "pinocchio/parsers/urdf.hpp"
#include "pinocchio/algorithm/joint-configuration.hpp"
#include "pinocchio/algorithm/model.hpp"
#include "load_robot_c_caller.hpp"
Include Directories:
C:\Users\haowenl\AppData\Local\anaconda3\Library\include
C:\Users\haowenl\AppData\Local\anaconda3\Library\include\eigen3
Source Files:
load_robot_c_caller.cpp
Libraries:
C:\Users\haowenl\AppData\Local\anaconda3\Library\lib\pinocchio_default.lib
C:\Users\haowenl\AppData\Local\anaconda3\Library\lib\pinocchio_parsers.lib
Compiler flags:
-DNOMINMAX
Linker Flags:
-lpinocchio_default -lpinocchio_parsers
System

OS: [windows 11]
Pinocchio version:  [3.1.0]

Hello @OoLoueLoueoO,
I don't know if Pinocchio can be build with MinGW, we don't have a CI setup to test that configuration.
Conda-forge Pinocchio package is build with MSVC compilers and I'm not sure MinGW and MSVC binaries are compatible. Is there a way to tell simulink to use MSVC compiler ?
Also, I see this message in the log you provided:

If your custom code uses C++ language syntax, ensure that 'Language' is set to 'C++' in Model Configuration Parameters > Simulation Target.

Did you choose C++ language ? Pinocchio is a C++ library.

Thank for your response jorisv. It does seems like a compiler config issue.  I was able to compile and generate the model and data using gcc in the MATLAB (in ubuntu) right now.

Thank for your response jorisv. It does seems like a compiler config issue.  I was able to compile and generate the model and data using gcc in the MATLAB (in ubuntu) right now.


ASM: Allow multiple machines to access the same machine instance

We currently don't have a way to declare a machine instance that is used by more than one calling machine, like this:
graph TD;
    Main --> Hash;
    Main --> Padding;
    Padding --> Hash;

Automatic witness generation for this should already be working!
As far as I'm aware, this is the last missing features we'd need to implement #1055!

I think the cleanest way to achieve this would be to make instanciation explicit rather than implicit now:
let arith = Arith {};

let other = Other { arith }
let main = Main { other, arith }

And interpret the main instance as the entry point just like we use the Main machine type now.

I agree, machines should be syntactic sugar for functions that return machine objects.
I think the main open question is where to place the arguments for machines (as in already instatiated sub-machines) if there are any.

We decided in a call to do something like this:
machine Main {
    mem: Memory;
    binary: Binary(mem);
}

machine Binary(mem: Memory) {
    use mem.addr or whatever
    ...
    #[operation_id]
    col witness operation_id;

    #[latch]
    col fixed latch(i) { if (i % 4) == 3 { 1 } else { 0 } };

    ...
}

Depends on https://github.com/powdr-labs/powdr/issues/1251

Was this merged @pacheco ?


sattrack loader: add roi argument

This change uses the new index_v2.csv format and adds roi as an optional argument for the sattrack loader.
see also https://github.com/orcestra-campaign/sattracks/issues/2

Thanks for adding this 👍


Add a type definition file in typings folder

You can trun on intelliSense when you use Vscode.

Nice work @lleohao!


Add .rbi for Sorbet files

Adds https://sorbet.org/docs/rbi. They are valid Ruby files, with a .rbi extension.
Checklist:

[x] I am associating a language with a new file extension.

[x] The new extension is used in hundreds of repositories on GitHub.com

Search results for each extension:

https://github.com/search?q=extension%3Arbi+typed&type=Code

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

https://github.com/sorbet/sorbet/blob/master/gems/sorbet/lib/gem_loader.rbi

[ ] I have included a change to the heuristics to distinguish my language from others using the same extension.

No, as this extension isn't used elsewhere.

I realise this was previously rejected in https://github.com/github/linguist/pull/4561 but since Sorbet is now public, usage appears to have gone up a lot.

since Sorbet is now public, usage appears to have gone up a lot.

It would appear so. ~3,538 .rbi results currently indexed, with the overwhelming majority of them being Ruby source:

2,396 results harvested
117 unique repositories
80 users

Not bad for something relatively-recent. 👍 LGTM. @lildude?

Not bad for something relatively-recent. 👍 LGTM. @lildude?

To be consistent with prior decisions, the usage doesn't quite meet the "hundreds of repos" requirement yet. Given it's jumped over 400 files in just over a week, I'm gonna give it a little longer before 👍  as it probably won't be long now.

There are now about 5k .rbi files. I downloaded 2555 of them using Harvester and counted 120 repositories by 84 users. If we extrapolate to the total 5k files, that would be approximately 240 repositories. I think we're now okay to merge this!


Added Youtube Video Downloader

Youtube Video Downloader

@yash-gulatii Don't forget to star the repo ✨


Don’t package binstubs

This line in is causing this gem to overwrite executables for appraisal, guard, pry,
rake, rspec, and yard. :disappointed:
Please merge this patch and release version 0.5.1.

Sorry about that. :sweat:

0.5.1 should be available on Rubygems now. Thanks for the fix!

@gevans Thanks for the quick fix!


BindAttribute doesn't work with FromBodyAttribute

The following code doesn't respect BindAttibute for /bind/frompost/ as all properties will be populated in MVC 1.1.0:
public class ComplexType
{
    public string Property1 { get; set; }

    public string Property2 { get; set; }
}

[Route("api/[controller]")]
public class ValuesController : Controller
{
    [HttpPost("bind")]
    public ComplexType Bind([Bind(nameof(ComplexType.Property1))] ComplexType complexType)
    {
        return complexType;
    }

    [HttpPost("bind/frompost")]
    public ComplexType BindFromPost([FromBody][Bind(nameof(ComplexType.Property1))] ComplexType complexType)
    {
        return complexType;
    }
}

As far as I understand the reason is that ComplexTypeModelBinder is the only binder that respects ModelMetadata.PropertyFilterProvider and his provider is the last one in the list of default ones. In case of FromBodyAttribute the binding is handled by BodyModelBinder which doesn't use this filter.

@alexb5dh BindAttribute.Include is not relevant in your scenario. [FromBody] and the BodyModelBinder use input formatters and input formatters use external serializers / deserializers with separate controls.
For example, if you wish to skip some values in submitted JSON, use model types that lack those properties. [JsonIgnore] attributes might also help.

Ok, so I see this is by design and issue can be closed.
I originally stumbled into this when trying to find a way to filter model properties based on HTTP request method used. Since input formatters doesn't have this this infromation and ModelMetadata.PropertyFilterProvider is not used by BodyModelBinder can you please give me some tips in this direction?


Implement the function that assigns a score based on a tuple of results

Actually we do not need to have such a function. All we need is a function that finds a score based on TPR and TNR


⚠ Jenkins Plugins Artifactory has degraded performance

In 47107e5, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1343 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 8906a68 after 25 minutes.


Added migration guide

Added migration guide for WindowsAzure.ServiceBus to Microsoft.ServiceBus.Messaging.

API change check
API changes are not detected in this pull request.

@JoshLove-msft,  @m-redding:   Last call for feedback.   I'd like to merge this in tomorrow if you don't have any blockers.

/check-enforcer override


Casadi better than sympy

Is casadi a better option for the symbolic solving then sympy?
https://web.casadi.org/

This would be interesting to find out, since CasADi seems to have both Python and C++ APIs. Both are capable of generating C code, which would suggest that in speed their maybe quite similar. This article suggests sympy has more "general purpose symbolic abilities", so I guess one way of moving forward with this is determining whether CasADi actually supports all the features of sympy which rednose and openpilot use or could benefit from.

Interesting. Sympy is quite slow for some of the more complicated jacobians. I assumed this would be better with cassadi, but maybe not.
Sympy also doesn't handle elements, vectors and matrices well together, there is a lot of weird code to deal with that


citrixworkspace recipe produces download error

FYI it looks to me, like Citrix withdrew Workspace for Mac v. 22.06 since today, and because of this the current recipe doesn't seem to work anymore in Jamf.
So some mac clients updated succesfully with Installomator 9.2 yesterday, I also downloaded 22.06 manually from Citrix.com (for PreStage purposes etc.); now I only find the previous version, 22.04 on Citrix and the Installomator recipe currently produces download errors (“URL using bad/illegal format or missing URL”). Tried some previous versions of Installomator, 9.1, 9.01, but makes no difference. Thanks in advance for hints
Installomator.log

It seems the site has been redesigned.. they are still pointing some links to https://www.citrix.com/downloads/workspace-app/mac/workspace-app-for-mac-latest.html but if you browse to https://www.citrix.com/downloads/workspace-app/mac/workspace-app-for-mac-latest1.html then you can download 2204, and updating that URL in the label works for now too.
2206 is now showing back under the Technology Preview version too.. we may not want to update the label yet if Citrix is doing something weird with this release, but you can update the URL so your label continues installing 2204 for now. I expect we'll need to watch the website carefully for any more changes.

We've got reports the label works as is now, like Citrix fixed the issue and published 2206. I'll let you verify and then we can close this Issue.

Thanks for the quick reply, the recipe is working again as expected.
I noticed by the way, when manually downloading the "new" 22.06 installer, it's not the same file - obviously a different build (although the 22.06 from 2 days ago wasn't downloaded from the Tech Preview channel... Haven't looked deeper into this though, in our case it's not that important).
Closing the issue...


VirtualBox startup failures even after usual post-CrOS upgrade fixes

Hmm. After the most recent CrOS upgrade, I'm getting the typical:
Kernel driver not installed (rc=-1908)
error when starting VirtualBox. However, my usual scripts to fix this -- setting the kernel flags, setting rw -- while apparently successful in their actions -- have not fixed the error this time.
Before I start screwing around with the kernel headers and dig myself in deeper, has anyone else seen a problem with this cycle?
I've already done a crouton update, and updated VirtualBox.
Any suggestions?  Thanks!

@laurenweinstein1 -
Same thing happenned to me and I'm still unable to fix it.
Normally, I run the change-kernel flags and sometimes I've had to run setup-headers in the chroot
but this time I get nowhere with that. I also upgraded VirtualBox, I guess I may have borked something doing that.
We'll have to wait until @divx118 or someone else has time to take a look or can hold our hand through it.
-DennisL

Thanks. Will hold off tinkering with it myself!
L
On Wed, Mar 30, 2016 at 3:00 PM, DennisL<EMAIL_ADDRESS>wrote:

@laurenweinstein1 https://github.com/laurenweinstein1 -
Same thing happenned to me and I'm still unable to fix it.
Normally, I run the change-kernel flags and sometimes I've had to run
setup-headers in the chroot
but this time I get nowhere with that. I also upgraded VirtualBox, I guess
I may have borked something doing that.
We'll have to wait until @divx118 https://github.com/divx118 or someone
else has time to take a look or can hold our hand through it.
-DennisL
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/dnschneid/crouton/issues/2508#issuecomment-203657347

Anyone have any ideas on this? Dead in the water here. Thanks.

New CrOS version came in. Did the usual fixes. Still no joy.

@laurenweinstein1 @DennisLfromGA  which kernel are you guys using? uname -a

Still 3.8.11. Thanks.
L
On 04/02 07:35, Maurice van Kruchten wrote:

@laurenweinstein1 @DennisLfromGA  which kernel are you guys using? uname -a

Ok 3.8.11 I am building now. It will be up in about 15 minutes

Super. What's the safest install technique? Thanks again!
L

Normally you can just run sudo apt-get update && sudo apt-get upgrade  on the commandline in the chroot.

Got it. Much appreciated.
L

Ok done they are up now https://github.com/divx118/crouton-packages/commit/4d0cc4786a3d4b4942e81a7397990db226bec5d2

Did the sudo apt-get update && sudo apt-get upgrade -- nothing header
related came in and drivers still not loading. Thanks.
L
On Sat, Apr 2, 2016 at 8:10 AM, Maurice van Kruchten <
<EMAIL_ADDRESS>wrote:

Ok done they are up now divx118/crouton-packages@4d0cc47
https://github.com/divx118/crouton-packages/commit/4d0cc4786a3d4b4942e81a7397990db226bec5d2
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/dnschneid/crouton/issues/2508#issuecomment-204731882

@laurenweinstein1 Try running the setup-headers script again.
Open a shell in your chroot:
$ cd ~
$ wget https://raw.githubusercontent.com/divx118/crouton-packages/master/setup-headers.sh
$ sudo sh setup-headers.sh

I'm at the:
You are attempting to install a kernel image (version 3.8.11) However,
| the directory /lib/modules/3.8.11/kernel still exists.  If this
| directory belongs to a previous linux-image-3.8.11 package, and if you
| have deselected some modules, or installed standalone modules packages,
| this could be bad.
selection, seeking OK or move /lib/modules/3.8.11/kernel out of the way.
How do I proceed. I've gotten burned by this before and had an unbootable chroot when
I apparently made the wrong choice.
Thanks.
L
On 04/02 08:44, Maurice van Kruchten wrote:

@laurenweinstein1 Try running the setup-headers script again.
Open a shell in your chroot:
$ cd ~
$ wget https://raw.githubusercontent.com/divx118/crouton-packages/master/setup-headers.sh
$ sudo sh setup-headers.sh

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/dnschneid/crouton/issues/2508#issuecomment-204741029--Lauren--
Lauren Weinstein (lauren@vortex.com): http://www.vortex.com/lauren
Founder:

Network Neutrality Squad: http://www.nnsquad.org
PRIVACY Forum: http://www.vortex.com/privacy-info
Co-Founder: People For Internet Responsibility: http://www.pfir.org/pfir-info
Member: ACM Committee on Computers and Public Policy
Lauren's Blog: http://lauren.vortex.com
Google+: http://google.com/+LaurenWeinstein
Twitter: http://twitter.com/laurenweinstein
Tel: +1<PHONE_NUMBER> / Skype: vortex.com
I have consulted to Google, but I am not currently
doing so -- my opinions expressed here are mine alone.

It's Groundhog day all over again. Even though I aborted the last step of
the header install as described above, headers seem confused. Ubuntu
updater keeps wanting to reinstall them, and crouton-update is failing with
header-related messages. Haven't I been here before? Also, after I ran the
header update shell file (and aborted at the last step), I'm now seeing an
rc.d error from apt-get. I have to grab it next time. I already moved back
my rc.local (that already has the module loading section in it).
On Sat, Apr 2, 2016 at 8:51 AM, Lauren Weinstein<EMAIL_ADDRESS>wrote:

I'm at the:
You are attempting to install a kernel image (version 3.8.11) However,
| the directory /lib/modules/3.8.11/kernel still exists.  If this
| directory belongs to a previous linux-image-3.8.11 package, and if you
| have deselected some modules, or installed standalone modules packages,
| this could be bad.
selection, seeking OK or move /lib/modules/3.8.11/kernel out of the way.
How do I proceed. I've gotten burned by this before and had an unbootable
chroot when
I apparently made the wrong choice.
Thanks.
L
On 04/02 08:44, Maurice van Kruchten wrote:

@laurenweinstein1 Try running the setup-headers script again.
Open a shell in your chroot:
$ cd ~
$ wget

https://raw.githubusercontent.com/divx118/crouton-packages/master/setup-headers.sh

$ sudo sh setup-headers.sh

---
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:

https://github.com/dnschneid/crouton/issues/2508#issuecomment-204741029--Lauren--
Lauren Weinstein (lauren@vortex.com): http://www.vortex.com/lauren
Founder:

Network Neutrality Squad: http://www.nnsquad.org
PRIVACY Forum: http://www.vortex.com/privacy-info
Co-Founder: People For Internet Responsibility:
http://www.pfir.org/pfir-info
Member: ACM Committee on Computers and Public Policy
Lauren's Blog: http://lauren.vortex.com
Google+: http://google.com/+LaurenWeinstein
Twitter: http://twitter.com/laurenweinstein
Tel: +1<PHONE_NUMBER> / Skype: vortex.com
I have consulted to Google, but I am not currently
doing so -- my opinions expressed here are mine alone.

@laurenweinstein1 You should have just completed the setup-headers script. by overwriting the kernel. Also it is wise to create a backup of your chroot, so you can always restore.
I just updated the 3.8.11 headers on my system with no problems. After rebuilding the vboxdrv modules they loaded with no problems.

I reran the setup-headers script and let it run to completion including the kernel overwrite without obvious errors. Ubuntu updater has stopped asking to update the headers. Crouton-update ran to completion OK. However, the drivers are still not loading, e.g.:
modprobe: ERROR: could not insert 'vboxdrv': Invalid argument

@laurenweinstein1 You need to recompile the drivers with the new headers installed. If you installed virtualbox from oracle just run sudo /etc/init.d/vboxdrv setup to do that. If you installed virtualbox from ubuntu repo then you have no /etc/init.d/vboxdrv script. I forgot how to do it then. I will see if I can find it again.

@divx118 That did it (using the script for directly installed VB). Back up. Thanks!!
L

@laurenweinstein1 Great, can you please close this issue then. Thanks


Automate generating release notes

What would you like to be added:
There should be some sort of automation that generates release notes and publishes it on the website at the end of each release.
Why is this needed:
The current manual process is tiresome and not scalable.

Closing in favour of https://github.com/submariner-io/releases/issues/454.


⚠ SceneTime has degraded performance

In ca77922, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1510 ms

Resolved: SceneTime performance has improved in 2e05403 after 6 minutes.


Eliminate Dependency on jQuery - Tasklist

[ ]  Create a list of all jquery functions SUI uses on the left and their modern alternatives on the right.
[ ] next task

Looks good. We can probably start with http://youmightnotneedjquery.com

I'm only a user of a few modules, not an active contributor. Jack is the expert here.

It could be worth looking what @hallister and @asvetliakov have done for eliminating the jQuery dependency in semantic-react.
There is also an interesting discussion about replacing jQuery with React based modules at jessy1092/react-semantify#20.

I was talking to @jlukic before about dropping jQuery from SUI ... it is basically not going to happen. The JS module code is heavily dependent on jQuery. I learnt to live with it.

What about migrating to jqLite first?
This way, the first step would be to replace DOM accessors. Angular users would benefit from it I guess.

Problem with migrating to jqLite is that it is possible to hit the problem of missing implementation.
It migth not be the easies but moving to vanila js or maybe TS(my preference) seems like a proper way.

@SzymonSasin what's TS?

I think TypeScript

Adding our React implementation to this list, Stardust :)  My previous comment on a related thread:

At TechnologyAdvice we are using https://github.com/TechnologyAdvice/stardust in production.  We have 6 humans and 2 robots working on it so far.  I spend the majority of my day job on Stardust.
I am rewriting the modules without jQuery.  We first tried to keep SUI DOM manipulations in sync with  React state through each of the library's lifecycle methods, in the end, this just was not realistic.
I've rewritten the Dropdown with selection, search, and complete keyboard navigation.  You can try it out here.  I will continue to move forward with all other modules as well.  If any one is interested in assisting, let me know.

@avalanche1: I mean typescript, I have edited my previus post.
@levithomason: it is great, but what is a point of moving from jQuery to React? People which are using any other framework than React might have problems. I think that possible solution is to add a thin layer to allow the use of differenet frameworks, similair solution to angular ones.

My comment was in regard to https://github.com/Semantic-Org/Semantic-UI/issues/3981#issuecomment-216186917

It could be worth looking what @hallister and @asvetliakov have done for eliminating the jQuery dependency in semantic-react.
There is also an interesting discussion about replacing jQuery with React based modules at jessy1092/react-semantify#20.

For those looking for a React implementation of SUI.  Since no matter the choice of JS SUI uses for DOM manipulation, it will always be incompatible with React's virtual DOM.

@levithomason Glad you're going fine with stardust :smiley:

Moving to "react" would be kind of bad for a lot of people.
@Assassyn one could also use Flow instead of TS, which is less intrusive.

To clarify, the React components are not intended to help non-react apps.  They are required for React apps as vanilla JS and jQuery DOM manipulations do not work in React.

Maybe something tiny like https://umbrellajs.com/ (I'm not affiliated) could handle this. Umbrella is mostly jQuery compatible, and would not also bring on the pain of recreating everything jQuery had been doing.
@franciscopop - any interest?

@alanguir thanks for proposing my library and yes, I'd be really interested. Though I have to ask first, what is the reason to remove jQuery? And is jQuery available for people using the library or just used internally?
The codebase is really modular though many modules are tightly coupled (to optimize for size) so you could just remove some parts if they are not needed. It stands at 3kb min+gzip right now. Some functions might also be missing so if it makes sense I'd be willing to implement them inside umbrella or as a separate file.
PS, great work with Semantic UI, I have known it for a while and I love it

As chamadas ao jquery no semantic são estas:

$.Deferred,
$._data,
$.address,
$.ajax,
$.api,
$.contains,
$.cookie,
$.data,
$.each,
$.easing,
$.events,
$.expr,
$.extend,
$.grep,
$.inArray,
$.isArray,
$.isEmptyObject,
$.isFunction,
$.isPlainObject,
$.isWindow,
$.merge,
$.removeCookie,
$.site,
$.tab,
$.test,
$.trim,
$.visit,
$.when

It also uses $.fn a lot.


FreeOrion.FreeOrion version 0.5

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

The number of graphics-based applications needing validation has significantly increased lately.


gps messages should not be stored if they are invalid

currently while gps message is parsed it overrides previously stored gps data.
we don't want to loose previous data if current message is invalid (this is in case we don't receive one radio message so it's safer to transmit last known good position instead of invalid gps data).

done


Checkpoint  can not access

Hi,
I am trying to download the checkpoints of your model but the link can not visit.
I have found the links in issues, but they all do not work.
Could you provide other available access links?
Thanks a lot.

Does this link work?

Does this link work?

It works. Thank you so much!!!


Update README.md

Fixed compilation error (it's a double now) and deduped the instrument name.

This PR does not fix compilation error, it introduces one.

Just FYI the README says to use version 0.1.3, which was released 1 day before the data type was changed from Double to String :-)

@ScalaWilliam new version released, thanks


Cannot decrease fold level if the current level contains empty (unfoldable) headings

Say you have the following note
# 1.
## 1.1
blabla
## 1.2
blabla
# 2.
## 2.1
## 2.2

This will prevent the plugin from decreasing the heading fold level to heading level 1

I'm glad you identified what's been causing me to be unable to fold! I have put a newline after each empty heading to allow `Decrease heading fold level` to work appropriately as a temporary solution.
I think it would be great if Creases folded all headings where it was applicable, and skipped those where it was not. Then it could continue to decrease heading fold at the next level.


Added delete permission to pod-reader

Summary
The RBAC yaml is missing the delete permission. This PR fixes that
Fixes dotnet/orleans/issues/8248

@dotnet-policy-service agree

closing and reopening for a fresh CI build.


[d3d11] Rendering artifacts in Barn Finders

Screenshot:

It's looks like what supposed to be light that get through trees on the back (or it's supposed to be shadows of these trees? I not sure) is rendered as constantly flickering black triangles.
After starting new game issue is reproducible with other effects too, so it's not specific to game main menu.
How part from screenshot is supposed to looks like: https://youtu.be/QiA5MHeWtQM?t=587
Software information
Barn Finders, default settings (high textures, all effects enabled)
Select low textures and disabling all effects doesn't make any difference.
System information

GPU: AMD Radeon Vega 64
Driver: Mesa 20.1.3
Wine version: Proton-5.11-GE-2-MF (game doesn't work with vanilla Proton 5.0-9)
DXVK version: v1.7-63-gc4f37d39-async

Apitrace file(s)
I don't have Windows so I can not record apitrace there.
I tried to create apitrace on Linux by following this DXVK wiki article (tried to do so with PROTON_USE_WINED3D=1) but trace file didn't appear in game folder.
Log files

d3d9.log: empty
d3d11.log: https://gist.github.com/RussianNeuroMancer/f7ca54c0c233e12f824f5c0246237d2b
dxgi.log: empty

but trace file didn't appear in game folder.

Check the desktop folder inside your wine prefix.

 How do I get around that?

Ok, got the game to run, works fine on my end (mesa-git, RX 480).

DXVK version: v1.7-63-gc4f37d39-async

Please don't use unofficial builds to report issues.

@doitsujin

works fine on my end (mesa-git, RX 480)

Tested with 20.2~git2007160730.081691 from Oibaf PPA, outcome is still the same:

Anyway, if you can't reproduce this - driver bug then?

Please don't use unofficial patched builds when reporting issues.

I tried to install official DXVK build via protontricks before writing this bugreport, but after I did this game stopped working, so I had to re-create WINE prefix.
@K0bin

Check the desktop folder inside your wine prefix.

Empty here. (Checked Desktop folder in both user profiles created by Proton.)

Please also try the game with Microsofts d3dcompiler and d3dx11 DLLs.

Tried, unfortunately artifacts didn't go anywhere.

Anyway, if you can't reproduce this - driver bug then?

Yeah, might be a Vega-specific issue.

Yeah, might be a Vega-specific issue.

Registered issue at Mesa bugtracker: https://gitlab.freedesktop.org/mesa/mesa/-/issues/3311


When searching for font, suggestFontSize is ranked lower than formatOnSave

Issue Type: Settings Search Issue
When searching for font in the settings search, suggestFontSize is ranked lower than formatOnSave
VS Code version: Code - Insiders 1.25.0-insider (b4a18da6e78f95295ea6538f5ae8dd0c9f0869a9, 2018-06-19T07:38:09.923Z)
OS version: Darwin x64 15.6.0
Extensions (7)

Extension
Author (truncated)
Version

npm-intellisense
chr
1.3.0

vscode-markdownlint
Dav
0.18.0

vscode-eslint
dba
1.4.12

tslint
eg2
1.0.32

vscode-npm-script
eg2
0.3.4

mssql
ms-
1.3.1

vscode-icons
rob
7.23.0

Results

Setting
Extension
Score

workbench.fontAliasing
core
5.004

editor.fontLigatures
core
5.001

editor.fontFamily
core
4.459

editor.fontWeight
core
4.293

editor.fontSize
core
4.143

terminal.integrated.fontSize
core
4.050

editor.suggestFontSize
core
3.838

terminal.integrated.lineHeight
core
3.789

editor.lineHeight
core
3.685

terminal.integrated.fontFamily
core
3.670

markdown.preview.fontFamily
core
3.372

markdown.preview.fontSize
core
3.202

markdown.preview.lineHeight
core
2.733

scss.lint.fontFaceProperties
core
2.666

less.lint.fontFaceProperties
core
2.493

css.lint.fontFaceProperties
core
2.436

terminal.integrated.fontWeight
core
0.968

terminal.integrated.fontWeightBold
core
0.389

editor.tokenColorCustomizations
core
0.319

editor.mouseWheelZoom
core
0.307

Query: font
Literal matches: 19

These results come from the local filtering, not bing search which is ranked by relevance. We can rank local results by relevance too, there's an issue for it but local search isn't set up for that right now.


Remove hash-base patch

Fixes # .
Changes proposed in this pull request:

This dependency is used by Torus, tested with Torus, I am able to vote and join space without any error


pre-commit hook adds unstaged files to the staging zone and then to commit itself

Describe the bug
Files from Untracked and Changes not staged for commit zone are pulled into the staging  (and subsequently into commit) by npm run license:fix script run by pre-commit githook.
How to reproduce

have three changed files in your tree: one in staged zone (Changes to be committed), one in Changes not staged for commit and another one in Untracked files:

user@mac:clarity$ git status
On branch func-return-value
Changes to be committed:
  (use "git reset HEAD <file>..." to unstage)

	modified:   packages/core/src/modal/modal.element.ts

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

	modified:   packages/core/src/alert/alert-actions.element.ts

Untracked files:
  (use "git add <file>..."  to unstage)

	new file:   .gitmessage.txt
	modified:   packages/core/src/alert/alert-actions.element.ts
	modified:   packages/core/src/modal/modal.element.ts

Both un-staged files got moved into the staged (Changes to be committed) zone and would have been included into resulting commit had I run git commit instead of npm run license:fix.
Expected behavior
Unstaged files are ignored by git hook scripts (linters, checkers, etc) and aren't pulled into the commit automatically.
Versions

Type: MacBook
OS: iOS
git: 2.21.1

Additional notes
Not sure whether this is bug or a feature but wanted to check before looking into ways of fixing it.
Thanks

Yes, this is a bug in the build that has not been addressed yet.

Looks like the issue is with this line of the script: https://github.com/vmware/clarity/blob/master/scripts/update-license.js#L34


Allow setting updateStrategy as helm value

According to
https://kubernetes.io/docs/concepts/workloads/controllers/statefulset/
"The RollingUpdate update strategy implements automated, rolling
update for the Pods in a StatefulSet. It is the default strategy
when .spec.updateStrategy is left unspecified."
So, this change will not introduce any behavior change unless the
defaults of the new values are overwritten.

Coverage decreased (-0.07%) to 76.599% when pulling 5b6e996c06b51a9a99851f64bf694789b3106828 on nbartos:helm into f272e4c80f3437e098e31516f9ef9e75c746a4e3 on contiv:master.

Hi,
this PR broke the default deployment:
k8s-master: error: error validating "/tmp/contiv-vpp.yaml": error validating data: ValidationError(Service.spec): unknown field "updateStrategy" in io.k8s.api.core.v1.ServiceSpec; if you choose to ignore these errors, turn validation off with --validate=false

(updateStrategy cannot be specified for a service)
I will prepare a fix.

Fixed in https://github.com/contiv/vpp/pull/831, tests are passing again.


Pass in form styling - DO NOT MERGE YET

This could be the corresponding changes to match the pass in form styling pull request on passport-verify. The template version here makes me slightly worried (particularly the inserting unescaped html into the template to render the form), although it seems likely that it's something rps may want to do (the alternative being to concatenate strings of html in the wrapper function - see the first of these commits).
See passport-verify pull request
It would need an actual version release of passport-verify and then updating the version number here before it can be merged.

I'm going to close this until someone needs the feature. When they do we can consider the right way to do it.


Revert renaming of type and next structure members.

These were renamed because these member names match python built-in functions, which could theoretically lead to conflicts.
But it's a small problem, which mostly affects the implementation details of the structure constructors. Meaning it might eventually be a small problem for us, but not for our users. Plus this has been a source of bugs #61. The known bugs are fixed but there still might be some unknown brittleness remaining.
In the interest of maintaining transparent similarity to the C API, we should use the original member names.
By the way, if we ever encounter a case where a python keyword (not just a built-in function) matches a member name or parameter name, we would probably need to re-invoke this mechanism for those keyword-matching parameters.
I'm still inclined to keep these parameters at the end of the constructor parameter list in the python API. In the C API they are at the beginning of the parameter list. I moved them to the end because they are some of the least-frequently needed parameters, so having them at the end allows us to omit mentioning them in the most common uses, when explicit parameter names are omitted.
@el-dee do you agree?

Fixed for 1.0.2401


*: use staticcheck instead of megacheck

What problem does this PR solve?
The CI of release-2.1 branch has been broken since megacheck is deprecated.
What is changed and how it works?
This PR fixes this problem by using staticcheck and also fixes a lint problem.
Check List
Tests

Unit test

/run-all-tests tidb=release-2.1 tikv=release-2.1 tidb-test=release-2.1

ci failed.

/run-all-tests tidb=release-2.1 tikv=release-2.1 tidb-test=release-2.1


[BUG] Get-PnPTeamsChannel 'MembershipType' property returns 'unknownFutureValue' if Teams Channel is a Shared Channel

Expected behavior
Returned value should be 'Shared'
Actual behavior
Returned value is 'unknownFutureValue'
Steps to reproduce behavior
$channel = Get-PnPTeamsChannel -Team "Team 1" -Identity "Shared Channel 1"
$channel.MembershipType
What is the version of the Cmdlet module you are running?
2.4.0
Which operating system/environment are you running PnP PowerShell on?

[ X] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

According to https://learn.microsoft.com/en-us/graph/api/resources/channel?view=graph-rest-1.0#properties :

Note that you must use the Prefer: include-unknown-enum-members request header to get the following value in this evolvable enum: shared.

Testing this in Graph Explorer without and with the include-unknown-enum-members header:

Without include-unknown-enum-members

With include-unknown-enum-members

A bit unsure if this header should be added everywhere when calling GET /teams/{team-id}/channels as that will change the value returned, so letting the maintainers decide.

I opened a PR at https://github.com/pnp/powershell/pull/4054 with the least invasive changes, include the header only when needed.


How to generate valid derived public addressed (on test network)

Hi, I am practicing generating multiple derived addresses from single master public address.
This code gives me an address:
$master_pub = "tpubD9EDMRKYRLyZNCZxvJDS6tueAkDVjdajrVZGv7rmPoaNZZEUGX5HJvDnBKi5EVG7ivx9tAcbjjEhEHwdkox3pP5iKQZ9FVFHHZoimQ5HwVh";

$key = HierarchicalKeyFactory::fromExtended($master_pub, NetworkFactory::bitcoinTestnet());

echo $key->derivePath("0/0")->getPublicKey()->getAddress()->getAddress();
// addr: 1DtkafCbLYJ2VJJZF4ws5uhnUyMUBitXTj

But it fails validation on testnet blockchain explorer:
https://www.blocktrail.com/tBTC/address/1DtkafCbLYJ2VJJZF4ws5uhnUyMUBitXTj
Why is that, what am I doing wrong?

Hmm, addresses don't have a definite size when encoded, it's possible there's a sanity check failing on blocktrail since bitcoind says it's valid too.

Hey, wait a second, you were putting that address onto testnet! I don't know how that escaped me. Of course it's invalid.

@lezhnev74 Hey, I see what the problem is now, you're querying testnet! That's why it's invalid, that's a mainnet address :)

@afk11 thanks for detailed examples! I will try them and will get back to this thread.

No worries at all! Would happily accept suggestions on make it a bit safer to use also. I don't want people to feel forced into setting the network globally to avoid these cases.
Might write something that would cause Bitcoin::getNetwork() to throw a DefaultsDisabledException if the default is ever relied upon so there's a way to guarantee run-time safety (even if you were consciously making an effort to pass the network everywhere, all it takes is forgetting in one place to cause application errors or having funds sent to the wrong network)

Closing, query resolved.


Downloading and uploading curves

It is possible to download curves (load on nodes and profiles). But this the value seems to be very strange and not corresponding to the load curve of nodes. 15 min values of more than 10^20 for just around 100 households.

Assigned @grdw

When I try to upload a direct downloaded csv file from ET Moses back to ET Moses it doesn't work. And this is Always the case for own made csv files.

But this the value seems to be very strange and not corresponding to the load curve of nodes. 15 min values of more than 10^20 for just around 100 households.

Can you paste a link to the LES where you see this behaviour? The CSV file describing the load of an LES node should be exactly the same as the load chart.

http://beta.moses.et-model.com/testing_grounds/251
It seems to correspend but only a factor 10^12 to big.
20 kW vs<PHONE_NUMBER>7165600

It seems to correspend but only a factor 10^12 to big.

Could it be that you use comma's as decimal separators? ETMoses uses periods. Perhaps the<PHONE_NUMBER>65600 are just the decimals after the 20 :smile:

We downloaded the load csv file from end point portiek1a1. Values seem to be correct using period as decimal separator. @JeroenvdLogt you can change the setting for decimal separator for Excel in Windows.

I changed the seperator accordingly to the instruction but it is still nog functioning.

For downloading it is functioning. For uploading I will make a new ticket.


fixes date/time formatting.  fixes error handling of object GET requests...

... that are out of range; will now properly display uframe message
@DanielJMaher
The big blocks are just being wrapped in try/except.  The error handling wasn't there before for better testing, now I think it's ready for a spit shine.
The "pass" in the except is to allow the error message from uframe, which is formatted well.

@DanielJMaher, need to get this in for Sontag's work.  Has a fix for post/put to events for uframe.


[IOS] onCurrentRecordingWaveformData returns currentDecibel with delay

onCurrentRecordingWaveformData returns incorrect values.
I speak fast to test and say "HEY"  and the values are like this
 LOG  0.012771714478731155
 LOG  0.012771714478731155
 LOG  0.012771714478731155
 LOG  0.012771714478731155
 LOG  0.012771714478731155
 LOG  0.012771714478731155
 LOG  0.012771714478731155
 LOG  0.012771714478731155
 LOG  0.012771714478731155
 LOG  0.012771714478731155
 LOG  0.01845887303352356
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6516658663749695
 LOG  0.6115133166313171
 LOG  0.5385453104972839
 LOG  0.47425442934036255

this is the output visualization of saying just  "HEY"
But on Android the output looks like this

As you can see values are jumping from 0.01 to 0.6 which is fine, but returns to 0 slowly. I have no idea what can cause this, but it works fine on Android.
UPDATE: tried to remove linear function in native side, but did not work

@Bayramito how do you access native events like onCurrentRecordingWaveformData?

@Bayramito how do you access native events like onCurrentRecordingWaveformData?

there are some hook provided, if you check the core files. useAudioRecorder and useAudioPlayer via these...


I modify CLASSES = ('aircraft',) in voc.py, but demo and eval result shows 'seroplane'?

I modified CLASSES = ('.....') to CLASSES = ('aircraft',) in voc.py. I trained my own dataset.
But when i ran demo.py,test.py and voc_eval.py, the result label is named aeroplane? Why does this happen?

i meat the same question. have you finish it?

I resolved it.
change the voc_classes() in mmdet/core/evaluation/classesnames.py

when install the mmdetection, you shold use python setup.py develop

@wizholy i use python setup.py, the problem still exists. It seems that the project only supports the coco dataset class, but not the voc class.

Have you solved this problem?

I met the same question , can you give some advise?

你解决这个问题了吗？

Have you solved this problem?
兄弟你解决了吗？


rename test folder

rename test folder and modify setup.py to run tests

Oh nice thanks for sending the pull to fix #20.  I'll check it out and integrate it shortly.

Awesome, the tests work great and are super easy to run now.  Thanks!  I also fixed up a few tests I had broken with some earlier changes so everything should be passing now.  I'll also update the readme to mention that tests can be run by installing mock and then using setuptool's runner.  Thanks!


Fetch signerFee from SwapContract and use instead of hard coded values

Currently 1.0007 is used in a few places for last look calculations, and text around the interface says 0.07%. We should use the onchain value for future proofing.

Hi @gpxl-dev, regarding this issue, do I fetch the signerFee from the SwapContract instance, ie.
const contract = ethers.Contract(contractAddress, contractAbi, provider); const signerFee = contract.protocolFee()
Please advise if I'm in the right direction?

@francisldn correct, this is the right direction. Note that AFAIK current deployments still use signerFee instead of protocolFee. Also note that contract.protocolFee() would return a promise, so you would need to assign signerFee to the value it resolves to rather than the promise itself.


Create env State from an observation and guide for how to create a LuxAI2022 Env and perform rollouts

+1
I guess this is about implementing this?:
    def from_obs(obs):
        # generate state from compressed obs
        pass

Yep its that, there's a bit of extra work though since the observation currently doesn't include some information (like global_id which is used to generate unit ids)

Another feature request related to this. In starter kits, provide an option in main.py to provide stacked future observation frames where they execute remaining actions for k turns.

part 1 done by #170


[VS code extension] vs code no responding when open a .lg file

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
v4.6.0-Daily-2019-07-31-01
Describe the bug
When I open this lg file and try to edit the last function vs code will have no responding.
Shared.txt
To Reproduce

open this .lg file with vs code
edit the last function
vs code will have no responding

Expected behavior
vs code works well.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
[bug]

@boydc2014 - FYI.

Fixed via #1241


Update links for better navigation in Open Source Page

What type of PR is this? (check all applicable)

[X] ♻ Refactor
[ ] ✨ New Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 👷 Example Application
[ ] 🧑💻 Code Snippet
[ ] 🎨 Design
[ ] 📖 Content
[ ] 🧪 Tests
[ ] 🔖 Release
[ ] 🚩 Other

Description
This PR fixes this the navigation error by taking the user to the top of the desired page.
Related Tickets & Documents
Resolves #808
Mobile & Desktop Screenshots/Recordings

Added code snippets?

[ ] 👍 yes
[X] 🙅 no, because they aren't needed

Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

No tests? Add a note

Added to documentation?

[ ] 📜 readme
[ ] 📜 contributing.md
[ ] 📓 general documentation
[X] 🙅 no documentation needed

No docs? Add a note
Documentation not required due to link update is making navigation easier and no major change in design or code.
[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

@Smartmind12 Thank you so much for looking into this. However, since Docusaurus resolves URLs on its own, If we change the URLs to this, we won't be able to test them locally because it'll always point to the website. I'm going to close out these PR's. The issue you created is helpful I just don't think this is the right solution. I'll comment on the issue and we can discuss it further.


Basic effect stack reshuffling utilities

You don't have raiseUnder, or any generalization thereof (e.g. Polysemy's insertAt). Just raise, subsume, and inject (and I don't consider inject -- a function that can't be used instead of raiseUnder without requiring you to carry a constraint wherever you go -- a valid solution.)
Pardon my french, but that is bonkers. Across all my work with effect systems, these kinds of effect stack reshuffling utility functions have proven essential whenever I experiment with any even remotely zany effect, and are very useful for any regular user as they can be leveraged to e.g. take an existing interpreter and turning it into a reinterpreter.
I recommend you have a few couple of basic raiseUnder variants, plus a more ambitious, general-as-possible-while-still-usable function like insertAt. Hell, you're free to crib insertAt's implementation if you like. I won't mind (though 9.2 breaking incoherent instances means it no longer generates as nice type errors... oh well.)
If you do introduce raiseUnder and friends, don't repeat our mistake: raiseUnder2 should introduce one effect under the two top effects of the stack, and raise2Under should introduce two effects under the top effect of the stack. Not the other way around, like Polysemy does it.

As I mentioned here, I have a plan to generalize inject so that these use-cases automatically resolve the Subset constraint.

general-as-possible-while-still-usable function like insertAt

Looks like after this generalization inject will have the same utility as insertAt (perhaps even better since inject doesn't need Nat, but maybe insertAt also works if you don't provide it).

I recommend you have a few couple of basic raiseUnder variants

My reason for not providing them was that inject will eventually be sufficient, but also, IIUC these functions should be needed very rarely, if only because effectful is not polysemy and the order of effects doesn't matter.

these kinds of effect stack reshuffling utility functions have proven essential whenever I experiment with any even remotely zany effect

Can you provide examples so that I can better understand this use-case?

Fixed by generalization of inject.


[YUNIKORN-1231] Add e2e tests for the entire preemption flow

What is this PR for?
Added e2e test for simple premptor with below cases:
Case 1: only one pod is running on specific node and same has been selected as victim
Case 2: three pods are running on specific node, 2 opted out pods and 1 regular pod. Regular pod has been selected as victim.
What type of PR is it?

[ ] - Improvement

Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-1231
How should this be tested?
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

recheck

@wilfred-s can you please take a final look and merge? @pbacsko had already reviewed the code.


Surya/core 5734/team create channel

syntax is
keybase chat create-channel westview_alums --team --set-channel 'cybersec-vanguard'

@mmaxim - done. I added a new resolver behavior that exits out if the conversation is already found. And I removed the initial metadata message, so all this does is send a message to a new channel name but error if it already exists. Is adding a new resolver behavior the right way to do it?


When using Azure DevOps CI/CD for a Linux web app how can I zip and publish the build artifact as a zip file only instead of copying to directory

Per these instructions I am creating devops project integrating CI/CD for a Linux Web App.
Here is my initial Pipelines pipelines/build job
trigger:
- master

pool:
  vmImage: 'ubuntu-latest'

steps:
- task: NodeTool@0
  inputs:
    versionSpec: '12.x'
  displayName: 'Install Node.js'

- script: |
    npm install -g @angular/cli
    npm install
    npm run build:ssr
  displayName: 'npm install and build'

- task: CopyFiles@2
  inputs:
    Contents: |
      dist/**
      node_modules/**
      package.json
    TargetFolder: '$(Build.ArtifactStagingDirectory)'
    CleanTargetFolder: true
    OverWrite: true

- task: PublishBuildArtifacts@1
  inputs:
    PathtoPublish: '$(Build.ArtifactStagingDirectory)'
    ArtifactName: 'drop'
    publishLocation: 'Container'

When because the CopyFiles@2 task there are 65000 files copying over to the directory to be published in the following PublishBuildArtifacts@1 task.
Because of this I thought I could add the Artifact task after the copy task and this would produce a .zip file and make the deployment more efficient and many times over faster.
However, this was not the case and all it did was serve to zip the folder up, un-zip the folder and proceed to do the same individual file processing like show below. At the time it took me to write this post it has been about 14 minutes and this is how far it has gotten.
 Should I not copy and then zip but rather just zip? Is this because it is the Linux pathway vs the Windows pathway.
As of now this would not be possible to utilize as it would take an hour for a deployment.

Try to use the  PublishPipelineArtifact instead of the PublishBuildArtifacts, the first upload will be slow, but the subsequent uploads will be way faster because the task will upload only the diff between the previous artifacts and the new artifacts.

@xtianus79 - are you still having any issues after the above suggestion of using PublishPipelineArtifact? We do recommend using Pipeline Artifacts instead of Build Artifacts.

Hi,
If you still experiencing this issue, please reopen it.
Thanks


Use "Football" instead of "Soccer" in market templates.

Description
Removes references of Soccer in the templates to Football (Soccer).
Issue
#5257

discussing with product


Add ADR for how we support Jenkins.

It looks good to me. However, there is something I would like to discuss with @Digitiain (Ian) before approving. I will do that at tomorrow's standup.


improve grid.svg

More familiar, Gnome's Dash to Dock like

This could be further advanced by changing a few colour settings, as majority may prefer a gnome shell like experience. However, all boils down to user preference. What do you think?

Well, I'm on vacation, with only an old laptop and a slow internet connection, but you seem to have changed 14 files. Check what's wrong with your PR, please.

well i ran a chmod +775

chmod +755 */*

why?

it allows user to modify files without requiring root

You want to change a single file, right? Do whatever you want on your machine, but not on the repo.

Alright, in that case I'll make more changes which you can review later and leave file permission as it is


Possible to authenticate with token via middleware?

Not an issue, but an authorization question:
After reading through the docs on middleware, I'm curious if there is a way to read an authentication token from a cookie and pass it to the authentication middleware (using either the API scheme or JWT scheme).
My global middleware looks like this:
'use strict'

class TokenGetter {

  * handle (request, response, next) {
    // Read the token from cookie and decorate the request
    const token = request.cookie('token')
    request.token = token

    // Pass it along
    yield next
  }

}

module.exports = TokenGetter

Is it possible to authenticate users with the built-in authentication middleware with a method like this?

You can set the Authorization header via middleware

Sorry if this is a silly question, but how would I set headers with Adonis? After reading the docs, I see that it is very easy to read header values, but am confused on how to set them.

You can say following inside a middleware
request.request.headers['authorization'] = 'your token'

Make sure to call the above middleware before the authentication middleware and define it here https://github.com/adonisjs/adonis-app/blob/develop/app/Http/kernel.js#L32

Got it. Thanks so much for your help with this.


Fixed aspect ratio to 1:1 for drop cards

lets make the imgs fixed aspect ratio of 1:1, so its perfect square, keep the layout breakpoints the same, the cards will just become taller

nominating ninja @hassnian as he did most of the landing page


how to log python file

Describe the bug
Hi, I have been analyzing some C files, and I have noticed how awesome your project is.
But when I try to analyze python files, I didn't find any documentation :(. I have tried this like if it were C file but obviously didn´t work:
CodeChecker log -b "python3 my_file.py" --output ./my_file.json
What I have understood about the "log" command, is that it catches compiler steps and saves it the json output. In C with gcc I can imagine how it captures compilation steps, but I was curious about how does it works with interpreted languages
CodeChecker version
6.17
To Reproduce
CodeChecker log -b "python3 my_file.py" --output ./my_file.json
But it results in an empty json
Expected behaviour
A complete json
Desktop (please complete the following information)
Ubuntu 20.04.3 LTS

CodeChecker log can be used to catch C/C++ compiler calls (g++, gcc, clang) and it will create a compilation database. CodeChecker analyze will work with this compilation database and it will create analyzer result files (basically plist files) to a given directory (report directory).  These result files can be parsed on the CLI (CodeChecker parse) or can be stored to a running CodeChecker server. This approach will work only if you want to run Clang Tidy or Clang SA analyzers.
For other analyzers we created a simple tool which can be used to convert an analyzer output to CodeChecker specific output. For example if you would like to run a python specific analyzer such as pylint you can do it on the following way: https://github.com/Ericsson/codechecker/blob/master/docs/tools/report-converter.md#pylint