Can Function::call borrow self immutably?

Is there a reason that vm::api::Function::call (and call_async and call_fast_async) take a mutable reference to self? The code compiles fine if they take immutable references.

I had some plans that Function could instead hold the Mutex lock that protects the vm execution which would then require &mut. This would let the function be called multiple times withot reqaquiring the lock on every call. Never got around to it so atm it doesn't do anything.
You should still be able Clone the Function values though to share them and call them from multiple places. Not sure yet if I should fall back to making them &.


Improve cache handling in Build environments

Our build environments (appveyor and semaphoreci) have a time limit.
We introduced caching to stay within these limits.
However our environment changes which sometimes invalidates the caching.
I would like to improve the link between the cache and the statements that fill it. E.g. the link between msys2 and stack -install -ghc-compiler is now implicit.
Furthermore, I want that when caches are invalidated and more time is needed to fill the new cache, automatically the most time consuming tests are skipped to ensure staying within the time limit.

Furthermore, I would like that it is automatically detected when some items are NOT created anymore. For example msys2-2015xxxx became msys2-2018xxxx yet this was NOT signaled.
@keremispirli What does .installed mean in caching, e.g. in %LOCALAPPDATA%\Programs\stack\x86_64-windows\ghc-integersimple-8.2.2.installed?

Why is .stack-work in test/sqatt not cached on appveyor?
Currently only:
80 cache:
81 - .stack-root -> stack.yaml
82 - .cache -> stack.yaml
83 - '%LOCALAPPDATA%\Programs\stack\x86_64-windows\ghc-integersimple-8.2.2'
84 - '%LOCALAPPDATA%\Programs\stack\x86_64-windows\ghc-integersimple-8.2.2.installed'
85 - '%LOCALAPPDATA%\Programs\stack\x86_64-windows\msys64'
86 - '%LOCALAPPDATA%\Programs\stack\x86_64-windows\msys64.installed'
87 - .stack-work -> stack.yaml

Strange, I expect it to be smaller than .stack-work in the root....

Maybe @keremispirli has some other insights ...

.installed files exist next to respective directories and signal that the respective installation exists. That's why we cache both the file and the directory.
I think .stack-work in test/sqatt is just forgotten. Since we also cache .stack-work in the root, it makes sense to cache this one, too. In that case it's condition better be test/squatt/stack_appveyor.yaml. Of course, we should also verify that we don't exceed cache size with it.
There may be a way to detect whether caches were hit or missed; we should check AppVeyor docs. Then we can start by skipping Model Exploration tests in case of a cache miss or invalidation.

@keremispirli thanks! Will pick it up after my holidays.


new steam update broke the theme

Hi 👋🏻, thank you for your interest in providing this feedback. ✨
But there is a better place to share the same, which is the project repository, where you can talk to the theme maintainers.
✅ Clicking here, you can open an Issue on it.


Using try except blocks while executing queries

In analytics/views.py, shouldn't the queries be executed in try except blocks ?
I might be mistaken, but that would help in case of unreliable internet connection.

Generally the queries happen locally, and even if the queries fail, it's not clear that catching the exception will help in troubleshooting.

Thanks for the observation! As Steve mentioned, this code is run on the server, and so is not subject to fluctuations of internet connectivity. In general, things in static/ are run in the browser, and everything else is run on the server.
The right place to put retry logic here would be in the failure handlers of the $.get calls in static/styles/stats/stats.js. In general though for Zulip we just write an error message to the screen and expect the user will reload.


Loading OS ITN GML files to SQL Server database

Hello,
I was wondering if you had a configuration file/example available to load Ordnance Survey ITN data in GML format into a SQL Server database.

Hi, I don't know of anyone that has used Loader to load ITN into MS SQL Server but it should be possible.
I would of thought that you would be able to use osmm_itn_postgres.gfs to define the output tables with little or no modification.
You would also need a suitable MS SQL Server ogr_cmd in your configuration. See the OGR MS SQL page for details.
The supplied views however will probably need some work as I'm not sure the array functions we use in Postgres are available in MS SQL Server.
I'd be happy to add a section to complement the [ITN Postgres section in the wiki]https://github.com/AstunTechnology/Loader/wiki/Configuration-examples#ordnance-survey-itn-to-postgresqlpostgis).


Consider changing our inline namespace name with each release

We use an inline namespace in our code to help with library versioning. For example:
https://github.com/googleapis/google-cloud-cpp/blob/b5b37acee96cb962ca2db34d62b5cdc037e09d89/google/cloud/spanner/numeric.h#L30-L33
which today resolves to v1 for all of our libraries. We rarely change this inline namespace name.
The purpose of using an an inline namespace is to allow for versioning at the API and ABI levels. However, for this to actually work, we should change the inline namespace anytime we make an API or ABI change. An easy way to do this is to simply change the inline namespace name with every release.
Proposal
We should change the GOOGLE_CLOUD_CPP_NS define
https://github.com/googleapis/google-cloud-cpp/blob/b5b37acee96cb962ca2db34d62b5cdc037e09d89/google/cloud/version.h#L22-L26
to include the whole released version.
For example, the currently released version is v1.25.0, and so the inline namespace name should probably expand to v1_25_0
Other considerations

We should update our Support docs to specify how we expect users to use inline namespaces. In general, users should not name the inline namespace. This shortens their code and gives them a path to upgrading to newer versions without breaking, as long as they recompile. Users should only spell the inline namespace name when they tightly depend on a specific version. This should be rare. By default, users should not spell the inline namespace name. I believe this is all standard C++ advice that would likely apply to any C++ code that uses inline namespaces.
We should announce this change in the CHANGELOG.md.
What should be the inline namespace name for the unreleased code at HEAD? We could use the next library version, but that's not really correct, because the eventual release with that version will be different. We could use the string head, which is what Abseil does. Or we could use something like v1_26_0_preview.

We have a few *.dox files that reference v1 explicitly (for bad Doxygen reasons I think).  We would probably want those files to change automatically in each release?
Also, consider just using google::cloud::spanner::v1_31.  I do not think we ever break API (or ABI) on patch releases.


nginx operation not permitted error

Hi, thanks for putting together this buildpack.  It worked like a charm for us on https://github.com/nycplanning/labs-planninglabs.nyc , but just tried deploying to a brand new dokku server and got this error when it was deploying:
nginx: [alert] sysctl(KERN_RTSIGMAX) failed (1: Operation not permitted)
nginx: [alert] sysctl(KERN_RTSIGMAX) failed (1: Operation not permitted)
nginx: [alert] sysctl(KERN_RTSIGMAX) failed (1: Operation not permitted)
nginx: [alert] sysctl(KERN_RTSIGMAX) failed (1: Operation not permitted)
nginx: [alert] sysctl(KERN_RTSIGMAX) failed (1: Operation not permitted)
nginx: [alert] sysctl(KERN_RTSIGMAX) failed (1: Operation not permitted)

Dokku version is 0.9.4
Host machine is Ubuntu 16.04.2 LTS
Has anyone ever encountered this error before?

Never seen this error before, but it's caused by nginx. I looked on Google and apparently someone had the same issue and resolved it by using a newer nginx version.
See https://botbot.me/freenode/dokku/2016-08-05/?tz=America%2FLos_Angeles&page=1

[Miguel Andrade] @savant @michaelshobbs i figured out the problem
[Miguel Andrade] unbelieveble
[Miguel Andrade] the nginx build the buildpack was using wasn’t running
[Miguel Andrade] changed it to a version I built myself
[Miguel Andrade] and a much more recent version

