shape of atten_scores

Hi,
tanks for this package!
I was replicating your example and I got an error in the last step when I call find_motifs.py
Traceback (most recent call last):
File "DNABERT/motif/find_motifs.py", line 110, in
main()
File "DNABERT/motif/find_motifs.py", line 91, in main
pos_atten_scores = atten_scores[dev_pos.index.values]
IndexError: index 5918 is out of bounds for axis 0 with size 5918
it seems there is a mismatch between the shape of the atten_scores/pred_results
atten_scores.shape - > (5918, 81)
pred.shape - > (5918, 81)
and the dev file shape
dev.shape  - > (5919, 81)
any idea what is causing this?
cheers
Michele

Thanks for letting me know! Our data reader regards the first line of .tsv file as a header and automatically skip it, and I forget to add a header for the sample data. That's why one data sample is missed. Please add a random header to the sample file, and it should work fine. I will change the sample data soon.

Hi, Thanks for the prompt response!
can you also please check the example code to run the step:

Motif analysis
I think is truncated at the end, probably missing the --return_idx flag

Hi @mtinti, thanks for pointing that out! We have removed the trailing '' at the end of the command.
For the purpose of running the example scripts, --return_idx flag is not necessary. It is only needed if you explicitly want the indices of those motifs.

Hi, thanks again to double check.
I'm getting this error now:
*** Begin motif analysis ***

pos_seqs: 0; neg_seqs: 0
Traceback (most recent call last):
File "DNABERT/motif/find_motifs.py", line 110, in
main()
File "DNABERT/motif/find_motifs.py", line 106, in main
return_idx  = args.return_idx
File "/content/DNABERT/motif/motif_utils.py", line 475, in motif_analysis
max_seq_len = len(max(pos_seqs, key=len))
ValueError: max() arg is an empty sequence

I think that the sequences in dev.tsv are not correctly tabbed separated, am I correct?

I would also like to report this bug.  Step 6 Motif finding does not work, even with the provided example data and ocmmands.  I really hope this bug can be fixed, becasue DNABERT is working ou great so far, I don’t want to be stopped at the last step.

The solution to the error posted by @mtinti on 19 Oct 2020 is the following:
I modified the script 'find_motifs.py', and now it runs without error. See below. However, no motifs are output for the provided example data/commands.
import os
import pandas as pd
import numpy as np
import argparse
import motif_utils as utils

def main():
    parser = argparse.ArgumentParser()
    parser.add_argument(
        "--data_dir",
        default=None,
        type=str,
        required=True,
        help="The input data dir. Should contain the sequence+label .tsv files (or other data files) for the task.",
    )

    parser.add_argument(
        "--predict_dir",
        default=None,
        type=str,
        required=True,
        help="Path where the attention scores were saved. Should contain both pred_results.npy and atten.npy",
    )

    parser.add_argument(
        "--window_size",
        default=24,
        type=int,
        help="Specified window size to be final motif length",
    )

    parser.add_argument(
        "--min_len",
        default=5,
        type=int,
        help="Specified minimum length threshold for contiguous region",
    )

    parser.add_argument(
        "--pval_cutoff",
        default=0.005,
        type=float,
        help="Cutoff FDR/p-value to declare statistical significance",
    )

    parser.add_argument(
        "--min_n_motif",
        default=3,
        type=int,
        help="Minimum instance inside motif to be filtered",
    )

    parser.add_argument(
        "--align_all_ties",
        action='store_true',
        help="Whether to keep all best alignments when ties encountered",
    )

    parser.add_argument(
        "--save_file_dir",
        default='.',
        type=str,
        help="Path to save outputs",
    )

    parser.add_argument(
        "--verbose",
        action='store_true',
        help="Verbosity controller",
    )

    parser.add_argument(
        "--return_idx",
        action='store_true',
        help="Whether the indices of the motifs are only returned",
    )

    # TODO: add the conditions
    args = parser.parse_args()

    atten_scores = np.load(os.path.join(args.predict_dir,"atten.npy"))
    pred = np.load(os.path.join(args.predict_dir,"pred_results.npy"))
    dev = pd.read_csv(os.path.join(args.data_dir,"dev.tsv"),sep='\t',header=0)
    #dev.columns = ['sequence','label']
    #dev['seq'] = dev['sequence']
    dev['sequence'] = dev['sequence'].apply(utils.kmer2seq)
    dev_pos = dev[dev['label'] == 1]
    dev_neg = dev[dev['label'] == 0]
    #print(dev_pos)
    #print(dev_neg)
    pos_atten_scores = atten_scores[dev_pos.index.values]
    neg_atten_scores = atten_scores[dev_neg.index.values]
    assert len(dev_pos) == len(pos_atten_scores)

    #print(dev_pos['sequence'])

    # run motif analysis
    merged_motif_seqs = utils.motif_analysis(dev_pos['sequence'],
                                        dev_neg['sequence'],
                                        pos_atten_scores,
                                        window_size = args.window_size,
                                        min_len = args.min_len,
                                        pval_cutoff = args.pval_cutoff,
                                        min_n_motif = args.min_n_motif,
                                        align_all_ties = args.align_all_ties,
                                        save_file_dir = args.save_file_dir,
                                        verbose = args.verbose
                                        #return_idx  = args.return_idx
                                    )

if __name__ == "__main__":
    main()

FYI - After editing the script as per above,the motif finding procedure worked on our data and identified motifs. GREAT! Thank you very much for the awesome tool! However, if I set the --max_seq_length to 56 instead of 64 during the (4) prediction and (5) visualization steps, then the following eror is thrown:
python find_motifs.py \
>     --data_dir $DATA_PATH \
>     --predict_dir $PREDICTION_PATH \
>     --window_size 24 \
>     --min_len 5 \
>     --pval_cutoff 0.05 \
>     --min_n_motif 3 \
>     --align_all_ties \
>     --save_file_dir $MOTIF_PATH \
>     --verbose
*** Begin motif analysis ***
* pos_seqs: 17938; neg_seqs: 18061
* Finding high attention motif regions
* Filtering motifs by hypergeometric test
Traceback (most recent call last):
  File "find_motifs.py", line 116, in <module>
    main()
  File "find_motifs.py", line 111, in main
    verbose = args.verbose
  File "/home/joneill/DNABERT/motif/motif_utils.py", line 514, in motif_analysis
    **kwargs)
  File "/home/joneill/DNABERT/motif/motif_utils.py", line 233, in filter_motifs
    pvals = motifs_hypergeom_test(pos_seqs, neg_seqs, motifs, **kwargs)
  File "/home/joneill/DNABERT/motif/motif_utils.py", line 196, in motifs_hypergeom_test
    motif_count_all = count_motif_instances(pos_seqs+neg_seqs, motifs, allow_multi_match=allow_multi_match)
  File "/home/joneill/DNABERT/motif/motif_utils.py", line 152, in count_motif_instances
    matches = sorted(map(itemgetter(1), A.iter(seq)))
AttributeError: Not an Aho-Corasick automaton yet: call add_word to add some keys and call make_automaton to convert the trie to an automaton.


Check invalid rules with DAL before fetching

1. Why is this change necessary?
Having the DAL check for invalid rules is faster than fetching them first. So I just removed the php check and moved it to the DAL.
2. What does this change do, exactly?
Remove a early-loop-quit as I assume #1310 will get this RepositoryIterator feature by default into the core.
Add a filter to the DAL query.
3. Describe each step to reproduce the issue or behaviour.

Have a loads of rules that you optimized by being invalid
Still loads a lot

4. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

A test would be nice

We tried to implement that in another PR. Some tests are failing, cause isInvalid is not same as Equals

When the DAL is not able to process a bool field then I'd understand the point that isInvalid is not the same as the equals check. But can you explain why it is not the same? I see that it is getting indexed but this should not change the way invalid works.

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


Fix some Metamask warnings and state upload

Fixes #2089
Short description
Metamask currently is deprecating ethereum.enable function and moving to a specific JSON RPC method to enable/list accounts. This PR adapts the dApp to use this new pattern.
It also fixes the store resetting the uploaded state when trying to connect, not allowing it to be passed to the SDK for upload.
Definition of Done

[ ] Steps to manually test the change have been documented
[ ] Acceptance criteria are met
[ ] Change has been manually tested by the reviewer (dApp)

Steps to manually test the change (dApp)

@taleldayekh maybe it'd be interesting to use this PR also to implement auto-connecting on state upload? There's really no visual indication when a state is uploaded, the user must go back and blindly click Connect for the state to be actually loaded.

Closed in favor of #2153


fix typo

rewriteCniPodSPec to rewriteCniPodSpec
Signed-off-by: Yuchen Dai<EMAIL_ADDRESS>

@stewartbutler The function which has the typo is probably the reason https://github.com/istio/istio/pull/23037 would work

/cherrypick release-1.5

@stewartbutler: new pull request created: #23040

In response to this:

/cherrypick release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Failed to query available provider packages

Hi there!
Thanks for great module!
One small thing, currently when I'm trying to use that module I'm getting this error:

I guess this is because I'm using older version of hashicorp/aws than you:
aws = {
  source  = "hashicorp/aws"
  version = "~> 3.74"
}

Could you support also version 3.74?

Hello @mits87 – I'll take a look at this later today to see if all the resources are supported and if it's possible, I will publish an update. Thanks!

Hey @mits87 – unfortunately the data resource aws_iam_openid_connect_provider requires v4.2.0 or later of the AWS provider and I don't want to break backwards compatibility for other users. Sorry about that.


feat: scale collections

Resolves https://github.com/paritytech/capi/issues/1113
Now, $.set and $.map decode to ScaleSets and ScaleMaps, respectively, which check their keys for equality once encoded. For example, the code sample in the issue:
// before
const value = [...publicKeys.entries()].find(([k, v]) =>
  bytesEqual(k, authenticationKey) // have to manually iterate and check by value
);

// after
const value = publicKeys.get(authenticationKey) // even though it is a different instance, this will still succeed, as it is checked by value

cc @rflechtner @weichweich

That was quick, thanks so much!


Change branch build to develop

What type of PR is this? (check all applicable)

[x] Refactor
[x] Feature
[x] Bug Fix
[x] New Query Runner (Data Source)
[x] New Alert Destination
[ ] Other

