TypeError: 'NoneType' object is not iterable when installing MarkupSafe v0.23 on MSYS2

I'm using MSYS2 (fork of Gygwin), Python 2.7.10, pip 7.1.2, msys2's gcc 4.9.2, on Windows 8.1 x64.
$ pip install sphinx
Collecting sphinx
  Using cached Sphinx-1.3.1-py2.py3-none-any.whl
Collecting sphinx-rtd-theme<0.2,>=0.1 (from sphinx)
  Using cached sphinx_rtd_theme-0.1.9-py2-none-any.whl
Collecting snowballstemmer>=1.1 (from sphinx)
Collecting six>=1.4 (from sphinx)
  Using cached six-1.9.0-py2.py3-none-any.whl
Collecting Pygments>=2.0 (from sphinx)
  Using cached Pygments-2.0.2-py2-none-any.whl
Collecting docutils>=0.11 (from sphinx)
Collecting babel>=1.3 (from sphinx)
Collecting Jinja2>=2.3 (from sphinx)
  Using cached Jinja2-2.8-py2.py3-none-any.whl
Collecting alabaster<0.8,>=0.7 (from sphinx)
  Using cached alabaster-0.7.6-py2-none-any.whl
Collecting pytz>=0a (from babel>=1.3->sphinx)
  Using cached pytz-2015.6-py2.py3-none-any.whl
Collecting MarkupSafe (from Jinja2>=2.3->sphinx)
  Using cached MarkupSafe-0.23.tar.gz
Building wheels for collected packages: MarkupSafe
  Running setup.py bdist_wheel for MarkupSafe
  Complete output from command /home/Ed/.virtualenvs/bugsy2/bin/python -c "import setuptools;__file__='/tmp/pip-build-wb1swt/MarkupSafe/setup.py';exec(compile(open(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" bdist_wheel -d /tmp/tmp1FkDLRpip-wheel-:
  running bdist_wheel
  running build
  running build_py
  creating build
  creating build/lib.mingw32_nt-6.3-2.3.0(0.290
  creating build/lib.mingw32_nt-6.3-2.3.0(0.290/5
  creating build/lib.mingw32_nt-6.3-2.3.0(0.290/5/3)-x86_64-2.7
  creating build/lib.mingw32_nt-6.3-2.3.0(0.290/5/3)-x86_64-2.7/markupsafe
  copying markupsafe/tests.py -> build/lib.mingw32_nt-6.3-2.3.0(0.290/5/3)-x86_64-2.7/markupsafe
  copying markupsafe/_compat.py -> build/lib.mingw32_nt-6.3-2.3.0(0.290/5/3)-x86_64-2.7/markupsafe
  copying markupsafe/_constants.py -> build/lib.mingw32_nt-6.3-2.3.0(0.290/5/3)-x86_64-2.7/markupsafe
  copying markupsafe/_native.py -> build/lib.mingw32_nt-6.3-2.3.0(0.290/5/3)-x86_64-2.7/markupsafe
  copying markupsafe/__init__.py -> build/lib.mingw32_nt-6.3-2.3.0(0.290/5/3)-x86_64-2.7/markupsafe
  running egg_info
  writing MarkupSafe.egg-info/PKG-INFO
  writing top-level names to MarkupSafe.egg-info/top_level.txt
  writing dependency_links to MarkupSafe.egg-info/dependency_links.txt
  warning: manifest_maker: standard file '-c' not found

  reading manifest file 'MarkupSafe.egg-info/SOURCES.txt'
  reading manifest template 'MANIFEST.in'
  writing manifest file 'MarkupSafe.egg-info/SOURCES.txt'
  copying markupsafe/_speedups.c -> build/lib.mingw32_nt-6.3-2.3.0(0.290/5/3)-x86_64-2.7/markupsafe
  running build_ext
  building 'markupsafe._speedups' extension
  creating build/temp.mingw32_nt-6.3-2.3.0(0.290
  creating build/temp.mingw32_nt-6.3-2.3.0(0.290/5
  creating build/temp.mingw32_nt-6.3-2.3.0(0.290/5/3)-x86_64-2.7
  creating build/temp.mingw32_nt-6.3-2.3.0(0.290/5/3)-x86_64-2.7/markupsafe
  gcc -mdll -O -Wall -I/usr/include/python2.7 -c markupsafe/_speedups.c -o build/temp.mingw32_nt-6.3-2.3.0(0.290/5/3)-x86_64-2.7/markupsafe/_speedups.o
  Traceback (most recent call last):
    File "<string>", line 1, in <module>
    File "/tmp/pip-build-wb1swt/MarkupSafe/setup.py", line 120, in <module>
      try_building_extension()
    File "/tmp/pip-build-wb1swt/MarkupSafe/setup.py", line 99, in try_building_extension
      run_setup(True)
    File "/tmp/pip-build-wb1swt/MarkupSafe/setup.py", line 93, in run_setup
      ext_modules=ext_modules,
    File "/usr/lib/python2.7/distutils/core.py", line 151, in setup
      dist.run_commands()
    File "/usr/lib/python2.7/distutils/dist.py", line 953, in run_commands
      self.run_command(cmd)
    File "/usr/lib/python2.7/distutils/dist.py", line 972, in run_command
      cmd_obj.run()
    File "/home/Ed/.virtualenvs/bugsy2/lib/python2.7/site-packages/wheel/bdist_wheel.py", line 175, in run
      self.run_command('build')
    File "/usr/lib/python2.7/distutils/cmd.py", line 326, in run_command
      self.distribution.run_command(command)
    File "/usr/lib/python2.7/distutils/dist.py", line 972, in run_command
      cmd_obj.run()
    File "/usr/lib/python2.7/distutils/command/build.py", line 127, in run
      self.run_command(cmd_name)
    File "/usr/lib/python2.7/distutils/cmd.py", line 326, in run_command
      self.distribution.run_command(command)
    File "/usr/lib/python2.7/distutils/dist.py", line 972, in run_command
      cmd_obj.run()
    File "/tmp/pip-build-wb1swt/MarkupSafe/setup.py", line 41, in run
      build_ext.run(self)
    File "/usr/lib/python2.7/distutils/command/build_ext.py", line 337, in run
      self.build_extensions()
    File "/usr/lib/python2.7/distutils/command/build_ext.py", line 446, in build_extensions
      self.build_extension(ext)
    File "/tmp/pip-build-wb1swt/MarkupSafe/setup.py", line 47, in build_extension
      build_ext.build_extension(self, ext)
    File "/usr/lib/python2.7/distutils/command/build_ext.py", line 528, in build_extension
      target_lang=language)
    File "/usr/lib/python2.7/distutils/ccompiler.py", line 691, in link_shared_object
      extra_preargs, extra_postargs, build_temp, target_lang)
    File "/usr/lib/python2.7/distutils/cygwinccompiler.py", line 189, in link
      libraries.extend(self.dll_libraries)
  TypeError: 'NoneType' object is not iterable

  ----------------------------------------
  Failed building wheel for MarkupSafe
Failed to build MarkupSafe
Installing collected packages: sphinx-rtd-theme, snowballstemmer, six, Pygments, docutils, pytz, babel, MarkupSafe, Jinja2, alabaster, sphinx
  Running setup.py install for MarkupSafe
    Complete output from command /home/Ed/.virtualenvs/bugsy2/bin/python -c "import setuptools, tokenize;__file__='/tmp/pip-build-wb1swt/MarkupSafe/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-wPGgLg-record/install-record.txt --single-version-externally-managed --compile --install-headers /home/Ed/.virtualenvs/bugsy2/include/site/python2.7/MarkupSafe:
    running install
    running build
    running build_py
    running egg_info
    writing MarkupSafe.egg-info/PKG-INFO
    writing top-level names to MarkupSafe.egg-info/top_level.txt
    writing dependency_links to MarkupSafe.egg-info/dependency_links.txt
    warning: manifest_maker: standard file '-c' not found

    reading manifest file 'MarkupSafe.egg-info/SOURCES.txt'
    reading manifest template 'MANIFEST.in'
    writing manifest file 'MarkupSafe.egg-info/SOURCES.txt'
    running build_ext
    building 'markupsafe._speedups' extension
    gcc -mdll -O -Wall -I/usr/include/python2.7 -c markupsafe/_speedups.c -o build/temp.mingw32_nt-6.3-2.3.0(0.290/5/3)-x86_64-2.7/markupsafe/_speedups.o
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-wb1swt/MarkupSafe/setup.py", line 120, in <module>
        try_building_extension()
      File "/tmp/pip-build-wb1swt/MarkupSafe/setup.py", line 99, in try_building_extension
        run_setup(True)
      File "/tmp/pip-build-wb1swt/MarkupSafe/setup.py", line 93, in run_setup
        ext_modules=ext_modules,
      File "/usr/lib/python2.7/distutils/core.py", line 151, in setup
        dist.run_commands()
      File "/usr/lib/python2.7/distutils/dist.py", line 953, in run_commands
        self.run_command(cmd)
      File "/usr/lib/python2.7/distutils/dist.py", line 972, in run_command
        cmd_obj.run()
      File "/home/Ed/.virtualenvs/bugsy2/lib/python2.7/site-packages/setuptools/command/install.py", line 61, in run
        return orig.install.run(self)
      File "/usr/lib/python2.7/distutils/command/install.py", line 563, in run
        self.run_command('build')
      File "/usr/lib/python2.7/distutils/cmd.py", line 326, in run_command
        self.distribution.run_command(command)
      File "/usr/lib/python2.7/distutils/dist.py", line 972, in run_command
        cmd_obj.run()
      File "/usr/lib/python2.7/distutils/command/build.py", line 127, in run
        self.run_command(cmd_name)
      File "/usr/lib/python2.7/distutils/cmd.py", line 326, in run_command
        self.distribution.run_command(command)
      File "/usr/lib/python2.7/distutils/dist.py", line 972, in run_command
        cmd_obj.run()
      File "/tmp/pip-build-wb1swt/MarkupSafe/setup.py", line 41, in run
        build_ext.run(self)
      File "/usr/lib/python2.7/distutils/command/build_ext.py", line 337, in run
        self.build_extensions()
      File "/usr/lib/python2.7/distutils/command/build_ext.py", line 446, in build_extensions
        self.build_extension(ext)
      File "/tmp/pip-build-wb1swt/MarkupSafe/setup.py", line 47, in build_extension
        build_ext.build_extension(self, ext)
      File "/usr/lib/python2.7/distutils/command/build_ext.py", line 528, in build_extension
        target_lang=language)
      File "/usr/lib/python2.7/distutils/ccompiler.py", line 691, in link_shared_object
        extra_preargs, extra_postargs, build_temp, target_lang)
      File "/usr/lib/python2.7/distutils/cygwinccompiler.py", line 189, in link
        libraries.extend(self.dll_libraries)
    TypeError: 'NoneType' object is not iterable

    ----------------------------------------
