evaluate vs test

Hi, it's not clear the difference between the evaluate.py and test.py scripts in the NLU folder. What do they evaluate? Since the result obtained is completely different even if they take as input the same test set.

evaluate.py uses the unified interface inherited from NLU such as class BERTNLU(NLU). Each NLU model should provide such a class so that we can compare different models given the same inputs. test.py is the test script for BERTNLU only, which may have different preprocessing. However, the difference should not be large.

Ok, so evaluate.py is used to compare the performance of different NLU while if I want to test only BERTNLU I should use test.py?
It is not clear to me why test.py calls the functions is_slot_da, calculate_F1 and recover_intent while evaluate doesn't do that. On what basis the overall performance is computed, in evaluate.py, if neither slots nor intents are recovered? Thks

evaluate.py will call recover_intent in BERTNLU: https://github.com/thu-coai/ConvLab-2/blob/master/convlab2/nlu/jointBERT/multiwoz/nlu.py#L106. And calculate_F1 will be called by both evaluate.py and test.py -> https://github.com/thu-coai/ConvLab-2/blob/master/convlab2/nlu/jointBERT/multiwoz/postprocess.py#L13
is_slot_da decides whether a dialog act (intent, domain, slot, value) is non-categorical, which means the value is in the sentence and we use the slot-tagging method in BERTNLU to extract it (e.g., inform the name of a restaurant). if is_slot_da is False, we use [CLS] to do binary classification to judge if such a dialog act exists (e.g. request the name of a restaurant). We evaluate two kinds of dialog act give slot F1 and intent F1 respectively. However, this metric may not be applied to other NLU models such as a generative model, so it is not included in evaluate.py
In evaluate.py we directly evaluate the dialog act F1, comparing two lists of tuple (intent, domain, slot, value).

Hi, thanks for your reply.
In test.py the Recall, Precision and F1 score were micro-averaged?

Yes, they are. TP, FP, FN are accumulated through the test set


Intermittent test failure

