Ignore unsupported tests

Addition of functionality for ignoring unsupported methods. This allows to parse supported data from files that may also contain data from unsupported method type. The unsupported method types are now ignored by the parsing function.

I closed this PR since we will not let unsupported methods pass silently.


Failed to launch external program tsc -p ts spawn tsc ENOENT

TypeScript Version:  2.4.1
VSCode Version: 1.3.1
OS Version: Linux Mint 18.1
node: 6.11.0
npm: 5.0.3
Steps to Reproduce:
Open VSCode insider from Nemo or from terminal
Try to run build task Ctrl+Shift+B (using tsc -p in task.json 'args')
tasks.json
{
    "version": "0.1.0",
    "command": "tsc",
    "isShellCommand": true,
    "args": ["-p", "ts"],
    "showOutput": "silent",
    "problemMatcher": "$tsc"
}

helloworld.ts in ts folder
class HelloWorld {

}

tsconfig.json
{
    "compilerOptions": {
        "target": "es5",
        "outFile": "../js/app.js",
        "sourceMap": true
    }
}

Empty folder js
Expected behavior:
I believe that this setup should compile the typescrpt files in my project to js files and put them in my js folder.
Actual behavior:
When I hit Ctrl+Shift+B i get:
Failed to launch external program tsc -p ts.
spawn tsc ENOENT
in the output window
Its worth noting that if I run the tsc command with args in the VSCode terminal window in my projects root directory (tsc -p ts) it DOES seem to work fine(I get the expected behavior).
This issues is similar to #8434
I do have custom code in my .profile and .bashrc, but in my case it does not matter if I open from the terminal or from the file explorer
running process.env.PATH in VScode dev tools console outputs the same path as running echo $PATH from my command line.

This is normally caused by tsc not being in your $PATH. Double-check if your global node_modules/.bin is reachable outside of bash, e.g. via running which tsc in sh/zsh.
While not a fix for your issue per se, consider running the local tsc in your project:
{
    "version": "0.1.0",
    "command": "node",
    "isShellCommand": true,
    "args": ["${workSpaceRoot}/node_modules/typescript/bin/tsc", "-p", "ts"],
    "showOutput": "silent",
    "problemMatcher": "$tsc"
}

This allows you to track the version of the TypeScript compiler you want to use in your package.json independent of the globally installed one (npm i -g typescript).

This worked.
Installed typescript locally with npm i typescript and changed tasks.json to
{
    "version": "0.1.0",
    "command": "node",
    "isShellCommand": true,
    "args": ["{pathToWorkSpaceRoot}/node_modules/typescript/bin/tsc", "-p", "ts"],
    "showOutput": "silent",
    "problemMatcher": "$tsc"
}

Which as you say "allows you to track the version of the TypeScript compiler you want to use in your package.json independent of the globally installed one". Seems like a good idea.
I think it is worth noting that: ~/.npm-global/bin is in my path which contains the tsc package. This should be equivalent to node_modules/.bin I believe.
Regardless, running 'sh' and 'tcs' I get 'tsc: not found'.

@kylerdanielster glad it solves your issue!
Just a little addendum, since you wrote "{pathToWorkSpaceRoot}", I assume you hard-coded the path - you can use the actual string ${workSpaceRoot} so it works regardless of where you checkout the project.

@leonadler "${workSpaceRoot}/node_modules/typescript/bin/tsc" did not work for me, that is why I hard coded it.
I played around with the path some more and "node_modules/typescript/bin/tsc" this did the trick as well.
Current task.json (in .vscode folder)
{
    "version": "0.1.0",
    "command": "node",
    "isShellCommand": true,
    "args": ["node_modules/typescript/bin/tsc", "-p", "ts"],
    "showOutput": "silent",
    "problemMatcher": "$tsc"
}

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


Build an aside system

Do authors just put <aside> tags in their articles?
Do we have a separate syntax for asides?
Do we have a separate interface for asides?

Rever to #458 closed issue for more context.
https://github.com/crimethinc/website/issues/458


Use syscall for modprobe

This implement a syscall based modprobe rather than calling out to a modprobe binary.
I have found during building my own non-overlay/takeover images for Raspberry Pi, that I needed to download busybox as /sbin/modprobe in order to boot properly.
With this patch, it is able to boot without the extra busybox binary.

I completely forgot about dependencies on modules, and probably should have searched for existing packages before writing my own code.
I've updated my branch to use that package, and just booted successfully with the 9p module as a test.


Says can not login into Mint

If I do it manually or automatically I can not get the program to think it is logged into mint. on MacBook Pro. have tried two now. abbe cuz I have multiple accounts?

Same here.. Did Mint make some changes?

Thank you both for the report and the screen shot. I'm taking a look now.

Good news; I was able to run the tagger without a problem. I think you both may be running old versions and it's not entirely your fault :) I haven't released a prebuilt version for windows for a while (And it looks like @alex-tomin was on windows based on the screen shot).
Can I bother either of you to attempt again, this time using HEAD (if from source), or using PyPI or rebuilt binaries with a version of 1.43.
Thanks!
Jeff

Oh... you are right, I forgot to mention the version.  Will check tonight and let you know.
Thanks for prompt look!

@jprouty I can confirm le latest works! Thanks a lot for uploading windows version and for the amazing tool 🎉

I am on mac. but honestly barley figured out how to run it from terminal.
or. no idea how to do this
(his time using HEAD (if from source), or using PyPI or rebuilt binaries
with a version of 1.43
https://github.com/jprouty/mint-amazon-tagger/releases/tag/1.43.).
looking forward to the update tho.
On Mon, Apr 18, 2022 at 6:07 PM Jeff Prouty @.***>
wrote:

Good news; I was able to run the tagger without a problem. I think you
both may be running old versions and it's not entirely your fault :) I
haven't released a prebuilt version for windows for a while (And it looks
like @alex-tomin https://github.com/alex-tomin was on windows based on
the screen shot).
Can I bother either of you to attempt again, this time using HEAD (if from
source), or using PyPI or rebuilt binaries with a version of 1.43
https://github.com/jprouty/mint-amazon-tagger/releases/tag/1.43.
Thanks!
Jeff
—
Reply to this email directly, view it on GitHub
https://github.com/jprouty/mint-amazon-tagger/issues/99#issuecomment-1101806179,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYT4LAB5JPFHZZZVP7U5XPDVFXMKFANCNFSM5TIIJCLQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks @aleto-ms and @alex-tomin for confirming it works now. @SmoothCamera, please reopen/reply if the tool continues to give you difficulty. Screen shots of any error messages/etc would be appreciated.


Suggested changes

Feel free to cherry-pick. I tried to be modular, but git being line based and the writing style of putting each paragraph on a single line makes that difficult at times.

Improve formatting, punctuation, grammar, phrasing, spelling of “Bézout”
Don't try to invert 0 in di
Add factor di to certain summands
Wrestle with GitHub's latex rendering engine

@aszepieniec, I believe you have considered the changes suggested in this PR, correct?


HuggingFaceEmbeddings takes the default model name and reinitializes self.client even while passing in client parameter.

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain_huggingface import HuggingFaceEmbeddings
from sentence_transformers import SentenceTransformer

model = SentenceTransformer(
   "all-MiniLM-L6-v2",
   backend="onnx",
   model_kwargs={
      "file_name": "model_qint8_avx512_vnni.onnx",
      "provider" : "CPUExecutionProvider"
  },
)

embeddings = HuggingFaceEmbeddings(client = model)

Error Message and Stack Trace (if applicable)
No response
Description
The HuggingFaceEmbedding class has a client parameter, which lets you pass in a pre-loaded model. But right now in the code, even after passing the client parameter, it has a DEFAULT_MODEL_NAME = "sentence-transformers/all-mpnet-base-v2" set and it starts downloading a loading that model
The reason for this is i think that
self.client = sentence_transformers.SentenceTransformer(
                self.model_name, cache_folder=self.cache_folder, **self.model_kwargs
            )

The self.client is initialized without checking if client has been passed into it or not, and regardless creates a new client loading the default model
This can be easily fixed by doing this
if not self.client: # Add a check to see if self.client has been passed, if yes no need to reinitialize the client
            self.client = sentence_transformers.SentenceTransformer(
                self.model_name, cache_folder=self.cache_folder, **self.model_kwargs
            )

Ill be happy to open a pr, fix this small issue then merge. Let me know how to proceed with this.
System Info
System Information

OS:  Darwin
OS Version:  Darwin Kernel Version 23.5.0: Wed May  1 20:19:05 PDT 2024; root:xnu-10063.121.3~5/RELEASE_ARM64_T8112
Python Version:  3.10.12 | packaged by conda-forge | (main, Jun 23 2023, 22:41:52) [Clang 15.0.7 ]

Package Information

langchain_core: 0.3.12
langchain: 0.2.0
langchain_community: 0.0.38
langsmith: 0.1.136
langchain_groq: 0.1.4
langchain_huggingface: 0.1.0
langchain_text_splitters: 0.2.0
langgraph: 0.0.50

Optional packages not installed

langserve

Other Dependencies

