fix(schema): Fix OS3 description on QueryParams and nullable options

Information

Type
Breaking change

Fix
No

Todos

[x] Tests
[x] Coverage
[ ] Example
[ ] Documentation

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

v6.11.2
GitHub release

Your semantic-release bot :package::rocket:


Merge pull request #6 from kevindimayuga/main

Models

no changes were made with this PR, next PR has model changes from Jacob


Error status is not updated

Expected behaviour
Error for incorrect dates shouldn't be displayed (step 6)
Value for Retail Turnover shouldn't be retained (Step 9)
Actual behaviour
Error for incorrect dates is displayed (step 6)
Value for Retail Turnover is retained (Step 9)
Steps to reproduce the behaviour

Go to questionnaire page (MCI)
Enter incorrect dates (From Date> To Date)
Enter Retail Turnover
Click on Save & Continue
Error message is displayed for Dates (as Expected)
Now correct the dates and remove the value for Retail Turnover (mandatory field)
Click on Save & Continue
Error message for both dates and retail value is displayed.
Value of retail turnover (from step 3) is retained.

Technical information
IE11
Browser
IE11
Operating System
windows 7
Screenshot

Confirmed, entering a value, then later removing it causes the previous value to be displayed in the input box but with an error that the value is missing.

Confirmed this is fixed by #385 on pre-prod. Closing.


Improve CMake Setup So CMake Users Can Easily Consume Library

Added

Library target for easy-socket. This name can be changed
Version of library in project declaration
All needed config files and install rules so distribution is easy for CMake users

This will make it easy to make this library available on vcpkg or something similar.

Option to turn off building of client/server tests so consumers aren't forced to build these projects.

Fixed

Windows build using CMake was broken. It was necessary to link to wsock32 and ws2_32.

Notes:

I only tested the Windows build. Not sure what is/could be broken on Linux.

Looks good to me!
I will test the Linux version on Monday and report back any issues, if any.
Thank you!

@masesk I was able to test building my changes on WSL via Visual Studio using Clang and GCC and everything built fine, just an FYI 👍

Everything seems to work correctly aside from the following:
make install  seems to output to
/usr/local/include-0.2.0/include/masesk/EasySocket.hpp
I believe this is due to line 54-57, where the include install puts it in the word include and version.
On Linux, this should be /usr/local/include/masesk/EasySocket.hpp
Also, I couldn't find any reason why we store the easy-socketTargets.cmake, easy-socketConfigVersion.cmake, and easy-socketConfig.cmake. I have no experience in this matter so any information would be helpful :)

Yes I see that now you're right, that's the wrong include path. That should be a simple fix

Also, I couldn't find any reason why we store the easy-socketTargets.cmake, easy-socketConfigVersion.cmake, and easy-socketConfig.cmake. I have no experience in this matter so any information would be helpful :)

These files are used by consumers of the library (assuming they are using CMake). So in a consumer's CMakeLists.txt, they going to require a dependency on your lib
find_package(easy-socket REQUIRED)
By default, CMake has a number of paths that it uses to search to find easy-socket. It's going to first try to find a Find Module and when it can't, it will look for a config file. All config files have to be named <library name>Config.cmake. So providing these files will make it easy to integrate your library into their project. Once the location of the easy-socketConfig.cmake file is specified they can just link to your project in one line:
target_link_libraries(MyProject PUBLIC easy-socket)
Hopefully that made sense; let me know if there are further questions.

I was able to fix the include directory install location.

Awesome work!
Similarly, I think the installation paths for all the config files are:
/usr/local/lib/cmake/easy-socket-0.2.0/easy-socket/easy-socketConfig.cmake
/usr/local/lib/cmake/easy-socket-0.2.0/easy-socket/easy-socketTargets.cmake
/usr/local/lib/cmake/easy-socket-0.2.0/easy-socket/easy-socketConfigVersion.cmake
I think the easy-socket-0.2.0 should not be there. Otherwise the find_package(easy-socket REQUIRED) won't find it. Correct me if I am wrong.

That is there on purpose to support side by side installation of multiple versions. If you don't think it's needed I can remove it.


[WIP] Enhancement: Added GitHub Repository Service

Added GithubService and moved controller methods to service

Merged after discussing with @ins0 on Skype.
Thanks a lot, @ins0!


[Question]: Pass props/data from liquid to vue component

Operating system
Mac
Node.js version
16
Shopify CLI version
2.0
Browsers
Chrome
Version
4 (Current)
Modifications
No response
Details
Question.
How can I pass an array of items/products into a vue component ? (I would prefer pass data instead of using slots)
For instance:
 <render-products-slider :items="{{ products }}"></render-products-slider>

I came from working with twig and in twig I would use a filter: :items="{{ products | json_encode }}"
What people use to do something similar to that, for example to pass images into a gallery, or other objects/arrays into a vue component ?
Thank you!
Notice

[X] I read the notice

Reading this fixed my issue:
https://themelab.uicrooks.com/guide/troubleshooting.html


Weird compiler error

Hi
I am trying to use the 0.4.3-kafka09 release and I get this really weird error when building my tests with maven.
[ERROR] error: missing or invalid dependency detected while loading class file 'KafkaUnavailableException.class'. [INFO] Could not access type RuntimeException in package scala.package, [INFO] because it (or its dependencies) are missing. Check your build definition for [INFO] missing or conflicting dependencies. (Re-run with-Ylog-classpathto see the problematic classpath.) [INFO] A full rebuild may help if 'KafkaUnavailableException.class' was compiled against an incompatible version of scala.package.
Have you seen this before? I'm building with Scala 2.11.6 using Scala Maven plugin 3.2.0

Hi,
No I haven't. I'm currently away with no laptop access but when I'm back next week I'll try to have a look. Are you able to access the RuntimeException class from your project?

Can it be related to me building with 2.11.x and the release is built on 2.10.x?

There is a 2.11.x available as well. If you're using 2.11 you should use
the 2.11 artifact.
On Fri, 18 Mar 2016 at 13:34, m-thread<EMAIL_ADDRESS>wrote:

Can it be related to me building with 2.11.x and the release is built on
2.10.x?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/manub/scalatest-embedded-kafka/issues/10#issuecomment-198359379