test_priority (tests.test_aioh2.TestServer) doesn't seem to be deterministic.
This causes a problem when packaging this library (eg https://github.com/NixOS/nixpkgs/pull/49158), because the package build will randomly fail. For now, we can probably disable that test, but it should probably either be made more consistent, or, if that's not possible, not be run by default?

Thanks for the report and sorry for the trouble! I might have no spare time on this project right now. The almostEquals does look like unstable, I think we can skip this test first, and fix it a bit later.


🐞 - Character descenders overflow the line

Which @taiga-ui/* package(s) are the source of the bug?
core
Please provide a link to a minimal reproduction of the bug
https://taiga-ui.dev/form (check first input)
Is this issue blocking you?
Non-Blocking
Description
Characters like "y", "p", "g" (i.e. which have descenders) are not fully visible in field placeholders (at least in tui-input).

Angular version
14.1.1
Taiga UI version
2.58.1
Which browsers have you used?

[X] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

Which operating systems have you used?

[X] macOS
[ ] Windows
[ ] Linux
[ ] iOS
[ ] Android

Not reproduce in v3


Support other languages

Format data standard
Metric configuration
Language specific configuration

Progress:

Complete 9 scenarios except IR task

Please help me verify the pipeline and wait for updating IR task

Too many files have changed -- could you provide a tutorial or a readme for others to understand the code structure?

@ledong0110 please resolve the conflict


Redefinition of substrate types can lead to cumbersome errors

There are some types redefined to get no-std deserialization, however this can lead to the following errors, which might be hard to resolve:

We should check if the no-std deserialization can be upstreamed.

Yeah. There's an open PR on substrate, didn't get a response for quite some time. Not sure what else I can do.
See https://github.com/scs/substrate-api-client/issues/391 for more info.

Closing this as duplicate. Please reopen if you disagree.


Fix setting constraints for number dropdown fields

Fixes #721

Happy this was not too bad - thanks!


[Bug]: pip install -r requirements.txt error!

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
aae84f6
Other environment information
Collecting accelerate==0.27.2 (from -r requirements.txt (line 1))
  Downloading accelerate-0.27.2-py3-none-any.whl (279 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 280.0/280.0 kB 6.4 MB/s eta 0:00:00
Requirement already satisfied: aiohttp==3.9.3 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 2)) (3.9.3)
Requirement already satisfied: aiosignal==1.3.1 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 3)) (1.3.1)
Requirement already satisfied: annotated-types==0.6.0 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 4)) (0.6.0)
Collecting anyio==4.3.0 (from -r requirements.txt (line 5))
  Downloading anyio-4.3.0-py3-none-any.whl (85 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 85.6/85.6 kB 13.8 MB/s eta 0:00:00
Requirement already satisfied: argon2-cffi==23.1.0 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 6)) (23.1.0)
Requirement already satisfied: argon2-cffi-bindings==21.2.0 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 7)) (21.2.0)
Collecting Aspose.Slides==24.2.0 (from -r requirements.txt (line 8))
  Downloading Aspose.Slides-24.2.0-py3-none-manylinux1_x86_64.whl (88.7 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 88.7/88.7 MB 2.6 MB/s eta 0:00:00
Requirement already satisfied: attrs==23.2.0 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 9)) (23.2.0)
Collecting blinker==1.7.0 (from -r requirements.txt (line 10))
  Downloading blinker-1.7.0-py3-none-any.whl (13 kB)
Collecting cachelib==0.12.0 (from -r requirements.txt (line 11))
  Downloading cachelib-0.12.0-py3-none-any.whl (20 kB)
Requirement already satisfied: cachetools==5.3.3 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 12)) (5.3.3)
Requirement already satisfied: certifi==2024.2.2 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 13)) (2024.2.2)
Requirement already satisfied: cffi==1.16.0 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 14)) (1.16.0)
Requirement already satisfied: charset-normalizer==3.3.2 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 15)) (3.3.2)
Requirement already satisfied: click==8.1.7 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 16)) (8.1.7)
Collecting coloredlogs==15.0.1 (from -r requirements.txt (line 17))
  Downloading coloredlogs-15.0.1-py2.py3-none-any.whl (46 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 46.0/46.0 kB 7.2 MB/s eta 0:00:00
Requirement already satisfied: cryptography==42.0.5 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 18)) (42.0.5)
Collecting dashscope==1.14.1 (from -r requirements.txt (line 19))
  Downloading dashscope-1.14.1-py3-none-any.whl (1.2 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.2/1.2 MB 86.1 MB/s eta 0:00:00
Collecting datasets==2.17.1 (from -r requirements.txt (line 20))
  Downloading datasets-2.17.1-py3-none-any.whl (536 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 536.7/536.7 kB 57.4 MB/s eta 0:00:00
Collecting datrie==0.8.2 (from -r requirements.txt (line 21))
  Downloading datrie-0.8.2.tar.gz (63 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 63.3/63.3 kB 9.7 MB/s eta 0:00:00
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Installing backend dependencies ... done
  Preparing metadata (pyproject.toml) ... done
Collecting demjson==2.2.4 (from -r requirements.txt (line 22))
  Downloading demjson-2.2.4.tar.gz (131 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 131.5/131.5 kB 17.2 MB/s eta 0:00:00
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> See above for output.

  note: This error originates from a subprocess, and is likely not a problem with pip.
  Preparing metadata (setup.py) ... error
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Actual behavior
pip install -r -r requirements.txt
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
Preparing metadata (setup.py) ... error
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
Expected behavior
pip install -r -r requirements.txt  will be successful
Steps to reproduce
1. pip install -r requirements.txt
2. Error happened again.

Additional information
Collecting accelerate==0.27.2 (from -r requirements.txt (line 1))
Using cached accelerate-0.27.2-py3-none-any.whl (279 kB)
Requirement already satisfied: aiohttp==3.9.3 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 2)) (3.9.3)
Requirement already satisfied: aiosignal==1.3.1 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 3)) (1.3.1)
Requirement already satisfied: annotated-types==0.6.0 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 4)) (0.6.0)
Collecting anyio==4.3.0 (from -r requirements.txt (line 5))
Using cached anyio-4.3.0-py3-none-any.whl (85 kB)
Requirement already satisfied: argon2-cffi==23.1.0 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 6)) (23.1.0)
Requirement already satisfied: argon2-cffi-bindings==21.2.0 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 7)) (21.2.0)
Collecting Aspose.Slides==24.2.0 (from -r requirements.txt (line 8))
Using cached Aspose.Slides-24.2.0-py3-none-manylinux1_x86_64.whl (88.7 MB)
Requirement already satisfied: attrs==23.2.0 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 9)) (23.2.0)
Collecting blinker==1.7.0 (from -r requirements.txt (line 10))
Using cached blinker-1.7.0-py3-none-any.whl (13 kB)
Collecting cachelib==0.12.0 (from -r requirements.txt (line 11))
Using cached cachelib-0.12.0-py3-none-any.whl (20 kB)
Requirement already satisfied: cachetools==5.3.3 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 12)) (5.3.3)
Requirement already satisfied: certifi==2024.2.2 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 13)) (2024.2.2)
Requirement already satisfied: cffi==1.16.0 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 14)) (1.16.0)
Requirement already satisfied: charset-normalizer==3.3.2 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 15)) (3.3.2)
Requirement already satisfied: click==8.1.7 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 16)) (8.1.7)
Collecting coloredlogs==15.0.1 (from -r requirements.txt (line 17))
Using cached coloredlogs-15.0.1-py2.py3-none-any.whl (46 kB)
Requirement already satisfied: cryptography==42.0.5 in /usr/local/lib/python3.10/dist-packages (from -r requirements.txt (line 18)) (42.0.5)
Collecting dashscope==1.14.1 (from -r requirements.txt (line 19))
Using cached dashscope-1.14.1-py3-none-any.whl (1.2 MB)
Collecting datasets==2.17.1 (from -r requirements.txt (line 20))
Using cached datasets-2.17.1-py3-none-any.whl (536 kB)
Collecting datrie==0.8.2 (from -r requirements.txt (line 21))
Using cached datrie-0.8.2.tar.gz (63 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Installing backend dependencies ... done
Preparing metadata (pyproject.toml) ... done
Collecting demjson==2.2.4 (from -r requirements.txt (line 22))
Using cached demjson-2.2.4.tar.gz (131 kB)
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
Preparing metadata (setup.py) ... error
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

try this:
pip install --upgrade setuptools==57.5.0

if you have installed blink>=1.7.0, you have to remove it first by following commands:
!sudo apt-get remove blinker
!sudo -H pip install --ignore-installed -U blinker


Request clarification on memory model and thread safety guarantees regarding read/write elimination/introduction

Citing dotnet/runtime#4329 and dotnet/runtime#14746:

ImmutableArray<T> goes to great lengths to be thread-safe by never dereferencing this more than once per public entrypoint (in dotnet/corefx#200 to fix bug dotnet/runtime#13908). Other structs in the Framework do not take this care (such as DateTime, Nullable<T>, or System.Int32), despite claiming to be thread-safe in MSDN documentation.
The community has been fascinated by the pattern in ImmutableArray<T> and brought up that it might make sense to spread it around more.

Personally, I also thought about this trick to make "thin" types thread-safe. However, my earlier research indicated otherwise, as explained in these two MSDN magazine articles:

The C# Memory Model in Theory and Practice -- MSDN Magazine -- December 2012
The C# Memory Model in Theory and Practice, Part 2 -- MSDN Magazine -- January 2013

They claim
public class ReadIntro {
  private Object _obj = new Object();
  void PrintObj() {
    Object obj = _obj;
    if (obj != null) {
      Console.WriteLine(obj.ToString());
    // May throw a NullReferenceException
    }
  }
  void Uninitialize() {
    _obj = null;
  }
}

that in the code above, running readIntro.PrintObj() and readIntro.Uninitialize() in parallel could result in NullReferenceException being thrown due to readIntro.obj being set to null.
It would be beneficial to know whether the current implementation in dotnet/runtime uses "insider knowledge" that the current JIT compiler will not introduce reads for ImmutableArray<T> (i.e., the particular implementation is not thread-safe as far as the specification is concerned), or whether the specification has changed to ensure reads are not introduced. Could a documentation clarifying this issue be provided at docs.microsoft.com?

@stephentoub Could you help to answer this question?

It would be beneficial to know whether the current implementation in dotnet/runtime uses "insider knowledge" that the current JIT compiler will not introduce reads for ImmutableArray (i.e., the particular implementation is not thread-safe as far as the specification is concerned), or whether the specification has changed to ensure reads are not introduced.

the current JIT compiler will not introduce reads

Correct
https://github.com/dotnet/runtime/issues/63474 tracks more formally documenting .NET's memory model.  A draft of that is available at https://github.com/VSadov/runtime/blob/memModel/docs/design/coreclr/botr/memory-model.md.  And as it states: "Reads cannot be introduced.".

Thanks @stephentoub! Since the documentation is tracked elsewhere, I'm going to close this issue.


Relax media parsing, fix data url parsing

This is a great package. I'm using it to parse CSS in the wild to extract data. A lot of it is invalid, so it is nice to have a parser that is as liberal as the browser one.
This PR fixes the regex similar to https://github.com/reworkcss/css/pull/110 and adds recovery after a rule with no selector.
@media screen and (max-width: 767px) { {
    width: 100%;
}
}
#id {
    position: fixed;
}

Awesome, thanks!


Reuse connection pool depending on host and connection options

fixes: https://github.com/Azure/azure-sdk-for-cpp/issues/1103
fixes: #1060

fixes: #1060

No wonder that test would fail occasionally. Keep an eye out for such failures going forward, because they are good canaries for pooling having some bug.

No wonder that test would fail occasionally. Keep an eye out for such failures going forward, because they are good canaries for pooling having some bug.

I updated the test in a more unit-test fashion way where there is no network calls. This will ensure the test is stable now and at the same time I am testing the expected bahavior from the pool


add the join today

Checklist

[x] I have read the Contributor Guide
[x] I have read and agree to the Code of Conduct
[x] I have added a description of my changes and why I'd like them included in the section below

Description of Changes
Describe your changes here
Related Issues
List related issues here

Hi @paulokenza! Welcome to the repo.
I'm not sure what the intent is with this change. Could you open an issue first if you see an improvement that could be made? Then we can discuss how to address it together 🙂


TFRecord load

Hej,
Thanks for your great tutorial on building my own object detection!
I'd like to create my own model with the records i used in your tutorial.
Can you/or anyone else provide a python function to load the records that were created by your generate_tfrecords.py?
I cant fully grasp, what the script does in detail.
Regards,
Jarvis

got a clue from doc
import tensorflow as tf

def _parse_image_function(example_proto):
    image_feature_description = {
        'image/height':  tf.io.FixedLenFeature([], tf.int64),
        'image/width': tf.io.FixedLenFeature([], tf.int64),
        'image/filename':  tf.io.FixedLenFeature([], tf.string),
        'image/source_id': tf.io.FixedLenFeature([], tf.string),
        'image/encoded':  tf.io.FixedLenFeature([], tf.string),
        'image/format': tf.io.FixedLenFeature([], tf.string),
        'image/object/bbox/xmin':  tf.io.FixedLenSequenceFeature([], tf.float32,allow_missing=True),
        'image/object/bbox/xmax':  tf.io.FixedLenSequenceFeature([], tf.float32,allow_missing=True),
        'image/object/bbox/ymin':  tf.io.FixedLenSequenceFeature([], tf.float32,allow_missing=True),
        'image/object/bbox/ymax':  tf.io.FixedLenSequenceFeature([], tf.float32,allow_missing=True),
        'image/object/class/text': tf.io.FixedLenSequenceFeature([], tf.string,allow_missing=True),
        'image/object/class/label':tf.io.FixedLenSequenceFeature([], tf.int64,allow_missing=True),
    }
    return tf.io.parse_single_example(example_proto, image_feature_description)

filenames="/path/to/records"
raw_image_dataset = tf.data.TFRecordDataset(filenames)

parsed_image_dataset = raw_image_dataset.map(_parse_image_function)
parsed_image_dataset

for image_features in parsed_image_dataset:
    print(image_features['image/height'])
    image_raw = image_features['image/encoded'].numpy()


 Is it because it is a file containing information specific to a machine?

It's mostly because the file has nothing to do with the code that is being tracked in git. It contains custom attributes set on the containing folder in the Finder. Unless those custom attributes are relevant to the code, that file is just noise. Additionally, if the file is not ignored and gets checked in unintentionally and later two devs each decide to put a pretty custom icon on the folder, then you could end up with a merge conflict on a binary file.
https://en.m.wikipedia.org/wiki/.DS_Store


Clean up the output that goes to stderr.

Info logging is a bit ugly, so messages that should be user-visible get
printed to stderr directly.
Also add a --quiet flag.

@dlorenc should we close this one?

Closing and tracking in this issue #370


can setOnItemClickListener work? or other method can use?

i have try setOnItemClickListener method for doing something when user click listview item like this:
playListView.setOnItemClickListener(new AdapterView.OnItemClickListener() {
@Override
public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
onStartVideo(position);
}
});
but it did not work, why? have some else method to realize this? setItemActionListener seems to work only when click action button.
thands to your help!

i have this problem too

is this issue already solved or what ?

This is not supported. Or look at the example code. there might be an
example.
I am not maintaining this repo anymore.
Please submit a pull request to fix it.
On Fri, Dec 16, 2016, 10:40 configurer<EMAIL_ADDRESS>wrote:

is this issue already solved or what ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tjerkw/Android-SlideExpandableListView/issues/103#issuecomment-267555038,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAx-saybV4G_hgYnGzNJE5dBjCeyTYrgks5rIlyjgaJpZM4Ggu6M
.

This repo is outdated due to the new RecyclerView and the fact that this
UIX is a bit old.
On Fri, Dec 16, 2016, 14:10 Tjerk Wolterink<EMAIL_ADDRESS>wrote:

This is not supported. Or look at the example code. there might be an
example.
I am not maintaining this repo anymore.
Please submit a pull request to fix it.
On Fri, Dec 16, 2016, 10:40 configurer<EMAIL_ADDRESS>wrote:
is this issue already solved or what ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tjerkw/Android-SlideExpandableListView/issues/103#issuecomment-267555038,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAx-saybV4G_hgYnGzNJE5dBjCeyTYrgks5rIlyjgaJpZM4Ggu6M
.


Investigate CSS attributes added for ScratchAddons

It looks like there have been a bunch of CSS attributes added to scratch to support https://github.com/ScratchAddons
Some PRs that did this:

https://github.com/scratchfoundation/scratch-blocks/pull/645
https://github.com/scratchfoundation/scratch-blocks/pull/638

Adding most of these CSS attributes involved monkey-patching things in Blockly, which is not happy =(
We need to figure out:

Whether we need to continue supporting these CSS attributes. (Ask a scratch person)
If so, what the attributes actually are. Which is going to be kinda tricky given that bugs won't show up in actual scratch. They'll only show up if you're using a given addon that references an attribute.
If we want to pull those into core Blockly or figure out a way to add them in a non-monkey-patch-y way to scratch-blocks.

Chris said that we didn't need to maintain these. And in general for ScratchAddons things, we shouldn't actively remove them, but we don't need to put extra effort into maintaining them.


Questions about Vehicle Control Calibration Tool

System information
OS Platform and Distribution (e.g., Linux Ubuntu 14.04): Ubuntu 16.04
Apollo installed from (source or binary): Source
Apollo version (1.0, 1.5, 2.0, 2.5, 3.0): 5.0
I'm doing the following instruction.
https://github.com/ApolloAuto/apollo/blob/master/docs/specs/calibration_table/control_calibration.md
Q1. There's not enough information to collect data. When I see data collection monitor, I couldn't make the progress bar changed. In order to see progress bar change, Is the Setting up Apollo Fuel Proxy essential?
Q2.When I do vehicle control calibration, should three components (GPS, Localization, Recorder) and modules (Canbus, Recorder, Sensors) work?

Q3. Aside modules mentioned in Q2, do I need to run other modules (Transform, Prediction, Planning, Perception, Localization, Control, etc)?
Q4. There are two vehicle calibration docs like the following.
How to Update Vehicle Calibration for Throttle and Brakes: https://github.com/ApolloAuto/apollo/blob/master/docs/howto/how_to_update_vehicle_calibration.md
Vehicle Control Calibration: https://github.com/ApolloAuto/apollo/blob/master/docs/specs/calibration_table/control_calibration.md
I think second one provides more precise calibration table because it has more specific cases like Go Straight, Reverse: Go Straight or Branch Left/Right. What's each calibration tool for? Are they separate calibration tool? Are they related each other?
Q5. I'm using with LGSVL simulator to test parking. When it comes to speed and acceleration, I think it seems like vehicle calibration table is wrong for this vehicle. What do you think about this one?

Best
Taehyung Kim

Thank you for your interest in Apollo platform.
Q1. The configure file is mkz7_data_collection_table.pb.txt. It is in apollo repo.
Q2. Yes.
Q3. No. It is not at autonomous driving mode. Thus, we don't need other modules.
Q4. https://github.com/ApolloAuto/apollo/blob/master/docs/specs/calibration_table/control_calibration.md
This one is the latest version.
Q5. To verify a calibration table, you may try a straight trajectory for test. Parking involves both longitudinal controller and lateral controller. It is hard to tell which controller went wrong.

Hi @cyberphysicalrobotics, hope our answer resolved your question. We will close the issue for now. If you have any additional question, please feel free to open a new issue. Our engineer team are more than happy to help that.
Thank you for supporting Apollo!


Display negative numbers in proper units, fix #335.

max_divisions will now work properly for negative numbers, instead of
always returning 0.

[x] Run the tests and see them pass
[x] Rebase your branch on top of develop
[x] Include tests for features you introduced / bugs you fixed

The failing checks seem to be totally unrelated to my PR, on my machine all tests passed.

The build fails because of something related to the ldap library triggered by the hey integration tests. This needs to be investigated.

pyasn1 was no longer compatible with ldap3 2.2.2, I set it to 2.3 now and the tests finally passed.


method to change app.context

Koa has option to change app.context (the prototype from which ctx is created from). Could we have a method to do that without modifying "kit" folder files?

I've got 'general purpose app middleware' on my to-do list, for manipulating the Koa app instance directly, which would make this possible. Will update this issue when released.

Released in 2.6.0. npm i -g reactql to upgrade.


Text widget render its content outside its boundary for Khmer language

Description
I am having a problem regarding displaying long Khmer Unicode text using Text widget. For some reason, the content goes outside the container cause rendering issue as below:

Steps to Reproduce
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: Scaffold(
        body: Center(
          child: Container(
            color: Colors.amber,
            width: 200,
            child: Text("និងក្មេងចង់ផ្ទៃសមុទ្រសែនខៀវស្រងាត់និងធំល្វឹងល្វើយសឹងតែរកកោះចិត្តរបសផ្ទៃសមុទ្រសែនខៀវស្រងាត់និងធំល្វឹងល្វើយសឹងតែរកកោះចិត្តរបស់អ្នកទេសចរទាំងចាស់និងក្មេងចង់ផ្ទៃសមុទ្រសែនខៀវស្រងាត់និងធំល្វឹងល្វើយកកោះចិត្តរបសផ្ទៃសមុទ្រសែនខៀវស្រងាត់និងធំល្វឹងល្វើយសឹងតែរកកោះចិត្តរបស់អ្នកទេសចរ"),
          ),
        ),
      ),
    );
  }
}

Logs
Here is my flutter doctor -v output:
[√] Flutter (Channel stable, v1.9.1+hotfix.2, on Microsoft Windows [Version 10.0.17134.1006], locale en-US)
    • Flutter version 1.9.1+hotfix.2 at D:\flutter_sdk\flutter
    • Framework revision 2d2a1ffec9 (2 weeks ago), 2019-09-06 18:39:49 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.0)
    • Android SDK at D:\Users\User\AppData\Android\Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.0
    • ANDROID_HOME = D:\Users\User\AppData\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    • All Android licenses accepted.

[√] Android Studio (version 3.5)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[√] VS Code (version 1.36.1)
    • VS Code at C:\Users\User\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.4.1

[√] Connected device (1 available)
    • Android SDK built for x86 64 • emulator-5554 • android-x64 • Android 9 (API 28) (emulator)

• No issues found!

Could you please give it a try and switch to master channel and try again?

Thank you for the quick response, I have switched to master channel but the problem is still there

flutter doctor -v output:
[√] Flutter (Channel master, v1.10.2-pre.34, on Microsoft Windows [Version 10.0.17134.1006], locale en-US)
    • Flutter version 1.10.2-pre.34 at D:\flutter_sdk\flutter
    • Framework revision 609a78fdca (12 days ago), 2019-09-10 20:03:52 -0700
    • Engine revision 7ea9884ab0
    • Dart version 2.5.0 (build 2.5.0-dev.4.0 be66176534)

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.0)
    • Android SDK at D:\Users\User\AppData\Android\Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.0
    • ANDROID_HOME = D:\Users\User\AppData\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    • All Android licenses accepted.

[√] Android Studio (version 3.5)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[√] VS Code (version 1.36.1)
    • VS Code at C:\Users\User\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.4.1

[√] Connected device (1 available)
    • Android SDK built for x86 64 • emulator-5554 • android-x64 • Android 9 (API 28) (emulator)

• No issues found!

Thanks for trying

Hi,
Is there any update on this issue yet?

Hi, not that I know but there have been some fixes made in this aera but I don't know if it also addresses this you could try again after upgrading your master.
The team has recognised the issue so they will take care of it as soon as possible.

ok. Thank you for the update.

Having the same with Kannada Language.
Steps to Reproduce
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Radio Tiles',
      home: HomeView(),
    );
  }
}

class HomeView extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.white,
      body: Container(
        width: MediaQuery.of(context).size.width,
        margin: EdgeInsets.only(top: 60),
        color: Colors.red,
        child: Text(
          'ಮೈ ಕಾಂಟ್ಯಾಕ್ಟ್ಸ್, ಫಾಲ್ಲೋಹರ್ಸ್ & ಫಾಲೋಯಿಂಗ್ ಫಾಲ್ಲೋಹರ್ಸ್ & ಫಾಲೋಯಿಂಗ್ ಫಾಲ್ಲೋಹರ್ಸ್',
          style: TextStyle(
            color: Colors.black,
          ),
          textAlign: TextAlign.left,
        ),
      ),
    );
  }
}

flutter doctor -v output
[✓] Flutter (Channel stable, v1.9.1+hotfix.5, on Mac OS X 10.15 19A583, locale en-IN)
    • Flutter version 1.9.1+hotfix.5 at /Users/vinothini/Documents/flutter
    • Framework revision 1aedbb1835 (2 days ago), 2019-10-17 08:37:27 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/vinothini/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.0, Build version 11A420a
    • CocoaPods version 1.8.3

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.39.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.5.1

[✓] Connected device (2 available)
    • iPhone 6s • 7ADBC258-5A57-4FED-BB3D-A1E21D34BA86 • ios • com.apple.CoreSimulator.SimRuntime.iOS-13-0 (simulator)
    • iPhone SE • 71794AB3-A52D-4798-A282-410BCEDB67F4 • ios • com.apple.CoreSimulator.SimRuntime.iOS-13-0 (simulator)

• No issues found!

@bala-vino What do you mean you have the same problem? I can't see a problem on the screenshot when comparing to your code.

Thanks for your response.
In the above attached screenshot the text ಫಾಲೋಯಿಂಗ್ went out of the screen.
For more clearness have added margin for the container, the text content goes outside the container.
Steps to Reproduce
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Radio Tiles',
      home: HomeView(),
    );
  }
}

class HomeView extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
        backgroundColor: Colors.white,
        body: Container(
        margin: EdgeInsets.only(top: 60, left: 16, right: 16),
        color: Colors.red,
        child: Text(
          'ಮೈ ಕಾಂಟ್ಯಾಕ್ಟ್ಸ್, ಫಾಲ್ಲೋಹರ್ಸ್ & ಫಾಲೋಯಿಂಗ್ ಫಾಲ್ಲೋಹರ್ಸ್ & ಫಾಲೋಯಿಂಗ್ ಫಾಲ್ಲೋಹರ್',
          style: TextStyle(
            color: Colors.black,
          ),
          textAlign: TextAlign.left,
        ),
      ),);
  }
}

I added the  will  need additional triage because this issue seems to affect multiple Asian languages

cc @GaryQian this might be worth a look to determine how serious it is

I would like to confirm that it is now fixed in master channel. Thank you for your effort team. Great job.


Add shared processing for head requests and move blocking activities

Added some functionality to have concurrent head requests for the same branch share the same result. This also adds a 60-second timeout to the head requests as well. Will result in a 404 on a timeout which is less than desirable though.

Just updated the PR to also include moving all blocking I/O activities to an appropriate thread pool. Actix performs the requests on its main worker pool and highly recommends moving I/O to another blocking pool.
See here: https://actix.rs/docs/server/


Packages and .NET9 update

Packages have been upgraded and test for .NET9 were also added.
Note that there is one TODO, where the test run into an unexpected message, when testing the "old" sender:
// TODO: Verify that the exception message is correct Assert.That(ex.Message, Does.StartWith("Unable to write data to the transport connection: Broken pipe."), "Bad exception message"); // Original message  //Assert.That(ex.Message, Does.StartWith("Authentication failed or server disconnected"), "Bad exception
Someone has to verify that the message is ok, or there is a bug.

When do you plan to remove the "Obsolete" stuff?

When do you plan to remove the "Obsolete" stuff?

@NerdsRule Never. Please change the usage and use Sender class instead, see more in README.md


Add  timestamp to agent commands

Agent now can get log level configuration from cmdline.
Changed logrus time format to : TimestampFormat: time.RFC3339Nano to get nanoseconds precision.
Start and end of commands is logged with the message:   {event}_start or  {event}_end
An special case is logged agent_started seconds.nanoseconds this get uptime with good clock precision.
Logrus format is JSON for easy parsing

LGTM
Let's wait for @jodh-intel comments before to merge it.

@jodh-intel I still need to rework the timestamp format...

@jodh-intel plase take a look to the log using logrus timestamp:
{"level":"info","msg":"Agent version: 0.1.0-alpha.0+-28cad9a9947ceced9a05b43ac3db5f4358ea5167","time":"2017-08-15T14:43:11.690803064Z"}
{"level":"info","msg":"agent_started 0.542392826","time":"2017-08-15T14:43:11.692726511Z"}
{"level":"info","msg":"Received \"startpod\" command","time":"2017-08-15T14:43:11.751076463Z"}
{"level":"info","msg":"startpod_start","time":"2017-08-15T14:43:11.751832257Z"}
{"level":"info","msg":"startpod_end","time":"2017-08-15T14:43:11.759136667Z"}
{"level":"info","msg":"Received \"newcontainer\" command","time":"2017-08-15T14:43:11.767398068Z"}
{"level":"info","msg":"newcontainer_start","time":"2017-08-15T14:43:11.767548997Z"}

@jcvenegas - I'm not clear on your last comment. This looks fine to me.
However, are you planning to either modify the logrus formatter to change the time format, or maybe add a special time format message for startpod_start and startpod_end or something?

@jodh-intel @grahamwhaley  I would like to get your approval in the format, now that I moved to RFC3339 as was suggested.

Yes I changed logrus time format to : TimestampFormat: time.RFC3339Nano to get nanoseconds precision.
To log start end events I just print  {event}_start or  {event}_end
An special case is logged agent_started seconds.nanoseconds this get uptime with good clock precision.
Logrus format is JSON for easy parsing

@jodh-intel @grahamwhaley could you give us your review on this one ?

lgtm.

I'm happy if @jodh-intel is happy, and he is - so merging....


How to use the parse_file or parse_str function?

I followed the code example on the documentation and I was prompted that could not find syn::parse_file, what should I do?
I have add
syn = "2.0.17"
quote = "1.0"

in Cargo.toml, and here is my code:
use std::error::Error;
use std::fs::File;
use std::io::Read;

fn run() -> Result<(), Box<dyn Error>> {
    let mut file = File::open("path/to/code.rs")?;
    let mut content = String::new();
    file.read_to_string(&mut content)?;

    let ast = syn::parse_file(&content)?;
    if let Some(shebang) = ast.shebang {
        println!("{}", shebang);
    }
    println!("{} items", ast.items.len());

    Ok(())
}

fn main(){
    run();
}

I'm confused and don't know what to do.
I have tried the code example of syn::parse_str, but it can't work either:

And this is my result:

syn::parse_file is available only when you enable both parsing and full features. See the tiny box in this page with text "Available on crate features ...".  parsing feature is default-enabled, but full is not. To enable it, you'll need following line in your Cargo.toml:
syn = { version = "2.0.17", features = [ "full" ] }

Similarly, the fmt::Debug trait on syn::Expr is available only when you enable extra-traits feature. Because you don't have it enabled, you're seeing the error while trying syn::parse_str.

Yeah, I have figured it out, thanks a lot!


Update the gemspec file

Update the current gemspec file to simplify it.

Removed date so gem pushes will always be current time
Simplified the runtime_dependency includes

this gem build is handled by Echoe, which scratch the gemspec on each run...
Your changes will be lost on next rake build....
Maybe you can wait a bit between a PR and his merge. juste time for a quick review

Ah I see. I guess we can remove echoe in favor of bundler. It seems like that's a dead project anyway, plus bundler is definitely simpler to use. I'll make a pr shortly to remove echoe, good thing is you can just do gem build with the current gemspec.

I agree with you


Could you please add QR code generation to filemanager

(Are you asking for help with using Caddy? Please use our forum instead: https://forum.caddyserver.com. If you are filing a bug report, please take a few minutes to carefully answer the following questions. If your issue is not a bug report, you do not need to use this template. Thanks!)
1. What version of Caddy are you running (caddy -version)?
0.9.5
2. What are you trying to do?
3. What is your entire Caddyfile?
(Put Caddyfile here)

4.
5. Please paste any relevant HTTP request(s) here.
(paste curl command, or full HTTP request including headers and body, here)
6. What did you expect to see?
add QR code generation to filemanager
7. What did you see instead (give full error messages and/or log)?
8. How can someone who is starting from scratch reproduce the bug as minimally as possible?
(Please strip away any extra infrastructure such as containers, reverse proxies, upstream apps, dependencies, etc, to prove this is a bug in Caddy and not an external misconfiguration. Your chances of getting this bug fixed go way up the easier it is to replicate. Thank you!)

Hi @deathrush - thank you for your feature request. This is the issue tracker for Caddy, but filemanager is a 3rd party plugin, as noted in the docs:

So, please feel free to file this issue at https://github.com/hacdias/caddy-filemanager. Thanks!


r-test and pfunit

Hi, I got some confusion while trying to test OpenFAST
I've done the installing and build the OpenFAST with visual studio 2019 and intel fortran. I'm trying to test OpenFast with r-test. I'm reading the documentation and it is said that I have to run pFunit first with cmake. I've cloned the openfast from the github, then r-test and pfunit in openfast folder that I've cloned are empty. Should I clone it separately out of OpenFAST folder or where should I put the r-test and pfunit?
if it is download separately from openfast folder, I tried to download the pfunit folder put it outside openfast folder from github and run it with cmake, then this error below appear:
Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.19042.
The Fortran compiler identification is Intel 2<IP_ADDRESS>10609
The C compiler identification is MSVC 19.29.30038.1
Detecting Fortran compiler ABI info
Detecting Fortran compiler ABI info - done
Determine Intel Fortran Compiler Implicit Link Path
Determine Intel Fortran Compiler Implicit Link Path - done
Check for working Fortran compiler: C:/Program Files (x86)/Intel/oneAPI/compiler/2021.3.0/windows/bin/intel64/ifort.exe - skipped
Checking whether C:/Program Files (x86)/Intel/oneAPI/compiler/2021.3.0/windows/bin/intel64/ifort.exe supports Fortran 90
Checking whether C:/Program Files (x86)/Intel/oneAPI/compiler/2021.3.0/windows/bin/intel64/ifort.exe supports Fortran 90 - yes
Detecting C compiler ABI info
Detecting C compiler ABI info - done
Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.29.30037/bin/Hostx64/x64/cl.exe - skipped
Detecting C compile features
Detecting C compile features - done
*** Setting default install prefix to C:/Users/dwino/pFUnit/installed.
*** Override with -DCMAKE_INSTALL_PREFIX=.
Performing Test _LOGICAL_DEFAULT_KIND: SUCCESS (value=4)
Performing Test _INT_DEFAULT_KIND: SUCCESS (value=4)
Performing Test _ISO_INT8: SUCCESS (value=1)
Performing Test _ISO_INT16: SUCCESS (value=2)
Performing Test _ISO_INT32: SUCCESS (value=4)
Performing Test _ISO_INT64: SUCCESS (value=8)
Performing Test _REAL_DEFAULT_KIND: SUCCESS (value=4)
Performing Test _DOUBLE_DEFAULT_KIND: SUCCESS (value=8)
Performing Test _ISO_REAL32: SUCCESS (value=4)
Performing Test _REAL32_IEEE_SUPPORT: FAILURE
Performing Test _ISO_REAL64: SUCCESS (value=8)
Performing Test _REAL64_IEEE_SUPPORT: FAILURE
Performing Test _ISO_REAL128: SUCCESS (value=16)
Performing Test _REAL128_IEEE_SUPPORT: FAILURE
what should I do? Thank you before.

Hi @saksiwi
The r-test and pfunit folders should exist inside of the OpenFAST folder and they should not be empty. These two repositories are included in the OpenFAST project as git-submodules. So, you should use git to clone the submodules directly. From your current OpenFAST directory, use this command: git submodule update --init --recursive. For reference, this portion of the documentation is here.
Keep in mind that the testing is only required for new software development. If you will not be making changes to the source code, I recommend that you download the executables from the GitHub Releases page or, for linux and Mac, use conda (conda install -c conda-forge openfast).

Hi @rafmudaf , thank you for replying.
I have to use the OpenFAST for modeling 2 MW wind turbine later, now I want to learn modeling 5MW wind turbine that have already been built inside the OpenFAST. If that the case should I do the testing?
If not, what do you recommend I do to start trying to model the wind turbine? I want to ask to execute the model is it only using command prompt or should I execute it from visual studio or using matlab? Sorry for asking such unimportant question, but hope you can help me.

Ok, in this case you do not need to compile the software. Compiling is a process that converts the code written in Fortran into an executable that you can run in the command line (cmd.exe). The OpenFAST software contains many input files to describe a turbine model and operating conditions, so you can change the input files as needed without modifying the code. You may know this, but I'm describing the process just in case.
I recommend that you download the files attached to the latest GitHub Release (linked above) in the "Assets" dropdown. Then, from the command prompt try to execute the file openfast_x64.exe with no additional arguments. The expected response is for the command prompt to display the syntax of the OpenFAST command line interface. The unexpected response is a pop-up saying something about a missing library.
After validating that the executable runs on your machine, you can move to establishing your turbine model. Please report back after verifying the executable will run.

Hi @rafmudaf thank you for the explanation!
I've tried to execute the openfast_x64.exe file, and this is the result:
Microsoft Windows [Version 10.0.19042.1165]
(c) Microsoft Corporation. All rights reserved.
C:\Users\dwino>openfast_x64.exe

OpenFAST
Copyright (C) 2021 National Renewable Energy Laboratory
Copyright (C) 2021 Envision Energy USA LTD
This program is licensed under Apache License Version 2.0 and comes with ABSOLUTELY NO WARRANTY.
See the "LICENSE" file distributed with this software for details.

OpenFAST-v3.0.0
Compile Info:

Compiler: Intel(R) Fortran Compiler 1900
Architecture: 64 bit
Precision: single
OpenMP: No
Date: Jun 23 2021
Time: 21:25:05
Execution Info:
Date: 08/23/2021
Time: 20:53:53+0100

Syntax is:
OpenFAST [/h] <InputFile>

where:
/h generates this help message.
<InputFile> is the name of the required primary input file.

Note: values enclosed in square brackets [] are optional. Do not enter the brackets.
Invalid syntax: no command-line arguments given.
Aborting OpenFAST.
Is it the expected result?
I'm sorry for asking this, but the primary input file (for example I want to try to model the 5MW turbine), can you tell me advice me which file to try to model first to see the wind field data from the turbine? I'm a bit lost with all the modules.
Thank you so much in advance!

Or maybe can you recommend any documentation for starting modeling the 5MW wind turbine?

Dear @saksiwi,
Yes, this is the expected result if no input file name is specified.  You'll have to add the name of the OpenFAST primary input file as indicated to execute a specific OpenFAST model.  The OpenFAST primary input file typically has a *.fst extension and in this file it lists which OpenFAST modules are enabled and what there input file names / paths are.  The OpenFAST primary input file the land-based version of the NREL 5-MW baseline wind turbine is available in the r-test repository here: https://github.com/OpenFAST/r-test/blob/main/glue-codes/openfast/5MW_Land_DLL_WTurb/5MW_Land_DLL_WTurb.fst.
I hope that helps.
Best regards,

Dear @jjonkman ,
Hi thank you for your answer. I tried to put the input file, I have 2 question:

I have 5MW turbine fst file in the openfast folder that I have cloned before, should I put the openfast_x64.exe file in all the folder that I want to try to run? Sorry if it is a silly question.
I have error while running the .fst file. The output is like this:

C:\Users\dwino\OpenFAST\reg_tests\r-test\glue-codes\openfast\5MW_Land_DLL_WTurb>openfast_x64.exe 5MW_Land_DLL_WTurb.fst

OpenFAST
Copyright (C) 2021 National Renewable Energy Laboratory
Copyright (C) 2021 Envision Energy USA LTD
This program is licensed under Apache License Version 2.0 and comes with ABSOLUTELY NO WARRANTY.
See the "LICENSE" file distributed with this software for details.

OpenFAST-v3.0.0
Compile Info:

Compiler: Intel(R) Fortran Compiler 1900
Architecture: 64 bit
Precision: single
OpenMP: No
Date: Jun 23 2021
Time: 21:25:05
Execution Info:
Date: 08/23/2021
Time: 22:14:21+0100

OpenFAST input file heading:
FAST Certification Test #18: NREL 5.0 MW Baseline Wind Turbine (Onshore)
Running ElastoDyn.
Nodal outputs section of ElastoDyn input file not found or improperly formatted.
Running AeroDyn.
AD15 Nodal Outputs: Nodal output section of AeroDyn input file not found or improperly formatted.
Skipping nodal outputs.

Reading a 31x31 grid (145 m wide, 17.5 m to 162.5 m above ground) with a characteristic wind
speed of 12 m/s. This full-field file was generated by TurbSim (v1.06.00, 21-Sep-2012) on
07-Jan-2014 at 12:50:45.
Processed 1442 time steps of 20-Hz full-field data (72.05 seconds).
Running ServoDyn.
Running ServoDyn Interface for Bladed Controllers (using Intel Visual Fortran for Windows, ).
FAST_InitializeAll:SrvD_Init:BladedInterface_Init:The dynamic library
.../5MW_Baseline/ServoData/DISCON.dll could not be loaded. Check that the file exists in the
specified location and that it is compiled for 64-bit applications.
OpenFAST encountered an error during module initialization.
Simulation error level: FATAL ERROR
Aborting OpenFAST.
Do you have any idea, why this is happening? Thank you.

Dear @saksiwi,
Regarding (1), that would work, but you can also set up an environment variable in Windows so that the OpenFAST executable can be called from any directory without having to duplicate copies of, or move, the executable.
Regarding (2), there are typically one of two reasons you'd git this error:
(a) The name or path to the dynamic library is specified incorrectly in input DLL_FileName in the ServoDyn input file.
(b) The dynamic library you have is compiled for 32 bits while you are using a 64-bit compiled executable of FAST / OpenFAST.
Best regards,

@saksiwi On your second question, verify that the DISCON.dll file exists at the path given in the error message. Note that the path is relative to the ServoDyn input file, so it would be
C:\Users\dwino\OpenFAST\reg_tests\r-test\glue-codes\openfast\5MW_Land_DLL_WTurb\.\../5MW_Baseline/ServoData/DISCON.dll

If it's not there, you should copy the 64 bit DISCON controller you downloaded from the GitHub Releases page to that location.

Dear @jjonkman and @rafmudaf ,
Thank you so much for your help! I've executed it successfully and get the output. Now will read the 5MW wind turbine technical report more for the detail. Hope I can understand more. Let me know if you have other suggestions or documents to read for modeling wind turbine.


A new branch for Scala 2.13

This pull-request will address #8

Hi Ali, thanks a lot for this! This looks good, except that the commit message "update scala version to 2.12.18" should probably be 2.13.12, right? Do you think you could fix that (I'm not sure about the easiest way, either in this PR or by pushing a new one if you have to?)

Thanks for mentioning this. I should've been mistaken.
I made some changes. Could you please see if it's ok?

Thanks a lot. Looks like a lot of good style changes overall, and the branch compiles for me and tests are passing.  I added two minor comments about things I didn't understand.

Looks good, thanks!


Use context for auth access controllers

The auth package has been updated to use "golang.org/x/net/context" for
passing information between the application and the auth backend.
AccessControllers should now set a "auth.user" context value to a AuthUser
struct containing a single "Name" field for now with possible, optional, values
in the future.
The "silly" auth backend always sets the name to "silly", while the "token" auth
backend will set the name to match the "subject" claim of the JWT.
closes #137

LGTM!
We may want to place the token into the context, along with the access that is available but this is fine for now.

It's failing to find the package.

@dmp42 that's because we're waiting to merge @stevvooe's work on #86 first

@jlhawn After thinking about it, there is no reason to hold up the merge of this PR waiting on #86.
Go ahead and add golang.org/x/net/context to godep. Add a WithRequest function (I'll move this to a new context package I'm working on):
func WithRequest(ctx context.Context, r *http.Request) context.Context {
    return context.WithValue("http.request", r)
}

Update app.go to push the request into the context before calling authorized (no reason to propagate context all throughout application and request context, yet):
ctx := WithRequest(context.TODO(), r)
if _, err := app.accessController.Authorized(ctx, accessRecords...); err != nil {
...
}
// We don't really need the auth'd context yet

This will be compatible with what I'm doing now and should be equivalent to the current behavior.

LGTM!


[MM-62205] Dynamically toggle widget window visibility when popout opens/closes

Summary
PR adds some logic to dynamically change the global Calls widget visibility when the popout window is open. This is to address the inconvenience of having this window always on top even when focusing on the popout.
Hiding the widget completely is also possible, but I still feel it's a more impactful change in behavior. My hope is that the proposed changes are enough to address the reported issue. That said, let me know if you feel strongly about this.
The previous conversation for extra context is at https://hub.mattermost.com/private-core/pl/wbf4jd67g7fnpmon8fafwnxmuy
Video
https://github.com/user-attachments/assets/c58e7dee-36a4-4b6c-a043-0acfd4b7da92
Ticket Link
https://mattermost.atlassian.net/browse/MM-62205
Release Note
Calls: while the popout window is open, the widget window's visibility will change so that it is not always on top of other windows.

Nice solve @streamer45. I like this as a solution and I think it will solve the biggest part of the issue. I've triggered a build to test and play with a bit and I'll report back.

I think this feels good to me. While testing a few things I noticed (that may or may not be as a results of this PR)

I'm not sure if this is a results of this change, but when I tried to share my screen I was unable to from the popout view. I had not yet provided permissions, it did not trigger the OS permission dialog either. It just did nothing. Once I cliked the share button from the widget, the OS prompt came up and allowed me to share. However, even after allowing the OS permission from there, I was still unable to share my screen from the popout.

The widget window shows up in the list of screens to use. I don't believe it does this currently in the desktop app and we should exclude it.

I think this feels good to me. While testing a few things I noticed (that may or may not be as a results of this PR)

I'm not sure if this is a results of this change, but when I tried to share my screen I was unable to from the popout view. I had not yet provided permissions, it did not trigger the OS permission dialog either. It just did nothing. Once I cliked the share button from the widget, the OS prompt came up and allowed me to share. However, even after allowing the OS permission from there, I was still unable to share my screen from the popout.
The widget window shows up in the list of screens to share. I don't believe it does this currently in the desktop app and we should exclude it.

Thanks @matthewbirtch . I think both are unrelated to this PR.
For 1, I am honestly surprised this wasn't reported before but I'll have a look asap. Created https://mattermost.atlassian.net/browse/MM-62241
For 2, from what I remember, that's always been the case since the widget is effectively its own window. Excluding it may require us to do some pattern matching on the window title because (as far as I know) there isn't a great way to uniquely identify that specific window from the sources returned by the desktop capturing engine. I'll see what can be done. Created https://mattermost.atlassian.net/browse/MM-62242

Hmm, interesting. I tried to replicate both of these things on hub with desktop v5.10.1 and didn't get the same result.

Hmm, interesting. I tried to replicate both of these things on hub with desktop v5.10.1 and didn't get the same result.

On Linux (both 5.9 and 5.10), I can definitely see the widget being among the windows available to share. But yeah, I can't reproduce the sharing from popout issue outside of master branch so there could be something there.

I can't reproduce the sharing from popout issue outside of master branch so there could be something there.

Ok, let me know if you want to look in to this as part of this PR or not. Happy to help test

@cpoile Check https://github.com/mattermost/desktop/pull/3253/commits/59a3d939d97aadd492fac3b507d6fcf3c6b8776b and let me know if it's better. I changed the method's name and also made its argument explicit so that it's easier to understand its meaning from the calling point.


[CRT runtime] Added functions TVMPlatformPreFuncCall and TVMPlatformPostFuncCall

See this thread for an explanation.

i commented on the discuss forum thread. let's resolve there and then continue this PR.

@areusch I changed the names accordingly like we discussed on the forum.

Hi @areusch ,
It seems the problem with the CI is more deeper than the changes I made:
[2022-05-23T14:13:06.372Z] Check Jenkinsfile generation
[2022-05-23T14:13:06.372Z] Traceback (most recent call last):
[2022-05-23T14:13:06.372Z]   File "jenkins/generate.py", line 18, in <module>
[2022-05-23T14:13:06.372Z]     import jinja2
[2022-05-23T14:13:06.372Z] ModuleNotFoundError: No module named 'jinja2'`

@areusch could we not test these functions by implementing the weak symbols in test code?

Hi @areusch ,
It seems the problem with the CI is more deeper than the changes I made:
[2022-05-23T14:13:06.372Z] Check Jenkinsfile generation
[2022-05-23T14:13:06.372Z] Traceback (most recent call last):
[2022-05-23T14:13:06.372Z]   File "jenkins/generate.py", line 18, in <module>
[2022-05-23T14:13:06.372Z]     import jinja2
[2022-05-23T14:13:06.372Z] ModuleNotFoundError: No module named 'jinja2'`

Have you rebased recently? This change landed which installs that dependency in one of the scripts (it will eventually move to the image itself):
https://github.com/apache/tvm/pull/11258/files#diff-137a515f2ac93c13003517d7a4ca908dcbaeb6dff50d05470579e633ddec354fR36

@areusch @Mousius just rebased to integrate that commit

hey @fPecc it looks like the merge has got github all confused (it's still using the old Jenkinsfile to run CI which won't work). This snippet should wipe out the local branch and rebase it to just your changes:
# get the git diff of your change
git fetch origin main
git checkout crt-pre-post-functions
git merge origin/main
git diff $(git merge-base origin/main HEAD) HEAD --stat > ~/change.diff

# get to a clean branch
git reset --hard origin/main

# replay your changes
cat ~/change.diff | patch -p1 -d .
git add .
git commit -m"[CRT runtime] Added functions TVMPlatformPreFuncCall and TVMPlatformPostFuncCall"
git push --force

@fPecc i think this one needs
$ docker/lint.sh -i clang_format


changing hover for back to top button

Description
Fixes #829
Screenshots

Checklist

[x] Tests have been added or updated to cover the changes
[x] Documentation has been updated to reflect the changes
[x] Code follows the established coding style guidelines
[x] All tests are passing

@SurajPratap10 Please review this as soon as you can


There's a package install problem.

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: Vscode oss - dev githup last version
OS Version: Windows 11 Pro

Steps to Reproduce:

When I do npm intall, @parcel/watcher throws an error when installing packages.
Paths also have a problem with windows possibly getting gaps
For 3 days I have not been able to install it in both linux operating system, windows and dev container.
When I clone the project and do npm install, this comes out.
@bpasero

Are you using a OneDrive folder for your VS Code repo clone?

Are you using a OneDrive folder for your VS Code repo clone?
Yes

Please test with the latest main which got some watcher updates that doesn't use node-gyp-build


Update "Update version" from config.json to config.yaml

New Update version for config.yaml instead of json. Since the json file needs to much brackets.
@andig: may you could have a double check if the sed will be update version: instead of "version": without quotes.
THX a lot!

No idea how I should check this


how to get the count of the line in this view

i use this view. i want to get the count of line in this view. How can i do?
in this picture, it has 2 line of textview. how can i get the  number 2?

At this moment, the API to retrieve the line count of the FlexboxLayout.
But the requirement sounds reasonable to me. I'll keep this issue open and convert it as a feature request.

ok. thanks

FlexboxLayout#getFlexLines() method was added in #102.
I'll release the updated version of the library anytime soon.

I need to see to!


remove folding and unfolding of batch dim and sequence dim in model.py

Stack from ghstack (oldest at bottom):

-> #190

Local experiments show that this does not slow down training. Actually it brings slight memory saving and speedup.
Experiment setup: devgpu, dp-degree=2, tp-degree=4, llama-13b, full AC
Before:

After this PR:

Discussing with @tianyu-l offline still: this PR means that we have extra cat after all-gather and reduce-scatter since we are no longer gathering/reducing on dim-0.

@awgu I am also curious why adding extra cat here would make perf better. I remembered in the past, when doing fold and unfolding, we did see perf gain since all-gather and reduce-scatter is performing on dim-0 which makes use to use all-gather-base and reduce-scatter-base. Have we checked the profiler to see what collectives we are calling @tianyu-l ?

After discussing with @awgu, I think the perf testing here is not convincing. Looking at the profile traces right now I do see it introduces extra cat.
Although, combining #314 with the no-folding way, we turn the beginning embedding communication from an all-reduce to a reduce-scatter. But as the number of layers grow, this save should matter less and less.
The PR was mainly to make distributed techniques not intruding model code, with the understanding that we probably should do it even with minor perf regression.
cc: @fduwjj

@tianyu-l Is it possible for the folding and unfolding to be part of the SequenceParallel API as an option (somehow)?

@awgu SequenceParallel is at layer level, I think we need to do the folding and unfolding outside all the layers, to avoid doing it num_of_layers times.

@tianyu-l I think what you said:

But as the number of layers grow, this save should matter less and less

This folding-unfolding happens every layer (for every atten and mlp layer) basically for every all-gather that happens for non-0 dim and then if torch.cat is called. we will see this perf save.
To anwser @awgu's ask, maybe this can be done at c10d level. I remember we used to let all_gather_into_tensor to handle it. But it has correctness issue. Because when a 3-D tensor is sharded by dim 1 for example, then techinically if you all-gather by dim 1 and cat by dim 1 the result is different compared to if we do folding -> all-gather by dim 0 -> unfolding. It's actually very surprising that this works for LLaMa training.

update: created #437 to showcase how to do folding and unfolding to accelerate Sequence Parallel, with no plan to land it (added a pointer in #438)

@fduwjj yeah when I say

But as the number of layers grow, this save should matter less and less.

I was referring to my previous sentence

Although, combining https://github.com/pytorch/torchtitan/pull/314 with the no-folding way, we turn the beginning embedding communication from an all-reduce to a reduce-scatter.

This folding-unfolding happens every layer

I guess you wanted to say as the number of layers grows, the save from doing collectives on dim 0 (thus avoiding aten.cat) happens on every layer, which I agree. (Technically speaking, the folding and unfolding only happens at the beginning and the end of an iteration, respectively.)


After few hours, chaind gets stuck

After having chaind running ok for few hours it gets stuck:

The cpu consumption goes to zero and stops synching.
No logs are produced in debug mode anymore.
t_metadata shows that there are some tables still not in sync.
I managed to reproduce this issue multiple times, it usually happens after 8 hours.

Restarting it fixes the issue, but perhaps there is deadlocking situation not so difficult to fix. I tried running a pprof with --profile-address but I'm not versed in profilers, so not sure where to look into. Wouldn't mind helping to debug and fix the issue with some assistance.
Thanks!

Which version of chaind is this with, and how is it run?  Also, which  beacon node is this using, and what are the beacon node parameters?
Most of chaind data gathering is triggered from receiving head events from the beacon node, so that would be a good place to start.  In your chaind.yml file please can you set the following:
eth2client:
  log-level: trace
blocks:
  log-level: trace

Note this will generate a lot of log entries, but we're only really interested in what happens once chaind gets stuck.  When it stops processing see if there are any log entries produced.

chaind version: 97df14a
chaind parameters, quite vanilla setup:

--eth1deposits.enable=true
--chaindb.url=xxx
--eth2client.address=xxx
--log-level=debug
--eth1client.address=xxx

beacon-node:

teku v21.12.1
--data-storage-mode=archive
--data-storage-archive-frequency=64
fully synched and plenty of cpu/memory allocated.

As discussed with other chaind users, other people are reporting the same issue. Not sure about the version or configuration they are using though.
Will enable those flags and let you know. Thanks for the help.

Has this problem shown up again?

It occurs ~weekly I would say and gets resolved with a restart. This behaviour has been consistent across all the recent versions (including 0.5.4). Our setup is using chaind with Lighthouse. My guess would be a rare race condition that leads to a deadlock either in PG or chaind itself. I have a ticket to check Postgres for clues next time this occurs.

Yes, in my case I would say it was every 2-3 days with the configuration above. As you say, restarting solves the issue. I ended up setting a liveness probe to fail every day or so that it was automatically restarted. Tried debugging the issue with a profiler but didn't get very far.

Have there been any more hangs?  I have been running chaind with lighthouse for the past 4 weeks and not seen any here.

I'm not running it, can't tell.

I haven't seen a dead hang in a couple of weeks. What does happen though sometimes is t_validators isn't updated, which is likely due the chain head listener not picking up new events.  I have the profiler on and will investigate when it happens again.

We may be able to close this one soon. We have tracked down this issue to some queries being occasionally blocked in Postgres (which leads to a deadlock). @mcdee has submitted a fix for this https://github.com/wealdtech/chaind/commit/36338be9de64f59a436459404a1700c6a86e4ece
We will need a few days to confirm but so far it looks good.

We haven't detected this problem since the last round of fixes was released.

Great to hear, I'll close this now and if you see it happen again feel free to reopen.


freezing

atom freeze after start and i tried atom --safe and still the same

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:


E2E Critical White Spots

Critical white spots should be implemented with e2e tests

Relates to #4131

@eliobischof as discussed, scope from b2b demo. everything that needs to be clicked in console to setup/teardown.


Add the ability to specify additional connection options

For example, if you need to specify ssl: { verify: false } in order to get
around a cerificate problem (which I don't recommend, but might be a stopgap),
you may specify when configuring:
config.connection_options = { ssl: { verify: false } }

This will accept any of faraday's options. See https://github.com/lostisland/faraday/blob/master/lib/faraday/connection.rb
I didn't see a changelog, so I wasn't sure how you wanted to handle what might be a breaking change for some people. In theory, it shouldn't break anything--and should just make everyone a little more secure--because the certificates should be valid, but there's a chance there's some certificate out there that isn't.
Also, I looked but didn't see any examples of your preferred hash syntax, so if it needs to support older =>, that's an easy change. And I'm not fluent in minitest (I spend most of my life in rspec), so if there's a preferred idiom, I'm happy to change it.
Addresses Issue #3.

This looks good, thanks!  I will get this merged and push out a release sometime soon.
Also changelog is a great idea.  I will add that and attempt to add the changes from previous releases as much as I can as well as some notes for this change.  Otherwise I was trying to use semantic versioning as a reasonable approximation of when there might be a breaking change for users.
As far as hash syntax goes, we don't have any requirements for that since we only support back to ruby 1.9.3 which has both hash syntaxes anyway.

@jhanggi just released this change as v2.1.0.  Thanks for the PR! 🎉 🎉 🎉


Switch to official geoserver docker, use wps extension

This switches to the new official geoserver docker image.
Non default passwords will be set manually once on the target servers.
@hblitza @JakobMiksch please review

As discussed in the last meeting - works for me, thanks!


Initial port of trafico over to actual/actions

Moving to fork: https://github.com/actualbudget/trafico/pull/1


Build failed - ld: framework not found Pods

Actions to get fail:
On build

tns platform add ios
tns build ios

Warning:
Pod installation complete! There is 1 dependency from the Podfile and 1 total pod installed.
[WARNING]: [!] CocoaPods did not set the base configuration of your project because your project already has a custom config set. In order for CocoaPods integration to work at all, please either set the base configurations of the target `iosfrases` to `Pods/Target Support Files/Pods-iosfrases/Pods-iosfrases.debug.xcconfig` or include the `Pods/Target Support Files/Pods-iosfrases/Pods-iosfrases.debug.xcconfig` in your build configuration (`iosfrases/build-debug.xcconfig`)

On XCode

Go to project on XCode
Build on a real device
Build does not complete, fail and give this message:

ld: framework not found Pods_iosfrases
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Workaround
I delete this file on Frameworks > TNSWidgets.framework > Pods_iosfrases.framework, it works.
Project Info
Nativescript<EMAIL_ADDRESS>CocoaPods: /Library/Ruby/Gems/2.0.0/gems/cocoapods-1.0.0/lib/cocoapods.rb
XCode: Version 7.3.1
Plugins I am using:

"nativescript-admob": "^1.1.5",
"nativescript-clipboard": "^1.1.4",
"nativescript-openurl": "^1.0.2",
"nativescript-social-share": "^1.3.0",
"tns-core-modules": "^2.2.1"

I found the erro. I was openning the project by the wrong file.
This is usually caused by having the .xcodeproj file open instead of .xcworkspace.
When you run 'pod install' for the first time, it will create an .xcworkspace file, which includes your original .xcodeproj and a Pods project. You'll need to close your .xcodeproj and open the .xcworkspace instead.
This is a common issue when creating a project through Xcode's new project wizard - I often forget that I'm not in a workspace, which is required to get Cocoapods to link correctly.


Updated my Project in Cqube

Project
Input File Validator
Changes
Added a new section under the Cqube project.
Notes
If there are any further notes.

yarn lock and package lock needs
to be reverted

yarn lock and package lock needs to be reverted

so what should i do for it

just remove them and commit again that it

just remove them and commit again that it

ok


Resolves #72

Allowed the table part of the path to be customised in the event listener, also added appropriate test and documentation.

Coverage increased (+0.97%) to 90.82% when pulling 48a91f6ce5726b3807dc520f6697ed9880e4bbaa on feature/path-table into 703ba1949106700e24c7cc5f493df657915d3ba6 on develop.


Wasm tests: add typed-funcref test showing example of desirable optimizations.

In order to have fast IC (inline cache) chains in AOT-compiled dynamic language Wasms, it would be great if we could make the "call to a typed funcref at a constant table index" pattern fast.
This use-case was discussed at the most recent Wasmtime biweekly and @jameysharp is working on some optimizations; the intent of this PR is to provide a concrete test-case whose blessed output we can see improve over time.
In particular, the following opts are still desirable:

With the use of non-nullable typed funcrefs, there shouldn't be a null check (there currently is, as noted by a comment in the code due to lack of type information at the right spot).
With the use of a constant table size and a constant index to the table.get, we should be able to load from the table without a bounds-check or any Spectre masking.

Other further optimizations for this pattern might be possible if we rearrange the table and function-reference data structures, and the lazy-initialization scheme thereof, but the above should be agnostic to that.

(table $ic-sites 100 100 (ref $ic-stub) (ref.func $ic1))

This example module initializes element zero of the table to point to a function

Oh this syntax is actually initializing the entire table with (ref.func $ic1), it's new syntax/binary stuff introduced as part of the function-references proposal (basically folded into the gc proposal)

Somewhat orthogonal question as well: I forget if I asked this in the past, but is the reason to use a typed table instead of typed globals to get the lazy initialization? If I change the above test to use two typed function globals it ends up codegenning (today)
function u0:1(i64 vmctx, i64, i32, i32, i32, i32) -> i32 fast {
    gv0 = vmctx
    gv1 = load.i64 notrap aligned readonly gv0+8
    gv2 = load.i64 notrap aligned gv1
    gv3 = vmctx
    sig0 = (i64 vmctx, i64, i32, i32, i32, i32) -> i32 fast
    sig1 = (i64 vmctx, i32 uext, i32 uext) -> i32 uext system_v
    sig2 = (i64 vmctx, i32 uext) -> i32 uext system_v
    stack_limit = gv2

                                block0(v0: i64, v1: i64, v2: i32, v3: i32, v4: i32, v5: i32):
                                    v8 -> v0
                                    v14 -> v0
@003b                               v9 = load.i64 notrap aligned table v0+80
@003d                               brif v9, block3, block2

                                block2 cold:
@003d                               trap null_reference

                                block3:
@003d                               v10 = load.i64 notrap aligned readonly v9+16
@003d                               v11 = load.i64 notrap aligned readonly v9+32
@003d                               v12 = call_indirect sig0, v10(v11, v0, v2, v3, v4, v5)
@004c                               v15 = load.i64 notrap aligned table v0+96
@004e                               brif v15, block5, block4

                                block4 cold:
@004e                               trap null_reference

                                block5:
@004e                               v16 = load.i64 notrap aligned readonly v15+16
@004e                               v17 = load.i64 notrap aligned readonly v15+32
@004e                               v18 = call_indirect sig0, v16(v17, v0, v2, v3, v4, v5)
@0057                               jump block1

                                block1:
@0052                               v19 = iadd.i32 v18, v12
                                    v6 -> v19
@0057                               return v19
}

which is pretty good given that we've got https://github.com/bytecodealliance/wasmtime/issues/5291 in our pocket

Thanks Alex, now I understand!
I'd forgotten about #5291; we just added a different use of memflags to indicate trap code so we have precedent now. But besides that, if I understood Chris correctly, this example is supposed to be typed as a non-nullable function reference. So if we thread that type information through correctly then we should be able to elide the null-check even without #5291, right?

Ah so the call_ref instructions are always done with a constant index, but the table.set instructions are done with a variable index? (if I'm understanding you right). I mostly wanted to confirm that all call_ref is done with a constant index or otherwise this codegen wouldn't be representative.

So if we thread that type information through correctly then we should be able to elide the null-check even without https://github.com/bytecodealliance/wasmtime/issues/5291, right?

True!
Thinking a bit more on this, if the table.get plus call_ref pair were replaced with call_indirect we wouldn't have to solve the problem of propagating validator type information into lowering. In such a situation we could inspect the table's type and notice that it's non-nullable and skip the null check. I think it should be the case that table.get plus call_ref should be the same codegen as call_indirect at least ...

This has inspired me to go off and do https://github.com/bytecodealliance/wasmtime/pull/8159 to handle the null check

Ah so the call_ref instructions are always done with a constant index, but the table.set instructions are done with a variable index? (if I'm understanding you right). I mostly wanted to confirm that all call_ref is done with a constant index or otherwise this codegen wouldn't be representative.

Right; the hope/aspiration is that that constant index can trickle through a carefully-placed series of optimizations so this turns into something closer to the good codegen we see with globals.
(Two other points that came to mind later re: use of tables: scalability is another factor -- the max of 100k (?) globals is a real limit if we use one for every IC site in a large program; and also, your point about lazy init, wherein we don't lazy-init globals and that'd be a big regression of instantiation latency on said ~100k-IC programs. The flipside is that we have the lazy-init dynamic checks on table load...)

call_indirect instead

Ah! I hadn't realized that call_indirect also participated in the "typed funcref" proposal; for some reason I had it pegged as "old dynamic sig check" territory. All the better if it lets us apply the optimizations more easily; I'll update this test to use it.

Thinking a bit more about lazy init: the overall behavior we want is that for a very large program with many ICs, we have fast instantiation and, once ICs are warmed up, fast IC invocation with as few dynamic checks as possible.
IC sites always start as linked to the "fallback IC"; my planned use for this "fast IC head" mechanism was to have a conditional at very IC site testing whether the IC-stub struct is a fallback IC, invoking a traditional (C++) function pointer if so, and then (hitting a weval intrinsic that leads to) doing this typed-funcref thing if not. The upshot of that is that we always have some slowpath action (attaching an IC) before we do the typed funcref invocation.
I guess what I'm getting at is: could we avoid the lazy-init checks if we had a nullable typed funcref table, with null as the default value? Then we do the usual init-the-anyfunc-before-you-take-its-address thing at the ref.func in the IC-attach path, and the just-instantiated state is a bunch of zeroes (kindly provided by mmap/madvise), and the invocation is load/load/call-reg. That's another new optimization but a special case for lazy-init instead. Stated succinctly: if a table's default funcref is null, don't do lazy-init, and make the 0 bit-pattern mean null rather than non-initialized-pointer-to-anyfunc.

Makes sense! It's always possible we can use the same optimization techniques on globals as well as tables, nothing saying we have to keep everything as-is for example. If tables work then there's no need to change though.

could we avoid the lazy-init checks if we had a nullable typed funcref table, with null as the default value

I like this idea!


$.live() may be has some problems.

I don't know hou to descripe in English...
使用dog初始化，然后使用 $.var(true) 进行绑定。
但是当我修改dog中的变量的值的时候，视图并没有立即刷新，当我第二次修改才会刷新。
Translate by Google:

Use dog to initialize, and then use $ .var (true) for binding. But when I modify the value of the variable in dog, the view does not immediately refresh, when my second change will refresh.

Maybe the gif can let you know what i said.

Here is the code:

My dog's name is

It is  years old now!

&
var dog = {
    name: 'doge',
    age: 12
}
var $ = new DisplayJS(dog)
$.var(!0)

i hope you can understand what i mean.

Ok, so of what I understand, when you change the dog variable, the $.live() function is taking one change before the last one. Tell me if I’m right 😊. But, just another question, why are you using $.var(!0) instead of $.var(true)?

Almost the same meaning.
!0 === true and  !1 === false , Do not you think it's so cool?

@JZLeung Yeah, it's cool 😎, I didn't know that. But, I already noticed this, but I don't have any current solution for that if you have one, you can do a PR, and If it works, I'll accept 😊.


simulation - plan_go_to Requires an Extra t Parameter in cffirmware, Causing Compatibility Issues

Discussed in https://github.com/IMRCLab/crazyswarm2/discussions/585
We need to update the SITL layer to support the latest firmware.

Not only the plan_to functionality need to be updated, the spiral function needs to be added too. I'll make an issue in the crazyflie-firmware


node-inspector reattach/refresh on source change.

I am trying to use node-inspector/chrome-dev-tools as an editor (with an awesome debugger).
And it kind of works. My setup is:

node-inspector --save-live-edit --preload=false
nodemon --debug app.js
one window for navigating the app and one for http://<IP_ADDRESS>:8080/debug?port=5858

The first annoyance that I find is that when I make a change which triggers
nodemon to restart which in turn triggers node-inspector to refresh/reattach
the http://<IP_ADDRESS>:8080/debug?port=5858 page
which takes a while (3-4 seconds) in which I can't see the script (and I don't like this).
The other is that when a syntax error happens nodemon crashes (awaiting for script changes) and node-inspector doesn't reload the debug page (http://<IP_ADDRESS>:8080/debug?port=5858) meaning the changes that would fix the syntax error can only be made via an external editor.
Is there a way around those two things?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

+1
Any updates?

enter rs to restart
easy quit!!
1.node-inspector
2.nodemon --debug-brk  app.js
3.one window for navigating the app and one for http://<IP_ADDRESS>:8080/debug?port=5858

So glad to have found this and quick update --debug-brk is now deprecated must use --inspect-brk
nodemon --inspect-brk app.js
works like a charm!


Updated concepts/distribution file

Updated link per customer feedback in concepts/distribution.md.

Docs Build status updates of commit 4043dc6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/concepts/distribution.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Updated per feedback in  #1023.

#sign-off and merged.


Metricbeat daemonset not reloading on configmap changes

Chart version:
7.7.0
Kubernetes version:
1.18.2
Kubernetes provider: E.g. GKE (Google Kubernetes Engine)
CAPI
Helm Version:
3.1.2
Describe the bug:
If i change the metricbeat config and do an helm upgrade, the metricbeat configmap gets changed, but the metricbeat daemonset and deployment never restart to grab the new config.
Steps to reproduce:

install metricbeat via helm
change for example the output server
the daemonset stays the same

Expected behavior:
The daeomset and deployment get restarted to use the new configmap

This is due to https://github.com/elastic/helm-charts/blob/fe52a62639031d8988e2ce4cc12479c3e85e2b69/metricbeat/templates/daemonset.yaml#L140
As stated here ( https://kubernetes.io/docs/concepts/storage/volumes/#configmap ): Note: A Container using a ConfigMap as a subPath volume mount will not receive ConfigMap updates.

Fixed by #634