Feature request: Ability to restart container

As of right now, there is no possible way provided to restart started containers. I would like to see a exposed method to restart a given container.
The use case for this may be specific to me (or may not be), but generally there is a need for me to change the configuration file and push for a restart after the config change. As of right now, I am trying to work it through via the use of Dockerode, which this is built on top of, but found no success.
I am willing to implement this myself for this feature, but would like to know the project maintainer's thought on this feature, considering it does not exists in testcontainer-java.
Related: https://github.com/testcontainers/testcontainers-java/issues/606

@cristianrgreco is there a straightforward way to wait for a particular log message after restarting a process in a container? I have this problem too and that's what stops me from doing that - if I create a new container I get Wait.forLogMessage but I have to wait for my container to start, which takes a long time because it needs to build everything from scratch due to being set up for hot reload. Meanwhile, if I restart the process I can't wait for the container to actually be ready to recieve messages particularly easily.

Hi @cruxicheiros, yes as per the README you can stream the logs of a running container:
const stream = await container.logs();
stream
    .on("data", line => console.log(line))
    .on("err", line => console.error(line))
    .on("end", () => console.log("Stream closed"));

I guess you want something like:
await new Promise(resolve => {
  const stream = await container.logs();
  stream.on("data", line => {
    if (line === "PROCESS READY") {
      resolve();
    }
  })
});

I would really like this feature as well, I want to test kafka auto-healing behavior, but as of right now when I do a stop() and start() I get new mappings. I want exactly the same host:port to test this behaviour. Mapping the ports to something I explicitly want is also not allowed/recommended anymore.
What is the correct way of testing such a feature?

Released in 8.12.0, thanks @cwkang1998


project: update infrastructure for bower/amd usage

For LaxarJS/laxar#179 and LaxarJS/laxar#225 to fully work, we need to update the widget infrastructure.
The widget.json should use the internal name: ax-command-bar-widget.
The bower.json should use the fully qualified name: laxar-command-bar-widget.
This makes widgets work exactly like controls.

Implemented on master (v2.2.0)


creae_ngrams missing 2 required position arguments

Hi guys,
Im getting the following error when training a pdf document. Any idea what this might be?
Thanks, Roy

Hi,
same problem for me.

Hi,
same problem for me.

Should be fixed in the latest commit

Should be fixed in the latest commit


About use GPU id

Hello,when I training ,I meet  two problems:
1.Whether I train the model or test , must use gpu_id 0,“ model= nn.DataParallel（model,device_ids=[0,1]）”Otherwise cannot runing .
2. I training using two gups, but my batchsize  only set 2,cannot set 4 or 8,if so ,always reminding me out of memory .Does someone meet this?

Check the model of your GPU or use nvidia-smi command, to get the memory size of your GPU, I have 3 Tesla K80 GPUs, setting batch_size=9, each GPU consumes about 10.3-1.4 GB memory

I have the same error ,have you solved it?


set_should_verify_ca equivalent in new api

There used to be set_should_verify_ca method on MqttOptions type, along with other methods that are all gone, which is fine but I'm not sure what is the equivalent of set_should_verify_ca method in new version. I can find most other methods have an equivalent but not this one.
There were self signed certs used for tests and lower environments in my case and I'm having to upgrade to newer version of rumqtt. I can see I need to use the new ConnectionMethod::Tls but there is no way of setting it to not verify the CA. Thanks

Rustls is the default now and I don't think it supports disabling ca verification.

@tekjar Seems you are correct, however this would be nice for development.
https://docs.rs/rustls/0.15.2/rustls/#rustls-takes-care-of-server-certificate-verification

@tekjar @TotalKrill - I haven't looked into rustls api in great detail, but possibly this is something that's not exposed in ring? I'll have a look to see what are the alternatives. This is a super handy feature for local dev/CI/lower environments which probably is useful for other servers/clients as well not just for MQTT broker/client.


ARROW-9405: [R] Switch to cpp11

This is very much work in progress. I've started by tackling code generation, as we cannot just use cpp11::cpp_register() as is, because e.g. of the #if defined(ARROW_R_WITH_ARROW) thing.
Once all code generation goes through cpp11 instead of Rcpp, then I'll remove uses of Rcpp classes, and then eventually we can remove the dependency.

@romainfrancois please rebase, it has some conflicts with the master.

Since this has a bunch of merge commits it will have to be git merge --squash'd to clean things up, let me know if I can help

@romainfrancois I squashed, be sure to git fetch origin && git reset --hard origin/cpp11

I somehow lost the ability to lint locally:
r % CLANG_FORMAT=$(which clang-format) ./lint.sh
Traceback (most recent call last):
  File "/Users/romainfrancois/git/apache/arrow/r/../cpp/build-support/run_clang_format.py", line 121, in <module>
    for filename, diff in pool.starmap(_check_one_file, checker_args):
AttributeError: 'Pool' object has no attribute 'starmap'

@github-actions autotune everything

I'm trying to debug what's wrong with the python tests that I previously silenced in this PR:
That looks fine:
library(reticulate)
#> Warning: package 'reticulate' was built under R version 4.0.2
pa <- reticulate::import("pyarrow")
py <- pa$array(c(1, 2, 3))
str(py)
#> [
#>   1,
#>   2,
#>   3
#> ]
class(py)
#> [1] "pyarrow.lib.DoubleArray"        "pyarrow.lib.FloatingPointArray"
#> [3] "pyarrow.lib.NumericArray"       "pyarrow.lib.Array"
#> [5] "pyarrow.lib._PandasConvertible" "python.builtin.object"

But then if I also load arrow I'm getting a segfault on pa$array() call:
library(reticulate)
library(arrow)
pa <- reticulate::import("pyarrow")
py <- pa$array(c(1, 2, 3))

The error seems to happen on this call:
x$`_export_to_c`(array_ptr, schema_ptr)

And I can confirm that this works on master locally:
library(reticulate)
#> Warning: package 'reticulate' was built under R version 4.0.2
library(arrow)
#>
#> Attaching package: 'arrow'
#> The following object is masked from 'package:utils':
#>
#>     timestamp
pa <- reticulate::import("pyarrow")
py <- pa$array(c(1, 2, 3))
py
#> Array
#> <double>
#> [
#>   1,
#>   2,
#>   3
#> ]

Created on 2020-08-19 by the reprex package (v<IP_ADDRESS>01)
More investigation needed.

Getting these details:

Looks like this is related to how cpp11 treats uintptr_t:
uintptr_t allocate_arrow_schema();
extern "C" SEXP _arrow_allocate_arrow_schema(){
BEGIN_CPP11
	return cpp11::as_sexp(allocate_arrow_schema());
END_CPP11
}

Rcpp makes a numeric and cpp11 makes a int:
library(cpp11)
library(Rcpp)

str(Rcpp::evalCpp("reinterpret_cast<uintptr_t>(new int)"))
#>  num 1.41e+14
str(cpp11::cpp_eval("reinterpret_cast<uintptr_t>(new int)"))
#>  int -1631515312

Created on 2020-08-20 by the reprex package (v<IP_ADDRESS>01)

Rcpp has these :
/* long are represented as numeric vectors which allows more precision
   to be preserved than int */
template<> struct r_sexptype_traits<long>{ enum{ rtype = REALSXP } ; } ;
template<> struct r_sexptype_traits<unsigned long>{ enum{ rtype = REALSXP } ; } ;

so uintptr_t aka unsigned long (at least locally for me) get converted to a double
I believe both are wrong (in terms of what this tries to do), but cpp11 is wronger.

