Add MPR badge

:clipboard: Description
add MPR package badges like AUR has.

it shouldn't be hard, because it's just the same webpage like AUR, but in different colors and with different URL. it's made for Debian and derivatives.
MPR: https://mpr.hunterwittenborn.com/
package example: https://mpr.hunterwittenborn.com/packages/disfetch/
i tried to do something myself via dynamic badges, but it just doesn't work with correct URL and correct XPATH.
this URL: https://mpr.hunterwittenborn.com/cgit/aur.git/?h=disfetch
and these XPATHs:

(//html/body/div[@id="cgit"]/div[@class="content"]/table[@class="list nowrap"]/tbody/tr/td/a)[1]/text()

(//html/body/div/div/table/tbody/tr/td/a)[1]/text()

//html/body/div/div/table/tbody/tr/td/a

//div/div/table/tbody/tr/td/a

each of them yields no result

:link: Data
you can just copy script for AUR and adapt it, so it works with MPR

:microphone: Motivation
MPR is pretty new project, but it becomes more popular and it would be nice to see MPR badges.

I'm going to close this for now. We do not add badges that scrape html, and accordingly the AUR badges, mentioned as a comparison, don't scrape html either; we get AUR data via an API call so there's nothing to "copy".
In order for this request to become actionable, we'd want to see the service increase in popularity and provide a standard JSON (or XML, but not preferred) API that provides the underlying package metadata. Please feel free to follow up if/when that occurs!

we get AUR data via an API call

You should also be able to do so from the MPR just fine, it actually uses the same backend platform as the AUR, so virtually everything should transfer over just fine.
P.s. I'm the maintainer of the MPR project; I thought I'd just clear up that misconception here.

@hwittenborn do you have a MPR API documentation link that can be shared, or at a minimum some example API calls?

@calebcartwright Should be exactly the same as for the AUR, just with a different base URL.
i.e.

https://mpr.hunterwittenborn.com/rpc/?v=5&type=info&arg[]=makedeb (returns results for makedeb)
https://mpr.hunterwittenborn.com/rpc/?v=5&type=info&arg[]=makedeb&arg[]=makedeb-beta (returns results for makedeb and makedeb-beta)

@calebcartwright Any update on this?
Honestly I think it might just work out better if Shields would just allow a custom endpoint for the AUR to be provided, then I think this any any future projects that use the AUR codebase could just start using this immediately.

Any update on this?

No. If anyone plans to work on it, they'll post here or submit a PR. Shields is an open contribution community project.
In general a pretty good heuristic for us as maintainers is that if we're going to take on the maintenance of code for a new service badge at this point then we'd like to see that there is at least enough popularity/interest in the service that one person in the community is willing to submit a PR to add the badge. Its not a particularly high bar, but also it is frequently not met.
In terms of implementation, sharing helper code with the AUR integration would be one valid way to address this if anyone is interested. Allowing AUR to take a registry param might be another a valid way to address this. It is an approach I suggested for Thunderbird Add-ons in https://github.com/badges/shields/issues/6994#issuecomment-914544264 This would seem particularly appealing if there are other registries using the same API. Do you know of any other registries that are API compatible with AUR.

:100: agree with Chris (apologies this slipped through my notifications list). The only concern I'd have with the query param path would be whether or not it's obvious and transparent to all end users that their registry has precise API compatibility. It's definitely a common, and often preferable, path for badges that can target multiple instances (e.g. Jenkins, Sonar, GitLab, etc.) but in those cases it's also completely obvious to the user that it's just another instance of the tool; they don't have know the underlying API surface that Shields consumes and whether other targets have that same contract.

Do you know of any other registries that are API compatible with AUR.

Besides the MPR I don't think any other project is using the AUR codebase, no.
What exactly was your concern @calebcartwright? I couldn't figure out what you meant :P
The only reason I can't do much of a PR right is I haven't gotten a chance to learn JavaScript yet, so I wouldn't know much of what to do. I'm sure I could spend some time on learning it, but that would require quite a bit of time that I'd really like to focus on other things, or at least right now.

What exactly was your concern @calebcartwright? I couldn't figure out what you meant :P

I'm asking clarifying questions that would, in my opinion, play a factor in the solution, particularly related to the user experience of Shields users.
One of the proposed solutions, as I understand it, is for our existing AUR badges to take a query parameter that allows the user to provide the link for an AUR endpoint (which would be a functionally required query parameter for MPR). It's not clear to me whether one of our users who is a maintainer of a package that's hosted on MPR will find it immediately clear that they should use our AUR badge route and or search/browse on our shields.io website to generate a badge for their MPR-hosted package.
My question is whether a reasonable MPR consumer would view/expect that to be a different registry of sorts, regardless of the fact that under the proverbial hood it happens to use the same API.
As an example, if the GitLab API uri path and response object for repo tags was identical to the GitHub API uri path and response object for tags with the only difference being the base URL, would it make sense for Shields to expose a "GitHub Tag" badge that GitLab users can also use by selecting the GitHub badge and then providing a query parameter for GitLab.
I'd argue pretty strongly that hypothetical would be a terrible idea :laughing: I'm not an Arch user, so I'm just not aware of how similar/different the AUR/MPR would be comparatively (i.e. are they basically synonymous, as divergent as say deb vs. snaps or rpm vs flatpack, somewhere in between, etc.)

Gotcha, that makes more sense :)
The entire ecosystem around MPR is pretty Arch Linux-centric, and most of the users know about the AUR and the tools required to build packages from the AUR. The MPR and it's installation tools are actually all forks of tools that originated in the Arch Linux ecosystem, providing further resemblance to there as well.


Unable to load extension: cannot import name _thread when creating a role

Issue Type

Bug report

Molecule and Ansible details
ansible --version:
ansible <IP_ADDRESS>
  config file =
  configured module search path = Default w/o overrides
  python version = 2.7.10 (default, Jul 14 2015, 19:46:27) [GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.39)]
molecule --version: molecule, version 1.24.0

Molecule installation method: pip
Ansible installation method: pip

Desired Behaviour
When creating a role with:
molecule init --driver docker --role ansible-meetup

I would like to have a role created, with the basic directory structure and molecule configuration
Actual Behaviour (Bug report only)
I'm trying to create a role with the molecule init command with the docker driver. But when executing the command, i get the following message:
molecule init --driver docker --role ansible-meetup
--> Initializing role ansible-meetup...
Traceback (most recent call last):
  File "/usr/local/bin/molecule", line 11, in <module>
    sys.exit(main())
  File "/Library/Python/2.7/site-packages/molecule/cli.py", line 41, in main
    cli(obj={})
  File "/Library/Python/2.7/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/Library/Python/2.7/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/Library/Python/2.7/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Library/Python/2.7/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Library/Python/2.7/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/Library/Python/2.7/site-packages/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/Library/Python/2.7/site-packages/molecule/command/init.py", line 141, in init
    util.sysexit(i.execute()[0])
  File "/Library/Python/2.7/site-packages/molecule/command/init.py", line 60, in execute
    self._init_new_role(role, role_path, driver, verifier)
  File "/Library/Python/2.7/site-packages/molecule/command/init.py", line 85, in _init_new_role
    util.process_templates(template, extra_context, role_path)
  File "/Library/Python/2.7/site-packages/molecule/util.py", line 115, in process_templates
    no_input=True, )
  File "/Library/Python/2.7/site-packages/cookiecutter/main.py", line 82, in cookiecutter
    context['cookiecutter'] = prompt_for_config(context, no_input)
  File "/Library/Python/2.7/site-packages/cookiecutter/prompt.py", line 188, in prompt_for_config
    env = StrictEnvironment(context=context)
  File "/Library/Python/2.7/site-packages/cookiecutter/environment.py", line 70, in __init__
    **kwargs
  File "/Library/Python/2.7/site-packages/cookiecutter/environment.py", line 40, in __init__
    raise UnknownExtension('Unable to load extension: {}'.format(err))
cookiecutter.exceptions.UnknownExtension: Unable to load extension: cannot import name _thread

I did update molecule to latest version, but did not help. Running MacOSX 10.10 (Yeah I know, pretty old)

@dj-wasabi odd, it works for me....
[jodewey:~/git/molecule] [molecule-env-2.7.13] master ± molecule init --driver docker --role ansible-meetup
--> Initializing role ansible-meetup...
Successfully initialized new role in /Users/jodewey/git/molecule/ansible-meetup.

What version of cookiecutter are you using?  Did you pip install -U all of your dependencies recently?
[jodewey:~/git/molecule] [molecule-env-2.7.13] master* 1 ± pip freeze | grep cookiecutter
cookiecutter==1.5.1

Sorry, probably bad installation. I did a virtualenv and installed everything and works like a charm. You may close this, on my other laptop it is working too.
P.S. Had no issues. :-) Was preparing for an Ansible Meetup here and will talk about Testing roles with Molecule .. :-)

@dj-wasabi awesome!  Share your preso sometime if you can!


Make text index query cache a configurable option

During design of text index feature, we had done some heap overhead experiments. As part of that we had determined that while Lucene's internal query cache helps with performance significantly for repeatable queries, it adds heap overhead. Therefore, it was disabled during index loading.
It is worthwhile to make this configurable on per-index basis for users to enable it. Of course, they need to know the downside of more heap overhead (which could potentially negate the perf improvements due to more GC).
As part of ongoing internal user acceptance testing, we learned that most of the queries are repeatable. The UI will more or less keep the text search filter constant and tweak the other filters. For such cases, it is good to see the performance improvements by enabling the query cache and if the heap overhead is not significantly high, user might want to keep it enabled.
Note: By default it is still disabled. So nothing really changed in the existing behavior.

Codecov Report

Merging #5176 into master will increase coverage by 9.39%.
The diff coverage is 70.58%.

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

Do we still need this?

Do we still need this?

Yes, by default nothing is changed. But if the user is interested, they can enable it

@Jackie-Jiang @jackjlli , I have addressed the review comments. Please take a look at it again


feat: add a contributors page

add a page to feature all the developers who worked on this awesome project.
assign me this under hacktoberfest and gssoc-ext , and i expect level 3 for this.

sure ...will consider it !!

Reduced to level1, as Level 3 isn't justified for such update.

@kvcops dont do this, i have already wrote more than 500 lines of code, and page is almost ready.

