python: more informative error

Checklist

[X] npm install && npm test passes
[X] tests are included
[X] commit message follows commit guidelines

Description of change
more informative error when failing to find a suitable python:
gyp info it worked if it ends with ok
gyp verb cli [ 'C:\\bin\\dev\\node\\node4.exe',
gyp verb cli   'D:\\code\\4node\\node-gyp\\bin\\node-gyp.js',
gyp verb cli   'configure',
gyp verb cli   '--loglevel=verbose' ]
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| win32 | x64
gyp verb command configure []
gyp verb check python checking for Python executable "python2" in the PATH
gyp verb `which` failed Error: not found: python2
gyp verb `which` failed     at getNotFoundError (D:\code\4node\node-gyp\node_modules\which\which.js:13:12)
gyp verb `which` failed     at F (D:\code\4node\node-gyp\node_modules\which\which.js:68:19)
gyp verb `which` failed     at E (D:\code\4node\node-gyp\node_modules\which\which.js:80:29)
gyp verb `which` failed     at D:\code\4node\node-gyp\node_modules\which\which.js:89:16
gyp verb `which` failed     at D:\code\4node\node-gyp\node_modules\isexe\index.js:42:5
gyp verb `which` failed     at D:\code\4node\node-gyp\node_modules\isexe\windows.js:36:5
gyp verb `which` failed     at FSReqWrap.oncomplete (fs.js:82:15)
gyp verb `which` failed  python2 { [Error: not found: python2]
gyp verb `which` failed   stack: 'Error: not found: python2\n    at getNotFoundError (D:\\code\\4node\\node-gyp\\node_modules\\which\\which.js:13:12)\n    at F (D:\\code\\4node\\node-gyp\\node_modules\\which\\which.js:68:19)\n    at E (D:\\code\\4node\\node-gyp\\node_modules\\which\\which.js:80:29)\n    at D:\\code\\4node\\node-gyp\\node_modules\\which\\which.js:89:16\n    at D:\\code\\4node\\node-gyp\\node_modules\\isexe\\index.js:42:5\n    at D:\\code\\4node\\node-gyp\\node_modules\\isexe\\windows.js:36:5\n    at FSReqWrap.oncomplete (fs.js:82:15)',
gyp verb `which` failed   code: 'ENOENT' }
gyp verb check python checking for Python executable "python" in the PATH
gyp verb `which` failed Error: not found: python
gyp verb `which` failed     at getNotFoundError (D:\code\4node\node-gyp\node_modules\which\which.js:13:12)
gyp verb `which` failed     at F (D:\code\4node\node-gyp\node_modules\which\which.js:68:19)
gyp verb `which` failed     at E (D:\code\4node\node-gyp\node_modules\which\which.js:80:29)
gyp verb `which` failed     at D:\code\4node\node-gyp\node_modules\which\which.js:89:16
gyp verb `which` failed     at D:\code\4node\node-gyp\node_modules\isexe\index.js:42:5
gyp verb `which` failed     at D:\code\4node\node-gyp\node_modules\isexe\windows.js:36:5
gyp verb `which` failed     at FSReqWrap.oncomplete (fs.js:82:15)
gyp verb `which` failed  python { [Error: not found: python]
gyp verb `which` failed   stack: 'Error: not found: python\n    at getNotFoundError (D:\\code\\4node\\node-gyp\\node_modules\\which\\which.js:13:12)\n    at F (D:\\code\\4node\\node-gyp\\node_modules\\which\\which.js:68:19)\n    at E (D:\\code\\4node\\node-gyp\\node_modules\\which\\which.js:80:29)\n    at D:\\code\\4node\\node-gyp\\node_modules\\which\\which.js:89:16\n    at D:\\code\\4node\\node-gyp\\node_modules\\isexe\\index.js:42:5\n    at D:\\code\\4node\\node-gyp\\node_modules\\isexe\\windows.js:36:5\n    at FSReqWrap.oncomplete (fs.js:82:15)',
gyp verb `which` failed   code: 'ENOENT' }
gyp verb could not find "python". checking python launcher
gyp verb could not find "python". guessing location
gyp verb ensuring that file exists: C:\Python27\python.exe
gyp ERR! configure error
gyp ERR!
******************************************************************
node-gyp can't use "python",
It is recommended that you install python 2.7, set the PYTHON env,
or use the --python switch to point to a Python >= v2.6.0 & < 3.0.0.
For more information consult the documentation at:
https://github.com/nodejs/node-gyp#installation
***********************************************************************
gyp ERR! not ok

CI: https://ci.nodejs.org/view/All/job/nodegyp-test-pull-request/33/

+1, I have similar issue, and  the error info is as following:
C:\Program Files (x86)\Microsoft Visual Studio 11.0>npm config set python "D:/Python27/python.exe"
C:\Program Files (x86)\Microsoft Visual Studio 11.0>node-gyp rebuild
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| win32 | x64
gyp ERR! configure error
gyp ERR! stack Error: Can't find Python executable "python", you can set the PYTHON env variable.
gyp ERR! stack     at PythonFinder.failNoPython (C:\Users\leon\AppData\Roaming\npm\node_modules\node-gyp\lib\configure.js:483:19)
gyp ERR! stack     at PythonFinder. (C:\Users\leon\AppData\Roaming\npm\node_modules\node-gyp\lib\configure.js:508:16)
gyp ERR! stack     at C:\Users\leon\AppData\Roaming\npm\node_modules\node-gyp\node_modules\graceful-fs\polyfills.js:284:29
gyp ERR! stack     at FSReqWrap.oncomplete (fs.js:152:21)
gyp ERR! System Windows_NT 10.0.15063
gyp ERR! command "D:\Program Files\nodejs\node.exe" "C:\Users\leon\AppData\Roaming\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Program Files (x86)\Microsoft Visual Studio 11.0
gyp ERR! node -v v8.0.0
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok

This should be definitely merged, the error message is vastly misleading. I just spent one hour figuring that I had the wrong python version installed.

Same

@bnoordhuis can someone please take a look at these python related PR's?

@refack I'm going to have to defer to you on the status of this and its suitability for merging, can you review and see if it's ready or not?

Landed in https://github.com/nodejs/node-gyp/commit/6b7c8e60c52e648d7384c25a5b10522dd5b590f9


UnboundLocalError: local variable 'SentenceID' referenced before assignment

