"Debugger engine has been disconnected unexpectedly"

All dump files are crashing, they were working before. I tried uninstalling and reinstalling WinDBG but got the same results. I tried to share the dump files but it says I can't share that without admin access which is weird, since I do have it on my account.

That error can happen when the debugger crashes. If you have Windows Error Reporting enabled, we might receiving dumps where we could investigate, though with how many we see, we'd also need a bit more information to try and find your specific crash.
If you can still reproduce the problem, maybe you could capture a dump or TTD trace of the debugger. The latter can be collected by passing -recordEngine some\directory - in that directory. If you can share the trace (probably EngHost01.run) then we could investigate.


refactor(example): examples overhaul

examples overhaul

[ ] We should also add a readme to the tortoise integration
[ ] We should add a brief top-level readme to the examples directory, explaining the separation between agent-core and network, how to navigate/use it

To improve maintainability (no redundancy, single source for documentation/definitions) I would suggest the following:

Only include example logic specific descriptions and explanations in the example's readme
put all "general" documentation into (a) separate file(s)
Link to any "general" description/documentation that is needed for an example (e.g., address, seed, bureau etc. basically everything that also should have a proper docstring)
If we switch from markdown to asciidocs we can even do includes to compile standalone files without linking and without copy-and-paste redundancies
eventually we should maintain valid, useful and up-to-date docstrings, which should be used to populate any base documentation (e.g., in example readmes or for general api documentation) and additional documentation would only be needed for concepts, architecture descriptions, examples, etc.

Good points @Dacksus. Definitely agree with both of the first tasks, and for these:

Only include example logic specific descriptions and explanations in the example's readme
put all "general" documentation into (a) separate file(s)
Link to any "general" description/documentation that is needed for an example (e.g., address, seed, bureau etc. basically everything that also should have a proper docstring)

Are you suggesting to provide links like this for all the relevant objects / concepts: https://fetch.ai/docs/references/uagents/uagents-api/context?


incorrect logged parameters in model file

There is a config2string after training model:
https://github.com/microsoft/LightGBM/blob/b1e5a843ab57c64b01c8896ef3cda6fdf907d417/src/io/config_auto.cpp#L549-L654
However, as some parameters are not visible in cpp side when using python/R package, such as num_iteration in python package. Therefore, some logged parameters are not correct.

to fix this, I think we should pass all parameters to cpp code, even though some of them are not used in cpp side.

So, what other configurations are needed here beside num_iteration? Would they differ based on the class where they are used; RF,GOSS or DART?

@Xiad32 these configurations just log the error values in the model file, it doesn't affect the normal usage.

Python-package has been fixed in #2589.
ping @jameslamb and @Laurae2 for R-package.

@StrikerRUS hey thanks for the ping, this one was not on my radar. I'll pick it up

@jameslamb Great! Thank you.


[FEATURE] Countdown the required Delta-V for a maneuver as you complete it

The required deltav will now dynamically update as the burn progresses. The burn time is not counting down though because that would require pretty expensive computation and the game itself does it in a very janky way as far as I can tell.

That's what I meant, excelent :)


Iroh-examples: Example how to sync a sqlite database

Use a sqlite database locally
Use the sqlite VFS for change tracking (or do brute force rehash)
Sync the sqlite database to other nodes
Have a "compute over data" thing that reads multiple sqlite dbs and updates an aggregated sqlite db.
https://www.sqlite.org/vfs.html

instead of VFS maybe just do selects on aggregate and update accordingly ?

iroh-node with sqlite has a table with sequential id's
when act on table iroh-node  notifies aggregate
iroh-node chrono aggregate select for last id compares with own and do a select on aggregate for fetch [new] range
if request is not in local (inbetween chrono sync) node selects on aggregate
aggregate need have trigger for id that changed [updated deleted] in separate table with ts-id, that iroh-node can select on and update its synced local since last ts-id range

besides syncing sqlites in iroh-band it could also index as mentioned elsewhere for gateway supernodes etc.

table with humanized key's containing tickets and / or addrs derp hash]type] fields

how about mix and match with DuckDB ? it has a sqlite extension a rust client and several imports
http://duckdb.org/docs/extensions/sqlite
http://duckdb.org/docs/extensions/inet


minor cleanup to avoid is_ok()

Minor style cleanup I noticed while reviewing https://github.com/sqlparser-rs/sqlparser-rs/pull/731 from @step-baby

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 86.335%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
12105

Relevant Lines:
14021

💛  - Coveralls


[08/12/2021] Third-party Usage: Swin-Transformer in Person Re-id

Thanks for the great repo.
I applied the swin transformer to the person re-id baseline, and enabling training and testing.
The code is at here https://github.com/layumi/Person_reID_baseline_pytorch
It would be great to involve the link in README/Third-party Usage and Experiments.
Thanks a lot~

Added, thanks!


BUG Add missing addStoredFields method

This has been documented in the readme for ages but never implemented.
Tested with localsolr with highlighting enabled.
cc @hafriedlander

cc @chillu

Thanks Damian, that was my bad around "docs without implementation" (78c5daa18613c04a6c0401b2257c101ab8f4631d). It's been a while, but from memory I've untangled a stream of work into half a dozen pull requests which partially relied on each other, which is why this one slipped through.


Support With Clause

https://dev.mysql.com/doc/refman/8.0/en/with.html#common-table-expressions-syntax

Hi, could I check if there are any plans for WITH statements to be supported?

I think an alternative workaround here is to use the parser in the Vitess project directly. Since PR https://github.com/vitessio/vitess/pull/8918 it "Add parsing for common table expressions using WITH clause"
I think the SQL Parser can be imported with import "vitess.io/vitess/go/vt/sqlparser" - but I think the API is different - it doesn't seem to have a sqlparser.Parse('[string of SQL]') method any more.

There's a second alternative workaround too - use DoltHub's hard-fork of the Vitess project, which is this one:
<EMAIL_ADDRESS>I got this working too, here: https://github.com/grafana/grafana/pull/96296/commits/45f9adf845baa030e74cc14e18d12b9453bd17d3


Implement Jetpack Compose for Jobs UI

JetPack Compose is a better way for generating UI from code. The entire site should be migrated to Jetpack Compose. Going to start with the Job page (that includes the workflow that is specific to the user).

https://developer.android.com/codelabs/jetpack-compose-migration#0

Progress as of Monday. Not likely to win any design awards, but a bit more

Here is the commit: https://github.com/dyor/scoop/commit/2abe795032848ef52f954d44cd4edfa1b6786cd3

Knocked the rest of the flow out last night. Still a lot of work to do in terms of actually using jetpack (I am hard coding a lot of things instead of relying on themes). But a reasonable start.

If you know Jetpack Compose, and would like to show me how it is done, drop a comment!!


WIP - CMake Argument testing

Pull Request Checklist:

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

This PR fixes #xyz

[ ] Tests for the changes have been added (for bug fixes / features)

[ ] (If applicable) Documentation has been added / updated (for bug fixes / features)

[ ] CHANGES.rst has been updated (with summary of main changes)

What kind of change does this PR introduce?

This forces the CMake build type via [tool.scikit-build.cmake.define] as well as [tool.scikit-build.cmake]

Does this PR introduce a breaking change?
No.
Other information:
I've created some development tags based on this branch and pushed packages to TestPyPI for testing on conda-forge. This is not working as intended yet.
https://github.com/conda-forge/raven-hydro-feedstock/pull/25

@huard
Update

The latest commit on main of RavenHydroFramework is building for Release!
This PR is currently set to the commit (but this must be tagged upstream before releasing raven-hydro).
The initial problem with the setting of the BUILD_TYPE is unresolved, but the default being set to Release should have no impact on conda-forge builds.

Next steps

Create a new release of RavenHydroFramework on GitHub
Update the commit of the submodule as well as the commit pointed to in the pyproject.toml, update changelog.
Merge this Pull Request to main.
Tag release of raven-hydro v0.3.1.
Update RavenPy to use the latest raven-hydro.


SOFTEXCO -The message "Cluj Napoca is not a city in Romania" is displayed instead of the job locations

ENVIRONMENT: production
URL: https://scrapers.peviitor.ro/
Browser: Chrome
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open the URL in the browser
Search for "Softexco" Company
Click on "Run Scraper" button
Check the job locations

EXPECTED RESULTS:
Job locations are the same in Scrapers-UI as on the company website
ACTUAL RESULTS:
The message "Cluj Napoca is not a city in Romania" is displayed instead of the job locations
ATTACHMENTS:

Fixed can you retest?

Re-Tested
Job locations are the same in Scrapers-UI as on the company website


ESPAsyncWebServer => PsychicHttp Migration

I am opening this issue to list and discuss  all the migration problems currently either preventing or making me abandon the move to Psychic in my project and libraries.
For now, if just migrating simple code like that requires me to rewrite completely my libraries and duplicate 90% of the code, this is a problem.
Example:
#ifdef ESPCONNECT_USE_PSYCHIC
  server.on("/", HTTP_GET, [&](PsychicRequest* request) {
    return request->reply(200, "text/plain", "Hello World!");
  })->setFilter([](__unused PsychicRequest* request) { return ESPConnect.getState() != ESPConnectState::PORTAL_STARTED; });
#else
  server.on("/", HTTP_GET, [&](AsyncWebServerRequest* request) {
    request->send(200, "text/plain", "Hello World!");
  }).setFilter([](__unused AsyncWebServerRequest* request) { return ESPConnect.getState() != ESPConnectState::PORTAL_STARTED; });
#endif

Proposition 1: Return references instead of pointers
This specifically applies to:
    PsychicEndpoint* on(const char* uri);
    PsychicEndpoint* on(const char* uri, http_method method);
    PsychicEndpoint* on(const char* uri, PsychicHandler *handler);
    PsychicEndpoint* on(const char* uri, http_method method, PsychicHandler *handler);
    PsychicEndpoint* on(const char* uri, PsychicHttpRequestCallback onRequest);
    PsychicEndpoint* on(const char* uri, http_method method, PsychicHttpRequestCallback onRequest);
    PsychicEndpoint* on(const char* uri, PsychicJsonRequestCallback onRequest);
    PsychicEndpoint* on(const char* uri, http_method method, PsychicJsonRequestCallback onRequest);

That I would like to see changed into:
    PsychicEndpoint& on(const char* uri);
    PsychicEndpoint& on(const char* uri, http_method method);
    PsychicEndpoint& on(const char* uri, PsychicHandler *handler);
    PsychicEndpoint& on(const char* uri, http_method method, PsychicHandler *handler);
    PsychicEndpoint& on(const char* uri, PsychicHttpRequestCallback onRequest);
    PsychicEndpoint& on(const char* uri, http_method method, PsychicHttpRequestCallback onRequest);
    PsychicEndpoint& on(const char* uri, PsychicJsonRequestCallback onRequest);
    PsychicEndpoint& on(const char* uri, http_method method, PsychicJsonRequestCallback onRequest);