Command "/home/Ed/.virtualenvs/bugsy2/bin/python -c "import setuptools, tokenize;__file__='/tmp/pip-build-wb1swt/MarkupSafe/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-wPGgLg-record/install-record.txt --single-version-externally-managed --compile --install-headers /home/Ed/.virtualenvs/bugsy2/include/site/python2.7/MarkupSafe" failed with error code 1 in /tmp/pip-build-wb1swt/MarkupSafe

This is an issue with Python and distutils, which don't support msys2 builds as far as I can tell, not with MarkupSafe.  There is nothing we can do here to fix this.


Added table listing services available by default for DI

Hi @ardalis, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@ardalis  nice addition.


Improve the documentation regarding contributions

Issue Link :link:
https://forums.swift.org/t/broken-unit-tests-for-the-ios-scheme/52371/5
Goals :soccer:

Document how to contribute to the project by sending pull requests.
Document how to configure the automated tests to be run.

Implementation Details :construction:
I've changed the CONTRIBUTING.md file, adding a section about pull requests. I've also linked it from the project  README.md.

Thanks for the correction and Improvements. I just applied them.


iOS: "Could not find the service definitions manifest. No custom services will be available."

 %flutter run --debug                                                                                [/src/game]
Running lib/main.dart on flutter_ipod_touch...
vm-service: Isolate creation error: (null)
[INFO:FlutterDynamicServiceLoader.mm(129)] Could not find the service definitions manifest. No custom services will be available.
[INFO:FlutterDynamicServiceLoader.mm(107)] 0 custom service definitions registered
vm-service: Isolate creation error: (null)
[INFO:FlutterDynamicServiceLoader.mm(129)] Could not find the service definitions manifest. No custom services will be available.
[INFO:FlutterDynamicServiceLoader.mm(107)] 0 custom service definitions registered
vm-service: Isolate creation error: (null)
[INFO:FlutterDynamicServiceLoader.mm(129)] Could not find the service definitions manifest. No custom services will be available.
[INFO:FlutterDynamicServiceLoader.mm(107)] 0 custom service definitions registered
[INFO:FlutterDynamicServiceLoader.mm(129)] Could not find the service definitions manifest. No custom services will be available.
[INFO:FlutterDynamicServiceLoader.mm(107)] 0 custom service definitions registered
[INFO:FlutterDynamicServiceLoader.mm(129)] Could not find the service definitions manifest. No custom services will be available.
[INFO:FlutterDynamicServiceLoader.mm(107)] 0 custom service definitions registered

Yeah, we should probably remove it.  It's the old way of integrating services.

Fixed by 07b59dff2c1906a0c637a6088697bb449f6074b7


hello,how to set WebSocket.binaryType?

WebSocket.binaryType

You should add more information. Currently your question is unclear.
binaryType is specific to client side JavaScript websockets. There is no need to set it when using this library. All binary messages will be received as ByteBuffers.

The server only supports connection by binary.I don't know how to connect. It is fail to connect when using this library.

Please post your code or give us a link to your project.

No response - closing issue!


Unfortunately, the playlist does not download.

Unfortunately, the playlist does not download. The percentages are full as if it is downloading but there is no video in the download folder.

Try running it as administrator

Yes, it downloads like this. You need to force the application to run as administrator


BBCode?

Heyho, I like your WYSIWYG-Editor very much, I would use it, but there is no BBCode-Support :(
Is something like this planned?
Would be happy if this will be answered, thanks and good greetings from germany,
Tim

WYSIWYG, but stores the value in the textarea to bbcode instead of HTML. See how SCEditor does it. http://www.sceditor.com/

Trumbowyg is based on a content editable and execComand API. So to support markdown, a complete rewrite is needed.
Maybe for an another project :)

I use Trumbowyg for every project I can. Love your work Alex. And so does the LiteCart Community.
For BBCode I've been using SCEditor which is practically dead and full of bugs.
Would it be able to throw in a bbcode converter as a plugin that converts the Trumbowyg HTML to bbcode right before submitting? With the support of bbcode I believe there is great potential to conquer the forum market with this editor. The only option up to date seems to be the massive hypernormous CKEditor.


[macos] add Mac OS X 10.0 through OS X 10.8

Saw that old versions were missing. Added them to the best of my knowledge

We can try getting more accurate data over time for the EOL dates, but EOL = lastDate is good enough for such old releases.


How to use adf with Angular 4/5/6/7 app instead of Angular.js ?

Please share any link or any way to implement same thing with Angular 4/5/6/7.

Hi Team,
Any updates?


:seedling: implement 1 QPS rate limiting for cron best practices worker.

What kind of change does this PR introduce?
bug fix

[X] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
We are getting connection reset requests from bestpractices.dev and 429 errors from our GCS bucket for too many writes.
bestpractices.dev has a limit of 1 QPS
GCS has a limit of 1000 QPS
What is the new behavior (if this is a feature change)?**
Use the lower limit of 1 QPS limit for bestpractices.dev when populating the cron data.
Each page has 30 projects, so 30 QPS is below the GCS limit.
The construct was taken from https://go.dev/wiki/RateLimiting which "works well for rates up to tens of operations per second."

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

