Help and feedback buttons missing.

We need to recover these buttons.  In addition, the three small glyphs (help/feedback/bookmark) should be moved to the top right of the left pane, not be in the search box.

If the links to help and feedback already exist in the top nav bar (where the Log In and Sign Up is), would the help and feedback buttons in the sidebar be too redundant?

I think it still might make sense to have help on the left pane.  We can make it more context sensitive too.
Also, I would change the text on the bookmark bar to say something like:
"Bookmark your current search results.”
Carl
On Oct 5, 2015, at 10:42 AM, Jessie Wong <notifications<EMAIL_ADDRESS>wrote:
If the links to help and feedback already exist in the top nav bar (where the Log In and Sign Up is), would the help and feedback buttons in the sidebar be too redundant?
—
Reply to this email directly or view it on GitHubhttps://github.com/informatics-isi-edu/chaise/issues/96#issuecomment-145609661.

What do you mean by context sensitive?
How about "Save Search | Help | Send Feedback" with a glyph before each of these?
"Save Search" because as @karlcz pointed out in another issue, we're actually saving the search criteria and not the results themselves. Also, it's more concise.

Lets just move the bookmark button and keep just the one.

Let's not call it "Save Search" that seems misleading. Let's simply call it Bookmark or Permalink.

I'd go with Permalink since 'Bookmark' could be ambiguous (e.g. bookmark what? will this bookmark be saved somewhere in my account?). Permalink is more established as a term that you know will give you a URL.

agreed

From: Jessie Wong<EMAIL_ADDRESS>Sent: Monday, October 05, 2015 5:07 PM
To: informatics-isi-edu/chaise
Cc: Robert Schuler
Subject: Re: [chaise] Help and feedback buttons missing. (#96)
I'd go with Permalink since 'Bookmark' could be ambiguous (e.g. bookmark what? will this bookmark be saved somewhere in my account?). Permalink is more established as a term that you know will give you a URL.

Reply to this email directly or view it on GitHubhttps://github.com/informatics-isi-edu/chaise/issues/96#issuecomment-145702465.
[https://avatars3.githubusercontent.com/u/3597150?v=3&s=400]https://github.com/informatics-isi-edu/chaise/issues/96#issuecomment-145702465
Help and feedback buttons missing. · Issue #96 · informatics-isi-edu/chaise · GitHub
We need to recover these buttons. In addition, the three small glyphs (help/feedback/bookmark) should be moved to the top right of the left pane, not be in the search box.
Read more...https://github.com/informatics-isi-edu/chaise/issues/96#issuecomment-145702465

@robes can you close this if it is really done?

@crisaless I don't think we ever recovered these.

I interpreted @carlkesselman's last comment in this thread to mean that that the help and feedback buttons are no longer needed. If this is right, then we can move this to Closed.

This has entered the realm of "we don't remember what this bug report means and therefore what should be done to resolve it." :-)
Thus we close it in and open a new issue as needed.


When a result has an invalid draw it can mess up the entire system

I need to ensure that a single invalid draw can't bring down the entire system.

See https://trello.com/c/XnvnyEjW for further details.

Commit https://github.com/dwayne/playwhe/commit/206bb0e4df5703312218c24fea1b26e437835ff5 attempts to fix this issue. Closing for now, but will reopen if it didn't fix it.


Update storage-format-(windows).md

this cost me an hour of my life

thank you for your sacrifice 🥲🥲

Then note must says about other using tools, not what is used in guide

If you using other tools to format sdacrd and it have option to choice partition table (MBR, GPT), you must use MBR


Added Extra Select Option for Group

Solves issue #837
As I said, We can add an extra Select for the Groups and now the dropdown is looks compact than before!
Let me know if there are any changes that I could do!!

@eltonmesquita will review this thanks

Sorry for the late answer, but the project advanced a lot and I'm changing too many things on the webpage to accept any changes to it now.
Thanks for all the work! Hope to get some contribution from you in the future!


removing a removePopup() method not used

popups are removed via the updatePopups() method

Coverage decreased (-0.005%) to 97.765% when pulling b389ca65b0002d68e2d54e7a0c1156064ecde3f4 on CarlyLanglois:remove-popup into bc137db230bdb90869acecc14ae0c2ada8610600 on boundlessgeo:master.

hmm does this make sense? I think we would still want a way for people to programatically remove a popup? cc @theduckylittle

oh good point. but if the map component accomplishes removing a popup w/o this method, would it make sense to just include a remove function inside of whatever app needs it?

it is true that popups can still be closed by calling the close method on the popup itself
I'm not sure if we need to have 2 ways, I'll let @theduckylittle decide since he wrote the original popup code

Please don't remove potentially useful API calls. :)
updatePopups does the real work but we should have an exposed method for removing a popup by ID.  The only real flaw here is that we should be returning the popup ID in addPopup so it's easier to reference.

ok, thanks for the notes. I'll close this then


Example in "Overview of Estimator" is missing an import statement

Environment

Qiskit Terra version: 0.37.1

Cutting and pasting the example given in "Overview of Estimator" almost works. But, Estimator is not imported.
This is the API doc for Primitives, not in the docstring for the Estimator class.
Adding
from qiskit.primitives import Estimator

to this list
https://github.com/Qiskit/qiskit-terra/blob/c318725d679b0eb3aa8874fb9755fa0567c5e94b/qiskit/primitives/base_estimator.py#L61-L62
does allow the example to run.
The same problem occurs for Sampler. Is this intentional? Maybe the idea is that in an example in a description of a class, the reader is expected to copy the example code and then add a line to import the class being described ?
Whether it's intentional or not, I think adding a single line to these examples would make them more convenient to use.
What is happening?
The example for Estimator fails because of a missing import statement.
How can we reproduce the issue?
Copy and paste the example into a Python session.
What should happen?
The example should run and print the output.
Any suggestions?
Add a line in the example to import the class.

Hi there! Thank you for the detailed description, this is my first contrib. I started off fixing this and updated the class descriptions + example usages to match the non-depricated parts of the code.

Thank you @sebastian-mair !


How to train LinkerInvent's prior model using my own data

Thank you very much for open-sourcing the project. I noticed that pre-trained prior models are provided in the 'prior/' directory. I am a beginner and would like to train a new prior using my own data, but I couldn't find any instructions on how to train the prior. I would greatly appreciate it if you could provide me with some guidance and assistance on this matter

你好
非常感谢您对 REINVENT 的关注，欢迎加入社区！
我们有四种不同的生成器“样式”，您对哪一种感兴趣（Reinvent、Libinvent、Linkinvent、Mol2Mol）。一般来说，我不一定会建议为生产制作新的先验，因为它需要一些知识和技能来制作高质量的先验。但是，制作新模型并使其表现良好当然也是一个有趣的学习练习。
非常感谢，Hannes。

First of all, thank you for your reply. My own dataset consists of a group of molecules represented in SMILES format. I want to train a prior model using LinkerINVENT starting from this data. I'm not sure where to find scripts for data preprocessing and retraining. I would greatly appreciate it if you could provide them.

Hi,
many thanks for your interest in REINVENT and welcome to the community!
We have four different generator "styles", which one are you interested in (Reinvent, Libinvent, Linkinvent, Mol2Mol). Generally, I would not necessarily recommend to produce new priors for production as it requires some knowledge and skill to make a high-quality one. But it can certainly also be an interesting learning exercise to make new models and get it to perform well.
Many thanks, Hannes.

First of all, thank you for your reply. My own dataset consists of a group of molecules represented in SMILES format. I want to train a prior model using LinkerINVENT starting from this data. I'm not sure where to find scripts for data preprocessing and retraining. I would greatly appreciate it if you could provide them.

Hi,
many thanks for your interest in REINVENT and welcome to the community!
We have four different generator "styles", which one are you interested in (Reinvent, Libinvent, Linkinvent, Mol2Mol). Generally, I would not necessarily recommend to produce new priors for production as it requires some knowledge and skill to make a high-quality one. But it can certainly also be an interesting learning exercise to make new models and get it to perform well.
Many thanks, Hannes.

First of all, thank you for your reply. I'm particularly interested in the 'styles' of the Linkinvent generator. My own dataset consists of a group of molecules represented in SMILES format. I want to train a prior model using Linkinvent starting from this data. I'm not sure where to find scripts for data preprocessing and retraining. I would greatly appreciate it if you could provide them.

Hi again,
the original publication describes how the input SMILES have been split but eventually is the same method as applied for Libinvent.  The repository of the original code may have some clues and data but the data splitting code is in this repository.

@halx thank you，i got it


Is similarity wrong ??

Inside the compare_files method, we are using
..
..
slices1 = get_copied_slices(idx1, file1_data.k)
..
token_overlap1 = np.sum(slices1[1] - slices1[0])
..
..
if len(file1_data.filtered_code) > 0:
        similarity1 = token_overlap1 / file1_data.token_coverage
..

to find the similarity.
But here, I think slices are the actual character(not tokens) slices. When we find the similarity, shouldn't we be divicing using len(file_data.raw_code) instead ? (Assuming token coverage is actually the number of tokens (not characters) in the code file)

Am I missing something here ??
Cause when I compared exact same file, token coverage and overlap are same so similarities are 100%. But when I compared with slighlty modified files, then when I took the actual hashes that got matched from find_fingerprint_overlap method and divide with actual number of hashes in the file, I'm getting different score.
..
copied_hashes, token_overlap, (sim1, sim2), (slices1, slices2) = compare_files(test_file, ref_file))
..
..
score = len(copied_hashes)/len(self.detector.file_data[test_f].hashes)
print(score, sim1)
..
..

here, in compare_files, I modified the find_fingerprint_overlap method to return the actual hashes that got matched
def find_fingerprint_overlap(hashes1, hashes2, idx1, idx2):
    """Finds the indexes of overlapping values between two lists of
    hashes. Returns two lists of indexes, one for the first hash list
    and one for the second. The indexes of the original hashes are
    provided in case boilerplate results in gaps.
    """
    overlap, ol_idx1, ol_idx2 = np.intersect1d(hashes1, hashes2,
        return_indices=True, assume_unique=True)
    return ol_idx1, idx1[ol_idx1], idx2[ol_idx2]

This I found out when I was trying to get the overall matched code for a single test_file when compared with all the ref_files.

The detection loop works over k-grams, so the tokenized code is stitched back together into a string after tokenization. A file post-tokenization might look like this, for example:
defF(V,V):SV=[hash(V[V:V+V])forVinrange(len(V)-V+1)]returnV.V(V)
It's been a little while since I've looked at this code but I believe both the token coverage and the slice indices refer to this string, not the raw code. So when I say "tokens", I mean characters in the tokenized string.
Hash overlap divided by number of hashes is actually how copydetect 0.4.0 and 0.4.1 computed similarity. The problem with this method is that hash overlap doesn't always map very well back to how similar the original files are: there's an example of this issue in #19.