aiohttp: 3.9.3
aiosqlite: Installed. No version info available.
aleph-alpha-client: Installed. No version info available.
anthropic: Installed. No version info available.
arxiv: Installed. No version info available.
assemblyai: Installed. No version info available.
async-timeout: 4.0.3
atlassian-python-api: Installed. No version info available.
azure-ai-documentintelligence: Installed. No version info available.
azure-ai-formrecognizer: Installed. No version info available.
azure-ai-textanalytics: Installed. No version info available.
azure-cognitiveservices-speech: Installed. No version info available.
azure-core: Installed. No version info available.
azure-cosmos: Installed. No version info available.
azure-identity: Installed. No version info available.
azure-search-documents: Installed. No version info available.
beautifulsoup4: 4.12.3
bibtexparser: Installed. No version info available.
cassio: Installed. No version info available.
chardet: Installed. No version info available.
clarifai: Installed. No version info available.
cloudpickle: 2.2.1
cohere: Installed. No version info available.
couchbase: Installed. No version info available.
dashvector: Installed. No version info available.
databricks-vectorsearch: Installed. No version info available.
dataclasses-json: 0.6.6
datasets: Installed. No version info available.
dgml-utils: Installed. No version info available.
docarray[hnswlib]: Installed. No version info available.
elasticsearch: Installed. No version info available.
esprima: Installed. No version info available.
faiss-cpu: Installed. No version info available.
feedparser: Installed. No version info available.
fireworks-ai: Installed. No version info available.
friendli-client: Installed. No version info available.
geopandas: Installed. No version info available.
gitpython: 3.1.32
google-cloud-documentai: Installed. No version info available.
gql: Installed. No version info available.
gradientai: Installed. No version info available.
groq: 0.5.0
hdbcli: Installed. No version info available.
hologres-vector: Installed. No version info available.
html2text: Installed. No version info available.
httpx: 0.27.0
httpx-sse: Installed. No version info available.
huggingface-hub: 0.26.0
huggingface_hub: 0.26.0
javelin-sdk: Installed. No version info available.
jinja2: 3.1.2
jq: Installed. No version info available.
jsonpatch: 1.33
jsonschema: 4.17.3
langchain-openai: Installed. No version info available.
lxml: 5.2.2
manifest-ml: Installed. No version info available.
markdownify: Installed. No version info available.
motor: Installed. No version info available.
msal: Installed. No version info available.
mwparserfromhell: Installed. No version info available.
mwxml: Installed. No version info available.
newspaper3k: Installed. No version info available.
nlpcloud: Installed. No version info available.
numexpr: 2.8.4
numpy: 1.25.2
nvidia-riva-client: Installed. No version info available.
oci: Installed. No version info available.
openai: Installed. No version info available.
openapi-pydantic: Installed. No version info available.
openlm: Installed. No version info available.
oracle-ads: Installed. No version info available.
oracledb: Installed. No version info available.
orjson: 3.10.3
packaging: 23.2
pandas: 1.5.3
pdfminer-six: Installed. No version info available.
pgvector: Installed. No version info available.
praw: Installed. No version info available.
premai: Installed. No version info available.
psychicapi: Installed. No version info available.
py-trello: Installed. No version info available.
pydantic: 2.8.2
pyjwt: Installed. No version info available.
pymupdf: Installed. No version info available.
pypdf: 3.17.4
pypdfium2: Installed. No version info available.
pyspark: Installed. No version info available.
PyYAML: 6.0.1
qdrant-client: Installed. No version info available.
rank-bm25: Installed. No version info available.
rapidfuzz: Installed. No version info available.
rapidocr-onnxruntime: Installed. No version info available.
rdflib: Installed. No version info available.
requests: 2.31.0
requests-toolbelt: 1.0.0
rspace_client: Installed. No version info available.
scikit-learn: 1.3.0
sentence-transformers: 3.2.0
SQLAlchemy: 2.0.30
sqlite-vss: Installed. No version info available.
streamlit: Installed. No version info available.
sympy: 1.12
telethon: Installed. No version info available.
tenacity: 8.3.0
tidb-vector: Installed. No version info available.
tiktoken: Installed. No version info available.
timescale-vector: Installed. No version info available.
tokenizers: 0.20.1
torch: 2.0.1
tqdm: 4.65.0
transformers: 4.45.2
tree-sitter: Installed. No version info available.
tree-sitter-languages: Installed. No version info available.
typer: 0.12.5
typing-extensions: None
upstash-redis: Installed. No version info available.
uuid6: 2024.1.12
vdms: Installed. No version info available.
xata: Installed. No version info available.
xmltodict: 0.13.0

@PraNavKumAr01 i don't think client is meant to be passed -- the field is marked as private. are you able to pass all of the relevant information for your model via model_kwargs in HuggingFaceEmbeddings?

Removed client here https://github.com/langchain-ai/langchain/pull/27495 -- going to close this issue, as users shouldn't be passing client directly. Feel free to open another issue if some functionality is missing!


Typescript Support

Could you create a type definition file for it.

:tada: This issue has been resolved in version 2.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


modifed d4mac and d4win download links to stable for v1.12

Modified the Docker for Mac and Docker for Windows download links to point to stable instead of beta in preparation for v1.12 publishing of docs tomorrow.
Signed-off-by: Victoria Bialas<EMAIL_ADDRESS>

@SvenDowideit @thaJeztah @sfsmithcha please review

02:06:34 ERROR: 2016/07/28 02:06:34 content_renderer.go:56: LinkResolver: No page found for "/docker-for-mac/index.md" on page "toolbox/overview.md".
02:06:34 ERROR: 2016/07/28 02:06:34 content_renderer.go:56: LinkResolver: No page found for "/docker-for-windows/index.md" on page "toolbox/overview.md".

expected error, can ignore
LGTM

@tiborvass are you willing to cherry-pick these (and possibly a few more?)


Add flamegraph target to Makefile

This PR adds a Makefile target that runs a copy of tests/deploy_account.rs with some differences to allow for better flamegraph visualization.

Codecov Report

Merging #205 (885ee78) into main (1ca883e) will decrease coverage by 0.14%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #205      +/-   ##
==========================================
- Coverage   73.64%   73.50%   -0.14%
==========================================
  Files          42       43       +1
  Lines        2660     2665       +5
==========================================
  Hits         1959     1959
- Misses        701      706       +5

Impacted Files
Coverage Δ

bench/benches.rs
0.00% <0.00%> (ø)

src/services/api/contract_class.rs
87.65% <0.00%> (-3.38%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Web] Clicking thumbnail image to view full size resorts in blank image.

Steps to Reproduce

Clicking onto thumbnail of posted image by user.
Image expands, sometimes showing for about a second at most before going blank.

Attachments
No response
What platform(s) does this occur on?
Web (Desktop)
Device Info
PC
What version of the app are you using?
Web - Chrome Browswer
Additional Information
When trying to view an image at full size on PC Web, the image sometimes shows for about a second or less before going blank.  The page is still dimmed and the large X on the top right corner is there to click to close out and brings it back to normal.
I've tried to delete all cookies/cache, disabled plug-ins and still no go.  Could not replicate on PC at work or elsewhere.

Could ad blocker be blocking the full image? More details about which image would help.