@jindalpriyanshu101 sir please refer other projects too, its considered as level 3 .

@AE-Hertz If you wrote 500 lines of code for a simple issue, then that is another issue. Adding a contributors page is a simple begineer level issue and no matter the number of lines in a code, it is the complexity of it which determines the level.

@techy4shri if thats the case, decide it during PR, . as far as i know handling apis is not a beginner level stuff.

@jindalpriyanshu101 sir please refer other projects too, its considered as level 3 . i am not making a readme file here.

can you link those OTHER PROJECTS, their points will be reduced as well.

Here it all depends upon the ui that you make...I was hoping you would make the ui as good as the index.html with animations and transitions

@AE-Hertz, labels are assigned strictly in accordance with the intrinsic complexity of the issue, not predicated upon an arbitrary metric such as personal knowledge. Furthermore, I have yet to encounter any verifiable evidence pertaining to these OTHER PROJECTS you referenced. I strongly advise you to either concentrate on the task at hand, or this issue will be summarily locked and permanently disregarded.


Damage checkbox value persisting to next record

Damaged specimen checkbox mark should not persist to next record (at the moment it is persisting)

Fixed with commit https://github.com/NHMDenmark/Mass-Digitizer/commit/b9b7f83189249b4049c0289a282d64f3a342de17 by adding the check box to the nonStickyFields list.


Add Unit tests to the package mil.nga.giat.geowave.adapter.raster.stats

There are currently no unit tests of any of the public methods in the java classes inside the mil.nga.giat.geowave.adapter.raster.stats package. It is important to understand what each statistic is intending to do and then within a test, you can call entryIngested() with completely synthetic data, and merge() a couple statistics together to ensure it works properly, as well as to and from binary methods for serialization.

complete


add a new hexo theme snark

Signed-off-by: litreily<EMAIL_ADDRESS>
[x] Read the theme publishing doc or plugin publishing doc.

Preview link doesn't seem to work.

@mkkhedawat The preview linkis work well, can  you check again?

Ohkay, Preview is fine.
But one issue, Theme name ('Snark') is not same as image name ('snark'), Because of that image is not loading on site. Change one of them.
Steps to reproduce :
$ cd site
$ npm install
$ hexo server

visit http://localhost:4000 and go to themes tab, search for your theme listing, Image should load properly.

@mkkhedawat update yet, and test pass, please help to merge, thanks!


Take AutoGenerateField into account for GetEnumSelectList

It would be convenient if the GetEnumSelectList helper methods only returned SelectListItems for those enum values that either don't have a DisplayAttribute or do have a DisplayAttribute with AutoGenerateField set to false.

It would be a breaking change to start doing that by default. Do you have any thoughts on what an API would look like that supports this? In general we like to avoid complex rules that are both difficult to describe and difficult to understand. E.g. in this case would someone be better off just having a different enum that they use for generating the list? Or just writing their own custom logic that processes the enum in a particular way and produces a SelectList that has only the desired items?

Or just writing their own custom logic that processes the enum in a particular way and produces a SelectList that has only the desired items?

This.
Thanks!


Module loader API rewrite - Make changes to work with the Language Server

Description:
$Title
Also need to fix all the failing LS tests.
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Will be implemented by project API


MNT: complete transition from flake8/black to ruff

Fixing the CI errors requires https://github.com/nipreps/fmriprep/pull/3280.


(react?) VisCrosshair is not compatible with timeline

the crosshair component as it stands today only operates on the x axis; this doesn't really make sense for a timeline; or at least it usually wont if you have multiple parallel items. having it be rotatable for the y axis would be useful.

Hi @mattp-
We have the horizontal version of Crosshair in our pipeline, it's not our highest priority though.
Can you please explain how would you like to use it with Timeline? We usually integrate them like this:

hi there, yep that looks like what i was looking for. mainly I was actually just looking for a workaround for https://github.com/f5/unovis/issues/103 to get my tooltip to appear. I thought I had it with crosshair until I realized it was not the right axis :)

@mattp- In that case, you'll need to set snapToData to false in Crosshair's config, and provide your own getCircles function for getting the position for the Crosshair circles. It's a bit tricky, but here is an example:
function findDataEntriesAt (x: number): SampleTimelineDatum[] {
  return data.filter(d => (d.time <= x) && ((d.time + d.duration) >= x))
}

function getCircles (x: number, data: SampleTimelineDatum[]): CrosshairCircle[] {
  return findDataEntriesAt(x).map((d, i) => ({
    y: 11 + i * 22 - scrollTop, // If your Timeline is scrollable you'll also need to store its current scrollTop position
    color: d.color,
  }))
}

// Optional: saving the scrollTop position and re-rendering the Crosshair
// Set this function via the `onScroll` config property of Timeline
function onScroll (top: number): void {
  setScrollTop(top)
  crosshairRef.component.render(0)
}


Unnecessary to set account sid / secret in function environment?

https://github.com/twilio-labs/plugin-rtc/blob/e4e4f930da74a6f76369d98a18b8f953184577bf/src/helpers.js#L127-L129
I believe it is unnecessary to copy account sid and secret to the function. Functions already have the account sid and secret of the account that owns them. They are available as context.ACCOUNT_SID and context.AUTH_TOKEN

Thanks for the tip @jesperfj!
It's true that ACCOUNT_SID is already available on the context object, so there's no need to pass that in.  However, AUTH_TOKEN cannot be used to mint video tokens. We will still need to pass in TWILIO_API_KEY_SID and TWILIO_API_KEY_SECRET.
I've created a PR to address your feedback: https://github.com/twilio-labs/plugin-rtc/pull/20

Fixed in #20
Thanks!

Why can it be used to mint twilio client tokens but not video tokens?
On Mon, Mar 23, 2020 at 8:30 PM Aaron Alaniz<EMAIL_ADDRESS>wrote:

Closed #18 https://github.com/twilio-labs/plugin-rtc/issues/18.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/twilio-labs/plugin-rtc/issues/18#event-3157932114,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACFJLDC6D6MYJ2O3IWKOETRJASLLANCNFSM4LRL3SLQ
.


Standard @Inject annotation

Flamewar beginning in 3, 2, 1…
Not all containers support annotations. And that's fine :) But several do, and the bad part is that end users are coupling themselves to a specific container.
This RFC is about standardizing an @Inject annotation (or similar annotation) so that end users can switch between containers supporting annotations.
This RFC is not about encouraging containers to support annotations.
References

Flow: Flow\Inject
JMSDiBundle for Symfony 2: JMS\DiExtraBundle\Annotation\Inject
JSR330 is the Java standardization of their @Inject
PHP-DI: DI\Annotation\Inject
Ray: Ray\Di\Di\Inject
Zend Framework 2: Zend\Di\Definition\Annotation\Inject

Comparison

Package
Annotation engine
Name
Apply to
Mandatory parameters
Default parameter

Flow
?
@Inject
Property
No
?

JMSDiBundle
Doctrine
@Inject@InjectParams
PropertyMethod
NoNo
Service ID

PHP-DI
Doctrine
@Inject
Property, Constructor, Method
No
Service ID

Ray
Doctrine
@Inject
Constructor, Method
No
None

Zend Framework
Doctrine
@Inject
Method, ?
No
?

I have only focused on @Inject type of annotations. Some packages offer @Service (or similar) annotations to declare and configure services, but this is not consistent enough across packages.
I have also not listed all the parameters (for now). The important thing is that none of them have mandatory parameters, which is good because it means there is a common compatible usage.
We can however see that they are not all usable everywhere, i.e. some packages support annotating properties, some support only annotating methods…
How?
It seems all (unless maybe Flow) containers use Doctrine's annotation system. So we could provide an annotation class like Interop\Container\Annotation\Inject.
I don't think there's a way to have an interface as annotation with Doctrine, so that's why we'll need to push a class and not an interface.
Example:
namespace Interop\Container\Annotation;
/**
 * @Annotation
 */
class Inject
{
}

Usage
Users can use the annotation with the FQN or using import aliases:
use Interop\Container\Annotation as DI;

class Foo
{
    /**
     * @Interop\Container\Annotation\Inject
     */
    public function __construct(/* ... */) { /* ... */ }

    /**
     * @DI\Inject
     */
    public function setBar(/* ... */) { /* ... */ }
}

Questions

Which "target" should be supported?

properties?
methods?
both?

Should we support an optional name parameter? => Need to evaluate support in every container.
Should we support any parameter for the annotation to let the package consuming it retrieve them?

Maybe as part of the PHPDoc PSR?
Also consider naming it @inject (note the lower-case i), see also phpDocumentor/fig-standards#96.

  Read Tom Butler's blog post at the link for a highly dissected discussion as to why.  https://r.je/php-annotations-are-an-abomination.html

See "good summary of the current annotation status in PHP" by Rafael Dohms.
http://www.slideshare.net/rdohms/annotating-with-annotations-forumphp-2012

I would not go so far to say that annotations are bad per se. I used to be totally against annotations in the past but my view has changed over the last couple years. In the end it resulted in removing the xml configuration format from our DI container and turn it into an annotation based DI container. I totally agree that spreading annotations throughout the whole code base is not the best idea. Using annotations for a single configuration class like we do with bitexpert/disco is perfectly valid in my opinion ;)

@cxj I am familiar with Tom's arguments against annotations. To call them an anti-pattern is quite extreme. Other languages have adopted annotations as part of their core implementations such as Java or attributes in C#. PHP is becoming more and more like these 2 languages and having the extra expressiveness that annotations provide I myself an other agree is a good thing.
Sure over-using any single pattern, or using a pattern inappropriately is not good. But lets not get hung up on several peoples conflicting opinions, in the Java world annotations have been widely accepted way to help many people get out of XML configuration hell. They have also been used with elegance in many very successful projects regardless of programming language.
As @mnapoli says

Let's not discuss wether we like them or not, that will get us nowhere.

Lets get back into the spirit of the conversation which is over standardisation. If you would like to suggest an argument for/against annotations then can we take those back to Reddit and keep this space for constructive discussion over how we can make @Inject annotation less implementation specific?

