Support dashDB

What version of Flyway are you using?
4.1.0
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)
All
What database are you using (type & version)?
dashDB (based on db2 10.5 running on Bluemix - the IBM cloud platform)
What operating system are you using?
MacOS
What did you do?
Do a migrate
What did you expect to see?
metadata table created
What did you see instead?
A dirty "DB2 SQL Error: SQLCODE=-1666, SQLSTATE=42613, SQLERRMC=CHECK, DRIVER=4.16.53"
How could you fix that?
Quickly: just add the "ORGANIZE BY ROW" directive in the createMetadataTable.sql script for db2:
CREATE TABLE ... ( ...) ORGANIZE BY ROW;
The row organization is the default mode for db2 but not for dashDB (column organization).

Duplicate of #1732


Failure to parse response from OAuth URL

Hey again!  Up until now, I've just been authenticating through arc.check_portal() / auth_binding() — worked great for testing, but trying to move to auth_client() using OAuth so we can use this new package in our automated scripts and stuff.  (Sometimes Arc Pro automatically logs you out of your portal after a period of inactivity, so unfortunately I don't think auth_binding() would work non-interactively in the long term for us.)
Unfortunately, running into a problem with auth_client() where I'm getting the following error:
Error in `httr2::oauth_flow_client_credentials()`:
! Failed to parse response from
  `client$token_url` OAuth url.
Caused by error in `resp_body_json()`:
! Unexpected content type "text/plain".
• Expecting type "application/json" or suffix
  "json".
Run `rlang::last_trace()` to see where the error occurred.

I thought it was probably something to do with the way our internal enterprise is set up, but I double checked with our administrator and we think we have the OAuth client ID all set up properly.  (And I have the ID set to my environment variables correctly, and have restarted R, etc.)
What stumped me is when I tried auth_code()— I'm able to log into our enterprise portal and receive an OAuth code just fine.  However, when I copy that code into R, I get the exact same error as above.  The code looks to be a 216-character long token that always ends in ..; I tried putting it in raw as well as in quotes and got the same result either way.
Not sure how exactly to give you a reprex for this one, given we're on an enterprise portal and it's probably not a good idea to paste the raw tokens here.  But let me know if there's anything I can give you that would make this any easier to look into; I'm happy to help any way I can!

This comes from {httr2}
httr2:::oauth_client_get_token
function (client, grant_type, ..., error_call = caller_env())
{
    req <- request(client$token_url)
    req <- req_body_form(req, grant_type = grant_type, ...)
    req <- oauth_client_req_auth(req, client)
    req <- req_headers(req, Accept = "application/json")
    resp <- oauth_flow_fetch(req, "client$token_url", error_call = error_call)
    exec(oauth_token, !!!resp)
}

What this is telling me is that the response from your Enterprise is not returning application/json as it should be, instead the content-type is text/plain. My guess, though, is that it is actually json in the response with the content type incorrectly set.

One thing to also ask your admin is if there is a reverse proxy or something which is modifying the headers.

Yeah, that would make sense to me.  Wonder if maybe there's a way to check for plain text responses and coerce them to JSON, if possible, from the package side?  Or if there's a way to verbose-ly get whatever that plain text response is, that might be helpful to look at too, for debugging purposes.
Our admin says we're on Enterprise 10.8 here!  I can ask about the reverse proxy and see if he knows anything on that front.
Not sure about the API key— the documentation here talks about them vis-a-vis the ArcGIS Platform or Places Service; do you know if there's a way to set up an API key in an enterprise portal?  We'd probably need to stick with the geocoder we have in there, for our purposes— but I'm on the data science side, not the IT admin side, so maybe there's a different way of setting this up that I just don't know about :)

There isn't any way for us to change the oauth client flow to accept plain text input. That would have to be reported in {httr2} or we would have to write the OAuth flow from scatch which feels a bit out of scope. Above all, I believe that accepting text/plain I think would be a security risk.
I'm not sure what the best way to address this is because the Enterprise server should be returning application/json in the Content-Type header and not text/plain.

Oh!— you're so right, of course {httr2} is handling all that, not sure why that didn't click for me earlier.  Yup, totally out of scope for this, then.  And the security concern is also valid, I hadn't thought of that either.
FWIW, just heard back from our GIS admin— he attests that the enterprise is returning JSON, and that we have no reverse proxies going on.

FWIW, just heard back from our GIS admin— he attests that the enterprise is returning JSON, and that we have no reverse proxies going on.

I am quite sure that the content itself is json. However, the request that it is sending has a header set incorrectly.
For example, look at this response header from a random website (you can inspect this by opening your browser's developer tools and going to network and clicking one of the requests)
Responses typically have this Content-Type header which specifies what kind of content is being returned. Perhaps they can check the devtools and confirm that the content type is application/json

Sure, I'll pass that info along— thanks, Josiah!

@aaronkrusniak wonderful! Let me know what you find. If it is text/plain perhaps we can make a support ticket and determine if this is a bug or a configuration issue. If that happens, please include jparry at esri dot com too :)


Making CreateCodeWriter protected so that it can be overridden in Mvc tests.

/cc: @NTaylorMullen

:shipit:

Thanks! https://github.com/aspnet/Razor/commit/1f480386f4fd36aacb4e0f3959469901aab67ed9


Remove unuseful toString method

What changes were proposed in this pull request?
Remove some toString() method.
Why are the changes needed?
Some toString() method is unuseful, so they should be deleted.
How was this patch tested?

Gentle ping, @deadwind4

Thank you, @deadwind4 . Mostly, the PR looks good.
cc @williamhyun , too

Here is a superset of this PR, @deadwind4 .
ORC-1219 will be assigned to you because you are the main contributor.

Here is a superset of this PR, @deadwind4 . ORC-1219 will be assigned to you because you are the main contributor. Please let me know your Apache JIRA ID.

I have commented on this Jira ticket so that you can assign me. Thanks a lot.

Thank you!


Add --force option

Feature request for a new option, --force, that will disregard and overwrite already existing files when installing a package.
I'd be happy to submit a PR for this, unless there is something I haven't considered and this is a bad idea.
Context
I maintain a number of Python projects in the Arch User Repository (AUR) and for projects that require tests to run on an installed package I use the following pattern:
build() {
  cd "$_archive"

  python -m build --wheel --no-isolation
}