Which issue(s) this PR fixes
Related to #4037 (but may not fix it yet)
Special notes for your reviewer
Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the release-note
(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

NONE

Also linking #1240 for historical cross references. As this was originally proposed to deal with rate limiting


fix: fixed for the issue (#549)

1. Description

1.1. Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for ALL new/updated/fixed functionality.
[ ] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[ ] I have updated/added relevant examples in examples.

1.2. Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

1.3. Related Issues

Codecov Report

Merging #550 (e20a854) into main (7eeb8ef) will increase coverage by 8.89%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #550      +/-   ##
==========================================
+ Coverage   67.52%   76.41%   +8.89%
==========================================
  Files         204       30     -174
  Lines        1946      564    -1382
==========================================
- Hits         1314      431     -883
+ Misses        632      133     -499

see 174 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


clicking on a menu item causes flicker

Bug Report
Ionic version:

[x] 4.x
Current behavior:
Clicking a menu item causes flickering
Expected behavior:
it should not cause flickering
Steps to reproduce:
run ionic serve
When the page is loaded, selecting other menu item cause flicker, it only happens once on freshly loaded page. a refresh triggers it again
Related code:
https://github.com/alainbelez/ionic-starter
Other information:
this is not happening to me on Angular, and i tested removing this { provide: RouteReuseStrategy, useClass: IonicRouteStrategy }, and it fixed the problem, i just don't know the side effects.
Ionic info:
Ionic:
ionic (Ionic CLI)             : 4.12.0 (/usr/local/lib/node_modules/ionic)
Ionic Framework               : @ionic/angular 4.1.1
@angular-devkit/build-angular : 0.13.6
@angular-devkit/schematics    : 7.3.6
@angular/cli                  : 7.3.6
@ionic/angular-toolkit        : 1.4.1
Cordova:
cordova (Cordova CLI) : 8.1.2<EMAIL_ADDRESS>Cordova Platforms     : android 7.1.4
Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 4.0.0, (and 6 other plugins)
System:
Android SDK Tools : 26.1.1 (/home/alain/Android/Sdk)
NodeJS            : v10.15.3 (/usr/bin/node)
npm               : 6.9.0
OS                : Linux 4.18

upon checking the file ionic-router-reuse-strategy.ts on first load, curr.component is null, and clicking a link on the menu bar will return true to this expression thus shouldReuseRoute will return false:
    if (future.component !== curr.component) {
      return false;
    }

suceeding clicks will cause false, thus skipping.

Hello, I thin this is the same issue as mine #17732

@mburger81 checked the repo and adding a trackBy in ngFor seems to fix your problem, i think it happens because angular is recreating the ion-item every time.
//view
    <ion-item *ngFor="let gw of gws; trackBy: trackByFn" button>
        <ion-avatar slot="start">

        </ion-avatar>
        <ion-label>

{{ gw.description }}

{{ gw.serialNumber }}

        </ion-label>
    </ion-item>

//controller
  trackByFn(index: number, item: any): number {
    return item.serialNumber;
  }


CI: pkg:protobuf – only analyze lib and test directories

Skip benchmarks since required protobufs are not generated yet

Feel free to squash/merge and delete the branch if you're happy @sigurdm


pub/sub browser client example?

Would be nice with an example on how one could interact with pub/sub over a browser client websocket?

Client:
ws=new WebSocket('ws://localhost:3000')
ws.onmessage=e=>console.log(e.data)

Server:
id=0
app=require('uWebSockets.js').App().ws('/',{
   open:ws=>{ws.subscribe('all');app.publish('all',`ws open id: ${++id}`)}
}).listen(3000,t=>{})


add drop table method

add drop table method
test drop table

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1839
#1838  👈
#1837
#1833
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @kariy and the rest of your teammates on  Graphite


scanpy.api.tl.diffmap documentation

Please consider mentioning explicitly in the documentation of scanpy.api.tl.diffmap that "the 0-th column is the steady-state solution, which is non-informative in diffusion maps", as mentioned here: https://github.com/theislab/single-cell-tutorial/blob/master/latest_notebook/Case-study_Mouse-intestinal-epithelium_1906.ipynb. Also that it's not plotted by default, cf. https://github.com/theislab/scanpy/issues/675.
It is very important for those who want to use these coordinates (adata.obsm["X_diffmap"]).

This issue has been mentioned on Scanpy. There might be relevant details there:
https://scanpy.discourse.group/t/how-are-the-diffusion-plots-generated/400/2


[WIP][FasterTransformer] use python 3.10.0 and upgrade pytorch

Description
Upgrade PyTorch and bump up python to 3.10

Doesn't this only change Python for FT but leave it the same (3.9) in the other containers?

defered


file type confusion in load

Here is output from spritz where we get an unrecognized file type error. I know this happens for other files, where I think that it shouldn't. There should be a way to isolate the final .### and identify it as the file type and ignore all the previous "."s
SRR1025675_1-trimmedAligned.sortedByCoord.outProcessed.out.fixedQuals.split.snpEffAnnotated.protein.fasta

This is how Path.GetExtension() works; it isolates the text after the last "." and uses this as the extension. I decided to not use that function because it messed up .xml.gz (it thought the extension was .gz). So what it does now it is assumes that whatever is after the first dot in the filename is the extension. There may be a more complex/robust way of handling that but I think the easy answer is just to remove the dots in the filename except the one before the extension

I'd like to have it handle the more complex scenerio. Anything ending in .gz should be examined again to see which bin it goes to (either raw file or database). That way we can use compressed for everything. This is not urgent, but I would like it to get done. Early job for the next undergrad.

see #1164


Crash - pointer being freed was not allocated

I'm getting a crash when running Kitura with Kitura-redis.
myFirstProject(72926,0x700000187000) malloc: *** error for object 0x7f8ca8e04b50: pointer being freed was not allocated
It seems that my code is executed properly as I receive a response from the server, but then the server crashes and I'm not sure how to go about debugging this. Can someone here point me in the right direction? Thanks!
Here's the code which crashes:
router.get("/visits", handler: {
    request, response, next in

    let redis = Redis()
    redis.connect("localhost", port: 6379, callback: { (error: NSError?) in
        if let error = error {
            Log.verbose(error.localizedDescription)
        } else {
            redis.incr("visits", by: 1, callback: { (visits: Int?, error: NSError?) in
                if let error = error {
                    Log.verbose(error.localizedDescription)
                } else {
                    if let visits = visits {
                        response.status(HttpStatusCode.OK).send("Visits: \(visits)")
                        next()
                    }
                }
            })
        }
    })
})

hmm not sure exactly, but next should be called for every condition

Hey @chrisozenne I want to follow up on this issue. Is this still happening in the latest version of Kitura? If you are interested, we can look at your project and see if we can figure out what's going on.

I haven't tried using Redis since I posted this over a month ago.

Sorry for the long delay @chrisozenne, we have been focused on development in the past but are now trying to make a renewed effort to address community issues. If there is anything we can do for you now, please let us know.

I'm glad to hear so. I was beginning to write a tutorial series on Kitura, but decided that the timing isn't quite right yet. For example, I have a pull request for Kitura-CouchDB which is now over a month old and being ignored: https://github.com/IBM-Swift/Kitura-CouchDB/pull/5

This is a duplicate of IBM-Swift/Kitura-redis#5


idk how to run this script

can you explain how to run this program?

Hey what are u doing here?  This code seems not very useful. 😵💫

Hey what are u doing here? This code seems not very useful. 😵💫

eumm i think this code is very useful for beginner, n yeah im beginner.. pleased to meet u!

Itu hanya program mencari kata di kamus bagi backend dev

It's just a program searching for words in the dictionary for backend dev

bisakah anda mengajariku bagaimana cara menjalankan script ini?

It's just a program searching for words in the dictionary for backend dev

bisakah anda mengajariku bagaimana cara menjalankan script ini?

Baca di readmeeeeeeeee


1.0-rc1 - Upgrade to 12.10-stable + Added support for Controller certificates + Better register prompts + tests + more

Changes:

Upgraded base to 12.10-stable
Added retries to any (doRequest) HTTP calls to the controller to handle when the controller crashes or returns bad data with exponential backoff
Added Controller certificate support:

   --anka-root-ca-path value             Specify the path to your Controller's Root CA certificate [$ROOT_CA_PATH]
   --anka-cert-path value                Specify the path to the GitLab Certificate (used for connecting to the Controller) (requires you also specify the key) [$CERT_PATH]
   --anka-key-path value                 Specify the path to your GitLab Certificate Key (used for connecting to the Controller) [$KEY_PATH]

It can now run independently of other gitlab-runners on the host.

config.toml -> anka-config.tml

Added --preparation-retries and defaulted it to 0
Better registration experience with errors thrown for bad data (prompt messages, etc)
Added a bunch of tests
Readme update with developer guide and changes we've made from the gitlabhq repo
Fixed all of the tests

TODO: Make tests for the executor

Tested the binary for darwin-amd64 and linux-amd64. It works!


need shuttle timer wallmounts for evac/travel time

ideally theyd be generic so they could be synced with arrivals shuttle as well

hey i built these (they work with evac and also travel) but i'm at a point where i need to know if they should be synced to individual shuttles or continue to be station-wide

@Emisse

there should be prototypes for arrivals shuttle timers that can be placed at terminals/arrivals and also ones for evac shuttle

@Emisse #19471 got merged. there is an arrivals screen proto and a regular screen proto which gets escape shuttle updates


5MP OV5642 Sensor without Arduino Shield

Hello! I am new to ArduCam and Arduino. I purchased an Arduino UNO and an OV5642 Sensor which did not include the Arducam Mini Camera Shield. I am (of course) now struggling to connect the two.
I am wondering if there is any way to buy a "mini camera shield" without the camera sensor.  I don't want the LCD Rev.C+, just looking for a simple way to grab a JPEG. Thanks for any suggestions; or even just confirming my suspicion that this is not possible.

@Pajama, You can't connect OV5642 camera module to Arduino UNO directly, you need additional ArduCAM shield like this one http://www.uctronics.com/arducam-f-camera-module-shield-for-arduino-uno-mega2560-due.html?___SID=U
It is just the one you needed without the LCD.

Can I do it with something like this? The "mini" shield? http://www.uctronics.com/mini-arducam-module-camera-shield-w-5-mp-ov5642-for-arduino-uno-mega2560-board.html?___SID=U

@Pajama, yes you can use this ArduCAM Mini 5MP camera which has build in OV5642 sensor. So you can't use your OV5642 module that you already have.

Okay I decided to just buy a new camera + mini shield due to some size constraints. Thanks for your help!


[iOS][WeexSDK] WeexSDK (0.26.0)第一次渲染textarea组件在ios端有卡顿情况

WeexSDK (0.26.0)第一次渲染textarea组件在ios端有卡顿情况

Try 0.28.0
If this still bothers you and you find a solution, you could give us a PR if you can, I am very happy to talk with you implemenation detail or review you PR in mailing list.
I have a busy schedule and I can't read Github issue every day, but I check mailing list every day. I am sorry if this bothers you.


Version confusion

I have done an "npm install --save simpl-schema" in my project and npm shows a successful installation of<EMAIL_ADDRESS>The documentation talks about release of "Version 2".
Can you explain the difference in the versions numbers?
Thanks.

 It is v2 in that it is a major rewrite with breaking changes, but I restarted the actual numbers since it's really an entirely new package.

Thanks
From: Eric Dobbertin<EMAIL_ADDRESS>Sent: Saturday, 11 February 2017 1:42 AM
To: aldeed/meteor-simple-schema<EMAIL_ADDRESS>Cc: jvella61<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [aldeed/meteor-simple-schema] Version confusion (#683)
"Version 2" switched from being a Meteor package to being a non-Meteor-specific JavaScript package on NPM. It is v2 in that it is a major rewrite with breaking changes, but I restarted the actual numbers since it's really an entirely new package.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/aldeed/meteor-simple-schema/issues/683#issuecomment-278978136, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATHQrWA7LIDnbKZi1NsYp55f60oeddK-ks5rbIUzgaJpZM4L815B.

The information contained in this message is intended only for the use of the individual or entity to which it is addressed. It is confidential and may be legally privileged. If the reader of this message is not the intended recipient, or is not responsible for delivering the message to the intended recipient, then please be aware that any use, dissemination, further distribution, or reproduction of this message in any form whatsoever is strictly prohibited. If the email is received in error, please notify the sender by return e-mail or telephone and immediately delete your copy of the message.
While iScape Pty Ltd does have security measures in place to prevent potential issues, no warranty can be provided that any email attachments are free from viruses or other defects and no liability will be accepted for any loss, damage or other consequences that may arise from opening or using email attachments.


Add missing model context attribute

Why are these changes needed?
The case, where model context is not None was missing, leading to a runtime error. Probably makes sense generally to add a test for that functionality.
Related issue number

Checks

[x] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[x] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[x] I've made sure all auto checks have passed.

Would be good to add a test.


Does not do a thing

Sadly does not do a thing.

More details please?

More details please?

I downloaded the plugin, it shows Green in plugin list.
I can type the command "/ttv" and "/ttvend"
but as i type "/ttv" it does not do a thing, i still right clicked a villager(with a profession and one without)
But both times it did not do a thing.
im half ish sure that i used the correct api key i used it from: https://platform.openai.com/account/api-keys
Created a new one and used that one in the config.yml -> "openai-key" its gpt 3.5.

additional info: i was now able to interact with a villager but got this error in console: https://pastebin.com/t9NDd3dX
Seems to also crash my server..
I WAS able to interact with a villager this time after re-doing my api key a few times... i was able to start a chat with a villager but after that it kept waiting and waiting untill it asked me to try again, and server crashed.

OpenAI is giving you this error:

You exceeded your current quota, please check your plan and billing details

Make sure you have a credit card set up in your account and that you haven't set up any limits here: https://platform.openai.com/account/billing/limits
Closing since this isn't an issue with the plugin itself


Fixed issue with error messages not being displayed during incorrect Login with false Username

Fixed issue with error messages not being displayed during incorrect logins with false Usernames

@kurtw

:+1: Nice!


Compat NgRx Store

Check if it is possible to integrate an existing NgRx Store into withRedux.
Coming from NgRx Store
const flightsFeature = createFeature({
  name: 'flights',
  reducer: createReducer(initialState, on(flightActions.searched, (state, {flights}) => ({...state, flights})))
})

@Injectable()
class FlightEffects {
  search$ = createEffect(() => ofType(flightActions.search));
}

Could be integrated like
signalStore(
  {providedIn: 'root'},
  withReduxCompat(flightsFeature, FlightEffects)
);

According to my tests while I was developing this lib, if you inject the effects in the store, you will have circular dependency injections if you try to inject the store on your effects class.
The best solution I found was to create this provider function that would inject the Effects when you use it with the EffectsDirective.

In that case, the  FlightEffect still goes against the @ngrx/store Store service and not against the SignalStore. I guess with your provider function, one could replace it under the hood.
So the effect would think it is working with the Store service although it runs against an adapter which uses the SignalStore.


anagramCheck.py : small changes

I concated the assert condition using and.
I added comments for the expected behavior of the tests.
I added a main function.  More Python like...

Good work


Dashboard : values file need to be improved

https://github.com/threefoldtech/tfgrid-sdk-ts/blob/v2.0.0-rc11/packages/dashboard/charts/tfgrid-dashboard/values.yaml#L107 , this line is not needed anymore as it's already be passed in env variables .

Verified from https://github.com/threefoldtech/tfgrid-sdk-ts/blob/development/packages/dashboard/charts/tfgrid-dashboard/values.yaml
The mentioned value was removed.


add e2e testcases for ci for stable branch

These cases have been merged into master branch and the difference is excluding zran format case since stable branch doesn't support zran for now.
Signed-off-by: Yiqun Leng<EMAIL_ADDRESS>

/retest


Allow configuring unstable flags via config file

Summary
This fixes #8127 by mapping the unstable key in .cargo/config to Z flags.
It should have no impact on stable/beta cargo, and on nightlies it gives folks the ability to configure Z flags for an entire project or workspace. This is meant to make it easier to try things like the new features resolver behavior, mtime-on-use, build-std, or timing reports across a whole project.
I've also included a small (but entirely independent) ergonomics change -- treating dashes and underscores identically in Z flags. That's along for the ride in this PR as the last commit, and is included here because it makes for more idiomatic toml file keys (print_im_a_teapot = yes vs print-im-a-teapot = yes). Please let me know if y'all would prefer that be in a separate PR, or not happen at all.
Test Plan
Apologies if I've missed anything -- this is my first cargo contrib and I've tried to hew to the contributing guide. If I've slipped up, please let me know how and I'll fix it.
NB. My linux machine doesn't have multilib set up, so I disabled cross tests.

cargo test passes for each commit in the stack
I formatted each commit in the stack with rustfmt
New tests are included alongside the relevant change for each change
I've validated each test by locally undoing the code change they support and confirming failure.
The CLI wins, for both enable and disabling Z flags, as you'd expect.

Keys in unstable which do not correspond with a Z flag will trigger an error indicating the invalid flag came from a config file read:
Invalid [unstable] entry in Cargo config

Caused by:
  unknown `-Z` flag specified: an-invalid-flag

If you'd like to see a test case which isn't represented here, I'm happy to add it. Just let me know.
Documentation
I've included commits in this stack updating the only docs page that seemed relevant to me, skimming the book -- unstable.md.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

I gather it's probably fair game since we're strictly talking about unstable flags / features here, but a hard error on Zflags in config would be a break with how mtime-on-use is configured today in that stable cargo simply ignores the setting.
I can see a reasonable argument either way --
Pro: A hard error matches the behavior of setting these same flags at the command line, and reduces the chances of a nasty surprise change-in-behavior if you switch channels and recompile.
Con: A hard error means you can't leave unstable options in config and toggle back and forth between stable and nightly, or configure CI runs on nightlies to assert that a new feature does not break a build that works on stable.
Let me know how you'd like to see it work, and I'll get it changed.

Oh right nevermind, I forgot that we ignore it on stable/beta! That seems fine for now!

@alexcrichton would you still like me to include a Deserialize impl in this PR, given it won't slot into the update path it's modifying?

Just a quick update — I'm still working on this, I just hit an oddity in the Deserializer that's treating missing fields as an error even with #[serde(default)] on the struct. Curiously, forcing every field to use the Option deserializer first and explicitly defaulting them works,
(a trick I shamelessly borrowed from dtolnay's answer to a totally unrelated question about serde usage elsewhere on the internet). That looks something like:
#[serde(deserialize_with="null_default")]
and
fn null_default<'de, D, T>(deserializer: D) -> Result<T, D::Error>
where
    D: serde::Deserializer<'de>,
    T: Deserialize<'de> + Default,
{
    let option = Option::deserialize(deserializer)?;
    Ok(option.unwrap_or_default())
}

I'm digging through the Deserializer now to see what's going on there.

Thinking out loud a bit here:
After talking it over with a buddy (thx @packysauce!) I think there's a bit of a layering problem in the de module that's preventing serde from doing defaults or deny_unknown here — when we're deserializing a struct, ConfigMapAccess is providing an iterator over the union of two sets: the fields we're interested in for this struct (the fields member slice), and k/v pairs available in the config table we're currently considering. As I understand them, serde MapAccess implementers are expected to serve up kv pairs of the underlying map without any particular care for what set of keys the caller is interested in. With that in mind, handling env reading inline complicates the Deserializer.
As I understand it, that means there are two problems:

Serde's handling for defaulting and denying-unknown will always be a noop, because we never hand back a k/v pair that the Deserialize impl didn't explicitly ask for as a field, and if we're missing a field we error out early ourselves instead of letting the derived Deserialize impl do it. This is caused by the union done in ConfigMapAccess, but fixing it would complicate point 2.
Environment variables don't fit super neatly in this model — if we modified ConfigMapAccess to return all k/v pairs from the config object under consideration plus all env vars which match the prefix under consideration, env vars which match member structs would show up as unexpected field keys (e.g. CARGO_UNSTABLE_SOMETABLE_SUBKEY would show up as an unknown key in CARGO_UNSTABLE even if CARGO_INSTABLE_SOMETABLE would have been valid).

While I could work around this for CliUnstable by annotating every non-Option field with a deserialize_with, I think the "Right" solution here is probably a larger refactor to split the env reading out of the Deserializer impl such that Config::get would try reading each source in decreasing priority order independently, with its own purpose-specific deserializer, rather than using a combined deserializer.

I'd definitely believe that the config bits with serde aren't exactly the best they could be, I've never really fully grok'd how to write a serde deserializer. That being said I'm happy with whatever works here, I think switching all the fields to Option would also be fine.

I've never totally understood the serde model myself, either — I'm learning my way through it on this PR. Definitely tell me if I've got it wrong!
Changes in this round:

Rebased on master
Added a default-to-false deserialize_with workaround to CliUnstable
Dropped a (probably helpful) test that's broken by that change.

I'm gonna keep hacking on top of this to flip the Deserializer impl around to allow serde's default/deny_unknown_fields to do the heavy lifting, but I didn't want y'all to have to wait on me finishing that up to land this and close the linked issue, if you'd like to.
In the meantime please don't hesitate to ask if you'd like me to make some more changes to this first.

Looks good to me! I'd be fine landing this as-is. FWIW I don't think we want to add deny_unknown_fields because that'll make it painful to use this from stable Cargo when nightly adds a new feature

Just kidding, this was way simpler than I was making it out to be — the new_map ctor for ConfigMapAccess does the right thing already, it's just the struct behavior is trying to be a little too clever in how it's prefiltering fields.
I patched new_struct and dropped the workaround. If this still looks good I think it's ready to rock!

Oh nice! So to make sure I understand here, the previous implementation with serde was blindly assuming that all the requested fields were in the config to lookup, but this implementation is changing that to instead only iterate over fields that are actually defined in the table/env. This filtering means that we don't assume that a field requested (which could be optional) was present in the config and iterate over it (unlike before).
One thing that makes me a bit worried is the prefix-checking for the environment. We've had issues of that in the past where you've looking for CARGO_DEBUG but CARGO_DEBUG_ASSERTIONS is present in the environment. It looks like though that this won't fall down in any way that it wasn't already falling down in, right? Specifically though if we add two unstable flags foo and foo-bar, then if you set CARGO_UNSTABLE_FOO_BAR in the environment then will that generate an error deserializing saying that the foo field wasn't defined? (even though it is set to #[serde(default)])

That's exactly it, yeah!
To summarize: we have 3 lists of keys:

The fields in the struct we're trying to deserialize
The list of environment variables which match our prefix
The CV::Table we're reading from

The previous behavior was for ConfigMapAccess to act as an iterator over #1, yielding the fields on the struct in order whether or not they exist in either the environment or the config file.
This patch causes ConfigMapAccess to instead iterate over the intersection of 1 and 2, unioned with 3 — (struct_fields ∩ env_keys) ∪ table_keys. Effectively, it asks "What keys are present in the sources of config?" and lets the caller decide whether or not those keys are significant to them. The environment prefiltering to only matches is meant to avoid spurious "unknown field" errors on all the environment keys that would never influence the current struct anyway.
The net results of that change:

Missing field errors can come from derived Deserialize impls when a key is not present in either the environment or the config table. Previously this condition would Err out in the Deserializer, typically in one of the deserialize_$type functions.
Derived Deserialize impls can choose to act on extra or missing fields as they'd like. Previously, extra keys would not be reported (because we only iterated struct fields), and missing fields would be reported as Deserializer errors. This means the #[serde()] annotations for default and deny_unknown_fields should work as expected now, with one exception (next point).
Environment variables which match a prefix will be reported. As you've pointed out, this does mean that env vars for sub-objects in config will show up in the set of keys for the containing object. In the case where either #[serde(default)] is set or no annotation is provided, this should be identical to the current behavior because the derived Deserialize will wind up simply skipping those env entries as unknown fields. The new failure mode is in combining env sub-object keys  with deny_unknown_fields — I think that will break in a potentially surprising way in that it'll cause an error for an otherwise-valid subkey setting.

I'll add some tests around that behavior now to make sure I've got the combination of behaviors right here — it occurs to me that we might even be able to fix that by doing exact matches on env keys in ConfigMapAccess, since subkeys shouldn't need to be reported there.

Looks like we've already got tests covering most of those cases!

config::struct_with_opt_inner_struct checks that setting an env key for a nested struct results in trying to deserialize the nested struct rather than defaulting it. This test correctly fails if I switch ConfigMapAccess to eq instead of starts_with on env keys, because that'd lead to skipping a member struct if there's no other indication that data is present to deserialize it.
config::overlapping_env_config asserts that optional keys with overlapping prefixes are correctly independent.

I've added a couple more using default annotations based on the existing ones. These demonstrate the failure mode you mentioned pretty clearly. I think that might be a somewhat unavoidable limitation of the mapping between environment variables and nested configs, unfortunately, but you're correct that this patch fails in the same way as the existing code in the overlapping_env scenario (and doesn't fail in any new ways).

On second thought — it seems preferable to error out when we encounter an ambiguous situation like this, versus incorrectly defaulting an inner field. I'll revert the env prefix tweak and patch the new tests accordingly.

Ok, with that done — your summary of the change is entirely correct, and now we've got tests asserting that.

Ok this all seems resonable to me, thanks again for helping to explain all this! It sounds like we'll want to be careful to avoid overlapping prefixes where possible until we fix these issues (if they're fixable at all, really), but that's something we can try to remain vigilant about in the future.
Thanks again for your work here!
@bors: r+

:pushpin: Commit d035daae4e26bfe144ceff41f4c9a0bb11fea605 has been approved by alexcrichton

:hourglass: Testing commit d035daae4e26bfe144ceff41f4c9a0bb11fea605 with merge a02e7655e7638a14e02584117a8b489e00ed8b0f...

Happy to help - thanks for the reviews!

:broken_heart: Test failed - checks-azure

Test failure, bummer.
I couldn't find a way to pull the proptest codegen'd case off the builder host, so I've left my workstation running a copy of the failing proptest (prop_limited_independence_of_irrelevant_alternatives) per core until I get a reproducer. Fingers crossed it doesn't take toooooo long.

@bors retry
https://github.com/rust-lang/cargo/issues/6258#issuecomment-656404282

:hourglass: Testing commit d035daae4e26bfe144ceff41f4c9a0bb11fea605 with merge f35ebafc74fed12ba7c5f7c497a7e751889c11dd...

thx ehuss!
I just got a repro case locally, but skimming the linked thread it looks like y'all already have a handle on where the edge case proptest is turning up is so I'll let it lie.

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing f35ebafc74fed12ba7c5f7c497a7e751889c11dd to master...

Technically there have to be an infinite supply of edge cases. The problem is NP-Complete so there have to be exponencial cases. So far (at least recently) the proptest are finding particularly un-realistic examples and my investigations have not found ways to make them fast that are worth the additional code complexity. But each one is a new adventure, that can be completely unrelated.

I just tried this feature with the latest nightly and it seems to work great overall. Thanks a lot @bearcage for implementing this!
One minor thing I noticed was that the -Zbuild=core command line argument behaves slightly different than setting unstable.build-std = ["core"] in a .cargo/config file. The former seems to automatically derive that compiler_builtins is also needed when building core, while the latter seems not. With the config option, a "can't find crate for compiler_builtins" error occurs unless I add compiler_builtins to the unstable.build-std list. This isn't a problem for me, but I wanted to report it in case this is unintended.

@phil-opp looks like that's a bug

Thanks a lot for the quick investigation and fix!

Thanks to this PR, now we can use the rust-analyzer with the unstable cargo option! Yay


Training Yolox-X on COCO

@FateScript  hi , i am training yolox -x backbone on coco dataset with image size of 800x1440 and the rest of the params are same as the default on 4 V100 gpu cards , below are the issues i am facing

to obtain 1 epoch its taking 73 mins
summary writer of the the map is not coming correctly the value is 0.3 and staturated
do we have training and val acc plots

All the above details is from observations from 40 epoch training on COCO dataset

What's your train time and iter time? 73mins is quite a long time.

train_log.txt
@FateScript  please find the training log attached

your iter time is a bit long. Could  you plz check  iter time on 1 v100 gpu to locate the communication related issue.


Remove db.instance.id

Changes
Removes db.instance.id and reverts the Elasticsearch usage of it to db.elasticsearch.node.name.
Reason
In many database systems, the idea of db.instance.id corresponds to server.address or network.peer.address.
After adding db.namespace, we're currently only using db.instance.id in one place, and so I think it may be too early to try to add this generalized attribute.
Also, "instance id" is a potentially confusing name given the different meanings it has in SQL Server and Oracle.

I didn't find any usage of db.instance.id in existing instrumentation libraries in otel repos.

should we update schema as well?

should we update schema as well?

Can you clarify what you had in mind? Thanks

@trask @lmolkova I proposed this attribute and use it here.
https://github.com/open-telemetry/opentelemetry-ruby-contrib/blob/743796d425de3aa6c13fd89dffb3ad869106d261/instrumentation/trilogy/lib/opentelemetry/instrumentation/trilogy/patches/client.rb#L76
The reason why server.address or net.peer.name at the time did not work for us was discussed here: https://github.com/open-telemetry/semantic-conventions/pull/345
I suppose we should follow the pattern that elasticsearch uses and have a separate name under db.mysql then?

thanks for the context @arielvalentin !
I'd suggest using server.address for cluster address and network.peer.address for the individual node. You can provide either one or both of them together on one span.
In case of mySQL I assume what you have there
as @connected_host is the network address of the mysql node?
Looking at this comment https://github.com/open-telemetry/semantic-conventions/pull/345#discussion_r1337175786 it seems it is

The configured hostname would appear as server.address: mysql-db.example.com vs node instance db.mysql.instance.address: mysql-e26b99z.example.com

And then network.peer.address is a good fit:

server.address: mysql-db.example.com
network.peer.address: mysql-e26b99z.example.com

You may populate either or both of them for single-db deployments, just in a consistent way.
I believe elastic uses db.elasticsearch.node.name because it's not an address.
PS: we don't always pay attention to comments on the closed issues/PRs, so please don't hesitate to create new issues.

Understood. We're very behind on semconv and haven't started work on introducing stability opt-in flags so I've been reluctant to start to migrate individual attributes over so we can avoid being in a more unspecified state than we already are.
I'll see what we can do next quarter to address stability migrations.


Add framebuffer type

would be nice to know if dealing with singles of halfs. Maybe just on the same line
framebuffer -> framebuffer of singles (or floats, or halfs)

Indeed, that was on my todo list. I am surprise I haven't added to the issues.
Now you're mentioning it, that's the good time to add this feature soon :)

It's now on master, I'll make release including this feature next week.

Thank you


Amount rounding off within Hundred's

If i Intiate a payment for Rs. 1000. It is taking it has 100.00.

@anandkumar-omnipay  This issue is for which environment? "SIMULATOR"?

stale issue, closing


Rename --provenance to ---provenance-path

Just a suggestion.
Wdut?
@asraa @ianlewis

I have no preference here, but maybe it makes sense to differentiate between inline content and files

Shall we do this before v1 announcement or too risky :-)? (update doc, fix e2e test, fix blog post)


Changing actual tab characters with escaped ones

With some editors, these actual tab characters are implicitly replaced with spaces, causing the lexer to break on the next compilation. These changes may need to be applied to the other branches.

While I agree that this is a great idea, it's worth noting that unfortunately, strtok doesn't handle escape
sequences. What I would suggest however, is to do something like this:
sprintf(delimiter, " \t,");
tok = strtok(line, delimiter);
(I might to make a patch/pull request with this later on today).
For reference:
Example code: http://codepad.org/isjkDeZL (shows failure of strtok(..., "\t"))
In a forum post at http://www.velocityreviews.com/forums/t512229-strtok-as-string-replace-function-problem.html,
Thomas J. Gritzan writes, "strtok doesn't handle escape characters."

I did further checking and it turns out that I'm wrong. Escape characters can be used in strtok because apparently all escape characters are converted to their real values by the preprocessor. Sorry about the confusion.


is it possible to change  icon on android like we do on iOS

I have gone through these following points

[] Check latest documentation: https://docs.nativebase.io/
[] Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
[] Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
[] Check examples from NativeBase KitchenSink https://github.com/GeekyAnts/NativeBase-KitchenSink
[] For discussion purpose make use of NativeBase Slack: http://slack.nativebase.io/
[] For queries related to theme, check Theme Variables from Docs and live NativeBase Theme Editor http://nativebase.io/customizer/

Issue Description

node, npm, react-native, react and native-base version, expo version if used, xcode version
Expected behaviour
Actual behaviour
Steps to reproduce

Is the bug present in both iOS and Android or in any one of them?
Any other additional info which would help us debug the issue quicker.
###Important
If you want your issue to be looked at quicker, attach a snack reproducible with your issue. Makes it easier for us!

Hey @KiranC10
Please paste your code here for reference. I am facing issue on the android platform.
The picker icon is completely invisible.
I am using react-native: 0.63.4 and native-base: 2.13.15


Restore copies of sync.yml files

Description

Since conda/infrastructure is now juggling two types of sync workflows (the old sync.yml and the new update.yml) we need to preserve two sets of templates (see templates and the new sync folder added here) to serve to the downstream repos in order to avoid pushing templated files.

@kenodegard oops, that seems to have broken https://github.com/conda/infrastructure/actions/runs/10221061129


AHQ Store DEV UI

Expected To be Added Soon

:timer: Under progress...
Creating functionality with the dev options!

Ui Created

Dont close

In Progress...

UI Done ✅
Releasing in next major update


Layers for Content and Style Reconstruction

Hi all,
I have a short question. On the main page it says:
'We perform style reconstructions using the conv1_1, conv2_1, conv3_1, conv4_1, and conv5_1 layers and content reconstructions using the conv4_2 layer. As in the paper, the five style reconstruction losses have equal weights.'
But the layers used in the code are 'relu1_1', 'relu2_1' etc.
Are 'relu' and 'conv' just different names for the same thing? Or am I missing something here?
Thanks in advance!

The README is a little out of date.
However convX_Y and reluX_Y both refer to the same block of activations; the difference is whether or not they have been passed through the ReLU nonlinearity. Based on the equations in the paper, it seems that they use the rectified activations, which is why we do the same here.
Probably the conv activations would work as well; however to save memory, most CNN implementations (including torch) perform ReLU in place, so we can't actually reconstruct from the convX_Y layers unless we change ReLUs to not be in-place, which would increase memory usage quite a bit.

Awesome, thanks for the quick reply!
That makes sense. Thanks again!


[6.0] Set the library version reported via CMake to 6.0.0.

Explanation: Sets the version of the library as built in CMake to "6.0.0".
Scope: 6.0 releases built with CMake instead of SwiftPM
Issue: N/A
Original PR: https://github.com/swiftlang/swift-testing/pull/637
Risk: Low
Testing: Manually tested as we don't have CI using CMake yet. See https://github.com/swiftlang/swift/pull/76034
Reviewer: @compnerd @briancroom

@swift-ci test

@swift-ci test


Image Caption Script

Hi author, thanks for your work. Can you provide an example script for the image caption downstream task?
Thanks.

Hello, you may check 'Evaluation on Downstream Tasks' in README.md and find whether it's what you want?

Hello, you may check 'Evaluation on Downstream Tasks' in README.md and find whether it's what you want?

Hi Ferry,
Thank you so much for your reply. Sorry that I didn't explain clearly about the task.
The image caption task is the task mentioned in your supplementary file, where the evaluation metrics are BLEU, METEOR, Cider and SPICE.

Hello, you may check 'Evaluation on Downstream Tasks' in README.md and find whether it's what you want?

Hi Ferry, Thank you so much for your reply. Sorry that I didn't explain clearly about the task. The image caption task is the task mentioned in your supplementary file, where the evaluation metrics are BLEU, METEOR, Cider and SPICE. In addition, may I ask about the decoder experiments as I cannot see the setting of adding the optinal decoder. Thanks for your kind help.

I should be sorry that I actually mistake this as script for image caption task which is indeed for image/text retrieval. I'll delete my reply...Plus, I am also trying to apply an additional decoder as to applying model on image caption :)

Hhaha, Never mind.
In the paper, the authors compare the performance with and without the decoder. Also, they verify their model on image caption task.
Therefore, I guess there is an implementation for the relevant parts and ask for the script and details, e.g., the implementation details of the decoder.
Maybe we can have more discussion later.
Thanks for your kind information.

Hello, you can find the image captioning code in our new FIBER repo https://github.com/microsoft/FIBER/

Hello, you can find the image captioning code in our new FIBER repo https://github.com/microsoft/FIBER/

Thanks for your reply. I will take a look. Again, thanks for sharing the code.


Default filter bank to boost point-cloud quality.

The windows original TI voxel viewer achieves a very nice point-cloud quality (outlier free) out-of-the-box probably due to a nice toolchain of filters that are set by default. It'd be nice to include an example/test where the point cloud is already pre-processed by such filter bank. I'm doing some tests right now with TintinCDK so I'll be happy to contribute if I succeed

By default, there are no filters applied. However, a default camera profile
is applied which sets various calibrations. The default camera profile is
applied whether you use SDK or viewer.
SDK has API to select camera profile while viewer has a menu to select the
same.
~ Prasad Bhat
On 01-Feb-2016 9:24 PM, "byfron"<EMAIL_ADDRESS>wrote:

The windows original TI voxel viewer achieves a very nice point-cloud
quality (outlier free) out-of-the-box probably due to a nice toolchain of
filters that are set by default It'd be nice to include an example/test
where the point cloud is already pre-processed by such filter bank I'm
doing some tests right now with TintinCDK so I'll be happy to contribute if
I succeed
—
Reply to this email directly or view it on GitHub
https://github.com/3dtof/voxelsdk/issues/62.

Hi Prasad, thanks for your answer. So after playing around I realized the is the amplitude threshold the parameter that was removing the outliers in the windows gui, so no filtering indeed.


SyntaxError: Identifier 'devImages' has already been declared

After running gulp command getting this error:
SyntaxError: Identifier 'devImages' has already been declared     at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:169:18)     at callTranslator (node:internal/modules/esm/loader:272:14)     at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:278:30)     at async link (node:internal/modules/esm/module_job:78:21)