IMO, annotations are an implementation detail, far from the consumer. I wouldn't try standardizing them, as every container will provide its own way to configure more fine-grained @Inject details anyway.
Also, as part of the project that actually tends to drive the annotations system, I kinda agree with the idea that they are an anti-pattern, as they couple implementation with injector (while DI is about decoupling dependencies).
Exactly like with a service locator there is though a place for them in non-reusable highly-app-specific code.
Standardizing them would lead to usage of annotations in open source projects (reusable code), which would not represent that use-case.

If you don't mind me copy-pasting the first lines of the issue:

This RFC is about standardizing an @Inject annotation (or similar annotation) so that end users can switch between containers supporting annotations.

This RFC is not about encouraging containers to support annotations.

I don't think it makes sense to discuss the point of annotations here, this will lead nowhere (and this is not the topic).

I apologize for going off topic.  It was wrong for me to do that.


Add onGherkinDocumentChange prop

We (Gherkin editor team) currently parse the gherkin document in our own code. That means for every keystroke in the document, two passes of the parser happen (one in gherkin editor, one in react-gherkin-editor). We'd like to rely on react-gherkin-editor to do the work, hence the request for a new prop looking like this:
onGherkinDocumentChange={(errors: GherkinParserError[], doc: GherkinDocument) => { /* ... */ }}

Jira issue [BDD-4]

Jira issue [BDD-4]

The property is available since https://github.com/SmartBear/react-gherkin-editor/commit/b8472cf486b0420a6a58b3f4d45808a57d4f2018
The property is named onParse (and not onGherkindocumentChange) and send back messages.

The property is available since https://github.com/SmartBear/react-gherkin-editor/commit/b8472cf486b0420a6a58b3f4d45808a57d4f2018
The property is named onParse (and not onGherkindocumentChange) and send back messages.


Correct RFC number

Correct RFC number (81821 -> 8182)
Add links to IETF datatracker and Wikipedia for context

Thank you for the PR! It looks good!
I know it is a bit silly, what with the changes only being documentation, but could you nonetheless perchance merge main into your branch so the CI goes green before merging?

Thank you for the PR! It looks good!
I know it is a bit silly, what with the changes only being documentation, but could you nonetheless perchance merge main into your branch so the CI goes green before merging?

Not silly at all, I have never understood git, and my work is based on cheatsheets other OSS contributers have given me.
I assume the conflicts were with the clippy commit.
Rebased (I think) on NLnetLabs:main , and pushed again.
Thank you, and to the NLnet/NLNog team, for all your wonderful projects

Thank you and apologies, I somehow forgot to click the buttons.


incorrect nomic embeddings

I was comparing the nomic embeddings and they are very different from the original version.
import pandas as pd
from more_itertools import chunked
from typing import List
import numpy as np
import torch.nn.functional as F
from sentence_transformers import SentenceTransformer
import torch
import os
from tqdm.notebook import tqdm
import json
from fastembed import SparseTextEmbedding, TextEmbedding

assert torch.cuda.is_available()
SEED = 25

model = SentenceTransformer("nomic-ai/nomic-embed-text-v1.5", trust_remote_code=True)

def embed(texts: List[str]):
    embeddings = model.encode(["clustering: " + t for t in texts], convert_to_tensor=True)
    embeddings = F.layer_norm(embeddings, normalized_shape=(embeddings.shape[1],))
    embeddings = F.normalize(embeddings, p=2, dim=1)
    return embeddings.cpu().numpy()

import types
embedding_model = TextEmbedding(model_name="nomic-ai/nomic-embed-text-v1.5")

def embed_fast(texts: List[str]):
    embeddings = embedding_model.embed(["clustering: " + t for t in texts])
    # Force computation if embed_func returns a generator
    if isinstance(embeddings, types.GeneratorType):
        embeddings = np.array(list(embeddings))
    return embeddings

res1 = embed(data)
res1
array([[ 0.0478127 ,  0.07791077, -0.16337295, ..., -0.09588917,
        -0.01815554, -0.0391101 ],
       [ 0.00486873,  0.05552602, -0.17271836, ..., -0.06137123,
        -0.01570066,  0.00191791],
       [ 0.06523891,  0.0010743 , -0.18216555, ..., -0.07239737,
        -0.04038522, -0.01559034],
       ...,
       [ 0.00218753,  0.07514183, -0.20761742, ..., -0.06362353,
        -0.01809935, -0.05537352],
       [ 0.04343107,  0.09780316, -0.18920647, ..., -0.04333348,
        -0.00627593, -0.05098606],
       [ 0.07419404, -0.00313356, -0.17786372, ..., -0.12185868,
         0.00074316, -0.03453541]], dtype=float32)

res2 = embed_fast(data)
res2
array([[ 0.02929905,  0.04431156, -0.15961514, ..., -0.06745377,
        -0.05103005, -0.03928511],
       [ 0.01270135,  0.05443013, -0.17877947, ..., -0.07663167,
        -0.02374521, -0.01003957],
       [ 0.04591728,  0.03366161, -0.22145797, ..., -0.06816148,
        -0.04856569, -0.00438772],
       ...,
       [ 0.00895302,  0.06434014, -0.22018844, ..., -0.06492045,
        -0.02143181, -0.06246027],
       [ 0.02247173,  0.06561402, -0.19789155, ..., -0.05780106,
        -0.01238827, -0.0473954 ],
       [ 0.05192114,  0.00191367, -0.1852103 , ..., -0.10932592,
        -0.01199157, -0.03163441]], dtype=float32)

# compute cosine similarity between nth element in res1 and res2
from sklearn.metrics.pairwise import cosine_similarity
cosine_similarity(res1[0].reshape(1, -1), res2[0].reshape(1, -1))
array([[0.85182214]], dtype=float32)

so we can see that the cosine similarity is only 0.85, the vectors are completely different.
The fastembed normalize https://github.com/qdrant/fastembed/blob/main/fastembed/common/models.py#L49-L54 does not follow the normalization in https://huggingface.co/nomic-ai/nomic-embed-text-v1.5#sentence-transformers it seems

Ohh, we missed this completely in our tests — I'll look into this. Thanks a ton for reporting this!

Any resolution on this?

Hey @k4u5h1k, sorry for the late response, yes, we're working on it, we'll fix it soon

