React-native support

Hello all and thanks for this very cool lib. It really seems like something we could use in our company, There's just one potential problem. We need to render the content on mobile devices with react-native. Theoretically, I could probably render it to a web view. But that comes with a lot of drawbacks and I would like to stay away from that. That's why I wanted to ask if there is any support for react-native (and/or react-native-web) existing or planned. If not, I might look into implementing something along the lines of the existing HTMLRenderer. But I would definitely appreciate some input on this first as chances are it can end up being a lot of work. Thanks

Hi, there are currently no plans to support react-native and also no existing work. Contributing something along the lines of HTMLRenderer would probably be the best idea!

Ok cool, good to know. I will see what the time allows. Currently we have a markdown solution which I am not very happy with. But implementing rendering for react native could take a while. I guess maybe keep this open or move to suggestions/feature requests if you have that. I will come back here if I have any progress. Probably in the following few weeks.

@PeterKottas We already wrote a renderer that outputs a table layout (for emails) instead. That was rather easy by forking https://github.com/ory/editor/blob/master/packages/renderer/src/index.js. You can probably do something similar by outputting React Native stuff here. Only complication I see: you probably need two plugin definitions for every plugin. One for editing (that uses HTML since I don't think that the editor will work in a React Native setup without major changes), one for rendering (that uses React Native)

@inyono good stuff mate. It rly seems it might not be that hard after all. Good point about the plugins though. To be fair, I have no idea right now what plugins I will want to support. I only looked at this lib recently so I will need to get working knowledge of it. By any chance, is the package you created for emails open sourced? Cause we might actually use this for emails as well. Just thinking out loud tbh

Just updating this thread with some new findings.
The layout plugin for RN would indeed be pretty straight forward.
Also the plugins in general would work imho. We would just need to find a clever way of switching between different components based on the platform. That require react-native package so we wood need some service that would return HTML be default if RN is not installed as peer-dependency. Again pretty straight forward.
However, the biggest problem and possibly a showstopper is the slate plugin. I in my naivety totally underestimated the size of it. slate.js apparently doesn't work with react-native even when there's some progress on this tracked here https://github.com/ianstormtaylor/slate/issues/1374, it still appears they will only internally wrap it in web view. That means that we are likely screwed when it comes to native support.
The options are either switching slate for something that supports react-native (not aware of anything just yet), wrapping the whole thing in webview, or letting slate finish the work on this and wrap theirs in webview while here, we do things properly.

Yeah slate will not work with native UI, it's completely different from native and wrapped around the DOM and the shitty contenteditable. Usually you're not writing rich text on mobile anyways...

Yeah, that's right, it wouldn't have to be editable. But my understanding is (might be wrong) I still need to render the whole editor even when I am in readOnly. Right? And that will mess up RN. For reference, this is the file where  is rendered.  in this context is the slate.js editor which is not compatible. No matter if it's readOnly or not. The only way would be if there was something like  that would accept the state and only render. Hmm ... that actually might be doable.

Right, the two models might not be compatible without a webview

Forgot to send the link to the file before but it's hardly necessary, just for completeness sake : https://github.com/ory/editor/blob/master/packages/plugins/content/slate/src/Component/index.js
Anyways I'll keep this in mind and check how hard would it be to create a light-weight standalone react-native renderer for slate. That would fix most of the problems.


Have you considered using DeepL's free windows integration?

So DeepL supports capturing from applications, translating directly in applications, and piping text back and fourth, it also supports screen capture and ocr.
As someone who uses it to communicate with a massive japanese team on the daily, it is by far the best and most accurate translation. And, it's completely free for the kind of use this application would be spitting out.
It's paid options are for team translations documents and cat tools, all of which you don't need.
I'm planning on using your tool to give myself jp closed captions in game (i'm less interested in the TTS portion)
The azure requirement is a huge turn off for many, and to be honest it's translations aren't even all that good in comparison, especially for Japanese.
This would create a completely free option for users that works with what is considered the best tool to work with japanese users.

@Lhun This is a fantastic suggestion since azure "text translation" isn't free which is why you can not translate TTS in the main app. I have begun testing the API on the lite version of the app https://github.com/VRCWizard/TTS-Voice-Wizard-Lite/releases/tag/v0.1.1 which you can also test here (keep in mind the lite version is missing many features added in the main app such as setting saving for example). I plan to include this in the main app in a future update.
Side note: Kill Frenzy Avatar Text seems to be missing many Japanese symbols I am not sure if these symbols can be converted into other symbols that are included but as of right now any symbols not in KAT will appear as "?" in game. For example the highlighted symbols in this picture ("声","出","読") will appear as "?" in VRChat

If you have any insight into the issue let me know, Here are the symbols supported by KAT

Actually, I commented there first because I recognized this issue as well. I tested the tts with a native speaker using the azure mode and it ... wasn't well received.
I made a comment on that particular issue with KFAT as well, s-silent is also engaged on this it seems.
https://github.com/killfrenzy96/KillFrenzyAvatarText/issues/6


Fix docker.save tar targets

Please provide a description of this PR:
Similar to https://github.com/istio/istio/pull/35761 if building non-dockerX targets with DOCKER_BUILD_VARIANTS=debug distroless default builder attempts to save <REPO>:<TAG>-default tar which doesn't exist

@sergii-ssh: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
a5af3b4b4287b3e44ce7c6e137adbbd83874b2c4
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

Verifying this works with DOCKER_V2_BUILDER=false make docker.save.

@sergii-ssh: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster-tests_istio
a5af3b4b4287b3e44ce7c6e137adbbd83874b2c4
link
unknown
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

Note: https://github.com/istio/istio/pull/36061 is needed to fix this with current commits which no longer use the Makefile logic unless DOCKER_V2_BUILDER=false

/cherrypick release-1.12

@sergii-ssh: #36041 failed to apply on top of branch "release-1.12":
Applying: Remove -default rather then DEFAULT_DISTRIBUTION
Using index info to reconstruct a base tree...
M	tools/istio-docker.mk
Falling back to patching base and 3-way merge...
Auto-merging tools/istio-docker.mk
CONFLICT (content): Merge conflict in tools/istio-docker.mk
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Remove -default rather then DEFAULT_DISTRIBUTION
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherrypick release-1.12

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@sergii-ssh: new issue created for failed cherrypick: #36072

In response to this:

/cherrypick release-1.12

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


ISSUE-3438: Program to find the determinant of a matrix

Why:
This PR implements ruby program to find the determinant of a matrix
Closes #3438
What's being changed:
program/find-determinant-of-a-matrix/FindTheDeterminantOfAMatrix.rb is added.

Hi @grraghav120
Please review when you're available :)

Hi @grraghav120
Any update on this?


Some important links should be added.

The add-in lacks the links some useful links.

I am working on this.


Minor bug: docs/core/proto-docs.md is not updated to proto change

Ever since the decision to move to tendermintdev/sdk-proto-gen:v0.7 in makefile, proto-docs.md is not generated even if proto is changed. I have tried this changing proto and proto-docs.md is not generated.
This is contributed to:

no rule for protoc-gen-doc like this in bug.gen.gogo.yaml

if we decide to include generating proto-docs.md feature again, we should move to tendermintdev/sdk-proto-gen:v0.3

cc @alexanderbez

Closed by #2834


B92 pm suspend

The following west manifest projects have been modified in this Pull Request:

Name
Old Revision
New Revision
Diff

hal_telink
https://github.com/telink-semi/hal_telink/commit/60ad4bf3940d6b8b9bb6c2ab933d931e9c90b74c
https://github.com/telink-semi/hal_telink/commit/9b3da6dd4b53f5867a2161b38faa503deb20d525 (telink_B92)
<EMAIL_ADDRESS>

Note: This message is automatically posted and updated by the Manifest GitHub Action.


Unpin gfortran

Trying to fix #1

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.


Layers support & Input Data issues

First of all, thanks for providing RTNeural. It is a really elegant way to get a model running in the audio c++ world!
I got two questions:

Is there any chance to see Conv1D Transpose and transformer Layers support in the future? It's quite a complex architecture..

The other one is probably a beginner question. I have a model with an input size of 64 Samples. Is there any way how to put in 64 Samples at a time and get 64 Samples back? The RTNeural examples and NeuralPi are all using an input size of 1

Hello! Glad you're enjoying the library.
For question 1, having more advanced layers would definitely be cool. I don't know much about the two layer types you mentioned here, except as high-level concepts. Would it be possible to share some articles about them, or (even better) some PyTorch or TensorFlow code that uses them in a neural net (preferably audio-related).
For question 2, you should be able to do that sort of thing like this:
float testInput[64] = { 0.0f, 1.0f, 2.0f, ... };  // create the input data
model->forward (testInput);                       // pass a pointer to the input data to the `forward()` method
float* testOutput = model->getOutputs();          // get a pointer to the output data

Hey, thanks for your quick & helpful response! Your work is really impressive :)
Regarding 1: I will go through some papers I read and send it to you the following days!
About question 2: I followed your example code, but get strange results (some float values > 700) and the python model outcome is completely different.
For test purposes, I am loading the model at run-time
Therefore, I declared the model like in my PluginProcessor.h :
std::unique_ptr<RTNeural::Model<float>> neuralNet[1];
I used this to load the json file:
void PluginProcessor::loadModel(std::string pathToJsonFile)
{
    this->suspendProcessing(true);

    std::ifstream jsonStream (pathToJsonFile, std::ifstream::binary);

    auto jsonInput = nlohmann::json::parse (jsonStream);

    neuralNet[0] = RTNeural::json_parser::parseJson<float> (jsonInput, true);

    neuralNet[0]->reset();

    this->suspendProcessing(false);
}

And this is how I am feeding the data into to model (I am using a buffer size of 64 in my DAW)
void PluginProcessor::processBlock (....)
{

     float testInput[64] = { };

     //filling in the audio data to testInput...
     ....

     neuralNet[0]->forward (testInput);
     const float* testOutput = neuralNet[0]->getOutputs();

     //writing the model output to audio buffer...
}

Is there something I'm doing wrong? Does it make a different if I declare the model architecture at compile time (except performance)?