This allows to chain calls with setFiter like ESPAsyncWebServer
And why would the user be anyway in control of a pointer that is managed by the library ?

Proposition 2: Either match ESPAsyncWS method names or provide a migration path (compile flag ?)
Examples:
send vs reply:
#ifdef PSYCHIC_TO_ESPASYNCWS
    esp_err_t send(int code) { return reply(code); }
    esp_err_t send(int code, const String& contentType = emptyString, const String& content = emptyString);
#endif

constructor:
PsychicHttpServer(uint16_t port = 80);

begin / end vs listen / stop:
#ifdef PSYCHIC_TO_ESPASYNCWS
    void begin(uint16_t port = 80) { listen(port); }
    void end() { stop(); }
#endif

etc...

Proposition 3: Provide a way to map callbacks and classes
Example:
#ifdef PSYCHIC_TO_ESPASYNCWS
  #include <espasyncws.h>
#endif

content:
#pragma once

#include "PsychicHttpServer.h"
#include "PsychicWebHandler.h"
#include "PsychicRequest.h"

using AsyncWebServer = PsychicHttpServer;
using AsyncCallbackWebHandler = PsychicWebHandler;
using AsyncWebServerRequest = PsychicRequest;

Note: I am not sure this is the right way to do it, it could also be done within the library classes, example right now I am trying to migrate by using that in a library:
#ifdef ESPCONNECT_USE_PSYCHIC
    using Server = PsychicHttpServer;
    using WebHandler = PsychicWebHandler;
#else
    using Server = AsyncWebServer;
    using WebHandler = AsyncCallbackWebHandler;
#endif

So this is scoped to only my classes, but if this concept works, why not extending it globally and have Psychic provide a mapping class.

