Issue with annotations without unique location

First off, thank you for the excellent work on the library. Much appreciated....
This might be more of a design question. I have an issue where annotations end up having the exact same location. I have coded it so that only unique annotations with a count of 1 show a custom callout view that allows you to open a details page on the item. But when multiple annotations are in the same spot, they show up at grouped and as such, there is no way to view the custom call out. What is the recommended way to handle this? Do I need a new callout that shows a list of the annotations in that point? Or is there a way to somehow show all the annotations in that location (e.g. spread them out).

You need to show a list of items in the cluster. In one of my apps, I show a screen with a list and then the user can select the item before ending up on the details page.

Thanks. That's what I've done and it worked well.
On Mar 19, 2016, at 7:51 AM, Claus Höfele<EMAIL_ADDRESS>wrote:
You need to show a list of items in the cluster. In one of my apps, I show
a screen with a list and then the user can select the item before ending up
on the details page.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/choefele/CCHMapClusterController/issues/122#issuecomment-198716458


Per Directory Settings Table Display Issue

Table options set to Hide Table Icons (No) Small Table Icons (No)
When viewing a Virtual Server in the apache module, with per directory options - the folders are displayed as tiles with folder icon, however you can't see which directory is which as the path gets truncated so all directorys seem the same, tooltips don;t work in safari on the buttons so the only way to find a directory is to go into each one. On our application server we have many directories and this is really frustrating - is the any way to either wrap the path or show path in tooltips?

Hi, Chris.
Is this how you see it?

Hi Ilia,
No it’s the one after when you click the globe icon on the screen you sent.
<EMAIL_ADDRESS>The per-directory options, you can only see /srv/www/htt on buttons so its difficult to find folders, they do seem to wrap if they are really long but you still miss some of the path.
Hope this help, let me know if you need anymore info
Cheers
Chris
From: Ilia Rostovtsev<EMAIL_ADDRESS>Reply-To: authentic-theme/authentic-theme<EMAIL_ADDRESS>Date: Wednesday, 5 June 2019 at 21:29
To: authentic-theme/authentic-theme<EMAIL_ADDRESS>Cc: Christopher Mills<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [authentic-theme/authentic-theme] Per Directory Settings Table Display Issue (#1365)
Hi, Chris.
Is this how you see it?
[image]https://user-images.githubusercontent.com/4426533/58988059-816c5100-87e9-11e9-9942-b78cad0e629c.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/authentic-theme/authentic-theme/issues/1365?email_source=notifications&email_token=AGVITHSCFGZOUPDWN75VNCTPZAOZHA5CNFSM4HUEMA2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXA5MEQ#issuecomment-499242514, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGVITHXN7UOL5AGG6MKKRFTPZAOZHANCNFSM4HUEMA2A.

I see now.
I surely needs a tooltip.

Great, I really like this theme and that’s the only issue I’ve seen.
Thanks for the response. Keep up the good work
From: Ilia Rostovtsev<EMAIL_ADDRESS>Reply-To: authentic-theme/authentic-theme<EMAIL_ADDRESS>Date: Thursday, 6 June 2019 at 11:53
To: authentic-theme/authentic-theme<EMAIL_ADDRESS>Cc: Christopher Mills<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [authentic-theme/authentic-theme] Per Directory Settings Table Display Issue (#1365)
I see now.
I surely needs a tooltip.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/authentic-theme/authentic-theme/issues/1365?email_source=notifications&email_token=AGVITHWAB4DDBX4JJM42CS3PZDUBNA5CNFSM4HUEMA2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXCPJPQ#issuecomment-499446974, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGVITHSNG5RY45QUCS3JJB3PZDUBNANCNFSM4HUEMA2A.

Please install latest dev version of the theme (from theme configuration) to give it a try.
Thanks for reporting.

Added another small, related patch. Update over again if you just did.

Cheers, Ill try it now
From: Ilia Rostovtsev<EMAIL_ADDRESS>Reply-To: authentic-theme/authentic-theme<EMAIL_ADDRESS>Date: Thursday, 6 June 2019 at 13:47
To: authentic-theme/authentic-theme<EMAIL_ADDRESS>Cc: Christopher Mills<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [authentic-theme/authentic-theme] Per Directory Settings Table Display Issue (#1365)
Added another small, related patch. Update over again if you just did.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/authentic-theme/authentic-theme/issues/1365?email_source=notifications&email_token=AGVITHQ5HQQBZUYG2JLHPVLPZEBPDA5CNFSM4HUEMA2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXCXCWQ#issuecomment-499478874, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGVITHSSQHLSZM6KIGGKVE3PZEBPDANCNFSM4HUEMA2A.

Updated, works much better thank you
From: Ilia Rostovtsev<EMAIL_ADDRESS>Reply-To: authentic-theme/authentic-theme<EMAIL_ADDRESS>Date: Thursday, 6 June 2019 at 13:47
To: authentic-theme/authentic-theme<EMAIL_ADDRESS>Cc: Christopher Mills<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [authentic-theme/authentic-theme] Per Directory Settings Table Display Issue (#1365)
Added another small, related patch. Update over again if you just did.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/authentic-theme/authentic-theme/issues/1365?email_source=notifications&email_token=AGVITHQ5HQQBZUYG2JLHPVLPZEBPDA5CNFSM4HUEMA2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXCXCWQ#issuecomment-499478874, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGVITHSSQHLSZM6KIGGKVE3PZEBPDANCNFSM4HUEMA2A.

You're welcome.


Add drop-down for DMI/LCD style

Request to add drop-down option to switch between Infotec DMI and TfWM LCD displays.

Complete


Found Easter Egg in quiz.dart

Found Easter Egg :)

https://github.com/appwrite/demo-todo-with-svelte/pull/32


InfluxDB 2.0: The "/sources/{sourceID}/buckets" response type is defined as "Buckets" but returns a list of "Bucket" [area/api]

The /sources/{sourceID}/buckets response type is defined as Buckets:
/sources/{sourceID}/buckets:
    get:
      tags:
        - Sources
        - Buckets
      summary: Get a sources buckets (will return dbrps in the form of buckets if it is a v1 source)
      parameters:
          - $ref: '#/components/parameters/TraceSpan'
          - in: path
            name: sourceID
            schema:
              type: string
            required: true
            description: ID of the source
          - in: query
            name: org
            description: specifies the organization of the resource
            required: true
            schema:
              type: string
      responses:
        '200':
          description: a source
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/Buckets"
        '404':
          description: source not found
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/Error"
        default:
          description: unexpected error
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/Error"

but HTTP response returns an array of Bucket:
-> GET
-> http://<IP_ADDRESS>:9999/api/v2/sources/039042392f62d000/buckets

<- 200
<- OK
<- [{
	"name": "telegraf",
	"rp": "autogen",
	"retentionPeriod": 0
}, {
	"name": "_internal",
	"rp": "monitor",
	"retentionPeriod":<PHONE_NUMBER>00000
}]

cc @bthesorceror


ADD SCSS?

It would be great to add SCSS or SASS for npm publish.

+1 for this request. It would be useful tu publish the .scss on npm so you can @import directly on your scss file and avoid extra resolvers in your webpack.


Error encountered when run synthesis.py  (File "synthesis.py", line 178, in synthesize)

I got the following errors when run the copy-synthesis with the "awb_arctic_a0040.wav" example.
Traceback (most recent call last):
File "synthesis.py", line 374, in
main(sys.argv[1:])
File "synthesis.py", line 371, in main
synthesizef(args.fs, shift=args.shift, dftlen=args.dftlen, ff0=args.f0, flf0=args.logf0, fspec=args.spec, ffwlspec=args.fwlspec, ffwcep=args.fwcep, fmcep=args.mcep, fnm=args.nm, ffwnm=args.fwnm, nm_cont=args.nm_cont, fpdd=args.pdd, fmpdd=args.mpdd, fsyn=args.synth, verbose=args.verbose)
File "synthesis.py", line 340, in synthesizef
syn = synthesize(fs, f0s, SPEC, NM=NM, nm_cont=nm_cont, verbose=verbose)
File "synthesis.py", line 178, in synthesize
if winlen>dftlen: raise ValueError('winlen({})>dftlen({})'.format(winlen, dftlen)) # pragma: no cover
ValueError: winlen(801)>dftlen(118)
I got the f0 spec and nm features with the analysis,py (i'm sure pyworld is used),
analysisf("awb_arctic_a0040.wav",
shift=args.shift,
dftlen=args.dftlen,
finf0txt=args.inf0txt, f0_min=args.f0_min, f0_max=args.f0_max, ff0="awb_arctic_a0040.f0", f0_log=args.f0_log,
finf0bin=args.inf0bin,
fspec="awb_arctic_a0040.spec", spec_mceporder=59, spec_fwceporder=args.spec_fwceporder, spec_nbfwbnds=args.spec_nbfwbnds,
fpdd="pdd/"+fileid+".pdd", pdd_mceporder=59,
fnm="awb_arctic_a0040.nm", nm_nbfwbnds=None,
preproc_fs=args.preproc_fs, preproc_hp=args.preproc_hp,
verbose=args.verbose)
and run the synthesis command as
python synthesis.py awb_arctic_a0040.resyn.wav --f0 awb_arctic_a0040.f0 --spec awb_arctic_a0040.spec --nm awb_arctic_a0040.nm
Please help.  Thx.

Did you replace the f0 estimator? Maybe changed the minimum value for f0?
I've updated the master with a fix for this case and some warning. please pull the master's updates.

@gillesdegottex  Thanks much for your reply. I used the REAPER estimator as recommended by default. I have updated to the latest version, but is seems that the same error still exists.

If you analyse with --nm_nbfwbnds, you have to use --fwnm then during synthesis.
Same for --spec_mceporder that needs --mcep at synthesis.
Try this first and tell me if there any more errors.

Seems to be solved.


Semantic Role Labelling

Adds semantic role labelling notebooks.
The visualisation is difficult in NetworkX so I used Gephi offline (~2000 ORG entities in the network). I will send the link to the input and output data files offline.

Very interesting analyses!
I have a few remarks and questions, some on documentation or code quality, others on the analysis:

flair should be added to requirements.txt
why are there two requirements.txt files? It seems like the one in the srl directory has more packages than required to run the notebooks
I see some duplicated code in ner_flair.ipynb and preprocess_ngram_named_entities.ipynb, probably worth moving into a python module at some point
In general, it's probably good to move the preprocessing into a separate script, to guarantee consistency for the different datasets and part of the pipelines
Docstrings in preproces_ngram_named_entities.ipynb seem to be copy-pasted and are not all correct ;)
Do I understand correctly that NER is applied on lowercase text? It looks like the FLAIR model can work on cased text, case should be informative for NER.
In oecd_semantic_role_labelling.ipynb step 3 loads the named entities, but these are not used after that.
The clean_srl_results function is very long and can be split up to make it more readable
It's not clear to me what the difference is between the highest_quality_result and the overall_results
It would be interesting to also see the verbs belonging to the edges in the graph. How often does it happen that there's multiple edges with possibly different labels between entities? if this happens often, then showing weighted edges can be informative.
Do I understand correctly that you only check whether the subject or object matches a possible organization name, but not the span found by the NER tagger? Do you know how many false positives this results in?
More in general: how well do the NER tagger and the SRL tagger match? How often are the subjects or objects entities, and how often do they partially overlap? Some of these statistics can provide a sanity check of how well the pipeline works

One other thing: I can really recommend the sigma-js plugin for gephi, to export a large network from gephi to an interactive static web page, very useful for less technical users.

One other thing: I can really recommend the sigma-js plugin for gephi, to export a large network from gephi to an interactive static web page, very useful for less technical users.

Thanks! I know of sigma-js but I did not know Gephi could export sigma-js. That will be very useful for the project leaders to use.


Create Docker App

Still need to think about how to connect from Docker to root system for access and mange apex image files etc.

Issue moved to OracleXE APEX Project: https://github.com/OraOpenSource/oraclexe-apex/issues/5


Add EVM serialization updates & FLOW transfer txn

Description

Updates EVM contract with changes from https://github.com/onflow/flow-go/pull/5932
Adds a FLOW transfer transaction directing to either a Cadence or EVM recipient from the signer's Cadence Vault

For contributor use:

[x] Targeted PR against main branch
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the standards mentioned here.
[x] Updated relevant documentation
[x] Re-reviewed Files changed in the Github PR explorer
[x] Added appropriate labels

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

Project coverage is 86.19%. Comparing base (02aab3f) to head (36d7acb).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #77   +/-   ##
=======================================
  Coverage   86.19%   86.19%
=======================================
  Files          18       18
  Lines         884      884
=======================================
  Hits          762      762
  Misses        122      122

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


[SourceKit] Migrate rename test to only check rename ranges and not the application of the rename

In an upcoming PR, I will remove the rename refactoring actions that apply the rename and return the renamed source code. Instead, clients should call into the sourcekitd requests that return the ranges to rename.
To keep that upcoming PR smaller, I’m migrating all the test cases to use find-(local-)rename-ranges now.

@swift-ci Please smoke test

@swift-ci Please smoke test


Centralize URL slug calculation and change its format.

https://github.com/themotte/qcvault/issues/20


Roadmap: v5

I never expected this small project to be this popular. It hit 500 stars yesterday! So thank you everyone for trusting this code and bringing it thus far.
There are a lot of things that I'd like to be improved

better build setup
better dev setup
tests and tests and tests
typescript

Hoping to make progress on this!

Update:
We're working hard to release the V5 version

it'll not have any breaking changes per-se
porting the code to TypeScript #130 (please see the progress on the main PR)
removed Webpack in favour of microbundle
adding tests

I just published a major version release 5.0.0! Please test it out and let me know! Thanks!


Add support for {% trans trimmed ... %}

Same behavior as in Django: All linebreaks and the whitespace surrounding them are replaced with a single space. Also added a policy ext.i18n.trimmed to enable this globally (can be overridden using the notrimmed modifier).
closes #504

Note: There is one minor case of backwards incompatibility which has no real-world implications. This was already valid Jinja code: {% trans somevar %}...{% endtrans %}, making the following ambiguous:
{% trans trimmed %}
    hello world
{% endtrans %}

However, trimmed is a somewhat unlikely variable name, especially in the context of a trans block and the fact that you can specify variables without an assignment there is undocumented. So I don't think anyone is using the combination of these two things. When using a variable assignment the behavior does not change - the modifier only applies if it is not followed by a =.

Works for me!


Semantic UI React raises warnings in StrictMode

Bug Report
Steps
Heya!  Well done for all your work on Semantic UI React!
You may already know (or not?) that as of React 16.3 3 lifecycle methods have been deprecated:

componentWillMount
componentWillReceiveProps
componentWillUpdate

If you turn on React's StrictMode it warns you about usage of the deprecated APIs.  I just dropped it into my app to experiment and:
Expected Result
I'd hope to see no warnings
Actual Result
But I do see warnings 💣
warning.js:33 Warning: Unsafe lifecycle methods were found within a strict-mode tree:
    in AppRaw (created by Route)
    in Route (created by withRouter(AppRaw))
    in withRouter(AppRaw)
    in Router
    in Provider
    in AppContainer

componentWillMount: Please update the following components to use componentDidMount instead: Route, Switch, t

componentWillReceiveProps: Please update the following components to use static getDerivedStateFromProps instead: Menu, Route, Switch

Learn more about this warning here:
https://fb.me/react-strict-mode-warnings
printWarning @ warning.js:33
warning.js:33 Warning: Unsafe lifecycle methods were found within a strict-mode tree:
    in AppRaw (created by Route)
    in Route (created by withRouter(AppRaw))
    in withRouter(AppRaw)
    in Router
    in Provider
    in AppContainer

componentWillReceiveProps: Please update the following components to use static getDerivedStateFromProps instead: DimmerInner

Learn more about this warning here:
https://fb.me/react-strict-mode-warnings

Some are React-Router specific - (They know about this) but things like Menu and DimmerInner are Semantic UI React I believe.  There's probably others too - I'm not using all components.
Version
0.81.1
Testcase
Probably the best way to see this would be to add:
    <React.StrictMode>
    </React.StrictMode>

around the whole docs project.
Hope this helps - thanks for your hard work!

Duplicate #2732.


[feature-request] Collection words from current file please

This plugin runs more fluently than All AutoComplete plugin, but one thing really hold me back is that All AutoComplete can suggest words from current documents.
It would be much helpful when I am inputing some terms not existing in the wordlist, for example my name, if I have entered my name before, All AutoComplete is capable to suggest my name if I am trying to type it again.
But All AutoComplete will stuck for seconds if the word list is too long (eg. 40k words), I really hope this plugin can collect words from current document.

Thank you for the feedback.
I know VSCode has word-based suggestions but only when there is no other completion provider (ref). And there is a good reason "We do this to prevent mixing good suggestions with word-based suggestions".
I guess the current dictionary already covers most of the words we use. You can add more words in your user settings (or even create a PR).

I have two concerns about this feature:

The quality of suggestions. This extension uses a relative small built-in dictionary rather than those used by some spell checkers (because they have, for example, many uncommon abbreivations).
Propose: collect words (space-split) from the current file, and use a filter to remove some unwanted words (e.g. Chinese words).

The performance.
Propose: 1) incrementally rebuild the dictionary, and 2) disable this feature when editing large files. (We already have such a feature (performance sense) in Markdown All in One.)

BTW, in case you don't know, this extension now offers suggestions if you are editing string/comment parts in Python/JS/TS files (need to enable the programmingLanguage option). I love it and am going to enable it by default starting from the next release. Feedback is welcome 😉.

I have two concerns about this feature:

The quality of suggestions. This extension uses a relative small built-in dictionary rather than those used by some spell checkers (because they have, for example, many uncommon abbreivations).
Proposal: collect words (space-split) from the current file, and use a filter to remove some unwanted words (e.g. Chinese words). (Probably we should only enable this feature in Markdown files. There may be problems to collect words from HTML/LaTeX/Python/… files.)
The performance.
Proposal: 1) incrementally rebuild the dictionary, and 2) disable this feature when editing large files. (We already have such a feature (performance sense) in Markdown All in One.)