Description

change branch build to develop

Related Tickets & Documents
Mobile & Desktop Screenshots/Recordings (if there are UI changes)

wrong


Imp push as an unsafe fn

This would give the 2nd motivation, to be used as a bag

Not relevant any more.


alerting setup issues

Describe the bug: (a clear and concise description of what the bug is)
Tried setting up alertmanager to send some alerts to opsgenie together with @staticvoid255. However, we ran into the following issues:

Communication between Alertmanager and Prometheus within a team-namespace is not possible if Otomi ingress networkpolicies are deployed. We deployed these netpols as temporary fix:

apiVersion: networking.k8s.io/v1
kind: NetworkPolicy
metadata:
  name: temp-fix-prometheus-to-alertmanager
  namespace: team-X
spec:
  ingress:
  - from:
    - namespaceSelector:
        matchLabels:
          name: team-X
      podSelector:
        matchLabels:
          prometheus: X-po-prometheus
  podSelector:
    matchLabels:
      alertmanager: X-po-alertmanager
  policyTypes:
  - Ingress
---
apiVersion: networking.k8s.io/v1
kind: NetworkPolicy
metadata:
  name: temp-fix-alertmanager-to-prometheus
  namespace: team-X
spec:
  ingress:
  - from:
    - namespaceSelector:
        matchLabels:
          name: team-X
      podSelector:
        matchLabels:
          alertmanager: X-po-alertmanager
  podSelector:
    matchLabels:
      prometheus: X-po-prometheus
  policyTypes:
  - Ingress

Communication between Kube-state-metrics and Prometheus within a team-namespace is not possible if Otomi ingress networkpolicies are deployed. (deployed a similar networkpolicy to the one above)

a team's Prometheus should contain a probeNamespaceSelector like below, so it wont allow teams to check all namespaces.

probeNamespaceSelector
    matchLabels:
        name: team-X

Teams are able to view alerts from other namespaces. This can be fixed by setting defaultRules.appNamespacesTarget to the team's namespace (in prometheus-operator values).

We noticed that when setting Opsgenie api-key and url in Otomi UI, these values were not stored in gitea and selfService values are duplicated now, see here the redacted content of the values/env/teams.yaml:
edit: Will move this issue to the right repo, not otomi-core.

    teamX:
        oidc:
            groupMapping: xxx
        alerts:
            receivers:
                - opsgenie
        resourceQuota:
            pods: '100'
        networkPolicy:
            egressPublic: false
            ingressPrivate: true
        selfService:
            service:
                - ingress
                - networkPolicy
            team:
                - alerts
            Service:
                - ingress
            Team:
                - alerts
            apps: []
        id: teamX

Alertmanager is not updated with new opsgenie configuration. Its config-reloader container does not seem to update alertmanager-teamx-po-alertmanager-generated secret while the alertmanager-teamx-po-alertmanager secret is updated. (not sure yet why).
edit: found the issue, opsgenie_configs[].tags is expecting a string (csv), not an array, will create a PR with fixes.

Otomi core version: v0.16.3
Otomi api version: 0.5.5
Otomi console version: 0.5.9
K8S version: 1.21.2

You are the best!


Coinbase integration

Tomorrow, Jazear and Tim will spec out what can be done with the Coinbase SDK. We want to execute specific workflows on Twitter that are deeply engaging.
We can also figure out what the threshold is for when the things are engaging enough that they would co-respond with a good base launch. We might wanna think about who we would partner with and other such things and would want to communicate with other base teams, etc.

@monilpat check this out https://www.npmjs.com/package/@coinbase/cdp-agentkit-core. Looks like it was just made, shared with me by Shaw


Username and password were incorrect on default CI.yaml

Hi, you closed this but I did indeed try using PYPY_API_TOKEN secret value to set MATURIN_PYPI_TOKEN env variable
I've tried with two different pypy accounts and with one of them I even tried setting the token value to the env variable without using a secret.. (as below) this isn't working in Maturin 0.14
Hi
Im just using the autogenerated CI.yaml
I added my pypyp API token to the Action secrets space as MATURIN_PYPI_TOKEN
Have also tried:
  - name: Publish to PyPI
    uses: messense/maturin-action@v1
    env:
      MATURIN_USERNAME: my_username
      MATURIN_PASSWORD: my_password
    with:
      command: upload
      args: --skip-existing *

I get the following error:
2022-12-02T13:22:59.6828501Z [command]/home/runner/work/_temp/d3a970aa-691e-42ba-be92-302145e91c5f/maturin upload --skip-existing /home/runner/work/FastQL/FastQL/fastql-0.1.0-cp310-cp310-macosx_10_9_x86_64.macosx_11_0_arm64.macosx_10_9_universal2.whl /home/runner/work/FastQL/FastQL/fastql-0.1.0-cp310-none-win_amd64.whl /home/runner/work/FastQL/FastQL/fastql-0.1.0-cp311-cp311-macosx_10_9_x86_64.macosx_11_0_arm64.macosx_10_9_universal2.whl /home/runner/work/FastQL/FastQL/fastql-0.1.0-cp311-none-win_amd64.whl /home/runner/work/FastQL/FastQL/fastql-0.1.0-cp37-cp37m-macosx_10_9_x86_64.macosx_11_0_arm64.macosx_10_9_universal2.whl /home/runner/work/FastQL/FastQL/fastql-0.1.0-cp37-none-win_amd64.whl /home/runner/work/FastQL/FastQL/fastql-0.1.0-cp38-cp38-macosx_10_9_x86_64.macosx_11_0_arm64.macosx_10_9_universal2.whl /home/runner/work/FastQL/FastQL/fastql-0.1.0-cp38-none-win_amd64.whl /home/runner/work/FastQL/FastQL/fastql-0.1.0-cp39-cp39-macosx_10_9_x86_64.macosx_11_0_arm64.macosx_10_9_universal2.whl /home/runner/work/FastQL/FastQL/fastql-0.1.0-cp39-none-win_amd64.whl
2022-12-02T13:22:59.6860026Z 🚀 Uploading 10 packages
2022-12-02T13:23:00.4949066Z ⛔ Username and/or password are wrong
2022-12-02T13:23:00.4976272Z 💥 maturin failed
2022-12-02T13:23:00.4976880Z   Caused by: Username and/or password are wrong
2022-12-02T13:23:00.5014062Z Error: The process '/home/runner/work/_temp/d3a970aa-691e-42ba-be92-302145e91c5f/maturin' failed with exit code 1
2022-12-02T13:23:00.5035140Z ##[error]The process '/home/runner/work/_temp/d3a970aa-691e-42ba-be92-302145e91c5f/maturin' failed with exit code 1
2022-12-02T13:23:00.5036757Z     at ExecState._setResult (/home/runner/work/_actions/messense/maturin-action/v1/dist/index.js:1702:25)
2022-12-02T13:23:00.5037520Z     at ExecState.CheckComplete (/home/runner/work/_actions/messense/maturin-action/v1/dist/index.js:1685:18)
2022-12-02T13:23:00.5038359Z     at ChildProcess.<anonymous> (/home/runner/work/_actions/messense/maturin-action/v1/dist/index.js:1579:27)
2022-12-02T13:23:00.5038814Z     at ChildProcess.emit (node:events:390:28)
2022-12-02T13:23:00.5039543Z     at maybeClose (node:internal/child_process:1064:16)
2022-12-02T13:23:00.5039885Z     at Socket.<anonymous> (node:internal/child_process:450:11)
2022-12-02T13:23:00.5040485Z     at Socket.emit (node:events:390:28)
2022-12-02T13:23:00.5040755Z     at Pipe.<anonymous> (node:net:687:12)
2022-12-02T13:23:00.5204636Z Cleaning up orphan processes

https://github.com/PyO3/maturin/issues/1322#issuecomment-1336027232
Please don’t keep opening the same issue again and again, you can always reopen the previous issue with a comment explaining the reason for reopening with more information.


How to visualization the view attention？Just like Figure 6 in your paper

Thank you for your wonderful work and look forward to your reply！

To visualize view attention, I use the attention matrix (Nrays x Npts x Nviews x Dim) from here and average across the channel dimension(Nrays x Npts x Nviews), further followed by computing the view number with maximum attention value (Nrays x Npts). Finally, the mode across points for each ray is computed and visualized (Nrays).
Unfortunately I cannot seem to find the code for it, but the above is the general idea. Let me know if you run into any problems


[DOC] Sorting and Search After in Hybrid Search

What do you want to do?

[ ] Request a change to existing documentation
[x] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request.
For Hybrid Query we have enabled the support for two features i.e sorting the hybrid query results and a customer can paginate the sort results by using search_after clause.
*Version: List the OpenSearch version to which this issue applies, e.g. 2.14, 2.12--2.14, or all.
2.15
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
https://github.com/opensearch-project/neural-search/issues/507

@vibrantvarun , do you have an ETA for this PR? Thanks!

Yes I am working on the PR. It will be out soon.

Thanks, @vibrantvarun !


feat(default_ad_api): add log when operation mode change fails

Description
Modify error response message.
Add warn log to guide user to GUI tool.
Tests performed

Launch planning simulation.
Request to change to autonomous mode without setting pose and goal.

ros2 service call /api/operation_mode/change_to_autonomous autoware_adapi_v1_msgs/srv/ChangeOperationMode
Effects on system behavior

Not applicable.
Interface changes

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Worked as expected:

@isamu-takagi Will you merge this or close it?

For some reason, the difference of this PR were included in #7432, so I will close it.


Prevent pull-to-refresh from triggering while scrolling in a nested scrollable element on the page

This fixes: https://github.com/hotwired/turbo-android/issues/294
This prevents pull-to-refresh from being triggered when (potentially) nested scrolling within an element on the page that is using overflow-y: scroll or overflow-y: auto. To address this, the following item were added:

On every touch start, check if the target element or any of its parent elements are scrollable. If any are scrollable, notify the TurboWebView that the currently touched element is scrollable.
On every touch end, reset the internal state of the TurboWebView.
Whenever the TurboSwipeRefreshLayout checks to see if its scrollable child can scroll up, check the state of the TurboWebView if a scrollable element is currently being touched.