Proposition 4: make handlers return `void:
typedef std::function<void(PsychicRequest *request)> PsychicHttpRequestCallback;
typedef std::function<void(PsychicRequest *request, JsonVariant &json)> PsychicJsonRequestCallback;

I don't understand why a handler would return an internal ESP status code: a handler is linked to serving a request and any status code within this context should lead to an http status code answer.
Those are acceptable:
    esp_err_t redirect(const char *url);
    esp_err_t reply(int code);
    esp_err_t reply(const char *content);
    esp_err_t reply(int code, const char *contentType, const char *content);

Because they are linked to the underlying API:
*  - ESP_OK : On successfully sending the response packet
 *  - ESP_ERR_INVALID_ARG : Null request pointer
 *  - ESP_ERR_HTTPD_RESP_HDR    : Essential headers are too large for internal buffer
 *  - ESP_ERR_HTTPD_RESP_SEND   : Error in raw send
 *  - ESP_ERR_HTTPD_INVALID_REQ : Invalid request
 */
esp_err_t httpd_resp_send(httpd_req_t *r, const char *buf, ssize_t buf_len);

And one would want to know if his request is sent correctly, and act upon this result within the handler.
But what is the use case for having a handler returning an error code ?

created individual threads for these for easier discussion


How to use the model trained with lifeiteng's code to inference

I have trained a model with lifeteng's code, so how can I used this model with this repository? Could you please provide a guideline to do that.

lifeiteng's model class does not contain ar_language_embedding and nar_language_embedding by default. To directly load your trained state dictionary, you have to manually initialize them to be all-zero matrix (if your trained model is monolingual, else you have to find a way to make your language embeddings compatible to this repo's), and then you should be able to use it flawlesslly.


CUDA out of memory with a 3090

When I try to run gradio_new.py I'm getting a CUDA OOM error, despite having >23GB of available memory. Is there something in the config I can tweak to overcome this?

Just push a small fix. Could you try again?

It works! Thank you


Added scikit-image

cc @jni
re-deploying with this commit now.

Our conda was pretty old, so had to update that so scipy 1.0+ could be found in the new defaults channel. Hopefully that doesn't break things 😬

@TomAugspurger SO COOL.
btw, I noticed the mem examples in the asv site, but I don't see any memory profiling happening. Is that a special setting, or something that was attempted but abandoned, or...?

https://github.com/scikit-image/scikit-image/pull/3215 for the memory test (need to return the object being tracked)


Focus search input when modal shown

This is a small hack for now.
We should find a cleaner way to solve this issue.
It works fow now :)
Closes #78

Part of me feels like we should honor the site/theme keyboard shortcut, even though it's in conflict with our UI. This is the main problem with relying on keyboard shortcuts, it's always an arms race of sorts to ensure your code captures the event first.
The approach with firing a timer off probably works for most cases, but it will effectively duplicate the key press event. If the theme/site handler for the existing key press has side effects other than focus, like showing up of a separate search modal or something, then the reader will get that UI and our search modal UI. That is, the use of a timer doesn't intercept the event, but duplicates it.
I know you can manipulate previously registered event handlers, however I've never done this for third party code. Normally, you know the exact function you are trying to deregister. If you can find the existing handler, it's possible to wrap that handler and intercept the keypress event for just /, bubbling all other event through to the handler.

Part of me feels like we should honor the site/theme keyboard shortcut, even though it's in conflict with our UI.

I'm not sure what this means. Can you expand on this?

The approach with firing a timer off probably works for most cases, but it will effectively duplicate the key press event. If the theme/site handler for the existing key press has side effects other than focus, like showing up of a separate search modal or something, then the reader will get that UI and our search modal UI. That is, the use of a timer doesn't intercept the event, but duplicates it.

Yeah, I've already hit this with Material for MkDocs, yeah. I'm not sure what to do in those cases.

I know you can manipulate previously registered event handlers, however I've never done this for third party code. Normally, you know the exact function you are trying to deregister. If you can find the existing handler, it's possible to wrap that handler and intercept the keypress event for just /, bubbling all other event through to the handler.

I researched about this and I wasn't able to de-register event handlers from others. I'm tracking this case and the one from the previous paragraph in https://github.com/readthedocs/addons/issues/25

I know the hack from this PR is not good, but I don't have anything better at the moment and I thought that at least it was better than the current behavior, but I know this is not the solution 🤷🏼

I'm not sure what this means. Can you expand on this?

Just that we shouldn't try to replace the override keyboard shortcut. These shortcuts may not be there for any particular reason, but unfortunately the shortcuts might also be important for the site.

Yeah, I've already hit this with Material for MkDocs, yeah. I'm not sure what to do in those cases.

Ah yes, that has a UI there too. There really isn't a great answer here, as we don't control the website we're injecting the features into. I don't think this is the primary method we want for triggering this UI long term anyways though, so maybe this is okay.
This, among other reasons like screen reader overlap, is why I'm generally -1 on implementing keyboard shortcuts (and I've been known to use qutebrowser!)

I researched about this and I wasn't able to de-register event handlers from others.

Yeah, I can only say it's possible when you have the instance of the function handy. Otherwise, I don't know how feasible deregistering is.

I know the hack from this PR is not good, but I don't have anything better at the moment

I might lean towards not doing anything magical or unexpected here. If the user wants our search/flyout/etc, they should disable whatever is interfering with the hotkeys. Long term, we should avoid relying on shortcuts to trigger our UI, but I understand this is just a short term solution to avoid UI that we haven't added yet.
We could make a secondary keyboard shortcut available perhaps? ie / or C-k?

Ah yes, that has a UI there too. There really isn't a great answer here, as we don't control the website we're injecting the features into. I don't think this is the primary method we want for triggering this UI long term anyways though, so maybe this is okay.

Agreed. I imagine we can probably just document "How to disable the default search on Material for MkDocs" and "How to disable the default search on Sphinx" and other doctools giving people a chunk of code or similar and rely on that.

This, among other reasons like screen reader overlap, is why I'm generally -1 on implementing keyboard shortcuts (and I've been known to use qutebrowser!)

Regarding using or not using keyboard shortcuts at all, we have this conversation opened: https://github.com/readthedocs/addons/issues/80. I was on the fence, but considering that GitHub makes extensive usage of them, I'm kind of convinced they are not bad.

I might lean towards not doing anything magical or unexpected here. If the user wants our search/flyout/etc, they should disable whatever is interfering with the hotkeys. Long term, we should avoid relying on shortcuts to trigger our UI, but I understand this is just a short term solution to avoid UI that we haven't added yet.

Yes, even having a UI element this may be required. Well, at least right now that we are using an input field inside the flyout that triggers the search modal. This is because the focus stays in the input field from the flyout instead of jumping to the input from the search modal. However, with this PR, it works fine and moves to the input from the search modal. At least, it fixes that issue 😄

We could make a secondary keyboard shortcut available perhaps? ie / or C-k?

Yes. I did some tests with C-k, but it requires a little more work since you also have to check that none of the other modifiers are pressed and things like that. It was not trivial / quick when I tried it. GitHub uses s or / to open the search bar (see #80).

Are we OK merging this half-baked solution for now? At least it solves the problem in our own theme. We can continue with the proper solution in the other issues that I linked.

Agreed. I imagine we can probably just document "How to disable the default search on Material for MkDocs" and "How to disable the default search on Sphinx" and other doctools giving people a chunk of code or similar and rely on that.

:+1: This seems like a great place to start for now. If we do have to hack things in later to support themes/tools, I'd consider that work polish level adjustments for later in our roadmap. I bet we'll find better solutions in the meantime though.

Regarding using or not using keyboard shortcuts at all, we have this conversation opened: https://github.com/readthedocs/addons/issues/80. I was on the fence, but considering that GitHub makes extensive usage of them, I'm kind of convinced they are not bad.

Aye, though I'm generally -1, I still feel shortcuts are okay as an addition to addons.
I just don't like focusing on this as a primary part of our UX as it's much harder to discover and use as a feature, especially for less technical users. As an alternative UX, it's usually okay though, as long as adaptive technologies and normal browser use aren't negatively affected.

Yes. I did some tests with C-k, but it requires a little more work

Yup, modifiers make this more complicated. Keyboard support is a rabbit hole and there is a lot that goes into doing keyboard support well.

Are we OK merging this half-baked solution for now?

There are definitely some downsides to this approach, and I feel there is probably a more standard fix for the problem. The focus issue is almost certainly either an async code issue or a matter of CSS transitions throwing the timing off.
I'd try to rule out async troubles first, as it seems the most likely. I believe we need to be using the updateComplete event, as the focus() call is firing before the template renders. For example, this fix describes updateComplete usage on an element in the template:
https://github.com/lit/lit-element/issues/605#issuecomment-471316709
Otherwise, I would have assumed that the Element.updated event is what we would have wanted to sync the event up correctly:
https://lit.dev/docs/v1/components/lifecycle/#updated
I'd give those a try first if you haven't already.

Otherwise, I would have assumed that the Element.updated event is what we would have wanted to sync the event up correctly

Excellent! This was it 🚀 . In fact, in the documentation link you shared, there is the exact same case I need here 😅 . It works perfect now without any type of hack.

Aye, though I'm generally -1, I still feel shortcuts are okay as an addition to addons.

Man, I don't know if this is what you are thinking here but maybe the "keyboard shortcuts" is an addon on itself that people can enable/disable 🤯 . I'm now convinced this is the way to go here. I will open an issue for this.

Man, I don't know if this is what you are thinking here but maybe the "keyboard shortcuts" is an addon on itself that people can enable/disable 🤯 . I'm now convinced this is the way to go here. I will open an issue for this.

Boom! 💣 I implemented this idea in https://github.com/readthedocs/addons/pull/118

Agreed. I imagine we can probably just document "How to disable the default search on Material for MkDocs" and "How to disable the default search on Sphinx" and other doctools giving people a chunk of code or similar and rely on that.

👍 This seems like a great place to start for now. If we do have to hack things in later to support themes/tools, I'd consider that work polish level adjustments for later in our roadmap. I bet we'll find better solutions in the meantime though.

Sphinx also catches / to focus on the search input from the navbar: https://github.com/sphinx-doc/sphinx/blob/fcc38997f1d9b728bb4ffc64fc362c7763a4ee25/sphinx/themes/basic/static/doctools.js#L146
It seems there is an option to disable it, DOCUMENTATION_OPTIONS.ENABLE_SEARCH_SHORTCUTS, but I didn't dig enough to know where to put that yet.

This PR is ready to be merged 👍🏼

Excellent, I'm glad this approach worked out. I think your idea to split out keyboard shortcuts as it's own addon is an interesting one, definitely worth testing that more. But I like that we have a way to surface configuration of hotkeys already :+1:


Simplify OSB Docker Image

Is your feature request related to a problem? Please describe.
OSB Docker Image currently copies over the github repository directories. It's also unwieldy.
A clear and concise description of what you want to happen.
Since PyPi and Dockerhub publishing workflows on Jenkins happen in chronological order (PyPi first then Dockerhub second), we can just install the PyPi package instead of git cloning and running python3 -m pip install -e ..
A clear and concise description of any alternative solutions or features you've considered.
Add any other context or screenshots about the feature request here.

Merged in revised Dockerfile


Added Resume Functionality for Full Finetuning Script (full.py)

Overview:
This pull request introduces an enhancement to the full.py script, enabling the capability to resume finetuning from the latest checkpoint. This was necessary because our research cluster has a 24 hour time limit for jobs.
Key Changes:
Resume Functionality: The updated script now includes the ability to resume finetuning from the most recent checkpoint.
Checkpointing Logic: Checkpoints will now save the full state, encompassing both the model and optimizer, contingent upon the full_checkpointing variable.
Conditional Handling: When the resume option is not activated and full_checkpointing is set to False, the script will operate in the same manner as before, ensuring compatibility and maintaining existing behavior.

cc @awaelchli since you originally wrote resume functionality


Camel cased transitions do not produce handlers

While attempting to name a transition startProcess, I found that the connect mappings did not produce a handler for this transition. After renaming it to "start process", the startProcess() handler was mapped as expected.
Are state names not allowed in camel case?
After reading the docs, I was expecting the behavior to be:

if spaces exist in transition name -> transform to camel case
if dashes exist in transition name -> transform to camel case
else -> leave as is

That's for sure a bug. I'll try to fix it asap.

@aryzing can you please try the new 5.0.3 version. Should fix the problem.
P.S.
by the way I'm interested in receiving feedback for the library. In what cases you guys used it. Is it working for you and so on.

Thanks for the quick fix! I'll update to the latest version now.
I'll catch up with you with feedback later: I only started working with it yesterday, but so far so good, I'm liking it quite a lot 👍

Just tested. It works as expected 👍  Good job guys !


Clicking on buttons requires a small drag to register on macOS Sonoma

On macOS Sonoma, UVR5 GUI does not recognize mouse clicks unless the mouse is dragged slightly while clicking.

I'm having the same issue on MacOS Sonoma 14.1 too, most mouse clicks are totally unresponsive.

I'm having the same issue on MacOS Sonoma 14.1 too, most mouse clicks are totally unresponsive.

I'm not able to test this on Sonoma yet. Is this happening with every button or just specific ones?

This happens on all interactive UI elements: dropdown menus, radio buttons, checkboxes... etc


I want to test pull requests

I want to test pull requests

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


chore(TU-11776): Update developers portal docs

[BOT] Preview available with hash d997c8466fc849d8bc0c9f3218ac37c35091e361 here.


Useless variable can't be deleted unless refresh the page

Repro steps:
• Navigate to https://makecode.microbit.org/
• Drag blocks as below picture
• Create a new variable named "test" and try to delete it
Expected result:
Variable "test" could be deleted
Actual result:
Variable "test" can't be deleted unless refresh the page

Won't fix. This is by design in blockly.


Feature Request: Support Long Press To Boost/Fav From Other Accounts

Hi @videah, like I've mentioned elsewhere I'm absolutely loving SkyBridge, it's helped me be a lot more active on Bluesky in lieu of a good native client. I did have a request though, it may not be possible but I was wondering if it's possible to add functionality for boosting/favoriting from other logged in accounts like this?

My goal is to boost posts from an alt account, one where I like to curate a collection of good posts. The way Ivory accomplishes this on Mastodon is by long-pressing on the boost or star icons. I was wondering if it's possible to similarly route the Bluesky boost/fav request to another logged in Bluesky account when a user long-presses and chooses a different Bluesky account [logged in on Ivory through SkyBridge].
Thank you so much!

Sorry for the wait, should now be implemented as of 67ecba910202e9f03ebfcfc049088cc04b32fb6d!

Oh wow, thank you so much @videah!


git background when repository has changes not showing

Hello everybody,
I checked the common issues sections and couldn't find an answer to my problem.
Describe Your Issue
My issue is, when having changes into a repository, the background for the branch is not showing.
I'll attach an image here:
https://ibb.co/rvkWkk5
Have you tried to debug or fix it?
I've tried to debug it, but as my shell knowledge is limited, I was not being able to solve it.
Environment Information

What version of ZSH are you using?
zsh 5.8 (x86_64-apple-darwin19.3.0)
Do you use a ZSH framework (e.g., Oh-My-ZSH, Antigen)?
Oh-My-ZSH
How did you install P9k (cloning the repo, by tarball, a package from your OS, etc.,)?
I cloned the repo
What version of P9k are you using?
v0.6.7
Which terminal emulator do you use?
iterm2

Thank you everybody for your help, I'm sorry if I'm too noob to be able to solve this issue by myself

Have you set any P9K specific variables or is that happening with no extra configuration? If you have tried to customize or set any variables then the settings from your .zshrc or like file would be great to see. It looks like your background color is just set to black or some dark value when the git segment detects that git is not in a clean state.

Hello @Thutm ,
First I want to thank you for your time :)
I was using this tutorial:
https://medium.com/@Clovis_app/configuration-of-a-beautiful-efficient-terminal-and-prompt-on-osx-in-7-minutes-827c29391961
So far, I've just used this p9k variables:
POWERLEVEL9K_MULTILINE_LAST_PROMPT_PREFIX="%{%B%F{black}%K{yellow}%} $user_symbol%{%b%f%k%F{yellow}%} %{%f%}"
source /usr/local/share/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh
POWERLEVEL9K_PROMPT_ADD_NEWLINE=true
POWERLEVEL9K_VCS_MODIFIED_BACKGROUND=’red’

My .zshrc file looks like this:
export ZSH="/Users/DaRo/.oh-my-zsh"

ZSH_THEME="powerlevel9k/powerlevel9k"

plugins=(
	git
	zsh-autosuggestions
	)

source $ZSH/oh-my-zsh.sh

POWERLEVEL9K_LEFT_PROMPT_ELEMENTS=(dir rbenv vcs)
POWERLEVEL9K_RIGHT_PROMPT_ELEMENTS=(status root_indicator background_jobs history time)
POWERLEVEL9K_PROMPT_ON_NEWLINE=true

# Add a space in the first prompt
POWERLEVEL9K_MULTILINE_FIRST_PROMPT_PREFIX="%f"
# Visual customisation of the second prompt line
local user_symbol="$"
if [[ $(print -P "%#") =~ "#" ]]; then
    user_symbol = "#"
fi
POWERLEVEL9K_MULTILINE_LAST_PROMPT_PREFIX="%{%B%F{black}%K{yellow}%} $user_symbol%{%b%f%k%F{yellow}%} %{%f%}"
source /usr/local/share/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh
POWERLEVEL9K_PROMPT_ADD_NEWLINE=true
POWERLEVEL9K_VCS_MODIFIED_BACKGROUND=’red’

# Colorise the top Tabs of Iterm2 with the same color as background
# Just change the 18/26/33 wich are the rgb values
echo -e "\033]6;1;bg;red;brightness;18\a"
echo -e "\033]6;1;bg;green;brightness;26\a"
echo -e "\033]6;1;bg;blue;brightness;33\a"

export NVM_DIR="/Users/DaRo/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"  # This loads nvm
export PATH="/usr/local/opt/mysql-client/bin:$PATH"

Where could I find the part where the colour for when git segment detects that git is not in a clean state is set? That's the part that I could never find.
If I can give you more info, please ask!
Thank you very much again!!!
Regards,
Daniel

Try these:
POWERLEVEL9K_VCS_CLEAN_BACKGROUND
POWERLEVEL9K_VCS_CLEAN_FOREGROUND
POWERLEVEL9K_VCS_MODIFIED_BACKGROUND
POWERLEVEL9K_VCS_MODIFIED_FOREGROUND
POWERLEVEL9K_VCS_UNTRACKED_BACKGROUND
POWERLEVEL9K_VCS_UNTRACKED_FOREGROUND

That's perfect!
I can confirm you that the parameter:
POWERLEVEL9K_VCS_UNTRACKED_BACKGROUND='yellow'
It was missing.
After adding it to the config, everything seems normal now.
Thank you for your help :)
Regards,
Daniel

Not a problem, glad you got it working. If all is well, could you close out your issue?

For sure, I'll do it, thank you again :)


Plugin installation fails after 2.2.3 release for existing apps

Expected Behaviour
Assuming semantic versioning, releasing a patch release 2.2.3 would not cause existing apps to stop building.
Actual Behaviour
Existing apps configured to user phonegap-plugin-push ^2.2.0 will stop working. The plugin will fail to install because 2.2.3 introduced a dependency on cordova-support-google-services:
Installing "phonegap-plugin-push" for android

Failed to get module path for cordova-support-google-services. Make sure it is installed manually via NPM.

Steps to Reproduce

Have an app that has been working for a long time with phonegap-plugin-push ^2.2.0
Try to do a fresh build today, npm will fetch 2.2.3 and will fail to install the plugin because of the missing dependency

I'm not sure if this plugin is using strict semver versioning, but this patch release breaks existing apps as described above. They will need to be changed to use an exact version (2.2.2), or install this new dependency.
In addition, there is currently no documentation for the change in installation requirements for new apps.

Here is the relevant PR that added this dependency:
https://github.com/phonegap/phonegap-plugin-push/pull/2293

@MichaelMarner Can you give me more details on do a fresh build? I'm trying to reproduce the issue but haven't hit upon the correct incantation.

This error came from our CI machines but I also replicated it locally. In this case a fresh build means:

Checkout source code from git
npm install
cordova platform add android (or iOS)

Fails when installing the push plugin with the error noted above.

Hey thanks for your time but I think this may have been an issue with our CI environment or some kind of caching issue. I've spent a bit of time looking into it and have found the google support plugin being added automatically when adding the platform:
Discovered plugin "phonegap-plugin-push" in config.xml. Adding it to the project
Installing "phonegap-plugin-push" for android
Installing "cordova-support-google-services" for android

Sorry for the mis-report.

@MichaelMarner okay, no problem. I'm just glad it got sorted for you.


[SR-8151] Class inheriting from class existential behaviour doesn't match SE-0156

Previous ID
SR-8151

Radar
None

Original Reporter
@hamishknight

Type
Bug

Environment
Apple Swift version 4.2-dev (LLVM 9d4565013d, Clang f12d22bf9f, Swift 6982d653de)
Target: x86_64-apple-darwin17.6.0

Additional Detail from JIRA

Votes
0

Component/s
Compiler

Labels
Bug

Assignee
None

Priority
Medium

md5: ee8dafed896ec4393455a7ad1df822bc

relates to:

SR-7405 Protocol compositions: improve composing diagnostics

Issue Description:
In SE-0156, the following rule is given:

To improve readability and reduce confusion, a class conforming to a typealias which contains a class type constraint does not implicitly inherit the class type: inheritance should stay explicit. Here are a few examples to remind what the current rules are and to make the previous sentence clearer:
The proposal does not change the rule which forbids using the protocol composition syntax in the inheritance clause:
protocol P1 {}
protocol P2 {}
class C {}
class D : P1 & P2 {} // Compiler error
class E : C & P1 {} // Compiler error

Class D in the previous example does not inherit a base class so it can be expressed using the inheritance/conformance syntax or through a typealias:
class D : P1, P2 {} // Valid
typealias P12 = P1 & P2
class D : P12 {} // Valid

Class E above inherits a base class. The inheritance must be explicitly declared in the inheritance clause and can't be implicitly derived from a typealias:
class E : C, P1 {} // Valid
typealias CP1 = C & P1
class E : CP1 {} // Compiler error: class 'E' does not inherit from class 'C'
class E : C, CP1 {} // Valid: the inheritance is explicitly declared

However this rule doesn't appear to match the current implementation. The following doesn't yield a compiler error, despite the proposal saying the inheritance must be spelled explicitly:
protocol P1 {}
protocol P2 {}
class C {}
typealias CP1 = C & P1
class E : CP1 {}

and the following doesn't compile, despite the proposal saying that it should:
protocol P1 {}
protocol P2 {}
class C {}
typealias CP1 = C & P1
class E : C, CP1 {} // error: Multiple inheritance from classes 'C' and 'C'

I believe this has become the expected behaviour by now.


Authentication Options

[Enter feedback here]
Is storage access key the only option to mount file share?
How about SAS or RBAC permission?
when mount file share to my Linux box, I use cifs protocol, what about the encryption in transit, is it encrypted?

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

ID: 37e08943-3758-7ed5-5a50-4c649fbf0993
Version Independent ID: 3ce971db-1349-3b63-cf23-153c62808f23
Content: Mount Azure File storage on Linux VMs using SMB - Azure Linux Virtual Machines
Content Source: articles/virtual-machines/linux/mount-azure-file-storage-on-linux-using-smb.md
Service: virtual-machines-linux
GitHub Login: @cynthn
Microsoft Alias: cynthn

@RonnieQuan Typically a file share is mounted using a Storage account key.
It is possible to use Identity based authentication using Active Directory on Windows VMs.
For Encryption, there is no additional encryption ontop of the SMB protocol. SMB does encrypt the data.
@SumanthMarigowda-MSFT can you confirm if SAS keys will work to mount a file share?

@RonnieQuan  You can't mount Azure file share using SAS.
You can share your feedback: https://feedback.azure.com/forums/217298-storage/suggestions/14498352-allow-azure-files-shares-to-be-mounted-using-sas-s

@RonnieQuan Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.


1.20.2

Includes the work from #336.
Might add #337.
Still a WIP but its good progress.

Stuff still needing to be tested:

[ ] Fabric Registry Sync API. This changed with 1.20.2
[ ] Status Effect Data Fixer

Stuff that still could be done:

[ ] Add #337

@LambdAurora if you could push a commit with the licenses updated that would be great since I am stuck on windows.

@LambdAurora if you could push a commit with the licenses updated that would be great since I am stuck on windows.

I'm a Linux user as well btw

I'm a Linux user as well btw

Ah yes there was another I couldn't remember.

Caused by: org.gradle.api.GradleException: License header checks have failed on 12 out of 71 files.
Might want to take a peek at that.

Build passes now

Ready for review!

woo!


[chore] re-organize railgun manager

What this PR does / why we need it:
This re-organizes some of the code in railgun, particularly around the controller manager in anticipation of the upcoming beta release.
One of the main features of this change is that manager and controller code is no longer part of our public API, and has been moved to our private API (e.g. we're not supporting linking to the manager/controller libraries from outside).

Controller/Operator mainly deploy stateful distributed applications - ingress.
What does it mean " this change is that manager and controller code is no longer part of our public API, and has been moved to our private API (e.g. we're not supporting linking to the manager/controller libraries from outside)".
Our controller does not have any endpoints for user consumption afaik, but we would like to migrate to service module in the long term ?

Controller/Operator mainly deploy stateful distributed applications - ingress.
What does it mean " this change is that manager and controller code is no longer part of our public API, and has been moved to our private API (e.g. we're not supporting linking to the manager/controller libraries from outside)".
Our controller does not have any endpoints for user consumption afaik, but we would like to migrate to service module in the long term ?

This means that the Golang packages for this code can not be included in packages outside this codebase, the compiler will complain that the packages are internal.

Controller/Operator mainly deploy stateful distributed applications - ingress.
What does it mean " this change is that manager and controller code is no longer part of our public API, and has been moved to our private API (e.g. we're not supporting linking to the manager/controller libraries from outside)".
Our controller does not have any endpoints for user consumption afaik, but we would like to migrate to service module in the long term ?

This means that the Golang packages for this code can not be included in packages outside this codebase, the compiler will complain that the packages are internal.

Confusing - Controller/Operator has not been SDK, API, Service, utility library, a module implement controller pattern, in what case user use the package code ? you mean they refer the way the controller is implemented.

Controller/Operator mainly deploy stateful distributed applications - ingress.
What does it mean " this change is that manager and controller code is no longer part of our public API, and has been moved to our private API (e.g. we're not supporting linking to the manager/controller libraries from outside)".
Our controller does not have any endpoints for user consumption afaik, but we would like to migrate to service module in the long term ?

This means that the Golang packages for this code can not be included in packages outside this codebase, the compiler will complain that the packages are internal.

Confusing - Controller/Operator has not been SDK, API, Service, utility library, a module implement controller pattern, in what case user use the package code ? you mean they refer the way the controller is implemented.

Here's an example that I hope will illustrate the problem:
$ pushd $(mktemp -d)
$ go mod init test
$ cat -> main.go << EOF
package main

import (
	"fmt"

	"github.com/kong/kubernetes-ingress-controller/railgun/pkg/config"
)

func main() {
	_ = config.Config{
		LogLevel: "debug",
	}
	fmt.Println("It works! able to import config.Config from KIC!")
}
EOF
$ go get github.com/kong/kubernetes-ingress-controller@next
$ go mod tidy && go mod verify
$ go run main.go
It works! able to import config.Config from KIC!

After this PR merges it would no longer work:
$ go run main.go
package command-line-arguments
	main.go:4:2: use of internal package github.com/kong/kubernetes-ingress-controller/railgun/internal/config not allowed

Let me know if that helps.


RowBasedKeySerde should use empty dictionary in constructor

Fixes #6255

Hi @gaodayue, I left a comment on https://github.com/apache/incubator-druid/issues/6255.

We should probably backport this to 0.12.3

Could you please add a comment about why the map is starting out empty (avoiding allocating too much when it's not needed)? Otherwise, a future contributor might not realize it.

Thank you @gianm . It's a good advice, but the PR is merged. Maybe we can add it in future optimization.


Saving and Loading Objax EMLPs yields slightly different predictions

It appears loading up an EMLP models saved with objax.io.save_var_collection yields slightly different predictions than the original model.
import emlp
from emlp.groups import SO
from emlp.reps import T,V
import numpy as np
import objax

net = emlp.nn.EMLP(T(1)+T(2),T(0),SO(3))
x=np.linspace(0,1,12)
print(net(x).T)

# Saving with file descriptor
with open('net.npz', 'wb') as f:
    objax.io.save_var_collection(f, net.vars())

Output: [0.4905533]
import emlp
from emlp.groups import S
from emlp.reps import T,V
import numpy as np
import objax

net = emlp.nn.EMLP(T(1)+T(2),T(0),SO(3))
x=np.linspace(0,1,12)
# Loading with file descriptor
with open('net.npz', 'rb') as  f:
    objax.io.load_var_collection(f, net.vars())

print(net(x).T)

Output [0.4904544]

It looks like this is due to randomness in the bilinear layer that is not captured as objax state variables. A workaround is to use the same numpy seed when initializing the model in both cases:
import emlp
from emlp.groups import SO
from emlp.reps import T,V
import numpy as np
import objax

np.random.seed(42)
net = emlp.nn.EMLP(T(1)+T(2),T(0),SO(3))
x=np.linspace(0,1,12)
print(net(x).T)

# Saving with file descriptor
with open('net.npz', 'wb') as f:
    objax.io.save_var_collection(f, net.vars())

Output: [0.03251831]
import emlp
from emlp.groups import S
from emlp.reps import T,V
import numpy as np
import objax

np.random.seed(42)
net = emlp.nn.EMLP(T(1)+T(2),T(0),SO(3))
x=np.linspace(0,1,12)
# Loading with file descriptor
with open('net.npz', 'rb') as  f:
    objax.io.load_var_collection(f, net.vars())

print(net(x).T)

Output [0.03251831]


Fix the unrecognized custom error issue in tests (probably hardhat error)

And remove the extra check in assert-revert.js when is fixed
Related to https://github.com/nomiclabs/hardhat/issues/1996

Checked if this was fixed today, but seems like not yet. I asked to bump the priority tho.

Had another look at this today. Still, no fixes from hh it seems.
Opened a PR to be a bit more explicit: https://github.com/Synthetixio/synthetix-v3/pull/667
The problem still seems to be that the first time tests are run errors are not parsed, but runs fine the second time.


cluster upgrade not working

What happen?

Relevant log output?
No response
What you expected to happen?
No response
How to reproduce it (as minimally and precisely as possible)?
No response
Anything else we need to know?
No response
What is the version of Sealer you using?
v0.10
What is your OS environment?
No response
What is the Kernel version?
No response
Other environment you want to tell us?

Cloud provider or hardware configuration:
Install tools:
Others:

Is the upgrade function removed?

@clcc2019 ， it not removed. but upgrade action also need special sealer image to do real upgrade

@clcc2019 ， it not removed. but upgrade action also need special sealer image to do real upgrade

how do change the mirror image

@kakaZhou719 The previous version was implemented by code, why do we need mirror support now, I think it has become complicated

I need to implement it through Plugin?

I need to implement it through Plugin?

yes, you need to build your own upgrade image with  upgrade plugin. FYI: http://sealer.cool/docs/concept/clusterfile.html#plugin-api


Pie Chart

As Chart JS Integration’s user,
I need to have pie chart
So that I can have pie chart view

Done in v.0.0.11


[Infrastructure] Create Mock ImageStore class for testing

Current Situation

All Provider API Scripts in the src/cc_catalog_airflow/dags/provider_api_scripts directory use the ImageStore class defined in src/cc_catalog_airflow/dags/provider_api_scripts/common/storage/image.py to verify and store image data that they collect.  A number of tests for these scripts use that class indirectly through some function in the script.  Because that class calls the internet and writes to disk, this can lead to problems with tests failing for network reasons, and files being written to disk unintentionally.
Suggested Improvement

We should create a MockImageStore class that can be used for testing Provider API Scripts.  This mock class should inherit most functionality from the original, but override anything calling the internet or writing to disk.
Benefit

We'll be able to make sure that tests using the ImageStore class to verify the data formats they pass will still test what they're intended to, without the problems listed above.
Alternatives

We could mock this in each test file, or try to avoid tests using the class for verification of data formats, but then we'd have to require much more knowledge from each implementer of a Provider API Script about what data is and isn't valid.

Another approach can be to create something like mock_resources, which can contain all the common mocks and patches used repeatedly in test scripts. These can be configured and used accordingly for different providers.
This would make it easy to import the mocks directly from the resources, which will reduce redundant code in testing scripts.
Their default behavior will be to not make calls to the internet or write to disk and internally use mock calls for that.
Some questions we might want to consider:

If a new MockImageStore class is created, will that also imply altering all the existing test scripts to use that?
Will this change also mean creating a MockDelayedRequester in future, since that's the class majorly causing the calls to internet issue in tests?

@akshgpt7 Currently there are three disabled tests in my pr #464 in various Provider API Scripts (you can search for @pytest.mark.skip in that PR's code to find them).  These are the inspiration for this issue, and the only tests which would absolutely need to be altered.
For other mocks, I'm totally happy if there's a directory somewhere with many of the repetitively created mocks (or pytest fixtures).
I do think it'd make sense to create a MockDelayedRequester in the future, but that one is less of a problem, since the only validation it does is on responses.
The trick of the ImageStore is that a large part of its functionality revolves around validating input from the Provider API Scripts, so it's actually quite useful for these scripts to test against something very similar to the real thing.
I haven't decided where the MockImageStore should live; I'm open to good ideas there

@mathemancer Noting @akshgpt7 's comment here I have the following two approaches in mind:-
Approach 1:

Create a mock_resources folder. This houses the MockImageStore file. Since we know which tests are calling the internet via ImageStore we can change the implementation in those code pieces to use MockImageStore.
The mock_resources folder will also contain all the common mocks and patches (as stated by @akshgpt7 ) which can be configured for each provider.
For performing the tests which need to call the internet we can store the appropriate responses (of each provider) in a sub directory under mock_resources and verify against those. The mock_resources folder (and hence MockImageStore) can live under src/cc_catalog_airflow/dags/provider_api_scripts/tests/.

Approach 2:

Same as Approach 1.
Go through each provider's test file and see which tests are calling the internet and writing to the disk. For each of these tests we make changes to use the MockImageStore class.
For data verification we still have to create valid data examples for each provider.

Just one doubt. For the tests which write to the disk, what would be the approach for data verification?

I think MockImageStore should just be used to override the ImageStore in every script that uses ImageStore (except for in test_image.py.  I.e., in src/cc_catalog_airflow/dags/provider_api_scripts/test_flickr.py, just after the import, this could look like:
import flickr
flickr.image_store = MockImageStore(args, go, here)

It shouldn't be necessary to configure it per provider, since if someone wants to mock out specific responses they can do that themselves.  I'm mostly interested in making sure that folks aren't inadvertently calling the internet.
I think the Mock class should actually just live in the directory with the class itself (maybe even in the same file).

@mathemancer So the implementation would be like this then:

A MockImageStore in the image.py file.
The sample responses (required for data verification) will live in some other directory.
The MockImageStore will be called whenever a test tries to call the internet or write to the disk.

Have I got it right? Please confirm. If possible I can take this up.

For the moment, just try to get it working for internet calls.  I don't think any sample responses should need to be added or moved, since they're already associated with the necessary tests.
Please let me know if you have other questions.  Thank you!

Some other work came up after I took this up. I will resume work on this from today.
Thank You!

@mathemancer After analysing image.py and related scripts I found that the only part where the internet is called by ImageStore is by the licenses.get_license_info function (which by a chain of function callings, calls urls.rewrite_redirected_url which in turn calls the internet).
Are there any other parts calling the internet? Please confirm.
For the above part, the solution I came up with is to pass a named tuple license_info to the class as an attribute. This tuple can be passed in the test file. For e.g.:
import flickr
license_info = LicenseInfo = namedtuple(
    'LicenseInfo',
    ['license', 'version', 'url']
)
flickr.image_store = MockImageStore(license_info)

while MockImageStore will look like:
class MockImageStore:
   '''docstring'''

    def __init__(
            self,
            provider=None,
            output_file=None,
            output_dir=None,
            buffer_length=100,
            license_info=None
    ):
    ...
    def _get_image(
            self,
            .....
    ):
    valid_license_info = self.license_info

Thank You!


fixed Issue table Creator - Web design #21 #372

Added the following code to fix the issue for the table Creator - Web design #21 #372. modules/tables/Table_21_Sandeep_varma_Sangaraju.html

Heartfelt Thanks for Your Contribution to Our Open Source Project!
            Dear @sangaraju-01,

            I trust this message finds you in good health and high spirits. I am reaching out with a heart full of gratitude for your recent contribution to our open source project. Your dedication has not only elevated the functionality of our project but has also exemplified the collaborative spirit that lies at the core of thriving open source communities.

            A heartfelt congratulations are in order for your exceptional work! Your commitment and expertise have played a pivotal role in the success of our project, and we consider ourselves truly fortunate to count you among our community members.

            I must extend my sincere apologies for the delay in responding to your pull request. Your patience and understanding during this time are immensely appreciated. As our project continues to grow, we acknowledge the imperative need to enhance the quality and response time for contributions. To address this, we've kickstarted a call for maintainers and reviewers who can help us uphold the high standards of our project and ensure swift responses to pull requests.

            Your involvement and insights would be invaluable in this initiative. If you, or someone you know, is interested in taking on a more active role in the project, please don't hesitate to reach out. We firmly believe that with a robust team of maintainers and reviewers, we can elevate both the quality of the project and the speed of our responses.

            On an exciting note, I am thrilled to share that we have activated the sponsorship button for our project. This development allows supporters like yourself to contribute to the sustainability and growth of the project. Your ongoing support, whether through code contributions or sponsorship, is genuinely appreciated.

            Once again, congratulations on your outstanding contribution, and thank you for being an indispensable part of our open source community. We eagerly anticipate your continued collaboration and shared success.

            Best regards,

            Ntem Kenyor K.
            Lead
            Kwickerhub


RtmpCamera2 cannot zoom to maxZoom

RtmpCamera2 when attempting to zoom to the bounds of maxZoom, the preview will freeze and sometimes crash. It would appear the value returned by getMaxZoom() is incorrect.
RtmpCamera1 does not have this issue and works as expected. (I know the API is a bit different)
This can most easily be tested by changing app/src/main/java/com/pedro/rtpstreamer/customexample/RtmpActivity.java to use RtmpCamera2
(tested on multiple android devices)

Fixed: https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/commit/ac8ddd61502c19a5571246b55e91c3c118295faa

Thanks, works perfectly now. Thank you for the blazing fast response


Support view attribute for Structurizr docs rendered with Kroki

Using asciidoctor-vscode to render Asciidoc with Structurizr diagrams containing multiple views, only the last view defined in the Structurizr diagram is rendered in the preview.
With a large Structurizr workspace (which might commonly be managed in an external file to the Asciidoc) there will be many views, so these aren't rendered correctly in the preview.
Asciidoctor-diagram defines the view attribute 1 for specifying the view to render.
Kroki defines the view-key option 2 to specify the view to render for Structurizr inputs.
It would be great if the renderer could map the standard view attribute on the diagram block to the Kroki view-key option so these diagrams are rendered correctly in previews.

I made it with this in my Antora project:
structurizr::example$system.dsl[renderer="mermaid",view-key="Containers"]

view-key is of course not the property specified for AsciiDoctor Diagram, but it works for me. Kroki now get's the query param it expects.

I guess Kroki (server) could support both. Not sure if it's better to convert view to view-key in the extension or support both options in Kroki... 🤔
I guess view-key should have a higher precedence (since it's the "long-form").


tasks/collection_objects/filter/set_area raises with bad(?) params - how?

This query raises. I can't replicate how it happened:
tasks/collection_objects/filter/set_area?utf8=%E2%9C%93&area_picker_autocomplete=&drawn_area_shape=%7B%22type%22%3A%22Feature%22%2C%22geometry%22%3A%7B%22type%22%3A%22Polygon%22%2C%22coordinates%22%3A%5B%5B%5B8.4375%2C16.04581345375217%5D%2C%5B8.4375%2C16.04581345375217%5D%5D%5D%7D%2C%22properties%22%3A%7B%7D%7D&selection_object=CollectionObject

"{"type":"Feature","geometry":{"type":"Polygon","coordinates":[[[8.4375,16.04581345375217],[8.4375,16.04581345375217]]]},"properties":{}}"
On the face of it, your 'Polygon' consists of only two points, and they are identical.
RGeo::GeoJSON.decode is apparently unable to decode the object into a valid GeoJSON Feature, and returns nil. This causes the raise for 'undefined method `geometry' for nil:NilClass'.
This CAN be duplicated (though not easily) by selecting the map, and clicking once (anywhere) and, without moving the mouse, double-click. This ends the polygon, and makes it ready for 'Set area'. In normal operation this will not result in a fault page, but the page will seem to stall with the spinner.
I'm looking at the code to see if there is a clean way to accommodate this action, without the raise.