@agroho Should be fixed. Please pull latest commit.


[Bug] resouce packs wont show in game after downloading

Minecraft Version

[ ] 1.12.2 (End of support)
[ ] 1.16.5 (End of support)
[ ] 1.18.2
[ ] 1.19.2
[X] 1.20/1.20.1

What happened?

open maid gui
2.go to download resource packs
download a resource pack
says its downloading but its not showing in resource packs in game

Relevant log output
No response
Contact Details
<EMAIL_ADDRESS>

Due to network problems, downloading will be slow, so you need to wait for a long time. This is normal.
You cannot see the file until it is completely downloaded (this is a protection mechanism I designed)

Due to network problems, downloading will be slow, so you need to wait for a long time. This is normal.
You cannot see the file until it is completely downloaded (this is a protection mechanism I designed)

nope wanst that it was showing in the folder but not in game had to restart it


Improve check for openssl on macOS

Only call openssl version once;

Make the check work with future version of LibreSSL after 2.2.7. This assumes that Apple might keep updating LibreSSL shipping with macOS, but that they will never provide header files.

Followup to https://github.com/rbenv/ruby-build/pull/1095
/cc @hsbt @osyoyu

@jeremy That's a good point. If there wasn't ever a /usr/bin/openssl that passed this check (since it was "0.9.8" since forever), nor will it ever (now that we blacklisted LibreSSL), I guess we can drop the openssl version check entirely?