HI, I just added on comment on the PR that was merged (See https://github.com/qdrant/fastembed/pull/280#issuecomment-2276112677)
The addition of the mean_pool solved cosine similarity issue but there are still inconsistencies about the normalization (especially when comparing to Nomic's documentation), I think this issue should be reopened in the mean time.


Fold starlark_module! macro a bit more

Introduce the intermediate step which does a transformation like:
x: String, ...

into
(named) x: String, ...

so both named and unnamed arguments are handled uniformly by matching
on $is_named:tt $t:ident (# x or (named) x).
Resulting macros are even more complicated than before but contain
fewer repetitions (and fewer LOC), so it's possible to extend them
further without drowning in copypasta.

Clippy: https://github.com/rust-lang/rust-clippy/issues/4133


Setting config options for substores not working

Summary
The gopass config --store <store> key value command sets the config option globally and not on the mount.
And why is the autosync option not available anymore? How can I disable the automatic sync?
Environment

OS: Arch
gopass Version: 1.10.1
Installation method: repo

Having individual settings for different mounts was deprecated in 1.10 actually.
I don't remember the exact rational, but the goal is to simplify the codebase and to have as little settings as possible.
Autosync should become the default I think, but @dominikschulz is the one who knowns the details.

Setting config options per mount has been removed. It complicated implementation and UX for little benefit.
AutoSync is now enabled by default. If you don't want sync remove the remotes.

The --store option should be removed from the help page if the feature has been removed.
However, this feature is important for me, because not all users of a substore are using gopass. And one substore I use, stores the public keys in a different location than gopass would with the exportkeys function. So I would like to disable exportkeys for this substore but let it enabled for the others.
And I want to disable autosync because it slows down my workflow as the syncing is done synchronously. I replaced the autosync by a systemd timer that runs every 30 minutes.
If it is done directly, creating and editing passwords takes too long, especially when using a YubiKey, which slows down the crypto operations.
So I will have to go back to 1.9.x

@dominikschulz I can see how having mount-specific config options can be useful.
Should we have a user-story with this ^ usecase and include it back?
I don't know how much complexity to the config handling it added? Most of the code is still able to handle it I'd guess.
For the autosync, if we don't want to have a config option we could maybe have a environment variable to disable it?
Meanwhile if you don't want to have the exportkey stuff messing with your substore, you could add the exportkey .public-keys folder to your .gitignore I guess?
(BTW, exportkey is not well documented, we might want to document what triggers it and how it behaves. Possibly add a way to specify a different location for its exportation?)

We certainly want to slow down our main use cases too much. Maybe having an option for autosync might be useful, but it should default to on. Environment variables are also OK. I'd prefer that for rarely used features.
The .gitignore workaround for exportkeys sounds good to me. Not sure if it's worth reintorducing the sub-store config compliexity (which was very annoying) just for that.

The problem is that the workaround is not working.
Gopass creates the commits despite the folder being listed in the .gitignore file.

If it's already track by git you might need to remove it first. Then it shuoldn't show up on the remotes anymore. But of course the folder will still be created locally.

It's not tracked by git, but the commits are created nevertheless. And my .gitignore file is correct.

Ha, yes. You are right. Sorry.
To work around some strange failure modes of git, we have added --force to git add.
Of course this will then ignore your .gitignore.
Seems like we need to find another way.
Actually I was considering to drop this feature, but I'm not entirely sure.
In certain settings it can help a lot.
Maybe we should have it enabled by default but provide some escape hatch, like checking if a special file in that mount exists and skipping key exports if we find it or so. But I'd really like to avoid reintroducing this cumbersome per-mount config options.

So, currently is autosync removed completely? I want to sync my stores, but not on every command I use. If I want to set autosync to false globally I get:

> gopass config autosync false
Error: Error setting config value

It's still mentioned in the docs.

this cumbersome per-mount config options.

@dominikschulz I wonder why that's cumbersome. It looks to me that's a common case from end users perspective, just like git config, where it has both the global config and per-repo config, and the per-repo one can overwrite the global one.

The way we did config and per-store config previously was cumbersome. But IMO we could probably come up with a better scheme.
For example, for the "per-repo" config: what if someone set just one option in a given repo?
Should we copy the entire current config to that repo and change just that value? Or should we make sure that we supprot "partial" config, where if a repo override just one option, the others are taken from the global config and if the global config isn't specifying everything we use the default value?
IMO I'd say the second way is "best", but it would currently require a complete re-write of the config handling.

I like the idea of "partial" config, as it appears that where there's a change at global level, it doesn't have to copy over all the stuff to all config files at the child level.


Convert dash case to camel case

When parsing HOCON, the mapper does not recognize dash case (e.g. 'sample-key: "a"') to convert it to camel case ('sampleKey')
PS: Sorry for the previous title. My co-worker was messing with my computer.

jotaporras changed the title from Your software sucks. to Convert Dash case to camel case 2 minutes ago

Sorry, my fault, I was fooling around. Hope you understand.

Have you tried using jackson's KebabCaseStrategy?

I'm not sure if this is what you want, but...
	val HOCON_MAPPER = ObjectMapper(HoconFactory()).apply {
		this.enable(MapperFeature.ALLOW_EXPLICIT_PROPERTY_RENAMING)

		this.propertyNamingStrategy = object: PropertyNamingStrategy.PropertyNamingStrategyBase() {
			override fun translate(p0: String): String {
				val newField = StringBuilder()

				for (ch in p0) {
					if (ch.isUpperCase()) {
						newField.append('-')
					}
					newField.append(ch.toLowerCase())
				}

				return newField.toString()
			}
		}
	}

This allows you to use properties with camelCase while the source config (in HOCON) uses dashed-variables
class TestConfig(
        @JsonProperty("camelCaseParam")
        val camelCaseParam: String
)

val testConfig = Constants.HOCON_MAPPER.readValue<QuirkyStuff.TestConfig>("camel-case-param = hello world")
println(testConfig.camelCaseParam)


Give Nokogiri::HTML::Document a parent which always returns nil

Hi,
take any node somewhere in a document and (for whatever reason) traverse up its parent chain. Inevitably when getting to the Nokogiri::HTML::Document node an error will be raised, because there's no parent method.
This makes traversing the parent cumbersome, because one can't simply rely on the parent method itself, e.g.
$ ruby -r pp -r nokogiri -e 'Nokogiri::HTML("

").search("span").each { |s| while s = s.parent; puts s; end }'

<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd">

-e:1:in `block in <main>': undefined method `parent' for #<Nokogiri::HTML::Document:0x00000000d28180> (NoMethodError)
        from /opt/ruby-1.9.1-p378/lib/ruby/gems/1.9.1/gems/nokogiri-<IP_ADDRESS>/lib/nokogiri/xml/node_set.rb:239:in `block in each'
        from /opt/ruby-1.9.1-p378/lib/ruby/gems/1.9.1/gems/nokogiri-<IP_ADDRESS>/lib/nokogiri/xml/node_set.rb:238:in `upto'
        from /opt/ruby-1.9.1-p378/lib/ruby/gems/1.9.1/gems/nokogiri-<IP_ADDRESS>/lib/nokogiri/xml/node_set.rb:238:in `each'
        from -e:1:in `'

It requires a check whether .parent is availabile
$ ruby -r pp -r nokogiri -e 'Nokogiri::HTML("

").search("span").each { |s| while s.respond_to?("parent") and s = s.parent; puts s; end }'

I can't foresee the implication if document would have a parent method which simply always returns nil, but I think it could be a useful addition.
thanks

Got into a similar problem. I need to traverse all parent DOMS but i can't move beyond the first element because a document has no parent. I second @mfn . Addition of the parent = nil for the document will be very useful.

@mfn Where did you add this? i'd like to do the same.

Since in Ruby you can open up any class, I think I just threw it into the script which did the parsing. But I'm not using ruby at the moment so there may be other/better best practices.


Pasting code into input

I'd like to be able to long-press one of the inputs or the whole component and then press the 'paste' button to paste the OTP that is stored in my clipboard. Is there a way to achieve that currently?
I also suggest adding textContentType="oneTimeCode" to the TextInput, it can be used to indicate that a field can be auto-filled by a code arriving in an SMS on iOS.

@sarahabuteen Thank you for such nice ideas! Feel free to open a PR and investigate it with me :)

@anday013 how did you handled pasting could you link me to the code snipped handling the same

@Biki-das here setting from the ref:
https://github.com/anday013/react-native-otp-entry/blob/7db6d76c2cea11969438069c21a97f7d74cd1b98/src/OtpInput/useOtpInput.ts#L26

@anday013 😅 could you explain how are you able to retrieve the copied text without using clipboard package and how are you specifically looking for paste events

@Biki-das I see what you're asking. Basically, there is no 'paste from clipboard' feature. This function sets the value 'programatically', i.e., you can call ref.setValue('123456') and it will be set. So, it does not paste from clipboard rather sets value that is passed as an argument to the function. Hope it's clear now 😅

@anday013 how you are then observing the paste event like when does this function gets called back appropriately such that

@Biki-das I think I did not fully get the question, may ask you to explain again please? :)
P.S. In my example, it calls handleTextChange where the state is being modified (by setText) with the passed argument, and new value is being set

@Biki-das I think I did not fully get the question, may ask you to explain again please? :)
P.S. In my example, it calls handleTextChange where the state is being modified (by setText) with the passed argument, and new value is being set

Like how are you differentiate when a user tries to paste and when trying to fill the inputs one by one.

@Biki-das  There is no difference. As I mentioned, there is no pasting from the clipboard. In the current implementation, there is only one possibility to fill out and no way to paste.
There is a possibility kinda "filling all at once" by calling that set function from ref, but as I said, it's just filling all at once, not pasting from clipboard :) I understand It may sound a bit confusing to you, but I am ready to explain, and you can also take advantage of looking at the codebase, as it's not that complicated compared to more sophisticated libraries :)

thanks @anday013 , i am sharing you a gist sort of a  small implementation of my own  otp component
https://gist.github.com/Biki-das/09385fd18a014904a90ca7a8b15612f4
? how can i do the same sort of functionality.

@anday013 if the paste button does not appears on long press? how shall a user on phone be able to paste any idea

thanks @anday013 , i am sharing you a gist sort of a small implementation of my own otp component
https://gist.github.com/Biki-das/09385fd18a014904a90ca7a8b15612f4
? how can i do the same sort of functionality.

You do not have pasting functionality either

@anday013 if the paste button does not appears on long press? how shall a user on phone be able to paste any idea

it does not appear because, long press on plain views (as each pin container is a view) does not affect/trigger actual TextInput (as it is small and invisible). It's the actual problem

hey guys I saw the component and I loved the idea it would be great if we could implement the copy paste and the auto fill functionalities lets break these two issues down and brainstorm how can we do it.

hey guys, thank you all for the interest. I've created a separate issue #31  to break down this functionality, which was a nice suggestion from @auronvila . I included all the required information; feel free to give it a try and ask any questions you have there.


[PyTorch] Prototype for operation-based API

Currently, Transformer Engine exposes fused operations with custom modules like LayerNormLinear. These are highly tuned for certain workloads (especially GPT), but are not easy to generalize to other models. This approach is especially cumbersome when the forward and backward passes have different fusion opportunities (e.g. forward GEMM+bias+gelu and backward dgelu+dbias+cast+transpose).
This PR adds a new API for specifying Transformer Engine models. Instead of using large compound modules (e.g. LayerNormLinear), users can build up a Sequential module out of small FusibleOperations (e.g. LayerNorm, Linear). The Sequential module (with a similar API as torch.nn.Sequential) will internally attempt to fuse operations together (possibly differently in the forward and backward passes).
Some of the more important components:

te.ops.FusibleOperation: A neural network operation that can be processed by the fuser. They have forward and backward functions similar to torch.autograd.Function.
te.ops.BasicOperation: A minimal FusibleOperation. Their forward and backward functions must be implemented and they should hold the model state and parameters.
te.ops.FusedOperation: A FusibleOperation that is interchangeable with multiple UnfusedOpeations. If it implements a forward or backward function, they must save the same context as the UnfusedOperations.
te.ops.Sequential: A container module with a similar API as torch.nn.Sequential.
te.ops.OperationFuser: A helper class that manages autograd, performs the operation fusions, and keeps track of corresponding BasicOperations and FusedOperations.

As a proof-of-concept, I've been able to fuse Linear and Bias operations, on a single GPU and with tensor parallelism. These modules have been implemented to support Float8Tensor, which simplifies the implementation and will be important for future work with e.g. FP8 attention. I've also added single-GPU and multi-GPU tests.
This work is heavily influenced by https://github.com/NVIDIA/TransformerEngine/pull/377 from @janekb04.
Remaining tasks:

[x] FP8 scaling factor updates
[ ] ~Checkpointing~
[x] Documentation

Future work:

[ ] Operations: layer norm, activations, attention
[ ] Fusions
[ ] Possibly reimplementing the existing modules using this infrastructure

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

/te-ci pytorch

Merging with approval from @ksivaman, @sudhakarsingh27, @ptrendx. This feature is still experimental and incomplete.


Privacy Policy Page

Hi,
if you use your website, with a developed Wordpress plugin you have to have a "Privacy Policy Page", but unfortunately you don't have one. Can you add to this?
Sincerely yours,
Domisiding

Sounds good, I'll add one!
Spoiler alert: We keep haproxy and crafatar log files for less than a week for debugging purposes and possible abuse handling. And that's about it. We don't use Google Analytics or the likes, but traffic is passed through Cloudflare (for caching).


🌐 Add Chinese translation for docs/zh/docs/tutorial/dependencies/dependencies-with-yield.md

Added Chinese page for Dependencies with yield
添加了Dependencies with yield的中文页面

📝 Docs preview for commit 8081cffca068fa2dfb9c26b5549740df6c23cc52 at: https://317ebf21.fastapitiangolo.pages.dev

📝 Docs preview for commit 8081cffca068fa2dfb9c26b5549740df6c23cc52 at: https://317ebf21.fastapitiangolo.pages.dev

Please let me know when this is ready @zhiquanchi 🤓 ☕

📝 Docs preview for commit 5954a06cf1c3494de63a755ef67a054409b2c22b at: https://f3f63204.fastapitiangolo.pages.dev

Thanks for the interest! @zhiquanchi 🚀
Do you think this one is ready with your suggested changes applied? @Aruelius 🤓