Hmm, the code in your post seems like ti should work... Would it be possible to provide an example JSON model file, as well as an example input and output? That would be useful for debugging.
The run-time and compile-time models should give identical results, the only difference besides performance is in how the models are loaded. For the run-time models, the model architecture is determined based on the contents of the JSON file. With a compile-time model, the user defines the model architecture in the code, and if it does not match the model size defined in the JSON file then the output will be incorrect. In both cases, any errors in the model loading process should be printed to the console. Would it also be possible to share the console output coming from the call to RTNeural::json_parser::parseJson<float> (jsonInput, true);?

thanks again for your help! I really appreciate you taking the time to help me with this problem
I just invited you to a private git repository, where I uploaded the json File, example audio input/output files (generated by the python model) and example input of an array of one's (note that this model is using now an input size of 144 (of course I updated the JUCE Plugin to this input size).
Note: don't be irritated by the speaker of the audio files, I had the audio data lying around and then I used it as training data ;)
The RTNeural Debug printout while loading the model looks like that (should be fine)
# dimensions: 1
Layer: conv1d
  Dims: 18
Layer: conv1d
  Dims: 18
Layer: conv1d
  Dims: 36
Layer: conv1d
  Dims: 36
Layer: lstm
  Dims: 32
Layer: dense
  Dims: 1

I know that the model is quite huge - I will definitely reduce the sizes of the layers at some but, it's just for a first prototype
According to your last hint of passing in the right input_size:
void PluginProcessor::processBlock (....)

                //monoBuffer.numofSamples --> 144
                float testInput[144] = { };

                for (int n = 0; n < 144; ++n)
                {
                    testInput[n] = monoBuffer[n];
                }

thanks for that hint! As I have experienced, it's not ethical to change the last index of the data tensor to 144. Instead our data tensor has the shape of :(batch_size = 4096, timesteps = 144, features = 1). Do you have any successions to proceed from here?
I upload the python training script on the private github repository as model.py. But here is also an overview about the model:
model = Sequential()
model.add(keras.layers.InputLayer(batch_input_shape=(4096, 144, 1)))
model.add(Conv1D(18, 12,strides = 1, activation=None, padding='same', name = 'Convolution1'))
model.add(Conv1D(18, 12,strides = 1, activation=None, padding='same', name = 'Convolution2'))
model.add(Conv1D(36, 12,strides = 2, activation=None, padding='same', name = 'Convolution3'))
model.add(Conv1D(36, 24,strides = 2, activation=None, padding='same', name = 'Convolution4'))
model.add(LSTM(32, name = 'LSTM_Layer', stateful = True))
model.add(Dense(1, activation=None, name = 'Dense_Layer'))
model.compile(optimizer=Adam(learning_rate=learning_rate), loss='mse', metrics=[error_to_signal])
model.summary()

Ah, that's interesting. The batch size shouldn't matter, since I'm assuming you only want to run one batch at a time when doing inferencing. The "timesteps" dimension is a little more interesting... I'm guessing those are consecutive samples in time? It looks like when you run the model in Tensorflow, the network output is an array of 144 values.
If I'm understanding everything correctly, then I think the correct implementation with RTNeural would be something like this:
void inference(float* output, const float* input) // both input and output are arrays of length 144
{
    for(int i = 0; i < 144; ++i)
    {
        output[i] = model.forward(&input[i]);
    }
}

Hmm yeah, I'm a little bit stumped by this one. I don't think I've tried training a network before where the input goes directly into a convolutional layer, without going through something else first. (Frankly I'm using recurrent networks much more than convolutional these days anyway.)
The fact that the LSTM layer has a 2D shape, rather than 3D is a little bit off-putting. I wonder if there's some internal "flattening" happening there.
One other thing I noticed is that the Conv1D layer in your script is using the argument padding='same', whereas the layer in the example script is using padding='causal'. I'd have to review the RTNeural implementation, but I believe I had intentionally set up the layer to expect causal padding given the expectation that it would be used in real-time context. If that is the case, it's definitely my fault for not properly documenting it. I'll take a look and see if I can confirm if that is the problem.


feat: allow blobName to be set dynamically with msg.blobName

Fixes: #1
This PR adds the ability to set blobName dynamically by passing in a value via msg.blobName. This allows you to upload multiple files at once, setting the name for each file dynamically.
I've also added a test that ensures that uploadData is called when blobName is present and mode is set to binary, as well as several tests that verify that blobName can be set dynamically by passing in msg.blobName.

Thank you for your feedback, the initial thought was to allow users who might require simple usage to use the node.
Of course, advanced improvement is always welcomed.

Thank you very much!


Is this project still maintained?

Hi,
Forgive me for asking but is this project still maintained?
I've been looking into stable HTTP/2 solutions within Crystal for the Invidious project and this seems like the best candidate.  Though the last meaningful commit is from all the way back in 2021.
Duo passes all of the HTTP specs, so I assume it still works. But I'd just like to know if this project is still open to any bug fixes we might discover when integrating it into Invidious.
Thanks in advance
Syeopite

Hello @syeopite,
The project is not actively maintained, and would be happy to collaborate to help you use it with the Invidious project. Please let me know how I can help, I will be available for any bug fixes.
Sincerely,

Thanks for maintaining duo! I'd be sure to let you know if I discover any new bugs.
For now though I'm encountering the same problem as #2 when trying to use the client.


add python builds to Dockerfile

I've successfully followed your doc and could run the eservice python tests
test-contract.py
test-request.py
test-secrets.py

In order to have a development environment, I thought I'd extend your Dockerfile to include building the python stuff up to running the tests above
Here's my try: https://github.com/encointer/encointer-sdk/raw/master/docker/Dockerfile.pdo-dev
Unfortunately I run into a supposed docker bug as the image grows indefinately during build (after building the "pdo/common" layer) until it fills up my partition. That bug is out of scope here, but do you have any plans in completing your Dockerfile.pdo-dev and yaml so developers could start to implement contracts directly?

@brenzi i don't think you failed due to a docker bug but the build of common generates very large test files.  See pull request #101 for a second dockerfile which also builds pdo code itself and deals (arguably a bit crudely with that test file).  Note, though, that this will with current build process work only for simulator mode.
For the bigger question regarding docker files and having end to end docker-compose files.  While that certainly could be interesting, the way pdo and sawtooth build and also don't necessarily match in base OS requirements, it's not immediately clear how best to do that.  Also so far no concrete requirements of such and intended uses were clearly articulated.  Any thoughts on this are very welcome!

Thanks, that pull request might work for me, I'll check that.
I'll articulate my requirement, but I'm open to alternative approaches:

docker-compose <some yaml file> up

this should start containers with validator, pdo transaction processor (pdo-tp), pdo dev shell (pdo-dev), set up transaction families for pdo, register pdo tp
my first attempt is here (very close to sawtooth example): https://github.com/encointer/encointer-sdk/blob/master/docker/sawtooth-pdo.yaml

I'd like to docker exec -it pdo-dev bash and run above PDO tests successfully
Then I'd like to use pdo-dev to develop my own Gipsy PDO contracts

The above should solve the problem of different ubuntu dist requirements  for sawtooth (16.04) and PDO (18.04)
Is there a simple way to mock sawtooth and just develop PDO contracts?

Just wanted to add that the above indeed worked. If this is of help for anyone, I completed a docker setup to run PDO tp (running on xenial) and PDO eservice tests or develop your own PDO contracts (running on bionic):
https://github.com/encointer/encointer-sdk

@brenzi can you provide a review for the docker PR that @g2flyer pushed? i don't have a setup for testing.

@g2flyer's commit worked for me. I just extended his script to include all python stuff

@brenzi thanks for your feedback. I'll merge the docker PR. If you want to be more formally part of the project (submit/review PRs), just let me know.

@cmickeyb  thanks for your offer. As I'm currently working with PDO for my project , I guess there might be pull requests from my side soon. So yes, I'd appreciate it if you could add me to the project. I'd be happy to contribute.

Just wanted to add that the above indeed worked. If this is of help for anyone, I completed a docker setup to run PDO tp (running on xenial) and PDO eservice tests or develop your own PDO contracts (running on bionic):
https://github.com/encointer/encointer-sdk

@brenzi Thanks for taking a stab at docker-compose. I've extended that now into PR#13 into somewhat more versatile configuration and also in docker support for building in SGX HW mode. I've tried to add you there as reviewer but similar to Mic it didn't let me add you. If you have some thoughs and feedback, though, on the PR, please let me know (as comments on that PR). Thanks!


Instance console output tailing

I couldn't find any command to display / tail the ec2 get-console-output equivalent. Is it hiding in some command that's not exposed well?
If not, I'd like to make this a feature request.

@viraptor Thanks for the suggestion. Indeed a long time ago I though of it, but instead we focused on other topics.
It could be indeed useful to point to a EC2 instance and stream the console output in the local terminal. You are right in saying that there are some hidden commands ;)
Basically what you want is a tailer: streamer of some AWS side events. There code is only in the aws/tailers directory, so that they do not disrupt the official awless. Although they are experimental we require that there implementation be tested, clean and robust.
Looking at the hidden command awless tail -h, we can see they are 2 already.
So you can have a go. I know that @Trane9991 - another contributor - as played the most with the tailers.

Hi
I'm looking for the same feature. Any evolution on this topic ?
Thanks for awless btw !


BZ-1211257 - Comments always show the 'current' time

(cherry picked from commit 7a51dcdf7468363426f68ec773589ec1b54f3b65)

Peer PR for the already merged https://github.com/droolsjbpm/kie-wb-common/pull/120

+1

Thanks.
master: dc76ebe217f5e1bcce5dc03f413e8b3f5cd83d1e


AWSSDK.Extensions.NETCore.Setup support for RDSAuthTokenGenerator

Describe the feature
RDSAuthTokenGenerator has static GenerateAuthToken and GenerateAuthTokenAsync methods to generate an auth token for RDS.  It cannot have the credentials resolved by the NETCore.Setup package because there is no AWS service client instance constructed. AWSCredentials can be passed to a GenerateAuthToken overload or FallbackCredentialsFactory is used. Either way currently requires separate configuration than what is provided by NETCore.Setup.
NETCore.Setup does not provide a way of directly getting the configured AWSCredentials either. It would be beneficial to be able to get an AWSCredentials instance from AWSOptions so that only one configuration method needs to be implemented for an application.
Use Case
Use AWSSDK.Extensions.NETCore.Setup to configure AWS credentials for RDSAuthTokenGenerator.
Proposed Solution
ConfigurationExtensions.GetAWSOptions can currently be used to get an instance of AWSOptions that has the Profile and Region populated according to the configuration. There is a Credentials property on AWSOptions but it is null on the returned instance. If the AWSOptions could provide the resolved AWSCredentials instance it could be passed to RDSAuthTokenGenerator.
A new method or property could be added to AWSOptions to get the AWSCredentials or the Credentials getter could create the AWSCredentials when not set externally.
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