Thanks for tracking it down, I figured it was something obscure like that.
On Tue, Apr 11, 2017 at 8:27 AM, James Tucker<EMAIL_ADDRESS>wrote:

"{"type":"Feature","geometry":{"type":"Polygon","
coordinates":[[[8.4375,16.04581345375217],[8.4375,16.04581345375217]]]},"
properties":{}}"
On the face of it, your 'Polygon' consists of only two points, and they
are identical.
RGeo::GeoJSON.decode is apparently unable to decode the object into a
valid GeoJSON Feature, and returns nil. This causes the raise for
'undefined method `geometry' for nil:NilClass'.
This CAN be duplicated (though not easily) by selecting the map, and
clicking once (anywhere) and, without moving the mouse, double-click.
This ends the polygon, and makes it ready for 'Set area'. In normal
operation this will not result in a fault page, but the page will seem
to stall with the spinner.
I'm looking at the code to see if there is a clean way to accommodate this
action, without the raise.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SpeciesFileGroup/taxonworks/issues/115#issuecomment-293262259,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABYSAVLY1LycHpvom3boWN4hqriRGoFks5ru3-sgaJpZM4M5i0W
.

@TuckerJD can we close this?

I think so, yes: Now, if the shape does not decode to a valid RGeo object, it will simply appear as though no objects were found.