I think the last commit (https://github.com/apache/arrow/pull/7819/commits/23b179e7d69d6042821a41878b86e37b18cdeb44) restore how things were before switching to cpp11 wrt how uintptr_t goes back and forth.
This uses a double as a host for a uintptr_t which IIUC is only faithful to 53 bits.
This should really use an external pointer but then I have no idea how python would go about extracting the actual pointer from the external pointer SEXP.
This seems to work, at least according to the tests, but I really don't know if the uintptr_t hosted in a double is fine with x$_export_to_c` expectations ?
cc @jjallaire @fsaintjacques @nealrichardson
py_to_r.pyarrow.lib.Array <- function(x, ...) {
  schema_ptr <- allocate_arrow_schema()
  array_ptr <- allocate_arrow_array()
  on.exit({
    delete_arrow_schema(schema_ptr)
    delete_arrow_array(array_ptr)
  })

  x$`_export_to_c`(array_ptr, schema_ptr)
  Array$create(ImportArray(array_ptr, schema_ptr))
}

We get a double for both schema_ptr and array_ptr Is that ok ?

I'm assuming there will be follow up, but it looks like the Rcpp -> cpp11 is complete. 🎉

@github-actions crossbow submit -g r


More features for bank organization.

What:

Bank organization grinds my gears.

Solution:

Implement a way to add more tabs. I'm thinking an arrow button next to the last tab to view more tabs.
Implement a way to have a blank space. I say this because it would be nice to have rows dedicated to (a) specific item(s).

We have bank tag tabs
There are placeholders in vanilla

That's cool and I didn't know about that, but I still don't think that's the same idea.
I attached a screenshot of my bank with a square around a blank space, and an arrow where I'd want to put it. Hopefully this helps you visualize what I'm talking about more.

You only have blank spaces at the end of groups of things in the bank, I want to be able to put them in the middle.

thats what fillers can do

just move the bank filler where you want them to go, you can drag them like an item.

That's because the bank doesn't have the concept of absolute positioning, it's just a collection of lists. This is not something we really want to change.
As others have said, Fillers can be used to add blank spots to any section you want.
Alternatively, there are situations in which you can make vertical groups of items using the leftover spaces on rows.

Fillers can be used to add blank spots to any section you want.

A downside of this is that it counts against your bank space.
I know that, at least for (late game) ironmen, extra bank space is pretty non-existent.


Fix for CHANGELOG formatting

CATEGORY
Choose one

[ ] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[X] Documentation

SUMMARY
<Select> was being parsed as HTML, and goofing all subsequent markdown in the file.

Codecov Report

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

@@          Coverage Diff           @@
##           master   #8187   +/-   ##
======================================
  Coverage    66.2%   66.2%
======================================
  Files         479     479
  Lines       22932   22932
  Branches     2524    2524
======================================
  Hits        15182   15182
  Misses       7616    7616
  Partials      134     134

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 15e6238...a205765. Read the comment docs.

This might not work because when we autogen the changelog again, it might just reset the formatting again. Can you test autogenning again to see if that happens?

When generating the CHANGELOG I tend to only do it for the version range that I need and overwrite that, so that should be ok.
I'm guessing at some point it would get fixed in github-changes, I'll upgrade my local one next time I generate it


Updated Robot.java with Gamepad Controls

It's vague because I don't have the details of the issue yet. We only got Li'l Mac working at the end of our last meeting, which is why we don't have the specifics of what needs to be fixed. But that's not the point anyway - I just wanted the gamepad controls to be in one place; the elevator issue is a a separate problem.


Please tell me how to set time out for request?

@duyanpham This is an Alamofire question, not an RxAlamofire question.

I used all the ways to set time out of Alamofire, but It 's not work.

Does it work in Alamofire itself, but not in RxAlamofire ?
If it doesn't work well in Alamofire you should open an issue in there repo, as this is only a Reactive wrapper for it.


Enhancement: Ability to override coin metadata

Description
Currently there is no way to update coin metadata link blockinfo, txinfo, addressinfo and etc. It would be a great enhancement if we can override this under pool coin config.

Fixed in Release 34.


Jquery calls

I've attempted to call the decode function from a jquery which I'm using to zoom, and I can't seem to figure out the proper way to formate the options to pass to the function. Can you please advise?

Could you provide an example source code? Are you getting any errors?

Hi, I gave up, but what I was trying was calling decodePCX() function in jquery and it seems that I don't know how to convert the options param to a properly formatted canvas object.  I'm using a file loader which calls a loadFromSelector() as follows:
function loadFromSelector() {
    decodePCX({ canvas: document.getElementById('canvas') });
    if(typeof(ctx) == 'undefined') {
        ctx = document.getElementById('canvas').getContext("2d");
    }
}

function decodePCX(options) {
    var canvas = options.canvas,
        context = canvas.getContext('2d');
    var loadedfile =
        initBrowserFileReader.loadFile(document.getElementById("gfile").files[0]).then((buffer) => {
            var pcx = new PCX(buffer);
            var pcxData = pcx.decode(context);
            initBrowserFileReader.render(context, pcxData);
        }).catch (function(error) {
        console.log("Error reading PCX File: ", error);
        });
}

Then, in the initBrowserFileReader render call, I do the following
   document.getElementById("sourcepic").src = ctx.canvas.toDataURL("image/png");
   sourcepic.src = document.getElementById("sourcepic").src;

This provides me with a source object to redraw the image after each zoom or pan action. I'm actually still working on that but I have some basic functionality working.
When I tried moving the loadFromSelector() function to a jquery assignment and then moved the decodePCX function into a $(document).ready function, I would get the error Uncaught ReferenceError: canvas is not defined when calling canvas.getContext("2d");, so it is not recognizing the canvas object which all other jquery code I'm calling recognizes. I am using that object for redraws of the canvas successfully.
If you have any ideas or a better approach, I'd really appreciate your advice.
Thanks again.

Do you have the full source code (with html) ? It seems like your canvas doesn't exists at the time decodePCX is called, so document.getElementById('canvas') returns null.
That would explain the error you are getting.

Well, currently the code works if I leave the functions above outside of jQuery, and since I tried calling from within jquery $(document).ready, I thought it should work since the canvas element exists. I could have made a mistake, so I'll have to try it again to see what I did wrong.

I close the issue as it's doesn't seem related to pcx-js.


Fix .data and .bss sections issue with unit tests

Migrate dummy Ram definitions to main_test.cpp in order to make sure
that it is defined for all builds.

@HuangDave you can merge this if it completes CI.


Integrate Boost release schedule into the new website

The review schedule is available in the current website here:
https://www.boost.org/development/index.html
It is very useful for planning library development around Boost releases.

You may like to comment on it further, but there is a "SCHEDULE OF EVENTS" on the homepage of https://www.preview.boost.org/ that's displaying the calendar information and release dates.

I have a page to push out that has the review schedule, I'll update/close this issue once I've gotten it out.  Thanks!

Thanks, I didn't notice it was listing future events. At first I thought it was listing past events.
I think the schedule on the current website is more visual though. I would prefer a more calendar-like view, with the ability to select the time span further into the future (beyond the current month).

I would prefer a more calendar-like view

https://www.boost.io/calendar/ shows the calendar.    Add links to this page from elsewhere on the website.

wow that's quite ugly :) but no worse than the old one.

@kennethreitz to close this issue, could you link this page ($HOME/calendar) from the homepage? My idea is that here:

we could add a link like this:

Thank you!


Please release v3.3.2 due to Ubuntu 18.04 / systemd fix.

Hello,
the not yet existing version v.3.3.2 would fix an issue at our Ubuntu 18.04 setup. The service redis-server.service cannot be enabled at systemd due a misconfiguration, which has been fixed with commit 1c5e2d67827be409da8571ca335b01db66f17aa2. Please release v.3.3.2.

Yes! Please do! :-) We also are waiting for it… Thanks a looooot!

The module has been migrated to the Vox Pupuli community group and version 4.0.0 has just been released. https://forge.puppet.com/puppet/redis

Yeah! Thanks! Works! Jippie!


Please allow creation of internet shortcut files (.url, .webloc, .desktop)

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.05.01. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.05.01

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

Description of your issue, suggested solution and other information
What?
youtube-dl should be able to create internet shortcut files in different formats. That is, instead of downloading a file, a small textfile is created -- based on a template for the desired format -- that contains the respective URL.
Why?
This is useful, e.g., when you want to binge-watch YouTube channels and want to take care yourself of what you already watched and didn't watch, and in what order, instead of relying on the abilities of the video platform, but don't want to get your disk space reduced unnecessarily.
You can also drag .url files on Windows into a player like MPC-BE to stream them, which is much more performant than in the browser! Unlike with downloaded videos, it's still easy with an internet shortcut to get access to the video comments in the browser.
After binge-watching a channel, you can share a zip file with shortcut files of the best content with others.
How?
Each major OS has its own file format for internet shortcuts. Though on macOS, while there is no system-level support for the Windows format, many apps support it, if I understood that correctly:

Windows: .url
macOS: .webloc
Linux: .desktop

Maybe, the --format switch could be extended with the identifier link or shortcut. This would default to the file format that is associated with the current platform.
To explicitly specify the file format, the following identifiers could be established: winlink, maclink, linuxlink.
Caveats for Windows: There are .url and .website files. "Based on exprimentation, I found programatically generating a .website file is non-trivial. If you don't get the parameters exactly right, it will not work properly. As of yet, I cannot find documentation describing the format. The name of the link appears to be embedded in the file." (source) So, we should stick to traditional .url shortcuts. (Firefox also creates .url files when dragging the address bar text.)
Caveats for Linux: .desktop files are described: "A Free Desktop Entry, used by Gnome and KDE, as well as Solaris (which I think uses Gnome)." (source) Therefore, I'm not sure, whether we can generalize .desktop files with linuxlink. (I'm not a Linux user.)
Alternatively or additionally, you may further specify the link identifier with the extension, or use it directly like --format mp4:

--format link[ext=url]
--format link[ext=webloc]
--format link[ext=desktop]
--format url
--format webloc
--format desktop

Proposed templates
The following placeholders are used in the templates that have to be replaced in a string-based manner, nothing fancy:

{{{url}}} -- Non-ASCII characters must be percent-encoded. E.g., http://www.%E4%B8%AD%E5%9B%BD%E6%94%BF%E5%BA%9C.%E6%94%BF%E5%8A%A1.cn/ means http://www.中国政府.政务.cn/. (Here are various shortcut files with this URL. I tested the percent-encoded URL in a handcrafted .url file on Windows 10, and it opened successfully in Firefox.)
{{{title}}} -- Only used for .desktop files. "Ubuntu...The file explorer displays the name embedded in the file." (source) Hence, it should probably be identical to the filename. Characters that are invalid in the file system should be okay according to these example files.

The file must be saved with UTF-8 encoding.
.url files

[InternetShortcut]\r\n
URL={{{url}}}\r\n

Chrome creates files of that structure.
Note: Windows internet shortcut files may also be created via COM. Here's a code example of MPC-BE for reading .url files via COM.
.webloc files

<?xml version="1.0" encoding="UTF-8"?>\n
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">\n
<plist version="1.0">\n
<dict>\n
	<key>URL</key>\n [\t for indent]
	<string>{{{url}}}</string>\n [\t for indent]
</dict>\n
</plist>\n

.desktop files

[Desktop Entry]\n
Encoding=UTF-8\n
Name=Link to {{{title}}}\n
Type=Link\n
URL={{{url}}}\n
Icon=text-html\n

Further information and things to consider

There's a Web Shortcut Utility Suite, that just has libraries for Java and Perl. However, the developer writes:

During development I collected shortcuts from various operating systems (both to study and to use for unit tests). I also manually created some examples for testing purposes. I made these available in their own repository with the hope that they will be useful to other developers. [link]

Windows is caching the URL of a .url file based on its filename or file path. This may mislead you when testing, or, possibly, a user when they run youtube-dl multiple times with simple output filenames.
It's important, that the --download-archive switch for continuing watching a channel works with internet shortcut files, also, even when previously created shortcut files were deleted.

This simply won't work: none of these shortcut formats apparently support passing HTTP headers. Needless to say it's extremely specific and out of scope.

@dstftw:

none of these shortcut formats apparently support passing HTTP headers.

What have HTTP headers got to do with this matter?
Imagine, I have a list of videos I want to watch, like: https://www.youtube.com/user/sxephil/videos. This list has videos with URLs of the form https://www.youtube.com/watch?v=4_jkeWgtZ18, which are the first thing youtube-dl encounters before trying to go deeper. These URLs must be written into the internet shortcut files, one of these URLs per file. MPC-BE, e.g., understands them. You can also double-click one of those and a browser tab will open.
Example .url file:
[InternetShortcut]
URL=https://www.youtube.com/watch?v=4_jkeWgtZ18

