Problems running mnist_estimator in distributed mode

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.12
Python version: 3.6.5
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A

Describe the current behavior
Running below code that I found in many pages on the net, I faced some problems:
import json
import os
import tensorflow as tf
from tensorflow.contrib.learn.python.learn.datasets.mnist import read_data_sets
data_dir = '.\\MNIST_data'
log_dir = '.\log_dist'
batch_size = 512
tf.logging.set_verbosity(tf.logging.INFO)

def keras_model(lr, decay):
    """Return a CNN Keras model"""
    input_tensor = tf.keras.layers.Input(shape=(784,), name='input')

    temp = tf.keras.layers.Reshape([28, 28, 1], name='input_image')(input_tensor)
    for i, n_units in enumerate([32, 64]):
        temp = tf.keras.layers.Conv2D(n_units, kernel_size=3, strides=(2, 2),
                                      activation='relu', name='cnn'+str(i))(temp)
        temp = tf.keras.layers.Dropout(0.5, name='dropout'+str(i))(temp)
    temp = tf.keras.layers.GlobalAvgPool2D(name='average')(temp)
    output = tf.keras.layers.Dense(10, activation='softmax', name='output')(temp)

    model = tf.keras.models.Model(inputs=input_tensor, outputs=output)
    optimizer = tf.keras.optimizers.Adam(lr=lr, decay=decay)
    model.compile(optimizer=optimizer, loss='sparse_categorical_crossentropy', metrics=['accuracy'])
    print(model.summary())
    return model

def main():
    """Main function"""
    data = read_data_sets(data_dir,
                          one_hot=False,
                          fake_data=False)
    model = keras_model(lr=0.001, decay=0.001)
    config = tf.estimator.RunConfig(
                model_dir=log_dir,
                save_summary_steps=1,
                save_checkpoints_steps=100)
    estimator = tf.keras.estimator.model_to_estimator(model, model_dir=log_dir, config=config)

    train_input_fn = tf.estimator.inputs.numpy_input_fn(
                         x={'input': data.train.images},
                         y=data.train.labels,
                         num_epochs=None,   # run forever
                         batch_size=batch_size,
                         shuffle=True)
    eval_input_fn = tf.estimator.inputs.numpy_input_fn(
                         x={'input': data.test.images},
                         y=data.test.labels,
                         num_epochs=1,
                         shuffle=False)

    train_spec = tf.estimator.TrainSpec(input_fn=train_input_fn,
                                        max_steps=2000)
    eval_spec = tf.estimator.EvalSpec(input_fn=eval_input_fn,
                                      #throttle_secs=1,
                                      steps=None    # until the end of evaluation data
                                      )

    evaluate_result = tf.estimator.train_and_evaluate(estimator, train_spec, eval_spec)
    print("Evaluation results:")
    for key in evaluate_result[0].keys():
        print("   {}: {}".format(key, evaluate_result[0][key]))

And then the rest of the code just included the TF_CONFIG definition for chief, worker and ps. I faced below issues:

I was able to run this code on Tensorflow 1.12 but not on Tensorflow 1.13, where I got the error ValueError: Can not squeeze dim[1], expected a dimension of 1, got 10 for 'metrics/acc/remove_squeezable_dimensions/Squeeze' (op: 'Squeeze') with input shapes: [512,10]. What is the reason?
I could get evaluation results printed at the end of training when I was running program in non-distributed mode, but I get below error when it tries to print the final evaluation results in distributed mode:

Traceback (most recent call last):
  File "mnist_estimator.py", line 81, in <module>
    main()
  File "mnist_estimator.py", line 62, in main
    for key in evaluate_result[0].keys():
TypeError: 'NoneType' object is not subscriptable

The final loss for distributed learning was higher than non-distributed learning (for the same number of training steps). What can be the reason? Is it the nature of distribution?
When running in distributed mode, the chief or worker are not waiting for the other party to start and immediately starts training (when the other party joins they do the task together, though). I thought they should wait for each other to be ready (as it was in my previous experiences with Tensorflow distributed training), isn't it?
What I read in Tensorflow-related pages about data-parallelism is that there are the same copies of code for different servers except in assignment in TF_CONFIG. The chief synchronizes the parameters update and parameter servers keeps the parameters, but I don't clearly understand who split the data between different workers. Is there just one copy at the chief server and it will split the data and send batches to the workers, or the workers each have a local copy of data and do the splitting and skip some data themselves?

Can you please help us in providing full code along with TF_CONFIG to reproduce the issue in our environment.Thanks!

@ravikyram
Thanks. The above is the full code. Just add below TF_CONFIG definition (replacing ... with your ip addresses and selecting any of chief/worker/ps according to the server):
if __name__ == '__main__':
    TF_CONFIG = {
        'task': {
            'type': 'chief/worker/ps',
            'index': 0
        },
        'cluster': {
            'chief': [...],
            'worker': [...],
            'ps': [...]
        }
    }
    os.environ['TF_CONFIG'] = json.dumps(TF_CONFIG)
    main()

Is it possible for you to try latest TF versions (!pip install tensorflow==1.15.0-rc1) and let us know whether the issue persists? There were lots of performance improvements in the latest versions. Thanks!

@ravikyram
I tried TF 1.15.0-rc1 but it can not be loaded on my system and I get this error:
Traceback (most recent call last):
  File "D:\Shahriar\tf1.15_venv\lib\site-packages\tensorflow_core\python\pywrap_tensorflow.py", line 58, in <module>
    from tensorflow.python.pywrap_tensorflow_internal import *
  File "D:\Shahriar\tf1.15_venv\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 28, in <module>
    _pywrap_tensorflow_internal = swig_import_helper()
  File "D:\Shahriar\tf1.15_venv\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
  File "D:\Shahriar\tf1.15_venv\lib\imp.py", line 243, in load_module
    return load_dynamic(name, filename, file)
  File "D:\Shahriar\tf1.15_venv\lib\imp.py", line 343, in load_dynamic
    return _load(spec)
ImportError: DLL load failed: The specified module could not be found.

@shahriar49,

Regarding ValueError: If you face problem in installing 1.15-rc0, you can try installing 1.14 and check if the issue still persists. Also, you can refer this Stack Overflow Issue and check if it helps.
I could get evaluation results printed at the end of training when I was running program in non-distributed mode, but I get below error when it tries to print the final evaluation results in distributed mode: ====> Can you please confirm if you able to run this in 1.12?

@rmothukuru
I checked again and I can run the code in 1.12 but not in 1.13. I will check 1.14 as well.

@rmothukuru
I ran the code in TF 1.14 and the below problems still exist:

I could get evaluation results printed at the end of training when I was running program in non-distributed mode, but I get below error when it tries to print the final evaluation results in distributed mode:

Traceback (most recent call last):
  File "mnist_estimator.py", line 81, in <module>
    main()
  File "mnist_estimator.py", line 62, in main
    for key in evaluate_result[0].keys():
TypeError: 'NoneType' object is not subscriptable

When running in distributed mode, the chief or worker are not waiting for the other party to start and immediately starts training (when the other party joins they do the task together, though). I thought they should wait for each other to be ready (as it was in my previous experiences with Tensorflow distributed training), isn't it?

What I read in Tensorflow-related pages about data-parallelism is that there are the same copies of code for different servers except in assignment in TF_CONFIG. The chief synchronizes the parameters update and parameter servers keeps the parameters, but I don't clearly understand who split the data between different workers. Is there just one copy at the chief server and it will split the data and send batches to the workers, or the workers each have a local copy of data and do the splitting and skip some data themselves?

@shahriar49,
Please confirm if we can consider this issue as duplicate of #32787, as the Code is same and as we to use tf.distributionstrategy for distributed training. Thanks!

@rmothukuru
Yes the code is almost the same, but the problems that I mentioned here is different than the other one. Here I am not using strategy clause and get some issues. There I am using strategy clause and it fails to work. So the problems discussed in each case is different.

@shahriar49 Can you try TF1.15.0rc3 and let us know whether issue persists with latest TF1.x version? Thanks!

@jvishnuvardhan
I will do, but can somebody at least answer my last question:

What I read in Tensorflow-related pages about data-parallelism is that there are the same copies of code for different servers except in assignment in TF_CONFIG. The chief synchronizes the parameters update and parameter servers keeps the parameters, but I don't clearly understand who split the data between different workers. Is there just one copy at the chief server and it will split the data and send batches to the workers, or the workers each have a local copy of data and do the splitting and skip some data themselves?

@yuefengz / @rchao  for the final question here.

Hello @shahriar49, if I understand correctly here we're using model_to_estimator without a tf.distribute strategy. With classic estimator training, I am not aware of any data splitting among the workers. The same copy of code is run on chief and workers, and if the training input_fn is configured to raise OutOfRangeError after one epoch, the total epochs trained will be the number of all workers.
You can find more information here: https://www.tensorflow.org/api_docs/python/tf/estimator/train_and_evaluate

@rchao
So you mean that without strategy clause each worker will do its job independently without data splitting? My observation says something else.

Mind sharing what you observed?


[Bug]: The second deletion of the custom environment is inconsistent with the design draft

ODC version
odc424
OB version

What happened?

What did you expect to happen?

How can we reproduce it (as minimally and precisely as possible)?
Delete custom environment
Anything else we need to know?
No response
Cloud
No response

Fixed In Commit e4fbbdbad75ee9c0b4f7111c64ce4d7dc1e93cb6
------
陈朝俊
Tue Mar 12 18:32:13 2024 +0800

pass


Lua question

What is your question?
Is it possible to force Psych Engine to use a different Lua runtime?
I have the .dylib file Which is basically the macOS equivalent of a .dll file.

Lua support for Mac was added yesterday on Linc LuaJIT's repo. Update your stuff with haxelib update linc_luajit

how do i fix the update linc_luajit


create "all tracks" report, sorted by title

key report if i can't get fastests/slowest to be dynamic. this will also be good for testing client interactive sort options

Thank you for contributing!Preview these changesLink expires: Sun Oct  7 07:29:28 UTC 2018Note: it may take a moment for the branch to deploy. Check all pending travis builds have completed


new-command: spo report activityusercounts

implements #1244