AWS .NET SDK and/or Package version used
AWSSDK.Extensions.NETCore.Setup 3.7.300
AWSSDK.RDS 3.7.309.5
Targeted .NET Platform
.Net 8
Operating System and version
Windows 10

Needs review with the team.

Reviewed this with team. This is a feature request to expose credentials resolved by AWSSDK.Extensions.NETCore.Setup package.

I have run into the same issue.
My ASP.NET Core application normally runs on EC2, but I run it locally for debugging purposes.
I have specified the profile in appsettings.json:
  "AWS": {
    "Profile": "AdministratorAccess-Preprod",
    "Region":  "ap-southeast-2"
  },

This works for all my other AWS SDK use(e.g. RDS, SQS, SNS), but not for the call to get a token for RDS IAM auth:
var password = RDSAuthTokenGenerator.GenerateAuthToken(hostname, (int)port, username);
Currently, I have to explicitly set the AWS credentials in environment variables or through appsettings.json to work around the issue. It would be preferable if RDSAuthTokenGenerator could load the application default credentials as the other services do.


[PROPOSAL] Specification for defining which subsystems, components and functionality is implemented in the microscope hardware

Each microscope model should define which subsystems (e.g. ion beam, manipualtor) is installed and available, and what functionality is available (e.g. manipulator rotation, plasma gas). Once defined this info can be used to determine how to execute functions, and prevent the API from trying to access undefined resources.
system.yaml:
system:
    ip_address: "localhost"
    manufacturer: "Thermo"
    model: "Helios"

Helios.yaml:

ion:
    enabled: True
    plasma: True
electron:
    enabled: True
manipulator:
    enabled: True
....

#34 related


How to recognize language other than english?

I am trying to recognize text which is spoken in spanish. But the controller is detecting only english language. Please help me.

Hey @vijaykharage
Great question, I just added a section on the README for how to deal with different locales.
Cheers!

Thanks for the quick help. Greatly appreciated.
PS. There will be crash if you use VoiceOverlayController() object without instantiating it as you are showing in your README.
Please update it with this, it will be crash free,
var voiceOverlayController: VoiceOverlayController = {         let recordableHandler = {             return SpeechController(locale: Locale(identifier: "es"))         }         return VoiceOverlayController(speechControllerHandler: recordableHandler)     }()


Location Tool: Irregular Polygon Shapes Do Not Save

Describe the bug
Irregular poly shapes (e.g. that cross over itself, like a bow tie shape) will not save. This becomes an issue when you try to close a polygon shape, but can't, and inadvertently cross the shape over itself.
To Reproduce
Steps to reproduce the behavior:

Go to /locations/new
Click on the polygon tool.
Create a bow tie shape (see image)
Click "Add"
Bug: Nothing happens.

Expected behavior

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Win 10, FF 72

Approved. Thank you!

Actually, this bug still exists in a slightly different version.

https://app.staging.crisiscleanup.io/locations/new
Choose "Primary Response Area"
Choose a random organization.
Add a normal, closed polygon.
Add a bow tie.
Bug: Bow tie won't save.

Win 10, FF 72

And this is a related bug. This is not a huge deal, but if you save a bow tie first (e.g. before saving another polygon). and then expand it, then only half of the bow tie expands. When you save the expanded bow tie, only the expanded half is saved.

I'm adding the "wontfix" label to this item. Although this is a known bug, the other fixes (e.g. snapping to the starting point) make this problem mostly moot. Not worth expending resources to address.

On second review, this is fixed, but the reference to the issue in #221 will be changed to "wontfix."


Issue: Air Drag possibly inconsistent and tolerance problems

WILL EDIT ISSUE LATER
Comparison of poliastro Cowell, Python sgp4 and TLE from Tiangong 1.
What can be seen is that for a torelance of 1e-5 the orbit decays, but for 1e-7 the orbit does not. Of course one needs to take into account the less accurate SGP4 than Cowell, however, based on the TLE's Cowell seems to be inaccurate.
Relative Tolerance = 1e-5

Relative Tolerance = 1e-7

🐞 Problem
The relative Cowell tolerance has an influence on the outcome of the propagation. SGP4 is known to be inconsistent for long propagation times, however Cowell/Dopri853 should be rather accurate/similar to TLE data.
🖥 Please paste the output of following commands

conda info -a (only if you have conda)
conda list (only if you have conda)
pip freeze

# Paste your output here:

python -c "import poliastro.testing; poliastro.testing.test()"

# Paste your output here:

🎯 Goal

💡 Possible solutions

📋  Steps to solve the problem

Comment below about what you've started working on.
Add, commit, push your changes
Submit a pull request and add this in comments - Addresses #<put issue number here>
Ask for a review in comments section of pull request
Celebrate your contribution to this project 🎉

Hmmm interesting. Please keep us posted when you're done with the analysis, this is important.

Finding 1
What I find the most peculiar is the too long Linear decay, compared to TLE and SGP4. The atmospheric model uses a scale height of 8 km with a rho_0=1.3 kg/m^3. This is true for an atmosphere till 100 km, but above that it is enormous erroneous compared to for example the data found in Space Mission Analysis and Design by Wertz for higher altitudes:
For example, at 300 km the current code will find a density of 6.73e-17 kg/m^3 while Wertz gives an average value of 1.95e-11. poliastro thus computes a factor 1e-6 lower atmospheric drag.
Finding 2
There is something going on with the rtol value compared to the stepsizes of the time of flights (tofs). Same propagations with different rtols lead to significant different results, to the point where for rtol=1e-7 no decay happens, where in TLE data the satellite has already re-entered. Reducing the rtol in my situation to 1e-5 lets the satellite re-enter more close to the TLE's.
It seems a high rtol makes the solution too stiff? Although I would expect the propagator to make smaller steps to account for this. Maybe a minimal step size in the settings lets the propagator use that with the allowed rtol, and thus non-accurate results? (as in rtol is adhered to, but a minimal step size is used for that, which should actually be lower than that minimal stepsize) Not sure how this propagator works exactly.
I think this normally is not an issue, but decaying satellites change their orbit significantly.
Also I had to implement an own termination condition for below 250km attitude, otherwise the propagator gets stuck.

Labeling this as a bug, we have to investigate it further.

Not sure how this propagator works exactly.

The Cowell method is described in many places, for example Vallado "Fundamentals of Astrodynamics and Applications". We use a high order Runge-Kutta method to integrate the equations of motion, specifically Dormand-Prince 8(5,3) (DOP853). We did not do a formal validation of our implementation, although you might be interested in trying the new one that will (hopefully) come with SciPy 1.4:
https://github.com/scipy/scipy/pull/10173
https://github.com/poliastro/poliastro/pull/675

Also I had to implement an own termination condition for below 250km attitude, otherwise the propagator gets stuck.

How did you do it? In theory the new solve_ivp function accepts events, so perhaps that would be a nice example of how to use them.

More on finding 1
I don't think the tolerance is the problem anymore, I just got more similar results with low tolerance because of the wrong atmospheric model. Do not have the Howard Curtis book myself, but it would be inconsistent if they use the normal exponential atmosphere above 100km.
Now I get under and over estimations of the decay with a rtol of 1e-9, depending on what atmosphere I use (for that matter 1e-7 and 1e-9 almost give the same results, which is more as expected).
I found this website: http://www.braeunig.us/space/atmmodel.htm, which gives the reference atmospehre up till 1000 km, based on a report by Jacchia (1971). 120km - 1000km tables: http://www.braeunig.us/space/pdf/Atmosphere_120-2K.pdf. Sadly the density is depending on solar maximum and exothermic temperature, but it is anyway more accurate than the exponential atmosphere used in the code.

How did you do it?
I got the propagator out of the loop and made my own loop and just checked the altitude to make it break the loop. So no solve_ivp was used, still getting the hang of poliastro, thus no advanced things yet.

because of the wrong atmospheric model

Oh, you're right! This atmospheric model that poliastro has is a simple, exponential one. It is more appropriate for "generic" stuff, perhaps for other planets (and especially big ones). I opened a new issue to implement a better model: https://github.com/poliastro/poliastro/issues/702

It works for a lot of simulations, for example for steeper re-entries (did my master thesis on that, written in octave though), but this all takes part at about maximum 100 km. But yes, for now the atmospheric drag is off by a lot. Changing it now manually to other values close to the ones in SMAD:

(no clue why inclination an eccentricity are so far off, relatively small changes though)

@jorgepiloto is adding more atmospheric models in #738.

Any chance we could test this again with the models @jorgepiloto added in #738?

The [link]http://www.braeunig.us/space/atmmodel.htm) that @WolfsSky shared is going to be really useful to fully implement pressure and density in COESA76 up to 1000km. After implementing locally the equation they use to compute previous state variables it seems to work fine. Pull request is coming soon.
On the other hand, they also include the martian atmosphere, really cool :heart_eyes:

Hi @WolfsSky, could you please share any gist or code to reproduce this issue and check if #756 solves it?

Hi @WolfsSky, I am reviewing old issues and was wondering if you still have any piece of code so this issue can be reproduced and checked against the new atmospheric models.

I am sure posting this analysis needed a lot of code: plotting the individual elements, comparing with TLEs... we will be glad to see that code published somewhere (in some personal repository perhaps). In the meanwhile, we added a new atmospheric model and this issue has been inactive for a while, so I will close it. Feel free to comment here or in the chat.


Directly assing mw_ret_t on declaring it and removed some mw_ret_t intermediate values which are not required

CI (packages up to rclcpp)

Linux
Linux-aarch64
macOS
Windows

CI looks good, thanks for the PR.


[Feature Request] Include/exclude obj from decompression matching regex

