[CT-1415] [Feature] Add adapter-specific aliasing for Google pseudo columns

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt-bigquery functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Background:
Google has a concept of metadata "pseudo columns" which are columns that can be queried in BigQuery, but do not materialize in the table itself.
Some examples of these columns are _PARTITIONDATE, _PARTITIONTIME, _FILE_NAME,  _TABLE_SUFFIX...etc, which always start with an underscore. _FILE_NAME for example, will be the name of the file being read in from Google Cloud Storage when an external data source is configured.
Issue:
Because these pseudo columns start with underscore, applying dbt tests to them is difficult.
For example, dbt_utils.unique_combination_of_columns and the default dbt uniqueness test select columns as is and create the dbt__test_audit table schema based on the selected columns.
Since columns with leading underscores are not allowed in BigQuery, applying uniqueness tests to these columns as is throws an error. The columns need to be aliased in order for test results to be stored in dbt__test_audit tables.
Describe alternatives you've considered
I've considered:

Writing a macro within my organizations dbt project to perform this aliasing, however because the aliasing would be used in tests defined in dbt-core and external packages, this doesn't feel like the best solution. In addition, being able to test on pseudo columns will likely benefit other dbt-bigquery users.

Aliasing pseudo columns in base models and materializing them there. Tests can then be run on the base model instead of on the source layer. The issue with that solution is we'd essentially be replicating data simply to add a uniqueness test.

Who will this benefit?
All dbt-bigquery users that use external data sources and want uniqueness testing on their source layer.
Are you interested in contributing this feature?
Yes. I will be submitting a PR for a potential solution!
Anything else?
No response

@ashleemtib Thanks for identifying the problem, offering a clear write-up, and even opening a PR with some proposed code to get to a solution!
I have a few questions about the user experience we should be aiming for. I'm also not sure that we'll be able to handle this pseudo-column aliasing behind-the-scenes in some cases. As a general principle, I'd rather opt for more-explicit, less-ergonomic code, over an abstraction that risks leaking.
As far as the second alternative you've proposed:

Aliasing pseudo columns in base models and materializing them there. Tests can then be run on the base model instead of on the source layer. The issue with that solution is we'd essentially be replicating data simply to add a uniqueness test.

Does this work if the base model is materialized as a view (or ephemeral)? I get that it adds an extra step into the modeling layer, but it at least avoids the need to store the data twice.

Sorry I'm late to the party here ;)
I need to make sure I understand how these columns are used, can you please confirm I got this right?
Let's say I want to materialize this model:
SELECT
  Column1,
  Column2,
  _PARTITIONDATE,
  _PARTITIONTIME
FROM {{ref('upstream_model_A')}}

The thing is if I define tests on these columns, some tests will break. The reasons are:

these tests will generate audit tables, re-using the column names of the model (Column1, Column2, _PARTITIONDATE and _PARTITIONTIME)
_PARTITIONDATE and _PARTITIONTIME are reserved names that can't be used as user defined column names

My naive take would be to manually update the model to:
SELECT
  Column1,
  Column2,
  _PARTITIONDATE AS my_model_PARTITIONDATE,
  _PARTITIONTIME AS my_model__PARTITIONTIME
FROM {{ref('upstream_model_A')}}

This is your second alternative:

Aliasing pseudo columns in base models and materializing them there. Tests can then be run on the base model instead of on the source layer. The issue with that solution is we'd essentially be replicating data simply to add a uniqueness test.

Which would work, but is not possible for sources (since there is no SQL file). Then what I would really want here is an identifier property  at the column level, to alias the column at the YAML level. @jtcohen6 is that something that was ever considered?
sources:
  - name: <source_name>
    tables:
    - name: <table_name>
      columns:
        - name: <column_name>
          description: <markdown_string>
          data_type: <string>
          alias/identifier: <string>
        - name: <another_column>
          ...

That's straightforward at the source level, but what does that mean if the model is a table. Are we overriding the name in the SQL query there?
The workaround here being to create a model on top of it, like a view, to hosts the tests. But that's more code and more objects, so not the best.

@Fleid your understanding is correct! It makes a lot of sense to use your "naive" take for aliasing the column in the model itself so this error does not occur, and that is what we are doing in our dbt project currently; however, this issue pertains specifically to testing on dbt sources, not models. I don't think I explicitly stated that, so my bad!
That being said, since we can't alias the column names in the sources, we'd need a way to do that. I like the idea of adding identifier to the yml to be used by the test, and I think it's the only solution we need in this case unless you all think it would be better to expand this to models as well. Thank you for your response!

I like that idea, I'm not sure how feasible it is but I like it.
After all, "sources make it possible to name and describe the data loaded into your warehouse by your Extract and Load tools" (from here). Let's name and describe columns and tables.
I'll create a new issue over to dbt-core, so @jtcohen6 can't ignore it, and close this one :)

There : https://github.com/dbt-labs/dbt-core/issues/6929


Project dead?

@almeynman you still actively developing the plugin? Would like to pick it up if you don't mind.

@leovanhaaren Thanks! Please do. Currently have no time to work on it.

@almeynman Would you be willing to accept PRs, or should we start a new module?

@pksunkara I am willing to accept PR. I am also willing to transfer ownership of this project if you are interested

I am willing to maintain this and improve this.

@almeynman Thanks for transferring the repo. I will be working on fixing the broken stuff this week. Could you also add me as owner on npm? https://www.npmjs.com/package/vue-cli-plugin-storybook

@pksunkara I have added you as admin. When you confirm, I will remove myself

Confirmed. Thanks for everything.


Update skaffold v2.0.2 windows release sha256 hash to be correct

The skaffold team republished the v2.0.2 windows binary due to an issue with signing the binary which was later resolved.  The sha256 value that the github action picked up is no longer correct as a result of this.  This PR contains the correct hash.
Skaffold v2.0.2 windows binary from our release page has the value in this PR:
https://github.com/GoogleContainerTools/skaffold/releases/download/v2.0.2/skaffold-windows-amd64.exe
https://github.com/GoogleContainerTools/skaffold/releases/download/v2.0.2/skaffold-windows-amd64.exe.sha256
aprindle@aprindle-ssd ~/Downloads $ sha256sum skaffold-windows-amd64.exe
7a1202a0674c7600d16b9f05c96185aaec01b6a1108b56891d0344dbf2dbd3a0  skaffold-windows-amd64.exe
aprindle@aprindle-ssd ~/Downloads $ cat skaffold-windows-amd64.exe.sha256
7a1202a0674c7600d16b9f05c96185aaec01b6a1108b56891d0344dbf2dbd3a0  skaffold-windows-amd64.exe

Thanks for the review here @zrma! lmk if there is anything I can do on my end to help expedite this - want to make sure our Scoop users can download Skaffold, thanks!

Can we please merge this it's making it difficult to upgrade my other installed apps with scoop update *

Friendly ping, are there any updates here or an ETA on when this might be merged?  Skaffold's Scoop users are blocked unable to use the latest Skaffold atm.

While I think this is a perfect PR, I'm not sure why code review by maintainers has been delayed...
I hope the maintainers approve it.


"Yandex Panoramas" is not suspect but "Yandex" is

I'm submitting a feature request
Brief Description
“Yandex Panoramas” is now an ok source for OSM. However “Yandex” on its own is still suspect, since that could be imagery or their maps
Not sure if this means change to the logic in
https://github.com/willemarcel/osmcha/blob/master/osmcha/changeset.py#L153
Suspect word list is at https://github.com/willemarcel/osmcha/blob/d9a8bc4fd55b859396a65e2e17863c6b4e93a5ce/osmcha/suspect_words.yaml#L12
What is the motivation / use case for this feature?
Yandex Panoramas are now an ok source
What  is the current behaviour, (attach relevant screenshots) ?
Anything with "Yandex" in it gets flagged

Fixed with:
https://github.com/willemarcel/osmcha/commit/f2fd7ea116ee3d26d81798d808ae7e44a4ab95f6
https://github.com/willemarcel/osmcha-django/pull/190


export: Use undeprecated fs.existsSync

fs.exists expects a callback so it won't work correctly anyway
path.exists might be removed in later change too
Change-Id: If6a07a0148df78525a7689cc557c556f10b039e6
Signed-off-by: Philippe Coval<EMAIL_ADDRESS>

For the record this is released in:
https://www.npmjs.com/package/gpio/v/0.2.10
maybe we could also tag the version (vX.Y.Z) before uploading to npm.


iPad, iPhone move tiles not correct

Moving tiles on iPhone or iPad is not possible, moved tile in horizontal position working, but if we moving tile in vertical position then, after touch the tile and hold them and moved in vertical position the tile is not moved, only whole page is moved.

Thanks, this is a know issue I don't plan on fixing. The UI disables tile dragging on mobile by default (on wider screens you might be able to unlock, but it's still not something I support).

OK
all clear.

Yeah I'm limited by what third party libraries are capable of when it comes to drag-and-drop. I haven't come across one that actually works 100%  in a mobile browser.

No problem,
in other thing iPadOS now uses full desktop Safari browser.


Fix memory leak in HttpPostMultipartRequestDecoder

Motivation:
We need to release all ByteBufs that we allocate to prevent leaks. We missed to release the ByteBufs that are used to aggregate in two cases
Modifications:
Add release() calls
Result:
No more memory leak in HttpPostMultipartRequestDecoder

/cc @fabienrenaud

Let me merge this one as it is an easy one...


Cannot enter "between" range on axis

Both X- any Y axis allow specifying "Between" as range selection. However there's nowhere to enter the min/max values for between:

I'm using v0.0.4.

I fixed this in 0.0.5... hopefully released soon :)
You can get it from master directly if you want

Great, thank you! Waiting for grafana to publish...

Same problem still. When selecting "between" no additional input fields are shown. Could you show how this is expected to look?

@ryantxu as this is still a problem could you please reopen so it doesn't get lost?

sorry -- it is fixed in master, but 0.0.5 got released before this was included.  Still fixed, but needs to be released.
You can use the master branch though


update sppull to 2.5.1

Thanks @tknaller

thank you very much for accepting my pull request. can you update the extension on the vscode marketplace?

yes, I will release this today.

awesome thank you

I just published SPGo 1.4.5 to the Marketplace.


laravel 8 + typo in endpoint

Next to making it compatible with Laravel 8, I've committed a correction on a typo (or Messagebird API change).
Hope you are interested in merging it.

Hi @fgd007
Thanks for the PR, however I am not maintaining this package anymore.
My suggestion is to use messagebirds package https://github.com/messagebird/php-rest-api or to fork this repo to continue work on it.


Optimization and bug fix

Optimized the callback code very slightly and fixed #28.

Hi Jemar, to be able to build you need to install Node JS. Then install Grunt (which is a build tool) and then run grunt in the main project folder. Have a look at the section Development.
To make a release:

Update version numbers in package.json and bower.json
Update version numbers in README.md
Add a small change log to README.md
Use Grunt to build the dist files
Check everything in and push to the repo
Then, inside Github, on tab Releases, press the button Draft a new release
Fill in the new version number for the tag
Add a title and a description (I copy this from README.md and edit a bit)
Upload these 3 files to the release:

dist/jquery.flip.js
dist/jquery.flip.min.js
dist/jquery.flip.min.js.map

Press the Publish button and you're done!

Manually merged and applied. Is in release v1.0.12.

Thanks! Can I do all/any of that on a branch before creating a PR without bower and the main release being updated prematurely?

Yes I think you can... Apart from the Release in GitHub step, I'm not sure about that one.

Oh I should add that there is a 'magic' branch called gh-pages that contains the site that's at https://nnattawat.github.io/flip. So you should, after the release is finish, checkout that branch and update the version numbers in the index.html that is in there. Also, if you added a nice feature, or some options, you can add docs / examples of it there. Once you push that the site gets updated automagically.


Error: Could not connect to device, handshake timeout

[Xiaomi Roborock S50 Vaccum 2nd Generation] ERR getDevice | miio.device, next try in 2 minutes | Error: Could not connect to device, handshake timeout.
wifi not changes, token same too. before all works as well. all packages version is last. What's wrong?

Do you have with the current version still problems?

@nicoh88 having the same error:
Jul 06 20:59:11 home.r.ringerl.de homebridge[10301]: [2019-7-6 8:59:11 PM] [Xiaomi Roborock S50] ERR getBatteryLow | No vacuum cleaner is discovered yet.
Jul 06 20:59:11 home.r.ringerl.de homebridge[10301]: [2019-7-6 8:59:11 PM] [Xiaomi Roborock S50] INF ensureDevice | undefined | The socket was destroyed or not initialised, initialising the device
Jul 06 20:59:34 home.r.ringerl.de homebridge[10301]: [2019-7-6 8:59:34 PM] [Xiaomi Roborock S50] ERR connect | miio.device, next try in 2 minutes | Error: Could not connect to device, handshake timeout

miio version:
npm list -g miio
/usr/lib
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>(git://github.com/nicoh88/miio.git#19e3660a4ed9cc4019b0ef887e432ea757eec922)

IP address and xiaomi token should be correct ... any ideas?

Try another Version. Like 0.50

Wrong token? Wrong IP?

sorry it was indeed the wrong IP address - my fault! everything working now ;)

To prevent a new ip, assign the roborock a static ip


Upgrade API dependencies

See https://github.com/hferentschik/balena-weather/security/dependabot/1


Treemap set selected node pragmatically

Version
4.2.1-rc.2
Reproduction link
https://jsfiddle.net/r2v4qwj0/
Steps to reproduce
Is there any way to setZoom or setCenter pragmatically for specific node in tree ?
In Addition, " lets suggest that I already have the path or the id of the node "
What is expected?
zoom to specific node
What is actually happening?
I can not do zoom to specific node

Currently, no.
But I found there is a treemapZoomToNode action. I tried to use it but it seems the view is not updated correctly.
myChart.dispatchAction({
  type: 'treemapZoomToNode',
  seriesIndex: 0,
  dataIndex: 1
});

I think a pull request is required to make it work. @MohanadI Are you interested in looking into the source code?


we don't support php 5.4

i activated travis for this repository and noticed it tries to build with php 5.4, then composer notices that this wont work.
i suggest we activate php 7 as well. its about to be released, and currently green.

hm, but hhvm setup seems to be broken

HHVM is an ugly bitch IMHO when it comes to cURL. PHP 7 can be enabled for this repo, but not for others, where PHPSpec is used, as 2.4 is not stable yet.

I have never dealt with HHVM. So I just copied existed setup from boilerplate.

i will see what i can do about HHVM

As I can see, there is a problem with the integration test suite.

there seems to also be an issue with the lowest version getting a too
old version of phpunit token stream

Code coverage fixed.

Tests currently needs PHP builtin server. So they can not run under HHVM. Add skipping them for now.


TypeError: unbound method init() must be called with Downloader instance as first argument (got InternUdemyLectureAssets instance instead)

This has only recently started to happen when using the udemy-dl.py to download a course.  This happens on both an update to an already downloaded course as well as downloading a new one.
Traceback (most recent call last):
  File "./udemy-dl.py", line 1432, in <module>
    main()
  File "./udemy-dl.py", line 1091, in main
    udemy.course_download(path=options.output, quality=options.quality, unsafe=options.unsafe)
  File "./udemy-dl.py", line 467, in course_download
    lecture_assets = lecture.assets
  File "/mnt/g/udemy-downloader/udemy-dl/udemy/_shared.py", line 341, in assets
    self._process_assets()
  File "/mnt/g/udemy-downloader/udemy-dl/udemy/_internal.py", line 119, in _process_assets
    assets  =   [InternUdemyLectureAssets(z, self) for z in self._info['assets']] if self._assets_count > 0 else []
  File "/mnt/g/udemy-downloader/udemy-dl/udemy/_internal.py", line 143, in __init__
    super(InternUdemyLectureAssets, self).__init__(parent)
  File "/mnt/g/udemy-downloader/udemy-dl/udemy/_shared.py", line 521, in __init__
    Downloader.__init__(self)
TypeError: unbound method __init__() must be called with Downloader instance as first argument (got InternUdemyLectureAssets instance instead)

@EnoqueEsteves can you please follow the issue reporting template?

@r0oth3x49 - apologies I have updated the issue following you template. Let us know if you require any further information or clarification.


Marking MIPs 35 and 36 as withdrawn

MIPs 36 and 35 were withdrawn by author, @Davidutro, @CPSTL.

Got it, thanks @blimpa. I see they have also been updated on the forums, so merging in the update 👍


nbclassic has no attribute base

Linking nbclassic 0.4.2 fails with:
module 'nbclassic' has no attribute 'base'

I believe is here.
I think there are some incomplete migrations bringing in legacy notebook things. The bundler extension also has a number of undefined names and imports that I don't think will work.

@minrk Can you paste here the commands to reproduce the issue?

Sure! Installing and enabling nbclassic in a clean env is enough:
python3 -m venv /tmp/test-nbclassic
source /tmp/test-nbclassic/bin/activate
pip install nbclassic
jupyter server extension enable --sys-prefix nbclassic # workaround #127
jupyter server

produces:
[W 2022-07-09 16:36:06.934 ServerApp] module 'nbclassic' has no attribute 'base'

Importing nbclassic.bundler.handlers also fails:
$ python3 -c 'import nbclassic.bundler.handlers'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/tmp/test-nbclassic/lib/python3.9/site-packages/nbclassic/bundler/handlers.py", line 10, in <module>
    from nbclassic.base.handlers import IPythonHandler
ModuleNotFoundError: No module named 'nbclassic.base'

pip freeze
anyio==3.6.1
appnope==0.1.3
argon2-cffi==21.3.0
argon2-cffi-bindings==21.2.0
asttokens==2.0.5
attrs==21.4.0
backcall==0.2.0
beautifulsoup4==4.11.1
bleach==5.0.1
cffi==1.15.1
debugpy==1.6.2
decorator==5.1.1
defusedxml==0.7.1
entrypoints==0.4
executing==0.8.3
fastjsonschema==2.15.3
idna==3.3
ipykernel==6.15.1
ipython==8.4.0
ipython-genutils==0.2.0
jedi==0.18.1
Jinja2==3.1.2
jsonschema==4.6.2
jupyter-client==7.3.4
jupyter-core==4.11.1
jupyter-server==1.18.1
jupyterlab-pygments==0.2.2
MarkupSafe==2.1.1
matplotlib-inline==0.1.3
mistune==0.8.4
nbclassic==0.4.2
nbclient==0.6.6
nbconvert==6.5.0
nbformat==5.4.0
nest-asyncio==1.5.5
notebook-shim==0.1.0
packaging==21.3
pandocfilters==1.5.0
parso==0.8.3
pexpect==4.8.0
pickleshare==0.7.5
prometheus-client==0.14.1
prompt-toolkit==3.0.30
psutil==5.9.1
ptyprocess==0.7.0
pure-eval==0.2.2
pycparser==2.21
Pygments==2.12.0
pyparsing==3.0.9
pyrsistent==0.18.1
python-dateutil==2.8.2
pyzmq==23.2.0
Send2Trash==1.8.0
six==1.16.0
sniffio==1.2.0
soupsieve==2.3.2.post1
stack-data==0.3.0
terminado==0.15.0
tinycss2==1.1.1
tornado==6.2
traitlets==5.3.0
wcwidth==0.2.5
webencodings==0.5.1
websocket-client==1.3.3

Strangely, there's an additional error in the jupyterhub tests (what brought me here) that prevents load as well:
2022-07-09T02:50:42.0277315Z [I 2022-07-09 02:50:40.173 MockSingleUserServer manager:340] jupyterlab | extension was successfully linked.
2022-07-09T02:50:42.0278001Z [W 2022-07-09 02:50:40.173 MockSingleUserServer manager:344] module 'nbclassic' has no attribute 'base'
2022-07-09T02:50:42.0278516Z [D 50:40.434 MockHub config_manager:93] Paths used for configuration of jupyter_notebook_config:
2022-07-09T02:50:42.0278878Z     	/tmp/nandy/.jupyter/jupyter_notebook_config.json
2022-07-09T02:50:42.0279336Z [D 50:40.435 MockHub config_manager:93] Paths used for configuration of jupyter_notebook_config:
2022-07-09T02:50:42.0279697Z     	/etc/jupyter/jupyter_notebook_config.json
2022-07-09T02:50:42.0280134Z [D 50:40.435 MockHub config_manager:93] Paths used for configuration of jupyter_notebook_config:
2022-07-09T02:50:42.0280491Z     	/usr/local/etc/jupyter/jupyter_notebook_config.json
2022-07-09T02:50:42.0280948Z [D 50:40.435 MockHub config_manager:93] Paths used for configuration of jupyter_notebook_config:
2022-07-09T02:50:42.0281381Z     	/opt/hostedtoolcache/Python/3.8.13/x64/etc/jupyter/jupyter_notebook_config.d/jupyterlab.json
2022-07-09T02:50:42.0281804Z     	/opt/hostedtoolcache/Python/3.8.13/x64/etc/jupyter/jupyter_notebook_config.json
2022-07-09T02:50:42.0282285Z [D 50:40.436 MockHub config_manager:93] Paths used for configuration of jupyter_notebook_config:
2022-07-09T02:50:42.0282678Z     	/home/runner/.local/etc/jupyter/jupyter_notebook_config.json
2022-07-09T02:50:42.0283130Z [D 50:40.436 MockHub config_manager:93] Paths used for configuration of jupyter_notebook_config:
2022-07-09T02:50:42.0283497Z     	/tmp/nandy/.jupyter/jupyter_notebook_config.json
2022-07-09T02:50:42.0284088Z [I 2022-07-09 02:50:40.436 MockSingleUserServer manager:340] notebook_shim | extension was successfully linked.
2022-07-09T02:50:42.0284722Z [I 2022-07-09 02:50:40.470 MockSingleUserServer manager:362] notebook_shim | extension was successfully loaded.
2022-07-09T02:50:42.0285476Z [I 2022-07-09 02:50:40.472 LabApp] JupyterLab extension loaded from /opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/jupyterlab
2022-07-09T02:50:42.0286206Z [I 2022-07-09 02:50:40.472 LabApp] JupyterLab application directory is /opt/hostedtoolcache/Python/3.8.13/x64/share/jupyter/lab
2022-07-09T02:50:42.0286870Z [I 2022-07-09 02:50:40.480 MockSingleUserServer manager:362] jupyterlab | extension was successfully loaded.
2022-07-09T02:50:42.0287461Z [D 2022-07-09 02:50:40.483 MockSingleUserServer manager:355] Traceback (most recent call last):
2022-07-09T02:50:42.0288149Z       File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/jupyter_server/extension/manager.py", line 351, in load_extension
2022-07-09T02:50:42.0288598Z         extension.load_all_points(self.serverapp)
2022-07-09T02:50:42.0289225Z       File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/jupyter_server/extension/manager.py", line 231, in load_all_points
2022-07-09T02:50:42.0289728Z         return [self.load_point(point_name, serverapp) for point_name in self.extension_points]
2022-07-09T02:50:42.0290402Z       File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/jupyter_server/extension/manager.py", line 231, in <listcomp>
2022-07-09T02:50:42.0290895Z         return [self.load_point(point_name, serverapp) for point_name in self.extension_points]
2022-07-09T02:50:42.0291612Z       File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/jupyter_server/extension/manager.py", line 224, in load_point
2022-07-09T02:50:42.0291995Z         return point.load(serverapp)
2022-07-09T02:50:42.0292581Z       File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/jupyter_server/extension/manager.py", line 150, in load
2022-07-09T02:50:42.0292946Z         return loader(serverapp)
2022-07-09T02:50:42.0293597Z       File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/jupyter_server/extension/application.py", line 453, in _load_jupyter_server_extension
2022-07-09T02:50:42.0293997Z         extension.initialize()
2022-07-09T02:50:42.0294656Z       File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/jupyter_server/extension/application.py", line 420, in initialize
2022-07-09T02:50:42.0295031Z         self._prepare_settings()
2022-07-09T02:50:42.0295645Z       File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/jupyter_server/extension/application.py", line 290, in _prepare_settings
2022-07-09T02:50:42.0296056Z         webapp = self.serverapp.web_app
2022-07-09T02:50:42.0296462Z     AttributeError: 'ServerApp' object has no attribute 'web_app'
2022-07-09T02:50:42.0296713Z
2022-07-09T02:50:42.0303345Z [W 2022-07-09 02:50:40.483 MockSingleUserServer manager:356] nbclassic | extension failed loading with message: 'ServerApp' object has no attribute 'web_app'