my understanding is following summary about OpenSSL on macOS.

< 10.11(El Capitan): macOS has openssl-0.9.8, but it's forked binary for Apple, so It's broken.

= 10.11(El Capitan): macOS was not provide OpenSSL header.

= 10.13(High Sierra): macOS changed LibreSSL from OpenSSL, but it was not provided header files same as El Capitan.(I do not confirm it yet)

So, OpenSSL/LibreSSL of macOS is completely broken. I agreed to use jeremy's condition for openssl veriication.


Failed to recompile original project with backdoor

Failed to recompile original project with backdoor

@niles1999 Please try verson 0.2.4.


Possible to handle Scalar bigger than 64 bits?

I am trying to add EVM (from Ethereum) as a processor.
EVM supports numbers up to 256 bits.
So there is for example the PUSH15 instruction, which pushes a 15 byte (120 bits) value to the stack.
I tried to implement it like this:
evm.sinc:
define token data120 (120)
     imm120 = (0, 119)
    ;

:PUSH imm120 is op=0x6e; imm120 {
    SP = SP - 15;
    *:15 SP = imm120;
}

But I get an exception from the Scalar class:
java.lang.IllegalArgumentException: Bit length must be >= 1 and <= 64
ERROR Command Failure: An unexpected error occurred while processing the command: Disassemble (BackgroundCommandTask) java.lang.IllegalArgumentException: Bit length must be >= 1 and <= 64
	at ghidra.program.model.scalar.Scalar.<init>(Scalar.java:62)
	at ghidra.app.plugin.processors.sleigh.SleighInstructionPrototype.addHandleObject(SleighInstructionPrototype.java:1474)
	at ghidra.app.plugin.processors.sleigh.SleighInstructionPrototype.getOpRepresentationList(SleighInstructionPrototype.java:718)
	at ghidra.program.database.code.CodeManager.addReferencesForInstruction(CodeManager.java:2915)
	at ghidra.program.database.code.CodeManager.addInstruction(CodeManager.java:674)
	at ghidra.program.database.code.CodeManager.addInstructions(CodeManager.java:531)
	at ghidra.program.database.ListingDB.addInstructions(ListingDB.java:412)
	at ghidra.program.disassemble.Disassembler.disassemble(Disassembler.java:610)
	at ghidra.program.disassemble.Disassembler.disassemble(Disassembler.java:473)
	at ghidra.app.cmd.disassemble.DisassembleCommand.doDisassembly(DisassembleCommand.java:223)
	at ghidra.app.cmd.disassemble.DisassembleCommand.applyTo(DisassembleCommand.java:157)
	at ghidra.framework.plugintool.mgr.BackgroundCommandTask.run(BackgroundCommandTask.java:101)
	at ghidra.framework.plugintool.mgr.ToolTaskManager.run(ToolTaskManager.java:315)
	at java.base/java.lang.Thread.run(Thread.java:834)