In Vagrant virtual box:
David-Laxers-MacBook-Pro:eldiablo davidlaxer$ vagrant up
/Applications/Vagrant/bin/../embedded/gems/gems/vagrant-1.6.3/lib/vagrant/pre-rubygems.rb:31: warning: Insecure world writable dir /Developer/NVIDIA/CUDA-6.5/bin in PATH, mode 040777
/Applications/Vagrant/embedded/gems/gems/bundler-1.6.2/lib/bundler/runtime.rb:222: warning: Insecure world writable dir /Developer/NVIDIA/CUDA-6.5/bin in PATH, mode 040777
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Checking if box 'ubuntu/trusty64' is up to date...
==> default: A newer version of the box 'ubuntu/trusty64' is available! You currently
==> default: have version '14.04'. The latest is version '20150817.0.0'. Run
==> default: vagrant box update to update.
==> default: Clearing any previously set forwarded ports...
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
default: Adapter 1: nat
==> default: Forwarding ports...
default: 22 => 2222 (adapter 1)
==> default: Running 'pre-boot' VM customizations...
==> default: Booting VM...
==> default: Waiting for machine to boot. This may take a few minutes...
default: SSH address: <IP_ADDRESS>:2222
default: SSH username: vagrant
default: SSH auth method: private key
default: Warning: Connection timeout. Retrying...
default: Warning: Remote connection disconnect. Retrying...
default: Warning: Remote connection disconnect. Retrying...
default: Warning: Remote connection disconnect. Retrying...
==> default: Machine booted and ready!
==> default: Checking for guest additions in VM...
==> default: Mounting shared folders...
default: /vagrant => /Users/davidlaxer/eldiablo
==> default: Machine already provisioned. Run vagrant provision or use the --provision
==> default: to force provisioning. Provisioners marked to run always will still run.
David-Laxers-MacBook-Pro:eldiablo davidlaxer$ vagrant ssh
/Applications/Vagrant/bin/../embedded/gems/gems/vagrant-1.6.3/lib/vagrant/pre-rubygems.rb:31: warning: Insecure world writable dir /Developer/NVIDIA/CUDA-6.5/bin in PATH, mode 040777
/Applications/Vagrant/embedded/gems/gems/bundler-1.6.2/lib/bundler/runtime.rb:222: warning: Insecure world writable dir /Developer/NVIDIA/CUDA-6.5/bin in PATH, mode 040777
Welcome to Ubuntu 14.04.1 LTS (GNU/Linux 3.13.0-32-generic x86_64)

Documentation:  https://help.ubuntu.com/

System information disabled due to load higher than 1.0
Get cloud support with Ubuntu Advantage Cloud Guest:
http://www.ubuntu.com/business/services/cloud
197 packages can be updated.
102 updates are security updates.
 Done
vagrant@vagrant-ubuntu-trusty-64:~$ ls
phoenix_pipeline     scraper             src               stanford_pipeline
pipeline_stdout.log  scraper_stdout.log  stanford-corenlp  stanford_stdout.log
vagrant@vagrant-ubuntu-trusty-64:~$ ls ..
ubuntu  vagrant
vagrant@vagrant-ubuntu-trusty-64:~$ ls phoenix_pipeline/
counter.txt        oneaday_filter.pyc  README.md               timex.pyc
docs               PETRARCH.log        requirements.txt        uploader.py
formatter.py       PHOX_config.ini     result_formatter.py     uploader.pyc
formatter.pyc      PHOX_pipeline.log   result_formatter.pyc    utilities.py
geolocation.py pipeline.py         scraper_connection.py   utilities.pyc
geolocation.pyc    pipeline_sched.py   scraper_connection.pyc
LICENSE            postprocess.py      source_keys.txt
oneaday_filter.py  postprocess.pyc     timex.py
vagrant@vagrant-ubuntu-trusty-64:~$ cd ../
vagrant@vagrant-ubuntu-trusty-64:/home$ ls
ubuntu  vagrant
vagrant@vagrant-ubuntu-trusty-64:/home$ cd vagrant/phoenix_pipeline/
vagrant@vagrant-ubuntu-trusty-64:~/phoenix_pipeline$ sudo !!
sudo python pipeline.py
Requires eGenix.com mx Base Distribution
http://www.egenix.com/products/python/mxBase/
Found a config file in working directory
PHOX.pipeline run: 2015-08-21 14:01:24.814406
Date string: 20150820
Total number of stories: 8708
Running Mongo.formatter.py
Running PETRARCH and returning output.
new_actor_length = 0
stop_on_error = False
write_actor_root = False
write_actor_text = False
require_dyad = True
code-by-sentence True
pause_by_sentence False
pause_by_story False
Comma-delimited clause elimination:
Initial : deactivated
Internal: min = 2    max = 8
Terminal: min = 2    max = 8
Verb dictionary: CAMEO.verbpatterns.150430.txt
Actor dictionaries: [u'Phoenix.Countries.actors.txt', u'Phoenix.International.actors.txt', u'Phoenix.MilNonState.actors.txt']
Agent dictionary: Phoenix.agents.txt
Discard dictionary: Phoenix.discards.txt
Issues dictionary: Phoenix.IssueCoding.txt
NO INFO
Traceback (most recent call last):
File "pipeline.py", line 142, in
run()
File "pipeline.py", line 138, in run
run_filter=file_details.oneaday_filter, version='v0.0.0')
File "pipeline.py", line 97, in main
parsed=True)
File "/usr/local/lib/python2.7/dist-packages/petrarch/petrarch.py", line 2896, in run_pipeline
updated_events = do_coding(events, 'TEMP')
File "/usr/local/lib/python2.7/dist-packages/petrarch/petrarch.py", line 2677, in do_coding
'{} has no parse information. Passing.'.format(SentenceID))
UnboundLocalError: local variable 'SentenceID' referenced before assignment
vagrant@vagrant-ubuntu-trusty-64:~/phoenix_pipeline$ cd ..
vagra
NO INFO
Traceback (most recent call last):
File "pipeline.py", line 142, in
run()
File "pipeline.py", line 138, in run
run_filter=file_details.oneaday_filter, version='v0.0.0')
File "pipeline.py", line 97, in main
parsed=True)
File "/usr/local/lib/python2.7/dist-packages/petrarch/petrarch.py", line 2896, in run_pipeline
updated_events = do_coding(events, 'TEMP')
File "/usr/local/lib/python2.7/dist-packages/petrarch/petrarch.py", line 2677, in do_coding
'{} has no parse information. Passing.'.format(SentenceID))
UnboundLocalError: local variable 'SentenceID' referenced before assignment
edited petrarch.py: added - SentenceID = ""
for sent in val['sents']:
SentenceID = ""
if 'parsed' in event_dict[key]['sents'][sent]:
$ python pipeline.py
...
NO INFO
NO INFO
NO INFO
NO INFO
NO INFO
NO INFO
NO INFO
NO INFO
NO INFO
NO INFO
NO INFO
NO INFO
NO INFO
...
Stories made it into Mongodb:

db.stories.count()
22736

T UnboundLocalError running in the ‘eldiablo’ Virtual box.
The data that got into MongoDB was running native on OS X  not in ‘eldiabl’ (e.g. - > db.stories.count()
8434
)

On Aug 31, 2015, at 1:22 PM, John Beieler<EMAIL_ADDRESS>wrote:
Closed #93 https://github.com/openeventdata/petrarch/issues/93.
—
Reply to this email directly or view it on GitHub https://github.com/openeventdata/petrarch/issues/93#event-397075265.


Global suffixes

Add support for globally-defined suffixes, which will make it easier for common values to be reused across a number of CSS properties.  For example, in config you may specify something like:
{
  globals: {
    foo: '10px'
  }
}

...which would allow you to write classes like H-foo and P-foo, which would output:
.H-foo {
  height: 10px;
}
.P-foo {
  padding: 10px;
}

...without requiring you to write custom config for every class using the foo suffix.

This is related to issue #90...

:+1: nice looks even better thanP-$foo. Are we supporting both?

@src-code actually, as @3den mentions it should be:
.H-\$foo {
  height: 10px;
}
.P-\$foo {
  padding: 10px;
}

The reason for this is that it is important that authors recognize those classes as global variables, hence the addition of the $ character.

So as we discussed offline this morning, Atomizer won't need to offer special support for $ or any other namespace - it'll be treated the same as any other string value used to denote a global/custom suffix.  It'll be up to a project to define their naming conventions for globals.  We'll be sure to document a recommendation around namespacing and dealing with conflicts like this.
On a side note, what we will want to ensure is that Atomizer properly warns when merging configs that define the same customs/globals but with conflicting values.  This is a need unrelated to the use of a namespace for customs/globals.

Addressed in pull #108


[SQL] Support MySQL Prepare SQL

What would you like to be added:
Why is this needed:
https://dev.mysql.com/doc/refman/8.0/en/sql-prepared-statements.html

@林国威

https://github.com/arana-db/arana/discussions/700

全局会话；
session 级别；

贴一下链接： @dongzl


Always Hidden Mode Not Working

MacOS Catalina.
Hidden Bar 1.8(12)

Cannot access "always hidden" mode
Image in tutorial show which key to hit is broken.

yeh thats why I closed it haha..seems the updater didn't finish (close and restart).
Appreciate the reply, Thank you.

I ask because I can't get it to work in the current version

@gingerbeardman oh haha, somehow thought you were the dev..I didnt really like the implementation of the "always hidden" mode so I wen back to Dozer, works much better for me.


Make the ImagePullPolicy Configurable

The pull policy of an APB should be set by the broker.
To configure it, set image_pull_policy in the config file.
The image_pull_policy setting will be verified that it's
a correct setting. The option is not case sensetive and will
error if an incorrect setting is used.
Fixes https://github.com/openshift/ansible-service-broker/issues/235

Let's default this to IfNotPresent

@jwmatthews Made the broker config default to IfNotPresent and the local config I left as Always. Atleast for me with dev work, I use Always because I often change around APBs to point at my own namespace. Let me know if you agree.


First cut at an example where the chatbot is wired to an Ollama Server.

So far an example, but mostly working :-) There are still some kinks though.

Preview: https://chatbot-pr-chatbot-383.surge.sh


[FLINK-33814][autoscaler] Autoscaler Standalone control loop supports multiple threads

What is the purpose of the change
When the job list has a lot of jobs, single thread isn't enough. So Autoscaler Standalone control loop supports multiple thread is very useful for massive production, it's similar to kubernetes.operator.reconcile.parallelism.
Brief change log
Introduce autoscaler.standalone.control-loop.parallelism.
Verifying this change

Added StandaloneAutoscalerExecutorTest#testScalingParallelism

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changes to the CustomResourceDescriptors: no
Core observer or reconciler logic that is regularly executed: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? docs

Thanks @mxm for the quick review and great suggestions!
CI is green, merging~


Cannot install Babel

I cannot install Babel with
uv tool install Babel
I get
error: Failed to find dist-info directory `babel-2.15.0.dist-info` in environment at XXX/.local/share/uv/tools/babel

However the dist-info exists:
ls ~/.local/share/uv/tools/babel/lib/python3.12/site-packages/
__pycache__  _virtualenv.pth  _virtualenv.py  babel  Babel-2.15.0.dist-info

I suspect this is due to the capital letter in the package name.
Maybe related: https://github.com/astral-sh/uv/pull/4686/files
Babel:

installation guide: https://babel.pocoo.org/en/latest/installation.html
pypi: https://pypi.org/project/Babel/
repository: https://github.com/python-babel/babel

Edit: I am sure. babel-2.15.0.dist-info is not Babel-2.15.0.dist-info

Found the "offending" line:
https://github.com/astral-sh/uv/blob/ba924d298f7aeb081711946e6ffc3db7c8d3b4fd/crates/uv-normalize/src/lib.rs#L31
There should be no conversion to lowercase.

Yeah we need to make that more robust.
Technically it's a bug in the package itself. The .dist-info directory is outlined here:

This directory is named as {name}-{version}.dist-info, with name and version fields corresponding to Core metadata specifications. Both fields must be normalized (see the name normalization specification and the version normalization specification), and replace dash (-) characters with underscore (_) characters, so the .dist-info directory always has exactly one dash (-) character in its stem, separating the name and version fields.

The Core Metadata specification asserts that the name should be normalized to lowercase. So it's correct to expect a lowercased directory, but we're supposed to be robust to it not being lowercase due to historical packages (and we are elsewhere).
Anyway, TLDR is: it's a misbehavior by the package, but we shouldn't fail.

https://github.com/python-babel/babel/pull/1110 fixed the root cause of this issue.

Released in https://pypi.org/project/babel/2.16.0/ - sorry for the trouble!


Allow adding extra tags into span

This allows addition of extra tags to span.
I'm uncertain about current implementation. Currently, the code first adds tags into span and then executes the target effect:
pipe(
  T.sync(() => {
    const span = r[SpanContext].spanInstance
    for (const [k, v] of Object.entries(extraTags)) {
      span.setTag(k, v)
    }
  }),
  T.apSecond(ma)
)

Should this be other way around?
Also, it is unclear to me to which span the tags will apply between these two cases:
pipe(
  T.unit,
  withExtraTags({ name: "hello" }),
  withChildSpan("operation")
)

and
pipe(
  T.unit,
  withChildSpan("operation"),
  withExtraTags({ name: "hello" })
)

it looks like this will add to the parent, I agree it's not a clear implementation. The only thing you can do is access the current span so the api would be more like AddExtraTag and not wrap a child