The short answer is that the use case you're describing just isn't possible with how the code is structured.
The detection code was designed around pairwise similarity, so pretty much all the functions assume comparison between two files. I think the easiest way to implement a one-to many comparison would look something like this (adapted from compare_files in detector.py):
slice_map = {}
similarity_statistics = {}
for file2_data in reference_files:
    # find any overlapping hashes
    overlap, ol_idx1, ol_idx2 = np.intersect1d(
        file1_data.hashes, file2_data.hashes,
        return_indices=True, assume_unique=True
    )
    if overlap.shape[0] > 0:
        # if there is overlap, store the copied slices under the name
        # of the copied file and remove the hashes from the fingerprint
        # object to avoid duplicate detection
        idx1 = file1_data.hash_idx[ol_idx1]
        slices = get_copied_slices(idx1, file1_data.k)
        slice_map[file2_data.filename] = slices
        file1_data.hashes = np.delete(file1_data.hashes, ol_idx1)

        # convert slices computed for the tokenized code into slices
        # for the original/untokenized code
        untokenized_slices = slices.copy()
        if len(file1_data.offsets) > 0:
            untokenized_slices += file1_data.offsets[:, 1][
                np.clip(np.searchsorted(file1_data.offsets[:, 0], untokenized_slices),
                0, file1_data.offsets.shape[0] - 1)
            ]

        # all three possible similarity metrics
        similarity_statistics[file2_data.filename] = {
            # number of fingerprints (n-grams) copied:
            "hashes": idx1.shape[0],
            # number of tokenized characters copied:
            "tokens": np.subtract(slices[1], slices[0]).sum(),
            # number of total characters copied (not considering tokenization):
            "characters": np.subtract(untokenized_slices[1], untokenized_slices[0]).sum()
        }