That was my initial thought, but I had ad-blocker disabled on that site.  I've tried to toggle on/off all the same, but nothing.  I use the same exact ad blocker on work PC and it works fine there whether on/off.
It does this for every image.
![Ex1](https:

//github.com/user-attachments/assets/591c77b6-aef7-4412-b4ac-ddd1bb5f5aa5)

That's very strange! Do you see the requests for the images coming through successful in the Network tab?

Never really done much with that, but I've gone ahead and opened it up and then clicked the image again.

Can you try

Selecting Img in the filter tabs (they're a bit below and to the right of the Network header)
Refreshing the page so it loads from scratch
Opening the image again
Screenshotting then

I want to see if it's just the thumbnail that loads successfully, or both thumbnail and full-sized image.

Like this?


Periodical dd-task-scheduler errors - Unable to create StatsD client - localhost:8125

Hi there,
I'm not sure if the following errors origins from this library or datadog-lambda-java, but we're seeing a lot of dd-task-scheduler errors from Java lambdas like this:

[dd.trace 2022-04-18 22:16:49:700 +0000] [dd-task-scheduler] ERROR datadog.communication.monitor.DDAgentStatsDConnection - Unable to create StatsD client - localhost:8125

We are using version 1.4.4 of this library in combination with the latest serverless-datadog-plugin - v5.0.0 which uses the lambda layer v21 of datadog-lambda-extension and v4 of the dd-trace-java layer and java11. For instrumenting things we've followed the official DataDog documentation. The errors doesn't seem to be specific to the Serverless plugin, as we also get this errors when we integrate the layers and configs directly without the plugin. Also the log level doesn't seem to take effect, as we're seeing the logs unaggregated on INFO level, despite the fact that we have configured logLevel: warn or DD_LOG_LEVEL: WARN.
Our serverless.yml config looks like this:
...
provider:
  environment:
    DD_ENV: ${self:provider.stage}
    DD_SERVICE: ${self:service}
    DD_TRACE_SAMPLE_RATE: ${self:custom.datadogTracingSampleRate.${opt:stage, self:provider.stage}}
    DD_VERSION: ${git:sha1}
...
custom:
  datadogTracingSampleRate:
    dev: 1.0
    int: 0.8
    prod: 1.0
  ...
  datadog:
    addExtension: true
    apiKey: ${ssm:.../apiKey}
    site: datadoghq.eu

    flushMetricsToLogs: true
    addLayers: true
    logLevel: warn
    enableTags: true
    injectLogContext: true
    integrationTesting: false

Any ideas what we could do to fix the error?

Fixed with https://github.com/DataDog/datadog-lambda-java/issues/78


feat(widgets): create widget tabs

Description
Please describe the reason related to the changes:
Type
Please select the type related to the changes:

[ ] bugfix
[ ] build
[ ] continuous delivery (CD)
[ ] continuous integration (CI)
[ ] documentation
[ ] feature
[ ] formatting
[ ] refactoring
[ ] styling
[ ] other (please describe below):

Breaking Change
Does your changes introduce a breaking change?

[ ] no
[ ] yes

Behavior
Please describe the current behavior:
Changes
Please describe the new behavior:
Information (issue, links, logs, etc.)
Please list any other useful information:
Checklist
Please make sure this pull request follow the requirements

[ ] commit guidelines @commitlint/config-conventional
[ ] 100% code coverage
[ ] tests passing

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

21 of 21 (100.0%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 98.63%

Totals

Change from base Build<PHONE_NUMBER>:
0.6%

Covered Lines:
71

Relevant Lines:
71

💛  - Coveralls


Improve speed by reduce stat syscall

From @kcwu on November 27, 2010 17:46
This is idea to improve file search speed.
Currently, ack use File::Next to search files.
It will stat each entry inside directory, to determine it is file or directory.
After File::Next know the entry is normal file (via -f), it will callback to ack's file_filter.
If we assume directory name won't look like file, it is possible to speed up:

Check entry name first. If the name is recognized by ack, but isn't interested by this search, we can just ignore such entry.
Only test it's file/dir/link if the name is not recognized by ack.

However, it may accident ignore directories with name recognized by ack. So this feature may need a option to let user turn on/off.
Copied from original issue: petdance/ack#149

From @hoelzro on August 28, 2013 21:7
Migrated to ack2 queue.

From @kcwu on March 3, 2011 4:14
By the way, linux's readdir(3) return "d_type" which specify a entry is directory or not. Thus save a lot of stat(2) calls.
Linux's find(1) takes this advantage and is much faster. I don't know how to apply this trick in perl though.

This was done in v3.1.0 and v3.1.1.

I can't find relevant commits in 3.1.0 and 3.1.1. Do you mean it is done earlier?

Sorry, I didn't explain enough in the Change log.  ack v3.1.1 uses File::Next 1.18, which is more economical with stat() calls.
I've updated the Changes file: cb7578e

I see. The change is in File::Next https://github.com/petdance/file-next/commit/470900a56b635d0516a3af71370912e6e520a185.
Thanks


Use event data

Note: This PR is rebased off of #3994 which should be merged first.
This PR contains all the remaining code needed for us to run our app fully in event-sourced mode.
With this merged, the system will be in the following state:

All inventory writes are double writes (to both events and inventory items)
Reads are put behind a feature flag. By default it still reads from inventory items. With the flag on, it instead reads from events.
On write, with the flag on, it uses the event sourcing validations to determine if there are any invalid events (i.e. trying to remove more inventory than a location has). With the flag off, it uses the current approach of inspecting inventory items.

Broadly, the PR contains the following changes:

Adds the event-read feature flag, which can be set on an organization level to turn event-source reading on.
Adds a new class, View::Inventory, which acts as a repository for event-calculated inventory. Since we aren't inspecting database rows, but getting the full state of the inventory as calculated from events, we want to ensure we aren't recalculating this more than once per request.
In order to facilitate this, code has changed to calculate the inventory once (generally in a controller action) and pass it through the view and any other methods necessary.
This includes making changes to the Exportable CSV-related methods to take an inventory object - the relevant classes simply overwrite the Exportable methods to enable this.
In general, the feature flag is checked at the top level (e.g in controller) and an inventory is generated if necessary. Then lower-down code will check for the existence of the inventory object to determine whether to use it or the current methods of inspecting inventory items.

One subtle change is that there are several places in the current system that differentiate between an inventory item not existing at all, or existing with a quantity of zero. The new event-driven method doesn't differentiate between these (since semantically they are the same case).
The PR has been split up into multiple commits, where each commit accomplishes one set of changes. Hopefully this will make the review more palatable.
For specs, I've introduced a new create_inventory method that ensures that all relevant events and inventory items are double-written whenever inventory needs to be created. For specs that currently look at the state of inventory items, I've added additional checks to validate the event-sourced inventory.
Ultimately, once this has been merged, tested and fully rolled out, we will remove all reference to inventory items from the app.

@cielf as per discussion, pushed a commit that handles dealing with storage location change on update.

I've done some light testing of this and I'm ok with merging it in.   Sanity check -- @awwaiid -- did your technical review include the changes in #3994?

per @awwaiid,  all the changes from #3994  are in #4000, so it can be closed and then we can merge this.

@dorner rspec (non-browser) failed and then I re-ran it without changes and it passed, so you should take a look -- https://github.com/rubyforgood/human-essentials/actions/runs/7862418075/job/21451870283?pr=4000


Error: INTERNAL Error: Unsupported compression function type

What happens?
I am getting this error when running a simple select query on a user uploaded duckdb database file:
Error: INTERNAL Error: Unsupported compression function type
    at pa.runQuery (duckdb-browser-eh.worker.js:1:741917)
    at Lo.onMessage (duckdb-browser-eh.worker.js:1:729928)
    at Bc.globalThis.onmessage (duckdb-browser-eh.worker.js:1:763450)
@duckdb_duckdb-wasm.js?v=597ea27d:11971 Uncaught (in promise) Error: INTERNAL Error: Unsupported compression function type
    at O.onMessage (@duckdb_duckdb-wasm.js?v=597ea27d:11971:15)

To Reproduce
 await db.registerFileHandle(
            "data_file.duckdb",
            file, // user uploaded file
            2,
            true
        );

const conn = await db.connect();
await conn.query("ATTACH 'data_file.duckdb'")
const arrowResult = await conn.query('select * from data_file.AAPL')

Browser/Environment:
Chrome, Version 126.0.6478.127 (Official Build) (arm64)
Device:
MacOS
DuckDB-Wasm Version:
1.28.1-dev106.0
DuckDB-Wasm Deployment:
Local development
Full Name:
William Walker
Affiliation:
Individual

Experiencing the same, but loading the file via ATTACH 'https://.... File can be opened just fine via duckdb CLI.

I think it depends on the bundle that was used. I changed it to the Vite approach, and I haven't seen those errors anymore.

Thank you @carlopi for looking into this.
Here is the sample file:
https://file.io/PNwQR8qqgict

Hello ! I got the same behavior and the same error.
Unfortunately, I cannot share the .duckdb file
One hint from my side: if we use force_compression=uncompressed, no issue anymore. But it's not convenient for me as the file size to huge


usage in multi sql schema

In my case , there's one User Model in 2 different mysql server or schema.(A and B)
how to use  A.User.or B.User explicitly?

The model definitions for A.User and B.User are the same? If so, sequelize-typescript is not supposed to work with one class for multiple different connections. This is because sequelize uses static functions for the typical ORM-methods like find(), findAll and so on.
But you could achieve it anyway with a base class:
folder structure:
models/
   base/
      BaseUser.ts
   connection-a/
      User.ts
   connection-b/
      User.ts

class BaseUser<T> extends Model<T> {
  @Column
  email: string;
}

// connection-a/
@Table
class User extends BaseUser<User> {}

new Sequelize({modelPaths: [__dirname + '/models/connection-a']});

and same for:
// connection-b/
@Table
class User extends BaseUser<User> {}

new Sequelize({modelPaths: [__dirname + '/models/connection-b']});

Thank you very much.


Initial commit of text navigation game task.

This is the initial commit of a text-based navigation task for an ASCII game world.

Thank you for submitting your task! We are in the process of pairing submissions with task reviewers and will be in touch soon.

I am one of the two reviewers assigned to review this task.
It looks very interesting and I like the idea of testing language models' ability to solve visual games. This area is fairly new to me so I have a low confidence level for my reviews.

Concreteness This task is concrete and code is well-written.
Specificity This task capture the LM's ability to solve visual games where the visual information is contained in text format.
Throughness As the author mentioned, playing randomly can solve some instances of the game with a small width. This could be a concern of the robustness of this task. Could you comment on how likely randomly choosing a direction could solve this task, say, with width 5 and interaction limited to 30.
Difficulty This task seems to be very difficult for language models. The author mentioned humans can solve this task instantaneously. But this should be conditional on the text being displayed in the right way. For example, if the text is printed in one line with new line replaced by <eos>, humans will also find it very difficult. I think this could be one reason that prevents the language model from solving this task: LM processes information in one line. Maybe explicitly instruct the LM to reason visually and interpret  <eos> as a new line operation could help?
Not solvable by memorizing the Internet Yes.
Novelty This task is novel. But there is a similar task #103. I would suggest collaboration to avoid duplicates and improve this ASCII Maze task together.
Justification The README.md provides clear and concise justification.

My major concern is the instruction and interaction part. I use examples to illustrate my concern but the author need not modify the instruction and interaction exactly as I mentioned. And because this area is relatively new to me, I am happy to change my opinion if the author could justify the way instruction and interaction are defined.

I am worried that the LM does not understand it's playing a game and could be doing something like writing a paragraph. Maybe You are playing a game with feedbacks from the environment. could be added at the beginning of the instruction to make it clear.
the instruction is "You enter a wicked maze.  As the (P)layer, you seek the (m)cguffin.  Find it, if you dare." I would prefer to put it in a more straightforward way, such as You have entered a maze. Your current position is at the letter P. Your task is to go to letter m. New line indicates a downward direction.
The LM does not know what is the vocabulary of directions (not mentioned in the instruction). It does not know there is an option for help, which reminds the LM the rules of the game. All such information should be made explicit in the instructions.
I think each interaction should explicitly update/inform the LM its new location and explains why. When bumping into a wall, it should be informed 'You remain in the previous location because you collide with a way.' When moving up one step, it should be informed 'You have moved one step upwards.'
When the LM's answer is unclear, the environment should remind the LM the rules of the game, i.e., the instructions, because this indicates the LM has not understand the rules of the game. The current feedback "Instructions unclear, which direction would you like to go?" does not provide sufficient information to help the LM.
As the author mentioned that he/she has done some experiments with existing LMs. Is there any empirical results w.r.t. my above comments?

In sum, my concern is the instruction does not adequately express the rules of the game and the environment does not provide sufficient feedback. There is a risk that with the information given, the game itself is actually ill-defined. At a high level, the instruction and ineraction should be constructed in a way that best helps the LM to succeed this game.

Hello! I am one of two community reviewers assigned to this task.
I like the idea of using a text adventure game to challenge an AI. My concern is that the game does not give enough information to the AI. I basically see only three levels an AI can perform at: can't interact, can move randomly,  knows the most traversing algorithm ("always follow the right wall") and follows it.
I think this task would be much improved if (a) the AI was always told about adjacent walls, (b) landmarks were added to the maze, and (c) it was occasionally given additional information about the location of the mcguffin, e.g.: "You here a sound like a siren go off in the distance behind you."

Correctness: Seems correct. I did not validate the code in detail.
Specificity. I don't know what this is trying to measure. The README talks about trying to understand a textual representation of an abstract space, but the AI is essentially blind, and has to literally bump into walls to navigate.
Thoroughness: A model would need to be trained on this task specifically to know what to do. Otherwise, it's just measuring what an AI expects to do when shown a maze, as explained in the limitations section.
Difficulty: Executing this strategy may not be sufficiently clued by the text, but there is a very short program that gets optimal scores on all mazes, as I understand it.
Not solvable by memorizing the Internet: Only if a similar problem appears on the Internet.
Novelty: I didn't spot any competitors in the PR list.
Justification: Criticism given above.
Size: Large enough
Compute: Simple enough

On the whole, I do not recommend acceptance of this task in its current form. It seems like a very coarse-grained measure of a very specific AI capability (knowing the maze-following algorithm).

I read Xudong's review.  I see that I missed task #103. More importantly, I see that I missed that they player is given the state at each step. That greatly changes my review; now I see the AI is indeed given the right feedback, and so this does measure how well it can read the ASCII.  This addresses most of my criticisms.
I upgrade my review to a very weak accept recommendation.

@chiafullo weak Accept.
The author has not responded nor addressed the comments from the reviewers.

Hey y'all, apologies for the late response.  Thanks for the thorough reviews!
@XudongOliverShen
Some responses to concerns:

Could you comment on how likely randomly choosing a direction could solve this task, say, with width 5 and interaction limited to 30.

Unrestricted random walks on a plane typically reach a mean displacement from the origin that scales like \sqrt(# of steps), thus for 30 steps, this expands out to about 5-units away from the origin.  This is simply to say, it is conceivable that a randomly moving agent could reach the endpoint for a width 5 maze in 30 steps, but at least two other factors are working against the language model here:

language models don't necessarily output valid moves every round
and
this problem is harder than simple diffusion on an empty grid because of the existence of walls

If you're worried about measuring this explicitly, I could certainly add something like a "fraction of language model outputs that achieve valid motion" measure, or even "distance to goal measure" to help disambiguate "progress on task" from random action.  How does that sound?

I think this could be one reason that prevents the language model from solving this task: LM processes information in one line. Maybe explicitly instruct the LM to reason visually and interpret  as a new line operation could help?

I suspect that this is an idiosyncrasy of existing large language models, and that future large language models (which may even include explicit visual-components) would not have this issue, so I'd rather not over-index on hacking the instructions to make them easier for this particular task.  This is a fair worry for existing language models though, I agree!

But there is a similar task #103. I would suggest collaboration to avoid duplicates and improve this ASCII Maze task together.

Good catch!  I'll sort out how best to achieve this.

In sum, my concern is the instruction does not adequately express the rules of the game and the environment does not provide sufficient feedback. There is a risk that with the information given, the game itself might be ill-defined. At a high level, the instruction and ineraction should be constructed in a way that best helps the LM to succeed in this game.

This is a completely valid worry, but I've found that modern large language models actually have text-based games vaguely of this flavor in their training corpora, so I'm not as worried as you are about this (even to the extent of "understanding" the "(P)layer" pattern, which is common in this subgenre of text-based games).
To be clear, I expect this game to be quite difficult for existing large language models, nearly by design.  I suspect language models will need at least a couple more new features to solve this kind of task (including something like a memory over previous queries, as well as some either explicit or implicit visual representation).  This is why I'm resisting making the game more obvious, and why I'm not being super explicit about the rules---I argue that this level of ambiguity should be solvable by a sufficiently powerful model.
@jkoppel  It seems most of your concerns were addressed by your followup, but I wanted to respond to these two comments:

Thoroughness: A model would need to be trained on this task specifically to know what to do. Otherwise, it's just measuring what an AI expects to do when shown a maze, as explained in the limitations section.

Difficulty: Executing this strategy may not be sufficiently clued by the text, but there is a very short program that gets optimal scores on all mazes, as I understand it.

I'm not really interested in whether a language model can be trained to solve a text maze task---this could clearly be done with supervised learning of course, and you're of course correct that there exists a simple algorithm to solve the problem generally.  I'm much more interested in whether large language models, without any explicit supervisory signal, can (inadvertently) solve problems as complicated as ASCII-parsing and maze-traversal, because this is suggestive of a much larger problem-solving potential.  I hope that makes sense!

I'm the meta-reviewer for this task.
Thank you very much for your interesting and novel submission!
I have a concern that the api assumes the model.generate_text(input) function is stateless. There are other tasks that specifically rely on the model only basing its prediction on what's in the input, and not on earlier calls to generate_text(). Currently this task only provides the state resulting from the most recent interaction to the model. Even if the model uses the help menu, or learns by trial and error how to move, it will lose that wisdom in the next round. I think the task should be modified to accumulate the entire history of interactions into the model's input.
Similarly to the concerns of the two reviewers, I think it will be difficult for the model will not know whether it's supposed to produce a move, or continue your text generating a description of the maze. Can I suggest appending "\nAction: " to the input before calling model.generate_text(), so it's clear that user input should be generated? I would also suggest calling generate_text() with stop_string="\n", so that the model doesn't then make up its own response to its command by starting to draw a new maze.
I am discussing the similarity of this task with https://github.com/google/BIG-bench/pull/103 with the other co-organizers, and will send a followup message on that.

@cdfreeman-google,
Ahah I see your point. I was worried that current LM models would fail badly and thought you might help make it a bit easier to produce meaning comparision between current LMs.

If you're worried about measuring this explicitly, I could certainly add something like a "fraction of language model outputs that achieve valid motion" measure, or even "distance to goal measure" to help disambiguate "progress on task" from random action. How does that sound?

That would be a great idea! Consider the test itself is very difficult, subtest progress measures could help set up a comparision even when LMs fail in general.

This is a completely valid worry, but I've found that modern large language models actually have text-based games vaguely of this flavor in their training corpora, so I'm not as worried as you are about this (even to the extent of "understanding" the "(P)layer" pattern, which is common in this subgenre of text-based games).

I agree that such genre of the text is common in text-based games and is reasonable to use in this test. But clear expression of the rules should not be diminshed. For example, I think the following is quite important?

The LM does not know what is the vocabulary of directions (not mentioned in the instruction). It does not know there is an option for help, which reminds the LM the rules of the game. All such information should be made explicit in the instructions.

@chiafullo Accept.

@Sohl-Dickstein for meta review and merge (if acceptable)

@Sohl-Dickstein  @XudongOliverShen Alrighty, just implemented all major suggestions with https://github.com/google/BIG-bench/pull/137/commits/f50268093ab0efdc37345472549cede44c1779c6
Please let me know if you'd like any further changes!
Summary of changes:

Added new partial-success diagnostic scores (fraction of queries that parse as valid inputs, and fraction of queries that result in moves that don't hit walls, in addition to the old "average success" score
Spruced up flavor text a tiiiiny bit to be marginally more helpful.
Added "Action:" to prompt text, as per meta-review request
Context now accumulates, so the entire trajectory of one game (And all associated interactions) accumulate within a context (as per meta-review request)

This looks good to me! Thank you for contributing it!
Merging.


Improve the module bundling story

[x] Stop getting the version from package.json with packageify because it needs a JSON loader when bundling with webpack.
[x] Compile js (es2015) and jsx files with babel to use them in the package.
[x] Remove the need for a css loader/transform.
[x] Stop storing compiled css in the repo.
[x] Add an example for browserify.
[x] Add an example for webpack.
[x] Add documentation for the examples.
[x] Update README instructions for NPM installation.
[x] Update deploy script to publish the compiled files instead of the original source code.

Looks great! Assumed you used babel plugins instead of browserify / webpack ones to avoid duplication and allow adding more bundlers easily :+1:. Tested the bundling process and works ok :+1:. Give a look to my "nitpicking" comment if you want then LGTM.

@daf-spr Thanks!


UDSource health checks fail and sometimes the source will hung in k3d cluster

This can be fixed by changing the base image to amazoncorretto:11. I updated our examples accordingly. but we need to figure out why. I am keeping this one open. Consider adding documentation on this.

Having a base image is the standard/recommended way to build an image and amazoncorretto is one of the best java base images. I don't know why we didn't have a base image before. I am closing this one.


Implement Interfacing to support varying Upstream Cache Types

In order to support multiple origin types (rather than just Prometheus), the Proxy code needs to be refactored to work with an Origin Type interface, so we can write the new interfaces as needed for additional TSDB's like InfluxDB.

I am starting work on this issue now and will be committing to and eventually merging from https://github.com/jranson/trickster/tree/origin-interface for those interested in helping, providing input or monitoring progress.


Remove version: 8x8.VirtualOfficeDesktop version <IP_ADDRESS>

Consolidating into 8x8.Work. Related to: #135119
Microsoft Reviewers: Open in CodeFlow

URL: https://vod-updates.8x8.com/ga/work-64-msi-v7.12.2-6.msi
Status Code: 200
(Automated message - build 656)


Remove hard-coded single-view assumption from ScrollPhysics

ScrollPhysics currently has hard-coded single view assumptions in two places, see the WidgetsBinding.instance.window in the snippet below:
https://github.com/flutter/flutter/blob/d1244b7c9b8d676a84bdba2a440ddae1e0907a8c/packages/flutter/lib/src/widgets/scroll_physics.dart#L386-L391
https://github.com/flutter/flutter/blob/d1244b7c9b8d676a84bdba2a440ddae1e0907a8c/packages/flutter/lib/src/widgets/scroll_physics.dart#L249-L258
In a multi-viewed world, this will not work anymore because the ScrollPhysics have no idea what view/window the scrollable they control is in. We need to refactor this, i.e. either the ScrollPhysics somehow need to look up the current view via View.of(context) or the relevant information need to be passed in.

cc @Piinks

Actually, in the second case, we can just look up the current view via the provided BuildContext.


Problem with angular.equals

angular.equals runs not exactly when comparing between empty object {} and other object with key starting with character $.
Example:
angular.equals (
 { },
 { "$or" : [1, 2] }
)

=> Expected Value is false, but true.
But when using jQuery.isEmptyObject, it's ok.

That's because $properties are supposed to be angular properties to be ignored by comparaison. Angular.equals is designed for angular needs and therefore works as designed.

I got it. Thank @frfancha very much :+1:


Add Utterance Comments

@jacogr you might hate this, you might love this, but this is something I have set up for some of my Docusaurus 2 projects.
https://utteranc.es/
Basically this PR integrates Utterances into your documentation so every page has its own comment section. New comments are placed into a github issue with the appropriate title which identifies which page it is for. As users post on github or in your comments, they will be shared on both sides.
Looks like this:

Ok, I didn't comment this morning on it since it is Monday and release day, so a bit pressed for time.
Ive actually seen this (for the first time) 2 weeks ago. Was intrigued, but didn't think of adding it here. Don't have strong feelings either way, think it can add value. Will play a bit with this PR.

@jacogr gonna close this cause it is stale, but still think it would be cool to have.

I was wondering if GH discussions could fill the void as well... but it is slightly different from "discussing on-page".


PowerToys Run doesn't work if Keyboard Manager is disabled

Environment
Windows build number: [Version 10.0.18363.836]
PowerToys version:  0.18.0
PowerToy module for which you are reporting the bug (if applicable): PowerToys Run

Steps to reproduce

Disable Keyboard Manager
Try to launch PowerToys Run with Alt+Space or other key paring

Expected behavior

The PowerToys Run launcher shows
Actual behavior

Either PowerToys Run should disable along with the Keyboard Manager, or the launcher should work regardless of the state of the Keyboard Manager
Screenshots

Hi @CalvT it works fine for me. Are either Alt or Space remapped by any chance on your Keyboard Manager? That could be a reason why Alt+Space only works when Keyboard Manager is disabled.

Alt+Space didn't work for me upon first install, I'd also disabled some toys before I got around to trying the Run toy.
I found however that by changing the shortcut key binding to something and then changing it back to Alt+Space, it now works -- so there's definitely something a bit fishing in the keyboard binding.

Yes, same as @RPitt - on updating to 0.18.0, I disabled Keyboard Manager, then tried to remap the Run toy to Caps.
The settings window wouldn't accept that, and switched back to Alt + Space, however Alt + Space wouldn't bring up Run. I tried switching to a different key combination, but that didn't work either.
Enabling Keyboard Manager then made it work, and it now works with Keyboard Manager disabled.
I'll try removing everything and seeing if I can reproduce with a fresh install in the morning.

closing this, reopen if you manage to repro!


Backchannel-related refactoring

Codecov Report

Merging #605 (e16fa9c) into main (de964a2) will decrease coverage by 0.15%.
The diff coverage is 29.41%.

:exclamation: Current head e16fa9c differs from pull request most recent head d162050. Consider uploading reports for the commit d162050 to get more accurate results

@@            Coverage Diff             @@
##             main     #605      +/-   ##
==========================================
- Coverage   59.25%   59.10%   -0.16%
==========================================
  Files         198      198
  Lines       19116    19175      +59
  Branches     4352     4367      +15
==========================================
+ Hits        11328    11334       +6
- Misses       3457     3508      +51
- Partials     4331     4333       +2

Flag
Coverage Δ

integration-aries-vcx
47.77% <29.41%> (-0.15%)
:arrow_down:

unittests-agency-client
29.33% <0.00%> (ø)

unittests-aries-vcx
47.46% <12.06%> (-0.13%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

agency_client/src/configuration.rs
0.00% <0.00%> (ø)

aries_vcx/src/handlers/issuance/issuer.rs
57.90% <0.00%> (-0.50%)
:arrow_down:

...ies_vcx/src/indy/primitives/revocation_registry.rs
46.03% <0.00%> (-1.44%)
:arrow_down:

...vcx/src/protocols/issuance/issuer/state_machine.rs
70.65% <0.00%> (-0.74%)
:arrow_down:

aries_vcx/src/indy/ledger/pool.rs
35.35% <8.33%> (-2.85%)
:arrow_down:

...s_vcx/src/indy/primitives/credential_definition.rs
44.44% <35.29%> (-1.41%)
:arrow_down:

aries_vcx/src/indy/primitives/credential_schema.rs
30.76% <36.84%> (+5.76%)
:arrow_up:

aries_vcx/tests/utils/devsetup_agent.rs
60.62% <50.00%> (-0.61%)
:arrow_down:

aries_vcx/src/handlers/connection/connection.rs
57.56% <63.63%> (-0.22%)
:arrow_down:

... and 3 more

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


Add new context api polyfill and migrate paralax

Polyfill is also required until major release with 16.4 restriction.

We need to add tests. Which ones would you prefer? I can help to setup puppeteer.

Sure this needs a polyfill? 16.3.1 contains the new context, already using in a couple of my apps.
Testing would be awesome, no preferences as you seem to know your way around pretty well.

Btw, i'm looking for help with maintenance, sharing authorship. If you're down i'll give you all the access you need to publish/push/pull/merge/review on your own.

The latest major release of this package included 16.0.0 and if we will publish minor release with new api those users can be surprised.
I can partially help with maintenance. However I'm not interested in project itself.
Probably you guys have some interest in it?
/cc @alexreardon @souporserious

Can you add a little more context @TrySound ?

This project generally or the parallax effect?

The project. The discussion moved to maintenance question. :)


Add XI for Northern Ireland after Brexit

According to https://en.wikipedia.org/wiki/ISO_3166-1#User-assigned_codes_in_wide_use XI has to be used for VAT numbers of trades between Northern Ireland and other countries of EU. It has not been added officially to ISO 3166-1 but is mandatory to be used for tax matters.

Thanks, but this data is coming form ICU database, so that's where you should report issues. This just converts it to other formats.


docs: use hambuger for navigation

Fixes #676
Demo: http://stephenplusplus.github.io/gcloud-node-gh-pages-again/#/docs/v0.15.0
The navigation will never be placed at the bottom of the page. Anytime there is a hamburger icon, it will expand the full set of links.

LGTM


ci: move helix queueing to a separate stage as well

This pull request moves the Helix queuing to a separate stage outside of the build stage.

The build is failing because we have ~30 tests that are failing in Helix :| The build is, otherwise, passing.


Doesnt work at chess.com

The bot goes out of sync after few moves, dunno why, it worked well till now maybe chess.com changed something.
Thanks

Ok, I will check it.

Now it works again, i've noticed that sometimes it got confused with the king castle, especially with 0-0-0 or long castle.

I already did it for lichess, and as I see - it works properly at both sites.

Duplicate #28. Closed.


kernel: Improve cooperative and preemptive performance as per thread_metric benchmark

This set of commits offers improvements to both the cooperative and preemptive results in the the thread_metric benchmark.
The following numbers were obtained with the thread_metric benchmark project on the disco_l475_iot1 board. MQ denotes use of CONFIG_SCHED_MULTIQ and DQ denotes use of CONFIG_SCHED_DUMB.
Before:
Time Period Total:  12436655  (Cooperative, MQ)
Time Period Total:  11268922  (Cooperative, DQ)
Time Period Total:  5730607   (Preemptive, MQ)
Time Period Total:  6404080   (Preemptive, DQ)
After:
Time Period Total:  16001868  (Cooperative, MQ) : +28.7 %
Time Period Total:  11595554  (Cooperative, DQ) : +2.9 %
Time Period Total:  6473166   (Preemptive, MQ) : +12.9 %
Time Period Total:  7018286   (Preemptive, DQ) : +9.6%

The main changes in this updated patch set are ...

Split out the halt_thread() reordering patch as per Andy's preference, so that it can be its own PR later.
Updated the implementation for caching the queue index in the muti-queue.

Apologies for all the updates. I kept realizing that I had forgotten a small detail to fix each time.  This should hopefully be it.

Rebased to resolve merge conflicts.


M5Stack Atom PoE connected via ethernet keeps connecting to WiFi

I have a Atom PoE that is working brilliantly, however every so often to reverts to WiFi instead of maintaining the ethernet connection.
I would love to get more logs, but I don't know how xD
Is there a fix or workaround?

I noticed the same problem every now and then, and I think I've found a bug in the connection logic. Please try this binary:
nuki_hub-8.19-pre-2.zip

I've flashed it and it's using Ethernet for now.
I'll monitor it over the next few days and report my findings.

You can upgrade to 8.19, the fix is included

In the Pre version and in the Final one the issue appears to be solved for me. I'm closing the issue.
Thanks for your hard work.


Migrate dusk-wallet-cli to be a command within dusk-blockchain #410

add wallet as cmd
fix some log and naming issues with dusk cmd
add dusk,voucher and wallet tasks to Makefile<
update require deps on go.mod

Codecov Report

Merging #414 into master will increase coverage by 4.53%.
The diff coverage is 53.46%.

@@            Coverage Diff             @@
##           master     #414      +/-   ##
==========================================
+ Coverage   52.34%   56.87%   +4.53%
==========================================
  Files         132      169      +37
  Lines        7499     9249    +1750
==========================================
+ Hits         3925     5260    +1335
- Misses       3063     3311     +248
- Partials      511      678     +167

Impacted Files
Coverage Δ

pkg/config/consts.go
0.00% <0.00%> (ø)

pkg/config/loader.go
61.03% <ø> (ø)

pkg/core/candidate/validator.go
65.21% <ø> (ø)

pkg/core/chain/collectors.go
57.14% <ø> (ø)

pkg/core/consensus/agreement/accumulator.go
90.90% <ø> (ø)

pkg/core/consensus/agreement/store.go
59.30% <0.00%> (ø)

pkg/core/consensus/candidate/factory.go
100.00% <ø> (ø)

pkg/core/consensus/candidate/genesis.go
71.42% <ø> (ø)

pkg/core/consensus/collector.go
0.00% <ø> (ø)

pkg/core/consensus/component.go
72.41% <ø> (ø)

... and 189 more

Continue to review full report at Codecov.

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


Start of windowing implementation

Implements "Stub" windowing API, the framework is there, but the meat and potatoes are not currently implemented
Summary of the PR
Essentially adds the interfaces as defined in the Windowing(3.0) proposal

I can answer any outstanding proposal wording questions in a bit, really annoyed this didn’t come up in #539 though.

Yeah me too - We've been less then thorough with that review. I guess I was just so into the idea cause we came up with it only few days before, and didn't see any problems with it.

Can you propose a change to the proposal itself? It's unfair to expect an external contributor to answer to this.

I don't really expect any contributor to answer this, I just figured tracking it here associated directly with the code seems easier. If you want I'll delete the proposal and re-submit it, and attach the comments there.

I was more saying propose changes to the proposal in response to your comments yourself, probably the most productive :)

Ah wups, yeah will do that once I'm done with my current work item. Just wanted to get my review out there, and will resolve any outstanding things once I get to windowing.

Ah wups, yeah will do that once I'm done with my current work item. Just wanted to get my review out there, and will resolve any outstanding things once I get to windowing.

Does this mean I other than some the slight changes I am stuck?

Not necessarily, your free to put in any work you want to, but no one expects you to do any of the proposal work.
Ill look at perkseys proposals in the morning and then also submit proposal changes, but heading to bed now.

This has an outstanding action on @HurricanKai to change the proposals, but there is little work being done here at this time. Should we merge just so we have something in the repo as this does technically implement the approved API as it currently stands?

I think this activity has proven that we're not necessarily ready to work on this right now.
Opinions for closing for now?

Doesn't seem like this will be implemented anytime soon, and there's lots of conflicts.


libtool error when building

Hi,
First of all, thanks for the script! I'm getting an error below and don't know what is causing it. I'm using Apple Silicon Mac but I guess that's not an issue here.
error: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool: file: /Users/mortar/Desktop/goodies/darwin-xnu-build/build/dtrace.obj/dtrace.build/Release/libelf.build/Objects-normal/arm64/clscook.o is not an object file (not allowed in a library)
Libtool /Users/mortar/Desktop/goodies/darwin-xnu-build/build/dtrace.obj/UninstalledProducts/macosx/libelf.a normal (in target 'libelf' from project 'dtrace')
Am I mssing something? I just run ./build.sh,that's all.
Thanks in advance

I guess problem is with my LLVM version ( #1 ) but GitHub action build get me "Failed to run query: There was no upgrade path to the target dbscheme" which seems like another problem with my own environment


Audit / log mechanism?

Does this platform contains or supports an extension mechanism for logging or auditing?
I mean the product, not the FHIR standard.
I would like to register all the activity related to all ingoing and outgoing messages.

Hi Jaime, what other have done is use the OnBeforeRequest and OnAfterResponse events on the FhirClient. These give you access to the http request just before it is sent and just after you have received a response, you can use these to debug!   As well, you can use tools like Fiddler to intercept requests and responses and debug your session.

Hi Ewout, the proposed client-side solution doesn't seem adequate for audit. The control of the mechanism shall be out of the client scope for security reasons. Moreover, if there are several applications accessing a FHIR repository and one of them do not or can not record an activity, there is no way an audit application can guess that an operation was done.
Other aspect is performance: if the audit is triggered at client side, it will produce an additional client-server traffic, while the typical server-side operation would spare that.

Sorry, Jaime, I am also maintaining the .NET client issues, and was thinking on the wrong side of the wire ;-)

Hi Ewout, thanks for keeping alive this suggestion. Is there anything done or planned?

Event logging has been planned in our roadmap for the coming month.

Some logging was implemented this year. Let's check how far this issue is now resolved.


Fix num batches in case of multiple dataloaders and percent_check

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure to update the docs?
[x] Did you write any new necessary tests?
[x] If you made a notable change (that affects users), did you update the CHANGELOG?

What does this PR do?
Fixes #1899.
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Still needs to figure out why tests are failing. Need some help.

you need to change the break condition in the loop that involves the max_batches variable, or do I miss something?

Hello @rohitgr7! Thanks for updating this PR.

In the file pytorch_lightning/trainer/data_loading.py:

Line 227:26: E226 missing whitespace around arithmetic operator
Line 239:17: E303 too many blank lines (2)

In the file pytorch_lightning/trainer/evaluation_loop.py:

Line 225:111: E501 line too long (128 > 110 characters)

I will help fix the rest of the tests tomorrow

I will write new test cases tomorrow.

@awaelchli I was thinking to add just 1 test in which we will test both multiple val and test dataloaders with different sizes by just comparing num_batches for both. This is sufficient or should I add more tests and conditions??

Testing the num_batches for multiple with different sizes is good. However, a more important test imo is checking also that the dataloaders get consumed by that amount.
We could easily do this with a custom dataloader that just returns the index of the sample. Then we check how many samples we have drawn from each dataloader. What do you think?

@awaelchli Yeah, good idea. But don't know how to do that within tests.

I think the test is good and covers the base case, thanks for adding it! I will try to generalize it a bit more if you don't mind and post here.

hey @rohitgr7 I was able to parameterize the test and make it consistent for train, val, test.
Would you mind to merge this branch into yours and push it?
https://github.com/awaelchli/pytorch-lightning/tree/fix_num_batches
Then I think the PR is ready for review/merge :)

There is also a PR #1959 that adds support for multiple train dataloaders, so if that one gets merged first we need to make a few adjustments here :)

I made a small mistake with git there! Apologies.

yes I saw, no big deal, now we just need to fix this small merge conflict :)

Just a suggestion! Can we have *_percent_check to be a list too where len(*_percent_check) == len(*_dataloaders)? In case if it is int then it will be same for all the dataloaders passed. Don't know how this can be useful in any case, just a thought.

Not sure how useful that is, but how about you open an issue for that?

Friendsly ping : )

This PR is not necessarily blocked by #1959. Either one could go first and the other has to adapt. To me it looks like this one is closer to the finish line.

@awaelchli Then let's merge this. Is there anything else I need to do?

I need to check why tests are failing.

this looks solid.
can we bring this fix into 0.8 @PyTorchLightning/core-contributors ?

@awaelchli  so far, in master is all related to 0.8 so we can do release from master not necessarily from v0.8.x It will be much easier to maintain for us...

@rohitgr7 mind resolveing conflict so we can get this done... :rabbit:

I'll add you as co-author to #2213 (sorry, didn't see this one)