Thanks for the interest! @zhiquanchi 🚀
Do you think this one is ready with your suggested changes applied? @Aruelius 🤓

Hi, maybe you should replace all the Python 3.6 to Python 3.8.

📝 Docs preview for commit 2f6b0a225a7bd414a6074dfd3a67121c21bdaeca at: https://6ea02cba.fastapitiangolo.pages.dev

Thanks for your help! @Aruelius @zhiquanchi 🚀✨


Permission issue on second docker-compose up

I could not use docker compose on my machine - I had to change to docker-compose everywhere. Otherwise I just followed the setup in the README.
On the first docker-compose up, everything worked fine. The server started, and I could visit it in a browser. The second time, I got multiple "permission denied" errors. I had to run chmod -R 777 . in order to be able to boot up.
Full backtrace: https://gist.github.com/danielittlewood0/a6762b0d411124bf6215da8a6573deed

Hi,
What OS are you using and how did you install Docker / Docker Compose? I'm just wondering what prevented you from using docker compose.
Also if you run id from your terminal, what output do you get?

@nickjj Gentoo linux; I installed docker ages ago but probably followed the instructions here: https://wiki.gentoo.org/wiki/Docker.
My docker version is 20.10.12 and my compose version is 1.29.2.
As for id:
$ id
uid=1001(daniel) gid=1001(daniel) groups=1001(daniel),7(lp),10(wheel),18(audio),19(cdrom),48(docker),85(usb),100(users),106(lpadmin),250(portage),272(plugdev)

Ah. I haven't used Gentoo in a long time but it looks like Docker Compose v2 is available at https://packages.gentoo.org/packages/app-containers/docker-compose. That'll install Docker Compose as a Docker plugin.
But, your error here is related to having a uid:gid that's not 1000:1000. I've left more details here: https://github.com/nickjj/docker-django-example/issues/18, while it's a Django project the same thing applies to Rails since this is a Docker issue.
I'm a little surprised that 777 works here, were you able to successfully update a CSS or JS file and see the changes with your volume mount?
You can test this by opening app/assets/stylesheets/application.tailwind.css and adding this to the bottom:
body {
  background-color: #00ff00;
}

You should be able to reload your browser and see a green background. Who ends up owning files in the public/ directory?
It's a tricky problem because it's due to how volume mounts work combined with user permissions on the Docker host. There's not much I can do to properly fix this.
You may run into this issue continuously tho when new files are created by Rails. For example if you up the project and then run ./run rails g model hey do you get a permission error again?

Thanks for bringing that to my attention! v2 is only available on the testing branch - I hadn't realised stable was so outdated :smiling_face_with_tear:
I am used to it being a file ownership issue - usually sudo chown -R daniel:daniel . will fix it - in this case I had to do chmod as well. I should have mentioned that I changed ownership first, but I was surprised that I had to do both.
I didn't update anything when I made the report. But I tried the update you suggested and refreshing didn't work; the server can't open this file for writing:
Permission denied @ apply2files - /app/tmp/cache/assets/sprockets/v4.0.0/yC/yCBlaVDE8E0WHr9DJ-e_2S6LI7Rl-32Jc_i6vGjq3Eo.cache

It's only when I chmod 777 again that it can access them (and then the screen changes colour). The owner of app/assets/builds is me - I guess if you are running the containers under user 1000 then it's not surprising that they can't write to my files.
I have seen this before with ownership issues, but I guess the containers I've been using run under root, where there is no issue for the container accessing files no matter who owns them.
Generating new files doesn't immediately raise problems for the server, but obviously I can't edit them. They're marked 644 and owned by user 1000 on my system.
I've never tried seriously to fix something like this (I don't tend to use rails generators, so the issue doesn't come up often). I guess one could provide a UID/GID at build time, and then at least the user who built the image could expect to be able to run it?
Nice project, by the way. And thanks for responding so quickly!

It gets tricky because it has to be set at build time and every example project I have except for Rails uses a base Node image which creates a node user for you beforehand but it doesn't let you set the uid/gid, they hard code it to 1000.
Basically I would need some solution that works for all of my example apps. It may involve having to have 2 sets of uid/gids, one for the assets stage and one for the main stage.

Generating new files doesn't immediately raise problems for the server, but obviously I can't edit them.

Right, this is a pretty big problem!

Hey, can you please do me a favor and try out this PR and report back whether it works or not in that PR? https://github.com/nickjj/docker-flask-example/pull/7
I know it's a Flask app and not Rails but I can do something similar for Rails too. I just want to make sure it works in that Flask example app because there's technically more moving parts than what it would be for this Rails app.

This is fixed in: https://github.com/nickjj/docker-rails-example/commit/6f6984697de53854a2c8e354f7bf7e5c62420606


Link in Bindings page is broken