Create module for building model features

This module should include the methods to transform the raw dataframe (with CAMS forecast and observed error) and produce both X and Y dataframe passed for the model.

It should consider different number of previous observations, between 0 and few days.

Cases (from higher to lower importance):

[ ] All stations considered at the same time. Train/test split should be made at station level.
[ ] All stations are considered individually.
[ ] Stations are clustered by aux file (csv ???). This clustering procedure needs also to be implemented.

Maybe predictions can be save at data/predictions/{ model } for easier posterior analysis.


Enable aqavit-test in alpine-linux container(#4625)

As we begin to add more and more of these examples of using containers instead of agents, we need to parameterize some things. For certain, we need to parameterize the image being used.
Instead of doing each platform individually, riscv and alpine-linux and whatever is next, we need to design the change to allow for us to do this with any platform where containers are supported, and be able to pass in an image or file that suits the purpose.

What kind of parameterization do you expect? Does it refer to the container images selected and used according to various configurations on nodes of a specific architecture? But like riscv involves qemu and two images, I feel that its execution method will not be too different from the current one.

Parameterizing image and args, instead of hardcoding to alibabadragonwelljdk images alibabadragonwelljdk/alpine_test_image etc, as there may be others using these scripts that will want to use their own images.
I am going to have a discussion with @sophia-guo and @llxia - it relates to broadening support for this for all platforms that can support containers and also relates to my desire to investigate the use of Testcontainers.

Parameterizing image and args, instead of hardcoding to alibabadragonwelljdk images alibabadragonwelljdk/alpine_test_image etc, as there may be others using these scripts that will want to use their own images.
I am going to have a discussion with @sophia-guo and @llxia - it relates to broadening support for this for all platforms that can support containers and also relates to my desire to investigate the use of Testcontainers.

All right. Looking forward to the outcome of your discussion


Parse arbitrary JSON or YAML documents as values

This would be a follow-up feature to #9. It would be useful to be able to read a JSON or YAML document and pick some paths from it.
I was trying to do this:
package main

import (
    "encoding/json"
    "tool/file"
)

imagesFile: file.Read & {
    filename: "images.json"
    contents: string
}

images: json.Unmarshal(imagesFile.contents)

values: {
    imageDigest: "\(images[0].digest)"
}

But it turns out tool/file only works in conjunction with  tool/cli, i.e. via cue cmd, as it compromises hermeticity.
So if there could be a way to achieve this in Timoni, it'd be great. It may require creating a custom Go extension to CUE, which would be possible. I do wonder how the hermeticity problem can be addressed.
Is the way JSON and YAML values work as of #51?
I suppose the way to spin this is that results will be reproducible as long as inputs don't change and that CUE prevents users from compromising hermiticity by not allowing reading files and generating either time-variant or random values from their code.
/cc @squaremo

We could extend the Runtime API with support for querying JSON and YAML files.
Something like:
runtime: {
	apiVersion: "v1alpha1"
	name:       "production"
	values: [
		{
			query: "file:https://example.com/values.json"
			for: {
				"id":   "obj.user.id"
				"user": "obj.user.info.name"
			}
		},
		{
			query: "file:https://example.com/values.yaml"
			for: {
				"id":   "obj.user.id"
				"user": "obj.user.info.name"
			}
		},
	]
}

That's a little awkward for a couple reasons: 1. embedding big xml or text documents inside cue is kind of ugly, and 2. my editors don't know to apply appropriate syntax highlighting and editing tools to embedded data.

@ansel1 I think what you listed are very cosmetic reason, the biggest reason why embedding sucks is because it's hard to update, if it's a separate file you can copy it verbatim and validate easily, which is much harder once it's embedded in another file. In some cases quoting or some other kind of reformatting may need be applied as well.

Found a discussion along similar lines in cue's forums.  https://github.com/cue-lang/cue/discussions/1582. Basically suggesting something like go's embed feature.

Basically suggesting something like go's embed feature.

That does broadly make sense, however in Go there is a compilation step, but CUE doesn't have it, albeit Timoni could potentially do it, but that would be rather akin to runtime API, which is what Stefan is suggesting and it makes a lot of sense to me.

Wouldn't cue export be the equivalent?


Warning: names(v) is deprecated, use fieldnames(v) instead.

The code to reproduce:
using Interact

@manipulate for c=checkbox(true)
    nothing
end

I would fix this issue myself if I had the time to fork and send a pull request. The modification is so small that I decided to simply open an issue.

No longer an issue on v0.1.6


feat(STONEINTG-957): validate ITS if git resolver

According to the Tekton Git Resolver  we should not enable ITS creation if both parameters exists in git resolver , URL & Repo+Org
STONEINTG-957
Maintainers will complete the following section

[ ] Commit messages are descriptive enough (hints)
[ ] Code coverage from testing does not decrease and new code is covered (check the PR coverage on codecov)
[ ] Controllers diagrams are updated when PR changes controllers code  (if applicable)

Codecov Report
Attention: Patch coverage is 28.57143% with 10 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@7173ad8). Learn more about missing BASE report.