I think, if more people had and knew these possibilities for binge-watching a channel and staying up-to-date with it or revisit the channel later for a further binge-watch session, the function would get its valid share of usage.
And what else is youtube-dl's goal but to enable people to get access to their videos under other circumstances than what the website provides for, even if it's "just" by acting as a means of checking what you have already watched (what you already have been presented via link files). (I'm referring to --download-archive.)

Needless to say it's extremely specific and out of scope.

Would you, please, alternatively, point me to the important locations in the codebase and tell me possible things to consider, so that I can try it myself -- at least off the top of your head?

Which route must I go to fetch the webpage URLs for single videos, based on a --format like link?
Where should the file writing take place?
Where/how must I change the parsing of the --format switch?
Where should I store the templates? (compiled-in strings [central location?], resource files)

Interesting idea!
You can use youtube-dl --get-id with your channel/playlist url, and it will give you a list of video ids, each one on a line. For example, in Bash (GNU/Linux commandline):
youtube-dl --get-id $URL | while read id
do
	echo "[InternetShortcut]" > "$id.url"
	echo "URL=https://www.youtube.com/watch?v=$id" > "$id.url"
done

This will create a file named ID.url with the contents you specified in your previous reply, for each line (or ID).
If you want to have the .url filename be the same as the video title, it's possible, but a little more complicated.
We need the --get-title switch. And then it will print the video titles and ids; each id or title is on a line by its own, and every title is before the corresponding id.
You can then use that output to generate these .url files. Using Bash (again), I can do something like this:
youtube-dl --get-title --get-id $URL | while read title
do
	read id
	echo "[InternetShortcut]" > "$title.url"
	echo "URL=https://www.youtube.com/watch?v=$id" >> "$title.url"
done

The problem is that using a video title as a filename directly is unsafe; you need to handle at least the forbidden characters in filenames in Windows (/, ?, <, >, \, :, *, |, ") by stripping them or replacing them by - for example.

@noureddin
Unfortunately, we also need to update a list of link files, and also find IDs of videos that were uploaded a while ago, but not made visible just until now, so that they appear inmidst of video IDs for which link files were already created. Further, I'd like the naming scheme ####. This is a video title.url, instead of yt_video_id.url that destroys the sort order, and still be able to ignore those already encountered files when doing updates in the future (even when the link files were deleted).
Using the ecosystem of youtube-dl with its --download-archive switch is much saner!

Thanks, @rautamiekka!
Edited. :)

@h-h-h-h
I just checked, and yes, you're correct; youtube-dl doesn't put anything in the download-archive file when simulating (e.g., using the --get-* switches), and I can't find anything in the help to make it do that.
About naming the files, the way I showed to create files using the video title can be easily changed to use the --get-filename switch instead of --get-title, and now, you have all the flexibility of the output templates.
The only thing is that an id is printed before the filename.
I think the only remaining problem is recording what videos are downloaded, even if their link files are deleted. This would be easy if there were a switch in youtube-dl to make it record simulated downloads in the archive file.
I agree that using youtube-dl is much saner. But if this functionality isn't going into youtube-dl, one can make a small Python script that uses youtube-dl to get all the needed metadata (in JSON maybe), read the archive file, write the link files, and append the newly “saved” videos to the archive file.
But I noticed that youtube-dl looks in the archive file, if given, even when simulating. This means we can handle recording the saved videos, and leave filtering old videos to youtube-dl itself.
My Bash prototype, again. :)
download_archive=.archive  # a hidden file
youtube-dl --get-id --get-filename --download-archive "$download_archive" \
		-o '%(playlist_index)s. %(title)s-%(id)s.%(ext)s' $URL |
while read id
do
	read filename
	filename="${filename%.*}.url"  # change the extension to .url
	echo "Downloading $id: $filename"  # be a little verbose
	echo '[InternetShortcut]' > "$filename.url"
	echo "URL=https://www.youtube.com/watch?v=$id" >> "$filename.url"
	echo "youtube $id" >> "$download_archive"  # record in the archive,
	# in the same format youtube-dl recognizes
done

@dstftw, @phihag, @jaimeMF, @yan12125, @remitamine, @kennell
Please re-evaluate this issue.


[C++][Parquet] Expose the API to fine tune the window_bits parameter of ZLIB compression

Describe the enhancement requested
ZLIB library supports history buffers of different sizes by setting windowBits parameter, while currently in Arrow it is set as a fixed value. It makes sense that setting the window size to the maximum number will give a better performance, while it does not provide too much flexibility. As we know there would be some scenarios where the memory efficiency is very limited, or the software stack is relatively old and do not have a large memory capacity. For those cases, the user may want to set the window bits to a small number to save some memory. At least it would be much more flexible to give users such an option to choose the window size when de/compressing. If that makes sense, we would like to add such a property for the user to set the ZLIB(GZIP) window_bits, and will ensure not affecting the existing function calls. Thanks!
Component(s)
C++, Parquet

Maybe you can just the style of compression_level to pass the argument in?

Maybe you can just the style of compression_level to pass the argument in?

Hi, thanks for your quick reply! Do you mean we could add another parquet writer property (maybe can call it window_bits) just like how compression_level passed in? That's also what we are trying to do now :)

By the way, currently, arrow compression arguments are just level. Maybe we can support argument like rocksdb: https://github.com/facebook/rocksdb/blob/main/include/rocksdb/advanced_options.h#L87

By the way, currently, arrow compression arguments are just level. Maybe we can support argument like rocksdb: https://github.com/facebook/rocksdb/blob/main/include/rocksdb/advanced_options.h#L87

Yes, it will give much more flexibility! Seems like rocksdb already support window_bits as an option, and many other arguments as well. Maybe I could help to draft a PR to add the window_bits option (or any other if needed) to Arrow as a kickoff?

Personally I'm +1 for this, but please not break original interface a lot, maybe just change it firstly:
std::unique_ptr<Codec> MakeGZipCodec(int compression_level, GZipFormat::type format) {
  return std::make_unique<GZipCodec>(compression_level, format);
}

And use MakeGZipCodec in src/arrow/util/compression_internal.h firstly. Later we can extend it. Do you think this is ok?

Personally I'm +1 for this, but please not break original interface a lot, maybe just change it firstly:
std::unique_ptr<Codec> MakeGZipCodec(int compression_level, GZipFormat::type format) {
  return std::make_unique<GZipCodec>(compression_level, format);
}

And use MakeGZipCodec in src/arrow/util/compression_internal.h firstly. Later we can extend it. Do you think this is ok?

Yes we would definitely not break the original interface. Change MakeGZipCodec is also what I want to do first! The only thing I need to double confirm is that since we want to pass the argument from user level to the function MakeGZipCodec, we may need to add another property similar to compression_level in parquet writer and pass it down to the compression_internal right? Then user can set the window_bits when creating the parquet writer :)

I'm ok on this.
@wjones127 @kou What's your opinion for adding a window_bits here?

I'm OK with make window_bits customizable.
But I don't think that calling MakeGZipCodec() from cpp/parquet/ directly is a good idea.
Currently, we use Codec::Create() instead of Make*Codec() directly in cpp/parquet/.
I don't think that adding a new parameter Codec::Create() such as Codec::Create(Compression::type codec, int compression_level, int window_bits) is a good idea.
If we choose this approach, we may need to add more parameters later.
I like that we introduce a new options class such as Codec::Options but it may be too complex.
I suggest that we discuss API on<EMAIL_ADDRESS>too.

Given the variety of usage scenarios, it does be quite helpful to expose the capability and flexibility to applications to decide the codec parameter.

I'm OK with make window_bits customizable. But I don't think that calling MakeGZipCodec() from cpp/parquet/ directly is a good idea. Currently, we use Codec::Create() instead of Make*Codec() directly in cpp/parquet/.
I don't think that adding a new parameter Codec::Create() such as Codec::Create(Compression::type codec, int compression_level, int window_bits) is a good idea. If we choose this approach, we may need to add more parameters later. I like that we introduce a new options class such as Codec::Options but it may be too complex.
I suggest that we discuss API on<EMAIL_ADDRESS>too.

Thanks for your suggestion @kou! I agree that introducing a new option class is a better design and will make the extension of the parameters much easier. And it's also true that it will be more complex to do such a refactor. Do I need to initialize a discussion on<EMAIL_ADDRESS>cause I have not used that before, thanks!

Just subscribe<EMAIL_ADDRESS>and send a request here, maybe you can follow others'<EMAIL_ADDRESS>

Just subscribe<EMAIL_ADDRESS>and send a request here, maybe you can follow others'<EMAIL_ADDRESS>
Thanks! So I need to send an email to add me into the dev list right? Then start a discussion

Yes, otherwise your mail would be blocked :)

Yes, otherwise your mail would be blocked :)

Got it, thanks!

+1 for @kou's suggestion. It is not a good idea to add a specific parameter for configs that are only acceptable by a few codecs. As I have replied to the discussion in the mailing list, I assume the proposed Codec::Options is more like the java.util.Properties with careful design of the supported keys.

+1 for @kou's suggestion. It is not a good idea to add a specific parameter for configs that are only acceptable by a few codecs. As I have replied to the discussion in the mailing list, I assume the proposed Codec::Options is more like the java.util.Properties with careful design of the supported keys.

Great suggestion and code reference. I think we could do something similar for C++ part.

When building parquet::WriterProperties, there could be 2 design options: one is to pass compression parameters separately (as the current implementation, with more properties to be added) and make CodecOptions internally, for example:
 std::shared_ptr<parquet::WriterProperties> props =
          builder.compression(parquet::Compression::GZIP)
              ->compression_level(9)
              ->set_gzip_format(parquet::GZipFormat::DEFLATE)
              ->gzip_window_bits(12)
              ->build();

Another is to let the user create a CodecOption and pass the option to the builder directly:
CodecOption opt;
opt.compression_level = 9;
opt.window_bits = 12;
std::shared_ptr<parquet::WriterProperties> props =
          builder.compression(parquet::Compression::GZIP)
              ->codec_option(opt)
              ->build();

Which one do you prefer? Thanks! @kou @wgtmac @mapleFU

I'd vote for (2) and adopt CodecOption proposed by @pitrou in the mailing list.

@wgtmac If we want it in mailiist, maybe we need a std::unique_ptr<CodecOption>?

@wgtmac If we want it in mailiist, maybe we need a std::unique_ptr<CodecOption>?

It would be a std::shared_ptr<CodecOption> or we cannot copy WriterProperties any more.

The mailing list discussion: https://lists.apache.org/thread/6gzmfhpfkflfhjmy3ws4y775tfx7g2f8

I prefer the latter too.
But I want to use CodecOptions not CodecOption because other existing options class has s.
For example, EqualOptions, ArithmeticOptions and so on.
I seems that we can use std::shared_ptr<CodecOptions> for this. Or we may want to copy CodecOptions instead of sharing it by std::shared_ptr.

Thanks for all your suggestions! So I'd go with the latter one with std::shared_ptr<CodecOptions>


Update all Azure components

Description
Updates all direct Azure SDK imports to their latest version.
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test


Update cookie name

cc @Tratcher @HaoK

https://github.com/aspnet/Coherence-Signed/issues/166

:shipit:


Unable to get the status of a lamp in universal mode