So my question:
Is it possible to add the EVM to Ghidra or are its numbers too big?
Probably problem like #206

Only varnodes can be of any number of bytes, even more than 8. The problem is in size of literals which must be splitted into smaller parts and then copied sequentially in some temporary register, not just unique, pcode varnode. Only then perform store operation. For example, here should be two unsigned imm fields so they are expanded by zero extension. They will be displayed as imm1, imm2, lower part in the right, higher in the left. Then they are copied to any dummy register and stored this way:
tmpImm[0,64] = imm2;
tmpImm[64,56] = imm1;
SP = SP - 15;
*:15 SP = tmpImm; # or *:15 SP = tmpImm[0,120];
Where tmpImm may be defined like this:
define register offset=0x100 size=15 [tmpImm];
Or you can do some bit shifts along with bitwise or over such numbers to achieve another result which will probably produce SUBs.


Streaming csv omits headers

Maybe I'm doing something wrong but the issue title hopefully explains it all. I'm trying to stream csv with stream_csv and everything works fine but the headers are not streamed. Let me know if I should provide a minimal reproduction

seems that instead of stream I tried to use stream_csv, which failed. Closing this

Oh, I think stream_csv is defined by Rails itself. Understandably confusing. 😅

@joeldrapper it seems that stream_csv is defined by Phlex::Rails::Streaming actually, see this, but I guess it doesn't work stream also sets some headers while stream_csv does not 🤔

Oh, it's part of the implementation! We should definitely hide that. We should probably standardize on naming all private methods with double underscores (__method__).

makes sense. I can make a pr for that later today 👍


Offer to fix missing http:// in website tag

A website tag without http:// is highlighted as an issue - it would be good if iD offered to "upgrade" or fix this by adding the protocol.
Also, you'll notice I had to scroll down to actually see the website field. Perhaps iD could move fields which are an issue higher up in the list, or even into the issues section just below the yellow box?

A website tag without http:// is highlighted as an issue

Would there be a way to make it check if the website has https first and then go with that instead of http if it does? If so, I think it would be a good idea to pick https when it's available.

I considered this when implementing the validation, but didn't follow it up for two reasons:

https is preferred when available, my understanding is that you'd need to send a request to every site to check that which seems like a bad idea (especially for a field that can have arbitrary values)
the value might not even be a website, in which case prepending the protocol only hides an issue that would hopefully be noticed when the user looks at the value

These don't necessarily mean a quick-fix can't be added, it would be convenient for users who're paying attention. I'm just not sure we can guarantee it's always a valid fix - in which case it seems unsuitable for a quick fix.

the value might not even be a website, in which case prepending the protocol only hides an issue that would hopefully be noticed when the user looks at the value

Well, if the website field is down at the bottom and you need to scroll down to see it, users might not even notice it.
Also, saying "invalid website" doesn't exactly explain the problem to users who might not realise http is needed - www.example.com looks like a valid website, so that needs some explaining.

I'm just not sure we can guarantee it's always a valid fix - in which case it seems unsuitable for a quick fix.

Not sure you can guarantee any fix in iD at the moment will always be a valid fix...?

A fix would be nice here, but it's true iD can't be sure whether a site is http or https. Sometimes they redirect to each other but sometimes they don't. There's possibly a safe way to query SSL status, but we can't just go loading any arbitrary site on the user's behalf.
I'm inclined to just remove this type of website validation check altogether. The data improvements seem relatively minor compared to the cost of prompting the user.

I can't see a safe way to query the site. Privacy forbids giving an arbitrary site the users personal information like they would get in an HTTP request

osmose checks for this error, gives the choice of both, and displays both so you can easily click and check the website: http://osmose.openstreetmap.fr/en/map/?#zoom=13&lat=55.93953&lon=-3.1811&item=3093&level=1%2C2%2C3&tags=&fixable=online
However before you remove the check I have some suggestions:

Tell the user why it is invalid - "missing http:// or https://" or similar

Move the invalid field up to the top under the issues heading, see below. Maybe could do the same with other error fields?

If it must be removed, then iD should highlight the form field with a red border when it is missing http - this happens already in Firefox but not Chrome.

I removed the warning. Perhaps eventually we can flag formatting at the field level instead of the feature level, as some browsers already do.


Theming: Double stylesheet file in node package

Bug
What is the expected behavior?
Building a custom theme with sass, gulp and gulp-sass
What is the current behavior?
When I try to build an own theme with sass I get a transpilation error because in material/core/overlay are both files overlay.css and overlay.scss and so sass doesn't know which it should pick.

