[eas-cli] analytics support

Why
support for analytics
https://linear.app/expo/issue/ENG-10/event-tracking-eas-build
How
implement analytics client
for now, it's only in eas build command
all events will go into a single project
Test Plan
Run build and checked Amplitude event explorer

@fson

One thing that's still missing is a way for the user to opt out of all analytics

These are the same analytics as in expo(this almost the same code), we gather only information we already have in db, the only information that we don't have server-side are timestamps of some steps.

@animeeples is going to follow up with @cruzach to get his perspective on this and also do some general research, then we can revisit

Updated according to https://linear.app/expo/issue/ENG-654/eas-cli-metrics-opt-out


field-counts: Can't be run if 001 and 008 aren't run first

field-counts calls flatten which is defined in the 001 SQL file. If a user runs add-view with --dontbuild, then they'll need to know to run the 001 SQL file before running field-counts.
Instead, field-counts should just define the function itself (nothing else uses flatten). While doing this, we can delete the flatten_with_values function, which nothing uses at all.
For these two functions, we also don't need drop function if exists because we use CREATE OR REPLACE.

I now notice that this command depends on release_summary_with_data, which is created in 008 (which itself depends on most other SQL files #83).
As such, for now, I'll document that field-counts must be run after refresh-views. For this issue, we should still:

[ ] Move the flatten function out of the 001 SQL file into the field-counts command
[ ] Delete the flatten_with_values function
[ ] Remove drop function if exists flatten;
[ ] Raise an error if field-counts' requirements aren't satisfied (i.e. release_summary_with_data doesn't exist)

flatten and flatten_with_values are used by analysts and are there as useful helper functions.  It is not suitable for colab notebooks as they have no write access to database.
Also https://github.com/open-contracting/kingfisher-views/issues/29 will need flatten.

Can you ask the analysts to share a notebook that uses the functions? Just to be clear on their usage.
In terms of where it should be defined, if it's used by analysts, by refresh-views, and also by field-counts, should it perhaps just be defined by add-view?


[Bug]: When using 'gemini-pro' and 'gemini-pro-vision', if you ask questions in Chinese, the Chinese responses you receive are likely to be garbled

What happened?
When using 'gemini-pro' and 'gemini-pro-vision', if you ask questions in Chinese, the Chinese responses you receive are likely to be garbled，Other models such as chatgpt and bing are normal.
Steps to Reproduce

ask questions in Chinese
Get a reply
Garbled code generation

What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
Screenshots

Code of Conduct

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

same
and these don’t seem to be sensitive words

same and these don’t seem to be sensitive words

yes,

be sensitive words

yes, I've turned off the censor function for sensitive words.


Support for environment variables in config

I've been attempting to run a setup locally involving mongodb, elasticsearch and transporter all on docker.
The dockerization works for all 3, but in order for them to communicate via docker links (which are preferred over exposed ports) the config must have access to environment variables, as docker link addresses/ports are defined in env variables.
This isn't just an issue with docker - anyone using a PaaS that exposes config info via env variables (e.g. heroku) may hit issues here.
Support for environment variables in the config.yaml or via command line args would be greatly appreciated!

:+1:  good idea, I'll get this onto the road map


ask about image failed

💚💜 Thank you for interest. ❤💛
Please make sure to check for more efficient package management. Please prioritize checking existing issues first. I will repay with higher-quality code.
To Reproduce

[x] I have read README.md. Please read me.
[x] I have searched for existing issues. (Most of the issues are duplicates.)
[x] I installed the appropriate version and confirmed that the previous version was properly removed upon reinstallation.
[x] I also checked the environment (not only on a single local machine but also on Google Colab, different Google accounts, and tried resetting cookies, etc. Please make sure to attempt these steps if necessary, and test the functionality in a new virtual environment if needed.)

Example
pip uninstall bardapi # and restart kernel
pip install bardapi==0.1.29 # check proper version
pip install -u bardapi==0.1.29 # upgrade version

import bardapi
bardapi.__version__

----------Please delete the content above this line, including this line.-------------
Version
OS: MacOS Monterey 12.6
Python: 3.10
Bard API: 0.1.34
Using proxy: no
Legion:
Error
ask pure text content fine, Web-ui fine, ask image failed and give following:
Response Error: b\')]}\\\'\\n\\n103\\n[["er",null,null,null,null,400,null,null,null,3],["di",43],["af.httprm",42,"474965566171913227",244]]\\n25\\n[["e",4,null,null,139]]\\n\'. \nUnable to get response.\nPlease double-check the cookie values and verify your network environment or google account.

I am having the same issue. Also I am making sure to instantiate multi-cookies by myself and still isn't working.
(Web-UI is fine, get_answer method is fine)

I am having the same issue.

No updates regarding the issue?

veonua #189
We currently cannot support the QA for the ask_about_image method in my current region and language. We've been unsuccessful even when trying to bypass this with accounts or IPs created in other regions.
There has been an update, and I'd appreciate it if you could check why it's still not progressing.
Also, when using the same version as the previous one, did you experience the same issue? (I'm curious if it used to work and then stopped working on the same version.)
It seems that only some data types need to be changed regarding this. We would be grateful if anyone could debug this issue and submit a Pull Request.

veonua #189
We currently cannot support the QA for the ask_about_image method in my current region and language. We've been unsuccessful even when trying to bypass this with accounts or IPs created in other regions.
There has been an update, and I'd appreciate it if you could check why it's still not progressing.
Also, when using the same version as the previous one, did you experience the same issue? (I'm curious if it used to work and then stopped working on the same version.)
It seems that only some data types need to be changed regarding this. We would be grateful if anyone could debug this issue and submit a Pull Request.

@dsdanielpark Is there any temporal solution to this issue so we could get the function working? Is kinda critical for our project to analyze some images.

Have you checked the installation of the GitHub version with the following command?
$ pip install git+https://github.com/dsdanielpark/Bard-API.git

There was a recent hot fix for 'ask_about_image', but we haven't officially included it in the version as QA hasn't finished yet.
What I continue to wonder is why there's a sudden response error when it was working fine in the original version.

Using 0.1.33 or 0.1.35 version for ask_about_image method
While wrapping some of the overall structures, version 0.1.34 could not properly support ask_about_image due to its omission. Please use the GitHub dev version or version 0.1.33 or 0.1.35. We apologize for the confusion. If you have any ideas for better refactoring of the structure, please feel free to contribute! Thank you.


jinja2 support would be nice

Add &[data-name$=".jinja2"]:before         { .jinja-icon;             .dark-red; } to styles -> file-icons.less:1011.

Done! I've used a different shade of red to help users differentiate between .jinja and .jinja2 files, if that helps. :)

@Alhadis awesome, thanks! I've been adding that line locally each time I update haha!

LOL dude, you should be using your user stylesheet.
@import "packages/file-icons/styles/items";
@{pane-tab-selector}, .icon-file-text {
	[data-name$=".jinja2"]:before { .jinja-icon; .dark-red; }
}

@Alhadis There are too many things to know in the world, i can't keep track of it all 👍


literal_parser.rb: fix scan_embeded_code

Writing conf like:
<source>
  type dummy
  tag "#{ENV['HOGE']}foo#{ENV['MOGE']}"
</source>

results in a error:
/home/sonots/src/github.com/fluent/fluentd/lib/fluent/config/literal_parser.rb:157:in `instance_eval': (eval):1: syntax error, unexpected tSTRING_DEND, expecting end-of-input (SyntaxError)
ENV['HOGE']}foo#{ENV['MOGE']
            ^
        from /home/sonots/src/github.com/fluent/fluentd/lib/fluent/config/literal_parser.rb:157:in `eval_embedded_code'
        from /home/sonots/src/github.com/fluent/fluentd/lib/fluent/config/literal_parser.rb:84:in `scan_double_quoted_string'
        from /home/sonots/src/github.com/fluent/fluentd/lib/fluent/config/literal_parser.rb:68:in `scan_string'
        from /home/sonots/src/github.com/fluent/fluentd/lib/fluent/config/literal_parser.rb:61:in `parse_literal'
        from /home/sonots/src/github.com/fluent/fluentd/lib/fluent/config/v1_parser.rb:128:in `parse_element'
        from /home/sonots/src/github.com/fluent/fluentd/lib/fluent/config/v1_parser.rb:93:in `parse_element'
        from /home/sonots/src/github.com/fluent/fluentd/lib/fluent/config/v1_parser.rb:41:in `parse!'
        from /home/sonots/src/github.com/fluent/fluentd/lib/fluent/config/v1_parser.rb:31:in `parse'
        from /home/sonots/src/github.com/fluent/fluentd/lib/fluent/config.rb:29:in `parse'
        from /home/sonots/src/github.com/fluent/fluentd/lib/fluent/supervisor.rb:356:in `read_config'
        from /home/sonots/src/github.com/fluent/fluentd/lib/fluent/supervisor.rb:126:in `start'
        from /home/sonots/src/github.com/fluent/fluentd/lib/fluent/command/fluentd.rb:167:in `<top (required)>'
        from /home/sonots/src/github.com/fluent/fluentd/bin/fluentd:6:in `require'
        from /home/sonots/src/github.com/fluent/fluentd/bin/fluentd:6:in `<top (required)>'
        from /home/sonots/.rbenv/versions/haikanko/lib/ruby/gems/2.1.0/bin/fluentd:23:in `load'
        from /home/sonots/.rbenv/versions/haikanko/lib/ruby/gems/2.1.0/bin/fluentd:23:in `<main>'

This patch fixes this issue.

cherry-picked to v0.10


add databricks volumes connectors

Description
Adds in source and destination connectors for the databricks managed UC volumes. At the moment, looking into ways for the source connector to recursively list files at a given path, but the destination connector is fully fledged out with a destination ingest test.

@rbiseck3 - Is there anything in this PR that you'd still like to merge in?


Declare stages for when the hook is relevant

I suspect not all git hook stages are relevant for this hook, so we should scope it to just those of relevance so its not run for no reason when it doesn't have to be. See https://pre-commit.com/#creating-new-hooks and the stages part.
PS: Adding a link to a related configuration reference for files like .pre-commit-hooks.yaml at the top of the file is a practice I really love, then anyone can quickly just visit the configuration reference and learn more about the kind of things that can make sense to do in a file. In this case, https://pre-commit.com/#creating-new-hooks is the relevant reference link for .pre-commit-hooks.yaml

I'm guessing it's just the commit stage?

@yuvipanda I think you are right - only commit (which is pre-commit).


fix C# resource dtor recursion issue

Give than the destructor function we export for a given exported resource will take care of removing the rep from the RepTable, we should not try to do that in the Dispose(bool) method; just call WasmInterop.wasmImportResourceDrop, which will call the host, which will call back into the guest, which will remove the entry.

Thanks to @kateinoigakukun for pointing this out!


Can not get the prune plugin to work.

Hi,
Im trying to delete field if its value = "N_A"; and I want this filter to run through all fields in the message, so i writing this in pipeline, but it's not working, the field with N_A values still exist in my kibana.
	prune {                 blacklist_values => {"." => "^N_A$"}        }
I dont know where I am doing wrong. Maybe this issue a little bit too basic but I am just a newbie to this.
Please help, thank you.

As the template text for new issues says:

Please post all product and debugging questions on our forum. Your questions will reach our wider community members there, and if we confirm that there is a bug, then we can open a new issue here.

So, please post your question there and make sure you include additional details (like what your input event looks like). I'm closing this issue.
That said, I don't understand what your intention is with the period in {"." => "^N_A$"}. The key in the hash is supposed to be a fieldname and you hardly have a field named ".".


Treeview project primary color only on focus

The project name in the treeview only has the primary color when the treeview is focused.
The tabs always (when setting is enabled) has the primary color and when treeview doesn't this looks a bit off.

Atom 1.5.0-beta0
Material UI https://github.com/silvestreh/atom-material-ui/commit/d7d8ea25d2425cb5980451adc102784261d46822
Windows 7

Never mind, missed the "Blend tree view with tab bar" option. Great solution.


any plan to support python3?

as titled

Python 3 is supported, we just haven't updated the metadata tags to reflect this on PyPi yet. Our Travis config runs tests against Python 3.


Does not detect repos

Let me know if there's any debug info I can send

Seems strange to me. This should ideally render instantly. This extension gets it's data from repo binary which is bundled in the extension. There is a bug in case of M1 where this returns nothing.
Can we check the output of

repo cmp -f
repo cmp -t
repo cmp -j

This binary should be here
I've around 40 repos in several levels and it takes under 100ms, so something is clearly off above.

Those commands work when run from iTerm.app, but hang when run from VSCode's terminal, and interestingly also hang when run from Terminal.app. I cannot paste the output as this is my work laptop and I may inadvertently publish confidential information (project names etc.), but iTerm.app does indeed print a list of git repos in my home directory.

macOS: 12.5
CPU arch: intel

I cannot paste the output as this is my work laptop and I may inadvertently publish confidential information (project names etc.)

Those commands were for you to check the output if we are getting repos, so sharing is not needed at all. Looks like we have the issue in the binary itself. something platform or shell specific.

Those commands work when run from iTerm.app, but hang when run from VSCode's terminal.
I guess something is wrong with the default terminal/shell.

Does iTerm and Terminal.app are having same shell ?
Is there any panic or it just hangs ?

Oh yes, forgot to mention, I use fish shell throughout.
It just hangs, nothing shown after waiting 30s or so (I just cancel it after that).
Happy to help 🙂 Although I'm a JS/TS bod only, so unfortunately I can't help with any Rust code.

Got some debug builds ready for you here. Let's start with repo cmp -f. In order to print info we require you'll need to run
INFO=1 repo cmp -f

Try this in iTerm, Terminal.app and VSCode Terminal. This should tell us where is it getting stuck.

Published v0.4.1 of extension. There was a bug in binary which caused early exit on traversal. This was affect macOS. While I'm still not sure about your issue, you may give it a try.

Published v0.4.1 of extension. It fixes a bug in binary which caused early exit on traversal. This was affecting macOS.
I'm still not sure about root cause of your issue because I tested it in M1 and Intel Macs and it worked fine expect traversal bug. You may give this release a try though.

Closing this as of now since I am unable to reproduce this and after several fixes, the extension seems to work fine for others. Feel free to reopen when we have more info on this.

Hi @mohitsinghs Sorry for not getting back to you sooner. Yes, that's working now! Thank you.

Glad that it worked. This entire time, I forgot that I have an old Macbook 😄, or else I would have fixed it sooner.


RemoteApp is jittery, misaligned and glitchy on Ubuntu 21.10 and Windows 11, freerdp-nightly

Ubuntu 21.10
This is FreeRDP version 3.0.0-dev (1adb26381)
Windows 11 Enterprise multi-session 21H2
[09:39:14:465] [38852:38853] [WARN][com.freerdp.client.x11] - xf_Pointer: Invalid appWindow
RemoteApp does not align the viewport, and cursor. Apps are not usable because they jitter around, and the local view does not quite correspond with the remote app. Most of the top bar looks cut off. Regular remote desktop session works perfectly.
To Reproduce
/opt/freerdp-nightly/bin/xfreerdp /v: /app:||7b9d2845-d91d-4b17-e6ed-08da11342132 /sound /microphone /multitransport /network:auto /rfx /gfx:AVC444 /cert-ignore /drive:hotplug,* +fonts /u: /p: /d:
Expected behavior
Remote application behaving like a local app.
Screenshots

Application details
Ubuntu 21.10 AMD64
This is FreeRDP version 3.0.0-dev (1adb26381)
Windows 11 Enterprise multi-session 21H2 64-bit

Output of xfreerdp /buildconfig

/opt/freerdp-nightly/bin/xfreerdp --version
This is FreeRDP version 3.0.0-dev (1adb26381)
root@ultrabook2:~# /opt/freerdp-nightly/bin/xfreerdp /buildconfig
This is FreeRDP version 3.0.0-dev (1adb26381)
Build configuration:
Build type:          RelWithDebInfo
CFLAGS:              -g -O2 -ffile-prefix-map=/build/freerdp-nightly-3.0.0+0~20220406014817.1107~1.gbp1adb26=. -flto=auto -ffat-lto-objects -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -fPIC -Wall -fvisibility=hidden -Wimplicit-function-declaration -Wredundant-decls -g -fno-omit-frame-pointer -fsanitize=address -fsanitize-address-use-after-scope
Compiler:            GNU, 11.2.0
Target architecture: x64

If available the log output from a run with /log-level:trace 2>&1 | tee log.txt

output of /log-level:trace 2>&1 | tee log.txt
log.txt

https://github.com/FreeRDP/FreeRDP/pull/8082
I have submitted the repair of this problem, which has not been merged for the time being

@vesystem2022 and I´ve stated the reasons why it was not merged ;)
Please address the issues for this to go forward or it will stall.

Revert 8e87e9d maybe resolve this issue.
Although the window will be iconise when it is restored from the minimized state, it needs to be clicked again to restore, but it will not display misaligned.
By trace the log when window restore from minimize, the issue about iconise from minimize is is caused by the TS server sending update orders like the following
Window updae  ... show maximize...
Window update ... clientAreaWidth=0 clientAreaHeight=0
Window update ... visibleWidth=160 visibleHeight=27

These order will maximize window, and then iconize window, so the remote app window will be show maximize and then iconise at last.
In the xf_event_MapNotify, regardless of sending SC_RESTORE or SC_MAXIMIZE, the remote app window will be show as iconise.
It maybe should be send a SC_MAXIMIZE after the window recv the last update order (eg. Window update ... visibleWidth=160 visibleHeight=27), But can not detect the "LAST" order or send time.

Try this this patch,  it will solve the misaligned and restore window from minimized.
diff --git a/client/X11/xf_event.c b/client/X11/xf_event.c
index 747e818b6..62ef0964a 100644
--- a/client/X11/xf_event.c
+++ b/client/X11/xf_event.c
@@ -761,7 +761,7 @@ static BOOL xf_event_MapNotify(xfContext* xfc, const XMapEvent* event, BOOL app)
        {
                appWindow = xf_AppWindowFromX11Window(xfc, event->window);

-               if (appWindow && (appWindow->rail_state == WINDOW_SHOW))
+               if (appWindow)
                {
                        /* local restore event */
                        /* This is now handled as part of the PropertyNotify
diff --git a/client/X11/xf_rail.c b/client/X11/xf_rail.c
index 770234f7d..a04ea4566 100644
--- a/client/X11/xf_rail.c
+++ b/client/X11/xf_rail.c
@@ -506,6 +506,13 @@ static BOOL xf_rail_window_common(rdpContext* context, const WINDOW_ORDER_INFO*
                                                    visibilityRectsOffsetY, appWindow->visibilityRects,
                                                    appWindow->numVisibilityRects);
                }
+               if (appWindow->rail_state == WINDOW_SHOW_MAXIMIZED && appWindow->clientAreaHeight == 0 && fieldFlags & WINDOW_ORDER_FIELD_WND_OFFSET && !(fieldFlags & WINDOW_ORDER_FIELD_WND_SIZE))
+               {
+                       xf_rail_send_client_system_command(xfc, appWindow->windowId, SC_RESTORE);
+                       xf_rail_send_client_system_command(xfc, appWindow->windowId, SC_MAXIMIZE);
+               }
        }

        /* We should only be using the visibility rects for shaping the window */

Strangely, it must be sent SC_RESTORE and SC_MAXIMIZE, otherwise, the window may still be an iconize.
This method of detecting the "LAST window update order" is too "HARD CODE". It is write by the LOG as below,
It seems that the window state is maximize and the width/height is smallest when restoring window from minimize state by clicking taskbar icon of app.
@akallabeth @jphein , Can you give some suggestions ?
[22:14:51:427] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:427] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 clientOffset=(0,19) visibleOffset=(0,0) visibilityRects=((0,0,1006,675))
[22:14:51:427] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:427] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 clientAreaWidth=1006 clientAreaHeight=656
[22:14:51:458] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:458] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 clientOffset=(4,23) visibilityRects=((0,0,1006,635))
[22:14:51:458] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:458] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 clientAreaWidth=998 clientAreaHeight=608
[22:14:51:489] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:489] [14735:14736] [DEBUG][com.freerdp.core.update] - ActivelyMonitoredDesktop activeWindowId=0x2b00d6
[22:14:51:490] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:490] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 show=maximized
[22:14:51:490] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:490] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 [ex]style=<0x15cfc000, 0x110 visible thickframe border caption> windowOffset=(0,0) windowClientDelta=(4,23) windowWidth=1006 windowHeight=635
[22:14:51:491] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:491] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 [ex]style=<0xb0000, 0x0>
[22:14:51:491] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:491] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 clientOffset=(4,632) visibleOffset=(0,609) visibilityRects=((0,0,160,27))
[22:14:51:492] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:492] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 clientAreaWidth=152 clientAreaHeight=0
[22:14:51:493] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:493] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 [ex]style=<0x15cfc000, 0x110 visible thickframe border caption> windowOffset=(0,609) windowWidth=160 windowHeight=27
[22:14:51:493] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:493] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 [ex]style=<0xb0000, 0x0>
[22:14:51:493] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:493] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 [ex]style=<0x15cfc000, 0x110 visible thickframe border caption>
[22:14:51:536] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:536] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 clientOffset=(4,23) visibleOffset=(0,0) visibilityRects=((0,0,1006,635))
[22:14:51:536] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:536] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 clientAreaWidth=998 clientAreaHeight=608
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 clientOffset=(4,631) visibleOffset=(0,608) visibilityRects=((0,0,1006,67))
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 visibilityRects=((0,0,160,27))
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 clientAreaWidth=152 clientAreaHeight=0
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 [ex]style=<0xb0000, 0x0>
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 [ex]style=<0x15cfc000, 0x110 visible thickframe border caption> windowOffset=(0,608)

**HERE, SENDING SC_RESTORE and SC_MAXIMIZE TO MAXIMIZE WINDOW.....**

[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 [ex]style=<0xb0000, 0x0>
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 [ex]style=<0x15cfc000, 0x110 visible thickframe border caption>
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - Alternate Secondary Drawing Order [0x0b] Windowing
[22:14:51:568] [14735:14736] [DEBUG][com.freerdp.core.update] - WindowUpdate windowId=0x2818262 [ex]style=<0xb0000, 0x0>


Review David's README pull requests

Task to do
Review the README.md Pull Requests: https://github.com/AgPipeline/transformer-soilmask/blob/update-readme/README.md and https://github.com/AgPipeline/transformer-canopycover/pull/6
Reason
Review the pull requests
Result
Better & approved README.md files

@jvanderleeuw What's the status of this?

It should be good; I talked with David the week before last and there were no large changes that needed to be made; I'll make sure with David today

merged PR


Bug: DEVMODEA is incorrectly sized and has wrong layout

Which crate is this about?
windows
Crate version
0.43.0
Summary
When trying to query info about a display, I found that the size is of the DEVMODEA is incorrect and that the layout also seems to be wrong. From the tests, the size of DEVMODEA should be 156, but size_of::() returns 164.
The layout is also incorrect, as for example dmPelsHeight value is in dmBitsPerPel.
By quicly checking the definitions, I found some of the incorrect values:

dmTTOption: DEVMODE_TRUETYPE_OPTION is 4-bytes, MSDN says 2-bytes (short)
dmColor: DEVMODE_COLOR is 4-bytes, MSDN says 2-bytes (short)
dmCollate: DEVMODE_COLLATE is 4-bytes, MSDN says 2-bytes
Not sure where the last bytes would be, but I guess padding.

Toolchain version/configuration
No response
Reproducible example
use windows::Win32::Graphics::Gdi::DEVMODEA;

fn main() {
    assert_eq!(std::mem::size_of::<DEVMODEA>(), 156);
}

Crate manifest
[dependencies.windows]
version = "0.43.0"
features = [
    "Win32_Graphics_Gdi",
    "Win32_Foundation"
]

Expected behavior
size_of::() == 156
value of dmPelsHeight is located in dmPelsHeight
Actual behavior
size_of::() == 164
value of dmPelsHeight is located in dmBitsPerPel
Additional comments
No response

Is this the same issue as #2138?

Yeah, it's the same, didn't find that other one, as I was looking for DEVMODEA  in the issue finder, not just DEVMODE


feat: add a real-time progress indicator for playground

Great addition! The real-time progress indicator for the playground will significantly improve the user experience by providing immediate feedback on the execution status of code snippets.
I have a few questions:

How does the indicator visually represent progress? Is it a percentage bar, a progress wheel, or something else?
Can the indicator be customized? For example, users might want to change the color or style.
How does the indicator handle errors? Does it provide a clear indication when something goes wrong?

Overall, this feature has the potential to be very useful and I'm looking forward to seeing the implementation details.


Added ability to send events to different channels depending on the value of a tag

This is something we required to send events coming from different portions of the same application to the right team in slack.
Provides configuration so that a specific tag can be used to direct the events to the right channel depending on that tag's value.

This change is

[Current coverage][cc-pull] is 66.46%

Merging [#43][cc-pull] into [master][cc-base-branch] will increase coverage by -4.21%

@@           master        #43   diff @@
========================================
  Files           5          5
  Lines         133        158    +25
  Methods         0          0
  Branches        0          0
========================================
+ Hits           94        105    +11
- Misses         39         53    +14
  Partials        0          0

Powered by Codecov. Last updated by [e99a61a...658a194][cc-compare]
[cc-base-branch]: https://codecov.io/gh/getsentry/sentry-slack/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/getsentry/sentry-slack/compare/e99a61a782cf37759e66b980c5817ce8b7d46b57...658a194d6dc726e74e3d2c7121d6133f77358066
[cc-pull]: https://codecov.io/gh/getsentry/sentry-slack/pull/43?src=pr

Would love to see this merged, we need it on our end as well.

Nice PR you should merge this!

Merge the beast, please!! We'd love to see this functionality, it would stop a load of people whinging about sentry in our organisation. Whingers do my nut in

The new slack integration supports this!


Add a simple tool to prepare job submission with simulate_prod for a HTCondor system.

Running small productions for testing reasons on local resources is a typical use case for simtools.
This PR adds a simtool simtools-simulate-prod-htcondor-generator which allows prepare a submission script and file for HT Condor system.
Example:
simtools-simulate-prod-htcondor-generator --output_path /lustre/fs24/group/cta/users/maierg/tmp/simtools-testing --apptainer_image /lustre/fs22/group/cta/users/maierg/apptainer/simtools-prod_20241218-090702.sif --priority 250 --simulation_software corsika_simtel --primary gamma --zenith_angle 20 --run_number_start 5 --number_of_runs 100 --azimuth_angle 180. --array_layout_name alpha --nshow 10 --site North --model_version 6.0.0 --log_level DEBUG

will generate the following condor submit file (with a queue of 100 jobs):
universe = container
container_image = /lustre/fs22/group/cta/users/maierg/apptainer/simtools-prod_20241218-090702.sif
transfer_container = false

executable = simulate_prod.submit.sh
error      = logs/err.$(cluster)_$(process)
output     = logs/out.$(cluster)_$(process)
log        = logs/log.$(cluster)_$(process)

priority = 250
arguments = "$(process) env.txt"

queue 100

and a run script:
#!/usr/bin/env bash

# Process ID used to generate run number
process_id="$1"
# Load environment variables (for DB access)
set -a; source "$2"

simtools-simulate-prod \
    --simulation_software corsika_simtel \
    --label simulate-prod \
    --model_version 6.0.0 \
    --site North \
    --array_layout_name alpha \
    --primary gamma \
    --azimuth_angle 180.0 \
    --zenith_angle 20.0 \
    --nshow 10 \
    --energy_range "3.0 GeV 330000.0 GeV" \
    --core_scatter "10 1400.0 m" \
    --run_number_start $((process_id + 5)) \
    --number_of_runs 1 \
    --submit_engine "local" \
    --data_directory /tmp/simtools-data \
    --output_path /tmp/simtools-output \
    --log_level DEBUG \
    --pack_for_grid_register simtools-output

Requirements:

availability of a apptainer of 'simtools-prod' (get it from the package registry on GitHub with e.g. apptainer pull --force docker://ghcr.io/gammasim/simtools-prod:20241218-090702
a env.txt file in the <output_path> containing the environment variables (e.g. DB access credentials)

Submitting with condor_submit simulate_prod.submit.condor will:

submit above jobs to an HT Condor managed queue
copy the env.txt file to each worker node
run the job in a container universe using apptainer image as given in the command line
all output is written relative to the <output_path>:

logs from HT Condor in <output_path/logs (includes the simtools printouts to the screen)
data, histogram, log files from the simulation execution into <output_path/simtools-output

No other output will be kept (all simtools output is written to the /tmp/ on each worker node. For testing, a job priority can be set to indicate the queue manager priorities.

Thanks for this. I will merge and add (either tomorrow or next year):

collection of corsika log files
possibility to collect simtools printout to screen (note that we are currently running this tools with the 'debug' option. Removing this makes the simtools printout not very informative and not worth to save.


update i18n and l10n for core.application/plugin.xml

update files: plugin.xml, bundle.properties, bundle_zh.properties , which under /plugins/org.jkiss.dbeaver.core.application/ .
core.application.zip

This PR still has conflicts..

Hi, Serge~ I have resolved the conflicts.


Admin and the Event Planner log ins need to be created

DoD: After clicking the link to create a profile, then it should be recorded in the Database. They should have an email and password as well as a Username. Then they should be able to log in and either manage events as the admin or request events as the planner.

Added in the scaffolding from Visual Studio and added in migrations to create the AppIdentity table to record any account created when registering an account through the web app. The admin is now able to approve/reject any events as well as being able to edit an event.


[BUG] Event Date Overlapping Issue on Card Display

Description
Describe the bug:
There is a bug affecting the card display feature in the GitHub repository, specifically related to the event date. The event date is not being displayed properly and is overlapping with other elements on the card. This issue hampers the readability and usability of the card, potentially leading to confusion for users attempting to view the event date.
Expected behavior:
The event date on the card should be displayed clearly and distinctly, without any overlap with other elements. This ensures that users can easily identify the event date and comprehend its significance without any visual hindrances.
Actual behavior:
Currently, the event date on the card is not displayed properly, resulting in an overlapping issue. The overlapping elements obstruct the visibility of the event date and may confuse users attempting to understand the date associated with the event.
Screenshots

Additional Information
Device (please complete the following information):

OS: Windows
Browser: Chrome

I am interested to work on this issue under GSSoC 2023 as a contributor

@akshitagupta15june please assign me this probem


Retry curl requests 3 times by default.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Make this the default instead of only using it in test-bot (https://github.com/Homebrew/homebrew-test-bot/pull/327).

Good call, thanks @reitermarkus!


Suppress Pasadena

Based on all the problems we are seeing with the Pasadena address data (see #9) and the over-complex building shapes (see #19) which also lack heights (see https://github.com/osmlab/labuildings/issues/3#issuecomment-67117627), I recommend we skip Pasadena as part of this import.
We can do this by skipping any features that have Source=PASADENA.GIS.

Suppressing Pasadena and Courtyards #27, we have ~299K out of ~314K buildings to import.

Closing this issue per #23, Pasadena will not be imported at this time due to it's complex geometry.


no action on the extension box

Hi, I cannot see any guide on left box.
I've installed serveral times, howerver it same. I can see some woding once I installed old version.
And, the icon is just double boxes, not "CODEGPT"

Hi @parkarck , we deprecate the old version. You can follow this instructions, but if you need assistance please setup a meeting here: https://calendly.com/pilar-hidalgo


Getting PgIndexFilter Error

Hey mate, so looks like this was solved before but has managed creep back in:
Error: Type 'PgIndexFilter' has already been registered!
at Object.newWithHooks (/home/ubuntu/server/backend_lnk/node_modules/graphile-build/src/makeNewBuild.js:603:17)
at /home/ubuntu/server/backend_lnk/plugins/postgraphile-filter-plugin/src/PgConnectionArgFilterPlugin.js:57:9
at Array.map (native)
at /home/ubuntu/server/backend_lnk/plugins/postgraphile-filter-plugin/src/PgConnectionArgFilterPlugin.js:52:40
at SchemaBuilder.applyHooks (/home/ubuntu/server/backend_lnk/node_modules/graphile-build/src/SchemaBuilder.js:289:20)
Would love any feedback

Can you provide the version numbers of postgraphile and postgraphile-plugin-connection-filter`?


window.set_position() has a weird behavior

use std::cmp::{max, min};
use minifb::{Key, MouseButton, MouseMode, ScaleMode, Window, WindowOptions};
use rand::Rng;

const WIDTH: usize = 640 / 2;
const HEIGHT: usize = 360 / 2;

fn main() {
    let mut window = Window::new(
        "WTF?",
        WIDTH,
        HEIGHT,
        WindowOptions {
            resize: true,
            scale_mode: ScaleMode::UpperLeft,
            ..WindowOptions::default()
        },
    ).expect("Unable to create window");

    while window.is_open() && !window.is_key_down(Key::Escape) {
        let window_pos = window.get_position();
        println!("window pos is {window_pos:?}");

        window.get_keys_released().iter().for_each(|key| match key {
            Key::Enter => {
                let window_pos = window.get_position();
                window.set_position(window_pos.0 + 10, window_pos.1);
            },
            _ => (),
        });

        window.update();
    }
}

Expected behavior:
The window jumps 10 pixels to the right
Current behaviour
The window jumped 28 pixels to the right and 110 pixels to the bottom
Wut?

Which platform/OS?

Which platform/OS?

Well, I'm running it in an LXC container. The host is KDE neon User - 5.24 aka Ubuntu 20.04 with both KDE and Gnome on Xorg. The container is Ubuntu 22.04, but it shouldn't matter at all.

Alright. I will have look

So I tried this locally on my Windows machine. Running Ubuntu 21.04 and i3 (I set the Window to floating) and the code above works just as expected for me so I'm not sure what is going on.

get_window position was a very crude hack, that was not very well tested. Especially on Linux.
Most systems disagree between setting the window position (usually without window decorations),
and getting the window position (usually with some window decorations).
This will more details on which desktop, window manager, etc is used.
It might not ever work 100% on all combinations.
If you have a detailed repro I might give it a shot again.

Closing until there is some more info about the issue. Please re-open if needed.


google-common [major] / google-webauth [minor] / google-gauth [minor]: Gemini functions

Adds support for function calling through langchain tools.
Adds support for bind().
Adjust testing, since AI Studio has gotten more strict.
Tested with AI Studio and Vertex AI. (Note this does not change the google-genai package.)
Missing:

Support for withStructuredOutput() (@jacoblee93 says he will handle)
Documentation

Fixes #4253
New exports:
@langchain/google-*/{types,utils}

I opted to export from separate entrypoints because if I exported all types/utils from the root entrypoint of common then we would be stuck with two bad options:

individually import & re-export modules inside packages like gauth/vertexai, etc so we aren't also exporting all the base classes from common. This means we will also need to remember to update the exports & cut releases for all these packages when a new type/util is added that we want to export.
re-export everything, including base classes. This would confuse users as they'd see ChatGoogle, ChatVertexAI, etc, etc in IDE autocompletes when looking at exports from, for example @langchain/google-vertexai

Added a new util zodToGeminiParameters since the google function definition does not cleanly allow for passing the result of zodToJsonSchema into the parameters field of a function.

Thanks both of you! Mostly nits, and looks like build is also failing.


[mirror] option protect mirror from commit removals (like branchprotection deny force push)

There was a case when an RPA tool is sold to another company and they revert to closed sourced licensing model while they are fully open and free. In this context they removed every single source code and commit history from their repositories and only left a readme file.  (repo link)
Most of RPA developers using automagica as their base tool have lost reach to source code and commit history in related repo. Some had the chance of having a local offline copy of the codebase and saved their robots in action.
However, What happens if I had a Gitea mirror repo to this remote and they would remove all commit history and source code. Do we lose all the codebase once the cronjob is triggered to sync mirror repositories?  Is there anything to avoid that? Is having a regular repository only option to avoid this risk of losing codebase caused by closing remote repo?
Cheers

short: yes in default mirror config
long: you can tell gitea to not remove old refs if they got deleted from source. This should safe souch thungs. But i didnt test it ...

Many thanks for your answer and beautifying my issue!

@UmutAlihan you can just disable mirror sync cron tab and do it manualy ...but i think it's not optimal

could you please provide an example app.ini line to only disable cron mirror sync w/out messing other regular cronjobs? It would really be very kind p2p act :)
(fyi I used docker-compose to spin up my gitea server)

@UmutAlihan -> https://docs.gitea.io/en-us/config-cheat-sheet/#cron---update-mirrors-cronupdate_mirrors
ENABLED = false

I'd suggest to close this duplicate in favor of #14076.


[BUG] Push notification was not delivered to iOS users

Describe the bug
@johnmpaul sent a push notification to users on Sunday October 11, 2020 at approximately 8:30 PM central time.
The push notification was sent to Android users of Safer Illinois, but was not received by iPhone users.
Expected behavior
The message should have been received by iOS users.
Screenshots
Here is a screenshot captured from an Android phone.

Smartphone (please complete the following information):

Device: iPhone 11
OS: 14.0.1

Today I have tested on Dev various situations and I don't see any problems client side.
My observations are:

When the target audience is set to iOS  - I receive notification on my test iPhone
When the target audience is set to Android  - I receive notification on my test Android device
When the target audience is NOT set to any platform  - I don't receive any notification neighter on iOS or/and Android
When I construct and send a json message through the Macbook terminal - the notification is received by the recipient no matter of the platform (iOS and Android). In other words no issues have been observed.

Additionally I cannot find anything meaningful that may explain this behaviour in the Firebase documentation and the on the Console itself
CC @johnmpaul @pmarkhennessy @isaac-galvan

Hi Mladen,
Did you try setting both platforms for a notification? You did not mention
that use case.
That was what I did.
Thanks, JP
On Wed, Oct 14, 2020 at 4:09 AM Mladen Dryankov<EMAIL_ADDRESS>wrote:

Today I have tested on Dev various situations and I don't see any problems
client side.
My observations are:

When the target audience is set to iOS - I receive notification on
my test iPhone
When the target audience is set to Android - I receive notification
on my test Android device
When the target audience is NOT set to any platform - I don't
receive any notification neighter on iOS or/and Android
When I construct and send a json message through the Macbook
terminal - the notification is received by the recipient no matter of the
platform (iOS and Android). In other words no issues have been observed.

Additionally I cannot find anything meaningful that may explain this
behaviour in the Firebase documentation and the on the Console itself
CC @johnmpaul https://github.com/johnmpaul @pmarkhennessy
https://github.com/pmarkhennessy @isaac-galvan
https://github.com/isaac-galvan
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rokwire/safer-illinois-app/issues/288#issuecomment-708332344,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJP53GYWJS5OWQ2LF263T3SKWBGXANCNFSM4SPATZ5A
.

--
Thanks, JP
cell:<PHONE_NUMBER>

@johnmpaul it's working

When the target audience is set to iOS & Android - I receive notification both on iOS and Android

Also please note the Firebase messages environment is determined during the build process. This means for example Dev pinned build will always use safer-in-illinois-dev	no matter of currently selected environment through the Debug panel. Prod pinned build will always use Prod Firebase configuration safer-in-illinois-prod - no matter of currently selected environment through the Debug panel. Also switching the Firebase configuration is not possible - it's determined build time.

Thanks Mladen. We will investigate further.
Thanks, JP
On Thu, Oct 15, 2020 at 1:17 AM Mladen Dryankov<EMAIL_ADDRESS>wrote:

@johnmpaul https://github.com/johnmpaul it's working

When the target audience is set to iOS & Android - I receive
notification both on iOS and Android

Also please note the Firebase messages environment is determined during
the build process. This means for example Dev pinned build will always use
safer-in-illinois-dev no matter of currently selected environment through
the Debug panel. Prod pinned build will always use Prod Firebase
configuration safer-in-illinois-prod - no matter of currently selected
environment through the Debug panel. Also switching the Firebase
configuration is not possible - it's determined build time.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rokwire/safer-illinois-app/issues/288#issuecomment-708983244,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJP53DOEP3SRNH7HRYATDTSK2VYJANCNFSM4SPATZ5A
.

--
Thanks, JP
cell:<PHONE_NUMBER>


Implement a diagnostic tool

New feature implementation
Implemented feature
Addresses most of #61, excluding the additional recommendations and the model issue (since lift / door models are not supported yet).
It is currently in draft and based on luca/drawing_editor because it needed some of the UI refactoring that went into that PR. However it also needs some of the refactoring that is in the fuel asset widget PR, so will move out of draft once that is finalized (and this widget is moved into a left panel rather than a floating window).
Implementation description
This PR adds the following features:

Checks for common types of issues, such as doors / lifts / docks with the same name, fiducials without a label and anchors that are very close but unconnected.
Suppressing issues by category.
Suppressing single issues.
Serializing / deserializing suppression data in the SiteProperties to make it persistent across sessions.

Example of the current behavior in the video below:
Screencast from 2023-07-25 11-20-31.webm
The architecture of the issues finding / inserting is fully decentralized. Issues are entities spawned as children of the workspace and systems are responsible for registering them (mapping a UUID to the issue type through a new App trait) and spawning them, for now only as a response to validation events.
There is one slightly brittle aspect to the solution, which caused the odd system ordering. Specifically, since issues are entities it is necessary to despawn them when a new validation request is received, to avoid having duplicates.
At the same time, it would be a fair bit of code duplication to have every system be responsible for cleaning up its own issues, then respawning them afterwards, so I created a system that performs the cleanup. Right now the events are sent in Update, cleanup is performed in PostUpdate and population with new issues in PreUpdate, if the ordering was changed in a refactoring this feature might behave unexpectedly (i.e. the cleanup system clearing all the newly created issues if it runs after their creation and the issue list always being empty).

Sadly there is a fair bit of noise coming from cargo fmt since let else was just stabilized, hopefully most of these changes should be straightforward


Support compiling operator &&

Is operator & a feasible workaround?

A workaround for this is that if you explicitly create logical variables (e.g. b <- logical(3)), then & should work for scalar or component-wise and, imitating R.  But you have revealed a true bug, which is that constructing logicals like  b <- TRUE or  b <- c(TRUE, FALSE) ends up treating them as numeric. Will fix.


Failed to establish a new connection/ connection refused

I am new to github and very novice at coding so I apologize for any errors in my issue submission. I am trying to use pyvizio to connect to my soundbar (smartcast soundbar 3651-E6) for use with Hassio. I am able to find the soundbar using the ip address and the discover command but sending commands from hassio terminal, attempting to pair and my hassio configuration all results in the same error. Any help would be greatly appreciated.
ERROR:pyvizio.vizio:Failed to execute command: HTTPSConnectionPool(host='<IP_ADDRESS>', port=7345): Max retries exceeded with url: /key_command/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x106603c50>: Failed to establish a new connection: [Errno 111] Connection refused'))
Hassio config
media_player:

platform: vizio
host: <IP_ADDRESS>
device_class: soundbar
suppress_warning: true

Can you try host: <IP_ADDRESS>:9000 instead? The package defaults to using port 7345 but I had to use 9000 for my sound bar

This worked perfectly! command line and the hassio component are now functioning properly. And I was also able to get my authentication token with the process detailed in the previous issues.


Robolectric 2.3 support without any Robolectric modifications

This branch pulls up all the private logic from RobolectricTestRunner into BurstRobolectricRunner.  Unfortunately, it's only compatible with Robolectric 2.3 right now, but with some minor changes to Robolectric (next major version) we could easily remove most of the logic from BurstRobolectricRunner.

Closing since this is old, but feel free to reopen it if you still want to pursue this.


Document time complexity

From my reading of the code, you're using arrays to store the keys and values and searching them linearly in get and set. I think you should document in your README that the polyfill has worst algorithmic complexity than a real Map implementation. The polyfill insert and lookup operations are O(n) whereas any decent native implementation will have O(1) (or perhaps O(log n)) operations. This difference may matter to your users.

@joebandenburg this is how you would naturally implement it (without overthinking optimisation in first place). It's also how ECMAScript specifies the lookup, (it's O(n)). So it's not a deviation from spec we're doing here, or other lie that needs to be exposed.
Still, of course lookup can be internally more optimal, and there's already opened issue for that. See https://github.com/medikoo/es6-map/issues/2

@medikoo : @joebandenburg is correct. See my question on point, including (for example, from the ES6 spec):

Set objects must be implemented using either hash tables or other mechanisms that, on average, provide access times that are sublinear on the number of elements in the collection

So using O(n) array lookup differs from the requirements of the spec, and so probably worth noting (IMHO).
Cheers

@brianmhunt yes, but it's not possible to obey to that paragraph without altering the objects which are added as map keys, and for frozen objects it's not possible at all.
So if you look at it from right perspective, every  Map polyfill should contain similar note :)
Anyway since v1 es6-map will do O(1) for most (not frozen of course) cases, I agree it's acceptable in ES5 world (it defnitely wouldn't in ES3 world, but this polyfil doesn't support ES3)

I noticed many people trying to use this library expected O(1), therefore I added note as requested to the readme.
It'll be addressed with v1, but due to limited time I can devote into it, I'm not able to state any certain date when it'll happen


Maximum number of prepared transactions reached

I'm trying to wrap up 2 NpgsqlCommand into 1 transaction.
using (var scope = new TransactionScope(TransactionScopeOption.Required, new TransactionOptions { IsolationLevel = IsolationLevel.ReadUncommitted }))
{
    try
    {
        using (NpgsqlConnection connection = new NpgsqlConnection(_db.Database.GetDbConnection().ConnectionString))
        {
            Transaction.Current.EnlistVolatile(new FakeEnlistmentNotification(), EnlistmentOptions.EnlistDuringPrepareRequired);
            connection.Open();
            connection.EnlistTransaction(Transaction.Current);
            _tablesService.Delete(companyId, 2, connection);
            _tablesService.Delete(companyId, 5, connection);
            scope.Complete();
            return Ok();
        }
    }
    catch (Exception e)
    {
        return InternalServerError();
    }
}

_tableService.Delete has defenition of NpgsqlCommand and calling ExecuteNonQuery() method. But after return Ok(); line i get an  PostgresException: 53200: maximum number of prepared transactions reached. I know that i can set max_prepared_transactions  in postgresql.conf file but what will happend if i even reach that param. As i understand i need to disable prepare transaction but i didnt find any way to do it. I'm already trying set Enlist to TRUE or FALSE but this doesnt do anything. IsPrepared property of command set to false.

There's no need to be using prepared transactions - these exist only to support distributed transaction scenarios, where you're trying to cover operations across different databases within a single transaction. It seems like in your scenario you're trying to do a simple transaction on a single database, so you should definitely not increase max_prepared_transactions. The fact that you enlist a second time likely triggers the escalation to a false distributed transaction.
To use the System.Transactions API (as you're trying), you should be able to remove the EnlistVolatile() and EnlistTransaction() calls, and Open() should automatically detect that an ambient TransactionScope is present and enlist to it. You can also use the simpler transaction ADO.NET API:
connection = Open();
var tx = connection.BeginTransaction();
_tablesService.Delete(companyId, 2, connection);
_tablesService.Delete(companyId, 5, connection);
tx.Commit();

Note also that PostgreSQL doesn't support the ReadUncommitted transaction isolation level - it will silently use ReadCommitted instead (see the docs).
Am going to close this as a usage issue, but feel free to post back here for more info or details.


Automatically infer local and remote path mappings for remote debugging

Currently we expect users to provide the local and remote path mappings when setting up remote debugging.
Then there's source references.
If we try to automatically infer local and remote path mappings, then source references might not work.
This issue is merely a reminder to look into the ability to automatically infer the path mappings.
https://github.com/Microsoft/vscode/issues/47914

We'll need to come up with a solution for this before we release the new debugger, as the default values in launch.json will affect this.

If we can then great, otherwise "oh well" 😉

Upstream issue https://github.com/Microsoft/ptvsd/issues/691

That would be great!
(I`m current using remote-workspace)

Possible solution - https://github.com/Microsoft/ptvsd/issues/691#issuecomment-425267344


android notif: On open when app in background, emit the event.

When the user has opened the app and then moved on to something else,
it may still be in memory for a while, including the JS environment.
In this state, if the user opens a notification and we simply launch
the app, the startup logic that looks at initialNotification won't run.
But that's what we've been doing. This is #3582 .
Fix that. First:

migrate this file to Kotlin, in keeping with our Kotlin/Java policy;
make a number of refactors that help this code better reflect the structure of what we're doing.

Tested end-to-end by hand in an emulator running Android 10.
Also clean up some long-standing noise in the logs, which was conspicuous in adb logcat -s ZulipNotif:V while end-to-end testing this.

EDIT: This is explicit on the issue thread, but I should mention it here too: thanks to @jainkuniya for spotting this issue and then investigating and diagnosing it!
I've gone and added the following lines, kernel-style, to the end of the key commit's message so the same point will be clear in our commit history too:
Reported-by: Vishwesh Jainkuniya<EMAIL_ADDRESS>Debugged-by: Vishwesh Jainkuniya<EMAIL_ADDRESS>
Separately, to connect some dots for GitHub's sake: this PR fixes #3582.

Thanks @ray-kraesig for the reviews! Based on your re-review, merging now.


examples: Add the jets and booktest examples

Move the end-to-end test into its own package: internal/endtoend
Add a new package for creating temporary PostgreSQL schemas:
internal/sqltest

Once this lands, the https://github.com/kyleconroy/sqlc-examples/ repository will be deleted.


Adopt plugin-hidden-parameter plugin

Description
I would like to adopt hidden parameter plugin, request post in mailing list: https://groups.google.com/g/jenkinsci-dev/c/lzFqejeiml8/m/7Pu2a3_UAgAJ .  See the requests to @wy-scm 9+ days ago asking for permission in https://github.com/jenkinsci/hidden-parameter-plugin/pull/7
Plugin: https://github.com/jenkinsci/hidden-parameter-plugin
Current Maintainer:

@wy-scm

Adding developer:

@madsjakobsen

Submitter checklist for adding or changing permissions

Always

[x] Add link to plugin/component Git repository in description above

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

Since the first request to @wy-scm was 9 days ago, I think we could allow this adoption request in as little as 5 more days.  Thanks very much @MadsJakobsen for being willing to adopt the plugin

Given last comment from Mark "ince the first request to @wy-scm was 9 days ago, I think we could allow this adoption request in as little as 5 more days." was 11 days ago already, I'm merging this.
Thanks

Thanks! and thanks @MarkEWaite for doing the videos and documentation for updating and releasing plugins, it's been tremendously helpful!


Modernize embulk-input-file with v0.11-ready API/SPI

embulk-util-file will be updated to 0.1.2 in another PR after this is merged to master. (#1410 needed to be on this branch to use 0.1.2, but this PR was developed before #1410. Rebasing everything would be hard...)

embulk-util-file:0.1.3 would work. Updated.

Thanks!


hook for estimating gas

To avoid having to hard-code the gas limits for a transaction, we need a hook that simulates the transaction. As JsonRpcProvider does not have a signer, we would also need the public address for an account with some ETH and various tokens, including stETH, wstETH, LDO, etc.

Discussed on the call, we will try to implement it in staking widget, if it will go nicely, we will move it to js-sdk


Ask about mechanical modeling in paper

Dear Adam Heins:
I'd like to ask some questions about mechanical modeling.

Is this equation in the Ee coordinate system？ In the Body frame, Fext is not zero because the object on Ee has velocity and acceleration.

How did you come up with this expression?
I don't have a good understanding of mechanics, and I hope I can get your answer.
It seems to me that F_GI is a minus sign equal to Fc, because in the Body frame:
$F_C + F_{GI} = ma$
So $F_{GI} =-(ma- F_{GI})$
The origin of $T_{GI}$ is because Fc does not provide Tau to objects?
Hope to get your reply！
Thanks！

For the first equation, it doesn't actually matter which frame it's in (being satisfied in one frame means it is satisfied in all others). In the paper it turns out to be convenient to represent it in the body frame, which is the representation for the second equation.
The second equation is the gravito-inertial wrench: the gravito-inertial force is just mass x acceleration minus gravitational force (rotated into the body frame). The torque is just the inertial torque. This is really just the Newton-Euler equations of motion for rigid bodies.
Your interpretation at the bottom is not quite correct. $ma$ is already part of the gravito-inertial force $f_{GI}$:
$f_{C} + f_{GI} = 0$
$f_{GI} = -(ma - mg)$

Thank you very much for your answer.
I still have some doubts about the first point. In the world  frame (ground  frame), the object has its own velocity and acceleration, but doesn't the first equation mean that it remains stationary or in a uniform straight line with zero external force.

The external forces acting on the system are the contact forces (contained in $w_C$) and gravity (contained in $w_{GI}$). This is true in the body frame and in the world frame. Depending on the contact forces, the object can certainly accelerate and/or rotate.


Add a way for manipulating FPS in startImageStream in camera plugin

Use case
 An option to manipulate FPS in startImageStream of camera plugin.

Proposal
 I'm using Camera plugin for one of my projects which is based on Real time face detection from an Image stream. Everything is working good except that face detection is catching even the blurred images. So I'm basically looking for a way to manipulate FPS of the image stream because I don't need images at that exact FPS. A lower FPS would help me to get clear picture of a face to run the model on.
P.S - This is my first issue report/feature request ever so I apologize if I haven't been able to explain what I wanted to say. If anything else required please let me know.

@av153k
There's similar issue describing your case, https://github.com/flutter/flutter/issues/54339.
Do you think that issue resemble yours ?

@darshankawar Yes it does. Thanks for pointing this out.

Closing this as duplicate of https://github.com/flutter/flutter/issues/54339. Please follow-up there for updates.


Its Not Work

I Try To Mining in my computer but nothing happen

What GPU do you have?
Also read the requirements for Ethereum mining: https://github.com/UnamSanctam/SilentETHMiner#requirements

im useing gtx 1650 4gb its not work

you have discord ? i need your help can you help me maybe ill doing something wrong

Read the requirements, 4 GB is not enough to hold the current Ethereum DAG.

what should i do

also why its not mining cpu like xmr

You should mine it with GPUs with higher VRAM (GPU Memory), Ethereum CPU mining is neither supported nor profitable at all, If you want to CPU mine use XMR.
XMR is for CPU mining, ETH is for GPU mining.

so i cant mining eth with 4gb ? how need gpu memory

No not Ethereum, your GPU needs 5 GB or higher. You can mine other Ethash coins like Metaverse, Ellaism, QuarkChain that require  have a smaller DAG. If you go to this page https://crypt0.zone/dag-file-size/CLO/7121742 and click the different coins it will say the required DAG size under "Calculated DAG file size:".


Can we customize the paypalbutton's onClick event? If can , how?

Can we customize the paypalbutton's onClick event? If can , how?
Cuz I want to deal with other things before I click the paypalbutton. Hope u can answer my question~

Hi @A-10ng! As of now you can not override the onClickListener.  As an alternate solution you could wrap the button inside a View and set the onClickListener to that View where inside it could call paymentButton.performClick once you have finished with your operations.
We'll take your suggestion into consideration.

Thank u ~

Closing due issue being resolved


When paused in landcape mode, the title & duration slider remain forever on screen obstructing the video

Version:0.4.3-c8d2db8
When paused, it would be nice if the title, duration slider & play/pause button etc, would fade out(just like when playing)...
This would help in watching (or better reading) videos that have text at the botton of the video.
Let the sliders come back.. may be.. after a single tap

Hmm, interesting use case. I admit I've never thought about that. My guess this is an issue when viewing the video in landscape mode. I should be able to squeeze this feature for 0.5.x. It does not sound too complicated.

Just noticed something I completely forgot about. You can actually hide video controls when playback is paused in landscape mode. You just need to manually perform down swipe gesture on the bottom bar.

Thanks for letting me know that.. :)
Hmmm... its almost an edge-start edge-end short distance swipe.....
IMHO swipes should have some distance coverage for an ergonomically satisfactory swipe.. :)

Already fixed on master and will be released in 0.5.0. Closing.


issue 1346 codecov is 400ing

Fix coverage and add info about OWNER_ALIASES

Fixes #1346

the codecov is fine i guess since main coverage was not working and behind


Fix: Limit timestep-pos-encoding to Rec-Sbale

What?
Minor fix to limit the timestep positional encoding to only rec-sable.
How?
Add a timestep flag checker in recurrent apply call

Hello, I tried to fix the issue
This is what I did:
Modify the SableNetwork class to include a flag for timestep positional encoding and only apply it in the recurrent version. Update the call and recurrent methods accordingly.
I also created pull request: https://github.com/instadeepai/Mava/pull/1131

[!CAUTION]
Disclaimer: This fix was created by Latta AI and you should never merge before you check the correctness of generated code!
The fix provided by Latta AI might not be complete and it can serve as an inspiration.

This bug was fixed for free by Latta AI - https://latta.ai/ourmission
If you no longer want Latta AI to attempt fixing issues on your repository, you can block this account.


GraphSONv2 vs. returned format

The documentation says that GraphSONv2 is returned, in fact GraphSONv2 looks different to the official format: http://tinkerpop.apache.org/docs/3.4.6/dev/io/#_edge_2
What is returned looks like GraphSONv1 with TinkerGraph serialization: http://tinkerpop.apache.org/docs/3.4.6/dev/io/#_tinkergraph
What is your recommendation for a local variant of the Cosmos Graph for development, as the Cosmos emulator does not support graphs? We're using the Tinkerpop Server, but having difficulties to configure it to returned the same format that is returned by Cosmos Gremlin.

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

ID: 0374b015-1139-9df8-6c96-a8657a088748
Version Independent ID: 2e18d44d-de3c-eee5-7932-0a5a967e9b17
Content: Azure Cosmos DB Gremlin support
Content Source: articles/cosmos-db/gremlin-support.md
Service: cosmos-db
Sub-service: cosmosdb-graph
GitHub Login: @LuisBosquez
Microsoft Alias: lbosq

@dArignac Thank you for bringing this to our attention. We are investigating and will update you when we have accurate information to share.

@dArignac A previous question explains your question a bit - https://github.com/MicrosoftDocs/azure-docs/issues/37857
Also the https://docs.microsoft.com/en-us/azure/cosmos-db/gremlin-compatibility document calls out v3 is not available and what is available in v2. Please check these and let us know what exactly you are looking for.

@NavtejSaini-MSFT
Let me clarify this. When I retrieve vertices from Azure Cosmos DB using GraphSONv2, or more specifically GraphSONMessageSerializerV2d0, then the returned JSON looks like the following:
[{
	"type":"vertex",
	"id":"8fff9259-09e6-4ea5-aaf8-250b31cc7f44",
	"label":"vert label",
	"properties":{
		"pk":[{
			"id":"8fff9259-09e6-4ea5-aaf8-250b31cc7f44|pk",
			"value":"test"
		}]
		,"email":[{
			"id":"80c0dfb2-b422-4005-829e-9c79acf4f642",
<EMAIL_ADDRESS>		}]
		,"abcd":[{
			"id":"4f5a5962-c6a2-4eab-81cf-5b530393b54e",
			"value":true
		}]
}}]

If I query the vertices in a Apache Tinkerpop server using the same serializer, the content looks like this:
[{
        "@type":"g:Vertex",
        "@value":{
        "id":{
            "@type":"g:Int64",
            "@value":30
        },
        "label":"vertex label"
        }
    }]

Though Azure only support the GraphSONv2 serializer, the returned format of the JSON is not GraphSONv2 but GraphSONv1 with TinkerGraph serialization (see http://tinkerpop.apache.org/docs/3.4.6/dev/io/#_tinkergraph). At least that is what I see if I compare the different GraphSON versions http://tinkerpop.apache.org/docs/3.4.6/dev/io/#graphson.
That is confusing, as we are using the TinkerPop Server for local development with the same serializer config and it returns the "real" GraphSONv2 format.
One more word to the Cosmos DB emulator - that setup is basically unusable for us, as we need administrative access to the notebooks we're using, which we don't have. The Windows Container version also does not work, as we're using Linux containers and Docker for Windows can only run one variant. That is why we've chosen the Tinkerpop Server. But we can't get it configured to return exactly the format that Azure Cosmos Graph returns, that's why I asked about it.

@LuisBosquez Please check the feedback from user and help here regarding the same.

@dArignac Gremlin API supports V2 as mentioned in the doc, can you point out where the format didn't match.
The emulator can be used with Gremlin API as well, there is a specific option you would use to enable. Here is more info on it: https://docs.microsoft.com/en-us/azure/cosmos-db/local-emulator#gremlin-api

@SnehaGunda Yes, the emulator does work. I missed the flag to enable the endpoint, my fault, sorry for that! However, as stated above, it is not of use for us.
Interesting is that the docs then require to set the serializer to GraphSONMessageSerializerV1d0 in the TinkerPop console to connect to it.

@dArignac Thanks for your patience, I had an email conversation with Luis internally.
Currently, the GraphSON format & serialization version are a little complex to explain. We don't have full support for GraphSONMessageSerializerV2d0 however, we are closer to it than V1. So we recommend using the V2 parameter in the connector. But the return the format is close to V1 style. Our team is actively working on a better implementation of this feature.
I made some fixes to the doc to remove GraphSON and just use JSON format. Hope this helps to some extent, we can document it well after the new version is available.

Let us know if you have any further questions, if not we will close this issue.

#please-close


CustomElements

Has anyone had any luck integrating CustomElements into this or the ASPNet/JavaScriptServices project
It doesn't seem to play well with webpack and es6 but it's required for a third party software, Clarity Design System, that I want to utilize in my app.

You shouldn't have much of a problem utilizing some Web Components with Angular in general, but, unfortunately Custom Elements cannot be server-rendered (as they only exist in the Browser-world).
Try Disabling server-side rendering, and you shouldn't have too many problems using them here or with JavaScriptServices.
Just remember that you'll need quite a few polyfills to get those working in older browsers, I believe you can only shim down to IE10+, I dont think IE9 supports them even with shims (I can't remember off hand).
Sidenote, also in the works, there is an initiative to make integrating Web Components with Angular even easier and more fluid 💯
I'd recommend taking a look at https://github.com/platosha/angular-polymer, as there's a helpful story on integrating Web Components with Angular there.
I'll close this out for now as overall Web Component support isn't something I can add to the repo, but feel free to message me on here!


Download different extra metadata for Spanish

According to a new requirement: https://docs.google.com/spreadsheets/d/1tTmp_CqjGHezSw_cpDMKjJZUTyTSykkGlzWpStzCs0U/edit#gid=0
There should be extra metadata imported for Spanish. And as I understand, this should be different than the extra metadata for English.
Currently: the metadata of every other manifest (content pack) is imported. The new requirement is to group the manifests by language and don't import English for Spanish or viceversa.

It verified pass in Narwhal v7.11.0 in spanish pack


Added logo in DG

Codecov Report
Base: 74.75% // Head: 74.75% // No change to project coverage :thumbsup:

Coverage data is based on head (8e38ae2) compared to base (6f7f573).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #238   +/-   ##
=========================================
  Coverage     74.75%   74.75%
  Complexity      556      556
=========================================
  Files            86       86
  Lines          1632     1632
  Branches        163      163
=========================================
  Hits           1220     1220
  Misses          352      352
  Partials         60       60

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

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


Add a light style to the Carbon components that support it

Detailed description

Describe in detail the issue you're having.

Provide an option to choose between the default style and the light style for the components that are supporting it in Carbon:

[ ] Dropdown
[ ] Input
[ ] Select

That new setting should have the same name for all components. For example, style: "" | "light" = ""; or light = false.

Is this a feature request (new component, new icon), a bug, or a general issue?

Yes

Is this issue related to a specific component?

Select

To whoever picks this up, we decided that @Input for controlling this should be named theme.

https://github.com/IBM/carbon-components-angular/pull/215 is fixing the Input and Textarea


some basic changes

would not know if you want these, however you can overlook them incase you want to merge this.
CHANGES:

allowed usernames in singleplayer mode (i tested, there is random numbers after Player.)
changed some stuff in core.glsl to make it a bit more "readable" and also bit better on performance
changed the pushout block code in the update in player to be in its own function, i do not know if this runs better or not.

also wrote some code for random splash texts if you want that (main menu), however i will not be pushing that to my fork.
oh that also doesn't use splashes.txt because uh, i just didn't think of that...

Wow, you got some stuff! Would you like me to add you to the contributors part of the README.md?

Wow, you got some stuff! Would you like me to add you to the contributors part of the README.md?

uh yes, please.
it would only be fair!

Alright, i'll merge the pull request and add you to the README.md.

Alright, i'll merge the pull request and add you to the README.md.

neato


Create Assignment One

TODO List

[ ] Stub out token class.
[ ] Stub out Interpreter (except for error handling). This isn't going to be used in the first assignment but it's simpler to include it from the start.
[ ] Add REPL loop.
[ ] Add token tests.
[ ] Add (to start with) simple instructions for how to get tests to pass.

NOTES
Don't stub out error handling. This is something we don't expect students to know about and it's nice to have errors which make sense that don't crash the calculator.

This was closed when we pulled Tuckers stubbed out version of the Token class and it's tests into master.


Fix alt-text-bad-value warnings

Ripple 3 validation cleanup for alt-text-bad-value warnings. Replace with new markdown syntax and use type='icon'.

Docs Build status updates of commit 92b521d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/edifact-transaction-sets-iftsai-wkgrre.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Hi @MandiOhlinger, this PR is ready to merge.

@dksimpson Oh geez. I hope these changes were a simple copy/paste. Either way, thank you!


QdrantIndex loses client in composability

When creating a ComposableGraph over a composite index of Qdrant indices, queries error in
File ...gpt_index/gpt_index/indices/query/vector_store/qdrant.py:61, in GPTQdrantIndexQuery.__init__(self, index_struct, embed_model, similarity_top_k, client, *
*kwargs)
     58     raise ValueError(import_err_msg)
     60 if client is None:
---> 61     raise ValueError("client cannot be None.")
     63 self._client = cast(qdrant_client.QdrantClient, client)

ValueError: client cannot be None.

Seems that the Qdrant client is not carried over to the composed index or graph. Created the query like this:
i1 = GPTQdrantIndex([d1], client=client, collection_name="i1")
i2 = GPTQdrantIndex([d2], client=client, collection_name="i2")
li = GPTListIndex([i1, i2])
graph  = ComposableGraph.build_from_index(li)

Any update?
If anyone knows how we can use Vector Stores Indexes with ComposableGraph, please help
https://stackoverflow.com/questions/75549286/gpt-index-issue-using-composablegraph-with-vector-stores

I dug in a bit, the clients (I assume this affects weaviate as well) are not part of index_struct that is copied over in composability, causing the error. If I read it right :) May be a bigger change needed to fix it.

@Mikkolehtimaki let me know if this is still an issue! apologies for the delay, 0.4.15 should have fixed this https://discord.gg/dGcwcsnxhU


discord_ui.Message.wait_for("select" ...) returns a ComponentContext, not SelectedMenu

According to the docs, discord_ui.Message.wait_for("select", ...) returns a SelectedMenu item; however, in my testing, it returns a ComponentContext object. Importantly, the latter lacks a selected_values item; in order to access the selection, I have to use component.data["values"].
This discrepancy also affects this line in the example code.

I tried to reproduce it with the example code but for me it works, it gives me a SelectMenu with .selected_values, maybe try to update?

ah wait now I got the issue! I'll fix it

Okay seems like I somehow already fixed it, I just gotta upload the new version, thx for reporting! ❤

okay this should be fixed in v4.2.14


Transform is applying twice when using makeImageFromView on android

Description
When using makeImageFromView, children views with transforms on them are displayed in the snapshot as if their transforms were doubled.
Version
0.1.216
Steps to reproduce
Run on an android device or emulator.
Copy below code, noting that one child has translation from a transform, and the other has top and left positioning. Compare the positions rendered versus the output from makeImageFromView. If you half the transform translations of child one, the views should show up in the exact same position in the skia image.
Snack, code example, screenshot, or link to a repository
      <View
        collapsable={false}
        ref={makeImageRef}
        style={{
          position: 'absolute',
          top: 0,
          left: 0,
          width,
          height,
        }}>
        <View
          style={{
            position: 'absolute',
            transform: [{translateX: 100}, {translateY: 100}],
            width: 80,
            height: 80,
            backgroundColor: 'red',
          }}
        />
        <View
          style={{
            position: 'absolute',
            top: 100,
            left: 100,
            width: 80,
            height: 80,
            backgroundColor: 'blue',
          }}
        />
    </View>

Thanks for reporting this. Is this only for absolute positioning?
Does this happens on iOS or android or both? I have to assume that it's an android-only issue?


⚠ Mobitopup has degraded performance

In 9dacb96, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3032 ms

Resolved: Mobitopup performance has improved in 2f5a21a.


Fix typo in basic-css-hex-code-colors. Add article -the- to description.

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no): Closes #13581

Description

This change is a first-timers-only contribution.
Adds article 'the' to a sentence in the challenge's description.
This is an additional change suggested by @erictleung after one made by @eti8.

closes #13581

@erictleung :
I did this:

Modified the file to add Greeenheart's suggestion
Commited the change --(I realized that my question of 'un-pull request' was wrong. I needed to squash commits according to FCC Contributing Guide) :open_mouth:
git pull --rebase upstream staging --(Update my local fork of FCC)
git push origin staging --(Push the update just done to my remote github fork. OPTIONAL )
git checkout fix/typo-in-html-and-css --(Change to my local branch where edits are made)
git rebase -i staging --(Squash the last two commits of my testing branch, with a rebase against the updated version of my local staging. This was a little painful, never done before) :cold_sweat:
git push -f origin fix/typo-in-html-and-css --(Push the squashed commits to my remote branch)

So that's it. I hope it was well done.
Thanks for your support to all FCC comrades! I've been learning very A LOT from this project.
:+1: :+1: :+1:

@homerrowsky Thanks for contributing! :smile: 🙌

@homerrowsky thanks for your contribution!


Tests (unit / int / accept) setup

Because we make great code and great software ( :) ), we need to have a proper way to test our code.
We need to determine here :

Minimum code coverage
Unit / Integration test lib (Jest is already there... might be a great fit since we are mainly using React. Open to suggestion if there is something better for electron app!)
3 : Acceptance tests ?

Not a big fan of 100% coverage. 90% is usually good enough for me.
I assume we will not be using a lot of electron native feature, except storing credentials. If so, Jest is probably good enough. If we add a lot of native electron code in the future, we might need another solution for testing it.
Spectron is the chromedriver for electron (https://electronjs.org/docs/tutorial/using-selenium-and-webdriver). Tescafe also has a plugin, but the latest built seems to be failing.

1 : Ok
2 : Let's stick to Jest for now since we might also stick on "web oriented" features at first (we're not electron pros yet ;) ). We'll consider other solution if implement more native feature
3 : I've had better experience with testcafe on web applications but don't know how it react on electron app. We should probably give it a try when we'll feel the need of adding end-to-end tests... For now, let's start by adding some features with simple unit test ;)


GE 45606 / ZW3001 in-wall paddle dimmer doesn't report correct state

Problem/Motivation
While this worked correctly with my old Vera hub, with Z-Wave JS UI the GE 45606 dimmer switch often does not show the correct device state.
Expected behavior
If the light is turned on, it should show as on in the UI.
Actual behavior
The light is turned on but shows as off in the UI.
Steps to reproduce
If the light is on, restart Home Assistant and it will invariably show as being turned off.
Proposed changes
Correctly detect the device state and display it properly.

I toggled the switch on and off manually repeatedly, and it's currently off but reporting a value of 47% in both Home Assistant and Z-Wave JS UI. So it's not the first issue. The logs show a number of values from when I was pressing on the switch, so it is getting updates, just not always reflecting the current value.
I don't know what a Lifeline group is, but when I look at Groups in the UI it shows 'No data available.'
The link you sent didn't really describe how to enable polling, but I did find it here:
https://zwave-js.github.io/zwave-js-ui/#/device-support/polling
From there I was able to set a polling interval of 10s and it found that the switch was turned off. I'm assuming this should work then, but if there's a better preferred approach would appreciate the advice.

1.13.3

That's the addon version, I need the version of zwave-js-ui. You copy/paste it from the info icon in top bar, on the right

zwave-js-ui: 8.18.0
zwave-js: 10.23.1

When I look at Groups in the UI it shows 'No data available.

This is an association issue. If no associations are set when you physically turn the light on and off it will not report information to Z-Wave JS UI.  I verified this with several of my own devices. Here's two pictures that show what should and shouldn't see on the groups page.

That doesn't seem to be the case for me - when I toggle the switch on/off manually it does report values to Home Assistant, but it doesn't necessarily report every value. So if I toggle it on and off several times in succession I'll see activity in the logs but the end state may not be reported correctly, which is where polling helps.


lines-between-class-members only supports exceptAfterOverload with 'always'

Description
PR with failing test for issue #240
The exceptAfterOverload: true option for the TS version of lines-between-class-members only applies if the first options is 'always'. If using the Object version with 'enforce', overloads are still flagged as an error.
Linked Issues
https://github.com/eslint-stylistic/eslint-stylistic/issues/240
Additional context

Could you add the test case at the end of the list with a comment linking to the issue? Thanks.

@antfu sure, done.


OpenTelemetry Service Operator for Kubernetes

As the setup of the OpenTelemetry Service might be complex, with a set of agents and collectors, it probably makes sense to create an Operator for Kubernetes. An additional advantage is that other Operators, such as the Jaeger Operator, might indirectly make use of this, by creating CRs based on the end-users requests.

I can start working on this right away. I just need this issue to be assigned to me, and a new GitHub repository, probably under open-telemetry/opentelemetry-operator.

@jpkrohling there is already an ongoing effort in this area (not yet captured in the issues here). You may want to sync with @owais to avoid duplicate work.

Is it public already, or should we use this issue here to discuss it?

There was some misunderstanding here: what @owais has been investigating is about enriching telemetry data with k8s information, not a k8s Operator to facilitate management/deployment of the service.

I'll go ahead with it then. @bogdandrutu, @SergeyKanzhelev, could any of you create a repository named opentelemetry-operator and concede me push rights?

Future versions of the operator might manage more OpenTelemetry pieces, not only the service. Also, the code for the service might be big enough to deserve its own lifecycle/release scheme/dependencies/...

I have started a new operator project under my account. Once I have an initial working version, I'll add a notice to this issue here and we can then decide if it fits this repository.

An initial version of the operator is ready. It still can't deploy a production-quality otelsvc, but it is very close already to how the operator will look like.
@tigrannajaryan, @bogdandrutu: once the final repository is created, let me know and I'll send a PR. Until then, I'll keep working on it under my own account.

It still can't deploy a production-quality otelsvc

@jpkrohling is this due to some problem in otelsvc? If so please create an issue in this repo so that we can fix it.

once the final repository is created, let me know and I'll send a PR. Until then, I'll keep working on it under my own account.

Thanks for your work. Based on Bogdan's suggestion this will need to wait until contrib org is setup. Until that when you feel that the operator is ready we (otelsvc maintainers) can review it and link to it from this repo.

is this due to some problem in otelsvc?

Not at all. It's more like it's missing features that I would expect from a production-grade operator, like opening the proper ports based on the configuration (ie, expose 14250 only when the receiver jaeger-grpc is used) or the ability to update/upgrade existing instances. It also lacks a mode property, to get the collector deployed as a DaemonSet (mode=agent). Currently, it's only being deployed as a Deployment (mode=collector).
The underlying otelsvc is using an image that I built and published under my Quay namespace, as the only image available seems to be under omnition's namespace, so, it would be nice to switch to an official image once that is available.

I believe the OpenTelemetry Operator is now ready to be reviewed. It's currently located here:
https://github.com/jpkrohling/opentelemetry-operator/
If this is good enough for an initial release and contains no blockers, I would like to request a new repository either under the official organization (open-telemetry), or under a "-contrib" organization. I certainly prefer this to be part of the official organization.
cc @objectiser, @tigrannajaryan, @pjanotti, @bogdandrutu.

ping @bogdandrutu, @SergeyKanzhelev

Consider putting the Operator under https://github.com/open-telemetry/opentelemetry-collector-contrib.

@songy23 unfortunately, all the tooling around operators expect the operator to reside on its own repository. To me, it doesn't matter much under which organization it resides (open-telemetry, open-telemetry-contrib, ...), and I think the other people who will help me maintain it won't mind that much as well.
The only thing that I think that would be beneficial is to have this operator close to the open-telemetry project, and not in some private account.

As discussed yesterday in the SIG call, I would like to have the operator reviewed, with the goal of placing it under its own repository owned by SIG Collector, under some organization managed by the OpenTelemetry project.
As promised, here are the parts that are of interest for a first review:

current repository: https://github.com/jpkrohling/opentelemetry-operator
the boilerplate code was generated by the operator-sdk
bootstrap code is located in cmd/manager/main.go, but that's pretty much all generated boilerplate
CRD is at deploy/crds/opentelemetry.io_opentelemetrycollectors_crd.yaml and an example is at deploy/crds/simplest.yaml
reconciliation loop starts at pkg/controller/opentelemetrycollector/reconcile.go. All sources in the same package deal with reconciliation of the objects required to build an OpenTelemetry Collector installation.
The upgrade mechanism to migrate from one OpenTelemetry Collector to the next is located at pkg/upgrade/upgrade.go.

Overall this is looking really good.  I have run it in a local k8s environment successfully and have used it to operate an otel-collector.
I agree with @jpkrohling this belongs in its own repo regardless of the organization.

Nice, thanks @joe-elliott!
@tigrannajaryan, do you, or anyone else, want to review it as well before moving it off of my personal account?

@jpkrohling I had a very cursory look and see that your repo already has an established CI and all the wiring for building is in place (which is good). If we wanted to move it to this or contrib repo we would need to make changes to the build process at least. I do not see good reasons why we would want to do that and why not keep it as a separate project.
I think a better approach would be to create a separate repo under open-telemetry org and move it there.
@open-telemetry/technical-committee what do you think?

@jpkrohling I took a quick look and LGTM. Since we now have the ability to build multiple distributions of the collector, would it make sense to add a field that allows to specify which build/distribution of Otel the operator should use?
We already have core, contrib and omnition distributions. I think Google has one they use internally as well. May be we can we add a docker_image or docker_repo field that allows fetching the image from a custom source? Also would be nice if the config allows to pin to a specific version of collector.
So may be something like:
$ kubectl apply -f - <<EOF
apiVersion: opentelemetry.io/v1alpha1
kind: OpenTelemetryCollector
metadata:
  name: simplest
  docker_image: custom-registry/image:tagname
spec:
  config: |
    receivers:
      jaeger:

    processors:
      queued-retry:

    exporters:
      logging:

    pipelines:
      traces:
        receivers: [jaeger]
        processors: [queued-retry]
        exporters: [logging]
EOF

@owais This can be done now:
$ kubectl apply -f - <<EOF
apiVersion: opentelemetry.io/v1alpha1
kind: OpenTelemetryCollector
metadata:
  name: simplest
spec:
  image: custom-registry/image:tagname
  config: |
    receivers:
      jaeger:

    processors:
      queued-retry:

    exporters:
      logging:

    pipelines:
      traces:
        receivers: [jaeger]
        processors: [queued-retry]
        exporters: [logging]
EOF

Great. Overall LGTM. I'll try it out on a local cluster today and share any feedback here.

Any further feedback on this?

@open-telemetry/technical-committee please post your thoughts on this. If you agree with what I wrote in my previous comment please create a repo for the operator.

@bogdandrutu, @SergeyKanzhelev, @carlosalberto, @yurishkuro, could any of you please create the repository if there are no objections?

Created https://github.com/open-telemetry/opentelemetry-operator
@jpkrohling you're an admin for now (https://github.com/open-telemetry/opentelemetry-operator/invitations), please finish the set up .

Thanks @yurishkuro! I accepted the invitation and was able to push my repo there, but looks like I can't see the "Settings" tab, which is required to setup the CI tokens under the "Secrets" page.

for some reason you only had Write access, I just changed it to Admin.

Closing, any further discussion can be done at https://github.com/open-telemetry/opentelemetry-operator