Works like a charm, thanks @jayohms!


What are the Design Ops tooling we need to establish for Theme Builder

Best practices for design and accessibility testing tooling and how they can be used to contribute to this project.
Best practices for evaluating accessibility of existing experiences and creating new accessible experiences.
Consider mobile and web -

@lwnoble @kburk1997 - tooling for  testing and design to help contributors
Penpot, Storybook, Chromatic, Anima, ... please comment here so that we can create a comprehensive list of tooling that can help our community evaluate and implement accessible solutions

Testing:

Accessibility Insights for Web - free Chrome/Edge extension that contains automated, guided, and manual test abilities. Very comprehensive - lots of tests that are available for free and it has a GitHub integration too that makes it easy to file issues. There is also tooling available for Windows + Android apps.
axe-devtools - freemium offering - Chrome extension with only automated tests are free. axe API is free up to a point - its Developer Hub offering is paid. Also has mobile offerings.
@storybook/addon-a11y - Storybook accessibility plugin - runs automated checks using axe api

Linting:

axe-linter - VSCode extension free, GitHub actions integration paid
eslint-plugin-jsx-a11y - plugin for ESLint - allows us to catch common accessibility issues + extend the configuration to map Material UI elements to HTML elements

For Designers:

axe for Designers - contains accessibility annotations + auto scan functionality
Stark - color contrast tool for designers - includes a lot more than just color contrast, including functionality for developers
Web Accessibility Annotation Kit - accessibility annotations toolkit for Figma from CVS Health

Integration Testing:

Cypress - offers an OSS Plan for open source projects for its premium offering, but basic functionality is also free

Beta Cypress Accessibility offering that has a waitlist
Accessibility testing can be achieved using plugins such as:

axe Developer Hub - axe offering that can integrate an accessibility scan into all Cypress tests
cypress-axe - barebones integration with axe api
cypress-audit - not up to date but does let you run Pa11y or Lighthouse

Playwright - free open source offering

Playwright guide for automated accessibility testing

Selenium WebDriver - multiple language bindings for browser automation

Accessibility testing can also be done using 3rd party plugins like axe-selenium-java
Very barebones compared to Cypress + Playwright

Regarding unit testing - #854 introduces Jest as a proposed unit testing framework. I did find a plugin that runs axe-core in Jest, so that could be an avenue for integrating automated tests:

jest-axe

However, since axe-core is so DOM-heavy, it may need to be done as part of the integration testing layer using Cypress or Playwright.


credential with liquid

I have event coming into Event format agent
{
"cNo": "c4"
}
c4 is a credential I have
I am trying to format it as
{%- credential {{cNo}} -%}
so the event would include credential c4, but it does not work
How can I do this?

How did you do it?


Update docker instructions

This PR includes documentation updates to take advantage of newly published docker containers.

Nice!
LGTM, +1


Basic type narrowing / flow analysis

Currently, no type narrowing / flow analysis is done in Luanalysis (as far as I can tell), which means I constantly have to annotate expressions with --[[--- @not nil ]] even if I just did a nil check. Example:
--- @type number | nil
local x = 5

if x then
	math.max(x, 5) -- warning! nil is not assignable to number
end

Luanalysis should be able to detect these cases and deduce that x is neither false nor nil. That means nil will be removed from type intersections type unions the | thing and boolean will be converted to an explicit true.
Note that this does not extend to table members or global variables, because those may have metatables. Therefore this should always be a warning:
--- @shape Something
--- @field x number | nil

--- @type Something
local x = {x = 5}

if x.x then
	math.max(x.x, 5) -- warning! x.x could have changed in the meantime
end

But you can still store x.x in a local so that the local itself does not change:
--- @shape Something
--- @field x number | nil

--- @type Something
local x = {x = 5}
local xx = x.x

if xx then
	math.max(xx, 5) -- ok! x.x could have changed in the meantime, but xx is fine
end

This is more complex than it seems though, because functions with upvalues can still change them. So upvalues need to be automatically detected & calls to those functions identified. But that's also more complicated than it seems, because functions are first-class citizens, so you don't necessarily know if a function you call is either one that has a certain upvalue, or one that transitively calls one of those. Here are some theoretical scenarios, with explanations...
--- @type number | nil
local x = 5

local function changeX()
	x = nil
end

if x then
	-- `changeX` was defined, but it wasn't hooked into anything, and it's local, so it can't possibly have been called by now.
	-- `x` is type-narrowed.
	math.max(x, 5) -- ok!
end

--- @type number | nil
local x = 5

function changeX()
	x = nil
end

if x then
	-- `changeX` was defined globally. The global env may have a __newindex metamethod - all bets are off.
	math.max(x, 5) -- warning: x may have changed in the meantime; changeX function is accessible
end

--- @type number | nil
local x = 5

local function changeX()
	x = nil
end

if x then
	-- `changeX` is defined locally and not hooked into anything, but we call it:
	changeX()
	math.max(x, 5) -- warning: x may have changed in the meantime; changeX function mutates it as an upvalue
	-- More advanced flow analysis (like detecting functions which conditionally modify upvalues) is... more difficult

	if x then
		math.max(x, 5) -- ok! changeX was not called
	end
end

--- @type number | nil
local x = 5

local function changeX()
	x = nil
end

registerSomething(changeX)

if x then
	-- `changeX` is defined locally, but got hooked into something
	math.max(x, 5) -- warning: x may have changed in the meantime; changeX function may have been called
	-- for all we know, `math.max` uses the `changeX` closure that we passed to `registerSomething`
	-- once again, all bets are off
end

Of course, there are more issues, like detecting whether a particular scope executes after changeX has been shared with something else. Sometimes it's both. You really have to be pessimistic here, and can't make any assumptions, which is really sad for ease of implementation, but it'll be better in the long run.
This is a large and difficult feature to add. I do not have any implementation to contribute and this issue will probably stay open for a good long while (or forever). But I feel it's worth tracking. Especially with the test cases.