check() {
  cd "$_archive"

  rm -rf tmp_install
  python -m installer --destdir=tmp_install dist/*.whl

  local site_packages=$(python -c "import site; print(site.getsitepackages()[0])")
  export PYTHONPATH="$PWD/tmp_install/$site_packages"
  pytest
}

I.e. to install the package to a temporary location and add that to PYTHONPATH before running the tests.
The rm invocation is necessary when the directory is not clean and the package has already been installed in the tmp_install directory. However, this invocation could be made redundant by installer having the option to --force the install and disregard/overwrite any existing package files.

--force is a pretty generic name. There are other causes of installation failures - such as problems with the RECORD file - that won't be ignored by --force. Maybe rename to something more specific like --overwrite-existing?


Facilitate the development of data loaders/writers

Currently we are limited to only few data loaders and writers and most are implemented as a Dask transform component. In addition, implementing them sometimes requires some deep knowledge of Dask and Fondant (setting index, renaming dictionary, how to only return subset of initial df, ...)
Ideally we would:

Enable pandas loader/writer component
Abstract away as much Fondant/Dask specifics as possible behind the scenes, if not possible maybe we can provide a boilerplate of functions that should be implemented (setting index, loading few rows, ..) or additional documentation/best practices on  how to develop them

@RobbeSneyders alright I think that makes sense. I created a ticket for the documentation #584


CPU run apache2 licensed model

what code downloads and installs the apache2 licenced models for CPU's I see no separate switch for enabling and disabling the CPU/GPU runtime
also is there a list of models this works with? or a way to query for the models available that also says what they are for?
Keep up the great work! much appreciated!

If you mean llama.cpp and GPT4All CPU models, then there's a specific base_model for 'llama' and 'gptj' and 'gpt4all_llama'.  Using those models triggers their own loaders, although if one builds llama-cpp-python with CUDA support, as in GPU docs, then it'll use GPU.
If you have a GPU model but want to use CPU, you can but it'll be slow and use alot of memory, but you can run CUDA_VISIBLE_DEVICES= python generate.py ... to do that.


Update ubuntu dependencies to fix CVEs

Two deps, libevent and rpcbind, cannot be updated without pulling in changes from ubuntu
"disco" distro, which is too far forward (not tested against K8s upstream)
What this PR does / why we need it:
Fix CVEs against dependencies
How can this PR be verified?
Runs correctly and the dependencies are not on the list of CVEs
Is there any change in kubo-deployment?
No
Is there any change in kubo-ci?
No
Does this affect upgrade, or is there any migration required?
No
Which issue(s) this PR fixes:
https://www.pivotaltracker.com/story/show/167631002
Release note:
PLEASE ASK PMs IF THEY WANT THE FOLLOWING IN THE RELEASE NOTES:
The following CVEs are NOT fixed in this version because patches have not been backported to Ubuntu Xenial:
libevent-2.0.21
1.) https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2016-10195
2.) https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2014-6272
3.) https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2016-10196
4.) https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2016-10197
rpcbind
1.) https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2017-8779
2.) https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2015-7236

Looks good learnt about $(($count+1)) ;)


Add information about  ibm dromedary

please add this model info
https://mitibmdemos.draco.res.ibm.com/dromedary

Hi, thanks for bring attention to this paper.
Since i am not very familiar with it, would you mind making a PR for it?
程信
发件人：bharathrajcl @.>
发送日期：2023-05-08 17:16:12
收件人：Hannibal046/Awesome-LLM @.>
抄送人：Subscribed @.***>
主题：[Hannibal046/Awesome-LLM] Add information about ibm dromedary (Issue #29)
please add this model info
https://mitibmdemos.draco.res.ibm.com/dromedary
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Done. Thanks again for bring this paper to Awesome-LLM!


fix: donation-alerts

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Absolutely. We have two donation alerts that we display to non-donors:

displayed on the learn landing page when users are signed in (not to interfere with the sign in CTA)
displayed on beta certifications

Both of the above-mentioned alerts have a donate button with a google analytics donation_related event.


Is pep8speaks working well?

I returned to do some work on xarray, and looks like there are lots of linting errors. Maybe we shouldn't worry ourselves with this; though let's make a deliberate decision
Is pep8speaks working well? One example is this PR: https://github.com/pydata/xarray/pull/2553, where it looks like pep8speaks initially complained, but doesn't have a test so was potentially lost in the discussion.
Any thoughts? I'm happy to help see if there are alternatives / pep8speaks can be added as a test / add a test in travis / fix these.
FWIW I've used black successfully in pandas-gbq and internally, but I think it's maybe too much change for a large project like xarray
./versioneer.py:943:-13592: W605 invalid escape sequence '\s'
./versioneer.py:943:-13408: W605 invalid escape sequence '\s'
./versioneer.py:943:-13228: W605 invalid escape sequence '\s'
./versioneer.py:943:-11196: W605 invalid escape sequence '\d'
./versioneer.py:943:-8162: W605 invalid escape sequence '\d'
./xarray/testing.py:148:1: W391 blank line at end of file
./xarray/core/options.py:66:80: E501 line too long (97 > 79 characters)
./xarray/core/combine.py:381:36: E226 missing whitespace around arithmetic operator
./xarray/core/combine.py:536:75: E211 whitespace before '('
./xarray/core/combine.py:537:57: E126 continuation line over-indented for hanging indent
./xarray/core/dataarray.py:19:1: F401 '.options._get_keep_attrs' imported but unused
./xarray/core/dataset.py:15:1: F401 '.computation' imported but unused
./xarray/core/dataset.py:18:1: F401 '..conventions' imported but unused
./xarray/core/dataset.py:27:1: F401 '.dtypes.is_datetime_like' imported but unused
./xarray/core/dataset.py:32:1: F401 '.pycompat.integer_types' imported but unused
./xarray/core/dataset.py:520:5: E303 too many blank lines (2)
./xarray/core/dataset.py:739:80: E501 line too long (81 > 79 characters)
./xarray/core/dataset.py:750:80: E501 line too long (80 > 79 characters)
./xarray/core/dataset.py:768:80: E501 line too long (88 > 79 characters)
./xarray/core/dataset.py:779:80: E501 line too long (84 > 79 characters)
./xarray/core/dataset.py:788:80: E501 line too long (84 > 79 characters)
./xarray/core/dataset.py:795:80: E501 line too long (80 > 79 characters)
./xarray/core/dataset.py:811:80: E501 line too long (84 > 79 characters)
./xarray/core/dataset.py:828:80: E501 line too long (80 > 79 characters)
./xarray/core/dataset.py:1178:80: E501 line too long (92 > 79 characters)
./xarray/core/dataset.py:1365:13: F401 'dask' imported but unused
./xarray/core/dataset.py:1365:80: E501 line too long (92 > 79 characters)
./xarray/core/dataset.py:1624:80: E501 line too long (82 > 79 characters)
./xarray/core/dataset.py:1763:80: E501 line too long (82 > 79 characters)
./xarray/core/dataset.py:1866:80: E501 line too long (80 > 79 characters)
./xarray/core/dataset.py:1867:80: E501 line too long (80 > 79 characters)
./xarray/core/dataset.py:1977:16: E111 indentation is not a multiple of four
./xarray/core/dataset.py:1991:37: E126 continuation line over-indented for hanging indent
./xarray/core/dataset.py:2908:39: F812 list comprehension redefines 'dim' from line 2900
./xarray/core/dataset.py:2976:80: E501 line too long (86 > 79 characters)
./xarray/core/dataset.py:2983:80: E501 line too long (82 > 79 characters)
./xarray/core/dataset.py:3027:80: E501 line too long (80 > 79 characters)
./xarray/core/dataset.py:3709:32: F812 list comprehension redefines 'dim' from line 3656
./xarray/core/dataset.py:3749:80: E501 line too long (80 > 79 characters)
./xarray/core/dataset.py:3832:80: E501 line too long (83 > 79 characters)
./xarray/core/dataset.py:3912:80: E501 line too long (82 > 79 characters)
./xarray/core/dataset.py:3918:80: E501 line too long (84 > 79 characters)
./xarray/core/dataset.py:3929:80: E501 line too long (82 > 79 characters)
./xarray/core/dataset.py:3932:80: E501 line too long (88 > 79 characters)
./xarray/core/dataset.py:3933:80: E501 line too long (83 > 79 characters)
./xarray/core/common.py:3:1: F401 'warnings' imported but unused
./xarray/core/common.py:4:1: F401 'distutils.version.LooseVersion' imported but unused
./xarray/core/common.py:209:80: E501 line too long (90 > 79 characters)
./xarray/core/common.py:429:80: E501 line too long (82 > 79 characters)
./xarray/core/common.py:465:80: E501 line too long (86 > 79 characters)
./xarray/core/common.py:468:80: E501 line too long (84 > 79 characters)
./xarray/core/common.py:472:80: E501 line too long (81 > 79 characters)
./xarray/core/common.py:473:80: E501 line too long (87 > 79 characters)
./xarray/core/common.py:486:80: E501 line too long (82 > 79 characters)
./xarray/core/common.py:496:80: E501 line too long (80 > 79 characters)
./xarray/core/common.py:505:80: E501 line too long (80 > 79 characters)
./xarray/core/common.py:527:80: E501 line too long (84 > 79 characters)
./xarray/core/common.py:560:80: E501 line too long (80 > 79 characters)
./xarray/core/common.py:564:80: E501 line too long (81 > 79 characters)
./xarray/core/common.py:568:80: E501 line too long (85 > 79 characters)
./xarray/core/common.py:637:80: E501 line too long (81 > 79 characters)
./xarray/core/common.py:641:80: E501 line too long (85 > 79 characters)
./xarray/core/common.py:648:80: E501 line too long (86 > 79 characters)
./xarray/core/common.py:654:80: E501 line too long (86 > 79 characters)
./xarray/core/common.py:661:80: E501 line too long (89 > 79 characters)
./xarray/core/common.py:677:80: E501 line too long (80 > 79 characters)
./xarray/core/common.py:679:1: W293 blank line contains whitespace
./xarray/core/utils.py:511:13: W605 invalid escape sequence '\:'
./xarray/plot/plot.py:17:1: F401 'xarray.core.alignment.align' imported but unused
./xarray/plot/plot.py:259:5: E303 too many blank lines (2)
./xarray/backends/netcdf3.py:12:14: W605 invalid escape sequence '\('
./xarray/backends/pynio_.py:12:1: I004 isort found an unexpected blank line in imports
./xarray/backends/api.py:18:1: I004 isort found an unexpected blank line in imports
./xarray/backends/api.py:26:9: F401 'netCDF4' imported but unused
./xarray/backends/api.py:30:13: F401 'pydap' imported but unused
./xarray/backends/api.py:41:9: F401 'Nio' imported but unused
./xarray/backends/api.py:46:9: F401 'cfgrib' imported but unused
./xarray/backends/api.py:59:9: F401 'scipy' imported but unused
./xarray/backends/api.py:68:9: F401 'netCDF4' imported but unused
./xarray/backends/api.py:72:13: F401 'scipy.io.netcdf' imported but unused
./xarray/backends/api.py:581:80: E501 line too long (80 > 79 characters)
./xarray/backends/api.py:648:80: E501 line too long (81 > 79 characters)
./xarray/backends/pseudonetcdf_.py:12:1: I004 isort found an unexpected blank line in imports
./xarray/backends/zarr.py:11:1: F401 '.common.ArrayWriter' imported but unused
./xarray/backends/zarr.py:240:80: E501 line too long (82 > 79 characters)
./xarray/backends/rasterio_.py:14:1: I004 isort found an unexpected blank line in imports
./xarray/backends/file_manager.py:6:1: I004 isort found an unexpected blank line in imports
./xarray/tests/test_combine.py:10:1: F401 'xarray.merge' imported but unused
./xarray/tests/test_combine.py:495:65: E211 whitespace before '('
./xarray/tests/test_combine.py:523:52: E231 missing whitespace after ','
./xarray/tests/test_combine.py:523:54: E231 missing whitespace after ','
./xarray/tests/test_combine.py:523:61: E231 missing whitespace after ','
./xarray/tests/test_combine.py:524:57: E241 multiple spaces after ','
./xarray/tests/test_combine.py:525:55: E241 multiple spaces after ','
./xarray/tests/test_combine.py:526:55: E241 multiple spaces after ','
./xarray/tests/test_combine.py:527:57: E241 multiple spaces after ','
./xarray/tests/test_combine.py:597:52: E203 whitespace before ','
./xarray/tests/test_combine.py:646:80: E501 line too long (85 > 79 characters)
./xarray/tests/test_combine.py:647:80: E501 line too long (85 > 79 characters)
./xarray/tests/test_combine.py:658:80: E501 line too long (83 > 79 characters)
./xarray/tests/test_combine.py:659:80: E501 line too long (87 > 79 characters)
./xarray/tests/test_plot.py:1164:13: F841 local variable 'g' is assigned to but never used
./xarray/tests/test_utils.py:12:1: F401 'xarray.core.options.set_options' imported but unused
./xarray/tests/test_dataarray.py:13:1: F401 'xarray.set_options' imported but unused
./xarray/tests/test_backends.py:660:80: E501 line too long (82 > 79 characters)
./xarray/tests/test_backends.py:1345:9: F841 local variable 'zarr' is assigned to but never used
./xarray/tests/test_backends.py:2171:9: E303 too many blank lines (2)
./xarray/tests/test_backends.py:2197:36: E127 continuation line over-indented for visual indent
./xarray/tests/test_backends.py:2204:36: E126 continuation line over-indented for hanging indent
./xarray/tests/test_backends.py:2242:5: F811 redefinition of unused 'test_open_mfdataset' from line 2155
./xarray/tests/test_backends.py:2965:80: E501 line too long (82 > 79 characters)
./xarray/tests/test_variable.py:30:1: I004 isort found an unexpected blank line in imports
./xarray/tests/test_distributed.py:3:1: F401 'distutils.version.LooseVersion' imported but unused
./xarray/tests/test_distributed.py:4:1: F401 'os' imported but unused
./xarray/tests/test_distributed.py:5:1: F401 'sys' imported but unused
./xarray/tests/test_distributed.py:7:1: F401 'tempfile' imported but unused
./xarray/tests/test_distributed.py:12:80: E501 line too long (81 > 79 characters)
./xarray/tests/test_distributed.py:14:1: F401 'dask.array' imported but unused
./xarray/tests/test_distributed.py:17:1: F401 'distributed.utils_test.loop' imported but unused
./xarray/tests/test_distributed.py:19:1: F401 'numpy as np' imported but unused
./xarray/tests/test_distributed.py:28:1: F401 '.raises_regex' imported but unused
./xarray/tests/test_distributed.py:66:1: F811 redefinition of unused 'loop' from line 17
./xarray/tests/test_distributed.py:76:49: F841 local variable 'c' is assigned to but never used
./xarray/tests/test_distributed.py:96:1: F811 redefinition of unused 'loop' from line 17
./xarray/tests/test_distributed.py:106:49: F841 local variable 'c' is assigned to but never used
./xarray/tests/test_distributed.py:121:1: E303 too many blank lines (3)
./xarray/tests/test_distributed.py:121:1: F811 redefinition of unused 'loop' from line 17
./xarray/tests/test_distributed.py:126:9: F841 local variable 'zarr' is assigned to but never used
./xarray/tests/test_distributed.py:133:49: F841 local variable 'c' is assigned to but never used
./xarray/tests/test_distributed.py:147:1: F811 redefinition of unused 'loop' from line 17
./xarray/tests/test_distributed.py:151:53: F841 local variable 'c' is assigned to but never used
./xarray/tests/test_distributed.py:158:1: F811 redefinition of unused 'loop' from line 17
./xarray/tests/test_distributed.py:161:49: F841 local variable 'c' is assigned to but never used
./xarray/coding/cftime_offsets.py:422:17: W605 invalid escape sequence '\d'
./xarray/coding/cftimeindex.py:71:2: W605 invalid escape sequence '\-'
./xarray/coding/cftimeindex.py:71:2: W605 invalid escape sequence '\:'
./xarray/coding/cftimeindex.py:72:6: W605 invalid escape sequence '\d'
./xarray/coding/cftimeindex.py:72:6: W605 invalid escape sequence '\d'
./xarray/coding/cftimeindex.py:72:6: W605 invalid escape sequence '\d'
./xarray/coding/cftimeindex.py:72:6: W605 invalid escape sequence '\d'
./xarray/coding/cftimeindex.py:72:6: W605 invalid escape sequence '\d'
./xarray/coding/cftimeindex.py:72:6: W605 invalid escape sequence '\d'

I think I need to be more careful to check the first comment for PEP8 issues before merging -- I've been relying on the green checkmarks.

@max-sixty Is strict pep8 compliance supposed to be mandatory in xarray? Sometimes it makes more sense to violate PEP8 to improve readability. My PR #2553 has some examples - e.g. here where I deliberately used a longer line rather than two lines so that it was clearer that I was testing what happens when you concatenate a square of 4 datasets. On the other hand there are also some violations of pep8 in the PR which are just oversights, and did get lost in the (lengthy) discussion.
Is strict pep8 compliance already supposed to be mandatory in xarray? Should any intentional pep8 violations be flagged using # noqa so that flake8 ignores that line?

We do aim for strict PEP8 compliance, but this is hard to enforce without the automated checks. Cases where it's violated intentionally can be marked with a comment.

Closed via #2632


Readline options?

JFYI,
It was mentioned in Ruby core, another ruby implementation:
https://github.com/aycabta/reline
I did test it with the Ruby Core readline tests, and it did fail some tests (missing methods, etc).
I'd help with it, but given that I'm a Windows type, I don't really have a 'baseline' for how readline should work...

Your fork is good for Windows until some progress is made on the official libraries :+1:

Sometime soon, I'll try it with byebug and see how the tests do...

Off topic - JFYI https://bugs.ruby-lang.org/issues/15313

Yesterday morning I ran tests with reline & ruby-head, some issues.  Haven't torn thru your code, how do I get typical --verbose test output?
From my perspective, reline should work with byebug...

I think with TESTOPTS=--verbose?
Yeah, I think it should work too, but when I tried tests wouldn't run. I think it's some problem when shelling out with Open3 and passing user input to the subprocess.

First thing I tried.  Dots...
IO.popen, Open3, etc.  I've worked plenty with STDOUT & STDERR, but STDIN, no.
I've tried tests from time to time also.  Also, I said 'reline should work with byebug'.  I really think that reline can only be considered a replacement for readline when it works with byebug...

First thing I tried. Dots...

Try bin/minitest --verbose (not sure why the other option is not working).
It'd be great to get this working, indeed.

Duh.  Looked at the code a bit
rake test --verbose

Back to frozen tests...

I worked a bit with reline using ruby-head, which I think is equivalent to reline master.  Tests were intermittent, but sometimes all did pass.  Used the following code for reline, most of which came from its test setup:
require 'reline'
Reline.send(:remove_const, :IOGate) if Reline.const_defined?(:IOGate)
Reline.const_set :IOGate, Reline::GeneralIO
Reline.core.config.instance_variable_set :@test_mode, true
Reline.core.config.reset
Reline.core.input = STDIN

Using something similar to the above, CI is passing on Windows using reline.  I'm considering this to be on hold until a determination is made as to reline's UI compatibility with Windows.


Add cookie consent

As an end user, I have a right to GDPR protections.
Add a cookie consent popup.

Resolved by removing the use of cookies.


Add scss-compile extension

Description

This extension helps developers to compile SCSS file into CSS file,
hope you will like it （　´∀｀）
Screencast/Video

https://github.com/raycast/extensions/assets/35317549/ae01eb69-80ae-451f-97c0-04e62801d766
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

It this view, what's the Watch Changes for?

How do I test with and without that enabled?

This

It this view, what's the Watch Changes for?

How do I test with and without that enabled?

Hello @pernielsentikaer, this option allows user to directly "watch" the "scss" file when they save the file. Please see below video as demonstration:
https://github.com/raycast/extensions/assets/35317549/4e6d7b36-4d6b-4fd0-9e81-112a8ba12b5e
But I can tell where you confusion comes from, they are able to turn on "watch" after save easily, so putting an extra configuration here isn't quite necessary, do you recon it is better to remove it ?

I think it's fine to keep it as is 🙂


Completion doesn't find local bindings

If I define let something, and then execute it, I would expect that typing som<tab> would autocomplete to something

It looks like you'd need access to toplevel_value_bindings, which is private :(
Would we be able to just grab the toploop.ml file and copy it into this project?

utop does this right -- I'm looking into what they're doing, seeing if we can use it.
https://github.com/diml/utop/blob/master/src/lib/uTop_complete.cppo.ml

That might be dangerous (theres some properly weird stuff going on in translmod around setvalue/getvalue).
It shouldn't be too hard to interpret the output of parsing and maintain our own table.
One thing I have been wondering about is whether it is possible to use utop (as a library) to control the  top level.  It tends to follow the ocaml compiler releases and features a bit better than iocaml (ie I would love to have ppx in iocaml, but it doesnt work at the moment).

(great minds and all that!)

:D yup. it would require forking utop (it doesn't currently expose all we would want it to), unless we pretend to be emacs and connect to utop over a socket

How come you've decided to use utop rather than merlin?
merlin also has a completion API and its already available for general use through its RPC.

merlin has a completion api that works with a running repl context? this isn't just for statically analyzable completions, this is for a repl

I see. I did not appreciate this difference at first. Isn't an ipython notebook
more like a buffer rather than a true repl though? And with OCaml, you shouldn't
need to actually run any of the source code to generate all completions. So I'm
not sure what a non "statically analyzable completion" would be.
In any case, @let-def could you please comment on this?
On 08/22, Jared Forsyth wrote:

merlin has a completion api that works with a running repl context? this isn't just for statically analyzable completions, this is for a repl
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/andrewray/iocaml/issues/50#issuecomment-241492004

In its current state Merlin is not compatible with a REPL because of the patches applied on compiler-libs.
We are looking into making it usable for a REPL, that should happen early 2017 and will probably be exposed as a library to use on top of compiler-libs.

@rgrinberg ipython is a true repl, with a live context that you can interact with and manipulate. though you're right that you could probably figure everything out statically, utop was built for the kind of interaction that people expect from ipython

@let-def any update with the Merlin then?

I didn't implement anything directly on Merlin side yet, but @SanderSpies extracted relevant code and made a proof of concept 👍 .
I have some ideas on how to improve the integration to toplevel, but I haven't had time to look at that :/.


@AndroidParcelable to check before adding blank constructor

Currently it forcefully adds a blank constructor and doesn't allow you to add your own

I tried fixing this today, but for some reason, the compiler does not acknowledge the empty ctors that were generated by the AndroidParcelable active annotation.
You may wanna have a go at it.

Not sure if we're missing each other - what I meant by this issue is
that if I create a bean and add an empty constructor, then add the
@AndroidParcelable annotation, I get a compile error because there
are now 2 empty constructors. If @AndroidParcelable first checked
that the empty c'tor exists, then it can bypass adding it's own, and
hence no compile error :-)

Yeah, I found it. Also, I found other issues. It's fixed. Patch coming up soon.


feat: Add crop image in profile fragment

Fixes: #1994
Fixes #[Add issue number here. If you do not solve the issue entirely, please change the message e.g. "First steps for issues #IssueNumber]
Changes: [Add here what changes were made in this issue and if possible provide links.]
Screenshots for the change:

resolved conflicts


Synced mode seems to be incompatible with nx 13 and project.json files

Hi everybody,
We have some projects using jscutlery/semver, and as we are migrating to nx 13 and the new project.json configuration structure, we are encountering the next problem: we cannot have a project.json at the workspace level because it causes some problems with the Nx cli (commands that use the dependencies graph are looping).
Did some of you encountered the same problem ?
For now, the solution we came with is to create two projects (let's call them apps-workspace and libs-workspace) with only a version target.
apps/apps-workspace/project.json
{
  "ignore": true,
  "projectType": "application",
  "root": "apps",
  "targets": {
    "version": {
      "executor": "@jscutlery/semver:version",
      "options": {
        "syncVersions": true
      }
    }
  }
}

libs/libs-workspace/project.json
{
  "ignore": true,
  "projectType": "library",
  "root": "libs",
  "targets": {
    "version": {
      "executor": "@jscutlery/semver:version",
      "options": {
        "syncVersions": true
      }
    }
  }
}

This allows to continue to use the sync mode and to run the target with nx run-many --all --target version. But it also makes possible to have divergences between the versions of the apps and the libs.
One solution would be to add a cwd option to the @jscutlery/version executor to allow to run it in another directory than the projectRoot directory, but not sure this is the most elegant solution.
Thank you in advance for your help!

I believe I'm running into this too.

For my use case, I don't version the apps, so just changing the root from . to libs worked:
    "workspace": {
      "targets": {
        "version": {
          "executor": "@jscutlery/semver:version",
          "options": {
            "syncVersions": true
          }
        }
      },
      "root": "libs",
      "tags": []
    },

You might notice I'm not using the project.json scheme. Others might run into a similar situation as me

Hi there, in your workspace.json file you can mix standalone configuration for your projects and the virtual "workspace" project for versioning the whole repo.
{
  "version": 2,
  "projects": {
    "workspace": {
      "root": ".",
      "targets": {
        "version": {
          "executor": "@jscutlery/semver:version",
          "options": {
            "syncVersions": true
          }
        },
      }
    },
    "demo": "apps/demo",
    "demo-e2e": "apps/demo-e2e"
  }
}

Hope it helps.

This seems to result in:
>  NX   The angular.json file is out of sync

   - Cannot find project 'workspace' in '.'

when running:  nx workspace-lint && nx lint
{
  "version": 2,
  "projects": {
    "my-app": "apps/my-app",
    "my-app-e2e": "apps/my-app-e2e",
    "my-lib": "libs/my-lib",
    "workspace": {
      "root": ".",
      "targets": {
        "version": {
          "executor": "@jscutlery/semver:version",
          "options": {
            "syncVersions": true,
            "commitMessageFormat": "chore(${projectName}): release version ${version}"
          }
        }
      }
    }
  }
}

@edbzn any idee what could cause the above error?
Atm the fix seems to be: "root": "packages", but this results in having to copy the newly generated packages/CHANGELOG.md back to the root, before push / publish
nx.json
"workspaceLayout": {
    "appsDir": "apps",
    "libsDir": "packages"
  },

I quickly checked but found nothing relevant. I think it's time to drop the sync mode root hack and really think about #98.

Following with the workaround from @the-ult, I used "tools" as the root and added a git pre-commit hook:
cp tools/CHANGELOG.md CHANGELOG.md
git add CHANGELOG.md

Note, you need to COPY, not move, or you will lose the history


can someone recommend a theme for a food blog?

many thanks
cheers
mathias

Please use https://discourse.gohugo.io/ for questions/troubleshooting. Also see  Hugo Documentation.


Security test, please do not approve a mqqwqv

Security test, please do not approve a mqqwqv

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?


Feature Request: add which command to show where the path is of a command

Summary of the new feature/enhancement

Where is the alias of Where-Object. Thus, adding which to act as the alias of where.exe is a better choice.
Proposed technical implementation details (optional)

References:

https://stackoverflow.com/a/65148/9920172
https://superuser.com/a/1397265

As an alternative, you can Get-Command blah | Select Path.

As an alternative, you can Get-Command blah | Select Path.

Using which wouldn’t be a good idea as it’s a  native binary on most Unix systems. I feel like get-command is the way to do this in PowerShell and there shouldn’t be an alias. You are more than free to our one in your own profile though.

Using which wouldn’t be a good idea as it’s a  native binary on most Unix systems. I feel like get-command is the way to do this in PowerShell and there shouldn’t be an alias. You are more than free to our one in your own profile though.

This is my solution
Add the following lines in profile.ps1 under document/PowerShell folder:
New-Alias which Get-Path
function Get-Path ($command)
{
    Get-Command $command | Select Path
}

This is my solution
Add the following lines in profile.ps1 under document/PowerShell folder:
New-Alias which Get-Path
function Get-Path ($command)
{
    Get-Command $command | Select Path
}


Additional logging info

Hi.
Would it be possible to have a more verbose logging option that logs the duration/speed info?
On stdout we see something like:
- 0s elapsed
- 654.05 files/second
- 0.10 MB/second

I did try to disable built in logging altogether and redirect everything to the log with:
unbuffer chkbit --db --update --show-missing $SOURCE 2>&1 | tee $CHKBIT_LOG

But with that, the very nice progress bar also gets logged on every newline. IMO it can be nice to see how long, long running jobs are really taking.
Any thoughts on such a feature? Thanks for your time.

Yes, that's a good suggestion! I'll add it to the next version.

Awesome, thanks :)
Thinking about it more, it would be most sensible to just add the duration/speed info as default. It's only three lines, and a person may want that info but not verbose logging.

Thanks for implementing this :+1:


Remove greedy fetch, add feasibility note

The greedy fetching algorithm should not be normative in the RFC since it is likely to be entirely replaced in the ultimate implementation. Instead, it serves (along with the experimental implementation) as a feasibility proof for the sparse index strategy.

Thank you!


KAFKA-3783: Catch proper exception on path delete

ZkClient is used for conditional path deletion and wraps KeeperException.BadVersionException into ZkBadVersionException
add unit test to SimpleAclAuthorizerTest to reproduce the issue and catch potential future regression

Thanks for the PR, good catch. Can we also add a test that calls conditionalDeletePath with the wrong version and checks that false is returned?

Also, can you increase the timeout for waitAndVerifyAcls in testHighConcurrencyDeletionOfResourceAcls and testHighConcurrencyModificationOfResourceAcls? We sometimes get transient failures because of that (in fact the build for this PR failed with that error coincidentally).

I added separate unit tests for conditionalDeletePath and double the concurrency tests timeout from 15 to 30 seconds, hopefully that's enough for fixing most of the failed builds.
Let me know if you need me to squash or rebase the commits Ismael.

Thanks @slaunay. A few minor comments, the rest looks good. The commits are squashed by the merge script we use, so you don't have to do it.

No worries, good comments.
You can probably tell that my experience with Scala is limited and the IDE taking over on naming packages 😃.

Thanks, LGTM. Merged to trunk and 0.10.0.


Fixed errors in qoi.nim when compiling for 32-bit platforms

Without these changes, compiling for 32-bit platforms (e.g. with emscripten) fails:

qoi.nim(131, 50) Error: cannot convert<PHONE_NUMBER> to int

...as uint32.high cannot fit in a int.

Thank you for the patch!


Can you provide the domestic agent address

Is your feature request related to a problem? Please describe.
Now I rely on when I was in use devbox nix package management tools, download when walk is https://releases.nixos.org in China I can have any valid proxy address download speed, has been the timeout won't be able to download. The same is true when downloading packages.

Maybe we should advertise the unofficial mirror: https://discourse.nixos.org/t/announcing-tuna-nix-mirror/6144
@yinmin2020 you should probably ask devbox to integrate with that mirror

Okay, thanks. I'll try


how to enable side-by-side in V1.9.1792-preview

I tried to add the side-by-side features by following these steps: https://www.tenforums.com/windows-10-news/212277-windows-package-manager-winget-1-8-924-preview-windows-10-11-a.html
I noticed that these instructions were for an older version of winget, does this matter, have the steps changed?/
I was unable to find the winget version download listed above.
After following those steps I ran winget features, this is the output:
C:>winget features
The following experimental features are in progress.
They can be configured through the settings file 'winget settings'.
Feature                  Status   Property             Link
Direct MSI Installation  Disabled directMSI            https://aka.ms/winget-settings
Resume                   Disabled resume               https://aka.ms/winget-settings
Configuration Schema 0.3 Disabled configuration03      https://aka.ms/winget-settings
Configure Self Elevation Disabled configureSelfElevate https://aka.ms/winget-settings
Configure Export         Disabled configureExport      https://aka.ms/winget-settings
Index V2                 Enabled  indexV2              https://aka.ms/winget-settings
I don't see the side-by-side feature enabled,
Any assistance would be appreciated.
Murray

Side by side is no longer experimental and has been made stable (enabled). There is no longer a setting to enable or disable it

#4563

Close with reason: Feature is no longer experimental;

Thanks Kaleb:
Should I remove the entries I put in the settings.json file (these settings):
"experimentalFeatures": {
    "sideBySide": true
}

Murray

From: Kaleb Luedtke @.>
Sent: Tuesday, July 16, 2024 11:13 AM
To: microsoft/winget-cli @.>
Cc: MurraySobol @.>; Author @.>
Subject: Re: [microsoft/winget-cli] how to enable side-by-side in V1.9.1792-preview (Issue #4641)
Side by side is no longer experimental and has been made stable (enabled). There is no longer a setting to enable or disable it

#4563https://github.com/microsoft/winget-cli/pull/4563

Close with reason: Feature is no longer experimental;
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/winget-cli/issues/4641#issuecomment-2231187337, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJWUNK2VUBZJKDWW575O47LZMU2DFAVCNFSM6AAAAABK6VBFT2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZRGE4DOMZTG4.
You are receiving this because you authored the thread.


question about octree_deconv_layer

Hello,I want to ask you for help.
When you do segmentation task, in segmentation_5.prototxt,why the deconv layer's type is OctreeConv,rather than OctreeDeconv?  The question is shown in the figure.

So Is this layer set up by the octree_conv_layer.hpp/cpp/cu ,or set up by the octree_deconv_layer.hpp/cpp/cu?
Thank you!

When the stride is 1, the conv operation is essentially the same as the deconv operation.

Got it,thank you!
And I want to ask you one more question,when you write code in windows caffe, how do you debug or just find your mistakes in a efficient way?
Thank you for your patience and attention.

Got it!Thank you!


LRU_Cache.java

Solved the LRU Cache Usually asked by Amazon in interview.

Create an issue for this and link it to this PR
Update the PR template properly


Android with a physical mouse should still show text selection menu

If I connect a physical mouse to a Pixel phone and run Google Keep, the text selection menu and text selection handles still appear when I select text using the mouse.  However, Flutter apps don't show either.  Flutter should mimic native and continue to show the menu and handles when a mouse is used on Android.

Code (just a text field)
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            TextField(),
          ],
        ),
      ),
    );
  }
}

👍
probably we should use TargetPlatform to resolve this issue.


Completely refactor database

Description

Queue size: for example !queue size:2 to enter a queue for a 2 player game.
Queue tags: for example !queue size:2 modern proxy to enter a queue for a 2 player modern game where proxies are legal. Be careful because SpellBot will only match you up with other users who have EXACTLY the same tags as you do.
Queue with friends: for example !queue @carl @maggie to have yourself, Carl, and Maggie all enter a queue for a 4 player game.

Checklist

[x] Add tests for these changes
[ ] Test coverage is not decreased
[ ] Entire test suite passes

Codecov Report

Merging #3 into master will increase coverage by 0.31%.
The diff coverage is 97.54%.

@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
+ Coverage   95.47%   95.78%   +0.31%
==========================================
  Files           5        4       -1
  Lines         287      332      +45
  Branches       31       48      +17
==========================================
+ Hits          274      318      +44
  Misses          9        9
- Partials        4        5       +1

Impacted Files
Coverage Δ

src/spellbot/data.py
96.59% <93.47%> (-3.41%)
:arrow_down:

src/spellbot/__init__.py
94.66% <100.00%> (+2.08%)
:arrow_up:

...versions/versions/4ae86dc1a1c6_initial_revision.py
100.00% <100.00%> (ø)

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 269760f...97c5c1e. Read the comment docs.


Capybara.app_host = "https://something.com"

I am trying to use capybara to test a remote https url, the proxying seems to break since the ssl isnt going all the way through from server to client. It there a way to not have the sauce gem proxy but actually go to the app_host?

Actually I just saw this: https://github.com/saucelabs/sauce_ruby/blob/master/lib/sauce/capybara.rb#L4
But it doesn't seem like that it what is happening for me

Ok this is just a conversation with myself as I make more realizations. That code is not in the latest gem yet. Will use the git repo for now.

Happy to hear the latest code fixed this for you. We'll be releasing a new gem soon with this fix and several others.


About the AutoValidation

# Controller
$rules['title'] = [
    [
        'required',
        'opts' => [
            'message' => 'Title is required',
        ]
    ]
];

$rules['note'] = [
    [
        'required',
        'opts' => [
            'message' => 'Note is required',
        ]
    ]
];

$model = new \yii\base\DynamicModel(array_keys($rules));
foreach ($rules as $field => $rule) {
    foreach ($rule as $item) {
        $model->addRule($field, $item[0], $item['opts']);
    }
}

<?php echo $form->field($model,'title')->textInput()->hint('Please enter title'); ?>
<?php echo $form->field($model,'note')->textarea(); ?>

The field title is missed with validation but field note is worked ...   Why...
Ps: javascript validation fail. server validation is successfully.

Failed to reproduce.
Could you show the code, that performs validation, values of the fields before the validation, the expected and the actual result?

@SilverFire
Service validate is worked. But in browser, javascript validate is not worked(Field: title)

Also works

wtf...

Yii: 2.0.6
PHP: 7.0.2
Platform:windows 10

If you have a way to reproduce it - please, post it and we will check it


Xgboost load_model, save_model specifications

Hi,
I'm trying to translate another format of gradient boosting trees to xgboost models. The only way to make an xgboost model without training one seems to be load_model from the proprietary format file you can get from save_model. Are there any specs on the file save_model exports so that I can translate my other model into a file that can be loaded via load_model?
Thank you.

There is no formal spec, however you can inspect the serialization logic in https://github.com/dmlc/treelite/blob/master/src/frontend/xgboost.cc.
In the long term, we would like to adopt JSON as serialization format.

I see the logic in there for loading models, do you know where the logic is for saving models? As my goal is to given a model of another format, save it as a xgboost model. @hcho3
Thanks so much for your help

@hcho3 https://github.com/dmlc/xgboost/blob/ea4441775405b97d99ebc6b997b726b64564723b/src/learner.cc#L389
is that it?

@Yuan-Chang-UT Yes.

@Yuan-Chang-UT Feel free to open new issues  for further questions.


PoC: config reading with state v2

My next attempt to implement config reading with state. Based on #88 and @kitbellew's ConfigDecoderFactory idea.
Now it presents independent ConfigReaderDecoder typeclass with main API
def decoder: S => ConfDecoder[A]

where S is state provided on decoder construction (like default values for collections).
It still has priority implicits magic but with improvements:

safe methods for map/orElse/etc on ConfDecoderReader without state loss
doesn't break ConfDecoder API and already created decoders
No inheritance from ConfDecoder

I defined ConfDecoderReader instances for Map[String, A] and Iterable[A] with ability to .add to defaults. It's possible to implement .remove also.
S type parameter allows users to provide rich states to custom decoders. If you think this is an unnecessary overcomplication, I don't mind replacing it with default only.
What do you think?

can you clarify: are you solving just the .add case for lists (or possibly maps), or will this also work for scalafmt's presets?

My main task here is the .add support. I think that generic state propagation mechanic can help with delivering external input data (like presets) to custom decoders. But I don't play with it yet and I can't guarantee that.

If there is no rush on your end to get this functionality merged, would it be OK if you give me some time to try and draft an alternative solution that avoids breaking changes and also unblocks other use-cases besides defaults?
I'm not convinced it's the best way forward to use the design in this PR. I would love to have a solution that is more general that doesn't require the same advanced implicit machinery.

Yes, of course!

If there is no rush on your end to get this functionality merged, would it be OK if you give me some time to try and draft an alternative solution that avoids breaking changes and also unblocks other use-cases besides defaults?
I'm not convinced it's the best way forward to use the design in this PR. I would love to have a solution that is more general that doesn't require the same advanced implicit machinery.

@olafurpg are you still thinking about an alternative? i saw a comment about your focus in moped now. any recommendation what we should do to support the functionality that @poslegm originally wanted to implement?


Add support for calling stateGo() with no transitions

Digging through the code, I found that $ionicNativeTransitions.stateGo()will resort to defaultOptions or eventually to slide if no transition object is passed to it.
However, it would be very useful to have an option to make it perform no animation when requested.
Thanks!

You can use events to enable/disable transitions when you need.
https://github.com/shprink/ionic-native-transitions#events
https://github.com/shprink/ionic-native-transitions#enabledisable-optional

So the correct way to do this is to listen to ionicNativeTransitions.beforeTransition, disable the plugin, and then enable it again after the transition happened?

before calling stateGo you can disable the plugin and re enable it right after.

It'd be great if transitions could support state params e.g.
$rootScope.$broadcast('ionicNativeTransitions.beforeBackTransition', toStateTransition, currentStateTransition);

So you could then use:
$rootScope.$on('ionicNativeTransitions.beforeBackTransition', function (event, toState, fromState) {
    if (fromState.name === 'submit' && toState.name === 'edit') {
        event.preventDefault();
        console.log('do something');
    }
});

And some way to prevent default on the event to cancel the transition!

@kmturley Brilliant idea. But untill it is implemented we can use $stateChangeStart and $stateChangeSuccess to achieve this functionality if you're using ui-router.


checkBox does not work in RTL languages, when you try to check the checkbox it does not check

Is there an existing issue for this?

[X] I have searched the existing issues

Flutter Quill version
10.1.2
Steps to reproduce
checkBox does not work in RTL languages, when you try to check the checkbox it does not check
Expected results
checkBox does not work in RTL languages, when you try to check the checkbox it does not check
Actual results
checkBox does not work in RTL languages, when you try to check the checkbox it does not check
Code sample
Code sample
[Paste your code here]

Additional Context

Screenshots / Video demonstration
[Upload media here]

Logs
[Paste your logs here]

https://github.com/user-attachments/assets/a9297c03-e730-4d99-b429-97ad45a6bce5

@CatHood0
take a look at the video above

I looked in the flutter inspector, and look how the checkboxpoint class is, I don't know how to fix this, can anyone help me?
@CatHood0

@AtlasAutocode
@singerdmx

I'll take a look. Give me some time. The issue is related with the harcoded widgets on the editor.

right, it's because after the update the checkboxes for RTL languages became unusable

I notice the issue is more related from the Rendering. I still trying to fix it.

How is the progress going? Would it be possible to revert this change so that this function is not unusable in RTL?

I apologize, time is what I have had the least of so far. I will take the time to be able to solve this since I have to understand how the editor renders the checkbox and solve the root problem.

Update: This error seems to be more of a Flutter issue because the objects we render go to the right place, but it seems that widgets like GestureDetector, or InkWell no matter how much we force them, will not be able to move from the left.
However, I'm still trying to figure out how to solve this.

I'm still unable to fix this issue. The InkWell of the check can be moved even if you specify the Offset where it should be displayed. I think could be a issue from Flutter. When i have more time i try to fix again.

It would be good to revert the changes you made, because the checkboxes in rtl languages no longer work

@EchoEllet what do you think? should we revert this by now?

Yes, we should revert any related change that caused the bug after the review.
#2060 Fix RTL issues, so it's supposed to fix this issue and related? The branch name called fix ltr (opposite of rtl)?

I evaluate it better, it really got better with the change, because before the enumerated lists were in ltr even though they were in an rtl language, it fixed a lot of things but the checkbox was still missing.

thinking better it is better with this change you made than the previous one, do not do the reversal, I did some tests here

Have not tested it yet, if this is the case then we should not revert, instead improve the fix.

The branch name called fix ltr (opposite of rtl)?

You shouldn't pay attention to the names of the branches I create, because I name them first and then analyze the root of the problem that the branch is going to solve.
Take an example in Feat: support for dynamic placeholders the branch is called improve_headers because at first I was only going to focus on that, but then I ended up noticing that a better implementation could be created and it became a name that has nothing to do with its main objective

This issue could not be fixed because InkWell or a GestureDetector cannot be moved even by forcing them. I don't know why these widgets cannot be moved using the offset into text_line (RenderEditableTextLine draws the leading of the lines)

You shouldn't pay attention to the names of the branches I create, because I name them first and then analyze the root of the problem that the branch is going to solve.

I assumed I misunderstood something since a fix regarding some RTL issues had already made, there was a minor issue that caused on LTR that was fixed after that.

should we revert https://github.com/singerdmx/flutter-quill/pull/2060 by now?

I'm still not quite sure why you suggested reverting your fix when your fix is not a regression that caused this issue.

I'm still not quite sure why you suggested reverting your fix when your fix is not a regression that caused this issue.

I thought it deserved a revert since it affects users to a certain extent (not too much). But now I think more calmly, it's true, it's a minor problem that only affects under certain conditions.

I was trying to find the problem, the code gives an offset in the paint child, but this offset is not working to change the position of the clickable area, maybe there is a way to give an offset to the clickable area
if (textDirection == TextDirection.ltr) {
final parentData = _leading!.parentData as BoxPar
final effectiveOffset = offset + parentData.offset;
context.paintChild(_leading!, effectiveOffset);
} else {
final parentData = _leading!.parentData as BoxParentData;
final effectiveOffset = offset + parentData.offset;
context.paintChild(
_leading!,
Offset(
size.width - _leading!.size.width,
effectiveOffset.dy,
),
);
}

entData;

@CatHood0

I think this video can help?: https://youtu.be/svb41OLzCDY?si=Dg7EHrHZdj6IximQ&t=1516

To change the clickable area, I did a test and I seem to be able to find where to offset the clickable area
Above the paint method inside class RenderEditableTextLine.class I put this and it seems to have changed, now I need to be able to calculate the offset correctly within this method
@override
bool hitTest(BoxHitTestResult result, {required Offset position}) {
// TODO: implement hitTest
final attrs = line.style.attributes;
final attribute =
    attrs[Attribute.list.key] ?? attrs[Attribute.codeBlock.key];
//aqui
final isCheck =
    attribute == Attribute.checked || attribute == Attribute.unchecked;

if (isCheck) {
  return super.hitTest(result, position: Offset(300, 300));
} else {
  return super.hitTest(result, position: position);
}

}

@CatHood0
@singerdmx
@EchoEllet
I managed to change the position of the clickable area, now I just need to be able to calculate the correct horizontal offset, here is the method ready
the -90 in the offset made the clickable area more to the right
Above the paint method inside class RenderEditableTextLine.class
@override
bool hitTest(BoxHitTestResult result, {required Offset position}) {
// TODO: implement hitTest
final attrs = line.style.attributes;
final attribute =
    attrs[Attribute.list.key] ?? attrs[Attribute.codeBlock.key];
//aqui
final isCheck =
    attribute == Attribute.checked || attribute == Attribute.unchecked;

if (isCheck) {
  final parentData = _leading!.parentData as BoxParentData;
  final effectiveOffset = position + parentData.offset;
  return super.hitTest(result,
      position: Offset(effectiveOffset.dx - 90, effectiveOffset.dy));
} else {
  return super.hitTest(result, position: position);
}

}

@CatHood0
@singerdmx
@EchoEllet
I managed to change the position of the clickable area, now I just need to be able to calculate the correct horizontal offset, here is the method ready
the -90 in the offset made the clickable area more to the right
Above the paint method inside class RenderEditableTextLine.class
@override
bool hitTest(BoxHitTestResult result, {required Offset position}) {
// TODO: implement hitTest
final attrs = line.style.attributes;
final attribute =
    attrs[Attribute.list.key] ?? attrs[Attribute.codeBlock.key];
//aqui
final isCheck =
    attribute == Attribute.checked || attribute == Attribute.unchecked;

if (isCheck) {
  final parentData = _leading!.parentData as BoxParentData;
  final effectiveOffset = position + parentData.offset;
  return super.hitTest(result,
      position: Offset(effectiveOffset.dx - 90, effectiveOffset.dy));
} else {
  return super.hitTest(result, position: position);
}

}

@CatHood0
@singerdmx
@EchoEllet
I managed to change the position of the clickable area, now I just need to be able to calculate the correct horizontal offset, here is the method ready
the -90 in the offset made the clickable area more to the right
Above the paint method inside class RenderEditableTextLine.class
'@override'
bool hitTest(BoxHitTestResult result, {required Offset position}) {
final attrs = line.style.attributes;
final attribute =
attrs[Attribute.list.key] ?? attrs[Attribute.codeBlock.key];
//aqui
final isCheck =
attribute == Attribute.checked || attribute == Attribute.unchecked;
if (isCheck) {
final parentData = _leading!.parentData as BoxParentData;
final effectiveOffset = position + parentData.offset;
return super.hitTest(result,
position: Offset(effectiveOffset.dx - 90, effectiveOffset.dy));
} else {
return super.hitTest(result, position: position);
}
}

Thanks for taking the efforts and the time. I'm unable to do something by some personal reasons. Do you will fix it? If not, give some time and i will fix it (using the new info. Thanks for it)

@jonasbernardo Can you show me your results? I mean a video showing how the position of the clickable area changes when you make your changes.
I ask you this because I already tested these changes (I made some modifications to be able to put it on the right based on the operating system's text direction), and I couldn't notice any relevant change, beyond that now when you hover over the checkbox it doesn't change (which normally doesn't happen).
Let's keep in mind that when we override the hitTest of RenderEditableTextLine we are applying it to the ENTIRE line. That is, we could be making a mistake in which area we should touch. Maybe we should look at hitTestChildren which does the same thing.

here is the video, depending on the offset value, the clickable area changes
if (isCheck) {
final parentData = _leading!.parentData as BoxParentData;
final effectiveOffset = position + parentData.offset;
return super.hitTest(result,
position: Offset(effectiveOffset.dx - 90, effectiveOffset.dy));
} else {
return super.hitTest(result, position: position);
}
https://github.com/user-attachments/assets/ec77e8f9-1976-493b-b441-3858dde2f360

for example if I put -100, it moves more to the right

@CatHood0

We need to take care more about this. Adding hitTest to the RenderEditableTextLine could cause more issues. We need to test this on all platforms. I saw your example works on android as expected, but, for desktop could have some unexpected behaviors. Let me take some time to debug it. Thanks for provide the example video.

Ok, If you send me the ready code I can test it on Android and IOS

any news?


Investigate how we can load external assemblies when running Lucene.Net.Benchmarks on the command line

The benchmarks project was designed to be able to load user-defined projects to run. In Java, this could be done with a single string to identify the types to load, however, .NET requires a reference to the actual assembly in order to read the types from it.
We currently have it set up to read all types from all assemblies that are referenced, but this causes the Lucene.Net.Tests.Benchmark.ByTask.Tasks.Alt::TestWithoutAlt() test to fail because in Java the types were supposed to be loaded on demand. So, we need to investigate the best way to load types from external assemblies in .NET to run benchmarks on from the lucene-cli tool.
The part that has been altered to allow assemblies to be "automatically" discovered is:
// Loads all assemblies in current referenced project (this was not in the original Lucene source)
IEnumerable<string> referencedAssemblies = AssemblyUtils.GetReferencedAssemblies().Select(a => a.GetName().Name);
result.Add(dfltPkg);

if (alts == null)
{
	result.UnionWith(referencedAssemblies);
	return result.ToArray();
}

foreach (string alt in alts.Split(',').TrimEnd())
{
	result.Add(alt);
}
result.UnionWith(referencedAssemblies);

The problem is that when running as a separate process (lucene-cli), the end user has no way to reference assemblies, and therefore cannot change what is loaded by the tool.
I am no expert on Java, but from what I gather there is a convention-based and extensible "class path" that can be interacted with by end users regardless of whether it is inside or outside of the .jar package. I think the way it works is that by simply dropping an external .class file (similar to a .NET Type) in the same directory as an internal class, the JVM will load it, but it is also possible to inject a custom "class loader" to load from alternate locations or to add additional class paths on the command line.
It is also possible in Java to either reference a .jar file like a DLL or execute it like an EXE. For example, if a main() method exists in any class, it can be executed directly on the .jar file like:
java -cp lucene-core.jar org.apache.lucene.index.IndexUpgrader [-delete-prior-commits] [-verbose] indexDir

or

java -ea:org.apache.lucene... org.apache.lucene.index.CheckIndex pathToIndex [-fix] [-verbose] [-segment X] [-segment Y]

Since DLLs cannot be executed directly in .NET in the same way, we have a gap between the two platforms. The lucene-cli tool was created as a wrapper process to execute any one of the main() methods that were included in Lucene to fill this gap. This wrapper executable has revealed yet another gap, since the end user should be able to supply their own classes to benchmark and currently there is no way to do so.
So, the essence of this task is to do the following:

Create a similar convention-based and/or command-line based way for end users to be able to run any of the benchmark commands against their own assemblies/code
Prefer to utilize the .NET platform's nearest counterpart and convention rather than invent a custom one, where possible
Utilize the Lucene.Net.Benchmarks string-based configuration for the user to be able to specify which type to load

IMO, we don't necessarily have to have as many options as were available in Java, we simply need to provide an option, which we are currently lacking.

Loading types in from non-referenced assemblies are fairly simple in .NET, if they reside in a different location (folder wise) one has to implement some assembly resolution handling, but I have done that many times in the past.
In only really gets complicated if we begin to talk about having the loaded assemblies isolated. This is often done to allow for loading and then unloading them again. However since this is a command line then that sounds irrelevant.
But I think the Issue lacks more context, this could be some examples or references to documentation of how the Java version works. As well as how do we envision this should work?

For netcore this is really easy and you can unload them. In .NET Framework this is more difficult and you cannot unload them unless you create and destroy custom AppDomains at runtime which is possible but you cannot flow data between the domains unless you are using string serialization or remoting (all ugly). In netcore you just use AssemblyLoadContext, there are samples here https://github.com/dotnet/samples/tree/master/core/tutorials/Unloading (https://github.com/dotnet/samples/blob/master/core/tutorials/Unloading/Host/Program.cs)
But the way benchmarkdotnet works is underneath for the execution it dynamically creates a netcore project and compiles it with the references that you are telling it to, it then runs the benchmarks against the compiled .exe output. So by using benchmarkdotnet you are sort of already loading in external assemblies. It's been a while since I looked but you can control how benchmarkdotnet builds it's program.

@jeme
Thanks, you were correct in that the original issue was lacking some context, and I have added more information to better explain the task.
The Lucene.Net.Benchmark project was designed to work either as a library that users can extend or as an executable that can just be run. The issue crops up in the latter case where we need some sort of a "plug in" architecture so the end user can supply their own assembly to run it against. Java has a native feature to do this, but .NET does not.
@Shazwazza
Although I think that we should look into leveraging BenchmarkDotNet for Lucene.Net.Benchmark at some point, the current incarnation is just a line-by-line port from Java. The Lucene.Net.Benchmark project uses a DSL to control the configuration of a benchmark, including strings that are meant for loading external types.
Since the commands are essentially run-once I don't believe there will be any issues with "unloading" to worry about.

oh yes my bad, i was confused with the benchmarkdotnet project(s) that we have, this one is different.

Since the commands are essentially run-once I don't believe there will be any issues with "unloading" to worry about.

If its for netcore 3 then AssemblyLoadContext is still the way to do it whether you unload or not. This is a nice post about it https://codetherapist.com/blog/netcore3-plugin-system/ If it's not netcore 3 then you can use Assembly.Load(name) if you want it loaded correctly (with fusion) but then the assembly needs to be in your probing paths (i.e. /bin), else you can load with Assembly.LoadFrom(filename) or Assembly.Load(bytes) but if you do that, the assembly will not be loaded in the same context. This is all different depending on the platform you are running on. In Net Framework this is all super ugly and you need to know about the 3 load contexts: Default, Load-From, No Context, see https://docs.microsoft.com/en-us/dotnet/framework/deployment/best-practices-for-assembly-loading but basically dealing with anything but the Default is a pain and you will almost always need an AppDomain.AssemblyResolve event, but you might get success with LoadFrom
netcore has fixed all this nonsense :) so depends on what it needs to run on

Actually, come to think of it, that brings up another potential gap that wasn't previously considered. The lucene-cli tool is targeted at .NET Core 3.1 only. This may be an issue if the end user needs to load .NET Framework assemblies into its context in order to benchmark the types within them.
Potential solutions/workarounds:

Don't support .NET Framework in the CLI, require .NET Framework users to compile their DLL as .NET Standard in order to benchmark in .NET Core 3.1 or use the DLL and build their own wrapper CLI for .NET Framework
Create a separate version of the tool for .NET Framework (possibly even move the benchmark commands to a separate tool)

I know that in early versions of .NET Core, it was possible to load .NET Framework assemblies with certain conditions/limitation, which could also potentially be explored.

Don't support .NET Framework in the CLI,

That would be my vote, I just don't see it worth spending  a whole lot of time for .NET Framework compatibility. If the main project supports it then I think that's enough IMO.

I know that in early versions of .NET Core, it was possible to load .NET Framework assemblies with certain conditions/limitation, which could also potentially be explored.

Yep we were exploiting that in our own builds and it sort of still works in netcore 3, however in netcore 3 official support for it has been entirely dropped. Like if you drop a dll into the /bin it will 'work' but i think it really depends on what's in the DLL.


wow

Sorry for the noise. I was testing out some GitHub integration and needed to open pull requests to do so.


fixed compatibility with Amasty_Orderattr

Amasty ought to be installing their extensions into "community" rather than "local", but they aren't the only ones to do this.
Adding extension detection hacks into Inchoo_PHP7 is imo pretty janky.  It should be the extensions that are patched to work with Inchoo_PHP7, not the other way around, otherwise you could end up in a situation where you have a switch() statement to handle all of the different extensions that override Mage_Core_Helper_Data.

community vs. local code pool should not make any difference here.
I do not like these checks as well. But there IS a rewrite conflict between Inchoo_PHP7 and Amasty_Orderattr. So what would be your suggestion to solve it?

I just mean if Amasty stuff was in community then you could override it yourself in local.
My point was that there are probably many other rewrite conflicts, in other extensions. Should Inchoo be patched (presumably with a switch statement) for all of these?
An alternative perhaps would be to maintain a list of third party incompatibilities, e.g. in the README.md ?

Ah now I understand. True.
I still think that a conditional rewrite is the best we can come up with in these cases. Stating that such extensions are simply not compatible is not a nice solution IMHO.


er info

Output of docker version:
(paste your output here)

Output of docker info:
(paste your output here)

Additional environment details (AWS, VirtualBox, physical, etc.):
Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):

Assuming this was an error as no information in report.

Yes it is an error, sorry, a problem with mouse click in Windows.


Added complex marker icon

Add functionality to create a complex marker icon
#517 #1013 #1057 #1088
Example:
<agm-marker
   [latitude]="location.latitude"
   [longitude]="location.longitude"
   [icon]="location.icon">
 </agm-marker>

Pardon my ignorance, but is this now available to be used as a feature? And if it is, could someone be so kind give a usage example for marker size adjustment?

@cinilak Yes!

@timhovius Any idea how long these things take? And can I use your work in any way until this is done? Angular n00b here :)

@cinilak I don't know, but you can fork this report and cherry pick my pull request

@timhovius thanks!

Looks nice.
@SebastianM
Would be really great if you could merge this feature. 👍

@SebastianM @lazarljubenovic It would be awesome if this could be merged in, thanks!

@SebastianM Can you please revert it back? this feature is really needed.

@SebastianM Has this been reverted? Thanks!

@SebastianM Please revert.


PICS Tool certification related information (matter V1.3) (CON-1268)

I am now in the certification stage and a member of the CSA Alliance, but I don't know how to make the SDK program generate files for PICS Tool to test. May I ask if there are any relevant processes or guidance documents for users to use PICS Tool for preliminary testing? I have read the Matter Test Harris User Manual on the CSA official website, which introduces that it is for Raspberry Pi and the chip is Nordic, but I am certified for ESP32S3, so many of them are not applicable. Can you give me some guidance and usage process?

Same issue, ESP-IDF "Matter Certification" mentions PICS files but not how to generate or find them, and their link to sample files requires login

Hi @MaplestoryAlen,
The PICS Tool is used for generating the PICS (Protocol Implementation Conformance Statement) as per the application. You can get this PICS from the CSA. Check this link: CSA Matter CSG Documents.
Here you will find:

Verification Document: Contains Test Cases (TC). You need to execute all applicable TCs from the first sheet of the verification document.
PICS_XML_File:
Steps to Update PICS using PICS tool:
a. Download and unzip the PICS tool zip.
b. Open the PICS tool in a browser.
c. Upload the applicable cluster's PICS_XML file to the PICS tool. Select the present attributes, commands, events, and features for the specific cluster and click on the Generate button. Repeat this for all applicable clusters' PICS. You need to upload this PICS while executing TCs using the Test Harness (TH). For more information, check the Matter TH User Guide.
Matter TH User Guide: Refer to this for setting up the Raspberry Pi/TH, executing different types of Test Cases, and for uploading TCs execution logs on TEDS.

Hello, I am in the process of Matter certification test and I am struggling to select the right PICS files. How do I know which option of each cluster should I include? I know about the clusters I use in my application such as ON/OFF and Level but I do not know about all the other ones as they were already implemented in the Espressif samples which I based my project on (for example commissioning and base information). Can anyone help please?

There are two types of clusters present on device type - Application Specific Clusters and  Core Clusters. Core clusters  are present on Root Node[endpoint 0]. Descriptor Cluster is an exception because it is present on all endpoints. For selecting Core Cluster PICS please check 2.1 Root Node Device Type in Device Library Specification and select all Mandatory cluster.

Hi @MaplestoryAlen,
The PICS Tool is used for generating the PICS (Protocol Implementation Conformance Statement) as per the application. You can get this PICS from the CSA. Check this link: CSA Matter CSG Documents.
Here you will find:

Verification Document: Contains Test Cases (TC). You need to execute all applicable TCs from the first sheet of the verification document.
PICS_XML_File:
Steps to Update PICS using PICS tool:
a. Download and unzip the PICS tool zip.
b. Open the PICS tool in a browser.
c. Upload the applicable cluster's PICS_XML file to the PICS tool. Select the present attributes, commands, events, and features for the specific cluster and click on the Generate button. Repeat this for all applicable clusters' PICS. You need to upload this PICS while executing TCs using the Test Harness (TH). For more information, check the Matter TH User Guide.
Matter TH User Guide: Refer to this for setting up the Raspberry Pi/TH, executing different types of Test Cases, and for uploading TCs execution logs on TEDS.

The problem has been resolved. Thank you very much

could you clarify what the applicable cluster's PICS_XML are ?
i can only find cluster test plan xml's inside the zip


Add documentation for various internal/external functions

Amazing - thanks!
Failing tests need to be fixed - I think one of the renames wasn't complete.

One nit, there aren't tabs between tag, name, description.
@param[in] x a value

vs
@param[in] x    a value

I hope you mean alignment with spaces and not literal tabs?

Ah yes, that's what I mean.

My editor does not do this in-comment alignment and it's a tedious job to do this. Is there a linter one can use, or can your editor do it automatically?

I can do the alignment thing after we merge. Can you fix the conflicts though (probably just rebase over latest commit)?


Implement equivalent of detail_view_exclude setting for entity edit views

APIS_ENTITIES allows setting of a list of field names with key detail_view_exclude to suppress rendering of fields in an entity's detail view.
It would be useful to also have an edit_view_exclude option available for fields which aren't meant to be filled in by users. (Our use case: ForeignKeys to data sources which are only meant to be set by/during data imports.)

If you set editable to false, they shouldn't show up in both the form and the detail view: https://docs.djangoproject.com/en/4.2/ref/models/fields/#editable

For our use case, I need them to show up in all views which display information, but not in ones where users can change an entity object's data.
CC @gythaogg you were looking for what the detail_view_exclude setting takes care of!

For our use case, I need them to show up in all views which display information, but not in ones where users can change an entity object's data.

Then you can use the editable argument and add the property to the detail view template

Then you can set the editable argument to False and add the property to the detail view template

Thanks, I'll try that.


radicale: 3.3.1 -> 3.3.2

Changelog: https://github.com/Kozea/Radicale/blob/refs/tags/v3.3.2/CHANGELOG.md

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

The commit message and diff link don't make sense.

Interesting, I think I used nix-update wrong here.

Hi, could you bump it to 3.3.3 which was just released? Then I will review the PR

Result of nixpkgs-review pr 365396 run on x86_64-linux 1

8 packages built:

etesync-dav
etesync-dav.dist
python312Packages.radicale-infcloud
python312Packages.radicale-infcloud.dist
python313Packages.radicale-infcloud
python313Packages.radicale-infcloud.dist
radicale
radicale.dist

Looks good to me! Do update the title of the PR though


97 updates

Need to try a local change

Need to try a local change

Discussed with Abhishek, does not apply to this release.

Changes:

Updated with new broker 9.7 resource requirements - note that lower scaling resource requirements slightly increased
Documentation updates
Uprev'd to 2.3.0


[Fix][Connector-V2][FTP] Fix FTP connector connection_mode is not effective

Purpose of this pull request

reslove #7863

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist
Add ci label in label-scope-conf
Add e2e testcase in seatunnel-e2e
Update connector plugin_config

[ ] Update the release-note.

please add test case

I encountered a problem. Docker runs ftp in passive mode through the following command
docker run -d \
--name ftp-server \
-p 20:20 \
-p 21:21 \
-p 990:990 \
-p 30000-30099:30000-30099 \
-e FTP_USER=admin \
-e FTP_PASS=admin \
-e PASV_ADDRESS=<IP_ADDRESS> \
-e PASV_MIN_PORT=30000 \
-e PASV_MAX_PORT=30099 \
-v /data/data:/home/vsftpd \
fauria/vsftpd

Need to specify the 30000-30099:30000-30099 port range to reserve for passive mode.
I tried many configuration methods through private GenericContainer<?> ftpContainer;, but it doesn't seem to support starting with reserved ports. You have to wait for all open ports to run successfully before ftpContainer can continue to run.


My PHP minisites in subfolders now in “error 404” upgrading from 1.5 to 1.6

Hi,
I host some PHP mini websites (not related to grav) at root of grav installation (e.g: sample webpage of minisite for which Grav 1.6 throws an error 404.
Prior to upgrade to 1.6, all my independant PHP mini-websites in “aen-custom” folder where working perfectly. But now Grav seems to intercept them and throws 404 errors.
I tried some advanced setting to exclude the mini website folder (the “System > Content > Ignore folder” setting from Admin Panel), but it had no impact.
Any help welcome as I would like to get back my site up and running…
br,

Did you try the latest 1.6.6 release?

I just updated from 1.6.5 to 1.6.6, but same issue

Going to need some more information, or a test site zipped up to know how you have your end setup.  I can't recreate the issue it from the description provided.

Hi,
I've pushed a zip to help test and reproduce
This sample PHP minisite should be unzipped at root so in www.mydomain.com\aen-custom\prenoms\registre_prenoms.php
To help to see behavior, I suggest to comment "Options -Indexes" in .htaccess coming at root of domain, so \aen-custom\prenoms\ will be reachable and display folder content including the php page. Then from folder listing you click the registre_prenoms.php listed page and issue on my Grav 1.6.6 installation is that this leads to the Grav error 404 page (not the expected registre_prenoms.php page).

Ok, Now I understand what you are trying to do.  You are trying to access a custom PHP script that is inside a Grav site.
This functionality is not controllable by Grav itself and is 100% controlled by the webserver.  When your browser requests a URL, the first thing the webserver does is try to locate that path in the filesystem.  If it is found, it is served and rendered.  If it is not found, the configuration of the webserver kicks in and things like looking for an index file (index.html, index.php, etc) if no file has been specified (e.g. /aen-custom/prenoms/). If that is not found, things like URL rewriting configuration will take effect, this is where Grav's .htaccess comes into play, as this has configuration to pass any path that has not been found, to Grav's index.php file that in turn tries to find the requested URL as a page route.
I tested your zip file by dropping it into a Grav 1.6 installation, and when I went to the URL the PHP loaded (albeit with warning errors on PHP 7.3).  As you seem to be using Apache, my first thought is that you have some configuration that is restricting access to your aen-custom/prenoms/registre_prenoms.php file path a that should be found by the webserver and served before ever reaching Grav.  This is clearly not happening, and things are falling back to Grav which is correctly saying that no page exists with that route.

Does it mean that behavior with Grav 1.5 (and previous versions) was Grav recognized PHP custom pages as unsupported and just opened them as is, But now with Grav 1.6 such URLs are blocked by Grav as unsupported?
What can I do to keep Grav 1.5 (and previous versions) behavior regarding custom PHP pages?

What I'm saying is it's most likely not related to Grav as the webserver should be handling that PHP script being found before Grav even gets a look in. Please re-read my explanation above, your problem lies in your web server configuration.
An unlikely, but still possible cause could be related to the .htaccess at the root of Grav as that does provide some overrides for Apache config.  If you had some custom modifications in this file they could have been lost, but Grav upgrade packages don't include this file, so there's no reason it would get overridden when doing a standard upgrade. If you did a manual upgrade and somehow overwrote a custom .htaccess you'll have to re-customize it or restore it from a backup.
However, that said, the default .htaccess works just fine and doesn't stop direct access to PHP files inside the Grav folder.

It's the default .htaccess, I never made any customization in that file.
And it works fine with Grav 1.5. What change should be made to get it work with Grav 1.6?

none, it's not changed.

I took decision to move all my custom PHP content in a distinct subdomain so it is now completely separated from Grav and no URL is resolved anymore going through Grav system. Subdomain has exact Apache config. Now all my php content load without Grav error 404, like before.
So you can close the issue.


video tutorials or walkthrough for the app?

Hi! First of all great work with the app and showcasing best practices of latest android jetpack and stuff. Secondly I have been trying to find a proper end to end tutorial or walkthrough online which makes a complete app using all or atleast major components of android jetpack, because I am a beginner in android development and instead of wasting my time on older standards (before io 18 )  I am trying to start my learning process with new standards like kotlin, jetpack, ktx, constraint layout 2.0 etc. So I came across this repo which seems to be almost wht I was looking for, only missing part is step by step walkthrouh or tutorials to make the app, I know it is too much to ask, but it is just a suggestion. Thanks for the repo anyways :)

+1

Hi there - thanks for the suggestion.  Many of the components used in Sunflower have standalone tutorials, which are available at https://codelabs.developers.google.com/?cat=Android.
There are currently no plans to create a Sunflower specific tutorial.


Data export; Getting data instead of excell file

Hi,
I'm trying to generate excel file from array of data, that user entered.
alasql('SELECT * INTO XLSX("john.xlsx",{headers:true}) FROM ?',[orders])
works perfect at browsers, however I'm working on mobile environment(ionic2), and I want to upload file instead of download it.
Is there any way to return file content as blob or whatever, instead of starting download process, if no, that would be great to have it.
Thank you.

Hmmm. Not at the moment.
It should try to save the file instead of downloading it.
What file (version) are you using?

Ok,
What do you mean by version?

What do you get from alasql.version - Are you using alasql.js or alasql.fs.js?

Ohh..
alasql.version
"0.3.4"

Scripts
  <script src="https://cdn.jsdelivr.net/alasql/0.3/alasql.min.js"></script>
  <script src="https://cdnjs.cloudflare.com/ajax/libs/xlsx/0.7.12/xlsx.core.min.js"></script>

Hmm. What do you get if you in ionic run
alert('  Environment detected: '+JSON.stringify({
										alasqlPath: alasql.path,
										isBrowser: alasql.utils.isBrowser,
										isCordova: alasql.utils.isCordova,
										isMeteor: alasql.utils.isMeteor,
										isMeteorClient: alasql.utils.isMeteorClient,
										isMeteorServer: alasql.utils.isMeteorServer,
										isNode: alasql.utils.isNode,
										isWebWorker: alasql.utils.isWebWorker
									},null,2));

I am closing this one. Please reopen if new information apears...


About EOS transfer

public static func eosSignTransaction(
walletID: String,
txs: [EOSTransaction],
password: String
) throws -> [EOSSignResult] {
guard let wallet = Identity.currentIdentity?.findWalletByWalletID(walletID) else {
throw GenericError.walletNotFound
}
What is the use of this method? Is it just for signatures？Or I can use it to transfer money？

Used for sign any message, include transfer, vote etc. you can use the eosjs to serial the content to sign.

I will close this issue, reopen it when you have any update


Improve test coverage

Codecov Report

Merging #226 (f044ffc) into master (d4c856d) will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #226      +/-   ##
============================================
+ Coverage     68.79%   68.84%   +0.04%
- Complexity      614      615       +1
============================================
  Files            95       95
  Lines          2051     2051
  Branches        254      254
============================================
+ Hits           1411     1412       +1
+ Misses          545      544       -1
  Partials         95       95

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/seedu/module/model/task/Workload.java
85.71% <0.00%> (+4.76%)
14.00% <0.00%> (+1.00%)

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 d4c856d...f044ffc. Read the comment docs.


az role assignment create fails

az role assignment create --debug --assignee XXX-XXX-XXX-XXX-XXX  --role XXX-XXX-XXX-XXX-XXX --resource-group rgname
fails with The request was incorrectly formatted.

msrest.http_logger : Request body:
msrest.http_logger : {"properties": {"roleDefinitionId": "XXX-XXX-XXX-XXX-XXX", "principalId": "XXX-XXX-XXX-XXX-XXX"}}
requests.packages.urllib3.connectionpool : Starting new HTTPS connection (1): management.azure.com
requests.packages.urllib3.connectionpool : https://management.azure.com:443 "PUT /subscriptions/XXX-XXX-XXX-XXX-XXX/resourceGroups/XXX/providers/Microsoft.Authorization/roleAssignments/XXX-XXX-XXX-XXX-XXX?api-version=2015-07-01 HTTP/1.1" 400 88
msrest.http_logger : Response status: 400

msrest.http_logger : Response content:
msrest.http_logger : b'{"error":{"code":"BadRequestFormat","message":"The request was incorrectly formatted."}}'
msrest.exceptions : The request was incorrectly formatted.

@yugangw-msft can you take a look?

@hoffmann, please use '--debug' and share with the verbose output

this thing is still a problem

Agreed with @lanpar this is still a problem, even in Terraform (I understand this isn't a Terraform problem, but I see it in Terraform just the same)
This works:
resource "azurerm_role_assignment" "Azure-Admins_roleassignment" {
  scope              = "/subscriptions/${var.subscription_id}/resourceGroups/${var.resource_group_name}/providers/Microsoft.Compute/virtualMachines/vmname-VM"
  role_definition_name = "Virtual Machine Administrator Login"
  principal_id       = "014a520d-9a93-4dee-9644-9f1b1f228b87"
}

azurerm_role_assignment.Azure-Admins_roleassignment: Creation complete after 2s (ID: /subscriptions/e465f7b1-95ef-4fab-69b6-...s/63644e80-73a8-0158-b49d-ac9426abe562)
This does not:
resource "azurerm_role_assignment" "Azure-Admins_roleassignment" {
  scope              = "/subscriptions/${var.subscription_id}/resourceGroups/${var.resource_group_name}/providers/Microsoft.Compute/virtualMachines/vmname-VM"
  role_definition_id = "1c0163c0-47e6-4577-8991-ea5c82e286e4"
  principal_id       = "014a520d-9a93-4dee-9644-9f1b1f228b87"
}

* azurerm_role_assignment.Azure-Admins_roleassignment: authorization.RoleAssignmentsClient#Create: Failure responding to request: StatusCode=400 -- Original Error: autorest/azure: Service returned an error. Status=400 Code="BadRequestFormat" Message="The request was incorrectly formatted."

@asc-adean are you still experiencing this issue?

@hasheddan I will try it again this afternoon, thank you for responding

Change your roleDefinitionId to absolute. Following is ARM resource snippet
"resources": [         {             "type": "Microsoft.Authorization/roleAssignments",             "apiVersion": "[parameters('apiVersion')]",             "name": "[variables('roleAssignmentName')]",             "properties": {                 "roleDefinitionId": "[resourceId('Microsoft.Authorization/roleDefinitions', parameters('roleDefinitionId'))]",                 "principalId": "[parameters('principalId')]",                 "principalType": "[parameters('principalType')]",                 "canDelegate": "false", 				"scope": "[resourceGroup().id]"             }         }     ],

This issue has long been fixed in #2667.
--role now supports 3 formats:

ARM resource ID, like /subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590/providers/Microsoft.Authorization/roleDefinitions/acdd72a7-3385-48ef-bd42-f606fba81ae7
GUID, like acdd72a7-3385-48ef-bd42-f606fba81ae7
Name, like Reader

Source code:
https://github.com/Azure/azure-cli/blob/df737edb429a03a16794f8eb8d63435507c84562/src/azure-cli/azure/cli/command_modules/role/custom.py#L561-L579

Issue Resolved on my end.
I was trying to give the role to the storage in the bicep.
var storageBlobDataContributorRoleID = 'ba92f5b4-2d11-453d-a403-e96b0029c9fe'
I was writing this property as :
roleDefinitionId: storageBlobDataContributorRoleID
The I tried this one:
roleDefinitionId: resourceId('Microsoft.Authorization/roleDefinitions', storageBlobDataContributorRoleID)
It worked perfectly.

@tzaroon, your issue doesn't seem to be related to Azure CLI. Please only report Azure CLI-related issues in this repo. Thanks.

@tzaroon, your issue doesn't seem to be related to Azure CLI. Please only report Azure CLI-related issues in this repo. Thanks.

An argument could be made that @tzaroon posting their findings are useful because this GH issue is one of the first results when you search for the error message in relation to role definitions. I had a similar issue and @tzaroon's post resolved it for me 🤷♂.

@jiasli
I came across this issue during deploying resources using ARM template (not az cli).
I hope the maintainers to improve documentations as there're no sample values to input.
c.f. https://learn.microsoft.com/en-us/azure/azure-resource-manager/bicep/scenarios-rbac#scope


Offset fix. Add vgg19

Some imagenet networks trained with a labels' offset. I fix this problem for  vgg19, resnet50, mobilenet also add vgg19 benchmark.

I've signed CLA

vgg not in current v0.5 scope.

closing because vgg is not in the mlperf model list.