The first link in this page is broken
<xref:Windows.UI.Xaml.Data.BindingOperations.SetBinding(Windows.UI.Xaml.DependencyObject%2cWindows.UI.Xaml.Dependency

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

ID: 1313ed23-758b-d84f-b3ea-38df54a695b6
Version Independent ID: 8b62eee5-b2d8-dd6a-f7ef-ba3d155c7418
Content: Binding Class (Windows.UI.Xaml.Data) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.UI.Xaml.Data/Binding.xml
Product: uwp
Technology: winui
GitHub Login: @jwmsft
Microsoft Alias: jimwalk

Link was fixed on Dec 9.

Link was fixed on Dec 9.


Add missing whitespace

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: While browsing through the docs this stood out to me as a typo. Fixed the typo.
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://665f722417da1c44822fa330--harness-developer.netlify.app


Fixes issue #10

Migrated the handler assignment logic from compile to link function, as the element reference was pointing to incorrect instance while placed inside an ng-if directive.
Also updated the test cases to wrap ng-if in dummy div elements, which otherwise was not getting compiled properly under Karma.

Thanks for this. I made this change myself after realizing that we had nothing to do in compile method, but was not getting the tests to pass...


ID widget doesn't restore going from small window to maximize

Small To Maximize
Screencast from 2023-06-06 02-02-21.webm

https://github.com/NickvisionApps/Denaro/releases/tag/2023.6.0-beta2


Further ambiguity breakers for UDC & SEDC

Without these ambiguity breakers, a decoding container that does not provide custom support for Int128 and UInt128 but conforms to both SingleElementDecodingContainer and UnkeyedDecodingContainer will fail to compile.

@swift-ci smoke test and merge

@swift-ci smoke test


Added utf-8 bom to json model files

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 95.708%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
3434

Relevant Lines:
3588

💛  - Coveralls


Add a Docker Compose File

I see a Docker File here, I would like to add on Docker-Compose file , then it would be more convenient to start this application in container, with just one command. Should I go for it, If you want?

It's a valid issue, that's why I am assigning you cause participants can't create issues.So please consider discussing issues on discord first then we will create them for u.
1.Add docker.yml properly.
2.Add a screenshot of the image this project running on docker in the PR

you are participating through swoc right ?

No, I came to Know about this project while searching and sorry for not discussing this on Discord First, I will remember this for next time.

It's ok , actually, these issues are under swoc , so as of now I am unassigning you ,later you can work on this project

Ok, one thing i would like to ask, Is registeration still open for swoc and can i work on this issue after getting registered? Just asking

Ok, one thing i would like to ask, Is registeration still open for swoc and can i work on this issue after getting registered? Just asking

no its currently closed

It's closed ...there will be one more open source in Jan named JGOC or Jwoc
so u can participate
On Sun, 26 Dec, 2021, 10:17 pm Ankita Puri, @.***>
wrote:

Ok, one thing i would like to ask, Is registeration still open for swoc
and can i work on this issue after getting registered? Just asking
no its currently closed
—
Reply to this email directly, view it on GitHub
https://github.com/ankitapuri/pixelvibe/issues/416#issuecomment-1001211429,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APRU7F2DCNRIUNVNXI2NTY3US5BLZANCNFSM5KXNH23Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Should addAccessibleInputListener automatically preventDefault?

In https://github.com/phetsims/circuit-construction-kit-common/issues/307 @jessegreenberg added preventDefault calls to prevent e.g., firefox from navigating pages back.  Should this be automatically implemented in addAccessibleInputListener so we don't end up scattering preventDefault everywhere?

I am afraid that preventing default in addAccessibleInputListener would be too aggressive.  For instance, calling event.preventDefault() on all keydown event listeners will also prevent use of tab key to go to the next item and ctrl + r to refresh the page.  We only want to preventDefault when specific keys are pressed in the listener.

Maybe we can factor out common interactions that will require us to prevent default. Like SimpleDragHadler we could have a KeyboardDeleteHandler (we can think of a better name later).  In CCK we have
        // on delete or backspace, the focused circuit element should be deleted
        if ( code === Input.KEY_DELETE || code === Input.KEY_BACKSPACE ) {

          // prevent default so 'backspace' and 'delete' don't navigate back a page in Firefox
          event.preventDefault();
          circuit.circuitElements.remove( circuitElement );
        }

Perhaps the API could look like
var myKeyboardDeleteHandler = new KeyboardDeleteHandler( {
  onDelete: function( event ) { /*do stuff*/ }
})
myNode.addAccessibleInputListener( myKeyboardDeleteHandler )

And the handler prevents  default for us so it isn't needed in sim code?

We could end up with a lot of these
KeyboardDeleteHandler
KeyboardSliderHandler
KeyboardDragHandler
KeyboardArrowKeyHandler
KeyboardNumberHandler
I am not sure how I feel about lots of these handlers. But at least the reasons for preventDefault would be isolated and documented in each one.

Perhaps we should see (or anticipate) what patterns come up frequently.  Perhaps an abstraction like:
new KeyHandler([Input.KEY_DELETE,Input.KEY_BACKSPACE],{
    preventDefault:true
},function(){ /*callback*/});

would work well?  It's difficult to say whether this would be necessary or useful.

Agreed, though an abstraction does seem useful.  I found 44 usages of addAccessibleInputListener, and inspection should about 1/4 use event.preventDefault() in some way.


Existing code no longer compiles when upgrading from 0.98.6 to 0.98.7

In response to this conversation: https://github.com/realm/realm-cocoa/commit/7a11e69c6e0a2af65d869a6fdb58a7611be70a0f
Here is the class that compiled in 0.98.6:
import Foundation
import ObjectMapper
import Realm
import RealmSwift

/**
 Base class for creating any API object from JSON
 */
public class APISchema: Object, Mappable, Equatable, Hashable {

    /**
     Mapping for this object, must be overriden in subclass
     */
    public func mapping(map: Map) {
        // override
    }

    /**
     Initialization
     */
    required public init(_ map: Map) {
        super.init()
    }

    /**
     Initialization
     */
    required public init(realm: RLMRealm, schema: RLMObjectSchema) {
        super.init(realm: realm, schema: schema)
    }

    /**
     Initialization
     */
    required public init() {
        super.init()
    }
}

/**
 Conform to equatable
 */
public func ==(lhs: APISchema, rhs: APISchema) -> Bool {
    return lhs.isEqual(rhs)
}

And here is the initializer I needed to add to compile on 0.98.7:
/**
 Initialization
 */
required public init(value: AnyObject, schema: RLMSchema) {
    super.init(value: value, schema: schema)
}

Okay, given that this object class is defined in Swift, this breaking compile-time change protects you from using this class in a way, which would have caused runtime errors before because required constructors wouldn't be correctly auto-inherited. I guess, you were not using this type in lists or queries? Because these would not have worked before and would have caused crashes.

@AndrewBarba can you use convenience initializers instead? Should be simpler: https://realm.io/docs/swift/latest#adding-custom-initializers-to-object-subclasses

@mrackwitz I heavily use these objects in lists and queries, and have a pretty hefty set of unit tests that's been passing for a number of realm versions now.
@jpsim The Mappable protocol requires the init(_ map: Map) initializer, once I add that the Swift compiler forces me to make that required and then it forces me to add all of the required initializers after that.

Ok, well nonetheless, this is all working as expected then. I see no reason to keep this issue open, unless you feel otherwise

No just wanted to give you guys a heads up. If you feel the need to investigate why something was working when it you don't think it should have been, just let me know, happy to share more code in a private thread.

Whoops, just noticed that you had already init(realm: RLMRealm, schema: RLMObjectSchema) defined before, which wasn't enforced to be there by the compiler before that change.
Thanks a lot for providing insides here. But what you're facing here is at least expected and all this initializers are actually required at runtime, while the latter might not be required for the specific subset of the API you use. Sorry for the inconveniences through that. But by defining it, you're definitively on the safe side.


How to upgrade platformio

How would I upgrade the docker container to a newer version of Platformio?

I triggered a new build now. It should have done it every month, but it seems GHA now disables workflow if there is no new commits in the repo within 60 days.

If you want to build it in the future, download docker and run docker build locally.

Thank you


Question/Feature Request: Adding multiple elements into a single cell

Hi, I using easytable from past few months, it is amazing. But, I found difficulty in adding multiple elements to a single cell. Like, adding multiple differently styled text in same cell and adding both image and text in same cell. I tried creating a custom cell following the Q/A in readme but was unable to do the bove things. Dont know if this feature is not present or I was not able to get the thing even it is possible.

Hi @KVSRAKESH
glad you like the library.
It's possible to create a cell type of its own in order to achieve what you want. But that's a bit more involved. Maybe check out the different cell type implementations for a little inspiration. You can modify what is drawn by a custom drawer class and there you can access the content stream with the coordinates of the current cell.
Best,
Stefan


Rule suggestion: require await of async functions

For example:
async function foo(){
}
async function bar(){
    foo(); // I need absent await warning
    return 234;
}

I need just simple example how to start, find method declaration and check that is async function

I'm not sure off the top of my head what exactly you'll need to do, but atom-typescript has a cool AST viewer which helps a lot when trying to write lint rules (see image below).
However, I'm not sure this is possible at the moment. TSLint lints files on a file by file basis currently, so it your function comes from another file, it has no way of knowing that it's an async function.
If you're only dealing with function declarations and calls from the same file, I imagine you could do something like this:
Override visitCallExpression and find all references to that function in the file, similarly to here. If you can find a reference that's a function declaration / function expression, see if it contains the async flag. If it does, and if the call expression isn't part of an async statement, generate a lint failure. This is a very rough sketch though, I'm sure there'd be more complications / there might be better language services features you can use.

blocked on #680

We just hit a bug where we weren't properly awaiting an async void function. This lint rule would be great!
One addition to the rule: it'd be nice if the rule automatically saw if you were legitimately wanted to run something in parallel, not await it — but you should still be thening or catching errors.
async function foo() { ... }

async function bar() {
    // This should fail:
    foo();

    // But these should pass:
    await foo();

    foo().then(
        () => { ... },
        (err) => { ... }
    )

    foo()
        .catch((err) => { ... });

    // Maybe ideally this should fail too, since it could lead to silent errors:
    foo().then(() => { ... });

yep, this feature request is now unblocked, accepting PRs

@JoshuaKGoldberg @cevek I still suspect this is resolved by #1632. See my comment: https://github.com/palantir/tslint/pull/1632#issuecomment-273357419

Agreed. I couldn't think of any situations in which #1632's didn't catch this behavior. Thanks for the bump!

Here's one situation that's not covered. I've just hit a bug as follows:
let value = some_async_function(); // forgot to await!
if (value === null) {
  // will never happen
} else {
  // will always happen
}

This is not the first time, and not the last. no-floating-promises does not cover this, and TypeScript doesn't tell me that I'm comparing Promise<x> to null, which is probably not what I intended.
I propose a consume-promises rule, that will require each promise to either have a p.then(success, error), p.then(success).catch(error) or await p

@tomholub have you tried enabling strict-boolean-expressions? It should complain on the value === null comparison.

@tomholub I think more better to disable primitive operations, to boolean checks and check against null and undefined
It is cover all buggy cases
async foo() {
    var b = bar();
    if (b) some();
    !b ? some() : some();
    x(+b);
    x(b + 'foo');
    if (b == null) x();
    if (b === undefined) x();
}

@bennyn This section is technically not a floating promise, since you handle it:
awaitMe() ? true : false

strict-boolean-expressions would handle that, since you're comparing a Promise in the ternary instead of the expected boolean. See discussion in https://github.com/palantir/tslint/issues/4117.
This is a common concern brought up - a little more +1 to making strict-boolean-expressions more configurable so more folks can use it :)

no-floating-promises doesn't work with the situation like this
   function awaitMe() {
       return new Promise(() => {
             resolve();
       });
   }

  awaitMe(); // gets warning from no-floating-promise
  const value = awaitMe(); // doesn't get any warning

Any solution to handle this?


fix: possible race on artefact upload

fixes: #2367

You mean to maintain an updated list of new artifacts in case there are multiple failures? I thought about I was not sure if it was worth extra complexity.


Test issue

Should the test be assert(Bundle(identifier:"org.cocoapods.StaticFrameworkResources") != nil)?
My understanding is the string should be the one from the bundle's Info.plist
I have a CocoaPods PR on the way that fixes the resource copy, but still doesn't run this app correctly.

With https://github.com/CocoaPods/CocoaPods/pull/7100, changing the test to

   assert(Bundle.main.path(forResource:"StaticFrameworkResources.bundle/banana", ofType: "png") != nil)

will succeed


lrzip: 0.650 -> 0.651

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/lrzip/versions.
meta.description for lrzip is: "The CK LRZIP compression program (LZMA + RZIP)"
meta.homepage for lrzip is: "http://ck.kolivas.org/apps/lrzip/"
meta.changelog for lrzip is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/1iff243blz6cy5gzk6ixi224ij5ndp1q-lrzip-0.651/bin/lrzuntar had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/1iff243blz6cy5gzk6ixi224ij5ndp1q-lrzip-0.651/bin/lrz had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/1iff243blz6cy5gzk6ixi224ij5ndp1q-lrzip-0.651/bin/lrzip had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/1iff243blz6cy5gzk6ixi224ij5ndp1q-lrzip-0.651/bin/lrunzip had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/1iff243blz6cy5gzk6ixi224ij5ndp1q-lrzip-0.651/bin/lrzcat had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/1iff243blz6cy5gzk6ixi224ij5ndp1q-lrzip-0.651/bin/lrztar had a zero exit code or showed the expected version
0 of 6 passed binary check by having a zero exit code.
0 of 6 passed binary check by having the new version present in output.
found 0.651 with grep in /nix/store/1iff243blz6cy5gzk6ixi224ij5ndp1q-lrzip-0.651
found 0.651 in filename of file in /nix/store/1iff243blz6cy5gzk6ixi224ij5ndp1q-lrzip-0.651

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

7 total rebuild path(s)

6 package rebuild(s)

6 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
libsForQt5.ark
libsForQt512.ark
libsForQt514.ark
libsForQt515.ark
lrzip
plasma5Packages.ark

Instructions to test this update (click to expand)

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

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

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

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

2 packages marked as broken and skipped:

libsForQt512.ark
libsForQt514.ark

2 packages built:

libsForQt5.ark
lrzip

Maintainer pings

@ofborg eval

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

3 packages marked as broken and skipped:

ark
libsForQt512.ark
libsForQt514.ark

2 packages built:

libsForQt5.ark
lrzip


Implement IEC 61853 IAM calculations for diffuse irradiance

[x] Adresses issue #751
[x] I am familiar with the contributing guidelines.
[ ] Fully tested. Added and/or modified tests to ensure correct behavior for all reasonable inputs. Tests (usually) must pass on the TravisCI and Appveyor testing services.
[ ] Updates entries to docs/sphinx/source/api.rst for API changes.
[ ] Adds description and name entries in the appropriate docs/sphinx/source/whatsnew file for all changes.
[ ] Code quality and style is sufficient. Passes LGTM and SticklerCI checks.
[ ] New code is fully documented. Includes sphinx/numpydoc compliant docstrings and comments in the code where necessary.
[ ] Pull request is nearly complete and ready for detailed review.

Thanks @adriesse this is a nice addition. It will go into the iam.py module when #783 is merged.  @wholmgren I think I have addressed the review comments for #783, test_get_psm3 still fails due to NSRDB-side data.

Ok, I hadn't really been paying attention to what you were doing.
Would it be possible to split these kinds of re-orgs into steps?  Like perhaps 1) add iam module with new functions 2) adjust your classes 3) remove the existing functions?


libovsdb doesn't have a way to unregister a monitor

