Unintuitive cli flag handling for help flag

Currently the cli flag handling for the -h/--help flag seems to be broken. When using
> ros2 run micro_ros_agent micro_ros_agent
Usage: 'MicroXRCEAgent <udp4|udp6|tcp4|tpc6|serial|pseudoterminal> <<args>>'
For a more detailed description about all the available arguments, please execute the agent with '-h/--help' option.

This seems fair, but following the advice:
> ros2 run micro_ros_agent micro_ros_agent -h
Error: chosen transport '--help' is invalid!
Usage: 'MicroXRCEAgent <udp4|udp6|tcp4|tpc6|serial|pseudoterminal> <<args>>'
For a more detailed description about all the available arguments, please execute the agent with '-h/--help' option.

> ros2 run micro_ros_agent micro_ros_agent --help
Error: chosen transport '-h' is invalid!
Usage: 'MicroXRCEAgent <udp4|udp6|tcp4|tpc6|serial|pseudoterminal> <<args>>'
For a more detailed description about all the available arguments, please execute the agent with '-h/--help' option.

Just a small qol issue, hopefully i'll have time to look at this and propose a pr for this soon.

Thank you for the insight! See related pull request here :slightly_smiling_face: https://github.com/eProsima/Micro-XRCE-DDS-Agent/pull/189

Closing since merged. Please reopen if any other issue.
And thanks for the tip @RobertWilbrandt


Added My data

Thanks for the opportunity for contributing

Good job @para-docx
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.

Star this repository. ⭐

SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .

Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.


Taskbaricons not centering 21H1

Hi,
i got on my latest installation of Windows an error while using TaskbarX.
Transparancy and animations are working fine, but the icons are not centered..
Console output: "@Calculator | The object reference was not set to an object instance"

Windows10
21H1 19043.928

It's probably because of TaskbarX not taking into account when the "News and interests" panel is not displayed. Described here.

It's probably because of TaskbarX not taking into account when the "News and interests" panel is not displayed. Described here.
An archive with fixed binaries according to the PR I made: TaskbarX.zip.

Getting this all the time. Sometimes there was "@calculator | The object reference was not set to an object instance"
But that got "fixed" by ur PR.
Due to that, ima add this here as a comment.
Waiting for Shell_TrayWnd
@Calculator | Error: Missing method 'instance object [TaskbarX] Accessibility.IAccessible::get_accRole(object)' from class 'System.Int32'.
@Looper1 | Error: Missing method 'instance object [TaskbarX] Accessibility.IAccessible::get_accRole(object)' from class 'System.Int32'.

Edition	Windows 10 Pro
Version	21H1
Installed on	29.10.2020
OS build	19043.1237
Experience	Windows Feature Experience Pack 120.2212.3920.0

It's probably because of TaskbarX not taking into account when the "News and interests" panel is not displayed. Described here.
An archive with fixed binaries according to the PR I made: TaskbarX.zip.

Tysm this helped my issue


option(s) to submit fixes