Coverage remained the same at 100.0% when pulling e897350abefe49833086be500dbdd4d31fd607f7 on plamber:report-sharepointactivityusercounts into f62e1ef8c7df7d30ab2e5898f036c4e999016f2b on pnp:dev.


Remove moot version property from bower.json

Per bower/bower.json-spec@a325da3
Also their maintainer says they probably won't ever use it: http://stackoverflow.com/questions/24844901/bowers-bower-json-file-version-property

Thanks! They should update http://bower.io/docs/creating-packages/, which still shows:
{
  "name": "my-project",
  "version": "1.0.0",
  "main": "path/to/main.css",
  "ignore": [
    ".jshintrc",
    "**/*.txt"
  ],
  "dependencies": {
    "<name>": "<version>",
    "<name>": "<folder>",
    "<name>": "<package>"
  },
  "devDependencies": {
    "<test-framework-name>": "<version>"
  }
}

I'm going to see if that's a github page and change that

bower/bower.github.io#143


player-cmus - position some times breaks the script

#99
Some times
cmus-remote -Q 2 "$info" | grep position | awk -F\  '{print $2}'

returns something similar to
23
comment

(23 being an example duration, followed by the word comment), which breaks the script outputting the name of the file. For what I understand, it happens when the archive has a comment beginning with \n, but without a tag comment.
When there is a tag comment, the script does not break and outputs a partial result (only artist, no title).
Some other times (with the tag comment or without it), the output is the expected.

Could you paste two complete outputs with that difference?
We could look what we need to change in that script.

Three outputs, one for each case :)
Working output (complete):
$ cmus-remote -Q 2 "$info" | grep position | awk -F\  '{print $2}'
28

$ cmus-remote -Q 2 "$info"
...
duration 240
position 38
tag artist Brian Hogan
tag album 26 Days
tag title Put It In Drive
tag date 2011
tag tracknumber 3
tag comment Recorded in February 2011 for the RPM Challenge. All songs copyright 2011 Brian P. Hogan.
set aaa_mode all
set continue trueset play_library true
set play_sorted false
set replaygain disabled
set replaygain_limit true
set replaygain_preamp 0.000000
set repeat false
set repeat_current false
set shuffle false
set softvol false
set vol_left 20
set vol_right 20

Partially working output (shows artist):
$ cmus-remote -Q 2 "$info" | grep position | awk -F\  '{print $2}'6

$  cmus-remote -Q 2 "$info"
status paused
duration 159
position 6
tag artist bella coola
tag album Light Years To Be Lonely
tag title Light Years to Be Lonely
tag date 2007
tag genre FOLK: Folk Blues
tag tracknumber 1
tag albumartist Bella Coola
tag composer Angelika Antalova / Angelica Antaltag comment Bella Coola is a Canadian singer/songwriter and guitarist currently based in Central Europe. She enjoys writing and collaborating with international artists and is currently involved in many diverse musical projects. Smokey, understated vocals and percussive guitar playing inform most of her songs on this debut EP.  All of her songs from Light Years To Be Lonely were conceived in the previous century when she was still residing in British Columbia.   All songs are available as digital downloads.
tag replaygain_track_gain -9.2 dB
set aaa_mode all
set continue false
set play_library true
set play_sorted false
set replaygain disabled
set replaygain_limit true
set replaygain_preamp 0.000000
set repeat false
set repeat_current false
set shuffle false
set softvol false
set vol_left 20
set vol_right 20

Broken output (shows file path):
$ cmus-remote -Q 2 "$info" | grep position | awk -F\  '{print $2}'
7
comment

$ cmus-remote -Q 2 "$info"
status paused
...
duration 262
position 7
tag artist Amanda Abizaid
tag album The Great Plan Vol. l
tag title Take It On Faith
tag date 2004
tag genre ROCK: Modern Rock
tag tracknumber 1
tag albumartist Amanda Abizaid
tag composer Amanda Abizaid, Shane Soloski
\nBorn in Beirut, Lebanon, AMANDA JO ABIZAID moved to The United States when she was nine years old.  She learned how to harmonize as a child by singing a wide range of music with her four older sisters. For many years, Amanda studied piano and flute with private teachers.  She continued to perfect her talents at The Catholic University of America, majoring in drama and music. She has also lived and modeled in Wash. D.C., New \nAmanda co-started and fronted the band "BLUE" that performed regularly in and around the Los Angeles area. Now as a solo artist, Amanda Abizaid performs her unique songs on piano and guitar and also with her band. Her original musical compositions include flute and violin parts that add an Arabian flavor to her poetic and classic songwriting: a mixture of sultry/soulful vocals and haunting melodies.  Her debut EP Album, "The Great Plan", Vol. I, was independently released last Summer 2004 and is sold internationally at www.CDBaby.com along with her first single "Lebanon", a song about the poli\nShe was the singing voice for Ally Sheady and Carisma Carpenter on VH1's show "STRANGE FREQUENCY", and has sung songs for WB's "CHARMED", "SUMMERLAND", "SMALLVILLE", Paramount's "THE 4400" (theme song, "A Place In Time"), and many more. Her feature film vocal and songwriting credits include,  "3 BELOW" and "FACING THE LION".  She performs regularly at Genghis Cohen, The Derby and Tangier Lounge in Los Angeles. She has just finished the follow up to "The Great Plan", Vol. II, which is set to be released this F\n3 Below- DITS                 3 Below, Fairlon Suite/writer & vocalist             S\nFacing The Lion-              Facing The Lion, Timeless/co-writer & vocalist       S\nThe Groomsmen-        featured/vocalist                                       Vision\nDevious Beings-       God Rest ye Merry Gentlemen, Come Alive/vocalist        Redbea\nOdd Girl Out-         Dreams In Ashes/vocalist                                Lifeti\nThe 4400      theme song- A Place In Time/vocalist                    Viacom/Paramou\nCharmed-s                     opening song, Not Gonna Drag Me Down/vocalist        W\nBig Man On Campus-    How Do I Make/vocalist                          Next Entertain\nOne Tree Hill-        At The E2 episodes/vocalist                             WB   W\nThe Division-         1 song/vocalist                                 Lifetime Netwo\nThat's Life-y CARISMA 3 episodes/vocalistlistvocalist,Viacom/VH1/Paramount EADViacom\nJC Penny-LS ctures    1 song/v1 song/vocalist                         WB      Boss H\nFelicity-                     1 song/vocalist                                 WB/Wad\nStephen Phillips & Tim P.-                                                    Boss H\nTom Gimbel-                                                           Aerosmith, For\nJennifer Stills-                                                              Three \nTaylor Mills- -                                       The Wallflowers,WilliamsBrian \nScott Bennett -                                                               Brian \nPepper Mashay-                                                                Jungle\nBird York -                                                                   Guided\nRoyal Caribbean Cruise Lines- Christmas CD                                    Wade H\nCarnival Cruise Lines- CD                                                     Wade H\nTaylor Daine-                                                         vocal demo
set aaa_mode all
set continue false
set play_library true
set play_sorted false
set replaygain disabled
set replaygain_limit true
set replaygain_preamp 0.000000
set repeat false
set repeat_current false
set shuffle false
set softvol false
set vol_left 20
set vol_right 20

The problem is that we are looking for the position with grep position. If you have now a string like that somewhere in your stream info you run into problems.
Look at this:
Her original musical compositions include ...
You could try to use grep "position ". But this is more a workaround than a solution.

I rewrote the conditions. Can you test it?

Thank you very much.
I quickly tested it and so far it works as intended. I will use it more in the next days and I will give you a more accurate feedback if I will find something.

Sounds good! :sunglasses:
I will close this issue. Open another one if you find something bad.


Automate generating a dist folder for Heroku

Write a npm run build command in the package.json that generates a dist folder containing the server src files and web files and have the server return the client files when hitting the server url. We also want to investigate how to push dist folders to a separate branch to be deployed from Heroku.

Additional bug fix: 278b9605e444a2a72db8d95788cb75b3a2a204a7

Additional bug fix for some Angular dependencies not being installed: move some angular cli + p5 packages from devDependencies to dependencies since devDependencies only installs when ran locally


Unable to access JSON value's attribute using Filters eq method.

I am writing an API services which accepts the below JSON contents as input and I am successfully able to save the contents for the first time.
input :
{
"primarykey": "payloadId"
,"payload":{
"payloadId": "payload12",
"payloadName": "payload name",
"payloadStatus": "true"
}
}
I am using this java code to verify the payloadId is already exist on not in the DB, but i am getting Document doc as null. please provide some assistance.
String primarykey = (String) feature.get("elementPrimayKey");
JSONObject payload = feature.getJSONObject("payload");
NitriteCollection nitritecollection = db.getCollection(collection);
Cursor cursor = nitritecollection.find(Filters.eq(primarykey, (String) payload.get(primarykey)));
Document doc = cursor.firstOrDefault();

The problem seems to be on this line
Cursor cursor = nitritecollection.find(Filters.eq(primarykey, (String) payload.get(primarykey)));

According to your sample json of payload
"payload":{
"payloadId": "payload12",
"payloadName": "payload name",
"payloadStatus": "true"
}

it does not contains any field called primarykey. And I am not sure what feature  is and what it returns by feature.get("elementPrimayKey");.
It is more likely a problem with your own application logic rather than of nitrite.
Why don't you use ObjectRepository instead of dealing manually with JSONObject which is very error prone?


chore(types): disable skipLibCheck and fix conflicting vitest-jest globals

:tada: This PR is included in version 1.0.70 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Removed duplicate branch on ci

What do these changes do?
Decreases number of workers are used in our pre-commit autoupdate ci

Are there changes in behavior for the user?
no

Related issue number
none

Checklist

[X] I think the code is well written
[ ] Unit tests for the changes exist
[ ] Documentation reflects the changes
[ ] If you provide code modification, please add yourself to CONTRIBUTORS.txt

The format is <Name> <Surname>.
Please keep alphabetical order, the file is sorted by names.

[ ] Add a new news fragment into the CHANGES/ folder

name it <issue_id>.<type> (e.g. 588.bugfix)
if you don't have an issue_id change it to the pr id after creating the PR
ensure type is one of the following:

.feature: Signifying a new feature.
.bugfix: Signifying a bug fix.
.doc: Signifying a documentation improvement.
.removal: Signifying a deprecation or removal of public API.
.misc: A ticket has been closed, but it is not of interest to users.