How about to add some function to unregister a NotificationHandler from ovs.handlers?

Fixed in 8fc30131bd541483827ce090eacfc5b92964e8a5


gRPC probe may not run enough times in case of connect failures

When connect fails, after a timeout, we just wait for another interval:
https://github.com/cloudprober/cloudprober/blob/6320b294730c82d76ed7b2df70ac0cb66ea80aa7/probes/grpc/grpc.go#L315
Say if interval and timeout both are 10s, connect will be attempted only 3 times in a minute instead of 6 times as per expectation. We should fix this behavior.
Assigning to myself as I worked on this code recently.

#815 as a possible fix.


Change a parameter name in percentile and quantile to support NumPy 1.22

Partially fix #6198.
percentile and quantile's interpolation parameter is renamed to method in NumPy 1.22, and some new interpolation methods are introduced as well. In this PR, I cover the change in the parameter name only to support NumPy 1.22, and will open another issue to implement the newly introduced methods.

/test mini

/test mini

/test head

@takagi Could you manually make a v10 backport of this PR excluding interpolation support (i.e., only fix tests to ignore DeprecationWarning)?

Simple cherry-pick seems fine because no method value is newly supported.
To backport only tests, the change to v10 would be

@testing.with_requires('numpy<1.22')
class TestQuantile:
    ...

class TestOrder:
    ...


Merge pull request #1109 from Aguafrommars/master

.Net 7.0.13 patch

:white_check_mark: Build TheIdServer 7.4.5+85.build.3387 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/f6d7431db7 by @aguacongas)

:x: Build TheIdServer 7.4.6.build.3393 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/dc0fc305a7 by @semantic-release-bot)


biling: Avoid slow jQuery :selected selector extension

