text:
Create of IndexedDB persister is slow when database doesn't exist

Describe the bug
I'm running the issue that initializing the IndexedDB persister is very slow, while it's fast on subsequent loads:
init_indexed_db_persister: 1858ms - timer ended
starting initial sync...
initial sync finished.
---
init_indexed_db_persister: 16ms - timer ended
store is initialized, skipping.

I thought this was IndexedDB being slow, but I then switched to pre-creating the database with the same layout tinybase expects and the whole thing is then very fast consistently in both cases:
ensure_database: 8ms - timer ended
init_indexed_db_persister: 20ms - timer ended
starting initial sync...
initial sync finished.
---
pre-creating database
ensure_database: 6ms - timer ended
init_indexed_db_persister: 24ms - timer ended
store is initialized, skipping

The init function does this:
async function initPersister(persister) {
      if (persister) {
        console.time("init_indexed_db_persister");
        // console.log("pre-creating database");
        // console.time("ensure_database");
        // await ensureDatabase("card-data");
        // console.timeEnd("ensure_database")
        await persister.startAutoLoad();
        await persister.startAutoSave();
        setStoreInitialized(true);
        console.timeEnd("init_indexed_db_persister");
      }
    }

The ensureDatabase function does this:
export function ensureDatabase(name: string) {
  return new Promise((resolve, reject) => {
    const request = window.indexedDB.open(name, 2);

    request.onerror = (event) => reject("Database error: " + event.target.error);
    request.onsuccess = (event) => resolve(event.target.result);

    request.onupgradeneeded = (event) => {
      const db = event.target.result;
      db.createObjectStore("t", { keyPath: "k" });
      db.createObjectStore("v", { keyPath: "k" });
      resolve(db);
    };
  });
}

The slowdown seems to be caused by awaitingstartAutoSave, it might be something related to the scheduling logic, I'm not sure though.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Create an indexedDBPersister and time how long it takes until startAutoSave resolves.
Expected behavior
No response
Screenshots or Videos
No response
Platform

OS :macOS
Browser: Firefox
Version: 121.0

Additional context
No response

Thanks! I'll take a look at this.

Hm, on both Chrome and Firefox this never takes more than 10ms, regardless of whether the database is there or not beforehand. Gonna keep looking...

OK, just documenting the analysis here. All with no database present:
console.time('save');
persister.save().then(() => {
  console.timeEnd('save'); // fast
});

console.time('load');
persister.load().then(() => {
  console.timeEnd('load'); // fast
});

BUT...
console.time('load');
persister.load().then(() => {
  console.timeEnd('load');
  console.time('save');
  persister.save().then(() => console.timeEnd('save')); // slow
});

🤔

I think I have a fix. The connection is not closed when the first load fails, so the save has to wait for it to timeout.

Assuming it's all good, closing out.

hey sorry, forgot to answer. yes, it's fixed on my end as well. thanks! :)

source: gharchive/issue

authors: fspoettel jamesgpearce

repo: tinyplex/tinybase

license: MIT

license_type: permissive

license_source: github-api


text:
[feat][ci] enable semantic pull request title check

This PR is for running tests for upstream PR https://github.com/apache/pulsar/pull/17961.
)

Upstream test already triggered. Closed.

source: gharchive/pull-request

repo: tisonkun/pulsar


text:
[BUG] Actions breaks if folders use parenthesis in the names

Is there an existing issue for this?

[X] I have searched the existing issues

Does this issue exist in the latest version?

[X] I'm using the latest release

Describe the bug?
SvelteKit has an advanced routing feature called layout groups: https://kit.svelte.dev/docs/advanced-routing#advanced-layouts-group - when a file within one of these folders is changed (e.g. src/routes/(app)/+page.svelte) then this action will incorrectly escape the parenthesis ending up with: INPUT_CHANGED_FILES: src/routes/\(app\)/+page.svelte and eslint will not error because it didn't find any files with that filename.
To Reproduce
In any of your examples, add parenthesis into a folder name and then commit to GitHub to see the action run and break.
What OS are you seeing the problem on?
ubuntu-latest or ubuntu-22.04
Expected behavior?
Action only runs lint on changed files and doesn't break.
Relevant log output
There is no log archive to download, but this section of the workflow run is applicable as you can see the correct filepath before the changed-files-patterns group runs but then it's wrong afterwards.
##[debug]Fetching target branch...
  /usr/bin/git fetch -q --no-tags --prune -u --progress --deepen=50 origin +refs/heads/main:refs/remotes/origin/main
  remote: Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
  Completed fetching more history.
  ##[debug]Getting current SHA...
  /usr/bin/git rev-parse --verify defa9e3ed1dab59e06e66688997ed2532c36bdf2^{commit}
  defa9e3ed1dab59e06e66688997ed2532c36bdf2
  /usr/bin/git rev-parse --verify defa9e3ed1dab59e06e66688997ed2532c36bdf2^{commit}
  defa9e3ed1dab59e06e66688997ed2532c36bdf2
  ##[debug]Current SHA: defa9e3ed1dab59e06e66688997ed2532c36bdf2
  /usr/bin/git merge-base 18db55d30244cc161d9885c65a42ffc8b2d497f5 defa9e3ed1dab59e06e66688997ed2532c36bdf2
  18db55d30244cc161d9885c65a42ffc8b2d497f5
  /usr/bin/git log --format=%H 18db55d30244cc161d9885c65a42ffc8b2d497f5..defa9e3ed1dab59e06e66688997ed2532c36bdf2
  defa9e3ed1dab59e06e66688997ed2532c36bdf2
  /usr/bin/git merge-base 18db55d30244cc161d9885c65a42ffc8b2d497f5 defa9e3ed1dab59e06e66688997ed2532c36bdf2
  18db55d30244cc161d9885c65a42ffc8b2d497f5
  /usr/bin/git log --format=%H 18db55d30244cc161d9885c65a42ffc8b2d497f5..defa9e3ed1dab59e06e66688997ed2532c36bdf2
  defa9e3ed1dab59e06e66688997ed2532c36bdf2
  /usr/bin/git rev-parse --verify 18db55d30244cc161d9885c65a42ffc8b2d497f5^{commit}
  18db55d30244cc161d9885c65a42ffc8b2d497f5
  ##[debug]Previous SHA: 18db55d30244cc161d9885c65a42ffc8b2d497f5
  /usr/bin/git merge-base 18db55d30244cc161d9885c65a42ffc8b2d497f5 defa9e3ed1dab59e06e66688997ed2532c36bdf2
  18db55d30244cc161d9885c65a42ffc8b2d497f5
  /usr/bin/git log --format=%H 18db55d30244cc161d9885c65a42ffc8b2d497f5..defa9e3ed1dab59e06e66688997ed2532c36bdf2
  defa9e3ed1dab59e06e66688997ed2532c36bdf2
  Retrieving changes between 18db55d30244cc161d9885c65a42ffc8b2d497f5 (main) → defa9e3ed1dab59e06e66688997ed2532c36bdf2 (debug-lint)
  /usr/bin/git diff --name-status --ignore-submodules=all --diff-filter=ACDMRTUX 18db55d30244cc161d9885c65a42ffc8b2d497f5...defa9e3ed1dab59e06e66688997ed2532c36bdf2
  M	site/src/routes/[mother=locale]/[learning=locale]/(app)/account/+page.svelte
  ##[debug]All diff files: {"A":[],"C":[],"D":[],"M":["site/src/routes/[mother=locale]/[learning=locale]/(app)/account/+page.svelte"],"R":[],"T":[],"U":[],"X":[]}
  All Done!
  ::endgroup::
::group::changed-files-patterns
changed-files-patterns
##[debug]Node Action run completed with exit code 0
##[debug]Set output added_files =
##[debug]Set output added_files_count = 0
##[debug]Set output copied_files =
##[debug]Set output copied_files_count = 0
##[debug]Set output modified_files = site/src/routes/[mother=locale]/[learning=locale]/\(app\)/account/+page.svelte
##[debug]Set output modified_files_count = 1
##[debug]Set output renamed_files =
##[debug]Set output renamed_files_count = 0
##[debug]Set output type_changed_files =
##[debug]Set output type_changed_files_count = 0
##[debug]Set output unmerged_files =
##[debug]Set output unmerged_files_count = 0
##[debug]Set output unknown_files =
##[debug]Set output unknown_files_count = 0
##[debug]Set output all_changed_and_modified_files = site/src/routes/[mother=locale]/[learning=locale]/\(app\)/account/+page.svelte
##[debug]Set output all_changed_and_modified_files_count = 1
##[debug]Set output all_changed_files = site/src/routes/[mother=locale]/[learning=locale]/\(app\)/account/+page.svelte
##[debug]Set output all_changed_files_count = 1
##[debug]Set output any_changed = true
##[debug]Set output only_changed = true
##[debug]Set output other_changed_files =
##[debug]Set output other_changed_files_count = 0
##[debug]Set output all_modified_files = site/src/routes/[mother=locale]/[learning=locale]/\(app\)/account/+page.svelte
##[debug]Set output all_modified_files_count = 1
##[debug]Set output any_modified = true
##[debug]Set output only_modified = true
##[debug]Set output other_modified_files =
##[debug]Set output other_modified_files_count = 0
##[debug]Set output deleted_files =
##[debug]Set output deleted_files_count = 0
##[debug]Set output any_deleted = false
##[debug]Set output only_deleted = false
##[debug]Set output other_deleted_files =
##[debug]Set output other_deleted_files_count = 0
##[debug]Finished: run

Has all relevant logs been included?

[X] I've included all relevant logs

Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I guess the real issue is in https://github.com/tj-actions/changed-files/blob/main/src/changedFiles.ts - I'll try to add a failing test there and see if I can't submit a PR to fix it.

This passes just fine so no issue here:
// Tests that getFilteredChangedFiles correctly handles parenthesis in folder names
    it('should not escape parenthesis in folder names', async () => {
      const allDiffFiles = {
        [ChangeTypeEnum.Added]: [],
        [ChangeTypeEnum.Copied]: [],
        [ChangeTypeEnum.Deleted]: [],
        [ChangeTypeEnum.Modified]: [
          'src/routes/[learning=locale]/(app)/+page.svelte'
        ],
        [ChangeTypeEnum.Renamed]: [],
        [ChangeTypeEnum.TypeChanged]: [],
        [ChangeTypeEnum.Unmerged]: [],
        [ChangeTypeEnum.Unknown]: []
      }
      const filePatterns = ['**/*.svelte']
      const filteredFiles = await getFilteredChangedFiles({
        allDiffFiles,
        filePatterns
      })
      expect(filteredFiles[ChangeTypeEnum.Modified]).toEqual([
        'src/routes/[learning=locale]/(app)/+page.svelte'
      ])
    })