@williamFalcon would it be easier to merge this one and rebase #2213 on actual master?
btw, there can be just one author, so co-author in Github does not work...

#2213 still needs the test implemented by @awaelchli here to check num_batches when dataloaders have different len with percent_check.

@Borda I got a little late. Looks like #2220 is merged already.

Rebase on actual master pls

this is not needed anymore... we already merged these fixes (and added everyone in this PR as co-author)

I agree with @rohitgr7. We both spent quite a bit of time debugging this and deriving a solid test for the specific bug reported in the original issue. I submitted a PR that adds it back. Hope that's ok with you william

oh sure! the test is perfect. i didn’t see that part

just make sure to rebase master and also look at the tests we added for that

but also... everyone on this PR was credited with that PR as well

I agree too, @rohitgr7 @awaelchli pls rebase on master...

@williamFalcon @Borda Thanks, it's very kind you were able to add us as authors there. I already submitted a new PR #2226  with the missing test, because was not able to push here.

we can still merge this one, @rohitgr7 :rabbit:

@Borda rebased.

@Borda Getting some problems with the tests here. Not sure why it's not showing any conflict to me locally when I rebase on the master. @awaelchli has already created a new PR that is working perfectly. I think we can close this one and merge that one.

@rohitgr7 @awaelchli the simples solution from this situation is that I merger this PR to #2226 and then the other PR will be merged in master so this PR will have status merged for sats purposes and also this PR will be listed in changelog to be in release contributors list... hole all fine :rabbit:

Thanks @Borda for the fix.


Nge add2

Associated with #14595

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14672/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14672/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14672/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14672/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14672/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14672/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14672/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14672/merge#BCU1XR#0


[Bug]: RPC client functions

https://github.com/modelscope/agentscope/tree/main/src/agentscope/rpc
In this repo below files are there
init.py
rpc_agent.proto
rpc_agent_client.py
rpc_agent_pb2.py
rpc_agent_pb2_grpc.py
In the rpc_agent_client.py set of functions are defined right? How can I call those? can someone help me to test any of these functions?
from agentscope.rpc.rpc_agent_client import RpcAgentClient
client = RpcAgentClient(host="localhost", port=12010)
agent_list = client.get_agent_list()
print(agent_list)
I get the below error. But my Server agent is running on that port
AttributeError: 'RpcAgentClient' object has no attribute 'get_agent_list'

You can find some examples in test_agent_server_management_funcs of tests/rpc_agent_test.py.
The wrong version of AgentScope may cause the reported error. Please reinstall AgentScope from the latest source code pip install -e .[full]


Create an example project inside our plugin

So we get more scores in our pub.dev package
"0/10 points: Package has an example"

tracked in PR #29


Appendable Excels not performed in the correct order

Description
Appendable Excels not performed in the correct order
Context