:selected is one of several jQuery-specific selector extensions that force jQuery away from the native document.querySelector fast path onto the Sizzle slow path, and also present an extra obstacle to migrating away from jQuery. Replace :selected by simply using .val()/.value or .selectedOptions on the <select> control.
Similarly, replace :input with HTMLFormElement.elements.
Cc @amanagr (#27673, #28939)

Followup to #25389.

OK, merged, thanks @andersk!


handles noarch builds

This change handles hw_info.log collection for "noarch" builds

All tests are passing with this minor change. This should help solve part of the problem we have seen with hw_info failing to be collected for some hosts ( issue #65 )


My colors changed after the update

Is this a temporary thing?

I "synced" it with the Sublime version, so yeah, some things might have changed. You can always install a previous version through APM:
apm install<EMAIL_ADDRESS>

cool. thanks.

For anyone else wondering why JS keywords  were suddenly italics, this was it.

Yeah, I'm starting to actually write the themes from scratch rather than just converting from sublime's version to avoid these kind of issues in the future.
The converted theme is pretty hard to maintain because the auto-generated CSS code isn't too good.
Stay tuned for future updates on all variants of the syntax theme 😄

Thank you @silvestreh Waiting for forever for the update!


fix: XCode 14.3 temp issues with config plugin

Why
Our App can't compile with the new XCode 14.3 currently.

How
Looks like there are packages which haven't updated their podspecs to target the correct deployment target. Added a Config plugin to fix this issue for now, since build properties alone are not enough to fix this.
Credit goes to: https://github.com/facebook/react-native/issues/34106#issuecomment-1493040686
The upstream bugfix is already merged but I don't know if this is going to be backported, so we need to monitor new releases.
https://github.com/facebook/react-native/pull/36759

Test Plan

hi there! hope react native will release a new patch version with the fix this week and we will update with a new expo package then. let's see if react native could release the fix this week.


DhruvDinesh

Pairplot incorrect

Upload a folder containing the csv and txt files. The folder name must be your name.


is src/kernel/expr.h the right reference for the lean AST?

partly because im poking at code extraction facilities in a few provers, is that the rep that would perhaps be used if one were to do a code extraction facility for lean?
https://github.com/leanprover/lean/blob/master/src/kernel/expr.h
thanks
-Carter

We already do (some) code extraction in Lean for evaluation in the VM, see the compiler and VM code.  This VM implements a straightforward stack machine; types, propositions, and proofs are erased during compilation.  You can inspect the internal bytecode representation by using set_option trace.compiler.optimize_bytecode true above the definition that you want to see.
If you want to experiment with your own code extraction techniques, you could start from expressions in the kernel type theory.  You can access them via the environment object from inside Lean, see our recent paper on metaprogramming.  The inductive definition for the kernel expression type is here.
@jroesch is working on a native compiler for Lean using this metaprogramming facilities.
I'll close this issue since it is not a bug report, but feel free to ask if have any questions.  You can also use our gitter channel.

(and the answer is yes, but see also the lean definition of exprs too for an easier high level view: https://github.com/leanprover/lean/blob/master/library/init/meta/expr.lean#L38)


Support for Broadcast

The current DeepShiba does not support broadcast.
However, it is a very powerful feature and is needed when defining complex functions.
It should be added as soon as possible.

I've implemented a broadcast function that works for now, but it's very slow and needs improvement.
Soure:
function set_same_dim(x::AbstractArray, shape)
    shape = [shape...]
    if length(size(x)) == length(shape)
        return (x, Tuple(shape))
    elseif length(size(x)) < length(shape)
        set_same_dim(reshape(x, (size(x)..., 1)), shape)
    else
        set_same_dim(x, pushfirst!(shape, 1))
    end
end

The dimensions of the input are aligned by this function.
And it is repeated by this function.
function broadcast_to(x, shape)
    x, shape = set_same_dim(x, shape)
    rep_count = ((x, shape) -> (Int(shape / x))).(size(x), shape)
    return repeat(x, (rep_count...))
end

broadcast_to(x, shape) behavior is exactly the same as x .+ zeros(shape) behavior.
A comparison of these speeds shows that
a = rand(1:10, (4, 100, 4, 4))
b = [1, 2, 3, 4]

@benchmark broadcast_to(b, size(a))

#=
BenchmarkTools.Trial:
  memory estimate:  397.34 KiB
  allocs estimate:  8127
  --------------
  minimum time:     706.482 μs (0.00% GC)
  median time:      770.686 μs (0.00% GC)
  mean time:        865.929 μs (4.13% GC)
  maximum time:     6.767 ms (83.20% GC)
  --------------
  samples:          5756
  evals/sample:     1
=#

@benchmark b .+ zeros(Int, (size(a)...))

#=
BenchmarkTools.Trial:
  memory estimate:  100.41 KiB
  allocs estimate:  11
  --------------
  minimum time:     10.796 μs (0.00% GC)
  median time:      50.540 μs (0.00% GC)
  mean time:        66.957 μs (19.16% GC)
  maximum time:     7.104 ms (99.34% GC)
  --------------
  samples:          10000
  evals/sample:     1
=#

Julia's broadcasts are fast using latency ratings and other techniques, but it is too slow and need to be improved.


[fix] SQL parsing of table names

CATEGORY
Choose one

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

SUMMARY
Presto supports using the optional cluster when specifying the table name, i.e., [[cluster.]schema].table however the currently SQL parse logic only returns at most the first two components, i.e., cluster.schema if a cluster was defined.
This PR extends the logic to support the [[cluster.]schema].table construct as well as:

Ensures the table construct is valid in terms of length and structure.
Ensures that the quotes are removed from each token. Previously this was only the case when only the table was specified which were removed via get_real_name().
The get_real_name() logic wasn't overly apparent though it's used when a table alias is present. I felt this logic wasn't overly clear and in the case of a TokenList representing schema.table AS foo the get_real_name() only returns table (though this wasn't invoked because there was the . token). From a table construct validity perspective it seems more prudent to strip the alias and then extract the full table name if valid.

TEST PLAN
CI.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
to: @graceguo-supercat @michellethomas @mistercrunch

Codecov Report

Merging #7490 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #7490      +/-   ##
==========================================
+ Coverage   65.42%   65.43%   +0.01%
==========================================
  Files         435      435
  Lines       21323    21331       +8
  Branches     2347     2347
==========================================
+ Hits        13950    13958       +8
  Misses       7257     7257
  Partials      116      116

Impacted Files
Coverage Δ

superset/sql_parse.py
99.2% <100%> (+0.05%)
:arrow_up:

Continue to review full report at Codecov.

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


Times in timepicker on disabled dates are not disabled

I have timepicker enabled and have initialize the datetimepicker with minDate/maxDate. When click on previous/next month and jump on some disabled date, I am still able to select times which results in the times in disabled dates being selected.

Expected behavior: all times in disabled dates are disabled.

Subscribe!

I'm having the issue. Is there a workaround for this?


Reduce reference sample collection time by using loop analysis

During reference sampling, in circuits with loops that have a large number of repetitions, the instantaneous stabilizer group can start to recur at the loop boundaries. This can be used to effectively skip simulating the remaining iterations by repeating the reference sample from the previous iterations.
This could noticeably reduce measurement sampling times at large code distances, depending on whether it was the data measurement layers or the stabilizer measurement layers dominating the execution time.
There may be an intermediate regime, where the unsigned tableau recurs but the signs differ. Unsure what's safe to do in that situation.

It turns out that this doesn't work consistently enough to be worth the 50% loss in performance when loop folding fails.
For example, surprisingly, the distance 5 planar honeycomb has a period of at least 100_000 iterations! Something to do with the observable moving around no doubt.

Since this doesn't work as expected, closing as WONTFIX. I put the test code on a tag: https://github.com/quantumlib/Stim/releases/tag/FEATURE-measurement-sample-period-folding


Update constructors for resources CatalogSrouce, ClusterRoleBinding, Configmap, ControllerRevision, CronJob, CSIStorageCapacity, DataSource, DV and DataImportCron

Short description:
This pr covers c-d resources, todo continue with e...z

/wip

@oharan2 I strongly suggest you split this PR into multiple PRs

Splitting this PR


add MCP server badge

This PR adds a badge for the HubSpot MCP Server server listing in Glama MCP server directory.

Glama performs regular codebase and documentation scans to:

Confirm that the MCP server is working as expected
Confirm that there are no obvious security issues with dependencies of the server
Extract server characteristics such as tools, resources, prompts, and required parameters.

This badge helps your users to quickly asses that the MCP server is safe, server capabilities, and instructions for installing the server.

Thanks! are you on the MCP Discord?
Let me know your Discord username and I will give you a special flair that's only for MCP server authors.

Yes! This is me #buryh
And thanks!
On Fri, Dec 27, 2024 at 11:31 Frank Fiegel @.***> wrote:

Thanks! are you on the MCP Discord https://glama.ai/mcp/discord?
Let me know your Discord username and I will give you a special flair
that's only for MCP server authors.
—
Reply to this email directly, view it on GitHub
https://github.com/buryhuang/mcp-hubspot/pull/1#issuecomment-2563974957,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJRU3B2OQDESOEVIFPDIHP32HWTK7AVCNFSM6AAAAABUIXAKS6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRTHE3TIOJVG4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Add unit test for zero fee transaction.

Some time ago there was a bit of thrashing around regarding the lone zero-fee transaction we support: creating the first regular key on an account.  We determined that you could only attach the regular key for free if you signed with the master key.  You can't attach a regular key to an account for free if you multisign.
Later I realized that we did not have that scenario covered in the unit tests.  Now it will be.
Reviewers: @ximinez, @nbougalis

Current coverage is 49.15%

Merging #1378 into develop will increase coverage by +0.03% as of 660673e

@@            develop   #1378   diff @@
=======================================
  Files           960     960
  Stmts         66698   66714    +16
  Branches          0       0
  Methods           0       0
=======================================
+ Hit           32763   32790    +27
  Partial           0       0
+ Missed        33935   33924    -11

Review entire Coverage Diff as of 660673e

Powered by Codecov. Updated on successful CI builds.

Good test. :+1:

:+1:

Merged as a8cc9033b49be348acf6053f01ed924703de344e.


Dev: Web - Workspace- Date is not clickable under Personal information page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open app
Go to settings > workspace >Bank account > connect manually
Fill the form until step 3
Select todays date - (in my case 2005-06-10) for the date of birth
Change the date to 2005-06-25

Expected Result:
Date is clickable under Personal information page
Actual Result:
Date is not clickable under Personal information page
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.26-1
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/cd8fa269-b7a1-4c35-a8e4-b85a5ec0ccbb
Expensify/Expensify Issue URL:
Issue reported by: @narzullome
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686396762249499
View all open jobs on GitHub

Got this sorted, btw @kbecciv for issues like this that are already in production, no need to add Dev to the title! 🙇♂

Unable to reproduce - @kbecciv can you confirm if still reproducible on your side?

@dylanexpensify Pls select todays date - for a year select 2005
For ex - 2005 -06-20 with this dates and year - just now I was able to reproduce

Reproduced

I think it is expected behavoir since user must be 18 years old, and any date of birth younger than this is disabled.
Maybe the problem is that datepicker in safari is not showing disabled these dates where in chrome it does

Hi, It's working fine on Chrome and Safari as well.
https://github.com/Expensify/App/assets/61835503/4e6a9bde-590e-4010-9184-d6f23d34d724

@dylanexpensify We can close this one. All date pickers will soon be replaced by our custom push to page date picker.

Not overdue. @dylanexpensify ^

Ahh nice! Thanks @s77rt !


Fix --use-core for functions, since we used the aster helper.

r? @fitzgen, @nox or anyone else?

 Warning

These commits modify unsafe code. Please review it carefully!

@bors-servo r=fitzgen

:pushpin: Commit 6645bbc has been approved by fitzgen

:zap: Test exempted - status


Making sure the checks are properly set up

For assignment 6

@JayChouteau Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

This looks like a work assignment that shouldn't lead to the creation of a Pull Request to this repository. Could you please reach out to the person responsible so that they clarify this part of the assignment? This creates unnecessary noise to this repository. Thanks.


Confusion about Boston Local vs OV IDs and thumbs missing thumbnail

thumbnails were dumped into s3 catalog for which there is no corresponding OV record

https://s3.amazonaws.com/openvault.wgbh.org/catalog/asset_thumbnails/V_UDAMVZGA4JEY06N.png exists
but http://ov-qa1.wgbh-mla-test.org/catalog/V_UDAMVZGA4JEY06N is 404

(Note that OV IDs are longer, and consist only of hex digits.)
This will be confusing over the long run.

The only thumbs under s3 catalog should be jpgs with OV IDs as their name
If that is not the case, they should go somewhere else.

S3 should be cleaned up, and there should be another PR pointing to the correct locations, and all 6 images should 4. (Only 5 do right now.)

@mccalluc: Fixed. Went back to my original plan and put the images in the special collections folder.

Waiting on PR.

@mccalluc: it's the fix-urls-1 and fix-urls PRs. (one for Open Vault and other for AAP[&#X1f60e]

looks good.


Strange behavior with sequence on re-bind

I am using CarouselView with data binding. My ItemsSource is bound to ObservableCollection and Position is bound to integer Position property. Everything works fine on first load. i.e. if I have "Item1", "Item2" and "Item3" in my collection, they show up in the right sequence. Then when I change the collection in my view model by clearing collection and then adding new items I get carousel showing old items. In particular, if I delete "Item1", "Item2" and "Item3" and add "Item4", "Item5" and "Item6", then my view model has an updated collection, but carousel is showing "Item5", "Item6" and "Item1"??? Not, it ignored first new "Item4" and appended old item "Item1" that was removed.
Here is my code:
XAML
<controls:CarouselViewControl Orientation="Horizontal" InterPageSpacing="0"
						Position="{Binding Position}" ItemsSource="{Binding ServiceProviders}"
						VerticalOptions="FillAndExpand" HorizontalOptions="FillAndExpand">
			<controls:CarouselViewControl.ItemTemplate>

ViewModel
this.Position = 0;
				await Task.Delay(200);
				for (var i = this.ServiceProviders.Count - 1; i >= 0; i--)
				{
					await Task.Delay(100);
					this.ServiceProviders.Remove(this.ServiceProviders[i]);
				}

				foreach (var result in results?.Value ?? new List<ServiceProviderSearchResultV1>())
				{
					this.ServiceProviders.Add(result);
				}

private ObservableCollection<ServiceProviderSearchResultV1> serviceProviders =
			new ObservableCollection<ServiceProviderSearchResultV1>();
		public ObservableCollection<ServiceProviderSearchResultV1> ServiceProviders
		{
			get
			{
				return this.serviceProviders;
			}
			set
			{
				if (this.serviceProviders != value)
				{
					this.serviceProviders = value;
					this.NotifyOfPropertyChange(() => this.ServiceProviders);
				}
			}
		}

I don't understand how it gets items taht are deleted. It is like it is shifted by one.

You are right. ObservableCollection CollectionChanged.Clear is not implemented.

But I am not using clear, I am doing for loop with remove

Thats exactly the problem :)

Can you try this?
while (list.Count > 0)
{
     list.RemoveAt(list.Count - 1);
}

Thanks, I will try this. In fact, can I just use normal list, not the observable collection?

You can but your wont be able to add and remove from the list. Only setting ItemsSource will refresh the UI.
I am implementing ObservableCollection Clear, Move and Replace actions.

Thought so. I hope RemoveAt will fix it for me. What is an ETA on clear? Also, when is xamarin adding your carousel?

ETA tomorrow. Carousel in Xamarin core? Dont know yet. PR is on idle state.

Unfortunately, RemoveAt is doing the same as my previous code.
First load I have
ServiceProviders collection: "Provider1", "Provider2", "Provider3",
Carousel: "Provider1", "Provider2", "Provider3"
User clicks search, I use you code snippet with "while" and "remove at"
ServiceProviders collection: "Provider4", "Provider5", "Provider6",
Carousel: "Provider5", "Provider6", "Provider1"
out of sync

If you need to Clear the list and start adding new elements, try this:
myCarousel.ItemsSource = new ObservableCollection<object>();
myCarousel.ItemsSource.Add(obj);

I can't do it. Because all my code is in viewmodel and I am binding to an observable collection

Not a problem. Assign the new collection in your VM.

All CollectionChanged Actions implemented. Will release soon.

Hey @alexrainman, do you have an ETA for when the new CollectionChanged actions will be released? I'm running into the same problem, and resetting the ItemsSource to a new ObservableCollection doesn't seem to resolve the issue for me, since bindings in the child views no longer update afterwards.

In a couple of days. I have to finish UWP implementation.

Great work, btw. Looking forward to when your PR gets merged into Xamarin Forms core.

Im looking forward to it also. I dont have time to maintain this repo anymore :(

Done.

public ObservableCollection flipperGetters;
flipperGetters = new ObservableCollection();
flipperGetters.Clear()
carousel.ItemsSource = flipperGetters;
After clearing and adding the carousel.ItemsSource is getting arrayondexoutofbound exception how to sove this problem. if i remove(flipperGetters.Clear()) it is working fine.


Update serverpod_auth dependency versions

Fixes #732.
Pre-launch Checklist

[X] I read the Contribute page and followed the process outlined there for submitting PRs.
[X] This update contains only one single feature or bug fix and nothing else. (If you are submitting multiple fixes, please make multiple PRs.)
[X] I read and followed the Dart Style Guide and formatted the code with dart format.
[X] I listed at least one issue that this PR fixes in the description above.
[X] I updated/added relevant documentation (doc comments with ///), and made sure that the documentation follows the same style as other Serverpod documentation. I checked spelling and grammar.
[X] I added new tests to check the change I am making.
[X] All existing and new tests are passing.
[X] Any breaking changes are documented below.

@lukehutch I'm not sure why the tests aren't running on this one. But you will also need to edit the pubspec in the templates directory as Serverpod uses that one to generate the actual pubspec used by the package.

Apart from that, it looks great!

@lukehutch I'm not sure why the tests aren't running on this one. But you will also need to edit the pubspec in the templates directory as Serverpod uses that one to generate the actual pubspec used by the package.

Thanks, I fixed both of the issues you raised.


Dependant Data Sources return no results

https://github.com/symphonycms/symphony-2/commit/cbeca16ee0ff34c975683b6f2bd11ee728f65579 has broken dependant datasources, they may return no results. The problem is in this line

But why???

The problem is definitely in class.frontendpage.php. But I don't manage to get it right.

Thanks!

I am lost. :-(

@michael-e check this :)

Thanks! Will you send a pull request to replace the quotes?

There: 16dc3b8 Like a cowboy.

:-)