Describe the bug
Sice the update to v2.6.2  I don't get a answer msg with lamp status, when I send a command to the lamp node in universal mode.
Before the update to v2.6.2, I sent a command to e.g. lamp 5 (Lamp node in universal mode) and I got a status information from lamp 5 on the output from lamp node.
Flow to Reproduce
`[{"id":"690e8418.6fa1d4","type":"debug","z":"3e8a3baf.b498ec","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"true","targetType":"full","x":1170,"y":540,"wires":[]},{"id":"7e2d74c0.9ce6a4","type":"hue-light","z":"3e8a3baf.b498ec","name":"","bridge":"1cda1aff.491725","lightid":"","colornamer":true,"skipevents":false,"x":1020,"y":540,"wires":[["690e8418.6fa1d4"]]},{"id":"17abdf1.a28b721","type":"function","z":"3e8a3baf.b498ec","name":"Inject lampnumber","func":"msg.payload = {\"brightness\":msg.payload};\nmsg.topic = \"32\";\nreturn msg;","outputs":1,"noerr":0,"x":810,"y":540,"wires":[["7e2d74c0.9ce6a4"]]},{"id":"aecd8e09.f560e8","type":"inject","z":"3e8a3baf.b498ec","name":"","topic":"","payload":"90","payloadType":"str","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":570,"y":460,"wires":[["17abdf1.a28b721"]]},{"id":"e46bf4f9.4286c","type":"inject","z":"3e8a3baf.b498ec","name":"","topic":"","payload":"50","payloadType":"str","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":570,"y":500,"wires":[["17abdf1.a28b721"]]},{"id":"cb093103.04b8d","type":"inject","z":"3e8a3baf.b498ec","name":"","topic":"","payload":"20","payloadType":"str","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":570,"y":540,"wires":[["17abdf1.a28b721"]]},{"id":"1cda1aff.491725","type":"hue-bridge","z":"","name":"Philips hue","bridge":"<IP_ADDRESS>","key":"Sl4IDYGDjVfZe5bAApaaWjMvK99LvzFLZd46gaDY","interval":"3000","disableupdates":false}]`

Expected behavior
Fix that problem in the next release.
Please complete the following information:

NodeRED version: v1.0.4
NodeJS version: v12.16.1
Device running NodeRED: Raps 3b+

Duplicate of #154 / fixed with v2.6.4


[Bug]: Static typed returns in the method "class_name"

Bug Description
There is a typing error here
Instead of return "PinconeVectorStore" it should have been return "PineconeVectorStore" (mind the misspelled e)
It should not have occurred in the first place. The class name attribute should be dynamically calculated rather than hardcoding it. This could've been done with builtin methods in python such as
dog = Animal()
dog.__class__.__name__  #returns the string "Animal"
Version
0.10.25
Steps to Reproduce
from llama_index.vector_stores.pinecone import PineconeVectorStore
print(PineconeVectorStore.class_name())

Relevant Logs/Tracbacks
No response

@saurabhmahra91 making it dynamic is not ideal. This should be a static name. This is intended to help with serializing and deserializing into proper classes. If the key can change any time, then it makes that task a lot harder

@logan-markewich Not dynamic like that.
Basically we should never have to type again the name of the class, and the class_name method should always return the class name based on its definition.
By dynamically calculated, I meant that the class_name method should return the class's actual name instead of a user-defined string that could differ from the actual class name.
I have also highlighted a typing error in the codebase. Please have a look at it to understand what I am trying to say.


to prevent horizontal scrolling

Should be fine now.

I think this needs background-position: center; as well.


无法创建镜像

In GitLab by @cuiaq123 on Jul 15, 2020, 02:21

In GitLab by @cuiaq123 on Jul 15, 2020, 02:27
closed


Notification should contain archive action

Expected behavior
Within a new mail notification I would like to move mail directly to the archive (and mark it as read).
Actual behavior
Now there are the following options (c.p. documentation) within the notification: reply, mark read, delete.
Environment
K-9 Mail version:5.403
Thanks
for K-9 with the great archive feature known from Thunderbird. :-)

It appears that this is a feature supported for wearables, but not phones: https://github.com/k9mail/k-9/blob/master/app/core/src/main/java/com/fsck/k9/notification/WearNotifications.java#L204
I am going to work on a code change to add this option to the device notifications as well.

See #3530


[Feature request]: Don't trigger break when mouse is down

Version

[X] I'm using version 1.9.0

Existing requests

[X] I've checked Existing requests

What operating system are you using?
macOS
Operating System Version
macOS Big Sur 11.6.2
Problem description
Stretchly pops up a break even when the mouse is currently down. If I'm in the middle of dragging something, odds are that Stretchly's banner covers where I'm dragging from and/or to.  If I release the mouse, I'll drop whatever I'm dragging and won't be able to see where I dropped it—perhaps hidden inside a folder—and I may not be able to Undo the operation.  If I hold the mouse down through the break, that's an ergonomic nightmare, even worse than no break at all.
Possible solution
If the mouse is down, don't trigger a break. Then go ahead and trigger a break, either immediately on mouse up, or whenever some clock cycle would ordinarily trigger Stretchly to check whether it's time for a break—either way is fine by me.  Of course, before breaking, check again to make sure the mouse isn't down again.
Compare #1007. My feature request should be easier because there's no question of what constitutes "currently" or "typing". Either the mouse is down or the mouse is up.
Alternative solutions
I try to find "safe" places to drop things I'm in the middle of dragging, but depending on what I'm doing there may not be a visible "safe" place.  Plus, it's physically and mentally fatiguing to hold the mouse down while scanning the edges of the screen (especially when Stretchly has just broken me out of my deep concentration on just one thing to suggest that I do one thing at a time!).
Additional information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I searched the web, but don't see any way how I could possibly detect that user is draging ;/

Thank you. It doesn't matter if the user is dragging or not—it only matters if the mouse is currently down.  Is it possible to detect that?  Here are my results of a quick search, but I confess I haven't read the pages I found.

https://developer.apple.com/library/archive/documentation/Cocoa/Conceptual/EventOverview/HandlingMouseEvents/HandlingMouseEvents.html
https://stackoverflow.com/questions/40886732/macos-detect-if-mouse-button-is-down-in-keydown
https://stackoverflow.com/questions/38278668/how-to-detect-right-mouse-down-and-other-mouse-down-while-left-mouse-dragged-on
https://apple.stackexchange.com/questions/142814/run-an-applescript-on-mouse-down
https://discussions.apple.com/thread/2264626
https://stackoverflow.com/questions/2166228/catching-mousedown-events-of-subviews-cocoa-osx
https://apple.stackexchange.com/questions/268529/mouse-click-event-being-fired-on-right-mouse-down-and-right-mouse-up
https://stackoverflow.com/questions/6753727/mouseentered-event-disabled-when-mousedown-nsevents-mac
https://developer.mozilla.org/en-US/docs/Web/API/GlobalEventHandlers/onmousedown
https://javascript.info/mouse-events-basics

Yes, that would work in case there is stretchly windows focused. App can't detect events if it's not active, as far as I know.

When Stretchly pops up, does it grab focus?  Maybe it could pop up its banner as usual, realize the mouse is down, and immediately go away again.  It would be a little disconcerting to see it jump up for a moment, but not really disruptive.  One could hold the mouse down for an extra instant while waiting for Stretchly to go away without much hand strain.  I'd love this feature.  If some people would hate it, I suppose it could be controlled by a preference setting.

Focus stays in the previous window, not Stretchly.

😞

As I don't see a way how to fix this, I will close the issue. Sorry :(


About using circuitnet.lef

Hello, I have the following two questions to ask you.
First of all, I want to ask if your def file is  obtained after the placement of the whole process runs?
Secondly, I want to view the placement results by importing your circuitnet.lef and def files into EDA business tools, but I found that there was an error when I first imported the circuitnet.ef file. Do you have any suggestions?
Error occurred when I imported the lef file：
Loading LEF file /mnt/hgfs/mkshare/circuitnet.lef ...
**ERROR: (IMPLF-53):	The layer 'M2' referenced in pin 'Z' in macro 'AN2_0010' is not found in the database. A layer must be defined in the LEF technology LAYER section before it can be referenced from a macro. Review the LEF files specified in the init_lef_file variable to see if the layer does not exist or is specified after the one that defines the macro.
Type 'man IMPLF-53' for more detail.
**ERROR: (IMPLF-3):	Error found when processing LEF file '/mnt/hgfs/mkshare/circuitnet.lef'. The subsequent file content is ignored. Refer to error messages above for details. Fix the errors, and restart tool again.
Type 'man IMPLF-3' for more detail.
**ERROR: (IMPLF-26):	No technology information is defined in the first LEF file.
Please rearrange the LEF file order and make sure the technology LEF file is the
first one, exit and restart tool.
**ERROR: (IMPLF-26):	No technology information is defined in the first LEF file.
Please rearrange the LEF file order and make sure the technology LEF file is the
first one, exit and restart tool.

Sorry for my late answer. The layer and via definition are not included in the current release of LEF due to NDA, and that's the reason for the errors. We are working on sanitizing them and the new LEF and DEF will be released in a few days, and the new DEF is dumped after routing. I will inform you in this thread when we are done.

That's fine. Thank you!

Hello, I found that you uploaded a new LEF file today, I re-downloaded it and put it into the commercial tool to read it when I reported an error, is it not completely replenished the LEF file content?Secondly, I would like to ask if the current def file has been modified?Please guide me, thank you！

Thank you for your report! I just upload a new LEF, please take a try.
Yes, the LEF has been modified this time so the new DEF is also modified accordingly.
We normalized coordinates when sanitizing. It is revertable with supplemental information which can be obtained upon email request.

First of all, thank you very much for your reply, and secondly, I have three questions, I hope you will not be bored, first: Regarding what you said about requesting this change by email, I would like to ask how to communicate by email, am I here to send you my email account or others? Second: Regarding your newly uploaded ispd2015 features, do you plan to upload the corresponding DEF file? Third: Is the newly uploaded DEF file of Circuitnet placement or global routing?

Hello, I just tried to read the LEF and DEF files, the LEF file did not report an error, but the following error occurred when reading the def file 1-RISCY-a-1-c2-u0.7-m1-p1-f0.def（For convenience, I changed its name to 1.def）

Hello, I just tried to read the LEF and DEF files, the LEF file did not report an error, but the following error occurred when reading the def file 1-RISCY-a-1-c2-u0.7-m1-p1-f0.def（For convenience, I changed its name to 1.def）

It seems that your tool do not support halo, you may need to remove the halo from DEF to proceed. The halo is defined for macros which are all in the first several lines of COMPONENTS part in the DEF, and you can remove them by removing the line starts with "+ HALO".
The missing tech-via error is quite strange, as these via definitions have been included in the DEF.
You can try to remove the halo first and see if there are more errors.