I have a simple pipeline where I read in three separate Excel sheets from the same xlsx, perform a task on each sheet, and rewrite the result to another xlsx. In my data catalog, "summary_quarter" is the main pandas.ExcelDataset and "summary_month" and "summary_week" are AppendableExcelDatasets of the same xlsx.
The problem is that Kedro is performing tasks on "summary_week" first. Naturally it throws an error because the xlsx initiated by "summary_quarter" does not exist yet.
I thought I could solve this problem by making an empty xlsx. However, this does not solve the issue because the results from "summary_week" are lost when the xlsx is overwritten by "summary_quarter".
I searched for a solution to force Kedro to start with a certain dataset but found nothing. If someone has a tip, id be really grateful if they could share it here.
Otherwise, I dont really have any ideas on how to solve this. Ideally, Kedro, by reading the order of appearance of the datasets in the catalog, should infer that quarter should be performed first, followed by month, then week.
Your Environment

Kedro 0.16.6
Python 3.8
Windows 10 Pro

Hi @lucasjamar! Thanks for flagging this. I hope the other parts of Kedro have been okay for you.
Can I play back if I understand what you're trying to do first? Is it: "You have a single Excel spreadsheet that in this case has three sheets, summary_quarter, summary_month and summary_week. You want to process each sheet individually and then combine the three sheets in a single Excel spreadsheet." And the reason you can't do this is because: "The way the AppendableExcelDataSet works is that it expects a default ExcelDataSet to append to. So summary_quarter has to go first."
If I got this right then I need to tell you that there is no way to specify explicit node order in Kedro at the moment since the datasets are the only defining factor in the execution order.
The workaround currently would be to add dummy dependencies between the nodes, something like:
node(make_pretty_df, summary_quarter, [summary_quarter_pretty, dummy_node1_out]),
node(make_pretty_df, [summary_week, dummy_node1_out], summary_week_pretty),
node(make_pretty_df, [summary_month, dummy_node1_out], summary_month)

dummy_node1_out doesn’t need to be registered in the catalog and can simply be None. Let me know if this resolves your issue and I'll close the ticket.