For now, you can try using a newer nginx version (see the buildpack's compile file), and if that doesn't work, we'll look into what's wrong with installing/running it this way.

Thanks, I found that exact thread and reached out to Miguel on twitter, but no response yet.  I am not very experienced with this and am not clear on how to just tell it to use another nginx version.  The other examples of nginx buildpacks I am finding all seem to be building nginx from source, and are much more complex.  Where does "https://s3.amazonaws.com/devmynd-github/heroku-nginx-1.6.0.tar.gz" originate from, and does whoever put it there also have other versions available?

I figured it would be a simple version change in the URL, but that doesn't seem to be working. I have no idea where it originates from either (if you check the fork tree at the bottom of the readme you might have an idea though).
Your best bet is to find a popular buildpack that's installing nginx and to copy the code from there :/

I was trying to go down that route, either bringing the jekyll stuff from yours into  https://github.com/dokku/buildpack-nginx or vice versa...  this one builds fine on its own but I don't know how to add the jekyll stuff properly.  Is there some trick to troubleshooting these buildpacks besides just making changes, pushing code to dokku and waiting to see what happens?

I'm afraid not. Afaik, you'll have to keep pushing code and monitoring builds.
You can also attempt to remove nginx code from this buildpack and use the dokku/buildpack-nginx together as described in Using multiple buildpacks…

So, persistence paid off, I was able to get it working with dokku/buildpack-nginx plus the jekyll bits from yours.  The result is here: https://github.com/NYCPlanning/dokku-buildpack-jekyll
buildpack-nginx wants to copy everything from the root of the project into www and serve from there, so I modified it to grab _site/* after jekyll does its thing. Works like a charm on Dokku 0.9.4/Ubuntu 16.04.2 LTS.
Let me know if you are interested in a PR for this... not sure if it will have any issues with older nginx/linux that yours is working fine on right now.
Thanks for your help!

That's great! 👏
Lots of people are using this buildpack so suddenly changing it in such a major way might break some systems, so I'm not looking to do that until it's necessary.
If other people have this problem with my buildpack, I'll refer them to yours. I'll keep the issue open so that it's easier to find!

Makes sense. Feel free to reach out if / when you start making major changes, we will be happy to collaborate.


[BUGFIX] Renamed show_net variable in userData additional to show_gro…

…ss, cause text in backend is 'display gross'
1. Why is this change necessary?
Variable show_net is confusing, cause when TRUE it will display the gross prices.
2. What does this change do, exactly?
Renaming the variable name and it's usage in smarty template

Hi @wolv-dev and thanks for your contribution!
Would you be so kind to rebase and retarget your PR for 5.4?
By renaming a smarty variable you are introducing a break.

@shopwareBot Rebased and retargeted done!

@wolv-dev something went wrong there :blush:

@mitelg Yeah, but I don't know what went wrong.
I rebased 5.4 into my branch, but all following commits are added, too.
I attached a screenshot:

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-20225
Please use this issue to track the state of your pull request.

CLA already signed.

Hey @wolv-dev,
thanks for your contribution. :blue_heart: We're very sorry for the late response but as this PR is a big break you need to do some changes on it:
Please don't remove the old show_net variable as it is used by many payment plugins and other plugins as well. To introduce the new variable, mark the old one as deprecated for SW 6.0.
In addition to that, there are some more files that need to be changed to use the new variable (e.g. tests, mail contexts).

This PR was closed due to inactivity. If this change is still important to you, feel free to create a new pull request.
For more information about our contribution guidelines, see https://developers.shopware.com/contributing/contribution-guideline/.


[KYUUBI #6594] Port HIVE-26633: Make thrift client maxMessageSize configurable

:mag: Description
Fix #6594.
This PR ports HIVE-26633(https://github.com/apache/hive/pull/3674): Make thrift client maxMessageSize configurable to fix a regression after upgrading Thrift 0.16 in 1.9.0.
Types of changes :bookmark:

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪

Checklist 📝

[x] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

cc @zhanghe-git, can you test this patch?

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

Project coverage is 0.00%. Comparing base (9cf2245) to head (0617ffd).

Files
Patch %
Lines

.../org/apache/kyuubi/jdbc/hive/auth/ThriftUtils.java
0.00%
17 Missing :warning:

.../org/apache/kyuubi/jdbc/hive/KyuubiConnection.java
0.00%
16 Missing :warning:

.../apache/kyuubi/client/KyuubiSyncThriftClient.scala
0.00%
12 Missing :warning:

...in/scala/org/apache/kyuubi/config/KyuubiConf.scala
0.00%
5 Missing :warning:

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #6631   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         680     680
  Lines       42007   42047   +40
  Branches     5729    5732    +3
======================================
- Misses      42007   42047   +40

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

cc @zhanghe-git, can you test this patch?

@pan3793 I've tested this patch, and everything is working fine. It has been successfully verified using our internal test cases."

Merging to master/1.9, thank you @zhanghe-git


"CONFIRM PASSWORD" FUNCTIONALITY

Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Fixes:  #1891
Description
This update adds a "Confirm Password" feature to the registration form of a Firebase authentication-based login system. The change includes an additional input field for confirming the password and validation logic to ensure that both password fields match before allowing the user to register.

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]
Checklist:

[x] I have made this change from my own.
[ ] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have tested the changes thoroughly before submitting this pull request.
[ ] I have provided relevant issue numbers and screenshots after making the changes.

Attach before & after screenshots in your PR & then please open a new PR for review.

@codercake I tested it !
It is not asking any confirm password field


Use Casdoor as the authentication system

We have developed the Casdoor: https://github.com/casbin/casdoor , which is also officially hosted at: https://door.casbin.com/ It is already successfully deployed to: https://oa.casbin.com/
We will move the authentication (user table, password login, Email, SMS, OAuth providers like GitHub, Google, etc.) of this forum to Casdoor finally.

@oceanlvr

@hsluoyz I will replace the existing user authentication with casdoor

Fixed by: https://github.com/casbin/casnode/commit/9f260a7a014a31b82478ab0d308612a130e1e208


Feature: Added reddit posts integration

This PR is to implement the reddit-posts feature, requested in Issue #4.
Summary of changes:

Uses embedly as the embed generator
Updated link type checker to recognize (Reddit) comments vs post
Updated test.html with Reddit post example
Added embedly-loader.js to load the embedly code
Udated manifest.json to inject embedly-loader.js
Updated CSS to properly hide and unhide new reddit post embed

I also have some additional notes to share that might help out with future integrations:
CSS
I had to modify your CSS to use display:none to hide the iframes that are created by the embed code. The problem with visibility: hidden, is that it can easily be overridden by children if they have visibility: visible (which reddit embeds do). Setting display none on the parent, however, will hide the parent and all its children, no matter what their settings are. See this S/O question for details.
Embedly
The main way that Reddit supports embeds is through Embedly. However, Embedly also supports hundreds of other sources other than Reddit, so you might want to evaluate using it for more of your planned integrations.
For example, for Issue #1 - Twitter integration, you could easily use Embedly for this by just slightly tweaking the reddit-post code.
Embedly vs reddit embed
These are actually one and the same. Reddit is an Embedly partner, and if you actually do a file diff between https://cdn.embedly.com/widgets/platform.js and https://embed.redditmedia.com/widgets/platform.js, you can see that they are identical.
Manually creating card code
Here is their embed generator.
Initialization
Since the embedly code uses mutation observers, you don't actually need to call any init methods, even when you are injecting new cards into the page. However, they do have a JS API if you need to explicitly generate cards.
Direct API
I also want to point out that Reddit actually has a public oEmbed API, that would be useful if you didn't like Embedly's styling and wanted to construct your own cards. The URL is https://www.reddit.com/oembed?url={reddit_post_or_comment_link}

@mdolr  Thanks for the feedback. Couple things right away:
Embedly:

Cards are part of Embedly's free plan, and are not rate-limited, but come with branding (see the twitter demo below). Details here.
Embedly does support twitter - in fact, since I could re-use most of the same reddit code, I hacked together a demo in a couple of minutes. Branch is here, and here is a demo GIF:

CSS:

The problem with the visibility CSS is two-fold. It takes up space even when the content is hidden, and simply does not work if children inside it override it.

empty space demo
Children issue demo

I've added an additional commit that still uses display: none, but puts the tooltip back in the correct position like you wanted. I would highly recommend using this over visibility, or else you are very likely to end up with future issues.

I'll still make an attempt at avoiding the use of Embedly and manually using fetch() to hit the Reddit API - probably tomorrow. I'll ping this thread again when I have an update.

If you find a solution for the positionning I’ll switch to ˋdisplay: none` don’t worry, I understand the benefits from switching but it wasn’t my main priority.
The problem with the twitter integration is kind of the same problem that embedly has, the twitter embed script tries to pull other scripts from a CDN, what I was suggesting with embedly is maybe we could try to modify platform.js by downloading everything that it refers to from the CDN to make it work locally, or maybe we could try and use the manifest Content Security Policy (I couldn’t get it to work with twitter but I don’t really know how it works either)
I’ve posted a question on the twitter dev forum to know how to inject their script as a content script but if we find a way to make embedly work we can use it while finding a better solution.
Finally thanks for all the time you’re putting into this it is really appreciated to have such contributions

@mdolr OK, as requested, I just rewrote my code to use a manually crafted embed, rather than the Embedly code. It doesn't look quite as nice, since it's using my hastily written CSS rather than Embedly's team of designers (😬), but it gets the job done.

Essentially, the new code manually crafts a GET request to the Reddit API (https://api.reddit.com/api/info/), parses the response, and generates an embed to place in the tooltip. I added a new CSS template to format this embed, and also backed out  / removed the Embedly code.
If you want to put the Embedly code back at some point, all the commit history is still there, so you could easily revert back.
Hope this is what you were looking for!

I've tried your changes and I'm having issues with CORS, you were right, since Q1 2019 CORS are a thing with content scripts see chromium security post.
I'll try to implement the request through a background script and merge your PR when I've the time.
I still haven't given up on the embedly thing I think it is a great solution but we need to find a way to make it work everywhere and not just locally.

@mdolr Sounds good, and that is interesting about the change to CORS.
Good luck on building this out more. Seems like a fun and useful extension idea!


[WIP] stage1: Implemented notifications from the app in the container to the host

After this patch, systemd on the host marks the container as "active"
when the init systemd in the container sends the ready message
instead of doing it after the container is created.
After this patch rkt sets --notify-ready=yes on systemd-nspawn.
Also appc/spec in godeps is patched to expose SupportsNotify.
Fixes: https://github.com/coreos/rkt/issues/1464
Warning:
If the application launched by rkt does not notify (with sd_notify())
that it is ready, then the application may be killed after a timeout.
Note: Requires systemd v231 in stage1, so this commit must not be merged before.

do we need an issue about bumping to systemd 231?

Dependency https://github.com/appc/spec/pull/626

Dependency https://github.com/coreos/rkt/issues/2843

Both dependencies are now cleared and I think we can proceed on this. Tentatively milestoning for 1.15.0.

@alepuccetti let us know if the ETA is ok for you, @sur can shepherd assist on this.

@lucab I think that the ETA is ok, I will start work on it this afternoon or tomorrow and give you a better feedback about it.

just a couple of nits in the documentation, thanks @alepuccetti! :+1:

@alepuccetti We should have functional tests for this, since the changes are minimal here, I think, we can do this in a follow-up issue.

@alepuccetti somehow master commits were mangled into this PR, can you correct this?

@alepuccetti your commits are mangled

There is a merge conflict with prepare-app.c. You'll have to add an additional , false field at the end of the files_mount_table.

ok, LGTM once green, I create a follow-up issue for functional tests.

Hmm, failed on Fedora, coreos flavor, TestNetDefaultRestrictedConnectivity. Investigating.

Known flake: https://github.com/coreos/rkt/issues/2943

Merging. The build was mostly green (except some known flakes), but there was a small documentation change.


Docker Logs weird behaviour

When reading logs it seems like the first line is missing and some weird characters appear at the start of every line (if casting it to int I get the value 1).
My college also said that his last line is missing. My mongodb example does not illustrate that, will try to find a good example.
I attached how the logs looks with "docker logs" and how it looks with Java:
DockerLogs.txt
I tried using charsets of UTF-8 and US-ASCII to see if that helped but same result

It seems that some pieces of the header are also displayed. Will have a look.

Now the headers works fine and the first line displays.
But it doesn't show all the lines in the end, they get suspiciously cut (attached log):
DockerLogsCut.txt

It now gets all the logs :)
The only thing to remark is that I get this logged every time at the start:
[pool-1-thread-1] ERROR java.io.InputStreamReader - Error while pumping stream.
java.io.EOFException
at okio.RealBufferedSource.require(RealBufferedSource.java:64)
at okio.RealBufferedSource.readByte(RealBufferedSource.java:77)
at io.fabric8.docker.client.utils.DockerStreamPumper.run(DockerStreamPumper.java:61)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)

Will look at fixing that too then :) So close!

Can you share a way to repro? Can't seem to hit it myself.


Unexpected behavior for cachedir

If you call the util.get_cache_netcdf function with a function (as the get_dataset_func argument) which in turn calls the util.get_cache_netcdf function unexpected errors may occur. The cachedir argument that was passed to the original util.get_cache_netcdf function is not passed onto the get_dataset_func function. Thus when util.get_cache_netcdf is called for a second time it is assumed that cachedir=None. If cachedir is None an attempt is made to create a temporary cache directory. Creating or accessing a temporary cache directory is not always possible and can result in an error. Also it is unexpected that a temporary directory is used for caching while a cache directory was specified in the original call of util.get_cache_netcdf.
some thoughts:

there is no good reason to call the util.get_cache_netcdf function with a function (as the get_dataset_func argument) that calls the util.get_cache_netcdf. Why cache the same thing twice? However it is also hard to check that this occurs.
The same error occurs when you use cachedir=None when calling the util.get_cache_netcdf function. A solution will be to raise an error when cachedir=None.

fixed with new caching module e531ac033c8524e3169fc8f41f38d00f189c089a


Unexpected behavior of cursor

Steps (mask for digits Us SSN --____):

Enter data into the input control.
Unselect the field.
Select the field one more time.
Select entered data.
Try to enter any other type of symbols except digits.
Unselect the input control.

Result: The cursor is displayed on the second position.

got this issue as well


Issue with Missing "Frame Relevance" Content in LLM Response

Thank you for open-sourcing this excellent work. I am currently reproducing your project and encountered an issue with a specific part of the code.
The function update_relevance_response is supposed to extract relevance information from the response text using the following code:
def update_relevance_response(text):
response = text
relevance_match = re.search(r"frame relevance: [([0-9, ]+)]", response)
if relevance_match:
relevance = list(map(int, relevance_match.group(1).split(',')))
return relevance
However, my response does not contain any "frame relevance" content, which is expected to be provided by the LLM. The specific response I received is:
'prediction: C, center_shift=0.000000, iteration=6, tol=0.000100] explanation: The actions described involve cleaning kitchen items such as'
Without this information, the function does not work as intended and will meet this error(UnboundLocalError: local variable 'relevance' referenced before assignment).
Have you encountered a similar issue before, where the LLM response lacks the "frame relevance" content? If so, could you provide any guidance or suggestions on how to ensure that the LLM includes this information in its response?

I think it is the OpenAI model problem that doesn't output correctly. I think the easiest solution is skip this sample when this sorts of issue encounter.
In the future (if time allows), I might update the code to tackle this edge cases due to the unstableness of GPT models. Thanks.


Kafka connections showing strange behaviors - not connecting, and not attempting to reconnect

Describe the bug
Strange localhost behaviors.  I see (using tshark) a connection attempt to <IP_ADDRESS>:9092 when I first start up the package, but then on repeated connection attempts there are no further packets sent to <IP_ADDRESS>:9092.
Additionally, it shows strange port numbers and IP addresses on first attempt - see the logfile.
To Reproduce
Set up kafka broker on localhost, configure system per config example below.
Expected behavior
I expect a connection to my kafka broker and topic.
Additional context
Kafka config file:
 - name: kafka-out
    kafkaproducer:
      remote-address: <IP_ADDRESS>
      remote-port: 9092
      connect-timeout: 5
      retry-interval: 10
      flush-interval: 30
      tls-support: false
      tls-insecure: false
      tls-min-version: 1.2
      ca-file: ""
      cert-file: ""
      key-file: ""
      sasl-support: false
      sasl-mechanism: PLAIN
      sasl-username: "user"
      sasl-password: "kmazqAIqrx7L"
      mode: flat-json
      buffer-size: 100
      topic: "my-topic"
      partition: 0
      chan-buffer-size: 65535
      compression: "none"

Output on log:
INFO: 2024/01/01 06:39:02.051467 [dnsdist-from-outside] collector=dnstap - is listening on tcp://[::]:59312
INFO: 2024/01/01 06:39:02.051637 [prom] logger=prometheus - is listening on [::]:8082
ERROR: 2024/01/01 06:39:02.194843 [kafka-out] logger=kafka - failed to dial: failed to open connection to debian:9092: dial tcp: lookup debian on <IP_ADDRESS>:53: server misbehaving
INFO: 2024/01/01 06:39:02.194915 [kafka-out] logger=kafka - retry to connect in 10 seconds
INFO: 2024/01/01 06:39:03.889922 [dnsdist-from-outside] collector=dnstap#1 - new connection from <IP_ADDRESS>:59888
INFO: 2024/01/01 06:39:03.889976 [dnsdist-from-outside] processor=dnstap#1 - initialization...
INFO: 2024/01/01 06:39:03.897504 [dnsdist-from-outside] processor=dnstap#1 - waiting dns message to process...
INFO: 2024/01/01 06:39:03.899095 [dnsdist-from-outside] collector=dnstap#1 - receiver framestream initialized
ERROR: 2024/01/01 06:39:12.051153 [kafka-out] logger=kafka - failed to established connection
INFO: 2024/01/01 06:39:12.196031 [kafka-out] logger=kafka - connecting to kafka=<IP_ADDRESS>:9092 partition=0 topic=my-topic
ERROR: 2024/01/01 06:39:12.196328 [kafka-out] logger=kafka - failed to dial: failed to open connection to <IP_ADDRESS>:9092: dial tcp <IP_ADDRESS>:9092: operation was canceled
INFO: 2024/01/01 06:39:12.196378 [kafka-out] logger=kafka - retry to connect in 10 seconds
ERROR: 2024/01/01 06:39:22.196729 [kafka-out] logger=kafka - failed to established connection
INFO: 2024/01/01 06:39:22.196908 [kafka-out] logger=kafka - connecting to kafka=<IP_ADDRESS>:9092 partition=0 topic=my-topic
ERROR: 2024/01/01 06:39:22.197070 [kafka-out] logger=kafka - failed to dial: failed to open connection to <IP_ADDRESS>:9092: dial tcp <IP_ADDRESS>:9092: operation was canceled
INFO: 2024/01/01 06:39:22.197093 [kafka-out] logger=kafka - retry to connect in 10 seconds

Strange localhost behaviors. I see (using tshark) a connection attempt to <IP_ADDRESS>:9092 when I first start up the package, but then on repeated connection attempts there are no further packets sent to <IP_ADDRESS>:9092.

Issue identified, the buffer is no more cleared after some connection attempts
fix to come...

Fixed in release v0.40.0
If you encounter any further issues, please feel free to open a new ticket.
Thank you for your feedback!


Newline required after -> in multiline when case.

when branch consisting of multi-line statement forces newline after ->
                    when (error) {
                        is NullPointerException -> repository.rxMethod()
                            .subscribeOn(Schedulers.io())
                            .andThen(Flowable.just(Unit))

                        is IllegalStateException -> doSomethingElse()

Expected Behavior
From the official style guide:
https://kotlinlang.org/docs/coding-conventions.html#control-flow-statements

In a when statement, if a branch is more than a single line, consider separating it from adjacent case blocks with a blank line

Reading this, the only requirement is that a blank line separates the NPE and ISE branches, which it does.
Note that IJ IDEA's "official kotlin style guide" formatter accepts this format (and will properly insert the blank line between the branches, as required by the style guide).
Observed Behavior
Ktlint treats this as an error:
Missing newline after "->"

But there is no requirement in the official style guide for this.
Ktlint instead wants this format (with or without braces around the case):
                    when (error) {
                        is NullPointerException ->
                            repository.rxMethod()
                                .subscribeOn(Schedulers.io())
                                .andThen(Flowable.just(Unit))

                        is IllegalStateException -> doSomethingElse()

Steps to Reproduce
The first code block reproduces the issue.
Your Environment

Version of ktlint used: 0.39.0
Name and version (or code for custom task) of integration used (Gradle plugin, Maven plugin, command line, custom Gradle task): Gradle org.jlleitschuh.gradle.ktlint v9.2.1
Version of Gradle used (if applicable): 6.7.1
Operating System and version: macOS 11.3

I just tested with v0.40.0, and this is no longer an error! I should have tried that first 🤦♂

I do have the same problem with 0.42.0. Was there a regression?


Video call got pause after few seconds and Some times Room Full errors

Its video calling works but its video call got pause after few seconds and change the screen and need to disconnect the call after this and then again I  create or want to jain this same name room then its give me error the Room size was full

Video lags many times, not gets connected quickly. How to clean room when it was full.

@niravPercept123 am facing the same issue when i connect with android device , if you find any solution please help me ?

I have same issue

I am facing same issue. Video is freeze at ios size after few seconds of call.
Anyone get the solution. Please suggest


Do not try to publish proofs for malicious ATXs during sync

Motivation
Publishing is blocked during sync because Syncer::ListenToATXGossip() returns false, and thus every malicious ATX being synced is causing an error resulting in an interruption of sync.
Description
Do not try to publish ATX malfeasance proofs during sync.
Test Plan
Sync a test node, look for sync interruptions / errors

@ivan4th should we merge the change?

Yes, let's merge

bors merge


Remove all references to NodeJS core packages

Closing this issue and let's create more specific issues 👍


TransE

Excuse me, after reading your paper, I don't quite understand that Transe is not suitable for multi hop reasoning. Can you explain it?

In TransE, if a vector for a relation is r, then vector for the inverse relation should be -r . So for a multi-hop path such as co-actor which requires path (acted in, acted in inverse), the resulting 2-hop path embedding would become 0 vector. So TransE may not be able to answer such questions.

In TransE, if a vector for a relation is r, then vector for the inverse relation should be -r . So for a multi-hop path such as co-actor which requires path (acted in, acted in inverse), the resulting 2-hop path embedding would become 0 vector. So TransE may not be able to answer such questions.

I see what you mean. Thank you for your answer!


submit test results

I submit the results on the DanceTrack website, but it keeps showing "Sorry, failed to upload file." I have strictly followed the submission format.

me too

Have you fixed it? I noticed some failure due to incorrect zipping / naming. Let me know if you have any existing question or trouble


Add justfile to simplify docker usage

This adds a justfile to simplify docker usage. Eventually more recipes can be added to it to make executing any kind of long complex commands simpler.

Just is a very popular task runner, with 19k stars and 7.5 million downloads. It supports autocomplete, IDE integration, CI installation, and even github highlighting
This is an optional feature, and does not have to be used. I think we may want to use this method to keep all CI scripts in one place as well - in order to be able to run them both locally or in a workflow.

@louwers while ubiqutous, Makefile is something many people are trying to get rid of due to its syntax complexity and combersomness as a TASK manager.  Also, this is not specifically for docker, but rather for any recipe project-wide.
Justfile file is OPTIONAL - developers do not have to use it, but it offers great value that we already use in martin, tilespec, and maplibre-rs.  Also, there are 19k usages on github excluding forks -- each of which is likely a full blown, often complex project.

You can use a Makefile as a simple task runner without dependencies too. That said, this is a monorepo, for better or worse. Many commands you might want to run are not even cross-platform. It does not make sense to have a project wide task runner.
It's also pretty embarrassing to have a task runner, but it cannot do the most important things developers want to do when they come to the project - build, run & compile for the platform they are interested in.

Nothing the maintainers put in a repo is really optional is the thing.

As clearly observed from my prior work in numerous repos, I am pretty good at maintaining things, and also pretty good at getting them to a better state. I am willing to invest a lot of time in this repo, move it forward, introducing clear ways to build all targets one after another on all platforms.  Justfile would help me achieve this goal by keeping things cross-platform and simple to build, rather than deal with Makefiles or other build systems (i.e. the death by a thousand cuts argument)

The Native repo had a whole lot of semi-broken dependences last year which the maintainers and team cleaned up.  We should be very careful when adding another level of build complexity.  It really needs to be compelling.

Justfile would help me achieve this goal by keeping things cross-platform and simple to build

It is simple for you because you have Docker installed and running, know Docker very well, have inspected and extended the Dockerfile, so you know what is happening when you run these commands and can debug any issues that occur.  But adding another level of indirection (or two in this case) is not helpful in general in my opinion.
I recommend keeping the justfile in the repo and create a personal .gitignore to exclude it from git.

@louwers lets not conflate two unrelated things - justfile and docker. They are not related, because next I will add things like  cmake-core and bazel-core, plus a build-core which will redirect to one of this targets.  And the main benefit of all this is that I can share these experiments with others while being on the shared repo - i.e. i do not want to fork the whole project because it will be a massive pain to merge it in later on.

This really sounds like it should be worked out in a fork.

I have made a temporary wrapper repo for this functionality to avoid prolonging this discussion. Hopefully there will be some positive feedback in the future and we could bring back this topic. For those who want to use simpler way of building maplibre-native, see https://github.com/nyurik/just-maplibre-native


Topic/authorization

This is a continuation of PR #264, adding HTTP Basic and Digest Authorization in the form of middleware. There is one open problem left concerning the composition of services using orElse. The solution of this problem was deferred to later; for details see the discussion in PR #264.

@frehn, I'm going to merge this and quickly take care of @julien-truffaut's suggestions in a very quick follow up commit. Thanks a lot for this work, and sorry the merge process has been so long: typically it would be faster where it not for the problem we still face with our monad stack ordering.
Thanks again.

@bryce-anderson, thanks for merging. I very much enjoyed the discussion and feedback on the PR and am looking forward to contributing to http4s again in the future.


Unauthorized Client

I am trying to run the app but when trying to login I get {"error": "unauthorized_client"} in the LogCat.
I am creating the Client ID in the admin panel, and then pasting it in the config.yaml. Is there a step I am missing?

Hi @dri94, you should change the Authorization grant type field from Client credentials to Resource owner password-based


Siad Crashed - EXT4 Filesystem Corrupted

I ran siad 1.3 on Debian 8.6 x64. It was in sync a few weeks ago, then I traveled. I came back, powered up my Linux box & ran siad to sync. Meanwhile, there were some package updates which I installed, while siad was running. Siad suddenly froze along with sia-ui and I closed it, then ran it again, and the whole machine died.
In the shell, when I typed "ls" it said "Input/Output Error" and after a reboot, the filesystem was gone. I spent 2 days recovering files and fixing stuff, and now have it half working (can login).
Where do I find the log files? What else do you need to know? I run Debian as the main OS on a Samsung NVMe.

log files are present in the same directory as the blockchain, wallet, etc. If you're using Sia-UI, I believe it's ~/.config/Sia-UI/sia.
I'm not sure if the logs will tell you anything interesting, though. My guess is that the failure is not attributable to siad or Sia-UI. (Of course, if it is, we'll make it our top priority to fix.)
Sorry about losing your fs, that's a major PITA. There may be some ext4 logs that you can query as well to identify the root cause.

Hey Luke,
I checked Sia and didn't find anything useful. I checked system logs and saw many NVMe errors from the kernel side.
It appears that Sia generated enough disk stress that it triggered a bug in kernel 4.8 used in Debian 8.x with NVMe.
However, none of the online reports showed anyone suffering from a filesystem corruption.
I don't think it's Sia's fault at this time, but let this be a warning to anyone using NVMe with 4.8 kernel on Linux.
Should I attach Sia & syslogs for others to see? Or simply close the ticket?

Sia does tax the disk more than we'd like. I'm running a Samsung NVMe myself, so fingers crossed. :P
Logs wouldn't hurt, but yeah I'll close this since I don't think there's anything actionable at this time

Alright. I'll post the logs soon and then close the ticket.

@tbenz9 this can be closed and reopened if/when the logs come. it sounds like it's not (substantially) sia's fault to me for a number of reasons.

auth.log
daemon.log
debug.log
dmesg-errors.log
dpkg.log
kdm.log
kern.log
messages.log
syslog.log
user.log


recaptcha error:  Unexpected character encountered while parsing value: <. Path '', line 0, position 0.

dear all,
the system work fine and very well for me, but if I try to enable the recaptcha feature I receive this error after the click on "change password" button:
Unexpected character encountered while parsing value: <. Path '', line 0, position 0.
The recaptcha seems to works fine: I see the image and when I select the correct squares and click on "verify" button I receive the "I'm not a robot" correct validation.
Please can you help me to correct this issue?
Thanks in advance
Bye

Possibly WAF issue. Those F5s are kinda terrible. Outside the scope.


Steps to add service account

The above documentation says how to use their personal inbox, It will be really helpful if the documentation provides the steps on how to add the connection of share account / service account.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a38b2975-5eb6-1aa4-c1fd-d0f033ace5ca
Version Independent ID: 85899731-9939-a779-29c4-4da652667b74
Content: Run flows based on email properties - Microsoft Flow
Content Source: flow-docs/email-triggers.md
Service: flow
GitHub Login: @MSFTMan
Microsoft Alias: deonhe

@NandaDuddala Thanks for your feedback. We'll look into creating a new doc for this.


Templates' buttons got an update... Again...

Now button types can be combined into one ugly message. Which, unfortunately, means the whole rewrite done for templates.ts should be mostly reverted...
I hate templates.
https://developers.facebook.com/docs/whatsapp/business-management-api/message-templates/components#buttons

At least now I have a great excuse for creating v2 :)


hiredis is not compiling in nodejs 12

Is there any way that this package will no longer depend on hiredis? Hiredis is not building properly using node-gyp in nodejs 12

I'm getting the same issue when building with parcel

Any news on this? As hiredis is pretty much abonded, it should be removed as dependency.

@sookoll could you install "redis"? I didn't quite understand your solution. Help?

If you can upgrade the latest node-redis you don't need hiredis, it uses this repo which is fastest than hiredis as authors claim.
If you are like me and stuck at some version, you can use this fork https://github.com/redis/hiredis-node/pull/144 until the PR will get merged

I really didn't need hiredis. I moved to use "ioredis" and it works good for me.

Hey all,
hiredis is no longer used in the latest parser, the the redis library has been updated to the latest parser version in https://github.com/NodeRedis/node_redis/commit/78936ac50c43ceb74654501a5da4b2994efe1217 (unreleased still, no eta).
Given that a fix is sat on master for this I'm going to go ahead and close this issue now. Thanks

I'm having this issue when trying to install redis-connect.... any clue @Salakar ?

It's been more than a year, can there be a new release to include this fix? redis-parser, and redis packages do not work on node 12 and node 14.


Opening Connection using SQLite JDBC

Hi fellas,
Im also working on android + activejdbc right now. And im trying to open a connection using activejdbc.
`try{
String db = "jdbc:sqlite:" + getDatabasePath("test.db").getAbsolutePath();
Base.open("org.sqlite.JDBC",  db, "root", "");
Rec_test test = new Rec_test();
}catch (Throwable e){
e.printStackTrace();
Toast.makeText(getApplicationContext(),e.toString(), Toast.LENGTH_LONG).show();
}finally {
Base.close();
}
`
But I have an error saying :
W/System.err: java.lang.UnsatisfiedLinkError: dalvik.system.PathClassLoader[DexPathList[[zip file "/data/app/ph.com.partnersolutions.testjdbc-1/base.apk", zip file "/data/app/ph.com.partnersolutions.testjdbc-1/split_lib_dependencies_apk.apk", zip file "/data/app/ph.com.partnersolutions.testjdbc-1/split_lib_slice_0_apk.apk", zip file "/data/app/ph.com.partnersolutions.testjdbc-1/split_lib_slice_1_apk.apk", zip file "/data/app/ph.com.partnersolutions.testjdbc-1/split_lib_slice_2_apk.apk", zip file "/data/app/ph.com.partnersolutions.testjdbc-1/split_lib_slice_3_apk.apk", zip file "/data/app/ph.com.partnersolutions.testjdbc-1/split_lib_slice_4_apk.apk", zip file "/data/app/ph.com.partnersolutions.testjdbc-1/split_lib_slice_5_apk.apk", zip file "/data/app/ph.com.partnersolutions.testjdbc-1/split_lib_slice_6_apk.apk", zip file "/data/app/ph.com.partnersolutions.testjdbc-1/split_lib_slice_7_apk.apk", zip file "/data/app/ph.com.partnersolutions.testjdbc-1/split_lib_slice_8_apk.apk", zip file "/data/app/ph.com.partnersolutions.testjdbc-1/split_lib_slice_9_apk.apk"],nativeLibraryDirectories=[/data/app/ph.com.partnersolutions.testjdbc-1/lib/x86, /system/lib, /vendor/lib]]] couldn't find "libsqlitejdbc.so"
Didn't you guys encountered this ? Im using sqlite-jdbc-3.23.1.jar right now downloaded from xerial.
Help would be much appreciated.
Originally posted by @VinsoyTisoy in https://github.com/javalite/activejdbc/issues/303#issuecomment-442076467

@VinsoyTisoy did you package all dependencies into your app, such as ActiveJDBC classes?
Code-wise, use try-with-resources: http://javalite.io/database_connection_management#use-try-with-resources

Also, seems that your error: couldn't find "libsqlitejdbc.so" - try to open a connection with just JDBC without the framework to ensure you have all the right libs in place. Closing as not related to framework. You can continue to comment if you want

@ipolevoy Tried it without using the framework and the error still occurs. Im just hoping that someone could give me a hint since other have tried implementing activejdbc on android.

@VinsoyTisoy if you have the same error without the framework, then clearly this is not activejdbc issue. I'm not a mobile developer, you need to ask the same question on Android forums - Stack Overflow?

Yeah, will try then. Thanks for the response.


Token

Coverage decreased (-0.7%) to 50.672% when pulling 5ebb82e355dcd33a4ca74e3de04c8fa4e474a5bd on token into 4276de22f7fa11b94b8a2157e7a6345c09f878ee on master.

Coverage decreased (-0.7%) to 50.672% when pulling 821c7851c60d73b1dbb2807a629208db443a6dcc on token into 4276de22f7fa11b94b8a2157e7a6345c09f878ee on master.

Coverage decreased (-15.7%) to 35.673% when pulling cfe5d34896a3e59016e460139295e49c76d35c0c on token into 4276de22f7fa11b94b8a2157e7a6345c09f878ee on master.

Coverage decreased (-3.2%) to 48.101% when pulling cc6d7f884e21be5f49ffda1c903932cf0ca5ba9b on token into 4276de22f7fa11b94b8a2157e7a6345c09f878ee on master.


Releases don't include the default configuration files

Current Behavior
I've downloaded the latest version of Neo Go and tried to run it using the following command:
.\neo-go-windows-amd64.exe node -m. It didn't work: config './config/protocol.mainnet.yml' doesn't exist.
Expected Behavior
Running neo-go.exe -m should work.
Possible Solution
The ./config should be shipped with the binaries, or the binary should create them if they don't exist (main net and test net).
Steps to Reproduce

Download Neo Go
Run the binary using the node -m option.

Context
It's not possible to run the node without the file. The configuration file is not small and can't be typed. Users must download it from the repository and store it in the config folder.
Your Environment

Version of the product used: Zephirantes
Server setup and configuration files used: Windows Server

-m/-t/-p have a bit different meaning, it can't be changed this easy, but we know about the problem. Still have not found any decent solution.

One alternative could be to add an init command that generates the files.
Another one would be to use the rpc endpoint and use RPC to get the network magic, standby validators and other config info. Some SDKs do that already, I'm not sure if Neo Go does (c# doesn't)

We can bundle our default ones and extract them using CLI. RPC would be lacking seeds at least, but it's an interesting idea for extension anyway.

Related to #3450.


[BUG]: CSM Operator fails to install CSM Replication on the remote cluster

Bug Description
When installing CSM Replication via CSM Operator on two OpenShift clusters, the csm-operator-manager pod is unable to communicate with the secondary cluster and is thus unable to complete validation pre-checks on the remote/secondary/disaster-recover cluster, blocking the install of the CSI driver and CSM Replication via CSM Operator.
Logs
2023-09-22T16:52:17.811Z INFO workspace/main.go:143 Use ConfigDirectory /etc/config/dell-csm-operator {"TraceId": "main"}
1.6954015401672292e+09 INFO controller-runtime.metrics Metrics server is starting to listen {"addr": "<IP_ADDRESS>:8080"}
1.695401540168204e+09 INFO setup starting manager
1.6954015401686692e+09 INFO Starting server {"path": "/metrics", "kind": "metrics", "addr": "<IP_ADDRESS>:8080"}
I0922 16:52:20.168696 1 leaderelection.go:248] attempting to acquire leader lease openshift-operators/090cae6a.dell.com...
1.6954015401686754e+09 INFO Starting server {"kind": "health probe", "addr": "[::]:8081"}
I0922 16:52:37.501589 1 leaderelection.go:258] successfully acquired lease openshift-operators/090cae6a.dell.com
1.6954015575017395e+09 DEBUG events Normal {"object": {"kind":"ConfigMap","namespace":"openshift-operators","name":"090cae6a.dell.com","uid":"10a95494-4bdf-4b41-b82a-bbe9b242d736","apiVersion":"v1","resourceVersion":"1042594"}, "reason": "LeaderElection", "message": "dell-csm-operator-controller-manager-57798fdd78-mndc2_1aaeb44f-5e7e-43fd-b14c-28663aca5b63 became leader"}
1.6954015575018597e+09 DEBUG events Normal {"object": {"kind":"Lease","namespace":"openshift-operators","name":"090cae6a.dell.com","uid":"b00994e6-cc9c-4669-89b6-e407116c33c4","apiVersion":"coordination.k8s.io/v1","resourceVersion":"1042595"}, "reason": "LeaderElection", "message": "dell-csm-operator-controller-manager-57798fdd78-mndc2_1aaeb44f-5e7e-43fd-b14c-28663aca5b63 became leader"}
1.6954015575019403e+09 INFO controller.containerstoragemodule Starting EventSource {"reconciler group": "storage.dell.com", "reconciler kind": "ContainerStorageModule", "source": "kind source: *v1.ContainerStorageModule"}
1.6954015575019846e+09 INFO controller.containerstoragemodule Starting Controller {"reconciler group": "storage.dell.com", "reconciler kind": "ContainerStorageModule"}
1.695401557602666e+09 INFO controller.containerstoragemodule Starting workers {"reconciler group": "storage.dell.com", "reconciler kind": "ContainerStorageModule", "worker count": 1}
2023-09-22T16:52:53.655Z INFO controllers/csm_controller.go:203 ################Starting Reconcile############## {"TraceId": "isilon-1"}
2023-09-22T16:52:53.655Z INFO controllers/csm_controller.go:206 reconcile for {"TraceId": "isilon-1", "Namespace": "dell-csm", "Name": "isilon", "Attempt": 1}
2023-09-22T16:52:53.656Z DEBUG drivers/powerscale.go:80 preCheck {"TraceId": "isilon-1", "skipCertValid": true, "certCount": 1, "secrets": 1}
2023-09-22T16:52:54.265Z INFO controllers/csm_controller.go:1221 proceeding with fresh driver install {"TraceId": "isilon-1"}
2023-09-22T16:52:54.276Z INFO controllers/csm_controller.go:1124 Driver not installed yet {"TraceId": "isilon-1"}
2023-09-22T16:52:54.336Z INFO record/event.go:285 Event(v1.ObjectReference{Kind:"ContainerStorageModule", Namespace:"dell-csm", Name:"isilon", UID:"020e0b89-6313-47a7-a048-9faece60dfb2", APIVersion:"storage.dell.com/v1", ResourceVersion:"1042687", FieldPath:""}): type: 'Warning' reason: 'Updated' Failed Prechecks: failed replication validation: Get "https://api.<endpoint>.com:6443/api?timeout=32s": dial tcp: lookup api.<endpoint>.com on <IP_ADDRESS>:53: no such host {"TraceId": "main"}
################End Reconcile##############
2023-09-22T16:52:54.350Z ERROR utils/status.go:345 failed replication validation: Get "https://api.<endpoint>.com:6443/api?timeout=32s": dial tcp: lookup api.<endpoint>.com on <IP_ADDRESS>:53: no such host *************Create/Update isilon failed ******** {"TraceId": "isilon-1"}
1.695401574350591e+09 ERROR controller.containerstoragemodule Reconciler error {"reconciler group": "storage.dell.com", "reconciler kind": "ContainerStorageModule", "name": "isilon", "namespace": "dell-csm", "error": "failed replication validation: Get \"https://api.<endpoint>.com:6443/api?timeout=32s\": dial tcp: lookup api.<endpoint>.com on <IP_ADDRESS>:53: no such host"}
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.1/pkg/internal/controller/controller.go:266
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.1/pkg/internal/controller/controller.go:227
2023-09-22T16:52:54.355Z INFO controllers/csm_controller.go:203 ################Starting Reconcile############## {"TraceId": "isilon-2"}

Screenshots
From Primary Cluster

Additional Environment Information
No response
Steps to Reproduce

Install CSM Operator.
Map the secondary cluster's IP to the OpenShift API endpoint in the /etc/hosts file.
Configure pre-requisites for CSM Replication.
a. Clone csm-replication repo and build repctl.
b. Add cluster info to repctl using repctl cluster add ...
c. Install replication CRDs.
d. Inject service account info
e. Create storage-classes.
Configure CSI driver pre-requisites.
a. Create driver namespace.
b. Create isilon-creds generic secret.
c. Create empty secret
Create a ContainerStorageModule instance in the OpenShift Console. Editing the config to enable replication, and providing the name of the target cluster under envs.name: TARGET_CLUSTERS_IDS and clicking 'Create'.

Expected Behavior
The csm-operator-manager pod should query the secondary cluster's API endpoint, complete pre-check validation, and install CSM Replication and the CSI driver on both primary and secondary cluster.
CSM Driver(s)
CSI PowerFlex v2.7.0
CSI PowerScale v2.7.0
Installation Type
CSM-Operator v1.2.0
Container Storage Modules Enabled
CSM-Replication v1.5.0
Container Orchestrator
OpenShift 4.12
Operating System
RHEL 9.2

Documentation has been updated to address this potential issue as part of dell/csm-docs#844.
It should be noted, this method is a workaround for users with environments that use multiple clusters and do not have the DNS configured to resolve queries against the FQDN of the remote cluster. It is highly recommended that the customer configure the DNS to support this name resolution.


feat: Add prometheus metrics for different version between latest and deployed

Is your feature request related to a problem? Please describe it.
I wanted to have a prometheus metric when deployed and latest version are not the same in an alert by my grafana instance.
Describe the solution you want
I add a prometheus metric deployed_version_different_than_latest_version that is set to 1 if version are different and 0 if not.
Additional context
Good luck with the review and the maintenance of this awesome project !

It's ok, I have worked a lot for understand every thing but if you think it's better like you do, I follow you. Thanks a lot, I will create a grafana dashboard latter this week 😄


(Partial) fix for bug around RVM detection.

When running multiple Jenkins build agents (slaves), particularly if
these are created and destroyed independently of the master instance,
the caching of the rvm_path on the master prevents the absence of
on rvm installation on the current slave being detected.
Note that the value is still being held globally, so two concurrently
executing jobs still have the potential to fail to detect the presence
of rvm, though the likelihood is deminished.

Like Pull Request #18 this appears to be failing on Jenkins because gem install bundler doesn't get called appropriately.

Nice work. I've got this error too, looking through the console logs it looks like this fixes the issue. Is there any way we can build this and get it working? Thanks :)

Same problem here. Hope this can be merged.

Thanks for PR, and sorry for taking long to respond. I took different approach ( 296a32399bfa70a8fa70e9ea6dd0c194e41d0dd4 ). version 0.6 should be released soon.


civo: 1.0.88 -> 1.0.89

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/civo/cli/releases.
meta.description for civo is: CLI for interacting with Civo resources
meta.homepage for civo is: https://github.com/civo/cli
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.0.89 with grep in /nix/store/dvdi3qhjnkh30gk62qpfz3af9skvwpab-civo-1.0.89
found 1.0.89 in filename of file in /nix/store/dvdi3qhjnkh30gk62qpfz3af9skvwpab-civo-1.0.89

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

civo

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dvdi3qhjnkh30gk62qpfz3af9skvwpab-civo-1.0.89 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A civo https://github.com/r-ryantm/nixpkgs/archive/da05627e4ea6b43c5eaca7d7d45a59c1ca265a06.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/da05627e4ea6b43c5eaca7d7d45a59c1ca265a06#civo

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dvdi3qhjnkh30gk62qpfz3af9skvwpab-civo-1.0.89
ls -la /nix/store/dvdi3qhjnkh30gk62qpfz3af9skvwpab-civo-1.0.89/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

civo

Maintainer pings
cc @techknowlogick for testing.

Add a :+1: reaction to pull requests you find important.

@ofborg eval

@NickCao could you trigger ofborg again?

@ofborg eval
(I think everyone could use the ofborg command, not restricted to committers?)

@ofborg eval

@NickCao ah thanks. Good to know :)


ReferenceError: useAuth is not defined

Environment

Operating System: Darwin
Node Version:     v18.14.2
Nuxt Version:     3.5.3
Nitro Version:    2.4.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      app, build, runtimeConfig, modules, css, plugins, postcss, vite, auth, experimental
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
No response
Describe the bug
In a component i use const { data } = useAuth(); everything works as expected but when writing tests with vitest for this component I get the following error while running the test
ReferenceError: useAuth is not defined
 ❯ setup components/Navigation.vue:43:18

Eventually this seem to be a problem with imports or something.
In my test I mock useAuth like this:
mocks: {
        useAuth: () => ({
          isAuthenticated: true,
          user: {
            name: 'John Doe',
          },
        }),
      },

but I just can't get pass this ReferenceError because the error is not in the test but in the component
anyone any idea on how to fix this?
Additional context
No response
Logs
No response

Mocking composables in nuxt is a big pain. In any case, you cannot mock useAuth like this. mocks in vue-test-utils is to mock entities in the vue component instance. For imported functions you will need to use vi.mock or jest.mock. If you are using vitest, you could also have a look at https://github.com/danielroe/nuxt-vitest#mocknuxtimport. However, I have had little success with this, probably due to https://github.com/danielroe/nuxt-vitest/issues/169, so even though have put some more info on this problem here, I haven't solved it myself yet either.

Here is a document about useAuth, you can try, if it helps you！！！
https://useauth.dev/docs/getting-started/

Hi @glennovische Did the response by @dschreij help you? I don't have much to add to what was said, so I will close this issue for now. If you need furthur help, feel free to respond and we can continue the conversation!

@dschreij Thanks for responding. In the other thread did you test mockNuxtImport?? If so I will give it a try aswell.

Hi @glennovische I did use mockNuxtImport which did the trick for me. I didn't have much success with vi.mock in a Nuxt environment so can't really tell if it is possible to get things to work with that too. I think mockNuxtImport does some 'magic' stuff with hoisting and auto imports, so it may be preferable to use it in any case if it is all the same to you.


waylogout: init at unstable-2023-06-09

Description of changes
Closes #242499.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] 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

[ ] 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
[ ] 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 242774 run on x86_64-linux 1

1 package built:

waylogout


[FLOC 3546] Explain why the plugin install instructions are optional

Fixes 3546
Explains why the plugin is an optional install step in the node services install doc, linking back to the introduction document on the plugin.

Thanks.  I'm not sure this quite does the job as-is.  What do you think?

Thanks @exarkun
I agree - I wasn't totally happy with using words like optional or essential. I've amended the text to highlight the scenario where users would want to install the plugin without having to say that anything is optional or essential - hopefully this is a bit clearer.

How about top-level, self contained documentation for Flocker Docker Plugin?

The Flocker Docker Plugin
Introduction
Installation
Configuration
Usage examples

I know I'm not supposed to say stuff like that in review so put this back up for review if you disagree.
Sorry to be a pain but it seems to me that this still doesn't address @Stephenitis 's original issue and the plugin documentation does seem...difficult to navigate.

Thanks @wallrj
I think it depends on whether we see the plugin as a part of Flocker, or a separate product. I'll get a second opinion.


edited local_task_queue.py to allow for async awaiting

Thanks! closing as changes made as of tag 0.2.7


feat: replaced the require attribute with aria-required

Content designers noticed that when a user hovers on a textarea in Chrome, the default 'Please fill in this field' box appears and it doesn't match our custom error message for that fireld.

This is coming from fields using the required attribute. For the public website forms we use the novalidate attribute and custom error messages so as far as I'm aware, required is useful for screen readers atm (please correct me if that's not the only use case here).
I've checked other services, e.g:

NHS login https://access.login.nhs.uk/enter-email
GOV register to vote https://www.gov.uk/register-to-vote

Those services don't use required for mandatory fields. NHS uses aria-required which I think would be useful to add in our case. This is to replicate the current screen reader behaviour.
Another question is whether we need aria-required at all as we sometimes have mixed forms - i.e text inputs and radio buttons being part of the same form. Radio groups don't have 'required' pronounced even when they are mandatory. If we remove aria-required completely my only concern is whether the assumption that the users will know which fields are mandatory because they don't have (optional) is enough.

This makes sense conceptually, but possibly isn't the cause of that message showing. As we provide our own error handling we should be adding the novalidate attribute to forms. We should do both though.

Do you think it's work checking this with Joanna? I'd be interested to know what problems the prompt is causing, apart from it having different wording.

We follow a similar rationale to https://design-system.service.gov.uk/patterns/validation/#turn-off-html5-validation, plus the fact that we can translate our error messages too. Would be good to check the pattern though.

This makes sense conceptually, but possibly isn't the cause of that message showing. As we provide our own error handling we should be adding the novalidate attribute to forms. We should do both though.

Do you think it's work checking this with Joanna? I'd be interested to know what problems the prompt is causing, apart from it having different wording.

We follow a similar rationale to https://design-system.service.gov.uk/patterns/validation/#turn-off-html5-validation, plus the fact that we can translate our error messages too. Would be good to check the pattern though.

We have novalidate and the required attribute already - this isn't really to do with validating a user's input tho, as the prompt shows on hover over a textarea with the required attr - before a user has even submitted the form. Looks like a thing Chrome has just decided to implement.
For me whether or not this is worth doing hinges on two things:

how big of a problem is it for content designers? It's not really a validation issue - our validation messages will still appear in the summary and by the question.
whether replacing required with aria-required has any other consequences we don't know about - this is what I think we should check with Joanna at AbilityNet.

We have novalidate and the required attribute already

Ooops! You are totally right I misread! Thought we only had them on the react forms in public website. Carry on 👍

Update: Asked about replacing the required attribute with aria-required in today's AbilityNet session. As we are not using the default required validations (i.e preventing the form from submitting and displaying a tooltip), there aren't further accessibility implications and aria-required will be fine.


perform environment calculation in Python

The first commit adds .dsv files describing the intended environment change in a machine readable form (similar to ament/ament_cmake#187).
The second commit duplicate the file prefix_util.py into prefix_util.py.em before changing its content.
The third commit extends the shell extension point to generate a shell-specific version of the prefix_util.py module. The module will utilize the information from .dsv files to compute the desired environment changes within Python.
The fourth commit updates the sh shell extension to use the .dsv files

Other shell extensions still need to be updated...

Codecov Report

Merging #209 into master will decrease coverage by 0.43%.
The diff coverage is 51.11%.

@@            Coverage Diff             @@
##           master     #209      +/-   ##
==========================================
- Coverage   77.85%   77.42%   -0.44%
==========================================
  Files          53       54       +1
  Lines        3071     3109      +38
  Branches      512      512
==========================================
+ Hits         2391     2407      +16
- Misses        652      675      +23
+ Partials       28       27       -1

Impacted Files
Coverage Δ

colcon_core/shell/dsv.py
0% <0%> (ø)

colcon_core/shell/sh.py
100% <100%> (ø)
:arrow_up:

colcon_core/shell/bat.py
100% <100%> (ø)
:arrow_up:

colcon_core/shell/__init__.py
98.37% <87.5%> (-1.06%)
:arrow_down:

colcon_core/shell/template/__init__.py
100% <0%> (+2.43%)
:arrow_up:

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 8aa8127...8b6b119. Read the comment docs.


Parser from Config toml to Graph-QL/Modular

User story

As a User
I want to have an abstract config file
So that it can be translated to any Query languages like Graph-QL

Acceptance criteria

Test that are passed for atleast 5 config files
The module should be open for different extensions

Definition of done
  1. Code review from peers has been completed and passed
    a. Code is fully implemented and commented
    b. Adheres to coding guidelines
    c. Commits are not squashed
    d. Code is checked into the repository
2. Unit tests/ Integration test have been written and are passing
3. Cleanliness of the Code
4. No critical bugs are open
5. Documentation has been updated
6. Code has been included into the release (candidate)
7. Code Coverage:
     60 % - for Features
     70 % - for Sprint Release
     80 % - for Product Release

Unit tests have to be written at a later point of the project


ability to set up one-way tls with the wizard

Thanks, @Skyllarr ! Done.

Hi @fjuma, can you please review the PR?
thanks,
ashwin

@theashiot Should we get this merged as-is - as an incubator project we can then decide to create issues and work incrementally as time allows / based on other priorites?

@theashiot Should we get this merged as-is - as an incubator project we can then decide to create issues and work incrementally as time allows / based on other priorites?

That sounds like a good plan, thanks!


[improve] [doc] A central page about Pulsar message sub throttling

Relate issue #18506
Motivation
The current documentation about Pulsar message sub-throttling is insufficient.
Modifications
Add a central page about Pulsar message sub-throttling, and the doc of Pulsar message pub-throttling will be provided by another PR
Documentation

[x] doc
[ ] doc-required
[ ] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

no needed.

@poorbarcode thanks for adding the docs! I will look into the content and get back to you.

Closing...
The source of docs has been moved to the site repo. Please resubmit the patch there. And IIRC cookbook series is to be replaced with tutorials series. FYI @momo-jun @DaveDuggins.

@poorbarcode @heesung-sn I think the existing draft has many improvement possibilities, so I put it into the Google doc to facilitate the collaborative effort on the review and Q&A more efficiently. I migrated Heesung's comments and also went through the content with a few IA changes and edits based on my understanding. Can you pls continue the review in the Google docs? After we get a more mature version, you can submit a PR to the pulsar-site repo to have more people review it through GitHub.


Add persistent state directory to python agents

Changes:

Added new Python AgentContext class to the API
An agent can access it by defining

from langstream import SimpleRecord, Processor, AgentContext
import logging, os

class Exclamation(Processor):
    def init(self, config, context: AgentContext):
        self.context = context
    def process(self, record):
        directory = self.context.get_persistent_state_directory()

if the agent declares init with only argument, it works as before

Added end to end test

LGTM. Just some minor comments.
Congrats on the Python PR !

LGTM. Just some minor comments. Congrats on the Python PR !

Thanks I addressed your comments


ci: reduce bootstrap time

i changed deployment method in the recent change to systest. in my tests that change allows to expect shorter bootstrap time

bors try

bors try

bors try

bors merge


fix: phone number format not recognised on verification screen

Description
Context: https://valora-app.slack.com/archives/C029MMTD529/p1677719177909599, https://valora-app.slack.com/archives/C025V1D6F3J/p1678194369634539?thread_ts=1677719863.631589&cid=C025V1D6F3J
We use the value in the phone number text input to do the phone number validation, but as soon as the validation succeeds, we add the country code to the text input. Subsequent validations are performed with a phone number that includes the country code, which always fails. This PR fixes this issue.
Test plan
This phone number can now be validated:
https://user-images.githubusercontent.com/20150449/223467814-ae6beaeb-6ccd-4dfb-8ca7-77c9fcb376e9.mp4
We can also backspace on a validated number:
https://user-images.githubusercontent.com/20150449/223469228-5e1232c3-de01-49d8-9cee-4020ad71a903.mp4
Related issues

Fixes RET-553, ENG-29, RET-646

Backwards compatibility
Y

If numbers for a given country can have different lengths, won't the "ghost text" we show be incorrect for some users?

i'm not sure that i see the problem - we just want to give an example of what the phone number should look like, if the example number is valid for the user's country (i.e. doesn't look unfamiliar to them) but is a different length to the user's actual number, that still seems fine?

This may warrant a unit test. Sorry I know it's extra annoying when the component doesn't have tests in the first place. We can follow up separately.

i agree, usually i'd add the tests before i merge but i haven't thought of a good way to test this yet. the bug only occurs while the user is typing, which is not possible (i think?) to reproduce with react native testing library since we only have fireEvent.changeText which dispatches a native event with the whole text. so a unit test isn't really helpful here :/ we could do it with an e2e test, but that also feels weird....although i lean towards having an e2e test


python3Packages.flower: disable failing tests

Motivation for this change
Fix build (https://hydra.nixos.org/build/157272835)
ZHF: #144627
Things done

Built on platform(s)

[x] x86_64-linux
[ ] 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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 144904 run on x86_64-linux 1

2 packages built:

python38Packages.flower
python39Packages.flower

Result of nixpkgs-review pr 144904 run on x86_64-linux 1

2 packages built:

python38Packages.flower
python39Packages.flower


Upgraded from RC2 to .NET Core 1.0

The version was bumped to 6.0.0-beta10 in all NuGet package projects.
There is one test failing, but I believe it is due to swagger.io/validator not being spec compliant.
The test is the following one, which is tested to return an empty JSON file, but instead returns an error complaining that definitions/Cart/default is not of type string...which it shouldn't. It should be a valid cart, which it is.
Albeit I don't quite understand why its Id property is set to a string value in AddCartDefault.cs although it is of type integer. That should have this test fail if the validator is truly spec compliant. Not that it matters for the test result today - it fails with the same error if I set it to -1.
[Theory]
[InlineData(typeof(Basic.Startup), "/swagger/v1/swagger.json")]
public async Task SwaggerRequest_ReturnsValidSwaggerJson(...)

The original PR for this - #112 has already been merged in.


Replace ask with sending request to make Mesos calls.

Summary:
The last MWT showed AskTimeoutExceptions when Mesos would not respond in time during
a Mesos master failover. This timeout was the result of an HTTP request timing out and is hard
to debug. It also leaves dead letters.
This change sends a promise instead. The promise is completed with the HTTP request result
which can include a timeout. It should ease debugging.

Exit status 1 when running "Unused imports / scalafix"

Details

[info] Running scalafix on 14 Scala sources
[info] Running scalafix on 2 Scala sources
--- /home/travis/build/mesosphere/usi/mesos-client/src/test/scala/com/mesosphere/mesos/client/SessionActorTest.scala
+++ <expected fix>
@@ -5,11 +5,9 @@
 import akka.http.scaladsl.model._
 import akka.http.scaladsl.model.headers.{BasicHttpCredentials, HttpCredentials}
 import akka.http.scaladsl.unmarshalling.Unmarshal
-import akka.util.Timeout
 import com.mesosphere.utils.AkkaUnitTest
import scala.concurrent.{Future, Promise}
-import scala.concurrent.duration._
class SessionActorTest extends AkkaUnitTest {
[error] (mesos-client / Test / scalafix) scalafix.sbt.ScalafixFailed: TestError
[error] Total time: 4 s, completed Jun 9, 2020 12:00:50 PM
restoring stty: 500:5:bf:8a3b:3:1c:7f:15:4:0:1:0:11:13:1a:0:12:f:17:16:0:0:0:0:0:0:0:0:0:0:0:0:0:0:0:0


local_server

Dear Authors,
I have tried the script for setting up run in local_server and it seems that it did not work successfully.
I have connect the colab saprothub to the local server and, I try click on the "1.1 install saprot", error message appears as below. Do you know what could be the problem? Thank you.
ModuleNotFoundError                       Traceback (most recent call last)
 in <cell line: 114>()
112 from transformers.models.esm.openfold_utils.feats import atom14_to_atom37
113 from string import ascii_uppercase,ascii_lowercase
--> 114 from saprot.utils.mpr import MultipleProcessRunnerSimplifier
115 from saprot.data.parse import get_chain_ids
116 from saprot.scripts.training import my_load_model
ModuleNotFoundError: No module named 'saprot'
Best wishes,
JQ

Hi JQ,
Have you installed the environment successfully before running the script run.sh?

Hi JQ,
Have you installed the environment successfully before running the script run.sh?

I run the $bash install.sh, and it runs normally.
I then have to go to the SaprotHub-main directory to run $bash local_server/run.sh
The version is not the latest. Updating...
HEAD is now at 23a69ad No public description
Already up to date.
ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'
WARNING: Skipping saprot as it is not installed.

| | | |_ __  | | | | ___
| || | ' /  / _ |  / -)
_/| ./_,_,|____|
|_|
Read the migration plan to Notebook 7 to learn about the new features and the actions to take if you are using extensions.
https://jupyter-notebook.readthedocs.io/en/latest/migrate_to_notebook7.html
Please note that updating to Notebook 7 might break some of your extensions.
[I 15:08:37.328 NotebookApp] Loading IPython parallel extension
[I 15:08:37.331 NotebookApp] Serving notebooks from local directory: /home/dc/.cache/SaprotHub/colabtools
[I 15:08:37.331 NotebookApp] Jupyter Notebook 6.5.5 is running at:
[I 15:08:37.331 NotebookApp] http://localhost:8888/?token=175d42a
[I 15:08:37.331 NotebookApp]  or http://<IP_ADDRESS>:8888/?token=175d42a
[I 15:08:37.331 NotebookApp] Use Control-C to stop this server and shut down all kernels (twice to skip confirmation).
[C 15:08:37.376 NotebookApp]
To access the notebook, open this file in a browser:
    file:///home/dc/.local/share/jupyter/runtime/nbserver-90025-open.html
Or copy and paste one of these URLs:
    http://localhost:8888/?token=175d42a
 or http://<IP_ADDRESS>:8888/?token=175d42a

It complains that saprot is not installed though, is that because it can find "requirements.txt", but even I run the command $bash run.sh, it stills complains cannot find the "requirements.txt". But this file is in the folder (local_server). How to fix the problem?

Maybe you should first go into the local_server directory? The system assumes you are in this directory and finds files using relative path.

when I run $bash run.sh  in local_server, it complains the same thing.

The version is not the latest. Updating...
HEAD is now at 23a69ad No public description
Already up to date.
ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'
WARNING: Skipping saprot as it is not installed.

| | | |_ __  | | | | ___
| || | ' /  / _ |  / -)
_/| ./_,_,|____|
|_|
Read the migration plan to Notebook 7 to learn about the new features and the actions to take if you are using extensions.
https://jupyter-notebook.readthedocs.io/en/latest/migrate_to_notebook7.html
Please note that updating to Notebook 7 might break some of your extensions.
[I 15:25:26.587 NotebookApp] Loading IPython parallel extension
[I 15:25:26.592 NotebookApp] Serving notebooks from local directory: /home/dc/.cache/SaprotHub
[I 15:25:26.592 NotebookApp] Jupyter Notebook 6.5.5 is running at:
[I 15:25:26.592 NotebookApp] http://jqlab:12315/?token=...
[I 15:25:26.592 NotebookApp]  or http://<IP_ADDRESS>:12315/?token=...
[I 15:25:26.592 NotebookApp] Use Control-C to stop this server and shut down all kernels (twice to skip confirmation).

And moreover, no jupyternote book is launched when I run the $bash run.sh in the local_server folder.
JQ

Can you run the installation from scratch in the local_server directory? Also, it looks like the jupyter notebook runs normally as expected.

Ok, I have tried to install from scratch.
I have first modify the install.sh to make a directory for cache in /home/dc/.cache/SaprotHub instead of /root/.cache/SaprotHub, the modified install.sh script is as below
#!/bin/bash
conda create -n SaprotHub python=3.10 --yes
source activate SaprotHub
mkdir -p /home/dc/.cache/SaprotHub
Install packages
git clone https://github.com/googlecolab/colabtools.git /home/dc/.cache/SaprotHub/colabtools
pip install /home/dc/.cache/SaprotHub/colabtools/
pip install -r ./requirements.txt
Overwrite files for properly calling colab
colab_dir=$CONDA_PREFIX/lib/python3.10/site-packages/google/colab
cp ./data_table.py $colab_dir/data_table.py
cp ./_reprs.py $colab_dir/_reprs.py
This step work normally.
Next, I also modify the run.sh script to change "/root/.cache" to "/home/dc/.cache" accordingly,  then
$bash run.sh
in the local_server directory, and it comes out with error message
[C 09:19:40.456 NotebookApp] Bad config encountered during initialization: No such notebook dir: ''/root/.cache/SaprotHub''

#!/bin/bash
source activate SaprotHub
Checkout the version in the cache
ori_dir=$(pwd)
if [ -d "/home/dc/.cache/SaprotHub/SaprotHub" ];then
cd /home/dc/.cache/SaprotHub/SaprotHub
local=$(git rev-parse HEAD)
remote=$(git ls-remote https://github.com/westlake-repl/SaprotHub.git | grep HEAD)
remote=(${remote//,/ }[0])
if [ "$local" != "$remote" ]; then
echo "The version is not the latest. Updating..."
 As if I don't change the /root to /home/dc directory, in the first step $bash install.sh, it will complain about the root access and won't install the environment correctly.
But I failed in the $bash run.sh step.
Best wishes,
JQ

Hi,
You also have to change the cache directory in jupyter_notebook_config.py and then everything should be OK。

Ok, "$bash run.sh"  works, but it did not provide the local server and connectable address. the message it gives as below

Read the migration plan to Notebook 7 to learn about the new features and the actions to take if you are using extensions.
https://jupyter-notebook.readthedocs.io/en/latest/migrate_to_notebook7.html
Please note that updating to Notebook 7 might break some of your extensions.
[I 11:30:04.460 NotebookApp] Loading IPython parallel extension
[I 11:30:04.462 NotebookApp] Serving notebooks from local directory: /home/dc/.cache/SaprotHub
[I 11:30:04.462 NotebookApp] Jupyter Notebook 6.5.5 is running at:
[I 11:30:04.462 NotebookApp] http://jqlab:12315/?token=...
[I 11:30:04.462 NotebookApp]  or http://<IP_ADDRESS>:12315/?token=...
[I 11:30:04.462 NotebookApp] Use Control-C to stop this server and shut down all kernels (twice to skip confirmation).
may I know how to deal with it?
Thanks.
JQ

And then you can follow the instructions to connect to you local server as here:

I have tried and the connect button is disabled in this way. Please see attached picture.
I also noticed that when I typed $bash run.sh, no web browser page was launched. Is this normal?
JQ

It is the same computer that I launch the colabsaprot webpage.
JQ

Could you set the IP address to <IP_ADDRESS> to see if it works?

ok, now it works by set the IP address to <IP_ADDRESS>, but again when I clicked the run button to install saprot, it ends up with error

Installing SaProt...
Cloning into 'SaprotHub'...
remote: Enumerating objects: 1609, done.
remote: Counting objects: 100% (772/772), done.
remote: Compressing objects: 100% (372/372), done.
error: RPC failed; curl 92 HTTP/2 stream 0 was not closed cleanly: CANCEL (err 8)
error: 2740 bytes of body are still expected
fetch-pack: unexpected disconnect while reading sideband packet
fatal: early EOF
fatal: fetch-pack: invalid index-pack output
ModuleNotFoundError                       Traceback (most recent call last)
 in <cell line: 114>()
112 from transformers.models.esm.openfold_utils.feats import atom14_to_atom37
113 from string import ascii_uppercase,ascii_lowercase
--> 114 from saprot.utils.mpr import MultipleProcessRunnerSimplifier
115 from saprot.data.parse import get_chain_ids
116 from saprot.scripts.training import my_load_model
ModuleNotFoundError: No module named 'saprot'
Do you think what is the problem and how can I troubleshoot
JQ

Looks like it ran into an error when installing saprot through git commands. Could you try manually cloning the repo to see whether it works?

following the above reported problem,
in the colabsaprot site, when I clicked the run button to install saprot, my terminal showed error message as below
[W 13:27:30.683 NotebookApp] 404 OPTIONS /api/colab/build-info?token=SaprotHub (<IP_ADDRESS>) 16.920000ms referer=https://colab.research.google.com/
[W 13:27:30.685 NotebookApp] 404 GET /api/colab/build-info?token=SaprotHub (<IP_ADDRESS>) 0.720000ms referer=https://colab.research.google.com/
[I 13:27:30.725 NotebookApp] Kernel started: 75e63041-a962-4f35-8f6d-8eb3d8ed3acd, name: python3
ERROR: Could not open requirements file: [Errno 2] No such file or directory: '/home/dc/.cache/SaprotHub/SaprotHub/requirements.txt'
ERROR: Invalid requirement: '/home/dc/.cache/SaprotHub/SaprotHub': Expected package name at the start of dependency specifier
/home/dc/.cache/SaprotHub/SaprotHub
^
Hint: It looks like a path. File '/home/dc/.cache/SaprotHub/SaprotHub' does not exist.
chmod: cannot access '/home/dc/.cache/SaprotHub/SaprotHub/bin/*': No such file or directory
Can I do something to fix the problem here?
If I would like to install saprot through git command, where should I install it?
Best wishes,
JQ

You could git clone the SaprotHub repo in /home/dc/.cache/SaprotHub/ and then rerun the installation button. I think the problem is that your computer cannot successfully clone the repo so the later installation would fail.


Fold "extra_arguments" into terragrunt when using "include" to retrieve remote TF configurations

So, I've got the following setup:

/terraform.tfvars

/preview/terraform.tfvars

The contents of /terraform.tfvars is:
terragrunt = {
  lock { ... }
  remote_state { ... }
  terraform {
    extra_arguments "global" {
      arguments = [ "-var", "account_id=12345" ]
    }
  }
}

The contents of /preview/terraform.tfvars is:
terragrunt = {
  include {
    path = "${find_in_parent_folders()}"
  }
  terraform {
    source = "git::git@github.com:foo/bar.git//app?ref=v0.0.1"
  }
}

The idea being that in app I would have access to var.account_id.  This is of course not possible (currently) because the terraform source-block will override the terraform extra_arguments-block.
The solution described in https://github.com/gruntwork-io/terragrunt/issues/143 could be applied (put my configuration into a separate tfvars), but it would mean I'd need to cut-and-paste the extra_arguments blocks into /preview/terraform.tfvars (and future staging environments).

Ah, yea, that's not ideal.
I think the way we are merging keys from include is fairly coarse grained, where we simply override top-level keys: lock, remote_state, terraform. We could merge the values together, but that will leave other holes, as then there would be no way to unset a value...
I'm open to ideas here.
One thing I've been considering is a new interpolation function:
read_var_from_file(VAR, FILE)

The idea is that it returns the value of a variable VAR  read from .tfvars file FILE. It could replace most use cases for include, as you could just explicitly pull in the values you want, but also allow much finer grained control:
terragrunt = {
  lock = "${read_var_from_file("terragrunt.lock", find_in_parent_folders())}"
  remote_state = "${read_var_from_file("terragrunt.remote_state", find_in_parent_folders())}"
  terraform {
    source = "git::git@github.com:foo/bar.git//app?ref=v0.0.1"
    extra_arguments {
      global = "${read_var_from_file("terragrunt.terraform.extra_arguments", find_in_parent_folders())}"
    }
  }
}

This may also solve #132, as it will allow you to reuse values of arbitrary variables (not just Terragrunt ones) from your .tfvars files:
aws_region = "us-east-1"

terragrunt = {
  remote_state {
    backend = "s3"
    config {
      encrypt = "true"
      bucket = "my-bucket"
      key = "terraform.tfstate"
      region = "${read_var_from_file("aws_region", self())}"
    }
  }
}

This is something I would like to see as well. If we could come to an agreement on a solution, I could put some work into making a PR.

@andrewrynhard My best guess for a solution is the one in the comment above yours: a read_var_from_file helper. On a related note, in order for path lookups to work well, we should also add support for a path.module style helper as described here.
I want to work on both of these, but have been utterly buried recently, and just haven't been able to find the time, so I'd be grateful for PRs!

Great, I will take a look and see what I can get done this week.

This sounds like a very needed feature for reducing duplication. Any update on this?

Nothing on my end. I submitted a flurry of PRs recently to update Terragrunt to support Terraform 0.9 and add a few other critical features, but have not had time for this one. @andrewrynhard, have you had a chance to take a look?

I have not had a chance yet. Hopefully soon.

A deep merge of the terraform key would be really nice to solve this!

On our fork, we made a lot of enhancements including:

Support of ${var.varname}
Deep merge of terragrunt/terraform config
Support for nested configurations (more than one level of include)
Add a criteria to determine uniqueness of a configuration by using separate temporary folder for each configuration (in our case, it is based on region/env)
Include of remote configuration to easy share our configuration between our multiple teams
Importing files from external source in the current temporary folder
Pre and Post hooks that are executed during the terragrunt process
Auto-discovery of configuration through some magic with tags and security groups
More flexible logging (using go-logging)
Possibility to ignore some folder in -all operations
Clean output on -all operations

Ultimately, our team only have to include that config file in their terraform folder and the rest (discovery of environment, region, providers, S3 backend, keys, extra_args for variables, locking, etc.) is magic:
project = "projectname"

terragrunt = {
    include {
        source = "${discover("bootstrap", var.env, "default")}"
    }
}

But having a new feature is one thing. Make a good PR from it with complete documentation and tests is another thing. We are trying to find time to package our changes in complete PRs to make them available to the root project.

@jocgir Wow, that's fantastic 👍
We'd love to hear more about each of these and get many of them into terragrunt itself. Please let me know if there's anything we can do to make that easier! Of course, we should discuss each item to make sure it's a good fit for the general Terragrunt community before you do the work to submit a PR, but I suspect a lot of these will be useful for a lot of people!

I like the read_var_from_file interpolation function as a solution for the problem originally posted by @rkr-kununu
I would use it in the parent tfvars file's terragrunt.terraform.source to look up a source variable from the child tfvars file.  This would allow me to put everything in the parent tfvars file.
I also see lots of other use cases for a read_var_from_file interpolation function.
Any chance this will be implemented soon?
I also like the ideas done/proposed by @jocgir , but I would hate to hold up implementation of a simple function while waiting on that much larger work.

After investigating adding the read_var_from_file option, I found that it was going to be pretty complex since the interpolation syntax and associated regexes were going to need modifying to support functions within functions (in order to support things like ${read_var_from_file("terragrunt.lock", find_in_parent_folders())}).
I then looked at @jocgir's deep merging approach for extra_arguments, and found it to be quite simple and elegant.  (Nice work @jocgir!)  So I have created PR #235 by isolating that change, and providing documentation and unit tests.
BTW, you can "unset" an extra_arguments block by creating an empty extra_arguments block in the child with the same name as a parent.  Fairly intuitive, I think.

I believe PR #235 fixes @rkr-kununu's original request.  So this can probably be closed.
Separate individual issues could be opened for the items implemented by @jocgir

@philsttr Good point. New issues for the other items discussed in this thread are more than welcome.


Bug report

Node JS 18
Latest Version
Debian 12
When i try to play music it does not play, its because of an issue of lavalink, when update the lavalink server. because of erela.js is outdated its not an easy task, please upgrade from erela.js and use any other package
discord error

console error

you need to connect to a music lavalink-node (v3) configable through the botconfig file


Get word level token embeddings?

I want to be able to pass in a individual word and retrieve a bert-embedding for this specific word.
Instead it splits the word, and returns character level embeddings instead, how do I overcome this problem?

Do the following to get the word embedding for word "split":
test_embedding = bert_embedding("test".split())