Make sure to use full sentences with correct case and punctuation, for example:
Fix issue with non-ascii contents in doctest text files.

Hmm the commit still doesn't run the ci...

@Andrew-Chen-Wang Sorry, I’m blanking on this for some reason. Can you help me understand what this PR was trying to fix?
Meanwhile, I’m going to release a beta without this PR merged, because it’s CI-related anyway!

@abrookins np, go for it!
Basically, every time pre-commit runs, the action doesn't run with it. I manually go into terminal to check that pre-commits running fine. If it isn't (say due to pyupgrade), then I commit the changes. By committing the changes, there are two workers per matrix env instead of the usual one.


Question about the Text Input to the LLM

Thanks for your amazing work!
I have a question about the textual input to the LLM. The input_ids from the dataset contains the long message about when the frames are sampled at. An example is:
The video contains 96 frames sampled at 0.0, 0.2, 0.5, 0.7, 0.8, 1.0, 1.3, 1.5, 1.7, 1.8, 2.0, 2.3, 2.7, 2.8, 3.2, 3.7, 4.0, 4.2, 4.5, 4.7, 4.8, 5.0, 5.3, 5.7, 5.8, 6.0, 6.2, 6.5, 6.7, 6.8, 7.0, 7.3, 7.5, 7.7, 8.0, 8.2, 8.8, 9.0, 9.2, 9.3, 9.5, 9.8, 10.0, 10.2, 10.5, 11.0, 11.3, 11.5, 12.0, 12.2, 12.3, 12.5, 12.7, 12.8, 13.0, 13.2, 13.5, 13.7, 13.8, 14.2, 14.3, 14.5, 14.8, 15.0, 15.2, 15.3, 15.5, 15.7, 15.8, 16.0, 16.2, 16.3, 16.7, 16.8, 17.0, 17.2, 17.3, 17.5, 17.7, 18.0, 18.2, 18.5, 18.7, 18.8, 19.0, 19.2, 19.3, 19.5, 19.8, 20.2, 20.3, 20.7, 20.8, 21.2, 21.5, 21.7 seconds.
However, the frame embedding the replaced by the Video QFormer features (Lines 417-440 in timechat.py) which do not represent individual frames anymore, since the Video QFormer incorporates temporal correspondence across frame embeddings. If so, isn't the above text input is confusing for the LLM, as the LLM might represent the visual features as standalone frame embeddding?
Moreover, since the frames are already timestamped by the Image QFormer, what is the requirement of this long text denoting which frames comes from which second at all?

Thanks Shuhuai, I appreciate your opinion.


Fix small spelling error in Norwegian translation

Signed-off-by: Matti<EMAIL_ADDRESS>

please use the Weblate for changing translations. In this case, I think you want to modify https://hosted.weblate.org/translate/prismlauncher/launcher/nb_NO/?offset=1&q=sansynelig&sort_by=-priority%2Cposition&checksum=


Added support for compiling under Windows/Visual Studio 2017.

Added support for compiling under Windows/Visual Studio 2017.
How to compile

Prerequisite: Visual Studio 2017 and its C/C++ compiler are installed

Either start "x64_x86 Cross Tools Command Prompt for VS 2017" or "x86 Native Tools Command Prompt for VS 2017"

'cd' into selfie folder

Type