There are already issues https://github.com/Benjamin-Dobell/IntelliJ-Luanalysis/issues/4, https://github.com/Benjamin-Dobell/IntelliJ-Luanalysis/issues/63 and closed issues https://github.com/Benjamin-Dobell/IntelliJ-Luanalysis/issues/100 and https://github.com/Benjamin-Dobell/IntelliJ-Luanalysis/issues/97.
I don't want too many duplicate issues so am closing this issue. Although this isn't new to me here (I touched on some of this in #4), I'd suggest copying this content into a comment on #63, so it's not lost. That should also make it easier to find #63 with search, since it'll then include both smart casting and narrowing terminology.

 :/
I thought I remembered looking at something like that but I couldn't remember the name and whether or not it was on the EmmyLua repository instead. Thanks.

I'd suggest copying this content into a comment on #63

The references you made by linking the issues should be enough (this one will appear on all of those).


Add createGrid unit tests.

I added some unit tests in another PR https://github.com/gionkunz/chartist-js/pull/654 which is still open and probably won't be merged.
Here are just the tests...

Awesome!


missing_device_type_components not checking Module Bays

The report "Missing Device Type Components" is not checking Module Bays correctly. It doesn't show when a Module Bay is missing from a device while it's present in the device type.

fixed it by changing
('devicebays', 'devicebaytemplates'), to ('modulebays', 'modulebaytemplates'),


Feat/autogenerate third party notices

:tada: This PR is included in version 0.3.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update fs2-core, fs2-io to 3.9.1

About this PR
📦 Updates

co.fs2:fs2-core
co.fs2:fs2-io

from 3.3.0 to 3.9.1
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "co.fs2" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #488.


chore: fix 404 status URL

fix 404 status URL
PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments

Thank you for your pull request! I appreciate the effort you've put into it. However, I will have to reject the change, as it is quite small. If you want to fix all of the remaining typos in the docs, you are more than welcome to submit a larger PR. Thank you for understanding!


talon_filter_transcript --minCounts with --maxFracA

I wanted to clarify that if we assign --minCounts and --minDatasets (e.g two replicates) but I keep the default for maxFracA.
I had assumed that if any read supporting a model has > 0.5 As following it then this model would be dumped regardless of whether both replicates at least 5 reads.
I saw a model that has 7 and 5 counts in rep1 and rep2 respectively but it doesn't output to whitelist, I then checked the read annotation file and indeed some of the reads have fractionA > 0.5 meaning one of the replicates only have 4 reads not 5 therefore only one of the replicates satisfies all criteria and the model is not output.
Is this correct?
And do these filtering only apply to TALONT? What about ENST? Are all ENST models with a least a read in any dataset output to the whitelist despite the use of --minCounts and --minDatasets ? I guess this would make sense, so long as its expressed in the dataset there would be no reason to filter out.

The filtering options create intersections of novel transcripts to output for the pass list, not unions. So yes, if you filter your transcripts using all three of those options, you will only get transcripts that satisfy all three criteria. And yes, TALON only filters novel transcript models, not known models that are annotated in the reference you used.
If you want to start making custom pass lists I would suggest trying to run the filter separately and performing the type of join you prefer on the resultant lists. As long as the pass list that you provide to the abundance and GTF utilities follows the same format as the output from the TALON filter, you can use whatever!


The command dotnet tool install -g Redth.Net.Maui.Check causes the screen to reboot without responding.

When I run dotnet tool install -g Redth.Net.Maui.Check from the command line, my PC freezes and reboots.
The OS is Windows Insider Preview Build 21387 and the version of dotnet is 6.0.100-preview.4.21255.9.
Visual Studio 2019 v16.11 Preview 1 is also installed on that PC.
Note that I tried to install other tools installed with dotnet tool install -g, but the same event occurs with dotnet-fsharplint.

This sounds really odd.  Can you run something like maui-check --verbose > mauicheck.log and see what's in the logfile after?

Nothing was recorded in the log, but I found the cause.
I terminated the resident anti-virus software and was able to install it successfully.


⚠ Bofrost.be has degraded performance

In 1f881f2, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1446 ms

Resolved: Bofrost.be performance has improved in 4b8a299.


Adding AimShootLife AI trips an assert

specifically in Ray.cs line 36:
DebugTools.Assert(FloatMath.CloseTo(_direction.LengthSquared, 1));

this is because it's normalising a zero length vector (presumably to the entity itself)

@DrSmugleaf no longer an issue because bulldozed


Unified login/logout methods

Fix https://github.com/huggingface/huggingface_hub/pull/1098.
This PR adds the login and logout methods as the main (only) entrypoint to login/logout in huggingface_hub. Most of the code is a copy from the existing code that was living in huggingface_hub.commands.user (which is now cleaner, with only CLI-related code).
I did not change any behavior on the git credential sutff (discussed in https://github.com/huggingface/huggingface_hub/issues/1051) as it is not the focus of this PR. Also unrelated from discussions happening in https://github.com/huggingface/huggingface_hub/issues/1094.
Colab
See example here
Example:
from huggingface_hub import login, logout

login()  # displays a widget in a notebook, a prompt in terminal otherwise

login("hf_xxx") # non-blocking login

logout() # logout completely

Still compatible:
from huggingface_hub import notebook_login

notebook_login()  # same as before

huggingface-cli login
huggingface-cli logout

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Here were my thoughts when building a single login method:

I feel it ease the user experience e.g. "you just have to know that there is 1 entrypoint to login" => it's more concise when sharing snippets of code in documentation or social media. No need to explain "if you have a token, use login(...) if you don't have a token and are in a notebook use notebook_login(), if you don't have a token and are in a script use interpreter_login()" (<- this one is not mandatory-)
If someone copies a snippet found on the internet (stackoverflow, documentation, existing notebook...), it doesn't need to adapt the from huggingface_hub import notebook_login to from huggingface_hub import login. Same if you take over/copy an entire notebook from someone else and want a non-blocking login you just need to paste your token as argument without changing anything else.
code doesn't have to be adapted when you convert your notebook to a script with jupyter nbconvert --to script my_notebook.ipynb
a downstream library can also benefit from it. Let's say you implement a lib that train something and push to the hub. You could have some logic like "if not logged in, then use login() before pushing" -> the downstream lib doesn't have to know if  the end user is in a notebook or a terminal

That's said, I fully agree that those are only marginal benefits. If you feel it's too magical, I'm fine with reverting back to have separate login methods.
(also cc @osanseviero for opinion)

(BTW just linking to internal convo)

is there any drawback to have this new login() behavior but still keeping (and documenting) a notebook_login too?
(which is what this PR does anyways, IIUC)
^ Personally I'd tend to do this, but I also don't feel that strongly about it

(which is what this PR does anyways, IIUC)

Yes, that's the case. I think keeping notebook_login is good just because it is widely used and known. I removed it from documentation (in favor for login()) but I can put it back, at least have it in the Reference section.

to me re-adding to doc would be a good solution, yes. But yeah let's wait for others to chime in =)

Yesterday I thought a single method for those very different developer experiences might lead to confusion and sometimes appear too much as a black box/magic.  A notebook_login makes it much clearer that the behavior is different, getting a widget is very different.
But after reading all these points, I think having this behavior + a still well-maintained notebook_login would make sense. I'm still not 100% convinced, though. I played with it in the colab and it's nice

Thanks everyone for the feedback. @sgugger I kept your suggestion to make both notebook_login and interpreter_login official so that the user can always force the use of the widget (or not). I have updated the documentation/docstrings accordingly. I think we are now good to go :tada:

As a note: the Codecov util make reviewing PR very difficult (and much longer for the reviewer) with all the comments. You should consider disabling it.

@sgugger understood. I agree that when code is not tested (like in this PR) it is very annoying. I changed the config to disable the comments in the "diff section". For your info (I just learnt about it from the doc), it is possible to toggle them by pressing "a" when you review the code in Files changed.


Fix 512

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #512

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Documentation has been added / updated (for bug fixes / features)
[x] HISTORY.rst has been updated (with summary of main changes)
[ ] bumpversion (minor / major / patch) has been called on this branch
[ ] Tags have been pushed (git push --tags)

What kind of change does this PR introduce?
Bug fix.

Does this PR introduce a breaking change?
No.

Other information:

Should we fix the Kmeans issues before merging this ?

The issue is quite small, so yes, I would suggest simply changing our "expected" value and pin scikit-learn at 0.23.2.

It feels heavy-handed to pin a dependency for no other reason that make sure tests pass (no new critical feature).

Then we could simple update the "importorskip" pytest statement to 0.23.2 (and change the expected output)?

Pull Request Test Coverage Report for Build 2114

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.005%) to 91.021%

Totals

Change from base Build 2109:
-0.005%

Covered Lines:
3112

Relevant Lines:
3419

💛  - Coveralls


📁Docs: missing elements in API reference for Context Menu

The Context Menu createContextMenu API Reference is missing 3 elements (item, group, groupLabel)

hello can i help?

Of course! Just assigned it to you! @FarhanAliRaza

what is the descriptions of these elements?

The preview component should also have these elements implemented.   Should it? And if yes do we need to add it in another issue ? or can it be added in this issue? I implemented an example.

hello,  i opened a pr here https://github.com/melt-ui/melt-ui/pull/761


Improve reproducibility of pipeline outputs

Add fields to the pipeline output that would help improve pipeline reproducibility. This would help with accuracy evaluation and improvement efforts.
Some ideas of fields to add to the output:

[ ] Input config (such as model parameters)
[ ] Pipeline config (such as whether to use the code search tool, whether the QA tool should return source documents)
[ ] Rendered prompts
[ ] Specific git commit hash when the input ref is a tag or branch

@shawn-davis per discussion today, I've tested that it's possible to get the current checked out commit hash in the cloned repo by running git rev-parse HEAD after the checkout step. However, we might need to do some testing to find the equivalent command for doing this in the git python library. We will also need to add a new attribute to the output schema to store this.
Since there are quite a lot of changes needed to improve reproducibility, it would be good to break the tasks above into separate issues/PRs to make the PRs smaller.


fixed: Project navigation link not linked to the project section #91

Issue No: #91
What does this pull request do?
Fixed Project navigation link not linked to the project section #91 by using the recommended fix
What is the relevant issue link:
Project navigation link not linked to the project section #91

you're so welcome


[Bug]: Upgrade fails when upgrading from 1.6.16 --> 1.6.17 using Docker/Watchtower

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Appsmith is deployed to DigitalOcean using the DO marketplace function (Docker) and using custom domain.  When watchtower attempts to apply upgrade, the app fails to come back online, throwing the following error:
2022-04-07 19:01:09,982 INFO success: editor entered RUNNING state, process has stayed up for > than 0 seconds (startsecs)
editor stdout | Load environment configuration
editor stdout |
editor stderr | + http_conf=/opt/appsmith/templates/nginx-app-http.conf.template.sh
editor stderr | + https_conf=/opt/appsmith/templates/nginx-app-https.conf.template.sh
editor stderr | + ssl_conf_path=/appsmith-stacks/data/certificate/conf
editor stderr | + APP_TEMPLATE=/opt/appsmith/templates/nginx-app-http.conf.template.sh
editor stderr | + mkdir -pv /appsmith-stacks/data/certificate/conf
editor stderr |
editor stderr | + cat
editor stderr |
editor stderr | + cat
editor stderr |
editor stderr | + [[ -n sub.domain.tld ]]
editor stderr | /opt/appsmith/run-nginx.sh: line 46: DYNO: unbound variable
editor stderr |
2022-04-07 19:01:09,995 INFO exited: editor (exit status 1; not expected)

Application never recovers.  In order to fix it, I had to roll back using a backup and manually disable watchtower.
Steps To Reproduce

Set up appsmith v1.6.17 using Digital Ocean
Wait for watchtower cron to run
???
Fail

Public Sample App
No response
Version
Self Hosted v 1.6.17

I believe this has been fixed with this commit. @mohanarpit can you confirm?

This has already been fixed with v1.6.18. Would request you to upgrade to this version.
Apologies for the poor experience here.


Samsung Galaxy S8 userdata partition problem

I think this is not actually an problem with Heimdall, but rather something around it. But as I understand here I would find many peoples who understands this area, maybe I can find some help.
So I am trying to generate and flash an userdata partition on galaxy S8 but the phone wont accept my image. I can do this without any problems for older samsung devices.
What I would do is

make an sparse image on ubuntu (with make_ext4fs from android-tools-fsutils package)
make_ext4fs -l [lenght] -s userdata.img [directory_to_my_content]
boot the device on Download mode
flash it with heimdall flash --userdata userdata.img

Now, on galaxy S8 the flashing goes ok for some time, but in around 40% it just stops and I have to reboot the device manually.
I checked many samsung firmware packages and usually they doesn't contain userdata image, but this one has it:
https://samsung-firmware.org/download/Galaxy__S8__/4v72/NEE/G950FXXU1AQJ1/G950FOXM1AQJ1/
and this userdata image I can flash ok with heimdall. It takes hours and days to download it so I have copied the userdata image here: https://1drv.ms/u/s!AlTfydQsL14YhKIifEMOccrcobcj6A
I did simg2img it and after mounting I can see it has plenty of apps and cache and some other files in it. First I thought that the userdata image has have something like this in it or it will fail. I tried to make an new sparse image with this content included, but the flashing still fails. I tried to make it a bit smaller, but it always fails.
If I see
file userdata.img
userdata.img: Android sparse image, version: 1.0, Total of 2621440 4096-byte output blocks in 447 input chunks.

Nothing special there, but if I see
simg_dump userdata.img
simg_dump: userdata.img: The file header size was expected to be 28, but is 32.

it wont dump the file because header size is something else than expected? Maybe this is the reason why the device wont accept the file?
So I am not that sure if there is a way for me to generate flashable userdata image for this phone? Or maybe I am lost and this header mismatch is not the problem, but there is something else I should check or do? Do you have any hints or clues from where I could begin?

Just in case somebody is also wondering the same thing, I came across another github project that has a D script to samsungify the images:
https://github.com/anestisb/android-simg2img/blob/master/simg_samsungify.d
So like expected, nothing wrong with the heimdall, it is just that samsung has their own format they like to use with some images. They have also better version of simg_dump tool that can dump images with 32 byte headers, so it is a nice project.

yeah, it all works with the samsung's own images. My problem was with generating the userdata image as it seems samsung has a bit special format.

You've used the flashing command without using the "userdata" argument? If not, try this way.
If you want to get your data back on the phone, there is many ways, the easier one is to use google backup system before. The second one if you didn't and if you've did a copy of your data image file is to unpack it and get your files back.

You can perform a compilation of the firmware.xml to change some value and try again too. Take a look on the readme file if you are interested to solve the flashing matter.


dotnet pack commands broken

Since the modules are in src/Orchard.Cms.Web

Fixed https://github.com/OrchardCMS/Orchard2/commit/a6c90b6b2c5b053d3d821c11eec9e5999a34dab3
@sebastienros I suggest to make the publishing of packages in MyGet

Thanks, it couldn't find it as I was searching file from VS.NET.
MyGet could be useful. But we need to update the version numbers in order to not be blocked later as we can't go down. So verly low priority.


amend: updated deps

:tada: This PR is included in version 2.0.26 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added size estimation for null values

pls rebase!

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@96edf86). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #5789   +/-   ##
=========================================
  Coverage          ?   82.94%
  Complexity        ?    11832