Thank you so much for this very cool extension!
I think @skywind3000 made a rather good suggestion which could become a very useful feature. This should be optional so that the default behaviour will not disturb the users with low quality suggestions. Users who are suffering from long typing works such as paper writing might benifit a lot by turnning this feature on.
I can't tell you how much I have benifit from this extension. Thank you very much again.
BTW: @skywind3000 Thank you very much for your cool works (you know what I mean :-)

I have two concerns about this feature:

The quality of suggestions. This extension uses a relative small built-in dictionary rather than those used by some spell checkers (because they have, for example, many uncommon abbreivations).
Proposal: collect words (space-split) from the current file, and use a filter to remove some unwanted words (e.g. Chinese words). (Probably we should only enable this feature in Markdown files. There may be problems to collect words from HTML/LaTeX/Python/… files.)
The performance.
Proposal: 1) incrementally rebuild the dictionary, and 2) disable this feature when editing large files. (We already have such a feature (performance sense) in Markdown All in One.)

Thank you so much for this very cool extension!
I think @skywind3000 made a rather good suggestion which could become a very useful feature. This should be optional so that the default behaviour will not disturb the users with low quality suggestions. Users who are suffering from long typing works such as paper writing might benifit a lot by turnning this feature on.
I can't tell you how much I have benifit from this extension. Thank you very much again.
BTW: @skywind3000 Thank you very much for your cool works (you know what I mean :-)

@starship863 Thanks for the feedback.
As you mentioned paper writing, do you expect this for LaTeX documents?

@starship863 Thanks for the feedback.
As you mentioned paper writing, do you expect this for LaTeX documents?

@starship863 Thanks for the feedback.
As you mentioned paper writing, do you expect this for LaTeX documents?

The present version works well for latex documents. I have been using this extension in latex documents for a long time.
For the new feature, I hope it may ignore the latex macros starting with ""

@starship863 Thanks for the feedback.
As you mentioned paper writing, do you expect this for LaTeX documents?

The present version works well for latex documents. I have been using this extension in latex documents for a long time.
For the new feature, I hope it may ignore the latex macros starting with ""

Just tried and realized we have to make use of some Markdown/LaTeX parsers, otherwise there will be tons of low-quality suggestions.

Just tried and realized we have to make use of some Markdown/LaTeX parsers, otherwise there will be tons of low-quality suggestions.

