Flutter 1.17.1 | Big Sur 11.4 | While running flutter doctor in terminal it is giving error Bad state: Future already completed error

Steps to Reproduce

Check Mac OS system version is 11.4
Download fresh flutter 1.17.1 from flutter official link
Install all the requirements
run flutter doctor

Logs
Building flutter tool...
Unhandled exception:
Bad state: Future already completed
#0      _AsyncCompleter.complete (dart:async/future_impl.dart:42:31)
#1      _NativeSocket.startConnect.<anonymous closure>.connectNext.<anonymous closure> (dart:io-patch/socket_patch.dart:642:23)
#2      _NativeSocket.issueWriteEvent.issue (dart:io-patch/socket_patch.dart:1045:14)
#3      _NativeSocket.issueWriteEvent (dart:io-patch/socket_patch.dart:1052:12)
#4      _NativeSocket.multiplex (dart:io-patch/socket_patch.dart:1073:11)
#5      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (9 tries left)

manish.patel@Manishs-MacBook-Pro-2 ~ % flutter --verbose create foo
Building flutter tool...
Unhandled exception:
Bad state: Future already completed
#0      _AsyncCompleter.complete (dart:async/future_impl.dart:42:31)
#1      _NativeSocket.startConnect.<anonymous closure>.connectNext.<anonymous closure> (dart:io-patch/socket_patch.dart:642:23)
#2      _NativeSocket.issueWriteEvent.issue (dart:io-patch/socket_patch.dart:1045:14)
#3      _NativeSocket.issueWriteEvent (dart:io-patch/socket_patch.dart:1052:12)
#4      _NativeSocket.multiplex (dart:io-patch/socket_patch.dart:1073:11)
#5      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (9 tries left)
Unhandled exception:

manish.patel@Manishs-MacBook-Pro-2 ~ % flutter doctor -v
Building flutter tool...
Unhandled exception:
Bad state: Future already completed
#0      _AsyncCompleter.complete (dart:async/future_impl.dart:42:31)
#1      _NativeSocket.startConnect.<anonymous closure>.connectNext.<anonymous closure> (dart:io-patch/socket_patch.dart:642:23)
#2      _NativeSocket.issueWriteEvent.issue (dart:io-patch/socket_patch.dart:1045:14)
#3      _NativeSocket.issueWriteEvent (dart:io-patch/socket_patch.dart:1052:12)
#4      _NativeSocket.multiplex (dart:io-patch/socket_patch.dart:1073:11)
#5      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (9 tries left)

Hi @tw-manishp,
Thanks for filing the issue, could you please try this solution

Hi @tw-manishp,
Thanks for filing the issue, could you please try this solution

I tried running same, but getting this error for that command  as well
manish.patel@Manishs-MacBook-Pro-2 ~ % flutter config --no-analytics
Building flutter tool...
Unhandled exception:
Bad state: Future already completed
#0      _AsyncCompleter.complete (dart:async/future_impl.dart:42:31)
#1      _NativeSocket.startConnect.<anonymous closure>.connectNext.<anonymous closure> (dart:io-patch/socket_patch.dart:642:23)
#2      _NativeSocket.issueWriteEvent.issue (dart:io-patch/socket_patch.dart:1045:14)
#3      _NativeSocket.issueWriteEvent (dart:io-patch/socket_patch.dart:1052:12)
#4      _NativeSocket.multiplex (dart:io-patch/socket_patch.dart:1073:11)
#5      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (9 tries left)

Unfortunately, the version you're using is outdated and is no longer supported, it's not feasible to provide any bug fixes for such an old version of Flutter.
I kindly advise you to update to the latest Fluter stable, if you encounter this issue, please comment below with the details from the latest stable, including steps to replicate the issue.
Can you please upgrade to the latest  stable and try again
Execute the following commands to upgrade
flutter channel stable
flutter upgrade --force
flutter doctor -v
If the problem persists, can you please provide your flutter doctor -v with the upgrade.
Thank you

Flutter 1.17.1 is required by the project. Do we have any work around.
I cannot disable Firewall because I don’t have access to disable it.

Please take a look at this stackoverflow thread if this helps.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Make a section in @NonNull match the one in @Nullable.

I'm wondering if the difference is just historical. (But I haven't checked.) Alternatively, there may be a reason behind it that didn't occur to me in all 5 seconds of thought that I gave it.


Remove Owen013.HikersModSHAddon from database

What is the reason for removal?

The mod is now obsolete, as Smol Hatchling and Hiker's Mod now communicate through APIs.


1.12.2 Mod Registry issue, doubles the mod's loading with certain launchers.

With certain mod launchers (GDLauncher for example), the mod is registered twice. This is a rare issue, but one I've only ever seen once before.
See:

When there is only one instance in the /mods folder. I have a friend who fixed his mod with this same issue, I can ask him how he fixed it.

This is a screenshot of a different mod.
Can you test on a new instance with only my mod, to see if the problem happens?

Oops, so sorry, went to the wrong issues page!


Incorrect phone number on email

Just received email titled
You now have Audio Conferencing for Microsoft Teams or Skype for Business Online – Here is your dial-in information and PIN
from Microsoft Audio Conferencing<EMAIL_ADDRESS>(Probably wrong place to submit this. Only place could find).
Email states
"Conference phone number:  +44 (20) 3787xxxx"
That number makes no sense. This is not a UK format number. It implies I should remove the 20 which I don't think is the intent.
Internal UK phone numbers are formatted like: 020 1234 5678
So when dialling in form outside UK, it becomes: +44 20 1234 5678
The international format is to put the zero in parenthesis so they can decide to either start the dialing with +44 or 0:
+44 (0) 20 1234 5678
(or if you're using some funky system that means I'm supposed to use 20 in a weird way, you'll have to explain that because I don't know how to use that).

@srandom Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

I am not an admin. I am an end-user.
This is not something I intend to follow-up. I was just hoping to inform Microsoft so this is the end of the matter as far as I'm concerned.
Looking at screenshots online, you seem to be using the American practice of using parenthesis on geographic areas when in fact when formatting international numbers, you're supposed to only using them around the initial zero or country code.:
https://howtophoneto.com/glossary/brackets/


Add Mongo DB check

It would be useful to have a library of commonly used health checks. In the scope of this issue, a health check should be implemented that checks if Mongo DB is alive (e.g. by pinging Mongo DB and see if the ping result is successful).
A few constraints to keep things clean and simple:

The check should be added to the checks module that you can find in the development branch.
As with all health check functions of this library, the check implementation should support accepting and forwarding context.Context that is being passed to each health check function.
If the health check requires external external modules or other dependencies, the check should be implemented as a separate Go module(similarly how it is done for the tests, that require dependencies to other packages as well). This is required to avoid that when people import the base library, they also need to import all dependencies of all provided checks.
The check should have a README.md in its root directory that describes how to use it and maybe what to look out for.
You should provide a test that checks at least the happy path. If a docker container is required to properly test the check, the docker container can be added to the docker-compose.yaml file in the checks base package.

Should you have questions or require help, please join our discord chat room.

hello @alexliesenfeld can you assign me this issue?

@amarps Sure!


[crash] Threaded application segfaults due to global_engine_lock being NULL