To compute similarity scores the first metric would be divided by fingerprint count (file1_data.hashes), the second by tokenized string length (len(file1_data.filtered_code)), and the third by original string length (len(file1_data.raw_code)).
Ultimately, however, this is just a thrown-together solution (it's not particularly efficient and modifying the hashes in place is not a good solution). A proper solution would unfortunately require a pretty comprehensive refactor of the current code.

True. Even if we manage to do that. There is one more case to handle. As hashes are simply a set of hashes, not a list, if there is any repeated fingerprint (repeated k-gram string), that got copied, we should not remove the option to compare the same thing elsewhere. It is a different problem altogether.  Even If we decide to solve it, as you said, it would be like writing the entire library all over again.
Anyway, Thank you for your response. I greatly appreciate the help. 😊

Closing this as the issue for matching similarity across multiple files is open in #44 and the handling of duplicate hashes was corrected by #48


Update link for: Django and SQL Database on Azure

The original link https://github.com/Microsoft/PTVS/wiki/Django-and-SQL-Database-on-Azure directs the user to a retired repo page, the link should be updated to direct the user to https://docs.microsoft.com/en-us/azure/app-service/containers/quickstart-python?tabs=bash (Quickstart: Create a Python app in Azure App Service on Linux)

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

ID: 7625e88c-5f6b-c5c8-7708-0123aa115c2f
Version Independent ID: e6548c63-31ef-31d1-f4b4-46234dc7e1a7
Content: Python Driver for SQL Server - SQL Server
Content Source: docs/connect/python/python-driver-for-sql-server.md
Product: sql
Technology: connectivity
GitHub Login: @David-Engel
Microsoft Alias: v-daenge

@hecj1 -- James, thank you for your feedback.  I am looking into this article.

Thank you for your feedback.  I have updated the documentation and it should be live by tomorrow.


[UX] implementation status of the --format

This is an umbrella ticket for tracking the implementation status of the --format option.

[X] docker events

[ ] docker history #30962 @TheHipbot

[X] docker images

[X] docker info

[X] docker inspect

[X] docker ps

[ ] docker search

[X] docker stats

[X] docker network ls

[ ] docker node ls #30424 @yongtang

[x] docker node ps #28213 @yongtang

[X] docker plugin ls

[X] docker plugin inspect

[ ] docker secret ls https://github.com/docker/docker/issues/30431#issuecomment-283687636 @ripcurld0

[X] docker secret inspect

[X] docker service inspect

[x] docker service ls #28199 @yongtang

[x] docker service ps #28213 @yongtang

[ ] docker stack ls

[x] docker stack ps #28213 @yongtang

[x] docker stack services #28199 @yongtang

[ ] docker system df https://github.com/docker/docker/pull/31482 @ripcurld0

[X] docker volume ls

[X] docker volume inspect

For following commands, maybe worth adding --format, but either is fine.

docker diff
docker port
docker checkpoint ls
docker top (difficult because currently invoking external top)

cc @albers @sdurrheimer

@AkihiroSuda docker stack ps, docker node ps, and docker service ps are being addressed in #28213 together; docker stack service and docker service ls are being addressed in #28199 together as well.

thank you for the information, updated the list

@TheHipbot feel free to open as a pull request, then code review takes place on the pull request

@TheHipbot
I left some comments to your commit.
Probably additional integration test is not needed.
(make test-integration-cli doesn't work for you? )

I guess BINDDIR= make test-integration-cli works for you
(Set BINDDIR to empty so that UNIX socket in the bundles dir works)

@AkihiroSuda can you put #31482 next to docker system df
Thanks 🐪

@AkihiroSuda - I am not sure why docker checkpoint ls needs format.
It has only one column (checkpoint name).
Are we going to add more attributes/columns to it?
Thanks.

@AkihiroSuda I think I will also take docker secret ls. It won't take me that long, I have made some progress https://github.com/ripcurld0/docker/commit/99773bbb8ec36b26ea1f255905ace3cb5a3cfce1

updated the list; moved checkpoint ls to the "either is fine" list
(I agree it is unneeded now, but maybe worth adding in future)

@AkihiroSuda -- I have taken some initiative and added the format option to the followings:

secret ls #31552
stack ls #31557

Now I will pursue to checkpoint-ls, diff and  port.
Thanks.

updated the list, thank you

@AkihiroSuda you can check docker diff, docker secret ls and docker checkpoint ls check-boxes

done

docker search (#31539) is still left, but let me close this issue, as CLI is no longer part of this repo


Firebase.configure() and GADMobileAds.configure make (independently) app crash on simulator with iOS10

[1] Environment

Xcode version: 9.2 and previous 9.x too
Firebase SDK version: 4.8.0
Firebase Component: Auth, Core, Database, AdMob
Component version: corresponding - 4.4.1, 4.0.13, 4.1.3

[2] Description:
Firebase.configure() and GADMobileAds.configure make (independently) app crash on simulator with iOS10 at the very start. Only if not to use configure() (and, of course any other commands) app can start.
I've wrote to support and got advise to check out the issue on one of sample projects, and it was reproduced!
[3] Steps to reproduce:
Take sample project, set it up with Firebase (using Pods file) and start on simulator or iPhone with iOS10
objc[5470]: Class PLBuildVersion is implemented in both /Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 10.2.simruntime/Contents/Resources/RuntimeRoot/System/Library/PrivateFrameworks/AssetsLibraryServices.framework/AssetsLibraryServices (0x1253f44c) and /Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 10.2.simruntime/Contents/Resources/RuntimeRoot/System/Library/PrivateFrameworks/PhotoLibraryServices.framework/PhotoLibraryServices (0x12380b78). One of the two will be used. Which one is undefined.
Dec 28 18:42:32  DatabaseExampleSwift[5470] : [Firebase/Analytics][I-ACS023007] Firebase Analytics v.40005000 started
Dec 28 18:42:32  DatabaseExampleSwift[5470] : [Firebase/Analytics][I-ACS023008] To enable debug logging set the following application argument: -FIRAnalyticsDebugEnabled (see http://goo.gl/RfcP7r)
2017-12-28 18:42:32.849862 DatabaseExampleSwift[5470:409304] [] __nwlog_err_simulate_crash_libsystem libsystem simulate crash unavailable "libsystem_network.dylib: nw_host_stats_add_src :: received error for SRC_ADDED: [22] Invalid argument"
2017-12-28 18:42:32.851078 DatabaseExampleSwift[5470:409304] [] nw_host_stats_add_src received error for SRC_ADDED: [22] Invalid argument, dumping backtrace:
[i386] libnetcore-856.30.16
0   libsystem_network.dylib             0x04e3d14d __nw_create_backtrace_string + 123
1   libsystem_network.dylib             0x04e54637 nw_get_host_stats + 1049
2   libnetwork.dylib                    0x051be48e nw_endpoint_resolver_start_next_child + 1560
3   libdispatch.dylib                   0x04bdb396 _dispatch_call_block_and_release + 15
4   libdispatch.dylib                   0x04c06cc3 _dispatch_client_callout + 14
5   libdispatch.dylib                   0x04be33ae _dispatch_queue_serial_drain + 1619
6   libdispatch.dylib                   0x04be3bc2 _dispatch_queue_invoke + 1109
7   libdispatch.dylib                   0x04be65e4 _dispatch_root_queue_drain + 470
8   libdispatch.dylib                   0x04be63a6 _dispatch_worker_thread3 + 143
9   libsystem_pthread.dylib             0x04f85fdc _pthread_wqthread + 1356
10  libsystem_pthread.dylib             0x04f85a6a start_wqthread + 34
2017-12-28 18:42:33.150878 DatabaseExampleSwift[5470:409292] [] __nwlog_err_simulate_crash_libsystem libsystem simulate crash unavailable "libsystem_network.dylib: nw_host_stats_add_src :: received error for SRC_ADDED: [22] Invalid argument"
2017-12-28 18:42:33.151841 DatabaseExampleSwift[5470:409292] [] nw_host_stats_add_src received error for SRC_ADDED: [22] Invalid argument, dumping backtrace:
[i386] libnetcore-856.30.16
0   libsystem_network.dylib             0x04e3d14d __nw_create_backtrace_string + 123
1   libsystem_network.dylib             0x04e54637 nw_get_host_stats + 1049
2   libnetwork.dylib                    0x051be48e nw_endpoint_resolver_start_next_child + 1560
3   libdispatch.dylib                   0x04bdb396 _dispatch_call_block_and_release + 15
4   libdispatch.dylib                   0x04c06cc3 _dispatch_client_callout + 14
5   libdispatch.dylib                   0x04be33ae _dispatch_queue_serial_drain + 1619
6   libdispatch.dylib                   0x04be3bc2 _dispatch_queue_invoke + 1109
7   libdispatch.dylib                   0x04be65e4 _dispatch_root_queue_drain + 470
8   libdispatch.dylib                   0x04be63a6 _dispatch_worker_thread3 + 143
9   libsystem_pthread.dylib             0x04f85fdc _pthread_wqthread + 1356
10  libsystem_pthread.dylib             0x04f85a6a start_wqthread + 34
2017-12-28 18:42:33.497519 DatabaseExampleSwift[5470:409292] [] __nwlog_err_simulate_crash_libsystem libsystem simulate crash unavailable "libsystem_network.dylib: nw_host_stats_add_src :: received error for SRC_ADDED: [22] Invalid argument"
2017-12-28 18:42:33.498277 DatabaseExampleSwift[5470:409292] [] nw_host_stats_add_src received error for SRC_ADDED: [22] Invalid argument, dumping backtrace:
[i386] libnetcore-856.30.16
0   libsystem_network.dylib             0x04e3d14d __nw_create_backtrace_string + 123
1   libsystem_network.dylib             0x04e54637 nw_get_host_stats + 1049
2   libnetwork.dylib                    0x051be48e nw_endpoint_resolver_start_next_child + 1560
3   libdispatch.dylib                   0x04bdb396 _dispatch_call_block_and_release + 15
4   libdispatch.dylib                   0x04c06cc3 _dispatch_client_callout + 14
5   libdispatch.dylib                   0x04be33ae _dispatch_queue_serial_drain + 1619
6   libdispatch.dylib                   0x04be3bc2 _dispatch_queue_invoke + 1109
7   libdispatch.dylib                   0x04be65e4 _dispatch_root_queue_drain + 470
8   libdispatch.dylib                   0x04be63a6 _dispatch_worker_thread3 + 143
9   libsystem_pthread.dylib             0x04f85fdc _pthread_wqthread + 1356
10  libsystem_pthread.dylib             0x04f85a6a start_wqthread + 34
2017-12-28 18:42:33.686690 DatabaseExampleSwift[5470:409304] [] __nwlog_err_simulate_crash_libsystem libsystem simulate crash unavailable "libsystem_network.dylib: nw_host_stats_add_src :: received error for SRC_ADDED: [22] Invalid argument"
2017-12-28 18:42:33.687432 DatabaseExampleSwift[5470:409304] [] nw_host_stats_add_src received error for SRC_ADDED: [22] Invalid argument, dumping backtrace:
[i386] libnetcore-856.30.16
0   libsystem_network.dylib             0x04e3d14d __nw_create_backtrace_string + 123
1   libsystem_network.dylib             0x04e54637 nw_get_host_stats + 1049
2   libnetwork.dylib                    0x051be48e nw_endpoint_resolver_start_next_child + 1560
3   libdispatch.dylib                   0x04bdb396 _dispatch_call_block_and_release + 15
4   libdispatch.dylib                   0x04c06cc3 _dispatch_client_callout + 14
5   libdispatch.dylib                   0x04be33ae _dispatch_queue_serial_drain + 1619
6   libdispatch.dylib                   0x04be3bc2 _dispatch_queue_invoke + 1109
7   libdispatch.dylib                   0x04be65e4 _dispatch_root_queue_drain + 470
8   libdispatch.dylib                   0x04be63a6 _dispatch_worker_thread3 + 143
9   libsystem_pthread.dylib             0x04f85fdc _pthread_wqthread + 1356
10  libsystem_pthread.dylib             0x04f85a6a start_wqthread + 34
2017-12-28 18:42:33.955698 DatabaseExampleSwift[5470:409380] [] __nwlog_err_simulate_crash_libsystem libsystem simulate crash unavailable "libsystem_network.dylib: nw_host_stats_add_src :: received error for SRC_ADDED: [22] Invalid argument"
2017-12-28 18:42:33.956429 DatabaseExampleSwift[5470:409380] [] nw_host_stats_add_src received error for SRC_ADDED: [22] Invalid argument, dumping backtrace:
[i386] libnetcore-856.30.16
0   libsystem_network.dylib             0x04e3d14d __nw_create_backtrace_string + 123
1   libsystem_network.dylib             0x04e54637 nw_get_host_stats + 1049
2   libnetwork.dylib                    0x051be48e nw_endpoint_resolver_start_next_child + 1560
3   libdispatch.dylib                   0x04bdb396 _dispatch_call_block_and_release + 15
4   libdispatch.dylib                   0x04c06cc3 _dispatch_client_callout + 14
5   libdispatch.dylib                   0x04be33ae _dispatch_queue_serial_drain + 1619
6   libdispatch.dylib                   0x04be3bc2 _dispatch_queue_invoke + 1109
7   libdispatch.dylib                   0x04be65e4 _dispatch_root_queue_drain + 470
8   libdispatch.dylib                   0x04be63a6 _dispatch_worker_thread3 + 143
9   libsystem_pthread.dylib             0x04f85fdc _pthread_wqthread + 1356
10  libsystem_pthread.dylib             0x04f85a6a start_wqthread + 34
2017-12-28 18:42:34.418738 DatabaseExampleSwift[5470:409304] [] __nwlog_err_simulate_crash_libsystem libsystem simulate crash unavailable "libsystem_network.dylib: nw_host_stats_add_src :: received error for SRC_ADDED: [22] Invalid argument"
2017-12-28 18:42:34.419438 DatabaseExampleSwift[5470:409304] [] nw_host_stats_add_src received error for SRC_ADDED: [22] Invalid argument, dumping backtrace:
[i386] libnetcore-856.30.16
0   libsystem_network.dylib             0x04e3d14d __nw_create_backtrace_string + 123
1   libsystem_network.dylib             0x04e54637 nw_get_host_stats + 1049
2   libnetwork.dylib                    0x051be48e nw_endpoint_resolver_start_next_child + 1560
3   libdispatch.dylib                   0x04bdb396 _dispatch_call_block_and_release + 15
4   libdispatch.dylib                   0x04c06cc3 _dispatch_client_callout + 14
5   libdispatch.dylib                   0x04be33ae _dispatch_queue_serial_drain + 1619
6   libdispatch.dylib                   0x04be3bc2 _dispatch_queue_invoke + 1109
7   libdispatch.dylib                   0x04be65e4 _dispatch_root_queue_drain + 470
8   libdispatch.dylib                   0x04be63a6 _dispatch_worker_thread3 + 143
9   libsystem_pthread.dylib             0x04f85fdc _pthread_wqthread + 1356
10  libsystem_pthread.dylib             0x04f85a6a start_wqthread + 34
[4]  Relevant Code:
Firebase.configure()
GADMobileAds.configure(...)

@Alexeichua Please share the project via github or a zipfile

database.zip

Running the DatabaseExampleSwift scheme doesn't crash for me on an iOS 10.0 iPhone 6s Plus simulator.  We'll have to investigate more deeply to understand what's different about your configuration.

This gives me a little positivity. I've tested on iPhone 6s iOS 10.3.1 and iPhone 5 iOS 10.2 simulators.

iPhone 5 iOS 10.2 succeeds for me:


Added support for CAN FD in monitor.py cli tool

Added a cli flag to monitor.py exposing the fd boolean argument in python-can's Bus constructor. Enables use of the monitor tool with CAN FD buses, for python-can bus interfaces that support it.

Coverage decreased (-0.5%) to 96.357% when pulling fee7b6b7c46d6af0d12a3050912ded64575b43bf on sfpetersen:master into 21769982f78847560f5223816c77898df8558bf9 on eerimoq:master.

This is now part of release 35.5.0, available on PyPI!


GCC can't find libstdc++ headers when cross-compiling back to same platform

Describe the bug
When cross-compiling back to the same platform, such as with pkgsLLVM.buildPackages.gcc, GCC can't find its own libstdc++ headers. If I look at pkgsLLVM.buildPackages.gcc.cc the include/ folder is empty and the libstdc++ headers are instead found in x86_64-unknown-linux-gnu/include/c++/12.3.0/. However, GCC doesn't actually look there. If I ask GCC to print its include paths with echo | result/bin/x86_64-unknown-linux-gnu-g++ -xc++ -E -v - it says
ignoring nonexistent directory "/nix/store/5wpf3nsdgdqfg2kxwb605sjc4gjqxi1h-x86_64-unknown-linux-gnu-stage-final-gcc-12.3.0/lib/gcc/x86_64-unknown-linux-gnu/12.3.0/../../../../include/c++/12.3.0"
ignoring nonexistent directory "/nix/store/5wpf3nsdgdqfg2kxwb605sjc4gjqxi1h-x86_64-unknown-linux-gnu-stage-final-gcc-12.3.0/lib/gcc/x86_64-unknown-linux-gnu/12.3.0/../../../../include/c++/12.3.0/x86_64-unknown-linux-gnu"
ignoring nonexistent directory "/nix/store/5wpf3nsdgdqfg2kxwb605sjc4gjqxi1h-x86_64-unknown-linux-gnu-stage-final-gcc-12.3.0/lib/gcc/x86_64-unknown-linux-gnu/12.3.0/../../../../include/c++/12.3.0/backward"
#include "..." search starts here:
#include <...> search starts here:
 /nix/store/5wpf3nsdgdqfg2kxwb605sjc4gjqxi1h-x86_64-unknown-linux-gnu-stage-final-gcc-12.3.0/lib/gcc/x86_64-unknown-linux-gnu/12.3.0/include
 /nix/store/5wpf3nsdgdqfg2kxwb605sjc4gjqxi1h-x86_64-unknown-linux-gnu-stage-final-gcc-12.3.0/include
 /nix/store/5wpf3nsdgdqfg2kxwb605sjc4gjqxi1h-x86_64-unknown-linux-gnu-stage-final-gcc-12.3.0/lib/gcc/x86_64-unknown-linux-gnu/12.3.0/include-fixed
 /nix/store/5wpf3nsdgdqfg2kxwb605sjc4gjqxi1h-x86_64-unknown-linux-gnu-stage-final-gcc-12.3.0/lib/gcc/x86_64-unknown-linux-gnu/12.3.0/../../../../x86_64-unknown-linux-gnu/include
 /nix/store/c35ndrdaral7brqky6nrgkyni6aic5ar-glibc-x86_64-unknown-linux-gnu-2.37-8-dev/include
End of search list.

When I compare this with a non-cross-compiled gcc the headers are all in include/c++/12.3.0/ and the same command prints
ignoring nonexistent directory "/nix/store/n8dryz4xf7ln028j37zapgzg4j47p743-gcc-12.3.0/lib/gcc/x86_64-unknown-linux-gnu/12.3.0/../../../../x86_64-unknown-linux-gnu/include"
#include "..." search starts here:
#include <...> search starts here:
 /nix/store/n8dryz4xf7ln028j37zapgzg4j47p743-gcc-12.3.0/lib/gcc/x86_64-unknown-linux-gnu/12.3.0/../../../../include/c++/12.3.0
 /nix/store/n8dryz4xf7ln028j37zapgzg4j47p743-gcc-12.3.0/lib/gcc/x86_64-unknown-linux-gnu/12.3.0/../../../../include/c++/12.3.0/x86_64-unknown-linux-gnu
 /nix/store/n8dryz4xf7ln028j37zapgzg4j47p743-gcc-12.3.0/lib/gcc/x86_64-unknown-linux-gnu/12.3.0/../../../../include/c++/12.3.0/backward
 /nix/store/n8dryz4xf7ln028j37zapgzg4j47p743-gcc-12.3.0/lib/gcc/x86_64-unknown-linux-gnu/12.3.0/include
 /nix/store/n8dryz4xf7ln028j37zapgzg4j47p743-gcc-12.3.0/include
 /nix/store/n8dryz4xf7ln028j37zapgzg4j47p743-gcc-12.3.0/lib/gcc/x86_64-unknown-linux-gnu/12.3.0/include-fixed
 /nix/store/4lap45ifj50lp980id60fnbvs81yj9lr-glibc-2.37-8-dev/include
End of search list.

Steps To Reproduce
Working example:
echo '#include <iostream>' | nix shell nixpkgs/5ba549eafcf3e33405e5f66decd1a72356632b96#gcc.cc -c x86_64-unknown-linux-gnu-g++ -xc++ -E -

That should print the contents of the iostream header.
Broken example:
echo '#include <iostream>' | nix shell nixpkgs/5ba549eafcf3e33405e5f66decd1a72356632b96#pkgsLLVM.buildPackages.gcc.cc -c x86_64-unknown-linux-gnu-g++ -xc++ -E -

This produces the following:
# 0 "<stdin>"
# 0 "<built-in>"
# 0 "<command-line>"
# 1 "/nix/store/c35ndrdaral7brqky6nrgkyni6aic5ar-glibc-x86_64-unknown-linux-gnu-2.37-8-dev/include/stdc-predef.h" 1 3 4
# 0 "<command-line>" 2
# 1 "<stdin>"
<stdin>:1:10: fatal error: iostream: No such file or directory
compilation terminated.

Additional context
I don't know if the problem is that the headers are in the wrong place, or if GCC is looking in the wrong place. It seems reasonable for GCC to conclude that because the target platform config matches the host platform config that it doesn't need the target-prefixed path, but I haven't managed to find any code on the Nix side that moves the includes so I think the GCC build process is putting them there and then forgetting about it.
My example here uses pkgsLLVM for convenience, but in my actual code I'm using cross-compiling with a different dummy crossSystem key in order to replace glibc using crossOverlays.
Notify maintainers
@Synthetica9 @vcunat @Ericson2314
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
❯ nix run nixpkgs#nix-info -- -m
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.51, NixOS, 23.11 (Tapir), 23.11.20230919.5ba549e`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.0`
 - nixpkgs: `/etc/nix/inputs/nixpkgs`

I've tested myself by overriding GCC with a new preFixup phase that moves the x86_64-unknown-linux-gnu/include/c++ directory back to include/c++ and now compiling a toy Hello World C++ program works. However there are other sub-folders in the x86_64-unknown-linux-gnu folder and I don't know if any of that stuff is a problem too.

I believe this is fixed by #240435.


Is there any way to disable "try out" feature in flasgger?

I have a REST API made with Flask and documented with flasgger and would like to disable "try out" feature in POST and PUT methods.

Looks like it is not even possible in Swagger UI, you may need to write a JS plugin and then load in the templates/flasgger/index.html
https://github.com/swagger-api/swagger-ui/issues/3725

If you are using ui3, could you just add a custom style to hide it?
.opblock-post .try-out {
  display: none;
}

... and repeat for PUT.
Of course this is only hiding/masking in the UI, but it's all UI anyway so you may need to secure those operations properly if you really don't want them called.

@fbiopereira let me know if some custom CSS as suggested above works for you.

Yes we tried and it worked.
But we decided to keep try out enabled for an while.

De: Brett Randall<EMAIL_ADDRESS>Enviado: terça-feira, 4 de setembro de 2018 20:36
Para: rochacbruno/flasgger
Cc: fbiopereira; Mention
Assunto: Re: [rochacbruno/flasgger] Is there any way to disable "try out" feature in flasgger? (#233)
@fbiopereirahttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ffbiopereira&data=02|01||2b1e321a5b744510938108d612bf4d4f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636717010171129237&sdata=LSl%2B4CLMnYMgRUh16d%2BBWGBT743Flwep6DD4DBFhbys%3D&reserved=0 let me know if some custom CSS as suggested above works for you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frochacbruno%2Fflasgger%2Fissues%2F233%23issuecomment-418551424&data=02|01||2b1e321a5b744510938108d612bf4d4f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636717010171129237&sdata=WVwbOpVkl5Ypsz3Tps6UPSMiMyIJtCnxdVrTkzqOBjY%3D&reserved=0, or mute the threadhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAozPkD2_vo9fT7tz3i4EkVlr5XTR_gABks5uXw6YgaJpZM4WQJh6&data=02|01||2b1e321a5b744510938108d612bf4d4f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636717010171129237&sdata=IGAJYjvNkgh2hhC3inor5Tfm8ZGnYqrpmyK0IfZIl60%3D&reserved=0.

@fbiopereira @rochacbruno @javabrett
You can specify which methods to show the "Try it out" button by  using "supportedSubmitMethods" attribute as documented in swaggerConfigDoc
Go to your flasgger library templates installed path, e.g:
C:\Python27\Lib\site-packages\flasgger\ui3\templates\flasgger
Edit swagger.html:

window.onload = function() {
...
// Build a system
const ui = SwaggerUIBundle(
Object.assign(
{
url: "{{ specs[0]['url'] }}",
dom_id: '#swagger-ui',
supportedSubmitMethods: ['get'],
...

Just add the line supportedSubmitMethods: ['get'], inside Object.assign, like in my example, is below dom_id attribute.
You can add more methods to display the "Try it out" button besides 'get', e.g.
supportedSubmitMethods: ['get', 'fetch'],
Cheers.

@IreneMaryanne thanks a lot!


Add a ? operator for FTS

Instead of naked strings implying FTS, we should have an operator for the expressions to avoid confusion, especially now that most backends won't support it. (Currently, only elastic and gmail adapters support FTS.)

[ ] Add spec / design doc (move content from Confluence)
[ ] Modify grammar
[ ] Update docs
[ ] Update examples

Copying internal proposal written by @demmer here:
Option 1: Unary Search Operator
Today a standalone string term in a filter expression is interpreted as a full text search for that term across all fields. This design was motivated by comparisons to the splunk language as well as the fact that we wanted to emphasize the search capability. However it is somewhat non-intuitive for a new user, so this proposal would the ? operator to indicate search.
For example the following would search for the string "error" in all fields and map the value "syslog" in the "appname" field:
read elastic ? 'error' appname='syslog'

Searching for multiple terms would require multiple ? operators:
read elastic ? 'error' ? 'host123' appname='syslog'

Option 2: Binary Search Operator:
Internally, FTS is implemented using the magic _all field in elasticsearch, which tells it to search across all fields and not just a specific field.
We could mimic those semantics in Juttle and make search a binary operator instead of a unary operator, and pass through _all as the hook to enable current behavior. For example:
read elastic _all?'error' appname='syslog'

If we were to do this ?= reads slightly better:
read elastic _all?='error' appname='syslog'

At that point it would be trivial to support FTS on a single field (which we can't do today):
read elastic message?='error' appname='syslog'

On balance it seems like the unary operator approach is cleaner than the _all magic field.

@demmer I’d like to discuss this tomorrow and ideally arrive to a resolution on the design so I can start implementing. Beside being important by itself I’d like to do this before I dive into #57.

The result of discussion with @demmer is that we’ll implement Option 1, but with ?= as the operator:
read elastic ?= 'error' appname='syslog'

After implementing this change (#232) me + @demmer agreed the new syntax looks ugly and we decided to scrap the whole idea of the FTS operator.


Improve wording on "some keybindings are dispatched to the workbench by default" notification.

I'm running QPs today and one of the participants had never used VS Code before. He saw the following notification when he opened the terminal:

He was very confused by what it meant (and so am I tbh as English is not my first language 😅), and he mentioned this was one of the low points of his experience with VS Code. I was wondering if there's a way the wording there can be improved so that less experienced/advanced users understand what is happening?
Thanks!

Yeah "workbench" is kind of an advanced term. @luabud how about something like this?

Some keybindings don't go to the terminal by default and are handled by  instead.

Suggestions welcome 🙂

I am commenting because I just saw this warning and was confused, so agree with @luabud.
@Tyriar - How about going a step further - For the Configure Terminal Settings piece, why not suggest/explain which setting addresses this keybinding issue? Even though Settings in this case offers two check boxes (one checked, one not checked in my case when opened), it is not 100% clear which of the two applies to this issue.

@chocaloca all 3 settings are relevant though, we try to keep notifications as small as possible as they're already annoying enough without a wall of text. The idea was to say "hey this is a thing btw" to allow users interested in configuring it to go to the right place, previously we did it by default and some users were confused by certain shell keybindings weren't working for them.

I do like your suggestion!

👍


[OpenShift] Support Thread

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
As part of #1286 ppl asked if I could share my config to get Uptime Kuma running on OpenShift.
So to provide a starting point for everyone interested this "question"/help issue now exists.
So for various reasons I ended up creating my own Docker image based on the official uptime-kuma image, but long story short:

I needed to for work (due to base image policy)
I wanted to have full control over how the image and configure how the permissions are set (according to the RedHat Docs)

Containerfile
FROM docker.io/louislam/uptime-kuma:1.11.4-debian AS app-src

FROM docker.io/node:14-bullseye-slim

ARG UID=3310
ARG GID=3310

COPY --from=app-src /app /app

RUN apt-get update -qqy \
    && apt-get install --no-install-recommends -qqy \
        python3 python3-pip python3-cryptography python3-six python3-yaml python3-click python3-markdown python3-requests python3-requests-oauthlib \
        sqlite3 iputils-ping util-linux dumb-init wget curl ca-certificates bash \
    && pip3 --no-cache-dir install apprise==0.9.7 \
    && rm -rf /var/lib/apt/lists/* \
    \
    && addgroup --system --gid ${GID} uptime-kuma || true \
    && adduser --system --disabled-login --ingroup uptime-kuma --no-create-home --home /nonexistent --gecos "uptime-kuma" --shell /bin/false --uid ${UID} uptime-kuma || true \
    && mkdir /app/data \
    && chown -R uptime-kuma:0 /app \
    && chmod -R g=u /app

WORKDIR /app
USER uptime-kuma
EXPOSE 3001
VOLUME ["/app/data"]
ENTRYPOINT ["/usr/bin/dumb-init", "--"]
CMD ["node", "server/server.js"]

Note:

Yes this is a tat outdated but the basics should work with new versions too.
Yes this version may copy over the entrypoint.sh as part of copying the /app folder, but am not using it. This is  due to OpenShift not permitting running as root/privileged user unless explicitly permitted and the script relying on that to change permissions on the persistent files.

Deployment
I hacked together a very basic Helm (3.x+) Chart to deploy a StatefullSet to our OpenShift cluster with a service and a Route. You can find that here: https://github.com/k3rnelpan1c-dev/uptime-kuma-helm
I will try and answer any questions and feedback to my best abilities and as time allows. If this issue violates the contribution guidelines in any way that I overlooked then please feel free to close this and my apologies in that case.
🐻 Uptime-Kuma Version
1.11.4
💻 Operating System and Arch
OpenShift 4.x
🌐 Browser
Firefox 99
🐋 Docker Version
cri-o
🟩 NodeJS Version
14.19.1

Is there now a fixed version of uptime-kuma docker image? If not, could you @k3rnelpan1c-dev share the one you built?

Well, I did share the Containerfile that I used to build my OpenShift compatible image in the issue text :thinking:
But I might as well setup a tiny GitHub action CI in my linked PoC Helm Chart repo to build it :man_shrugging:

Hi,
We also built this finally, but getting websocket errors like:
Cannot connect to the socket server. [Error: timeout] Reconnecting...
our service on OpenShift level is like
kind: Service
apiVersion: v1
metadata:
  name: example
  namespace: uptime-kuma
spec:
  ports:
    - name: webservice
      protocol: TCP
      port: 80
      targetPort: webservice
  selector:
    app: uptime-kuma
  clusterIP: xxxxx
  clusterIPs:
    - xxxxx
  type: ClusterIP
  sessionAffinity: None
  ipFamilies:
    - IPv4
  ipFamilyPolicy: SingleStack

and route is like
kind: Route
apiVersion: route.openshift.io/v1
metadata:
  name: uptime-kuma
  namespace: uptime-kuma
spec:
  host: uptime-kuma.xxxxx
  to:
    kind: Service
    name: example
    weight: 100
  port:
    targetPort: webservice
  wildcardPolicy: None

What could be wrong?

Okay, where exactly do you get [Error: timeout] Reconnecting... when trying to connect to uptime-kuma or with uptime-kuma trying to ping som other service?

 Sorry, this question could be deleted.

Ah yes proxy issues, my favourite :sweat_smile:
btw. hacked something together in terms of CI and with an updated Kuma version:

Source
Quay

@louislam, hope you are alright with me repackaging the image this way, if not please let me know and I will purge its existence but keep the sources available.

It is ok.
If you think it is stable, you could also add it into How to Install section as an unofficial way by creating a pull request.
https://github.com/louislam/uptime-kuma-wiki/blob/master/🔧-How-to-Install.md

Thank you :slightly_smiling_face:
I will keep on testing a bit more and will then give it another thought.
Depending on how stable my repackage CI is going to be I may omit my repackage in that though, as I want to avoid guiding ppl to an install path that I would not be able to maintain (even if it is just unofficial).
Thank you for this great tool btw :+1:

well, went ahead and finally opened https://github.com/louislam/uptime-kuma-wiki/pull/11, as I am happy with my automated workflow to re-bundle.
Additionally, it helps everyone wanting/needing a rootless uptime-kuma image, as it is implicitly required by OpenShift.

if I can dare to advise, it is possible to simplify the part relating to the assignment of permissions. In fact, you just need to add the following without the creation of a new user/group.
RUN chgrp -R 0 directory && \
        chmod -R g=u directory

USER 1001

Explanation from the doc: The user account that runs the container is always a member of the root group, hence the container can read and write to this directory. The root group does not have any special permissions (unlike the root user) which minimizes security risks with this configuration. The g=u argument from the chmod command makes the group permissions equal to the owner user permissions, which by default are read and write. You can use the g+rwX argument with the same results. The user 1001 instead is the Red Hat convention.
I modified the Dockerfile accordingly and works like a charm.

overall you right, but one thing that I observed (not with uptime IIRC, but other NodeJS and Python Apps) is that they sometimes rely on a writable home directory, which is an easy fix if you set the envVar HOME to a volume or emptyDir.
However, I generally added the extra user for other container runtimes that don't act like OpenShift (running as arbitrary UID) so they have a user to their UID uptime runs as. (is it overkill, maybe :man_shrugging:, but it works)

@k3rnelpan1c-dev's Container Image works perfectly for me on OKD/OCP 4.11 with the Restricted-v2 SCC.
Ping doesn't work, but I don't use it anyways.
      securityContext:
        capabilities:
          drop:
            - ALL
        runAsUser: N/A in Openshift
        runAsNonRoot: true
        readOnlyRootFilesystem: true
        allowPrivilegeEscalation: false
        seccompProfile:
          type: RuntimeDefault

@k3rnelpan1c-dev
There don't seem to remain any open issues in this issue, am I correct?
If yes, don't forget to close it.
(Only you or Louis can do so. Otherwise, this will become a zombie that never dies)

ak, right, sorry. thank everyone who chimed in :+1:


Allow Named Assets to Begin with A; Allow Longer Names

This is a proposed protocol change:

From a specified future block, interpret new registrations with ID >= 1.82e19 as named assets beginning with A.

This implies that new registrations above that threshold no longer become numeric assets.
Names will get mapped with base 27 (b27_digits = ' ABCDEFGHIJKLMNOPQRSTUVWXYZ'). If a   (space) is detected, the registration is invalid. If the first character is not A, the registration is invalid.
JS functions for converting between ID and name:
 ` function asset_name_b27(id) {
    const k = BigInt(1.82e19);
    let b27_digits = ' ABCDEFGHIJKLMNOPQRSTUVWXYZ';
    let n = BigInt(id);
    let name = '';
    n -= k;
    do {
      let mod = n % 27n;
      name = b27_digits[mod] + name;
      n -= mod;
      n /= 27n;
    } while (n > 0);
    return name;
  }

  function asset_id_b27(name) {
    const k = BigInt(1.82e19);
    let b27_digits = ' ABCDEFGHIJKLMNOPQRSTUVWXYZ';
    let name_array = asset_name.split("");
    let n_bigint = BigInt(0);
    for (i = 0; i < name_array.length; i++) {
      n_bigint *= BigInt(27);
      n_bigint += BigInt(b27_digits.indexOf(name_array[i]));
    }
    n_bigint += k;
    return n_bigint.toString();
  }`

Note that the largest A-name (assuming max 12 chars long; ZZZZZZZZZZZZ) will get ID ~ 1.84e19, slightly below the theoretical maximum. We can of course decide on a different k to have ZZZZZZZZZZZZ match the exact maximum, or to minimize collisions with existing numeric assets, etc.
Also note that the above functions are very similar to how names have been registered (and will continue to be registered). The differences are k = 0 and base 26. With b26 it just happens that names cannot begin with A.

This is an interesting proposal. Thank you for taking the time to write it out, including all the technical details :)
Unfortunately, at this time I oppose this change as I feel it breaks the fundamental rules that were established when the name system was created.

No names that start with A
4 Letters is shortest allowed asset

If users are now able to register A names, or register shorter names, I feel it essentially screws over many of the long-time counterparty users who registered X names like XAPPLE with the understanding that there would never be any asset names starting with A allowed.
While we may be able to do this from a technical perspective, I object to this change, as I feel it changes the fundamental rules of asset names, and is unfair to existing users and asset holders.

No names that start with A
4 Letters is shortest allowed asset

I agree @jdogresorg. This is basic asset name validation logic for Counterparty applications. 3 letter assets are the protocol ones (so, I wouldn't mind adding another 3 letter asset if it has special functionality like XCP does). And A assets serve as clear indication these are either numeric or longnames.
Relevant related discussion: https://github.com/CounterpartyXCP/cips/discussions/87.

I think that there is alot of value in expanding the asset names.  Adding A helps, but doesn't really solve the issue that the 4-12 latin character namespace is a severe hindrance on global adoption.
Another way to do this that could be to create a special purpose named asset called cp (or something else, does't really matter), that allows any user (not just owners) to create a subasset. Then these assets are not part of the original namespace and this new namespace is much broader.  So much so, that creating any number of metaprotocols with this namespace is possible. The simplest of which would be a fully functional global nameservice utilizing punycode.
This type of change would set of a massive land grab and lead to alot of interest and new users entering the counterparty ecosystem.
If there is a major concern with squatting on these assets, a dutch auction in xcp starting at something like 10-20xcp and reducing the cost by 0.01% could allow for a slow phase in of a year or so?

Then these assets are not part of the original namespace and this new namespace is much broader. So much so, that creating any number of metaprotocols with this namespace is possible. The simplest of which would be a fully functional global nameservice utilizing punycode.

If we decide to go the domain route, why not implement a Namecoin-like strategy on Bitcoin using Counterparty with an expiration mechanism for these new 'domains' to allow for more experimentation?

If there is a major concern with squatting on these assets, a dutch auction in xcp starting at something like 10-20xcp and reducing the cost by 0.01% per block could allow for a slow phase in of a year or so?

The expiring domains won't appeal to the collectors' market, and requiring renewal using XCP will help prevent squatting.
Meanwhile, the existing asset class will remain unchanged—set in stone, without unnecessary modifications—as collectibles must be treated with caution and care because their value hinges on their history, and any unwarranted alterations could prove disastrous to their worth.


CFPB_2Tone_isolated_RGB_352 is unused

This image https://github.com/cfpb/cfgov-refresh/blob/flapjack/static/img/CFPB_2Tone_isolated_RGB_352.png appears to be unused. Can it be deleted? cc @Scotchester

I can't remember what this might've been used for. I suppose if it's not being referenced anywhere, might as well delete it.


BasicAuth HTTP filter: emit metadata containing username

Title: Emit metadata with username after successfully authenticated
Description:
Once a request has been successfully authenticated using the BasicAuth filter, emit metadata containing username. The username can be used by the subsequent filters, such as the RBAC filter for authz.

@zhaohuabing Side note: Please add BasicAuth in https://github.com/envoyproxy/envoy/blob/main/CODEOWNERS
Looks like you and @wbpcode are the expert in this filter

SGTM. :)


⚠ API - Lens has degraded performance

In 8f41f81, API - Lens ($LENS_API_URL) experienced degraded performance:

HTTP code: 503
Response time: 595 ms

Resolved: API - Lens performance has improved in 9d7d366 after 7 minutes.


Add precommit

Successor of #4, after the repo visibility has changed.

This doesn't seem to work on macOS, filepath is relative to the root of the repo while notebook_tutorials_dir is absolute.
I had the same problem in Haystack, I think we can hardcode paths here and use relatives everywhere.

@masci In this case I made a first attempt to make pathlib work with absolute paths...
Could you try it in macOS?


Add Latex support

I added latex support that renders latex during build of site so it does not bloat the site with javascript.

Hey thanks for contributing! This is a great addition, but I don't think everyone would utilise it. For some it will be useful for displaying formulas on their blog, but not everyone is doing that and for most it'll just add bloat and degrade the performance of their blog. For that reason I'm going to close this. Hope you understand!

Understandable. Have a nice day.


refactorize the whole test suite

Make a call to assert_internally_consistent automatically when snapshotting the scheduler. There is no point in snapshotting something broken and expecting the dumb humans to notice.
Replace every possible call to assert_internally_consistent with a snapshot of the scheduler. It uses the same amount of lines and ensures we never change something without noticing in any tests ever.
Name every snapshot: it's easier to debug when something goes wrong and easier to review in general.
Stop skipping breakpoints; it's too easy to miss something. Now you must explicitly show which path the scheduler is supposed to use.
Add a timeout on the channel.recv, it eases the process of writing tests; now, when something files, you get a failure instead of a deadlock.

@irevoire the tests fail

bors merge


Debian image openssl vulnerability?

It seems that a Dockerfile used to build an image FROM python:3.5 i.e. python:latest will be using a debian image with a vulnerable openssl.
I don't understand precisely why, as an image I built yesterday FROM debian:jessie uses the more recent openssl but an image I built today FROM python:3.5 seems to use an older one.
python:3.5 -> buildpack-deps:jessie -> buildpack-deps:jessie-scm -> buildpack-deps:jessie-curl -> debian:jessie
Is an outdated upstream debian:jessie image being inadvertently cached perhaps?
This arose from a vulnerability warning I received from a python:3.5-based  image I pushed to my registry at IBM Bluemix Containers.

Security Notice: DSA-3673-2
Package        : openssl
Debian Bug     :<PHONE_NUMBER>59
It was discovered that the original patch applied for CVE-2016-2182 in
DSA-3673-1 was incomplete, causing a regression when parsing
certificates. Updated packages are now available to address this
problem.
For the stable distribution (jessie), this problem has been fixed in
version 1.0.1t-1+deb8u5.
We recommend that you upgrade your openssl packages.

This was fixed with https://github.com/docker-library/official-images/pull/2179.
 It contains the general-purpose command line binary /usr/bin/openssl,
 useful for cryptographic operations such as:
  * creating RSA, DH, and DSA key parameters;
  * creating X.509 certificates, CSRs, and CRLs;
  * calculating message digests;
  * encrypting and decrypting with ciphers;
  * testing SSL/TLS clients and servers;
  * handling S/MIME signed or encrypted mail.
root@253a43173a3c:/#

Perhaps you have an older python:3.5 image and need to pull a new one?  Just pulling a new debian:jessie won't update your images that are from it.

Oh good, that's reassuring that the official python is up-to-date .... a bit worrying, though, that it is so easy for me  to have stale basis images.


DrawVisibleBounds

Hi @joshbduncan ,
I haven't seen a bug in getVisibleBounds() in a long time. In the FOR loop that finds the mask in the clipping group (119 line).
} else {
          clippedItem = curItem;
          break;
        }

This ELSE will break the FOR loop on the first element, and there may be cases where the mask is moved in the hierarchy and then the function returns the dimensions of the top object instead of the mask =).

Hmm, I never really thought about someone moving the clipping path in the layer stack. I'll dig into this when I have time and try to sort out a solution. Thanks for letting me know!

For example, https://github.com/nimbling/Nimbling_Scripts#clip-scripts scripts, when drawing with a script, the bottom object is assigned a mask and remains the bottom object in the stack.That is, cases where the mask is not only at the top of the group are real :)

So, I think that piece of code was actually a relic from an early version of the script. Seems that if the script actually makes it to that part then something funky is going on and in that case, I'll just return the visible bounds.

So, it turns out that the last else statement was needed. For example, if a clipped group was then grouped again, the script were produce incorrect results. I added a catch to see if the clippedItem is undefined after iterating over all of a clipped group pageItems, and if so, the function just sets the clipped item into the original curItem.

I changed the function code in a similar way on Sunday when I needed to get bounds in a script. It's an acceptable solution if we don't find any more complicated cases =)

Oh, I'm sure there are plenty more cases that would break this script. There are so many junk files out there, especially ones exported to .ai and .pdf from other software. I'll close this for now. Thanks again for your help!

So, I found something interesting... If you move the from the first position in the layer stack like pic above, then save and reopen the file, Ai actually moves it back to the correct spot. The clipping isn't moved to you actually close the file, so it seems that Ai does some cleanup when opening files (tested with both .ai and .pdf).

Wow. I didn't know about this because I don't use masks much in my work. And a friend said he knew about this feature in Illustrator. In any case, the recursive function makes sense, because the user can create a group in the document with the mask moved and run the script when the mask is not yet on top :)

Yes, your bug issue definitely needed to be fixed. I could see many situations where a file being worked on had unordered clipping mask layers. I was just surprised to see Ai actually changing a file during the open process and wanted to share it. Thanks again!


Removing nullability possibilities.

Simplifying code by removing the possibility of LastModified being null when a hash file is found.

Codecov Report

Merging #46 (d4b352c) into main (6a07527) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main      #46   +/-   ##
=======================================
  Coverage   17.91%   17.91%
=======================================
  Files           5        5
  Lines          67       67
=======================================
  Hits           12       12
  Misses         55       55

Impacted Files
Coverage Δ

Functions/BlobStorageEntry.cs
0.00% <0.00%> (ø)

Functions/Program.cs
0.00% <0.00%> (ø)

Functions/Range.cs
0.00% <0.00%> (ø)

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 6a07527...d4b352c. Read the comment docs.


Update to cursive 0.15

Thanks for the work! The library has now been updated.

Thanks for the work! The library has now been updated.


Only getting 404 responses

Routes do not seem to be registered using my version of node & yarn. Running the app works and there is no critical error on startup, only a warning (see screenshot below).
However none of the routes return anything but a 404 Not Found!
Please include supported versions of node and yarn into the readme (if they are the root cause)
I am running it in Docker using this Dockerfile.
Operating System: Debian
Node Version: 18.9.0
Yarn Version: 1.22.19
Everything in a screenshot:

(5100 on host is mapped to 5001 in the container)

@PoolPirate this happened to me simply because I forgot to append "/callbacks/crm" on the Twilio Frontline config page.


It doesn't work anymore (?)

Hi, I love your extension.
However it seems that it stopped working today, maybe something changed in Spotify Web Player?
I'm using Firefox.
Thanks!

Hey, thanks for reporting!
I'm sorry to hear that, but unfortunately I won't be able to take a proper look at this because I don't have any computer or laptop available right now... :(
I'll do my best to fix it asap at the beginning of next week, though.
Thanks

Hello,
I'm having the same issue as Oneiros90.
I'm seeing the issues with Google Chrome ver. 89.0.4389.90 (Official Build) (64-bit) on Windows 10 Home.
Looks like chrome had some major changes with their recent updates. Could be that it's a simple change on their part that's effecting how your extension works.  I'm hoping that it's really simple to find and adjust for for when you have the ability to. Really appreciate this extension.

If that's of any help, can also confirm it also stopped working for me properly since a day or two ago.
Firefox 86.0 (64-bit) on Ubuntu 20.04

Same problem with Google Chrome ver. 89.0.4389.90.

Same problem here with Firefox 86.0.1 on MacOS.

I'm also having the same issue (both on a mac and windows computer. To try to help with troubleshooting when you get time to look at this, I'm attaching a screen shot of the 'web console' messages when the add on tries to activate:

Hope this gets worked out soon, I really like this add on!

Hi folks!
I've tested this new change for a couple of hours on Firefox/Chrome and it seems to be working like it used to.
So I'll be releasing version 2021.3.23 very soon in both Firefox Addons and Chrome Webstore very soon, so don't forget to update before testing it out. :)
And as always, if you guys find any issues, feel free to post them here or create a new issue.
Thanks


Always return an array from getInstancesByAppId

It makes sense to return an array all the time as it allows
consumers to do stuff like client.getInstancesByAppId('id').map(...).filter(...)
without error checking first.
Error checking can still be done by checking instances.length !== 0.

As this is coded now, it breaks the module. It is throwing a re-assignment to const error. To get around it you could do this instead:
const instances = this.cache.app[appId.toUpperCase()] || [];
if (!instances.length) {
  this.logger.warn(`Unable to retrieve instances for appId: ${appId}`);
}

Could a convenience function be added, something like getInstancesArrayByAppId, and then deprecate the legacy call? Just to avoid breaking backwards compatibility.

Of course. I edited inline on github and didn't have a linter available.

Thanks noseglid! While it is a minor change, there are people using this in production, and I know that some of the implementations have the potential to break, and causing a random dependency bug in a production AWS setting, sounds like a great way to ruin any dev's day :).
A couple additional changes that would have to be made before this is fully ready for merge:

[ ] change getInstancesByVipAddress to have the same contract
[ ] Fix tests for getInstancesByVipAddress and getInstancesByAppId

The idea of providing  a convenience function getInstancesArrayByAppId is an interesting one. Ideally, it would be called something else because that particular name implies the other function doesn't return an array ever, and it makes an already long function name longer.

Coverage decreased (-0.7%) to 93.502% when pulling 15c214a547c26d314cfb5c797e17d3799fe6db5a on noseglid:patch-1 into 6e56e233ce4026ba010a8349ecda431357df91ba on jquatier:master.

Coverage remained the same at 94.224% when pulling 51ecdb851774a2b17609350027c491fcd182479c on noseglid:patch-1 into 6e56e233ce4026ba010a8349ecda431357df91ba on jquatier:master.

Coverage remained the same at 94.224% when pulling a5e4631c8c9a8d7927c5e015b11a943fe842ceca on noseglid:patch-1 into 6e56e233ce4026ba010a8349ecda431357df91ba on jquatier:master.

I gather from your comment that you're fine with a major bump of the version with this change?
I think it's better to do a major bump and keep interfaces clean rather than augmenting and making maintenance challenging due to a lot of redundant interfaces.

I've addressed the comments you had.

Awesome, thanks!
I will push up a v4 branch and we will have to repoint this PR to that branch

I manually merge this PR into this branch:
https://github.com/jquatier/eureka-js-client/tree/v4


Semantic error

MPN mans Manufacturer Part Number.
This is a different meaning to distributor part numbers, such as LCSC Part # (recently encouraged by JLC to be renamed to JLCPCB Part #).
However, in your plugin, about which I learned after watching this MCH video seems to be positioned as the de-facto predominant interface for JLC for Kicad 6, I would recommend best practice approach, ie.

Do not call distributor part numbers 'MPN'
Realise that these fields represent different information and may both be present
In a situation in which more than one field is present, clarify in documentation which fields will be used in which order.

Given that so many people come to KiCad as a first EDA experience I believe it is extra important to get this right.
As I have not yet upgraded to KiCad 6.x from KiCad 5.1.x (mostly due to the huge amount of custom tooling in use) I regret that I do not have a test environment or familiarity to provide a pull request myself.

Thank you for letting me know,
I guess I could try and search for all the unique fields first and if they are not preset, fallback to the MPN field. (even due that they are means different things this might help with the component selection).
Does the sounds good for you?

Ended up implementing it the following way (for simplicity):

Primary fields: LCSC Part #, JLCPCB Part #

Failing the above (correct) fields, the fields LCSC or JLC in capitals could be used.

Failing any of those, the field MPN in capitals could be used.

Following those, the fields MPN in any other case could be used.


[communication-identity] [Reland] Add methods to translate CommunicationIdentifier to / from a stable string format

This PR relands #25668 (with a minor fix on top from #24460)
Previous PR was reverted in order to allow an unrelated release from main while the API in this PR was under active API council review (across all languages).
Description
This PR address the API review council feedback on this feature branch.
The changes are minimial:

Add a previously missing type in the package exports.
Add type annotations to newly added functions.

All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-communication-identity


Cannot run .sql file on mysql

System information:

Operating system (distribution) and version : Win10
DBeaver version 6.1.2
Java version java version "1.8.0_221" Java(TM) SE Runtime Environment (build 1.8.0_221-b11)
Java HotSpot(TM) 64-Bit Server VM (build 25.221-b11, mixed mode)

Connection specification:

Database name and version mysql 8 - MariaDB JDBC driver
Driver name org.mariadb.jdbc.Driver
Do you use tunnels or proxies (SSH, SOCKS, etc)?  no

I wrote a .sql file that create a store procedure on server and run it.
If I run this file under heidisql everything works fine, but under DBeaver I cannot run...
DELIMITER $$
use placetransfer;

DROP PROCEDURE migrar_db$$
CREATE PROCEDURE migrar_db()
	BEGIN
		/* Configuracao das variaveis para uso em todo o script !!!! */
		declare v_id_loja int default 7;

		create table migrar_log (ID int NOT NULL AUTO_INCREMENT, hora DATETIME NULL, texto VARCHAR(10000) NULL, PRIMARY KEY (ID));
		insert into migrar_log(hora,texto) values (now(),concat('Script iniciado'));
		insert into migrar_log(hora,texto) values (now(),concat('ID da loja : ', v_id_loja));

		show tables like 'tbl_vitrine';
		select found_rows() into v_num_rows;
		if (v_num_rows > 0) then
			insert into migrar_log(hora,texto) values (now(),'Importando tabela de vitrine');
			delete from tbl_vitrine where id_loja <> v_id_loja;
			insert into place.tbl_vitrine select * from tbl_vitrine;
		end if;

		insert into migrar_log(hora,texto) values (now(),'Script concluido');
	END $$
DELIMITER ;

call migrar_db();
select * from migrar_log;

This is a small piece of code, if I run, everything ok but on DBeaver:
SQL Error [1064] [42000]: (conn:166279) You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'DROP PROCEDURE migrar_db' at line 3
Query is : use placetransfer;

You have a wrong delimiter after use placetransfer; - you must use $$ as delimiter as you redefined it one line earlier..

@serge-rider I understood, but why I can run this on heidisql without errors ?
Considering that run on server, doesn't make sense because the error will be fired on both (heidi/dbeaver)


Crash on large file

i'am trying to open 1 hour of podcast on my phone, event if i puting this code
var wavesurfer = WaveSurfer.create({
    container: '#waveform',
    backend: 'MediaElementWebAudio',
    scrollParent: true,
    waveColor: '#D9DCFF',
    progressColor: '#4353FF',
    cursorColor: '#4353FF',
    barWidth: 2,
    barRadius: 3,
    barHeight: 1,
    barGap: 2
  });
  wavesurfer.load('<%=<EMAIL_ADDRESS>%>');
  wavesurfer.on('ready', function () {
    wavesurfer.play();
  });

my phone crash and i have this information/issue on my dev console
Uncaught TypeError: this.Backend is not a constructor
    value wavesurfer.js:586
    value wavesurfer.js:387
    value wavesurfer.js:230
    <anonymous> hey:692

when is a tiny file, everythink is ok, reading audio file work perfectly, but able to read large file.
need help, thanks
PS: using .opus file as audio format

Which browser?

[x] Chrome Android Browser (Android 7.1, LG V20, crash browser app)
[x] Opera Touch Browser on Android (Android 7.1, LG V20, crash browser app)
[x] Firefox, Opera and EDGE on Windows 10
[x] Same desktop browsers on Linux OS

The phone probably runs out of memory while loading the entire file to draw the waveform.
Sadly the easiest solution here for you is to calculate the waveforms for all your audio ahead of time. So you store the waveform data as .json files on your server, and when you load an audio file you also request the waveform .json and give it to wavesurfer. Then you can use the MediaElement backend which can play big files without any problems.

Which browser?

Android browser, chrome desktop - Linux and windows (Google Chrome, opera browser)


google - ml_video_face_detection

I believe line 12 of serverless-faas-workbench/google/cpu-memory/model_serving/ml_video_face_detection/main.py should be changed from fourcc = cv2.VideoWriter_fourcc(*'XVID') to fourcc = cv2.VideoWriter.fourcc(*'XVID')
That is change the _ to a .

Hi @baddoghaus
Thank you for the report. In the code, we are referring to a case in the following page.
https://www.programcreek.com/python/example/89348/cv2.VideoWriter_fourcc
Do you see any issue that it should be changed to fourcc method?

Hello @kmu-leeky and thanks for the response.
My apologies, I was just able to successfully run my function with the source code you've provided. My true error was a mismatch of video codec formats.

thank you for the update. good to hear that.


wip: v5

Technically usable, but needs a lot of cleanup.
Also some response fields are still interface{}, so might be annoying to use

gonna regret not squashing this in the future but yolo


Icons Not Showing For Associated Files

I recently installed the extension and all the files for which I setup associations are not showing an icon based on the filetype.
Ex.

When I force reloading of the window, I see this small failure pop-up

Cannot read properties of undefined (reading 'label')
Which I'm assuming is some sort of config issue.
This is not keeping the extension from working, but it is annoying and ugly.

Hi @Stuart-Wilcox, thanks for bringing this issue up. We are working on higher prioritized tasks at the moment, but we will get back to this one as soon as possible.

Thanks for the acknowledgment. Is there anything I can do in the meantime to try to debug myself?

No icon is expected (I'm not quite sure how to fix that, it might be up to icon packs to add a relevant icon?).
The label issue is interesting. I'm going to optimistically say that I fixed it in #553 which hasn't been released yet.


⚠ ClamAV has degraded performance

In 09e9521, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 49 ms

Resolved: ClamAV performance has improved in 1e6c739.


[Ubuntu] Rework Сhromium installation

Description
Changing the Chromium installation from apt-get to manually, because even using apt-get install Chromium was installed as a snap package. It led to an issue like mkdir: cannot create directory ‘/run/user/1001’: Permission denied when we tried to use Chromium.
Related issue:
#2967
Check list

[x] Related issue / work item is attached
[x] Documentation is updated (if applicable)
[x] Changes are tested and related VM images are successfully generated

@nikita-bykov @maxim-lobanov I wonder if we should take the version corresponding to the google chrome one as it is described here https://www.chromium.org/getting-involved/download-chromium

@nikita-bykov @maxim-lobanov I wonder if we should take the version corresponding to the google chrome one as it is described here https://www.chromium.org/getting-involved/download-chromium

@miketimofeev @nikita-bykov
I think it makes sense to make sure that Chromium is compatible with ChromeDriver.
Looks like we just need to retrieve installed Chrome version and put it to the url: https://omahaproxy.appspot.com/deps.json?version=44.0.2403.157. This url will return chromium_base_position that we should use instead of LAST CHANGE content.

Is it correct that we will now install Chromium to all Ubuntu versions and not on Ubuntu 20.04 only?

Is it correct that we will now install Chromium to all Ubuntu versions and not on Ubuntu 20.04 only?

We now have enough free space on all Ubuntu images, so maybe we can add Chromium not only on Ubuntu 20. If not, I'll add a condition to install only on Ubuntu 20.

Is it correct that we will now install Chromium to all Ubuntu versions and not on Ubuntu 20.04 only?

We now have enough free space on all Ubuntu images, so maybe we can add Chromium not only on Ubuntu 20. If not, I'll add a condition to install only on Ubuntu 20.

Change the PR title, please

@nikita-bykov @maxim-lobanov we need to change the condition in software report generator since we've decided to add chromium everywhere
https://github.com/actions/virtual-environments/blob/3195add3cdc31759b7f2999b84192c703f2fb620/images/linux/scripts/SoftwareReport/SoftwareReport.Generator.ps1#L220-L222

Please change the condition in software generator
https://github.com/actions/virtual-environments/blob/3195add3cdc31759b7f2999b84192c703f2fb620/images/linux/scripts/SoftwareReport/SoftwareReport.Generator.ps1#L220-L222

The same for tests: https://github.com/actions/virtual-environments/blob/main/images/linux/scripts/tests/Browsers.Tests.ps1#L21

Please change the condition in software generator
https://github.com/actions/virtual-environments/blob/3195add3cdc31759b7f2999b84192c703f2fb620/images/linux/scripts/SoftwareReport/SoftwareReport.Generator.ps1#L220-L222

The same for tests: https://github.com/actions/virtual-environments/blob/main/images/linux/scripts/tests/Browsers.Tests.ps1#L21

it's already done here, don't know why the test moved to the chrome section, but anyway
https://github.com/actions/virtual-environments/pull/3029/files#diff-4c476e8348e741443009d80dc08c7f6b6913320238aae669042cd9da015fe952L21


[Highlight] Dark spaces or ... in cpp code.

I've tried 3 different themes now, each displays the same artifact. A dark space in templates or dark ... in templates.

The problem seems to be spans, which have a dark background set :
<span style="color:#960050;background-color:#1e0010"> </span>
Here is a live example : https://p-groarke.github.io/blog/2018/05/19/declarative-functional-apis-a-k-a-abusing-lambda-parameters/
Note there also seems to be a conflict with that particular theme highlighting.

This is your theme/CSS issue. Also the info you provided is very inadequate.
This issue manager is not for "how-to" questions. For those, use https://discourse.gohugo.io/.

I changed themes 3 times. Also the info you require isn't very clear.

@p-groarke
Please open a topic at the Forum https://discourse.gohugo.io/ and include a link to a repository with the source code of your project so that people can look into your issue.
If you cannot share it as is then include some dummy content.
This is the Hugo bugs issue tracker.
Thanks.


'Uncaught TypeError: Illegal Invocation' when root layout contains body tag

Describe the bug
Recently I migrated an old app (Sveltekit 1/ Svelte 3) to Svelte 5 via the migration scripts. That worked great for the most part, but my app would no longer work correctly. It rendered the page initially but quickly repainted it completely blank. In the console it printed the error "Uncaught TypeError: Illegal Invocation" (see logs).
I found the fix/workaround in a Reddit Thread that had the same issue: a wrapping <body> tag inside the root layout +layout.svelte was causing the issue. Removing this wrapping body element fixed the issue.
The issue occurs both when using slots or snippets. It does not occur when you remove the slot/snippet from the layout.
In my original project it also gave me a hydration_mismatch warning before the Illegal Invocation error, though I wasn't able to reproduce that.
Reproduction
https://github.com/BaumianerNiklas/svelte5-illegal-invocation-repro
This is a barebones project created with the sv CLI, with the only change being the layout that was added.
If you remove the wrapping body tag in there, everything works fine.
Logs
chunk-TFYL3GQY.js?v=4e242971:2346 Uncaught TypeError: Illegal invocation

	in root.svelte

    at get_next_sibling (chunk-TFYL3GQY.js?v=4e242971:2346:30)
    at Module.sibling (chunk-TFYL3GQY.js?v=4e242971:2394:5)
    at _layout (+layout.svelte?t=1734871119624:16:15)
    at chunk-DIK5XZVB.js?v=4e242971:283:58
    at update_reaction (chunk-TFYL3GQY.js?v=4e242971:1308:23)
    at update_effect (chunk-TFYL3GQY.js?v=4e242971:1399:21)
    at create_effect (chunk-TFYL3GQY.js?v=4e242971:2474:7)
    at branch (chunk-TFYL3GQY.js?v=4e242971:2622:10)
    at chunk-DIK5XZVB.js?v=4e242971:277:17
    at update_reaction (chunk-TFYL3GQY.js?v=4e242971:1308:23)
get_next_sibling @ chunk-TFYL3GQY.js?v=4e242971:2346
sibling @ chunk-TFYL3GQY.js?v=4e242971:2394
_layout @ +layout.svelte?t=1734871119624:16
(anonymous) @ chunk-DIK5XZVB.js?v=4e242971:283
update_reaction @ chunk-TFYL3GQY.js?v=4e242971:1308
update_effect @ chunk-TFYL3GQY.js?v=4e242971:1399
create_effect @ chunk-TFYL3GQY.js?v=4e242971:2474
branch @ chunk-TFYL3GQY.js?v=4e242971:2622
(anonymous) @ chunk-DIK5XZVB.js?v=4e242971:277
update_reaction @ chunk-TFYL3GQY.js?v=4e242971:1308
update_effect @ chunk-TFYL3GQY.js?v=4e242971:1399
process_effects @ chunk-TFYL3GQY.js?v=4e242971:1548
flush_queued_root_effects @ chunk-TFYL3GQY.js?v=4e242971:1468
process_deferred @ chunk-TFYL3GQY.js?v=4e242971:1505

System Info
System:
    OS: macOS 14.6.1
    CPU: (6) x64 Intel(R) Core(TM) i5-8500B CPU @ 3.00GHz
    Memory: 6.82 GB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 22.9.0 - /usr/local/bin/node
    npm: 10.8.3 - /usr/local/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Safari: 17.6
  npmPackages:
    @sveltejs/adapter-auto: ^3.0.0 => 3.3.1
    @sveltejs/kit: ^2.0.0 => 2.14.0
    @sveltejs/vite-plugin-svelte: ^4.0.0 => 4.0.4
    svelte: ^5.0.0 => 5.15.0
    vite: ^5.4.11 => 5.4.11

Severity
annoyance
Additional Information
I am not quite certain if this is a bug, an undocumented breaking change, or something else I am missing.
I could not find any documentation that suggested not using body tags in the root layout like this.
The issue happened after the migration from Svelte 4 to Svelte 5 (though I filed the issue in the Kit repo since it has to do with the layout/routing).

You cannot have two body tags. There already is one in src/app.html, one of them has to be removed and the browser will automatically remove the one in the +layout. The missing element then causes an error during hydration because an element was expected where there now is none.

That makes sense, thank you! I guess that was always wrong, it just lead to errors now. Might be worth mentioning somewhere?

Might be worth mentioning somewhere?

Possibly, though the docs are extensive, chances of people missing that would still be high.
Maybe there could be SvelteKit-specific validations that error if a body tag is found anywhere in the components (and if <body> is missing in app.html).


Add progress charts to Release Plan overview

In the new UI there are progress bars for each Milestone to give the user a quick look at their progress for that Release Plan.
The charts on Figma Design are not achievable with the native charts available in Power Apps, so we have to come up with another approach or our own custom graphs.

For this iteration, I was able to create custom column graphs using rectangles in a vertical gallery.


Test Issue created at Tue, 21 Dec 2021 10:09:51 GMT

Test issue body  Tue, 21 Dec 2021 10:09:51 GMT

Comment on issue at Tue, 21 Dec 2021 10:09:54 GMT


[Activation] Merge to Activation branch - 22082018

58 tests failing

Can @rowandh @codingupastorm @zeptin @thefazzer and @monsieurleberre give this a look, see if I haven't forgotten anything?

@codingupastorm here you go

Thanks all good from me now @bokobza

Looks good to me too

@bokobza Approved.

Ok let's merge


Highlighting removal is not working

Issue / Feature -
Platform (android, iOS or both) -
Crash / Error -
Steps to reproduce / Describe in detail -

Care to explain with screenshots what this issue means?

Hi mate,
Thank you for your fast response and sorry for not explaining the problem well enough.
What I mean is that when you highlight the text you can not unhighlight it.

Mate sorry for taking your time it is working properly my bad.


still getting java.lang.NoClassDefFoundError with rest-assured:4.1.2

hello,
according to
https://github.com/rest-assured/rest-assured/issues/1168
https://github.com/rest-assured/rest-assured/issues/1175
this should have been fixed but sadly I'm still getting java.lang.NoClassDefFoundError: io/restassured/path/json/mapper/factory/JohnzonObjectMapperFactory
this exception is only thrown when I have the following piece of code in my test base
@BeforeClass
public static void setup() {
	RestAssured.port = 9001;
}

using
java 11
spring-boot:2.1.9
rest-assured:4.1.2
best regards,
peter

I get the same exception on 4.1.2. In my case, it's only apparent when adding the following config to my test:
RestAssured.enableLoggingOfRequestAndResponseIfValidationFails()

I solved it importing additional modules:
    test "io.rest-assured:json-path:4.1.2"
    test "io.rest-assured:xml-path:4.1.2"

which is not ideal, since I'm not 100% sure I'd be using them, but it might become useful for someone.

Yeah same issue with JDK13 as well, had to revert to 4.1.1

ok I was was wrong, fix for this is to import both:
testCompile group: 'io.rest-assured', name: 'rest-assured', version: '4.1.2'
testCompile group: 'io.rest-assured', name: 'rest-assured-all', version: '4.1.2'

I hope there is a proper fix for this

ok I was was wrong, fix for this is to import both:
testCompile group: 'io.rest-assured', name: 'rest-assured', version: '4.1.2'
testCompile group: 'io.rest-assured', name: 'rest-assured-all', version: '4.1.2'

I hope there is a proper fix for this

This worked for me too -- I was having the same issue.

I had the same issue with io.rest-assured:rest-assured:4.1.2 and io.rest-assured:spring-mock-mvc:4.1.2  but I am using Java 8 with gradle 7.
Importing rest-assured-all fixed the problem too.

I had a same problem with Java 11 and rest assured 4.2.0.
I only had this dependency
testImplementation "io.rest-assured:rest-assured:${restAssuredVersion}"

After I added these two, everything worked fine
testImplementation "io.rest-assured:json-path:${restAssuredVersion}"
testImplementation "io.rest-assured:xml-path:${restAssuredVersion}"

With
testImplementation('io.rest-assured:rest-assured:4.3.0')

I am getting
java.lang.ClassNotFoundException: io.restassured.path.json.mapper.factory.JsonbObjectMapperFactory

Downgrading to 4.2.0 and include -all fixed it for me:
testImplementation('io.rest-assured:rest-assured:4.2.0')
testImplementation('io.rest-assured:rest-assured-all:4.2.0')

@mamachanko answer worked for me in Java 11

With
testImplementation('io.rest-assured:rest-assured:4.3.0')

I am getting
java.lang.ClassNotFoundException: io.restassured.path.json.mapper.factory.JsonbObjectMapperFactory

Downgrading to 4.2.0 and include -all fixed it for me:
testImplementation('io.rest-assured:rest-assured:4.2.0')
testImplementation('io.rest-assured:rest-assured-all:4.2.0')

I came across the same problem with 4.3.3:
        <dependency>
            <groupId>io.rest-assured</groupId>
            <artifactId>rest-assured</artifactId>
            <version>4.3.3</version>
        </dependency>

And I downgrade to 4.2.0 and include -all, then it works

With
testImplementation('io.rest-assured:rest-assured:4.3.0')

I am getting
java.lang.ClassNotFoundException: io.restassured.path.json.mapper.factory.JsonbObjectMapperFactory

Downgrading to 4.2.0 and include -all fixed it for me:
testImplementation('io.rest-assured:rest-assured:4.2.0')
testImplementation('io.rest-assured:rest-assured-all:4.2.0')

I came across the same problem with 4.3.3:
        <dependency>
            <groupId>io.rest-assured</groupId>
            <artifactId>rest-assured</artifactId>
            <version>4.3.3</version>
        </dependency>

And I downgrade to 4.2.0 and include -all, then it works

The same issue is resolved by switching back to the 4.1.1 version

io.rest-assured
rest-assured
4.1.1

io.rest-assured
rest-assured-common
4.1.1

io.rest-assured
rest-assured-all
4.1.1

after some time I gave it another try with
@BeforeAll
public static void init() {
    RestAssured.port = 9001;
}

using

spring-boot:2.6.3
rest-assured:4.4.0
both java 11 and 17

and now it seems to work .. ¯_(ツ)_/¯


Reduce code duplication in context flyout for folders and files

I'll work on this

Is this the same as #2747? If so let's keep this tracked there

Is this the same as #2747? If so let's keep this tracked there

@gave92 Yes I didn't check before

@gave92 Yes I didn't check before


Correct the Density{Unknown} scitype for UnivariateFinite distributions

I suggest this issue is not addressed until after UnivariateFinite is moved out to it's own package https://github.com/JuliaAI/MLJBase.jl/issues/504

No, rather, the new package CategoricalDistributions.jl should implement this natively. Transferring this issue to that repo.


Add PrimeReact to UI components list

Fixes #1179

Add PrimeReact to UI components list in the community section.

Thank you!!!

@cagataycivici it's not merged yet though

Hi, should this link be organized alphabetically like the rest?

@httpJunkie if it's suggested then I can push the update

When can we expect the merge and the deploy to live?

Yes, can the list please remain sorted alphabetically?

Yes, can the list please remain sorted alphabetically?

Done the changes , Can you please review and merge?


request help: how to get request body and url parameters

In apisix core.request it only expose apis to get/set header, get remote ip.  I wanna develop a plugin to check request signature is valid or not, How to get request body and url parameters?

read request body, please take a look at this document: https://github.com/openresty/lua-nginx-module#ngxreqread_body
read URL parameters, you can take a look at this document: https://github.com/openresty/lua-nginx-module#ngxreqget_uri_args

I think it's better to expose apis to get body and url parameters in core.request. Users can start to develop quickly based on apisix. @membphis

yes, that is better. welcome PR ^_^


[Question] How To Handle Modular Startup Config from Plugins?

hello all,
please read below scenario:
in a plugin based application where a single host application serves with micro plugins everything is working well and good, but i need to do startup service registration from these plugins as needed so say plugin1 introduces a middleware, plugin2 a pipeline, plugin3 some mediatr service etc... i dig into OrchardCore & Orchard does that by using a StartupBase class but i am unable to find out how they are doing it [if my assumption is correct orchard uses msbuild for plugins unlike loadcontext of this library].
my requirements and structure is different from orchard, but i like the idea of having a StartupBase class
where i can define configuration order and service init order and it gets called on main host app initilization
can someone guide me to the right way to do this, i am ok with even minimal flow steps as long as its clear to understand.
the Plugin Startup files must handle the defined order in host and be injected into main startup bus.
thanks

thanks didnt saw that and went on implementing my own logic... which is similar to this :)


Some questions about implementation details

Hello, thank you for an interesting paper and nice code.
I have two questions concerning implementation details.

Does the "one-by-one" block reconstruction mentioned in the paper mean that input to each block comes from already quantized preceding blocks, i.e. each block may correct quantization errors coming from previous blocks? Or maybe input to each block is collected from the full-precision model?
Am I correct in my understanding that in block-wise reconstruction objective you use gradients for each object in calibration sample independently (i.e. no gradient averaging or smth, like in Adam mentioned on the paper)? Besides, what is happening here in data_utils.py, why do you add 1.0 to the gradients?

cached_grads = cached_grads.abs() + 1.0
# scaling to make sure its mean is 1
# cached_grads = cached_grads * torch.sqrt(cached_grads.numel() / cached_grads.pow(2).sum())

Thank you for your time and consideration!

Hi, I also found point 2 confusing, have you figured out the rationale behind it?


frugal: 3.14.10 -> 3.14.11

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/frugal/versions.
meta.description for frugal is: "Thrift improved"
meta.homepage for frugal is: "https://github.com/Workiva/frugal"
meta.changelog for frugal is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/yb2r78m93qkmnp6ybk6109xqlixqqf0z-frugal-3.14.11/bin/frugal passed the binary check.
1 of 1 passed binary check by having a zero exit code.
1 of 1 passed binary check by having the new version present in output.
found 3.14.11 with grep in /nix/store/yb2r78m93qkmnp6ybk6109xqlixqqf0z-frugal-3.14.11
found 3.14.11 in filename of file in /nix/store/yb2r78m93qkmnp6ybk6109xqlixqqf0z-frugal-3.14.11

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

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
frugal

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

frugal

Maintainer pings
cc @diogox @kalbasit for testing.

Change looks good to me


Fixed NMS Interface for 1.20.6

In 1.20.6 it would not load without the change because the API thinks it is not supported. The 1.20.6 is only a hotfix

Just waiting for the workflows to run, then merging it into pathetic. Thanks for the contribution :)


你好，想问下这个方法的精度是多少？

跟距离有关，10m以内，0.1m精度
在 2023-05-17 19:23:47，"ycj1124" @.***> 写道：
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


How to replicate "Time to Train Facebook’s Fairseq" data: 1.5 Days using DGX-2

I have found that "Time to Train Facebook’s Fairseq" is done in 1.5 Days using DGX-2 system.
https://devblogs.nvidia.com/tensor-core-ai-performance-milestones
Can anyone help me to replicate the data?

In this paper, "Scaling Neural Machine Translation", The system used DGX-1 with V100 GPUs.
What is the memory size of these GPUs?

16GB

Is there any suggestion for 32GB V100 gpus to reduce the training time?

See my reply in the other thread: https://github.com/pytorch/fairseq/issues/491