Hi @yetudada ! Yes Kedro is great tool for my data science projects!
Yes you understood what it is I am trying to do
Ok I see. Thanks for the hack. I will add to my pipeline.
Many thanks!

Awesome! Glad this is sorted! I'll close the ticket now.


index: support version aware checkout

adds checkout(latest_only=True) param for index.checkout. When true, checkout will enforce that the file is always checked out as the latest version in version-aware contexts. When false, if the previous version exists, a new version will not be checked out (regardless of whether or not the existing version is also the latest version)
adds progress bar and file xfer count support in checkout/save

Codecov Report
Base: 54.98% // Head: 55.29% // Increases project coverage by +0.30% :tada:

Coverage data is based on head (5379630) compared to base (fcaa82b).
Patch coverage: 75.92% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #232      +/-   ##
==========================================
+ Coverage   54.98%   55.29%   +0.30%
==========================================
  Files          49       49
  Lines        2906     2930      +24
  Branches      498      417      -81
==========================================
+ Hits         1598     1620      +22
- Misses       1233     1234       +1
- Partials       75       76       +1

Impacted Files
Coverage Δ

src/dvc_data/index/checkout.py
63.46% <63.88%> (+3.93%)
:arrow_up:

src/dvc_data/hashfile/db/__init__.py
55.67% <100.00%> (+0.46%)
:arrow_up:

src/dvc_data/index/save.py
86.07% <100.00%> (+2.74%)
:arrow_up:

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

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


godot-4-beta16 freeze when steamVR and/or windows-mixed-reality-portal

Godot version
godot-4-beta16
System information
win 11
Issue description
when steamVR and window-mixed-reality-portal is running and you want to select "Project->export" to build your project, godot freezes and you must kill the process with task-manager.
Steps to reproduce
when steamVR and window-mixed-reality-portal is running and you want to select "Project->export" to build your project, godot freezes and you must kill the process with task-manager.
Minimal reproduction project
none

Not sure what is going on here, so far can't reproduce on Windows. Could this be related to the issues we were having with beta 16 and exports?

still got the problem with godot-4.0-stable. I always need to close steamVR and windows-mixed-reality application.
Now i found out, godot tells me about scripts like RightHand, MovementDirect etc. it shows me a BLUE-Script icon (instead of white script icon) and it says:  This script is currently running in the editor ??  I dont know what that means.
But i can double-click it and they (the blue-script-iconed scripts) open.  But when i start mixed-reality-portal and steamVR, i cannot open the blue-iconed-scripts anymore ???   If i close mixed-reality-portal and steamVR , i again can open the blue-iconed-scripts ??
Perhaps this is a hint/info , if perhaps others got this problem in future too

@flobotics Can you reproduce this after enabling Single Window Mode in the Editor Settings then restarting the editor?


TitleAVAIL

Title
Konstru
Description
Version control for Structures
Website
Version control for Structures
Image
logos/konstru.png
Crunchbase
https://www.crunchbase.com/organization/konstru
Industries

engineering

Tags:

web-app

Duplicate of #24


Security fixes from Michelin CERT

Hello Team,
You will find in this PR, the remaining fixes for the last 3 vulnerability we cannot report on HackerOne due to Signal restrictions.
For fix Broken Access Control on endpoint migrate, we guess that only superuser could access to this feature.
Don't hesitate to come to us for details about the vulnerability found.
Thanks again for your reactivity for our first bugs.
++++

@maxime-escourbiac-bib Thanks for the PR.
However, we only take PRs against the bugfix or dev branch. This is one that should be based on the bugfix branch considering what it's doing. Details on contributing to DefectDojo are at https://github.com/DefectDojo/django-DefectDojo/blob/master/readme-docs/CONTRIBUTING.md

@mtesauro Hello, thanks for the quick reply!
I'm currently on vacation I will have a look when I will have some availability :)
++

Thanks for pull request , I close this one


Reorder monaco TS deps based on imports

Some parts of the code I'm not proud of, but it works

Some parts of the code I'm not proud of, but it works

It's not that bad! Just a few comments.

PTAL

PTAL

Maybe because I'm reviewing from my phone, but I can't see the changes to the commented out tests. Also, what about Preston's q re: dep type?

@tessaSAC I actually forgot to push, should be good now


truncate option with jquery not working

Hi,
I am struggling with the truncate option using the jquery API. I have the
impression that this option is not at all implemented. Could that be or am I doing something wrong?
In my application the URLs are linkified. Thus I know that the library is working. Just the truncate business is
not working. Any suggestions?
<script src="linkifyjs/linkify.js"></script>
<script src="linkifyjs/linkify-html.js"></script>
<script src="linkifyjs/linkify-jquery.js"></script>

<script>
const options = { truncate:30  };
$("td").linkify(options);
</script>

Hi @dingolino67, this is a known issue and due to be fixed in the next release. In the meantime, I suggest the format option as a workaround
$("td").format({
  format: (value) => {
    if (value.length > 50) {
      return value.slice(0, 50) + '…';
    } else {
      return value;
    }
  }
});

Thanks a lot. That works.

Fixed in latest v4 release.


memory

Hello, the device around me now has only 12g video memory. Your code has been reporting video memory errors. Can you run it by modifying parameters?

Is there a lighter version that can be reproduced in 12g video memory?

try using embed_dims = 48, which is tiny version of vt-unet.

OK, thank you very much for your reply. Is there a big difference in the performance of the three versions?

Hello, after I run the code,  threshold_values=True/False is deprecated, please use threshold=value instead.
fused_weight_gradient_mlp_cuda module not found. gradient accumulation fusion with weight gradient computation disabled.
appears. I wonder if it has any impact?

Hello, in training, the training data set is divided into Train dataset number of batch: 519
Val dataset number of batch: 130
Bench Test dataset number of batch: 130. What is the reason for this operation?

I look forward to your answer very much

"threshold_values=True/False is deprecated" is a warning, there is no impact from that for model training. In our preprint version in arxiv you can find all the implementation details including dataset splitting.
During model training, we divide the entire dataset into 3 sets: training (used for model training), validation (used for model validation and saved the best model) and test (evaluated on saved best model).

OK, thank you very much. Do you still need to save some data in test.py, or just put all the data in the training set without dividing the test set.

There is also fused_weight_gradient_mlp_cuda module not found. gradient accumulation fusion with weight gradient computation disabled.
appears. I wonder if it has any impact?

No, you can use bench test dataset which is 130 scans in your case. In test.py https://github.com/himashi92/VT-UNet/blob/main/test.py#L102 use get_datastes instead of get_test_datasets. If you want to generate predictions for brats2021 validation dataset, then have a separate folder and use get_test_datasets method.
Regarding, "fused_weight_gradient_mlp_cuda module not found". This seems a torch version problem. Try updating torch. I am not sure whether that'll affect model training.

OK, thank you very much for your reply. What version of torch do you use? Let me adjust my version.

I am using pytorch=1.9.1 torchvision=0.10.1 and cuda=11.3

OK, thank you very much. Did you use all the data in brats2021 in the experiment? Whether some data with bad marks are eliminated.

yes, Brats2021 organizers updated the dataset.

I'm sorry, I don't quite understand. Did you use all the data of brats2021, that is, 1251 data?

we divide 1251 scans into 834, 208, 209 for training, validation and testing, respectively.

OK, thank you very much for answering for me, and then express my thanks.

Hello, because of the small version I use, you only provide the link of vt-unet-b pre training model. Can you provide the link of vt-unet-m and vt-unet-t pre training model?

We'll release all the trained models with the new version of the code soon.

Hello, I used the small version for training. If no pre training model is provided, put it into saved_ Model file, how can I test and verify the performance?

You can train from scratch and evaluate the performance.

Hello, do you mean to train the generated model_ best_ base. pth. Put tar into saved_ Under the model folder.

Hello, I'd like to ask you something. Normally, you should not train first and then test with the trained model. I don't know why you are saving here_ Put the model provided in the model instead of the best model trained by yourself?

Hello, when will the new version be released? I'm looking forward to it.

At the moment we cannot provide you with trained best models for vt-unet T and S versions as the paper is still under review process. We'll update our repository soon.
If you want to evaluate the VT-UNet model performance as given in the ReadME file, first train the model by yourself from scratch (That does not need our trained model weights, only pretrained weights of imagenet is needed. All the steps are given. Please follow instructions).  While training, the best model weights will be saved in the given respective path which is evaluated on val dataset. For example model_best.pth.tar.
This model_best.pth.tar can be then loaded using test.py and evaluated on the bench test dataset.
We followed the conventional process of training a deep learning model like other researchers. Hope I addressed your concerns. For further details, go through the paper.
Thanks

OK, thank you very much for your reply.

Hello, what codes need to be modified if I want to use use bench test dataset which is 130 scans in your case. In test.py https://github.com/himashi92/VT-UNet/blob/main/test.py#L102 use get_datastes instead of get_test_datasets..

We look forward to your reply.

What do you mean? I cannot understand what you are saying.
For training you can use train.py.
For testing during inference you can use test.py.
Feel free to modify those scripts as per your requirements. Everything is written in basic python and pytorch. If you don't understand the code and outcomes, try debugging and understand the code and how tensors work.

I'm really sorry. Maybe I'm not accurate. Run test The data set tested in py needs to redefine a test set. In the code, it is get_ test_ Datasets, which I mentioned earlier, has been divided into a part as test. You said that these data can also be used in the test. Use get_ Datasets is enough, so I want to ask what code needs to be modified if you want to do so?

I'm really sorry. Maybe I'm not accurate. Run test The data set tested in py needs to redefine a test set. In the code, it is get_ test_ Datasets. As I said earlier, a part has been divided into tests in the training. You said that these data can also be used in the test. Use get_ Datasets is enough, so I want to ask what code needs to be modified if you want to do this

https://github.com/himashi92/VT-UNet/blob/2c5c5451a8e61d0884d5ddc3b487494602fd6e37/test.py#L102
change this line of code to following:
_, _, bench_dataset = get_datasets(args.seed, fold_number=args.fold)
So that you can evaluate your 130 test set using test.py
Hope I answered your question