Good point. But it still fails with the same issue :( It's probably something on my side.

Hi again. I managed to get things to work. It turned out that I had to use 2.11.7 in my build to get things to work. I used 2.11.6 before and that gave me the compiler error.
Cheers
/Fredrik

Thanks for the update Frederik!

Had the same issue, thanks for the solution

Make sure scalaVersion the embedded-kafka is compiled against is same as the version you are using in your project. With the latest embedded-kafka, it works for 2.11.7, but not 2.11.5.


helmfile sync never process deployment CentOS 7.3, Ubuntu 16.04.4 LTS

Hi there,
I just wanted to rewrite this piece of working code:
helm install stable/prometheus --name prom --set rbac.create=false --namespace=prometheus

my simple helmfile:
vim charts.yaml

...
context:  <my_context>
releases:
  # Published chart example
  - name: prometheusnorbac                            # name of this release
    namespace: prometheus                       # target namespace
    chart: stable/prometheus     # the chart being installed to create this release, referenced by `repository/chart` syntax
    set:                                   # values (--set)
      - name: rbac.create
        value: false
...
wq!

When I run:
./helmfile -f charts.yaml sync
exec: helm repo update
Hang tight while we grab the latest from your chart repositories...
...Skip local chart repository
...Successfully got an update from the "stable" chart repository
Update Complete. ⎈ Happy Helming!⎈

Then chcek helm list
 helm list
NAME            REVISION        UPDATED                         STATUS          CHART                   NAMESPACE
prom            1               Thu Mar 15 14:22:19 2018        DEPLOYED        prometheus-5.4.3        prometheus
promt           1               Fri Mar 16 11:50:31 2018        FAILED          prometheus-5.4.3        prometheus
promtemp        1               Fri Mar 16 10:58:31 2018        DEPLOYED        prometheus-5.4.3        prometheus

So from helm list output can be seen that deployment for prometheusnorbac was never finished.
Tested OS's

Ubuntu 16.04.4 LTS
Centos 7.3

Hi @xjantoth Sorry for the inconvenience. We've recently changed the key from charts: to releases:, and the binary had not been released since then.
But now, we have v0.10 available for you! Would you mind trying once again with the new version? Thx.

Hi @mumoshu thanks for a new version 0.10:

vim helmfile.yaml

...
context: <my_context>                   # kube-context (--kube-context)

releases:
  # Published chart example
  - name: prometheusnorbac                            # name of this release
    namespace: prometheus                       # target namespace
    releases: stable/prometheus     # the chart being installed to create this release, referenced by `repository/chart` syntax
    set:                                   # values (--set)
      - name: rbac.create
        value: false
...

Does this error message mean that I have to take my code  form helmfile.yaml and place  to
/home/jantoth/.helm/repository/Charts.yaml
 ./helmfile -f helmfile.yaml sync
exec: helm repo add roboll http://roboll.io/charts --kube-context <my_context>
"roboll" has been added to your repositories
exec: helm repo update --kube-context <my_context>
Hang tight while we grab the latest from your chart repositories...
...Skip local chart repository
...Successfully got an update from the "stable" chart repository
...Successfully got an update from the "roboll" chart repository
Update Complete. ⎈ Happy Helming!⎈
exec: helm upgrade --install prometheusnorbac  --namespace prometheus --set rbac.create=false --kube-context sandbox.aws.zoomint.com
Release "prometheusnorbac" does not exist. Installing it now.
Error: no Chart.yaml exists in directory "/home/jantoth/.helm/repository"
err: exit status 1

@xjantoth Hi!
I've tried that myself and it turned out that you should change releases: stable/prometheus to chart: stable/prometheus.

@mumoshu Hi!
Thank you so much 👍 I have successfully deployed basic prometheus setup on k8s by using helmfile.
Much appreciate your instant help.
We will be testing it for a while from now.
Maybe you would like to post this update to your git README file:
-Jan
cat helmfile.yaml

 cat charts.yaml
context: <my_context>

releases:
  # Published chart example
  - name: promnorbacxubuntu         # name of this release
    namespace: prometheus              # target namespace
    chart: stable/prometheus             # the chart being installed to create this release, referenced by `repository/chart` syntax
    set:                                   # values (--set)
      - name: rbac.create
        value: false

Usage:
./helmfile -f hemlfile.yaml sync

@xjantoth Thanks for your confirmation and the suggestion. Glad it worked for you!
Regarding the suggestion, I've submitted #57. Would you mind taking a look? Thx!


Lots of duplicated resources in hello-visualization

Webpack configurations, font files, HTML, etc. The two examples are so similar that it feels weird to not have them on a combined page instead and share resources. Would be easier for the end-users getting started to as it's way less code to read/learn.

It's basically line-chart.js and scatterplot.js that differs out of 10+ files, everything else is pretty much identical (1000+ lines of code), shared or not I think the folders should be merged, but perhaps that's what you mean with a common folder?


fix: deduplicate resources when creating policies

This change looks looks fine, just need to make sure that it deduplicates on the same level as the original implementation. The reason is that there is an policy application limit for AWS Lambda accounts that this was implemented to address, but it appears that this change may undo that.
The intention here was to ensure that all resource types that shared a common set of actions were grouped into a single policy, so in the case of something like read-only secrets we could merge them to save on IAM policy length.
e.g.
functionA:read:secret1 + functionA:read:secret2 + functionA:read:secret3 => functionA:read:secret1,secret2,secret3


Imlementation of Entirely new Embedded Architecture for Rust

I was wondering how one would go about creating a target for unsupported embedded architectures (as of now) for the Rust Embedded Ecosystem ?. Eg the 8051 families of Microcontrollers

Step one: Figure out a way to get 8051 support into LLVM, similar to the AVR project https://github.com/avr-llvm/llvm.


Driver versions needed?

Does this not support AMD GPUs?
I'm getting this error:
terminate called after throwing an instance of 'tvm::runtime::InternalError'
  what():  [18:53:29] /home/runner/work/utils/utils/tvm/src/runtime/vulkan/vulkan_instance.cc:111:
---------------------------------------------------------------
An error occurred during the execution of TVM.
For more information, please see: https://tvm.apache.org/docs/errors.html
---------------------------------------------------------------
  Check failed: (__e == VK_SUCCESS) is false: Vulkan Error, code=-9: VK_ERROR_INCOMPATIBLE_DRIVER
Stack trace:
  [bt] (0) /home/loganm/miniconda3/envs/llama_index/bin/../lib/libtvm_runtime.so(tvm::runtime::Backtrace[abi:cxx11]()+0x27) [0x7f6fa5518a37]
  [bt] (1) /home/loganm/miniconda3/envs/llama_index/bin/../lib/libtvm_runtime.so(+0x3f375) [0x7f6fa54b6375]
  [bt] (2) /home/loganm/miniconda3/envs/llama_index/bin/../lib/libtvm_runtime.so(tvm::runtime::vulkan::VulkanInstance::VulkanInstance()+0x1a47) [0x7f6fa5605857]
  [bt] (3) /home/loganm/miniconda3/envs/llama_index/bin/../lib/libtvm_runtime.so(tvm::runtime::vulkan::VulkanDeviceAPI::VulkanDeviceAPI()+0x40) [0x7f6fa5601a60]
  [bt] (4) /home/loganm/miniconda3/envs/llama_index/bin/../lib/libtvm_runtime.so(tvm::runtime::vulkan::VulkanDeviceAPI::Global()+0x4c) [0x7f6fa5601dfc]
  [bt] (5) /home/loganm/miniconda3/envs/llama_index/bin/../lib/libtvm_runtime.so(+0x18ae3d) [0x7f6fa5601e3d]
  [bt] (6) /home/loganm/miniconda3/envs/llama_index/bin/../lib/libtvm_runtime.so(+0x6bdc4) [0x7f6fa54e2dc4]
  [bt] (7) /home/loganm/miniconda3/envs/llama_index/bin/../lib/libtvm_runtime.so(+0x6c367) [0x7f6fa54e3367]
  [bt] (8) mlc_chat_cli(+0xd950) [0x55ed543c8950]

Aborted

Here's my driver info

Yes, i think there are ppl successfully running on AMD GPUs, please try to upgrade to the latest driver and see if it works

the same error
mlc_chat_cli
terminate called after throwing an instance of 'tvm::runtime::InternalError'
what():  [21:35:46] /home/runner/work/utils/utils/tvm/src/runtime/vulkan/vulkan_instance.cc:111:
An error occurred during the execution of TVM.
For more information, please see: https://tvm.apache.org/docs/errors.html
Check failed: (__e == VK_SUCCESS) is false: Vulkan Error, code=-9: VK_ERROR_INCOMPATIBLE_DRIVER
Stack trace:
[bt] (0) /root/miniconda3/envs/mlc-chat/bin/../lib/libtvm_runtime.so(tvm::runtime::Backtraceabi:cxx11+0x27) [0x7f4190835a37]
[bt] (1) /root/miniconda3/envs/mlc-chat/bin/../lib/libtvm_runtime.so(+0x3f375) [0x7f41907d3375]
[bt] (2) /root/miniconda3/envs/mlc-chat/bin/../lib/libtvm_runtime.so(tvm::runtime::vulkan::VulkanInstance::VulkanInstance()+0x1a47) [0x7f4190922857]
[bt] (3) /root/miniconda3/envs/mlc-chat/bin/../lib/libtvm_runtime.so(tvm::runtime::vulkan::VulkanDeviceAPI::VulkanDeviceAPI()+0x40) [0x7f419091ea60]
[bt] (4) /root/miniconda3/envs/mlc-chat/bin/../lib/libtvm_runtime.so(tvm::runtime::vulkan::VulkanDeviceAPI::Global()+0x4c) [0x7f419091edfc]
[bt] (5) /root/miniconda3/envs/mlc-chat/bin/../lib/libtvm_runtime.so(+0x18ae3d) [0x7f419091ee3d]
[bt] (6) /root/miniconda3/envs/mlc-chat/bin/../lib/libtvm_runtime.so(+0x6bdc4) [0x7f41907ffdc4]
[bt] (7) /root/miniconda3/envs/mlc-chat/bin/../lib/libtvm_runtime.so(+0x6c367) [0x7f4190800367]
[bt] (8) mlc_chat_cli(+0xe010) [0x5654d12ee010]
gpu info

Yea I've updated all my drivers. I wish there was an easier way to debug haha

@logan-markewich does it work now with your AMD GPU? If so, would you mind sharing your tok/sec info here?

@junrushao I should have clarified, I updated everything but couldn't get it to work haha
Not sure how to debug it either

I just bought an AMD GPU and will see if it works locally on my machine


Export the reload_service Icinga plugin

Story: https://trello.com/c/8U5ou5rr/77-gracefully-kill-unicorn-workers-if-they-use-too-much-ram
The reload_service plugin needs to be installed on every machine, so
make it an exported resource that's collected by the icinga::client
class.

I understand that <| |> is a resource collector and <<| |>> is an exported resource collector (docs) but I can't really connect the dots here.
Changing to a @@ means the plugin is exported from the monitoring machine and now will be collected on all machines, which makes sense. But then what does the line Icinga::Plugin <| |> actually do?

<| |> will realise the virtual resources; <<| |>> will collect the exported resource I've created. The Icinga::Plugin <| |> is still needed as many of our modules have virtual Icinga::Plugin resources which need to be realised by that line.

Is there any reason this needs to be exported and can't just be a normal Icinga::package resource?

@deanwilson monitoring::event_handler::app_high_memory is only installed on the monitoring box, whereas this plugin needs to be installed on all boxen. I wanted to keep its definition close to the thing that uses it in the code for better encapsulation, rather than separate it into another class.

What can I do to help get this merged? Happy to talk through the code with someone.

This makes sense to me. Would also like to get this merged. @alexmuller @deanwilson are you happy with it?

Pleeeeeeease?

Can we add a tag to the export and collection please? That'll stop all the plugins being realised when that code is hit. https://github.com/voxpupuli/puppet-lint-spaceship_operator_without_tag-check explains the thinking behind it.

@deanwilson what should the tag be? Is virtual == true sufficient in this case or should it be something specific?

Something context specific. I'm being a little vague as naming things is difficult and to be honest the name isn't as important as the existence of the restricting tag.

We should get this merged today. I'm going to speak to @deanwilson and gang later.

@deanwilson From what I understand of the bug report, the bug only applies to virtual resources (as opposed to exported resources, which this PR adds) and only poses a problem if specifying dependencies. Is that correct? So from my understanding, the code added here would not make the problem worse.
If so, can I suggest that we tackle it in a separate PR so that we can unblock this one?

After another discussion we've looked at having the plugin as a distinct resource from the handler so it can be reused by other handlers, such as high CPU. This also removed the complexity of exporting and collecting.

Superseded by https://github.com/alphagov/govuk-puppet/pull/3899.

The above link should be #4074.


⚠ Bofrost.de has degraded performance

In 8b011df, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1397 ms

Resolved: Bofrost.de performance has improved in 9b6810f.


Stuck resource for <IP_ADDRESS>/24 that doesn't reflect reality

Found while running an IRRExplorer query for AS1280, there's a block:
<IP_ADDRESS>/24 | APNIC |   |   | 1280 | Route objects exist, but prefix not seen in DFZ No (covering) RPKI ROA found for route objects
Drilling down to the apnic registry yields:
whois -h whois.apnic.net <IP_ADDRESS>/24
route:          <IP_ADDRESS>/24
origin:         AS1280
descr:          APNIC Research and Development
6 Cordelia St
mnt-by:         MAINT-AU-APNIC-GM85-AP
last-modified:  2018-11-30T00:16:42Z
source:         APNIC
Except that that whois query does not yield that.  I suspect IRRexplorer has it cached somehow and there's no way to force a clear.  (This has been like this for more than a week).

This problem seems to be specifically related to the hosted version on https://irrexplorer.nlnog.net, my local copy doesn't show this.
There were a large number of old entries in the database. I've cleaned it, things seem to be better now.


awx_job_template not wait for the first scm update of the awx_project to completed prior to it creation

Hi
At the moment we are trying to using this provider to deploy our projects and templates in several AWX instances, and we notice that the default behavior of the awx_job_template is that it would not wait for the first scm update of the awx_project to completed prior to it creation, leading to cases where the awx_job_template creation failure (since at that time the playbook(s) of the projects still haven't been cloned/checked out yet.
Is this the intended behavior? Are the provider going to support some flag that ensure the latest job of /project_updates/ endpoint for that project is done before marking the awx_project as complete? Any recommended workaround at the moment?
I have checked the https://github.com/denouche/goawx/ repo but doesn't seem like the client support doing that yet.

Hi,
I'm facing the same issue, and the best I've come up with so far is a pretty ugly Terraform resource targeting:
- terraform ${TF_GLOBAL_OPTIONS} apply -auto-approve -target=awx_project.project
- sleep 10
- terraform ${TF_GLOBAL_OPTIONS} apply -auto-approve -target=awx_job_template.check -target=awx_job_template.run


Can't retrieve any story

Hey!
First of all, this seems to be an amazing tool!
I've downloaded the story scraper.
I can't scrap any story, I tried with many usernames.
Here are the errors:
[SUCCESS]: Got into the story link.

Traceback (most recent call last):
  File "c:\Users\Kevin\Desktop\IG Scraper\insta_story_scraper.py", line 64, in <module>
    driver.find_element_by_xpath("/html/body/div[1]/section/div[1]/div/section/div/div[1]/div/div/div/div[3]/button").click()
  File "C:\Users\Kevin\AppData\Local\Programs\Python\Python39\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 394, in find_element_by_xpath
    return self.find_element(by=By.XPATH, value=xpath)
  File "C:\Users\Kevin\AppData\Local\Programs\Python\Python39\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 976, in find_element
    return self.execute(Command.FIND_ELEMENT, {
  File "C:\Users\Kevin\AppData\Local\Programs\Python\Python39\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 321, in execute
    self.error_handler.check_response(response)
  File "C:\Users\Kevin\AppData\Local\Programs\Python\Python39\lib\site-packages\selenium\webdriver\remote\errorhandler.py", line 242, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.NoSuchElementException: Message: no such element: Unable to locate element: {"method":"xpath","selector":"/html/body/div[1]/section/div[1]/div/section/div/div[1]/div/div/div/div[3]/button"}

Thank you!

Thank u for the info. As u can see, the code wasn't maintained for over a year, so it was a good opportunity for me to get back to it, and introduce some changes. Now, it works fine and u can also download the scraped content. I tried with multiple usernames, I took a sreenshot of the last example:

I hope it helps you. I'll also do a code review for the feed and hashtag scraping in the next days.


How should I get my SBI implementation ID?

I implemented my own SBI implementation for my team's project, and wonder how I can get my SBI implementation ID. Is there a test, registry or something else that I should complete first before I can get my own specific ID for my SBI?
Here is my SBI implementation, written in Rust. It implements Base, Time, sIP extensions, and a subset of Legacy ones. I'm now using it on my team's project to replace the old version of RustSBI that's now in use. The PsicaSBI can now run on k210 platform and QEMU.

@retrhelo
maybe you can reference this issue

I see. But this rule seems to be for Extension ID. Is SBI ID following the same rule?

@retrhelo
in my understand
Document that in a patch for the SBI document Send it to the Mailing List (You need to be subscribed to be able to post) and create a PR on GitHub
e.g [PATCH] Add system reboot extension

All Right, I'll check it out. Thanks. :)

No activity on this issue. Closing this one. Please reopen if you have still some issues.


Redundant Space at top of Bar Chart

Hi,
Just wondering if it's possible to get rid of the redundant space at the top of the bar chart - highlighted the background in red to show the space at the top:

Thanks
.

Sorry, also, is there a way that we can set the space on the left of the chart to 0 for times when we don't want the y axis showing?

Hi @verofy-simon-denny 👋
Regarding your first question -
Yes, we can remove the extra padding at the top, by using the maxValue prop.
Here's a chart without using the maxValue prop-

The code for the above chart is -
const barData = [{value: 15}, {value: 30}, {value: 26}, {value: 40}];
<BarChart backgroundColor={'red'} data={barData} />

Now, if we add maxValue={40} -
const barData = [{value: 15}, {value: 30}, {value: 26}, {value: 40}];
<BarChart backgroundColor={'red'} maxValue={40} data={barData} />

The output becomes-

Notice the padding at the top is now gone!
How it works-
If we don't provide the maxValue, it computes the maxValue as follows-
It takes the biggest value in the data array and the next reading on the y axis becomes the maxValue.
For example, in the above case, the biggest value in the data array is 40 so the next reading on y axis, i.e. 50 becomes the maxValue.

Thank you for the answers, appreciate the time and the great library.
It makes sense, but I can't seem to set the height and max value at the same time, am I doing something wrong here?
I've used your example, but added height:
<BarChart             backgroundColor={'red'}             maxValue={40}             data={barData}             height={140}           />

Many Thanks, Simon.

Yes @verofy-simon-denny, Even I faced the same issue. But the fact is there's no issue as such. I reloaded the app and it worked like a charm💫✨
Please try reloading the app after changing any of these props- maxValue, stepValue, stepHeight, height
If it's not solved even after reloading, please do let me know, I will work on it.

Thanks that works now.

Thank you for the answers, appreciate the time and the great library.
It makes sense, but I can't seem to set the height and max value at the same time, am I doing something wrong here?
I've used your example, but added height:
<BarChart backgroundColor={'red'} maxValue={40} data={barData} height={140} />

Many Thanks, Simon.

Hi @verofy-simon-denny ,
I am facing the same issue. But it only happens when I use the topLabelComponent. Otherwise, it is working fine.
Please suggest any answer.
Thanks in advance.


"No map" for routes monitored without reference

The map is empty and the message "No map" and smileys are displayed.
Related to #299
Example: R2 Réunion - Traversée de la Réunion

KPN display segment list but display nothing.
This is probably because you don't have OSM data for this part of the world?
Is it the same issue or another ?
GR R1 Tour du Piton des Neiges

@pyrog thanks for reporting this issue
Fixed in release 4.0.35.
The issue with both routes was the same: the map did not get displayed for latitudes less than zero. This has been fixed in the code.
The knooppuntnet overpass database contains all OSM data for the entire world (and the complete history since september 2012).
The knooppuntnet background tiles however are only generated for Austria, Belgium, Germany, Luxembourg, Netherlands, Poland, Spain and France, but all of this only within bounding box -9.3, 36.0, 24.5, 55.15. This is why for given routes by default no background tiles are displayed. You can enable the OpenStreetMap tiles in the layers popup menu. The OSM tiles have to be re-enabled every time a new map is displayed, this is done because we still want the knooppuntnet tiles to always be the default (osm tile usage policy).

Thank you for the fix and the explanations.
OSM tiles are nice (to see i.e. guideposts) 😎
…so I added feature request #301


Migrate to ProjectPythia org for publishing

Hi @cmdupuis3! I've invited you to be a member of the ProjectPythia organization, for which you should receive an email and notification. Once you accept this, please go to your repo Settings > Danger Zone > Transfer Ownership and Transfer it to the ProjectPythia organization (you'll still have full access to your repo.) Once you do so, then we can restart the machinery in ProjectPythia/cookbook-gallery#119 to publish this!

Done


Update doc of global_norm

@findmyway: What's wrong with the documentation? It is correctly documenting squared operation vs multiplication.

Emm, I thought it was l2_loss. Sorry to disturb.


IP not assigned to VM error when updating the VM object

2020-11-19 02:09:34,920 - INFO: Creating new NetBox 'virtual machine' object 'sav-v1-vcenter' with data: {'tags': [{'name': 'NetBox-synced'}, {'name': 'Source: sav'}], 'name': 'sav-v1-vcenter', 'cluster': 4, 'role': 5, 'status': 'active', 'memory': 19456, 'vcpus': 4, 'disk': 499, 'platform': 9, 'comments': 'VMware vCenter Server Appliance', 'primary_ip4': 252}
,
 'nat_inside': None,
 'nat_outside': None,
 'role': None,
 'status': {'label': 'Active', 'value': 'active'},
 'tags': [{'color': '9e9e9e',
           'id': 11,
           'name': 'NetBox-synced',
           'slug': 'netbox-synced',
           'url': 'http://scrubbed/api/extras/tags/11/'},
          {'color': '9e9e9e',
           'id': 12,
           'name': 'Source: sav',
           'slug': 'source-sav',
--
header: X-Content-Type-Options: nosniff
header: Referrer-Policy: same-origin
header: X-Frame-Options: SAMEORIGIN
{'primary_ip4': ['The specified IP address (<IP_ADDRESS>/24) is not assigned '
                 'to this VM.']}
{'_body_position': None,
 '_cookies': <RequestsCookieJar[]>,
 'body': b'{"tags": [{"name": "NetBox-synced"}, {"name": "Source: sav"}], "name'
         b'": "sav-v1-vcenter", "cluster": 4, "role": 5, "status": "active", "m'
         b'emory": 19456, "vcpus": 4, "disk": 499, "platform": 9, "comments": "'
         b'VMware vCenter Server Appliance", "primary_ip4": 252}',
 'headers': {'User-Agent': 'netbox-sync/0.0.1', 'Accept-Encoding': 'gzip, deflate', 'Accept': '*/*', 'Connection': 'keep-alive', 'Authorization': 'Token 79c5341a8229cef4dc3800b526585d459d04f4b4', 'Content-Length': '257', 'Content-Type': 'application/json'}2020-11-19 02:09:34,921 - DEBUG2: Sending POST to 'http://<IP_ADDRESS>:8000/api/virtualization/virtual-machines/?limit=200&exclude=config_context' with data 'b'{"tags": [{"name": "NetBox-synced"}, {"name": "Source: sav"}], "name": "sav-v1-vcenter", "cluster": 4, "role": 5, "status": "active", "memory": 19456, "vcpus": 4, "disk": 499, "platform": 9, "comments": "VMware vCenter Server Appliance", "primary_ip4": 252}''.
2020-11-19 02:09:34,951 - DEBUG: http://<IP_ADDRESS>:8000 "POST /api/virtualization/virtual-machines/?limit=200&exclude=config_context HTTP/1.1" 400 89
2020-11-19 02:09:34,952 - DEBUG2: Received HTTP Status 400.
2020-11-19 02:09:34,952 - DEBUG: Response Body:
2020-11-19 02:09:34,952 - ERROR: NetBox returned: POST /api/virtualization/virtual-machines/?limit=200&exclude=config_context Bad Request
2020-11-19 02:09:34,952 - ERROR: NetBox returned body: {'primary_ip4': ['The specified IP address (<IP_ADDRESS>/24) is not assigned to this VM.']}
2020-11-19 02:09:34,952 - ERROR: Request Failed for virtual machine. Used data: {'tags': [{'name': 'NetBox-synced'}, {'name': 'Source: sav'}], 'name': 'sav-v1-vcenter', 'cluster': 4, 'role': 5, 'status': 'active', 'memory': 19456, 'vcpus': 4, 'disk': 499, 'platform': 9, 'comments': 'VMware vCenter Server Appliance', 'primary_ip4': 252}
2020-11-19 02:09:34,956 - INFO: Creating new NetBox 'virtual machine interface' object 'vNIC 1 (2000-sav_management-<IP_ADDRESS>%2f24) (sav-v1-vcenter)' with data: {'tags': [{'name': 'NetBox-synced'}, {'name': 'Source: sav'}], 'name': 'vNIC 1 (2000-sav_management-<IP_ADDRESS>%2f24)', 'mac_address': '00:0C:29:DD:6B:55', 'description': 'Network adapter 1 (VirtualVmxnet3) (vlan ID: 2000)', 'enabled': True, 'mtu': 1500, 'mode': 'access', 'untagged_vlan': 63}
2020-11-19 02:09:34,956 - DEBUG2: Sending POST to 'http://<IP_ADDRESS>:8000/api/virtualization/interfaces/?limit=200&exclude=config_context' with data 'b'{"tags": [{"name": "NetBox-synced"}, {"name": "Source: sav"}], "name": "vNIC 1 (2000-sav_management-<IP_ADDRESS>%2f24)", "mac_address": "00:0C:29:DD:6B:55", "description": "Network adapter 1 (VirtualVmxnet3) (vlan ID: 2000)", "enabled": true, "mtu": 1500, "mode": "access", "untagged_vlan": 63}''.
2020-11-19 02:09:34,977 - DEBUG: http://<IP_ADDRESS>:8000 "POST /api/virtualization/interfaces/?limit=200&exclude=config_context HTTP/1.1" 400 47
2020-11-19 02:09:34,977 - DEBUG2: Received HTTP Status 400.
2020-11-19 02:09:34,977 - DEBUG: Response Body:
2020-11-19 02:09:34,977 - ERROR: NetBox returned: POST /api/virtualization/interfaces/?limit=200&exclude=config_context Bad Request
2020-11-19 02:09:34,977 - ERROR: NetBox returned body: {'virtual_machine': ['This field is required.']}
2020-11-19 02:09:34,977 - ERROR: Request Failed for virtual machine interface. Used data: {'tags': [{'name': 'NetBox-synced'}, {'name': 'Source: sav'}], 'name': 'vNIC 1 (2000-sav_management-<IP_ADDRESS>%2f24)', 'mac_address': '00:0C:29:DD:6B:55', 'description': 'Network adapter 1 (VirtualVmxnet3) (vlan ID: 2000)', 'enabled': True, 'mtu': 1500, 'mode': 'access', 'untagged_vlan': 63}
2020-11-19 02:09:34,977 - DEBUG2: Adding unresolved dependencies back to object: ['virtual_machine']
2020-11-19 02:09:34,977 - DEBUG2: Parsing 'virtual machine interface' data structure: None (sav-v1-vcenter)
2020-11-19 02:09:34,992 - INFO: Updating NetBox 'IP address' object '<IP_ADDRESS>/24' with data: {'assigned_object_type': 'virtualization.vminterface'}
2020-11-19 02:09:34,993 - DEBUG2: Sending PATCH to 'http://<IP_ADDRESS>:8000/api/ipam/ip-addresses/252/?limit=200&exclude=config_context' with data 'b'{"assigned_object_type": "virtualization.vminterface"}''.
2020-11-19 02:09:35,084 - DEBUG: http://<IP_ADDRESS>:8000 "PATCH /api/ipam/ip-addresses/252/?limit=200&exclude=config_context HTTP/1.1" 200 890
2020-11-19 02:09:35,084 - DEBUG2: Received HTTP Status 200.
2020-11-19 02:09:35,084 - DEBUG: Response Body:
2020-11-19 02:09:35,084 - DEBUG2: Adding unresolved dependencies back to object: ['assigned_object_id']
2020-11-19 02:09:35,084 - DEBUG2: Parsing 'IP address' data structure: <IP_ADDRESS>/24
2020-11-19 02:09:35,085 - DEBUG: Ip address '<IP_ADDRESS>/24' attribute 'assigned_object_id' changed from 'None' to 'vNIC 1 (2000-sav_management-<IP_ADDRESS>%2f24) (sav-v1-vcenter)'
2020-11-19 02:09:35,085 - DEBUG: Pruning disabled. Skipping
2020-11-19 02:09:35,085 - INFO: Completed NetBox Sync in 37.17 seconds

Not seeing much to go on here, let me how to help debug further!

I realized what's going on here, this IP already had a manual entry in netbox. On removing it, this synced properly.
Maybe just need a more descriptive error message here and leave it up to the user to manually resolve?

Hi, yes this was an issue. Fix will be pushed today,
primary_ip caused some headaches as we run into circular dependencies.
For an IP to be assigned as "primary_ip" the IP needs to be assigned to an interface of the machine.
New machine flow:

create device
create interfaces
add IPs to interfaces
assign an IP of an interface as primary IP.

But this is solved with the next commit.

Can you test the "main" branch again please.
thank you.

Seems to be working, however it's not checking to see if a VM has a disconnected nic before syncing the IP over. I've got two VMs with the same ip configured, but one has its network cards disconnected. Sync associates the IP to the one with the disconnected nics since it sees it first, then throws an error on the connected one.

added new branch with the changed beahvior. Can you test it please?

Better! Although it doesn't seem to remove a disconnected nic's IP unless it conflicts with another VM. IE, I have two VMS with two nics, the IP conflict is only on one of the nics. One VM has both nics disconnected and the conflicting IP was removed, but the non-conflicting IP address is still there.
For consistency of "we don't associate an IP unless the NIC is connected", that other IP address should probably get removed as well.
Can test this by:

Adding a NIC to a VM, having it connected, putting an IP on it
Run sync, make sure it gets associated
Disconnect the nic in vmware
Run sync again and the IP will still be there after

Hi,
yes this is sort of desired behavior.
It Is possible that other setups would like to have the IP still assigned to the interface, even though it's offline. And netbox would still show that.
What do you think?

If netbox allowed the same IP to be assigned to multiple interfaces and had some sort of concept of connected/disconnected I'd say leave it, but since we're constrained by netbox here I think a sensible default is just ignoring any that are in a disconnected state. You could have this as a configuration param though ie, "add_ip_even_if_disconnected"

Maybe we have a communication issue here.
scenario One:

the VM is running
one interface is disconnected
but the OS has still an IP address assigned to this interface

The IP will not be deleted/marked as orphaned because it is still configured in a running VM.
scenario Two:

the VM is running
one interface is disconnected
NO IP address is assign to this interface

The IP will be marked as orphaned and be deleted after a grace period of prune_delay_in_days days.
This will not remove the IP immediately.
If you want to delete everything (except interfaces) that has been unconfigured immediately then you need to set prune_delay_in_days to 0.
What do you think?

Since netbox refuses to allow the same IP to be added to multiple interfaces, I think if an interface is disconnected in vmware it should be immediately removed from netbox even if it's still active in the OS.
This could be a configurable option, or maybe you only delete the IP address immediately if it's found on another VM?

Sounds good to me!

Added following checks:

If the same IP address is found on a different interface (of a different device/VM) within the same realm
(both using same VRF or both are global) then following test are performed:

If the current interface is enabled and the new one disabled:

IP stays at the current interface

If the current interface ia disabled and the new one enabled:

IP will be moved to the enabled interface

Both interfaces are in the same state (disabled/enable)

IP will also stay at the current interface as it's unclear which one would be the correct one

any chance you could test this?

Due to no response in over a month I will close the ticket for now.

Due to no response in over a month I will close the ticket for now.


Rancher v2.5 backup and restore documentation

outstanding questions -

What command should be used to upgrade Rancher after a restore? This command was used to test the feature but I'm not sure which Rancher image tag should be used.

helm upgrade rancher rancher-alpha/rancher \
  --version 2.5.0-alpha1 \
  --namespace cattle-system \
  -set hostname=<same hostname as first rancher server> \
  --set rancherImageTag=master-head

What is the new link to where the rancher-backup helm chart will be? The current link is https://github.com/mrajashree/charts/tree/backup-restore
is 'backup-restore-operator' going to be changed everywhere? Because in these docs I've changed it everywhere, including in the commands and the examples.

I'm going to merge this into staging so that it will be easier to review.
The docs that need to be reviewed are at this link: https://staging.rancher.com/docs/rancher/v2.x/en/backups/ (this page and the child pages in the nav).

@catherineluse The backup-rollbacks for rancher installed with docker (single-node-backups) should continue to use docker volumes https://rancher.com/docs/rancher/v2.x/en/upgrades/upgrades/single-node/
right now in the staging docs this section is also moved to legacy folder under backups, but it should be its own section under the main backups/ folder

@catherineluse I created this PR to suggest changes https://github.com/rancher/docs/pull/2730

For the outstanding questions:

What command should be used to upgrade Rancher after a restore? This command was used to test the feature but I'm not sure which Rancher image tag should be used.
helm upgrade rancher rancher-alpha/rancher
--version 2.5.0-alpha1
--namespace cattle-system
-set hostname=
--set rancherImageTag=master-head

@deniseschannon @maggieliu @cloudnautique I want to confirm that rancher v2.5 will be tagged as latest before release right? so will the helm install command for installing rancher will be what is in our HA install docs right now:
helm install rancher rancher-latest/rancher \
  --namespace cattle-system \
  --set hostname=<same hostname as first rancher server>

What is the new link to where the rancher-backup helm chart will be? The current link is https://github.com/mrajashree/charts/tree/backup-restore

new link: https://github.com/rancher/charts/tree/main/charts/rancher-backup

is 'backup-restore-operator' going to be changed everywhere? Because in these docs I've changed it everywhere, including in the commands and the examples.

The changes in commands and examples are correct

I merged Rajashree's PR https://github.com/rancher/docs/pull/2730 and made additional changes on top of it https://github.com/rancher/docs/pull/2734


[todo-mvvm-databinding] Scrolling if there are more TODOs than fits on the screen calls repository methods if checkbox value changes

Scrolling if there are more TODOs than fits on the screen calls repository methods if checkbox value changes. This is because two-way data binding calls setCompleted() even when list item check box value is set to new task completed value.

We removed that call when we updated to Room. Feel free to reopen if it's not fixed.

This issue has bot been fixed.


"Guest specific operations were attempted" on boot, Rsync not syncing either

Hi,
I have a box hosted on digital ocean, and whenever I try to reboot it, I get the following error:
Bringing machine 'default' up with 'digital_ocean' provider...
==> default: Powering on the droplet...
Guest-specific operations were attempted on a machine that is not
ready for guest communication. This should not happen and a bug
should be reported.

Because of this, none of my changes are getting synced to the DO box, as the vagrant rsync command does not appear to be working either.

On the readme vagrant rsync is not in the supported commands. Does this mean this is unsupported?

What is the OS of the droplet?

I was able to get the same error. Everything works as normal and the Droplet is powered on despite the error.
Droplet is powered down
$ vagrant status
Current machine states:

default                   off (digital_ocean)

off

Start the Droplet and trigger error
$ vagrant up
Bringing machine 'default' up with 'digital_ocean' provider...
==> default: Powering on the droplet...
Guest-specific operations were attempted on a machine that is not
ready for guest communication. This should not happen and a bug
should be reported.

The droplet is powered on despite the error
$ vagrant status
Current machine states:

default                   active (digital_ocean)

active

Vagrantfile
Here is a copy of the configuration.
Vagrant.configure('2') do |config|

  config.vm.provider :digital_ocean do |provider, override|
    override.ssh.private_key_path = '~/.ssh/id_rsa'
    override.vm.box = 'digital_ocean'
    override.vm.box_url = "https://github.com/smdahlen/vagrant-digitalocean/raw/master/box/digital_ocean.box"

    provider.token = ENV['DIGITAL_OCEAN_TOKEN']
    provider.image = 'ubuntu-14-04-x64'
    provider.region = 'sfo1'
    provider.size = '512mb'

  end


Patches

Allow multiple (default 10x) tries to access earthscope (FDSN) data before giving up.
Should no longer fail with BadGZipFileError.

Codecov Report
Attention: Patch coverage is 19.04762% with 51 lines in your changes are missing coverage. Please review.

Project coverage is 60.58%. Comparing base (47d46a3) to head (8d93770).
Report is 43 commits behind head on master.

Files
Patch %
Lines

mth5/clients/fdsn.py
15.38%
44 Missing :warning:

mth5/io/phoenix/readers/base.py
0.00%
5 Missing :warning:

mth5/io/phoenix/readers/calibrations.py
50.00%
1 Missing :warning:

mth5/io/phoenix/readers/receiver_metadata.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #206      +/-   ##
==========================================
- Coverage   60.62%   60.58%   -0.05%
==========================================
  Files         141      141
  Lines       15634    15729      +95
==========================================
+ Hits         9478     9529      +51
- Misses       6156     6200      +44

Flag
Coverage Δ

tests
60.58% <19.04%> (-0.05%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Correct tiny typo

The first error is a mistranslation, probably a mix-up between "Algorithmus" and "Logarithmus". In any case, I believe that "Format" works best here. If you want to translate "JPEG encoding" literally, then use "Aufgrund der Art und Weise wie die JPEG-Kodierung funktioniert, ...".
The second error is just an accidental double word insertion.
Sorry for touching the EOL at the end of the file (GitHub web editor).

hey @iblech thank you for the pull request!
(i'm glad that someone actually reads the descripton text)


None of the requested configurations were supported.

In Chrome 49 and prior we are getting the error "None of the requested configurations were supported." when using codec="avc1.640814". However MediaSource.isTypeSupported('video/mp4; codecs="avc1.640814"') returns true. This isn't a Shaka issue because 49 is using native MSE as-is but I can't find any discussion of this inconsistency. Has anyone encountered something similar?

Is the content encrypted or clear?

The content is encrypted.

Does it work in Chrome 50 and later or is it still a problem? document.createElement('video').canPlayType('video/mp4; codecs="avc1.640814"') returns "probably" in Chrome 51, so it should work, though I'm not familiar with the Shaka API

Yes it works in later versions.

If it works in Chrome stable (51) but not in Chrome 49, then I have to assume it's a bug in Chrome that was fixed already.
If the content is encrypted, MediaSource is not the only component that must support the MIME type.  MediaKeys must also support it.  The error you mentioned occurs when EME negotiation fails.  So the EME query for that type/codec must be failing.
Here's a simplified version of Shaka's EME query for that type.  You can run this in the JS console to see what happens on your version of Chrome:
config = {videoCapabilities: [{ contentType: 'video/mp4; codecs="avc1.640814"' }]};
navigator.requestMediaKeySystemAccess('com.widevine.alpha', [config]).then(
    function() { console.log('okay'); },
    function(e) { console.log(fail, e); }
);

Unfortunately, though, we will be unable to give direct support for outdated versions of Chrome, since Chrome is meant to be auto-updated.
If you have a fork or an embedded version of Chromium, you may be able to patch the bug yourself.  Since it appears that the bug has been fixed in Chrome stable, though, you may be able to identify the change that fixed it and back port that to your version.

@joeyparrish thanks for the info. I don't expect any Shaka support for this, just trying to get a handle on what's going on. Only reason we were concerned with 49 was that it is the lats version available on OSX 10.7 I believe.

Alright, after checking all the codecs in the manifest in question, they all fail the test you provided in Chrome 49. Thanks for informing me that EME and MediaSource support are independent of each other.

Oh, I see.  I was not aware of that.  If you can track down the specific issue in crbug.com, it might be worth raising with Chrome team that the bug persists on OSX 10.7.

Okay, good point. I'll see what I can do.


fix: upgrade tree sitter to v0.21.0

Two updates

upgrade the treesitter version to v0.21.0
use the same version as the treesitter

We will be merging next into main soon enough, and this has that update and many more (it brings us to current tree-sitter), so I'm going to close this in favor of that, but thank you!


Support for Electra iOS 11.2-11.3.1

Can this be updated to support the recent Electra jailbreak?
https://coolstar.org/electra/
Currently I'm getting the following error: [!] Unknown jailbreak. Aborting.

I haven't used the Electra iOS 11.2-11.3.1 jailbreak yet. But based on the error, is the bfinject script pointing to the correct directory? You might have to edit the path outlined below as well as others accordingly.

@svedm thanks for this fix! I can confirm it's working on my 8+ 11.3.1.

Hello guys somebody can help me? I trying to inject cycript but it said unknown jailbreak aborting. I write this command:
bash bfinject -p 3094 -L cycript
[!] Unknown jailbreak. Aborting.

Someone can help me ? It said no Team id

@svedm testing with the reddit app with iphone8 on 11.3.1. I get this error when running /var/mobile/Documents/bfinject root# bash bfinject -P Reddit -L decrypt
bfinject: line 43: wc: command not found
[!] "Reddit" was not uniquely found, please check your criteria.

MJavad's fork work under ios 11-11.4b3 even with tweaks turned on. you just need to place it under / , /private /bin /sbin /var wont work bec. it doesnt allow code execution . However it doesnt sign correctly but you can do it manually via ldid -S path/to/binary or use this automated script on the ipa signer.sh. usage bash signer.sh path/to/ipa

It works perfectly on 11.3.1 without tweaks. It fails with tweaks enabled.

Yes. MJavad however found a way to run it with tweaks enabled. I’ve tested it works up to 11.4b3

I created the PR for you.


Create Player-Action function

to interact with the game choices. i.e. "a)go in next room b)go to previous room c)look around d)save and exit game

I created the function and made a pull request.
The function I programmed takes the different choices the user is offered for a decision as an array. It presents the different choices to the user and asks him/her to decide by using numbers on the keyboard. It checks if the number typed in is valid (else, the user has to decide again) and if the user is drunk.
If the user is drunk, the decision will be made randomly and the user is informed that his decision has been changed due to him being drunk.
The validated, and maybe drunk-changed, user input is returned.
Let me know what you think!

the function was reviewed and merged!


Attribute issue while executing build.py file

I installed all the dependencies and tried to run build.py file for llama2, but the following issue
Traceback (most recent call last):
File "/home/shavak/wk-atharv/WebScrappingChatBot/llm/udemy/TensorRT-LLM/examples/llama/build.py", line 25, in
from weight import (get_scaling_factors, load_from_awq_llama, load_from_binary,
File "/home/shavak/wk-atharv/WebScrappingChatBot/llm/udemy/TensorRT-LLM/examples/llama/weight.py", line 25, in
import tensorrt_llm
File "/home/shavak/wk-atharv/WebScrappingChatBot/llm/udemy/TensorRT-LLM/examples/llama/tensorrt_llm/init.py", line 15, in
import tensorrt_llm.functional as functional
File "/home/shavak/wk-atharv/WebScrappingChatBot/llm/udemy/TensorRT-LLM/examples/llama/tensorrt_llm/functional.py", line 25, in
from . import graph_rewriting as gw
File "/home/shavak/wk-atharv/WebScrappingChatBot/llm/udemy/TensorRT-LLM/examples/llama/tensorrt_llm/graph_rewriting.py", line 11, in
from .network import Network
File "/home/shavak/wk-atharv/WebScrappingChatBot/llm/udemy/TensorRT-LLM/examples/llama/tensorrt_llm/network.py", line 26, in
from ._common import set_network
File "/home/shavak/wk-atharv/WebScrappingChatBot/llm/udemy/TensorRT-LLM/examples/llama/tensorrt_llm/_common.py", line 21, in
from ._utils import str_dtype_to_trt
File "/home/shavak/wk-atharv/WebScrappingChatBot/llm/udemy/TensorRT-LLM/examples/llama/tensorrt_llm/_utils.py", line 81, in
int64=trt.int64,
AttributeError: module 'tensorrt' has no attribute 'int64'
'
Can anayone please help

Can you check your tensorrt version? TensorRT-LLM requires TensorRT 9.1.

tensorrt version:- 8.6.1.post1 with python version of 3.9

is there any command using pip or conda for the upgrading the tensorrt

Sorry for inconvenience. TRT 9 is not updated to pypi.
You can download on https://developer.nvidia.com/downloads/compute/machine-learning/tensorrt/secure/9.1.0/tars/tensorrt-<IP_ADDRESS>.linux.x86_64-gnu.cuda-12.2.tar.gz, and install the .whl file in TensorRT-<IP_ADDRESS>/python/ after extraction.

ok thanks, do I  need to uninstall the previous version of tensorrt

ok thanks, do I need to uninstall the previous version of tensorrt

I think it is fine. But you can uninstall it first.

tensorrt-9.1.0.post11.dev4-cp310-none-linux_x86_64.whl
tensorrt-9.1.0.post11.dev4-cp311-none-linux_x86_64.whl
tensorrt-9.1.0.post11.dev4-cp36-none-linux_x86_64.whl
tensorrt-9.1.0.post11.dev4-cp37-none-linux_x86_64.whl
tensorrt-9.1.0.post11.dev4-cp38-none-linux_x86_64.whl
tensorrt-9.1.0.post11.dev4-cp39-none-linux_x86_64.whl
tensorrt_dispatch-9.1.0.post11.dev4-cp310-none-linux_x86_64.whl
tensorrt_dispatch-9.1.0.post11.dev4-cp311-none-linux_x86_64.whl
tensorrt_dispatch-9.1.0.post11.dev4-cp36-none-linux_x86_64.whl
tensorrt_dispatch-9.1.0.post11.dev4-cp37-none-linux_x86_64.whl
tensorrt_dispatch-9.1.0.post11.dev4-cp38-none-linux_x86_64.whl
tensorrt_dispatch-9.1.0.post11.dev4-cp39-none-linux_x86_64.whl
tensorrt_lean-9.1.0.post11.dev4-cp310-none-linux_x86_64.whl
tensorrt_lean-9.1.0.post11.dev4-cp311-none-linux_x86_64.whl
tensorrt_lean-9.1.0.post11.dev4-cp36-none-linux_x86_64.whl
tensorrt_lean-9.1.0.post11.dev4-cp37-none-linux_x86_64.whl
tensorrt_lean-9.1.0.post11.dev4-cp38-none-linux_x86_64.whl
tensorrt_lean-9.1.0.post11.dev4-cp39-none-linux_x86_64.whl
There multiple files provided which .whl file do i need to install

You can install tensorrt-9.1.0.post11.dev4-* and choose the python version you use.

also after installation do i need to export path

As  I install .whl file but still encountered the following error
File "/home/shavak/wk-atharv/WebScrappingChatBot/llm/udemy/TensorRT-LLM/examples/llama/build.py", line 21, in
import tensorrt as trt
File "/home/shavak/wk-atharv/WebScrappingChatBot/llm/udemy/TensorRT-LLM/anaconda/lib/python3.9/site-packages/tensorrt/init.py", line 71, in
from .tensorrt import *
ImportError: libnvinfer.so.9: cannot open shared object file: No such file or directory

Do you export TensorRT-<IP_ADDRESS>/lib/ into LD_LIBRARY_PATH.

yes, and the error is resolved. I encountered another error regarding lib file for tensorrt-llm. I git pull the files, but able to see any lib file
self._handle = _dlopen(self._name, mode)

OSError: /home/shavak/wk-atharv/WebScrappingChatBot/llm/udemy/TensorRT-LLM/examples/llama/tensorrt_llm/libs/libnvinfer_plugin_tensorrt_llm.so: cannot open shared object file: No such file or directory

It looks you don't install tensorrt_llm successfully. You can refer the document https://github.com/NVIDIA/TensorRT-LLM/blob/release/0.5.0/docs/source/installation.md to install tensorrt_llm.

ok, thanks for the help. Really appreciate for your patience

I GET THIS ERROR WHILE INSTALLING LATEST TENSOR RT
Collecting tensorrt==9.1.0.post11.dev4
Downloading tensorrt-9.1.0.post11.dev4.tar.gz (6.8 kB)
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
Preparing metadata (setup.py) ... error
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Hi @arjunanand13 ,
First, it might not be needed to use uppercase to report issues. The members of the TensorRT-LLM team are trying their best to help.
Regarding the issue, can you tell us which command leads to such a result, please? I suspect it's not coming from build.py but I may be wrong.
Thanks,
Julien

I get this error while installing the package
pip install tensorrt-dispatch-bindings==9.1.0.post12.dev4

I get this error while installing the package pip install tensorrt-dispatch-bindings==9.1.0.post12.dev4

Why do you want to install the package? You can install tensorrt by
pip install tensorrt-9.1.0.post12.dev4-cp311-none-linux_x86_64.whl


first try at getting lensed CMB maps in

This is a tweak of a similar implementation in actsims https://github.com/ACTCollaboration/actsims/blob/96f4b98d9e580c33fb73f1334f476817b0286782/actsims/simTools.py#L24
This is just draft code and has not been tested!  But we (Mat and Alex) are submitting it for a first review.
It is decoupled from the actsims repo but by default will load up the same lensed alm's.

Indeed.  Even putting aside bandpass integration for now, passing even just the central frequencies would be better than passing an integer nfreqs.  In actsims I pass around lists such as  [‘f150’] or [‘f090’, ‘f150’].  What will we do in mapsims?

On Dec 6, 2018, at 12:45 PM, Mathew S. Madhavacheril<EMAIL_ADDRESS>wrote:
@msyriac commented on this pull request.
In mapsims/cmb.py https://github.com/simonsobs/mapsims/pull/4#discussion_r239548765:

   returns (nfreqs,ncomp,Ny,Nx) rectangular pixel ndmap.

'''
ncomp = 3 if pol else 1
filename = _get_cmb_map_string(cmb_dir,iteration_num,cmb_set,lensed,aberrated)
#The hdu = (1, 2,3) means get all of T, E, B
#Note the alm's are stored as complex32, so upgrade this for processing
alm_teb = np.complex128(hp.fitsfunc.read_alm(filename, hdu = (1,2,3) if pol else 1))
#Here we can multiply the alms by the appropriate beam;
#hp.sphtfunc.almxfl can be used.  Not included yet.  Once we
#do, we will have to call the inverse SHT nfreqs times, as in actsims.
if nside is not None:

   #Then we are outputting a healpix map

   map_tqu = hp.alm2map(alm_teb, nside)

   output = np.tile(map_tqu, (nfreqs, 1, 1)) # if nfreqs>1 else map_tqu

   #Here we want to multiply the map by the modulation factor.  FIXME: not implemented yet

So instead of nfreqs, we'll need some way to specify the frequency bandpasses in the interface to this function right? Should we go with some effective central frequency? Or do we want to do a bandpass integration? What will PySM be doing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/simonsobs/mapsims/pull/4#discussion_r239548765, or mute the thread https://github.com/notifications/unsubscribe-auth/APAXHDMLYVBDTYsJrTDDJfo0bigj7MFhks5u2VfEgaJpZM4ZFxor.

I've made a first pass at integrating mapsims.cmb and so_pysm_models.alms. Could you have a look?

@ajvanengelen I've put a placeholder for modulation here: https://github.com/msyriac/so_pysm_models/blob/precomputed_alms_with_car/so_pysm_models/alms.py#L64

thanks @msyriac !


[CI] Random selection of a team member is not working.

Fix the error in the Select a member from managers team randomly step
https://github.com/RasaHQ/rasa/runs/2759366368?check_suite_focus=true
jq: error (at <stdin>:1): Cannot index string with string "login"

We recently introduced a change in our internal team processes and as a result of that we don't want to randomly assign these issues to a team member. Would it be possible to completely get rid of that step? 🙂

@dakshvar22 So you want to just create the issue and send the slack message. If this is the case, definitely we can remove the Random selection of a team member step.

Yes, so just to be clear 🙂  :

Create an issue
Add relevant tags
Send the slack message
Add the issue to the research project inbox.

@dakshvar22 Thanks for the clarity. 🙇♀  I can work on this next week as I am on vacation this week.

For sure! Please enjoy your vacation! Didn't mean to disturb you mid-vacation. 🙂


Wrapping in braces is broken

Apparently the plugin actually does wrap the selected code in braces but then does some stuff extra that unindents the code and removes the opening brace. Doing a couple of "undo" actions takes the code back to where it was properly wrapped in braces.

Yes, it is definitely not perfect. Do you have a snippet I can use to test?

Here's what happens in a step-by-step process:
Here's a bit of code:
var a = b;
Here's what happens after pressing Ctrl+B
var a = b;
That's it. You may think nothing happened but here's what happens each time I press Command+Z:
var a = b;
Mind you, this is AFTER I press Undo meaning the text stays the same but is now selected if it wasn't before and stays selected if it was.
{
    var a = b;
}

{
var a = b;
}

var a = b;
At this point, the next undo undoes whatever I did before I pressed Ctrl + B. BTW, the first line

Heyoo this is no longer an issue. Wrapping in braces works as expected. Thanks a bunch!


Run second shell script

I need to run second script even if first steal running. Now im using this:
shell.exec('node vexample2.js ',{silent:true});
but if i send second socket message to run next script when this already finished - have and error.

Can you post a more detailed description (including minimal scripts)?

Minimal scripts? Im using node

Please post the code which produces the issue. It's hard to understand your problem if you don't clearly describe where the problem is.

Its more my mistake, than error. I need to run shell script and continue check socket server. But while script is working socket cant get new messages.

Its more my mistake, than error.

Sounds like user error, I'm assuming this is not a shelljs bug. Closing.
If this is indeed a shelljs bug, then please post a minimum code example (the smallest possible script to reproduce the issue). And please paste it in a markdown code block (no screenshots or unformatted code please).


Add v2 predector notebook

Hi, Great project with a great approach!
To make it easy to run, I added a notebook that also works with colab. How do you like it?

example: https://colab.research.google.com/drive/1C5P8gJlg8xFYnz0G3o5X2ahLDO8uJyf4?usp=sharing

This is amazing! Thanks so much.
I have been trying to update it to accept a CSV file w a list of urls, and then spit out the file w a new column adding the computed scores per row.
Any chance you could help me update this?
Thanks again!


End to end test error output is unclear

I currently have a failing test suite that looks like this:
=== RUN   TestAuthors
--- PASS: TestAuthors (0.03s)
##[error]    db_test.go:14: db: postgres://postgres:postgres@localhost:5432/postgres?sslmode=disable
##[error]    db_test.go:25: []
##[error]    db_test.go:35: {1 Brian Kernighan {Co-author of The C Programming Language and The Go Programming Language true}}
##[error]    db_test.go:42: {1 Brian Kernighan {Co-author of The C Programming Language and The Go Programming Language true}}
PASS
ok  	github.com/kyleconroy/sqlc/examples/authors	0.044s
?   	github.com/kyleconroy/sqlc/examples/booktest/mysql	[no test files]
=== RUN   TestBooks
--- PASS: TestBooks (0.04s)
##[error]    db_test.go:15: db: postgres://postgres:postgres@localhost:5432/postgres?sslmode=disable
##[error]    db_test.go:128: Book 1 (FICTION): my book title available: 24 Feb 20 19:56 +0000
##[error]    db_test.go:133: Book 1 author: Unknown Master
##[error]    db_test.go:137: ---------
        Tag search results:
##[error]    db_test.go:143: Book 4: 'never ever gonna finish, a quatrain', Author: 'Unknown Master', ISBN: 'NEW ISBN' Tags: '[someother]'
##[error]    db_test.go:143: Book 3: 'the third book', Author: 'Unknown Master', ISBN: '3' Tags: '[cool]'
##[error]    db_test.go:143: Book 2: 'changed second title', Author: 'Unknown Master', ISBN: '2' Tags: '[cool disastor]'
PASS
ok  	github.com/kyleconroy/sqlc/examples/booktest/postgresql	0.057s

The test suite failed (maybe because of this section??) but it's not super clear what the issue is from this snippet.

never mind, turns out the failure was elsewhere

Yeah, this is a problem with GitHub Actions. t.Log output shows up as errors in the logs.

Going to mark this issue as closed, let me know if the output is still confusing.


Gem structure changes

Changes for spinning out the drivers into their own gems. These changes will be released under 3.0.0.

Good news : https://github.com/blog/2141-squash-your-commits
I did see anything with your module in it.


fix(cli): delete "\f" and adjust double quotes in docs of 5 files

Pull Request Test Coverage Report for Build 849947776

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.973%

Totals

Change from base Build 849890302:
0.0%

Covered Lines:
4243

Relevant Lines:
5240

💛  - Coveralls


[SPARK-21715][WebUI] History Server should not respond history page html content multiple times for only one http request

What changes were proposed in this pull request?
Add the images required by dataTables jQuery plugin that will show up in the history page table header. All the images are downloaded from the official github site for dataTables. Change the path for the images in the dataTables.bootstrap.css to make sure the images can be found. Disable the favicon.ico downloading by changing the header in the html.
How was this patch tested?
Build and unit test passed.
Deployed and check the UI changes. The images are getting correctly downloaded and History Server can handle these requests correctly.

Can one of the admins verify this patch?

The PR title does not match what the PR summary says. The title is about one change, the summary is about a different change, and the code seems to handle both. It's all pretty confusing.
It seems like Alex's comments weren't yet addressed either.


Generating patients in Hospice care

Hello,
I'm relatively new to the Synthea Generator and need to generate some patients that are currently in hospice and have the corresponding value sets/OID codes.
After "run_synthea -p 1000" , the 1000 were generated.  However, after grepping for HOSPICE/hospice/Hospice I did not see any matches.  I tried removing all the providers except the hospice providers in the "synthea.properties" file and encountered errors.  Then I pointed all the providers toward providers/hospice.csv .  Many matching files were outputted, though on inspection it just has the providers named with hospice associated businesses.   What I need are patients who flow through any particular Synthea workflow and end up in Hospice with hospice associated Valueset codes.  Is there a way to do this?
Warmly,
Doug

Hospice care is being added in the bb-export branch.
See #770. When that PR is merged, hospice, home health, and skilled nursing facility support will be added.


Only found chapter, videos not found

Is this because of udemy update? or is it because of my pc?
windows 10, python 3.6

already fixed check : #179

but where is the updated release? I dont see any upgrade from 0.4

@kamalhm you should check commits instead version upgrade.

I see, thank you, sorry for bothering you i'm still new to github :D


ModuleNotFoundError: No module named 'gi'

I just tried setting up pantalaimon in docker, and can't get panctl to run from within the container.
Docker commands:
docker run -d --restart=unless-stopped -v /home/core/data:/data -p 8008:8008 matrixdotorg/pantalaimon
docker exec -it <instanceID> /bin/bash
# panctl
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/pantalaimon/panctl.py", line 28, in <module>
    from gi.repository import GLib
ModuleNotFoundError: No module named 'gi'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/panctl", line 5, in <module>
    from pantalaimon.panctl import main
  File "/usr/local/lib/python3.8/site-packages/pantalaimon/panctl.py", line 30, in <module>
    from pgi.repository import GLib
ModuleNotFoundError: No module named 'pgi'

I'm assuming https://github.com/matrix-org/pantalaimon/pull/122/commits/86060a2f7561d40bd142c8bc1847e151c3d40dff didn't totally fix things.

https://gist.github.com/aspacca/6320d86692353001f7a0be1ae4471113

https://gist.github.com/aspacca/6320d86692353001f7a0be1ae4471113

This gives me
Step 8/22 : RUN pip wheel .[ui] --wheel-dir /wheels --find-links /wheels
 ---> Running in xxxxxxxxxx
ERROR: Directory '.[ui]' is not installable. Neither 'setup.py' nor 'pyproject.toml' found.
ERROR: Service 'pantaleimon' failed to build: The command '/bin/sh -c pip wheel .[ui] --wheel-dir /wheels --find-link

do you run from the build from the root of the git repo?
What does it matter, you're pulling all the code from pip

RUN pip --no-cache-dir install --find-links /wheels --no-index pantalaimon
RUN pip --no-cache-dir install --find-links /wheels --no-index pantalaimon[ui]

What does it matter, you're pulling all the code from pip

not in the builder: https://gist.github.com/aspacca/6320d86692353001f7a0be1ae4471113#file-dockerfile-L13-L15
COPY . /app

RUN pip wheel .[ui] --wheel-dir /wheels --find-links /wheels

that's exactly where you have the error

Don't get me wrong, I didn't modify your gist. Here is the complete result:

Used your gist as Dockerfile
Made a docker-compose.yml
Started it with command docker-compose up --build

version: '3.3'

services:
  pantalaimon:
    build: .
    container_name: pantalaimon
    restart: unless-stopped
    volumes:
     - ./data:/data
    ports:
     - 8124:8124

$ docker-compose up --build
Building pantalaimon
Step 1/22 : FROM python:3.8-slim-bullseye AS builder
 ---> 61c56c60bb49
Step 2/22 : RUN mkdir -p /app
 ---> Using cache
 ---> a92d1585ff31
Step 3/22 : RUN apt-get update && apt-get install -y git gcc clang cmake g++ pkg-config python3-dev wget libcairo2-dev libgirepository1.0-dev libdbus-1-3 libdbus-1-dev
 ---> Using cache
 ---> 7c90eaf6b3fc
Step 4/22 : WORKDIR /app
 ---> Using cache
 ---> 6b50fa3344b1
Step 5/22 : RUN wget https://gitlab.matrix.org/matrix-org/olm/-/archive/master/olm-master.tar.bz2     && tar -xvf olm-master.tar.bz2     && cd olm-master && make && make PREFIX="/usr" install
 ---> Using cache
 ---> 697925527e27
Step 6/22 : RUN pip --no-cache-dir install --upgrade pip setuptools wheel
 ---> Using cache
 ---> 3c19921e5934
Step 7/22 : COPY . /app
 ---> a16c24218f89
Step 8/22 : RUN pip wheel .[ui] --wheel-dir /wheels --find-links /wheels
 ---> Running in 730471605f13
ERROR: Directory '.[ui]' is not installable. Neither 'setup.py' nor 'pyproject.toml' found.
ERROR: Service 'pantalaimon' failed to build: The command '/bin/sh -c pip wheel .[ui] --wheel-dir /wheels --find-links /wheels' returned a non-zero code: 1

@aspacca You're absolutely correct, I didn't spot that and I didn't copy the git repsitory to my folder. I feel like this is all a bit off topic. Though. Perhaps your gist should become the actual dockerfile, see #131

I'm not sure I'm following the above replies. Was there a tried and true solution for this? As it stands, Pantalaimon is unusable for me, as I cannot operate panctl at all.

@jph
starting the container building from the dockefile at https://gist.github.com/aspacca/6320d86692353001f7a0be1ae4471113 solves the issue
it starts a tmux session that you can attach and from there start a new one running panctl

Thanks @aspacca, good workaround.


correct ipv6 domain name format

Ipv4 and ipv6 URL domain name is different in format. Make vc-manager compatible with ipv6 only cluster

/lgtm
/approve


Support representations on PhaseSpacePosition

          I see the support for representations and for reference frames is missing here. Should we discuss plans for that? I like the way, in gala, one can do:

psp = PhaseSpacePosition(...)
psp.cylindrical.rho, psp.cylindrical.v_rho

Originally posted by @adrn in https://github.com/GalacticDynamics/galax/pull/96#discussion_r1468888277

We're close enough now.
from coordinax import SphericalVector
from galax.coordinates import PhaseSpacePosition

psp = PhaseSpacePosition(...)
psp.represent_as(SphericalVector).q.rho

Maybe worth returning to, but this is now essentially a supported feature.


dirhtml builder errors with files called index

I'm using this plugin for a Sphinx site that has some pages called index.rst (e.g. docs/mydir/index.rst). When using the dirhtml builder, Sphinx does not create an extra folder for index (e.g. docs/mydir/index/index.html) and instead just leaves the index file as-is (docs/mydir/index.html), but this extension searches for this file when checking whether this file exists. Has anybody else run into this issue or know a workaround?

Rediraffe currently does not properly account for index.rst files in folders. The linked pr should resolve it.
When redirecting to a file, rediraffe makes sure that the file actually exists.

Wow that was a fast turnaround!! Thanks so much 🙂


[17.12] Do not make graphdriver homes private mounts.

Backport of moby/moby#36047
git cherry-pick -s -S -x -Xsubtree=components/engine 9803272f2db84df7955b16c0d847ad72cdc494d1

The idea behind making the graphdrivers private is to prevent leaking
mounts into other namespaces.
Unfortunately this is not really what happens.
There is one case where this does work, and that is when the namespace
was created before the daemon's namespace.
However with systemd each system servie winds up with it's own mount
namespace. This causes a race betwen daemon startup and other system
services as to if the mount is actually private.
This also means there is a negative impact when other system services
are started while the daemon is running.
Basically there are too many things that the daemon does not have
control over (nor should it) to be able to protect against these kinds
of leakages. One thing is certain, setting the graphdriver roots to
private disconnects the mount ns heirarchy preventing propagation of
unmounts... new mounts are of course not propagated either, but the
behavior is racey (or just bad in the case of restarting services)... so
it's better to just be able to keep mount propagation in tact.
It also does not protect situations like -v /var/lib/docker:/var/lib/docker where all mounts are recursively bound
into the container anyway.
Signed-off-by: Brian Goff<EMAIL_ADDRESS>(cherry picked from commit 9803272f2db84df7955b16c0d847ad72cdc494d1)
Signed-off-by: Brian Goff<EMAIL_ADDRESS>

ping @kolyshkin PTAL

Added git cherry-pick -s -x -Xsubtree=components/engine 2fe4f888bee52b1f256d6fa5e20f9b061d30221c to cherry-pick moby/moby#36237


Try to reproduce supervised results from ideas.ipynb

Once reproduced, convert into an example script and doc page.

Don't forget to remove the notebook afterwards.


StringShouldBeRawString false positive with prependIndent

Might not be a false positive, let's see what you think!
Context
In some cases when interpolating raw strings, we need to use multiple levels of indentation, instead of hacking with .trimMargin() (and its drawbacks), we can simply do this:
val x = """
    ...
""".trimIndent()
val usage = """
    ...
${x.prependIndent("\t\t\t\t")}
"""

val x = """
    ...
""".replaceIndent("\t\t\t\t")

Expected Behavior
Above cases are not flagged for StringShouldBeRawString. I would say all usages of prependIndent/replaceIndent should be exempt, because if we replace it with raw, we lose the readability. We know that prepend/replace will very likely contain whitespace.
Observed Behavior
Above codes are flagged.
Your Environment

Version of detekt used: 1.23.0
Version of Gradle used (if applicable): 8.1.1
Gradle scan link (add --scan option when running the gradle task): N/A
Operating System and version: all
Link to your project (if it's a public repository): N/A

Note: could the problem be that tabs shouldn't be flagged?, because this is also flagged "Summary\t(total: 1)\t   1\t          0" (an assertion on an aligned table)

Similar one: val triplet = "\"\"\"" or val TQ = "\"\"\"", do we really want to encourage converting these to """"""""" 🤨?

I would say all usages of prependIndent/replaceIndent should be exempt

+1 on this

Hi, @TWiStErRob there is a config option ignoredCharacters will that solve your first issue? You have ignore the \t charcter

But in case of "\tat foo\n\tat bar\n\tat qux" it makes sense and that should be detected. It is a good rule, but there are some usages where it shouldn't be enforced, one such group is the stdlib indent methods, there might be more.

For """"""""" I don't even know what to do, probably just add an direct hard coded exception?


Allow replying to thread messages in system messages

Good idea

But there should be option to disable it in config

Superseded by 993f2f42688746bcc815ebb3fb6ff73a054f46f9


Issues in docker demployement

After I docker-compose up, the micro services are started running. I can see the 'mainflux' welcome text logo. However, I am getting the following error,
Error: Uncaught, unspecified "error" event. (Could not connect to server: Error: connect ECONNREFUSED <IP_ADDRESS>:4222)
mainflux-ws      |     at Client.emit (events.js:144:17)
mainflux-ws      |     at Socket. (/mainflux-ws/node_modules/nats/lib/nats.js:389:14)
mainflux-ws      |     at emitOne (events.js:77:13)
mainflux-ws      |     at Socket.emit (events.js:169:7)
mainflux-ws      |     at emitErrorNT (net.js:1253:8)
mainflux-ws      |     at doNTCallback2 (node.js:441:9)
mainflux-ws      |     at process._tickCallback (node.js:355:17)
mainflux-ws      | [09:53:57] [nodemon] app crashed - waiting for file changes before starting...
Also not sure how to view/access IOT platform on the browser. No such getting started document available on the wiki page.
TIA,
Prakash.

@asprakash yes, there is tones of issues now with Docker as we are switching to Go (from NodeJS).
It should be stabilized once @janko-isidorovic and @nmarcetic take care of https://github.com/Mainflux/mainflux/issues/43 and  especially https://github.com/Mainflux/mainflux/issues/44.
If you have some knowledge and you want to give them a hand, every help is highly appreciated!

@asprakash this whould be fixed now.
Can you please re-test and tell us if it works.


Add new gcs hooks, add expected mounts to security policy

Introduce a new wait-paths binary, which polls file system
until requested paths are available or a timeout is reached.
Security policy has been updated to have ExpectedMounts entries,
which will be used in conjunction with "wait-paths" hook for
synchronization purposes.
Refactor oci-hook logic into its own internal package and update
existing code to use that package. Copy runc HookName and constants
definitions to break dependency on runc
Introduce ExpectedMounts as part of security policy language and
the logic to enforce the policy, which resolves the expected mounts
in the UVM and adds a wait-paths hook to the spec.
Signed-off-by: Maksim An<EMAIL_ADDRESS>

@SeanTAllen @KenGordon @svolos FYI

I checked this out a while ago, but forgot to comment that it looked good and I didnt see anything problematic with the approach.

Do we have any tests for the expected mounts work in here?

Do we have any tests for the expected mounts work in here?

If you're talking about CRI integration tests, then it's currently blocked on the integrity validation PRs. I am planning on adding the tests once those are merged in as a separate PR. I've done local testing though.

@dcantah , @katiewasnothere please, revisit, when you get a chance

@dcantah , @katiewasnothere please, revisit, when you get a chance

Carved off time tomorrow for a second look!

@katiewasnothere PTAL when you get a chance.


prismlauncher: add canonicalize-jars-hook

Description of changes
this strips timestamps from java build artifacts, fixing reproducibility issues
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 250757 run on x86_64-darwin 1

4 packages built:

prismlauncher
prismlauncher-qt5
prismlauncher-qt5-unwrapped
prismlauncher-unwrapped


[Community] Add integration on serving OPT-175B using Alpa backend

Motivation
Thanks for open sourcing the OPT model weights!
With much help from this thread, we recently have integrated OPT serving with the Alpa system (developed by UC Berkeley).
Alpa offers unique advantages to train or serve large OPT models with more flexible/heterogeneous cluster setup and GPU specs -- such as using many in-house clusters with lower-end GPUs (e.g., 40GB A100, V100, or even Titan X, 2080Ti), as long as the total memory is sufficient.
Alpa can also provide various types of parallelism strategies beyond FSDP and tensor model parallel, which might be more advantageous in certain cluster setups, e.g., multiple-node cluster, or cluster with limited communication bandwidth.
Pitch
Like the HuggingFace integration with OPT, We want to contribute a guide on how to setup OPT-175B serving (and later, inference or training if enough interest), so people who do not have 8x 80Gb A100 can still benefit from these weights by using Alpa.
We have generated a setup guide on our end. For the OPT-175B model, we have tested it on various cluster setups, such as 32x 16GB V100 (AWS p3.16x), or 12x 40GB V100, etc.
Plan
I can draft a PR to Metaseq to describe how to use this integration and might need your review, what do you think? @stephenroller

Very cool

Hey guys - somewhat related, do you have any guide on how to run this on a single GPU?
Found this comment in opt_baselines.py that led me to this PR.


Consider shortening Asset.repr?

The repr for asset currently displays all the information that can be associated with an asset, and the verbosity makes it unpleasant to use, especially in collections where the default behavior is to repr the elements of the collection.  For example, this is what a list of 5 assets from the Q database currently looks like:
In [7]: assets[:5]
Out[7]:
[Equity(2, symbol=u'ARNC', asset_name=u'ARCONIC INC', exchange=u'NYSE', start_date=Timestamp('2002-01-01 00:00:00+0000', tz='UTC'), end_date=Timestamp('2017-02-02 00:00:00+0000', tz='UTC'), first_traded=None, auto_close_date=Timestamp('2017-02-07 00:00:00+0000', tz='UTC'), exchange_full=u'NEW YORK STOCK EXCHANGE'),
 Equity(6, symbol=u'AACE', asset_name=u'ACE CASH EXPRESS INC', exchange=u'NASDAQ', start_date=Timestamp('2002-01-01 00:00:00+0000', tz='UTC'), end_date=Timestamp('2006-10-05 00:00:00+0000', tz='UTC'), first_traded=None, auto_close_date=Timestamp('2006-10-10 00:00:00+0000', tz='UTC'), exchange_full=u'NASDAQ GLOBAL SELECT MARKET'),
 Equity(14, symbol=u'AALA', asset_name=u'AMERALIA INC', exchange=u'NASDAQ', start_date=Timestamp('2002-01-01 00:00:00+0000', tz='UTC'), end_date=Timestamp('2006-03-30 00:00:00+0000', tz='UTC'), first_traded=None, auto_close_date=Timestamp('2006-04-04 00:00:00+0000', tz='UTC'), exchange_full=u'NASDAQ GLOBAL MARKET'),
 Equity(21, symbol=u'AAME', asset_name=u'ATLANTIC AMERICAN CORP', exchange=u'NASDAQ', start_date=Timestamp('2002-01-01 00:00:00+0000', tz='UTC'), end_date=Timestamp('2017-02-02 00:00:00+0000', tz='UTC'), first_traded=None, auto_close_date=Timestamp('2017-02-07 00:00:00+0000', tz='UTC'), exchange_full=u'NASDAQ GLOBAL MARKET'),
 Equity(24, symbol=u'AAPL', asset_name=u'APPLE INC', exchange=u'NASDAQ', start_date=Timestamp('2002-01-01 00:00:00+0000', tz='UTC'), end_date=Timestamp('2017-02-02 00:00:00+0000', tz='UTC'), first_traded=None, auto_close_date=Timestamp('2017-02-07 00:00:00+0000', tz='UTC'), exchange_full=u'NASDAQ GLOBAL SELECT MARKET')]

By comparison, the current __str__ is relatively compact:
In [8]: map(str, assets[:5])
Out[8]:
['Equity(2 [ARNC])',
 'Equity(6 [AACE])',
 'Equity(14 [AALA])',
 'Equity(21 [AAME])',
 'Equity(24 [AAPL])']

While it's occasionally nice to have all the repr information available, I think those cases are outweighed by the need to do processing to generate readable error messages any time an asset might appear in a structure.
Since the asset repr is a major part of Zipline/Quantopian's UI, I'd like to invite some discussion on whether/how the Asset repr should be changed.  Some questions:

Do other people agree that the asset repr is too verbose?
If repr is too verbose, how should we shorten it?  Do we want to reduce it all the way down to what __str__ currently shows, or is there a middle ground between __str__ and the current __repr__?  The non-sid fields I use most often are, in order:

symbol
start/end date
asset_name  # Significant dropoff from start/end
exchange
auto_close_date  # Significant dropoff from exchange.

If we want to truncate Asset repr, should we provide an option (ala pd.set_option) to restore the verbose repr and/or set a custom repr?  Something like this might be useful for PyFolio/Alphalens where visual space is at a premium.

cc @twiecki @gusgordon @llllllllll @ehebert @richafrank @mtydykov @kglowinski @dmichalowicz @jmccorriston @jbredeche @ahgnaw for feedback

:+1: on shortening.
Your order seems right for Equity instances.
For Future instances, I would rank auto_close_date the same as start_date and end_date.

Agree with Eddie, I look at auto_close_date frequently for futures.

Agree about shortening. I also like the idea in (3) of optionally allowing the repr to be more verbose.

I agree with Eddie and David. Your list is spot on for equities, but for futures, auto_close_date should be in the same tier as start/end date. My vote would be to just have a simplified version that matches what __str__ shows. In my own experience, if I want to see the start/end date, I explicitly ask for the relevant attribute. I do this because the verbosity makes it too hard to read, or my log line gets truncated on Q. Obviously, if we only cut down to a subset, this would change at least a little bit, but I think that the most common use case here would be someone just trying to identify an asset, which can be done with (sid [SYMB]).
I'm not against the option to restore verbosity, but I think the value would be pretty limited in the community.

Makes sense to me.
Relatedly, it might be worth it to have something that's even less verbose than __str__. Consider pyfolio, where we are displaying asset names all the time. Using __str__ is too verbose (we already know it's an Equity!), and symbol can sometimes lead to confusion if there are multiple assets with the same symbol. We've elected to use a symbol-sid convention, e.g. AAPL-24. The ideal solution in pyfolio's case is probably to only include the sid if there is symbol overlap, otherwise only the symbol needs to be displayed.

we already know it's an Equity!

What happens when someone wants to analyze a futures (or, especially, a mixed-asset algorithm) with pyfolio?  symbol-sid is still sufficiently distinguishing here, of course, but I wouldn't advise baking in the assumption of a single asset type in pyfolio's display formatting.

True, though most people will probably trade primarily one asset type in their strategy. If someone is looking at a long list of assets they've traded, they don't need the type displayed unless there could be some uncertainty about the properties of the asset. Of course it's also nice to have a uniform format, so there are tradeoffs.

We're going to need to make some pyfolio updates when futures roll through ;)

@jbredeche expressed frustration in slack today that this has not yet been done.

@ssanderson in what case would we worry about not having a more verbose repr?
I think shortening it, making it similar to __str__ would be useful. While working on the Transaction repr change I was looking at Asset as an example and thought it had too much information.

@FreddieV4 I do find it occasionally useful to have all the info at hand.  I think those cases are outweighed, however, by the cases where the repr is too verbose.
At this point I think just changing __repr__ to match the current __str__ is a strict improvement that's worth doing.  In the future, we can add hooks for contexts like PyFolio that might want to contextually-override the repr, but I think bikeshedding on that design isn't worth the opportunity cost of not improving the repr now.
Want to put together a PR for making __repr__ match __str__?  I think the easiest way to do that is to just rename the current __str__ to __repr__, and delete the current __str__ implementation (__str__ defaults to __repr__ if no explicit __str__ is defined).

@ssanderson Sure I can put together a PR for this.

@ssanderson Sure I can put together a PR for this.
How does the __repr__ works between Asset and Equity though? In the Equity object we define the repr and it's pretty much the same as the repr for Asset. Would the repr for Equity not override the repr for Asset because it's a subclass?
i.e. Wouldn't I have to change the repr for both Asset and Equity?

In the latest release so going to close this. Feel free to re-open for any extended discussion on this 🙂


Passing list-likes to .loc or [] with any missing label will raise KeyError in the future, you can use .reindex() as an alternative.

Dear Dr. Greg,
I was trying to run your process_data.py script in nbconverted scripts. This resulted in an error, which I believe was because of the missing column 'hugoSymbol' in tsg_df and also because that indexing with list with missing labels is deprecated. Please find the exact output below. The documentation says:

Changed in version 1.0.0.

Using .loc or [] with a list with one or more missing labels will no longer reindex, in favor of .reindex.

Could I simply use .reindex() instead of .loc() here in this case? I would be very grateful for your help with this issue. Thank you.
Kind regards,
Yagmur
Passing list-likes to .loc or [] with any missing label will raise
KeyError in the future, you can use .reindex() as an alternative.

See the documentation here:
http://pandas.pydata.org/pandas-docs/stable/indexing.html#deprecate-loc-reindex-listlike
  status_gain = copy_gain_df.loc[:, oncogenes_df['hugoSymbol']]
/mnt/lsf-nas-1/os-shared/anaconda3/envs/tybalt/lib/python3.5/site-packages/pandas/core/indexing.py:1367: FutureWarning:
Passing list-likes to .loc or [] with any missing label will raise
KeyError in the future, you can use .reindex() as an alternative.

See the documentation here:
http://pandas.pydata.org/pandas-docs/stable/indexing.html#deprecate-loc-reindex-listlike
  return self._getitem_tuple(key)
tybalt/scripts/nbconverted/process_data.py:255: FutureWarning:
Passing list-likes to .loc or [] with any missing label will raise
KeyError in the future, you can use .reindex() as an alternative.

See the documentation here:
http://pandas.pydata.org/pandas-docs/stable/indexing.html#deprecate-loc-reindex-listlike
  status_loss = copy_loss_df.loc[:, tsg_df['hugoSymbol']]
tybalt/scripts/nbconverted/process_data.py:270: FutureWarning:
Passing list-likes to .loc or [] with any missing label will raise
KeyError in the future, you can use .reindex() as an alternative.

See the documentation here:
http://pandas.pydata.org/pandas-docs/stable/indexing.html#deprecate-loc-reindex-listlike
  copy_status = copy_status.loc[:, mutation_status.columns].fillna(0).astype(int)```

nice catch @yagmuronay - although I am not sure that your solution will work. It is possible that the underlying data the notebook loads has changed in some way.
response = requests.get('http://oncokb.org/api/v1/genes')
oncokb_df = pd.read_json(response.content)
oncokb_df.to_csv(oncokb_out_file, sep='\t')

# Integrate copy number, oncokb gene-type, and mutation status to define status matrix
oncogenes_df = oncokb_df[oncokb_df['oncogene']]
tsg_df = oncokb_df[oncokb_df['tsg']]

# Subset copy gains by oncogenes and copy losses by tumor suppressors (tsg)
status_gain = copy_gain_df.loc[:, oncogenes_df['hugoSymbol']]
status_loss = copy_loss_df.loc[:, tsg_df['hugoSymbol']]
copy_status = pd.concat([status_gain, status_loss], axis=1)

what does tsg_df look like?

Dear Dr. Greg (@gwaygenomics),
thank you so much for your quick reply. The parameters tsg_df and oncogenes_df indeed have a column named "hugoSymbol". I found out later that the output was written as expected. Previousy, I looked into the wrong path, which was for raw data. All in all, these were only some warnings. Thank you so much for your time!


Stories: Add example of onEscape for Overlay

Add story to override onEscape with Overlay
Story: https://primer-react-gqp6t9wxq-primer.vercel.app/react/storybook?path=/story/internal-components-overlay--override-on-escape

Covered in https://github.com/primer/react/pull/1861


[DPTOOLS-2252] Publish Docker image in CI

Summary of Changes
Publish Docker image in CI
Tests
Performed make build-push-image
Documentation
N/A
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[x] PR passes make test
[x] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message"

Codecov Report

Merging #26 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #26   +/-   ##
=======================================
  Coverage   76.45%   76.45%
=======================================
  Files          10       10
  Lines         344      344
  Branches       37       37
=======================================
  Hits          263      263
  Misses         67       67
  Partials       14       14

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1a05eb2...97d4475. Read the comment docs.

Thanks @jinhyukchang!
There’s now a PR #27 which bumps the package to 1.0.2 and fixes #24 - perfect opportunity to test the new Docker CI 🙂