I use openssl-1.1.1g with mysql 5.7.30 client (mysql's standard CLI application) and additional GOST engine for OpenSSL.
The command mysql segfaults at start when it calls SSL_load_error_strings() that is a wrapper for OPENSSL_init_ssl(). The problem does not manifest for non-threaded applications like openssl CLI application itself that works just fine with used external GOST engine. The problem does not occur if no external engines are configured within openssl.cnf. Also, the problem is specific for OpenSSL version 1.1.1 as it does not occur for OpenSSL 1.0.2 that has GOST engine built in.
The crash occurs in the engine_table_register() internal function of OpenSSL that calls CRYPTO_THREAD_write_lock(global_engine_lock) while it is still NULL (not initialized yet).
Here is the backtrace:
Reading symbols from /usr/local/bin/mysql...
[New LWP 100085]
Core was generated by `mysql'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x000000080202c136 in ?? () from /lib/libthr.so.3
(gdb) bt
#0  0x000000080202c136 in ?? () from /lib/libthr.so.3
#1  0x0000000803650dc8 in CRYPTO_THREAD_write_lock (lock=0x0)
    at crypto/threads_pthread.c:78
#2  0x00000008035abcce in engine_table_register (
    table=0x803951328 <cipher_table>,
    cleanup=0x8035ad0a0 <engine_unregister_all_ciphers>, e=0x802e303c0,
    nids=0x80325bd80 <gost_cipher_nids>, num_nids=12, setdefault=0)
    at crypto/engine/eng_table.c:88
#3  0x00000008035ad05c in ENGINE_register_ciphers (e=0x802e303c0)
    at crypto/engine/tb_cipher.c:30
#4  0x000000080300f760 in bind_gost (e=0x802e303c0, id=<optimized out>)
    at /home/ports/security/gost-engine/work/engine-7ed64cd/gost_eng.c:269
#5  bind_engine (e=0x802e303c0, id=<optimized out>, fns=0x7fffffffdd98)
    at /home/ports/security/gost-engine/work/engine-7ed64cd/gost_eng.c:312
#6  0x000000000069bcde in dynamic_load (e=0x802e303c0, ctx=0x802e1c420)
    at crypto/engine/eng_dyn.c:480
#7  0x000000000069b6a2 in dynamic_ctrl (e=0x802e303c0, cmd=206, i=0, p=0x0,
    f=0x0) at crypto/engine/eng_dyn.c:336
#8  0x00000000006fcf66 in ENGINE_ctrl (e=0x802e303c0, cmd=206, i=0, p=0x0,
    f=0x0) at crypto/engine/eng_ctrl.c:174
#9  0x00000000006fd6b5 in ENGINE_ctrl_cmd_string (e=0x802e303c0,
    cmd_name=0x705632 "LOAD", arg=0x0, cmd_optional=0)
    at crypto/engine/eng_ctrl.c:289
#10 0x00000000006fc964 in int_engine_configure (name=0x802e201a0 "gost",
    value=0x802e90600 "gost_section", cnf=0x802e7be00)
    at crypto/engine/eng_cnf.c:90
#11 0x00000000006fc6a6 in int_engine_module_init (md=0x802e28210,
    cnf=0x802e7be00) at crypto/engine/eng_cnf.c:171
#12 0x00000000004f2f6d in module_init (pmod=0x802e280f0,
    name=0x802e20190 "engines", value=0x802e90520 "engine_section",
    cnf=0x802e7be00) at crypto/conf/conf_mod.c:312
#13 0x00000000004f206f in module_run (cnf=0x802e7be00,
    name=0x802e20190 "engines", value=0x802e90520 "engine_section", flags=50)
    at crypto/conf/conf_mod.c:171
#14 0x00000000004f1ee7 in CONF_modules_load (cnf=0x802e7be00, appname=0x0,
    flags=50) at crypto/conf/conf_mod.c:102
#15 0x00000000004f2230 in CONF_modules_load_file (filename=0x0, appname=0x0,
    flags=50) at crypto/conf/conf_mod.c:138
#16 0x000000000064ec5f in openssl_config_int (settings=0x0)
    at crypto/conf/conf_sap.c:69
#17 0x0000000000553ad4 in ossl_init_config () at crypto/init.c:293
#18 0x0000000000553669 in ossl_init_config_ossl_ () at crypto/init.c:291
#19 0x000000080202c5b8 in pthread_once () from /lib/libthr.so.3
#20 0x00000000005ad6ed in CRYPTO_THREAD_run_once (once=0xd2b0a8 <config>,
    init=0x553660 <ossl_init_config_ossl_>) at crypto/threads_pthread.c:118
#21 0x00000000005530ca in OPENSSL_init_crypto (opts=2097230, settings=0x0)
    at crypto/init.c:701
#22 0x0000000000498b5e in OPENSSL_init_ssl (opts=2097230, settings=0x0)
    at ssl/ssl_init.c:205
#23 0x0000000000464e23 in ssl_start ()
    at /home/ports/databases/mysql57-client/work/mysql-5.7.30/vio/viosslfactories.c:442
#24 0x00000000004373fc in mysql_server_init (argc=0,
    argv=0xd22940 <embedded_server_args>,
    groups=0xca0f00 <embedded_server_groups>)
    at /home/ports/databases/mysql57-client/work/mysql-5.7.30/libmysql/libmysql.c:132
#25 0x0000000000413e1e in main (argc=2, argv=0x802e210e0)
    at /home/ports/databases/mysql57-client/work/mysql-5.7.30/client/mysql.cc:1317

The operating system is FreeBSD 11.4-RELEASE/amd64, if this matters.

What version of the GOST engine do you use?
Could you also check whether the engine is loaded once or loaded/unloaded/loaded again?

I use gost-engine built from FreeBSD port: https://svnweb.freebsd.org/ports/head/security/gost-engine/
This is Github's revision 7ed64cd of gost-engine master branch (2019-04-21). Modern revisions of master branch do not support OpenSSL 1.1.1 requiring OpenSSL 3.0 and I could not build gost-engine's branch openssl_1_1_0 because its cmake's configuration lacks "install" targed needed for FreeBSD port.

Could you also check whether the engine is loaded once or loaded/unloaded/loaded again?

I doubt OPENSSL_init_ssl() does it. MySQL source code line is here:
https://github.com/mysql/mysql-server/blob/5.7/vio/viosslfactories.c#L444

Hmmm... It does not look like a GOST-specific problem, but maybe a general one.

That's why I submitted the problem here and not for gost-engine.

The crash occurs in the engine_table_register() internal function of OpenSSL that calls CRYPTO_THREAD_write_lock(global_engine_lock) while it is still NULL (not initialized yet).

Hmm. The thing that makes no sense to me about this stack trace is this line:
#8  0x00000000006fcf66 in ENGINE_ctrl (e=0x802e303c0, cmd=206, i=0, p=0x0,
    f=0x0) at crypto/engine/eng_ctrl.c:174

So to get to the crashing line we've already passed trhough ENGINE_ctrl, and the beginning of that function is defined like this:
https://github.com/openssl/openssl/blob/2d9f56e9992ef3725b87a0a8e6165a18d038b784/crypto/engine/eng_ctrl.c#L125-L134
So, in other words, by the time we get to the crashing line, we've already acquired and released global_engine_lock at least once - so how can it be NULL? AFAICT there is no where in the code that ever explicitly sets it to NULL.

Is it possible to put this in a debugger to see if we can figure out where in the process global_engine_lock becomes NULL?

Nice catch. Something very fish is happening here:
#8  0x00000000006fcf66 in ENGINE_ctrl (e=0x802e303c0, cmd=206, i=0, p=0x0, f=0x0)
at crypto/engine/eng_ctrl.c:174
174         return e->ctrl(e, cmd, i, p, f);
(gdb) p &global_engine_lock
$1 = (CRYPTO_RWLOCK **) 0xd2c288 <global_engine_lock>
(gdb) p global_engine_lock
$2 = (CRYPTO_RWLOCK *) 0x802e20040
(gdb) frame 1
#1  0x0000000803650dc8 in CRYPTO_THREAD_write_lock (lock=0x0) at crypto/threads_pthread.c:78
78          if (pthread_rwlock_wrlock(lock) != 0)
(gdb) p &global_engine_lock
$3 = (CRYPTO_RWLOCK **) 0x803954778 <global_engine_lock>
(gdb) p global_engine_lock
$4 = (CRYPTO_RWLOCK *) 0x0

# ldd `which mysql`
/usr/local/bin/mysql:
        libedit.so.0 => /usr/local/lib/libedit.so.0 (0x800e93000)
        libz.so.6 => /lib/libz.so.6 (0x8010ca000)
        libm.so.5 => /lib/libm.so.5 (0x8012e3000)
        librt.so.1 => /usr/lib/librt.so.1 (0x801513000)
        libexecinfo.so.1 => /usr/lib/libexecinfo.so.1 (0x801719000)
        libc++.so.1 => /usr/lib/libc++.so.1 (0x80191c000)
        libcxxrt.so.1 => /lib/libcxxrt.so.1 (0x801be6000)
        libgcc_s.so.1 => /lib/libgcc_s.so.1 (0x801e05000)
        libthr.so.3 => /lib/libthr.so.3 (0x80201d000)
        libc.so.7 => /lib/libc.so.7 (0x802245000)
        libncurses.so.8 => /lib/libncurses.so.8 (0x8025fc000)
        libelf.so.2 => /lib/libelf.so.2 (0x802851000)
# nm /usr/local/bin/mysql | grep global_engine_lock
0000000000d2c288 B global_engine_lock

It seems, mysql binary contains OpenSSL code statically compiled in, so nm(1) finds global_engine_lock symbol name. Maybe dynamic loading of libgost.so somehow results in creation of second copy of OpenSSL library, so gdb shows two different symbols global_engine_lock, one initialized and another not initialized?

I've just found that mysql CLI application build process links it with OpenSSL libraries statically. So, the problem provided to be MySQL-specific and not in OpenSSL.

#3  0x00000008035ad05c in ENGINE_register_ciphers (e=0x802e303c0)
    at crypto/engine/tb_cipher.c:30
#4  0x000000080300f760 in bind_gost (e=0x802e303c0, id=<optimized out>)
    at /home/ports/security/gost-engine/work/engine-7ed64cd/gost_eng.c:269
#5  bind_engine (e=0x802e303c0, id=<optimized out>, fns=0x7fffffffdd98)
    at /home/ports/security/gost-engine/work/engine-7ed64cd/gost_eng.c:312
#6  0x000000000069bcde in dynamic_load (e=0x802e303c0, ctx=0x802e1c420)
    at crypto/engine/eng_dyn.c:480

In frame 6 we are presumably in the "dynamic_load" function statically linked against the mysql binary. In frames 4 and 5 we're inside code in the GOST engine. And then we can see in frame 3 that the GOST engine itself has made an OpenSSL call - ENGINE_register_ciphers. Since the GOST engine is dynamically linked its picking up an entirely different instance of OpenSSL and this is where the problem start - because we pass into that function an "ENGINE" instance that was not created from that version of OpenSSL.
Unfortunately I have no idea what the solution to this is. Maybe @levitte has some ideas.

I suspect right solution is to force MySQL link with OpenSSL dynamically instead of using static linking. AFAIK, this is already done for MySQL 8.0.x branch but 5.7.x still links it statically.
I'm testing it now.

Yes, I've fixed it locally backporting small change to ssl.cmake from MySQL 8.0.20 to 5.7.30.
I guess this issue may be closed. Sorry for noise and thank you for help.

I guess this issue may be closed.

Perfect. Thanks.

Sorry for noise and thank you for help.

Not noise at all. This issue maybe helpful for someone else that comes across the same problem.


feat: add unit tests

Summary 📌

Closes #164
This PR focuses on setting up unit testing and initial test cases for the ArkMarket project.

Changes Made ⚡
1. Testing Environment Setup

Configured Vitest as the testing framework, adding it to the project dependencies.
Created the initial configuration in vite.config.ts to support testing with jsdom environment and @testing-library/react for rendering components.
Added scripts in the package.json to run the tests, including:

test: Runs all tests.
test:watch: Runs tests in watch mode.
test:coverage: Runs tests and generates code coverage reports.

2. Unit Tests

Created a directory __tests__ inside the src folder to house test files.
Added unit tests for key components:

copy-button.test.tsx: Tests the functionality of the Copy Button component, including rendering and simulating a button click.
authentication.test.tsx: Verifies the rendering of the Authentication component, including wallet connection, mobile view image, and ArkMarket branding.
auth-switcher.test.tsx: Ensures the AuthSwitcher component toggles between login states.
buy-now-dialog.test.tsx: Verifies the functionality of the Buy Now Dialog component.
external-link.test.tsx: Tests that external links open correctly.

3. Test Coverage 🥷🏼

Added basic unit test cases for the components mentioned above to ensure rendering and user interaction logic work as expected.
Integrated coverage reporting to track and improve test coverage for components across the project:

Configured coverage reports to be generated in text, html, and json formats for better insights.

4. Test Results ✅

The initial test suite has been executed successfully, with 5 unit tests added to cover some components.

5. Pending 🔑

I will add more tests for the pending components, I do not know if for all of them but at least 10 more or let me know your preference 🫡

@coxmars looks good! there is just an issue with updated packages/ui tsconfig, I'll merge when this is fixed.

@coxmars looks good! there is just an issue with updated packages/ui tsconfig, I'll merge when this is fixed.

Thanks for the feedback! I noticed the issue with the packages/ui tsconfig. Also, I'd like to add a few more tests for additional components before the merge (today).
Let me know if that works for you! 🫡

@coxmars existing react tests are sufficient for now as the goal for now is to have test setup and later decide what to tests. You can try to add unit tests for lib functions like ownerOrShortAddress.ts.
Also, removingpackages/ui tsconfig changes fixes CI lint and typecheck issues.
Lmk if you have issues or need other informations 🚀 .

Hi @gershon, unit test for ownerorShortAddress.ts applied, restored packages/ui tsconfig file and merge applied, I think now it looks good, if you need something else just tell me. Thanks 🫡


Payload path name with UNIX sockets

Resolve #204

I'm just noting that this reverses the naming of the files...
What was /pid_remoteip_remoteport:localip_localport.out
Is now       /pid_localip_localport:remoteip_remoteport.out

Did we also change the CLI to accommodate?


Main hapi 234 rename provider code/name to hdx_provider_stub/name

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

Comparison is base (6896896) 93.94% compared to head (dca4229) 93.94%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #51   +/-   ##
=======================================
  Coverage   93.94%   93.94%
=======================================
  Files          90       90
  Lines        1849     1849
=======================================
  Hits         1737     1737
  Misses        112      112

Files
Coverage Δ

hdx_hapi/db/dao/dataset_view_dao.py
84.61% <100.00%> (ø)

hdx_hapi/db/dao/operational_presence_view_dao.py
93.18% <100.00%> (ø)

hdx_hapi/db/dao/population_view_dao.py
94.73% <100.00%> (ø)

hdx_hapi/db/dao/resource_view_dao.py
90.62% <100.00%> (ø)

hdx_hapi/db/models/db_dataset.py
100.00% <100.00%> (ø)

hdx_hapi/db/models/db_resource.py
100.00% <100.00%> (ø)

hdx_hapi/db/models/views/db_dataset_view.py
100.00% <100.00%> (ø)

...pi/db/models/views/db_operational_presence_view.py
100.00% <100.00%> (ø)

hdx_hapi/db/models/views/db_population_view.py
100.00% <100.00%> (ø)

hdx_hapi/db/models/views/db_resource_view.py
100.00% <100.00%> (ø)

... and 11 more

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


Support NLog.Config API

Currently, Windsor logging facility for NLog only supports configure by configuration file.
NLog finally supports configuration, https://github.com/nlog/NLog/wiki/Configuration-API. I hope Windsor should support, so we can do:
container.AddFacility<LoggingFacility>(facility => facility.UseNLog(NLogConfigInstance));

@davidshen84 it would be great if you could provide a pull request introducing this support, you'll likely also need to extend the Castle.Core NLog integration.

@jonorossi
I tried to add NLog 4.4.1 Nuget dependency to Castle.Facilities.Logging project, but I got the following errors, when I try to build from VS and using the build.cmd tool.
Severity	Code	Description	Project	File	Line	Suppression State
Error	CS0012	The type 'CLSCompliantAttribute' is defined in an assembly that is not referenced. You must add a reference to assembly 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'.	Castle.Facilities.Logging(netstandard1.6)	C:\Users\david\github\Castle\Windsor\src\Castle.Facilities.Logging\CSC	1	Active

Severity	Code	Description	Project	File	Line	Suppression State
Error	CS3001	Argument type 'LoggingConfiguration' is not CLS-compliant	Castle.Facilities.Logging(netstandard1.6)	C:\Users\david\github\Castle\Windsor\src\Castle.Facilities.Logging\LoggingFacility.cs	484	Active

But the Castle.Services.Logging.NLogIntegration project in the Core solution has the same dependency, and it build successfully on my machine.
Can I get some help?
Secondly, I found in the NLogFactory.cs in the Castle.Services.Logging.NLogIntegration project already has a factory overwrite that accepts LoggingConfiguration.
public NLogFactory(LoggingConfiguration loggingConfiguration)
So I think I do not need to modify the Core.

I'm of the opinion that that is the wrong direction for Windsor. While log injection is incredibly useful it is trivial to with it up yourself with either a sub resolver or regular property injection. By creating logging abstractions you end up missing out on a lot of the power of using a specific logging framework (e.g. serilog).

@davidshen84 you shouldn't need to modify Windsor for this work since we've deprecated the built-in helpers for log4net and NLog, Windsor only cares about an ILoggerFactory and an ILogger now.
@jonnii agreed, however Windsor and other Castle projects need somewhere to log. Now that we've simplified things in Windsor around logging set up I think most of the pain should disappear.
I'm going to close this issue now that UseNLog is deprecated, you should be able to do something like:
container.AddFacility<LoggingFacility>(f => f.LogUsing(new NLogFactory(loggingConfiguration)));


Refactor parsing of function in CodeInput using ast

The problem with inspect is that ignores the actual formatting provided by the user and that it sometimes stores directly the related objects (e.g. type annotation x: int stores the type int instead of the string int which makes it hard to pass as string later on). With ast this should be resolved. See example
def mock_function_1(x: int, y: int):
    """
    This is an example function.
    It adds two numbers.
    """
    if x > 0:
        return x + y
    else:
        return y

import ast, inspect
module = ast.parse(inspect.getsource(mock_function_1))
ast.dump(module)

#Out: "Module(body=[FunctionDef(name='mock_function_1', args=arguments(posonlyargs=[], args=[arg(arg='x', annotation=Name(id='int', ctx=Load())), arg(arg='y', annotation=Name(id='int', ctx=Load()))], kwonlyargs=[], kw_defaults=[], defaults=[]), body=[Expr(value=Constant(value='\\n    This is an example function.\\n    It adds two numbers.\\n    ')), If(test=Compare(left=Name(id='x', ctx=Load()), ops=[Gt()], comparators=[Constant(value=0)]), body=[Return(value=BinOp(left=Name(id='x', ctx=Load()), op=Add(), right=Name(id='y', ctx=Load())))], orelse=[Return(value=Name(id='y', ctx=Load()))])], decorator_list=[])], type_ignores=[])"

📚 Documentation preview 📚: https://scicode-widgets--86.org.readthedocs.build/en/86/

Sadly the ast module by design ignores comment which makes it not applicable for this approach

For reference this is how get_function_body could have been changed, but we cannot take this approach since it removes comments that are essential for the function body
    @staticmethod
    def get_function_body(function: types.FunctionType) -> str:
        function_source, function_definition = CodeInput._get_function_source_and_def(
            function
        )

        ast_functon_body = function_definition.body[-1]

        source_function_body = ast.get_source_segment(
            function_source, ast_functon_body, padded=True
        )
        breakpoint()
        if source_function_body is None:
            raise ValueError("Could not extract function body.")
        source_function_body = textwrap.dedent(source_function_body)
        return source_function_body

We still need to decide the formatting for the parsing to a for the parameters but it definitely improved


Left align table headers

closes #1246

Super elegant fix, @TraGicCode !

Codecov Report

Merging #1257 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1257   +/-   ##
======================================
  Coverage    93.6%   93.6%
======================================
  Files          60      60
  Lines        1329    1329
======================================
  Hits         1244    1244
  Misses         85      85

Continue to review full report at Codecov.

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

Sure here you go

Looks good, thanks @TraGicCode !


rename metrics

Two200_OK_rate
Can not start with numbers or contain whitespace

Closed in https://github.com/bjartnes/bounded-disturbances/commit/da7604f59e2a2f1a71f2cf880252d37f40507250


remove statement: TiSpark provides HTAP solution

What problem does this PR solve?
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change
Has persistent data change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to update the tidb-ansible repository
Need to be included in the release note

/run-tests

/run-all-tests tidb=release-6.1 tikv=release-6.1 pd=release-6.1

/run-all-tests tidb=release-6.1 tikv=release-6.1 pd=release-6.1

/run-all-tests tidb=release-6.1 tikv=release-6.1 pd=release-6.1

/merge


boto3 + moto + s3 AttributeError

it appears that boto2 and boto3 have different methods for s3 at least so using @mock_s3 with boto3 library causes errors such as AttributeError: 's3.ServiceResource' object has no attribute 'delete_object'
delete_object() is from boto3, the equivalent in boto2 is delete_key()
is there a different version of moto that I need to use?

@atharvai try this:
import boto3
s3 = boto3.resource('s3')
s3.Object("bucketname", "hello.txt").delete()

If you are still having errors, can you post some code to recreate the error?


Evaluate the use of a general purpose Image

As capi images will not be mandatory on SCS-clouds, capo can not rely on the existence of them. Instead, one flavor on SCS is actually mandatory (Ubuntu 22.04). That means, that every capo can rely on the existence of this image. The task of this issue is to evaluate how much of an effort it is to use such a general purpose image and use it as the base-images for cluster-api workload-cluster nodes.

I rather wonder why Ubuntu 22.04 is manadory when CAPI, for example, is not. What if I am a CSP who has a partnership with RedHat or SUSE and don't want to offer a Canonical product on my cloud? I think it would be better to discuss why a single image of Canonical is mandatory and whether it should not also be marked as non-mandatory.

This issue is about dealing with a general purpose image whose availability is guaranteed by the standard (currently Ubuntu 22.04). A better place to debate which images should be mandatory is here. Anyways, i would welcome Debian as (commercial) vendor-neutral image :)

    preKubeadmCommands:
      - sysctl -w net.ipv4.ip_forward=1
      - curl -fsSL https://pkgs.k8s.io/core:/stable:/v1.30/deb/Release.key | gpg --dearmor -o /etc/apt/keyrings/kubernetes-apt-keyring.gpg
      - curl -fsSL https://pkgs.k8s.io/addons:/cri-o:/prerelease:/main/deb/Release.key | gpg --dearmor -o /etc/apt/keyrings/cri-o-apt-keyring.gpg
      - echo "deb [signed-by=/etc/apt/keyrings/kubernetes-apt-keyring.gpg] https://pkgs.k8s.io/core:/stable:/v1.30/deb/ /" | tee /etc/apt/sources.list.d/kubernetes.list
      - echo "deb [signed-by=/etc/apt/keyrings/cri-o-apt-keyring.gpg] https://pkgs.k8s.io/addons:/cri-o:/prerelease:/main/deb/ /" | tee /etc/apt/sources.list.d/cri-o.list
      - apt-get update -y
      - apt-get install -y  cri-o kubelet kubeadm
      - modprobe br_netfilter
      - echo "br_netfilter" >> /etc/modules
      - systemctl enable crio --now

seems to be enough to use Ubuntu 22.04 out of the box, i did not make any time-measurements, but it does not feel to provision slower than the CAPI images.

sysctl -w net.ipv4.ip_forward=1

I don't know if it's needed, but as you persisted all other commands on the system, let me point out, that this command does not persist the sysctl setting but only changes it for the currently running kernel.

Good point, thanks for highlighting it :)


Saved tenant endpoints

Category: (New feature)
When users log in, their previously selected tenant is stored locally. I created a new feature that stores (internal users') last used tenant(saved_tenant) in persisted storage.

Github Issue # or road-map entry, if available:
N/A

Description of changes:
Created two new routes under the ACCOUNT endpoint that allows (internal) users to GET and PUT their saved_tenant. Modified InternalUserV7 to accommodate these changes (added saved_tenant parameter, along with a getter, setter, and modification to toString(). (Internal) user saved_tenant information is persisted on the user configuration on the security index. This does increase server-side latency, but the additional latency is in the scale of around 10 ms per action, which is negligible in the scale of typical human reaction time (~250 ms). (Statistics of measured additional latency can be found in the "Impact" section of the Quip document.

Why these changes are required?
When connected to the frontend, this new feature will allow (internal) users to have a seamless experience automatically loading into their previously used tenant across browsers and devices.

__What is the old behavior before changes and new behavior after changes?
Before:
When switching tenants, users' last selected tenant is stored locally. As a result, users may need to manually change their current selected tenant across browsers and devices.
E.g. Bob logs into a private tenant in Google Chrome. Bob decides to log into OpenSearch on Firefox, and is sent to the default tenant. Bob needs to manually switch his tenant on Firefox in order to user the tenant he was using on Google Chrome.

After:
Tenants are stored for (internal) users, so (internal) users will have a seamless process switching between browsers and devices on the same cluster.
E.g. Joe logs into a private tenant in Google Chrome. Bob decides to log into OpenSearch on Firefox, and is sent to the same tenant. Joe does not need to manually switch his tenant across browsers/devices.

Testing done: (Please provide details of testing done: Unit testing, integration testing and manual testing)
Unit testing (added to existing fileAccountApiTest.java)

TO-DOs, if any: (Please describe pending items and provide Github issues# for each of them)

Modify frontend connections to allow users to GET and PUT their saved_tenant via Kibana UI
Expanding functionality to accommodate GET/PUT saved_tenant for external users

Is it backport from main branch? (If yes, please add backport PR # and commits #)
N/A

By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here. 1. Category: (New feature)
When users log in, their previously selected tenant is stored locally. I created a new feature that stores (internal users') last used tenant(saved_tenant) in persisted storage.

Github Issue # or road-map entry, if available:
N/A

Description of changes:
Created two new routes under the ACCOUNT endpoint that allows (internal) users to GET and PUT their saved_tenant. Modified InternalUserV7 to accommodate these changes (added saved_tenant parameter, along with a getter, setter, and modification to toString(). (Internal) user saved_tenant information is persisted on the user configuration on the security index.

Why these changes are required?
When connected to the frontend, this new feature will allow (internal) users to have a seamless experience automatically loading into their previously used tenant across browsers and devices.

__What is the old behavior before changes and new behavior after changes?
Previously (before), when switching tenants, users' saved tenant is stored locally. As a result, users may need to manually change their current selected tenant across browsers and devices. (E.g. Bob logs into a private tenant in Google Chrome. Bob decides to log into Opensearch on Firefox, and is sent to the default tenant. Bob needs to manually switch his tenant on Firefox in order to user the tenant he was using on Google Chrome.)
Now (after), tenants are stored for (internal) users, so (internal) users will have a seamless process switching between browsers and devices on the same cluster.
E.g. Joe logs into a private tenant in Google Chrome. Bob decides to log into Opensearch on Firefox, and is sent to the same tenant. Joe does not need to manually switch his tenant across browsers/devices.

Testing done: (Please provide details of testing done: Unit testing, integration testing and manual testing)
Unit testing (performed in a new file SavedTenantApiTest.java

TO-DOs, if any: (Please describe pending items and provide Github issues# for each of them)
N/A

Is it backport from main branch? (If yes, please add backport PR # and commits #)
N/A

By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Could you please fix CI?

https://github.com/opensearch-project/security/pull/1390#discussion_r683839989
@cliu123
This suggestion is inconsistent with existing tests, e.g. https://github.com/opensearch-project/security/blob/main/src/test/java/org/opensearch/security/dlic/rest/api/AccountApiTest.java#L54

Codecov Report

Merging #1390 (75a86a2) into main (2b69d9c) will increase coverage by 0.04%.
The diff coverage is 83.78%.

@@             Coverage Diff              @@
##               main    #1390      +/-   ##
============================================
+ Coverage     64.73%   64.77%   +0.04%
- Complexity     3193     3210      +17
============================================
  Files           247      248       +1
  Lines         17230    17279      +49
  Branches       3045     3058      +13
============================================
+ Hits          11153    11192      +39
- Misses         4527     4535       +8
- Partials       1550     1552       +2

Impacted Files
Coverage Δ

...arch/security/dlic/rest/api/AbstractApiAction.java
76.89% <ø> (+0.98%)
:arrow_up:

.../security/securityconf/impl/v7/InternalUserV7.java
84.31% <ø> (+1.33%)
:arrow_up:

...earch/security/dlic/rest/api/AccountApiAction.java
80.23% <81.25%> (-2.13%)
:arrow_down:

...y/dlic/rest/validation/InternalUsersValidator.java
100.00% <100.00%> (ø)

...ity/dlic/rest/validation/SavedTenantValidator.java
100.00% <100.00%> (ø)

...org/opensearch/security/rest/TenantInfoAction.java
77.94% <0.00%> (-10.30%)
:arrow_down:

...security/configuration/DlsFlsFilterLeafReader.java
59.76% <0.00%> (-0.71%)
:arrow_down:

...a/org/opensearch/security/tools/SecurityAdmin.java
47.52% <0.00%> (+0.26%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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

enant is kibana feature. Should we store saved_tenant into centralized index instead associating it with internal user.?  We should not tie it to internal user and store it to security_index… this will block us or add lot of pain for migration when we will  extend this feature for non internal user (like SAML or LDAP)...  Storing saved_tenant into centralized index (possibly .kibana or .session_info etc... ) will help to extend this feature for non internal user and same index can be used in future for storing cookie on server side..

I don't think that security plugin should use .kibana as it may conflict with the Dashboards/Kibana own functionality (may require additional POC to see how conflict can be avoided). Note that it is also not a session option, it is longer term user preference options.

(this force-push is a reversion to last commit on August 12th)

@benjaminxu251 I know there has been a lot of discussion on this issue, and its taken a considerable effort - thank you, unfortunately, this discussions have stalled out, so I am closing this issue.
If you would like to see this feature in OpenSearch, please open a feature proposal and we can work together to figure the best way to support this functionality.


Cannot run example on android:  Unable to resolve module react-native-snap-carousel

Is this a bug report, a feature request, or a question?
Question
Have you followed the required steps before opening a bug report?

[x] I have read the guidelines regarding bug report.
[x] I have reviewed the documentation in its entirety, including the dedicated documentations :books:.
[x] I have searched for existing issues and made sure that the problem hasn't already been reported.
[x] I am using the latest plugin version.
[x] I am following the issue template closely in order to produce a useful bug report.

Have you made sure that it wasn't a React Native bug?
yes
Is the bug specific to iOS or Android? Or can it be reproduced on both platforms?
Android
Is the bug reproductible in a production environment (not a debug one)?
In an emulator, I didn't went that far to tell it's production
Environment
Emulator
Expected Behavior
I expect to run the examples on android
Actual Behavior
Right after cloning the master branch, running npm run android produce the following
dka@dka:[~/workspace/github.com/archriss/react-native-snap-carousel/example (master)]: npm run start

><EMAIL_ADDRESS>start /home/dka/workspace/github.com/archriss/react-native-snap-carousel/example
> react-native start

┌──────────────────────────────────────────────────────────────────────────────┐
│                                                                              │
│  Running Metro Bundler on port 8081.                                         │
│                                                                              │
│  Keep Metro running while developing on any JS projects. Feel free to        │
│  close this tab and run your own Metro instance if you prefer.               │
│                                                                              │
│  https://github.com/facebook/react-native                                    │
│                                                                              │
└──────────────────────────────────────────────────────────────────────────────┘

Looking for JS files in
   /home/dka/workspace/github.com/archriss/react-native-snap-carousel/example

Loading dependency graph, done.

To reload the app press "r"
To open developer menu press "d"

info Reloading app...
error: bundling failed: Error: Unable to resolve module `react-native-snap-carousel` from `src/index.js`: react-native-snap-carousel could not be found within the project.

If you are sure the module exists, try these steps:
 1. Clear watchman watches: watchman watch-del-all
 2. Delete node_modules: rm -rf node_modules and run yarn install
 3. Reset Metro's cache: yarn start --reset-cache
 4. Remove the cache: rm -rf /tmp/metro-*
    at ModuleResolver.resolveDependency (/home/dka/workspace/github.com/archriss/react-native-snap-carousel/example/node_modules/metro/src/node-haste/DependencyGraph/ModuleResolution.js:186:15)
    at ResolutionRequest.resolveDependency (/home/dka/workspace/github.com/archriss/react-native-snap-carousel/example/node_modules/metro/src/node-haste/DependencyGraph/ResolutionRequest.js:52:18)
    at DependencyGraph.resolveDependency (/home/dka/workspace/github.com/archriss/react-native-snap-carousel/example/node_modules/metro/src/node-haste/DependencyGraph.js:282:16)
    at Object.resolve (/home/dka/workspace/github.com/archriss/react-native-snap-carousel/example/node_modules/metro/src/lib/transformHelpers.js:267:42)
    at /home/dka/workspace/github.com/archriss/react-native-snap-carousel/example/node_modules/metro/src/DeltaBundler/traverseDependencies.js:426:31
    at Array.map (<anonymous>)
    at resolveDependencies (/home/dka/workspace/github.com/archriss/react-native-snap-carousel/example/node_modules/metro/src/DeltaBundler/traverseDependencies.js:423:18)
    at /home/dka/workspace/github.com/archriss/react-native-snap-carousel/example/node_modules/metro/src/DeltaBundler/traverseDependencies.js:275:33
    at Generator.next (<anonymous>)
    at asyncGeneratorStep (/home/dka/workspace/github.com/archriss/react-native-snap-carousel/example/node_modules/metro/src/DeltaBundler/traverseDependencies.js:87:24)
 BUNDLE  [android, dev] ./index.js ▓▓░░░░░░░░░░░░░░ 16.0% (2/94)::ffff:<IP_ADDRESS> - - [07/Apr/2020:17:23:51 +0000] "GET /index.bundle?platform=android&dev=true&minify=false HTTP/1.1" 500 - "-" "okhttp/3.12.1"
 BUNDLE  [android, dev] ./index.js ▓▓░░░░░░░░░░░░░░ 16.0% (2/94), failed.

Reproducible Demo
The examples  directory of this repository
Steps to Reproduce
git clone https://github.com/archriss/react-native-snap-carousel.git
cd react-native-snap-carousel
npm i
cd example
npm i
npm run start
# press r

add  react-native-snap-carousel

It seems to work with yarn instead of npm (I can reproduce the issue with npm). Don't know the reason yet
I'm interested to investigate further and will update if I have more information.

any solution with npm yet?

yarn add react-native-snap-carousel @types/react-native-snap-carousel
two packages, both

I have the same issue can someone helppppp

Thanks

TypeError: Cannot read properties of undefined (reading 'hookChildProcess') in react native:
Running jetifier to migrate libraries to AndroidX. You can disable it using "--no-jetifier" flag. error Failed to run jetifier. TypeError: Cannot read properties of undefined (reading 'hookChildProcess') at module.exports (/Volumes/Os/ReactNativePROJECT/Educational/node_modules/react-native/node_modules/execa/index.js:308:22) at Object.runAndroid [as func] (/Volumes/Os/ReactNativePROJECT/Educational/node_modules/react-native/node_modules/@react-native-community/cli-platform-android/build/commands/runAndroid/index.js:104:34) at Command.handleAction (/Volumes/Os/ReactNativePROJECT/Educational/node_modules/@react-native-community/cli/build/index.js:192:23) at Command.listener (/Volumes/Os/ReactNativePROJECT/Educational/node_modules/commander/index.js:315:8) at Command.emit (node:events:527:28) at Command.parseArgs (/Volumes/Os/ReactNativePROJECT/Educational/node_modules/commander/index.js:651:12) at Command.parse (/Volumes/Os/ReactNativePROJECT/Educational/node_modules/commander/index.js:474:21) at setupAndRun (/Volumes/Os/ReactNativePROJECT/Educational/node_modules/@react-native-community/cli/build/index.js:271:24) at run (/Volumes/Os/ReactNativePROJECT/Educational/node_modules/@react-native-community/cli/build/index.js:212:11) at Object. (/Volumes/Os/ReactNativePROJECT/Educational/node_modules/@react-native-community/cli/build/bin.js:8:11) info Run CLI with --verbose flag for more details.

yarn add react-native-snap-carousel @types/react-native-snap-carousel
two packages, both

Its work for me thanks for your help.


Add docker compose setup for local development, demo

Signed-off-by: darshanime<EMAIL_ADDRESS>Closes https://github.com/thanos-io/thanos/issues/2249
Add a docker-compose setup for local dev environment, demos etc.

[ ] I added CHANGELOG entry for this change.
[ ] I added documentation for this new flow
[X] Change is not relevant to the end user.

@bwplotka the PR is not yet done, but wanted some early feedback on the idea

I agree this seems to be best done in a separate project.

Happy to set up the project for https://github.com/thanos-community/ if you want.

sure, that sounds good @bwplotka 👍

I mean, this also mean we need longer term maintainer for  such project. Also my worry is that such project will be super small.
Maybe something like deployments would do better :thinking:

I think having a sample docker-compose.yml would be of great help. Maybe we can add the example to the documentation.

As disussed in community call, this is not required currently.
Can I close this?


Skip the email step for autocomplete-python

This change removes the email step from the workflow. Instead the Copilot workflow is used at the end.
Changes:

Removes step to create an account
New copy text and screenshot in the screen displayed while downloading kite
kite is launched with a new flag to force onboarding in the copilot

How to test
We have to make kite-installer, kite-api, and autocomplete-python make use this. This can be achieved with the npm link and apm link commands.

Make sure that you check out the branch copilot-workflow for each of kite-connect-js and kite-installer!
Make sure to uninstall the kite and autocomplete-python plugins from Atom

cd ~/kite-connect-js
git checkout copilot-workflow
npm link

cd ~/kite-api-js
npm link kite-connector

cd ~/kite-installer
git checkout copilot-workflow
npm link kite-api-js
npm link kite-connector

cd ~/autocomplete-python
npm link kite-api-js
npm link kite-connector
npm link kite-installer
apm link

After these steps you can launch Atom, it should use the local sources of autocomplete-python.
If it's not showing up, remove the references to autocomplete-python in the config at Edit->Config... (in Atom). Right after saving this change the kite wizard should pop up in a new tab.

Codecov Report

Merging #80 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #80   +/-   ##
======================================
  Coverage    90.9%   90.9%
======================================
  Files          18      18
  Lines         286     286
======================================
  Hits          260     260
  Misses         26      26

Continue to review full report at Codecov.

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


"No signal" on shutdown after recent update.

I couldn't find any other information that works.
After updating to 3.1.9 from 3.1.5 shutting down my PC no longer turns off the TV, the TV remains on displaying "no signal".
I have verified the network information is correct and that the TV has a static IP.
Here is the log after shutting down and booting the PC.
https://zerobin.net/?9316758317b4f7d3#xTaGtkzrQA/1+TNFhEFNvRy8ZlA0wiJ0kX1yF7QcKSo=
Config:
https://zerobin.net/?eb9262ed51eccf37#txgnjCBhfKU480/FTRCjTPZDMlLefQC9LKvreHlP/0c=
Hopefully I didn't miss something and this isn't a duplicate, please let me know if you'd like any more information.
Thanks :)

Hi @soma811 there is no change per se between 3.1.5 and 3.1.9 that should cause the issue.
However, shutting down the TV on system shutdown is a race to communicate with the TV before it's impossible to do so. Changing the method to "early" from "auto" in the shutdown section in general options may hopefully resolve your issue. If this does not help please paste a log here and I will have a look.

Hello, I tried setting timing to early to no avail. TV still displays "no signal" on shutdown.
Log:
https://zerobin.net/?c90527465e26853a#MHHr8nohZgXigyTY8FVZEv0dDukE4/fxF69TAdISze0=
Config:
https://zerobin.net/?0f170da9ad1b41ff#j1Gd5PzRq8PB4DetOsEWSxNp/4YSfomNk3Tve8jHVTg=
Thanks! :)

Hi thanks, I noticed nothing out of the ordinarie in the log but see this in the config:
"LocalEventLogRestartString": [
"restart",
"shutdown",
"power off"
],
It seems you may have checked a few checkboxes too many in the "shutdown" configuration. Try unchecking "shutdown" and "power off". Hope that helps

Awesome, this worked, thank you so much! Didn't realize that checking all of them could cause it not to work.


Notesbooks can be tested with the built package of karabo

Currently, when testing the notebooks on the installed package, the pytest is killed. This happens also locally for me. I don't know why.
Example on a GitHub runner: https://github.com/i4Ds/Karabo-Pipeline/actions/runs/5858833188/job/15884912205
The notebooks are tested nightly on the GitHub checkout, so that should be ok.
Thus, the notebooks are not tested on the user installation (package).

@kenfus no logs available anymore. However, I think this issue is the same as #531 I think? If yes, please close this issue. If it's not the same, I think you should elaborate more on this issue.


Allow multiple Tags on the same commit

Hi,
currently we build a new release version with every build that Jenkins does. This causes the problem that we might want to do another build that starts with a different configuration. But this second build is built on the same commit.
The versioning plugin then refuses to do that because it can not create a tag since the commit is already tagged.
Is it possible to allow this? Since the plugin should be able to resolve which tag is the latest on one commit it should be able to determine a proper version.

What would the second tag look like? Are you just looking to change the prefix on the tag?

Closing due to lack of information.

Hey, we also need this to be able to have different build artifacts from the same working tree (say e.g. a debug build and a production build).
A suffix would be enough for us (e.g. 1.10.2-1)
Would be happy about re-opening!

@master-lincoln The tag identifies a particular version of the code. If you want to build and release multiple artifacts, couldn't you just add a classifier to the jar task for each of those artifacts? I guess I'm having a hard time understanding the value of having a tag 1.10.2 and 1.10.2-debug pointing to the exact same commit. 1.10.2 applies to the entire codebase as a whole, and it's saying that the particular commit identifies an explicit version of the code. Even if there are multiple artifacts corresponding to that version, all of those artifacts must have been generated from that particular tag. Hence the second tag is superfluous because it is attached to the exact same commit.
You shouldn't rely on the version number to identify these multiple artifacts. Because let's say you have artifact-1.0.0.jar and artifact-1.0.0-source.jar, all it means is that both were generated from version 1.0.0 of the code. So if you wanted to check it out or rebuild them, you would be checking out the 1.0.0 tag regardless of whether you wanted to build artifact-1.0.0.jar or artifact-1.0.0-source.jar.
I think if you look at how artifacts are stored in maven repositories, for example, it works this way well -- you have a single version number but can have multiple artifacts with different classifiers (e.g., a "source" classifier if you want to package the source as well).

Thank you for this elaborate answer. You are right, I lost the big picture. As this versioning plugin is only concerned about versioning the code base, we should either make the configuration part of the code base or append classifiers manually to the version when building different configurations.


Backtracking Tabs

When hitting settings button on main page, brings you backwards to whatever page you were on before hitting it (either create or login page). Crashed once upon spamming this, not again.
Not sure where the problem lies, going to keep testing breakpoints.
Any help would be appreciated, I can stream code for this, do not want to push in-case it's a deeper issue.
Maybe time to make a bug report branch, so we can push only for broken versions, and pull that separately.

Bug squashed. ImageButton and Button needed to be swapped for the declarations.


What causes the spinner to display each time I view one of my contracts?

Isn't this data already local? Could it be served up instantly? Seems like it takes a few seconds to display each one of contracts every time I view it and I wonder if this is necessary.

Pulling data from the local server isn't instant.

Could caching help? The experience just feels a bit off. I first see a giant spinner that overtakes the entire screen and then the smaller spinner displays and it takes a few seconds to render.
Clicking followers or following on my page loads instantly, wondering if it could work the same for contracts.

The followers, following, about and store all load when the page loads, since each of those is a single call to the server. Individual contracts don't load until someone clicks on one.
If a store had a lot of contracts, loading all of them when the page loads could take a while, since each one is a separate call to the server, especially if you're looking at someone else's store.

I would agree with Mike, I feel like spinners should only show if it's going to take a while. 1 or 2 seconds to load something doesn't seem like the place to use a spinner.

How do you tell how long it's going to take before you show the spinner?
We could delay showing the spinner, but it would seem weird to show nothing, wait, and then show the spinner.

I think a spinner should still display if it's fetching content, but seeing two different types of spinners and having to wait to load something that I looked at a few seconds ago seems like there is some room for optimization.

Status @morebrownies ?

Just gonna say this if fixed.


keypair issue-Terraform

Error: Error launching source instance: InvalidKeyPair.NotFound: The key pair 'terraform' does not exist
status code: 400, request id: 39c1fbd2-ff42-42c1-9500-63ba50c28f21
on webserver.tf line 1, in resource "aws_instance" "appserver1":
1: resource "aws_instance" "appserver1" {
Error: Error launching source instance: InvalidKeyPair.NotFound: The key pair 'terraform' does not exist
status code: 400, request id: 8eeb2889-cf76-46a9-936a-a862d4dec5c4
on webserver.tf line 15, in resource "aws_instance" "webserver1":
15: resource "aws_instance" "webserver1" {

Please ensure that AWS Key-Pair has been generated for the respective Region of your usage..
and try once again..
Ramkoti


[IDE] Two more preparation commits for solver-based cursor info

[IDE] Make isDynamicRef take a getType function (@bnbarham)
This allows isDynamicRef to work with types from a constraint system solution

[IDE] Move getSelectedOverloadInfo to be a function on Solution (@xedin)
We will need this for solver-based cursor info.

[Sema] Add a solution callback that is called when the constraint system produces a solution (@xedin)
When doing solver-based cursor info, we’ll type check the expression in question using a normal typeCheckASTNodeAtLoc call (not in code completion mode) and listen for any solutions that were produced during the type check.

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test


[deliver] iOS screenshot uploader: wrong order

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
If 10 screenshots are uploaded and the naming of the screenshots is [1...10], then the 10th screenshot will be ordered to the front and show up as first screenshot.
This issue has been reported before, but automatically closed instead of fixed: https://github.com/fastlane/fastlane/issues/14785
Command executed

Complete output when running fastlane, including the stack trace and command used

Environment

summary>🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.15.2

Ruby
2.6.4

Bundler?
false

Git
git version 2.21.0 (Apple Git-122)

Installation Source
~/.rbenv/versions/2.6.4/bin/fastlane

Host
Mac OS X 10.15.2 (19C57)

Ruby Lib Dir
~/.rbenv/versions/2.6.4/lib

OpenSSL Version
OpenSSL 1.0.2s  28 May 2019

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
11.0

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.140.0
🚫 Update available

Loaded fastlane plugins:

Plugin
Version
Update-Status

fastlane-plugin-bugsnag
1.4.1
✅ Up-To-Date

fastlane-plugin-badge
1.3.0
🚫 Update available

Loaded gems

Gem
Version

did_you_mean
1.3.0

slack-notifier
2.3.2

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

terminal-table
1.8.0

multipart-post
2.0.0

word_wrap
1.0.0

public_suffix
2.0.5

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

excon
0.71.1

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
0.13.1

gh_inspector
1.1.3

mini_magick
4.9.5

multi_xml
0.6.0

rubyzip
1.3.0

security
0.1.3

xcpretty-travis-formatter
1.0.0

bundler
1.17.2

jwt
2.1.0

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

mini_mime
1.0.2

signet
0.12.0

googleauth
0.10.0

httpclient
2.8.3

google-api-client
0.36.4

nanaimo
0.2.6

colored2
3.1.2

claide
1.0.3

CFPropertyList
3.0.2

atomos
0.1.3

xcodeproj
1.14.0

unicode-display_width
1.6.0

plist
3.5.0

addressable
2.7.0

tty-screen
0.7.0

tty-cursor
0.7.0

tty-spinner
0.9.2

babosa
1.0.3

faraday
0.17.3

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

fastimage
2.1.7

json
2.3.0

dotenv
2.7.5

naturally
2.2.0

simctl
1.6.7

retriable
3.1.2

multi_json
1.14.1

memoist
0.16.2

os
1.0.1

digest-crc
0.4.1

google-cloud-env
1.3.0

google-cloud-core
1.4.1

google-cloud-storage
1.25.1

emoji_regex
1.0.1

forwardable
1.2.0

logger
1.3.0

stringio
0.0.2

ipaddr
1.2.2

openssl
2.1.2

ostruct
0.1.0

strscan
1.0.0

date
2.0.0

fileutils
1.1.0

etc
1.0.1

io-console
0.4.7

zlib
1.0.0

rexml
3.1.9

psych
3.1.0

mutex_m
0.1.0

xml-simple
1.1.5

git
1.5.0

fastlane-plugin-bugsnag
1.4.1

curb
0.9.10

badge
0.11.0

fastlane-plugin-badge
1.3.0

generated on: 2020-01-31

This bug has not been fixed.

Still not fixed.

Still not fixed.

I don't think this has been fixed.

Your need to use leading zeros.
01,...09,10,11 etc

You need to use leading zeros.
01,...09,10,11 etc

I know what the workaround is, but the main problem is that the sorting is done lexicographically instead of numerically.
As it seems screenshot names always start of with numbers, sorting lexicographically is nonsensical.

AFAIK it is not fixed.

I'm considering working on this (no promises!), but I have a few questions. @janpio, in https://github.com/fastlane/fastlane/issues/14785#issuecomment-503469720 you mentioned that the order of uploads mattered, however uploads can be multithreaded, can an order even be guaranteed then? Or is Apple sorting the screenshots by filename on their side? I'm not sure which of the moving parts is responsible. I guess for single-threaded uploads, guarantees can be made.
https://github.com/fastlane/fastlane/blob/5e03f6aed7ab9abb620b67107146bc615ee3184d/deliver/lib/deliver/upload_screenshots.rb#L118-L127
At work we use https://github.com/public-law/naturally for a similar problem, in case padding screenshots with a leading 0 is not an option.

This is a fair request, imo. Although the solution probably won't be that easy to achieve 😅 unless we add a dependency such as the one suggested by @aried3r.
A quick test shows the behavior:

This is how ruby naturally sorts things: if it's a number, sort them numerically. If it's a string, sort it alphabetically (and 1 comes before 2, in "10" vs "2")
What is even more complicated is that if we decide to change the sorting behavior, this change probably needs to be backwards compatible, because some people might be relying on the existing behavior. So an option to adopt "the new sorting algorithm" will have to be created, likely.
Adding a dependency just for this case doesn't seem to be in question because we aim to keep fastlane with as few dependencies as possible. However, the gem suggested above (https://github.com/public-law/naturally) seems to be so small that we could consider it? @joshdholtz thoughts on this? 😁
By the way @aried3r the relevant code you're looking for is here - nothing related to the order of upload 😊
https://github.com/fastlane/fastlane/blob/5e03f6aed7ab9abb620b67107146bc615ee3184d/deliver/lib/deliver/upload_screenshots.rb#L218-L228
I'm open to hear other suggestions, alternatives...

@fastlane-bot Not fixed.
@rogerluan I get your point regarding backwards compatibility, but to me this feels more like a bug than an actual intended behavior, but I think it's also fine to add an additional flag to sort naturally.
@joshdholtz What's your take on this?

Worth adding a dependency or should this be implemented in fastlane itself?
Is preserving the current behavior important, e.g. by adding an sorting-method param?

This is a good point! I have never thought of this...
@EmDee
Can't you deal with this issue by prefixing/suffixing starting from 0? (or you can use whatever works; e.g. alphabets )

irb(main):005:0> [*0...10].map {|x| "6.5_#{x}.jpg" }.sort
=> ["6.5_0.jpg", "6.5_1.jpg", "6.5_2.jpg", "6.5_3.jpg", "6.5_4.jpg", "6.5_5.jpg", "6.5_6.jpg", "6.5_7.jpg", "6.5_8.jpg", "6.5_9.jpg"]

I know this is not human friendly nor what you want. However, for those reasons @rogerluan mentioned for the backwards compatibility, this is a fair workaround until the point where we can make breaking changes. No changes are required but you can easily fix the issue on your ends. It's definitely worth documenting on its page though for people having the same issue.

cc: @aried3r JFYI: Uploads using multiple threads does break the order but deliver also re-order screenshots after uploading is done here. And this is where the origin issue causes app_screenshots.sort_by(&:file_name)😂
https://github.com/fastlane/fastlane/blob/e89cdcfb869ad0640438cc5f6ed261bb50887641/deliver/lib/deliver/upload_screenshots.rb#L233-L250

@EmDee
Can't you deal with this issue by prefixing/suffixing starting from 0? (or you can use whatever works; e.g. alphabets )

@ainame Yes, that’s been my workaround ever since.

@fastlane-bot Not yet fixed.

@fastlane-bot Not yet fixed.

Hey @EmDee I just opened a PR that fixes this issue, based on our discussions 😊 let me know if that works for you.
https://github.com/fastlane/fastlane/pull/18200

The order was wrong on 2.180.1:
[11:35:43]: Uploaded './screenshots/iMessage/fi/1-Finnish-iPhone 8.png'... (3.451629 secs)
[11:35:43]: Uploaded './screenshots/iMessage/fi/2-Finnish-iPhone 8.png'... (3.359283 secs)
[11:35:44]: Uploaded './screenshots/iMessage/fi/3-Finnish-iPhone 8.png'... (3.142242 secs)
[11:35:45]: Uploaded './screenshots/iMessage/fi/6-How to-iPhone 8.png'... (2.889228 secs)
[11:35:45]: Uploaded './screenshots/iMessage/fi/4-Finnish-iPhone 8.png'... (3.07833 secs)
[11:35:45]: Uploaded './screenshots/iMessage/fi/5-Finnish-iPhone 8.png'... (3.091414 secs)
[11:35:45]: Uploaded './screenshots/iMessage/fi/7-How to-iPhone 8.png'... (3.07154 secs)


🏍 Support Vehicles

Challenges

Look direction isn't shared with the server while on a vehicle. no longer using look direction, anyway 😉

maybe use C (wheelie/jump) to go up and X (honk) to go down? Not sure if I can intercept these

Increasing elevator multi-dim size doesn't seem to increase the range that BuffsWhenWalkedOn activates

Closing issue as part of cleanup; this was determine to more or less be impossible from server-side due to limitations discovered in how the game transfers info through a network while in a vehicle.


does not work in 1.16.5

tried other mods, when using this one though, it crashes and says errors occurred within the seedcracking mod, any thoughts? or is it just not compatible?


Proposal for readability fixes

Fixes #27

Max-width for content
Adjustments of line height and spacing
Making block quotes distinct
Heading colors & spacing

Love it!  Thanks!


Fix issues with SARTSATraces

This fixes #70 and fixes #71.
Issue #70 : SARTSA Traces cannot be sampled correctly.
The sampleable_inds field of the episodes buffer containing a SARTSATrace (CircularArraySARTSATraces or ElasticArraySARTSATraces) was not correctly keeping track of which indices are sampleable. In a SARTSATraces, information from 3 steps are required to complete a single trace. The initial step of the episode in which only the state is pushed. The next step in which the action and next_state are pushed. The following step in which the next_action is pushed. Now, there is one sampleable index, but two unsampleable ones. Thus, the last two indices in the trace are typically unsampleable during the episode. This change was made to the function Base.push!(eb::EpisodesBuffer, xs::NamedTuple) in episodes.jl.
Issue #71 : Action and state go out of sync in CircularArraySARTSATraces
When more traces are pushed into the Traces than its capacity, the state does not match the appropriate action. To fix this, the capacity of the state trace should be one more than that of the action trace. The capacity of all traces are incremented by 1 so that the Traces can hold capacity amount of full traces.

The tests were also modified to check that it works correctly with the following usage:

Push first state at the start of the episode (PreEpisodeStage in RL.jl)
Push state, action, reward, terminal during the episode (PostActStage)
Push action after the episode ends in a PartialNamedTuple (PostEpisodeStage)

This is the behaviour of the agent from RLCore.jl and all trajectories should work well with that. The CircularArraySARTSATraces are a bit restrictive with regards to usage and cannot be used very differently than the typical usage above. I have not been able to make it work while represented next_action and working with all general usage.
The CircularArrarySARTSATraces do not currently work with CircularPrioritizedTraces as these add additional keys like keys and priorities which are updated outside of the Traces. Some changes need to be made with CircularPrioritizedTraces to make it work with CircularArrarySARTSATraces also. Thus, there are 2 errors during the testing involving CircularPrioritizedTraces.

@dharux Thank you for the PR. I've updated the main branch to eliminate unrelated test failures. Would you able to resolve the remaining test errors (in CircularPrioritizedTraces)?

I could take a shot at it. It might require completely rewriting how tuples are pushed into a CircularPrioritizedTraces and how it is sampled for the case when it contains a CircularArraySARTSATraces.

@HenriDeh any tips or pointers for @dharux

@dharux Thanks, looks good to me, I’ll let @HenriDeh do a final check.

Needed some final fixes. Everything should be working now!


Dependencies not resolved by maven?

After compiling Gaffer, we needed to manually add the following files to the java project:
accumulo-core-1.7.1.jar
commons-collections-3.2.2.jar
commons-configuration-1.10.jar
commons-io-2.5.jar
commons-lang-2.6.jar
commons-logging-1.2.jar
guava-19.0.jar
hadoop-auth-2.2.0.jar
hadoop-common-2.2.0.jar
jackson-annotations-2.1.2.jar
jackson-core-2.7.4.jar
jackson-databind-2.2.3.jar
libthrift-0.9.1.jar
slf4j-nop-1.7.21.jar
If it's normal (and not due to our problem!) it could be a good idea to write it somewhere in the wiki (better with links to the various projects).

Hi @unclassified01,
Thanks for raising the issue. There are a few different jars produced from the build for different usages.

If you are simply running the example then there is a maven profile ‘exampleJar’, which produces an example jar with all dependencies.

If you are deploying the jars onto Accumulo for use by the iterators then you need to use the following jars produced by the default maven build:

accumulo-store-iterators-[version].jar

simple-serialisation-[version]-library-serialisers.jar

simple-function-[version]-library-serialisers.jar

We will get the wiki updated with this extra information. Hope that helps, if not then let us know which jar has the missing dependencies and we will look into it.

Closing as we have updated the wiki with the additional information. Please let us know if you have any other questions.


Title disappears on experimental button

Hello,
The new on/off icons is cool! Is it just me that the title vanishes when the icon toggles?
Off:

On:

Domain: Light
Service: Toggle
Title cleared, custom set to friendly name.

(Default Settings act the same)
Icons used are from material design: Bulb-Outline & Bulb-on.

Hi,
i can reproduce this issue. It seems like StreamDeck ignores the "setTitle" command, when the icon state is set to 1 (on). Debugging output shows, that my plugin sets the title in both cases:
Switching off:

Switching on:

I will take a closer look, but I think this issue may be "unfixable" (from plugin side of view).

Hmm, second look:

Suddenly, a suspicious "state: 0" appears in setTitle call, when state should be 1...

Hi @cgiesche - Can confirm this is fixed in 1.5.1 🙏🏼


fix(app-tools): add changeset for fixing app-tools exported types

Bug
Fix app-tools exported types.

[x] Related issues linked using fixes #433

Example
Try "@modern-js/app-tools": "0.0.0-canary-1641112447939",.

Tried "@modern-js/app-tools": "0.0.0-canary-1641112447939", but the following error message present in VS Code:
Parsing error: "parserOptions.project" has been set for @typescript-eslint/parser.
The file does not match your project config: modern.config.ts.
The file must be included in at least one of the projects provided.eslint

Meanwhile, despite pnpm run dev will run, the the following modern.config.ts config was not effective. Maybe related to #364 :
import { defineConfig } from '@modern-js/app-tools';
import ArcoWebpackPlugin  from '@arco-design/webpack-plugin';

export default defineConfig({
    tools: {
      webpack: {
        plugins: [
            new ArcoWebpackPlugin({
              theme: '@arco-themes/react-choral-music',
            }),
          ],
      }
    }
 })

@jiahao-c
Add modern.config.ts to tsconfig.json should fix the error.
{
  "include": ["src", "modern.config.ts"]
}

Currently, webpack can only be modified with webpack-chain. See #364


fix: reset cursor to eof on append

Description
When the same file handler is being used, once a read happens, the cursor needs to be set to the end of file before appending again to avoid override.

Got merged mid-review 😂  Would have approved anyway though. Thanks!

Thank you @phughk, will keep note of the PR for version bump


Use environment package

As pointed out by @joefitzgerald there is a package avialable that provides a unified environment on OS X that matches what you would expect from process.env on other platforms. Since most packages use this to run their linters we should probably implement this here to simplify things for the OS X users.
Thoughts?

Asked him in #packages of slack, see https://atomio.slack.com/archives/packages/p1448052728000048

Bringing that conversation over here, since there is a limit to that history:
@steelbrain:

Ping @joefitzgerald
[12:52] I am interested in using your environment package in linter providers, but I have a question
[12:53] For almost all the linter providers, the module that actually spawns the process is atom-linter, it's an npm module that these apm modules depend on. The fact that you are exporting a service instead of setting a global limits it's usage in atom-linter, to use that all the apm packages need to consume that service and then pass it on to atom-linter::exec, it could've been simplified if you were, say setting a global, it would be accessible everywhere and would simplify the implementation

@joefitzgerald:

maybe you should use an executor service
[1:22] that depends on environment

@Arcanemagus Thank you for bringing this up, I had almost forgotten about it :)
@joefitzgerald I would love to create a service for that, but we would then have to publish this package on apm instead of npm and all packages would have to use it via atom-package-deps instead of directly. Also it goes against the concept of packages IMO, 'cause the user will see it in their list and will be given options to uninstall it

@steelbrain this package has been proposed to fix this issue in a few of the linter-* packages by @lexicalunit: https://www.npmjs.com/package/fix-path
Maybe we could use it if we can't work around the service requirement?

I am very much in favor of that module you linked to, but the fact that it alters a global when it could return a value would mean that when it's used by multiple packages, it would add duplicate entries. Other than that, it's using hard-coded values opposed to @joefitzgerald's package that finds out the paths by actually executing it, which is a better way IMO

As discussed over in https://github.com/AtomLinter/linter-shellcheck/pull/35, this really is something that should be fixed Atom side in the end, although we should probably work around this shortcoming temporarily till it lands on stable. It looks like the issue to follow for getting this into Atom: https://github.com/atom/atom/issues/4126

I've created a new consistent-env package that should make things easier for us, it's results are isolated and it's an npm package. It also has internal caching as opposed to fix-path by sindre. Thanks for the idea @lexicalunit


My Resource page revamp

Can we have this deployed somewhere?

Can one of the admins verify this patch?

+1: good work

I just pushed some work that fixes the toolbar buttons on the resource landing page not showing up for discoverable resources.

@mauriel133 Have updated beta with your latest push. Check it out and see if the issue has been fixed.

@hyi or @alvacouch Can one of you review the latest changes pushed by @mauriel133 then either me or @mauriel133 can merge this. I need this merged before I can start working on #795

@mauriel133 @pkdash +1 for the latest push regarding toolbar button fixes on resource landing page. You can merge this as needed.


Exclude HC config items from the manifest payload

https://issues.redhat.com/browse/CMCS-99
Signed-off-by: Philip Wu<EMAIL_ADDRESS>

@jnpacker Can you have a look at this?
Not sure if this is the direction we want to go. Other ways we could handle this is:

A label in the configuration item that would exclude it from the manifest payload. The resource on the HC would have this label but maybe we could remove this label when we process it in the hypD controller.
Maybe configuration items generally do not need to be on the hosting cluster? So we don't need to include them in the manifest payload. It is of type []runtime.RawExtension, unlike Secrets and ConfigMaps.


2024 Fall Cohort Process - Call 3 (Sep 4)

To do list

[x]  2+ weeks prior to call: confirm & coordinate logistics with guest speaker; schedule prep call if needed
[x] 1 week prior to call: Finalize lesson plan/slides
[x] 1 week prior to call: Email reminder/confirmation to guest speakers (if any)
[x] Add team members to GitHub Team; confirm notification acceptance (Kyber + Openscapes Staff)
[ ] By Fri after call: Thank you email to guest speaker
[x] By Fri after call: Initiate guest speaker honorarium
[x] By Fri after call: Download, format, post call recording
[x] #51 @vlzimmer
[ ] By Tues following call: Review digest @AHolder1 @tures23
[ ] By  Wed following call: Send digest @vlzimmer

Ok all - I finished a draft of the digest!

AWESOME! @tures23 can you please take the first round of review and tag me back in when you're done? I won't be able to take a look until Fri. Thankssssss!

wonderful digest @vlzimmer  ! I love the pathway reminder at the end :) @AHolder1 please review if you haven't already. thank you!!

The digest looks great @vlzimmer ! Send awayyy!
Some reminders:

you may need to download the images and then upload them (simple copy paste might not work)
Be sure to @ the cohort's team (2024-swrcb-fall-cohort) at the top of the issue so that folks get an email when the digest is posted.


Systematic organization of map legend sources

I see a lot of photos in this repo's issues and PRs that are of the form "Here's a picture of Utah's official state map legend" (or Cincinnati, or wherever).
Is it worth organizing them into some massive chart (or data structure from whence multiple charts can be derived) such that it's easy to see what each stroke or fill style is empirically used for? This would be a reference for asserting that "80% of states use a pink line to mean a motorway."

Many of these photos come from my map collection. I’ve posted most of these legends to Google Drive, but I’m reluctant to formally make them into part of this repository’s codebase, since most are still under copyright.

While our use of pictures of bits of maps for scholarly analysis almost certainly qualifies as fair use, I'm similarly uncomfortable with putting pieces of copyrighted maps into the repository code base and calling it public domain.  Creating some sort of archive of these clips for cartographic analysis would certainly be useful but it would need to exist somewhere outside of this github repo (and for the same reason, outside of the OSM wiki).

I am fine with this outcome, but I did want to clarify, RE

I'm similarly uncomfortable with putting pieces of copyrighted maps into the repository code base and calling it public domain.

I did not expect to put photos in the data source. Not even necessarily an SVG. Something that would let one build a chart like this:

Shape
Fill
Border
Zoom
Most used as
Most %
Next Most
Next %

Line
3px pink
1px red
12
motorway
68%
primary
12%

Rounded Square
5px white
2px black
12
junction
83 %
city 4
7%

Say from data like this:
{
    "sources": [
        {
            "id": 12345,
            "name": "City of ____ DOT",
            "url": null,  // maybe?
            "zoom": 12,
        },
    ],
    "features": [
        {
            "source": 12345,
            "name_source": "primary highway",
            "name_osm": "motorway",
            "shape": {
                "type": "line",
                "description": "3x center with 1x casing",
            },
            "colors": {
                "fill": {
                    "original": "#RRGGBB",
                    "simple": "pink",  // Some simplified palette
                    "width": 3,
                },
                "outline": {
                    "original": "#RRGGBB",
                    "simple": "red",   // Some simplified palette
                    "width": 1,
                },
            },
        },
        {
            "source": 12345,
            "name_source": "junction",
            "name_osm": null,
            "shape": {
                "type": "rounded square",
                "description": "5x center with 2x border",
            },
            "colors": {
                "fill": {
                    "original": "#FFFFFF",
                    "simple": "white",  // Some simplified palette
                    "width": 5,
                },
                "outline": {
                    "original": "#040404",
                    "simple": "black",   // Some simplified palette
                    "width": 2,
                },
            },
        }
    ]
}


LAPS integration in WAC on a Windows  Server Gateway 2019

We have a particular use case.
We want to do our support of our Windows 10  machines with WAC and this in combination with LAPS.
This specific use case could help us to eliminate the administrator role for our support guys and gives us support guys a bunch of build in tools to help them with particular support tasks.
And in combination with LAPS we can use the local administrator to manage  the pc if need admin rights even without knowing the password for each computer.
When WAC installed on a Windows 10 machine(my machine)  we see a very nice integration with LAPS (see printscreen).
Laps is nicely integrated with WAC so if we connect this machine automatically  through “Manage as” we have some admin rights to do some support things and seamless admin connection with this local user (we do not need tho give the credentials)
For the obvious reasons we do not want to install the WAC on A Windows 10 machine but on a fully managed  Windows server Gateway (on OS windows 2019 ).
So we can manage this centrally for all our support guys.
What we are missing in the “manage as” ,is the fully integration with LAPS (see below  prtscreen on our sever )

Hey @Hermanvgys,
For LAPS to be an option for Windows Server 2019, your WAC gateway needs to be running on a Windows 10/11 PC.
If your WAC gateway is running on a Windows Server, then it will not be supported, and we currently do not have this on our roadmap to be addressed.

@Hermanvgys You should really have techs connecting with domain accounts, I believe all the msft security baselines recommend disabling remote access via the builtin local admin account - and also disabling it entirely, to be used only in a person-at-the-system boot to safemode scenario, which allows the builtin admin account to be accessed even if it's disabled; see https://docs.microsoft.com/en-us/windows-server/identity/ad-ds/plan/security-best-practices/implementing-least-privilege-administrative-models under the heading "Controls for Local Administrator Accounts".


[INTERNAL] Remove unused expected build results

Folders have been added by mistake and are not used by any test.

Coverage remained the same at 94.686% when pulling<PHONE_NUMBER>f3b116c921bb84b15172e20195ad23 on next-test-cleanup into af075edf784d9f1ba162a34f0bf150dbcbc0f479 on next.


Android does not work

I couldn't open pdf in android.
I used the pdf path 'assets/data/myFile.pdf'.
And the error message is : Relative URIs are not supported.
Ios runs well.
I am very confused.
Does anyone meet the same problem?
Thanks.

'assets/data/myFile.pdf' is not an absolute url. have a look at the sample app.
you could use window.resolveLocalFileSystemURL or prepend a vaild path from the cordova file constants (cordova.file.*).

Where is the sample app ??
"you could use window.resolveLocalFileSystemURL or prepend a vaild path from the cordova file constants (cordova.file.*)."
please explain this point.
Please help.

sample app is linked in the docs: https://github.com/sitewaerts/cordova-plugin-document-viewer#using-the-plugin
Sorry, I have no capacities on teaching you cordova basics.

Okay thank U . (y)

Correct path for android is 'file:///android_asset/www/assets/'

what is the path for ios ?

@ketchup208 I'm using this code for IOS                                                                                baseUrl = location.href.replace("/index.html", "");                                 this.document.viewDocument(baseUrl + '/assets/your-file.pdf', 'application/pdf', {title: 'your-file-title'});

Thank you, i will test it all and maybe submit a pr to change the docs.


[Usage]: how to use guided_decoding_backend?

Your current environment
I found that in EngineArgs, guided_decoding_backend can choose to support outlines or lm-format-enforcer. How can I use it?
Specifically, the following code can be executed by outlines
import outlines
model = outlines.models.transformers(model_path)
prompt = "fish is delicious"
unstruct_answer = outlines.generate.choice(model,["positive","negative"])(prompt)

How can I achieve this in vllm？
How would you like to use vllm
I want to run inference of a [specific model](put link here). I don't know how to integrate it with vllm.

Hi @estuday! You can check out this documentation.


SIngle stepping in disassembly is always step-in

I've noticed that there is no difference between step-in and step-over when stepping in the disassembly view. Looking at the GDB log, I see that VS is always issuing the "stepi" command in this case, where it should be using "nexti".
-joe porkka
https://visualstudiogallery.msdn.microsoft.com/35dbae07-8c1a-4f9d-94b7-bac16cad9c01/view/Discussions

Fixed #292


Custom Character Sheet Backgrounds

Would be cool to inject a "pick a custom background" into the background selector (just ask for a URL)

Just re-read.  I mean background image on the background image selector on the character sheet view

So I am not sure if @emfmesquita would rather this be apart of his extension or mine. But I did start working on this for the character sheet. I already have a preview for the theme selector, so you can pick any valid color and that will be your theme.
If you go to this post, you'll see a preview of the theme colors working. I am going to start on the background image next.
I do personally think that this fits better with the theme of beyondhelp more than it does D&D Beyond Toolbox. So if this is something you (@emfmesquita) would rather include in your extension, I can help implement it.


Reverted tabs to spaces

@cmpich your IDE is committing with tabs. I would be more than happy to switch this repo over to using tabs vs spaces, everyone else is using tabs.

Can we just switch your ENV to using spaces and let everyone else use tabs? as in that is what everyone is already doing?

I am used to always use 4 spaces per tab to make it always look the same.


Typo on Semantic Search documentation

for page https://docs.microsoft.com/en-us/azure/search/semantic-search-overview there is a typo:
"Semantic search is collection of features"
should be
"Semantic search is a collection of features"
It's right under the "What is semantic search?" headline

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

ID: a3b38bed-e5a0-2c9b-18cb-f16a7423c822
Version Independent ID: adc69a3d-c53a-6c7a-7b22-8f2bc08c8df3
Content: Semantic search - Azure Cognitive Search
Content Source: articles/search/semantic-search-overview.md
Service: cognitive-search
GitHub Login: @HeidiSteen
Microsoft Alias: heidist

Thanks for pointing this out @TheCodingCanal. We a PR pending to fix the issue.


Anyone can help me?

--2022-06-08 09:28:46--  https://github.com/rssnyder/discord-stock-ticker/releases/download/v2.0.0/discord-stock-ticker-v3.3.0-linux-amd64.tar.gz Resolving github.com (github.com)... <IP_ADDRESS> Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected. HTTP request sent, awaiting response... 404 Not Found 2022-06-08 09:28:46 ERROR 404: Not Found.

what is the command you are running that gives you this?


[AutoPR devspaces/resource-manager] DevSpaces new path

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6087

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/4191