I'm not 100% sure what's different there, but #130 appears to fix it.

Thx @minrk I have run those commands on my env and it works fine (was also working fine in my existing conda env).
python3 -m venv /tmp/test-nbclassic
source /tmp/test-nbclassic/bin/activate
pip install nbclassic
jupyter server extension enable --sys-prefix nbclassic # workaround #127
jupyter server
open http://localhost:8888/tree

It is true that there is a warning module 'nbclassic' has no attribute 'base'. Starting with --debug does not give more details. I think jupyter-server is silently catching some errors (cc/ @Zsailer)
[W 2022-07-10 07:45:24.924 ServerApp] module 'nbclassic' has no attribute 'base'
[I 2022-07-10 07:45:25.169 ServerApp] notebook_shim | extension was successfully linked.
[I 2022-07-10 07:45:25.225 ServerApp] notebook_shim | extension was successfully loaded.
[I 2022-07-10 07:45:25.229 ServerApp] nbclassic | extension was successfully loaded.
[I 2022-07-10 07:45:25.236 ServerApp] Serving notebooks from local directory: /Users/echarles/datalayer-platform/ext/jupyter-notebook-classic
[I 2022-07-10 07:45:25.236 ServerApp] Jupyter Server 1.18.1 is running at:
[I 2022-07-10 07:45:25.236 ServerApp] http://localhost:8888/?token=14ba66dd9fea8e60dd6a5f3ba06224d61f68555044bcbed1
[I 2022-07-10 07:45:25.236 ServerApp]  or http://<IP_ADDRESS>:8888/?token=14ba66dd9fea8e60dd6a5f3ba06224d61f68555044bcbed1
[I 2022-07-10 07:45:25.236 ServerApp] Use Control-C to stop this server and shut down all kernels (twice to skip confirmation).
[C 2022-07-10 07:45:25.244 ServerApp]

    To access the server, open this file in a browser:
        file:///Users/echarles/Library/Jupyter/runtime/jpserver-25920-open.html
    Or copy and paste one of these URLs:
        http://localhost:8888/?token=14ba66dd9fea8e60dd6a5f3ba06224d61f68555044bcbed1
     or http://<IP_ADDRESS>:8888/?token=14ba66dd9fea8e60dd6a5f3ba06224d61f68555044bcbed1
[W 2022-07-10 07:45:40.370 ServerApp] 404 GET /nbclassic (::1) 5.60ms referer=None
^C[I 2022-07-10 07:45:48.629 ServerApp] interrupted
Serving notebooks from local directory: /Users/echarles/datalayer-platform/ext/jupyter-notebook-classic
0 active kernels
Jupyter Server 1.18.1 is running at:
http://localhost:8888/?token=14ba66dd9fea8e60dd6a5f3ba06224d61f68555044bcbed1
 or http://<IP_ADDRESS>:8888/?token=14ba66dd9fea8e60dd6a5f3ba06224d61f68555044bcbed1
Shutdown this Jupyter server (y/[n])? ^C[C 2022-07-10 07:45:48.845 ServerApp] received signal 2, stopping
[I 2022-07-10 07:45:48.847 ServerApp] Shutting down 2 extensions
[I 2022-07-10 07:45:48.848 ServerApp] Shutting down 0 terminals

FYI I had the same issue as well I presume is the same issue as well https://stackoverflow.com/questions/72883216/jupyterhub-fails-to-spawn-new-session-with-no-module-named-ipythonhandler when running the JupyterHub Docker image. My JupyterHub service stopped working after I rebuilt the container.

@minrk @jongillham nbclassic 0.4.3 is released https://github.com/jupyter/nbclassic/releases/tag/v0.4.3
I hope it solves the issues you have encountered.


🆕 Software Suggestion | HardenedBSD

Basic Information
Name: HardenedBSD
Category: Advanced Operating Systems
URL: hardenedbsd.org
GitHub github.com/HardenedBSD/hardenedBSD
.onion link lkiw4tmbudbr43hbyhm636sarn73vuow77czzohdbqdpjuq3vdzvenyd.onion
Description
A security-enhanced fork of FreeBSD. The HardenedBSD Project is implementing many exploit mitigation and security technologies on top of FreeBSD. The project started with Address Space Layout Randomization (ASLR) as an initial focal point and is now implementing further exploit mitigation techniques.
Why I am making the suggestion

HardenedBSD has many mitigations OpenBSD does not such as CFI, SafeStack, SEGVGUARD, a proper W^X implementation and more but it lacks things OpenBSD has such as LibreSSL, a hardened malloc etc...

(Taken from here)
My connection with the software
None.

[x] I will keep the issue up-to-date if something I have said changes or I remember a connection with the software.

Looks to be an interesting project, however I am not sure how I feel about adding it.
There were other things that were not added (Gentoo etc), because if a reader thinks they require that for their use case then they probably don't need us to tell them.
I imagine projects like HardenedBSD would be best suited to specialist applications (mostly in the server segment) where those extra mitigations are useful.


Image is being thrown as a white blank image

I am experiencing a very strange error that is happening everywhere in my app where I return an image. When a image is being returned in browser, it looks like this white little square:

The code used to return this:
return response(File::get(public_path().'/images/my-image.png'), 200)->header('Content-Type', 'image/png');

I have ensured the image path is valid, that the image is valid (I can access it without issues if I open it directly from its public path), and checked any error logs (nginx + laravel.log).
If I die and dump the values of File::get(public_path().'/images/my-image.png') I get PNG encoded data.
If I cURL to the path, it looks like this:

It happens on both staging, production and local machine (3x local machines testet - 1 windows, 2x mac osx). It occurred suddenly - it used to work fine previously.
I am stuck with this bug. Any suggestions?

Please ask on the forums. I don't think this is a bug.

Are you sure? Any ideas how I can debug further? I am not the only one experiencing this.

I have tried removing all vendor service providers to see if anything could conflict, but that does not work either.

If you have any ideas to point me in a direction of debugging further I'd be grateful!


rpm spec: Skip rust build for debug binary

The make command also build debug binary which is not required for
rpm.

Force merging as this is test only code change.


ktk.Player should warn if markers of same name were found in different input TimeSeries

At the moment, Player quietly merges every TimeSeries provided as an input. But if two TimeSeries have similar names, one will take precedence and it's (1) not documented which; (2) confusing if we really wanted to explore two movements and see only one.
I don't think that marker names should be modified automatically, because it would break the interconnections argument. However we should get a warning.

Will be integrated in #215

Now related to #251


[Accessibility][Screen Reader -New Trigger]: On activating 'New Trigger' button, narrator narrates all the info available in the dialog box in a single flow.

User Experience
The screen reader dependent user will get confused if all the information is narrated in a single flow.
Test Environment
Browser: Microsoft Edge Canary Version: 82.0.425.0 (Official build)
OS build: 2004 (19564.1005)
Screen Reader: Narrator
URL: Bot Framework Composer
Prerequisite Here
Repro Steps:

Open the above mentioned URL in EDGE.
Select 'Design Flow' from left navigation.
Now, navigate to 'New Trigger' button available in the filter dialog pane and activate it..
Observe the narrator narration.

Actual:
The narrator narrates all the information available on the 'New Trigger' dialog in a single flow.
Expected:
The narrator should only narrate the tittle of the pane along with close button.
Observation:
Same issue observed with NVDA.
MAS Reference: MAS4.2.1
Attachment:
MAS4.2.1_Narrator narrates all the info in a single flow.zip

Verified this is still happening. Working on it now.

This looks like a problem with the interaction between Dialog and Dropdown. I can reproduce it in a CodePen just using off-the-shelf Fabric UI pieces: https://codepen.io/beyackle/pen/ExjOdBq

@ashish315 could we have this bug filed with Fabric team?
@beyackle I'm going to move this to 'In Review' list for visual clarity, feel free to change if you disagree