=========================================
  Files             ?     1400
  Lines             ?    45507
  Branches          ?     5638
=========================================
  Hits              ?    37748
  Misses            ?     5119
  Partials          ?     2640

Impacted Files
Coverage Δ
Complexity Δ

...in/java/io/crate/breaker/SizeEstimatorFactory.java
72.72% <100%> (ø)
7 <0> (?)

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 96edf86...7c08169. Read the comment docs.


This isn't working anymore

Most of the examples at http://ignitersworld.com/lab/contextMenu.html no longer work (tested in Chrome 81 - windows) the project founder has stated that this project s/he is no longer supporting. I will try to take a look for other better alternatives, or otherwise think about fixing the code for my purposes

So what I found was that part of my problem is that I am using Google Chrome. A lot of the projects I found of this ilk seem to work perfectly fine in Firefox, but not in Chrome. Eventually, I created my own small 100% javascript solution, based a lot on how other projects were generating contextmenus (2 files - .js and .css in zip)
MyMenuWidget.zip


refactor: remove one component

Summary

remove one component as unnecessary
add missing package exports

Test plan

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4ed7a56) 100.00% compared to head (df04542) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #34   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           11        11
  Lines          137       133    -4
  Branches        32        32
=========================================
- Hits           137       133    -4

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


Reg : Testing Model

I am trying to just use pretrained models to test. How do I generate clips for test data is that using ffmpeg or clips are generated in test_net.py?

this is the output which i got but not sure I am doing it right. I used r2.5d_d18_l16.pkl model

Check out this https://github.com/facebookresearch/VMZ/tree/master/tutorials

Thanks...what would be the ideal test size to test(I am running using CPU)


Why removed lot of extensions?

Describe why this should be added
I don't want use template
Other details
https://github.com/aniyomiorg/aniyomi-extensions/commit/cc6cffd7003b57368b0fe4a96a32bac583874080
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open (or closed) issue.
[X] I have written a short but informative title.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Because we were hit with a DMCA notice. See #3341


ValueError: Expected metadata value to be a str, int, float or bool, got None which is a NoneType

got prompt
WARNING: AnyNode.IS_CHANGED() got an unexpected keyword argument 'prompt'

RUN-1 01-ai_Yi-1.5-9B-Chat-16K-8_0bpw_exl2 Take the input and multiply by 5 1 2

Finding Nodes in Workspace. kwargs: {'id': '1'}
{'prompt': 'Take the input and multiply by 5', 'function': 'An error occurred: API request failed with status code 500: Internal Server Error', 'imports': '', 'comment': None, 'input_types': "Type: <class 'int'>\nType: <class 'int'>", 'version': '0.1.2'}
!!! Exception during processing !!! Expected metadata value to be a str, int, float or bool, got None which is a NoneType
Traceback (most recent call last):
  File "C:\AI\StableDiffusion\Stable Diffusion ComfyUI Aki V1.2\execution.py", line 317, in execute
    output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
  File "C:\AI\StableDiffusion\Stable Diffusion ComfyUI Aki V1.2\execution.py", line 192, in get_output_data
    return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
  File "C:\AI\StableDiffusion\Stable Diffusion ComfyUI Aki V1.2\execution.py", line 169, in _map_node_over_list
    process_inputs(input_dict, i)
  File "C:\AI\StableDiffusion\Stable Diffusion ComfyUI Aki V1.2\execution.py", line 158, in process_inputs
    results.append(getattr(obj, func)(**inputs))
  File "C:\AI\StableDiffusion\Stable Diffusion ComfyUI Aki V1.2\custom_nodes\anynode\nodes\any.py", line 414, in go
    self.last_hash = registry.add_function(prompt, self.script, self.imports, self.last_comment, [variable_info(any), variable_info(any2)])
  File "C:\AI\StableDiffusion\Stable Diffusion ComfyUI Aki V1.2\custom_nodes\anynode\nodes\util_functions.py", line 57, in add_function
    self.add_function_to_chromadb(prompt_hash, prompt, function_code, imports, comment, input_types)
  File "C:\AI\StableDiffusion\Stable Diffusion ComfyUI Aki V1.2\custom_nodes\anynode\nodes\util_functions.py", line 86, in add_function_to_chromadb
    collection.add(
  File "C:\AI\StableDiffusion\Stable Diffusion ComfyUI Aki V1.2\python\lib\site-packages\chromadb\api\models\Collection.py", line 80, in add
    ) = self._validate_and_prepare_embedding_set(
  File "C:\AI\StableDiffusion\Stable Diffusion ComfyUI Aki V1.2\python\lib\site-packages\chromadb\api\models\CollectionCommon.py", line 271, in _validate_and_prepare_embedding_set
    ) = self._validate_embedding_set(
  File "C:\AI\StableDiffusion\Stable Diffusion ComfyUI Aki V1.2\python\lib\site-packages\chromadb\api\models\CollectionCommon.py", line 182, in _validate_embedding_set
    validate_metadatas(maybe_cast_one_to_many_metadata(metadatas))
  File "C:\AI\StableDiffusion\Stable Diffusion ComfyUI Aki V1.2\python\lib\site-packages\chromadb\api\types.py", line 336, in validate_metadatas
    validate_metadata(metadata)
  File "C:\AI\StableDiffusion\Stable Diffusion ComfyUI Aki V1.2\python\lib\site-packages\chromadb\api\types.py", line 302, in validate_metadata
    raise ValueError(
ValueError: Expected metadata value to be a str, int, float or bool, got None which is a NoneType

Prompt executed in 0.01 seconds

and this is my workflow.it is a simple one created for testing.

local llm was loaded. please ignore last error, it was not caused by anynode.

API URL was right, and there was no API key.

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

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

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

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

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

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

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

DO NOT DOWNLOAD ANYTHING I HAVE POSTED AS A COMMENT OR ANY OTHER MEDIAFIRE LINKS, MY ACCOUNT WAS COMPROMISED AND BEING USED TO COMPROMISE OTHERS AND YOURS WILL END UP BEING USED AS WELL
I AM WORKING ON DELETING AND POSTING THIS ON ALL THREADS IN WHICH THEY USED MY ACCOUNT TO POST THIS

I'm getting this same issue, only im using OpenAI but everything is pretty much the same, my API key is in the env file, but no matter what i put into the node, i get the same output error like you have

I also encountered the same error, and there was a rejection from the server, I tried to replace localhost with <IP_ADDRESS>, and also tried to add/or /v1 at the end, but in fact, it is not useful, but there is no problem with the LLM service, I tested in the application and it can return normally
In addition, the local LLM node can not accept the input of pictures or videos, and it can not directly process the pictures as demonstrated. It has been tested for a long time, but it is a pity that it did not succeed. If there is any latest progress, please remember @me, thank you

@XJF2332
I tried to change comfyui manager security to weak and it works,
..\ComfyUI\custom_nodes\ComfyUI-Manager
you will find file config.ini , open it with notepad
and set security_level = weak
I hope this will fix your issue

I'm getting this same error even with weak security for the manager. Is there any way around this? It seems like such a great node, but it always gives this same error no matter what I try.


bpf: Get rid of unused lb6_{key,service} structs

They were used by the legacy LB SVC which was removed in v1.6.

This change is

test-me-please

Coverage decreased (-0.004%) to 44.029% when pulling 95693c7818bcfc5100615239fe80c4a31a037e09 on pr/brb/rm-lb6-legacy into 88f3fd5f08c3ca5671cf401744f4f3c0ee56d7ba on master.


[Bug]: BUG IN THE DIV WHERE GAMES ARE REPRESENTED

Give a brief about the bug discovered✍
Hey @panwar8279
I found a BUG in the game selection where all the games are stored
Following are the bugs:

The text in card is not aligned and looks very odd, like it is spreaded all over the box
The number of game is also not right in every box in some box it is fully visible and in some it's not

What is the expected behavior?
The name of game should be in one or two line in the bottom of card so that is should visible and the number of game should be visible clearly
Give a brief description to get the expect behaviour.
NONE
Are you contributing for a program?
GSSoC2023
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hi @marshadkhn , I have assigned this issue to you. Kindly complete within 4 days. Good luck!

For sure

This issue is done


do some code refactoring

@smileyborg :)

Overall, I would like to merge in the refactoring of the error status description, but all of these other changes are extremely minor or subjective improvements.
I appreciate the effort you put into these, but remember that the more changes we make the more likely it is for unintentional bugs to get introduced. It's definitely a good idea to avoid changes just for the sake of making changes.
Please do feel free to submit PRs for more substantial changes or enhancements, ideally to the library itself. It's also a good idea to communicate your intent before making changes (especially large ones) to get feedback and make sure it's aligned with the project -- you can do this by opening an Issue.
Thanks!

think that u are right.


Indicate Text Formatting on Highlight

issues
fix https://github.com/jaredreich/pell/issues/16
desc
add keyup & mouseup event listeners on editor element
use queryCommandState
add pell-button-selected class to button

Question: should I commit the dist/ files as well?
Do you have some sort of travis for build?

Excellent PR, thanks @ronapelbaum.


Does it also support websocket ?

Thanks for pointing this out. It's not supported now, but we'll look into it for future updates.


chore(build): multi branch pipeline upgrade goversion

For all golang project's multi-branch pipelines, need support go1.20.
Tested: https://cd.pingcap.net/view/Dev-build-master/job/build_pd_multi_branch/job/master/765/console

/lgtm


configure batch size

Description
The batch size is now at 500 content.
When we have a lot of data (more than 10k) it's should be faster to have a bigger amount.
It should be great if we could configure the batch upload number for upgrade in plugin.

Hey @LiquidITGuy
I'm currently implementing this option! I'll come back to you as soon as it is live :)