Files with missing lines
Patch %
Lines

api/v1beta2/integrationtestscenario_webhook.go
28.57%
6 Missing and 4 partials :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #877   +/-   ##
=======================================
  Coverage        ?   69.18%
=======================================
  Files           ?       43
  Lines           ?     4449
  Branches        ?        0
=======================================
  Hits            ?     3078
  Misses          ?     1049
  Partials        ?      322

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

/test images

/test images


[triton] New version with no boost dependencies

Describe the pull request
Update triton port so it does not require Boost

What does your PR fix?

Compilation time has been reduced by orders of magnitude

Does your PR follow the maintainer guide?

Yes

All feastures are tested successfully in the following triplet:

x86-windows
x64-windows
x64-windows-static

Can you test Linux and osx please?
On Tue, Apr 12, 2022 at 7:48 PM JonLiu1993 @.***> wrote:

All feastures are tested successfully in the following triplet:

x86-windows
x64-windows
x64-windows-static

—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vcpkg/pull/24081#issuecomment-1096455661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHT3KGNWPB2OEWUM2DZ22DVEVBG7ANCNFSM5TDGXYDA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Alberto García Illera
GPG Public Key https://goo.gl/yshdwh

@illera88, I can test linux and osx on my machine, can you explain what I reviewed?

I updated Triton to latest commit the author has done.
In the latest commits boost has passed to be optional reducing greatly the compilation time. Some changes on how Capstone is being included helped to get rid of one of the patches (capstone).
I haven't tested the python feature btw: triton[python].
Regards

I updated Triton to latest commit the author has done.
In the latest commits boost has passed to be optional reducing greatly the compilation time. Some changes on how Capstone is being included helped to get rid of one of the patches (capstone). I haven't tested the python feature btw: triton[python].
Regards

I have tested features successfully in x64-linux but test x64-osx failed about vcpkg_tool

Unfortunately I don’t have an OS X to test
On Wed, Apr 13, 2022 at 12:27 PM JonLiu1993 @.***>
wrote:

I updated Triton to latest commit the author has done.
In the latest commits boost has passed to be optional reducing greatly the
compilation time. Some changes on how Capstone is being included helped to
get rid of one of the patches (capstone). I haven't tested the python
feature btw: triton[python].
Regards
I have tested features successfully in x64-linux but test x64-osx failed
about vcpkg_tool
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vcpkg/pull/24081#issuecomment-1097486254,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHT3KFLPQPZKOEROQCL3R3VEYWH5ANCNFSM5TDGXYDA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Alberto García Illera
GPG Public Key https://goo.gl/yshdwh

@illera88, updating one port to test these triples is enough

that's not clear to me. Is triton working on OSX?