OK, thank you very much for answering these questions for me. Thank you very much.

No worries, good luck with your research!


Add support for mathematical expressions using MathJax

Fixes #165. Ref: https://github.com/utterance/utterances/issues/582, https://github.com/utterance/utterances/issues/637
https://github.blog/2022-05-19-math-support-in-markdown/
https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/writing-mathematical-expressions
The vendored library is adapted from GitHub's client code that you can access from the DevTools:

I don't know how legal this is, but I've put a disclaimer just in case.
P.s. thanks GitHub for providing sourcemaps!

Wow! 🤩 Came here from Utterances, which still does not support MathJax
Giscus rocks!


Added 3 new Libs

Bridge by Aidan Follestad
Glide by Bump Technologies
Hawk by Orhan Obut

Please check if I added the custom Licence for Glide correctly. I'm not so sure about that... :D

thanks for the PR  will check it ;)


Add an optional (relative) time to the start and stop commands

Some times, you realize you have started working on a task/project without tracking it. It would be nice to have a CLI option to indicate that you actually started earlier. For instance
watson start --for 10min 'World Domination Project' +'first analysis'
watson start --since 14:35 'Work on personal organisation'

or something similar.
In the same way, the option can be useful for the stop command (e.g. back from lunch and see that your task is still started).

:+1:

Current workflow to solve this kind of issues is:
# Start your project with tags
$ watson start "World Domination Project" +"first analysis"
# Edit current (running) frame
$ watson edit

# or (for stop)
$ watson stop
$ watson edit

--for and --since options seems cool, but I hope this won't be too confusing.

@yloiseau I am afraid this would add too much complexity to the start command.
@k4nar @SpotlightKid WDYT?

I don't use watson enough to have a valid opinion on this. But FYI, for now we don't have any option for watson start, so there's room for some :) .

I'm using the workflow with watson edit described above fairly and I have to admit it's a bit cumbersome. I would welcome a way to at least set the absolute start time directly with watson start. Or maybe the watson edit command should take options to change a frame non-interactively?

:+1:
At times it happens to me as well.

+1

👍

I hacked a quick implementation proposal for this kind of feature at https://github.com/TailorDev/Watson/pull/224

Indeed, finding a good naming is essential. I do find @yloiseau's version more intuitive though :) .
What do you find confusing in the current proposal?

@k4nar Two things are confusing me:

I am used to command line options being named either as "intended actions" (like "add last …" in my proposal) or to make sense in English natural language when taken together with the command ("start at" in my proposal). Trying to apply such an interpretation to "start for …" and "start since …" fails, so it's not intuitive to me.

I am not an English native speaker, so the semantic difference between "for" and "since" is also not intuitive to me. I know it, but I always have to think for a few seconds to make sense of the difference when seeing both side-by-side such as in the proposed options.

As example of another tool having an interface for this, doing CLI has something similar and very semantic:
https://brettterpstra.com/projects/doing/#adding-entries
doing now --back 15m working on GH-13

Also, a watson amend command (inspired by git commit --amend) might make an intuitive way to noninteractively make edits to last entry, include watson amend --back 15m and similar such modifications

I'm not sure we have to add a new command that would be an edit alias (even if I like the amend semantic).
:+1: for the --back 15mn option; hence non interactively editing the current frame would look like the following:
$ watson edit --back 15mn

Just for the sake of inspiration and because there are more issues related to dates, I think it might be interesting to learn how other projects are dealing with dates.
For example:

https://taskwarrior.org/docs/durations.html
https://timewarrior.net/docs/backdated.html

In my opinion, all date issues should be tackled at once to develop a consistent solution through all Watson commands.

I just ran into this problem and was going to file an issue.
stop already supports --at.  For symmetry, why not add --at to start?  Otherwise the routine for stopping a late task would be
watson stop --at 14:00
but the routine for starting a late task would be
watson start someproj
watson edit --back 15m
I like the --at syntax in stop because it is concise and doesn't make me invoke two commands.

I like the --at syntax in stop because it is concise and doesn't make me invoke two commands. Why not add --at to start to make it symmetric to stop?

I would expect --back and --at would be interchangeable (though not functionally so) and so could both be implemented for start and stop -- relative vs absolute times :)
I would venture that --back should always work relative to current time. (e.g., rather than "pushing back" an existing state, like the start time.)
For examples:
# with absolute time
watson start --at 14:00

# with relative time
watson start --back 15m

# start a task, backdating start time to 15m ago
watson start --back 15m someprog

# modify start time of running task (relative to time edit command run)
watson start someprog
# < ~5 min of time passes >
watson edit --back 20m

# modify stop time of last stopped task
watson stop
# < ~5 min of time passes >
watson edit --back 20m

My assumption (though this should be vetted), is that edit modifies the start time for running tasks, and the stop time for stopped tasks.


googleapiclient.errors.HttpError: <HttpError 404 when requesting https://www.googleapis.com/calendar/v3/calendars/[...]&alt=json returned "Not Found">

As of a few days, I'm getting this traceback using gcalcli master branch (tag: v4.2.1):
$ gcalcli agenda
Traceback (most recent call last):
  File "[...]/gcalcli/cli.py", line 247, in <module>
    main()
  File "[...]/gcalcli/cli.py", line 155, in main
    gcal.AgendaQuery(start=parsed_args.start, end=parsed_args.end)
  File "[...]/gcalcli/gcal.py", line 1197, in AgendaQuery
    return self._display_queried_events(start, end)
  File "[...]/gcalcli/gcal.py", line 1157, in _display_queried_events
    event_list = self._search_for_events(start, end, search)
  File "[...]/gcalcli/gcal.py", line 1104, in _search_for_events
    events = self._retry_with_backoff(
  File "[...]/gcalcli/gcal.py", line 112, in _retry_with_backoff
    return method.execute()
  File "/usr/lib/python3/dist-packages/googleapiclient/_helpers.py", line 130, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/googleapiclient/http.py", line 856, in execute
    raise HttpError(resp, content, uri=self.uri)
googleapiclient.errors.HttpError: <HttpError 404 when requesting https://www.googleapis.com/calendar/v3/calendars/[...]&alt=json returned "Not Found">

If I manually retrieve that URL, I get:
{
  "error": {
    "code": 403,
    "message": "The request is missing a valid API key.",
    "errors": [
      {
        "message": "The request is missing a valid API key.",
        "domain": "global",
        "reason": "forbidden"
      }
    ],
    "status": "PERMISSION_DENIED"
  }
}

Well, and a few tries later, I now get:
{
 "error": {
  "errors": [
   {
    "domain": "usageLimits",
    "reason": "dailyLimitExceededUnreg",
    "message": "Daily Limit for Unauthenticated Use Exceeded. Continued use requires signup.",
    "extendedHelp": "https://code.google.com/apis/console"
   }
  ],
  "code": 403,
  "message": "Daily Limit for Unauthenticated Use Exceeded. Continued use requires signup."
 }
}

I'll try the steps outlined in #497.

I'll try the steps outlined in #497.

That worked.

hi have same problem.how you solved it ?
<HttpError 403 when requesting https://serviceusage.googleapis.com/v1/projects/loyal-theater-314713/services/iam.googleapis.com:enable?alt=json returned "Project 'loyal-theater-314713' not found or permission denied.". Details: "[{'@type': 'type.googleapis.com/google.rpc.PreconditionFailure', 'violations': [{'type': 'googleapis.com', 'subject': '?error_code=210002&type=Project&resource_id=loyal-theater-314713'}]}, {'@type': 'type.googleapis.com/google.rpc.ErrorInfo', 'reason': 'RESOURCES_NOT_FOUND', 'domain': 'serviceusage.googleapis.com', 'metadata': {'type': 'Project', 'resource_id': 'loyal-theater-314713'}}]">
<HttpError 403 when requesting https://serviceusage.googleapis.com/v1/projects/loyal-theater-314713/services/drive.googleapis.com:enable?alt=json returned "Project 'loyal-theater-314713' not found or permission denied.". Details: "[{'@type': 'type.googleapis.com/google.rpc.PreconditionFailure', 'violations': [{'type': 'googleapis.com', 'subject': '?error_code=210002&type=Project&resource_id=loyal-theater-314713'}]}, {'@type': 'type.googleapis.com/google.rpc.ErrorInfo', 'reason': 'RESOURCES_NOT_FOUND', 'domain': 'serviceusage.googleapis.com', 'metadata': {'type': 'Project', 'resource_id': 'loyal-theater-314713'}}]">
Creating accounts in saf-ppn1s-l0iktiz1bl7hadxwovrt
Creating accounts in transfer-315010
Creating accounts in loyal-theater-314713
Traceback (most recent call last):
File "gen_sa_accounts.py", line 323, in
download_keys=args.download_keys
File "gen_sa_accounts.py", line 233, in serviceaccountfactory
_create_remaining_accounts(iam,i)
File "gen_sa_accounts.py", line 33, in _create_remaining_accounts
sa_count = len(_list_sas(iam,project))
File "gen_sa_accounts.py", line 93, in _list_sas
resp = iam.projects().serviceAccounts().list(name='projects/' + project,pageSize=100).execute()
File "/usr/local/lib/python3.7/dist-packages/googleapiclient/_helpers.py", line 134, in positional_wrapper
return wrapped(*args, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/googleapiclient/http.py", line 915, in execute
raise HttpError(resp, content, uri=self.uri)
googleapiclient.errors.HttpError: <HttpError 404 when requesting https://iam.googleapis.com/v1/projects/loyal-theater-314713/serviceAccounts?pageSize=100&alt=json returned "Requested entity was not found.". Details: "Requested entity was not found.">

{
"error": {
"code": 403,
"message": "The request is missing a valid API key.",
"status": "PERMISSION_DENIED"
}
}

(... if only for my own later reference...)  ;-)
I've ran into this again, and followed the steps of https://github.com/insanum/gcalcli#login-information to restore access.

Today I figured out that in my last steps, https://github.com/insanum/gcalcli/issues/530#issuecomment-1277241149, Reset Client Secret is not actually necessary, but one may instead Continue [...] with the existing credentials as displayed on the page where you'd Reset Client Secret.