Hi Jay,
First and foremost, I would like to thank you for your great tool that I use on 100k of pdf’s per day.
Recently I came up with a requirement, where I only would want to de-compress/filter certain objects, e. g. containing BT..ET, but no /Image or /FontFile using either the --qdf option or --stream-data=uncompress.  There is already a filter for certain encodings, but I would like to filter the in-memory uncompressed objects for regex like “/BT\s.*\sET/” and then keep all other objects compressed.
The new filter should have two options: inclusive or exclusive.  This way one could ex- or include certain obj contents to be de-compressed in the output providing a JSON of 1 or multiple regex.
E. g.

--decompress-filter={"include":["BT\\s.*\\sET"],"exclude":["^\\s*/FontFile.{1,2}(\\d+)  \\d+ .*","^\\s+/Image\\d+ (\\d+) \\d+ .*"]}

The first group in the regex would match the obj to be included or excluded.  If no group is defined then one match would qualify the surrounding obj to be in- or excluded.
I tried to create a new version of the c++ fix-qdf tool to copy certain objects from an uncompressed version of the pdf back into the QDF document in process.  However, it looks like that the fix-qdf parser only supports QDF formatted pdf files and not any other regular pdf’s.
Is this a feature that you would deem feasible for a future release or could you point me to a workaround or qpdf source code positions I would need to look at to enhance?
Thank you,
Stefan

Hello.
I'm not sure I think a feature like the one your suggesting would be useful for the CLI tool. I think when you get to that level of specificity, you are better off coding against the library API. Using the qpdf library, it would certainly be possible to implement something like what you're suggesting. That said, I am definitely open to a discussion of how we could enhance the CLI to make it possible to implement something like what you're trying to achieve.
Presumably if you're processing that many files with complicated requirements, you must be doing something programmatically. What language or languages are you using? I'm not sure if you're aware of pikepdf, which is a set of Python bindings for the qpdf library. I am not the author, but I use it and think it's of very high quality. It might be possible for you to use it to do what you're trying to do. Perhaps there may be some library enhancements I could make that would improve the ease of doing this kind of operation, but I haven't thought it through at this time.
Anyway, if you let me know how you're using qpdf, maybe I can help think of a good solution.

If you are using the C++ library, you can use QPDFWriter::setCompressStreams(false) to prevent QPDFWriter from compressing streams automatically when it writes. Streams that are already compressed will stay compressed. For streams that you want to remain uncompressed, you can explicitly replace their stream data in the original QPDF object with the uncompressed data from the stream. For example, you can use QPDF::replaceStreamData with the PointerHolder<Buffer> that was returned by QPDF::getStreamData on the same stream, thus replacing the compressed stream with an uncompressed stream. Writing some code that walks through streams applying matching logic and then operates in this way could probably get you what you want, and I suspect it would be pretty easy to write this in Python with pikepdf, or it could be written in C++. I'm happy to discuss further. I may work on other language bindings to qpdf in the future, but I don't have any immediate plans to do so right now.

Hi Jay,
Thank you for your swift and helpful reply.
The application I use your tool for is long time archival of PDF, having their texts in pseudo readable form, but not in a separate file (that would be easy to achieve).
A subsequent tool I need to use in the process runs in O((i+k) log i) time for i/k = different file sizes.  In my case a regular compressed pdf file of 200 KB becomes 36 MB.  That is why I would rather not want to use the completely uncompressed versions of the pdf to feed into the above tool, but rather partial uncompressed to optimize the runtime according to Big O time-dependency above.
I am still using the CLI version of QPDF, but massive parallel on multiple nodes on a hyperscaler.
Since I must process this many pdf only one time (later incremental archival) I thought I could get by using the CLI version alone.
I am not shy in implementing the partial decompression in C++ applying the tips you gave me.  At first it surely will be faster to implement it in Python though as you pointed out.
I am sorry, I am new to the QPDF library.  Would the pdf-invert-images.cc example be a good template to start with?  The fix-qdf tool uses QUtil::read_lines_from_file which returns a nice vector of lines which are easy to match with regex.  I just wonder how would I identify the obj in a stream?
When I think about the above algorithm then testing the regex for each line will also lead to a performance penalty.  Oh well, there is no free lunch.  I could take shortcuts and code specific to my internal pdf file structure, but that will make it not flexible and thus unusable for other pdf files.
I am glad I reached out to you.  In the past I always tried to figure out things on my own which I still consider the best way to learn, but I simply don’t have the time any more to do the research myself.  I miss the old days writing TSR programs for fun in Assembler on M$ DOS without the ability to Google/Altavista code snippets.
Thanks again for the hints and support you provided.
Thank you,
Stefan

pdf-invert-images.cc would be a reasonable starting point. If I have some time in the next few days, I could probably produce some example code. There were some recent additions to qpdf to allow you to get information about where in a file certain things appear. The code was contributed and first appeared in 9.1.0. Anyway, my hunch is that it shouldn't take much code to get this to work. I'm happy to look over any code you've written or offer suggestions along the way.
We must have started programming at around the same time. I wrote my first programs on graph paper, but the first ones I actually ran on a computer were on a TRS-80 circa 1981. I had already been programming for 13 years before I had access to the web. I had to learn stuff by reading books, reading source code, and figuring it out. :-)

Closing for inactivity.


updating controls to use resources.

Updating controls to use resources so they work with InSpec v4.18.

I'm having issues with the acceptance tests on this and I'm not sure why. Debugging.

Working on verifying this but it looks like this may only fail:

On a real VM (docker works)
After the lockdown from the SSG

My guess is that some logic path is being triggered in only that case (and it's quite baffling)

Confirmed that this only happens on a real VM after the SSG remediations have been applied.
This is not this PR or the version of inspec, so I'm going to push this as working and we can figure out what's going on in another PR.


Fix ellipse and rectangle problems with arithmetics

Errors may caused by conversion to int.

Download:
Environment: QT=5.12, ARCH=32, PYTHON_PATH=C:\Python37
Environment: QT=5.12, ARCH=64, PYTHON_PATH=C:\Python37-x64

Download:
Environment: QT=5.12, ARCH=32, PYTHON_PATH=C:\Python37
Environment: QT=5.12, ARCH=64, PYTHON_PATH=C:\Python37-x64


Junction files without SAM writes

Create the Junction bed files on the fly without the need to write the bulky JUNCT.sam files.

Junction file now on the fly created from BAM, piping the samtools view output to awk and then perl get_juncts.pl

Junction file now on the fly created from BAM, piping the samtools view output to awk and then perl get_juncts.pl


Block creation of external mode when Provider mode is already present

After fix:

Bug link: https://bugzilla.redhat.com/show_bug.cgi?id=2312442

/cherry-pick release-4.17

/cherry-pick release-4.17-compatibility

/test odf-console-e2e-aws


Size of the Scale factor

Hi. Is there any way to change size of the scale factor? What should I do if I want outputs with x4 or x8 scaling?
When I change the scale factor from 3 to 4, I receive this error:
ValueError: Cannot feed value of shape (128, 96, 96, 3) for Tensor 'labels:0', which has shape '(?, 128, 128, 3)
Thanks.

Hi. Is there any way to change size of the scale factor? What should I do if I want outputs with x4 or x8 scaling?
When I change the scale factor from 3 to 4, I receive this error:
ValueError: Cannot feed value of shape (128, 96, 96, 3) for Tensor 'labels:0', which has shape '(?, 128, 128, 3)
Thanks.

scales = xx in utils.py

Thank you for you response. So, what does "scale" factor in main.py mean? (which is 3 by default)


Uncaught Error: Cannot find module './kernel-manager'

[Enter steps to reproduce:]

Atom started.

Atom: 1.25.0 x64
Electron: 1.7.11
OS: Mac OS X 10.13.3
Thrown From: Hydrogen package 2.3.0
Stack Trace
Uncaught Error: Cannot find module './kernel-manager'
At module.js:472

Error: Cannot find module './kernel-manager'
    at Module._resolveFilename (module.js:470:15)
    at Module._resolveFilename (/Applications/Atom.app/Contents/Resources/electron.asar/common/reset-search-paths.js:35:12)
    at Function.get_Module._resolveFilename (/Applications/Atom.app/Contents/Resources/app/src/module-cache.js:354:58)
    at Module.require (/app.asar/static/index.js:40:43)
    at require (/Applications/Atom.app/Contents/Resources/app/src/native-compile-cache.js:66:33)
    at /packages/Hydrogen/lib/signal-list-view.js:21:22)
    at /packages/Hydrogen/lib/signal-list-view.js:120:3)
    at Module.get_Module._compile (/Applications/Atom.app/Contents/Resources/app/src/native-compile-cache.js:106:36)
    at Object.value [as .js] (/Applications/Atom.app/Contents/Resources/app/src/compile-cache.js:240:29)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (/app.asar/static/index.js:47:45)
    at require (/Applications/Atom.app/Contents/Resources/app/src/native-compile-cache.js:66:33)
    at /packages/Hydrogen/lib/main.js:35:23)
    at /packages/Hydrogen/lib/main.js:643:3)
    at Module.get_Module._compile (/Applications/Atom.app/Contents/Resources/app/src/native-compile-cache.js:106:36)
    at Object.value [as .js] (/Applications/Atom.app/Contents/Resources/app/src/compile-cache.js:240:29)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (/app.asar/static/index.js:47:45)
    at require (internal/module.js:20:19)
    at customRequire (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:96:26)
    at Package.requireMainModule (/Applications/Atom.app/Contents/Resources/app/src/package.js:782:33)
    at Package.activateConfig (/Applications/Atom.app/Contents/Resources/app/src/package.js:267:16)
    at PackageManager.module.exports.PackageManager.packageHasSettings (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/package-manager.js:68:20)
    at PackageCard.hasSettings (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/package-card.js:794:40)
    at PackageCard.updateSettingsState (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/package-card.js:439:22)
    at PackageCard.updateInterfaceState (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/package-card.js:431:18)
    at new PackageCard (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/package-card.js:92:16)
    at PackageDetailView.completeInitialization (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/package-detail-view.js:206:34)
    at PackageDetailView.loadPackage (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/package-detail-view.js:225:20)
    at new PackageDetailView (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/package-detail-view.js:100:16)
    at callback (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/settings-view.js:366:26)
    at SettingsView.getOrCreatePanel (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/settings-view.js:371:25)
    at SettingsView.showPanel (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/settings-view.js:429:30)
    at SettingsView.showDeferredPanel (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/settings-view.js:416:20)
    at SettingsView.initializePanels (/Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/settings-view.js:250:18)
    at /Applications/Atom.app/Contents/Resources/app/node_modules/settings-view/lib/settings-view.js:122:26
    at _combinedTickCallback (internal/process/next_tick.js:73:7)
    at process._tickCallback (internal/process/next_tick.js:104:9)