I tried implementing both ways: pipe(ma, T.apFirst(T.sync(() => /* addTag */ ))) and pipe(T.sync(() => /* addTag */), T.apSecond(ma)). It doesn't seem that there is a difference in which span tags are added, but it is slightly easier to do with apSecond approach, since apFirst might not execute addition of tags on error in ma, since ap is implemented in terms of chain.
However, it seems that tags need to be added into child before wrapping into withChildSpan call, or else tags will get added into parent. I'm not sure how improve api in order to make it more obvious other than by adding the comment about it.

I don't think you are getting the point both apFirst & apSecond have access to the same env the only env you have access to is the "current" env (parent), you cannot add it to the child span because the environment of the child is only existing for the child, it is created inside "withChildSpan" and provided to "ma". The only way to add it to the child span would be to have "withChildSpan" be aware of tags, for example by adding a second optional argument to "withChildSpan" or create a separated function that also accepts tags or create a ChildTags environment that is fed from parent and consumed by "withChildSpan"

Yes, I understand that apFirst and apSecond have access to same current environment, which might or might not have child span.
Yes, withChildSpan creates the new span and provides it to ma, so that ma can set tags and when it is doing so, they will get added to that child span. I didn't mean to have an ability of passing tags created in parent span to child span.
With current implementation, if I have something like this (not real code, just mockup):
const tracedEff = pipe(
  T.unit,
  T.liftDelay(100),
  addExtraTags({ tag: 'custom tag' })
)

const main1 = withControllerSpan(tracedEff)
const main2 = withControllerSpan(withChildSpan(tracedEff))

The execution of main1 will add the extra tag into parent span, but in main2 that extra tag will be in the child span.

Have you thought about failure scenarios? we might still want the tag to apply even if "ma" fails

Yea, i quickly checked: applySecond/applyFirst are implemented in terms of chain, so it executes first and if it succeeds, it executes second. So if I use applyFirst anywhere like so:
const x = applyFirst(ma, T.sync(() => { /* set tags */ }))

Then if ma fails, then tags wont get set, since ma will be executed and then tags will be chained after ma.
But if I use applySecond, like so:
const x = applySecond(T.sync(() => { /* set tags */ }), ma)

Then tags will get set first and as long as setting tags doesn't fail, ma will get chained after tags set and executed.
To be fair though, I think it is a good idea to add a test case for this scenario, when ma fails. I will add it later tonight.

targeting legacy


Introduce a script to profile MARL Roundabout

What changes do you make in this PR?

Please describe why you create this PR

Checklist

[x] I have merged the latest main branch into current branch.
[x] I have run bash scripts/format.sh before merging.
Please use "squash and merge" mode.

related to #262


Preview 7 Blazor ServerSide Input's @Bind doesn't call the EventCallback

I've created this simple input generic component
@typeparam T

<input @bind="val" />

@code {

    [Parameter]
    public T val { get; set; }

    [Parameter]
    public EventCallback<T> valChanged { get; set; }
}

but when used from another parent component like this:

@page "/"

<Inp T="@decimal" @bind-val="@val"></Inp>
<div>@val</div>

@code
{
    public decimal val { get; set; }
}

The value in the parent UI component doesn't get updated when you update the input textbox,  it seems that the EventCallback<T> valChanged is not being called at all.
Regards!

Thanks for contacting us, @chrdlx.
Rather than using the @bind on the input element, you should instead directly set the value and add the event handler explicitly for onchange.
We're going to document how to do this as part of  https://github.com/aspnet/AspNetCore.Docs/issues/13539


DropIn "Select payment method " show only Paypal

Hello, I call the dropIn with this code but the only payment method showed is Paypal:

BraintreePayment braintreePayment = new BraintreePayment();
var data = await braintreePayment.showDropIn(nonce: ppToken,amount: currentValue.list_price.toStringAsFixed(1),inSandbox: true,enableGooglePay: true);

I have added inside Application in Manifest
<meta-data android:name="com.google.android.gms.wallet.api.enabled" android:value="true"/>

But I'm not able to see Credit Card and Google Pay on dropIn.
Am I doing something wrong?
Thanks.

Hi @alevittoria ,
Have you pasted this activity inside your manifest.xml?
<activity android:name="com.braintreepayments.api.BraintreeBrowserSwitchActivity"
        android:launchMode="singleTask">
            <intent-filter>
                <action android:name="android.intent.action.VIEW" />
                <category android:name="android.intent.category.DEFAULT" />
                <category android:name="android.intent.category.BROWSABLE" />
                <data android:scheme="com.example.htmlfun.braintree" />
            </intent-filter>
</activity>

Yes, I have done it.
The version of the library is 1.2.2
I'm thinking... Is it possible to disable some options like credit card payment by backend? Is it possible that it's the nonce that disables some options?

Ok, it's not a problem linked with the library, it is an option in Paypal account that disables credit cards payment, thanks for the help


protocol-frame-format-type:: PROTOCOL_FRAME_FORMAT_TYPE_SNAP8021H incorrect _

The  "PROTOCOL_FRAME_FORMAT_TYPE_SNAP8021H " value of the protocol-frame-format-type enumeration has an unnecessary blank at the end in the UML. This results in a "_" at the end in the YANG.
Proposal:
The blank shall be removed in the UML.
So the "_" will no longer occur in the YANG.

according to IF call 2021-09-21 we consider this as a simple "bug" which can be corrected without further approval by ONF 5G-xHaul

Solved with VlanFd_1.0.0-tsp.220407.1740.


How to use Gilde as a StreamProvider