Rescue from ScriptError

I found that sneakers worker becomes frozen after raised NotImplementedError.
Code to reproduce:
class MyWorker
  include Sneakers::Worker

  from_queue  'lolwut'

  def work(msg)
    raise NotImplementedError
    ack!
  end

This PR fixes it.

I think this PR shouldn't be merged. If you have error in your code then you have to fix it in your code.

Background processing should be reliable. It should not become frozen if coder writes bad code leading to error.

Background processing should be reliable. It should not become frozen if coder writes bad code leading to error.

Coder should cover his/her code with tests to detect ScriptErrors and not use system layer NotImplementedError in application layer.
@gabrieljoelc
https://ruby-doc.org/core-2.5.0/NotImplementedError.html

Raised when a feature is not implemented on the current platform. For example, methods depending on the fsync or fork system calls may raise this exception if the underlying operating system or Ruby runtime does not support them.

It's not ok to catch error when some feature is unavailable in current ruby implementation or OS.
http://chrisstump.online/2016/03/23/stop-abusing-notimplementederror/

We catch the base class here not NotImplementedError, so this could allow maintaining Sneakers error handling behavior for other sub-types of ScriptError as well (i.e. LoadError). Bottom line: we want want Sneakers to pass unhandled exceptions to worker_error instead of freezing.
NotImplementedError usage discussion is for another forum, please.

@gabrieljoelc Unfortunately, the problem of stale workers is still actual. Some gems introduce their own exceptions not based on StandardError and SystemStackError also leads to frozen state of consumer. I've made another attempt to fix the problem by introducing ensure. https://github.com/jondot/sneakers/pull/441


Unable to find key definition for defined key

This was reported by one of our customers and we tried to minimize the sample.
Try to publish to an XHTML-based output the following DITA Map:
http://www.oxygenxml.com/userFiles/TA_Help_ditamap.zip
The publishing reports that a certain key (which is actually defined) cannot be found:

[DOTJ047I] Unable to find key definition for keyref="kusing_debugger", href may be used as fallback if it exists.

Relevant console messages:

keyref:
[keyref] Reading C:\Users\radu_coravu\Desktop\LASS\TA_Help_reduced\temp\xhtml\TA_Integration_ditamap.ditamap
[keyref] Reading C:\Users\radu_coravu\Desktop\LASS\TA_Help_reduced\temp\xhtml\TA_Test_execution_ditamap.ditamap
[keyref] [Fatal Error] :2:11: The element type "topicref" must be terminated by the matching end-tag "".
[keyref] Failed to parse key definition: The element type "topicref" must be terminated by the matching end-tag "".
[keyref] null
[keyref] Processing C:\Users\radu_coravu\Desktop\LASS\TA_Help_reduced\temp\xhtml\test.dita
[keyref] C:\Users\radu_coravu\Desktop\LASS\TA_Help_reduced\test.dita:6:46: [DOTJ047I][INFO] Unable to find key definition for keyref="kusing_debugger", href may be used as fallback if it exists.

I will try to keep the ZIP with the samples on our web site for a couple of days.
DITA-OT 1.7

Closing as obsolete - I've tested with Radu's 2013 samples in DITA-OT 2.2.4, and the key is resolved properly. This must have been fixed with other key updates in either 1.8, 2.0, 2.1, or 2.2.


Make credentials optional for S3Bucket block

Description
S3Bucket blocks implicitely require a set of credentials to work, even if a the block can be declared without one. For instance,
it is possible to register this block without issue :
from prefect_aws import S3Bucket

block = S3Bucket(
    bucket_name="foo",
    bucket_folder="bar",
)

block.save("foo")

Yet, using the block (as a result_storage option, for instance) will trigger :
Failed("Task run encountered an exception AttributeError: 'dict' object has no attribute 'get_params_override'")

Proposal

Either make the credentials argument mandatory (not desirable)

Or make the boto3 session defaults to system parameters at run time (probably best)

[x] I would like to help contribute a pull request to resolve this!

hi @rmnvncnt - can you show your flow code that resulted in this error?

Failed("Task run encountered an exception AttributeError: 'dict' object has no attribute 'get_params_override'")

something seems a little odd about that error

FWIW, I have an S3 Bucket block with no credentials, as we have prefect running on an EC2 that it can get the creds from, and it works fine, no errors


Updated image repository enpoind and tag on values.yaml for HELM deployment.

Update the endpoint of the registry on "master" to be synced with the latest release of the driver v2.3.2

Thanks @jtorrex for the contribution! The image repo address is registry.cern.ch/kubernetes/cvmfs-csi, so without the /charts part and the tag is v2.3.2. Can you please fix this?


Include error counts in 5.1 feed overviews

This PR adds high level info about errors in your feed. Total Errors in Feed are a combination of errors with severities of fatal, critical, and errors; Warnings in Feed are just the warnings. See the Pivotal card for more infomation.
You might look at index.html and say to yourself: "Why would we add an ng-if directive? We don't need it here!" If the world were a sane place, you'd be right, but the world is not always so. It turns out that if you omit this directive, V5AsideController will be called before FeedOverview51Ctrl, and $routeParams.vipfeed will be undefined. You won't be able to request any errors with an undefined public_id!
"There must be a better way!", I can hear you saying, and I hope you're right. Let's talk about that part and see if we can find the better way.

I'm afraid "a better way" would involve starting from scratch, so this LGTM.


How to inject struct that implements an interface as interface

Hi!
type User struct{}

type UserRepository interface {
	GetUserByID(ctx context.Context, id int64) (*User, error)
}

func DatabaseBuilder() *sql.DB {
	return nil
}

// Implements UserRepository
type MySQLUserRepository struct {
	db *sql.DB
}

func NewMySQLUserRepository(db *sql.DB) *MySQLUserRepository {
	return &MySQLUserRepository{
		db: db,
	}
}

func (r *MySQLUserRepository) GetUserByID(ctx context.Context, id int64) (*User, error) {
	return nil, nil
}

type Handler struct {
	userRepository UserRepository
}

// Requires interface, but we will provide a struct, that implement this interface
func NewHandler(userRepository UserRepository) *Handler {
	return &Handler{
		userRepository: userRepository,
	}
}

How to describe injectors, so that the NewMySQLUserRepository will return struct, but this struct will be injected into NewHandler as interface (because it is implementing interface).
I know that i can replace signature of NewMySQLUserRepository to NewMySQLUserRepository(db *sql.DB) UserRepository, so it will return interface, but this is not always appropriate for me.
I tried wire.InterfaceValue(new(UserRepository), NewMySQLUserRepository), but this will not work, because second argument requires implementation, not injector, so i need to make function call.

You're looking for wire.Bind:
func inject() *Handler {
  wire.Build(
    DatabaseBuilder,
    NewMySQLUserRepository,
    wire.Bind(new(UserRepository), new(MySQLUserRepository)),
    NewHandler,
  )
  return nil
}

Thanks for your reply, small fix: there must be *MySQLUserRepository (pointer to a struct) in a Bind, because pointer type implements interface, or you will get error: MySQLUserRepository does not implement interface UserRepository
Correct is:
...
wire.Bind(new(UserRepository), new(*MySQLUserRepository)),
...

Alternative to @danf0rth answer is to use a nil instance:
wire.Bind(new(UserRepository), (**MySQLUserRepository)(nil)),


Wrapper simplification and bools_as_numbers option

Added opts_wrapper_t, which allows new glz::opts based wrappers to be created.
Added bools_as_numbers option to glz::opts for #981

@jbbjarnason, just wanted to make you aware of this improvement, which is adding raw wrapper handling to reading as a side affect. This makes it easier to create new wrappers for booleans in glz::opts.


Automatic rotation of tokens?

I don't think there's a way to automatically rotate the pem key for the github app. We will just have to do it manually. I put reminders on my calendar to rotate, but we should talk offline about a rotation schedule.


source side bpe

Implemented bpe subword segmentation on source side. Also added some tigrinya endpoints.

Hi @alpoktem, can you turn tick the “Allow edits from maintainers” checkbox so that I can add commits to this pull request?

Fixes #9

Merged separately. Thanks @alpoktem


feat:add option of favicon

serve({
            favicon:path.resolve(__dirname,'./favicon.ico')
          })

can use like this

I would rather remove all favicon.ico related code, not sure why it's there.

I don't like green

In that case, you can use the contentBase option to configure from which folder it should load the favicon.


Make it possible to initialize a theme with arbitrary theme zip

Timber is great and all, but not everyone wants to use it. Perhaps an agency has their own baseline theme they use on all projects.
Instead of replacing how theme bootstrap works, instead another command could be added such as initialize which takes in a --url specifying where to go to download the theme.
bootstrap should probably become deprecated after this feature ships.

I added this functionality to the bootstrap command however in the future I would like to add all this functionality in the the new command described in #283


同步mongodb，有修改动作报错。