Thanks for forwarding the question along. I put a question in the Fabric React channel in their team and haven’t gotten a response yet, but you can see the CodePen in there.
From: Corina<EMAIL_ADDRESS>Sent: Thursday, March 26, 2020 1:42 PM
To: microsoft/BotFramework-Composer<EMAIL_ADDRESS>Cc: Benjamin Yackley<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/BotFramework-Composer] [Accessibility][Screen Reader -New Trigger]: On activating 'New Trigger' button, narrator narrates all the info available in the dialog box in a single flow. (#2059)
@ashish315https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fashish315&data=02|01|Benjamin.Yackley%40microsoft.com|29dc6dede8a245bd1a5308d7d1c630cf|72f988bf86f141af91ab2d7cd011db47|1|0|637208521460878064&sdata=8qfUY9NoaULDKVg4D6Un3G7x66bYOSbbMi1XZIFfUyE%3D&reserved=0 could we have this bug filed with Fabric team?
@beyacklehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbeyackle&data=02|01|Benjamin.Yackley%40microsoft.com|29dc6dede8a245bd1a5308d7d1c630cf|72f988bf86f141af91ab2d7cd011db47|1|0|637208521460878064&sdata=96qn2e%2BGfHHK9y1JGzYhtXzTAUAEnmnPHUQYahTaKdI%3D&reserved=0 I'm going to move this to 'In Review' list for visual clarity, feel free to change if you disagree
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FBotFramework-Composer%2Fissues%2F2059%23issuecomment-604674411&data=02|01|Benjamin.Yackley%40microsoft.com|29dc6dede8a245bd1a5308d7d1c630cf|72f988bf86f141af91ab2d7cd011db47|1|0|637208521460888058&sdata=DpiCa6N9E%2Blr9E6DLxO%2FLmZWjqIvQXA%2F37Of8mvEu08%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOY6QCMY7WAVLVLCPBZOMJTRJO43BANCNFSM4KZ2LSUQ&data=02|01|Benjamin.Yackley%40microsoft.com|29dc6dede8a245bd1a5308d7d1c630cf|72f988bf86f141af91ab2d7cd011db47|1|0|637208521460888058&sdata=aMQ4sITpY15URMLwjzGf0Y0DjuZuGhOhhQmcvYyak04%3D&reserved=0.

@corinagum Will let you know once confirmed with Fabric Team.

@corinagum will share the bug ID once the fabric team provide us. Meanwhile we are closing the issue.

As checked with fabric team the issue is not repro at their end as well as 'New Trigger' dialog UI gets updated and currently narrator narration is acceptable.


Repo code doesn't match code  npm package

https://github.com/ekalinin/sitemap.js/blob/8ff1e17884a36c6c9a8280bae0847a81d37b7f91/lib/sitemap.js#L299
2.1.0 version both

There is no in npm package :
  if (url instanceof Object) {
      lastmod = url.lastmod ? url.lastmod : lastmod;

      url = url.url;
    }

I have no idea how it could be.
Just uploaded 3.0.0 version (in typescript).


Do we need to set mixed-precision explicitly or is it handled if tensor cores available?

I following your awesome guide on timm: https://towardsdatascience.com/getting-started-with-pytorch-image-models-timm-a-practitioners-guide-4e77b4bf9055.
I am running training on an A100-based VM which should support mixed-precision training. Does Trainer from PyTorch Accelerated take care of that automatically?

Hi, thank you for the kind words! The Trainer will handle mixed precision for you, so there will be no changes needed to your code, as long as it is set in your accelerate config when launching training.
As described here, you can create a config file using the command accelerate config --config_file accelerate_config.yaml. You can verify that mixed precision is set by inspecting the config file which, depending on your answers, should look something like this:

If you then launch with that config, using:
accelerate launch --config_file accelerate_config.yaml train.py [--training-args]
mixed precision will be enabled!

Thank you!
I am using a similar main.py script as yours (shown in your tutorial) so I assume there won't be a need to launch with config file or am I mistaken?

Hi, if you launch without a config file, it will assume default settings, which will use 1GPU and mixed precision will be disabled.
The idea behind the config file and the accelerate cli is that you have a consistent launch command regardless of your infrastructure; you don't need to bother with python -m torch.distributed .. and all of that stuff. If you want to use multiple GPUs, or toggle fp16, then all you need to do is update that file, and not the code. Of course, launching without the accelerate CLI will still work, but you will need to make some code changes to enable mixed precision.
Personally, I would recommend always launching with accelerate launch and the config file, as it offers the most flexibility with the least amount of effort.

Thanks for clarifying, that helps.

Of course, launching without the accelerate CLI will still work, but you will need to make some code changes to enable mixed precision.

Could you also hint at how those changes might look like? It's good to also be aware of that I think.

Thanks so much for being thorough with your explanations. And yes it makes sense to use config files to launch training whenever possible.


Compatibility issues with Python 3.8

requirements.txt say "streamerate>=1.0.0", but latest version is not compatible with python 3.8 due to it using Python 3.10 syntax.

Fixed in this commit: https://github.com/asuiu/pyxtension/commit/594dfc5660e2e24abf601bd556d22bf8bdc5ef77
Built & deployed new version on PyPi: https://pypi.org/project/pyxtension/1.16.11/
Also feel free to raise PRs.
@kiranzo thanks for contributing and creating issues!
Feel free to check and come back if it doesn't fix the issue

@asuiu Fixed, thanks! Problem is, I came across this error trying to use bintablefile library, and I still can't get it to work each time for different reason! Guess I'll just use gzipped parquet for my table data.

@kiranzo
Just curious what errors you've got with the bintablefile. Feel free to add an issue on the project's github page, will be glad to try fix the usecase.
As a side note, instead of Parquet, I found ORC format slightly more friendly for the Python ecosystem.
Also don't apply the GZip over the Parquet/ORC, as these formats are performing internal compression AFAIK, and performance can degrade if compression is applied over the whole file (i.e. headers and metadata)

@asuiu Fixed, thanks! Problem is, I came across this error trying to use bintablefile library, and I still can't get it to work each time for different reason! Guess I'll just use gzipped parquet for my table data.

@kiranzo Just curious what errors you've got with the bintablefile. Feel free to add an issue on the project's github page, will be glad to try fix the usecase.
As a side note, instead of Parquet, I found ORC format slightly more friendly for the Python ecosystem. Also avoid applying the GZip over the Parquet/ORC, as these formats are performing internal compression AFAIK, and performance can degrade if compression is applied over the whole file (i.e. headers and metadata)

https://github.com/eSAMTrade/bintablefile/issues/6
I created it today. Thanks for the ORC suggestion, I'll look into it. Also, I created my parquet.gz with Pandas compression options, so it should be ok.
I test everything on a table that has uuid, filepath and 3 categorical columns (they were enums in PostgreSQL), so bintablefile doesn't suit my needs, unless I convert PostgreSQL enums to int-backed Python enums (we discussed it today, but it's something to consider in the future for now).


Custom css or class name?

Hi there,
I was wondering if there is a possible way to add custom styles instead of ct-series-c, ct-series-b etc?
I had a look at this thread https://github.com/gionkunz/chartist-js/issues/79 particularly at DSpeckhals comment. Something like this would be fantastic.

Terrible sorry found all the info on the site.


[CI] Run tests with PySide and PyQt5 on Windows

Make appveyor runs 3 tests:

python 3.5, PyQt5 (wheel from pypi)
python 2.7, PySide (wheel from pypi)
python 2.7, PyQt4 (wheel from www.edna-site.org)

Relates to #48

Great!


this.context.pageContext.list.serverRelativeUrl doesn't refresh

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Windows
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[X] 💥 Microsoft Edge
[X] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version

Chrome: 99.0.4844.51 (Official Build) (64-bit)
Edge 99.0.4844.51 (Official Build) (64-bit)

SPFx version: 1.14.0
Node.js version: 14.15.0

Describe the bug / error
I have a List View Command Set with a Command that should only be visible for one specific list (e.g. Travel Requests).
In order to hide the Command in other lists, I need to check the content.
The listViewStateChangedEvent is not executed on first render, so I cannot use it to hide button. I'm therefore using onInit()
TestCommandSet.ts
//...
public onInit(): Promise<void> {
    console.log("------------- onInit -------------");
    console.log("this.context.pageContext.list.serverRelativeUrl: " + this.context.pageContext.list.serverRelativeUrl);
    console.log("this.context.listView.list: " + this.context.listView.list);
    console.log("this.context.listView.list.serverRelativeUrl: " + this.context.listView.list.serverRelativeUrl);

    Log.info(LOG_SOURCE, 'Initialized TestCommandSet');
    this.context.listView.listViewStateChangedEvent.add(this, this.onListViewUpdatedv2);
    return Promise.resolve();
  }
//...

I have two problems here:

this.context.listView.list returns undefined when debugging
After deploying the solution to the SPO site, the this.context.pageContext.list.serverRelativeUrl has correct list url when I select a list. However, after selecting another list from the QuickLaunch, the  this.context.pageContext.list.serverRelativeUrl  is not updated

Steps to reproduce

Create two lists: "Travel requests" and "Recruitment tracker" in your SPO site. Make sure both lists are displayed in the Quick Launch
Deploy the List View Command Set solution to the (site) Apps Catalog
Open the site (home page), open DevTools (F12)
Click on the  "Travel requests" in the left navigation, see the following output in the console:

Click on the  "Recruitment tracker" in the left navigation, see  the following output in the console:

I could use this.context.listView.list.serverRelativeUrl to detect the list context, but  this.context.listView.list returns undefined when debugging.
Expected behavior
Would be great if both properties this.context.pageContext.list.serverRelativeUrl and this.context.listView.list.serverRelativeUrl returned correct values when debugging and when deployed.

should be rolled out ww by now


IndexError: list index out of range

Hello,
Running BeagleBone Black.  Trying to run the provided example python script.
root@AWD1:/home/debian/beaglebone_pru_adc# uname -r
4.9.82-ti-r102
root@AWD1:/home/debian/beaglebone_pru_adc# python examples/basic.py
Traceback (most recent call last):
File "examples/basic.py", line 1, in
import beaglebone_pru_adc as adc
File "/usr/local/lib/python2.7/dist-packages/beaglebone_pru_adc-0.0.3-py2.7-linux-armv7l.egg/beaglebone_pru_adc/init.py", line 11, in
SLOTS = glob.glob('/sys/devices/bone_capemgr.*/slots')[0]
IndexError: list index out of range
Is this compatible with the kernel version that I am running?

Nope, looks like you have a different cape driver. Code is looking for files:
/sys/devices/bone_capemgr.*/slots
and in your case it can not find any.

Yes, I agree.  My understanding is the current BeagleBone Black version did
away with the cape manager, and only uses uboot for device overlays.  Wasnt
sure if someone had updated this code or knew of a fix to "bypass" the need
for the cape manager...
On Fri, Aug 31, 2018 at 11:11 AM, Mike Kroutikov<EMAIL_ADDRESS>wrote:

Nope, looks like you have a different cape driver. Code is looking for
files:
/sys/devices/bone_capemgr.*/slots
and in your case it can not find any.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pgmmpk/beaglebone_pru_adc/issues/16#issuecomment-417713304,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD2rJ2o2vqG4W1_YVYQzxXNCKWa3FnF1ks5uWWAugaJpZM4WVRxC
.

--
Thomas Remmert
Cell:<PHONE_NUMBER>
Office:<PHONE_NUMBER>

Is there a workaround for this for updated versions of Debian that don't expose the slots?

A separate project implements streaming ADC capture on Debian, see:
https://github.com/pgmmpk/bbb_pru_adc


Need to support repositories endpoints

Need a support for https://developer.travis-ci.com/resource/repositories.

I will work on these issues within a few weeks, but I would really appreciate it if some one would take these.


Create an AI Art Generator Platform