For a PR it would be a commit to the same branch. (we do similar for adding changelog entries to PRs upon a specific label being present: https://github.com/datalad/release-action/blob/master/add-changelog-snippet/action.yml#L23 )
For a run within a non-PR branch - would be a new PR against that branch.

@yarikoptic
Hmm. Certainly doable from the codespell end, since it has a -w mode that writes back changes, and when run that way it reports:
FIXED: relative/path/to/first/file.xtn
FIXED: relative/path/to/other/code.xtn

and etc.
So it'd just be a matter of git add <foo> for each file listed, and then commit the changes.
...Arguably that functionality may be out of scope for this action, though, because it's "codespell with annotations" — and there's no real point in making annotations for spelling errors that are automatically fixed. It's kind of taking things in a different direction, functionality-wise.
A separate action to run codespell and auto-apply the changes, instead of making annotations, might actually make more sense, despite the seeming redundancy of two different actions to run codespell. (In truth there are multiple codespell actions already, so adding an "auto-apply codespell fixes" action would just be expanding the existing set of choices.)

IMHO there is no need for a separate action and indeed could be just a matter of adding an option and committing/pushing. FWIW, here is a "live" fresh request for such a feature

https://github.com/cryptnono/cryptnono/pull/37#issuecomment-2267111135
Do you think we could pull off this quickly (e.g. if I sent a PR) or should I redo the workflow to just do "manual" installation of codespell and pushing?

@yarikoptic ¯\_(ツ)_/¯ I'm not a dev/committer here, just another user... but since I'm also the only one who's responded here in over a year, I think it's safe to assume that "quickly" is probably way off the table.

Would it be enough to allow actions-codespell to have a "write" mode and then you use autofix.ci or something similar to push the fix to the given PR?
I also wonder if it's easier instead to use pre-commit.ci to do it instead for most people

but pushing from CI is easy for a PR, isn't it?
people who use pre-commit.ci typically do not even wan to use actions-codespell since they have codespell in their pre-commit config.


svn: Failed to build

I'm having an issue installing svn on my Laravel Homestead VM running on osx 10.10.
Here's the gist
Thanks.

serf seems to compile fine for me but it failed during "make install"
Then I tried brew install --debug subversion
When it raise from ruby

raise
ignore
backtrace
irb
shell

I chose 5
execute "make install"
exit shell
then choose 4
do nothing and exit
and the installation continued and succeeded.
Not sure why.

Does brew install homebrew/dupes/krb5 before brew install subversion help subversion to install correctly?
Should --without-serf be the default for subversion in Linuxbrew?
What functionality does this disable if we do disable serf by default?

Installing homebrew/dupes/krb5 does not seem to be enough to get subversion's serf to compile correctly. I'm going to disable serf by default.
scons: Reading SConscript files ...
/bin/sh: 1: /usr: Permission denied
OSError: '/usr --cflags gssapi' exited 126:
  File "/tmp/subversion--serf20160224-17504-1skjz6v/serf-1.3.8/SConstruct", line 360:
    env.ParseConfig('$GSSAPI --cflags gssapi')
  File "/home/linuxbrew/.linuxbrew/Cellar/scons/2.4.1/libexec/scons-local/SCons/Environment.py", line 1576:
    return function(self, self.backtick(command))
  File "/home/linuxbrew/.linuxbrew/Cellar/scons/2.4.1/libexec/scons-local/SCons/Environment.py", line 594:
    raise OSError("'%s' exited %d" % (command, status))


resholve: 0.6.0 -> 0.6.6, add resholveScript* fns

Motivation for this change
This tracks a few upstream changes in resholve and dependencies:

extract argument-handling functions from resholvePackage to enable easier re-use in additional APIs...
address minor review feedback in the course of #138080
add resholveScript and resholveScriptBin functions to close #138442

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 139253 at d88181ba run on x86_64-linux 1

5 packages built successfully:

arch-install-scripts
bashup-events32
bashup-events44
dgoss
resholve

Hey, I was just reading the whole doc again to make sure I understood and found a couple of typos.
This doesn't need to be handled in that PR, but I thought you might want to know
convienience. -> convenience
isnert -> insert
you have the following paragraph
# --fake 'f:setUp;tearDown builtin:setopt source:/etc/bashrc'
fake = {
  # fake accepts the initial of valid identifier types as a CLI convienience.
  # Use full names in the Nix API.
  function = [ "setUp" "tearDown" ];
  builtin = [ "setopt" ];
  source = [ "/etc/bashrc" ];
};

shouldn't the flag to --fake be function instead of just f: ?
in the following paragraph
# --fix 'aliases xargs:ls $GIT:gix'
fix = {
  # all single-word directives use `true` as value
  aliases = true;
  xargs = [ "ls" ];
  "$GIT" = [ "gix" ];
};

I still can't figure out what the xargs = [ "ls" ]; does.
in the following paragraph
# --keep 'which:git;ls .:$HOME $LS:exa /etc/bashrc ~/.bashrc'
keep = {
  which = [ "git" "ls" ];
  "." = [ "$HOME" ];
  "$LS" = [ "exa" ];
  "/etc/bashrc" = true;
  "~/.bashrc" = true;
};

I still can't understand the following lines
"." = [ "$HOME" ];
 "$LS" = [ "exa" ]; shouldn't that line be in the fix section ?
The doc is otherwise amazing!
I understand these comments don't pertain to the current PR, so feel free to ignore them if you don't have time.

I understand these comments don't pertain to the current PR, so feel free to ignore them if you don't have time.

No worries. I'm leery of picking at the trivial stuff atm not because I don't think it's worth fixing--I prefer fixing them before I have a chance to forget--but mostly because the ofborg darwin checks are slow atm.

This doesn't need to be handled in that PR, but I thought you might want to know
convienience. -> convenience
isnert -> insert

Fixed locally.
A lot of the remaining parts hinge on information in the manpage. resholve is still lacking some important automation (particularly wrt single-sourcing the docs and maintaining the Nix expressions on both ends). I'd like to have ~full docs for the Nix and CLI APIs, but for now I'm offloading as much as I feel like I can get by with on the manpage.
I've added a local note to make the necessity of the manpage more evident.

...
shouldn't the flag to --fake be function instead of just f: ?

No, but it's a fair question! In the Nix API I generally model verbose-but-explicit forms, but there's also an abbreviated version of the fake forms supported at the CLI. The doc here is trying to bridge between the two APIs by demonstrating how the terse manpage form translates to the verbose Nix form. But yeah, you need both to decode it.
(If anyone reading this has a handle on patterns for granular single-sourcing like this, I have an issue open at abathur/resholve#19).

...
I still can't figure out what the xargs = [ "ls" ]; does.
...
I still can't understand the following lines
"." = [ "$HOME" ];
 "$LS" = [ "exa" ]; shouldn't that line be in the fix section ?

I can explain these (and don't mind doing so!) but I am curious if the manpage clarifies them for you (or if you've already read it and they were still unclear.) Don't feel obliged, though.

Thank you for your answers!
For other people who are following, here is a link to the man page https://github.com/abathur/resholve/blob/master/resholve.1.txt
The relevant section for --keep says
    −−keep ’directives’ | RESHOLVE_KEEP=’directives’
         Ignore (omit errors for) some things it isn’t safe to fix/ignore by
         default. Directive forms:

         outer_command:inner_command[;inner_command]...
            ignore inner_command(s) in the arguments to outer_command

         $variable (Note: $ may need escaping)
            ignore $variable or ${variable} used as all or part of a first
            word (command)

            Such as: ‘$GIT status‘ or ‘$BUSYBOX/ls‘

         absolute_path
            ignore a specific absolute_path

         ~/path
            ignore a specific home‐relative_path

         Ex: −−keep ’which:git;ls source:$HOME $LS:exa /etc/bashrc ~/.bashrc’

reading that I'm guessing that the line
"$LS" = [ "exa" ]; means ignore the error for the variable $LS that part is clear. reading from the doc, perhaps the second part means that you ignore it when it's used like $LS exa ? This is the part that doesn't make sense, I couldn't figure out why you would write such a command.
then the
"." = [ "$HOME" ]; would mean ignore the . path relative to $HOME ? (not sure about this one). It is one of the two paths possibilities, however not sure which one.

Oof. Thanks for pulling on this thread--it flushed out a few inconsistencies here (which is frustrating, since this exactly what I'm hoping to avoid by not fully-duplicating documentation until I can get it single-sourced...)
From your previous post:

I still can't figure out what the xargs = [ "ls" ]; does.

And from this one:

reading that I'm guessing that the line "$LS" = [ "exa" ]; means ignore the error for the variable $LS that part is clear. reading from the doc, perhaps the second part means that you ignore it when it's used like $LS exa ? This is the part that doesn't make sense, I couldn't figure out why you would write such a command.

Both of these forms are syntactically valid but unimplemented. They leaked into the examples from tentative syntax forms that are commented out in the manpage source.
(long story: I was having trouble settling on a CLI API. One way I sorted out my thoughts was by writing a manpage that included all of the options I imagined it gaining over the next few years to make sure there was conceptual/grammatical room. Then I came back and commented out what wasn't implemented, but obviously missed some forms in the examples :/)

then the "." = [ "$HOME" ]; would mean ignore the . path relative to $HOME ?

If I'm reading you right, this is roughly correct. Technically, it's just telling resholve not to object to $HOME in this argument, without regard for whether it is the whole path, a prefix, in the middle, or a suffix.
Also, there's another good catch here. I missed updating the "." form here to track API changes that did get updated in the manpage. resholve now internally normalizes instances of . to source, so "source" = [ "$HOME" ]; or source = [ "$HOME" ]; are now the only sensical forms here.
I was wrapping up a bug-fix when you posted this, so I went ahead and made these doc fixes and and included them in the push.

For clarity, since I keep adding commits to this PR: this PR is ready to be merged at any time.
I'm just tacking on new resholve releases as long as it stays open.

Then let's merge, ti think this adds hugely valuable features!
Thank you for the hard work!


Newline in string not working C#

Newline in string not working C# what i do help me
Problematic image

i want this

Yes this would really helpful. Even i tried with the markdown options given in the link

I have found that if I use a StringBuilder and do the following, it works
sb.AppendLine(myText);
sb.AppendLine("");
db.AppendLine(otherText);

To create a line break in markdown you have to end a line with two spaces.
Take a look at the sample on Wikipedia: https://en.wikipedia.org/wiki/Markdown

Let is know if that does not work for you.

Its not working for me, In Microsoft Bot Framework Emulator it working fine but when i upload on web for final check and try on the skype its same as before its not working in skype
Please help me what i do

@inornos  hey!  '\n' works when i deployed it on skype. However in emulator two spaces gives a new line.

<br/> work for me in emulator


Set up acts_as_tenant

I rule and have completed some work on Case Manager that's ready for review!
This sets up the acts_as_tenant gem, to enable us to do multitenancy.
The general approach here is to move a lot of the stuff that's in env vars now to a Fund object, and read from that rather than env vars. The rest of it is, frankly, mostly grepping around for FUND and removing that in favor of some form of current_tenant.name. (Exceptions to this are lines - more on lines later - and fund-mailer-domain, which has been daria-services for a little while now.) The diff is large, but most of it turned out to be find and replace.
Deployment-wise, what this will mean in the immediate term is that every fund's instance will get turned into multitenant apps with only one tenant. We'll need to declare a brief downtime to reconfigure, but the two step plan is basically:

Create a Fund object with the proper subdomain set
Run some sql to set every row in every table's fund_id to 1

This approach means we can punt on lines for now, but most likely we'll need to retool to make Line a proper object. (Having line be set on the Fund level doesn't work super well because of the enums and validation on other objects right now.) But we can roll this out as is without a problem.
To test:

try the migration strategy:

git checkout main && rails db:drop db:create db:migrate db:seed && git checkout tenants-2 && rails db:migrate
echo "Fund.create name: 'CatFund', domain: 'supercat', subdomain: 'fluffy', full_name: 'Catte Fund" | rails c
sql="
begin;
update archived_patients set fund_id = 1;
update call_list_entries set fund_id = 1;
update calls set fund_id = 1;
update clinics set fund_id = 1;
update configs set fund_id = 1;
update events set fund_id = 1;
update external_pledges set fund_id = 1;
update fulfillments set fund_id = 1;
update notes set fund_id = 1;
update patients set fund_id = 1;
update practical_supports set fund_id = 1;
update users set fund_id = 1;
update versions set fund_id = 1;
commit;
"
echo $sql | rails db

spin up the app as normal - git checkout tenants-2 && rails db:drop db:create db:migrate db:seed then rails s and try logging in as normal - localhost:3000 - to confirm you can hit the first fund
log in to the alt development tenant - http://catbox.lvh.me:3000 to confirm you can hit the second fund

This pull request makes the following changes:

sets up acts as tenant
adjusts a lot of stuff to be tenant

no significant view changes
It relates to the following issue #s:

bumps #1817

For reviewer:

Adjust the title to explain what it does for the notification email to the listserv.
Tag this PR:

feature if it contains a feature, fix, or similar. This is anything that contains a user-facing fix in some way, such as frontend changes, alterations to backend behavior, or bug fixes.
dependencies if it contains library upgrades or similar. This is anything that upgrades any dependency, such as a Gemfile update or npm package upgrade.

If it contains neither, no need to tag this PR.

per discussion/self-realization, breaking this into two so it's much easier to review

splitting this across two PRs instead


WARNING:tensorflow:Entity <function Function._initialize_uninitialized_variables..initialize_variables at 0x000001CF6FFB6CA8> could not be transformed and will be executed as-is.

The attachments contain details of the warning that I encountered while working on some dataset. Kindly review it. In case of bug, fix it.
WARNING:tensorflow:Entity <function Function._initialize_uninitialized_variables..initialize_variables at 0x000001CF6FFB6CA8> could not be transformed and will be executed as-is. Please report this to the AutoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: module 'gast' has no attribute 'Num'
WARNING: Entity <function Function._initialize_uninitialized_variables..initialize_variables at 0x000001CF6FFB6CA8> could not be transformed and will be executed as-is. Please report this to the AutoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: module 'gast' has no attribute 'Num'

Same here, I hope this is why TF2 is not working (otherwise I have no idea why on my RTX 2060 super the script is not working while on other machines it is fine).
The script :
import math
import pickle
import os

import numpy as np
os.environ['TF_CPP_MIN_LOG_LEVEL'] = '3'
import tensorflow as tf

def load_data(path: str):
    data = []
    labels = []
    for batch_index in range(1, 6):
        with open(os.path.join(path, 'data_batch_{}'.format(batch_index)), 'rb') as data_file:
            data_dict = pickle.load(data_file, encoding='bytes')
            data.append(np.reshape(data_dict[b'data'], (len(data_dict[b'data']), 3, 32, 32)))
            labels.append(data_dict[b'labels'])
    return np.concatenate(data), np.concatenate(labels)

def main():
    data_path = 'data/cifar-10-batches-py'
    data, labels = load_data(data_path)
    data = np.transpose(data, (0, 2, 3, 1))

    max_epoch = 2
    batch_size = 32

    image_input = tf.keras.Input(shape=(32, 32, 3))

    data_op = tf.cast(image_input, tf.float32) / 255.0
    network_output = tf.keras.layers.Conv2D(8, 5, strides=(2, 2), activation=tf.nn.relu)(data_op)
    network_output = tf.keras.layers.Conv2D(16, 3, strides=(2, 2), activation=tf.nn.relu)(network_output)
    network_output = tf.keras.layers.Flatten()(network_output)
    network_output = tf.keras.layers.Dense(120, activation=tf.nn.relu)(network_output)
    network_output = tf.keras.layers.Dense(60, activation=tf.nn.relu)(network_output)
    network_output = tf.keras.layers.Dense(10)(network_output)

    model = tf.keras.Model(inputs=image_input, outputs=network_output)

    model.compile(
        optimizer=tf.optimizers.Adam(learning_rate=0.001),
        loss=tf.losses.SparseCategoricalCrossentropy())

    history = model.fit(data, labels, batch_size=batch_size, epochs=max_epoch)

if __name__ == '__main__':
    main()

The output :
Train on 50000 samples
Epoch 1/2
WARNING:tensorflow:Entity <function Function._initialize_uninitialized_variables.<locals>.initialize_variables at 0x7f28df337c10> could not be transformed and will be executed as-is. Please report this to the AutoGraph team. When filing the bug, set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output. Cause: Bad argument number for keyword: 1, expecting 2
   32/50000 [..............................] - ETA: 33:42Traceback (most recent call last):
  File "tf2_cifar.py", line 49, in <module>
    main()
  File "tf2_cifar.py", line 45, in main
    history = model.fit(data, labels, batch_size=batch_size, epochs=max_epoch)
  File "/usr/lib/python3.8/site-packages/tensorflow_core/python/keras/engine/training.py", line 709, in fit
    return func.fit(
  File "/usr/lib/python3.8/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 313, in fit
    training_result = run_one_epoch(
  File "/usr/lib/python3.8/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 123, in run_one_epoch
    batch_outs = execution_function(iterator)
  File "/usr/lib/python3.8/site-packages/tensorflow_core/python/keras/engine/training_v2_utils.py", line 86, in execution_function
    distributed_function(input_fn))
  File "/usr/lib/python3.8/site-packages/tensorflow_core/python/eager/def_function.py", line 457, in __call__
    result = self._call(*args, **kwds)
  File "/usr/lib/python3.8/site-packages/tensorflow_core/python/eager/def_function.py", line 520, in _call
    return self._stateless_fn(*args, **kwds)
  File "/usr/lib/python3.8/site-packages/tensorflow_core/python/eager/function.py", line 1823, in __call__
    return graph_function._filtered_call(args, kwargs)  # pylint: disable=protected-access
  File "/usr/lib/python3.8/site-packages/tensorflow_core/python/eager/function.py", line 1137, in _filtered_call
    return self._call_flat(
  File "/usr/lib/python3.8/site-packages/tensorflow_core/python/eager/function.py", line 1223, in _call_flat
    flat_outputs = forward_function.call(
  File "/usr/lib/python3.8/site-packages/tensorflow_core/python/eager/function.py", line 506, in call
    outputs = execute.execute(
  File "/usr/lib/python3.8/site-packages/tensorflow_core/python/eager/execute.py", line 67, in quick_execute
    six.raise_from(core._status_to_exception(e.code, message), None)
  File "<string>", line 3, in raise_from
tensorflow.python.framework.errors_impl.UnknownError:  Failed to get convolution algorithm. This is probably because cuDNN failed to initialize, so try looking to see if a warning log message was printed above.
         [[node model/conv2d/Conv2D (defined at /usr/lib/python3.8/site-packages/tensorflow_core/python/framework/ops.py:1751) ]] [Op:__inference_distributed_function_950]

Function call stack:
distributed_function

I reinstalled driver, CUDA and cudnn, nothing changed (tensorflow 2.0.0)

same here, did someone find some answers ?

Sorry for the delayed response! These may be two bugs.
The first error that @Victor-99 reported is the same as #1 and can be fixed by running pip install --user gast==0.2.2. Newer versions of TF should fix it.
I'm not sure if the error from @Corentin-pro is the same - I'm investigating a similar issue in https://github.com/tensorflow/tensorflow/issues/34433. If downgrading gast doesn't fix it, could you re-run the snippet with this added to the top: tf.autograph.set_verbosity(3, True) it should give us additional clues to resolve it.

Is this a bug of tensorflow 2.0?

@Capriciousman in your case, there seem to be two issues -

The warning you see indicates a bug, but since you use only built-in Keras components, it should be safe to ignore. I did a quick investigation, but it doesn't seem to reproduce at head. What version of TF were you using?

Your training seems to generate NaNs. That is more likely due to an incorrect configuration in your model. For instance, the output layer has only one unit, but you need 10 - fashion_mnist is a multi-class model. In fact when I try to run your code it gives me an error about that (Received a label value of 9 which is outside the valid range of [0, 1)). Setting units to 10 trains the model correctly. I suspect the error message that I'm seeing was only added in a more recent version, which would explain why you didn't get one.

@mdanatg I am using the latest version TF 2.0, Ignore would not lead to the solution.
Yes, it is generating Nans, there is an issue in the optimization configuration. I have corrected the # classes. I have tried the same code in google colab and it's working fine there.! Hence seems an issue in my tf/keras/optimization configs.
#######code########
import tensorflow as tf
print(tf.version)
mnist = tf.keras.datasets.fashion_mnist
(training_images, training_labels), (test_images, test_labels) = mnist.load_data()
import matplotlib.pyplot as plt
plt.imshow(training_images[0])
print(training_labels[0])
print(training_images[0])
training_images  = training_images / 255.0
test_images = test_images / 255.0
model = tf.keras.models.Sequential([tf.keras.layers.Flatten(),
tf.keras.layers.Dense(128, activation=tf.nn.relu),
tf.keras.layers.Dense(10, activation=tf.nn.softmax)])
model.compile(optimizer = tf.compat.v1.keras.optimizers.Adam,     #Error is here
loss = 'sparse_categorical_crossentropy',
metrics=['accuracy'])
model.fit(training_images, training_labels, epochs=5)
model.evaluate(test_images, test_labels)

Hi All
Is there a solution about this bug ? Because untill now the bug isnt fix for me


docs: translate formatter/differences-with-prettier.md to pt-br

Summary
Minion 👾 task of:

#1065

Test Plan

Native speaker, please review my translation doc.

@ImBIOS I think we can translate "nós bogus" to "nós falsos"


Can we use mediapipe eye track detection for multiple persons?

Please make sure that this is a feature request.
System information (Please provide as much relevant information as possible)

MediaPipe Solution (you are using):
Programming language : C++/typescript/Python/Objective C/Android Java
Are you willing to contribute it (Yes/No):

Describe the feature and the current behavior/state:
Will this change the current api? How?
Who will benefit with this feature?
Please specify the use cases for this feature:
Any Other info:

HI @PurushottamaRaoK,  could you please provide your use case with the above feature request. Thanks!


jsoncom: gracefully deal with EMSGSIZE errors

The jsoncomm code is using socket.SOCK_SEQPACKET and is written with the assumption that each send/recv transmit exactly one msg. Unfortunately there is a size limit for these messages defined by the kernel (/proc/sys/net/core/{r,w}mem_max) which can be hit by large messages, e.g. from a org.osbuild.files:map().
The default limit I observed is around 200kb. This commit will try to increase the socket buffer size when this happens with setsockopt(socket.SOL_SOCKET, socket.SO_{RCV,SND}BUF, size).
This should give us some breathing room, however there is stil a question about how to deal very large jsoncom message, as the kernel has the above sysctl limits. We could use SO_{RCVB,SND}UFFORCE to bypass them but that feels a bit dangerous as it potentially hand a way to DoS the system with very large jsoncomm messages.
For pure json we could use a streaming protocol like jsonseq and avoid all limitations but jsoncom is also doing fd transfers so it will be a little bit more involved (plus jsoncomm as it is now is an essential part of the architecture right now).
Closes: HMS-4537

Generally LGTM, one question: If I understand correct when _sendmsg_handle_emsgsize will throw an exception we will resize the buffers and send data again. To the point where we will hit a limit in /proc/sys/net/core/wmem_max. If in this case setsockopt wont return exception too?

That is my understanding, /proc/sys/net/core/(r|w)mem_default also exists which is the initial size of the buffers. But once we run into the limit there's nothing you can do but increase it.

My question here would be:

What if setsockoptwill fail, or will not fail but the buffer size will not be changed?
Do we need also wrapper for recv counterpart?

My question here would be:

Thank you for your review and your questions.

1. What if `setsockopt`will fail, or will not fail but the buffer size will not be changed?

This can happen but it will not make anything worse, we call setsockopt() and then try again. If setsockopt() failed the retry will raise the same EMSGSIZE exception we had before.

2. Do we need also wrapper for `recv` counterpart?

I was wondering the same. The included test does not require it (it does both send and recv) and when playing around it seems as long as the send buffer is big enough the recv buffer will not error. I suspect that internally this all points to the same kernel memory anyway so that would make sense but I did not very in the kernel code so only ~60% confident in this theory).

When we hit the limit with large lists of packages for the input to the rpm stage, we switched to fds, which I think is a good general solution for this.  What are the downsides with that?
https://github.com/osbuild/osbuild/pull/1331
Also, before that: https://github.com/osbuild/osbuild/pull/824

2. Do we need also wrapper for `recv` counterpart?

I was wondering the same. The included test does not require it (it does both send and recv) and when playing around it seems as long as the send buffer is big enough the recv buffer will not error. I suspect that internally this all points to the same kernel memory anyway so that would make sense but I did not very in the kernel code so only ~60% confident in this theory).

I think we should then add a comment about that in the code. I can imagine the situation when someone opens PR adding "missing" recv wrapper in future :)

When we hit the limit with large lists of packages for the input to the rpm stage, we switched to fds, which I think is a good general solution for this. What are the downsides with that?
#1331 Also, before that: #824

I opened https://github.com/osbuild/osbuild/pull/1836 with a complementary PR, this one may still be useful. If we don't want to use I would like to cherry pick the bit that gives the buffer size in the error message.

It seems the prefered way is to not do this and use FDs (which is fine with me). I think there is still value in making the error nicer, so I opened https://github.com/osbuild/osbuild/pull/1840


Add max version constraint for fugue

It looks like the recent release of Fugue 0.7.0 has bumped its qpd dependency to a version that only has python support up to 3.8. I'm not sure if this is the cause for the recent Fugue-related failures, but it does mean that at least for now, we should constrain to fugue<0.7.0, where 3.9+ support is guaranteed.
In the long run, we should probably see what the blockers are to allowing 3.9+ support on qpd again, cc @goodwanghan in case you have any additional context to provide here.

Hi, I apologize for the inconvenience, the qpd constraint should not exist. How about I make the change on the QPD side so that you don't need this PR?

@goodwanghan that would be great! I'm not 100% sure if that's the cause of the failures we've been seeing in CI, but resolving that would certainly narrow things down a lot 🙂

It looks like the failures here persist even with the pip package of qpd=0.3.1 (which doesn't have the python<3.9 constraint) 😕 so think for now we might want to go ahead and merge this in to unblock tests, and follow up later with a resolution for whatever the underlying issue is.

It looks like the failures here persist even with the pip package of qpd=0.3.1 (which doesn't have the python<3.9 constraint) 😕 so think for now we might want to go ahead and merge this in to unblock tests, and follow up later with a resolution for whatever the underlying issue is.

I see so the issue isn't necessarily qpd and python 3.9+ related but still shows up only on python 3.9 and above right?

I think the problem is on qpd,
 classifiers=[
        # "3 - Alpha", "4 - Beta" or "5 - Production/Stable"
        "Development Status :: 3 - Alpha",
        "Intended Audience :: Developers",
        "Topic :: Software Development :: Libraries :: Python Modules",
        "License :: OSI Approved :: Apache Software License",
        "Programming Language :: Python :: 3.6",
        "Programming Language :: Python :: 3.7",
        "Programming Language :: Python :: 3.8",
        "Programming Language :: Python :: 3 :: Only",
    ],

I will go ahead adding the versions and publish 0.3.2 to unblock you guys

I just published qpd 0.3.2 can you bump up the version to see if that works for you?

Codecov Report

Merging #639 (c66a095) into main (0db4506) will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #639      +/-   ##
==========================================
+ Coverage   88.45%   88.59%   +0.13%
==========================================
  Files          69       69
  Lines        3500     3498       -2
  Branches      707      701       -6
==========================================
+ Hits         3096     3099       +3
+ Misses        317      309       -8
- Partials       87       90       +3

Impacted Files
Coverage Δ

dask_sql/physical/rel/custom/predict.py
95.34% <0.00%> (-0.11%)
:arrow_down:

dask_sql/utils.py
100.00% <0.00%> (ø)

dask_sql/_version.py
34.00% <0.00%> (+1.44%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Adds an unannotated view to the BPBrowser

Hides the Annotations in source code by shrinking and not rendering them to create the illusion of an unannotated source version. Adds a "unannotated" option to display it in the BPBrowser.
.
Solves https://github.com/hpi-swa-lab/babylonian-programming-smalltalk/issues/86

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

19 of 101 (18.81%)  changed or added relevant lines in 10 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 65.596%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/Babylonian-UI.package/BPBrowser.class/instance/showUnannotated..st
0
4
0.0%

packages/Babylonian-UI.package/BPBrowser.class/instance/showingUnannotated.st
0
4
0.0%

packages/Babylonian-UI.package/BPUnannotatedStyler.class/instance/insertExamplesMorphFrom.into.basedOn..st
0
5
0.0%

packages/Babylonian-UI.package/BPBrowser.class/instance/showingUnannotatedSourceString.st
0
8
0.0%

packages/Babylonian-UI.package/BPUnannotatedStyler.class/instance/annotationMorphFor..st
0
9
0.0%

packages/Babylonian-UI.package/BPBrowser.class/instance/toggleUnannotatedSource.st
0
10
0.0%

packages/Babylonian-UI.package/BPBrowser.class/instance/sourceStringUnannotated.st
0
17
0.0%

packages/Babylonian-UI.package/BPBrowser.class/instance/selectedMessage.st
0
25
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.6%

Covered Lines:
5430

Relevant Lines:
8278

💛  - Coveralls


refactor: rename babylon to @babel/parser

Closes #621

@skyrpex thanks for the pr! you'll want to also update the babel-related deps in package.json for this to work

Oops, you're right! I was lazy and just made the PR using the Github editor... 😛
I'm now facing some problems: @babel/parser's minimum version is<EMAIL_ADDRESS>If I update every babel dependency to beta.48 (or beta.49) tests fail.
What should I do?


fix the format checking, mentioned in the issue Extract audio #9

Extract audio #9

Seems good for me


INT B-20316 Actual Pro Gear Weights in PrimeV3

B-20315
Summary
Pro Gear Weights were getting added to the Prime V1 and Prime V2 APIs at the same time Prime V3 was getting created. That timing caused those fields to not make it into Prime V3. Later, the Simulator was updated to use Prime V3 in places that used Pro Gear Weights and this issue became much more noticeable. <- All of this was task assigned to our team.
Brooklyn noticed this issue and fixed Prime V3 to accept and persist Pro Gear Weights. This ticket returns those stored values.
I was eager to move this to PI 24.5 so it could be fixed before Home Safe switched to V3.
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

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

How to test

Access the Prime Simulator
Find a move for which you can update actual pro gear weights (Unrelated fields can prevent you. If it has 0 for the estimated weight and it's after the pickup date it won't let you edit anything.)
Add values for both or either pro gear weights and save.
Verify the values you just saved appear on the shipment summary
Verify the values appear when you go to re-edit the same shipment.

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys.
[ ] Have been communicated to #g-database.
[ ] Secure migrations have been tested following the instructions in our docs.

Screenshots

It was running the main test suite.


Github Pages for KubeVela homepage

Folder docs serves the source site for the Markdown Github homepage, and docs/kubevela remains as the static homepage site.

For a quick glance, please visit the temporary site https://zzxwill.github.io/RudrX/.


❓ [QUESTION] installation with lammps

I have done the cmake part successfully in our cluster.
while doing make -j 32 i got the following error.
/data/home/reddy/softwares/lammps-29Sep2021/src/pair_nequip.cpp: In member function ‘virtual void LAMMPS_NS::PairNEQUIP::coeff(int, char**)’:
/data/home/reddy/softwares/lammps-29Sep2021/src/pair_nequip.cpp:209:30: error: lvalue required as left operand of assignment
209 |   torch::jit::getBailoutDepth() = jit_bailout_depth;
I was able to install in my own laptop but got the above error in all of our clusters.
please help me to resolve this issue.
thank you.

Hi @deleep225, please provide a full set of information what platform you are on and what you ran for the install.

Hi Simon,
please check the information below.
uname -a
Linux nanochem3 3.10.0-1160.42.2.el7.x86_64 #1 SMP Tue Sep 7 14:49:57 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
my cmake command includes the following
cmake ../cmake -DCMAKE_PREFIX_PATH=python -c 'import torch;print(torch.utils.cmake_prefix_path)' -DCUDA_TOOLKIT_ROOT_DIR=/usr/local/cuda -DCUDNN_LIBRARY_PATH=/data/home/reddy/anacoda3/lib -DCUDNN_INCLUDE_PATH=/data/home/reddy/anaconda3/include -DMKL_INCLUDE_DIR=/usr/include/
OUTPUT for the above command.
-- Running check for auto-generated files from make-based build system
-- Found OpenMP_CXX: -fopenmp
-- Found OpenMP: TRUE
-- Could NOT find FFMPEG (missing: FFMPEG_EXECUTABLE)
-- Generating style headers...
-- Generating package headers...
-- Generating lmpinstalledpkgs.h...
-- Could NOT find ClangFormat (missing: ClangFormat_EXECUTABLE) (Required is at least version "8.0")
-- The following tools and libraries have been found and configured:

Git
MPI
OpenMP

-- <<< Build configuration >>>
Operating System: Linux CentOS Linux 7
Build type:       RelWithDebInfo
Install path:     /data/home/reddy/.local
Generator:        Unix Makefiles using /usr/bin/gmake
-- Enabled packages:
-- <<< Compilers and Flags: >>>
-- C++ Compiler:     /data/home/reddy/softwares/gcc-10.3.0/bin/c++
Type:          GNU
Version:       10.3.0
C++ Flags:     -O2 -g -DNDEBUG
Defines:       LAMMPS_SMALLBIG;LAMMPS_MEMALIGN=64;LAMMPS_OMP_COMPAT=4;LAMMPS_GZIP
-- <<< Linker flags: >>>
-- Executable name:  lmp
-- Static library flags:
-- <<< MPI flags >>>
-- MPI_defines:      MPICH_SKIP_MPICXX;OMPI_SKIP_MPICXX;_MPICC_H
-- MPI includes:     /usr/local/mpich-3.1.4/include
-- MPI libraries:    /usr/local/mpich-3.1.4/lib/libmpicxx.so;/usr/local/mpich-3.1.4/lib/libmpi.so;
-- Looking for C++ include pthread.h
-- Looking for C++ include pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- Found CUDA: /usr/local/cuda (found version "11.4")
-- The CUDA compiler identification is NVIDIA 11.4.120
-- Check for working CUDA compiler: /usr/local/cuda/bin/nvcc
-- Check for working CUDA compiler: /usr/local/cuda/bin/nvcc -- works
-- Detecting CUDA compiler ABI info
-- Detecting CUDA compiler ABI info - done
-- Caffe2: CUDA detected: 11.4
-- Caffe2: CUDA nvcc is: /usr/local/cuda/bin/nvcc
-- Caffe2: CUDA toolkit directory: /usr/local/cuda
-- Caffe2: Header version is: 11.4
-- Found CUDNN: /data/home/reddy/anacoda3/lib
-- Found cuDNN: v7.6.5  (include: /data/home/reddy/anaconda3/include, library: /data/home/reddy/anacoda3/lib)
-- /usr/local/cuda/lib64/libnvrtc.so shorthash is edac1e2e
-- Autodetected CUDA architecture(s):  6.1 6.1 6.1 6.1 6.1 6.1 6.1 6.1
-- Added CUDA NVCC flags for: -gencode;arch=compute_61,code=sm_61
-- Found Torch: /data/home/reddy/softwares/libtorch/lib/libtorch.so
-- Configuring done
-- Generating done
-- Build files have been written to: /data/home/reddy/softwares/lammps/lammps-29Sep2021/build
Afeter his I have used make -j 32 command, which was 100 percent build but at the last step the installation gave the error.
[100%] Building CXX object CMakeFiles/lammps.dir/data/home/reddy/softwares/lammps/lammps-29Sep2021/src/write_restart.cpp.o
/data/home/reddy/softwares/lammps/lammps-29Sep2021/src/pair_nequip.cpp: In member function ‘virtual void LAMMPS_NS::PairNEQUIP::coeff(int, char**)’:
/data/home/reddy/softwares/lammps/lammps-29Sep2021/src/pair_nequip.cpp:209:30: error: lvalue required as left operand of assignment
209 |   torch::jit::getBailoutDepth() = jit_bailout_depth;
|   ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
make[2]: *** [CMakeFiles/lammps.dir/data/home/reddy/softwares/lammps/lammps-29Sep2021/src/pair_nequip.cpp.o] Error 1
make[1]: *** [CMakeFiles/lammps.dir/all] Error 2
make: *** [all] Error 2
Thank you.

Hi @deleep225 ,
What is your PyTorch version? If it is 1.11, I have a suspicion.
Thanks.

Hi,
print(torch.version)
1.11.0
should I use other version of pytorch?
Thank you.

Hi @deleep225 ,
1.11 is the newest version, just released, and it looks like there is some compatibility work to be done on the LAMMPS plugin. I would advise using 1.10 for now; at some point in the near future we will update the plugin to be compatible with 1.11.

Hi everyone,
Sorry for adding my comment here and not creating a separate discussion, but I'm having the same issue as @deleep225, and as @Linux-cpp-lisp suggested, I tried changing the pytorch version from 1.11 to 1.10 but still getting the error.
My error is a little different where it fails at the cmake command, I'm attaching the details here
(nequip) nisarg@Nisargs-MacBook-Pro build % cmake ../cmake -DCMAKE_PREFIX_PATH=`python -c 'import torch;print(torch.utils.cmake_prefix_path)'`
-- Running check for auto-generated files from make-based build system
-- Checking for module 'mpi-cxx'
--   No package 'mpi-cxx' found
-- Could NOT find FFMPEG (missing: FFMPEG_EXECUTABLE)
-- Generating style headers...
-- Generating package headers...
-- Generating lmpinstalledpkgs.h...
-- Could NOT find ClangFormat (missing: ClangFormat_EXECUTABLE) (Required is at least version "8.0")
-- The following tools and libraries have been found and configured:
 * Git
 * PNG
 * ZLIB

-- <<< Build configuration >>>
   Operating System: Darwin
   Build type:       RelWithDebInfo
   Install path:     /Users/nisarg/.local
   Generator:        Unix Makefiles using /usr/bin/make
-- Enabled packages: <None>
-- <<< Compilers and Flags: >>>
-- C++ Compiler:     /Library/Developer/CommandLineTools/usr/bin/c++
      Type:          AppleClang
      Version:       <IP_ADDRESS>00029
      C++ Flags:     -O2 -g -DNDEBUG
      Defines:       LAMMPS_SMALLBIG;LAMMPS_MEMALIGN=64;LAMMPS_PNG;LAMMPS_GZIP
-- <<< Linker flags: >>>
-- Executable name:  lmp
-- Static library flags:
CMake Warning at /opt/anaconda3/envs/nequip/lib/python3.9/site-packages/torch/share/cmake/Torch/TorchConfig.cmake:22 (message):
  static library kineto_LIBRARY-NOTFOUND not found.
Call Stack (most recent call first):
  /opt/anaconda3/envs/nequip/lib/python3.9/site-packages/torch/share/cmake/Torch/TorchConfig.cmake:127 (append_torchlib_if_found)
  CMakeLists.txt:922 (find_package)

-- Configuring done
CMake Error in CMakeLists.txt:
  Imported target "torch" includes non-existent path

    "MKL_INCLUDE_DIR-NOTFOUND"

  in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

  * The path was deleted, renamed, or moved to another location.

  * An install or uninstall procedure did not complete successfully.

  * The installation package was faulty and references files it does not
  provide.

CMake Error in CMakeLists.txt:
  Imported target "torch" includes non-existent path

    "MKL_INCLUDE_DIR-NOTFOUND"

  in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

  * The path was deleted, renamed, or moved to another location.

  * An install or uninstall procedure did not complete successfully.

  * The installation package was faulty and references files it does not
  provide.

CMake Error in CMakeLists.txt:
  Imported target "torch" includes non-existent path

    "MKL_INCLUDE_DIR-NOTFOUND"

  in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

  * The path was deleted, renamed, or moved to another location.

  * An install or uninstall procedure did not complete successfully.

  * The installation package was faulty and references files it does not
  provide.

CMake Error in CMakeLists.txt:
  Target "lammps" contains relative path in its
  INTERFACE_INCLUDE_DIRECTORIES:

    "MKL_INCLUDE_DIR-NOTFOUND"

CMake Error in CMakeLists.txt:
  Target "lammps" contains relative path in its
  INTERFACE_INCLUDE_DIRECTORIES:

    "MKL_INCLUDE_DIR-NOTFOUND"

CMake Error in CMakeLists.txt:
  Target "lammps" contains relative path in its
  INTERFACE_INCLUDE_DIRECTORIES:

    "MKL_INCLUDE_DIR-NOTFOUND"

-- Generating done
CMake Generate step failed.  Build files cannot be regenerated correctly.

I am working on macOS Monterey 12.1, and have also tried running the same on Linux but the same error. Not sure if there's any changes to be made in installing cmake as without it, it throws cmake not found  error.
Thanks in advance for the help!
Nisarg

@nisargjoshi10  This is covered in the pair_nequip README, please follow the steps there for MKL.
@deleep225 I am migrating this issue to the pair_nequip repository, since it is more directly relevant to that code.


Create profiles/working groups

It would make sense to split ADM-OSC into different profiles.  Since implementations are likely to not need all of the messages, it would make it easier to describe what is being implemented.  For example

Core (this is most of the 0.4 spec -- position data)
Broadcast (These might include many of the 0.5 proposals)
Trackers (?)
Live (?)
Control (?)

This might help clarify which messages correlate directly with ADM (like position) and which do not (like touch/release).
Also, it might be a good way to distinguish between high-density realtime data and low-density configuration data.

Core makes sense.
Broadcast as well.
Trackers is almost identical to Core potentially, I would not push for such a profile initially.
Live might be similar to Core
Control requires a profile indeed.

Is "Core" related to the ADM specification ?
There are many industries and segments in ours using Core as a name, and would think that the profile could "simply" be called "Position".

Core doesn't have any direct correlation to the ADM spec.
I think Position might be a better name.  The current (0.4) spec is about controlling a renderer for object-based audio.  I can't think of a simple name for "generic renderer controls", though.

We talked during last PlugFest in NY as next step to build a workgroup specifically on "interactive" topics, specifically the interactions that happen during production, from different ADM-OSC sources like console surround panner controller, DAW plug in automation record/playback, touch control interface, musician object animators etc.
Specifically, the goal should be to sort out priority parameters as talked in issue #10 and #28.
Who would be interested to join this group (we thought first with current users of such features like Steinberg, Meyer Sound, Flux)


Draft: Add export

Initial implementation of adding export functionality.
Right now it is only logging the export, but I was planning to add it directly to the user's clipboard. Looks like there is a slim svlete plugin for managing copy/paste.
Would love feedback on the work so far, and any design considerations

Addresses Issue #2


What happens when more than two edges connect to a single input port?

What happens when more that two edges connect to a single input port? This came up in our weekly meeting and we didn't know the answer.

Also being worked on here: https://github.com/ModECI/MDF/tree/cyclic_graphs

See: https://github.com/ModECI/MDF/commit/6b409a019781b9c8881e5362df3023743fbe1409 for initial implementation.
Note: still some open questions about what to do when a single node is run multiple times, e.g. as part of a graph wide condition. Should the current input to the node be added up each time? Probably not, so added a 3rd option overwrite to add and multiply, which is the default for now...


Add App.module.css instead of App.css on project initialization

The idea behind this change is to show how CSS Modules works in CRA.
In the end a new project shows how to import a regular stylesheet in the index.js and how to import CSS Module.
This change came in my mind after this thread:
https://twitter.com/dan_abramov/status/1195440508681646080?s=20

Hi @esvyridov, I understand the intention here - but we want to keep CRA's base templates very light, and not too confusing for new-starters.

@mrmckeb I see your point. Ok, let me close this PR. Sorry that I didn't create an issue before the PR.

Sorry for being late to the game. I love css modules to make styling scale in big projects. let's demonstrate the power of that in a separate template.

@andriijas Sorry, I think I don't understand what do you mean by "separate template". Do you mean separate template for injecting css, like I did in this PR or separate app template like it was implemented for typescript support?

3.3.0 is shipping soon with template support :)

@andriijas Sorry, I didn't know about that :) I've checked that change and if we want to add a template with css modules we have to add two more templates... for js and ts. Am I right?


cheat manager

Hi all, so Gecko website has been down for some time now, the owner said his going to close down the site, he said all the codes will be still available but from gcforever, or something like that, why hasn't the download link on dolphin been updated?

Please update the download link for codes to https://web.archive.org/web/20191001120556/https://www.geckocodes.org/index.php?chid=R&r=*&l=m

We updated the URL in Dolphin a year and a half ago. See https://github.com/dolphin-emu/dolphin/pull/9133.


Excess Text in Tests 1.9 and 2.7

From j1939-84 created by ericthomasswenson: Equipment-and-Tool-Institute/j1939-84#1164
Excess Text in Tests 6.1.9 and 6.2.7
Start Test 1.9 - Component ID: Make
14:51:44.9738 Destination Specific org.etools.j1939tools.j1939.packets.ComponentIdentificationPacket Request to Engine #1 (0)
14:51:44.9782 18EA00F9 [3] EB FE 00 (TX)
14:51:45.1418 1CFEEB00 [49] 49 4E 54 20 20 2A 37 35 37 31 32 34 32 30 32 31 2A 58 58 58 58 58 58 58 58 58 58 58 58 58 58 2A 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 2A
Component Identification from Engine #1 (0): {
Start Test 2.6 - DM56: Model year and certification engine family
14:53:46.0076 Destination Specific DM56 Request to Engine #1 (0)
14:53:46.0100 18EA00F9 [3] C7 FC 00 (TX)
14:53:46.0165 18E8FF00 [8] 01 FF FF FF F9 C7 FC 00
14:53:46.0195 Destination Specific
org.etools.j1939tools.j1939.packets.ComponentIdentificationPacket Request to Engine #1 (0)
14:53:46.0220 18EA00F9 [3] EB FE 00 (TX)
14:53:46.1807 1CFEEB00 [49] 49 4E 54 20 20 2A 37 35 37 31 32 34 32 30 32 31 2A 58 58 58 58 58 58 58 58 58 58 58 58 58 58 2A 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 2A
Component Identification from Engine #1 (0): {
Make: INT
Model:<PHONE_NUMBER>
Serial: XXXXXXXXXXXXXX
Unit:<PHONE_NUMBER>000000

Previously fixed:


multihandshake idea

Right now, for each type of connection upgrade, we perform a multistream handshake in order to be able to support several protocols on top of the same transport. This adds a considerable amount of RTT's  to the connection before it is ready, which might represent a problem for slow links.
0 handshake can bring considerable amounts performance for streaming scenarios. What if we take the multistream handshake and make a special case for a multihandshake such that we send: /multihandshake/all/the/protocols/that/will/be/stacked/ as one message, instead of handshaking each at all times. This adds the problem that by narrowing down the number of protocols + versions we are interested in talking (muxer1 vs muxer2 or encryptedChannel1 vs encryptedChannel2), we can miss a successful handshake, but for previous known peers, it should work like a charm.

i don't think you need anything different. multistream is designed to do this, you can send everything straight up and achieve a 0-RTT open, given the protocols are known a-priori (or you guess right)


Problem to test docker api scan

I've tested this command with API scan, but the scan doesn't get all the Swagger endpoints URLs in the scan, all my environments use the keycloak auth for the access control, I follow the zap API scan documentation to run the auth commands, and for me, doesn't work.
I've tested another application with keycloak and doesn't work to find the URLs automatically, the zap app has any function to run with keycloak or the zap scan doesn't work well with the keycloak authentication?

Hi, can you post the full output?
And do you have a demo environment available? The scanner should be able to authenticate, but I don't know what kind of token Keycloak uses to manage the session.


Low-priority, very minor (but probably very quick) fix needed

There is a sizing problem in the Safari display.  The lefthand column needs to be a little bit wider to accommodate the icons (or the icons could be made smaller):

This issue is only visible when you are logged in.

Fixed


When standalone containers with exposed port become unhealthy and get deleted , new containers are not able to start successfully.

Rancher Versions: Build from wagyu
Steps to reproduce the problem:
Create a standalone container which is healthcheck enabled and having 1 exposed port.
Make the container go unhealthy ( In my case i made the healthcheck url unavailable).
This results in container being marked unhealthy and then deleted.
New container that gets created is not able to start successfully because of scheduling errors:
2017-03-28 16:50:43,152 INFO  [afb8b341-88be-4811-a9a6-3c8968388c0b:182697] [instance:4887] [instance.start->(InstanceStart)->instance.allocate] [] [torService-1183] [c.p.e.p.i.DefaultProcessInstanceImpl] Allocation failed: No healthy hosts with sufficient resources available
2017-03-28 16:50:43,152 DEBUG [afb8b341-88be-4811-a9a6-3c8968388c0b:182697] [instance:4887] [instance.start->(InstanceStart)] [] [torService-1183] [c.p.e.p.i.DefaultProcessInstanceImpl] Exiting [UNKNOWN_EXCEPTION] process [instance.allocate:null] on resource [4887]
2017-03-28 16:50:43,152 INFO  [afb8b341-88be-4811-a9a6-3c8968388c0b:182697] [instance:4887] [instance.start->(InstanceStart)] [] [torService-1183] [i.c.p.process.instance.InstanceStart] Failed to Scheduling for instance [4887]

Happens because for standalone container we create a replacement before killing the unhealthy instance. Should be fixed by swapping this order.

I do have other hosts in the setup where this instance could have landed , why where the other hosts in the setup not considered for scheduling ?

I do have other hosts in the setup where this instance could have landed , why where the other hosts in the setup not considered for scheduling ?

Fixed in wagyu branch

Tested with build from master(07/07)
When standalone containers with exposed port become unhealthy and get deleted , new containers are  able to start successfully.


Should return 400 for AmbiguousApiVersionException instead of 500

Is there a reason the versioning middleware returns HTTP 500 (server error) when it encounters AmbiguousApiVersionException instead of 400 (user error)? Is there a way to configure library to return 400, or must we handle this in our exception handler?
We are using this version:
<PackageReference Include="Microsoft.AspNetCore.Mvc.Versioning" Version="5.0.0" />

When there's a request containing two api-version params in query string, service would return 500.
Microsoft.AspNetCore.Mvc.Versioning.AmbiguousApiVersionException: The following API versions were requested: 2021-06-01-preview, foo. At most, only a single API version may be specified. Please update the intended API version and retry the request.
   at Microsoft.AspNetCore.Mvc.CollectionExtensions.EnsureZeroOrOneApiVersions(ICollection`1 apiVersions)
   at Microsoft.AspNetCore.Mvc.Versioning.QueryStringApiVersionReader.Read(HttpRequest request)
   at Microsoft.AspNetCore.Mvc.Versioning.ApiVersionReader.CombinedApiVersionReader.Read(HttpRequest request)
...
   at Microsoft.AspNetCore.Builder.Extensions.MapWhenMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)

The middleware never returns 500 directly; however, AmbiguousApiVersionException will be thrown any time multiple, different API versions are requested and they are not exactly the same (e.g. ambiguous; even semantic equivalence is not allowed). When an unhandled exception occurs, ASP.NET Core propagates that as 500.
The one and only place this should happen is via IApiVersioningFeature.RawRequestedApiVersion which can propagate through IApiVersioningFeature.RequestedApiVersion. Extension methods such as GetRequestedApiVersion() and any other approach ultimately go through IApIVersioningFeature. There's no clear path to transition from accessing a property to a HTTP response, which is why an exception is necessary. API Versioning knows where ambiguous API versions might appear,  appropriately handles it, and propagates 400 instead (as seen here). I suspect you have some code somewhere that is accessing the value early in the pipeline. By the time the routing system goes through API Versioning and dispatches to your endpoint, it should be impossible to encounter AmbiguousApiVersionException. If that's not the case, you'll need to elaborate on your scenario some more.
Why does this happen? The crux of the problem comes down to IApiVersionFeature.RawRequestedVersion and the original design of IApiVersionReader.Read. Both of these only allow a single, raw API version value to be read. If there are none, then null can be used. What should it do if there are multiple? It shouldn't be null as that would imply no API version. There's no way to return multiple API versions; hence, AmbiguousApiVersionException.
In 5.1.0, IApiVersioningFeature adds RawRequestedApiVersions which is a IReadOnlyList<string> of all raw API versions. This will always work whether there are 0 or many API versions requested. Starting in 6.0.0, IApiVersionReader.Reader now returns IReadOnlyList<string> which supports multiple, raw API versions from a single reader (ex: the query string) as well as composition (ex: the query string and header). Supporting the possibility of multiple API versions end-to-end removed a lot of cases for exception handling (and improves performance). It also gives a clear way to test if there are multiple API version were requested and what the original, raw values were (for debugging and error reporting).

Whenever Microsoft finally wraps up the code signing certificate, I'll be able to officially publish the RawRequestedApiVersions enhancement. 😞

I hope that helps clear things up.

@commonsensesoftware Thanks for the detaild explanation! You were absolutely spot on that there's something reading the API version. We have a metrics middleware that tries to read the API version as the request comes in. Swallowing the exception there allows the rest of the pipeline to properly return 400 to user. Thanks again!


cargo-charon fails to parse arguments correctly

Cargo version: cargo 1.66.0-nightly (7e484fc1a 2022-10-27)
I installed cargo-charon as a Cargo CLI extension using cargo install. Invoking cargo charon fails for me on a simple repository due to what appears to be an argument-parsing error:
error: Found argument 'charon' which wasn't expected, or isn't valid in this context

USAGE:
    cargo-charon [FLAGS] [OPTIONS]

For more information try --help

Manually running the cargo-charon executable from where Cargo installed it to works as expected.

I took the above approach because the executable is named like a Cargo plugin, and because the documentation suggests something which no longer works (cargo run errors out due to the lack of a default executable.) If cargo-charon isn't meant to be installed like this, I'm happy to update the docs in some way.

I'm really sorry for my late answer: for some reason I didn't receive a notification for your issue. I updated the README in a recent PR: could you confirm that the new explanations work for you? Also note that I changed the name cargo-charon to charon as it was indeed confusing.

Looks good! I think that this is a non-issue with a renamed executable, since it doesn't look like a cargo plugin.


CFT scanning doesn't support short form intrinsic functions

terrascan version: v1.19.1
Operating System: MacOS

Description
CFT scanning doesn't support short form intrinsic functions and treats a use of short form as a vulnerability.
I've seen this issue occur for multiple resource types in our YAML templates.
What I Did
An example yaml resource is seen below - notice that it uses the short form of the ImportValue function (https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/intrinsic-function-reference-importvalue.html#w45aac23c24c46b9) :
  SQSQueue:
    Type: AWS::SQS::Queue
    UpdateReplacePolicy: Delete
    DeletionPolicy: Delete
    Properties:
      QueueName: test
      KmsDataKeyReusePeriodSeconds: 300
      KmsMasterKeyId: !ImportValue imported_value
      MessageRetentionPeriod: 345600 #4 days
      RedrivePolicy:
        deadLetterTargetArn: <insert_dead_letter_arn>
        maxReceiveCount: 5

Running a scan on that resource returns an error (terrascan scan -i cft -f sqs-example.yaml --severity high -o yaml
- rule_name: sqsSseDisabled
          description: Ensure that your Amazon Simple Queue Service (SQS) queues are protecting the contents of their messages using Server-Side Encryption (SSE). The SQS service uses an AWS KMS Customer Master Key (CMK) to generate data keys required for the encryption/decryption process of SQS messages. There is no additional charge for using SQS Server-Side Encryption, however, there is a charge for using AWS KMS
          rule_id: AC_AWS_0366
          severity: HIGH
          category: Security Best Practices

but updating the resource to use the long form OR updating the resource to use a standard string will fix the problem, for example changing to Fn::ImportValue):
  SQSQueue:
    Type: AWS::SQS::Queue
    UpdateReplacePolicy: Delete
    DeletionPolicy: Delete
    Properties:
      QueueName: test
      KmsDataKeyReusePeriodSeconds: 300
      KmsMasterKeyId: Fn::ImportValue imported_value
      MessageRetentionPeriod: 345600 #4 days
      RedrivePolicy:
        deadLetterTargetArn: <insert_dead_letter_arn>
        maxReceiveCount: 5

This happens for Fn:Ref (!Ref) and others.

I did check the v1.19.1 code and it seems that this should be supported, so I am not sure why it is failing:
https://github.com/tenable/terrascan/blob/v1.19.1/pkg/iac-providers/cft/v1/yamlparse.go


feat: dpop

:white_check_mark: Build TheIdServer 7.3.1-feat-DPoP1-0035 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/6dfb6ce08f by @invalid-email-address)


prototype support for 1xx informational responses

This is a prototype intended to spur discussion about what support for 1xx informational responses should look like in a Hyper server.  The good news is that it works great (for HTTP/1 only, so far).  The bad news is it's kind of ugly.  Here's what I did:

Add ext::InformationalSender, a type which wraps a futures_channel::mspc::Sender<Response<()>>.  This may be added as an extension to an inbound Request by the Hyper server, and the application and/or middleware may use it to send one or more informational responses before sending the real one.

Add code to proto::h1::dispatch and friends to add such an extension to each inbound request and then poll the Receiver end along with the future representing the final response.  If the app never sends any informational responses, then everything proceeds as normal.  Otherwise, we send those responses as they become available until the final response is ready.

TODO items:

[ ] Also support informational responses in the HTTP/2 server.
[ ] Determine best way to handle when the app sends an informational response with a non-1xx status code.  Currently we just silently ignore it.
[ ] Come up with a less hacky API?
[ ] Add test coverage.

Cool start, thanks!
I'd suggest trying to figure out what the API looks like for users as the next step. That's the part that we need to get right, and can't really change once it's stable. The implementation details inside we can optimize and refactor whenever.

I'd suggest trying to figure out what the API looks like for users as the next step.

@vikanezrimaya's proposal looks fine to me, and could easily be implemented using this PR as a foundation.  Any concerns with that approach?


fix: Link to simple example in README

See d8ce7e60739447427f0a705e5995e9c13644b4f8

Oh thank you. Somehow I didn't receive a notification about your contribution!


OLS-1229: Refactor runner.py - SSL-related utility functions

Description
Refactor runner.py - SSL-related utility functions
Type of change

[x] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)
[ ] CI configuration change
[ ] Konflux configuration change

Related Tickets & Documents

Related Issue #OLS-1229

/test e2e-ols-cluster-4-15

/test e2e-ols-cluster-4-15

Codecov Report
Attention: Patch coverage is 92.30769% with 2 lines in your changes missing coverage. Please review.

Project coverage is 94.52%. Comparing base (dcd093e) to head (349e003).
Report is 7 commits behind head on main.

Files with missing lines
Patch %
Lines

runner.py
33.33%
2 Missing :warning:

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1910      +/-   ##
==========================================
+ Coverage   91.60%   94.52%   +2.91%
==========================================
  Files          65       66       +1
  Lines        2894     2903       +9
==========================================
+ Hits         2651     2744      +93
+ Misses        243      159      -84

Files with missing lines
Coverage Δ

ols/utils/ssl.py
100.00% <100.00%> (ø)

runner.py
55.12% <33.33%> (+7.25%)
:arrow_up:

... and 7 files with indirect coverage changes

----
🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests

/test e2e-ols-cluster-4-15

/approve
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

/lgtm

/override "ci/prow/e2e-ols-cluster-4-17"


docs(datasets): Use kedro-sphinx-theme

Description

Resolve https://github.com/kedro-org/kedro-plugins/issues/662
Development notes

Very similar to https://github.com/kedro-org/kedro/pull/3675
Checklist

[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added a description of this change in the relevant RELEASE.md file
[ ] Added tests to cover my changes

🔥

Spotted a minor issue, possibly a misconfiguration on the theme https://github.com/kedro-org/kedro-sphinx-theme/issues/3


Support multibinding - map

Support multibinding for map

Resolved: https://github.com/Park-SM/HiltBinder/pull/11


harmonist: init at 0.1

Motivation for this change
To add a new roguelike game.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

:alien:

@GrahamcOfBorg build harmonist

@etu @JohnAZoidberg  It's done. :alien:

@freepotion Thanks! :)

@etu Thank you too! :)


Having an issue with NAT Gateway in 0.9.1

Been using terraforming for a while and I am using 0.9.1, but NAT Gateway support is not available as a resource. Tried clean install with gem, same issue.
*** LOCAL GEMS ***
aws-sdk (2.3.21, 2.3.19)
aws-sdk-core (2.3.21, 2.3.19)
aws-sdk-resources (2.3.21, 2.3.19)
terraforming (0.9.1)
.....

I'm sorry I haven't released the version including NAT support yet.
If you'd like to use this feature now, please install from the source code.
$ git clone https://github.com/dtan4/terraforming.git
$ cd terraforming
$ rake install

I'll release the new version in a week.

Thank you.

@alexmurray100 Sorry for being late! Terraforming v0.10.0 w/ NAT Gateway support was released.
You can get the latest version by gem install terraforming -v 0.10.0.


Version?

Hello to everyone, today i was adding ban implementation to my bot, but i've seen it was added in the 0.2.0 version.
I'm using the 0.1.5 and when i try to change, eclipse give me an error (in the pom.xml) saying: "missing artifact for skype4j:0.2.0"

Hello.
The skype4j:0.2.0 don't published in central maven, yet: http://search.maven.org/#search|ga|1| skype4j
You can published current master version 0.2.0-SNAPSHOT in local maven repo on your machine.
for it:
git clone https://github.com/samczsun/Skype4J.git && cd Skype4J && mvn clean install
and include this version to your pom.xml file.

Also you can add github repo to maven and get artifacts by commit
<repositories>
	<repository>
	    <id>jitpack.io</id>
	    <url>https://jitpack.io</url>
	</repository>
</repositories>

<dependency>
    <groupId>com.github.samczsun</groupId>
    <artifactId>Skype4J</artifactId>
    <version>-v0.1.5-g1fc4e07-42</version>
</dependency>

[this site] (https://jitpack.io/) helps to define version of artifact
Also there are some errors in ban implementation described in #134. Be careful=)

When new version will be published to Maven repo?

@slavias don't count on it being anytime soon, sam appears to have been very busy recently.


Broadcasting from webrtc enabled browser to non-webrtc enabled browsers

Say I have a scenario where:

Browser A (Chrome) is broadcasting webcam
Licode server receives
Licode relays the video to Browser B (Safari) and Browser C (IE)

Safari and IE don't support WebRTC, so with licode are they able to view A's video (as passive participants) just using a <video> tag or something like that?

I think what you need is live transcoding the stream and broadcast it with HLS or RTMP. Check out #145. I'm still figuring out how to do it :)

Hmm. Why do you need to transcode the stream? WebRTC and  can both use VP8 right? (sorry for the basic questions, this is new to me)

I am new to Licode so please correct me if I'm wrong. From what I understand, Safari and IE don't support WebRTC and VP8. So we need to transcode the stream and broadcast the data with streaming protocols like RTMP or similar.

Your best bet is to use a plug-in that implements WebRTC, such as the temasys one.  Converting to HLS or RTMP won't help you.

have you had any progress?
I am thinking of using some binaryjs fallback, it would be could If I could do it directly from licode instance, any thoughts ?


[SPARK-32405][SQL] Apply table options while creating tables in JDBC Table Catalog

What changes were proposed in this pull request?
Currently in JDBCTableCatalog, we ignore the table options when creating table.
    // TODO (SPARK-32405): Apply table options while creating tables in JDBC Table Catalog
    if (!properties.isEmpty) {
      logWarning("Cannot create JDBC table with properties, these properties will be " +
        "ignored: " + properties.asScala.map { case (k, v) => s"$k=$v" }.mkString("[", ", ", "]"))
    }

Why are the changes needed?
need to apply the table options when we create table
Does this PR introduce any user-facing change?
no
How was this patch tested?
add new test

Test build #130302 has started for PR 30154 at commit edf98d0.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34902/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34904/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34902/

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-K8s/34902/
Test PASSed.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34904/

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-K8s/34904/
Test PASSed.

Test build #130302 has finished for PR 30154 at commit edf98d0.

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

Merged build finished. Test FAILed.

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

retest this please

Test build #130306 has started for PR 30154 at commit edf98d0.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34908/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34908/

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-K8s/34908/
Test PASSed.

Test build #130306 has finished for PR 30154 at commit edf98d0.

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/130306/
Test PASSed.

cc @cloud-fan @maropu @MaxGekk @ScrapCodes

We also need to retrieve the table comment in TableCatalog.loadTable

We also need to retrieve the table comment in TableCatalog.loadTable

So we will retrieve table comment using a query and then set this comment in JDBCOptions in JDBCTable, right? This query is different for different databases, for example, MySQL has SELECT table_comment FROM INFORMATION_SCHEMA.TABLES WHERE table_name='tablename', DB2 has SELECT remarks from syscat.tables where tabname='tablename'

This query is different for different databases

ah, it's so complicated. Let's forget about it for this PR.

Test build #130480 has started for PR 30154 at commit 74469a6.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35084/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35084/

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-K8s/35084/
Test PASSed.

Test build #130480 has finished for PR 30154 at commit 74469a6.

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

Merged build finished. Test FAILed.

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

retest this please

Test build #130485 has started for PR 30154 at commit 74469a6.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35089/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35089/

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-K8s/35089/
Test PASSed.

Test build #130485 has finished for PR 30154 at commit 74469a6.

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/130485/
Test PASSed.

Test build #130542 has started for PR 30154 at commit e267481.

Test build #130544 has started for PR 30154 at commit d4b1cf9.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35142/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35143/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35142/

Build finished. Test FAILed.

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

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35143/

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-K8s/35143/
Test PASSed.

Test build #130542 has finished for PR 30154 at commit e267481.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
abstract class JdbcDialect extends Serializable with Logging

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/130542/
Test PASSed.

Test build #130544 has finished for PR 30154 at commit d4b1cf9.

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/130544/
Test PASSed.

I think the principle is: when users specify some table properties, JDBC V2 should fail if the underlying database can't support the properties.
I prefer a dialect API to generate CREATE TABLE SQL statement, because comment is not the only table property. It's cumbersome to add one dialect API for each table property, like comment, charset, etc.

when users specify some table properties, JDBC V2 should fail if the underlying database can't support the properties.

I agree JDBC V2 should fail if the underlying database can't support the properties. However, I feel it's hard to come up with a complete list of the supported properties for each of the databases. It is easy to have a complete list of the supported properties for MySQL because in CREATE TABLE syntax, it explicitly lists the table_options:
table_option: {
    AUTO_INCREMENT [=] value
  | AVG_ROW_LENGTH [=] value
  | [DEFAULT] CHARACTER SET [=] charset_name
  | CHECKSUM [=] {0 | 1}
  | [DEFAULT] COLLATE [=] collation_name
  | COMMENT [=] 'string'
  | COMPRESSION [=] {'ZLIB' | 'LZ4' | 'NONE'}
  | CONNECTION [=] 'connect_string'
  | {DATA | INDEX} DIRECTORY [=] 'absolute path to directory'
  | DELAY_KEY_WRITE [=] {0 | 1}
  | ENCRYPTION [=] {'Y' | 'N'}
  | ENGINE [=] engine_name
  | ENGINE_ATTRIBUTE [=] 'string'
  | INSERT_METHOD [=] { NO | FIRST | LAST }
  | KEY_BLOCK_SIZE [=] value
  | MAX_ROWS [=] value
  | MIN_ROWS [=] value
  | PACK_KEYS [=] {0 | 1 | DEFAULT}
  | PASSWORD [=] 'string'
  | ROW_FORMAT [=] {DEFAULT | DYNAMIC | FIXED | COMPRESSED | REDUNDANT | COMPACT}
  | SECONDARY_ENGINE_ATTRIBUTE [=] 'string'
  | STATS_AUTO_RECALC [=] {DEFAULT | 0 | 1}
  | STATS_PERSISTENT [=] {DEFAULT | 0 | 1}
  | STATS_SAMPLE_PAGES [=] value
  | TABLESPACE tablespace_name [STORAGE {DISK | MEMORY}]
  | UNION [=] (tbl_name[,tbl_name]...)
}

but other databases don't have explicitly defined table_options. I am not sure which of the properties should be considered as valid table properties. For example, for postgresql, https://www.postgresql.org/docs/9.1/sql-createtable.html, should we treat TABLESPACE tablespace or CONSTRAINT constraint_name as valid table properties? I feel it might be better to send properties to underlying databases and let databases decide whether to fail the CREATE TALBE or not, than trying to have a complete list of the supported properties on Spark side.

... send properties to underlying databases and let databases decide whether to fail the CREATE TALBE or not

This works. Does it conflict with the dialect API to generate CREATE TABLE statement?

Does it conflict with the dialect API to generate CREATE TABLE statement?

No, it doesn't conflict with the dialect API to generate CREATE TABLE statement. The table properties are appended to the end of the CREATE TABLE statement as createTableOptions.
s"CREATE TABLE $table ($strSchema) $createTableOptions"

Test build #130640 has started for PR 30154 at commit bb9c4fc.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35250/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35250/

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-K8s/35250/
Test PASSed.

Test build #130640 has finished for PR 30154 at commit bb9c4fc.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #130670 has started for PR 30154 at commit 39057aa.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35281/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35281/

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-K8s/35281/
Test PASSed.

Test build #130670 has finished for PR 30154 at commit 39057aa.

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/130670/
Test PASSed.

Test build #130696 has started for PR 30154 at commit ecc9a4e.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35306/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35306/

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-K8s/35306/
Test PASSed.

Test build #130696 has finished for PR 30154 at commit ecc9a4e.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #130731 has started for PR 30154 at commit ecc9a4e.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35340/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35340/

Merged build finished. Test PASSed.

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-K8s/35340/
Test PASSed.

Test build #130731 has finished for PR 30154 at commit ecc9a4e.

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/130731/
Test PASSed.

thanks, merging to master!

Thank you! @cloud-fan


Protocol buffer definitions for baremetalhost agent

Yes! I set up a meeting for tomorrow to close in on this quickly. Thanks!
From: Madhan Raj Mookkandy @.>
Sent: Thursday, March 11, 2021 4:47 PM
To: microsoft/moc @.>
Cc: Vincent Nguyen @.>; Author @.>
Subject: Re: [microsoft/moc] Protocol buffer definitions for baremetalhost agent (#36)
@madhanrm commented on this pull request.

In rpc/baremetalhostagent/agent/moc_baremetalhost_agent.protohttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fmoc%2Fpull%2F36%23discussion_r592832664&data=04|01|vinguyen%40microsoft.com|fe3fa4331bf8450a948308d8e4f06342|72f988bf86f141af91ab2d7cd011db47|1|0|637511068400365532|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=9mHLcHkScckdwOMbODIjLhsbjeyRJodrF6jKyS7efLk%3D&reserved=0:

HighAvailabilityState highAvailabilityState = 13;

  Tags tags = 14;

+}

+message BareMetalHostRequest {

  BareMetalHost bareMetalHost = 1;

+}

+message BareMetalHostResponse {

  BareMetalHost bareMetalHost = 1;

  google.protobuf.BoolValue Result = 2;

  string  Error = 3;

+}

+service BareMetalHostAgent {

  rpc CreateOrUpdate(BareMetalHostRequest) returns (BareMetalHostResponse) {}

can we call for a small meeting and close on the gap here

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fmoc%2Fpull%2F36%23discussion_r592832664&data=04|01|vinguyen%40microsoft.com|fe3fa4331bf8450a948308d8e4f06342|72f988bf86f141af91ab2d7cd011db47|1|0|637511068400365532|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=9mHLcHkScckdwOMbODIjLhsbjeyRJodrF6jKyS7efLk%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQWDTQL4E7664QJVTFZSPMLTDFJALANCNFSM4Y266CRQ&data=04|01|vinguyen%40microsoft.com|fe3fa4331bf8450a948308d8e4f06342|72f988bf86f141af91ab2d7cd011db47|1|0|637511068400375528|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Js4JYqzu658tDhLxhNEtt62C9iFBLSyFUhjY3g8AkDE%3D&reserved=0.

Madhan, how is this credential monitor used for other agents? What is the scenario? Thanks.
From: Madhan Raj Mookkandy @.>
Sent: Friday, March 12, 2021 5:54 PM
To: microsoft/moc @.>
Cc: Vincent Nguyen @.>; Author @.>
Subject: Re: [microsoft/moc] Protocol buffer definitions for baremetalhost agent (#36)
@madhanrm commented on this pull request.

In rpc/baremetalhostagent/admin/credentialmonitor/moc_baremetalhostagent_credentialmonitor.protohttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fmoc%2Fpull%2F36%23discussion_r593540959&data=04|01|vinguyen%40microsoft.com|c5e3f3d8f64544108b4708d8e5c2d991|72f988bf86f141af91ab2d7cd011db47|1|0|637511972326599693|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=z%2Ff5EMPUcOJYsaYyqx6Rh2meq80k16ULfHXooScnHqc%3D&reserved=0:

+message CredentialMonitorRequest {

  CredentialMonitor CredentialMonitor = 1;

+}

+message CredentialMonitorResponse {

CredentialMonitor CredentialMonitor = 1;

string  Error = 2;

+}

+message CredentialMonitor {

bytes certificate = 1 [deprecated = true];

CertificateStatus status = 2;

string newCertificate = 3;

+}

+service CredentialMonitorAgent {
Whats the feature/requirement here.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fmoc%2Fpull%2F36%23pullrequestreview-611370675&data=04|01|vinguyen%40microsoft.com|c5e3f3d8f64544108b4708d8e5c2d991|72f988bf86f141af91ab2d7cd011db47|1|0|637511972326609688|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=aUCdU8E%2BKtz6mgy2qNfqqQMQyomWzeFmMqnmVd3X7h8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQWDTQNCJH5PL3DQP2YLZ23TDKZR5ANCNFSM4Y266CRQ&data=04|01|vinguyen%40microsoft.com|c5e3f3d8f64544108b4708d8e5c2d991|72f988bf86f141af91ab2d7cd011db47|1|0|637511972326609688|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2FUw60kkeSxGA%2F8mFyksYLV%2Bmo0XFuacOViMhSboGMgs%3D&reserved=0.

@madhanrm I resolved all your comments. Thanks for the feedbacks. I need to check in this change in to unblock Davut for Cloud agent change. Please feel free to review it and let me know if you have any more feedbacks. I will address them in the next change.


Search backend

closes #10

Seems fine and clear
Spelling error noticed /closestchools/
// get the closest school
app.get('/closestchools', (req, res, next)=>{

Done!


zinit is not leveraging $TMPDIR

Describe the bug
Using Termux I noticed that a few errors were spat out when updating using zinit update -p 20:
[...]
2019-10-24 1eb4759 better completion with filetypes
2019-10-23 f6eff10 Add cppcheck support
Updating a6e641b..d4e54f1
Fast-forward
Installed 146 completions. They are stored in $INSTALLED_COMPS array. .zinit-install-completions:67: no such file or directory: /tmp/zinit.installed_comps.20640.lst
.zinit-install-completions:68: no such file or directory: /tmp/zinit.skipped_comps.20640.lst
[7]    done       .zinit-update-or-status update "$user" "$plugin" &>>| $PUFILE
The update took 55.71 seconds

This is due to /tmp being inaccessible in Termux
To Reproduce
Run zinit update on Termux.
Expected behavior
zinit shouldn't assume /tmp is accessible.
If $TMPDIR is set use its value instead of assuming /tmp.
TMPDIR=${TMPDIR:-/tmp}
Versions:

$ZSH_VERSION == 5.8
Operating system name and version (uname -a):
Linux localhost 4.4.218-Xiaomi_SDM660-LA.UM.8.2.r1-06300+ #1 SMP PREEMPT Fri Apr 17 22:12:22 MSK 2020 aarch64 Android

It's fixed, thanks.


is there any way to add dark mode?

@jatinhemnani01 I think carbon has already. Is it the gray100 theme that you are looking for?
Under Styling.

Yeah. It's supported for all the components?

Yes. On the docs, if you switch the them it will be preserved while you are navigating on the other components. so you can see how they would like. Also, under each component you can switch the theme.


Way to change SurfaceAndGroundControl based on percantage

In Mega Taiga there normally are these SurfaceAndGroundControl settings:
DIRT:2, DIRT, -0.95, DIRT:1, DIRT, 1.75
Is there a way to make it like in WorldEdit, where you can add more than one material and give them percentages like "75%Dirt;2;25%Dirt:1, Dirt, -0.95, [...]" ?
If not, can you add it or is it impossible?

As a quite circumstantial solution i would try to list a lot of Noise Values like:
DIRT:2, DIRT, -6.5, DIRT:1, DIRT, -6.0, DIRT:2, DIRT, -5.5, and so on...
but i guess it consumes a lot of power and makes the generation a lot slower, right?

So... Apperantly it doesn't cost much power, world generation is maybe 1 second slower than before, but i can live with it :D


526 season health checks

Resolves #526

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

178 of 191 (93.19%)  changed or added relevant lines in 14 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 95.894%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

CourageScores/Services/Health/Checks/FixturesAfterStartDate.cs
4
5
80.0%

CourageScores/Services/Health/Checks/FixturesBeforeEndDate.cs
4
5
80.0%

CourageScores/Services/Health/Checks/TeamsHaveBothLegs.cs
30
31
96.77%

CourageScores/ClientApp/src/components/division_health/DivisionHealth.js
16
18
88.89%

CourageScores/ClientApp/src/components/division_health/ViewHealthCheck.js
2
10
20.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
6913

Relevant Lines:
6995

💛  - Coveralls