The issue happens in this block. So either in the inputs.escapeJson or the JSON.stringify or the getChangeTypeFiles function. That's about as far as I can debug. But yes, definitely this issue should be moved to https://github.com/tj-actions/changed-files as the issue is either in that package or me not knowing how to use the settings properly.

Hi @jacob-8 This issue has been resolved in the latest release.
You'll need to set the escape_paths input to false explicitly
      - name: Run ESLint on changed files
        uses: tj-actions/eslint-changed-files@v25
        with:
          ...
          escape_paths: false
          ...

repo: tj-actions/eslint-changed-files


text:
criacao da tarefa expedir guia de execucao penal no fluxo de controle…

… de prazos e retorno de expedientes

Resolvi os conflitos @eriquim.

repo: tjrn/pje-fluxo

license: mit

license_source: bigquery


text:
test: Adding Cypress integration testcase for Sensitive input

Description
Please include a brief summary of the change and include the relevant issue(s).
Closes #(issue)
What change does this PR introduce?
Please select the relevant option(s).

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation update (changes to docs/code comments)

What is the proposed approach?
Please give a short overview/explanation on the approach taken to resolve the issue(s).
Checklist:

[ ] The commit message follows our adopted guidelines
[ ] Testing has been done for the change(s) added (for bug fixes/features)
[ ] Relevant comments/docs have been added/updated (for bug fixes/features)

Images:

sensitive text

Markup language

LGTM! On a side note, the current integration test approach is not sustainable, reason being we're only relying on a single configuration of settings for testing. Ideally, we should have multiple instances of the chatbot with different configurations to cover more grounds (e.g. markup language should not work if dangerouslySetInnerHtml is false). That's an issue for another time though, thanks for working on this 😊

I agree! , I also had this thought. I was not able to figure out a approach to pass different configurations. using state to load different configuration is one way.

LGTM! On a side note, the current integration test approach is not sustainable, reason being we're only relying on a single configuration of settings for testing. Ideally, we should have multiple instances of the chatbot with different configurations to cover more grounds (e.g. markup language should not work if dangerouslySetInnerHtml is false). That's an issue for another time though, thanks for working on this 😊

I agree! , I also had this thought. I was not able to figure out a approach to pass different configurations. using state to load different configuration is one way.

This can likely be broken down into a few issues:

Create toggle buttons to switch between different chatbot configurations
Create issue for different chatbot configurations (can be 1 issue per config e.g. all settings enabled, all settings disabled etc)

This is not pressing but if you are keen, we can talk more on discord 🙂

LGTM! On a side note, the current integration test approach is not sustainable, reason being we're only relying on a single configuration of settings for testing. Ideally, we should have multiple instances of the chatbot with different configurations to cover more grounds (e.g. markup language should not work if dangerouslySetInnerHtml is false). That's an issue for another time though, thanks for working on this 😊

I agree! , I also had this thought. I was not able to figure out a approach to pass different configurations. using state to load different configuration is one way.

This can likely be broken down into a few issues:

Create toggle buttons to switch between different chatbot configurations
Create issue for different chatbot configurations (can be 1 issue per config e.g. all settings enabled, all settings disabled etc)

This is not pressing but if you are keen, we can talk more on discord 🙂

Yes we can do that.

repo: tjtanjin/react-chatbotify


text:
Rename projection_back to scale_resotration

Summary
Rename use_projection_back to scale_restoration.
close #118

I have to update notebooks as well.

Codecov Report

Merging #120 (7dddcc5) into develop (6871f55) will not change coverage.
The diff coverage is 88.70%.

:exclamation: Current head 7dddcc5 differs from pull request most recent head 317fc12. Consider uploading reports for the commit 317fc12 to get more accurate results

@@           Coverage Diff            @@
##           develop     #120   +/-   ##
========================================
  Coverage    96.30%   96.30%
========================================
  Files           20       20
  Lines         2516     2516
========================================
  Hits          2423     2423
  Misses          93       93

Impacted Files
Coverage Δ

ssspy/bss/fdica.py
93.08% <86.36%> (ø)

ssspy/bss/iva.py
95.11% <87.50%> (ø)

ssspy/bss/ilrma.py
98.38% <93.75%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


text:
Use pytest-xdist

Summary
To accelerate the tests in GitHub actions, pytest-xdist is installed.
By default, the test is performed by 16 parallel.

Codecov Report
Patch coverage has no change and project coverage change: -0.43 :warning:

Comparison is base (0a5eadf) 94.42% compared to head (1e9972d) 93.99%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #235      +/-   ##
==========================================
- Coverage   94.42%   93.99%   -0.43%
==========================================
  Files          43       43
  Lines        5163     5163
==========================================
- Hits         4875     4853      -22
- Misses        288      310      +22

see 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


text:
Card TinyMCE text fields: paragraphs separated by spaces instead of line breaks

Expected behavior
Multiple paragraphs entered into the TinyMCE fields in the backend should be formatted properly as separate paragraphs in the frontend.
Current behavior
Multiple paragraphs are smashed together and only separated by a couple of spaces.
This is likely due to CSS settings in the frontend.
Steps to recreate the problem
In the admin view, in any empty textbox of any cards, do

type "Hello"
hit Enter
type "World"

Expected output:

Hello
World

Actual output:

Hello World

Created a pull request to fix this issue.

This fix has been verified - the test paragraphs entered on the backend are correctly formatted when viewing the card in the CAE frontend.
Test Course: https://umich.beta.instructure.com/courses/587202/external_tools/48681

[ ] From the CAE tool on the test course, click Admin link at the upper right  to access the Admin UI
[ ] Select 'Lti tools' to see the list of cards
[ ] Click on a tool: Adobe Creative Cloud
[ ] Delete the current content of the Short Description field ("Access Creative Cloud Express projects and apps within Canvas, and use them to create and complete their assignments.")
[ ] In the Short Description field, type "Hello" -> hit Enter - > type "World".
[ ] Toggle the HTML editor button on the toolbar to verify  tags for each word
[ ] Click Save - LTI tool list loads
[ ] Click View Site link to return to the CAE tool on the course
[ ] Verify that "Hello" and "World" display as separate paragraphs, one word above the other with a line break separating the paragraphs
[ ] Return to Admin UI -> Lti tools ->  click on the ACC listing
[ ] Enter the original text into the Short Description field and click Save
[ ] Return to the course and verify that the ACC edit has been saved with the original text

repo: tl-its-umich-edu/canvas-app-explorer


text:
Levels of the quantifier bounds were effectively ignored.

Fixes #188.

Looks good to me, thanks for catching this!

repo: tlaplus/tlapm


text:
Add java and latex to container

Changes

Add java and latex to devcontainer to be able to run the tlaplus tools when debugging the extension.
Updated the library for running the integration tests
Clean up instalation of the packages required to run the integration tests from the terminal
Update documentation
Removed the changes to the default user in the container (if necessary the required permissions to the folder can be set on the host side)

LGTM.  Thanks!

authors: afonsonf lemmy

repo: tlaplus/vscode-tlaplus


text:
[VM] Refactor and improve vm.

Have a separate function for RunInstCall.
Cache func_index lookup by table to avoid repeative lookup by str.
Move PackedFunc call arg stack to Frame to increase locality and avoid re-allocation in repeative calls.
Make frame stack of unique_ptr to avoid frame re-allocation and copy during frame.resize.
Pass curr_frame as arguments into sub-functions to make it explicit.
Reduce rtol to avoid flaky test

Thanks TQ! thanks for the review! @psrivas2 @YuchenJin @MasterJH5574

authors: tqchen yongwww

repo: tlc-pack/relax


text:
Typescript support

@tleunen first thanks a lot for creating react-mdl. I came from .NET and angular world and I surprisingly recognized that there is not any feasible ui framework for React so far (apart from horrible and not mobile friendly Material-UI or apart from react-bootstrap). I appreciate a lot your approach of react component wrapping up Google JS and CSS and I find it very promising.
However, I am missing the Typescript support. Typescript is not so popular in React community but its usage in JavaScipt and node.js (and angular 2) world is incredibly growing.
Are you interested in contributing react-mdl.d.ts Typescript definition file?
If yes:

for what r-mdl version (1.5, 2.0 or both)? Alternatively, will both versions have the same interface?
How are you preparing ## Configuration table in documentation? I think that content of this table could be a good source for semi-automatic .d.ts file generation.

I'm not a typescript user, but indeed it would be great having the typescript description file for all the typescript users out there :)
Even if I'm thinking about v2 (which btw will be fully React, without the initial material.js from MDL), it's still in early development. So if you want the typescript support asap, it's better to do it against v 1.5 first ;) Ideally, both versions will have a really close interface. Most components won't change.
For the conf table in the doc, it's manually, but I like the idea of scanning the component and generating it automatically :)

OK, I will try to parse 1.5's .propTypes objects.

I prepared the first .d.ts version. I have problem with the functional properties signature. Some of them I recognized from your code - can you check it out please? And there are three of them which I cannot recognize.
Thanks.
Problems:

TableHeader.cellFormatter - ??
Dialog.onCancel - ??
TableHeader.sortFn - ??

Check it out please:

Checkbox.onChange - event

Radio.onChange - event

Textfield.onChange - event

IconToggle.onChange - event

RadioGroup.onChange - event

Slider.onChange - event

Snackbar.onActionClick - event

Switch.onChange - event

TableHeader.onClick - (event, name:string)

HeaderTabs.onChange - tabId:string

Tab.onTabClick - tabId:string

TabBar.onChange - tabId:string

Tabs.onChange - tabId:string

Snackbar.onTimeout - void

TableHeader.cellFormatter can take 3 params, but only the first one is really needed. The 2 others are just in some specific usecases. (String, Object, Number) which are the value in the cell, the row object, and the index of the row.
Your definition for TableHeader.sortFn is correct. And the others as well.

All elements are expected to have intrinsic HTML attributes like style, I can see this from examples. Also, React.Props<T> is marked as obsolete. In addition, we need to have support for importing individual components. I started a repo for Typings, adding new elements continuously but only when I need them in my project. It may take about a week until I finish all types.

@tleunen thanks, I modified TableHeader.cellFormatter to (val:string, row: any, rowIndex:number) => string.
@alexeyzimarev, thanks for info about your project.

All my d.ts code is generated from @tleunen 's propTypes objects (including oneOfType or oneOf, which is very usefull for typing, see e.g. FooterSectionProps.type?: 'top' | 'middle' | 'bottom' | 'left' | 'right'; etc.). So it contains all valid properties and events - you can use it as source for your repo.
inspecting @tleunen 's code, I am not sure that all elements are expected to have intrinsic HTML attributes. But maybe I am wrong.

I don't mind adding a typescript definition file in this project, if one of you guys want to send a PR for that ;)
As @alexeyzimarev mentioned, almost all elements can have intrinsic attributes.

intrinsic attributes
I added them to all components except of Table. The reason is that official react.d.ts definition file contains following definitions for some of the intrinsic attributes:
value?: string | string[];
rows?: number;