Description: We're combining art and technology. Your task is to design a platform where users can generate AI-powered artworks with a modern and artistic design.

Difficulty: intermediate
Estimated Time: 5-6 hours

Hi Jayesh,
Hi there! I'm interested in working on this issue. Could you please assign it to me? I believe I can make a valuable contribution to this task.
Looking forward to your response. Thanks!

Hello User,
Thank you for your interest in working on this issue. I've assigned the issue to you, and you can now start working on it. If you have any questions or need any assistance, feel free to ask. Good luck with your contribution!
Best regards,
Jayesh


[entropy_src/dv] Synchonize ExtHT and other HT scoreboarding

Prior to this commit, the ExtHT vs. the Repcnt and windowed tests all ran in  separate threads.  This commit makes the health test scoring driven by ExtHT events (sample received, window closed), thus all health tests can be scored in the same task.
ExtHT Req events are now checked to see that they match the RNG data and that the window-close events are occuring at the correct time.
The original intent was to find a predict/access collision in the alert count registers.   This commit does not fix that bug unfortunately. It does however simplify the code, and remove one more thread from the scoreboard.   It also supports better scoring of ExtHT events.
Signed-off-by: Martin Lueker-Boden<EMAIL_ADDRESS>

Thanks for the reviews! It looks like there is a bug in this one, I'll merge it once I figure that out.


closes 84: The django.forms.util module has been renamed. Use django.forms.utils instead.

Thank you for contribution.
But we should consider <1.7 users. (like https://github.com/django-ckeditor/django-ckeditor/pull/176/files)

@lqez 1.6 is no longer supported by the Django team. See: https://www.djangoproject.com/download/
I do understand if you want to keep compatibility for those still on it. Would you like me to update the PR with those changes? Not a problem.

Got it. And I added support for deprecated django versions. https://github.com/summernote/django-summernote/commit/5fb6b51fb7ec8124bf9a3d45043b4ab02fc0a4af


Support Telescope path_display option

Telescope is able to shorten the displayed file paths using the path_display option. It would be nice if the harpoon extension would respect this setting (or offer something similar).

I have made some adjustments in a fork that you ight want to try https://github.com/RomanoZumbe/harpoon


Update and simplify recipe

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you but ran into some issues, please ping conda-forge/core for further assistance.

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi @bgruening, I slightly simplified the recipe (this now loads the source as dist tar.gz and includes a minor update of the source that I missed in my first recipe) - can it be merged?

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

@conda-forge-admin, please update team

Hi @bebatut, could you help me: how do I get this merged? Is there something missing? Thanks, S


Mediatype: add avif to mediatype inference

Adds .avif files to mediatype inference, as it was not previously contemplated by the logic.

I went ahead and merged, but let me if you believe something's missing.
Pre-released as 0.4.193-commit-38002cb.

And thanks for sponsoring :green_heart:

Hi @brillout! I opened the PR as a draft out of habit, so no worries from my side. Thanks for the quick review & merge.


Descriptive stats: duckdb.Error: Invalid Error: don't know what type

Happens e.g. for xnli for the all_languages config for the train split
"Traceback (most recent call last):\n",
" File \"/src/services/worker/src/worker/job_manager.py\", line 160, in process\n job_result = self.job_runner.compute()\n",
" File \"/src/services/worker/src/worker/job_runners/split/descriptive_statistics.py\", line 404, in compute\n compute_descriptive_statistics_response(\n",
" File \"/src/services/worker/src/worker/job_runners/split/descriptive_statistics.py\", line 323, in compute_descriptive_statistics_response\n con.sql(\n",
"duckdb.Error: Invalid Error: don't know what type: \u000e\n"

cc @polinaeterna

after refresh it's another error (file size doesn't match):
","  File "/src/services/worker/src/worker/job_manager.py", line 171, in process
    job_result = self.job_runner.compute()