cmd控制台
2022-06-24 14:19:56.068494 I | transfer run from position(mysql-bin.000003 304999263)
2022-06-24 14:19:57.069412 I | transfer stop
2022-06-24 14:19:57.071406 I | dumper stopped
2022-06-24 14:19:57.115291 I | transfer run from position(mysql-bin.000003 304999263)
2022-06-24 14:19:58.115368 I | transfer stop
2022-06-24 14:19:58.118153 I | dumper stopped
2022-06-24 14:19:58.193949 I | transfer run from position(mysql-bin.000003 304999263)
2022-06-24 14:19:59.194247 I | transfer stop
2022-06-24 14:19:59.200250 I | dumper stopped
2022-06-24 14:19:59.284036 I | transfer run from position(mysql-bin.000003 304999263)
2022-06-24 14:20:00.284258 I | transfer stop
2022-06-24 14:20:00.290240 I | dumper stopped
2022-06-24 14:20:00.366031 I | transfer run from position(mysql-bin.000003 304999263
log日志
error	bulk write error: [{[{Performing an update on the path '_id' would modify the immutable field '_id'}]}, {}]

请问如何解决呢？


fix(firestore): Call document iterator stop function

Description
Fixes #4230
Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.
Checklist

[x] I have followed Contributing Guidelines from CONTRIBUTING.MD
[ ] Tests pass:   go test -v ./.. (see Testing)
[ ] Code formatted:   gofmt (see Formatting)
[ ] Vetting pass:   go vet (see Formatting)
[ ] These samples need a new API enabled in testing projects to pass (let us know which ones)
[ ] These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
[ ] This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
[ ] This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
[x] Please merge this PR for me once it is approved

Hi @sakayaparp , I'm curious if you could measure that:

before the change, the memory wasn't released,
after the change, the memory was released

Thanks!

Hi @sakayaparp , I'm curious if you could measure that:

before the change, the memory wasn't released,
after the change, the memory was released

Thanks!

Hi @Deleplace , The image below shows the memory usage encountered on my production environment.

Awesome data, thank you @sakayaparp !


misleading error message

2015/09/03 11:48:13 FATAL: error: not a URL, server: "http://localhost:8091"
Please check that your -server parameter is a valid URL
(http://HOST:PORT), such as "http://localhost:8091",
to a couchbase server -- main.main() at main.go:142
This seems weird, since I was using the exact same command line I had been before.
It turns out the message means "Couchbase Server isn't running"

This might have been fixed with latest/master, as the line number no longer matches, and I see a different err msg.  Note: this was when manually launched, not launched using the toy/watson build...
Launched with simple Cfg provider...
$ ./cbft -server=http://localhost:8091
...
2015/09/04 09:40:05 FATAL: error: could not connect to server ("http://localhost:8091"), err: Get http://localhost:8091/pools: dial tcp <IP_ADDRESS>:8091: connection refused
  Please check that your -server parameter ("http://localhost:8091")
  is correct, the couchbase server is accessible,
  and auth is correct (e.g., http://USER:PSWD@HOST:PORT) -- main.main() at main.go:158

When launched with the couchbase bucket Cfg provider...
$ ./cbft -cfg=couchbase:http://cfg-auth:password@localhost:8091 -server=http://<IP_ADDRESS>:8091 -bindHttp=<IP_ADDRESS>:9090 -data=data0 -tags=feed,janitor,pindex,queryer
2015/09/04 09:37:38 main: ./cbft started (v0.2.0-105-ge878a7a/4.0.0)
...
2015/09/04 09:37:38 FATAL: main: could not start cfg, cfgConnect: couchbase:http://cfg-auth:password@localhost:8091, err: Get http://localhost:8091/pools: dial tcp <IP_ADDRESS>:8091: connection refused
  Please check that your -cfg/-cfgConnect parameter ("couchbase:http://cfg-auth:password@localhost:8091")
  is correct and/or that your configuration provider
  is available. -- main.main() at main.go:135

Versions of cbft/cbgt...
$ git describe
v0.2.0-105-ge878a7a
$ (cd ../cbgt && git describe)
v0.2.0-208-gb721c2c


Profile / Reduce Memory Limits

For Kubernetes & Marathon, we could measure how much memory the services use and update the k8s/marathon metadata accordingly.
Per @huntc in #97:

Tip: Use pmap to profile - you may get away with considerably less than that.

@longshorej is this still an ongoing concern? (I'm trying to figure out which issues need to be migrated to the new repos.)

This repository has been archived in favour of the specialised:

lagom-java-sbt-chirper-example which is a fork of this but only uses sbt.
lagom-java-maven-chirper-example which is a fork of this but only uses Maven.


Propagate PRs onto 6.6 branch

To resolve issue with large volume creation, we are forced to put PRs #2153 and #2151 onto 6.6 branch.
It also adopts https://github.com/lf-edge/eve/commit/d48bb4e4c40eedb995caadbaf69243e69db5c296 commit to resolve dependency issue.

It comes with https://github.com/lf-edge/eve/commit/d48bb4e4c40eedb995caadbaf69243e69db5c296 inside (461 files changed there).

Is there a need to cut a 6.6.2 release? If so, who is asking for it @giggsoff ?


If people does not select anything return directory path for dir_type and both_type

issue #44

Hey, thanks for helping me out. However, there is a problem with this. The only way developer can judge whether a file has been selected or not is the size of the paths passed to him as a parameter. Since you are returning the present directory path, there won't be a possible way to check whether directory was selected or last visited. That will affect the existing library users. I hope you are getting it.

Got it! Thanks.


Semaphore can acquire multiple permits

Thanks for your contribution! Unforntunatly it breaks the idea of PermitExpireableSemaphore - id per permit. Whereas, in your code multiple permits bounded to single id. Method for multiple permits aquiring should return list of ids.


[feature request] anchor link like GitHub

Example:

In order to support above, need to set anchorLinkSymbol to . But unfortunately it is escaped.
I will take a look and see if I can provide a PR.

Fixed in PR https://github.com/MoOx/markdown-it-toc-and-anchor/pull/6

Closed by #6


Update model on stop drag model.

Hi people,
I have just a little question about some work which I can't get working. Is it possible to load from a model a list and alter the position after dragging an element around in the callback?
somehow I can't get it working.

Does the code here help? https://github.com/codef0rmer/angular-dragdrop/issues/216

Reopen if the issue still persists.


Ref<L> for Entity does not work if you don't already HAVE the link object

What: You can't get the backing link if you don't already have the link to identify it.
public ref L Ref<L>(Link<L> link) where L : class => ref _world.GetComponent<L>(this, link);
Potential Fix:
Probably need to create a specific function just for this, or allow ref to return an array-like for a Wildcard match expression.

Resolved in 0.5.8


[WIP] Allow to ignore specific errors for specific files

This allows to exclude specific known errors for specific files, meaning
that instead of skipping all checks for a particular file, we can only
skip some specific ones (or, rather, ignore their results).
The rules can be specified globally, or per-tag, or per-component,
or per-rpm.
TODO: convert configs to use this.

getting
./errors.go:33:16: undefined: KnownErrors
./main.go:93:22: undefined: KnownErrors

For now, you can run go generate (since I can't update this PR at the moment).


Strip markup from achievement tooltips

Fixes #27
Removes the color tags from achievement descriptiosn and requirements.
It uses the same regex that GW2Efficiency does (modified slightly for C#), so should catch all cases.
It also converts
 tags into \n.
However, when it finds multiple it only ever uses a single newline.
This is what GW2Efficiency does and is a choice.

Originally "Look what I can do" was an extra line down. And it's with these single line quotes that the

 combo tends to be used.
There are some achievements that use multiple \n, which I haven't crushed into a single line.
Weirdly, I find their use cases to be more reasonable:

Anyway, happy to change or modify this. Just let me know.

I like it and am happy how the ones look that use the double \n.  Looks nice.
Thank you!


Connexion 3.0.0a4, GZIP Middleware

Description
In 3.0.0a2 I was able to use the Starlette GZipMiddleware class but in 3.0.0a4 I get an exception at runtime.
Expected behaviour
I would hope to be able to use Starlette middleware in my app's middleware stack.
Actual behaviour
Using
middlewares = [GZipMiddleware] + ConnexionMiddleware.default_middlewares
super().__init__(__name__,
                 lifespan=self.lifespan,
                 middlewares=middlewares,
                 specification_dir="spec/",
                 resolver=RelativeResolver(root_path="app.controllers"),
                 *args, **kwargs)

in my application constructor. But in 3.0.0a4 I get TypeError: GZipMiddleware.init() got an unexpected keyword argument 'lifespan'
It's clear that this is a result of the lifespan changes in the previous release.
Am I doing something wrong here?
Steps to reproduce
As above
Additional info:
Output of the commands:

python --version
pip show connexion | grep "^Version\:"

Thanks for the report @SpudInNZ
The issue is that we're passing the lifespan to every middleware, which isn't a problem for Connexion middleware, but could be for others. I'll look into how we can do this differently.

We ran into the same issue, here is our implementation for a CORSMiddleware - does this look right?
from starlette.middleware.cors import CORSMiddleware as StarletteCorsMiddleware

from starlette.types import ASGIApp, Receive, Scope, Send

class CORSMiddleware(StarletteCorsMiddleware):
    def __init__(self, next_app: ASGIApp, *args, **kwargs):
        kwargs["allow_origins"] = ["*"]
        kwargs["allow_methods"] = ["*"]
        kwargs["allow_headers"] = ["*"]
        kwargs["allow_credentials"] = True

        if "lifespan" in kwargs:
            kwargs.pop("lifespan")

        super().__init__(next_app, *args, **kwargs)

    async def __call__(self, scope: Scope, receive: Receive, send: Send) -> None:
        # Needs to be set so starlette router throws exceptions instead of returning error responses
        scope["app"] = "connexion"
        await super().__call__(scope, receive, send)


[Privacy] Add a storage option

Hello!
We need to use Amplitude without using cookies or local storage.
Now we use the disableCookies option, but Amplitude-TypeScript doesn't have the same option to disable localStorage.
I found an issue for Amplitude-JavaScript about this topic and the pull request https://github.com/amplitude/Amplitude-JavaScript/pull/320 to add a new option: storage. Will Amplitude-TypeScript have a storage option or option to disable localStorage?

Hi @Nikitina-Ann, thank you for choosing Amplitude. This configuration is available. Here's an example code to configure the Amplitude SDK to not use cookies or local storage:
``
import { createInstance } from '@amplitude/analytics-browser';
import { MemoryStorage } from '@amplitude/analytics-core';
const amplitude = createInstance();
amplitude.init('API_KEY', 'USER_ID', {
cookieStorage: new MemoryStorage(),
storageProvider: new MemoryStorage(),
})

I hope this is helpful. Additionally, if you want to check the different storage options one may use, I added some tests to document the different storage options/scenarios here: https://github.com/amplitude/Amplitude-TypeScript/pull/294/files.

@Nikitina-Ann I see what you mean here. I'm reopening this issue. I can confirm that the SDK does not allow opting out of cookie storage and local storage. We'll have to make some code changes to allow this to happen. One thing I will call out is, using a non persistent storage for web attribution feature will disable trackNewCampaigns option.

Hi @Nikitina-Ann, I'm trying to get back to my team and make sure we provide the appropriate solution for you. Meanwhile, wanted to take this chance to ask if anything has changed since we last been in contact? Would love to incorporate if you have change with how you want to use Amplitude SDK with regards to storage.


copy filters, inventory, docs and test from openshift-ansible

generate openshift-tools-ansible-inventory rpm

@kwoodson tamed the buildbot by ignoring files with extension "adoc" (/usr/bin/file was miss-identifying them as python and running pylint on them)

lgmt :+1:


julia.withPackages: fix artifact builds by removing extraneous env-vars

Description of changes
It turns out the derivation we've been using for archive-like Julia artifacts has been introducing an env-vars file into the output. I don't know why this is happening, but it causes problems for some artifacts. I first noticed this with the TZJData.jl package, which downloads an artifact containing a directory of time zone databases. When we put an env-vars file in this folder, some code in TimeZones.jl tries to read it as a time zone database and crashes.
Maybe there's a cleaner way to keep stdenv.mkDerivation from creating the env-vars file?
This brings our test failures from 52 down to 39 out of the top 1000 packages. So this PR brings our success rate from 94.8% to 96.1%.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[X] Fits CONTRIBUTING.md.

BTW, I was wondering if we could create a GitHub issues label called topic: julia or similar. It would help me to organize the different Julia-related issues. @NickCao, any chance you could do that? Thanks!

The problem is with sourceRoot = "." and cp -r . $out in installPhase. We shall either extract src into a subdirectory, or set dontUnpack and do the unpacking/copying ourselves.

BTW, I was wondering if we could create a GitHub issues label called topic: julia or similar. It would help me to organize the different Julia-related issues. @NickCao, any chance you could do that? Thanks!

I'm not entirely sure about the process of creating new labels, would ask around.

The problem is with sourceRoot = "." and cp -r . $out in installPhase. We shall either extract src into a subdirectory

Ah, thanks for the tip. It seems to be fixed now using a preUnpack hook.

Yep it looks good, the top 1000 tests completed and we're at 39 failures still.


Update metrics.md

update the different assignment unit vs. analysis unit

🚀 Deployed on https://660f3dd5bd22c214bc9b4153--cozy-fox-0defec.netlify.app


docker --format not working

I have 2 ubuntu 14.04.1 hosts that both return the following output on any container, but same docker version.  I also got this error when using docker version 1.x.

Distributor ID: Ubuntu
Description:    Ubuntu 14.04.1 LTS
Release:        14.04
Codename:       trusty
uname -a
Linux goAwayS 3.13.0-38-generic #65-Ubuntu SMP Thu Oct 9 11:36:50 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
docker ps
CONTAINER ID        IMAGE                   COMMAND             CREATED             STATUS              PORTS                           NAMES
334a0bd7ec99        sameersbn/bind:latest   "/start"            38 hours ago        Up 53 seconds       10000/tcp, <IP_ADDRESS>:53->53/udp   bind
docker inspect -f '{{ .NetworkSettings }}' bind
Template parsing error: template: :1: unexpected unclosed action in command
2014/10/26 23:09:02 Template parsing error: template: :1: unexpected unclosed action in command
docker inspect -f '{{ .ResolvConfPath }}' bind
Template parsing error: template: :1: unexpected unclosed action in command
2014/10/26 23:14:45 Template parsing error: template: :1: unexpected unclosed action in command
docker inspect bind | head
[{
"Args": [],
"Config": {
"AttachStderr": false,
"AttachStdin": false,
"AttachStdout": false,
"Cmd": [
"/start"
],
"CpuShares": 0,
docker inspect bind | grep IPA
    "IPAddress": "<IP_ADDRESS>",

Just for the record, I have came across same issue once, it turned out docker command was wrapped in a script similar to this:
#!/bin/sh
/opt/bin/docker $@

So the gotcha is in how shell interprets $@, it breaks up anything with whitespace as separate arguments, dropping the quotes you have passed to the wrapper. To fix this you simply need to change such script to look like this:
#!/bin/sh
/opt/bin/docker "$@"

it turned out docker command was wrapped in a script similar to this

Was that the way it was shipping with your distro? Or did you create that script yourself?

@thaJeztah well, it was on Ubuntu Snappy, which is like alpha or pre-alpha, I suppose.

@errordeveloper hm okay, still important I think. I'll ask on docker-dev if this should be made into a new issue on docker, or reported to ubuntu somewhere

@errordeveloper I had a quick chat on #docker-dev (here). There's no need to create a separate issue on Docker here, but could you report this issue to Ubuntu? Thanks!

Sure, it's beimg worked on. I just posted here in case anyone comes across
the same issue, they can have at least one clue, cause it was quite obscure
for me and did take a few days to find.
On Sun, 11 Jan 2015 22:46 Sebastiaan van Stijn<EMAIL_ADDRESS>wrote:

@errordeveloper https://github.com/errordeveloper I had a quick chat on
#docker-dev (here
https://botbot.me/freenode/docker-dev/2015-01-11/?msg=29279266&page=3).
There's no need to create a separate issue on Docker here, but could you
report this issue to Ubuntu? Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/issues/8792#issuecomment-69515724.

Okay, good to hear it's taken care of. And thank you for posting it here to
assist others. Much appreciated.


support suggestion when debugging.

a = []
a.append('string')
b = a[0]

in debugging when hover on b we can see 'string'. but when write b. it doesn't show StringClass methods and attributes.

@byehack, I presume that your sample is meant to say b = a[0].
You haven't included any type annotations in this code, so Pylance attempts to use type inference to determine the type of a and b. Its inference for a is based on the assignment a = [], which does not provide any information about the type contained within the list, so Pylance infers the type of a to be List[Any]. Its inference for b is based on the assignment b = a[0], and b is inferred to have type Any.
This is a situation where you will need to provide a type annotation to get the behavior you're looking for. You can provide one for either a or b (or both).
a: List[str] = []
a.append('string')
b = x[0]

or
a = []
a.append('string')
b: str = x[0]

Note that debugging means VS Code is going to ask the debugger for the values, and can offer a different experience. This is similar to (and more powerful than) using notebooks with IPython.

@erictraut thanks for reply
i mean can't you add option to suggest on type from real value in debugging?

The language server has no knowledge of anything but the files themselves, and has no idea what debugging is or how it would access it.
Additionally, you can't write code during a debugging session without the debugger going out of sync, so the LS will think one thing and the debugger will think another, and the types that a type checker reads statically do not have to match the actual types in a running program (e.g., the source may say it's a Protocol, but protocols don't "exist", they just describe a value that has certain methods/properties).
There's some discussion on how to reconcile something like a notebook kernel and a static type checker (where you are naturally in an environment where you can edit and run at the same time), but I can't think of any solution that would make this feasible for a debugging session...

but I can't think of any solution that would make this feasible for a debugging session...

it would be great if debugpy could connect to LSP for povide suggestion.
but if you sure this can't be happening you can close this issue.
i will leave this issue open for doing on later time.

I'm going to close this; I don't think we're going to be calling out into the debugger to gain extra type information.


bug: gelguy/wilder unusable with latest main commit

Did you check docs and existing issues?

[X] I have read all the lazy.nvim docs
[X] I have searched the existing issues of lazy.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.8.3
Operating system/version
Linux
Describe the bug
On current main (5af93806aaa33fd9e8b4a7a32e9f847a3ad64c2a), the plugin https://github.com/gelguy/wilder.nvim cannot be used anymore. However, with stable (8077428e63feb0f3bf795d53b23ba1695b28ab0e) it works as expected.
Steps To Reproduce

Start nvim with the config below via nvim -u repro.lua
Press :. A popup menu appears from wilder
Run :Lazy update to self-update lazy to latest main state
Restart nvim
Press :. No more popup appears.

Expected Behavior
Wilder just works.
Repro
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({
		"git",
		"clone",
		"--filter=blob:none",
		"https://github.com/folke/lazy.nvim.git",
		"--branch=stable",
		lazypath,
	})
end
vim.opt.rtp:prepend(lazypath)

require("lazy").setup({
	{
		"gelguy/wilder.nvim",
		config = function()
			local wilder = require("wilder")
			wilder.setup({ modes = { ":", "/", "?" } })
			-- Disable Python remote plugin
			wilder.set_option("use_python_remote_plugin", 0)

			wilder.set_option("pipeline", {
				wilder.branch(
					wilder.cmdline_pipeline({
						fuzzy = 1,
						fuzzy_filter = wilder.lua_fzy_filter(),
					}),
					wilder.vim_search_pipeline()
				),
			})

			wilder.set_option(
				"renderer",
				wilder.renderer_mux({
					[":"] = wilder.popupmenu_renderer({
						highlighter = wilder.lua_fzy_highlighter(),
						left = {
							" ",
							wilder.popupmenu_devicons(),
						},
						right = {
							" ",
							wilder.popupmenu_scrollbar(),
						},
					}),
					["/"] = wilder.wildmenu_renderer({
						highlighter = wilder.lua_fzy_highlighter(),
					}),
				})
			)
		end,
		dependencies = { "romgrk/fzy-lua-native" },
	}
})

ok, I've added back a hack to work around incorrect require statements by some plugins.
The reason this no longer worked is because fzy-native-lua does a require that is not correct. See here https://github.com/romgrk/fzy-lua-native/blob/master/lua/fzy-lua-native.lua
That actually works with the default Neovim loader, because of a bug there. Lazy does it in the correct way.
Would be great if you could post a bug upstream to the fzy-native-lua repo to let them know about this.
It's also really bad practice to have lua/init.lua in a plugin. They should namespace all their lua modules under fzy-lua-native

Thanks for the revert. I think I am still missing the details on how the require statement should look like before I could open a bug upstream..

For anyone having some problems with Wilder, I had some problems with using wilder search and :UpdateRemotePlugins helped me solve it. My configuration is as simple as @languitar above.


Refactor pipeline_cli_factory into class

Allow snk to be used as a library.
from snk import Snk
app = Snk()

[project.scripts]
workflow = "workflow.cli:app"

✅