I also encountered this problem. I achieved this requirement by modifying the DictionaryCompletionItemProvider class implemention in completion.js file under ~\.vscode\extensions\yzhang.dictionary-completion-1.2.2\out\src, where ~ represents the windows user path. The specific modifications are as follows:
class DictionaryCompletionItemProvider {
    constructor(fileType) {
        this.fileType = fileType;
    }
    provideCompletionItems(document, position, _token) {
        // add the document text wordlists
        let allWords_for_completion = [...allWords];
        for (let i = 0; i < document.lineCount; ++i) {
            const line = document.lineAt(i);
            const text = line.text;
            const words = text.split(/\b(\w+)\b/);
            words.forEach((word) => {
                if (!allWords_for_completion.includes(word)) {
                    allWords_for_completion.push(word);
                }
            });
        }

        const lineText = document.lineAt(position.line).text;
        const textBefore = lineText.substring(0, position.character);
        const docTextBefore = document.getText(new vscode_1.Range(new vscode_1.Position(0, 0), position));
        const wordBefore = textBefore.replace(/\W/g, ' ').split(/[\s]+/).pop();
        const firstLetter = wordBefore.charAt(0);
        const followingChar = lineText.charAt(position.character);
        const addSpace = vscode.workspace.getConfiguration('dictCompletion').get('addSpaceAfterCompletion') && !followingChar.match(/[ ,.:;?!\-]/);
        if (wordBefore.length < vscode.workspace.getConfiguration('dictCompletion').get('leastNumOfChars')) {
            return [];
        }
        switch (this.fileType) {
            case "markdown":
                // [caption](don't complete here)
                if (/\[[^\]]*\]\([^\)]*$/.test(textBefore)) {
                    return [];
                }
                return this.completeByFirstLetter(firstLetter, allWords_for_completion, addSpace);
            case "latex":
                // `|` means cursor
                // \command|
                if (/\\[^ {\[]*$/.test(textBefore)) {
                    return [];
                }
                // \begin[...|] or \begin{...}[...|]
                if (/\\(documentclass|usepackage|begin|end|cite|ref|includegraphics)({[^}]*}|)?\[[^\]]*$/.test(textBefore)) {
                    return [];
                }
                // \begin{...|} or \begin[...]{...|}
                if (/\\(documentclass|usepackage|begin|end|cite|ref|includegraphics|input|include)(\[[^\]]*\]|)?{[^}]*$/.test(textBefore)) {
                    return [];
                }
                return this.completeByFirstLetter(firstLetter, allWords_for_completion, addSpace);
            case "html":
                // <don't complete here>
                if (/<[^>]*$/.test(textBefore)) {
                    return [];
                }
                //// Inside <style> or <srcipt>
                let docBefore = document.getText(new vscode.Range(new vscode.Position(0, 0), position));
                if (docBefore.includes('<style>')
                    && (!docBefore.includes('</style>')
                        || docBefore.match(/<style>/g).length > docBefore.match(/<\/style>/g).length)) {
                    return [];
                }
                if (docBefore.includes('<script>')
                    && (!docBefore.includes('</script>')
                        || docBefore.match(/<script>/g).length > docBefore.match(/<\/script>/g).length)) {
                    return [];
                }
                return this.completeByFirstLetter(firstLetter, allWords_for_completion, addSpace);
            case "javascript":
            case "typescript":
                //// Multiline comment
                if (/\/\*((?!\*\/)[\W\w])*$/.test(docTextBefore)) {
                    return this.completeByFirstLetter(firstLetter, allWords_for_completion, addSpace);
                }
                //// Inline comment or string
                const tmpTextBeforeJs = textBefore.replace(/(?<!\\)('|").*?(?<!\\)\1/g, '');
                if (/\/{2,}/.test(tmpTextBeforeJs) //// inline comment
                    || (/(?<!\\)['"]/.test(tmpTextBeforeJs) //// inline string
                        && !/(import|require)/.test(tmpTextBeforeJs.split(/['"]/)[0]) //// reject if in import/require clauses
                    )) {
                    return this.completeByFirstLetter(firstLetter, allWords_for_completion, addSpace);
                }
                return [];
            case "python":
                //// Multiline comment (This check should go before inline comment/string check)
                const tmpDocTextBefore = docTextBefore.replace(/('''|""")[\W\w]*?\1/g, '');
                if (/('''|""")((?!\1)[\W\w])*$/.test(tmpDocTextBefore)) {
                    return this.completeByFirstLetter(firstLetter, allWords_for_completion, addSpace);
                }
                //// Inline comment or string
                const inlineCheckStr1 = textBefore.replace(/('''|""")/g, '').replace(/f?(?<!\\)('|").*?(?<!\\)\1/g, '');
                const inlineCheckStr2 = inlineCheckStr1.replace(/((?<!\\){).*?((?<!\\)})/g, '');
                if (/#+/.test(inlineCheckStr1)
                    || /(?<!\\|f)['"]/.test(inlineCheckStr1)
                    || /f(?<!\\)['"][^{]*$/.test(inlineCheckStr2)) {
                    return this.completeByFirstLetter(firstLetter, allWords_for_completion, addSpace);
                }
                return [];
            // TMP adapted from JS/TS
            case "c":
                //// Multiline comment
                if (/\/\*((?!\*\/)[\W\w])*$/.test(docTextBefore)) {
                    return this.completeByFirstLetter(firstLetter, allWords_for_completion, addSpace);
                }
                //// Inline comment or string
                const tmpTextBeforeC = textBefore.replace(/(?<!\\)(").*?(?<!\\)\1/g, '');
                if (/\/{2,}/.test(tmpTextBeforeC) //// inline comment
                    || (/(?<!\\)"/.test(tmpTextBeforeC) //// inline string
                        && !/#include/.test(tmpTextBeforeC.split(/"/)[0]) //// reject if in include clauses
                    )) {
                    return this.completeByFirstLetter(firstLetter, allWords_for_completion, addSpace);
                }
                return [];
            // TMP not tested
            case "vue":
                // <don't complete here>
                if (/<[^>]*$/.test(textBefore)) {
                    return [];
                }
                //// Multiline comment
                if (/\/\*((?!\*\/)[\W\w])*$/.test(docTextBefore)) {
                    return this.completeByFirstLetter(firstLetter, allWords_for_completion, addSpace);
                }
                //// Inline comment or string
                const tmpTextBeforeVue = textBefore.replace(/(?<!\\)('|").*?(?<!\\)\1/g, '');
                if (/\/{2,}/.test(tmpTextBeforeVue) //// inline comment
                    || (/(?<!\\)['"]/.test(tmpTextBeforeVue) //// inline string
                        && !/(import|require)/.test(tmpTextBeforeVue.split(/['"]/)[0]) //// reject if in import/require clauses
                    )) {
                    return this.completeByFirstLetter(firstLetter, allWords_for_completion, addSpace);
                }
                return [];
        }
    }
    completeByFirstLetter(firstLetter, wordlist, addSpace = false) {
        if (firstLetter.toLowerCase() == firstLetter) {
            // Lowercase letter
            let completions = wordlistToComplItems(wordlist.filter(w => w.toLowerCase().startsWith(firstLetter)));
            if (addSpace) {
                completions.forEach(item => item.insertText = item.label + ' ');
            }
            return new Promise((resolve, reject) => resolve(completions));
        }
        else {
            // Uppercase letter
            let completions = wordlist.filter(w => w.toLowerCase().startsWith(firstLetter.toLowerCase()))
                .map(w => {
                let newLabel = w.charAt(0).toUpperCase() + w.slice(1);
                let newItem = new vscode.CompletionItem(newLabel, vscode.CompletionItemKind.Text);
                if (addSpace) {
                    newItem.insertText = newLabel + ' ';
                }
                return newItem;
            });
            return new Promise((resolve, reject) => resolve(completions));
        }
    }
}

The main change is to add the words of the current file to the original matching string list. Since I am not very familiar with code optimization, there may be areas where this code needs further optimization. For example, each completion requires re-reading the characters of the entire file for segmentation, which may slow down the completion of long documents.

Thanks for sharing. Would you mind opening a PR?

Thanks for sharing. Would you mind opening a PR?

Ok, I have made a PR. But I think my way maybe not so efficient, so you'd better make a toggle command between this and the original version. Thank you for creating this plugin!

It is now supported in v1.3.0 (needs to set option collectWordsFromCurrentFile to true).


cilium bpf ct list global -o json prints map details, not ct entries

$ sudo cilium bpf ct list global | head -n 5
TCP IN [f00d::a0f:0:0:4870]:80 -> [f00d::a0f:0:0:4264]:34274  expires=71101 rx_packets=6 rx_bytes=615 tx_packets=4 tx_bytes=581 flags=0 revnat=0 proxyport=0 src_sec_id=42212
TCP OUT [f00d::a0f:0:0:ad]:80 -> [f00d::a0f:0:0:dc06]:34770  expires=70766 rx_packets=0 rx_bytes=0 tx_packets=3 tx_bytes=282 flags=0 revnat=0 proxyport=0 src_sec_id=42212
ICMPv6 OUT [f00d::a0f:0:0:4870]:32768 -> [f00d::a0f:0:0:1150]:0  expires=71092 rx_packets=10 rx_bytes=1180 tx_packets=10 tx_bytes=1180 flags=0 revnat=0 proxyport=0 src_sec_id=41924
TCP OUT [f00d::a0f:0:0:ecfe]:80 -> [f00d::a0f:0:0:f8a]:48700  expires=70964 rx_packets=4 rx_bytes=581 tx_packets=6 tx_bytes=615 flags=0 revnat=0 proxyport=0 src_sec_id=41924
TCP OUT [f00d::a0f:0:0:ecfe]:80 -> [f00d::a0f:0:0:10da]:57812  expires=70921 rx_packets=0 rx_bytes=0 tx_packets=3 tx_bytes=282 flags=0 revnat=0 proxyport=0 src_sec_id=42212

$ sudo cilium bpf ct list global -o json
{
  "MapType": 1,
  "KeySize": 40,
  "ValueSize": 48,
  "MaxEntries": 1000000,
  "Flags": 0,
  "OwnerProgType": 0,
  "NonPersistent": false
}
{
  "MapType": 1,
  "KeySize": 14,
  "ValueSize": 48,
  "MaxEntries": 1000000,
  "Flags": 0,
  "OwnerProgType": 0,
  "NonPersistent": false
}

This command is intended to print the contents of the map, not the details of the IPv4/IPv6 map structures.

Hi, I'd be willing to do this but I'd need much more words and/or resources to understand where to find the problem/how to solve it

Having errors in getting cilium set up on my machine and am trouble shooting at the moment. Once finished if I run into trouble in implementing the issue I'll check back on here or on the Cilium slack, thanks!

@jMuzsik Are you using the vagrant box? (http://docs.cilium.io/en/v1.0.0-rc4/contributing/#setting-up-a-development-environment) It's by far the easiest way to get a dev environment up and running.

Yes I am but I thought it was because I had issues related to a previous installation of the virtualbox but rather I believe that it may be a problem in the getting-started that needs to be updated:
Its a simple bug, and I'll post it later tonight as an actual issue, as I'm in a bit of a rush:
I took some pics and such to post but the gist of it is this:
In the vagrant file the Vagrant config uses  bento/ubuntu-17.04 but this causes an error b/c this version of ubuntu will fail during the download of docker compose - check out this article.
So basically all instances of  bento/ubuntu-1704 must be replaced with bento/ubuntu-1710

@jMuzsik - the issue with the VM you are running is fixed by https://github.com/cilium/cilium/pull/3020 . However, the VM you are running is not the best VM for compiling / testing Cilium with. Please refer to the using the provided Vagrantfile section.

Hi, sorry but this is impossible for me, it is not as simple as I supposed.
First of all, something like this has to be written within cilium/cmd/bpf_ct_list.go:
	bpfCtList := make(map[string][]string)
	if err := ctMap.CtMap.Dump(bpfCtList); err != nil {
		os.Exit(1)
	}

As, this form of function always happens in other instances of outputting JSON through the command line. Such as in:
bpf_endpoint_list.go or bpf_tunnel_list.go
Then,  the best reference I saw that had a similarity to what needed to be done was within bpf_endpoint_list.go's function:  lxcmap.LXCMap.Dump(bpfEndpointList)
This first creates the specified bpf map then Dumps into the matrix.
As so:
var (
	// LXCMap represents the BPF map for endpoints
	LXCMap = bpf.NewMap(MapName,
		bpf.MapTypeHash,
		int(unsafe.Sizeof(EndpointKey{})),
		int(unsafe.Sizeof(EndpointInfo{})),
		MaxKeys,
		0,
		func(key []byte, value []byte) (bpf.MapKey, bpf.MapValue, error) {
			k, v := EndpointKey{}, EndpointInfo{}

			if err := bpf.ConvertKeyValue(key, value, &k, &v); err != nil {
				return nil, nil, err
			}

			return k, v, nil
		},
	)
)

A function like this does not exist in ctmap.go which I believe must be created. From my experiments (which I am surely incapable of doing in the precise way you guys need) it appears that not only that function, but other functions to fit the type and the requirements of ctMap must be created.
Also, I am quite sure that within pkg/bpf a new file needs to be created to handle the creation of the CtMap.
Sorry and thanks for letting me put my hand into trying to implement this, I learned quite a bit in the process, and if there are any simple problems I'd be happy to give it a try. Thanks!

@jMuzsik thanks for digging in to take a look, it seems that my instructions were underestimating the work required as I didn't realise how different the approach is in CT map vs. other map types. Regardless, thanks for trying it out, I think that your analysis should be useful in future for implementing this functionality. I hope that you were able to learn some things along the way.

FYI https://github.com/cilium/cilium/pull/5480 is refactoring the CT map code to make the CT maps more accessible, so this bug should become much easier to address once that goes in.

Will look into it!


Styling Overhaul Results Page

Description
Results page should reflect figma design
Technical Notes
[Add Technical notes]
Risks
[Add Risks]
Related Issues Or Discussions
[...Issues]

https://www.figma.com/file/SESTgQliRsgV72Q8n4r46n/Zwell-Desktop?type=design&node-id=0-1&mode=design


Wrong behaviour when a page dose not exists

I scanned my website with a security scanner and there are a lot warnings when it try to get the pages from the _next/HASH/page/* url
Expected Behavior
GET /_next/2025e8cd-80d3-454a-bf9f-ddd04aedcd3d/page/admin.aspx
should return the 404 error code with my 404 error page
Current Behavior
GET /_next/2025e8cd-80d3-454a-bf9f-ddd04aedcd3d/page/admin.aspx
I get the 200 status code and this text in the browser
window.__NEXT_REGISTER_PAGE('/admin.aspx', function() {
        var error = new Error('Page does not exist: /admin.aspx')
        error.statusCode = 404

        return { error: error }
      })

FWIW, this is quite problematic when using a cloud platform like Fastly.  During a purge, you could get a stale html doc returned that references the js from a previous immutable build.  If this didn't return a 200, fastly would be able to more accurately deal with the transition to the new deploy.

Fixed in next@canary.


PHP Fatal error:  Uncaught Hitrov\Exception\CurlException: curl error occurred: Could not resolve host: iaas.eu-amsterdam-1-AD-1.oraclecloud.com

Hey Guys,
Can anyone help me out with this error message?
<EMAIL_ADDRESS>sudo php index.php
PHP Fatal error:  Uncaught Hitrov\Exception\CurlException: curl error occurred: Could not resolve host: iaas.eu-amsterdam-1-AD-1.oraclecloud.com, response:  in /home/ubuntu/.local/bin/oci-arm-host-capacity/src/HttpClient.php:31
Stack trace:
#0 /home/ubuntu/.local/bin/oci-arm-host-capacity/src/OciApi.php(257): Hitrov\HttpClient::getResponse()
#1 /home/ubuntu/.local/bin/oci-arm-host-capacity/src/OciApi.php(132): Hitrov\OciApi->call()
#2 /home/ubuntu/.local/bin/oci-arm-host-capacity/index.php(74): Hitrov\OciApi->getInstances()
#3 {main}
thrown in /home/ubuntu/.local/bin/oci-arm-host-capacity/src/HttpClient.php on line 31

"iaas.eu-amsterdam-1-AD-1.oraclecloud.com" cannot be resolved?
Any pointers are much apreciated.
Cheers,
Mitch

I believe the proper region name must be eu-amsterdam-1. and if for example there's only one availability domain within this region, you can completely skip setting OCI_AVAILABILITY_DOMAIN

I believe the proper region name must be eu-amsterdam-1. and if for example there's only one availability domain within this region, you can completely skip setting OCI_AVAILABILITY_DOMAIN

Good morning Hitrov,
Thanks for your reply.
I believe the proper region name must be "eu-amsterdam-1".
yes I got that from the OCI portal:
OCI_REGION=eu-amsterdam-1-AD-1
and if for example there's only one availability domain within this region, you can completely skip setting "OCI_AVAILABILITY_DOMAIN"
Yes again there is only1 availability domain and i skipped it but there are 3 "Fault domains" is that an issue?

This is my redacted .env:
###########################################################################################
OCI_REGION=eu-amsterdam-1-AD-1
OCI_USER_ID=ocid1.user.oc1..aaaaaaaa***********************
OCI_TENANCY_ID=ocid1.tenancy.oc1..aaaaaaaa*************
OCI_KEY_FINGERPRINT=*************************************
absolute path (including directories) or direct public accessible URL
OCI_PRIVATE_KEY_FILENAME="/.pem"
OCI_SUBNET_ID=ocid1.subnet.oc1.eu-amsterdam-1.aaaaaaaaq2****************
OCI_IMAGE_ID=ocid1.image.oc1.eu-amsterdam-1.aaaaaaaa******
Always free ARM: 1,2,3,4. Always free AMD x64: 1
OCI_OCPUS=4
Always free ARM: 6,12,18,24. NB! Oracle Linux Cloud Developer Image requires minimum 8. Always free AMD x64: 1
OCI_MEMORY_IN_GBS=24
Or "VM.Standard.E2.1.Micro" for Always free AMD x64
OCI_SHAPE=VM.Standard.A1.Flex
OCI_MAX_INSTANCES=1
Your public key ~/.ssh/id_rsa.pub contents
NB! No new lines / line endings allowed! Put inside double quotes
OCI_SSH_PUBLIC_KEY="ssh-rsa *************************************"
Is now optional for ARM since Always Free ARMs can be created in any AD.
ListAvailabilityDomains API call will be used for retrieval https://docs.oracle.com/en-us/iaas/api/#/en/identity/20>

NB! Always free AMD x64 instances should be created only in "main" (Always Free Eligible) AD,
you must set it manually in this case!

If you wanna specify more than one, set as a PHP array in OciConfig constructor (inside index.php file)
OCI_AVAILABILITY_DOMAIN=
CACHE_AVAILABILITY_DOMAINS=1
Optional. If not set, default value depends on OCI_IMAGE_ID will be used.
Minimum is 47 for AMD x64 or 50 for ARM, maximum is 200 for Always Free accounts
OCI_BOOT_VOLUME_SIZE_IN_GBS=200
Optional. If set, will be used instead OCI_IMAGE_ID. Cannot be used together with OCI_BOOT_VOLUME_SIZE_IN_GBS.
Open https://cloud.oracle.com/block-storage/boot-volumes, click non-used and copy it's OCID
NB! You must specify OCI_AVAILABILITY_DOMAIN also as they should be located in the same one
OCI_BOOT_VOLUME_ID=
TOO_MANY_REQUESTS_TIME_WAIT=600
###########################################################################################


TASK: Static factory method for aggregate roots

This turns AggregateRoot::reconstituteFromEventStream() into a static
factory method that class the protected constructor.
This way we can ommit the unserialize() trick and ensure consistency.
Resolves: #67

Everytime we add a static method I see some red flags

I'm with you on that. But factory methods are the only good use for static methods IMO ;)


Set up longer timer and notify user when TX will take long to sign.

Moved to https://github.com/BitLox/copay/issues/8


QoL improvements: PRs only modifying markdown files in the repo (such as docs) should still have link verification, cspell, and other CI checks run

Currently, whenever a PR modifies just the markdown files in the repo, the CI leg is stuck at checkenforcer and none of the verification legs that are part of GenerateReleaseArtifacts run. These are important to catch link verification and spelling errors.
Otherwise, errors slip through, causing other PRs to be impacted and builds fail (like in here https://github.com/Azure/azure-sdk-for-cpp/pull/4273, https://github.com/Azure/azure-sdk-for-cpp/issues/4286).
For example, look at this PR which highlights the unwanted behavior:
https://github.com/Azure/azure-sdk-for-cpp/pull/4283

For the appropriate checks and CI pipelines to run, I had to cosmetically modify other files, such as the CMakeLists.txt file. Otherwise, no CI legs run. Only then do we see things like:
https://dev.azure.com/azure-sdk/public/_build/results?buildId=2142299&view=logs&j=a129effc-2dd1-54d1-fb5a-ad7bdc0e851d&t=ccad100f-2904-5913-2083-c525ad929a03

This has the other unfortunate side effect of the PR not being mergeable, since it can't be green.
cc @weshaggard, @danieljurek, @RickWinter, @LarryOsterman

One option to consider is to change sdk\core\ci.yml to add the "doc" tree to the list of paths which should be included. That should cause the core pipeline to be triggered whenever we modify any of our documents.

That's a good idea!
How do we handle and merge doc changes at the root of the repo, like the README.md or the contributing guide?

@ahsonkhan you could add a path trigger for the root of the repo to trigger the core pipeline as well.

Wes, wouldn't the path trigger on the root mean that the core pipeline is triggered on every change to the repo, not just file changes in the root of the repo?

Yes if you do the full path but you can do wildcards based on what I'm reading at https://learn.microsoft.com/en-us/azure/devops/pipelines/repos/github?view=azure-devops&tabs=yaml#paths. Perhaps *.md would work to only trigger on those files.

Partially fixed in https://github.com/Azure/azure-sdk-for-cpp/pull/4296


Text Alignment Solution

Text Alignment HackerRank problem Solution in Python

Hi there!
I am the Mentor of this project .
Although this repo is not part of Hacktoberfest 2024, we have some great alternatives for you. Welcome to SkillShow, a repository focused on skill enhancement!
Guidelines:
Please adhere to the existing guidelines.
No additional rules, thanks!
Check it out here: SkillShow.
Note: Start by creating an issue. Once it’s assigned to you, feel free to open a pull request (PR). Directly opening a PR will lead to an invalid label.
Looking forward to your contributions!

Hey Kushal!
Thank you for the Opportunity to Contribute to SkillShow.
I have made a PR and Raised an Issue regarding the "Text Alignment" Problem
on Hackerrank. Would be grateful if you can look into it and assign the
issue to me / merge the PR.
Thanking you,
Tarun Balaji K S
On Fri, Oct 11, 2024 at 5:01 PM Kushal Das @.***> wrote:

Closed #282
https://github.com/Kushal997-das/Hacktoberfest-2024/issues/282 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/Kushal997-das/Hacktoberfest-2024/issues/282#event-14607288041,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APS6NIPKXK47GH32SI5HH63Z26ZJ7AVCNFSM6AAAAABPRNMO6SVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGYYDOMRYHAYDIMI
.
You are receiving this because you were mentioned.Message ID:
@.***
.com>


Allow name spaced models

I get the following error when using namespaced model Unsakini::User. This is the fix I came up with.
 Failure/Error: authenticate_for Unsakini::User

 NameError:
   `@_current_unsakini/user' is not allowed as an instance variable name

Closing, working on tests. Will add it back once done with tests


Lock not immediately released after reboot

I've been wondering for a while if there's some way to improve the time it takes to release the lock after reboot. At least in my case the nodes often times stay cordoned some time after all the pods that are unaffected by the cordoning have been recreated.
Here's an example lock:
time="2023-07-15T06:24:03Z" level=info msg="Binding node-id command flag to environment variable: KURED_NODE_ID"
time="2023-07-15T06:24:03Z" level=info msg="Kubernetes Reboot Daemon: 1.13.1"
time="2023-07-15T06:24:03Z" level=info msg="Node ID: k3s-server-0a8bc"
time="2023-07-15T06:24:03Z" level=info msg="Lock Annotation: kured/kured:weave.works/kured-node-lock"
time="2023-07-15T06:24:03Z" level=info msg="Lock TTL not set, lock will remain until being released"
time="2023-07-15T06:24:03Z" level=info msg="Lock release delay not set, lock will be released immediately after rebooting"
time="2023-07-15T06:24:03Z" level=info msg="PreferNoSchedule taint: "
time="2023-07-15T06:24:03Z" level=info msg="Blocking Pod Selectors: []"
time="2023-07-15T06:24:03Z" level=info msg="Reboot schedule: SunMonTueWedThuFriSat between 00:00 and 23:59 UTC"
time="2023-07-15T06:24:03Z" level=info msg="Reboot check command: [test -f /var/run/reboot-required] every 5m0s"
time="2023-07-15T06:24:03Z" level=info msg="Reboot command: [/bin/systemctl reboot]"
time="2023-07-15T06:25:32Z" level=info msg="Holding lock"
time="2023-07-15T06:25:32Z" level=info msg="Uncordoning node k3s-server-0a8bc"
time="2023-07-15T06:25:33Z" level=info msg="Releasing lock"

Apparently it takes almost 90 seconds between starting kured and uncordoning the lock, even though no lock release delay has been set.
Is this expected?

Yes, this is expected. There's a random delay after the reboot to avoid random apiserver issues.

Closing for now, please feel free to comment again for further discussions.


Are digital audio FPGA files needed in package deployment

When deploying compiled code with the Digital Audio Toolkit, need to check if package build needs to be updated to include the FPGA (or other) files for the digital audio piece to work.

Bitfiles are stored here: C:\Program Files\National Instruments\Shared\PDM Acquisition Toolkit\Bitfiles
VI that loads the bitfiles:


Drop Python 3.9 support

We should drop Python 3.9 for the upcoming 2025 R1 releases of all PyGranta packages, so we are aligned with SPEC 0 and ADR 21.
This should result in a new minor release, unless a major release is required for any other purposes.

We should do this at the same time as #107


Decorator breaks when the deprecated arg is not in the docstring

Very similar to #5 this line assumes that the deprecated kwarg is in the docstring:
description_start = re.search(f"\n{indent}{arg}\s*:", params_section).end()

...but again the search returns None if it's not found.

fixed in latest commit


Typo in hw_wordentail.ipynb

I wanted to create PR but I do not have create branch permission.

Minor thing, but could improve understanding. In hw_wordentail.ipynb:
"That is, if a word w appears in a training pair, it does not occur in any text pair. "
should read as:
"That is, if a word w appears in a training pair, it does not occur in any test pair. "

Hi @votrubac
If you can create a PR, what you can do is

fork the repo, and clone to your env
create a branch from your local env, and make the change and push to your forked branch
from there you can create a PR.

This is a longer version of how to do it, in case https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/creating-a-pull-request

@votrubac @insop Thanks for this! I appreciate the typo correction. I'll just make the change -- seems easier!


[$500] Desktop - Deeplinks - Concierge chat opens when paste other chat link in web and click "allow"

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Precondition: user should be signed in on desktop and web to the same account

Paste any deeplink conversation with other user on your account
Click "Allow"

Expected Result:
Desktop app should opens and lands on the same conversation as web
Actual Result:
User redirected to the concierge conversation
Workaround:
Unknown
Platforms:

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/c4884770-fa47-4266-a125-4e06698a94ed
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01febb654d868b11de
Upwork Job ID:<PHONE_NUMBER>748978176
Last Price Increase: 2023-11-10

Can't reproduce. Seems to be working when I copy the link from a conversation in my browser to another tab and opening the desktop app. I have included a video of the steps I took.
https://github.com/Expensify/App/assets/48998844/2f483a78-783c-454a-b074-a4c271a3829b

Not able to reproduce on latest main

@kbecciv @narefyev91 the issue is not reproducible as is but while investigating it I see we have a crash when deeplinking to Concierge via /concierge route, in both desktop and web.
Should we update this issue to address that or create another issue? I can post a proposal to fix it, it's a pretty bad crash

I think we need to follow the same process for specify bugs (which we already have) - we need a new one bug ticket - maybe someone already flagged that issue somewhere - just to be sure we will not make duplications.
@tienifr you can sync with @kbecciv and check if we do not have already any equal bug tickets. Thanks!

@tienifr Can you please share reproduction steps and video for the crash?

Ok, sounds like this is no longer an issue. Comment if I'm wrong!


Apple Maps not able to find Share Extension

Describe the bug
I am trying to share from apple maps to my main application through the share extension, I even went into the ShareExtension-Info.plist and added an NSExtensionActivationRule, that should allow the data type, however the extension is still not showing up and I cannot figure out why.
 <key>NSExtension</key>
    <dict>
      <key>NSExtensionAttributes</key>
      <dict>
        <key>NSExtensionActivationRule</key>
        <dict>
          <key>NSExtensionActivationSupportsWebURLWithMaxCount</key>
          <integer>1</integer>
          <key>NSExtensionActivationSupportsWebPageWithMaxCount</key>
          <integer>1</integer>
          <key>NSExtensionActivationSupportsImageWithMaxCount</key>
          <integer>1</integer>
          <key>NSExtensionActivationSupportsMovieWithMaxCount</key>
          <integer>1</integer>
          <key>NSExtensionActivationRule</key>
          <string>SUBQUERY(extensionItems, $extensionItem, SUBQUERY($extensionItem.attachments, $attachment, ANY $attachment.registeredTypeIdentifiers UTI-CONFORMS-TO "public.url").@count >= 1).@count == 1</string>
        </dict>

expo version (ex: 49) : 50
using firebase : no
using static build : no
routing : expo-router
package version (ex: 1.2.0) : 1.5.2
platform target : iOS
device : simulator and real device
type: local build and expo build

Sharing for apple maps it's not supported, i mean i didn't test it but i think it should need some ajustement in shareextension to handle this data type.
However the extension should be showing with the good activation rule, can you try to add it in the plugin configuration :
"expo-share-intent",
{
    "iosActivationRules": {
        "NSExtensionActivationRule": "SUBQUERY(extensionItems,$extensionItem,SUBQUERY($extensionItem.attachments,$attachment,SUBQUERY($attachment.registeredTypeIdentifiers, $uti, $uti UTI-CONFORMS-TO \"public.url\" AND NOT $uti UTI-CONFORMS-TO \"public.file-url\").@count >= 1).@count >= 1).@count >= 1"
    },
}

totally resolved, I changed the attributes to the following and it worked well! Thank you :)
   <key>NSExtension</key>
    <dict>
      <key>NSExtensionAttributes</key>
      <dict>
        <key>NSExtensionActivationRule</key>
        <string>SUBQUERY(extensionItems, $extensionItem, SUBQUERY($extensionItem.attachments, $attachment, ANY $attachment.registeredTypeIdentifiers UTI-CONFORMS-TO "public.url").@count >= 1).@count == 1</string>
      </dict>
      <key>NSExtensionMainStoryboard</key>
      <string>MainInterface</string>
      <key>NSExtensionPointIdentifier</key>
      <string>com.apple.share-services</string>
    </dict>

Thank you for being a super start btw! @achorein

Just a heads up if you are going to place the rule within the app.json, it needs to be this:
"expo-share-intent",
{
"iosActivationRules":  "SUBQUERY(extensionItems,$extensionItem,SUBQUERY($extensionItem.attachments,$attachment,SUBQUERY($attachment.registeredTypeIdentifiers, $uti, $uti UTI-CONFORMS-TO "public.vcard" ).@count >= 1).@count >= 1).@count >= 1"
}
Otherwise you get a nested NSExtensionActivationRule
Why Apple can't just have a pre-set for VCARD I have no idea - but hope this helps people coming to this


Add extractor for batterydata.energy.gov data

Initial support support for files from batterydata.energy.gov.
To do list:

[x] Find a file example I can host on GitHub (or download as needed) for testing
[x] Build a converter for "summary" data
[ ] Test that the content differs with store_all or not
[x] Cross-reference column documentation with NREL's codebase (@pauljgasper, need help here), update our definitions accordingly
[ ] Screen out the EIS data, which we are not yet storing

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

-10 of 72 (86.11%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.9%) to 78.148%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

batdata/extractors/batterydata.py
59
69
85.51%

Totals

Change from base Build<PHONE_NUMBER>:
0.9%

Covered Lines:
540

Relevant Lines:
691

💛  - Coveralls


Debian/Ubuntu .deb package issue: 'Problem parsing dependency Depends'

Running sudo apt-get update today (coinciding with the v1.0.14-0 release presents me with the following error:

Reading package lists... Error!
E: Problem parsing dependency Depends
E: Error occurred while processing keybase (NewVersion2)
E: Problem with MergeList /var/lib/apt/lists/dist.keybase.io_linux_deb_repo_dists_stable_main_binary-amd64_Packages
E: The package lists or status file could not be parsed or opened.

dist.keybase.io_linux_deb_repo_dists_stable_main_binary-amd64_Packages reads as follows:

Package: keybase
Version: 1.0.14-0
Architecture: amd64
Installed-Size: 33876
Maintainer: Keybase Bugs<EMAIL_ADDRESS>Depends:
Priority: optional
Section: misc
Filename: pool/main/k/keybase/keybase_1.0.14-0_amd64.deb
Size: 5639838
SHA256: f58114299f06a8ddda009f8cb1b51f9d925c7eed40d0df2fee899352bfcce2af
SHA1: 326f8a901350fa3998c8f0cda34ab81abb45be85
MD5sum: 5a5748b57e9f4157e67863fa8d02ebe7
Description: Keybase's command line client

I suspect the empty Depends: line is causing the issue.
Perhaps related to f27b600ec78b78ad69f5020f81bf514b04949fff?

I'm also affected by this issue.

Same problem, messes with the munin monitoring that checks every hour for new updates of the system and fails because of this issue, resulting in an alert email per hour.
I have commented out the keybase repository in /etc/apt/sources.list.d/keybase.list in the meantime...

Thanks everyone, we'll fix this soon.
@oconnor663 Dependencies is interpolated in all modes, but only set in the prerelease mode.

This should be fixed now with version 1.0.14-1. @brbsix and others, can you confirm?

Confirmed it's fixed in the latest revision. Thanks


The demo requires Depth camera with Color sensor

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ R200 / F200 / SR300 / ZR300 / D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
{ legacy / 2.. }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
<Describe your issue / question / feature request / etc..>

The demo requires Depth camera with Color sensor

Hi @rongyaoxiaohuihui  Which demo program are you referring to, please?
If it is the align_depth2color.py one that has an RGB sensor check, this program will not work with camera models that do not have an RGB sensor.  With the D405 camera model, it has RGB but does not have a separate RGB sensor as RGB comes from the depth sensor.  align_depth2color.py can work with the D405 model if the RGB sensor check is removed from the script.  A RealSense user at https://github.com/IntelRealSense/librealsense/issues/11329 has a version of the script that has this removal.


Change from react-app-rewired to CRACO

React App Rewired is not being updated anymore, but Create React App is.
This is the reason why CRACO  was developed and I think for the future the checkout would benefit from using this:
https://medium.com/gsoft-tech/why-i-built-craco-33ff39f4fc94

Maybe we could even look towards something like ViteJS. This will solve problems like PostCSS8 being not supported by CRA.
I already did the intergration and it's very easy. There is just one issue left with the dev enviroment combined with Preact, it now only works with React.
What do you think about this @progammer-rkt, is this something to consider?

@jesse-deboer I will checkout these packages soon. We will be going with React App Rewired for the major release. But, surely we can reconsider it.

Sounds good to me, let's go for V1 and look to this after the release! 😄
I did some work already, got a few small issues, but will fix them later. This also makes jit mode and PostCSS8 possible without the need for an extra npm command, and it's superrrrrrr quick (just milliseconds for the rebuild)
https://github.com/jesse-deboer/magento2-hyva-checkout/commit/ea1ca5d9250d3348e985c8ffc297ca496d6e71ab

So instead of using CRACO I used ViteJS as a better toolkit compared to Create React App and React App Rewired.
I think the basic from which ViteJS starts is way better compared to CRA because it also doesn't need something like CRACO to extend it and has a lot of other benefits like HMR build in + PostCSS 8 support.
Here's the PR I made to change to ViteJS: https://github.com/hyva-themes/magento2-hyva-checkout/pull/189

Closing this issue as we consider ViteJS in the next major version of Hyva checkout 2.x

Craco is also not being updated anymore. It's getting discountinued too


Mean key -> most common key

Start Date
No response
Implementation PR
No response
Reference Issues
No response
Summary
In the Playlists details, there is the "Mean key" but it doesn't make any sense to get an average out of the key.
What would be more logical is to have the most common key, so like the genres or the artists, get the number of each key, and then return only the most common one.
Basic Example
Mean key : 5.13
Could be now Most common key : F
More information here : https://developer.spotify.com/documentation/web-api/reference/get-several-audio-features
And here : https://en.wikipedia.org/wiki/Pitch_class
Drawbacks
Allow to really know the most used key in the playlists.
Unresolved questions
No response

I'll like to work on this.
but I am stuck on loading and not able to get details from spotify API

I have sent the mail to<EMAIL_ADDRESS>regarding authorization

Thanks for you will to contribute to our project ! Eveything should be fine since you are authorized now

Should I replace Mean key card with Most common Key
something like this?

or add a new card instead?
Also to get most common key I'm storing the keys from each track in an array (since the keys would be in range -1 to 11 ) and then finding index of maximum value in that array
like this
 return_value.keys=mapKeyToPitch(return_value.key_array.indexOf(Math.max(...return_value.key_array)));
In case of 2 or more keys with same number of occurrences in playlist should show only one key or all the keys with max occurrence?

Please review PR #64 and let me know if any changes need to be done


Unexpected Closing Parentheses and Braces When Executing Code Chunks in R

Describe the bug
When executing certain chunks of R code using VSCode with the R extension and Radian, I sometimes observe additional closing parentheses or braces being sent to the console, which were not present in the original code. This issue seems to occur after a small delay.
To Reproduce
This issue occurs sporadically and I have been unable to identify a consistent pattern. However, I've noticed that it happens frequently when I run the following chunks of R code multiple times:
data_test <- reduce(rep(list(mtcars), 2), bind_rows)

Can you fix this issue by yourself? (We appreciate the help)
No
(If applicable) Please attach setting.json
{
    "[r]": {
        "editor.defaultFormatter": "REditorSupport.r",
        "editor.formatOnSave": false,
        "editor.wordSeparators": "`~!@#%$^&*()-=+[{]}\\|;:'\",<>/?"
    },
    "[rmd]": {
        "editor.defaultFormatter": "REditorSupport.r",
        "editor.formatOnSave": false
    },
    "r.rmarkdown.knit.openOutputFile": true,
    "r.rmarkdown.codeLensCommands": [
        "r.runCurrentChunk",
        "r.runAboveChunks",
        "r.runBelowChunks"
    ],
    "r.source.encoding": "UTF-8",
    "r.source.focus": "editor",
    "r.alwaysUseActiveTerminal": false,
    "r.sessionWatcher": true,
    "r.rtermSendDelay": 8,
    "r.plot.useHttpgd": true,
    "r.lsp.debug": true,
    "r.rterm.windows": "C:\\Users\\brian\\AppData\\Local\\Programs\\Python\\Python311\\Scripts\\radian.exe",
    "r.liveShare.defaults.commandForward": true,
    "r.rterm.linux": "/usr/bin/radian",
    "r.liveShare.defaults.shareBrowser": true,
    "r.bracketedPaste": true,
    "r.session.levelOfObjectDetail": "Normal"
}

Expected behavior
I expect the code to be sent to the console as it is, without any additional characters.
Screenshots

Environment (please complete the following information):

OS: Windows 11
R Version: 4.3.0
VSCode Version: 1.79.2
Radian Version: 0.6.4

Additional context
I noticed that when I format the code differently, the problem does not occur. For instance, the following code does not produce the issue:
data_test <- reduce(
  rep(list(mtcars), 2),
  bind_rows
)

In addition, it also does not play when instead of using bind_rows, I use rbind :
data_test <- reduce(rep(list(mtcars), 2), rbind)

I have been aware of this issue for the past two years and have been searching for a solution, as having this problem renders Radian completely unusable. However, I have been unable to find a solution thus far.

Well, this seems to be an old and already known problem: https://github.com/REditorSupport/vscode-R/issues/419, https://github.com/REditorSupport/vscode-R/issues/820, https://github.com/randy3k/radian/issues/303, https://github.com/randy3k/radian/issues/277, https://github.com/randy3k/radian/issues/230 . And it is only happening on windows, because Bracketed paste mode is not supported.
The first one associated with the addition of parentheses or braces when sending a code to the console. It seems as if there is a brief desynchronization when sending the code to the console, and radian tries to close parenthesis before seeing that it was already closed. This can be easily fixed by disabling auto-completion in the .Rprofile or .radian_profile file.
options(radian.auto_match = FALSE)

The second is a bit more problematic and is apparently occurring when the code block is very large, or when a chunk is executed that has several code blocks and these have to be sent all at once to the console. There is a very temporary workaround proposed here that reduces the number of times the problem occurs, although it still happens: https://github.com/REditorSupport/vscode-R/issues/820#issuecomment-1002937100

I remembered that someone told me this before, but I don't know where ".radian_profile" is. Can you tell me how to find ".radian_profile", thanks.

@brianmsm There's also, in my experience at least, two new showstopping bugs that started recently: Radian eventually stops printing any output at all, and prior to that it will begin freezing for tens of seconds or more at a time in the middle of executing even a single line of code.
What's interesting to me is all of these feel like regressions. When I first began using VSCode-R with Radian the only issue I had consistently was the buildup of background processes from the language server, and only something as huge as an entire shiny app would fail to auto-execute on ctrl+enter. Both VSCode-R and Radian have steadily gotten less reliable and performant with time.
For example starting an R session with the built in command palette function on an SSH remote is unusable, but calling R at the terminal to create an R session works great.

There's also, in my experience at least, two new showstopping bugs that started recently: Radian eventually stops printing any output at all, and prior to that it will begin freezing for tens of seconds or more at a time in the middle of executing even a single line of code.

@D3SL Can you reproduce this on my reference deployment at https://demo.jupyter.b-data.ch?

OS: Debian 12
Code Version: 1.79.2
Radian Version: 0.6.6

r executable: /usr/local/lib/R/bin/R
r version: 4.3.1
python executable: /usr/local/bin/python
python version: 3.11.4

ℹ These seem to be Windows specific issues.

New to VSCode-R, and love the idea of it more than the experience at this point. These bugs - persisting even after trying solutions - make Radian completely unusable. Hard to believe these problems have persisted for years apparently.

Unbelievable that this hasn't been fixed or addressed.


Set up CI to audit for public dependency types

We want strict control over what types from dependencies we expose, and there is a tool that can help us check that in CI: cargo api-linter.
We should start by adding a new CI job to .github/workflows/CI.yml to use it. Any questions about what goes in the config file can be clarified here or in a PR.

Looks like that has been refactored into cargo-check-external-types. I'll pick up this up.

I ran this tool against the master branch (982e6a51385).


Add ability to write tests inside of a  block inside of SFCs

Feature Description
I would like the ability to write unit tests inside of a custom block <test> inside of the Vue file that is under test.
Problem
Having a file and subsequent test file creates noise in the project directory. Changing a file in a way that changes the tests means there are two files to update, rather than possibly having all of the changes in one file, which makes for noisy commits.
Expected behaviour
Using the example found here https://vue-test-utils.vuejs.org/guides/#getting-started, you could do the following:
// MessageComponent.vue
<template>
<p>{{msg}}</p>
</template>

<script setup lang='ts'>
defineProps<{msg: string}>()
</script>

<test>
// Import the `mount()` method from Vue Test Utils
import { mount } from '@vue/test-utils'
import MessageComponent from<EMAIL_ADDRESS>// reimport?

test('displays message', () => {
  // mount() returns a wrapped Vue component we can interact with
  const wrapper = mount(MessageComponent, {
    propsData: {
      msg: 'Hello world'
    }
  })

  // Assert the rendered text of the component
  expect(wrapper.text()).toContain('Hello world')
})
</test>

I'm not sure exactly what the interaction would be like, such as when these blocks would run. I think it'd be nice to have them run in watch mode if the project is running in development mode and then run all of the blocks during a build.
Here are a couple examples of other Vue related projects utilizing custom blocks:
https://kazupon.github.io/vue-i18n/guide/sfc.html#basic-usage
https://github.com/ktsn/vue-route-generator#route-custom-block
I think this is possible using https://vue-loader.vuejs.org/guide/custom-blocks.html#custom-blocks and/or https://vitejs.dev/guide/migration.html#custom-blocks-transforms, but that starts to get into things that are beyond my expertise.
As an example, Rust puts tests alongside the code: https://doc.rust-lang.org/book/ch11-03-test-organization.html.
Alternatives
One alternative to implementing this as a part of vue-test-utils would be to extend it somehow as a part of a separate project. Another alternative solution could be to continue making separate test files and using the library as it is intended now.

I like the idea. It reminds me of unit tests in Rust.  Another advantage could be that tests don't feel like an extra layer to add on top, but more like an integral part of software development and its workflow.

not that vue isn't already leading in the dx department but this would really make it standout of the rest.


Investigate and track the development of each species

We need to set up a system for this, and I'm trying to figure out what the easiest way is.
We could just make a table or super long list of each species and the standards (#2) that it does or doesn't meet, but I wonder if there's better ways to go about this, like using the project boards (but then would we make a board for every species? Or stick all the cards on one board? That's a lot of boards/cards!). Maybe sticking a checklist file into each species folder and going from that would suffice, but it seems a pity to not use the automation functions that are there.
I'm still learning my way around github so if there's anyone more experienced who has organization suggestions,  I would appreciate it.

[x] Apparently task lists
[ ] might also be an option?

Additionally, until we get a formal system in place, you can feel free to comment on what's already been investigated/finished here, just for the sake of keeping track of things.

The Purbana, Pinky Plant and Bluebell have been updated with push, pull, and hit scripts for the proper stimuli (#84).


dbeaver/pro#2716 [ERD] Ortho layout should be reorganized

New composition applied for graph

verified


More setup bugfixes

Added the missing .idea folder on gitignores
Docker compose was complaining about a duplicated volume so i removed it
2 packages were missing from package.json (reflect-metadata and eslint-config-airbnb-typescript) so i added then
Had to downgrade the NodeJS version until ts-node-dev fixes the bug with fs.watch recursive

Greate job :scotland:
i'll probably clean up the server_develop repository today,
it's really messy right now since i was trying things out yesterday.


bin: make rustfmt happy.

@ngaut @BusyJay @disksing @zhangjinpeng1987

It may conflict with #777 , because I did the same thing.


Slim\Http\Uri has no way to disable authorization

When you have an Uri that has a user and password set, you can not get a Uri without the auth info.
I would assume this would work:
$uri = Slim\Http\Uri::createFromString('https://user:pass@host.com/path');
echo (string) $uri . "\n";
#<EMAIL_ADDRESS>
$uri = $uri->withUserInfo('','');
echo (string) $uri . "\n";
#<EMAIL_ADDRESS>
As you can see, the password is not unset and an incorrect URL is constructed.

@akrabat  ... I honestly can't tell if this is a problem in the PSR-7 spec... or not.
@splitbrain I'll look at other PSR-7 implementations and see what they do; this is very likely a huge edge case that no-one thought of.

The Interface comment says:

Password is optional, but the user information MUST include the user; an empty string for the user is equivalent to removing user information

I would interpret this as removing all user information including the password, when an empty user is passed.
Some info on the background: My application runs behind mod_kerberos which sets PHP_AUTH_USER and PHP_AUTH_PW which are used in createFromEnvironment. I my application I then need to pass the baseUrl to my Twig template handler, however getBaseUrl now returns urls with the password in clear text which are then added to all my links. So I need  a way to pass a baseUrl without the credentials. I worked around this by doing a parse_url myself, but it would be nicer to simply unset the authentication on the Uri object.

yeah looks like a 🐛  👍

I just noticed that this seems to be fixed by #2332

Yeah, this one was an issue I fixed for some time ago. Reverting back to 3.8 also fixes the issue.

#2332 is now merged to 3.x


Eat UTF-8 BOM from scene file

Resolves https://github.com/tangrams/tangram-es/issues/1265
This will be handled within yaml-cpp later.

Is this something I should fix in the style?

@nvkelso No. You could modify the file to fix this, but it's a difference that's not even visible in a text editor so it's unreasonable to try to maintain.


sql: virtual table changes for implicit column definitions

PR: https://github.com/cockroachdb/cockroach/pull/58749
From release notes:

Added a new implicit column to crdb_internal.index_columns, which signifies whether the column is implicitly added onto the index through PARTITION BY with an implicit column or a hash-sharded index. [#58749][#58749] {% comment %}doc{% endcomment %}

Related to https://cockroachlabs.atlassian.net/browse/CRDB-1523.
Related to https://cockroachlabs.atlassian.net/browse/CRDB-1504.

I don't think we need to document this.


Unauthrole and Authrole

I unfortunately decided to delete the unauth role and auth role from the console. This was a bad idea, as I could no longer push to amplify. I recreated the roles, which now allows me to push, but is there any way to restore the settings of these roles?

Sorry, I don't think this was the proper place to ask a question like this. Closing.


Backup & restore settings & local database if a user installs the app on a new phone

Low-priority but nice to have.

Tested that already works properly in Android 6.0+: https://developer.android.com/guide/topics/data/testingbackup.html. So a key/value B&R would be nice to have in older versions, but that could be annoying with the database having so much data. Since this works correctly in new Android, let's consider it very low priority.
"Back up data in a database. If you have an SQLite database that you want to restore when the user re-installs your application, you need to build a custom BackupAgent that reads the appropriate data during a backup operation, then create your table and insert the data during a restore operation."
-https://developer.android.com/guide/topics/data/keyvaluebackup.html


Write a TypeScript program to find surface area of a cuboid

Description
Write a TypeScript program to find surface area of a cuboid
Surface area of cuboid = 2 × (Length × Width + Length × Height + Width × Height)
Length : 2
Width  : 3
Height : 4

Surface area : 52

How to contribute

Save the solution in program/FindSurfaceAreaOfACuboid.ts file
Add FindSurfaceAreaOfACuboid.ts file in find-surface-area-of-a-cuboid folder

!assign


Composing with refl gives different path

In the context of the interval, I found the following.
fromUnitK' : (a : I) -> Id I (fromUnit (toUnit a)) a = split
  zero -> <i> zero
  one -> <i> seg {I} @ i
  -- the usual case here is:
  -- seg @ i -> <j> (seg {I} @ i /\ j)
  -- but try p (which works) vs. p' (which doesn't):
  seg @ i -> p'
    where
    p : Id I zero (seg {I} @ i) = <j> (seg {I} @ i /\ j)
    p' : Id I zero (seg{I} @ i) =
      compId I zero (seg{I} @ i) (seg{I} @ i)
        p
        (refl I (seg{I} @ i))

Type checking failed: Faces in branch for "seg" don't match:
got
[ (i = 0) -> <!0> hComp I zero [ (!0 = 0) -> <!1> zero, (!0 = 1) -> <!1> zero ], (i = 1) -> <!0> hComp I (seg {I} @ !0) [ (!0 = 0) -> <!1> zero, (!0 = 1) -> <!1> one ] ]
but expected
[ (i = 0) -> <!0> zero, (i = 1) -> <!0> seg {I} @ !0 ]

I should look more at the example, but in general
we cannot expect that a path p composing with refl  is
the same as p. (We cannot ensure this for composition in the universe.)
Given this, I am not sure if we should expect fromUnitK’
to be well-typed?

On 11 Feb 2016, at 13:30, Auke Booij<EMAIL_ADDRESS>wrote:
In the context of the interval, I found the following.
fromUnitK' : (a : I) -> Id I (fromUnit (toUnit a)) a = split
zero ->  zero
one ->  seg {I} @ i
-- the usual case here is:
-- seg @ i ->  (seg {I} @ i /\ j)
-- but try p (which works) vs. p' (which doesn't):
seg @ i -> p'
where
p : Id I zero (seg {I} @ i) =  (seg {I} @ i /\ j)
p' : Id I zero (seg{I} @ i) =
compId I zero (seg{I} @ i) (seg{I} @ i)
p
(refl I (seg{I} @ i))
Type checking failed: Faces in branch for "seg" don't match:
got
[ (i = 0) -> <!0> hComp I zero [ (!0 = 0) -> <!1> zero, (!0 = 1) -> <!1> zero ], (i = 1) -> <!0> hComp I (seg {I} @ !0) [ (!0 = 0) -> <!1> zero, (!0 = 1) -> <!1> one ] ]
but expected
[ (i = 0) -> <!0> zero, (i = 1) -> <!0> seg {I} @ !0 ]
—
Reply to this email directly or view it on GitHub https://github.com/mortberg/cubicaltt/issues/28.

Okay, after some more discussion and thinking, I am now convinced this is not a bug (or at least none to do specifically with HITs, but just a property of how identity paths in cubicaltt behave in general). Feel free to close this "bug".


Python 2.7.x: AttributeError: 'module' object has no attribute 'version'

Setup tools install pre-release version of MarkupSafe for Python 2.7 which cause the following AttributeError exception.
Writing /var/folders/jm/nszzwsfd7tb76tv9b58rx6980000gn/T/easy_install-5rvlNb/MarkupSafe-2.0.0a1/setup.cfg
Running MarkupSafe-2.0.0a1/setup.py -q bdist_egg --dist-dir /var/folders/jm/nszzwsfd7tb76tv9b58rx6980000gn/T/easy_install-5rvlNb/MarkupSafe-2.0.0a1/egg-dist-tmp-lH_2sa
Traceback (most recent call last):
  File "setup.py", line 46, in <module>
.
.
.
.
  File "/Users/praveenpremaratne/.config/system-config/.envs/toolkit/lib/python2.7/site-packages/setuptools/config.py", line 515, in _parse_version
    version = self._parse_attr(value, self.package_dir)
  File "/Users/praveenpremaratne/.config/system-config/.envs/toolkit/lib/python2.7/site-packages/setuptools/config.py", line 350, in _parse_attr
    value = getattr(module, attr_name)
AttributeError: 'module' object has no attribute '__version__'

MarkupSafe version is NOT defined in the setup.py.
ps: I'm not sure whose end is the error and why Setuptools trying to install a pre-release by default.

Os: macOS Mojave (10.14.6)
Python: 2.7.15
Setuptools: 39.2.0

By the way, this happens on regardless of which version of Setuptools I use.

are you using easy_install by any chance? that's deprecated since a long time...

No, I'm using pip.
pip                           20.0.2

I'm using Virtualenv tho.

I can't reproduce this issue. I created a fresh Python 2.7 virtualenv with the given versions of pip and setuptools, and pip installed MarkupSafe 1.1.1 as expected. Even the --pre flag still returns 1.1.1 for Python 2.7.
$ virtualenv -p 2.7 test
$ . ./test/bin/activate
$ pip install pip==20.0.2 setuptools==39.2.0
$ pip install MarkupSafe
DEPRECATION: Python 2.7 reached the end of its life...
Collecting markupsafe
  Downloading MarkupSafe-1.1.1-cp27-cp27mu-manylinux1_x86_64.whl (24 kB)
Installing collected packages: markupsafe
Successfully installed markupsafe-1.1.1

It seems there's something else wrong, perhaps with the env you've created, or the way you're installing the package. This doesn't affect my result, but for some reason you've got the latest pip but a two-year-old version of setuptools. If you can't diagnose the issue further, the best course of action is probably to pin your dependency versions, like MarkupSafe<2.

With pip the installation as expected just like yours. But with Setuptools it doesn't.
I've put together a repo that can reproduce the issue on my end, could you check if this reproduces the issue on your end as well?
https://github.com/praveenprem/markupsafe-issue-119
ps: I've used an older version of Setuptools hoping it will force packages to Python2. But even with the latest version, I get the same outcome.

You said you're using pip, not setuptools. Setuptools does not understand modern installation metadata, only use pip to install packages.


Java Indexer: add test for the Plugin framework [phab:D1869]

Created by schroederc at 2017-08-14 18:45:31:

shahms commented inline (context lost during migration) at 2017-08-14 19:26:02:
com_google_auto_service

shahms commented inline (context lost during migration) at 2017-08-14 19:26:02:
com_google_auto_common

schroederc wrote at 2017-08-14 19:55:53:
Namespacing

shahms accepted this revision at 2017-08-14 20:13:33

schroederc closed via commit aafc6073bbe6cad19bf1a769670d4e6b0fe375f3 at 2017-08-14 20:17:08


Support nested options

Does the current syntax make sense? #2

We can split nested props by dot: one.two: 1

@ai +1

@ai, that looks really clever, but splitting properties with a dot . is not very CSS, IMHO.

@jonathantneal also siggested
one: {
  two: 2
}


Weekly Meeting Schedule

when2meet link

Update: https://doodle.com/meeting/participate/id/en5ZGw4d


feat: Add git as a tool to the image to support git dependencies

fixes: #41
Curious what it does with image size, but it doesn't seem like a bad idea to add it.

This won't fix the issue as the pixi binary is installed into the target container

It will also add some additional complexity since there are different package managers in different containers.


Typo in README.md

I've noticed a minor, but common typo in the word length in the section with examples of overriding DJANGO_REST_PASSWORDRESET_TOKEN_CONFIG:

You can configure the length as follows:

DJANGO_REST_PASSWORDRESET_TOKEN_CONFIG = {
    "CLASS": "django_rest_passwordreset.tokens.RandomStringTokenGenerator",
    "OPTIONS": {
        "min_length": 20,
        "max_lenght": 30
       # ________^
    }
}

It is mistyped as lenght instead of length.

facepalm
Thank you, I'll try to fix this asap (alternatively PR welcome :D)

@anx-abruckner I'll submit a PR in a moment. :)

Thank you!


Example problem crashing due to un-picklable objects

I've been trying to run the a2dr example program from the readme but it doesn't seem to work on my computer (MacOS with Python 3.9). The error message I get is as follows:
Traceback (most recent call last):
  File "test2.py", line 22, in <module>
    drs_result = a2dr(prox_list, A_list, b, anderson=False)
  File "/usr/local/lib/python3.8/site-packages/a2dr/solver.py", line 261, in a2dr
    procs[-1].start()
  File<EMAIL_ADDRESS>line 121, in start
    self._popen = self._Popen(self)
  File<EMAIL_ADDRESS>line 224, in _Popen
    return _default_context.get_context().Process._Popen(process_obj)
  File<EMAIL_ADDRESS>line 284, in _Popen
    return Popen(process_obj)
  File<EMAIL_ADDRESS>line 32, in __init__
    super().__init__(process_obj)
  File<EMAIL_ADDRESS>line 19, in __init__
    self._launch(process_obj)
  File<EMAIL_ADDRESS>line 47, in _launch
    reduction.dump(process_obj, fp)
  File<EMAIL_ADDRESS>line 60, in dump
    ForkingPickler(file, protocol).dump(obj)
AttributeError: Can't pickle local object 'precondition.<locals>.proto.<locals>.<lambda>'

I have tried running the program on previous versions of Python, and found that it works on Python 3.7 but fails in a similar fashion on Python 3.8. The Python changelogs for version 3.8 note the following update to the inbuilt multiprocessing library, which I think is the cause of the problem: "On macOS, the spawn start method is now used by default.". The relevant discussion about this update is here, and they mention there that there is potential loss of functionality when dealing with non-pickable objects when switching from fork to spawn for starting new processes.
From the error message, it seems as though the precondition function in a2dr/a2dr/precondition.py might be the culprit. Indeed, just removing the lambda function from the readme example isn't enough to fix the error. Is there a way to implement precondition without lambda functions to see if that fixes things? I suppose an alternative would be switching from spawning back to forking, though I don't know what the side effects of that would be (there's some talk on that in the previously linked discussion).

I managed to work around the issue by starting the script/notebook with the following fragment :
import multiprocessing
multiprocessing.set_start_method('fork')

Thanks a lot Luke for sharing this! Sorry it has been extremely busy at Amazon in the past two months and so I haven't got time to resolve this issue. We will test your solution and update the code as soon as possible. Thanks again!


Native review for trotha01/elm-google-analytics

https://github.com/trotha01/elm-google-analytics
I understand that Native reviews are on hold and waiting on 0.17.0
The reason I'm posting is I believe the original reason for the reviews was to see the different use cases for using Native, so this is just to notify of one use case.
This library is a wrapper around the google analytics js function. The js function is small, ~10 lines of code, but I don't believe the functionality it does can be written in Elm currently.
It adds a script tag with src 'https://www.google-analytics.com/analytics.js' to the dom (which can be done in elm), but it also modifies the window object (which can't be done in elm) for analytics.js to use.
The original javascript function can be found at the top of the page here:
https://developers.google.com/analytics/devguides/collection/analyticsjs/#the_javascript_tracking_snippet

I am moving towards the plan outlined here and discussed a bit further in the 0.17 announcement.
So I get that this is simple, but I don't think this is the right direction for Elm.

Thanks, I'll check out those links and update

@trotha01 can you post updates as you go? This is a snippet I'd use on any application, so I'm willing to help out what's needed to make this smooth.

Hi @domenkozar, I would love help with this. Right now I'm trying to figure out how to structure it.
Evan seems to want to move towards a port module instead of using native, but from what I understand (and from the docs) ports are not allowed in libraries. Even if they were allowed in libraries, elm-reactor doesn't support ports (as far as I can tell). Additionally, to compile an elm module with ports, we need to use --output something.js. So all users of this library wouldn't be able to use elm-reactor, and would need to compile with the --output flag as well.
If you have ideas on how to structure this to make a nice user experience, let me know. I may try reaching out to the community in the slack channel as well. They may have ideas there.

@evancz Since I can't use Native, and since I can't use ports in libraries, I am not sure how to move forward. How do you recommend making elm wrappers for js libraries?
Use cases include:

This google analytics snippet
An Elm library for webaudio
An Elm library for any of the js apis

Elm seems to severely limit libraries by not allowing use of native js web functionality. Am I missing something here?

After a discussion in the slack #native room, it sounds like native is the only way to use javascript in a library. I made updated the code and continued the thread here:
https://github.com/elm-lang/package.elm-lang.org/issues/176


Add hyperlinks and/or diagrams to TSV tables

The TSV tables display well in local tools and also in Github. It would be very useful when checking data to have links to WWW or other references. For example I would like structure diagrams for chemistry so we can easily triage false positives.
There are methods for adding hyperlinks and images but they may not be portable across platforms.
May be best to download all the images into a separate directory with a file for each image.

Sir,
2D images and 3D images for chemical structure diagram.


ernie-3.0-nano-zh 和 ernie-3.0-micro-zh是不是没有发布？

AutoModelForTokenClassification.from_pretrained(pretrained_model_name_or_path="ernie-3.0-nano-zh", num_classes=2)
[2022-07-12 20:11:05,019] [    INFO] - Downloading model_config.json from https://bj.bcebos.com/paddlenlp/models/community/ernie-3.0-nano-zh/model_config.json
[2022-07-12 20:11:05,173] [   ERROR] - Downloading from https://bj.bcebos.com/paddlenlp/models/community/ernie-3.0-nano-zh/model_config.json failed with code 404!
Traceback (most recent call last):
File "/home/liuts/.conda/envs/liuts/lib/python3.7/site-packages/paddlenlp/transformers/auto/modeling.py", line 257, in _from_pretrained
default_root)
File "/home/liuts/.conda/envs/liuts/lib/python3.7/site-packages/paddlenlp/utils/downloader.py", line 162, in get_path_from_url
fullpath = _download(url, root_dir, md5sum)
File "/home/liuts/.conda/envs/liuts/lib/python3.7/site-packages/paddlenlp/utils/downloader.py", line 199, in _download
"{}!".format(url, req.status_code))
RuntimeError: Downloading from https://bj.bcebos.com/paddlenlp/models/community/ernie-3.0-nano-zh/model_config.json failed with code 404!

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "/home/liuts/.conda/envs/liuts/lib/python3.7/site-packages/paddlenlp/transformers/auto/modeling.py", line 527, in from_pretrained
**kwargs)
File "/home/liuts/.conda/envs/liuts/lib/python3.7/site-packages/paddlenlp/transformers/auto/modeling.py", line 261, in _from_pretrained
f"Can't load weights for '{pretrained_model_name_or_path}'.\n"
RuntimeError: Can't load weights for 'ernie-3.0-nano-zh'.
Please make sure that 'ernie-3.0-nano-zh' is:

a correct model-identifier of built-in pretrained models,
or a correct model-identifier of community-contributed pretrained models,
or the correct path to a directory containing relevant modeling files(model_weights and model_config).

是的 版本问题 我刚刚也发现了 。。。谢谢


Practice

👷 Deploy Preview for upbeat-jennings-3ee703 processing.
🔨 Explore the source changes: b8d848d2f79a3ee8190779b851c228b002859d1c
🔍 Inspect the deploy log: https://app.netlify.com/sites/upbeat-jennings-3ee703/deploys/620447a264e4aa0008c5a9b2


Build(deps): Bump xml-formatter from 3.6.0 to 3.6.2 (#5608)

Pull request opened by the merge tool on behalf of #5608

Merged in 15d1cf26a40ba7bf3f94eae9caf6cd61dcd291fe


Fix package bundling (UMD)

This PR adds standard UMD bundling fixing the following issues:

Current AMD bundling is using wrong name for strophe (it’s now “strophe.js”), requiring you to alias the package to make it work.
Moving “strophe.js” to peerDependency as having it as normal dependency will install a separate instance when using NPM. This means that the disco plugin get’s registered in the wrong instance and thus currently doesn’t work at all.

I have tested this new UMD bundle format on the following platforms:

Webpack
Node.js
React Native
Browser (no bundling, just

This is great, thanks!
It doesn't look like you're including the rollup generated UMD file in the commit. I think it should also be included in the repo so that other projects don't need to perform a build step first.

It get's built as soon as it's published to NPM. Bower has been discontinued, so I don't see how anybody would consume it otherwise?
It's generally pretty bad standard to include build artifacts in your repo.

It gets built as soon as it's published to NPM.

Ah ok, I didn't know that.

Can we agree on including an MIT license in all of them?

The package.json does say that the license is MIT. Do you want to include the license file in the repo? That's probably a good idea because we need the original copyright header.

It gets built as soon as it's published to NPM

That's an awesome idea to include the rollup script only in the package and not in the git repo. Thanks for your work.

@jcbrand true, but package.json is targeted NPM consumers. Github also recommends that you make clear license attribution in the repo itself as a LICENSE.txt file.
I think we should have it in both the file and package.json, and move it out of the source files to avoid confusion. My company's legal department will also be happy with that :)


Export - Typography alias incorrectly exported

Bug
When exporting an alias typography style with 'Expand typography' option selected, the json renders a Name/Value pair of the original style reference. e.g. {s, t, y, l, e, 0, 1} instead of the associated fontSize, fontWeight, lineHeight etc.
Reproduce
Steps to reproduce the behavior:

Create a single typography style using Import (style01)
Create a new typography style (alias02) and select 'alias mode' button (4 circles)
Enter the original style in the aliasName {style01}
Hit 'Update'
Hit 'Export' and check 'Expand typography'
The exported alias typography is transformed into an object with Name Value pairs that reference the Name of the original style, not its values.

Note: If you don't 'Expand typography' you just get [Object, object] in the output.
Expected behavior
Expect the new alias style to have the same references as the original style. e.g. If original fontSize = 16, then the alias fontSize should also = 16 etc.
Screenshots

Figma file
Figma file containing a reproducible example

Thanks for creating this, seems like this one slipped through when we worked on the new alias mode for typography tokens.