What are the steps to reproduce?
Follow the steps at [https://github.com/angular/material2/blob/master/docs/theming.md#defining-a-custom-theme](your theming guide)
Run a gulp task to compile the scss files to css. Use at least the following devDependencies
{
   // ...
   "gulp": "^3.9.1",
   "gulp-sass": "^2.3.2",
   "sass": "^0.5.0",
   // ...
}

Which versions of Angular, Material, OS, browsers are affected?
Angular: 2.1.0
Material: 2.0.0-alpha.9 cobalt-kraken
OS: Microsoft Windows [Version 6.3.9600] and [Version 6.1.7601]
other versions see in above

Yeah, overlay.scss should be a partial (_overlay.scss).


[Beam-59] Remove IOChannelFactory usage from Numbered/ExplicitShardedFile

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @dhalperi

Coverage increased (+0.01%) to 69.806% when pulling 97aae4f5c61383725ff0f368e7ce4f955c9d6aba on vikkyrk:numberedshardedfile into 14a90d7c5196d96de66e897687d28202d790439f on apache:master.

Also added ExplicitShardedFile

Coverage increased (+0.02%) to 69.682% when pulling 9f4ebfcad9cddc3793e8b919865627a4da6f83c8 on vikkyrk:numberedshardedfile into 948f7ef2fc37b30809cf1adb2631eeb6a1e5eff3 on apache:master.

Coverage increased (+0.02%) to 69.681% when pulling 9f4ebfcad9cddc3793e8b919865627a4da6f83c8 on vikkyrk:numberedshardedfile into 948f7ef2fc37b30809cf1adb2631eeb6a1e5eff3 on apache:master.


Replace the assert statement in connection::Connection::put with an error packet and terminate the transmission

A misbehaving peer could send us an ack with the wrong block number; however, panicking via an assert is not a helpful way to deal with this.
We don't need to go out of our way to try to get the connection back on track. Create a courtesy Error packet, send it to the peer, and then return early from this method with an err.

Hey Connor, I would like to work on this!

Awesome! Thanks 🙂


Translation for templates and forms

I found two <blocktrans> tags in the templates, but they were not done extensively. I was about to undertake the task of tagging more of these texts with {% trans%} tags, as well as using ugettext_lazy  in python files. I am thinking about using _ notation:
from django.utils.translation import ugettext_lazy as _

I was wondering how the maintainer wants to handle the resulting .po files. They will diverge from the json files that Numbas compiler generates. If kept separate, it will result in a diverged effort in translating on poeditor.com; combining them could be a maintenance hassle and an extra dependency. Two repositories are already coupled, and perhaps guidance on how de-coupled they need to be would be helpful.
The primary motivation behind the effort is how I disagree with the term "Group," "Exam," "Question." I want to use "Section," "Deck," and "Slide" instead. I think they are more generic. For URL endpoints, LocalMiddleware can be used, but this is not a priority.
If there are other details I should read over, let me know! I will catch up with the documentation soon.

Yes, there isn't any real translation at the moment. I haven't prioritised it, but it should be done at some point.
I think the Numbas runtime translation can be considered completely separate to the editor. I certainly don't want to have to include strings that are only to do with the editor in the Numbas runtime.


SDK TextRewriteTool doesn't work in 2.17.3

In 2.17.3 the premade tool text_rewrite_tool gives error when trying to use it:
File "C:\Code\python\steamship\agent-starter\src\api.py", line 88, in prompt
self.run_agent(self.incoming_message_agent, context)
File "C:\Python311\Lib\site-packages\steamship\agents\service\agent_service.py", line 59, in run_agent
self.run_action(action=action, context=context)
File "C:\Python311\Lib\site-packages\steamship\agents\service\agent_service.py", line 25, in run_action
blocks_or_task = action.tool.run(tool_input=action.input, context=context)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python311\Lib\site-packages\steamship\agents\tools\text_generation\text_rewrite_tool.py", line 52, in run
output_blocks = llm.complete(prompt=prompt)
^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'complete'

@MikPoik I believe that is saying that llm is None. To use it in an agent, you need to set an LLM in the context.
Example (from the main in that tool): context=with_llm(llm=OpenAI(client=client)).
@eob this is related to your get_or_default question, earlier, I believe.

@MikPoik thank you for catching this.
I think the tool main  methods fell out of sync with some of the changes in the base SDK and it didn't yet have unit test coverage to block the merge.
We're pushing some changes today to fix and will update you and close the issue when they're in.
Thanks again!


Intermittent Error when running dbt models

Describe the bug
intermittent  Error:
When runiing dbt run --model +tfm_stock_history_major_currency
Running with dbt=0.19.1
[WARNING]: Configuration paths exist in your dbt_project.yml file which do not apply to any resources.
There are 1 unused configuration paths:

models.dbt_hol.l30_mart

Found 7 models, 0 tests, 0 snapshots, 0 analyses, 316 macros, 0 operations, 0 seed files, 2 sources, 0 exposures
Encountered an error:
Runtime Error
cannot import name 'SnowflakeOCSPAsn1Crypto' from partially initialized module 'snowflake.connector.ocsp_asn1crypto' (most likely due to a circular import) (c:\snowflake\lab1\dbt-env\lib\site-packages\snowflake\connector\ocsp_asn1crypto.py)
URL of where you see the bug
https://guides.snowflake.com/guide/data_engineering_with_dbt/#7
To Reproduce
Steps to reproduce the behavior:

run command:  $ dbt run --model +tfm_stock_history_major_currency

Expected behavior
Running with dbt=0.19.1
[WARNING]: Configuration paths exist in your dbt_project.yml file which do not apply to any resources.
There are 1 unused configuration paths:

models.dbt_hol.l30_mart

Found 7 models, 0 tests, 0 snapshots, 0 analyses, 316 macros, 0 operations, 0 seed files, 2 sources, 0 exposures
21:04:41 | Concurrency: 200 threads (target='dev')
21:04:41 |
21:04:41 | 1 of 6 START view model l10_staging.base_knoema_stock_history........ [RUN]
21:04:41 | 2 of 6 START view model l10_staging.base_knoema_fx_rates............. [RUN]
21:04:44 | 2 of 6 OK created view model l10_staging.base_knoema_fx_rates........ [SUCCESS 1 in 2.62s]
21:04:44 | 3 of 6 START table model l20_transform.tfm_fx_rates.................. [RUN]
21:04:44 | 1 of 6 OK created view model l10_staging.base_knoema_stock_history... [SUCCESS 1 in 2.69s]
21:04:44 | 4 of 6 START view model l20_transform.tfm_knoema_stock_history....... [RUN]
21:04:46 | 4 of 6 OK created view model l20_transform.tfm_knoema_stock_history.. [SUCCESS 1 in 2.41s]
21:04:46 | 5 of 6 START view model l20_transform.tfm_stock_history.............. [RUN]
21:04:49 | 5 of 6 OK created view model l20_transform.tfm_stock_history......... [SUCCESS 1 in 2.47s]
21:04:52 | 3 of 6 OK created table model l20_transform.tfm_fx_rates............. [SUCCESS 1 in 7.91s]
21:04:52 | 6 of 6 START view model l20_transform.tfm_stock_history_major_currency [RUN]
21:04:55 | 6 of 6 OK created view model l20_transform.tfm_stock_history_major_currency [SUCCESS 1 in 2.79s]
21:04:56 |
21:04:56 | Finished running 5 view models, 1 table model in 19.96s.
Completed successfully
Done. PASS=6 WARN=0 ERROR=0 SKIP=0 TOTAL=6
Screenshots

Additional context
rerunning the prevevous model from step 6 seemed  help.. but not sure why

Also seem to happen when dbt run  (again it intermittent)  but it seem to fail more that work

This seem be realted to my install
Building wheel for tree-sitter (PEP 517) ... error
ERROR: Command errored out with exit status 1:
command: 'c:\snowflake\lab1\dbt2\scripts\python.exe' 'c:\snowflake\lab1\dbt2\lib\site-packages\pip_vendor\pep517\in_process_in_process.py' build_wheel 'C:\Users\eskim\AppData\Local\Temp\tmpxclxl9z_'
cwd: C:\Users\eskim\AppData\Local\Temp\pip-install-ct586pwj\tree-sitter_f722a907dbad4e07b56a4e7839cd794a
Complete output (10 lines):
running bdist_wheel
running build
running build_py
creating build
creating build\lib.win-amd64-3.9
creating build\lib.win-amd64-3.9\tree_sitter
copying tree_sitter_init_.py -> build\lib.win-amd64-3.9\tree_sitter
running build_ext
building 'tree_sitter.binding' extension
error: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
ERROR: Failed building wheel for tree-sitter
Failed to build tree-sitter
ERROR: Could not build wheels for tree-sitter which use PEP 517 and cannot be installed directly


Fix issues creating endpoints and with wireup

Fix bug with create_endpoint, where a UCXMessageTruncatedError may not be properly raised, as well as its use in distributed-ucxx not raising the exception expected by distributed.
Additionally, fix wireup issues in tests, where only sending (or receiving) a wireup message does not seem to suffice, but sending and receiving suffices to prevent errors in tests.

/merge

Thanks for the approval @wence- !


Fix windows and AZERTY shortcuts for undo redo

Follow up of https://github.com/mui/mui-toolpad/pull/1225 - add support for windows key shortcut
Partial of https://github.com/mui/mui-toolpad/issues/226

We're currently using Cmd+S in some places, does it have the same problem?
Any way we can abstract the platform specific behavior so that we don't have to write this everywhere we add a shortcut?

We're currently using Cmd+S in some places, does it have the same problem?

Any way we can abstract the platform specific behavior so that we don't have to write this everywhere we add a shortcut?

Those places are using useShortcut, so my assumption is that when we do metaKey we also wanna include ctrlKey - this change should kinda abstract this so places that use cmd + s should also work with ctrl + s now. Does that make sense?

@oliviertassinari please take a look again when you get a chance

Personally, I would recommend using destructuring as least often as possible as it makes it harder to run quick regex search on the codebase

Regarding this - IMO that warrants a broader discussion that might be outside of scope for this PR. "as least often as possible" sounds pretty subjective, I'm not even sure how we would apply that in practice, it might either lead to inconsistent usage or simply we could go to other extreme of not using it anywhere 🤔

I would be curious about the team's takes on the merit of this coding style

I don't have strong opinions about the general use of destructuring.
In this particular instance, personally, I find that it reduces the overall readability. a property named code has more context when attached to event than when free floating. It all feels a bit unnecessary here.

@oliviertassinari @Janpot destructuring removed, please take a look


test: invoke python scripts with the interpreter

Ensure that we explicitly provide the path to the interpreter when
running the python scripts during the tests.  This is needed to have the
tests work on Windows which does not honour the shebang in the file.  It
also ensures that we use the correct interpreter for the tests.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @graydon @jrose-apple

@swift-ci please test and merge


Add Hash#compact and Hash#compact!

Ruby 2.4 has Hash#compact and Hash#compact!. This pull request introduces them into Crystal.
Hash#compact makes a new Hash without all nil values:
h1 = {"hello" => "world", "foo" => nil}
h1.class # => Hash(String, String | Nil)
h2 = h1.compact
h2       # => {"hello" => "world"}
h2.class # => Hash(String, String)

And, Hash#compact! removes all nil values from self, and returns self or nil if it were made no changes like Hash#reject!:
h1 = {"hello" => "world", "foo" => nil}
h1.compact! # => {"hello" => "world"}
h1          # => {"hello" => "world"}
h1.class    # => Hash(String, String | Nil)
h1.compact! # => nil

Their behaviors are same as Ruby.

I don't think compact and compact! are very useful in Crystal, because nil as a key/value in array/hashes is not very common, but for the few cases where one uses that (maybe as a result of a map or other operations) it can be useful. So let's merge this :-)
Thank you!