This is x64-osx test error, It doesn't seem to have anything to do with this pr
[1/1] cmd /c "cd .. && "F:/Feature-test/triton/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe" "F:/Feature-test/triton/vcpkg/scripts/detect_compiler" "-DCMAKE_MAKE_PROGRAM=F:/Feature-test/triton/vcpkg/downloads/tools/ninja/1.10.2-windows/ninja.exe" "-DCMAKE_SYSTEM_NAME=Darwin" "-DBUILD_SHARED_LIBS=OFF" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=F:/Feature-test/triton/vcpkg/scripts/toolchains/osx.cmake" "-DVCPKG_TARGET_TRIPLET=x64-osx" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=external" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=F:/Feature-test/triton/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_LINKER_FLAGS_RELEASE=" "-DVCPKG_LINKER_FLAGS_DEBUG=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=F:/Feature-test/triton/vcpkg" "-DZ_VCPKG_ROOT_DIR=F:/Feature-test/triton/vcpkg" "-D_VCPKG_INSTALLED_DIR=F:/Feature-test/triton/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF" "-DCMAKE_OSX_ARCHITECTURES=x86_64" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=F:/Feature-test/triton/vcpkg/packages/detect_compiler_x64-osx""
FAILED: ../CMakeCache.txt
cmd /c "cd .. && "F:/Feature-test/triton/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe" "F:/Feature-test/triton/vcpkg/scripts/detect_compiler" "-DCMAKE_MAKE_PROGRAM=F:/Feature-test/triton/vcpkg/downloads/tools/ninja/1.10.2-windows/ninja.exe" "-DCMAKE_SYSTEM_NAME=Darwin" "-DBUILD_SHARED_LIBS=OFF" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=F:/Feature-test/triton/vcpkg/scripts/toolchains/osx.cmake" "-DVCPKG_TARGET_TRIPLET=x64-osx" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=external" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=F:/Feature-test/triton/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_LINKER_FLAGS_RELEASE=" "-DVCPKG_LINKER_FLAGS_DEBUG=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=F:/Feature-test/triton/vcpkg" "-DZ_VCPKG_ROOT_DIR=F:/Feature-test/triton/vcpkg" "-D_VCPKG_INSTALLED_DIR=F:/Feature-test/triton/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF" "-DCMAKE_OSX_ARCHITECTURES=x86_64" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=F:/Feature-test/triton/vcpkg/packages/detect_compiler_x64-osx""
-- The C compiler identification is unknown
CMake Error at F:/Feature-test/triton/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/share/cmake-3.22/Modules/CMakeFindBinUtils.cmake:224 (message):
  Could not find install_name_tool, please check your installation.
Call Stack (most recent call first):
  F:/Feature-test/triton/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/share/cmake-3.22/Modules/CMakeDetermineCCompiler.cmake:196 (include)
  CMakeLists.txt:11 (enable_language)

-- Configuring incomplete, errors occurred!
See also "F:/Feature-test/triton/vcpkg/buildtrees/detect_compiler/x64-osx-rel/CMakeFiles/CMakeOutput.log".
See also "F:/Feature-test/triton/vcpkg/buildtrees/detect_compiler/x64-osx-rel/CMakeFiles/CMakeError.log".
ninja: build stopped: subcommand failed.

Don't have an OSX. Do you mind debugging it @JonLiu1993? If it's a test error, is the test functioning fine for other ports?
I would like to clear this up to move forward with the merge.
Cheers

@illera88, triton tested successfully in the x86-windos、x64-windows、x64-windows-static and x64-linux triplet, x64-osx test error, But it looks like this error is not caused by this pr, so I think we can merge this pr


Hopefully better version check.

Works on Windows to where by default php cannot use git. Uses .git filestructure then. If that fails to, return some static text

Maybe we could change this feature by allowing to refresh the screen by a command line command. Usually after a git pull, you still need to change the config file. Maybe we could do something like
touch refresh

And let the mirror check the edit date of refresh
Any thoughts?

I don't need to change the config. My Config is in a seperate branch which is the master branch of the PI but not published to github. So my dev machine has both branches and I just push in one way or the other. Because of that I prefer to not log in to the pi.

This will be fixed in V2.


Repository Interface and destination in different packages from Model

Nice generator package!
Is your feature request related to a problem? Please describe.
With dependency injection pattern, it's common that controller package defines the repository interface and accept it as input. However current implementation requires that repository interface and model type located within the same package.
Describe the solution you'd like
It would be nice if it could support the setup like this.
examples/ref_exp
├── ctrl
│   └── ctrl.go
├── repo
│   └── user_repo.go // generated
└── user.go

// ctrl.go
package ctrl

//go:generate repogen -pkg=.. -dest=../repo/user_repo.go -model=UserModel -repo=UserRepository

// UserRepository is an interface that describes the specification of querying
// user data in the database.
type UserRepository interface {
	// InsertOne stores userModel into the database and returns inserted ID
	// if insertion succeeds and returns error if insertion fails.
	InsertOne(ctx context.Context, userModel *ref_exp.UserModel) (interface{}, error)
	// FindByUsername queries user by username. If a user with specified
	// username exists, the user will be returned. Otherwise, error will be
	// returned.
	FindByUsername(ctx context.Context, username string) (*ref_exp.UserModel, error)

        ...
}

func NewController(userRepo UserRepository ... )...

Additional context
Just play around the code base. For this feature these might be related:

Package ID of where Model located. (to be used in RepositoryGenerator.Imports() )
in interfaceMethodParser there are some operation mode checking. And about the type comparison. Not sure how to allow imported types like *ref_exp.UserModel in args and return. (Currently it will check with p.StructModel).

Great improvement proposal! I was trying to do that but for my use case, I don't need that at the moment. However, if you can work on this, here are the high-level specification that I want:

When the model struct and the repository interface are in the different package, -pkg flag should reference the package of the repository interface, not the model struct. The generator should be able to derive the location of the model struct automatically.
-model flag should be able to read the imported type, e.g. ref_exp.UserModel.

And for the detailed specification, I want to separate the responsibilities as follows:

internal/code package should be able to provide the imported packages (lazily is ok)
internal/spec package should be able to validate the referencing fields
internal/mongo package should be able to derive bson tags and construct the query

Thanks for the specification!
What I am thinking is:

package.Package add Imports and collect all imports and Path to locate itself
in generateFromRequest, check structModelName  if it's from the same package.

if not, then parse package for structModel's package to get structModel  (roughly)

mongo generator will need additional import provide by package.Path

However in step 2, I am not sure: given -mode=ref_exp.UserModel and package.Imports, how to locate the model go file ? go/parser need a file path, but what I got is package path

For step 1, how does it need to be changed? Are there any concerns?
And for step 2, I've just noticed that concern. In my opinion, -model-dir flag should be a possible option. I have ever thought about -pkgs flag that read multiple packages and construct references by itself. It should be able to support the case that the model references a struct in a third package, but it may be too complicated.

thanks for the prompt reply
Then I will add -model-dir as flag.
for step 1,

Path is required for model pkg import in generated code.
no need Imports anymore since -model-dir should be enough

I also found destPkgName := flag.String("dest-pkg", "", "destination package name") might also needed :P

Resolved in https://github.com/sunboyy/repogen/pull/49


Update /security/forgotten-link/{code}

In gitlab by @A-Lawrence on Aug 20, 2016, 20:01
Update this route to utilise the {token} route model binding search.
Linked to #495.

In gitlab by @A-Lawrence on Aug 24, 2016, 12:31
mentioned in issue #495

In gitlab by @A-Lawrence on Sep 4, 2016, 18:18
Assignee removed

In gitlab by @A-Lawrence on Sep 4, 2016, 18:18
Milestone removed


Error when accessing a file from cached namespace bucket

Environment info

NooBaa Version: 5.6.0
Platform: Kubernetes 1.18

Actual behavior

Create cached namespace bucket
Able to list the files of the remote bucket
Not able to copy a file

Expected behavior

Create cached namespace bucket
Able to list the files of the remote bucket
Able to copy a file

More information - Screenshots / Logs / Other output
The main error reported on the endpoint pod is:
Apr-7 8:26:42.267 [Endpoint/14] [ERROR] core.rpc.rpc:: RPC._request: response ERROR srv object_api.update_endpoint_stats param
s { namespace_stats: [ { io_stats: { read_count: 607, write_count: 0, read_bytes: 554063233, write_bytes: 0, error_write_bytes: 0, error_write_count: 0, error_read_bytes: 0, error_read_count: 0 }, namespace_resource_id: '60658c3da2a804002ee6026b' }, [length]: 1 ], bucket_counters: [ { bucket_name: SENSITIVE-64a717a11f8d324e, content_type: 'binary/octet-stream', read_count: 700, write_count: 0 }, { bucket_name: SENSITIVE-64a717a11f8d324e, content_type: 'application/octet-stream', read_count: 4, write_count: 0 }, [length]: 2 ] } reqid 34499@fcall://fcall(7psnzeet) took [1.1+1.1=2.2] Error: not anonymous method update_endpoint_stats

Attaching more verbose logging as well
noobaa.log

Hey @YiannisGkoufas I think that these errors on update_endpoint_stats should not interrupt the S3 service.
Do you see that the read fails? Because I see in your log that it might have completed?

Read request log
Apr-7 8:25:42.451 [Endpoint/14]    [L0] core.endpoint.s3.s3_rest:: S3 REQUEST
    GET /bucket1-cached/TPCDS-TEST-100G/call_center/part-00000-1f4a5e91-eac5-4190-8499-65d95212f848-c000.snappy.parquet
    op get_object request_id kn76sf1t-8tcokv-gsr
    {
        host: 's3.noobaa.svc',
        'accept-encoding': 'identity',
        'user-agent': 'aws-cli/2.1.35 Python/3.8.8 Linux/4.15.0-140-generic exe/x86_64.debian.10 prompt/off command/s3.cp',
        'x-amz-date': '20210407T082542Z',
        'x-amz-content-sha256': 'e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855',
        authorization: 'AWS4-HMAC-SHA256 Credential=fOXTchSSFBmsUIHyLnY9/20210407/us-east-1/s3/aws4_request, SignedHeaders=host;x-amz-content-sha256;x-amz-date, Signature=036b7375ceaef24cc0df8a646a456c4bb398398a16330e0623081582d260b3a6'
    }

Apr-7 8:25:42.452 [Endpoint/14]    [L0] core.server.object_services.object_server:: object_server.read_object_md:
    {
        bucket: SENSITIVE-64a717a11f8d324e,
        key: 'TPCDS-TEST-100G/call_center/part-00000-1f4a5e91-eac5-4190-8499-65d95212f848-c000.snappy.parquet'
    }