Those definitions are in conflict with the following react-mdl definitions:
value: string | number for Radio and RadioGroup
rows: Array<{}>  for Table.
Possible solution is to keep value?: string | string[] for Radio and RadioGroup + to ommit intrinsic attributes for Table
In order to have intrinsic attributes for Table, it is necessary to rename Table's rows attribute.

@PavelPZ as you can see, I define interfaces for both Props and for ComponentClass. This is because I find it not correct to define classes in type definitions although I realize that these are ambient references, still it actually would generate some real JS code, whilst interface definitions do not generate anything. In order to export a component type, I declare a constant that has interface as its type and export it.
This approach is used in type definitions for many popular component libraries like react-bootstrap.
Regarding the HTML attributes, all components are in fact HTML elements and normally should have all intrinsic HTML attributes available. These attributes are widely used in samples and this is where I got issues using your d.ts file.
I am not sure if machine generation is good or not. I am still working on my definitions file and I test all components with it. Adding oneOf is not a problem for me too but of course it will need changes if the source library changes.

@alexeyzimarev, HTML attributes are already solved.
d.ts machine generation is based on correct propTypes in @tleunen code, see #273.
I am successfully using react-mdl.d.ts in my eLearning project now.

I see that typescript are available in the DefinitelyTyped repo. See https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/react-mdl/

authors: PavelPZ alexeyzimarev tleunen

repo: tleunen/react-mdl


text:
How to improve training efficiency and shorten training time

I have two T4 on my machine, and I want to improve training efficiency， because it has enough memory when I use the default params

I tried to update the batch_size to 256 but it doesn't seem to be working

maybe deepspeed you can try

you should increase micro_batch_size param @Tungsong

authors: ChrisXULC Tamminhdiep97 Tungsong

repo: tloen/alpaca-lora


text:
Permission error 13 when opening pre-existing folder.

Hi! I'm evaluating several todo.txt apps, and ntodo is a promising candidate.
I ran into a hang condition immediately at first start.
After installing, I chose "offline". Then I tapped the  path, and navigated to my pre-existing shared folder.
This resulted in a blank, grey screen with no way to proceed.
My guess is that the app chokes on some other, preexisting files in the folder. My shared folder (managed by Sync Thing) also contains an encrypted MyExpenses database, so it has subfolders and some textfiles with "gibberish".
It did not make a difference if a todo.txt exists in the folder root or not. The gray screen happens both with and without that file.
The only way out of the gray screen I've found is to uninstall the app, clear the cache from device settings, and then re-install, which is really elaborate.
screenshots
The offline-login screen

The "evil" folder, containing a MyExpenses DB + todo.txt

Resulting hang: gray screen

Thank you for reporting this issue.
Which version you are using? The issue should not fix with version 0.6.0. However, a precise error description should be displayed.
Normally the file picker should ask you for authorization for the folder. I assume this does not happen because the folder already exists and was not created.
I check this and make sure that the authorization is also correct for existing folders.

I have checked this. With version 0.6.0 you should receive an error message if the permissions of the folder does not match. However, the real problem remains.

Ah, I am on 0.5.1, fresh from F-droid, I should have mentioned that 😅
Tapping the local path does indeed get me the (system) filepicker, and "deze map gebruiken" (Dutch, "use this folder") does indeed get a system-pop-up asking for permission, which I granted.
Is v0.6.0 something you just created? Or can I get it somewhere?

Is v0.6.0 something you just created? Or can I get it somewhere?

Version 0.6.0 should be in the fdroid-store since today.

Aha, I needed to refresh my F-droid source lists, now they see 0.6.0.
The error is indeed a permissions problem:
OS error: permission denied;error 13.
I did grant the requested "storage access" permission when first-tapping the path.
I wonder if this is due to my usage of android 12. That, I believe, has an additional finer grained, per-folder system, which may be infering.
Alternatively, it could be that that folder is already "claimed" by the MyExpenses app, though I am not an Android permission expert..

P.S. I want to stress again that I mean my feedback in the most positive way!
Getting anything at all working on Android is non-trivial, so, thanks for making this, making it in the open, and making it in a privacy friendly way without tracking ❤

The issue still does not seem to have been fixed (exactly the same problem running v0.6.1 from F-Droid).

Positive feedback: Last night I received the update that fixed this. I can now open existing todos 🎉
It seems the app now requires the "manage all files" permission to solve this? Regardless, Android asked me if I wanted to grant permission to the specific folder as well.

It seems the app now requires the "manage all files" permission to solve this? Regardless, Android asked me if I wanted to grant permission to the specific folder as well.

Yes, because I don't restrict the path where the todos may be saved and for accessing the Android's file system the app needs the MANAGE_EXTERNAL_STORAGE permission (Permission.manageExternalStorage - "manage all files").
The dialog that asks for permission for a specific folder comes from the file_picker plugin. The dialog is not really necessary, but I don't see any way to turn it off either.

authors: juleskers kiddae tmaegel

repo: tmaegel/ntodotxt


text:
textsplitter: allow configuration of RecursiveCharacter

Description
Think it'll be helpful to allow clients to configure text splitters while keeping some of the sane defaults in place. I recently had to configure ChunkSize and ChunkOverlap but couldn't do so without manually creating a RecursiveCharacter:
rc := textsplitter.RecursiveCharacter{
		Separators:   []string{"\n\n", "\n", " ", ""},
		ChunkSize:    1024,
		ChunkOverlap: 0,
}

I've tried to follow the patterns we have for functional options in other packages.
PR Checklist

[x] Read the Contributing documentation.
[x ] Read the Code of conduct documentation.
[x] Name your Pull Request title clearly, concisely, and prefixed with the name of the primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. Fixes #123).
[x] Describes the source of new concepts.
[x] References existing implementations as appropriate.
[x] Contains test coverage for new functions.
[x] Passes all golangci-lint checks.

Actually, I just realised I had a brain-fart moment and could have just done:
rc := textsplitter.NewRecursiveCharacter()
rc.ChunkSize = 1024
rc.ChunkOverlap = 0

I'm happy to close this if you think functional options isn't that important here.

I'm not sure this circumstance warrants functional arguments -> I generally want to reserve those for the configuration of key clients and methods that are going to be used in many cases and should have a dead simple set of defaults. I think specifying a/the RecursiveCharacter struct literal is fine here

Sure, makes sense.

authors: tmc tobiade

repo: tmc/langchaingo


text:
fix: rename test exp packages and fix all test compiling error

Summary
Part of the fixes of #18.
Details

Placed the test files near by the code file it tested instead of nesting with one more test sub-dir in order to meet the idiomatic requirements
Removed suffix _test from the package names of test files in order to align the package names and meet the idiomatic requirements
Fixed all the imports issue by adding exp sub-dir into the import path

@nekomeowww thanks for this fix! I see the main benefit of aligning test package names with the tested package name as a means of doing greybox style testing of unexported functions/types/fields. In this/our case I can't help but think we should preserve "_test" in the package names (but very much agree with moving the test files into the corresponding directories).

@nekomeowww thanks for this fix! I see the main benefit of aligning test package names with the tested package name as a means of doing greybox style testing of unexported functions/types/fields. In this/our case I can't help but think we should preserve "_test" in the package names (but very much agree with moving the test files into the corresponding directories).

Yes it is for unexported functions/types/fields since we are utilities library and we will only export the needed ones to users while we may have a lot of helper functions to implement the algorithms.

Btw, thanks for helping on fixing those incorrect tests. I didn't include it because I thought it would be nice to separate them into different PRs to speed up the review 😆

authors: nekomeowww tmc


text:
Microschema

Add it.
https://schema.org/Recipe

Done

repo: tmcw/old-fashioned


text:
🛑 Portal 6 is down