> nmake -f Makefile.win TARGETNAME
to compile TARGETNAME. For compiling selfie.exe type
> nmake -f Makefile.win selfie.exe
(only the Makefile target "selfie" was renamed to "selfie.exe"; all other targets kept their existing name).
For an explanation of Makefile.win:
"diff" was replaced by "fc" (cf. https://stackoverflow.com/q/6877238/497193). Unluckily fc has no "-q" option. Powershell seems to support 'diff', but "x64_x86 Cross Tools Command Prompt for VS 2017" and "x86 Native Tools Command Prompt for VS 2017" currently have no Powershell counterparts. So I considered using "fc" the better solution.
I get no trouble compilation problems when I removed the 'main(a,b)=main(a,char**argv)' definition - so I did remove it.
On the other hand I had to define "atoi" as something different (I chose "xatoi"), since otherwise one gets a name collission with the existing "atoi" function of the C Runtime
What works and where are still bugs?
> nmake -f Makefile.win sat
seems to work.
For the tests in target "test":
> selfie.exe -c selfie.c -o selfie1.m -s selfie1.s -m 2 -c selfie.c -o selfie2.m -s selfie2.s
and
> selfie.exe -c selfie.c -o selfie.m -m 2 -l selfie.m -m 1
seem to work. On the other hand
> selfie.exe -c selfie.c -o selfie3.m -s selfie3.s -y 3 -l selfie3.m -y 2 -l selfie3.m -y 2 -c selfie.c -o selfie4.m -s selfie4.s
outputs:
selfie.exe: this is selfie compiling selfie.c with starc
selfie.exe: 191548 characters read in 7576 lines and 1032 comments
selfie.exe: with 105824(55.24%) characters in 31199 actual symbols
selfie.exe: 270 global variables, 311 procedures, 455 string literals
selfie.exe: 2094 calls, 809 assignments, 71 while, 627 if, 287 return
selfie.exe: 129704 bytes generated with 30755 instructions and 6684 bytes of data
selfie.exe: 129704 bytes with 30755 instructions and 6684 bytes of data written into selfie3.m
selfie.exe: 1293223 characters of assembly with 30755 instructions written into selfie3.s
selfie.exe: this is selfie executing selfie3.m with 3MB of physical memory on hypster
selfie3.m: 129704 bytes with 30755 instructions and 6684 bytes of data loaded from selfie3.m
selfie3.m: this is selfie executing selfie3.m with 2MB of physical memory on hypster
selfie.exe: context NULL throws uncaught invalid address
selfie.exe: this is selfie terminating NULL with exit code -8 and 0.52MB of mapped memory
selfie.exe: profile: total,max(ratio%)@addr(line#),2max(ratio%)@addr(line#),3max(ratio%)@addr(line#)
selfie.exe: calls: 427289,64954(15.22%)@0x140B8(~5375),64953(15.22%)@0x22CC(~1134),64920(15.19%)@0x13EE8(~5354)
selfie.exe: loops: 32836,32425(99.00%)@0x1A6D4(~6704),176(0.53%)@0x4F34(~1850),71(0.21%)@0x3B10(~1535)
selfie.exe: loads: 2444977,64954(2.65%)@0x140CC(~5375),64953(2.65%)@0x22E0(~1134),64920(2.65%)@0x13EFC(~5354)
selfie.exe: stores: 1776820,64954(3.65%)@0x140BC(~5375),64953(3.65%)@0x22D0(~1134),64920(3.65%)@0x13EEC(~5354)

I take this as a failure. :-(
> selfie.exe -c selfie.c -o selfie5.m -s selfie5.s -min 4 -l selfie5.m -y 2 -l selfie5.m -y 2 -c selfie.c -o selfie6.m -s selfie6.s
again seems to work. Finally for
> selfie.exe -c -mob 1
I get the output
selfie.exe: nothing to compile, only library generated
selfie.exe: error in library in line 0: procedure main undefined
selfie.exe: 280 bytes generated with 69 instructions and 4 bytes of data
selfie.exe: this is selfie executing library with 1MB of physical memory on mobster
selfie.exe: library exiting with exit code 0 and 0.00MB of mallocated memory
selfie.exe: this is selfie terminating library with exit code 0 and 0.01MB of mapped memory
selfie.exe: profile: total,max(ratio%)@addr(line#),2max(ratio%)@addr(line#),3max(ratio%)@addr(line#)
selfie.exe: calls: 1,1(100.00%)@0x48(~1),0(0.00%),0(0.00%)
selfie.exe: loads: 2,1(50.00%)@0x24(~1),0(0.00%),0(0.00%)
selfie.exe: stores: 1,1(100.00%)@0x4C(~1),0(0.00%),0(0.00%)

I don't know whether I should take this as success or failure, since on one hand there is an error message ("selfie.exe: error in library in line 0: procedure main undefined"), but on the other hand this error message also when I compiled and tested selfie under GNU/Linux (Ubuntu).

As i already mentioned in #29 , the first error is totally ok. I'm working on a fix for this bug.
The second "error" is nothing to bother about. Maybe the error message is a bit misleading. During library generation, it's actually a good sign if main is missing 😄

In https://github.com/nubok/selfie/commit/51236c39912616c32652d112c37c90cc5db48932 I added another small change for Makefile.win, which makes it a little bit more clear what the output file is.

@nubok Thanks for your work, but in the meantime we figured out, that we probably don't want to be compatible with Win32. There are totally different syscalls in Win32, compared to the POSIX standard we rely on. (especially when it comes to more advanced syscalls)


Feat: vela cluster export config

Signed-off-by: Somefive<EMAIL_ADDRESS>Description of your changes

Support export multi-cluster kubeconfig.
Export multi-cluster kubeconfig

 Load existing cluster kubeconfig and list clusters registered in KubeVela. Export the proxy access of these clusters to
KubeConfig and print it out.

Usage:
  vela cluster export-config [flags]

Examples:
  # Export all clusters to kubeconfig
  vela cluster export-config

  # Export clusters with specified kubeconfig
  KUBECONFIG=./my-hub-cluster.kubeconfig vela cluster export-config

  # Export clusters with specified labels
  vela cluster export-config -l gpu-cluster=true

  # Export clusters to kubeconfig and save in file
  vela cluster export-config > my-vela.kubeconfig

  # Use the exported kubeconfig in kubectl
  KUBECONFIG=my-vela.kubeconfig kubectl get namespaces --cluster c2

Flags:
  -h, --help              help for export-config
  -l, --selector string   LabelSelector for select clusters to export.

Global Flags:
  -y, --yes   Assume yes for all user prompts

I have:

[x] Read and followed KubeVela's contribution process.
[x] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[x] Run make reviewable to ensure this PR is ready for review.
[x] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Special notes for your reviewer

Right. ClusterPedia use this implementation and I think it is also helpful for KubeVela users.
P.S. Not only for debugging but also for resource inspecting.


Remove obsolete scripts for contrastive phenotyping

Let's merge these changes into representation while we release VisCy 0.2.1 with the examples we discussed this morning.  We will merge representation into main.


Charshes when someone talks

when any play says something the bot crashes and i get this error
Hint: hit control+c anytime to enter REPL.
Listening on port 8000
[2023-03-13T04:05:40.662] [INFO] default - Bot joined to the server
/home/runner/Afk-Bot/node_modules/prismarine-chat/index.js:390
      const fstr = format.formatString
                          ^

TypeError: Cannot read properties of undefined (reading 'formatString')
    at ChatMessage.fromNetwork (/home/runner/Afk-Bot/node_modules/prismarine-chat/index.js:390:27)
    at Client.<anonymous> (/home/runner/Afk-Bot/node_modules/mineflayer/lib/plugins/chat.js:121:25)
    at Client.emit (node:events:513:28)
    at Client.emit (node:domain:489:12)
    at Client.<anonymous> (/home/runner/Afk-Bot/node_modules/minecraft-protocol/src/client/chat.js:197:14)
    at Client.emit (node:events:513:28)
    at Client.emit (node:domain:489:12)
    at FullPacketParser.<anonymous> (/home/runner/Afk-Bot/node_modules/minecraft-protocol/src/client.js:91:12)
    at FullPacketParser.emit (node:events:513:28)

Node.js v18.12.1
repl process died unexpectedly: exit status 1

(To exit

https://github.com/urFate/Afk-Bot/issues/67


Bug fix for gen-certs

Fixing bug so that bash doesn't have to be run in admin mode/with root access.
http://www.herongyang.com/crypto/OpenSSL_Signing_keytool_CSR_5.html
Basically allowing OpenSSL to manage it to create and manage the serial number, instead of having to read the serial number from the .srl file.

I was actually working on a workaround that copied some files from the home folder into a temporary folder. I've not tested it extensively but it seemed to work, I can submit a PR for the team to have a look, in case that would be preferrable?

Does that still require admin/root access to work? I'm interested in removing that requirement if possible as it does catch some people out.

Not sure if you were asking me or @dsingh07 😄, but in any case, these were my changes: https://github.com/oscarrenalias/adop-docker-compose/commit/13979fd30aba606b9130b1ba61cf89d799090f1b.
I've not submitted a PR yet because I  haven't had time to thoroughly test them, but I'm happy to do so if you think that this is a better approach.

@oscarrenalias That was a question for you, both changes look basically the same to me unless they are addressing separate issues. This PR doesn't need anything copied out of the home directory which seems simpler.
@dsingh07 Can you take a look at this please?

I'm not familiar with OpenSSL but it would appear that indeed this PR doesn't need to copy any files, so I agree it would be a preferable solution.

I can also confirm that the command works fine for me in OS X 10.11 with OpenSSL 0.9.8zh 14 Jan 2016:
sh -x cmd/compose gen-certs /root/.docker
+ SUB_CMD_NAME=compose
+ DEFAULT_MACHINE_NAME=default
+ export MACHINE_NAME=default
+ MACHINE_NAME=default
+ export VOLUME_DRIVER=local
+ VOLUME_DRIVER=local
+ export LOGGING_DRIVER=syslog
+ LOGGING_DRIVER=syslog
+ export CUSTOM_NETWORK_NAME=local_network
+ CUSTOM_NETWORK_NAME=local_network
+ export OVERRIDES=
+ OVERRIDES=
+ export TOTAL_OVERRIDES=
+ TOTAL_OVERRIDES=
+ export PULL=YES
+ PULL=YES
+ getopts m:f:F:v:l:n:i: opt
+ shift 0
+ SUBCOMMAND_OPT=gen-certs
+ '[' 2 -ge 1 ']'
+ shift
+ ADOPFILEOPTS='-f /docker-compose.yml -f /etc/volumes/local/default.yml -f /etc/logging/syslog/default.yml'
+ ELKFILEOPTS='-f /compose/elk.yml'
+ case ${SUBCOMMAND_OPT} in
+ gen_certs /root/.docker
+ echo 'Generating client certificates for TLS-enabled Engine'
Generating client certificates for TLS-enabled Engine
+ CERT_PATH=/root/.docker
+ '[' -z /root/.docker ']'
++ uname
+ HOST_OS=Darwin
+ CLIENT_SUBJ=/CN=client
+ echo Darwin
+ grep -E 'MINGW*'
+ TEMP_CERT_PATH=/Users/oscar.renalias/docker_certs
+ rm -rf /Users/oscar.renalias/docker_certs
+ mkdir -p /Users/oscar.renalias/docker_certs
+ set +e
+ openssl genrsa -out /Users/oscar.renalias/docker_certs/key.pem 4096
+ openssl req -subj /CN=client -new -key /Users/oscar.renalias/docker_certs/key.pem -out /Users/oscar.renalias/docker_certs/client.csr
+ echo 'extendedKeyUsage = clientAuth'
+ openssl x509 -req -days 365 -sha256 -in /Users/oscar.renalias/docker_certs/client.csr -CA /Users/oscar.renalias/.docker/machine/certs/ca.pem -CAkey /Users/oscar.renalias/.docker/machine/certs/ca-key.pem -CAcreateserial -CAserial temp.seq -out /Users/oscar.renalias/docker_certs/cert.pem -extfile /Users/oscar.renalias/docker_certs/extfile.cnf
+ set -e
+ cp /Users/oscar.renalias/.docker/machine/certs/ca.pem /Users/oscar.renalias/docker_certs/ca.pem
+ docker --tlsverify --tlscacert=/Users/oscar.renalias/.docker/machine/certs/ca.pem --tlscert=/Users/oscar.renalias/docker_certs/cert.pem --tlskey=/Users/oscar.renalias/docker_certs/key.pem -H= version

I haven't run through the full process yet but it would appear to work.

One last thing. When creating temporary folders as in TEMP_CERT_PATH, we may want to use something like mktmp to create temporary folders in the correct place for doing so (/tmp or /var/tmp, let the shell decide) instead of creating them in places like $HOME/docker_certs, as users may be led to believe that this is an official folder.
Alternatively, we could create the folder relative to the script's current path, so that everything remains within the adop directory.

I agree that we should either be using mktmp or a relative directory instead of cluttering the home directory - I've spun this off into #77.


Problems building from NatNetSDK

My lab is working on transitioning to Ros2.
We are using Ubuntu 22.04 on an OrangePi 5 (arm64) with Ros2 Humble. The build error is from the NatNetSDK portion due to libNatNet.so being in the wrong file format. The following is the output:
Starting >>> mocap_optitrack_driver
Starting >>> rqt_mocap_control
--- stderr: mocap_optitrack_driver
/usr/bin/ld: /home/darc/darc_ws/src/mocap4ros2_optitrack/mocap_optitrack_driver/NatNetSDK/lib/libNatNet.so: error adding symbols: file in wrong format
collect2: error: ld returned 1 exit status
gmake[2]: *** [CMakeFiles/mocap_optitrack_driver_main.dir/build.make:265: mocap_optitrack_driver_main] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:165: CMakeFiles/mocap_optitrack_driver_main.dir/all] Error 2
gmake: *** [Makefile:146: all] Error 2
Failed   <<< mocap_optitrack_driver [19.7s, exited with code 2]
Aborted  <<< rqt_mocap_control [22.2s]
Summary: 8 packages finished [1min 23s]
1 package failed: mocap_optitrack_driver
1 package aborted: rqt_mocap_control
2 packages had stderr output: mocap_optitrack_driver rqt_mocap_control
My question is what system are you using to run the NatNetSDK? The documentation says it should work on Ubuntu but has no other specifics, and all their examples are in Windows. Any help around this would be great. Thanks

Hi @flapjack48 ,
We employ the NatNet SDK on AMD architecture, conducting thorough tests on both Intel i7 and i9 processors. For ARM compatibility, it is advisable to utilize the corresponding libraries. However, since this software originates from Optitrack, we regret to inform you that we are unable to provide assistance in this matter.
Upon researching, I came across a forum post indicating that ARM architecture may not be supported. I recommend reaching out to Optitrack directly and inquiring about any potential solutions for integrating ARM support into our package (https://forums.naturalpoint.com/viewtopic.php?t=20512).
Should you encounter any other issues or require assistance with different concerns, please feel free to contact us again. We are here to help.
Regards.

Hi @jmguerreroh,
Thanks for the quick reply. For anyone who comes up with the same problem, I installed the SDK on an AMD64 computer, and that worked great!


[github] Fix and move prover-inconsistency-test

Fixed broken prover-inconsistency-test CI job and moved it from daily to ci-pre-land as prover-tests, since there seems to be no reason to have it in the daily run instead of the PR related run.
Also cleaned it up, removing Slack references and other dead code.
Motivation
The daily CI tasks fails. This solves half of the problem (by omission), while #301 solves the other half.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
In addition to this PR's CI tests, the new CI job also succeeds in my repository: https://github.com/villesundell/move/runs/7498332280?check_suite_focus=true

In addition to @awelc & @tnowacki, this might be of interest to @bmwill (CI) and @emmazzz (Prover).

The historic reason why we had it in daily instead of CI were flakes in the DPN verification. But perhaps we can make it the default now? @junkil-park WDYT?

The historic reason why we had it in daily instead of CI were flakes in the DPN verification.

Another reason was that the inconsistency check is 10x ~ 20x slower than the normal run of Prover depending on the timeout limit with the -T option (see https://github.com/diem/diem/pull/7984). The inconsistency check runs Prover after injecting false specs on purpose. Prover often does timeout when the verification of Diem Framework fails. It's not ideal, but we treat timeouts as expected results (= negative verification results) in the inconsistency check.

CI changes LGTM, but i'll leave it to @junkil-park and @wrwg to decide if they want to move this to land blocking

Given the experience with timeouts and flakiness, perhaps we can update this PR so it runs nightly (but this time for real as this seemed to have been broken)?

So, would the next course of action be, that I will move the test back to daily and fix it properly? ( @wrwg & @junkil-park )

So, would the next course of action be, that I will move the test back to daily and fix it properly? ( @wrwg & @junkil-park )

Yes please!

So, would the next course of action be, that I will move the test back to daily and fix it properly? ( @wrwg & @junkil-park )
@villesundell , can the inconsistency test come back to daily?


Update scalatest to 3.2.16

About this PR
📦 Updates org.scalatest:scalatest from 3.2.11 to 3.2.16
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (3.2.11).
You might want to review and update them manually.
project/Version.scala

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalatest", artifactId = "scalatest" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalatest", artifactId = "scalatest" }
}]

labels: test-library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #858.


Extend the AES block modes with OFB (Output Feedback)

Description
This PR extends the AES module with the OFB (Output Feedback) block mode, including test cases taken from NIST SP800-38A. It also extends the cipher abstraction module with OFB mode and associated test cases.
Status
READY
Requires Backporting
NO
Migrations
YES
This PR extends the AES module API and cipher abstraction to include the OFB block mode.
Todos

[x] Tests
[x] Documentation
[x] Changelog updated

Rebased against development to remove conflicts.

Now passing CI. @RonEld can you please re-review? And either @mpg or @yanesca - can you please review? Thanks.

@sbutcher-arm I'll review this one, so that @yanesca can focus on reviewing https://github.com/ARMmbed/mbedtls/pull/1410 :)

Thanks @mpg ! I am removing this PR from my queue.

Updated following review comments, and rebased.

All review feedback has now been responded to, if not acted on, and the PR now needs re-review.

New feedback addressed! Concerning the documentation, I honestly think it's best done all at once, and not just for OFB, so I'd suggest raising a GitHub issue for the /general/ concerns about the documentation.

Rebased against development to remove conflicts.

Created https://github.com/ARMmbed/mbedtls/issues/1682 to track doc enhancements.

retest

The CI was broken by the rebasing because the introduction of ARIA conflicted with OFB, as ARIA hadn't implement OFB. This has now been fixed in the cipher module. Unfortunately the PR now needs re-reviewing.

I had a look at the benchmark application, and realised it doesn't benchmark everything, and concluded that adding it wasn't that useful as the performance won't be that different from other block modes.
@Patater - I think this is your choice. Do you want OFB to be in the benchmark application? If so, I think it needs to be raised as an issue so we can merge what we have.

I've raised the benchmark issue as #1720.


UI issue.

Text is cutting from left side.
Device: iphone 7

Check the distance between device name and description.

@cakesoft-swati Not able to reproduce second issue on iphone 7.

Verified this issue on keeper app v 1.3.1(434)


Transaction on adding lock + updating balance

@abarmat I tried a few times with mocked values and worked fine!

@NicoSantangelo I'm good with the change too.

@NicoSantangelo done, will merge.


build: delete umd.js from the final package

This file is not necessary and breaks closure compiler.

Coverage remained the same at 97.544% when pulling 4f7f59b8338392827ea0fe1d364244f7a3351a70 on IgorMinar:build/remove-umd.js into 80d05afead0aaea75b8a18b42b13c1c725697674 on ReactiveX:master.


2023.1/2023.2: add backport-916767.patch

https://review.opendev.org/c/openstack/kolla/+/916767
Nova: fix swtpm and swtpm-tools missing from deb installs
Closes osism/issues#1008

Not longer required. Upstream back ports merged.


no working

I upload it to wemos 8266 and echo dows no activate relay

Updated the missing relay code. Thanks for creating an issue

Thanks now it works
Question  I need to learn  to code arduino any good place to start.?
Sent from my Verizon Wireless 4G LTE smartphone
-------- Original message --------
From: Aruna Tennakoon<EMAIL_ADDRESS>Date: 8/15/2016 12:47 PM (GMT-05:00)
To: kakopappa/arduino-esp8266-alexa-wemo-switch<EMAIL_ADDRESS>Cc: ekerp<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [kakopappa/arduino-esp8266-alexa-wemo-switch] no working (#1)
Updated the missing relay code. Thanks for creating an issue

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/kakopappa/arduino-esp8266-alexa-wemo-switch/issues/1#issuecomment-239856729, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AT3FaJu_hHuPC_BzSG7z1OXd6_W0-cbEks5qgJgkgaJpZM4JcIK-.

This is my first real Arduino project as well. Prior to this I had very
little experience in Arduino. If you want to get into Arduino programming I
recommend you to start with basics like turning on of LEDs and then move to
more advanced stuff. I used YouTube alot to search for this and that. You
can find almost anything there
Louis Rossman has good tutorials on YouTube  to explain components like
Transistors, capacitors in plain English
https://youtu.be/4prl_6eSJLY
Good luck, let me know if you need any help
On Tuesday, August 16, 2016, ekerp<EMAIL_ADDRESS>wrote:

Thanks now it works
Question I need to learn to code arduino any good place to start.?
Sent from my Verizon Wireless 4G LTE smartphone
-------- Original message --------
From: Aruna Tennakoon<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
Date: 8/15/2016 12:47 PM (GMT-05:00)
To: kakopappa/arduino-esp8266-alexa-wemo-switch <
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','arduino-esp8266-alexa-wemo-switch@noreply.github.com');>

Cc: ekerp<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','ekerp@hotmail.com');>>, Author <
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','author@noreply.github.com');>>
Subject: Re: [kakopappa/arduino-esp8266-alexa-wemo-switch] no working (#1)
Updated the missing relay code. Thanks for creating an issue

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/
kakopappa/arduino-esp8266-alexa-wemo-switch/issues/1#
issuecomment-239856729>, or mute the thread<https://github.com/
notifications/unsubscribe-auth/AT3FaJu_hHuPC_BzSG7z1OXd6_W0-
cbEks5qgJgkgaJpZM4JcIK->.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/kakopappa/arduino-esp8266-alexa-wemo-switch/issues/1#issuecomment-239888165,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHIM5iq0BPEgBywLgfiPKK8TSNOyK9Omks5qgLGcgaJpZM4JcIK-
.

thanks for your big help ....another thing how can i add a power bottom to this project so i can turn it from local switch too.
[Image by FlamingText.com]http://www.flamingtext.com/hmail.html

From: Aruna Tennakoon<EMAIL_ADDRESS>Sent: Monday, August 15, 2016 9:43 PM
To: kakopappa/arduino-esp8266-alexa-wemo-switch
Cc: ekerp; Author
Subject: Re: [kakopappa/arduino-esp8266-alexa-wemo-switch] no working (#1)
This is my first real Arduino project as well. Prior to this I had very
little experience in Arduino. If you want to get into Arduino programming I
recommend you to start with basics like turning on of LEDs and then move to
more advanced stuff. I used YouTube alot to search for this and that. You
can find almost anything there
Louis Rossman has good tutorials on YouTube to explain components like
Transistors, capacitors in plain English
https://youtu.be/4prl_6eSJLY
Good luck, let me know if you need any help
On Tuesday, August 16, 2016, ekerp<EMAIL_ADDRESS>wrote:

Thanks now it works
Question I need to learn to code arduino any good place to start.?
Sent from my Verizon Wireless 4G LTE smartphone
-------- Original message --------
From: Aruna Tennakoon<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
Date: 8/15/2016 12:47 PM (GMT-05:00)
To: kakopappa/arduino-esp8266-alexa-wemo-switch <
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','arduino-esp8266-alexa-wemo-switch@noreply.github.com');>

Cc: ekerp<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','ekerp@hotmail.com');>>, Author <
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','author@noreply.github.com');>>
Subject: Re: [kakopappa/arduino-esp8266-alexa-wemo-switch] no working (#1)
Updated the missing relay code. Thanks for creating an issue

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/
kakopappa/arduino-esp8266-alexa-wemo-switch/issues/1#
issuecomment-239856729>, or mute the thread<https://github.com/
notifications/unsubscribe-auth/AT3FaJu_hHuPC_BzSG7z1OXd6_W0-
cbEks5qgJgkgaJpZM4JcIK->.

You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/kakopappa/arduino-esp8266-alexa-wemo-switch/issues/1#issuecomment-239888165,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHIM5iq0BPEgBywLgfiPKK8TSNOyK9Omks5qgLGcgaJpZM4JcIK-
.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/kakopappa/arduino-esp8266-alexa-wemo-switch/issues/1#issuecomment-239979591, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AT3FaIwFxzoPtlQNoqvZrrQVEDWB_fxbks5qgRXUgaJpZM4JcIK-.

Look for a push button example
https://www.arduino.cc/en/Tutorial/Button
On Tuesday, August 16, 2016, ekerp<EMAIL_ADDRESS>wrote:

thanks for your big help ....another thing how can i add a power bottom to
this project so i can turn it from local switch too.
[Image by FlamingText.com]http://www.flamingtext.com/hmail.html

From: Aruna Tennakoon<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
Sent: Monday, August 15, 2016 9:43 PM
To: kakopappa/arduino-esp8266-alexa-wemo-switch
Cc: ekerp; Author
Subject: Re: [kakopappa/arduino-esp8266-alexa-wemo-switch] no working (#1)
This is my first real Arduino project as well. Prior to this I had very
little experience in Arduino. If you want to get into Arduino programming I
recommend you to start with basics like turning on of LEDs and then move to
more advanced stuff. I used YouTube alot to search for this and that. You
can find almost anything there
Louis Rossman has good tutorials on YouTube to explain components like
Transistors, capacitors in plain English
https://youtu.be/4prl_6eSJLY
Good luck, let me know if you need any help
On Tuesday, August 16, 2016, ekerp<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Thanks now it works
Question I need to learn to code arduino any good place to start.?
Sent from my Verizon Wireless 4G LTE smartphone
-------- Original message --------
From: Aruna Tennakoon<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>');>>
Date: 8/15/2016 12:47 PM (GMT-05:00)
To: kakopappa/arduino-esp8266-alexa-wemo-switch <
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','arduino-esp8266-alexa-wemo-switch@noreply.github.com');>
<javascript:_e(%7B%7D,'cvml','arduino-esp8266-alexa-wemo-
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','arduino-esp8266-alexa-wemo-switch@noreply.github.com');>
');>

Cc: ekerp<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','ekerp@hotmail.com');>
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','ekerp@hotmail.com');>');>>, Author <
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','author@noreply.github.com');>
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','author@noreply.github.com');>');>>
Subject: Re: [kakopappa/arduino-esp8266-alexa-wemo-switch] no working
(#1)
Updated the missing relay code. Thanks for creating an issue

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/
kakopappa/arduino-esp8266-alexa-wemo-switch/issues/1#
issuecomment-239856729>, or mute the thread<https://github.com/
notifications/unsubscribe-auth/AT3FaJu_hHuPC_BzSG7z1OXd6_W0-
cbEks5qgJgkgaJpZM4JcIK->.

You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<https://github.com/kakopappa/arduino-esp8266-alexa-wemo-
switch/issues/1#issuecomment-239888165>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AHIM5iq0BPEgBywLgfiPKK8TSNOyK9Omks5qgLGcgaJpZM4JcIK->
.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/
kakopappa/arduino-esp8266-alexa-wemo-switch/issues/1#
issuecomment-239979591>, or mute the thread<https://github.com/
notifications/unsubscribe-auth/AT3FaIwFxzoPtlQNoqvZrrQVEDWB_
fxbks5qgRXUgaJpZM4JcIK->.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/kakopappa/arduino-esp8266-alexa-wemo-switch/issues/1#issuecomment-239981076,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHIM5iMWeOfnJKbw9Cl3mIy6mXKXDfUNks5qgRiRgaJpZM4JcIK-
.


feat: added bootstrap node detection in config

Description
Added a boolean value for detecting a bootstrap node in the config file.
Related issue(s)

Fixes #756

/claim #756

@melsonic
Thanks for your PR. We are going to detect the Bootstrap node without changing the config file.
We clearly mentioned that:
Another way is to read all bootstrap peers in the config, and
if the node's peer ID is the same as one of the bootstrap nodes, it can be marked as a bootstrap node.
In this case, we keep the config intact.


Derive Ord for GamepadButtonType.

Objective
Use GamepadButtonType with library that requires Ord.
Motivation
KeyCode derives Ord that I'm using with a trie for recognizing input sequences. I would like to do the same for GamepadButtonType but am stymied by it not deriving Ord.
Solution
This PR add derivations PartialOrd and Ord for GamepadButtonType.
Workaround
If deriving Ord is not possible, I'd be happy to know how I might coerce GamepadButtonType into a u32 or something else that is Ord, so I can wrap GamepadButtonType in a newtype. I suppose serializing with serde may work or reflect?

Reflect Workaround
I did find this cursed workaround.
#[derive(PartialEq, Eq, Clone, Debug)]
pub(crate) struct GamepadButton(GamepadButtonType);

impl From<GamepadButtonType> for GamepadButton {
    fn from(a: GamepadButtonType) -> Self {
        Self(a)
    }
}

impl PartialOrd for GamepadButton {
    fn partial_cmp(&self, other: &Self) -> Option<Ordering> {
        use bevy::reflect::Enum;
        self.0.variant_index().partial_cmp(&self.0.variant_index())
    }
}

impl Ord for GamepadButton {
    fn cmp(&self, other: &Self) -> Ordering {
        use bevy::reflect::Enum;
        self.0.variant_index().cmp(&self.0.variant_index())
    }
}


this adds option to configure S3 style access (path-style URLs or vir…

this adds option to configure S3 style access (path-style URLs or virtual hosted-style URLs) with the environment variable AWS_ENABLE_PATH_STYLE_ACCESS.
Note that when this variable isn't provided the default hard-coded value is used depending on the module. With this environment variable one is able to override the default value. https://github.com/instaclustr/esop/issues/53 provides some more context.
Besides this option the Ceph module is updated to use the builder in configuring/providing AmazonS3.

Hi,
I added some explanation of using this environment variable. I implemented in a way one can also use it for the other S3 modules. Except when running in Kubernetes. I'm not sure if this option should have effect in that case.
thnx for the feedback


Fixed minor misspell and gofmt simplify notices

👍

@rafaeljesus we already have it, https://github.com/hellofresh/janus/blob/master/build/test.sh#L52, and it did not help, but we also have https://goreportcard.com/report/github.com/hellofresh/janus

:shipit:


fix: prevent gatsby file detection from breaking path field

Fixes #1088
This should fix the path issues that people intermittently run into.
Gatsby sees the relative path (and the parent node with an absolute
path), joins on the parent path and the relative path, and presumes it's
a file... which is normally good, but which here is not what we want
as we're then getting a resolver of type file instead of type
string referring to the relative path of the document.
This seems to fix a (long standing?) bug where the "Edit this Page"
button is broken, as well. e.g. see a current community page vs. deploy preview community page
I'd need to think more as to whether there's a cleaner fix here, but
this seems to work pretty well!

Awesome work!

Thank you!!


How to determine which line of post to be in thumbnail?

I've been searching in config and the post, but it's still give a full text on the pagination page. How to set which line to be in thumbnail?

Hi, in the readme, I guess you are talking about excerpt.
Post excerpt
The excerpt are the first lines of an article that is display on the blog page. The length of the excerpt has a default of around 250 characters and can be manually set in the post using:
---
layout: post
title: Sample Page
excerpt_separator: <!--more-->
---

some text in the excerpt
<!--more-->
... rest of the text not shown in the excerpt ...

The html is stripped out of the excerpt so it only display text.


#25658 More responsive card columns

Replace $columns-count with $columns-sm-count and $columns-md-count that responsively span cards in 2 and 3 columns respectively.
Include media-breakpoint mixin for sm and md devices to use the new variables

This variable change breaks backward compatibility.


Update Sign-in Burst from Multiple Locations.yaml

Fixes #
Proposed Changes

@mgladi @petebryan WDYT?

Code logic looks good, just check out why the build is failing

Duplicate


Reverse unixtime

Hey, here a small feature request.
I recently wanted to get unixtime's into a usual date format (like RFC 3339).
In date I do:
$ date --date=@1458647617
Tue Mar 22 11:53:37 AM UTC 2016

But I often forget it, and I think it's a nice addition to your unixtime binary.

Hi @fossdd,
I like that idea! But if we do this, we need to keep also the --millis and --nanos option in mind. Maybe something like --from-millis and --from-nanos.
Also I would prefer to keep time zones and custom date formats out of scope for this utility. Otherwise we can just rewrite the date utility in Rust. Maybe only support for RFC 2822 and RFC 3339!?
@fossdd Do you have some time to do a PR?

I like that idea! But if we do this, we need to keep also the --millis and --nanos option in mind. Maybe something like --from-millis and --from-nanos.

Yes, I think this is fine.

Also I would prefer to keep time zones and custom date formats out of scope for this utility. Otherwise we can just rewrite the date utility in Rust. Maybe only support for RFC 2822 and RFC 3339!?

I have to look into it, but of course.

@fossdd Do you have some time to do a PR?

A bit, if it doesn't bring too much errors :-)

@fossdd Thank you very much for your PR!
I thought about it as well and did an implementation myself (#5). What do you think?

@fossdd Released in version 0.2.0


can't reproduce the results in your papers

I find my results using given data and codes are different from your results presented in your paper. And I can't find the use of GAN can improve the performance.

1 can't reproduce the results in the paper as well, I question the process of reviewing the paper, the paper doesnt even mention the batch size.

There are errors in preprocess.py,  firstly.  You incorrectly divided the datas.
And there are lots of errors in parameter settings.
And the loss function is wrong.
You may need check your files carfully!

Can't reproduce the results as well. Could you please provide the proper parameter settings and correct codes that can reproduce the results in your paper on UCSB dataset? THANKS A LOT

Please note that this code isn't provided by the original author of the paper. In fact, it's just the practice of some other people trying to implement the model.  Some key steps aren't taken in this implementation, including the (1) normalization and re-normalization of edge weights for the input and output, (2) the reshaping of the GCN feature output to a long vector, etc. Although (2) may significantly increase the complexity,  it's necessary, because TLP for dynamic weighted graphs is a hard graph-level problem, but GCN is originally designed to extract the node-level features to support node-level graph inference. Moreover, the training process is also different from the description in the paper, where the model parameters need to be continuously updated when it comes to a new timeslice.
Some improvements can also be made to reduce the complexity, e.g., (1) combining the pre-training loss (minimization of reconstruction error) and GAN loss of G into one objective, which merges the pre-training and formal training into one step; (2) using additional FC layer to reduce the dimension of the long reshaped vectors before fedding it into RNN, etc.
Some papers follow GCN-GAN and made some improvements, e.g., Yang et al. An Advanced Deep Generative Framework for Temporal Link Prediction in Dynamic Networks, IEEE Trans. on Cybernetics, etc.

你好在跑这个代码时，train.py文件运行报错："Traceback (most recent call last):
File "D:\pycharm\Project\LNG_networks\GCN-GAN-pytorch-master\train2.py", line 102, in
generator_loss.backward()
File "D:\pycharm\Project\venv\lib\site-packages\torch_tensor.py", line 487, in backward
torch.autograd.backward(
File "D:\pycharm\Project\venv\lib\site-packages\torch\autograd_init_.py", line 200, in backward
Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.FloatTensor [512, 1]], which is output 0 of AsStridedBackward0, is at version 2; expected version 1 instead. Hint: enable anomaly detection to find the operation that failed to compute its gradient, with torch.autograd.set_detect_anomaly(True).
"这是为什么呢？求大神指点，阿里嘎多！！！

你好在跑这个代码时，train.py文件运行报错："Traceback (most recent call last): File "D:\pycharm\Project\LNG_networks\GCN-GAN-pytorch-master\train2.py", line 102, in generator_loss.backward() File "D:\pycharm\Project\venv\lib\site-packages\torch_tensor.py", line 487, in backward torch.autograd.backward( File "D:\pycharm\Project\venv\lib\site-packages\torch\autograd__init__.py", line 200, in backward Variable._execution_engine.run_backward( # Calls into the C++ engine to run the backward pass RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.FloatTensor [512, 1]], which is output 0 of AsStridedBackward0, is at version 2; expected version 1 instead. Hint: enable anomaly detection to find the operation that failed to compute its gradient, with torch.autograd.set_detect_anomaly(True). "这是为什么呢？求大神指点，阿里嘎多！！！

This is not the official implementation. Follows are FIY.
https://github.com/yanghaoxie/GCN-GAN-for-Weighted-Dynamic-Networks
https://github.com/KuroginQin/OpenTLP


about other metrics

Thank you for your great work!👍
I would like to know how other metrics are calculated, such as compute semantic sequence score？
What file should be passed as segmentation_map_dir parameter？
please

Hi @cenchaojun , have you found the answer? I am stuck at that part too.


[[Metadata]] property not exists.

The semantics say that Metadata is stored in Object's [[Metadata]] property，but in fact  Metadata is stored in a global WeakMap in the implement codes . Object never has a property named [[Metadata]]。
Do I understand something wrong?

The implementation uses a WeakMap to emulate the existence of a possible future [[Metadata]] internal slot or similar mechanism.

@rbuckton I see，thnx.😄😄


Allow live stream to start from a relative point to the live edge

Have you read the FAQ and checked for duplicate issues:
What version of Shaka Player are you using:
Can you reproduce the issue with our latest release version:
Can you reproduce the issue with the latest code from master:
Are you using the demo app or your own custom app:
If custom app, can you reproduce the issue using our demo app:
What browser and OS are you using:
What are the manifest and license server URIs:
(you can send the URIs to<EMAIL_ADDRESS>instead, but please use GitHub and the template for the rest)
What did you do?
What did you expect to happen?
What actually happened?

We will add a way for an app to specify a relative offset from the live edge when calling Player.load().  For example, playback starting at -10 in a live stream would be 10 seconds before the live edge.  This is in contrast to an absolute media timestamp (as we expect now).


Updated the climate container to Zeplin version 1.5

Features added

converted the old climate container to the new design
the new container contains three IonCards, for fan intensity, temperature control, and fan direction
temperature is now controlled via up and down buttons that increase and decrease the temperature by one degree of the selected unit
replaced sliderValue with celsiusTemp in redux to accommodate this
added slider-background.svg to src/assets/ as the background image of the intensity slider to give it the notched look

The branch is out of date, causing the merge conflicts. Looks like it just needs to be rebased onto the current version of main and then we'll check it again.


Fixed content of signed in view of Cosmos DB service dialog.

Noticed some issues with the Cosmos DB service dialog when running through the release checklist.
This PR fixes the following issues:

missing dialog header
missing "Go to " button text
missing "Go to " button link (now links to https://azure.microsoft.com/services/cosmos-db/)

Before:

After:

Coverage decreased (-0.0003%) to 67.926% when pulling b1a777a59401444d20494a5e64a70cae49d92ab4 on toanzian/cosmos-db into e164c01d98cbe175cc34fcba29b8d32f2492a7b4 on master.


Flare stars

Added flare star triggering
Please double check that the source_type==11 finds all the flare star names in the SWIFT database that Gemma gave me. This is the list of names it produces:
['1A_1742-294 ', 'Aql_X-1 ', 'Cyg_X-2 ', 'GRS_1724-308 ', 'GX_5-1 ', 'H1254-690 ', 'H1608-522 ', 'H1624-490 ', 'H1658-298 ', 'H1705-440 ', 'H1820-303 ', 'KS_1731-260 ', 'X_1732-304 ', '4U_1735-44 ', 'Ser_X-1 ', '4U_1702-429 ', '4U_1746-370 ', 'GS_1826-238 ', 'X_1745.6-2901 ', 'X_0836-429 ', 'EXO_0748-676 ', 'EXO_1745-248 ', 'EXO_1747-214 ', '4U_1916-053 ', '4U_1812-12 ', 'GRS_1747-312 ', '2S_1711-339 ', '2S_0918-549 ', 'SLX_1737-282 ', 'SLX_1744-300 ', 'SAX1752.3-3138 ', 'SAX1808.4-3658 ', 'RXS171824-4029 ', 'SAX1750.8-2900 ', 'X_0512-401 ', '4U_1323-619 ', 'Cir_X-1 ', 'Cen_X-4 ', 'X_1832-330 ', 'SLX_1735-269 ', '4U_1715-321 ', 'XTE_J1723-376 ', 'SAX1747.0-2853 ', '4U_1850-087 ', 'XTE_J2123-058 ', '4U_1708-23 ', '1A_1246-588 ', 'DG_CVn ', 'V1216Sgr ', 'Wolf424 ', 'YZ_CMi ', 'CN_Leo ', 'V1054Oph ', 'V645Cen ', 'ROSS1280 ', 'DM-216267 ', 'Algol ', 'II_Peg ', 'HR_1099 ', 'UX_Ari ', 'AR_Lac ', 'TZ_CrB ', 'CF_Tuc ', 'HD_8357 ', 'Sigma_Gem ', 'AT_Mic ', 'AU_Mic ', 'DM_UMa ', 'HR_5110 ', 'Lambda_And ', 'YY_Gem ', 'UV_Cet ', 'LQ_Hya ', 'AD_Leo ', 'EV_Lac ']

That sounds like a good idea as I think she also wanted to include magnetars and maybe be able to filter by source type

Gemma said she'll have to look into it so I'm going to merge this in for now as there are no current flare star proposal and left it as an issue as a reminder #39


Update Validator Plugin (Javax)

This will prevent the eventual NoSuchMethod error when used with inject 9.0-javax

@rbygrave please let me know when you've deployed this so I can close the PR

We have released 3.2 (Jakarta) and 2.10 (Javax). So this PR can be closed now.


get rid of all migration code

Migration code exists on the migration module but also in random places across the codebase.
We should get rid of it, since this should apply only for 1.5.0

I would rather get rid of it in 1.6

Ok to do it right after we release 1.5.1


Fix multiple cloudwatch composite alarm nuking (#467)

Description
Fixes #467.

TODOs
Read the Gruntwork contribution guidelines.

[ ] Update the docs.
[x] Run the relevant tests successfully, including pre-commit checks.
[x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[x] Include release notes. If this PR is backward incompatible, include a migration guide.
[x] Attention Grunts - if this PR adds support for a new resource, ensure the nuke_sandbox and nuke_phxdevops jobs in .circleci/config.yml have been updated with appropriate exclusions (either directly in the job or via the .circleci/nuke_config.yml file) to prevent nuking IAM roles, groups, resources, etc that are important for the test accounts.

Release Notes (draft)

Updated available to delete multiple CloudWatch composite alarms.
Migration Guide

@Jennas-Lee are you able to address the review comment on this PR?

Closing this PR as there has been no response for the past month or so. Feel free to re-open the PR.
It may involve quite a bit of conflict resolution required due to some refactoring changes we've made recently.


added changes to edge and prop load

Changed some things to speed up loading files into neo4j.
Loading edges is down to 30s seconds from a few days.
Loading properties is down to a few minutes from a few hours.

Looks good to me +1

@algmzestrd is this good for you?

Could've sworn I +1'd this a while ago.
On Tue, Feb 2, 2016 at 7:25 PM, Andrew Bowler<EMAIL_ADDRESS>wrote:

@algmzestrd https://github.com/algmzestrd is this good for you?
—
Reply to this email directly or view it on GitHub
https://github.com/rwhite226/Graphalyzer/pull/19#issuecomment-178943467.


Watson / Sherlock

Hey Ben,
Would it be OK to get Rasta's CSharp implementation of Sherlock (which is now deprecated) into Posh pls? I see lots of other CSharp assemblies are making their way in too :)
https://github.com/rasta-mouse/Watson
Thanks!

Fancy doing a PR :D

In on the dev branch, will get merged in next release.


Some bookmarks not linked correctly

If you click "Table of contents", "List of Tables", "List of Figures", they link to the preface instead of their respective pages

I just saw this comment. Did you manage to fix it? Otherwise I'll take a look.


onHideAfterCancel or just onHide

Is there any way to trigger function when modal closes after pressing Cancel? There is only onHideAfterConfirm callback. It would be nice if there would be onHideAfterCancel or just onHide (similar to react-native-modal's onModalHide)

Hey @ofilipowicz, Isn't onCancel enough for your use case? Am I missing something? 🤔

@mmazzarolo it triggers on button press, I need a hook on animation ends. I want to display another modal, but this one needs to be hidden first.

Oh I got it!
Well, you should be able to pass onHide function to reactNativeModalPropsIOS, which is an Object o props that will be passed to the modal.

My bad, I just noticed.
onHideAfterConfirm seems to get triggered even when pressing the cancel button

@ofilipowicz yeah sorry
yep, it is not doable (yet) for Android since the "modal" is controlled natively by Android


Kodi 19

Hi, really appreciate this add-on. Just wanted to ask if Kodi 19 support is there or if it's a possibility?

Yes, there is Kodi 19 support

Oh yeah, should have checked 🤦♂ thanks

Oh yeah, should have checked 🤦♂ thanks

No problem, just leave a message if you run into trouble :)


Robert Longson: Watch eiffellubricants.com.au

Robert Longson requests the watch of the watch_keyword eiffellubricants\.com\.au. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

eiffellubricants\.com\.au has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Tracking issue for release

This bug is to keep track of issues involving the next release of Blockly Games.  The last release was pushed 28 June 2016, so this one is long overdue.
Staging URL:  https://beta-dot-blockly-games.appspot.com/
Main features:

Music game that teaches functions.
New levels for movie game.
Internal gallery, replacing Reddit.
New translations, including Kabyle: https://github.com/google/blockly-games/issues/66
Separate execution highlighting from selection highlighting.

Upstream Blockly issues:

[x] https://github.com/google/blockly/issues/1810 Category selected UI event has null element.
[x] https://github.com/google/blockly/issues/1425 Mutators in FF are broken in RTL.
[x] https://github.com/google/blockly/issues/503 (probably the same as https://github.com/google/blockly/issues/772) Functions called "do something" don't survive round-tripping to XML.
[x] https://github.com/ajaxorg/ace-builds/issues/123 package 31.01.18 doesn't contain latest code.
[x] https://github.com/google/blockly/issues/1699 Blockly.Mutator.findParentWs is @package
[x] https://github.com/google/blockly/issues/1726 Blockly.un/bindEvent_ is @package

Blockly Games issues:

none

Release pushed!


TLS record HMAC 2b: constant-time computation

Provide an implementation of mbedtls_ct_hmac() using only PSA APIs, when MBEDTLS_USE_PSA_CRYPTO is defined.
Depends on: #5176 - to have the keys available in struct mbedtls_ssl_transform.
Related: #5177
Follow-up: #5178

@superna9999 I initially assigned this to me, partially to save myself the trouble of writing a detailed description as I did for the other tasks, but it's now clear that you'll be able to start this way before I do, so here's a quick dump of what I had in mind, hoping it helps.
The current implementation of mbedtls_ct_hmac() already breaks the HMAC abstraction and re-implements part of HMAC: it uses mbedtls_md_xxx() functions, not mbedtls_md_hmac_xxx() functions (see the comments in the code). The PSA-based version is going to do that too: use psa_hash_xxx() functions instead of psa_mac_xxx()). It will  actually go further, in that instead of picking up ikey and okey from the context, we're going to have to compute them ourselves from the key (see the RFC or the code of mbedtls_md_hmac_starts()).
That will require, unfortunately, making the key exportable when we are creating it in populate_tranform(). Then in mbedtls_ct_hmac() we can export the key, compute ikey/okey and do like the existing implementation.
Then there are generic differences between MD and PSA, not specific to what this function does: MD has a "context" that both holds the key and the state of the current operation, while in PSA those are distinct objects: the key (held in ssl_tranform, which we're going to export) and the psa_hash_operation_t, of which we'll have two, local to the function.
The signature of the function will have to change, to accept a PSA key as it's first argument instead of a mbedtls_md_context, so we'll probably end up with a declaration somewhat like this:
#if !defined(MBEDTLS_USE_PSA_CRYPTO)
int mbedtls_ct_hmac( mbedtls_md_context_t *ctx,
 /* other arguments */ );
#else
int mbedtls_ct_hmac( mbedtls_svc_key_id_t key,
 /* other arguments */ );

(or something to that effect).
Final point, regarding testing: this function is unit-tested by ssl_cf_hmac() in test_suite_ssl. This test function depends on MBEDTLS_TEST_HOOKS, but also is only fully effective if MBEDTLS_TEST_CONSTANT_FLOW_MEMSAN or MBEDTLS_TEST_CONSTANT_FLOW_VALGRIND is used - see component_test_memsan_constant_flow() and component_test_valgrind_constant_flow() in tests/scripts/all.sh.
I hope this makes sense, if you have any question I'm still here tomorrow morning, after that I'll be on holiday for one week, but other team members can help.

@mpg Thanks, for now I replaced the mbedtls_ct_hmac() with PSA MAC API calls to finish replacinc MD by PSA API, this will certainly help me finish this task !

Yes, replacing with a call to psa_mac_sign() (or rather the multi-part equivalent) is functionally correct and just doesn't have constant-time behaviour, which means it should allow all tests to pass except the two test_xxx_constant_flow() components of all.sh, which is a good intermediate step.


Mechanism for declaring arrayed/bundled interfaces with autowrap

Issue reported by @azaidy.
Proposal is to have an optional shape field like numpy when declaring interfaces:
interfaces = {
    'umi_in': dict(type='umi', dw=dw, aw=aw, cw=cw, direction='input', shape=(m,n), txrx='umi'),
    'umi_out': dict(type='umi', dw=dw, aw=aw, cw=cw, direction='output', shape=(m,n), txrx='umi'),
    'umi_aux': dict(type='umi', dw=dw, aw=aw, cw=cw, direction='output', shape=(p,))
}

Elements in arrays should be accessible with standard array notation; this should apply to both coupled UmiTxRx interfaces and decoupled interfaces.
dut.net.intfs['umi'][i][j].write(...)
dut.net.intfs['umi_aux'][k].send(...)

Note: this should also be possible with bundled clocks and resets


Added unstable-schema generation for Cargo.toml

What does this PR try to resolve?
Added unstable-schema feature that generates JsonSchema for Cargo.toml
See #12883
finished first step of plan
Information
In cargo-util-schemas,  run cargo test --feature unstable-schema  . If there have been any changes to manifest.schema.json, rerun with env variable SNAPSHOTS=overwrite  and it will update them.
How should we test and review this PR?
In cargo-util-schemas run cargo test --features unstable-schema and it will generate manifest.schema.json

Please cleanup the commits for how you'd like them reviewed / merged.

Thanks!
@bors r+

:pushpin: Commit 1f419ba6fc934a053f598442c018bb8fb007904b has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 1f419ba6fc934a053f598442c018bb8fb007904b with merge 4d4130a45e45f6d5fce25edc930725bf99a60c40...

:sunny: Try build successful - checks-actions
Build commit: 4d4130a45e45f6d5fce25edc930725bf99a60c40 (4d4130a45e45f6d5fce25edc930725bf99a60c40)

Thanks!
@bors r+

:pushpin: Commit 21ce69ef376153efbe1308e466a9df795712a13b has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 21ce69ef376153efbe1308e466a9df795712a13b with merge cf9152007eb7221c91b6c298520dd0707d11bb9c...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing cf9152007eb7221c91b6c298520dd0707d11bb9c to master...


Angular 1.5.5+ One-way and 2-way binding no longer copies primitives to child scope

Note: for support questions, please use one of these channels: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#question. This repository's issues are reserved for feature requests and bug reports.
Do you want to request a feature or report a bug?
Possibly a bug after angular 1.5.5 versions. Works fine prior to angular 1.5.3
What is the current behavior?
When using angular.component() in angular 1.5.5+, One-way and 2-way bindings no longer copies primitives from parent scope to child scope (component). We have to explicitly copy from parent using $onChanges() method.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar (template: http://plnkr.co/edit/tpl:yBpEi4).
Create 2 directives parent and child directive and bind Primitives from parent to child (also, using controllerAs:vm) using one way binding. Make change to value in parent scope. No longer refreshes child directive with value
What is the expected behavior?
When using angular.component() in angular 1.5.5+, One-way and 2-way bindings no longer copies primitives from parent scope to child scope (component). We have to explicitly copy from parent using $onChanges() method.
This works correctly in angular 1.5.3 and prior versions
What is the motivation / use case for changing the behavior?
Which versions of Angular, and which browser / OS are affected by this issue? Did this work in previous versions of Angular? Please also test with the latest stable and snapshot (https://code.angularjs.org/snapshot/) versions.
Other information (e.g. stacktraces, related issues, suggestions how to fix)

It is difficult to identify the problem without a reproduction. The description is quite vague. Can you please post a runnable demo of the issue (e.g. using CodePen, Plnkr etc) or a code sample that we can use to reproduce the problem?

Closing due to lack of response


[BUG] GSE not working on one character and specification, working fine on others

🔵 Describe the bug:
GSE was working fine for me most of the day, then I went to edit a sequence and made a minor tweak.  Since then, I can no longer save or import sequences on that character, and even with them properly keybound, the sequences don't fire.  Other toons are working fine.
🔵 To reproduce: (Steps to reproduce the behavior)

I just ran a dungeon and realized my Beast Mastery Hunter did not have the Black Arrow in the rotation.  I went to add that spell.  It is talented, and it fires manually.
After saving that update, I went to make some other changes to the sequence.  The message on GSE said, "Save pending for..." and never finished saving.
I attempted to re-import the sequence that I had exported as a backup.  That import did nothing.
I attempted to import a different macro from wowlazymacros.com.  That import did nothing.
I switched specifications to Marksman.  The pre-patch sequence that I was using for that spec seems to fire off fine with the keybinds.  I cannot, however, save or import macros in that specification.

🔵 The error:
Paste the error message in this blockquote.
No error messages, except for the continuous "Save pending..." message when I try to save any changes.

🔵 Screenshots:
If applicable, add screenshots to help explain your problem.
🔵 Expected behavior:
A clear and concise description of what you expected to happen.
🔵 GSE.lua file:
Please provide your GSE.lua file or the export string for the specific macro that is causing an issue.

To find the GSE.lua file:

First, make sure you have enabled the "File Name Extensions" checkbox in Explorer (View tab)
Browse to C:\path\to\wow\_retail_\WTF\Account\YourAccountName\SavedVariables\ or /path/to/WoW/_retail_/WTF/Account/YourAccountName/SavedVariables
Copy the GSE.lua file to your Desktop
Rename this copy to GSA.lua.txt
Then just drag it here in your message

Note: If you do not include your GSE.lua file you are wasting your time and mine. 90% of the time, the first thing I will ask for is "Can you please upload your GSE.lua file?".  This file is needed with the error to work out Why that error occured for you.

GSE.lua.txt
The sequences in question are BM-AOE-MACRO, and BM-ST-MACRO.
🔵 Desktop (please complete the following information):

OS: Windows 11 64-bit
Game Version: Retail <IP_ADDRESS>311

🔵 GSE Version:

Version: 3.2.11-a
Downloaded From: Curseforge Application

🔵 Additional context:
Thank you for what you do, and for anything you can do to help with this.

Update:  I could go to another character and import the sequence, and then go back to my Beast Mastery Hunter and resetting the keybind gets it working again...but I am still unable to save any edits from that character in any specification.

One of your sequences on that character class is breaking the bucket.  This isn't a bug as much as its a corruption.  WHat you need to do is export one by one. Delete the one you exported and log out and back in. Its a case of finding the broken piece and removing it.
After you have found the culprit - reimport the others that you had exported.
GSE stores sequences per class.  While you can import from another class if something is rotten in the bucket when you go back to that class you cant change anything in that class until you fix whats broken.

Moving this to a discussion as it isn't a bug but doing what it is supposed to.


Update Akka to 1.1 stable release

Update all Akka packages to 1.1 stable release

closed as part of https://github.com/andreyleskov/GridDomain/releases/tag/1.4.144


Delete width property from buttons to fix button text crashing

With the original fixed with buttons, Tweeter is fine but Facebook button is crashing texts.

Hi, thanks for opening this request. Can you share a screenshot of the problem? I don't understand the issue is.


Fix deploy via juju api

They deprecated the adding charm to api server before deploy and now
require this step to be handled by the user of the api. yay. This fix
allows us to deploy services again.
Signed-off-by: Adam Stokes<EMAIL_ADDRESS>

Going to go ahead and merge this as it is a small 1 line addition and it's blocking my other work


Change paypal to be optional

- Summary
PayPal should be an optional, additional payment processor.
- Test plan
Automated tests need to be added around payment creation. I added some additional logic and a test to ensure you do not attempt a refund of a PayPal transaction (since that is not implemented yet #58).
- Description for the changelog
Change PayPal to be an optional, additional payment processor.

Superseded by #60