","  File "/src/services/worker/src/worker/job_runners/split/descriptive_statistics.py", line 467, in compute
    compute_descriptive_statistics_response(
","  File "/src/services/worker/src/worker/job_runners/split/descriptive_statistics.py", line 352, in compute_descriptive_statistics_response
    hf_hub_download(
","  File "/src/services/worker/.venv/lib/python3.9/site-packages/huggingface_hub/utils/_validators.py", line 118, in _inner_fn
    return fn(*args, **kwargs)
","  File "/src/services/worker/.venv/lib/python3.9/site-packages/huggingface_hub/file_download.py", line 1364, in hf_hub_download
    http_get(
","  File "/src/services/worker/.venv/lib/python3.9/site-packages/huggingface_hub/file_download.py", line 547, in http_get
    raise EnvironmentError(
","OSError: Consistency check failed: file should be of size 301736502 but has size 293192246 (0002.parquet).
We are sorry for the inconvenience. Please retry download and pass `force_download=True, resume_download=False` as argument.
If the issue persists, please let us know by opening an issue on https://github.com/huggingface/huggingface_hub.

https://datasets-server.huggingface.co/statistics?dataset=xnli&config=all_languages&split=train

{"error":"Consistency check failed: file should be of size 301736502 but has size 293192246 (0002.parquet).\nWe are sorry for the inconvenience. Please retry download and pass force_download=True, resume_download=False as argument.\nIf the issue persists, please let us know by opening an issue on https://github.com/huggingface/huggingface_hub."}

After a refresh for https://datasets-server.huggingface.co/statistics?dataset=xnli&config=all_languages&split=train it is working fine


Setting to center the layout when turning on the zen mode

@isidorn as discussed in #40757 this PR adds a setting to centered the layout when activating the zenmode

I simply used the logic used with zenmode.fullScreen. Should work fine. Was not sure when naming the setting... could also be called zenmode.centerLayout.

@SrTobi works great. Code changes are good as well.
Awesome work, merging 🍻

@SrTobi yeah, centerLayout is better! I will change on master

Any idea when this will be available to use?

@vraa this is available in vscode insiders. In vscode stable it will be availble in around 10 days.


feat: add ellipsis to text overflow in sidebar

Currently, sidebar items don't display ellipses when overflowing. This is particularly problematic in the "sidebar-only" theme, where the sidebar cannot be resized, making it difficult to understand when items were truncated.

Before
After

Thank you very much for this PR! I appreciate your contribution!


*** buffer overflow detected ***: terminated with -D_FORTIFY_SOURCE=3 with GCC 12

I see the following failure while testing latest GCC:
[   12s] [==========] 1 tests from sratom ran (12 ms total)
[   12s] *** buffer overflow detected ***: terminated

It's caused by the following code:
https://github.com/lv2/sratom/blob/c46452c83d442de137fa6470ba544e3ba142e923/src/sratom.c#L337-L343
where the second argument of snprintf should be size * 2 + 1 - 2 * i.
GCC upstream discussion is here:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104969

@siddhesh

Thanks.  clang-tidy (or perhaps clang itself, I don't quite remember) has highlighted this issue as well, but I haven't propagated a fix back to master yet.

Good, can you please propage it now?

Fixed in 2edb78c


💬 [Question] Authentik Design Options

💬 Question

Present Hardi the design options for Authentik with all data so he can provide the correct formats, Images and other Ressources.

This describes what options we have to customize the design in Authentik.
Processes
We currently have the following processes which is each individually designable:

Authentication (login)
Authorization
Enrollment (register)
Invalidation (logout)

Layout
In Authentik we can display processes in 5 different layout modes:

Stacked:

Content left:

Content right:

Sidebar left:

Sidebar right:

Darmkode
There is also a darkmode

Options
We can define the following things out of the box:

Background
Logo (global)
Favicon (global)
Almost all texts

Furthermore we can mount a custom.css file to adjust the style like colors, margins, ...
Library
Authentik also has an userinterface, which we wont use as of current planning.
He we see the global logo and the AppIcon we can define for each App.

Ressources

https://docs.goauthentik.io/docs/interfaces/user/customization#custom-css
https://github.com/goauthentik/authentik/issues/2364
https://www.youtube.com/watch?v=YawgyM509ng
https://www.reddit.com/r/selfhosted/comments/18ymm49/how_do_i_enable_dark_mode_in_authentik/

Login.pdf
Register.pdf
Mein Vorschlag für die FELDER


Marketplace deploy: store access_token as env variable during spawner process

I now very doubt it is necessary to store the token inside aiidalab app implementation. The initial purpose of storing the token in jupyterhub is to interact with other marketplace applications by the GUI of QeApp. Do we still want to do interaction in this way or we just want to call and run simulation on QeApp application by its Marketplace API?

The purpose of storing the token is to be able to communicate with other MarketPlace applications, including for instance a data source or data sink application. That would be used to transmit artifacts like input structures and parameters and output data like band structure results to preceding or succeeding steps in the same workflow.
Without the token, a user can start the application, but we can't communicate any results back.

Ah, true! even not talking with GUI we still need the token to talk to other applications for data transmission. My bad.

@unkcpz Is this ready for another review?

I add a lot of useless commits for test purposes, will clean it up and finish this today. Sorry for the delay.

Hi @csadorf, the commits are rebased. I think it is ready for review.
I add the openapi yaml files for both QeAPP and SSSPapp, but there is a problem registering them both in one jupyterhub, since the two marketplace applications have different id and secret for authenticator, but in our condition they share the same authenticator.
I think there are two ways for this issue, 1. we deploy them in different servers. 2. Modify the _get_hearder method of authenticator, so it can authenticate for both application.
The second option is more what we expected from start, and I don't think have only one aiidalab app in a server is a good idea. I will try to implement the second option but in another PR.

Having multiple different deployments would be rather expensive. How difficult would you estimate the customization of the authenticator to be?

Thanks @csadorf .


Edit EasyCLA comment when /easycla is added regardless of the previous status

Summary
We need EasyCLA comment to be updated/edited when /easycla trigger is added, even if the status doesn't have any changes.
Example:

Background
Currently the EasyCLA comment will only get updated/edited when there is a change in the commit, author or signature. However, it doesn't update when there are no changes, even when adding /easycla comment in the PR.

The proposed solution involves updating the comment each time the /easycla comment is added to a PR. This fix will not be included in the upcoming EasyCLA release, but we plan to implement it in the subsequent releases.

@mlehotskylf @jarias-lfx
This proposal would contradict with issue raised for https://github.com/communitybridge/easycla/issues/167
Should this be overruled ?

@nickmango https://github.com/communitybridge/easycla/issues/167 hides the comment when EasyCLA check passes automatically. This case is when it doesn't, we are asking to update the comment regardless of the change when adding /easycla comment.

@nickmango is working on this.

This will only update both previously EasyCLA failed and passed comments.

Tested and verified that changes get updated irrespective of the passed or failed status.
@thakurveerendras kindly verify

@thakurveerendras kindly verify

It is working as expected for tested use cases and @thakurveerendras will try to test with more corner cases.

@thakurveerendras tested corner cases and it working as expected.

Tested on prod & found that edit easycla comment appears when add /easycla in comment on prod

What determines whether or not a comment appears at all?
see https://github.com/istio/community/pull/1406 for a test - the test passes, but there's no comment (which is fine), and adding an /easycla comment doesn't add one (which is probably fine?)

What determines whether or not a comment appears at all?
When the status checks change, a comment is made against the PR. Case in point :- if the Easycla previously failed and then passes the comment will be added and vice versa. However for the easycla comment being run we just ensure the comment was edited (though in effect there was no change) , to keep track of the easycla command

@nickmango Thanks, but in the case mentioned above, there was never a comment, even though the check presumably had to run and pass for the PR to merge.  Just wanted to check that was expected behaviour.

@nickmango Thanks, but in the case mentioned above, there was never a comment, even though the check presumably had to run and pass for the PR to merge. Just wanted to check that was expected behaviour.

Got the case. Currently this isnt happening as the comment only appears on status changes fail->pass., pass->fail or fail(new PR). It wont appear for pass(new PR). This however can be enhanced as the issue has been previously raised internally.


R200 Stream Documentation

Required Info

Camera Model
R200

Firmware Version
<IP_ADDRESS>

Operating System & Version
ubuntu 16.04

Kernel Version (Linux Only)
4.4.0-22-generic

Build System
Makefile

The only published documentation I can find on valid stream resolutions and viable combinations is located on this web page:
https://software.intel.com/en-us/articles/realsense-r200-camera
Available Resolutions
@60FPS, depth at 320x240,, color can be 640x480
@60FPS, depth at 480x360, color can be 320x240 or 640x480
@30FPS, depth at 320x240,, color can be 640x480, 1280x720 or 1920x1080
@30FPS, depth at 480x360, color can be 320x240, 640x480, 1280x720, or 1920x1080
However, the 1280x720 stream does not exists on the R200 with FW <IP_ADDRESS>.
Is there better, or more current documentation available for the R200? Also for the other RealSense cameras?

For each of the cameras in /dev/video you can get a complete list of
resolutions using a Linux video tool.
v4l2-ctl --list-formats-ext -d /dev/video

The DCM release notes include this chart, is it what you're looking for?
http://registrationcenter-download.intel.com/akdlm/irc_nas/9078/release_notes_RealSense_SDK_2016_R2.pdf

Yes, exactly what I wanted for reference.
How current is this document?
I ask because the R200 with the FW I listed above doesn't support any 1280x720 color streams.

@mdhorn all documentation is old. The firmware and spec documents are never in sync. The cross-section of validated modes is indeed smaller than what the firmware provides as well. Because of this, librealsense hardcodes streaming modes most commonly used (and many uncommon ones) because we were routinely finding modes that didn't work as published by the UVC descriptors on the camera firmware (as noted by @teknotus above).

So the descriptors don't always match what formats the camera will actually
stream? Does the camera report back unsupported mode, or is it a worse
error?

If you ask the camera FW what it is supports via librealsense (get_stream_mode), it only reports valid modes.
But, if you use 'v4l2-ctl --list-formats-ext -d /dev/video2', unsupported modes are reported.

Correct. v4l2-ctl actually reaches out to the camera and parses the UVC desciptors. librealsense consciously does something a little more 'dangerous' by hardcoding the subset of modes that are almost guaranteed to work. We've discussed it internally a few times and librealsense may eventually support a get_stream_mode that actually asks the camera first.

@mdhorn does v4l2-ctl report a 1280x720 mode still? Just checking to confirm it wasn't removed in the firmware side. We should probably support it in librealsense, but we will have to interpolate intrinsics.

1280x720, as well as several other "interesting" modes are reported:
$ v4l2-ctl --list-formats-ext -d /dev/video2
ioctl: VIDIOC_ENUM_FMT
	Index       : 0
	Type        : Video Capture
	Pixel Format: 'YUYV'
	Name        : YUYV 4:2:2
		Size: Discrete 1920x1080
			Interval: Discrete 0.033s (30.000 fps)
			Interval: Discrete 0.067s (15.000 fps)
		Size: Discrete 1280x720
			Interval: Discrete 0.033s (30.000 fps)
			Interval: Discrete 0.067s (15.000 fps)
		Size: Discrete 960x540
			Interval: Discrete 0.033s (30.000 fps)
			Interval: Discrete 0.067s (15.000 fps)
		Size: Discrete 848x480
			Interval: Discrete 0.033s (30.000 fps)
			Interval: Discrete 0.067s (15.000 fps)
		Size: Discrete 640x480
			Interval: Discrete 0.017s (60.000 fps)
			Interval: Discrete 0.033s (30.000 fps)
			Interval: Discrete 0.067s (15.000 fps)
		Size: Discrete 640x360
			Interval: Discrete 0.033s (30.000 fps)
			Interval: Discrete 0.067s (15.000 fps)
		Size: Discrete 424x240
			Interval: Discrete 0.033s (30.000 fps)
			Interval: Discrete 0.067s (15.000 fps)
		Size: Discrete 320x240
			Interval: Discrete 0.017s (60.000 fps)
			Interval: Discrete 0.033s (30.000 fps)
			Interval: Discrete 0.067s (15.000 fps)
		Size: Discrete 320x180
			Interval: Discrete 0.033s (30.000 fps)
			Interval: Discrete 0.067s (15.000 fps)

	Index       : 1
	Type        : Video Capture
	Pixel Format: 'pRAA'
	Name        : 10-bit Bayer RGRG/GBGB Packed
		Size: Discrete 2400x1081
			Interval: Discrete 0.033s (30.000 fps)

@mdhorn okay, so then we need to add the 1280x720 color stream to the internal R200 mode list. It will complicated the generation of stream intrinsics. There's an existing issue (#135) to implement this functionality. Closing this out and will track #135.


Does SmallMoleculeSetProposalEngine assume molecule to be modified is at end of topology?

Looking over SmallMoleculeSetProposalEngine.propose(), it seems to assume that the small molecule is always at the end of the topology.  Is there a way we can avoid this assumption?

I don't see that. SmallMoleculeSetProposalEngine.propose() always finds the small molecule via its residue name (not index) and converts it to canonical isomeric smiles. In the the atom mapping, it does put the small molecule at the end of the topology immediately before creating the atom map, but I don't see what difference this makes. Nothing should interrupt propose and change the indices of the small molecule before propose returns.

Ah, OK!  My bad for misreading this.


Class property type inference failure when destructuring return value of generic function

Bug Report

🔎 Search Terms
Class Property
Destructuring
Type Inference
Generic
🕗 Version & Regression Information

This is a crash: No
The issue occurs on the latest nightly (v4.9.0-dev.20221017), as well as on version 4.7.4. I haven't tested on other versions, but my suspicion is that this has been a hidden bug for a while.

⏯ Playground Link

Playground link with relevant code
💻 Code

class Foo {
    constructor() {
        [this.x, this.y] = init();
    }

    x; // Any
    y; // Any
}

function init<T>() {
    return [0, ""] as [number, string];
}

🙁 Actual behavior
The x and y properties are typed as any.
🙂 Expected behavior
The x and y properties should be typed as number and string respectively.
Further investigation
If the return value of the function is captured in a local variable, and then destructured from there, everything works as expected:

class Foo {
    constructor() {
        const bar = init();
        [this.x, this.y] = bar;
    }

    x; // number
    y; // string
}

function init<T>() {
    return [0, ""] as [number, string];
}

Or, if the generic argument is omitted, everything works as expected:

class Foo {
    constructor() {
        [this.x, this.y] = init();
    }

    x; // number
    y; // string
}

function init() {
    return [0, ""] as [number, string];
}

Making use the generic type argument T  does not appear to resolve the issue:

class Foo {
    constructor() {
        [this.x, this.y] = init(false);
    }

    x; // Any, should be number
    y; // Any, should be boolean
}

function init<T>(value: T) {
    return [0, value] as [number, T];
}

As an additional note: If you're wondering why the compiler thinks T might depend on the type of x and y, it's due to the same contextual typing mechanism that makes this work:
let x: Promise<number> = new Promise(resolve => resolve("foo"));  // error on resolve call

Well, I'm not familiar enough with the granularities of the type checker logic in order to refute what you're saying, but the error is still generated in the case when a default generic type argument is defined for T:
class Foo {
    constructor() {
        [this.x, this.y] = init();
    }

    x; // Still fails
    y; // Still fails
}

function init<T = string>() {
    return [0, ""];
}

If I'm understanding what you're saying correctly, then I would imagine that such a work-around should at least dodge the issue. But the error persists. Also worth noting that if the generic argument is provided at the call site, everything works as expected:
class Foo {
    constructor() {
        [this.x, this.y] = init<string>();
    }

    x; // number
    y; // string
}

function init<T>() {
    return [0, ""];
}

The default is only used when TS can’t successfully infer a type for the generic.  It’s still going to try to infer it through the usual means, and thus hit the aforementioned circularity (which will indeed end with T getting its default type, to be clear).  At that point though the property in question now has an “implicit any” as its type and the compiler can’t walk it back because that would require more than one round of inference.  Hence the design limitation.
That providing the type argument at the call site dodges the error is fully expected - it bypasses generic inference (there’s no reason to prefer an inference over a type provided explicitly).

In short, any case where type inference X depends on type inference Y and vice versa will be detected as circular, even if there’s a different way to infer X that avoids the circularity, because inference is only done in a single pass.  TS doesn’t do full unification (see #30134).

Short version: TS tries to infer both T and prop “at the same time”.  Both fail because the inferences depend on each other.  T, being a type parameter, has a working failure case (fall back to default/constraint), prop does not and gets flagged as implicit-any.  There is no mechanism to try inference again and you get stuck with implicit-anys. 😢
Fix would be to do type inference in multiple rounds (allowing failed inferences to be retried using information from the previous round), but responses to similar issues in the past have suggested it’s a performance issue.

This is a true circularity, as outlined above.

Well, technically, it’s a finite spiral—T has a working fallback mechanism that would resolve the circularity if not for the restriction that types must all be inferable in a single pass.


Adminstrator Window cannot utilize Layout

Environment
Windows build number: 1809
PowerToys version: v0.19.1
PowerToy module for which you are reporting the bug (if applicable):
Layout

Steps to reproduce
Run any application as adminstrator
Ensure Layout is enabled on monitor
Click and drag adminstrator window and hold shift (as you would for a normal use of layout)
Notice you cannot apply the layout to the window
Expected behavior
Layout should work on any running application, should not be limited to the user's profile.
Actual behavior
Layout appears to be based on the user profile of the application.

Are you running PowerToys as Administrator? This is probably the issue. If so, this needs to be closed.

Are you running PowerToys as Administrator? This is probably the issue. If so, this needs to be closed.

No im running power toys as my user, elevating an applications privileges using my admin account.  That application does not snap with PowerToys

Open PowerToys Settings and choose "Restart as administrator". FancyZones cannot utilize apps which have higher privileges than itself.

Open PowerToys Settings and choose "Restart as administrator". FancyZones cannot utilize apps which have higher privileges than itself.

Will try that on my personal computer, this kind of makes sense given the nature of privleges.
I'll close for now.


Sometimes I can't send babbles response

Quite frequently when I try to send reply to a babble message, it clears the text field seemingly sending the message but the message does not appear. It does not throw any error anywhere and is not always fixed by refresh. Not sure under what conditions this happens, I can reply to some babbles and not to others. Sometimes sending first reply does not work but sending second one does.

If you see this again can you please make a screenshot of the console errors?

I am not necessary looking for errors, but there is also some debug logging (about connections and subscriptions) that might give a clue


Feat/add command history support

Added history-navigation support to the TorXakis command line interface.

The issue reported by code climate was already there.

I am very curious to work with this new feature!

I added documentation as well here. Let me know if there are any issues, and give Ctrl-R a try :wink:


Android Camera Lag

android camera lag

I have a strange bug on android. I tested my game on an emulator with Android Studio and everything works correctly! however when I install the game on my phone (release or debug) here is the weird distortion effect that occurs. The distortion effect seems to only occur on the X axis, when the camera moves vertically (Y) there is no problem...
https://github.com/julesgrc0/wispy/assets/60539590/fb14da89-aaef-4057-b2be-cc57e1ed18c2

what I tried

the camera example with raylib (everything works)
with a low float in x (close to 0)
without adding the deltatime in x
threadless
without rendering texture
without camera (only matrix float[16])

everything works fine on PC (windows/linux) and on the emulator, but as soon as I test it on my phone the X axis is completely buggy

fixed by removing the camera for Android and replacing it using a simple vector


Why I use cookiecutter to clone git responsitory always said "No such file and directory: u'/my/home/.cookiecutters/myproject/cookiecutter.json'?

HI ALL,
I'm newer to cookiecutter, when I use cookiecutter first time, it always report error as follow:
Please help. Thanks
You've cloned /xxxxx before. Is it okay to delete and re-clone it? [Y/n] Y
Cloning into 'python_lesson'...
remote: Counting objects: 3, done.
remote: Compressing objects: 100% (2/2), done.
remote: Total 3 (delta 0), reused 0 (delta 0), pack-reused 0
Unpacking objects: 100% (3/3), done.
Checking connectivity... done.
Traceback (most recent call last):
  File "/usr/local/bin/cookiecutter", line 11, in <module>
    sys.exit(main())
  File "/Library/Python/2.7/site-packages/click/core.py", line 610, in __call__
    return self.main(*args, **kwargs)
  File "/Library/Python/2.7/site-packages/click/core.py", line 590, in main
    rv = self.invoke(ctx)
  File "/Library/Python/2.7/site-packages/click/core.py", line 782, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Library/Python/2.7/site-packages/click/core.py", line 416, in invoke
    return callback(*args, **kwargs)
  File "/Library/Python/2.7/site-packages/cookiecutter/cli.py", line 70, in main
    cookiecutter(template, checkout, no_input)
  File "/Library/Python/2.7/site-packages/cookiecutter/main.py", line 92, in cookiecutter
    extra_context=extra_context,
  File "/Library/Python/2.7/site-packages/cookiecutter/generate.py", line 43, in generate_context
    file_handle = open(context_file)
IOError: [Errno 2] No such file or directory: u'/my/home/.cookiecutters/myproject/cookiecutter.json'

@zhaozhiming has full certainty that the repository you tried to clone with cookiecutter has a file named cookiecutter.json in the root directory of repository ?

Hi @zhaozhiming!
Cookiecutter requires a template to have two things:

a file named cookiecutter.json that defines variables and the corresponding default values
a folder that is templated, f.i. {{cookiecutter.repo_name}}

There are a few tutorials which you may want to read at http://cookiecutter.readthedocs.org/en/latest/tutorials.html or you could have a look at other templates that are featured on the README file of this project.
Hope this helps! :bow:


Debian Wheezy repo error for libsodium13

Hi on Debian Wheezy server i got this error when try to update (or install on new debian 7 server)
Prendre : 1 http://repo.saltstack.com/apt/debian/7/amd64/latest/ wheezy/main libsodium13 amd64 1.0.3-1 [161 kB]
Prendre : 2 http://repo.saltstack.com/apt/debian/7/amd64/latest/ wheezy/main libzmq3 amd64 4.0.5+dfsg-3 [467 kB]
Prendre : 3 http://repo.saltstack.com/apt/debian/7/amd64/latest/ wheezy/main bsdmainutils amd64 9.0.6 [214 kB]
Prendre : 4 http://repo.saltstack.com/apt/debian/7/amd64/latest/ wheezy/main python-jinja2 all 2.7.3-1 [171 kB]
Prendre : 5 http://repo.saltstack.com/apt/debian/7/amd64/latest/ wheezy/main python-msgpack amd64 0.4.2-1 [126 kB]
Prendre : 6 http://repo.saltstack.com/apt/debian/7/amd64/latest/ wheezy/main python-yaml amd64 3.11-2 [176 kB]
514 ko téléchargés en 0s (11,6 Mo/s)
localepurge: Disk space freed in /usr/share/locale: 0 KiB
localepurge: Disk space freed in /usr/share/man: 0 KiB
Total disk space freed by localepurge: 0 KiB
E: impossible de récupérer http://repo.saltstack.com/apt/debian/7/amd64/latest/pool/main/libs/libsodium/libsodium13_1.0.3-1_amd64.deb : Taille incohérente
i see that you solved this problem long time ago for Debian Jessie. (#29812)
source_list config :
deb http://repo.saltstack.com/apt/debian/7/amd64/latest wheezy main
Setup
(Please provide relevant configs and/or SLS files (Be sure to remove sensitive info).)
Steps to Reproduce Issue
(Include debug logs if possible and relevant.)
Versions Report
(Provided by running salt --versions-report. Please also mention any differences in master/minion versions.)

Ok i found the problem.
i think we stick with an old problem of size in the repo that was keeped in our apt-cacher-ng, as we had start look salt on a test server some time ago.
And since this time then he should have refused to update and reread the repo.
Or there is some correction on the repo.
but i totally clean all traces of saltstack repo on servers and apt-cache-ng, and restart upload. seems ok.
I test on a new empty debian 7 client.

Ok i confirm, when local cache  (or a server cache like apt-cache-ng) had an older version inside, he seems not update anymore with this error.
I test on 3 different  environment cases same result, and same solution : clean/reset all caches (apt-cache-ng i completely remove any data speaking of salt), and upload the new
--> OK for new install or update.


Try ssh command and freshly booting vms

This step may fail when the ssh service is not ready
for auth, even tough

it is root login
we have rng dev
the port was listening

Retry.
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running

/approve


Added API caveat for mmctl user migrate-auth

Based on TAM experience and recommendations.

This PR isn't needed. Closing.


[BUG] Partially does not work downloading images through the phone

Version information
Laravel: v5.6.35
Voyager: v0.11
PHP: 7.2
Database: MySQL 5.7
Description
Faced the problem of downloading images through the phone. When you try to download images with the format jpg, png in large register JPG, PNG gives an error that the images are loaded in the wrong format. If you upload 1 image, not a set, then you receive a notification about the successful download, but the image itself does not load.
Recently checked the same problem on the PC

Are you really using Voyager 0.11?
If so, please upgrade. There's no way someone is able to give your support or even bugfixes for a 2 year old product.

Are you really using Voyager 0.11?
If so, please upgrade. There's no way someone is able to give your support or even bugfixes for a 2 year old product.

Sorry, wrong version - 1.1

Developers, you can try it yourself - you should have the same error - try downloading the image in the above mentioned formats with upper case type on mobile devices and on PC

Are you really using Voyager 0.11?
If so, please upgrade. There's no way someone is able to give your support or even bugfixes for a 2 year old product.

Developers, you can try it yourself - you should have the same error - try downloading the image in the above mentioned formats with upper case type on mobile devices and on PC

@xzACzx are you still experiencing this problem with last Voyager release 1.3.1?
Honestly I don't see how Voyager can have anything to do with the download of an image.

The error was fixed by the latest update.
The problem was not in the pictures, but in their type
Voyager counted pictures only with a lowercase type.
For instance:
image.png - read
image.PNG - error
ср, 1 янв. 2020 г. в 11:42, MrCrayon<EMAIL_ADDRESS>
@xzACzx https://github.com/xzACzx are you still experiencing this
problem with last Voyager release 1.3.1?
Honestly I don't see how Voyager can have anything to do with the download
of an image.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/the-control-group/voyager/issues/4042?email_source=notifications&email_token=AHDZJCTZWNMG6YBIL3QOPHLQ3RQQRA5CNFSM4HBQJATKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH5BOWI#issuecomment-570038105,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHDZJCVNVV57NMVAVILAHSTQ3RQQRANCNFSM4HBQJATA
.

But Voyager doesn't check anything when you download an image unless the problem was on upload.
Anyway good it's solved.


Updates commandLine switches to occur before app start

Updates the commandLine switches to occur before the app start so they can take effect.

thanks!

Hey @aaronabramov! Sorry to ask! Is it possible to publish this change to npm?

oh no. i didn't realize i didn't publish it. i'll do it right now
sorry about that @smacpherson64

No worries at all, thank you!

<EMAIL_ADDRESS>


[3.0-Triple]Remove auto wrap when attachment is an object instance

What is the purpose of the change

Do not wrap object in attachment
Fix triple server thread pool name's typo
Remove hanlded exception info from metadata

Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #8473 (41b18d7) into 3.0 (6808f8f) will decrease coverage by 0.15%.
The diff coverage is 22.22%.

@@             Coverage Diff              @@
##                3.0    #8473      +/-   ##
============================================
- Coverage     62.84%   62.68%   -0.16%
+ Complexity      312      311       -1
============================================
  Files          1112     1112
  Lines         46633    46574      -59
  Branches       7016     6979      -37
============================================
- Hits          29306    29196     -110
- Misses        14064    14114      +50
- Partials       3263     3264       +1

Impacted Files
Coverage Δ

...ache/dubbo/rpc/protocol/tri/UnaryClientStream.java
24.59% <0.00%> (-0.41%)
:arrow_down:

.../apache/dubbo/rpc/protocol/tri/AbstractStream.java
45.45% <28.57%> (+2.59%)
:arrow_up:

...ng/transport/dispatcher/all/AllChannelHandler.java
62.06% <0.00%> (-20.69%)
:arrow_down:

...ng/transport/dispatcher/WrappedChannelHandler.java
43.47% <0.00%> (-15.22%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
50.00% <0.00%> (-11.54%)
:arrow_down:

...in/java/org/apache/dubbo/common/utils/JVMUtil.java
81.13% <0.00%> (-11.33%)
:arrow_down:

...mmon/threadpool/support/fixed/FixedThreadPool.java
80.00% <0.00%> (-7.50%)
:arrow_down:

.../remoting/transport/netty4/NettyClientHandler.java
62.06% <0.00%> (-6.90%)
:arrow_down:

...moting/transport/netty4/NettyEventLoopFactory.java
30.00% <0.00%> (-6.37%)
:arrow_down:

.../threadpool/support/limited/LimitedThreadPool.java
83.33% <0.00%> (-5.56%)
:arrow_down:

... and 39 more

Continue to review full report at Codecov.

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


Problem with storage of mmul operation

Issue
I'm trying to loop through and individually multiply the inner matrices of two nD tensors, and mmul seems to have issues storing data when the calling matrix is stored within an INDArray. The two "for" loops below are essentially the same except for one difference: in the first loop, the matrix which calls mmul is referenced as a matrix within dir100, whereas in the second loop, the calling matrix is a standalone 2d matrix. For some reason, when executing the first loop, mmul fails to store any values in the output tensor (productArr). The second loop executes without an issue.
The code is in scala, not java, but i'm pretty sure nD4s and nD4j use the same mmul function, and the nd4j page is much more active.
My Code
` var arr1:INDArray = null
var dir100:INDArray = Nd4j.zeros(1,6,3,1)
dir100.getRow(0).putRow(0,Nd4j.create(Array(Array(1.0),Array(0.0),Array(0.0))))
dir100.getRow(0).putRow(1,Nd4j.create(Array(Array(1.0),Array(0.0),Array(0.0))))
dir100.getRow(0).putRow(2,Nd4j.create(Array(Array(1.0),Array(0.0),Array(0.0))))
dir100.getRow(0).putRow(3,Nd4j.create(Array(Array(1.0),Array(0.0),Array(0.0))))
dir100.getRow(0).putRow(4,Nd4j.create(Array(Array(1.0),Array(0.0),Array(0.0))))
dir100.getRow(0).putRow(5,Nd4j.create(Array(Array(1.0),Array(0.0),Array(0.0))))

var productArr = Nd4j.zeros(1,6,3,3)
print("Before")
println(productArr)
for(i<-0 until 6){
  arr1 = dir100.getRow(0).getRow(i)
  arr1.mmul(arr1.transpose(),productArr.getRow(0).getRow(i))
}
print("after")
println(productArr)

var arr4 = Nd4j.zeros(1,6,3,3)
var arr3 = Nd4j.create(Array(Array(1.0),Array(0.0),Array(0.0)))
for(i<-0 until 6){
  arr3.mmul(arr3.transpose(),arr4.getRow(0).getRow(i))
}
print("arr4: ")
println(arr4)`

Version Information

Scala Version: "2.11.12"
Nd4s Version: "0.9.1"
Platform: Windows x64

Contributing
Happy to contribute for a fix, mostly just want to know if I'm doing something wrong.

Note that the issue only appears to arise when the dimensions of the product don't line up with the dimensions of the calling matrix. In this case, mmul is 3x1 times 1x3, giving a 3x3 product. If the calling matrix is in a tensor of 3x3 matrices, there's no trouble.


puppet: Upgrade sentry-cli.

This version causes sentry-cli monitors run to not fail if Sentry is down (getsentry/sentry-cli#2169).

Merged, thanks @alexmv!


add support for proxying 100 continue headers

Fixes https://github.com/istio/istio/issues/21382 and https://github.com/istio/istio/issues/36278
Added a feature flag that can be removed later.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Dual Stack
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

AFAIK, it does not have any negative impacts on any routes.  https://github.com/istio/istio/issues/21382#issuecomment-669525370 also confirms the same

If we don't think it will break anything and should be the new default, but it is a behavioral change, do we want this to be affected by compatibility version?

If we don't think it will break anything and should be the new default, but it is a behavioral change, do we want this to be affected by compatibility version?

It should be compatible with all versions and I do not think this change should impact any one negatively. That is why I defaulted it to true and if there are any cases that need it to be false for some reason, they can turn it off. My plan is to remove this env var after couple of releases.
Other option is an API in VS but that IMO that is an overkill here.


Downgrades Some Deps for Expo-Flavor.

There are some open issues mentioning Ignite not running and reference expo-linear-gradient (e.g. #1963 , #1954). Those aren't 100% related (since they are specific to vanilla-flavor), but I did confirm that a freshly ignited app (using expo-flavor) does not build.

iOS Output
Android Output

While starting the Expo dev server, I noticed this:

Recently, there have been some new Ignite releases (upgrading to 0.68.x). The expo core dep got updated in vanilla-flavor, but not in expo-flavor (possibly due to expo-cli prerequisite issue?). In any case, it's a good idea to always use recommended expo library versions. Upgrading core without libraries (or vice versa) can cause things to break. The real fix to this is to upgrade the expo-flavor boilerplate to latest and make sure we continue overriding the expo-flavor deps/libs according to the recommended.
I downgraded the packages (in the expo-flavor package.json) that have jumped major versions compared to the recommended. The app now builds:

PS: this won't fix the tagged issues unless they re-ignite their apps which isn't practical.

This PR looks good, closing in favor of #1930 (which includes this fix also)


Windows installer failing: SignTool Error: File not found: C:\Users\jenkins\windows.p12

https://ci.adoptopenjdk.net/job/build-scripts/job/release/job/sign_build/11503/console
find: 'basename' terminated by signal 13
.... lots of these...
find: 'basename' terminated by signal 13
Signing Windows release
Signing ./bin/fixpath.exe
SignTool Error: File not found: C:\Users\jenkins\windows.p12

RETRYWARNING: Failed to sign ./bin/fixpath.exe at 00:05:34: Possible timestamp server error - RC 0 ... Retrying in 10 seconds
SignTool Error: File not found: C:\Users\jenkins\windows.p12

Seems an issue on build-azure-win2012r2-x64-1 marked offline

As suggested by @gdams on slack , we're going to move the windows.p12 cert over to C:\openjdk, as often there are differences between machines in that sometimes the Jenkins user is at C:/users/jenkins.<Something_random>. Hopefully this will stop these errors

With the above build PR being merged, this should hopefully be fixed. This installer run started about 10 minutes after the PR was merged and it looks okay. I'll wait until the nightlies run before I close this issue though

Looks okay from last night, closing


'sequences' in continuous coalescent shiny app

am I correct - it seems that the number of mutations mapped onto the tree in the continuous coalescent app does not always match how it is distributed across the 'sequences' in plot below - seems sometimes an extra row (mis-counting the number of mutations signaled) gets added. double-checking because have been using for grad teaching. Example attached that I struggle to explain to my students based on n or number of mutations. Sorry if I'm off.

Interesting! I definitely agree the alignment doesn’t match mutations on the tree. I can make similar things happen on my end too. Weird that’s it’s rare. Must be some bad conditional somewhere or something.
Well I welcome the excuse to refactor the code for this app and try to figure out what’s going wrong.
Sorry about that! Guess just let folks know there a bug. Will let you know if and when it’s fixed.
From: John Wares @.>
Date: Thursday, February 2, 2023 at 11:46 AM
To: silastittes/shiny_popgen @.>
Cc: Subscribed @.***>
Subject: [silastittes/shiny_popgen] 'sequences' in continuous coalescent shiny app (Issue #3)
am I correct - it seems that the number of mutations mapped onto the tree in the continuous coalescent app does not always match how it is distributed across the 'sequences' in plot below - seems sometimes an extra row (mis-counting the number of mutations signaled) gets added. double-checking because have been using for grad teaching. Example attached that I struggle to explain to my students based on n or number of mutations. Sorry if I'm off.
[Screen Shot 2023-02-02 at 1 44 47 PM]https://user-images.githubusercontent.com/9868830/216420557-34b915ff-192f-4073-90f0-84bbf5bf1bbe.png
—
Reply to this email directly, view it on GitHubhttps://github.com/silastittes/shiny_popgen/issues/3, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABLZ2X7BPCDJQAYE62QS75TWVP6HZANCNFSM6AAAAAAUPMHCZM.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Ok I think I have a fix. There were several issues.
I made some tweaks to the sequence visualization while I was at it.
They look nice ...

But kinda cluttered when there's a lot of mutations ...

Do you like the new or old viz more?
Will wait for feedback before merging the updates. Thanks!

 Still feel free to comment on alignment viz.

Yeah I still see it not quite tracking correctly. Thanks so much for working on this.
Dr. John Wares
Professor – Dept of Genetics/
Odum School of Ecology
Curator – Genomics and Aquatic Invertebrate Diversity, Georgia Museum of Natural History
University of Georgia
He/Him
twitter @wareslab
If I say no to your request, it is because I recognize my current commitments and wish to honor them.
On Feb 3, 2023, at 11:23 AM, Silas Tittes @.***> wrote:
[EXTERNAL SENDER - PROCEED CAUTIOUSLY]
Oh wait, it's still wrong. lol. Still feel free to comment on alignment viz.
—
Reply to this email directly, view it on GitHubhttps://github.com/silastittes/shiny_popgen/issues/3#issuecomment-1416099397, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACLJMHRLOF6MFKOFO6NKMFDWVUWHHANCNFSM6AAAAAAUPMHCZM.
You are receiving this because you authored the thread.Message ID: @.***>


feat: add code to generate not crossed P4J features

Hi @martinezmatias，
In this commit, we added functions to extract not crossed features.  In the generated JSON files, the features are constructed into three keys: SRC, FORMER, and LATER, to indicate the features of the changed AST, three statement blocks in the AST former the changed AST and later. I did not delete any original code but add an option to configure crossed and non-crossed features.
In addition, I fixed the bug to obtain the former/later statements. They were the same as the changed statement.

Hi @SophieHYe
I'm a bit confused by the terminology: what does it mean not crossed features?  If they are the "normal" features that we usually consider, why to call them "not crossed" and simply "P4J features"?

@martinezmatias, It's good to call them P4J features.

Hi @SophieHYe

It's good to call them P4J features.

Great, so they are the "normal" P4J features. Could you rename the occurrences of not-crossed?
Thanks a lot

Hi @SophieHYe
Merged!
Thanks a lot for the changes.


[Bug] Redis plugin: use connection pool for thread safety.

Description

Currently, redis plugin uses a single connection instance, which is not thread safe.  It would be better to move to a connection pool instead.

Closing it since it was fixed in one of the earlier commits.


"brimcapd" server

At the moment Brimcap only allows for populating and querying a local "Brimcap root". This means that if a Brim app is connected to a remote Workspace and accesses a Pool that was created by loading a pcap via Brimcap at that remote side, when they click the Packets button, their local Brimcap root will still be queried and the flow will not be found. If the user is savvy enough to run brimcap index locally against the same pcap to populate their personal Brimcap root, that would make the Packets button work as expected. But this is probably asking too much of users.
When contemplating this feature gap, we recognized there's room for something like a "brimcapd server" such that the local Brimcap could do a remote "search" by connecting to the remote brimcapd, which could then extract the relevant flow and return it over the network to be displayed locally in Wireshark.

Note that issue is linked to from the "Remote Workspaces (v0.25.0+)" article in the Brim wiki. If/when this gets addressed, the article should be updated.

https://www.qacafe.com/analysis-tools/cloudshark/tech-integrations/pcapdaemon is an example of an OEM partnership, where Cloudshark/qacafe.com had done the heavy-lifting for this problem domain. Maybe the 85% solution is to Partner/OEM program with them?


For Cud?

Hi Patrick;
Maybe possible to adapt theme for CudaText?
It has visual dialog to edit theme (Options/ Settings-more/ colors)

Hey @Alexey-T I'm not using this Theme right now because I migrated from Sublime to ATOM. Never used CudaText before, don't know how to create a theme for it 😢

well, dialog  (Options/ Settings-more/ Sett-colors) is theme configurer. it is colors for ui, and styles for lexers/syntax.