First of all, thank you very much for your reply, and secondly, I have three questions, I hope you will not be bored, first: Regarding what you said about requesting this change by email, I would like to ask how to communicate by email, am I here to send you my email account or others? Second: Regarding your newly uploaded ispd2015 features, do you plan to upload the corresponding DEF file? Third: Is the newly uploaded DEF file of Circuitnet placement or global routing?

My email is<EMAIL_ADDRESS>Yes, we can upload DEF for ispd15.
The DEF is writ after detailed routing.

If you want to load LEF DEF with openroad, you need to remove HALO in components part and virtual net in nets part, and add NDR via definition in LEF. We have prepared a LEF with NDR via definition, and you can contact me with email<EMAIL_ADDRESS>to request it.


Result does not match pytorch implmentation

Hi,
I am using version 2.0.1, and my gpu is A100(40G), I am testing with the following code:
import math
import torch
import torch.nn.functional as F
from flash_attn import flash_attn_func

inten = torch.randn(3, 2, 768, 32, 128).cuda().half()
query_states, key_states, value_states = inten[0], inten[1], inten[2]
inv_norm_factor = 1. / math.sqrt(128)

attn_output_fl = flash_attn_func(
        query_states,
        key_states,
        value_states,
        dropout_p=0., softmax_scale=inv_norm_factor,
        causal=False)

attn = torch.einsum('bihd,bjhd->bhij', query_states, key_states).mul(inv_norm_factor)
attn = F.softmax(attn, dim=-1, dtype=torch.float32).to(query_states.dtype)
attn_output = torch.einsum('bhij,bjhd->bihd', attn, value_states)
print((attn_output_fl - attn_output).abs().sum())

And I found the result is not 0, which means that the results of pytorch implementation and flash-atten does not align with each other. What is wrong with my implementation?

Some numerical difference is expected. Floating operations aren't exactly associative (e.g. a + b + c will give slightly different result compared to a + c + b), you can try that out.
I usually compare:
(FlashAttention in bf16/fp16 - standard attention in fp32) vs (standard attention in bf16/fp16 - standard attention in fp32).
This gives you an idea if what's the expected numerical difference.

Hi Dao,
Thanks for giving such a quick reply !!!
I do not think it is numerical difference, I got an result of 199:

Do you have any insights on this please?

Note that you're summing the numerical differences over 6M elements. Sum of differences of 200 means on avg the numerical difference is 3e-5.  What's the numerical error when you compare (standard attention in fp16 - standard attention in fp32)?

To clarify:
standard attention in fp32 means you convert q, k, v to fp32, do all calculation in fp32.
standard attention in fp16 means you keep q, k, v in fp16, do calculations in fp16.

Hi Dao,
When I try to use max() instead of sum(), and I run for several time, the largest different is about 1e-3:
print((attn_output_fl - attn_output).abs().max())

Is this margin of 1e-3 expected?

What's the numerical error when you compare (standard attention in fp16 - standard attention in fp32)?