Apr-7 8:25:42.455 [Endpoint/14]    [L0] core.sdk.namespace_cache:: NamespaceCache.read_object_md use md from cache
    {
        obj_id: '606c85f72c09c0000e78e8fd',
        bucket: SENSITIVE-64a717a11f8d324e,
        key: 'TPCDS-TEST-100G/call_center/part-00000-1f4a5e91-eac5-4190-8499-65d95212f848-c000.snappy.parquet',
        size: 13964,
        etag: 'a2a7f60ad6165b1f9d621094c8ddd069',
        md5_b64: 'oqf2CtYWWx+dYhCUyN3QaQ==',
        content_type: 'binary/octet-stream',
        create_time:<PHONE_NUMBER>898,
        last_modified_time:<PHONE_NUMBER>000,
        cache_last_valid_time:<PHONE_NUMBER>436,
        num_parts: 1,
        is_latest: true,
        xattr: { 'noobaa-namespace-s3-bucket': 'yiannis-tpcds-100' },
        stats: { reads: 0, last_read: 0 },
        tag_count: 0,
        should_read_from_cache: true
    }

Apr-7 8:25:42.456 [Endpoint/14]    [L0] core.sdk.namespace_cache:: NamespaceCache._read_object_stream
    {
        params: {
            object_md: {
                obj_id: '606c85f72c09c0000e78e8fd',
                bucket: SENSITIVE-64a717a11f8d324e,
                key: 'TPCDS-TEST-100G/call_center/part-00000-1f4a5e91-eac5-4190-8499-65d95212f848-c000.snappy.parquet',
                size: 13964,
                etag: 'a2a7f60ad6165b1f9d621094c8ddd069',
                md5_b64: 'oqf2CtYWWx+dYhCUyN3QaQ==',
                content_type: 'binary/octet-stream',
                create_time:<PHONE_NUMBER>898,
                last_modified_time:<PHONE_NUMBER>000,
                cache_last_valid_time:<PHONE_NUMBER>436,
                num_parts: 1,
                is_latest: true,
                xattr: { 'noobaa-namespace-s3-bucket': 'yiannis-tpcds-100' },
                stats: { reads: 0, last_read: 0 },
                tag_count: 0,
                should_read_from_cache: true
            },
            obj_id: '606c85f72c09c0000e78e8fd',
            bucket: 'bucket1-cached',
            key: 'TPCDS-TEST-100G/call_center/part-00000-1f4a5e91-eac5-4190-8499-65d95212f848-c000.snappy.parquet',
            content_type: 'binary/octet-stream',
            noobaa_trigger_agent: false,
            md_conditions: undefined,
            encryption: undefined,
            read_size: 13964
        }
    }

Apr-7 8:25:42.456 [Endpoint/14]    [L0] core.sdk.namespace_cache:: NamespaceCache: bucket usage
    {
        bucket_free_space_bytes:<PHONE_NUMBER>,
        bucket_usage: {
            size: 15201595,
            size_reduced: 516854505,
            free:<PHONE_NUMBER>,
            available_for_upload:<PHONE_NUMBER>,
            last_update:<PHONE_NUMBER>573
        }
    }

Apr-7 8:25:42.457 [Endpoint/14]    [L0] core.util.http_utils:: HTTP REPLY RAW
    GET /bucket1-cached/TPCDS-TEST-100G/call_center/part-00000-1f4a5e91-eac5-4190-8499-65d95212f848-c000.snappy.parquet

 https://github.com/noobaa/noobaa-core/issues/6441

Hi @guymguym yeah the file shows up but with 0 size

I noticed this error on the endpoint pod:
Apr-21 11:15:24.492 [Endpoint/11]    [L0] core.endpoint.s3.ops.s3_get_object:: request aborted: undefined
(node:11) UnhandledPromiseRejectionWarning: Error: Semaphore Timeout
    at Semaphore._on_timeout (/root/node_modules/noobaa-core/src/util/semaphore.js:215:25)
    at Timeout.<anonymous> (/root/node_modules/noobaa-core/src/util/semaphore.js:211:53)
    at listOnTimeout (internal/timers.js:554:17)
    at processTimers (internal/timers.js:497:7)
(node:11) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 102)
(node:11) UnhandledPromiseRejectionWarning: Error: Semaphore Timeout
    at Semaphore._on_timeout (/root/node_modules/noobaa-core/src/util/semaphore.js:215:25)
    at Timeout.<anonymous> (/root/node_modules/noobaa-core/src/util/semaphore.js:211:53)
    at listOnTimeout (internal/timers.js:554:17)
    at processTimers (internal/timers.js:497:7)
(node:11) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_

@jeniawhite @dannyzaken @nimrod-becker Did you see issues with caching on 5.6?
@YiannisGkoufas What is your hub resource? AWS S3? S3 compatible? Also note that we recently released 5.7 perhaps you would prefer to use it on next runs. Another option is to use weekly master builds - @liranmauda do we have a wiki on weekly master builds so that it will be clear how to find the CLI, and the images etc... I see that @jeniawhite uploaded the last CLI to https://noobaa-operator-cli.s3.amazonaws.com/ but I wonder if the weekly build process pushed there too.

I believe we were not able to bring all the fixes of caching into 5.6, @YiannisGkoufas would it be possible for you to use the 5.7 version ? we have the images in the dockerhub repos

@guymguym @nimrod-becker It's IBM COS, I tried version 5.7 and builds from master with the same behaviour

@liranmauda do we have a wiki on weekly master builds so that it will be clear how to find the CLI, and the images etc...

We Don't have a Wiki for that, AFAIK.

I see that @jeniawhite uploaded the last CLI to https://noobaa-operator-cli.s3.amazonaws.com/ but I wonder if the weekly build process pushed there too.

The weekly build process should push noobaa-operator-cli unless something is not working.

Created wiki for that - https://github.com/noobaa/noobaa-core/wiki/Weekly-Master-Builds

@YiannisGkoufas Can you share the CR's - BackingStore, NamespaceStore, BucketClass?

Sure @guymguym, using the latest images+cli
https://github.com/noobaa/noobaa-operator/commit/bf95bd6e00a8a65e191915bc4b852761b005ad71
https://github.com/noobaa/noobaa-core/commit/4ce639a13908fa898bb31be8d44aac3b97ebd512
I am not able to do even an ls
[yiannis@oc3656886087 ~]$ aws s3 --endpoint-url=https://<IP_ADDRESS>:443 --no-verify-ssl ls
/usr/local/aws-cli/v2/2.1.3/dist/urllib3/connectionpool.py:1020: InsecureRequestWarning: Unverified HTTPS request is being made to host '<IP_ADDRESS>'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings

expected string or bytes-like object

From the CRDs you suggested I only had a backinstore and a bucket class (added the connection and the namespace bucket via the UI)
noobaa_crds.zip


After install the blueretro on gamecube

so i just installed the blue retro bluetooth and when i push the power button it lights up blue and doesn't turn on but the fan is spinning non stop. i also have a pico installed and a sd card in the bottom port of the gamecube. how do i fix this?

Try and press the reset button. This issue is documented on the GameCube board page in laser bear industries.

@yozan2450 You should contact Laser Bear support https://www.laserbear.net/pages/contact-us
This look like an installation issue.

Yea the ribbon cable had a defect in it
On Wed, Jan 18, 2023, 2:37 PM Jacques Gagnon @.***>
wrote:

@yozan2450 https://github.com/yozan2450 You should contact Laser Bear
support https://www.laserbear.net/pages/contact-us
This look like an installation issue.
—
Reply to this email directly, view it on GitHub
https://github.com/darthcloud/BlueRetro/issues/529#issuecomment-1387661609,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APYTYLJEAHWBVT3FO5GUP2LWTBBA3ANCNFSM6AAAAAATSWZI7M
.
You are receiving this because you were mentioned.Message ID:
@.***>


feat: use new asynchronous resolvo code

get_dependencies and get_candidates now spawn tasks to process asynchronously.

Happy to see it worked out in the end!

Quick benchmark:
There are two versions:

rip without async resolvo code.
rip-async with async resolvo code.

Both were compiled with --release and copied over. Don't know about the statistical interference, I had no discord or IDE running.
Benchmark 1 without a cache
Command: hyperfine "./rip quetz-server" "./rip-async quetz-server" --prepare "rm -rf /Users/tdejager/Library/Caches/rattler/"
Benchmark 1: ./rip quetz-server
  Time (mean ± σ):      2.952 s ±  0.129 s    [User: 0.578 s, System: 0.363 s]
  Range (min … max):    2.802 s …  3.129 s    10 runs

Benchmark 2: ./rip-async quetz-server
  Time (mean ± σ):      1.975 s ±  1.242 s    [User: 0.481 s, System: 0.395 s]
  Range (min … max):    1.343 s …  5.483 s    10 runs

  Warning: Statistical outliers were detected. Consider re-running this benchmark on a quiet system without any interferences from other programs. It might help to use the '--warmup' or '--prepare' options.

Summary
  ./rip-async quetz-server ran
    1.49 ± 0.94 times faster than ./rip quetz-server

Benchmark 2 with a cache
Command: hyperfine "./rip quetz-server" "./rip-async quetz-server" -w 2
Benchmark 1: ./rip quetz-server
  Time (mean ± σ):      1.567 s ±  0.074 s    [User: 0.339 s, System: 0.457 s]
  Range (min … max):    1.469 s …  1.707 s    10 runs

Benchmark 2: ./rip-async quetz-server
  Time (mean ± σ):      1.094 s ±  0.428 s    [User: 0.371 s, System: 2.439 s]
  Range (min … max):    0.803 s …  2.021 s    10 runs

  Warning: Statistical outliers were detected. Consider re-running this benchmark on a quiet system without any interferences from other programs. It might help to use the '--warmup' or '--prepare' options.

Summary
  ./rip-async quetz-server ran
    1.43 ± 0.56 times faster than ./rip quetz-server

Seems like a nice improvement

Before merge

Get rid of the tracing statements.

This is ready for a final review! @tdejager @nichmor @wolfv


PHPUnit assertions not shown in autocomplete suggestions

When writing regular PHPUnit tests, all the available PHPUnit assertions are suggested:

However, when using the $this->assert... syntax in PEST tests, none of them show up:

This should have been fixed now. I'll release a new alpha version tomorrow :)


demos: Redirect error messages to stderr

Rerouted error messages from stdout to stderr.
Regular, human readable output and json output both print to stdout (depending which is specified) but error messages sent to stdout when generating json result in invalid json output.

@mikew-lunarg Okay, in ERR_EXIT I fflush both stdout and stderr for good measure and removed fflushes that were happening just before an exit.


Goldsky mirror to Clickhouse

What is it?
Let's set it up using yaml files and the Goldsky CLI so we can version control it.

Let's start with ENS, Optimism traces, and Base

Trying to see if we can use someone else's Clickhouse instance for this

Deprioritizing this for now.