Commands

Non-Core Packages
autocomplete-html-entities 0.2.0
autocomplete-java 1.2.7
autocomplete-python 1.10.5
color-picker 2.2.5
gpp-compiler 3.0.7
html-to-css 0.2.2
Hydrogen 2.3.0
language-65asm 5.0.0
language-x86-64-assembly 2.2.11
platformio-ide-terminal 2.8.1
python-indent 1.1.4
script 3.17.3

Could you try reinstalling hydrogen using the command line?
apm uninstall hydrogen
apm install hydrogen

An aside for hydrogen devs: a number of similar issues have been popping up during the past year. Reinstalling always seems to fix things.
But does anyone know why this is happening in the first place? There are enough bug reports to suggest a real problem with installation.

That helped!
W dniu 22/03/18 20:37 Nikita Kitaev<EMAIL_ADDRESS>napisał:

Could you try reinstalling hydrogen using the command line?
apm uninstall hydrogenapm install hydrogen
An aside for hydrogen devs: a number of similar issues(https://github.com/nteract/hydrogen/issues?utf8=✓&q="Uncaught+Error%3A+Cannot+find+module") have been popping up during the past year. Reinstalling always seems to fix things.
But does anyone know why this is happening in the first place? There are enough bug reports to suggest a real problem with installation.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub(https://github.com/nteract/hydrogen/issues/1265#issuecomment-375332371), or mute the thread(https://github.com/notifications/unsubscribe-auth/AM8JugYa7j4ubD3pnKhED6Z8_wglfbr4ks5tg7lLgaJpZM4S0qCx).


Auditor app crashes on initial device pairing

The Auditor app keeps stopping when attempting to perform the initial pairing between a phone and tablet.
Auditor: Google Pixel 3a XL [Model: G020B]
Auditee: Galaxy Tab A 10.1 2019 (Wi-Fi) [Model: SM-T510]
Both devices are running Auditor version 17.
Steps to reproduce issue:

Launch Auditor on both devices.
Tap 'auditor (device performing verification)' on the Pixel.
Tap 'auditee (device being verified)' on the Tab A.
Scan QR code shown on the other device.
The app crashes.

No issues when auditor and auditee are switched; pairing and verification are successful.
Please let me know if additional information is needed.

@Jawshy Can you check if this is still a problem and send another bug report?

The app still crashes on the tablet. I've sent a bug report to<EMAIL_ADDRESS>

@Jawshy I think https://github.com/GrapheneOS/Auditor/commit/ce8c0f464ec25daa7c690a2030620c34b84f300c will fix the issue. I can make a test release for you to try on the Auditee.

It might be better to just make a normal release due to how Auditor releases impact attestation.

Adjusted this to https://github.com/GrapheneOS/Auditor/commit/a0cc37c35dbcad8463886bff3d44666a2cd9467a to avoid weakening the check on other devices.

This will be fixed by the next release when I finally find the time to put that together...

@Jawshy This is FINALLY released in version 18. Sorry it took so long.

I updated Auditor to version 18 on both devices and tried verifying the tablet again. The app didn't crash, but the verification failed on the first attempt. Upon clearing the auditee parings, the initial pairing was successful on the second attempt. Cheers.

It probably failed due to the previous failed pairing. If the pairing is incomplete, the device being audited doesn't actually know that.


Update README.md add config before deploy

fixes #305

The actions/configure-pages Actions is optional and will not help resolve an issue with artifacts if you are having one.


the forest for the trees

Hi, in the ever ongoing thought process of trying to improve the early alert tool, I still think that for species with a lot of observations (so species that are not too rare), the alerts are kinda ineffective. An example is Ludwigia grandiflora for which you can easily get several hundreds of records in one month from various databases.
Thinking about a number of potential options to make the alerts more targeted:

Solution 1: Move from "observation alert" to "place-based alert" by grouping records based on a spatial unit to avoid repetition of the species being observed by several persons in the same location. Of course, this should partly be solved when people can issue alerts for a specific spatial delimiter. To go place-based or not could actually be a user option when tailoring the alerts since some users will probably want everything at least for some species.
Solution 2: increase the number of areas that can be used for setting alerts (going to smaller spatial scales)
Solution 3: allow users to make a polygon themselves
Solution 4: include a "time delimiter" in the alerts, for instance you want weekly alert emails but you only want in that alert email the observations of that last week, or even last day (on other words, focus more on recent reports)

I'll keep thinking...

Thanks @timadriaens, that might deserve a discussion to understand the details better and plan appropriately!
I have the feeling that most of it could be solved with a good "custom polygon management" system, which is in place but currently being polished and improved.
We also have to see how to do this best given the timing constraints.
@damianooldoni, do you want to set a chat soon? Or you have another preference?

Solution 3 has been deployed.
We are already speaking about solution 4 in https://github.com/riparias/gbif-alert/issues/227. The user has to choose how old the occurrences can be, e.g. max 1 year, or 1 month. This allows us to mobilize more and more datasets in the future without having people disturbed by old occurrences they are not actually interested. Default value could be 1 year. See #227 for more info.
So, I think that 3 + 4 will solve this issue. And as 4 is already discussed in an issue, this one can be closed.


Add ros2_controllers as exec_dependency in next Foxy synchronization

Reference to this comment https://github.com/ros-simulation/gazebo_ros2_control/pull/44#discussion_r569765040
Add effort_controller, velocity_controller and joint trajectory controller as exec_dependency

While this may sound intuitive, I think we may run into a bunch of packaging and dependency issues. The plugin itself shouldn't even have launch files, a much more maintainable setup would be to have a gazebo_ros2_control_demos package where you can shove all scripts and asset files + runtime dependencies.

I honestly think this should not be an enhancement rather than an actual bug. I just ran into this in a freshly installed ubuntu and the launch file crashes pretty badly without any hints:
~/workspace/ros2/ros2_control_ws $ ros2 launch gazebo_ros2_control_demos cart_example_position.launch.py
[INFO] [launch]: All log files can be found below /home/knk1pal/.ros/log/2021-02-11-19-24-40-095406-knk1pal-desktop-94280
[INFO] [launch]: Default logging verbosity is set to INFO
[INFO] [gzserver-1]: process started with pid [94282]
[INFO] [gzclient   -2]: process started with pid [94284]
[INFO] [robot_state_publisher-3]: process started with pid [94287]
[INFO] [spawn_entity.py-4]: process started with pid [94289]
[robot_state_publisher-3] Parsing robot urdf xml string.
[robot_state_publisher-3] Link slideBar had 1 children
[robot_state_publisher-3] Link cart had 0 children
[robot_state_publisher-3] [INFO] [1613100280.531889377] [robot_state_publisher]: got segment cart
[robot_state_publisher-3] [INFO] [1613100280.531991254] [robot_state_publisher]: got segment slideBar
[robot_state_publisher-3] [INFO] [1613100280.532006048] [robot_state_publisher]: got segment world
[spawn_entity.py-4] [INFO] [1613100280.884423657] [spawn_entity]: Spawn Entity started
[spawn_entity.py-4] [INFO] [1613100280.884755934] [spawn_entity]: Loading entity published on topic robot_description
[spawn_entity.py-4] [INFO] [1613100280.885949316] [spawn_entity]: Waiting for entity xml on robot_description
[spawn_entity.py-4] [INFO] [1613100280.991258736] [spawn_entity]: Waiting for service /spawn_entity, timeout = 5
[spawn_entity.py-4] [INFO] [1613100280.992204321] [spawn_entity]: Waiting for service /spawn_entity
[spawn_entity.py-4] [INFO] [1613100281.749584266] [spawn_entity]: Calling service /spawn_entity
[spawn_entity.py-4] [INFO] [1613100281.885642581] [spawn_entity]: Spawn status: SpawnEntity: Successfully spawned entity [cartpole]
[gzserver-1] [INFO] [1613100281.907895669] [gazebo_ros2_control]: Loading gazebo_ros2_control plugin
[gzserver-1] [INFO] [1613100281.913425389] [gazebo_ros2_control]: Starting gazebo_ros2_control plugin in namespace: /
[gzserver-1] [INFO] [1613100281.913928641] [gazebo_ros2_control]: Starting gazebo_ros2_control plugin in ros 2 node: gazebo_ros2_control
[gzserver-1] [INFO] [1613100281.920382961] [gazebo_ros2_control]: connected to service!! robot_state_publisher
[gzserver-1] [INFO] [1613100281.921563856] [gazebo_ros2_control]: Recieved urdf from param server, parsing...
[INFO] [spawn_entity.py-4]: process has finished cleanly [pid 94289]
[gzserver-1] [INFO] [1613100281.965261873] [gazebo_ros2_control]: Loading joint: slider_to_cart
[gzserver-1] [INFO] [1613100281.965319936] [gazebo_ros2_control]: 	Command:
[gzserver-1] [INFO] [1613100281.965382091] [gazebo_ros2_control]: 		 position
[gzserver-1] [INFO] [1613100281.966137730] [gazebo_ros2_control]: 	State:
[gzserver-1] [INFO] [1613100281.966167436] [gazebo_ros2_control]: 		 position
[gzserver-1] [INFO] [1613100281.966763995] [gazebo_ros2_control]: 		 velocity
[gzserver-1] [INFO] [1613100281.966799265] [gazebo_ros2_control]: 		 effort
[gzserver-1] [INFO] [1613100281.968665842] [gazebo_ros2_control]: Loading controller_manager
[gzserver-1] [WARN] [1613100281.980678942] [gazebo_ros2_control]:  Desired controller update period (0.01 s) is slower than the gazebo simulation period (0.001 s).
[gzserver-1] [INFO] [1613100281.980875189] [controller_manager]: Loading controller 'joint_trajectory_controller'
[gzserver-1] [ERROR] [1613100281.980908832] [controller_manager]: Available classes:
[gzserver-1] [ERROR] [1613100281.980920531] [controller_manager]: controller_manager/test_controller
[gzserver-1] [ERROR] [1613100281.980927902] [controller_manager]: Loader for controller 'joint_trajectory_controller' not found.
[gzserver-1] terminate called without an active exception
[gzserver-1] Aborted (core dumped)
[ERROR] [gzserver-1]: process has died [pid 94282, exit code 134, cmd 'gzserver                                                                    -s libgazebo_ros_init.so   -s libgazebo_ros_factory.so       '].
[INFO] [gzclient   -2]: process has finished cleanly [pid 94284]
^C[WARNING] [launch]: user interrupted with ctrl-c (SIGINT)
[robot_state_publisher-3] [INFO] [1613100291.621538047] [rclcpp]: signal_handler(signal_value=2)
[INFO] [robot_state_publisher-3]: process has finished cleanly [pid 94287]

AS I mentioned in the PR Holding this PR to the next Foxy sync


Dannyglatstein/sc 618196/update contentful personalization to have

Needs a description to outline what the PR is trying to achieve


Multimodal (Visual) LLM

Hi it looks like you updated the API a little bit in this commit
https://github.com/guinmoon/llmfarm_core.swift/commit/e4e8aa7617e2e86af434677cc4196462a0005ea9
Would you mind giving an updated working example in this repo here and/or here

Hi.

If you could send me a sample code too? Thank you so much @guinmoon If all multimodal models that can be run with llama.cpp can be run with llmfarm_core?


Improve developer ergonomics with easier access to query parameters, cookies, etc

I recently ported over a small project of mine (https://github.com/ajusa/250) from Jester to Mummy, and ended up creating a file (https://github.com/ajusa/250/blob/master/src/mummy_utils.nim) which has several utility functions that I thought were "missing" from mummy itself.
It'd be nice if some of these could be added to mummy itself - or if there were easier ways to set cookies, do redirects, and parse objects from query parameters/post bodies. Those were my main pain points.
I believe you mentioned on Discord that you were exploring a pattern to handle parsing query parameters and path parameters, if that's the case that would probably fix most of the issues I've described above.

Thanks for linking to this file, it is useful to see. I think me continuing to make more examples will go a long way to help in these cases.
I consider it a great thing you were able to add those conveniences in just a handful of lines.
Why I'm less than eager to build these in directly:

Redirects -- they are very simple, it can be 1 line. It is also the case that people need to specify 301 vs 302 and may want to add additional headers. For these reasons, I currently think this is best shown in a new example instead of trying to create a proc that ends up taking everything as an optional parameter.

Cookies -- this should be its own little import in webby so you can choose to use that one or choose to use the Nim stdlib version. I like keeping the choice open.

URL / URI handling -- again, it is a webby or Nim stdlib option. Request will have .uri on it and you can choose to parse it in your handler. This is important because parsing that .uri could fail and I consider it important that the programmer has the option to choose how to handle that.

getOrDefault is a totally reasonable, should add that to webby etc.

I don't write this as a dismissal, just that sharing how I'm thinking now. All of this will get better, it takes a long time to really nail all the aspects of server QoL and I am not for sure not there yet!

I have added a couple examples, one showing redirecting, the other showing an approach to custom handler types with additional parameters https://github.com/guzba/mummy/pull/78

I looked through #78 and I like the custom handler approach, I'll have to try it out. Never thought to use callbacks in that way.
For the points you raise though:

I'm a bit split on this - on one hand I agree, on the other hand redirects are so common that I feel that including a slightly opinionated way of doing them (while still allowing additional headers like you mentioned) would be useful. I can see myself misspelling "Location" and not realizing it, it'd be nice if I didn't have to worry at all about the specific header value.
Agreed, once webby supports cookies that would be pretty nice to have.
Fair enough, mummy shouldn't be too opinionated here
Awesome, I'll look into sending a PR for that soon.

Feel free to close this issue as I think #78 addresses most of my concerns (unless there's something else you think can be added).

I have added some of the ergonomics in my previous issue with focus on the parameters - both named but also query params - https://github.com/guzba/mummy/issues/98 (code: https://github.com/ThomasTJdev/mummy/issues/2).
For the response and redirect I have something like:
template resp(resp: string) =
  request.respond(200, @[("Content-Type", $ContentType.Html)], resp)

template resp(httpStatus: HttpCode, contentType: ContentType, resp: string) =
  request.respond(httpStatus.ord, @[("Content-Type", $contentType)], resp)

template resp(resp: JsonNode) =
  request.respond(200, @[("Content-Type", $ContentType.Json)], $resp)

template resp(httpStatus: HttpCode, resp: JsonNode) =
  request.respond(httpStatus.ord, @[("Content-Type", $ContentType.Json)], $resp)

template redirect(httpStatus: HttpCode, path: string) =
  request.respond(httpStatus.ord, @[("Location", path)])

I have updated my personal package with utilities. For reference for others: https://github.com/ThomasTJdev/mummy_utils

I redid url parsing in webby and have decided to go further with mummy/routers features in this work-in-progress PR: https://github.com/guzba/mummy/pull/111
Relevant to this issue since it would make query parameters much easier to access as suggested.

This is really great to see, thank you guzba! Two questions:

Will there be any work to be able to extract a list of query parameters out? I see that the underlying implementation is just seq[(string, string)], so do I just want to loop over the keys/values and add them to my own seq if I have a key with multiple values? Maybe this is more of a webby issue than a mummy issue.
Will you also be adding a similar query field to parse a form URL encoded body? This one is a bit more opinionated (as the body is really just a string), it's just convention that we format it a certain way based on the browser implementations of form encoding and multipart encoding.

Will there be any work to be able to extract a list of query parameters out? I see that the underlying implementation is just seq[(string, string)], so do I just want to loop over the keys/values and add them to my own seq if I have a key with multiple values? Maybe this is more of a webby issue than a mummy issue.

The API support for the QueryParams type can certainly be improved but yeah you can iterate the (key, value) pairs and do whatever you'd like. The choice of seq internally was done to enable multiple values for the same key, though the easy [] accessor will not be what you want in that case.

Will you also be adding a similar field to parse a form URL encoded body?

I agree that it is a bit less clear how to improve things for the request body. The body can be form-encoded, could be JSON, could be mulitpart etc and these all have different implications. I do think the body being a string is a good baseline not a height of convenience haha.
Even if it is not an ideal name since it was intended for parsing the URL query string, parseSearch in webby will decode a form-encoded body. Just a note in case I've not showed that anywhere. let parsedBody = parseSearch(request.body)

Revisiting the initial 4 topics that were discussed as of today:

Redirects -> nothing to report here
Cookies -> still need to be added to webby
URL / URI handling -> ended up changing my mind here and went ahead and added a parsed and decoded .path and .queryParams to Request in release 0.4.0
getOrDefault was added ina PR a while back


[OSD-16305] Initial pass at the deletion controller

spelling fixes, README updates
Fixes existing context.TODOs
Minor readme updates
Context aware service tests
Additional finalizer controller logic
Additional controller logic
Updated go.mod
controller fixes
return lower case when fetching clusterID name
logging key/value updates
Updated RespDelete DeleteAll bool
Deletion controllers should target syncsets, not selectorsyncsets
Tested in stg, looks like it was successful removing the Goalert Service for a cluster in stage.

/hold

/unhold


UNMET PEER DEPENDENCY

I found this error in my console. I tried several times, until I updated my software.
Any suggestions will be appreciated.
NODE: 4.4.7
NPM: 2.15.8
REACT-NATIVE: 0.31.0
NATIVE-BASE: 0.5.4
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed. npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed. npm WARN<EMAIL_ADDRESS>No license field.

@totouraMan I guess you did not go through system requirements.
Check here


add workflow activation and scheduling lock TTL

Moved the "check for and delete expired locks" logic into the lockAndCreate method. Also refactored lockTTL to lockTimeout and deleteOldLocks to deleteExpiredLocks.


indent error with pipe (|)?

abc:
  def:
    - |
      some message
      some message

Getting this error:
error  Expected indentation of 8 spaces but found 6 spaces  yml/indent

Is this correct? The formatter I use one VS Code yaml format the yaml to this.
Even if I try to change it to:
abc:
  def:
    - |
        some message
        some message

after format, it goes back to 6 spaces.

Thank you for posting this issue.
It looks like a bug to me.

Thanks for the quick fix! 🎉


Generate code for servicemanagement

I am trying to use the Service Management API: https://github.com/google/go-genproto/blob/master/googleapis/api/servicemanagement/v1/servicemanager.pb.go
However the corresponding generated package is not present in google-cloud-go.

Use https://godoc.org/google.golang.org/api/servicemanagement/v1. I don't think we'll be generating anything nicer than that in the near future, but hopefully that is nice enough for your needs.


Add support for Tag Helper opt-out operator

Summary of the changes

Adds support for the Tag Helper opt-out operator !. My knowledge of HTML is limited so I'm not sure if I missed any cases, let me know if I did!
The LSP side of this PR has been merged in: https://devdiv.visualstudio.com/DevDiv/_git/VSLanguageServerClient/pullrequest/344266

Fixes:
https://github.com/dotnet/aspnetcore/issues/33126

Actually, it looks like I messed something up a few cases. Temporarily closing for a bit to figure out what's going on. 😅


 Weird I didn't see any issues around this in dev, thanks for sorting this out

@Nick-Lucas I think this has to do with the e2e setup. Jordan wanted to rework tests completely so this would get caught, we have more issues related to not catching broken types. I'd love to be able to check the generated code too as so many times people report something doesn't build correctly and that would be trivial to test

Yeah the snapshot tests are really useful for checking changes to generated code, but I think they could be shifted to run on the uncompiled code to tighten up the dev loop. Build is quite slow
The suite as a whole looks really good but has a few gaps (some of which I filled in)
I did most of my dev using the dryrun suite for this reason

@Nick-Lucas mind sharing how did you use dry run? I think we all have different ways of developing/testing, curious what was your process

Oh I just added one new test for the transformers in that file (src/index.ts)and disabled the dryrun flag so I could inspect the generated output and place breakpoints. Enabled the flag again before finishing up
It felt pretty productive!

I think some committed config for Vitest would help, I never got the vs code extension to work for the repo but the js debug shell is fine

Ah yeah, I tend to do something similar to see the output fast. Are you on Discord btw? Are you planning to contribute more stuff?

Oh yeah I'm happy to join on discord if you guys have a space there. I'll probably have some more contributions on the transformers based on my experiences at work, and keen to help figure out the instantiable client stuff since my team depends on that 🙂

We don't have a group chat at the moment due to lack of stable contributors, we've been just chatting there with Jordan. I am on Discord mrlubos if you want to talk there


virtual clusters: add upstream_rq_retry_overflow to stats

Description: adding one more stat in the style of #10474. This should give a full picture of retries happening at the router level for virtual clusters
Risk Level: low
Testing: modified unit test
Docs Changes: added docs
Release Notes: release notes from #10474 are still valid.
Signed-off-by: Jose Nino<EMAIL_ADDRESS>

@mattklein123 thanks for approving and merging #10474. I was giving one last pass and decided I wanted this stat as well to give us a view for the retry behavior on par with the cluster stats.


How do I modify the start directory

Description of the new feature/enhancement
The starting directory is now the system32, I want to change it to the specified directory.
What should I do?

Proposed technical implementation details (optional)

#6166


node installation detail

About the PR

node Installation instructions updated in the README
minor typo fix

thank you 🙏🏽  since this was whipped up super quickly we did oversaw the requirements


404 Error - Download Prerequisites

Describe the issue
If you try to download the artifacts with the include plattform tag you receive a 404 error during download of the url rewrite module.
Scripts used to create container and cause the issue
Flush-ContainerHelperCache
$msToken = "..."
$artefactUrl = Get-BCArtifactUrl -type Sandbox -country de -storageAccount bcinsider -sasToken $msToken -select SecondToLastMajor
Download-Artifacts -artifactUrl $artefactUrl -includePlatform

Full output of scripts
Downloading platform artifact /sandbox/16.5.15639.0/platform
Downloading C:\Users\<user>\AppData\Local\Temp\3f8e0d63-1eaf-4483-89f1-bd76f6463eac.zip
Unpacking platform artifact to tmp folder using 7zip
Downloading Prerequisite Components
Downloading c:\bcartifacts.cache\sandbox\16.5.15639.0\platform\Prerequisite Components\Open XML SDK 2.5 for Microsoft Office\OpenXMLSDKv25.msi
Downloading c:\bcartifacts.cache\sandbox\16.5.15639.0\platform\Prerequisite Components\IIS URL Rewrite Module\rewrite_2.0_rtw_x64.msi
Download-File : Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an error: (404) Not Found."
At C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\Misc\Download-Artifacts.ps1:206 char:45
+ ...             Download-File -sourceUrl $url -destinationFile $path -tim ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Download-File], MethodInvocationException
    + FullyQualifiedErrorId : WebException,Download-File

Screenshots
n/a
Additional context

I checked the urls in Prerequisite Components.json.
If I throw the URL for the rewrite module into a browser I receive a 404.
did it use to work? yes

Same for
Flush-ContainerHelperCache
$artefactUrl = Get-BCArtifactUrl -type Sandbox -country de
Download-Artifacts -artifactUrl $artefactUrl -includePlatform

https://freddysblog.com/2020/08/18/one-of-these-days/

Fix shipped in BcContainerHelper 1.0.3
Fix will ship in NavContainerHelper <IP_ADDRESS>

Sorry, I did not check your blog & twitter.

No problem - it is fine to have an issue here - will leave it open for other people to find.

The error is still the same in version 1.0.3
PS C:\WINDOWS\system32> Get-Module -All -ListAvailable -Name BcContainerHelper
Directory: C:\Program Files\WindowsPowerShell\Modules
ModuleType Version    Name                                ExportedCommands

Script     1.0.3      BcContainerHelper                   {Add-FontsToBcContainer, Add-GitToAlProjectFolder, Backup-BcC...
Script     1.0.3      BcContainerHelper                   {Add-FontsToBcContainer, Add-GitToAlProjectFolder, Backup-BcC...
But with execution of New-BcContainer command it says that is still version 1.0.2
New-BcContainer     -accept_eula
-containerName $containerName     -credential $credential
-auth $auth     -artifactUrl $artifactUrl
-imageName 'scaptify'     -includeTestToolkit
-includeTestLibrariesOnly     -licenseFile $licenseFile
-dns '<IP_ADDRESS>'     -usessl -installCertificateOnHost
-memoryLimit 8G `
-updateHosts
BcContainerHelper is version 1.0.2
BcContainerHelper is running as administrator
Host is Microsoft Windows 10 Pro - 2004
Error
Downloading c:\bcartifacts.cache\sandbox\16.4.14693.15627\platform\Prerequisite Components\IIS URL Rewrite Module\rewrite_2.0_rtw_x64.msi
Download-File : Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an error: (404) Not Found."
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\1.0.2\Misc\Download-Artifacts.ps1:206 char:45

...             Download-File -sourceUrl $url -destinationFile $path -tim ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Download-File], MethodInvocationException
FullyQualifiedErrorId : WebException,Download-File

I still get the error with version <IP_ADDRESS>
Script output:
Create BuildPV2-build from https://bcinsider.azureedge.net/sandbox/16.5.15646.0/w1
NavContainerHelper is version <IP_ADDRESS>
NavContainerHelper is running as administrator
Host is Microsoft Windows Server 2019 Datacenter - ltsc2019
...
Downloading Url Rewrite
Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an error: (404) Not Found."
at InstallPrerequisite, C:\Run\HelperFunctions.ps1: line 579
at , C:\Run\navinstall.ps1: line 26
at , C:\Run\start.ps1: line 209
at , : line 1
At C:\Program
Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\ContainerHandling\Wait-NavContainerReady.ps1:44 char:17

...             throw "Initialization of container $containerName failed" ...

That is not the same as above.
Above you used Download-Artifacts - this is something in the container.
Somehow you haven't gotten the latest generic image - you need to provide full script + full output.


Collect code coverage and add badges to README file

Strictly speaking, it's an unrelated clean-up, but I thought that while I was there I could also update the caching wokflows :innocent:


ProgramTest.java:52-55: Two tests are disabled here because...

The puzzle 144-b64024e6 (from #144) in eo-compiler/src/test/java/org/eolang/compiler/ProgramTest.java (lines 52-55) has to be resolved: "Two tests are disabled here because there is something wrong with our ANTLR syntax. We should be able to parse the files provided, but they fail. Let's find out and fix."
The puzzle was created by Yegor Bugayenko on 06-Aug-17.
Estimate: 30 minutes, role: IMP.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@yegor256 please, pay attention to this issue

@0crat assign @g4s8

@0crat assign @g4s8  (here)

@yegor256 Job gh:yegor256/eo#145 assigned to @g4s8, please go ahead (policy). @g4s8 is not a member of this project yet, but can request to join. @g4s8 check your Zerocracy profile and follow the instructions.

+15 points just awarded to @0pdd, total is +135.

@g4s8 would be great if you can take a look. I'm stuck now, because of this weird problem with the syntax.

@yegor256 sure, I'll see what can I do

@yegor256 I fixed grammar rules, but broke java code generation. Now eo code can be parsed with strict ANTLR error handling. I suppose to complete this task within week.

@g4s8 thanks!

The puzzle 144-b64024e6 has disappeared from the source code, that's why I closed this issue.

Job gh:yegor256/eo#145 is now out of scope.

+30 points just awarded to @g4s8, total is +1605.


Reverse a string - not accepting correct solution

Challenge Reverse a String has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.81 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
I can complete the assignment with unshift but wanted to create this function to reuse in the other algorithm challenges. My output seems correct but is not registering as being correct.
My code:

function reverseString(str) {
  var retstr="";
  var l=str.length;
  for (var i=0; i<str.length; i++){
    retstr+=(str[l-i-1]);
  }

  retstr="\""+retstr+"\"";

  return retstr;
}

reverseString("Howdy");

i meant split not unshift

I think your line
retstr="\""+retstr+"\"";
is the problem. There is no need to add the double quotes around the reversed string.

+1 @tpoikela
Just remove that line and you're good to go, and close the issue as well! :)
Also, since you've stored str.length in l you can use it in the for loop:
function reverseString(str) {
  var retstr="";
  var l=str.length;
  for (var i=0; i<l; i++){
    retstr+= str[l-i-1]; // no need for brackets here
  }

  return retstr;
}

Yeah, that is not the correct solution @tgardiner88
For reverseString("Howdy");, your function returns ""ydwoH"", when it should return "ydwoH"

function reverseString(str) {
var retstr="";
var l=str.length;
for (var i=0; i<str.length; i++){
retstr+=(str[l-i-1]);
}
return retstr;
}
reverseString("Howdy");
it works now :D

Closing as resolved.


Make testCafe.close wait until connections are closed (closes #216)

\cc @AlexanderMoskovkin @georgiy-abbasov

\r-

@testcafe-build-bot \retest

FPR

\r-

New modules: onceDone and mapReverse.

@testcafe-build-bot \retest

lgtm, just add BrowserSet update pls

@testcafe-build-bot \retest

FPR

lgtm
/cc @VasilyStrelyaev please, check comments

\r-

FPR

lgtm

lgtm


ENT-9396: Bumped inventory-clamav to 1.1.0

In testing I think you have two cases which need work. On a fresh install I was rate-limited from pulling down definitions and got first
ClamAV definitions date 	$(_clamscan_version_data[definitions_date])
and then
ClamAV definitions date 	$(with)
as inventory.

Yes, that's an existing issue (not new with this version bump). It is present until you have successfully done freshclam at least once.


Comments Query failed

If we use any quotes, colons,etc in our comments  the comment query failed .
Files to change:

post.php

Hint: use real_escape string for content

Hi @chaitak-gorai Can you assign this task to me?

Hello,
@chaitak-gorai, I'm working on it, at the latest tomorrow it will be ready
Thank you for your understanding

Ok no issue

@chaitak-gorai ,
Can I ask for more information about the problem, because when I typed ": it showed up in the comments.

@mmedery
try using quotes ie '' ''  it will give a query error.


get_system_metadata() with Format=STANDARD-XML does not capture SystemID, SystemDescription or TimeZoneOffset attributes

<?xml version="1.0"?>
<RETS ReplyCode="0" ReplyText="SUCCESS" >
    <METADATA>
        <METADATA-SYSTEM Version="19.10.36933" Date="2019-10-25T15:33:19.7Z">
            <SYSTEM SystemID="ABCMLS" SystemDescription="ABC MLS" TimeZoneOffset="Z">
                <COMMENTS></COMMENTS>
            </SYSTEM>
        </METADATA-SYSTEM>
    </METADATA>
</RETS>

A call to rets_client.get_system_metadata() with Format=STANDARD-XML and rets 1.7.2 does not return SystemID, SystemDescription or TimeZoneOffset attributes due to a small bug in the code.
I will create a PR with the fix.
In RETS 1.7.2 specification (https://www.rapattoni.com/rapdocs/support/mls/retsdocs/RETS_1_7_2.pdf), the SystemID, SystemDescriptions and TimeZoneOffset are the attributes of the <SYSTEM> tag as opposed to actual separate tags.

Here is the RETS 1.5 specification (https://www.reso.org/download/rets-1-5-specification-dtd-documents/)

Bump! Any way to get this fixed?


Trees are placed in the air.

Hello,
Trees are placed in the air. How can i fix this?

Unable to find plugin 'Bridge' (referenced via aiden_geo_tutorial.uproject). Install it and try again, or remove it from the required plugin list.	aiden_geo_tutorial	D:\2022\work\cesium-procedural-foliage-main\cesium-procedural-foliage-main\Intermediate\ProjectFiles\UnrealBuildTool
这个问题怎么解决的，bro

Have you achieved it? Can you send the source code


Dashboard

Description
Minor changes and bug fixes
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[X] The code runs locally and deployed

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

per amanda we are closing this pr bc she incorporated the changes alex made into pr #82


Show message in notification

Reference #162

+1 Receiving notifications about new messages would be the most important improv imho

That is already present. This is about showing the actual message in the notification popup.

Ok I'm going to work on that based onhttps://github.com/erroneousboat/slack-term/pull/162#issuecomment-534346343
Based on the PR i also can see that it was added a new config element to show it, for me it would be not necessary, I would always show it, but I would like to know your opinion @erroneousboat .
Also I want to have format more like this:

Which will need to include the a new attribute to the configuration which is the actual Application Name, so if you have many you can also know which one is the message from (we may be able to get this from the API of slack but IDK really hehe).
If this is ok with you I'll work on it.
Waiting for confirmation.


ClosureBuilder fails with UnicodeDecodeError

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=603

Duplicate of #118


Where is the driver module for Mobileye? How can Mobileye messages be published?

I do not find any source file that implements Mobileye driver in the latest Apollo code base. So how can Mobileye messages get published? I do not find any implementation of PublishMobileye but I do notice that there is a subscriber callback ThirdPartyPerception::OnMobileye in third_party_perception module that takes into a Mobileye message.

you can find Mobileye driver in this branch https://github.com/ApolloAuto/apollo/tree/mobileye_radar/modules/drivers/mobileye.

@mickeyouyou Oh that one needs to be completely removed, and the one there is out-dated... @hongkedavid We can not open source this part due to license issue.

@mickeyouyou @Capri2014 Got it. Thanks!


fix: Use dynamic partition value

Description
Replacing hardcoded aws partition with local.partition
Motivation and Context
Solves the issue hen trying to deploy to gov-cloud accounts.
Breaking Changes
None
How Has This Been Tested?
Nope

This PR is included in version 1.6.1 :tada:


--all-namespaces shorthand

As a beginner to kubectl, I must have typed (and re-typed due to typos) --all-namespaces many hundreds of times before I stumbled across the -A shorthand notation for this.  I sincerely would be grateful on behalf of all kubectl beginners if this very useful cheat could be added to this cheatsheet.  I have proposed in this PR a location quite near the beginning of the cheatsheet for this so that it is very hard to miss.

Oops, reviewed before CLA was signed. Review will likely still apply after signing!

Oops, reviewed before CLA was signed. Review will likely still apply after signing!

I just clicked on the CLA and it does not permit individuals only corps, but I am between jobs so I don't have anyone to sign it.  My new job starts in a few weeks so I could do it then?

/easycla

Hi @bwolmarans ,
Just a reminder to [lease sign the EasyCLA
https://github.com/kubernetes/community/blob/master/CLA.md

done, ty

Thank you for addressing the feedback
/lgtm
/approve


Form elements rendering with custom attributes

If i want to use custom attributes on the alements ( eg. if i use angular ng-click, or data-ng-click )
and that is not posible because each element has the predefined $validTagAttributes protected property
formText($form->get('name')->setAttribute('ng-click','checkName()')); ?>

This is still a valid issue. I cannot use the valid attribute of "aria-label" or in fact any other aria attribute, which are key attributes to use if you care about accessibility, which everyone should.
I don't mind Zend parenting me making sure I don't accidentally add an invalid attribute, but at least allow me to use the valid ones I need.
I also have to agree with @wildlyinaccurate. Whilst I don't mind it, it shouldn't really be Zend's job or responsibility to ensure the attributes I'm using are valid (but if you ARE going to do that, at least let me use what I need!)


Refactor partitions

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5810/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5814/
Test PASSed.

alluxio-bot, merge this please


Input Channel Settings

Thanks for sharing the code.
I tried to apply the source code on my own dataset. I changed the data loader following the BRATS example to have one channel as the input and the segmentation as the second channel. Then I changed the "in_channels=5" to "in_channels=2" for the UNetModel. However, it shows the error that the expected input channel should be 3.
In addition, gaussian_diffusion.py file, the it has been set to img.shape != (1, 5, 224, 224). what's this mean?
Any suggestions will be much appreciated. Thanks!

Where did you change "in_channels=2"? In the file script_util.py? Then it should work with a 2-channel input.
Could you show me the detailed error message?
As for the second point, this means that if your image is already of the shape [conditional image, segmentation mask at time t], then you skip this step. Otherwise, you need to concatenate the original input image with the segmentation mask at time t. In your case, you would need to change this line to:
img.shape != (1, 2, image_height, image_width)

Thanks for your reply.
I can confirm that the output shape from the dataloader is torch.Size([2, 256, 256])
In the script_util.py file, I modified the in_channels for the UNetModel to 2, which shown in below.
return UNetModel(
image_size=image_size,
in_channels=2,
model_channels=num_channels,
out_channels=2,#(3 if not learn_sigma else 6),
num_res_blocks=num_res_blocks,
attention_resolutions=tuple(attention_ds),
dropout=dropout,
channel_mult=channel_mult,
num_classes=(NUM_CLASSES if class_cond else None),
use_checkpoint=use_checkpoint,
use_fp16=use_fp16,
num_heads=num_heads,
num_head_channels=num_head_channels,
num_heads_upsample=num_heads_upsample,
use_scale_shift_norm=use_scale_shift_norm,
resblock_updown=resblock_updown,
use_new_attention_order=use_new_attention_order,
)
As suggested, I have modified the gaussian_diffusion.py file to img.shape != (1, 2, image_height, image_width)
the error message is like the followings.
creating model and diffusion...
creating data loader...
training...
torch.Size([2, 256, 256])
torch.Size([2, 256, 256])
torch.Size([2, 256, 256])
torch.Size([2, 256, 256])
torch.Size([2, 256, 256])
torch.Size([2, 256, 256])
torch.Size([2, 256, 256])
torch.Size([2, 256, 256])
Traceback (most recent call last):
File "/home/anaconda3/envs/py36/lib/python3.6/pdb.py", line 1667, in main
pdb._runscript(mainpyfile)
File "/home/anaconda3/envs/py36/lib/python3.6/pdb.py", line 1548, in _runscript
self.run(statement)
File "/home/anaconda3/envs/py36/lib/python3.6/bdb.py", line 434, in run
exec(cmd, globals, locals)
File "", line 1, in
File "/mnt/Research/Diffusion-based-Segmentation/scripts/segmentation_train.py", line 3, in
"""
File "/mnt/Research/Diffusion-based-Segmentation/scripts/segmentation_train.py", line 65, in main
lr_anneal_steps=args.lr_anneal_steps,
File "/mnt/Research/Diffusion-based-Segmentation/guided_diffusion/train_util.py", line 186, in run_loop
self.run_step(batch, cond)
File "/mnt/Research/Diffusion-based-Segmentation/guided_diffusion/train_util.py", line 207, in run_step
sample = self.forward_backward(batch, cond)
File "/mnt/Research/Diffusion-based-Segmentation/guided_diffusion/train_util.py", line 238, in forward_backward
losses1 = compute_losses()
File "/mnt/Research/Diffusion-based-Segmentation/guided_diffusion/gaussian_diffusion.py", line 937, in training_losses_segmentation
model_output = model(x_t, self._scale_timesteps(t), **model_kwargs)
File "/home/anaconda3/envs/py36/lib/python3.6/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/home/anaconda3/envs/py36/lib/python3.6/site-packages/torch/nn/parallel/distributed.py", line 799, in forward
output = self.module(*inputs[0], **kwargs[0])
File "/home/anaconda3/envs/py36/lib/python3.6/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/mnt/Research/Diffusion-based-Segmentation/guided_diffusion/unet.py", line 657, in forward
h = module(h, emb)
File "/home/anaconda3/envs/py36/lib/python3.6/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/mnt/Research/Diffusion-based-Segmentation/guided_diffusion/unet.py", line 74, in forward
x = layer(x)
File "/home/anaconda3/envs/py36/lib/python3.6/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/home/anaconda3/envs/py36/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 443, in forward
return self._conv_forward(input, self.weight, self.bias)
File "/home/anaconda3/envs/py36/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 440, in _conv_forward
self.padding, self.dilation, self.groups)
RuntimeError: Given groups=1, weight of size [128, 2, 3, 3], expected input[8, 3, 256, 256] to have 2 channels, but got 3 channels instead
Uncaught exception. Entering post mortem debugging
Running 'cont' or 'step' will restart the program
Thanks for your help!


Use custom media types for request and responses

Custom media types will make it easier for us to iterate on request and response formats for API methods. It will also make migrating to new formats and maintaining backward compatibility much more straightforward.
Implementation plan:

Experiment with media types in webhook implementation #42. For this, we'll try to use application/vnd.docker.distribution.events.v1+json for the event format.
Set baseline use of application/json to refer to the current specification.
Define custom media types for other formats (manifest, layers, etc.). Here are some possible examples:

Layer: application/vnd.docker.distribution.layer+tar
Manifest: application/vnd.docker.distribution.manifest.v1+json
Signed Manifest: application/vnd.docker.distribution.manifest.v1+prettyjws

Discussion now a part of OCI artifacts spec