Hi,
When I try to run in fp32 mode, I got the error message of:
  File "/home/.conda/envs/py39/lib/python3.9/site-packages/flash_attn/flash_attn_interface.py", line 222, in forward
    out, q, k, v, out_padded, softmax_lse, S_dmask, rng_state = _flash_attn_forward(
  File "/home/.conda/envs/py39/lib/python3.9/site-packages/flash_attn/flash_attn_interface.py", line 42, in _flash_attn_forward
    out, q, k, v, out_padded, softmax_lse, S_dmask, rng_state = flash_attn_cuda.fwd(
RuntimeError: FlashAttention only support fp16 and bf16 data type

I modified your code to do the comparison:
import math
import torch
import torch.nn.functional as F
from flash_attn import flash_attn_func

inten = torch.randn(3, 2, 768, 32, 128).cuda().half()
query_states, key_states, value_states = inten[0], inten[1], inten[2]
inv_norm_factor = 1. / math.sqrt(128)

attn_output_fl = flash_attn_func(
        query_states,
        key_states,
        value_states,
        dropout_p=0., softmax_scale=inv_norm_factor,
        causal=False)

attn = torch.einsum('bihd,bjhd->bhij', query_states, key_states).mul(inv_norm_factor)
attn = F.softmax(attn, dim=-1, dtype=torch.float32).to(query_states.dtype)
attn_output = torch.einsum('bhij,bjhd->bihd', attn, value_states)

attn_ref = torch.einsum('bihd,bjhd->bhij', query_states.float(), key_states.float()).mul(inv_norm_factor)
attn_ref = F.softmax(attn_ref, dim=-1, dtype=torch.float32)
attn_output_ref = torch.einsum('bhij,bjhd->bihd', attn_ref, value_states.float())
print((attn_output_fl - attn_output_ref).abs().max())  # 0.0002
print((attn_output - attn_output_ref).abs().max())  # 0.0012

FlashAttention is more accurate in this case.

Thanks for telling me this!!
I found my code does not converge after switching to flash-attention, I will check my code more carefully.


[Help] Failed to initialize account

template next-tailwind-counter
errors as bellows

reproduce steps
npx create-solana-dapp@latest and select the corresponding template .After that npm run dev
@beeman thank you for taking a glance

Same Issue for me as well - unable to create the account.
Tried with @latest and @next. Also tested with react + nextjs
Account page seems to work besides the faucet being dry...

Account Create Error:

Same issue.

Holy bananas beemans gonna fix this! Awesome!!!

Hey all, the program deployed on Devnet was different than the one we shipped which caused this issue.
I've deployed the current version to Devnet under this account coUnmi3oBUtwtd9fjeAvSsJssXh5A5xyPbhpewyzRVF
To update it, search for CounNZdmsQmWh7uVngV9FXW2dZ6zAgbJyYsvBpqbykg and replace it with the new account.
If you create a new account and select either next-tailwind-counter or react-vite-tailwind-counter you should get a working version.
To manually make the changes, refer to this commit and apply the changes in anchor/src/counter-exports.ts and src/components/counter/counter-data-access.tsx.
Please let me know if there are any other issues!


Backward compatibility for Github Action

We have discussed this briefly with @encukou today. When we switch to F35, we might break someone's CI because F35 does not contain Python 3.5.
My idea is to build container images and create manifests with the fedora version in their name and release a new version of Github action every time we switch to a newer Fedora release.
We can build all the container images with the original tags like amd64 and amd64_fedora34. The same applies to the latest manifest where we can use latest and latest_fedora34. We can then change the github action to use latest_fedora34 explicitly. Then, when we switch to Fedora 35, the tags and manifests using _fedora34 suffix will stay the same, and therefore the github action will stay stable and we can release a new version of the github action using _fedora35. Also, those who use the images directly can choose between a stable and possibly obsolete version of the images or rolling tags.
Does this make sense?

It does.

I have some questions.

Do we build e.g. Fedora 36 and 35 tags in parallel, offering the users to pick any Fedora version, or do we still only build one version (and the users can pick an older one, but not up to date)?
Do we add a warning to the EOLed tag in the last build we do?

My idea was that we would regularly build both f35 and f36 tags (from different branches here maybe). Once Fedora 35 goes EOL, we would (potentially add a warning and) cancel the cron in that branch.

And the GH action would be tagged to pull in that very image, so you could do @f35, @f36, etc.

Keeping just the newest works for me. I was just trying to figure out what is the end goal here.


More load options - assets and throttleMs

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

g1style_gyunay_halmi

By by by


How does BrowseAdapter inject automatically without any @Provide method?

Hi,
I wonder how does BrowseAdapter inject automatically without any @Provide method?

BrowseAdapter has @Inject at the constructor

Check for annotation processors info: https://google.github.io/dagger/users-guide.html. Hope it will help :-)


Is there a way to remove Guest Directory Tenant from my Azure Stack Deployment

Hi there is no cmdlet Unregister-AzsGuestDirectoryTenant to remove  Guest Directory Tenant from Azure Stack Deployment. Please provide such option.

FYI. I created "Unregister-AzsGuestDirectoryTenant" and send Pull Request(https://github.com/Azure/AzureStack-Tools/pull/394).

Awesome. Thank you.

Hello, we've been working on this feature internally and have prepared PR #403 for both parties to "unregister" a directory tenant from an Azure Stack deployment. You'll notice there are 2 new cmdlets added; one for the admin to revoke access from the directory, but also one to cleanup all the applications and permissions which were installed into that directory. PR #394 linked above by @yusukeMatsu shows how to do the first portion of this, which just involves deleting that resource through ARM. Note that you first have to "delete" any subscriptions which were created in that directory, otherwise ARM will block the deletion.


Add WorkerName to ErrCreatedVolumeNotFound

The ErrCreatedVolumeNotFound is logged on the ATC, not the
workers. Determining which worker is failing is very difficult.  This
adds the worker name so operators can troubleshoot more quickly.

Agreed. I've update the message to this:
volume 'fake-handle' disappeared from worker 'fake-worker' for pipeline 'fake-pipeline' job 'fake-job' task 'fake-task'
Or this if we can't pull the details:
volume 'fake-handle' disappeared from worker 'fake-worker', could not fetch task information err 'fake-error'


Add Ruby linter

Who is this PR for?
developers
What problem does this PR fix?
ruby -wc has helpful checks like variables that are unused, so let's add these warnings into the IDE and the build
What does this PR do?
Adds a rake lint and calls it in the Travis build

This one seems like an actual migration bug: https://github.com/studentinsights/studentinsights/blob/master/db/migrate/20161222175109_add_summer_boolean_to_service_type.rb#L18

This is what this looks like in Sublime, the most helpful bit for catching actual bugs being that you can see in the IDE when you have unused variables (with the dot in the gutter, just like with eslint):

And here's the status bar:

Actually reading more Rubocop can do this better and faster (Lint/UselessAssignment), but it doesn't seem to be included in our config right now.
@alexsoble I see there's a rubocop.yml file, but that many of the items that are "enabled by default" according to the docs aren't caught: http://rubocop.readthedocs.io/en/latest/cops_lint/ because of DisabledByDefault: true at the top.  I figure this came about from you tuning this when you first installed it, want to chat more about this at some point?  I'd vote for putting in a bunch of these, especially the ones that are lint cops aimed at catching bugs as compared to style cops (where I don't have any preference).

Closing this.  @alexsoble let's chat more in person or over https://github.com/studentinsights/studentinsights/pull/1521?


xcode can't be installed

Version
Tell us which versions you are using:

react-native-router-flux v4.?.? (v3 is not supported)
react-native v0.?.?

Expected behaviour
Actual behaviour
Steps to reproduce
For non-obvious bugs, please fork this component, modify Example project to reproduce your issue and include link here.
1.
2.
3.

react-native

Please, provide information in the ticket to get assistance.


Add Shared Library Build to CI

Build the shared library version of base64pp on the CI.
TODOs:

Call the Windows build template with extra -DBUILD_SHARED enabled.
Install the library and push artifacts.

This was closed with #78


Automatic creation of simplified reports issue #81

Adjusted to different inputs
Customization of reports possible, by defining a format
New report folder with pdf for each specified lead
Gitignore for created pdf's

Superseeded by #245


Chinese localization suggestions

As a native Chinese speakers, I wish you could completely delete the Chinese localization of swaylock-effects. It is way better to show in English instead of such a worth Chinese.

Just do that, it’s already supported (https://github.com/jirutka/swaylock-effects/pull/52).

(or suggest proper translations!)

I don’t wanna add any translations to swaylock-effects. There are only a few strings and they can be changed via CLI options and/or a configuration file – if someone wants to localise them, they can set localised strings themselves.


Running inference: ModuleNotFoundError: No module named 'vedacore'

When I run the model on colab:

!python tools/infer.py '/content/vedadet/configs/infer/tinaface/tinaface_r50_fpn_gn_dcn.py' '/content/vedadet/images/1.jpg'

I get the error:

Traceback (most recent call last):
File "/content/vedadet/tools/infer.py", line 7, in
from vedacore.image import imread, imwrite
ModuleNotFoundError: No module named 'vedacore'

Here is my notebook 'https://colab.research.google.com/drive/1Nr4oAgQagwZUybqfVG78HBws2HhMFksW?usp=sharing'

If you correctly clone this directory, this is mainly caused by absolute path error.
your execute file is in "tools" folder, while vedacore folder exists one step above.
so interpreter is finding vedacore module inside "tools" folder.
you can solve this problem by add
import os, sys
sys.path.append(os.getcwd())
to your execute file (Ex. tools/infer.py)


EXT-6-2 error can't download

Hi,
https://sourceforge.net/p/radiusdesk/code/1959/tree/extjs/ext-6-2-sencha_cmd.tar.gz?format=raw
Can't be downloaded thus script fails :( help

Download manually from https://gitlab.com/RADIUSDesk/other_packages/-/blob/master/ext-6-2-sencha_cmd.tar.gz to /tmp/rd_sources/ and rename file to ext-6-2-sencha_cmd.tar.gz and run installer again


chore(test): fix client tests in older IE versions

By locking sinon to the older version. The problem was that new version of sinon brought newer versions of several dependencies, which use ES2016 Map, which is not available in the older IE versions. The solution is fragile as we can't reliably lock transitive dependencies, but should do for now to make master green.
As a proper solution we should consider cutting a new major release, where we would drop support for ancient browsers and more clearly document which browsers karma aims to support.

I'll re-open from the fork as two Travis builds conflict with each other.


Camera

We want to have a camera on the rocket, but the protocol that will be used to communicate with it is at this time unclear. Whatever it ends up being will need to be implemented in firmware.

I think we'll use an off board camera that doesn't rely on a communication protocol (just locally logs to SD), so I don't think there are any firmware requirements here and I'll close this.


Align labeled text boxes in the middle

The first commit removes the bottom margin in order to perfectly align labeled text boxes vertically in the middle
Before

After

@ewpatton I made changes as per your comment. Please kindly review it. Thanks

Merged. Thanks @ethanhon!


Storybook autodocs

Description
When I import a for example Box from styled-system/jsx and create a Story with it, I expect the autodocs to be able to generate the properties for it. This however does not happen.
Link to Reproduction
.
Steps to reproduce
Create a Panda with Storybook integration, create a box.stories.tsx:
import { Box, BoxProps } from '@aurora-ui/styled-system/jsx';
import type { Meta, StoryObj } from '@storybook/react';

// More on how to set up stories at: https://storybook.js.org/docs/react/writing-stories/introduction#default-export
const meta = {
  title: 'Layout/Box',
  component: Box,
  tags: ['autodocs'],
} satisfies Meta<BoxProps>;

export default meta;
type Story = StoryObj<typeof meta>;

// This story uses a render function to fully control how the component renders.
export const Primary: Story = {
  render: (props) => {
    const { children, ...rest } = props;
    console.log(props);
    return <Box {...props}>I'm in a box!</Box>;
  },
};

JS Framework
React  TS
Panda CSS Version
0.7.0
Browser
Google Chrome
Operating System

[X] macOS
[ ] Windows
[ ] Linux

Additional Information

This doesn't seem to be a Panda bug.
The type definitions for all components are generated. Kindly re-open this in the Storybook repo.

When this works, that prop table will be very massive since it includes all common CSS properties.


[Request] Create new release?

Any chance you could release a new version @corenting ? The recent improvements have been very useful, and with a new release other packages can depend on them.

Hello ! Sorry for the time it took to do a release, I didn't have much free time.
I just released 4.0.0 with the latest changes on PyPI.

Thank you!


Cleanup UAA job spec and templates

This removes default references to run.pivotal.io in the UAA specs and enables consumers to disable the signup and password reset links from the default login page on the UAA.
The links.home, links.network, and links.signup-network do not appear to be used in the UAA so they were removed entirely. If they are used, the Pivotal Network related links should likely be extracted and separated from the open source release.

@sykesm Looks good. Will get this merged in

@mbhave @sreetummidi Can you confirm whether or not the login.links properties are being used any more?  I see the template does login.links.marshal_dump (so you don't directly see login.links.home referenced, for example), but does the UAA code in cf-release actually use them?

login.links is being used.
login.tiles is no longer in use

Actually login.tiles is used. @sykesm is right about login.links.home, login.links.network and login.links.signup-network not being used. We'll merge this in. Thanks.


License?

Thanks for taking the time to write this code. It's particularly valuable, seeing as the authors of the original paper chose not to release their version. At time of writing, 89 forks and 367 stars.
Nevertheless, it'd be a lot easier to reuse if it had an explicit license attached.
Thanks,
Darren..

Hi Darren, somehow I missed that when I created the project. I added a Licence file (MIT Licence).
Thank you for the notice!


azure bot service internal error occurred

Hi,
I have encountered an internal error in my azure bot service since yesterday. (I didn't do any change)

I also tried to create a new bot service yesterday and it was successful to be created, but show the same internal error message today.
I have tried to open in different browsers in Windows 10 and Mac OS. It's still show the error.
However, the bot is still work fine when I test in the bot framework website.
Is there anything wrong in azure bot service these days or how can I do to fix the error?
Thanks!

hi @eternaltung - There appears to be an error in the Azure Bot Service portal.  Thank you for reporting, we will pass this along to the Azure team.

I encounter the same error

Please download the Azure Storage Explorer from here: http://storageexplorer.com
Login, and navigate to your resource group's Storage Account.  When you open it, you'll see File Shares.  There you'll find the source for your bot.

This bug has been fixed, so future Bot Service bots will not experience this issue.  The Bot Service and Bot Framework are currently in preview and being improved upon daily.  We apologize for this inconvenience, and hope your future experiences with the Bot Framework are productive and enjoyable.

Same here. I got my source code on GitHub, but I can't deploy it because of 500.
P.S. I've created this bot yesterday

Hey kudinovdenis,
Bots created in the past few days have this issue.  Newly created bots will not.  Please try again today, and let us know if you run into anything.
Thanks.


[WIP] Space-efficient syncing

Syncing rounds currently requires the node to hold the entire diff in memory (both for sending and receiving). If the diff happens to be really big, it can easily overwhelm the node and taking it down. This PR improves the memory usage of this process by paging(?) the diff to disk if it gets too big.

Codecov Report

Merging #177 into master will increase coverage by 0.23%.
The diff coverage is 73.33%.

@@            Coverage Diff             @@
##           master     #177      +/-   ##
==========================================
+ Coverage   48.62%   48.86%   +0.23%
==========================================
  Files          39       39
  Lines        6577     6586       +9
==========================================
+ Hits         3198     3218      +20
+ Misses       3033     3028       -5
+ Partials      346      340       -6

Impacted Files
Coverage Δ

protocol.go
28.98% <0%> (ø)
:arrow_up:

avl/node.go
72.38% <100%> (+0.96%)
:arrow_up:

avl/tree.go
48.22% <66.66%> (-0.15%)
:arrow_down:

tx_json.go
2.2% <0%> (ø)
:arrow_up:

ledger.go
39.76% <0%> (+0.57%)
:arrow_up:

graph.go
81% <0%> (+0.59%)
:arrow_up:

gossip.go
94.44% <0%> (+8.33%)
:arrow_up:

index.go
35% <0%> (+10%)
: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 1a64f98...2f566cd. Read the comment docs.

From a single run:

Badger with 500MB diff:

SSD: 9s (accounts.Commit took 700ms)
HDD: 51s (accounts.Commit took 7.2s)

LevelDB with 500MB diff:

SSD: 9.9s (accounts.Commit took 2.2s)
HDD: 1m20s (accounts.Commit took 40s)

Badger DB implementation is removed from this PR. Merge #217 instead to get Badger db.


NPM Install not found

Node version: v18.16.0
I run this command:
npm i -g rust-to-npm-cli

Which outputs:
changed 1 package in 759ms

But then I try to use it:
rust-to-npm build

And it says not found...
-bash: rust-to-npm: command not found

🤔

Fixed in v0.4.12. The refactor on the crate to split into a CLI and rust lib ended up breaking a few things 😅.
Re-run npm i -g rust-to-npm-cli to get the latest.
Thanks again!


Is there a way to filter out data from the excel while reading the file and then converting it to JSON format ?

The excel sheet I am trying to import has a column called "status" which can have the value true/false. I want to import only the rows that have the status true, is there a way to achieve this by filtering out the data while reading the excel and then converting it to JSON?

No, as filtering out rows during parsing has other complex effects on worksheet metadata (it would effectively be the same as deleting rows from the worksheet).  The easier and recommended approach is to filter after using sheet_to_json


Clear db cache on startup

Clear the cache for the users/roles/permissions datastructures on app startup. This ensures that the cache is always up to date, for example if a database import has been done before running the app.

@zaidka What about this one? ;-)

Won't this cause cache to clear whenever a new worker process starts?

I'm not sure. I verified the execution of the clear_permissions_cache with log output. I will have a look again tomorrow.

I had a look at this once more. The clear_permissions_cache method is called per worker. This means that it is called more than once at startup if you run puma in clustered mode.
There's another possible place to clear the cache on startup, in config/puma.rb:
before_fork do
  # action
end

But I don't have access to the Rails or the GenieacsGUI object there AFAIK.
Do you haven an idea how to fix this?

Maybe clear the cache manually from console or using a rake command.

Well, then I'd rather have the disadvantage of getting the cache cleared too often than having to run a command manually :grin:

Why not cache in memory instead?

You mean using MemoryStore as the cache store? (https://guides.rubyonrails.org/caching_with_rails.html#activesupport-cache-memorystore) Well, that's an option too, but then there's a cache per worker as I have understood it.


Deleting a channel from the database results in an error relating to OpenAI

yt-fts delete --channel UC4tlrTXCBw6NPZ9nCA3_s9w

Traceback (most recent call last):
  File "/opt/homebrew/bin/yt-fts", line 8, in <module>
    sys.exit(cli())
             ^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/yt_fts/yt_fts.py", line 144, in delete
    delete_channel(channel_id)
  File "/opt/homebrew/lib/python3.11/site-packages/yt_fts/db_utils.py", line 159, in delete_channel
    from .vector_search import delete_channel_from_chroma
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/yt_fts/vector_search.py", line 7, in <module>
    from .embeddings import get_embedding
  File "/opt/homebrew/lib/python3.11/site-packages/yt_fts/embeddings.py", line 42, in <module>
    def get_embedding(text, model="text-embedding-ada-002", client=OpenAI()):
                                                                   ^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/openai/_client.py", line 92, in __init__
    raise OpenAIError(
openai.OpenAIError: The api_key client option must be set either by passing api_key to the client or by setting the OPENAI_API_KEY environment variable

This was a bug effecting 0.1.40 and was fixed on 0.1.41. see #121


Active minio kill

Test case for killing a vm of active minio

nosetests -s -v  testcases/s3_redundant/test04_minio_active_passive.py:TestActivePassive.test001_kill_active_minio_vm
test001_kill_active_minio_vm ... [Thu08 14:28] - test04_minio_active_passive.py:46  :j.s3_failures        - INFO     - Get the active minio vm (VM1)
[Thu08 14:29] - test04_minio_active_passive.py:55  :j.s3_failures        - INFO     - upload file (F1)to the active minio
[Thu08 14:29] - s3.py             :45  :j.s3demo             - INFO     - get s3 client : ParseResult(scheme='http', netloc='<IP_ADDRESS>:1024', path='', params='', query='', fragment='')
[Thu08 14:29] - s3.py             :169 :j.s3demo             - INFO     - create bucket
[Thu08 14:29] - s3.py             :170 :j.s3demo             - INFO     - bucket : ntw4jcyqnwa9fd4m
[Thu08 14:29] - s3.py             :188 :j.s3demo             - INFO     - upload 09va30awltczbiyf file to ntw4jcyqnwa9fd4m bucket
[Thu08 14:29] - test04_minio_active_passive.py:58  :j.s3_failures        - INFO     - kill VM1
[Thu08 14:29] - test04_minio_active_passive.py:61  :j.s3_failures        - INFO     - Check that a new vm has been deployed and acting as active vm
Thu08 14:29] - test04_minio_active_passive.py:120 :j.s3_failures - INFO - Download the file again from the passive minio, should succeed
[Thu08 14:30] - s3.py :199 :j.s3demo - INFO - Download x995wjvaur9mrqo1 file form 9zjkci86q0kj9baj bucket
Ran 1 test in 298.639s
OK

Codecov Report

Merging #47 into master will decrease coverage by 0.27%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #47      +/-   ##
==========================================
- Coverage   52.36%   52.09%   -0.28%
==========================================
  Files         269      259      -10
  Lines       42151    32197    -9954
==========================================
- Hits        22074    16774    -5300
+ Misses      18140    13488    -4652
+ Partials     1937     1935       -2

Impacted Files
Coverage Δ

cmd/gateway-startup-msg.go
50% <0%> (-11.12%)
:arrow_down:

cmd/posix-errors.go
38.33% <0%> (-11%)
:arrow_down:

cmd/fs-v1-rwpool.go
63.21% <0%> (-9.2%)
:arrow_down:

cmd/web-router.go
69.69% <0%> (-8.09%)
:arrow_down:

cmd/sts-handlers.go
14.63% <0%> (-8.01%)
:arrow_down:

pkg/quick/errorutil.go
78.43% <0%> (-6.38%)
:arrow_down:

cmd/retry.go
81.81% <0%> (-5.69%)
:arrow_down:

cmd/fs-v1-helpers.go
63.82% <0%> (-5.29%)
:arrow_down:

pkg/quick/encoding.go
50.66% <0%> (-4.55%)
:arrow_down:

cmd/config-migrate.go
48.39% <0%> (-4.4%)
:arrow_down:

... and 236 more

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 632a534...0aa12fd. Read the comment docs.


The function update() does not exist.

After copying the example code of Particle I get an error saying : The function update() does not exist.

That's some OOOOLD code. I've taken out that line, which wasn't doing anything anyway.

That's some OOOOLD code. I've taken out that line, which wasn't doing anything anyway.

Now there´s a particle! Thank you very much.

Now there´s a particle! Thank you very much.


rover_motor_controller controller_node crashing with coredump

Hello,
The supplied custom controller with package is crashing with core dump.
Here is an excerpt:
[controller_node-4] terminate called after throwing an instance of 'boost::wrapexceptboost::system::system_error'
[controller_node-4]   what():  write_some: Bad file descriptor
[ERROR] [controller_node-4]: process has died [pid 113153, exit code -6, cmd '/home/can/ros2_foxy/install/rover_motor_controller_cpp/lib/rover_motor_controller_cpp/controller_node --ros-args -r __node:=controller_node --params-file /tmp/launch_params_hf8tz4lm'].
Here is another:
can@hurricane:~/ros2_foxy/src/ros2_rover/rover_bringup/launch$ /home/can/ros2_foxy/install/rover_motor_controller_cpp/lib/rover_motor_controller_cpp/controller_node
terminate called after throwing an instance of 'boost::wrapexceptboost::system::system_error'
what():  write_some: Bad file descriptor
Aborted (core dumped)
I am running on ubuntu20.04 - ros foxy.
Any ideas or help debugging this problem greatly appreciated.
Best Regards,
C.A.

Are you running it in raspberry with the same hardware components (motors)?

@mgonzs13 no. i am trying to run your software on a x64 ubuntu 20.04 regular computer. how can i fire the gazebo part, and experiment with it in simulation?

If you want to run the simulator, you have to use the launch files from rover_gazebo. For instance, to launch the Moon world you have to use the command ros2 launch rover_gazebo moon.launch.py. This will launch the Moon world, the rover simulator and RTAB. To move the rover use ros2 run rover_teleop teleop_keyboard_node in a new shell.


Great refactor of the build definition

The old build had a number of problems:

NPM dependencies were defined in two different places. As a consequence, the versions used by the Scala.js compiler didn't match the versions required by the extension manifest.
The build artifacts were being placed outside the target directory, and sbt clean open didn't actually clean them. This could cause a lot of mistakes and confusion when testing new changes.
The build redefined parts of the functionality provided by existing SBT plugins.

The new build has these problems fixed:

Every configuration value and dependency version is defined in a single place.
All of the above are visible to the SBT build.
When you clean the build artifacts, they actually get cleaned.
Instead of defining a custom task, the build uses npmInstallDependencies from the sbt-scalajs-bundler plugin.

Hey, thanks for this ! I was away the past few days, thanks for your patience. I'll give it a try today.
While we're talking : do you mind if I ask about the context of this work ? Don't get me wrong, I appreciate the contribution, but I'm not sure where the motivation comes from, considering this is housecleaning work. Are you interested in adding functionality later ?

I am writing my own VS Code extension (which isn't published yet), and I have found that there is no standard way to do it with Scala.js and SBT. I have used your project and @pme123/vscode-scalajs-hello as examples, but I needed both the JS part and the Scala part (client and server) configured in the same multi-project build.
I have found out that the easiest way to achieve this is to write a SBT plugin that would configure the extension project and define the additional tasks. Having applied this solution to my own project, I decided to share it with other projects that could make use of it.
I consider publishing said plugin as a Maven artifact at some point. However, my own extension project is at a very early stage, and it doesn't yet use many of the features that are used by other projects such as yours. Thus I am also interested in getting my code tested and obtaining the feedback.
One feature that I'm going to add is publishing to the VS Code Marketplace. Looks like @scalacenter/accessible-scala already has it implemented, so I think I could reuse their code.

Awesome, thanks for the context. My day has been derailed a bit and unfortunately I've not had time to give it a try today, but I'll definitely do so over the weekend !

Sorry for taking your time, but I'm really interested in merging this, so that I may better understand any possible problems with my approach. I will be happy to answer any questions you might have.

Alright I've tested and mostly it works fine, and I'm reasonably happy with the workflow it provides, but I can't merge the PR until vsce package and vsce publish are handled by SBT somehow

The complete build output is assembled in the directory target/scala-2.13/scalajs-bundler/main. When I run vsce package in that directory, an installable VSIX file is generated. I haven't tested vsce publish (because, for one thing, I haven't registered as a publisher), but I gess it would work as well.

The complete build output is assembled in the directory target/scala-2.13/scalajs-bundler/main. (I have added a new command for doing that.) When I run vsce package in that directory, an installable VSIX file is generated. I haven't tested vsce publish (because, for one thing, I haven't registered as a publisher), but I assume it would work as well.

I've given this a go and the resulting vsix file is around 10MB large, whilst the workflow on the main branch gives me a ~160KB large file. I presume something is off (possibly due to typescript appearing as a dependency in the package.json when it's not the case).
I cannot accept the PR without this being addressed ... : even if the workflow seems nicer in surface, the resulting artifact should be of the same size as what the current workflow gives me.
Additionally, my point wrt vsce package and vsce publish commands is that without bespoke SBT commands handling that for me, the workflow provided by an SBT plugin is incomplete imho. Though, I'm willing to overlook the absence of these bespoke commands in this PR and implement them myself. But I don't want to have to cd to the correct directory in order to package the vsix file.

On a totally unrelated note : it's usually deemed a bad idea to amend commits / force-push after you've opened a pull-request and people started commenting on it. It makes it really hard for the reviewer to understand what changes in between iterations and whether their comments are addressed at all.
It's much preferable to just send normal commits, and ask the blessing of the maintainer/reviewer to squash the commits before the PR is merged.

The latest beta of org.scalablytyped.converter, which has been released a couple weeks ago, fixes the problem of excessive VSIX size (now it is about 260 KiB large). I have updated my changes accordingly and rebased them on top of the latest main. Can you please review this PR again?

I'm gonna be honest : after giving it a think, I'm just not particularly interested in merging this, as I don't think managing npm dependencies in SBT is a worthwhile endeavour here.
The current workflow is satisfying to me, considering how infrequently I need to touch this project. Going forward, I think it'd be better serving to the community if this project was folded directly in metals, but I have neither the time nor drive to take that on.
I apologise for not getting back to you earlier. I appreciate the intent, but the reality of open-source is that maintainers don't have to accommodate the vision of perfection of everyone. If you have a vision for what this project should be, feel free to fork, you have my blessing.


Update bug_report.md

- What I did
Changed app name from @ buzz to @wavi to prevent spamming mr. buzz
- How I did it
I just took out buzz completely and replaced with @wavi
- How to verify it
File a bug report and see if it annoys
- Description for the changelog

Closing this as it's already dealt with by #149


Submit a Cloud Definition

Do NOT copy/paste a definition from somewhere else. Read about the word you want to define and come up with your own definition. Copy/Paste submissions will be closed and not added.
Fill out the JSON with your submission:
    {
    "word": "Internet Protocol",
    "content": "The Internet Protocol is a Layer 3 network communications protocol which adds cross network addressing and routing capabilities to move data (encapsulated in datagrams) across Layer 2 networks from source host to destination host without direct links",
"learn_more_URL":"https://en.m.wikipedia.org/wiki/Internet_Protocol",
    "tag":"networking",
    "abbreviation": "IP",
    "author_name":"Clay Hickman",
    "author_link": "https://github.com/clayhickman"
    }

Fill out the JSON below with the following.
Word (REQUIRED)
The word you are defining. Check this URL for all words we currently have.
Content (REQUIRED)
The definition. No more than 3 sentences.
learn more URL (REQUIRED)
Website where people can visit to learn more about the word.
tag (REQUIRED and select one)
Tech category the word fits in. Options:

compute
security
service
general
analytics
developer tool
web
networking
database
storage
devops
ai/ml
identity
iot
monitoring
cost management
disaster recovery

abbreviation (OPTIONAL)
If the word is commonly abbreviated, please provide it. For example, command line interface is often abbreviated as CLI.
author name (REQUIRED)
Your name.
author link (REQUIRED)
The URL you want your name to link to.

Thank you, added :)


Remove format! calls in SsrNode

Using format! is bad for performance because it pulls in the formatting infrastructure. Instead, replace all instances where format! is used with push and push_str. Also use String::with_capacity when possible to avoid unnecessary allocations.

I can help with this

Seems this issue is still open. I would like to work on this.

@vishalsodani please, go ahead.

@lukechu10 Hi, you have commented here

Only files in packages/sycamore/generic_node/ssr_node.rs could use this optimization because they are part of the hot path.

What does this mean? Is there specific usage of format! in this file which needs to be replaced?

@vishalsodani instead of implementing std::fmt::Display for the different ssr nodes, create a new function called write_* that takes a &mut String. Call .reserve() on the string with the computed length and then use .push_str() or .push_char(). This prevents creating an instance of Arguments which is no always zero-cost. Hopes this helps.


#[derive(Debug)] seems expensive

The code generated by #[derive(Debug)] uses core::fmt::DebugStruct and its siblings: DebugList, DebugMap, DebugSet, and DebugTuple. Internally, these types use &dyn fmt::Debug, and it appears that in Rust userspace apps this defeats LLVM's devirtualization. This causes most or all of the Debug implementations (including Debug implementations for types from libcore) to be compiled in.
I ran some tests on a stripped-down libtock-rs that removes all of its #[derive(Debug)] instances. In these tests, I manually derived Debug using the core::fmt::Debug* types as well as implemented it entirely manually. Here are the resulting sizes of .text (in bytes):

DebugStruct: 15664
DebugTuple: 15504
DebugList: 15584
DebugSet: 15584
DebugMap: 16576
Manual implementation (no Debug*): 504

Based on this test, I think it may be wise to avoid #[derive(Debug)] in libtock-rs and instead derive Debug manually. I count 12 uses of #[derive(Debug)] in libtock-rs so this seems reasonable.
Any comments before I make this change?

This could also be gated by one (or several) feature(s). cfg_attr can be used to conditionally derive Debug:
#[cfg_attr(feature = "derive_debug", derive(Debug))]
struct Foo { ... }

And the release/debug mode (or another feature) could be checked to decide whether to implement Debug at all (whether hand-rolled or derived).
What do you think?

I personally would be okay with just removing all of the debug implementation as I don't use them for debugging and we shouldn't use them in production code at all.

I personally would be okay with just removing all of the debug implementation as I don't use them for debugging and we shouldn't use them in production code at all.

This has been affecting our test binaries significantly, where we need Debug or similar functionality. Note that this also applies to Display, although I'm not sure how necessary that is in production builds.
I've been working on code size tooling to help understand how we can mitigate this issue.

I just found out that we are not the only ones wanting a more lightweight formatting in Rust: https://twitter.com/japaric_io/status/1196203591737991168
There is now the ufmt crate, streamlining a lightweight Debug-like trait: https://crates.io/crates/ufmt

After talking to some Rust people they pointed me to https://github.com/rust-lang/rust/pull/101568 which should help here.
It should allow more optimisations when using the core fmt

[..] this defeats LLVM's devirtualization.

Working on it!
(No promises yet, but it looks like at least one of my experimental approaches doesn't break devirtualization.)


Index Advisor: Missing Indexes issue docs

Add a Missing Indexes issue. This is like the existing Missing Index
issue, but it looks at all index recommendations on a single table
together, rather than recommending individual indexes in separate
issues.

@keiko713 I think that's the plan, yeah. Having all index recommendations on a single table together should be strictly superior.


API: take PIN in request param and return link object

write a controller (GET endpoint) that takes PIN and return valid link obj from db.
Let's assume PINs are valid for 30min. Hence, it will NOT return the link obj if currentTime - pinCreationTime > 30601000 i.e. status 404.

hey @muke1908, I am intrested in taking up this issue can you expand a little more

Hi @yatharthMogra sure!
You'll have to write an API for example:  GET- > /chat-link/<pin>
router.get(
  '/:pin',
  asyncHandler(async (req, res) => {
    const { pin } = req.params;
    ... your code to find link from db.
);

use util function findOneFromDB to find link by pin.
Most of the controllers are written here: https://github.com/muke1908/chat-e2ee/blob/master/backend/api/chatLink/index.js
You can take a look for example.
Let me know if you have any issue in setting up the project.


Feature Request: Mark Note Requests as Fulfilled Upon Upload

Describe the feature
Currently, in the ExamTime web application, users can request notes from others, creating a platform for knowledge sharing and collaboration. However, there is a lack of clarity regarding the status of these requests once notes are uploaded by other users.
I propose the addition of a new feature where users uploading notes can specify which request they are fulfilling. Upon upload, the corresponding request will be marked as fulfilled, providing clarity to other users and preventing redundant uploads of the same notes. This enhancement will streamline the note-sharing process, improve user experience, and promote efficient utilization of resources within the community.
Key Features:

Option for users uploading notes to select the request(s) they are fulfilling.
Automatic marking of corresponding requests as fulfilled upon successful upload.
Clear indication on the platform interface that a request has been fulfilled, preventing duplicate uploads and providing transparency to users.
Improved user engagement and satisfaction through efficient note-sharing processes.

Add ScreenShots

I want to show fullfilled status for the request, when any user uploads the pdf for that request
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'24 contributor
[X] I want to work on this issue

@Praveenkumar5048 I want something like there should be a button or button from which you will be able to fulfill notes request
then the request will be removed
Also don't forget to implement this in the View All Notification Page also


enterogastritis, disalicylide (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


⚠ Bofrost.es has degraded performance

In 693f7dd, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1515 ms

Resolved: Bofrost.es performance has improved in 5ecb837.


LLaMA-7B-q4 inference only 4 threads are used

LocalAI version:

Jun 8, 2023
Environment, CPU architecture, OS, and Version:

Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz
Describe the bug

ggml-model-q4_0.bin
docker-compose up -d --pull always
curl http://localhost:8080/v1/completions -H "Content-Type: application/json" -d '{
     "model": "your-model.bin",
     "prompt": "A long time ago in a galaxy far, far away",
     "temperature": 0.7
   }'

only 4 cpus are used, while I have 40 on a single socket.

@imajiayu  did you set the threads here? https://github.com/go-skynet/LocalAI/blob/6bb562272dada1da893f8fb1bfc768b6d819d2de/.env#L3

Thanks a lot.


Enhancements to Entity Management and Display Features

This merge introduces several updates and new features to improve the management and display of various entities within the project. The following changes have been made:

Updated Coordenador, Beneficiario: Enhanced the management of coordinators and beneficiaries.
Added Projecto and Created Beneficiario Show: Introduced project management and created a detailed view for beneficiaries.
Updated Beneficiario Show: Improved the detailed view for beneficiaries with additional information and features.
Created Financiador Show: Developed a detailed view for financiers to display their information.
Created Sub Rubrica and Added Rubrica Projecto Show: Added functionality for sub-rubrics and created a detailed view for project rubrics.
Added Show Rubrica Projecto: Enhanced the project rubric view with additional details and features.

These updates collectively enhance the system's capability to manage and display detailed information for various financial and project-related entities, providing users with more comprehensive and user-friendly interfaces

great


change text of "greeneries adjacent to cities"

Stephen pointed out that this is above what is technically a list of cities, not a list of greeneries

also fixed in #39


Register v0.1.1

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/39674
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.1 -m "<description of version>" 588fa5cbd348d9864a406084de46bbd3ed69f00d
git push origin v0.1.1


Remove struct tags from pkg/api/types.go

WIP, we'll see how tests fare

Should we have a hook that checks for these, ie opposite of existing hook
for internal versions?
On Thu, May 14, 2015 at 8:20 PM Tim Hockin<EMAIL_ADDRESS>wrote:

WIP, we'll see how tests fare
You can view, comment on, or merge this pull request online at:
https://github.com/GoogleCloudPlatform/kubernetes/pull/8294
Commit Summary

Remove struct tags from pkg/api/types.go

File Changes

M pkg/api/types.go
https://github.com/GoogleCloudPlatform/kubernetes/pull/8294/files#diff-0
(748)

Patch Links:

https://github.com/GoogleCloudPlatform/kubernetes/pull/8294.patch
https://github.com/GoogleCloudPlatform/kubernetes/pull/8294.diff

—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/8294.

I don't think we need a hook for this - it's easy enough to keep correct once it is bulk removed.

Is this still a wip or can i merge on green?

We should abandon this PR.
Sadly, the codebase is not ready for this.  Specifically:

clientcmd depends on api.TypeMeta (not documented) and the JSON tags therein.
kubectl does a JSON marshal/unmarshal which breaks without the core JSON tags.


ci.yml uses codecov/codecov-action@v3, which uses a deprecated version of Node.js

https://github.com/TanStack/form/actions/runs/9727402334