In 0449a47, Portal 6 (https://portal6.hcmus.edu.vn/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Portal 6 is back up in 54dc6d8.

repo: tmkhiem/hcmus-web-uptime


text:
Using multidex + "dexopener" - java.lang.NoClassDefFoundError

Reposted from: https://github.com/mockito/mockito/issues/1082#issuecomment-315651419
Starting 0 tests on emulator-5554 - 4.4.2
Tests on emulator-5554 - 4.4.2 failed: Instrumentation run failed due to 'java.lang.NoClassDefFoundError'

com.android.builder.testing.ConnectedDevice > No tests found.[emulator-5554 - 4.4.2] FAILED
No tests found. This usually means that your test classes are not in the form that your test runner expects (e.g. don't inherit from TestCase or lack @Test annotations).

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':connectedDebugAndroidTest'.
> There were failing tests. See the report at: file:///Users/<>/repo/android-gif-example/build/reports/androidTests/connected/index.html

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED in 6s
94 actionable tasks: 39 executed, 55 up-to-date

Version of dexmaker - 0.10.1
I was using my example project here: https://github.com/jaredsburrows/android-gif-example

Your project seems to use neither Multidex nor DexOpener.
Could you tell me the steps to reproduce?

I just applied multidex and your library along with mockito-android library.

Are you using Multidex for your test apk?
If so, AndroidJUnitRunner does not support mulitdex for test apks, so DexOpener does not support it.
https://developer.android.com/studio/build/multidex.html?hl=en#testing

Notes:

Don't use MultiDexTestRunner, which is deprecated; use AndroidJUnitRunner instead.
Using multidex to create a test APK is not currently supported.

@tmurakami  Multidex 1.0.2 is out:
https://developer.android.com/topic/libraries/support-library/revisions.html#25-4-0

Concurrent with this support library release, we are also releasing multidex version 1.0.2. This version includes the following important changes:
Allows multidexing of instrumentation APK.
Deprecates MultiDexTestRunner (AndroidJUnitRunner should be used instead).
Provides better protection against some bad archive extraction management of the app.
Fixes a bug that could lead to abandoned temporary files.
Provides faster installation when done in concurrent process.
Fixes an installation bug on API 19 and 20.
https://developer.android.com/topic/libraries/support-library/revisions.html#25-4-0

In theory multidex for the test APK should work. But the fact of the matter is using your library + mockito-android makes it multidex anyways.

Thank you for your information.
First I tried Multidex 1.0.2 without using DexOpener, but your instrumented tests does not seem to work.
$ ./gradlew clean connectedDebugAndroidTest

[snip]

Starting 0 tests on Nexus_5_API_19(AVD) - 4.4.2
Tests on Nexus_5_API_19(AVD) - 4.4.2 failed: Instrumentation run failed due to 'java.lang.NoClassDefFoundError'

com.android.builder.testing.ConnectedDevice > No tests found.[Nexus_5_API_19(AVD) - 4.4.2] FAILED
No tests found. This usually means that your test classes are not in the form that your test runner expects (e.g. don't inherit from TestCase or lack @Test annotations).

FAILURE: Build failed with an exception.

Logcat is here:
E/AndroidRuntime( 3574): FATAL EXCEPTION: Instr: android.support.test.runner.AndroidJUnitRunner
E/AndroidRuntime( 3574): Process: burrows.apps.example.gif.debug, PID: 3574
E/AndroidRuntime( 3574): java.lang.NoClassDefFoundError: org.junit.runner.manipulation.Filter$1
E/AndroidRuntime( 3574): 	at org.junit.runner.manipulation.Filter.<clinit>(Filter.java:21)
E/AndroidRuntime( 3574): 	at android.support.test.internal.runner.TestRequestBuilder.<init>(TestRequestBuilder.java:83)
E/AndroidRuntime( 3574): 	at android.support.test.internal.runner.TestRequestBuilder.<init>(TestRequestBuilder.java:525)
E/AndroidRuntime( 3574): 	at android.support.test.runner.AndroidJUnitRunner.createTestRequestBuilder(AndroidJUnitRunner.java:370)
E/AndroidRuntime( 3574): 	at android.support.test.runner.AndroidJUnitRunner.buildRequest(AndroidJUnitRunner.java:343)
E/AndroidRuntime( 3574): 	at android.support.test.runner.AndroidJUnitRunner.onStart(AndroidJUnitRunner.java:260)
E/AndroidRuntime( 3574): 	at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1701)
W/ActivityManager( 1568): Error in app burrows.apps.example.gif.debug running instrumentation ComponentInfo{burrows.apps.example.gif.test/android.support.test.runner.AndroidJUnitRunner}:
W/ActivityManager( 1568):   java.lang.NoClassDefFoundError
W/ActivityManager( 1568):   java.lang.NoClassDefFoundError: org.junit.runner.manipulation.Filter$1
I/ActivityManager( 1568): Force stopping burrows.apps.example.gif.debug appid=10067 user=0: finished inst
I/ActivityManager( 1568): Killing 3574:burrows.apps.example.gif.debug/u0a67 (adj 0): stop burrows.apps.example.gif.debug
I/ActivityManager( 1568): Delay finish: com.google.android.gms/.stats.service.DropBoxEntryAddedReceiver
I/ActivityManager( 1568): Resuming delayed broadcast

Could you please point out any problems with the following changes?
diff --git a/build.gradle b/build.gradle
index c45e8e0..94567bb 100644
--- a/build.gradle
+++ b/build.gradle
@@ -55,6 +55,7 @@ android {
     versionName "1.0"
     minSdkVersion rootProject.ext.lollipop ? 21 : rootProject.ext.minSdkVersion // Optimize build speed - build with minSdk 21 if using multidex
     targetSdkVersion rootProject.ext.targetSdkVersion
+    multiDexEnabled true
     testApplicationId "burrows.apps.example.gif.test"
     testInstrumentationRunner "android.support.test.runner.AndroidJUnitRunner"
     resConfigs "en"                                                             // Optimize APK size - keep only english resource files for now
@@ -158,9 +159,11 @@ android {
 configurations.all {
   resolutionStrategy.force deps.supportAnnotations
   resolutionStrategy.force deps.kotlinStdlib
+  resolutionStrategy.force 'com.android.support:multidex:1.0.2'
 }

 dependencies {
+  compile 'com.android.support:multidex:1.0.2'
   // Android/Google
   compile deps.design
   compile deps.cardviewv7
@@ -187,8 +190,9 @@ dependencies {
   androidTestCompile deps.junit
   androidTestCompile deps.assertjCore
   androidTestCompile deps.mockitoKotlin, { exclude group: "net.bytebuddy" }     // DexMaker has it"s own MockMaker
-  androidTestCompile deps.mockitoCore, { exclude group: "net.bytebuddy" }       // DexMaker has it"s own MockMaker
-  androidTestCompile deps.dexmakerMockito, { exclude group: "net.bytebuddy" }   // DexMaker has it"s own MockMaker
+//  androidTestCompile deps.mockitoCore, { exclude group: "net.bytebuddy" }       // DexMaker has it"s own MockMaker
+//  androidTestCompile deps.dexmakerMockito, { exclude group: "net.bytebuddy" }   // DexMaker has it"s own MockMaker
+  androidTestCompile 'org.mockito:mockito-android:2.8.47'
   androidTestCompile deps.runner
   androidTestCompile deps.espressoCore
   androidTestCompile deps.espressoIntents
diff --git a/src/main/kotlin/burrows/apps/example/gif/App.kt b/src/main/kotlin/burrows/apps/example/gif/App.kt
index ae5c2ea..baed7f9 100644
--- a/src/main/kotlin/burrows/apps/example/gif/App.kt
+++ b/src/main/kotlin/burrows/apps/example/gif/App.kt
@@ -1,7 +1,7 @@
 package burrows.apps.example.gif

 import android.annotation.SuppressLint
-import android.app.Application
+import android.support.multidex.MultiDexApplication
 import burrows.apps.example.gif.presentation.di.component.ActivityComponent
 import burrows.apps.example.gif.presentation.di.component.AppComponent

@@ -9,7 +9,7 @@ import burrows.apps.example.gif.presentation.di.component.AppComponent
  * @author [Jared Burrows](mailto:jaredsburrows@gmail.com)
  */
 @SuppressLint("Registered")
-open class App : Application() {
+open class App : MultiDexApplication() {
   lateinit var appComponent: AppComponent
   lateinit var activityComponent: ActivityComponent

Perhaps I found a solution.

Create your own JUnitRunner as follows

class JUnitRunner : AndroidJUnitRunner() {
  override fun onCreate(arguments: Bundle?) {
    MultiDex.installInstrumentation(context, targetContext)
    super.onCreate(arguments)
  }
}

Set JUnitRunner as the default test instrumentation runner

android {
    defaultConfig {
        // testInstrumentationRunner "android.support.test.runner.AndroidJUnitRunner"
        testInstrumentationRunner "your.own.JUnitRunner"
    }
}

If you use DexOpener, change the base class of your own JUnitRunner to DexOpenerAndroidJUnitRunner.
class JUnitRunner : DexOpenerAndroidJUnitRunner() {
  override fun onCreate(arguments: Bundle?) {
    MultiDex.installInstrumentation(context, targetContext)
    super.onCreate(arguments)
  }
}

But now, due to the issue #8, DexOpener does not work with your project.
As a workaround, add DataBindingUtil::class.java into App#onCreate().
open class App : Application() {
  lateinit var appComponent: AppComponent
  lateinit var activityComponent: ActivityComponent

  override fun onCreate() {
    super.onCreate()

    DataBindingUtil::class.java // Add this line

    // Setup components
    appComponent = AppComponent.init(this)
    activityComponent = ActivityComponent.init(appComponent)
  }
}

We have released version 0.10.2 which fixed the issue #8.

@tmurakami Thank you so much! I will try it in my project soon!

@tmurakami If you do not mind me asking, how did you know about installInstrumentation ? I do not see it in the documentation: https://developer.android.com/reference/android/support/multidex/MultiDex.html.

@jaredsburrows
https://android.googlesource.com/platform/frameworks/multidex/+/master/library/src/android/support/multidex/MultiDex.java#127
https://android.googlesource.com/platform/frameworks/multidex/+/master/instrumentation/src/com/android/test/runner/MultiDexTestRunner.java#35

I had a mistake.
I overlooked that your app uses applicationIdSuffix.
There is a problem with the version 0.10.2, so you should use 0.10.1.
DexOpenerAndroidJUnitRunner does not work well with applicationIdSuffix.
Instead you should use DexOpener#builder(Context) on your JUnitRunner.
class JUnitRunner : AndroidJUnitRunner() {
  override fun newApplication(cl: ClassLoader, className: String?, context: Context): Application {
    DexOpener.builder(context)
      .openIf { it.startsWith("burrows.apps.example.gif") }
      .build()
      .installTo(cl)
    return super.newApplication(cl, className, context)
  }
}

@tmurakami Thanks for sharing the source. I guess the documentation is not up yet.
Thanks, I will look into it.

This issue seems to be solved, so I will close this.
If there is any problem, please reopen this issue.

Thank you.

authors: jaredsburrows tmurakami

repo: tmurakami/dexopener


text:
Lesson 3 World War II

We need to create markdown for the below

chapter-4.pdf

authors: rmhari sathishk


text:
Appstudio update test-component-pac-kmhx

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-kmhx-on-pull-request has failed.

StatusDurationName

✅ Succeeded
24 seconds
init

✅ Succeeded
31 seconds
clone-repository

❌ Failed
1 minute
build-container

✅ Succeeded
7 seconds
show-summary

Failure reasontask build-container has the status "Failed":
STEP 1/3: FROM quay.io/jitesoft/nginx:latest
Trying to pull quay.io/jitesoft/nginx:latest...
Error: creating build container: initializing source docker://quay.io/jitesoft/nginx:latest: unable to retrieve auth token: invalid username/password: unauthorized: Invalid Username or Password

Pipelines as Code CI/test-component-pac-kmhx-on-pull-request has failed.

StatusDurationName

✅ Succeeded
24 seconds
init

✅ Succeeded
31 seconds
clone-repository

❌ Failed
1 minute
build-container

✅ Succeeded
7 seconds
show-summary

Failure reasontask build-container has the status "Failed":
STEP 1/3: FROM quay.io/jitesoft/nginx:latest
Trying to pull quay.io/jitesoft/nginx:latest...
Error: creating build container: initializing source docker://quay.io/jitesoft/nginx:latest: unable to retrieve auth token: invalid username/password: unauthorized: Invalid Username or Password


text:
ruoyi-vue 项目无法正常启动 报错

可以贴一下你的项目目录吗？最好是可以精确到此处引用的文件粒度

import vue 文件必须带有 .vue 后缀

        有啥好的解决方案没啊

给个demo？

你俩为啥不信我说的，上面报错就是 import 要加 .vue 后缀

我是信你的，不过我感觉他引用的不是vue文件，如果是引用的vue文件的话，在不改动项目的基础上，可以加个插件来实现。

这引用的显然是 vue 文件。且 vite 不支持不加 vue 后缀，vue-cli 也推荐加，vue-cli 5.x 也是必须加。理由就是决定是否编译 Vue SFC 靠文件后缀，不再检测内容，为了速度

如果这是jsx语法的呢？为什么只有detailTitle的引用报错了，前面两个没有报错呢？我认为这需要提供一个代码目录截图或者demo，来确认一下到底问题出在哪里

有道具 而且这个还不是用的 alias @.
不过这用户逼逼叨半天也不给个复现代码，也是醉了

把这个项目拉下来试一试就知道了呀

把这个项目拉下来试一试就知道了呀

哪个项目？地址给下？

https://gitee.com/y_project/RuoYi-Vue?_from=gitee_search

https://gitee.com/y_project/RuoYi-Vue/tree/master/ruoyi-ui

@dravenww

大佬 有试过吗

你提供的demo，没有看到截图所对应的代码

那这个demo你那边能跑起来吗 使用vite后

你这个项目需要改造下才能跑起来，这个项目跑不起来的原因是 @IndexXuan 所说的引用vue文件须加上.vue的后缀。

暂时先关闭此issue了，如有问题，可重新提交

authors: IndexXuan dravenww liangbaika


text:
“Array to string conversion” Error when using {{ title }}

In the “Title-Template” field, when trying to type {{ title }} and the brackets are left empty at first, as soon as the first closing bracket is typed (like this {{}), I get a modal prompting “Array to string conversion”.

If, however, I enter at least one character or more after the two opening brackets, everything is fine.

I wonder if this is related to an issue in Kirby core or Kirby SEO

Thought I'd add that I also have the same issue. I'm using a Page model with the metaDefaults function, but if I set the og:image manually on the page, I get that error when saving the page.

Thank you!

authors: linusrogge sswany tobimori

repo: tobimori/kirby-seo


text:
Remove our use of the stack-sizes tool.

As of version 0.5.0, stack-sizes is no longer a binary crate. This breaks our CI, which tries to install the latest version of the stack-sizes binary.
This PR removes all references to the tool from libtock-rs, so that CI can continue working.
stack-sizes appears to have been replaced by cargo-call-stack, which we may add support for in the future.

bors r+

repo: tock/libtock-rs


text:
Laravel 10.13.2 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v10.13.2 branch.
Before merging, you need to:

Checkout the shift-ci-v10.13.2 branch
Review all pull request comments for additional changes
Run composer update
Thoroughly test your application (no tests?, no CI?)

:warning: The Shift build servers run PHP 8.1. If you are running a different version of PHP you should run composer update locally to ensure you have the proper versions of your dependencies to reflect your PHP environment.


text:
Replace parquet.js with hyparquet

Uses hyparquet for javascript parquet parsing. It is a small, pure js implementation of parquet parsing with no dependencies. Parquet.js that this replaces is unmaintained and has not been updated in 5+ years.
Fixes #102 and #104 by using a well-maintained parquet library that supports modern parquet files.
I tested this with the parquet file generated by together-python and confirmed that upload works and fixes issue #104.
Let me know if I can help with anything!

@Nutlope? Anyone?

This should be straightforward to review.

authors: nicolasembleton platypii

repo: togethercomputer/together-typescript


text:
[hub-mirror] 请求执行任务

{
"platform":"",
"hub-mirror": [
"https://docker.1panel.dev/uusec/nanqiang:latest",
"https://docker.1panel.dev/percona/percona-server:5.7"
]
}

{
"platform":"",
"hub-mirror": [
"docker.1panel.dev/uusec/nanqiang:latest",
"docker.1panel.dev/percona/percona-server:5.7"
]
}

repo: togettoyou/hub-mirror


text:
Add evernote support

I added evernote support for the contest. It shows up with the cards that display the title of the note.

It seems that if I have multiple notes all the start time links are attached to the first note. Clicking the link gets data of the correct note.

Evernote integration added with pull request #426
Thank you for your contribution and for taking part in the challenge.

authors: IndrekV JustinDFuller

repo: toggl/toggl-button


text:
feat: connect created styles to local variables

In relation to this question: https://tokens-studio.slack.com/archives/C03C60J1XUP/p1687428596648179?thread_ts=1687367476.523889&cid=C03C60J1XUP
This is probably a very simplified implementation, as it should probably consider some other settings as well, like whether the first part of token path is removed from style names or not. But seemed to work okay for my test, I now have styles connected to vars!
Feel free to iterate on, orpoint out what I should do additionally to get this merged, or close at some point :)

I'll be closing this, we're working on something similar for our current epic: https://github.com/tokens-studio/figma-plugin/issues/2250

repo: tokens-studio/figma-plugin


text:
Update FreeBSD CI environment to 12.4

12.3 will soon be EoL

Thanks @asomers.

authors: Thomasdezeeuw asomers

repo: tokio-rs/mio


text:
Adds support for UnixStream and UnixListener on Windows

This PR supersedes PR https://github.com/tokio-rs/mio/pull/1610 and solves issue https://github.com/tokio-rs/mio/issues/1609
Background Info
I have spoken with Thomasdezeeuw and his main complaints with the last PR was

It's been a while since I've looked at the pr. But basically I don't want any (substantial) changes to the tests as that would be we would break already existing (Unix only) code that people have written using Mio. Ideally we pave over an platform differences between Unix and Windows

So I was looking into cleaning up the PR and cleaned up some messy edges. I am making this draft pull request just to show in the public where it is at and what I am working on.
Ready for review !!!
The PR is ready review I made it so we didn't have to change the tests. The only test what is different is the try_io doc test which is because the unix one uses pair which isn't supported on Windows. But all the tests in the tests folder matches.
The only addition to the tests would be in tests/UnixStream.rs
assert_would_block(stream.read(&mut buf));

This is because of https://github.com/tokio-rs/mio/issues/1611 which is known about and intentional done this way.
sys/windows/stdnet is a implementation matching the standards libraries implementation of Unix UDS. So if the standard library ever adds support for Unix UDS for windows it should be a drop in replacement.

Any plan for reviewing/merging this PR?

I'm going to close this because it still needs some work and hasn't seen any activity recently. @KolbyML thanks for the effort!

authors: KolbyML Thomasdezeeuw leiless


text:
rename generic apps

calculator
minesweeper
???

I'm not convinced that we really care to do this. Are we saying that no package can have a generic name ever? In my opinion it should really be first come, first served with names, like other repositories do.

I agree with @Eeems on this one. Moreover, we should strive to follow the upstream package name as much as we can to avoid confusing users who are looking for a specific package. So name changes should be discussed per package at the upstream level IMO.

I guess topic this is based on the discussion in #97. I agree that a package should match the package as close as possible. Though the points in the mentioned PR should also be considered.
Someone making another calculator or minesweeper game would always look like a second class citizen and those are trivial apps, other may do as well.
I didn't think it possible for chess to eventually collide but I was proven wrong as well.

Edit: At least minesweeper clould probably be renamed on the source as well.

i'm thinking freemines or something similar for it

we should strive to follow the upstream package name as much as we can to avoid confusing users who are looking for a specific package

what happens when two packages have same name upstream?

what happens when two packages have same name upstream?

Then we could prefix the two packages with the name of their author, for example. This is something we need to mention in docs/package.md#pkgname.

I fully agree with you @raisjnn, but, if I may reiterate, I think we want to avoid a situation where a package has a name in Toltec that is too far away from its upstream name. Would it be sufficient to add a sentence to https://www.github.com/toltec-dev/toltec/tree/testing/docs/package.md#pkgname asking for authors who submit their packages from now on to avoid choosing a generic name for their project?

I fully agree with you @raisjnn, but, if I may reiterate, I think we want to avoid a situation where a package has a name in Toltec that is too far away from its upstream name.

if two people choose the name 'foo', one of them will end up with repository/foo or something similar. the people who come looking for foo, will be pointed at the other package with the same name and will inadvertently install it. we will make this situation much more likely to happen by allowing generic names. one possible solution is namespacing all packages (or only ones we think are generic?) so they include a github username or something similar.

Would it be sufficient to add a sentence to https://www.github.com/toltec-dev/toltec/tree/testing/docs/package.md#pkgname asking for authors who submit their packages from now on to avoid choosing a generic name for their project?

i'd still consider banning generic names or advising that if the name is too generic, it might get renamed in the future when there is a collision.
the calculator app is a good example of why i advocate this: the app is super simple and any number of calculators would be more interesting. why do we give preference to one over the other based on time (who reserved the name first), when what would benefit the users is preference of name based on functionality and user experience.

#137 adds to the tooling the features needed for renaming packages while providing a seamless transition to end-users.

authors: Eeems LinusCDE matteodelabre raisjn raisjnn

repo: toltec-dev/toltec


text:
Heavily improve automatic model card generation

Hello!
Pull Request overview

Heavily improve automatic model card generation

Example generated output
These model cards were fully generated - none of this was edited except for one sentence: See [train.py](train.py) for the training script.:

tomaarsen/span-marker_bert-base-uncased-keyphrase-inspec: no evaluation during training
tomaarsen/span-marker-bert-base-uncased-bionlp: with evaluation during training

Details
Add the following sections to the README body:

Model Description (encoder model, maximum sequence length, maximum entity length, training dataset, language, license)
Model Sources (repo link, thesis link)
Model labels (a table)
Direct uses (inference, a code snippet)
Downstream use (finetuning, a code snippet)
Training set metrics (sentence length, entities per sentence)
Training hyperparameters
Training results
Environmental Impact (emissions, training time)
Training hardware
Framework versions

Beyond that, I added some values to the README metadata:

language
license
library_name
tags
datasets
metrics
widget
pipeline_tag
co2_eq_emissions
model-index (f1, recall, precision)
base_model

The values for these are automatically computed via the Trainer and a ModelCardCallback.
TODO

[ ] Tests
[ ] Additional documentation

cc: @davanstrien I think this might really interest you. Perhaps you have some useful feedback!

Tom Aarsen

Hey  @tomaarsen ,
would be awesome if the base_model metadata tag could also be written, see discussion here https://twitter.com/vanstriendaniel/status/1699102884246479214 :hugs:

That sounds like a great idea!
@davanstrien I noticed that the Model Card metadata documentation is slightly lacking: https://huggingface.co/docs/hub/model-cards. In particular, e.g. base_model and pipeline_tag (and perhaps others) are missing. Notably, this modelcard.md specification that the documentation links to seems incomplete (it also doesn't have co2_eq_emissions, etc.).

Thanks for the ping. This looks sick!! A few comments/questions/suggestions

train.py to clarify, does this come from the spanmarker library? Will this also work if someone pushes from a notebook?
Really happy to see how much metadata you are putting into the model cards automatically, super cool to see the model card attempting to find language info automatically!

Suggestion: I love all the stuff that's captured in the model card at the moment. This is probably a matter of preference, but it might also be nice to include sections that would be nice for the user to complete i.e. bias. I think it's difficult to reliably do this automatically, but it could be nice to include this in the template to give people a gentle nudge to complete this. One way of doing this if you don't want to model card to get full of empty sections would be to include these parts wrapped in comments markers that way, people are encouraged to add this information when they are editing the card, but it doesn't clutter the visible card too much.

@davanstrien I noticed that the Model Card metadata documentation is slightly lacking: huggingface.co/docs/hub/model-cards. In particular, e.g. base_model and pipeline_tag (and perhaps others) are missing. Notably, this modelcard.md specification that the documentation links to seems incomplete (it also doesn't have co2_eq_emissions, etc.).

I'll take a look at that today, IRC there are some parts of the model card which are validated hub side which might be why some of these aren't included but I'll confirm and get back to you. The base_model wasn't used regularly before but I'll make sure that gets added to the docs since I think this is a really useful piece of metadata and often very easy to generate for libraries focused on fine-tuning :)

Hello!
The train.py I uploaded manually. I don't want to automatically include the "current file" into the HF model repository, as it may contain private keys or other data that should not be uploaded. I also included the sentence linking to train.py manually.
At the moment the language(s) are just user provided, but it would actually be interesting to see if the dataset metadata contains a language.
As for the commented out sections: that's a great idea. I'll definitely throw that on my TODO.
I appreciate your insights!

Tom Aarsen

At the moment the language(s) are just user provided, but it would actually be interesting to see if the dataset metadata contains a language.

If it's a hub dataset, this would be fairly easy to do via the huggingface_hub library (you'd want to wrap this in some try/except blocks)
from huggingface_hub import dataset_info
data_info = dataset_info('imdb')
data_info.cardData.get('language')

I personally think this would be nice to try and add this information automatically, but it might be unexpected for users and wouldn't necessarily work well for all datasets i.e. ones which have multiple languages which the user is only partially using (English split of a dataset for example).
One potential option would be to surface this info to the user when they are pushing to the hub in a log message. Alternatively, this could also go into a 'suggested metadata' section that is commented out by default with an explanation of where the metadata comes from. i.e. something like
---
tags:
- chemistry
library_name: generic
# From the dataset used to train this model (https://huggingface.co/datasets/imdb)
# we have inferred the following languages as being likely for your model
# If this is correct, uncomment the lines below.
#language
# - en
---

Do you know if it's possible to have comments in the metadata section of a Hub README?
I'll definitely try to look into the automated language info. Even if the automatically generated language is incomplete (e.g. it finds English but the model is trained for English, French and German), then that's already better than not having a language whatsoever.

Tom Aarsen

Do you know if it's possible to have comments in the metadata section of a Hub README? I'll definitely try to look into the automated language info. Even if the automatically generated language is incomplete (e.g. it finds English but the model is trained for English, French and German), then that's already better than not having a language whatsoever.

Yes (https://huggingface.co/davanstrien/test/raw/main/README.md), it's a bit ugly when you look at the raw version but the rendered version just doesn't display those parts.

That repo is private, but I believe you on your word 😄

That repo is private, but I believe you on your word 😄

sorry just made it public!

That looks totally fine I think, I'll try to see if I can easily do something like that - I'm not sure as I currently use some yaml dumping utils for it.

The language is now automatically inferred from the metadata from the dataset, when possible.
Beyond that, I've added commented-out sections to the model card template in 179971f9703de0bb54fd8a1263c07b30866e2684. @davanstrien perhaps in the coming days you'll have some time to give your thoughts on the sections? They're inspired by this docs page, but they're not a 1 to 1 copy.
I think this PR is starting to be feature complete - just the docs and tests remain.

Tom Aarsen

Note to self: Ensure that the model card is created correctly on push_to_hub.

I focused mainly on the model card template so didn't really review the code at all.

That's no worries! I meant the template, so this is perfect :)

authors: davanstrien stefan-it tomaarsen

repo: tomaarsen/SpanMarkerNER


text:
Seems the widget is not updated when orderby changes

Hi
I'm trying to sort List by using sortable: true option of TableColumn
And it seems the widget is showing previous datas.
I have to manually press refresh button to update my rows.
Your demo page seems to have a same problem. sorting by Gender column does not work at all until pressing refresh.

I've figured out that in controller.dart:390 is the main problem
If I comment  the whole block below  and put
_currentDataset.clear();
_currentDataset.addAll(items);

it seems to be working.

I've figured out that in controller.dart:390 is the main problem
If I comment the whole block below and put
_currentDataset.clear();
_currentDataset.addAll(items);

it seems to be working.

Did you remove the else if statements too?

Closed as fixed in 2.0.1

authors: LAMXkE tomasweigenast

repo: tomasweigenast/paged-datatable


text:
amavisd-new stops in the container

I noted for the second time this behavior.

Start the container.
It works about half an hour.
Then he ceases to receive and send emails.

See full_syslog.txt

Well, it is the same mistake that here - https://github.com/tomav/docker-mailserver/issues/6

repo: tomav/docker-mailserver


text:
ol3 API is minified

When I need to do something using underlying ol3 api I can't use the real methods and properties names. I need to acces it by its minified name.
By example in drawend event of Draw object I need to do something linke this.
draw.on('drawend', function (e) {
   var array = e.feature.U.geometry.B;
    console.log('Shape', array);
});

I want to access without the U nor the B and using its real API names

Hi,
try to use ol-debug.js instead of ol.js
Bye, BizioX

@BizioX ol-debug.js is toooooooooo big :( :-1:

authors: BizioX lautarobock


text:
Mousemove stops hideAnimation

Steps to reproduce:

Use a hideAnimation with movement such as fadeOutDown
Toggle a toast
Click to dismiss it
Move mouse out of toast then quickly move into it again
The hideAnimation is interrupted and return to the origin position.

@KochiyaOcean  Thank you for the query. I believe this is an intended feature and can be very helpful.
Explanation: If the user has hovered over a toast notification, then we can assume that the user is currently reviewing it. If the user clicks on it by mistake or hovers "in and out" which would close the toast notification, by focusing (hovering in) on the toast, it will not be closed until the user has completely moved out from it.

I got it. Thanks for your explanation :-)

authors: KochiyaOcean RangarajKaushikSundar

repo: tomchentw/react-toastr


text:
Can't view profile results in latest Chrome dev tools

In latest Chrome (36.0.1985.125 for me, OS X version) I can't see anything after loading the cpuprofile file generated from example/cpu-blocking.js. The Chart is all empty, and Tree and Heavy reports seems to show not all the information.
Looks like the format of cpuprofile has changed in previous months or chrome-cpu-profiler doesn't fill all the needed fields.

Facing the same issue.
Chrome: 39.0.2171.95 (64-bit)
Node: 0.10.35

authors: bardt tusharmath

repo: tomgco/chrome-cpu-profiler


text:
Handle digit separator _

Since Go 1.13, numbers may contain _ between digits.
It would be nice that when a number is added to the ignored list, all its variants containing _ be ignored as well: --ignored-numbers 1000 should match 1000 and  1_000.
Currently, we have to specify --ignored-numbers 1000,1_000 manually.

@benoitmasson
Thanks for the report. I'll check this asap.

Hi @benoitmasson,
I have made two attempts to implement this so far. Because I got your point. But the documentation isn't really helpful here and I also couldn't find a package which does a conversion of the numbers. Because for numbers like 1000 it seems easy on the first glance. But for example you can write 1234567890 like 1234_567_890 or 1_234_567_890 but also like this 1_234_567_89_0.  So I need to generate this combinations in a general way to find every time all occurrences when you give me for example the number 1234567890 to ignore.
But unfortunately I couldn't find one so far. So that I'm stuck here and didn't have the time to go further here.

Generating all combinations can be done, as follows using recursion for example:
func generateNumberVariants(num int) []string {
	sign := ""
	if num < 0 {
		sign = "-"
		num = -num
	}

	return generateNumberVariantsAux(strconv.Itoa(num), sign)
}

func generateNumberVariantsAux(num, prefix string) []string {
	if len(num) <= 1 {
		return []string{prefix + num}
	}

	variantsWithoutUnderscore := generateNumberVariantsAux(num[1:], prefix+num[0:1])
	variantsWithUnderscore := generateNumberVariantsAux(num[1:], prefix+num[0:1]+"_")
	return append(
		variantsWithoutUnderscore,
		variantsWithUnderscore...,
	)
}

but it is computationally heavy, and generates a lot of elements if the input number is large (2^log(n) variants for n).
Wouldn't it be easier to do the opposite, i.e., for every magic number found in the code, normalize it by removing _'s and check whether this normalized form should be ignored?

@benoitmasson That's totally right. Thanks for putting me on the right path! It's so easy sometimes 🙈

authors: benoitmasson tommy-muehle

repo: tommy-muehle/go-mnd


text:
Ensure consistent code conventions

Consistent code conventions became an issue in PR #128.
In reaction to that @hnrkdmsk proposed in https://github.com/tomorrow-one/transactional-outbox/issues/128#issuecomment-1609784199:

I'd like to have an editorconfig here so we won't have the issue in the future.

This should be a good thing to define conventions, and IDEs should take this into account.
For checks as part of the CI pipeline we could also try https://megalinter.io/latest/

For now sufficiently done with #133

repo: tomorrow-one/transactional-outbox


text:
some key orders when pressed fast would activate unpressed keys

For eg:-
If I press 'ser' fast enough it would write as 'serf' and if i write 'esa' fast enough I will get 'esqa'
For reference: I am using qmk with Arch Linux
you can see my config here
Ask me if you need anything else
Thanks for the Hard work!!

Hey! Did you customize the qmk code? I have some code to avoid ghosting, snd I think specifically those combnations Are part of it... does it happen if you Flash my example firmware?

Checkout the qmk fork mentioned in the firmware guide, there is some code there to avoid this I think

I used the same firmware mentioned in the firmware.md guide and customized the keymaps.
Also using your keymap this still happens , for eg if I type 'rfp' it will type 'rfpt'
I am using cheapino v1 btw
Just to be sure I followed this firmware.md guide
https://github.com/tompi/cheapino/blob/master/doc/firmware.md and created my own keymap after that which was linked above

Okay, i assumed you were using v2.  I will make an attempt to backport a fw fix for this for v1 during the weekend.

Hey, sorry for the delay, could you check if this fixes your problem: https://github.com/tompi/qmk_firmware/commit/bad9c6d3639c779faf75e22ab21cb0a0988282e3
If so I will update the vial fw and the v1 releases

Yes this fixes it thank you very much

authors: Clutchnp tompi

repo: tompi/cheapino


text:
feat: add INNERNET_NAT_EXCLUDE_CIDR env variable

This comes in handy when nodes can talk to each other on multiple interfaces (eg. in a k8s cluster), but only some of them should be used for the innernet connection.

Hey @pzeinlinger sorry for the AFKness. I'm in favor of something like this, but I think it fits better as a command-line argument since that's where similar features sit. Also, we have the ipnetwork crate for CIDR math, so no need to import netaddr2 here :).

This was just an idea and meant as a quick workaround.
I don't work with Rust regularly, so if you want to implement this as a command-line argument (possibly allowing to exclude multiple cidrs from consideration) please go for it. :-)

authors: mcginty pzeinlinger

repo: tonarino/innernet


text:
Add CLI parameters for disable/enable peer

Adds a --name parameter and --yes parameter to the enable-peer/disable-peer. Helps with large network management where we need some automated tools to disable/enable peers.
Fixes tonarino/innernet#214.

Overall this looks good, thanks for the PR! I think this is good to merge after following up on the clarifying questions.

Thanks for the quick review - I have been travelogn but will take a look and address the comments next week.

Overall this looks good, thanks for the PR! I think this is good to merge after following up on the clarifying questions.

Sorry for leaving this dangling for so long! I just lost of track of it. Updated as per the comments.

Good to test. For reference - we have been running this patch for the last
6 months or so on our innernet instance!
(Deployed it and kind of forgot that it was a patch we added, so forgot
about this PR! Sorry!)
On Wed, 13 Dec 2023 at 14:25, Matěj Laitl @.***> wrote:

@.**** approved this pull request.
Looking great, thanks, @linuskendall https://github.com/linuskendall!
Good to merge, though would be good to test in real life first.
Want anyone in the community to quickly test? Ideally both the server and
client variants of the new options.
—
Reply to this email directly, view it on GitHub
https://github.com/tonarino/innernet/pull/248#pullrequestreview-1779136082,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHOYROX4RHY7452AZCR5F3YJFUPPAVCNFSM6AAAAAAVEWY426VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONZZGEZTMMBYGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

@strohel Taking you attention back to this. I think this could me deemed as tested. Can we merge?

Thanks for ping, I've hit the merge button. Thanks everybody contributing ans sorry for the delays!

authors: goodhoko linuskendall strohel


text:
No backup codes!!!

Hi, why my backup code is not showing??

can anyone help ME ??? I have this information about my wallets too,
"auth": {
"kind": "keychain",
"keychainStoreKey": xxxxxx
},
I can't send,
Error =>
index.js:2 00:20:20.592 › TypeError: Cannot read properties of null (reading 'split')
at b (index.js:5814:42446)
at async g (index.js:5814:41672)
at async m (index.js:5814:41463)
at async Object.mutationFn (index.js:5509:71462)

@mk0xf1e6e It sounds like your query would be best dealt with by the support team.
I'll have to refer you to the official support live chat with the ticket ID TON07638
Please see the link below to our dedicated support line:
Tonkeeper Network DApps Support Ticket Requests.
Click on the live chat icon at the bottom corner of the page to initiate chat with a live support agent.

all passwords are stored in the Windows Credentials?? is there any other backups?

repo: tonkeeper/tonkeeper-web


text:
x509 not working because of a bug in zeep

authentication.rst mentions that x509 requests are not working because of a bug in zeep. Can you elaborate on what that bug is?

The exact bug you raised https://github.com/mvantellingen/python-zeep/issues/788 I did start reading through the Zeep code and saw some (yet unreleased) fixes in master to some of the codebase, but it doesn't fix the entire issue.

Also, thanks for raising the issue. This is an early version of this integration with Workday, so I'm very keen for any feedback!

Added a test to demonstrate issue https://github.com/tonybaloney/workday/blob/master/tests/unit/test_auth.py#L115
(env) ➜  workday git:(master) ✗ tox -e py36
GLOB sdist-make: /Users/anthonyshaw/repo/workday/setup.py
py36 recreate: /Users/anthonyshaw/repo/workday/.tox/py36
py36 installdeps: -r/Users/anthonyshaw/repo/workday/requirements_dev.txt, zeep[xmlsec]
py36 inst: /Users/anthonyshaw/repo/workday/.tox/dist/workday-0.4.0.zip
py36 installed: alabaster==0.7.11,appdirs==1.4.3,atomicwrites==1.1.5,attrs==18.1.0,Babel==2.6.0,cached-property==1.4.3,certifi==2018.4.16,chardet==3.0.4,coverage==4.5.1,defusedxml==0.5.0,docutils==0.14,idna==2.7,imagesize==1.0.0,isodate==0.6.0,Jinja2==2.10,lxml==4.2.3,MarkupSafe==1.0,more-itertools==4.2.0,packaging==17.1,pkgconfig==1.3.1,pkginfo==1.4.2,pluggy==0.6.0,py==1.5.4,Pygments==2.2.0,pyparsing==2.2.0,pytest==3.6.3,pytest-cov==2.5.1,pytest-mock==1.10.0,pytz==2018.5,requests==2.19.1,requests-staticmock==1.4.0,requests-toolbelt==0.8.0,six==1.11.0,snowballstemmer==1.2.1,Sphinx==1.7.6,sphinxcontrib-websupport==1.1.0,tox==3.1.2,tqdm==4.23.4,twine==1.11.0,urllib3==1.23,virtualenv==16.0.0,workday==0.2.0,xmlsec==1.3.3,zeep==3.0.0
py36 runtests: PYTHONHASHSEED='3206515548'
py36 runtests: commands[0] | pytest
===================================================== test session starts =====================================================
platform darwin -- Python 3.6.6, pytest-3.6.3, py-1.5.4, pluggy-0.6.0
rootdir: /Users/anthonyshaw/repo/workday, inifile: setup.cfg
plugins: mock-1.10.0, cov-2.5.1
collected 27 items

tests/unit/test_auth.py ..F                                                                                [ 11%]
tests/unit/test_client.py ................                                                                              [ 70%]
tests/unit/test_soap.py ........                                                                                [100%]Coverage.py warning: No data was collected. (no-data-collected)

========================================================== FAILURES ===========================================================
__________________________________________ test_wssec_signature_cred_authentication ___________________________________________
tests/unit/test_auth.py:144: in test_wssec_signature_cred_authentication
    assert isinstance(
.tox/py36/lib/python3.6/site-packages/workday/soap.py:151: in call_soap_method
    result = getattr(self._client.service, attr)(*args, **kwargs)
.tox/py36/lib/python3.6/site-packages/zeep/proxy.py:42: in __call__
    self._op_name, args, kwargs)
.tox/py36/lib/python3.6/site-packages/zeep/wsdl/bindings/soap.py:115: in send
    options=options)
.tox/py36/lib/python3.6/site-packages/zeep/wsdl/bindings/soap.py:89: in _create
    envelope, http_headers = client.wsse.apply(envelope, http_headers)
E   AttributeError: 'list' object has no attribute 'apply'

---------- coverage: platform darwin, python 3.6.6-final-0 -----------
Name                        Stmts   Miss  Cover
-----------------------------------------------
src/workday/__init__.py         4      4     0%
src/workday/auth.py            17     17     0%
src/workday/client.py          33     33     0%
src/workday/exceptions.py       7      7     0%
src/workday/soap.py            52     52     0%
-----------------------------------------------
TOTAL                         113    113     0%

============================================= 1 failed, 26 passed in 0.30 seconds =============================================
ERROR: InvocationError for command '/Users/anthonyshaw/repo/workday/.tox/py36/bin/pytest' (exited with code 1)
___________________________________________________________ summary ___________________________________________________________
ERROR:   py36: commands failed

See related branch and PR https://github.com/tonybaloney/python-zeep/tree/xmlsec_signed_cred

I pulled the changes from the zeep master and am now getting "Invalid username or password" both with and without your proposed changes. The UsernameToken only has a username in it.

The July 28th, 2018 release of Zeep 3.1.0 pulled in the wsse fixes and that resolved a few of the issues.
A few more notes:
For session.verify, that needs to point to the public keychain. including the root CA certificates as well as the public key that is given to Workday.
For the wsse signature, it needs to be
 signature = Signature('privatekey.pem','public_key_only.pem')
In other words, the public key in the signature ONLY, no root CAs.
Now the fun part.
We had to edit zeep/wsse/signature.py

Comment out the Security Token reference:

    #sec_token_ref = etree.SubElement(
    #    key_info, QName(ns.WSSE, 'SecurityTokenReference'))
    #sec_token_ref.append(x509_data)

Change:

    # Add reference to signature with URI attribute pointing to that ID.
    ref = xmlsec.template.add_reference(
        signature, xmlsec.Transform.SHA1, uri='#' + node_id)

To:
    ref = xmlsec.template.add_reference(
        signature, xmlsec.Transform.SHA256, uri='')

Change:
    # This is an XML normalization transform which will be performed on the
    # target node contents before signing. This ensures that changes to
    # irrelevant whitespace, attribute ordering, etc won't invalidate the
    # signature.
    xmlsec.template.add_transform(ref, xmlsec.Transform.EXCL_C14N)

To:
    # This is an XML normalization transform which will be performed on the
    # target node contents before signing. This ensures that changes to
    # irrelevant whitespace, attribute ordering, etc won't invalidate the
    # signature.
    xmlsec.template.add_transform(ref, xmlsec.Transform.ENVELOPED)

You could also comment out the timestamp security node. Workday does not seem to care about it.
There is a C# example on Workday community which shows a little bit about how they build the headers.

repo: tonybaloney/workday

license: ISC


text:
[BUG] ... Edit button not working

The edit button is not working after installing toolkits on safari.

Clicking on the Budget
Clicking on the edit button
Receive error message

I should have been able to edit the transaction.

MacBooks (Safari)

Same here

This will be fixed in v3.4.0. We're currently waiting on Apple to approve the new version of the extension. Should be soon!

authors: Aysha1970 Lichtbringer williammck

repo: toolkit-for-ynab/toolkit-for-ynab


text:
Subpackages do not import properly in a clone outside of GOPATH

I have a program with import path go.company.com/team/prog1 which has "subpackages" inside it, committed to its repo as e.g. go.company.com/team/prog1/helper. I've vendored all the external (other repo) dependencies with godep. Now I'm trying to automate the build and test steps in an external build tool, and prog1's dependencies on its own subpackages cause build failures when cloned outside of a GOPATH:
$ unset GOPATH # only here for clarity, it was actually never set in the first place
$ git clone go.company.com/team/prog1
$ cd prog1
$ godep go test ./...

sourcefile.go:13:2: cannot find package "go.company.com/team/prog1/helper" in any of:
    /usr/lib/golang/src/pkg/go.company.com/team/prog1/helper (from $GOROOT)
    /home/user/prog1/Godeps/_workspace/src/go.company.com/team/prog1/helper (from $GOPATH)
...repeated likewise for all the other subpackage imports...

I've worked around this issue by instead doing this:
$ mkdir -p src/go.company.com/team
$ git clone go.company.com/team/prog1 src/go.company.com/team/prog1
$ export GOPATH=`pwd`
$ cd src/go.company.com/team/prog1
$ godep go test ./...

But this feels like an unneccessary and unintended complication, especially since the exact same script works perfectly fine for a program that does not import any of its own "subpackages".
Before godep was around (near Go 1.0), the build script I wrote for another project to do similar things would copy the repo's contents inside the vendoring workspace, which allowed these "subpackages" to work. That doesn't seem reasonable for godep though, since committing your workspace is the intended use case...

godep relies on the go tool and by extension things being compatible with a $GOPATH workspace layout.

authors: encryptio freeformz

repo: tools/godep


text:
Catalog Error: Table with name file_metadata does not exist!

Also had challenges with forming the graph: https://motherduck.com/blog/streamlining-ai-agents-duckdb-rag-solutions/

Hey, seems you don't have the dataset needed to load this example notebook on your local.
What was the issue with the graph generation?

repo: topoteretes/cognee


text:
chore: migrate Slider to MUIv5

FX-2674
Description
Describe the changes and motivations for the pull request.
How to test

FIXME: Add the steps describing how to verify your changes

Screenshots

Before.
After.

Insert screenshots or screen recordings
Insert screenshots or screen recordings

Review

[x] Read CONTRIBUTING.md and Component API principles
[x] Annotate all props in component with documentation
[x] Create examples for component
[ ] Ensure that deployed demo has expected results and good examples
[ ] Ensure that tests pass by running yarn test
[ ] Ensure that visuals tests pass by running yarn test:visual. If not - check the documentation how to fix visual tests
[ ] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.

PR commands

List of available commands:

@toptal-bot run all - Run whole pipeline
@toptal-bot run build - Check build
@toptal-bot run visual - Run visual tests
@toptal-bot run deploy:documentation - Deploy documentation
@toptal-bot run package:alpha-release - Release alpha version

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

According to our workload and planning, we will do the migration at another time. So, I'm closing this PR.

authors: OleksandrNechai ozgurkececioglu

repo: toptal/picasso


text:
[FX-5057] Migrate ButtonAction to TailwindCSS

FX-5057
Description
We are going to migrate ButtonAction to TailwindCSS and new MUI Base Button
How to test

Temploy

Development checks

N/A Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[x] Annotate all props in component with documentation
N/A Create examples for component
[x] Ensure that deployed demo has expected results and good examples
[x] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[x] Covered with tests (visual tests included)

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

Cool to see Toptal on https://twitter.com/Base_UI

authors: dmaklygin oliviertassinari


text:
docs(suspensive.org): add docs for @suspensive/jotai

Overview
PR Checklist

[x] I did below actions if need

I read the Contributing Guide
I added documents and tests.

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

Project coverage is 84.23%. Comparing base (02f0aec) to head (46d101b).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1084   +/-   ##
=======================================
  Coverage   84.23%   84.23%
=======================================
  Files          54       54
  Lines         552      552
  Branches      117      117
=======================================
  Hits          465      465
  Misses         82       82
  Partials        5        5

Components
Coverage Δ

@suspensive/react
97.05% <ø> (ø)

@suspensive/react-query
75.28% <ø> (ø)

@suspensive/react-query-4
0.00% <ø> (ø)

@suspensive/react-query-5
0.00% <ø> (ø)

@suspensive/jotai
0.00% <ø> (ø)

@suspensive/cache
100.00% <ø> (ø)

@suspensive/react-await
100.00% <ø> (ø)

@suspensive/react-image
80.39% <ø> (ø)

I wrote the component example with reference to the examples in the Jotai official documentation to make it easy for Jotai users to understand quickly.
I verified that the example works in my local environment!

@manudeli  Thank you for the quick feedback! https://github.com/toss/suspensive/pull/1084/commits/d9943f05d11035bd9cfc85b2f6ba16552aa96672

@manudeli Thank you! the documentation has become even better!

@manudeli I have reviewed everything once more!

@manudeli @SEOKKAMONI Could you provide feedback on the updated documentation? I'm curious if you have any better ideas for the examples.

authors: codecov-commenter gwansikk

repo: toss/suspensive


text:
Strange start/end times

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
In my list of events, the start and end times have some numbers I cant quite decipher.
Example is an event tomorrow which starts at 10:45 and ends at 11:45. The card shows: 2410-2411
I am running with 24h on v. 6.1.2
Expected behaviour
The start time should show 10:45 and the end time should show 11:45
Steps To Reproduce
Its consistent
Card Version
6.1.2
Home Assistant Version
core-2021.12.4
Configuration
type: custom:atomic-calendar-revive
europeanDate: true
hoursFormat: 24h
entities:
  - entity: calendar.torben_kalender
    color: orange
  - entity: calendar.louise
    color: green
    blacklist: tømmes
  - entity: calendar.men_with_manners
    color: red
timeColor: grey
dimFinishedEvents: true
calShowDescription: false
disableCalLink: false
showProgressBar: false
showCurrentEventLine: false
showMonth: false
showWeekDay: true
showFullDayProgress: true
showEventIcon: false
showHiddenText: false
fullDayEventText: Hele dagen
untilText: Indtil
noEventsForNextDaysText: Ingen aftaler
noEventText: Ingen aftaler
firstDayOfWeek: 1
maxDaysToShow: 7
refreshInterval: 60
dateFormat: LL
showRelativeTime: true
hideFinishedEvents: false
showDeclined: false
showLoader: true
showDate: true
sortByStartTime: false
showHours: true
showNoEventDays: true
hideDuplicates: true

Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

please remove below from your config, this setting was changes a few versions back due to changes in HA
hoursFormat: 24h

Oh sorry, i must have missed that. Thanks a lot

repo: totaldebug/atomic-calendar-revive


text:
🛑 Tracky Website is down

In 119754c, Tracky Website (https://tracky.louiscreates.com) was down:

HTTP code: 0
Response time: 0 ms

Is this on purpose or not @tototomate123? I thought you messed something up but the react favicon is making me skeptical of that stance.

This is on purpose


text:
Added "Point" and "UUID"

PointField will be serialized to { "x": value, "y": value }
Test case is provided.
UUIDField will be serialized with UUID, which is provided by marshmallow itself.

Currently does not support 3D point since I only get coordinates[0] and coordinates[1] from PointField.

Awesome !

What is the reasoning behind serializing PointField to { "x": value, "y": value }? It seems to me that PointField should be serialized to a GeoJSON point object.

@etrochim no real reason, I think it was from @kevin0571 needs
However you can use register_field to overload the mashmallow field to use by a subclass of PointField wuth a custom serialization.

The reason I'm asking is because I just discovered this project and thought it might be useful for another project I am working on where I need to serialize mongoengine documents with MultiPolygon and Polygon fields. After finding that those field types were not implemented I decided to implemented them myself but quickly ran into this strange serialization format.
In my mind the default Point, Polygon, LineString, etc fields should serialize to a standard geo interchange format; GeoJSON in particular. If the user requires something different they should go down the subclassing route you suggested.
In the interest of implementing these unimplemented fields, what should the other geo field serialization formats look like? Using a non-standard format built off the current PointField would be useful to few users (because few other tools would recognize that format). However, changing PointField to emit GeoJSON has backward compatibility concerns. I'm curious about what your thoughts are?

@etrochim the reason I implemented this with x, y instead of longitude, latitude is that I think Point in this case is a generic point instead of a geo point, that is why it's described with a coordinate(x, y). Correct me if I'm wrong.

authors: etrochim kevin0571 touilleMan

repo: touilleMan/marshmallow-mongoengine


text:
Google Doodle search header backgrounds get replaced instead of the actual Google logo

Behaviour
What the header looks like normally:

What Old Google does to it:

What it should look like:

Cause
The code run on the /search page is as follows:
var img = document.getElementsByTagName("img")[0];
img.src = replacementurl;

Simply, the header background is the first <img> element on the page (index 0), whilst on non-Google-Doodle days, the Google logo is at index 0.

As of commit 77ea131, the header background no longer displays and the Google logo is replaced properly. I am yet to test how this works with doodles which don't have header backgrounds, but I think it should be okay (given that I used a boilerplate-ish system and Google usually doesn't reuse classes in that way (I THINK I THINK)).

repo: toydotgame/old-google


text:
Cards API metoda securesale zwraca nieprawidłowy kod HTTP 602

Serwer TPay zwraca nieprawidłowy kod odpowiedzi HTTP 602 w przypadku udanej transakcji securesale.

Witaj @pszalko
Kod zwracany przez serwer Tpay jest niepoprawny i rozwiązanie problemu leży raczej po stronie aplikacji Tpay, a nie biblioteki.
Kod 602 jest w zasadzie nieprawidłowym kodem HTTP, więc dostosowanie biblioteki do takiego kodu mija się z celem.
Niebawem błąd zostanie usunięty po stronie aplikacji.
Dziękujemy za zgłoszenie.

authors: piotrjozwiak pszalko

repo: tpay-com/tpay-php


text:
Nav should indicate if you are not on mainnet

It can be confusing to know what network you're on. The nav should tell you what network you are on if you are not on mainnet.

Handled via merge of #248

authors: mycodecrafting obstropolos

repo: tqtezos/minter


text:
Update google-auth-library-credentials to 1.10.0

Updates com.google.auth:google-auth-library-credentials from 1.8.1 to 1.10.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.auth", artifactId = "google-auth-library-credentials" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.google.auth", artifactId = "google-auth-library-credentials" }
}]

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

Superseded by #197.


text:
Update trace4cats-context-utils, ... to 0.14.4

About this PR
📦 Updates

io.janstenpickle:trace4cats-context-utils
io.janstenpickle:trace4cats-core
io.janstenpickle:trace4cats-kernel
io.janstenpickle:trace4cats-testkit

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

🔍 Files still referring to the old version number
The following files still refer to the old version number (0.14.0).
You might want to review and update them manually.
README.md

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

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

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

Superseded by #261.


text:
[Fix] Prevent various OOB accesses and discontiguous buffer bugs

Pull Request Template
Checklist

[x] Confirmed that run-checks all script has been executed.
[x] Made sure the book is up to date with changes in this PR.

Changes
Fixes OOB bugs in the reduce kernels and col2im.
Fixes bugs when running scalar ops with discontiguous buffers (buffers with empty memory regions created by things like slice) by making them contiguous first (this is almost always faster than strided access).
Testing
All tests now pass with no weird values or segfaults

Windows are failing with a file permission error, seems like a CI bug

repo: tracel-ai/burn


text:
ABOAT-1480 - User authentication for Neb 2

Fixes # .
https://traderinteractive.atlassian.net/browse/ABOAT-1480
What does this PR do?
Added refresh flag to allow cache to be refreshed. This was needed for new PHP library to be able to update refresh tokens on the fly.
Checklist

[  ] Pull request contains a clear definition of changes
[  ] Tests (either unit, integration, or acceptance) written and passing
[  ] Relevant documentation produced and/or updated

@chrisryan as per your previous comment, i did notice the refresh flag on the constructor. i dont think this would work but i could be wrong. ill need to look into it a bit more before releasing. the issue i was having was needing to update a redis key/value pair on the fly while still being able to use the cache to fetch cached data. i think based on your comment, it would not allow me to fetch from the cache and it would update the cache on every call.

@chrisryan Hi. can you confirm if we are able to release this ? I dont think the refresh flag in the constructor will work as we are relying on an existing object instantiation of the memoize class from within Application.php i believe. we need to be able to refresh with the existing instance of memoize.

I don't see any problems with the changes. We will be tagging it as a major change since it changes the Interface methods.

authors: chrisryan williamhuntjr

repo: traderinteractive/memoize-php