try fedors dashing fork

Description
Replace this paragraph with a description of what this PR is doing. If you're modifying existing behavior, describe the existing behavior, how this PR is changing it, and what motivated the change. If you're changing visual properties, consider including before/after screenshots (and runnable code snippets to reproduce them).
Related Issues
Replace this paragraph with a list of issues related to this PR from our issue database. Indicate, which of these issues are resolved or fixed by this PR. There should be at least one issue listed here.
Tests
I added the following tests:
Replace this with a list of the tests that you added as part of this PR. A change in behavior with no test covering it
will likely get reverted accidentally sooner or later. PRs must include tests for all changed/updated/fixed behaviors. See Test Coverage.
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I signed the CLA.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] All existing and new tests are passing.
[ ] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[ ] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

It should. I built it yesterday in my fork but the task failed with some weird issues and I got distracted by some other things. Could you please also try to set USE_IN_MEMORY_DISK to true?

It should. I built it yesterday in my fork but the task failed with some weird issues and I got distracted by some other things. Could you please also try to set USE_IN_MEMORY_DISK to true?

ahh, ok, maybe it's pulling in the cache from your test yesterday. switching to in memory disk.

🤞

Also, it looks like we (maybe me) updated docs.sh after disabling dashing on pre-submit, so if all goes well this will fail at the end trying to deploy. Still, we should be able to benchmark docset generation time, and if we land this we can just update the script to handle PRs.

@fkorotkov switching to memory disk resulted in OOM killed

You can try to bump to 12G. 🤔 Don't think it will need more then 4G.

The task failed, but because of an unrelated bug: https://cirrus-ci.com/task/6251878486900736.
However, it concerns me that it took so long, 54:39. This doesn't seem faster than upstream Dashing. Any thoughts @fkorotkov ?


Numbers DSL deprecation

Description
Deprecate the DSL for building numbers in the kotools.types.number package.
Checklist

[x] Deprecate the DSL.
[x] Test.
[x] Refactor.
[x] Update Work in progress section in changelog.

Done.


Verbs are not working? [internal properties are not working]

This code and calling it with arguments "add test1 test2"
[Verb("add", HelpText = "Add repository to the group.")]
internal class AddVerb
{
	[Value(0, Required = true)]
	internal string RepositoryPath { get; set; }

	[Value(1, Required = true)]
	internal string GroupName { get; set; }
}

Parser.Default.ParseArguments<AddVerb, object>(args)
			.WithParsed<AddVerb>(v => AddRepositoryToGroup(v));

calls AddRepositoryToGroup but all v members are null.
If I add option
[Verb("add", HelpText = "Add repository to the group.")]
internal class AddVerb
{
	[Option]
	internal string Opt { get; set; }

	[Value(0, Required = true)]
	internal string RepositoryPath { get; set; }

	[Value(1, Required = true)]
	internal string GroupName { get; set; }
}

Parser.Default.ParseArguments<AddVerb, object>(args)
			.WithParsed<AddVerb>(v => AddRepositoryToGroup(v));

and call it with arguments "add --Opt asdf test1 test2"
i get option Opt is unknown.

Figured out, internal properties are not working for some reason.

Properties should be public to be set by Parser during Reflection.  Internal Properties are not supported.

Properties should be public to be set by Parser during Reflection. Internal Properties are not supported.

Maybe that should be specially stated in the documentation, because at first I had a wtf moment...
Internal classes work, internal properties do not.

Wiki Documentation are updated  to state public properties.


add Bcast

Tests broken because of jax#5463

Tests broken because of jax#5463

Tests pass locally. I will merge.
Jax is still broken for AD of anything with tokens until a new release will be out.

Tests pass locally. I will merge.
Jax is still broken for AD of anything with tokens until a new release will be out.


WildPokemon either empty or small amount.

The WildPokemon array seems to always either empty or small amount.
[{"EncounterId":{"low":570066557,"high":1837351680,"unsigned":true},"LastModifiedMs":{"low":580492510,"high":342,"unsigned":false},"Latitude":52.07300839137805,"Longitude":-1.3258175187096903,"SpawnPointId":"487726438ff","pokemon":{"Id":null,"PokemonId":90},"TimeTillHiddenMs":655742}]
Have checked with mobile app and other location based tools and there should be at least a few more. The NearbyPokemon array doesnt match up either and says there are more.
This is the code I am using -
for (var i = hb.cells.length - 1; i >= 0; i--) {
    if(hb.cells[i].WildPokemon[0])
    {
        console.log(JSON.stringify(hb.cells[i].WildPokemon));
     }

and for nearby -
for (var i = hb.cells.length - 1; i >= 0; i--) {
    if(hb.cells[i].NearbyPokemon[0])
    {
        console.log(JSON.stringify(hb.cells[i].NearbyPokemon));
     }

Also, while I'm here-
What are the "high" and "low" attributes of some fields, i.e. EncounterId?
Thanks

This library only returns nearby pokemon. Your client needs to update the location to scan for pokemon that are farther away
EncounterId is a 64-bit integer See https://github.com/dcodeIO/long.js/issues/34 for how to convert it back to base64

How do you 'update the location' setlocation doesn't update the location as it stays the same in the heartbeat

Since a few days, I have the same issue. I have a script and didn't change it's content, but since a few days the script only shows 1-3 Pokemon. Before I've had about 20 Pokemon for the same radius (and yes I did update the location to increase the radius).


Incorrect request URL?

From logs:  GET /piwik/piwik.php/?_cvar
The last "/" should be remove, I think.

Good comment.
The / is added by AFHTTPSessionManager in order support appending adding additional path components to the base url when building the request. Since the base URL already contains the full path nothing gets appended and the hence the trailing /.
Do not think it matters much but you are right.
Fixed by separating out the path when constructing the AFHTTPSessionManager and including it each time as path component when building the request.


Error: SignTool SIGN: Password is required with Certificate path but not specified.

What You Are Seeing?
When I use the Sign method with a CertPath specified but no Password, Sign fails with the following error:

Error: SignTool SIGN: Password is required with Certificate path but not specified.

What is Expected?
It should sign the file; the certificate I use has no password (it's a self-signed certificate for dev purpose). signtool.exe doesn't enforce a password when using a certificate file.
What version of Cake are you using?
0.18.0
Are you running on a 32 or 64 bit system?
64 bit
What environment are you running on?  Windows? Linux? Mac?
Are you running on a CI Server?  If so, which one?
No
How Did You Get This To Happen? (Steps to Reproduce)
void SignFiles(IEnumerable<FilePath> paths)
{
    var settings = new SignToolSignSettings
    {
        DigestAlgorithm = SignToolDigestAlgorithm.Sha256,
        TimeStampUri = new Uri(timeStampUrl),
        TimeStampDigestAlgorithm = SignToolDigestAlgorithm.Sha256
    };

    if (!string.IsNullOrEmpty(certificateThumbprint))
    {
        // On build server, use the real certificate; the thumbprint is in an environment variable.
        settings.CertThumbprint = certificateThumbprint;
    }
    else
    {
        // On dev machine, use a self-signed dev certificate
        settings.CertPath = devPfxFile.FullPath;
    }

    Sign(paths, settings);
}

Output Log
https://gist.github.com/thomaslevesque/79b70af96b15440d363c24f6942debce

Looking at code it password is currently required by Cake, but as you say you can have passwordless certs.
https://github.com/cake-build/cake/blob/0632cae5ae8d0cf1af925c794d31db58aa458460/src/Cake.Common/Tools/SignTool/SignToolSignRunner.cs#L150-L155
            if (settings.CertPath != null && string.IsNullOrEmpty(settings.Password))
            {
                const string format = "{0}: Password is required with Certificate path but not specified.";
                var message = string.Format(CultureInfo.InvariantCulture, format, GetToolName());
                throw new CakeException(message);
            }

@thomaslevesque is this something you would like to send a PR for?

@thomaslevesque is this something you would like to send a PR for?

Absolutely (in fact I intended to, but only had time to submit the issue). It should be simple enough to fix, so you can probably expect a PR later today.

@thomaslevesque said...
but only had time to submit the issue

Not a problem.  This is exactly the correct process 👍

@devlead should we class this as a bug fix, and release 0.18.1?

@gep13 It's been like this since start, so don't think a hotfix is needed - we can ship it with next version, we can push some of the v0.19.0 features to v0.20.0  and release it sooner than later,


DNB requirement: need a single packed file of all content

Is your feature request related to a problem? Please describe.
The DNB (german national library) uses the OAI-PMH metadata formats to harvest URN's and metadata and content of the document repos of german libraries. If there is more than one file belonging to a document a zipped version of all the content is required (they accept only one transfer url).
Describe the solution you'd like
Provide a worker to zip content. Add a marker in the file metadata to make the transfer file recognisable.
Describe alternatives you've considered
Perhaps there is the possibility to do a on-the-fly compression while delivering the file. A virtual download URL to trigger the compression and download would be needed (it's CPU vs. space).
Additional context
xMetaDissPlus spec
[xepicur (german)]https://wiki.dnb.de/display/URNSERVDOK/xepicur+-+XML-Datentransferformat+zur+Verwaltung+von+URN

related: we would like to see a "Download all" button in the user-interface

This has to be a .zip? Because if not, this might also be a OCFL-usecase?

related: we would like to see a "Download all" button in the user-interface

See also: https://github.com/inveniosoftware/product-rdm/issues/86

This has to be a .zip? Because if not, this might also be a OCFL-usecase?

No, it hasn't be a .zip, just phoned with DNB as here are no docs available ragarding this question. Allowed format are tar, tar.gz, .gz, .zip and .7z.

...but it's still relevant when you look at this comment. I don't know how realistic this is, though.

It depends largely on what exactly you want to achieve. The DNB just wants exactly one URL they can call to download all the content at once. That would also be a sufficient solution for  if you don't want to select what's in the download file beforehand.

See also proposed changes to DataCite v4.5 for the new distribution tag: https://datacite.org/documents/DataCite_Metadata_Schema_4.5_RFC.pdf


Cleanup cluster operator options

Signed-off-by: Zhonghu Xu<EMAIL_ADDRESS>What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

/hold It is a mess from goland refactor


Avoid removing newlines between docstring headers and rST blocks

Given:
def func():
    """
    Example:

    .. code-block:: python

        import foo
    """

Removing the newline after the Example: header breaks Sphinx rendering.
See: https://github.com/astral-sh/ruff/issues/11577

\cc @harupy

@charliermarsh thanks for fixing this!