Hi,
I try to use Glide (https://bumptech.github.io/glide/) to load Images. (we like only one Image Library in our project.)
For this reason I try to write a GlideStreamProvider but faild because I have no idea how to get the InputStream
Regards

A quick google yielded the following results:
https://github.com/bumptech/glide/issues/2803 suggests to just use the underlying network layer (OkHttp) that Glide already uses, or...
https://groups.google.com/forum/#!topic/glidelibrary/6X_Q0Z9ERB4 and https://groups.google.com/forum/#!topic/glidelibrary/wneghdP2v2o/discussion seem to indicate how to use Glide APIs to do the same thing.


Regresion: Terminal group relative size not preserved

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: stable and from sources
OS Version:

Steps to Reproduce:

Open terminal panel
Split terminal and resize
Reload window
:bug: terminal relative sizes are not preserved

PR fixing it https://github.com/microsoft/vscode/pull/192519


[./cyber-dojo update] should update language images

At present it only updates service images.

Done


changeset for monorep fix

Current dependencies on/for this PR:

main

PR #28   👈

This comment was auto-generated by Graphite.


Feat: Build Pipeline

https://asaphub.atlassian.net/browse/CRN-606
This is the build pipeline for PRs. It's the build, test and check up to but not including Squidex Sync.

[x] Build
[x] Rebuild
[x] Unit Test
[x] Integration Test
[x] Test Build Output
[x] Test Browser
[x] Bundlewatch
[x] Check Constraints
[x] Check Format
[x] Check Packages
[x] Serverless Package

Moved to seperate PR:

Prepare Squidex
Verify Packages
Notify on Error

Should we remove the duplicate jobs in .gitlab-ci.yml in context of this PR?


Different Variants in datadog kotlin gradle cannot be resolved

I follow your guidelines Plugin Configuration Options  and cannot change serviceName field depending on build variant. I don't know why this example is written in groovy and in your sample is commented out https://github.com/DataDog/dd-sdk-android-gradle-plugin/blob/develop/samples/variants/build.gradle
datadog {
      site = "EU1"
      mappingFileTrimIndents = true
      create("Flavour1") {
          serviceName = "AppServiceOne"
      }
      create("Flavour2") {
          serviceName = "AppServiceTwo"
      }
  }

Hello @KamilPieta! If you are using Kotlin Build Script, can you check if the following answer https://github.com/DataDog/dd-sdk-android-gradle-plugin/issues/154#issuecomment-1463730497 solves your issue?

That helped, thanks!


Issue with "Set" in @site/docs/reference/scripts/set.md

Describe the problem

Page: Set
Source: https://github.com/demisto/content/blob/master//Packs/CommonScripts/Scripts/script-Set_README.md

The documentation mentions "The value to set to the key. Can be an array. Usually, a DQ expression.", but what is a DQ expression?

@matan-xmcyber thanks for reporting this. We will update the doc.


Feature reuqest: Add ALL_PROXY to list of supported variables for Build-time arguments

In https://github.com/docker/docker/issues/14634, variables were whitelisted to add support for proxying via cURL and other libraries. ALL_PROXY should be added to that list. A possible bonus feature is to emulate ALL by adding entries for HTTP, HTTPS, and FTP.

We're okay with adding this, would you be interested in opening a pull request?

@thaJeztah  Were you asking me if I would open a pull request? This isn't code I want to deal with.

Discussing this with other maintainers, and we didn't get more requests and actual use-cases for this; users can already add them as a custom ARG, so there's no big reason to add this. Let's close for now

Hi @thaJeztah
In my company, I'm working with/on Clearlinux and its primary tool is swupd. It requires the ALL_PROXY environment variable at least I've noticed inside a container environment (it uses libcurl and ALL_PROXY is one of the environment variables that libcurl understands.
Looking at the popularity of libcurl and Clearlinux, it might not be a bad idea to add ALL_PROXY to the list of supported-by-default build-time arguments.


FAISS should save/load its index_to_id map

The save_local, load_local member functions in vectorstore.faiss seem firstly to need to read/write the index_to_id map as well as the main index file. This would then fully separate the storage concerns between the Docstore and the VectorStore.
As it is now, the index_to_id map is an additional component the user must separately serialize/deserialize to reload an FAISS index associated with a Docstore.

Check out https://github.com/hwchase17/langchain/issues/789 and https://github.com/hwchase17/langchain/pull/880.

Thanks for your work on this. It looks like the code currently in tree is broken and any change could be an improvement. Closing this as it is a duplicate.


8.4.0RC2 - Search presets cannot be deleted in bulk (as the context menu suggests)

Steps to reproduce:

Select one or more search presets.

Select "Delete" from one of the menus (right-click context menu or title bar menu).

Result: The progress bar moves slowly, then stops just before the end, the spinner turns forever, nothing happens.

This bug, along with others, is still present in 8.4.0.
In the light of Franz's recent blog post about the file manager, i think it would be a good idea to fix know bugs and consider feature request posted here on GitHub first, in order to not just "giving it some polish" but primarily just "make it work better".
https://www.concrete5.org/about/blog/community-blog/file-manager-user-testing


Update Enlightenment, rename e19 -> enlightenment, drop e16

Things done:

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
[x] Built on platform(s): NixOS
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

More
This updates Enlightenment to the next point release (with libraries updated to the next major release), drops e16 and renames e19 -> enlightenment. Tested in VM and by my friend (who used it for several days).
See #12599, #13428 for background and discussion. cc @matejc @ftrvxmtrx

:+1:

Merging this in few days to leave a discussion window.


SPD command takes entered value as m/s instead of kts

Simply put, the command SPD acid X now interprets X as m/s instead of kts.
Example:
00:00:00>CRE D1 M600 52.27065 2.04949 90 30 20
00:00:00>SPD D1 5

D1 will then decelerate to 10 kts, equivalent to approximately 5 m/s.

This is WIP as discussed at the last hackathon, but 253860a should be a quick-and-dirty fix.

This is WIP as discussed at the last hackathon, but 253860a should be a quick-and-dirty fix.

I think this broke the CRE command now. After creating an aircraft at any speed, it proceeds to decelerate to half its speed. So
00:00:00>CRE D1 M600 52.27065 2.04949 90 30 20
```

decelerates to 10 kts.


Blog page, return to home

As blog page doesnt include a nav bar, there should be atleast a Home button to return.
Also along with this if we click on OS-Code text on top left in main page we redirect to home. We can include the logo also to be clickable

@engorgedspirit We are currently creating the blog page through hashnode, so will not be able to give access to everyone!


Sandbox Examples Problem

I am learning Vue.
Sorry, my English is not good.
https://codesandbox.io/s/bar-chart-kog87?file=/src/App.vue
I would like to know what is this for?
options: {
    min: 0,
    max: 100,
}

Whether I delete or change the value, I don't see a difference.

If the chart data is not empty, it is probably balancing itself automatically. Under normal conditions, whichever direction (x or y) indexAxis of the chart is the values entered in the options are the limit values.

If the chart data is not empty, it is probably balancing itself automatically. Under normal conditions, whichever direction (x or y) indexAxis of the chart is the values entered in the options are the limit values.

Thank you.


Create meshery configuration management page

Signed-off-by: Joseph Midura<EMAIL_ADDRESS>Description
This PR fixes #3812
Notes for Reviewers
This is a work in progress. Does creating this page also update docs.meshery.io, or does that site need to be updated separately?
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit be2d90382e3d6c59fbc2d98607be2b92b2c6267f at: https://61228b17f6716595bf9bb357--meshery-docs-preview.netlify.app

Actually, it looks like the netlify preview above answers my question. The page "docs.meshery.io" updated as a result of the configuration management page I created. I didn't think there was a way to build docs.meshery.io page during testing.

hi @josephmidura, thanks for your diving into it,
let's add sth more details into it. could you take a look here again?

@josephmidura do you mean running docs.meshery.io locally?

I didn't think there was a way to build docs.meshery.io page during testing.

@josephmidura that is possible. https://docs.meshery.io/project/contributing-docs
https://docs.meshery.io/project/contributing-docs#serve-the-site (after you install the requirements for that)

It looks to me like the desired configuration management information is currently combined with lifecycle management here. Should this pull request also remove configuration management info from that page?

🚀 Preview for commit 8da6d18079abd9506cecb2800646d908736ba97c at: https://612ed4c5f71da82155c1f686--meshery-docs-preview.netlify.app

🚀 Preview for commit 7b3c6a7bd0b1e33c1e144339f525bbd6253b78cd at: https://612ed5fa3958e12886100df4--meshery-docs-preview.netlify.app

Thanks, GJ @josephmidura!
cc: @leecalcote

@leecalcote is this good to go.?

Oh, my goodness. How did I leave @josephmidura's PR hanging here so long? 😬 I'll have a look...

Closing this PR as separate functionality docs for patterns, filters and applications and guides for each have been created.


ARTEMIS-5002 AMQP producer not unblock if the disk space is freed

(test adapted from the one on the Jira)

@clebertsuconic can you take a look of this PR see if the fix makes sense? Thanks!

@clebertsuconic @gemmellr I've updated the PR. Thanks!


Proposal: Introduce defineCssVar API for Enhanced CSS Variable Management

Summary
I'm proposing a new API, defineCssVar, for Kuma UI to enhance the management and usage of CSS variables within our CSS-in-JS setup. This would allow for more dynamic and maintainable styling by treating CSS variables as JavaScript entities.
Current Challenge
Currently, when working with CSS variables in Kuma UI, we need to hardcode the CSS variables within our components, leading to duplication and complexity in managing responsive and theme-based styles.
Example of current usage:
// Existing method of defining and using CSS variables
import { k, css } from "@kuma-ui/core";
// ...component code...
<body className={css`
  --header-height: 54px;
  @media (min-width: 920px) {
    --header-height: 80px;
  }
`}>
// ...component code...
</body>

Proposed Solution
Introduce the defineCssVar API to create a more structured and less error-prone approach to defining and using CSS variables.
Usage example with defineCssVar:
import { defineCssVar } from "@kuma-ui/core";

const cssVar = defineCssVar({
  headerHeight: {
    name: "--header-height",
    default: "54px",
    media: {
      "(min-width: 920px)": "80px",
    },
  },
});

const { name, className } = cssVar.headerHeight;

Then we can use these variables in our components:
// Use CSS variables in the component
const { name, className } = cssVar.headerHeight;

// ...component code...
<body className={className}>
// ...component code...
</body>

Kuma's Compiler would handle these defined variables by converting them to the appropriate CSS at compile time.
Benefits
Maintainability: Easier to manage and update CSS variables across different components.
Reusability: Promotes the reuse of CSS variables, reducing code duplication.
Scalability: Makes it simpler to scale styling for larger projects with many components.
I look forward to discussing this API design and its implementation further with maintainers @kotarella1110, @yukukotani, @taishinaritomi, but also welcome feedback from the community.

I can see this being extremely beneficial. For not only my use case, but for several others including the ones you listed here. What would it look like changing the variables values at runtime?


request: Use custom param decorator

Hi, really excited to find this powerful library. I plan to migrate to trpc from GraphQL. Is it possible to use createParamDecorator or something similar to get custom param decorators?
import { createParamDecorator, ExecutionContext } from '@nestjs/common';

export const UserDecorator = createParamDecorator((data: unknown, context: ExecutionContext): User => {
  // some logic to get data from context
  return ctx.user;
});

To achieve this:
@Router()
export class AppRouter {
  constructor() {}

  @Query({ output: z.array(someSchema) })
  async test(@UserDecorator() user: User) {
    return [{ _id: user._id }];
  }
}

Hey @yar2001,
Thanks for taking the time and opening an issue!
I actually never tried it, I think it will works but I don't really know since I had to build a custom implementation for the router in order to get everything to work.

Thanks for your reply! I didn't realize it would be so complicated. Anyway, the current library is good enough for production use.
For anyone who wants custom params, you can generate them in context and write code like this:
@Router()
export class AppRouter {
  constructor() {}

  @Query({ output: z.array(someSchema) })
  async test(@ctx() { user }: { user: User }) {
    return [{ _id: user._id }];
  }
}


Incorrect transpilation makes DevTools check always falsy

// This is wrong.
typeof Deno < "u" && window.__PREACT_DEVTOOLS__ && window.__PREACT_DEVTOOLS__.attachPreact("10.15.1", d, {
    Fragment: A,
    Component: T
});

Could esm.sh be serving up a Deno target and rewriting the modules here?

Any updates or insight on this issue? I was able to load Preact devtools after changing the import map to include npm:preact/debug and npm:preact/devtools; however, I cannot get past a message that says:

Connected, listening for Preact operations...
If this message doesn't go away Preact started rendering before devtools was initialized.
You can fix this by adding the preact/debug or preact/devtools import at the top of your entry file.

I added these imports to the top of dev.ts to no avail. I tried injecting the import to the main entry through a Fresh plugin too, but that also had no effect.

The preact/debug import is already injected by Fresh. No need to manually add it. I haven't checked yet with npm: imports, but that's no my to do list.

Happy to report that dev tools work in Fresh 1.5.1 with Preact 10.18.1 🙌


HTTP/2: add exception error messages and log them

TODO:

[x] Actually log exceptions
[x] Verify exceptions in tests

This is ready for review.

Updated w/ logs for Http2StreamErrorException.
I've also done one minor change, which is to always close the connection when receiving a WINDOW_UPDATE frame. You can see the change here, as well as the explanation for it in the comments: https://github.com/aspnet/KestrelHttpServer/pull/2108/commits/1f2b55587504bfc0a9d384d66a263434a752d61a#diff-4ab77515238e5e964ae9a3a4615205b1R553


How to disable GuestUser Middleware

Hi,
I'm trying to integrate this (great) package with Cartalyst Sentinel which I'm familiar with.
So far I have completed all auth system, but I found that GuestUser Middleware is pushing a $user variable to all views, which I think should be optional (that's a very common variable name).
Take this scenario (which I'm dealing with): I'm making an user create/edit form, which is linked to a model, $user. If the form founds a $user variable, it fills the form with the attributes of the $user variable, and if no $user variable, all the inputs are blank.
But when GuestUser Middleware creates a $user for you, a lot of problems can arise (when you try to call a Eloquent relationship, or use some function from the model that clearly doesn't exists).
So, I think there should be a way to disable this GuestUser Middleware, but I didn't found any (the middleware is pushed from a GuestUserProvider which is registered via AdminLTETemplateServiceProvider. If I overload this file i'll have to manually track the updates to that provider... almost no useable :-(
Is there any way to disable or override this GuestUser Middleware?

Some options:

You can use another variable name for you user ;-)
Use a custom service provider extended from mine but whithout registering GuestUserProvider
Create your custom Middleware or custom ViewComposer, execute it after mine and put what you want as user variable in views

Hi @acacha
Thanks for your answer.

You can use another variable name for you user ;-)

Sure, but that's going to generate problems on the future. When I or another dev takes the project and sees an $user variable on his view, I/he/she will make wrong assumptions (Is this my $user or acacha's guest $user?)
Also, that would be a mess when using resources.

Use a custom service provider extended from mine but whithout registering GuestUserProvider

Yes, that's the way I'm taking, but will have to track down changes to your service provider, because it's registering the GuestUserProvider on register() function which registers other providers.
I would be a good option:

To register GuestUserProvider on a separate function inside AdminLTETemplateServiceProvider, so overloading this separate function (registerGuestUserProvider() ?) would suffice.
To make it optional via a config option.

Create your custom Middleware or custom ViewComposer, execute it after mine and put what you want as user variable in views

No, that won't work. I don't want an $user variable when no user has logged in.
Also, this applies to the GravatarServiceProvider, which is now mandatory.
Thanks anyway for your great package and support!

HI @underdpt I agree with you to make this last features as optional only thinking about how to implement that... fill you free to propose a PR if you have any useful idea! Thanks!

Anywat some easy proposed changes are being implemented. See https://github.com/acacha/adminlte-laravel/commit/03a2b9ada39cc5edb75948ffbb1d20ee5b7f1acb

Added config variables to allow disabling extra providers at https://github.com/acacha/adminlte-laravel/commit/8be5bb1d4a8c386de7bccad55f656f6bf5454556

Thanks!  That's very helpful


Extension not working

I've downloaded the exntesion
Did the instalation
Enabled it
And finally tested it at google keep

Result: Nothing happen
Expected: Markdown format

My fault, never mind. I've confounded with another extension, it was not active


compatibility

hey! could it be possible to move the hud more in the left? it would be cool...

its not very beautiful with inventorio's off hand

hello! there are already existing options in the config screen to adjust the vertical and horizontal offset of the armor hud!

Could you show me please?
Le jeu. 19 déc. 2024, 18 h 37, uku @.***> a écrit :

hello! there are already existing options in the config screen to adjust
the vertical and horizontal offset of the armor hud!
—
Reply to this email directly, view it on GitHub
https://github.com/uku3lig/armor-hud/issues/6#issuecomment-2555958483,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BG7DN6UBF7UU4SDB2SXOLY32GNKC7AVCNFSM6AAAAABT5XJERCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJVHE2TQNBYGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

go to options, then click on the uku button and find uku's armor hud. you can configure the mod from there

alternatively, you can also just go to modmenu and find the mod there

Thanks! Merry Christmas by the way
Le mer. 25 déc. 2024, 15 h 31, uku @.***> a écrit :

go to options, then click on the uku button and find uku's armor hud. you
can configure the mod from there
image.png (view on web)
https://github.com/user-attachments/assets/5c13c1d4-d304-4041-8b65-419cc4abebcd
alternatively, you can also just go to modmenu and find the mod there
—
Reply to this email directly, view it on GitHub
https://github.com/uku3lig/armor-hud/issues/6#issuecomment-2561994343,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BG7DN6U3K3KGXI2I5O3QVPL2HMI2BAVCNFSM6AAAAABT5XJERCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRRHE4TIMZUGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

merry christmas and happy holidays!


Auto fixes for rules

I really like what has been done with this project, I haven't contributed much to it due to circumstances of my development stack, but it has been managed really well.
After recently trying out Golang, I'm completely sold on go fmt. This does exactly what html linters used to do and produces the right output for you.
Now I realize this isn't quite as easy in coffeescript, but if there were a directory for fixes that you could optionally run against the failed rules I think that some good things might come.
I counted up the rules that could be easily auto-corrected, it ends up being 19/24. I think that could save some annoyance. Think of the time saved digging, and if you put the command in a pre-commit hook, think of the format changes that will be saved when doing diffs.
What do you think of this idea? I would be willing to help get it started.
Cheers,
Bryan

I know it has been a couple years, but I finally figured out what I really wanted to accomplish with this: https://github.com/brysgo/jscodemigrate


Querying the response content-length never does I/O.

... And if it ever had to, it could return -1.


Stack update, allow unencrypted JSON to be published, support channel groups

@tsloughter if you wouldn't mind doing a review again. I added support for channel groups today. Do you know who to contact that has the permissions to push to this repo?

We can try pinging @stephenlb


MH-12139  Admin UI new event media upload progress bar

This pull causes a new event media upload to show a prominent yellow progress bar containing the percentage of upload across the top of the Admin UI, below the main banner.
The motivation is to let the user know what is going on the background when a large file upload is started and an indication of when it will be complete. Showing this information helps to stop the user from accidentally closing the browser, or starting another upload, or anything that will jeopardize the current large file upload.
This pull adds a new resource, chieffancypants angular-loading-bar tool. There is already a chieffancypants hot key tool with the same license. The angular-loading-bar is included in the same way the first tool was included (a checked in bower_component).
This pull also changes the behavior of the event success/fail notification, in that it does not automatically remove the notification after a few seconds, and it gives the notification a global context so the user can see it if the user has moved onto other tabs while the event was still being processed.

The progress bar UI in the current state of this pull becomes conflicted when more than one event is at the same time. I'm modifying the code to add a new progress bar row for each active upload. I'm hesitant to remove the green "uploading in progress"  banner in the lower area, that duplicates the upload message, because users are accustomed to looking there.

@karendolan Ok, thanks for the info. Please let me know once I should continue the review.
Note that you might encounter similar problems when it comes to the creation of (multiple) scheduled events.

Hi, just a reminder, next Monday (April 1st) is the feature freeze for opencast 7.0

@karendolan Just to be sure: The ball is at your side currently, isn't it?

@karendolan Please note that https://github.com/opencast/opencast/pull/852 (still looking for a reviewer...) might be relevant for this work and that this PR currently has conflicts.

@karendolan If there is nothing happening here, it might be better to close this PR and refile once it's ready.

@karendolan any ETA for further development?

Conflicts @karendolan

Hi @gregorydlogan I'm actually on vacation this week, I fixed the immediate conflicts but not the issue of the multiple long uploads, nor resolved the color of the progress bar. As I was looking at it today, also, I think it's worth revisiting the idea of putting the progress bar right into that existing green "event is being uploaded..." notice bar. since it is already there and already trying to tell the user that something is happening. So it might as well be giving status during a very long upload.
Do you mind if I attempt to migrate the bar into the green "upload in progress notification" and revisit the reasons why this was rejected originally? If I cannot get it resolved by Aug 17. Then I will close this pull.

Per discussion in the technical meeting today, Karen's going to continue with with this pull. If we still can't get this done then we can revisit closing this later.

The pull is now simplified to add an upload percentage to an already existing new event upload notification (see scree shot below). The controversial additional progress bar UI element is omitted.
Purpose: Adding a percentage indicator to the existing upload notification gives the user confidence that a large upload is still progressing (and not a stick spinner). The user can gage how fast the upload is progressing: if the user should stay and wait for a fast upload, or go out for a coffee and come back for a slow one.

I'll take a look at this in the next couple of days since I've been bugging you 'bout it :)

This looks good to me.  My only complaint is that my tests froze at 98.x%, but that freeze appears to be related to waiting on the system to percolate.  When the pause ended the was an update message sent via ActiveMQ, so I'm guessing that it might have been waiting on disk or some other processor intensive thing to actually finish.  Regardless, this works better than nothing, so let's merge it.


Clean model params and init args

Before submitting

[X] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[X] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Replaces hparams from the init of the RL models. Instead there are now explicit arguments for each model. Also updated the model docstrings to match the format seen in the self-supervised models
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
:+1:

Hello @djbyrne! Thanks for opening this PR.

In the file pl_bolts/models/rl/dqn/model.py:

Line 33:121: E501 line too long (138 > 120 characters)

In the file pl_bolts/models/rl/reinforce/model.py:

Line 33:121: E501 line too long (175 > 120 characters)

In the file pl_bolts/models/rl/vanilla_policy_gradient/model.py:

Line 33:121: E501 line too long (189 > 120 characters)

Do see the Hitchhiker's guide to code style


Add guide for contributors related to fork repository mechanism

to propose a change, contributors should follow guide about forking mechanism. I think we could state it in README for clarity

how about state it in #319 separated files?

@andriawan could you please track this issue inside the related epics for the sake of discoverability?
I have invited you as a Collaborator to the repository so you can edit the epic issue descriptions by yourself to do so.
Thanks, @andriawan!

@zainfathoni, now that #319 is merged & closed, is there anything else left to be done for this issue?

I think it could be taken mas @apraditya

I don't quite get it @andriawan. Do you think we can close this issue?

I think adding this infromation to documentation is quite important for new comer as i found several contributors is not quite familiar with forking mechanism

I don't quite get it @andriawan. Do you think we can close this issue?

No, he meant that you could take this issue, @apraditya. @andriawan, please try to avoid passive sentences as much as possible to avoid ambiguity. Your "it could be taken" sentence is unclear on who gets to take the issue. 😅
Please create a draft pull request as explained in our contributing guidelines.

sorry, i lost the context mas @zainfathoni. Sorry for my miss understanding 😅  🙏🏼  @zainfathoni @apraditya


PR: Remove "No match" entry in editor's code completion widget

Description of Changes

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[x] Included a screenshot or animation (if affecting the UI, see Licecap)

Remove no match entry on code completion widget when there's no completions available.
Issue(s) Resolved

Fixes #10002
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: Steff456

@steff456, please see why test_automatic_completions is failing.


Upload button does not activate

When using the php-local bridge reads good the files but when uploading, the modal opens, I select a file and the upload button keeps disabled.
But, if I clic again in the "select files" button makes a request to the server and saves the file.
It is a bad configuration that I'm missing?
Tested with apache and the build in server, here is the basic implementation
http://doctorassistant-dev.us-west-2.elasticbeanstalk.com/filemanager
thanks

I confirm having the same problem.

On the demo if I request bower.json it says 1.5.1
My bower.json says 1.5.1
This problem does not exist on the demo, but does on my localhost. I confirmed the inProcess flag is incorrect. If I remove that flag it does let me upload.


Document cancellation safety of muxink

Through careful analysis, we should document the cancellation safety of every Sink and Stream-based method call of wrapped parts provided by the muxink module. In other words, make it possible for the user to know whether a call like SinkExt::send will yield a cancellation safe future or not.

Done in 306d6f8ea..a844725d6.


Pick up ERC1155 from transfer APIs

Refactor the way ERC1155 tokens are picked up. This also required a little re-work of some other areas to ensure it works correctly.
Etherscan and other TX APIs added a new token1155tx action so the simple boolean NFt toggle needed to be removed and handled a different way.
Fixes #3062

Later maybe we can refactor to have subclass for Token such as ERC1155Token.

Do you mean move the code to the ERC1155Token class or is this in a different context, a different Token class for use only in the Transaction fetch?
The UpdateTokenBalance method does need that refactor.

Oh, we already have an ERC1155Token subclass, but it's large enough. I'm considering having a strategy class for each type of token. To gain a better understanding, we need to read the code in more detail.


Slippery components spin after being slipped on

While hilarious it probably shouldn't happen as it looks silly.

Can't replicate.


cli: Add server OS/Arch info to 'version' cmd

Added 'OS/Arch (server)' line to docker version.
This info already is in /version endpoint but was missing from docker version output, it may come handy when we have a Windows daemon.

Client API version: 1.17
Go version (client): go1.4.1
OS/Arch (client): darwin/amd64
Server version: 1.5.0
Server API version: 1.17
Go version (server): go1.4.1
Git commit (server): a8a31ef
OS/Arch (server): linux/amd64 :new::new::new::new::new::new::new::new::new::new:

Signed-off-by: Ahmet Alp Balkan<EMAIL_ADDRESS>cc: @vieux @jfrazelle @unclejack @tiborvass @tianon

LGTM

+1 useful if people run their client on a separate machine from the daemon.

+1! can you please add the same info to the info section in cli.md?

@SvenDowideit there's not really much about docker version in cli.md what do you want me to add for instance?

@ahmetalpbalkan always +1 to more details :)

@SvenDowideit added some stuff lmk if it looks good.

:+1:

does that count as LGTM? :joy:

Guess it's a down-under "ɯʇƃl" 😊

@ahmetalpbalkan You should also add info to api docs and to whats new section.

@LK4D4 that's not an API change?

@ahmetalpbalkan :) yeah, then api docs is wrong :/

@LK4D4 updated remote API docs as well.

Okay, at least it documented now :)
LGTM

@LK4D4 yeah, better late than never. :smiling_imp:

LGTM

LGTM