Geometry shader input primitive type - control point patch

Seems not able to submit an issue to https://github.com/MicrosoftDocs/win32. Not sure here is the right place to go for this issue.
In the shader model 4.0 for geometry shader page, https://github.com/MicrosoftDocs/win32/blob/docs/desktop-src/direct3dhlsl/dx-graphics-hlsl-geometry-shader.md, there is a primitive type 1-32 control point patch in the Geometry Shader Input Primitves diagram. But the type does not exist in the primitive type table.

Primitive Type
Description

point
Point list

line
Line list or line strip

triangle
Triangle list or triangle strip

lineadj
Line list with adjacency or line strip with adjacency

triangleadj
Triangle list with adjacency or triangle strip with adjacency

The question is how to use the control point patch type in the geometry shader.

Looks like the correct name is "patchN", e.g. "patch1" for a 1-control-point patch.

Thanks Jesse, I have found the answer in https://learn.microsoft.com/en-us/windows/win32/direct3dhlsl/sm5-object-inputpatch, it needs shader model 5.0 support.


Update lading to 0.25.0

What does this PR do?
Motivation
Describe how you validated your changes

Possible Drawbacks / Trade-offs
Additional Notes

#32271  👈 (View in Graphite)
main

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

/merge


Remove unnecessary nose test dependency

[x] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

nose was our original test framework.  It was replaced with pytest in #204, but kept as a dependency due to being some kind of indirect requirement that I don't really understand (https://github.com/pvlib/pvlib-python/pull/232#discussion_r75128631).  In any case it doesn't seem to be necessary today, so this PR gets rid of it.

a pandas test function that requires nose

said @wholmgren in your linked comment, so does pandas still depend on nose? doubt it.

I say merge it!


Decoding did not generate any output

Hi there,
I am trying to use transformer model for a text summarization task. the training seems go well, but decoding did not give me any output.
the problem definition and Tensorboard results is at this gist url.
and my command for training and decoding:
for training:
t2t-trainer --generate_data --train_steps=35000 --eval_steps=1000 --save_checkpoints_secs=3600 --keep_checkpoint_max=10 --t2t_usr_dir=~/problem/src --data_dir=~/problem/data --problems=my_problem --model=transformer --hparams_set=transformer_my_problem _hparams_set --output_dir=~/problem/out
for decoding:
t2t-decoder --t2t_usr_dir=~/problem/src --data_dir=~/problem/data --problems=my_problem --model=transformer --hparams_set=transformer_my_problem_hparams_set --output_dir=~/problem/out --decode_hparams="beam_size=4,alpha=0.6" --decode_from_file=/home/flynn/problem/data/dev.src
I'm sure there is not empty target in my training data. So the empty output for all the decoding file is not expected. Please help me if you see anything wrong. Thanks!

It’d be helpful if you provide the following:
Output of ls in the output_dir
Output of the t2t-decoder command

Sorry for the delay. Is this still an issue with recent releases?

seem it's not a say for you to debug. I'll try new version and see. Thanks!


Failed to run automated scorecard-action@main test-organization-ls/scorecard-action-private-repo-tests

https://github.com/test-organization-ls/scorecard-action-private-repo-tests/actions/runs/2178685695

@laurentsimon @rohankh532 @azeemshaikh38  FYI...

Have we set the COSIGN_EXPERIMENTAL=1?

Just set the ENV Variable

Hang on :-) Nice fix, but this should be fixed in code (os.Setenv()) because that's what the action will run, right?

I have updated the action with this id-token: write
https://github.com/test-organization-ls/scorecard-action-private-repo-tests/blob/bcb770f821774f3d7db16bdb339f77950cdc2774/.github/workflows/scorecards-golang.yml#L21

closing, as per https://github.com/ossf/scorecard-action/issues/180#issuecomment-1105508516


Update the TOTP related docs with the latest changes.

Is your feature request related to a problem? Please describe.
The TOTP flow was changed in My Account and in the authentication portal. These changes can be found in the following PRs:

https://github.com/wso2/identity-apps/pull/1546
https://github.com/wso2/identity-apps/pull/1534
https://github.com/wso2/carbon-identity-framework/pull/3334
https://github.com/wso2-extensions/identity-outbound-auth-totp/pull/84

The relevant documents should be updated to accommodate the changes introduced through the above PRs.
Related issue: https://github.com/wso2/product-is/issues/11017

Updates done

Screenshots updated according to https://github.com/wso2/identity-apps/pull/1546
Added Additional configurations according to https://github.com/wso2/carbon-identity-framework/pull/3334

Related PRs:

https://github.com/wso2/docs-is/pull/3184
https://github.com/wso2/docs-is/pull/3164

Moving the issue to Milestone 6.1.0-checkpoint to complete the other tasks

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


ISPN-7543 Run kill-server before start-server in integration tests

https://issues.jboss.org/browse/ISPN-7543

Pushed to master


Initialize Foundry for testing and scripting

Discussed new strategy for implementing Foundry with @Brean0.

Too many JS tests to migrate at this time. Focusing on their coverage & efficacy before doing any sort of language migration.
Merge this into docs
Update remappings to simplify module refactor
In the short term, use Foundry for fuzz tests and fast compiling
In the long term, consider a migration to a Solidity-based test suite

Note that sunrise improvement tests on #133 are currently Solidity-based.


Meeting SDK join meeting disappears after bringing app to foreground

Hello,
Android works well with minSdkVersion 24. But on iOS 17.1.1, after joining a meeting everything’s working correctly but after sending the app to the background and then bringing it to the foreground I can’t no longer see the Meeting view controller (only the view controller from which I joined the meeting) but camera and audio still work. Basically this and this issue.
Is it related to Zoom SDK? Could you upgrade Zoom SDK to v<IP_ADDRESS>60?

Hi @patriksabol thanks for letting me know - I'll update the SDKs as soon as possible.

Hi, have you also met with this issue about backgrounding the app and lost Meeting view controller?


ICSharpCode.SharpZipLib.Zip.ZipException: Wrong Central Directory signature

File to test: https://dl.google.com/android/repository/sys-img/android-wear/x86-P_r02.zip.
SharpZipLib fails to unzip this file, while Windows10 is able to do it using Extract All... menu item.
Steps to reproduce

Download the test file;
Try to unzip it using SharpZipLib;
Observe the Wrong Central Directory signature error.

Version of SharpZipLib
1.1.0
Obtained from (only keep the relevant lines)

Package installed using NuGet

Confirmed. This is due to the fact that the file has a Zip64 Central Directory, while not using any Zip64 extensions. It also contains a non-zip64 CD, but it seems to be corrupt.
The way SharpZipLib detects whether to use Zip64 is if any of the fields are set to all 1s (0xffff etc), and only then tries to find the Zip64 signature.
This means that  unzip and other programs that just tries to find both headers at once works fine, while SharpZipLib (albeit correctly) complains about the invalid Central Directory.
We should probably attempt to find the Zip64 signature at once as well, this would my the library work more like most other zip-implementations and handle non-compliant zip files better.

I made a new issue #357 for the root cause to link this and other issues into.


connexion.readthedocs.io (RTD) not up-to-date

Description
It seems that readthedocs site is not updated with recent documentation updates.
I was looking at issues related to MethodView and found PR #847. That PR includes documentation changes, namely a section in routing named Automatic Routing with MethodViewResolver but i'm not able to find it.
Expected behaviour
The section is available on RTD https://connexion.readthedocs.io/en/latest/routing.html.
Actual behaviour
Section is not there.
Steps to reproduce
Visit link: https://connexion.readthedocs.io/en/latest/routing.html
Additional info:
N/A

This has been fixed.


Stream timeout with asset issuances

The api seems to be having issues with asset issuances at the moment.
https://api.counterparty.io:4000/v2/addresses/bc1qy25gwr9vdrvxx0w35wa9vrpc3hc5h6rd0p56f4/compose/issuance?asset=A12590236770677120000&quantity=1&divisible=false&lock=true&description=STAMP%3A&exact_fee=15095&return_psbt=true

Yeah Bitcoin Core is overloaded because we're updating some nodes. We need to migrate to using electrs for initial mempool synchronization because it's not single-threaded for RPC. Cf. https://github.com/CounterpartyXCP/counterparty-core/issues/2837 and https://github.com/CounterpartyXCP/counterparty-core/issues/2843


Inconsistent appendixes in table of contents

A and B have their labels in the ToC, but C doesn't. This should be made consistent.

Fixed. In the process, I changed some spacing around the appendix titles. Let me know if you think it looks too dense.


Feature request: Automatic device discovery

Is your feature request related to a problem? Please describe.
It would be great to have a parameter for automatic device discovery with an option to disable it. Often, devices that are no longer on the network still appear in the device list. Additionally, new devices might appear on the network over time, and to see them, you need to manually initiate a discovery each time, which isn't always convenient. As for the interval for automatic discovering, it would be excellent if it could be set manually. However, having a fixed interval of 10, 15, or 30 seconds would also be quite sufficient.
Describe the solution you'd like
Described above.
Describe alternatives you've considered
No response
Additional context
No response

in my company there a policy that forbid to do scans like device discovering, it would be nice to have an option to disable it for me to keep using this nice app.


npm ERR! request to https://registry.npmjs.org/yarn failed, reason: self-signed certificate in certificate chain

HI, thanks for your great work, when I am building viser, this is the error:
(viser) No client build found. Building now...
(viser) nodejs is set up!
npm ERR! code SELF_SIGNED_CERT_IN_CHAIN
npm ERR! errno SELF_SIGNED_CERT_IN_CHAIN
npm ERR! request to https://registry.npmjs.org/yarn failed, reason: self-signed certificate in certificate chain

npm ERR! A complete log of this run can be found in: /home/ma-user/.npm/_logs/2024-11-07T06_52_03_790Z-debug-0.log

I would appreciate if anyone could help me!

Hi, there are another question...
AttributeError: module 'viser.extras' has no attribute 'Record3dLoader_Customized'

Hi, sorry for missing this issue!
I'm not sure about the nodejs error unfortunately... perhaps this is a VPN or network issue? If you install viser from pip instead of source the client will also come pre-built, which would avoid this error.
For the AttributeError perhaps you should be using this fork of viser? https://github.com/Junyi42/viser
We don't have a class called Record3dLoader_Customized in the main/official repo.

Oh, many thanks !!!! indeed I am using this :https://github.com/Junyi42/viser
Thank you!!
And how can I customize the npm path? I found that viser has its own npm/nodejs. So I have no idea to change it.

ok i've solve the problem, just add npx path to $PATH and cancel the certificate:
export PATH=/home/data/miniconda3/envs/base/lib/python3.10/site-packages/viser/client/.nodeenv/bin
npm set strict-ssl false


setTimeout / setInterval in event handler doesn't work correctly

[x] I have searched the issues of this repository and believe that this is not a duplicate.

In my project using next.js, I was trying to make carousel for mobile.
But for some reason, setTimeout / setInterval in event handler doesn't work correctly.
Timer takes few seconds even if set 100ms.
This is affect ONLY for Android Chrome. iOS Safari and Chrome work fine.
Here is my sample component code.
handler is called right after touchend, but the code inside setTimeout run after few seconds.
This component works fine in the project by using create-react-app .
import React from 'react';

class Scroller extends React.Component {
  constructor(props) {
    super(props);
    this.state = {
      scrolled: [],
    };
    this.handler = this.handler.bind(this);
  }
  handler(e) {
    setTimeout(() => {
      this.setState({
        scrolled: [...this.state.scrolled, 'yes'],
      });
    }, 100);
  }
  render() {
    return (

        {[1,2,3,4,5,6,7,8,9,10].map(key => (

{key}

        ))}

        {this.state.scrolled.map((str, i) => (

{str}

        ))}

)
  }
}
export default Scroller;

Your Environment

Tech
Version

next
3.2.2

node
8.4.0

OS
MacOS 10.12.6

browser
Android Chrome

I guess this is related to React and not Next 🤔

https://stackoverflow.com/a/11613327

Thanks @pruhstal! ❤

@timneutkens I thought so, but like I wrote, pure react project works fine.
@pruhstal thanks. but its not woking even i added preventDefault. and its about Android <4, really old post.
Could someone try my code?

@hiromitz yes, it's old, but if you expand the comments, someone said it's still broken in 4.4.4 (~2015). Which could mean that it's still not fixed today.

@pruhstal Oh, I see. Thanks.


Add initial version of deploy_on_spaces

Pull Request Template

Add default deployment options for Hugging Face spaces through the Hugging Face hub api.
Closes #5108

Added utils directory under the assumption we will soon add telemetry too
Added huggingface_hub as core dependency under the assumption it is lightweight and we will soon use it for telemetry too.

TODO:

docs
tests
evaluation w.r.t. oauth deployment for using argilla vs argilla-space-with-oauth @frascuchon do you know what the difference is when using these templates?

Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested

Checklist

I added relevant documentation
follows the style guidelines of this project
I did a self-review of my code
I made corresponding changes to the documentation
I confirm My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

it would fit quite nicely in a "huggingface spaces sdk" , which is something to look forward to

@burtenshaw not sure why the OAuth is an issue. I thought of adding it as an extra but I think the value of the integration also lies in the fact that it works on the fly as an entry point so I would then propose showing it on import or function execution instead of mentioning it everywhere through the docs. WDYT?

@frascuchon argilla-space vs argilla-space-with-oauth do you know what the difference is between these Hugging Face spaces templates?

@burtenshaw not sure why the OAuth is an issue. I thought of adding it as an extra but I think the value of the integration also lies in the fact that it works on the fly as an entry point so I would then propose showing it on import or function execution instead of mentioning it everywhere through the docs. WDYT?

Sorry. I didn't explain this point properly. I meant that oauth, and other challenges, could increase the complexity of the module and so might be better placed in another package.
That said, I'm not 100% against this inclusion in the SDK. But I do think it's a significant change that we should find the best place for. My main concern is that we end up with too much utilities code in the SDK that's difficult to maintain on a daily basis. And doesn't offer much value to the user beyond a well documented Spaces deployment guide.
If we do include this in the SDK, rather than an extension, I'd say that we should interface it consistently with the Argilla class.
But, I'd like to also here from @frascuchon and @dvsrepo .

it would fit quite nicely in a "huggingface spaces sdk" , which is something to look forward to

@Josephrp what's the latest on this?
@davidberenstein1957 should we follow this up?

@frascuchon argilla-space vs argilla-space-with-oauth do you know what the difference is between these Hugging Face spaces templates?

The official is this one: https://huggingface.co/spaces/argilla/argilla-template-space (the one that is used by the New Space UI HF flow too).
The other one is an unofficial one I created to be able to launch oauth spaces more easily

@dvsrepo @burtenshaw and I decided to go forward with this. @frascuchon do you have any specific opinion about the PR and location of the Mixin within the package?

@davidberenstein1957 Is this ready to go into 2.1.0 version or should I move it to 2.2.0?


Added new "geohashField" for more flexibility

I added this field because we needed it in our project as we had a predefined database structure that had the geohash already present but with a different field name than the one hard coded in the package's code.
Feel free to review this small change and mention any issues you can see in it.

@iabdousd
Thank you so much for your PR!
Looks good at first glance. I'll give it a closer look before giving my approval, please wait a little more!

@iabdousd
Now geoflutterfire_plus new version (v0.0.15) including your contribution is available🎉
https://pub.dev/packages/geoflutterfire_plus/changelog
Thank you for your contribution again!

Thanks for the rapid review and approval @KosukeSaigusa !


adding gallery

Hey,
This website has a theme moody and without having a gallery of moody pictures, its incomplete.
Hence, I added few such pictures to it.
Updated the gallery in the header section too.
Also, used tailwind css for this.
Can this PR be merged?
Thank you !

@aishwarya1735 Kindly share a ss here about what things you have added.

Closing this as not required


Luma wont update.

I havent used my 3ds in a few months and my luma is still on 6.6, when i use the luma updater 2.2 or 2.1.2 (tried both) and choose to install stable version, it says it installs it, but when it reboots and i open luma updater it says im still on 6.6

That's because you need boot9strap. Using my fork of Luma Updater does nothing for you right now as you still have a9lh. Follow this guide and then you'll be able to use my fork of Luma Updater. Next time you have an issue, please post the issue in the correct repo.


Every time, when closing git-bash tab, the application crashes (not responding)

How to fix/debug this, cos debug console doesn't show any error, before terminal becomes unresponsive.
What I tried:

exe: c:\git\git-cmd.exe arg: --no-cd --command=usr/bin/bash.exe -l -i
exe: c:\git\bin\bash.exe arg: -l -i

Wanted to note, that this only happens when using login option.


[Enhancement] Dump mem image before be exit to check mem leak

Why I'm doing:
With the help of jemalloc's mem_leak logic, the memory image is output, which is convenient for users and tests to check memory leaks.
What I'm doing:
How to usage:

./bin/start_be.sh --daemon --check_mem_leak
kill starrocks_be (without -9)
When be exit, will output one heap profile

<jemalloc>: Leak approximation summary: ~33504584 bytes, ~21783 objects, >= 4201 contexts
<jemalloc>: Run jeprof on dump output for leak detail

jeprof.1747309.0.f.heap

convert the file to pdf

./bin/jeprof --show_bytes --nodefraction=0.000000001 --edgefraction=0.000000001 --pdf lib/starrocks_be.debuginfo jeprof.1747309.0.f.heap >b.pdf

What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

can we update this tip in sr's doc? It's useful.

can we update this tip in sr's doc? It's useful.

Good suggestion, i will add this to doc


sap.deviceInfo not working in particle setup

Node: v4.6.0
OS: Windows 10 x64
Particle-cli: 1.17
When particle setup runs (inside commands/WirelessCommand.js):
sap.deviceInfo(pubKey).on('error', function(err) {

            if (err.code === 'ECONNRESET') {
                return;
            }
            retry = setTimeout(info, 1000);
        });

Err returns:

Also, if I run softap-setup (4.0.1) standalone same issue occurs:

The device is connected to the laptop in listening mode (blinking blue), and before that I did a factory restore.
Another ticket that shows this issue: https://community.particle.io/t/solved-unknown-issue-with-photon-edit-photon-cannot-connect-to-cloud/19293/25

OK. Thanks for reporting this.

@danielmapar  thanks for the report. I'm not able to reproduce it with the information we have. I hope you can provide more details:

Please tell me the system firmware version on the Photon that is experiencing this issue. (Put the device in listening mode, then run particle serial identify. )
Can you post your test.js script for the standalone test so I can be sure we are testing the same thing.
does the problem happen every time, or just sometimes?

I've managed to reproduce this by modifying example/basic-setup.js.


[enhancement] [WBT + WbW] Set upper and lower limit for contours

I guess this applies to both WBT and WbW. When creating contours (from a raster or vector), we have the option to set the base contour level, but we can't set a 'top' limit. Having this option would allow us to create contours for a slice of a DEM, for instance (such as creating two/more sets of contours, with different spacing based on the slope when working on areas with both mountains and plains)

This is a good idea Carlos. I can certainly add this feature.


Error 500 in GET new path or PATCH new file

Expected behavior
Create new link or upload file in existing path.
Current behavior
Return 500 and given the error page.
Your Environment
I was using the public env (http://dontfile.com)

@mateusmanuel We're running on free dynos on Heroku. Unfortunately, they have a limit on the number of rows the database can have. That's why we can create more pages/files. The best I can do for now is delete some pages =(
Also, as Heroku is removing all free Dynos, we'll have to change providers soon.


Checklist: Large Elements with Link Copy Update

Following #9654 - copy update per @choumx :
https://github.com/google/web-stories-wp/pull/9654#discussion_r748635732

Elements with links must not cover most of the page. Decrease the size of elements with links.

Instead of

Links cannot cover a large majority of the page. Use smaller elements for links.

Verified in QA


Re-render

@bekozi I should have done this a long time ago. Sorry for dropping the ball here.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Failing with:
======================================================================
ERROR: test_calc (ocgis.test.test_simple.test_simple.TestSimple)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/ocgis/test/test_simple/test_simple.py", line 590, in test_calc
    ret = self.get_ret(kwds={'calc': calc, 'calc_grouping': group, 'aggregate': True, 'calc_raw': calc_raw})
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/ocgis/test/test_simple/test_simple.py", line 91, in get_ret
    ret = OcgInterpreter(ops).execute()
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/ocgis/api/interpreter.py", line 125, in execute
    ret = conv.write()
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/ocgis/conv/numpy_.py", line 14, in write
    for coll in self:
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/ocgis/conv/numpy_.py", line 9, in __iter__
    for coll in self.colls:
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/ocgis/api/subset.py", line 75, in __iter__
    for coll in self._iter_collections_():
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/ocgis/api/subset.py", line 139, in _iter_collections_
    coll = self.cengine.execute(coll, file_only=self.ops.file_only, tgds=tgds)
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/ocgis/calc/engine.py", line 138, in execute
    out_vc = function.execute()
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/ocgis/calc/base.py", line 154, in execute
    self._execute_()
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/ocgis/calc/base.py", line 627, in _execute_
    fill = self._get_temporal_agg_fill_(value, shp_fill=shp_fill)
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/ocgis/calc/base.py", line 433, in _get_temporal_agg_fill_
    fill[ir, it, il, :, :] = self.aggregate_spatial(cc, weights)
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/ocgis/calc/base.py", line 119, in aggregate_spatial
    ret = np.ma.average(values, weights=weights)
  File "/feedstock_root/build_artefacts/recipe_root_1486655374736/_t_env/lib/python2.7/site-packages/numpy/ma/extras.py", line 592, in average
    "Axis must be specified when shapes of a and weights "
TypeError: Axis must be specified when shapes of a and weights differ.

My guess is that ocgis is not OK with latest numpy. I am trying to pin numpy to <1.12 to test.

That looks familiar and is caused by a newer version of numpy. Something changed in ma.average. I first noticed it in 1.11.2. Fixed in the trunk, but not in this release.

@bekozi it seems that you need numpy <1.11 but conda-forge dropped support for that numpy to avoid strain on the CIs. That is a problem because we moved to icu 58.* , making the old fiona, compiled with numpy 1.10 and icu 56.* incompatible with the rest of the stack (gdal for example).
Maybe we could add a patch to make ocgis compatible with latest numpy?

Crossing posts here...yeah, let me get a patch together...

@bekozi I still see the failure with that patch :unamused:
Do you want to merge this to get the feedsotck updates in and work on the patch in another PR?

Patch is not being applied from what I gather. Any thoughts on why this may be happening? The failure is:
File "/feedstock_root/build_artefacts/recipe_root_1486657558700/_t_env/lib/python2.7/site-packages/ocgis/calc/base.py", line 119, in aggregate_spatial
    ret = np.ma.average(values, weights=weights)

Relevant patch line is:
https://github.com/conda-forge/ocgis-feedstock/pull/7/files#diff-73bbb073487aa6de16abad709c7f28d7R13
+-        ret = np.ma.average(values, weights=weights)
++        ret = np.ma.average(np.squeeze(values), weights=weights)

Target code line is not updated. Does it make sense to test the patch locally?

Patch is not being applied from what I gather.

The problem is that you got a bad reviewer (me :stuck_out_tongue_winking_eye:) who did not paid the proper attention. Commit https://github.com/conda-forge/ocgis-feedstock/pull/6/commits/b49159bb8f4635c51d03a1beb11d7747678469b1 should fix that.

Confirmed this fix addresses the package naming issue. Thanks @ocefpaf for all the help!


Sample rename

Purpose
Update to project naming
Does this introduce a breaking change?

[ ] Yes
[x ] No

Pull Request Type
What kind of change does this Pull Request introduce?
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

good


Fix regression in handling of spaces in dependent library names

Highlights from CHANGELOG.md
When switching to recursive library dependency detection, I inadvertently deleted the code that converts "official" library names to the allowed character set for the directory names. This PR restores that, with a contributed test case to validate it.
Fixes #192

When I check out tdd and try running the test in SampleProjects/BusIO the test still fails with the same error. It appears that you haven't merged #212 into master and it isn't being tested yet.


Instantiable Handlebars + Custom Configuration with Expression Name Resolver

Closed via PR #15 (branched from #14)

Updated NuGet with 1.1.0


Remove uses of features deprecated in 5.0

There are a sizeable number of warnings generated in the integration tests that revolve around using deprecated end points or features, like so:
[2016-10-24T15:24:52,486][WARN ][org.elasticsearch.deprecation.rest.action.admin.indices.RestTypesExistsAction] [HEAD /{index}/{type}] is deprecated! Use [HEAD /{index}/_mapping/{type}] instead.

These should be hunted down and rectified. Care needs to be taken to ensure backwards compatibility in these places.

This specific deprecation has been addressed in the connector with 2f4e3a5d8a50aeeb5875c0fdcb267610d61f5350


Dataset problem

the miniImageNet dataset used in your paper:
miniImageNet_category_split_train_phase_train.pickle
miniImageNet_category_split_train_phase_val.pickle
miniImageNet_category_split_train_phase_test.pickle
miniImageNet_category_split_val.pickle
miniImageNet_category_split_test.pickle
I know ：
①miniImageNet_category_split_val.pickle  is the validation set of miniImageNet which has 16 classes and Each class has 600 images
②miniImageNet_category_split_test.pickle a test set of miniImageNet which has 20 classes and Each class has 600 images.
③miniImageNet_category_split_train_phase_train.pickle a train set of miniImageNet which has 64 classes and Each class has 600 images.
but I don't know the detail of miniImageNet_category_split_train_phase_val.pickle and
miniImageNet_category_split_train_phase_test.pickle.??????????????
Is there any overlap between these two data sets and the data inminiImageNet_category_split_train_phase_train.pickle or something else?

Hi. This miniImageNet is downloaded from the project: https://github.com/gidariss/FewShotWithoutForgetting.
The train_phase_train/val/test are used for testing base classes, they are all from the 64 base classes, and the images in each set does not overlap to each other.

I used this data, and I ran your experiment. I found that  train_phase_train has 64*600=38400 images which are from 64  base classes. According to the instructions which says base class has 64 classes and each class has 600 images, I think train_phase_train has all the training data from the 64  base classes.
However, I found that The train_phase_val has 18748 images which are from 64  base classes.
So I've been wondering whether the images in train_phase_train/val/test may be overlap  each other or Additional data is used???????

It seems that miniImageNet_category_split_train_phase_test.pickle is never used?


Move Outliers.tukey to Signal and change result type to Interval

Description
Testing.Outliers.tukey creates an dedicated Outliers type consisting of a lower and upper border.
For convenience it would be beneficial to make use of the Interval type. Additionally the outlier detection may fit better to Signal than Testing.

closed by 34565ae6a739aacd7e09f3090bdcbf3fba717b56


Add regex stuff

Implement the logic for matching code with regex.
Add tests for :

query as regex
filter as regex
propagate when hole is captured by regex
propagate built in rules when query is regex

Stack from ghstack (oldest at bottom):

-> #529

Addressed comments https://github.com/uber/piranha/pull/529/commits/3ac21b03af456692c94d66ec4fc2eab8312afb08 and rebased


Logseq 0.10.5 breaks the theme

Upgrading to Logseq 0.10.5 (changelog) seems to break the theme. Text is now black on the Dracula background color.

I think it is broken by Logseq 0.10.4.

I'm seeing this too and had to switch to the default dark theme.  The menu for the graphs turns transparent.  I'm seeing the issue on 0.10.5 for macOS, iOS, Windows and Linux.  So it seems to be a fundamental change/issue.  I can also confirm it does NOT happen on 0.10.3, so it's a recent change.

Facing the same issue after upgrading to 0.10.5. Had to roll back to 0.9.20 for it to work again.
Theme used : logseq-monokai : https://github.com/tzcl/logseq-monokai

Facing the same issue after upgrading to 0.10.5. Had to roll back to 0.9.20 for it to work again.
Theme used : logseq-monokai : https://github.com/tzcl/logseq-monokai

It appears that theme is having the same issue.

Looks like there's a Logseq issue for this as well.

@k7hpn - Great link to the Logseq Issue... there is a comment there that recommends forcing an "accent" color and that helped fix the Dracula theme for me....
https://github.com/logseq/logseq/issues/10873#issuecomment-1897882037

me too.

Does anyone know if this is being looked at?  I see there was an update to the README.md talking about it but not sure if anyone is actively working on it.  I'm not trying to push for something as I'm guessing this is all volunteer, I just wanted to understand if it is worth trying to look at this over the weekend or if someone with much more knowledge and history with this project is about to fix this with 2 hrs of effort.

I just tried the workaround forcing an "accent" color from https://github.com/logseq/logseq/issues/10873#issuecomment-1897882037 and it seems to mostly fix/work for Logseq 0.10.6, although some of the colors still seem off.

I love the addition of the word yet 😁 Would be awesome if this could be fixed for the latest version ❤

This seems to be working today in Logseq 0.10.8

I confirm that with the theme installed post-upgrade from Logseq 0.10.7 to 0.10.8 the theme was working. However, after I removed and reinstall it, it stopped to working. @k7hpn, if you have a moment, could you please try removing and reinstalling the theme to see if the issue persists on your end?

Ahh, @PixelTux I think I had inadvertently selected an accent color which fixed it. When I deselect the accent color it is still broken. Sorry about that, that's on me.

I merged this fix #24 which is supposed to fix the issue, can you guys confirm that it is fixed?

@SlyBouhafs I tested both on 0.10.7 and 0.10.8, it works for me! But I had to manually copy this repos custom.css. It seems neither the marketplace version nor the the cdn link have been updated?

I merged this fix #24 which is supposed to fix the issue, can you guys confirm that it is fixed?

It works with 0.10.8! Great!

On Logseq 10.8, the theme does not work through the theme's plugin. It works if you register it in custom.css. But the color of the links and dates has changed from pink to cyan. On version 10.3, the theme from the plugin and from custom.css works and the colors are correct there, with pink highlighting of links and other data.

Hi @AristarhSamos , you're seeing cyan instead of pink because it's actually defined in the code.

Not sure if it's intended or a bug, but if you prefer pink, you can go to
Settings > General > Custom Theme and click on Edit custom.css
add this to overwrite the theme colore:
.external-link {
  color: var(--pink);
}

.external-link:hover {
  color: var(--pink-hover);
}

Remember to remove it if you change themes, as it will default back to white since the --pink color is specific to this theme and isn't universally defined.

I've manually copied and pasted the custom.css on version 10.0.8 and it seems to be working fine. The only broken thing that I noticed until now were the drop-down menus. Either the background color or the text color must be changed but I failed to find the corresponding CSS rule for that.

@alexbraga @dev0T this bug originates from the upstream main build when an accent color is not selected and the dark theme is used. I'll look into it when I have some time.

Uninstalling and reinstalling the Dracula theme today in LogSeq v0.10.9 (c67b8b5) still results in black-on-dark-grey text for me if I do not have an accent color selected.


Remove apt-transport-https package declaration

This package is installed in recent versions of apt::default. As a result,
this package is declared twice resulting in CHEF-3694.

@redguide/nodejs seems legit to me

LGTM after a rebase :+1:

Rebased


NERDTree always shows line numbers

Hi,
NERDTree no longer honours the setting for hiding line numbers. I tried adding let nerdtreeshowlinenumbers=0 to the end of the update function to no avail. Is there an easy way to make NERDTree exempt from showing the line numbers please?
Thanks for a neat plugin!

Reappeared with 2.0 update.

Instead of adding exceptions for tools like NERDTree, I'd like numbertoggle to disable itself automatically in buffers without line numbers. This means users would have to set number and relativenumber in their ~/.vimrc files themselves, which I think would be acceptable.
I opened a pull request in #38 to disable numbertoggle automatically when number is not set, which works for me in Goyo. @Leandros since I don't have NERDTree installed, could you switch your install to the if-number branch to check if this fix works with NERDTree as well?

Yes, seems to work. NERDTree isn't showing line-numbers anymore after switching to it.


Opening a Pedersen commitment

Is there a well defined way to open a Pedersen commitment? I was hoping to find a function that I could pass the commitment and blind value to and it would return the commitment value, but I did not find such a function. Do point me in the right direction if you can.
Thanks!

That's not how Pedersen commitments work. You cannot retrieve the value of a Pedersen commitment with only the blinding factor. The only thing you can do is create a function that generates a Pedersen commitment from a given value and blinding factor and then use that function to check if a given Pedersen commitment effectively commits to some value.
That's why it is called a commitment, it is not an encrypt/decrypt mechanism.


docs: Add documentation for SRIOV

Most of the documentation has been moved from 2.x with
some cleanup and references to the 3.0 runtime.
Contributions-by: Archana Shinde<EMAIL_ADDRESS>Signed-off-by: Archana Shinde<EMAIL_ADDRESS>

@mcastelino Yes, I am working on a separate doc "Passing-devices-to-Clear-Containers.md" that describes steps involved in passing block and vfio device groups. Expect a PR today for that. Thats what you meant right? We dont have support for autodetecting vfio capable devices today.

@jodh-intel thanks for taking a look. I have pushed changes for your comments.

@iphutch Can you take a look at this?

Hi @iphutch - could you take a look at this?

@klynnrif could you please take a look?

@klynnrif Ah, I see you used the 'Review Changes' method to group comments here - many thanks! (and for the reviews in general ;-) )

@rcaballeromx @iphutch @klynnrif I have addressed some of your feedback. I have not addressed some of the comments that change the technical accuracy. If there are some changes that you think need to be done lets open a separate PR for it, as the comments on this PR are causing the page to not load for me on multiple occasions and I would like to get some documentation merged in for users interested in SRIOV.

@amshinde - again I see changes that need to be applied that don't change the technical content.
Also, I think pullapprove is confused because the author is Eric but the sign-off is from you. Maybe if @egernst is happy you could change the commit ownership to you and add a Contributions-by for him to appease the pullapprove Gods?

@jodh-intel @amshinde -- fine by me

@klynnrif Can you take a look at this PR and approve if this looks good to you.

@klynnrif Can you take a look?

@klynnrif @rcaballeromx Can you take a look please?

@klynnrif I have addressed your comments, please take a look.

Thanks @amshinde.
@rcaballeromx - are you happy with this now?


CLI: support newline to break up long commands

Feature request: I would like to do this in a shell script:
influx -host $HOST << EOF
CREATE CONTINUOUS QUERY cq_hourly_foo_readings ON $DB_NAME BEGIN
  SELECT
    mean(current) AS mean_current,
    mean(voltage) AS mean_voltage,
    mean(temp) AS mean_temp
  INTO hourly.foo_readings
  FROM realtime.foo_readings GROUP BY time(60m), foo, baz
END

CREATE CONTINUOUS QUERY cq_hourly_bar_readings ON $DB_NAME BEGIN
  SELECT
    mean(current) AS mean_current,
    mean(voltage) AS mean_voltage,
    mean(temp) AS mean_temp
  INTO hourly.bar_readings
  FROM realtime.bar_readings GROUP BY time(60m), foo, bar
END

-- etc multiple other long statements here..
EOF

This of course doesn't work today, the CLI attempts to parse a whole command when it encounters a newline.  Easiest way I'd guess would be to treat a '\' as a continuation indicator to ignore the newline.  Re: #4279

Amazed this isn't implemented.. or am I missing something?

still not implemented...


Fixed react prop errors for cloud_manager

Fixed these console errors in the cloud provider form and changed implementation of hidden text-field.

@miq-bot add_reviewer @agrare
@miq-bot add-label bug

Backported to morphy in commit 53a1cbe43817525197b3d2fae9680b853824e455.
commit 53a1cbe43817525197b3d2fae9680b853824e455
Author: Adam Grare<EMAIL_ADDRESS>Date:   Mon Jul 12 09:43:31 2021 -0400

    Merge pull request #18 from GilbertCherrie/fix_cloud_manager_console_error

    Fixed react prop errors for cloud_manager

    (cherry picked from commit fd28a7da7931b145262e6f40bf0ca67747619855)


Topic 12

Topic 12: Euler Line
The Goal
To make animations of the Euler Line
The definition of Euler Line can be found at
Topic 12
The Euler Line Picture is here:

The Task
Given Triangle ABC, with points A,B,C movable, draw the orthocenter, the centroid, and the circumcenter, and they form the Euler Line. For technical details, see Topic 08.
The trilinear coordinates of the symmedian can be found at Wikipedia.
Some issues discussed during  the meeting on 6/25/2022 on the file GdpTopic12View.vue

Line 3, still "Morley-wrapper"
We suggest to make the following changes

For an obtuse triangle, wish that the orthocenter is always connected to pedals. See below

For the Euler line doesn't match with the points when slope is positive,

Can we replace
x1: -canvas.getWidth()
y1: -eulerLineLinearEquation.m * canvas.getWidth() + eulerLineLinearEquation.b,
to see if that solves the problem?

I have solved the issue 1 to 3, but the solution for issue 4 still don't work.


pull request for e2e test at 01/03/2017 05:15:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


updates for v1.4.0

Found a small misspelling

You can review notebooks like this ☝ by installing GitNotebooks in this repo (it's free)


[BUG] Cached operations not available in score_fn of ScoreSubpopulation

Describe the bug
Example in README using subpopulations uses a function which relies on cached spacy results, but these cached results are stripped from the dataset/batch before the length function is reached.
To Reproduce
Steps and code snippet that reproduce the behavior:

Code snippet (from the README)

from robustnessgym import Dataset, Spacy, ScoreSubpopulation

dataset = Dataset.load_dataset('boolq', split='train')
spacy = Spacy()
# .. on the 'question' column of the dataset
dataset = spacy(batch_or_dataset=dataset,
                columns=['question'])

def length(batch, columns):
    """
    Length using cached Spacy tokenization.
    """
    column_name = columns[0]
    # Take advantage of previously cached Spacy informations
    tokens = Spacy.retrieve(batch, columns, proc_fns=Spacy.tokens)[column_name]
    return [len(tokens_) for tokens_ in tokens]

# Create a subpopulation that buckets examples based on length
length_subpopulation = ScoreSubpopulation(intervals=[(0, 10), (10, 20)],
                                          score_fn=length)

dataset, slices, membership = length_subpopulation(dataset, columns=['question'])

Instructions: Run example above
Errors and traceback:

ValueError: Retrieval failed: couldn't find a key called Spacy in cache.
ValueError                                Traceback (most recent call last)
 in
14                                           score_fn=length)
15
---> 16 dataset, slices, membership = length_subpopulation(dataset, columns=['question'])
17 # dataset is updated with slice information
18 # slices is a list of 2 Slice objects
/opt/notebooks/sandbox/Robustness/robustness-gym/robustnessgym/slicebuilders/slicebuilder.py in call(self, batch_or_dataset, columns, mask, num_proc, batch_size, *args, **kwargs)
116                 batch_size=batch_size,
117                 *args,
--> 118                 **kwargs,
119             )
120
/opt/notebooks/sandbox/Robustness/robustness-gym/robustnessgym/slicebuilders/subpopulations/score.py in prepare_dataset(self, dataset, columns, batch_size, *args, **kwargs)
128             batch_size=batch_size,
129             *args,
--> 130             **kwargs,
131         )
132
/opt/notebooks/sandbox/Robustness/robustness-gym/robustnessgym/slicebuilders/slicebuilder.py in prepare_dataset(self, dataset, columns, batch_size, *args, **kwargs)
263                         columns=columns,
264                         *args,
--> 265                         **kwargs,
266                     )
267                 except NotImplementedError:
/opt/notebooks/sandbox/Robustness/robustness-gym/robustnessgym/slicebuilders/subpopulations/score.py in prepare_batch(self, batch, columns, *args, **kwargs)
140             self.scores.extend(self.score.retrieve(batch=batch, columns=columns))
141         elif isinstance(self.score, Callable):
--> 142             self.scores.extend(self.score(batch=batch, columns=columns))
143         else:
144             raise RuntimeError("score function invalid.")
 in length(batch, columns)
7     column_name = columns[0]
8     # Take advantage of previously cached Spacy informations
----> 9     tokens = Spacy.retrieve(batch, columns, proc_fns=Spacy.tokens)[column_name]
10     return [len(tokens_) for tokens_ in tokens]
11
/opt/notebooks/sandbox/Robustness/robustness-gym/robustnessgym/core/cachedops.py in retrieve(self_or_cls, batch, columns, proc_fns, identifier, reapply, **kwargs)
126                 if not identifier:
127                     raise ValueError(
--> 128                         f"Retrieval failed: couldn't find a key called "
129                         f"{target_ident_key} in cache."
130                     )
ValueError: Retrieval failed: couldn't find a key called Spacy in cache.

Include any relevant screenshots.
Expected behavior
The length function would be able to use the Spacy results already computed on the 'question' column.
System Information

OS: Linux
Versions for RG and relevant dependencies [robustnessgym (v0.0.3), spacy (v2.3.5)]

Additional context
It appears a function called _filter_prerequisite_columns in slicebuilder is filtering out all columns except for
"question" that was explicitly passed in.

Seeing now that the example code in a closed issue mentions the need for prerequisites(Spacy) until v0.0.4

Yeah! Sorry about this issue, we should have a fix for this in the next release.


Analytics client

Issue: #533

[ ] Tick here to confirm that you have run RunCodeFormatter.ps1

Description:
Major performance improvements in sending events. Also got the first start of a Client .NET SDK by moving code for sending events to the EventHub library.
Test:
Run normal tests on Nether Analytics and monitor send performance.

Hold this PR for a while. Ran into some issues with the client using too many Threads/Tasks


Australian Pacific College added

https://www.apc.edu.au/

@s40067091 Pull request merged. Thank you!


Storefront: Limit pagination button display based on state

Don't display "Previous <<" or "Next >>" when there is no such page. For instance on the first or the last page, this should be either gone (preferred) or disabled.

Some additional problematic points that relate to this: the current << and >> buttons only jump to the start and end. We might want additional buttons for that purpose if we want this functionality. The buttons also only display "[current page link]/#" on hover instead of a useful prompt. We might want to remove this hover prompt or adjust the content.


added products screen

Acceptance Criteria for 1

[x]  Mentor is added to pr
[x]  AC copied to pr description
[x]  Link to deployed version is added to pr
[ ]  The ProductsList component correctly receives product data as props.
[ ]  ProductCard component created and renders a single product.
[ ]  Header and Footer elements are present.
[ ]  ProductList displays right on mobile devices

Acceptance Criteria for 1*

[ ]  The Header component has clickable elements that toggle the display between the About and ProductsList components.
[ ]  State management is handled in App.tsx, with state determining which component to show.
[ ]  LocalStorage added to save items in card after page realod

Danil, good job well done. I like your design.
Could you please fix one issue?

David, you've done a huge job and your app looks good, and you need to fix some layout staff:

You need to find out how to remove this scroll bar (number 1)
Your shopping cart should be empty until you add an item to it.

Why you use "overflow-y: auto;"? Do you need it there?

You need to fix this counter issue


DRILL-5940: Add support for Avro messages with schema registry for Kafka

DRILL-5940: Add support for Avro messages with schema registry for Kafka
Description

Refactored KafkaRecordReader to use EVF.
Updated JsonMessageReader to use V2 JSON reader
Added AvroMessageReader for processing messages serialized as Avro records and added schema registry support.

To be able to deserialize Kafka messages of Avro type, schema registry URL should be specified in the Kafka storage plugin kafkaConsumerProps (and other schema registry options if required):
"schema.registry.url": "http://localhost:8081"

After that, store.kafka.record.reader session option should be set to org.apache.drill.exec.store.kafka.decoders.AvroMessageReader.
Documentation
NA
Testing
Added UT that reads Avro messages.

@vvysotskyi  Thanks for this PR!   Does this PR address the single topic issue with the Kafka Plugin?

@vvysotskyi  I approved this PR.  We should probably update the docs as well. @dzamo Can I tag you for that effort?

A user asked Support confluent questions yesterday, update the docs for better. thanks.


Get style at index

Is this possible with ansi package?
http://stackoverflow.com/q/27165569/1420197

https://github.com/IonicaBizau/node-ansi-parser


Add Integration with Terraform Configuration Files

Vendor overview/documentation is here
Initial task list:

[x]  Create new plugin in ManageIQ (https://github.com/ManageIQ/manageiq-providers-ibm_terraform)
[x] Define configuration manager with connection to CP4MCM Terraform (CAM)
Discover and persist Terraform content in MIQ inventory

[ ] templates (configuration profiles)
[ ] managed virtual machines (configured systems)

Configuration Manager UI

[ ] view list of templates, details
[ ] add configured systems for all VMs under Terraform management

Provider repository - https://github.com/ManageIQ/manageiq-providers-ibm_terraform

Initial task list:

[ ]  Create new plugin in ManageIQ
[ ] Define configuration manager with connection to CP4MCM Terraform (CAM)
Discover and persist Terraform content in MIQ inventory

[ ] templates (configuration profiles)
[ ] managed virtual machines (configured systems)

Configuration Manager UI

[ ] view list of templates, details
[ ] add configured systems for all VMs under Terraform management

@chadh1313 what is the PR for Define configuration manager with connection to CP4MCM Terraform (CAM)?

@chessbyte I believe it is this one that finished it off (some was already in place for the POC): https://github.com/ManageIQ/manageiq-providers-ibm_terraform/pull/19

Closing in favor of ManageIQ/manageiq-providers-ibm_terraform#28


Reactivity

I'm working on a Vue project and I tried to use vue-query. However, I ended up encountering this behavior:
When I use
const { isLoading, isError, data, error } = useGetResidencialByAnoMes(anomes, 10, filters)

it works normally, i can access the properties of data both in setup() and in template
But when I use:
const test = useGetResidencialByAnoMes(anomes, 10, filters)

And try access, for example, test.data.foo in setup() I get undefined and in template is not rendering anything. I tested to run a v-for loop in test.data and the reactivity properties are displayed (__object, __v_isRef).
I fixed that wrapping with reactive, like this:
const test = reactive(useGetResidencialByAnoMes(anomes, 10, filters))

But shouldn't it work without it?

Hi,
Composables exposed by vue-query library return all attributes as refs.
This means that isLoading, data etc. are all refs.
To get the actual value of data you need to access it's like data.value.
Therefore if you do not spread the returned object you should access your data like test.data.value.
Reasoning for returning refs instead of reactive object is that reactivity is lost whenever someone spreads such object. Therefore this would not work
const { isLoading, isError, data, error } = useGetResidencialByAnoMes(anomes, 10, filters)
Your fix works, because wrapping refs object with reactive uwraps all refs that are inside and it's a valid solution.
Another solution will be compiler syntax sugar which is currently still in RFC phase https://github.com/vuejs/rfcs/discussions/369

Reactivity some time feels a little bit confusing to me, I'll try study to understand these behaviors.
Thanks for the answer!


Material Properties Docs

Describe the bug
Docs not clear with relation to how properties are reported when material properties are used - even if only one material is used!

rewrite docs!

Resolved with #127.


Fix uninitialized file position in ms3_readtracelist_selection()

When calling ms3_tracelist() (and thereafter
ms3_tracelist_selection()), previously the position within the
file from which records were first read was undefined, because
fpos was used uninitialized.  This in general did not cause
problems, but repeated calls to the library could leave fpos
taking an arbitrary value, including a negative one.  In that
case, when the negative, non-0 value of fpos was read in
ms3_readmsr_selection() the first time, it was taken to be a
'legacy' request to start reading records at some non-0 offset
within the file.
Fix this by initializing fpos in ms3_tracelist_selection()
to 0.
This potential error and its fix was spotted by @bakerb845.
Closes #70.

Thanks @anowacki !


Change event not working

I'm trying to use Bootstrap-toggle within an angular application but the change event is not working.
<input type="checkbox" data-toggle="toggle" data-on="ON" data-off="OFF" data-onstyle="success" (change)="doSomething()">
If I do not include bootstrap-toggle libs (min.js + min.css) the change event is triggered successfully and the method invoked. As soon as I load the bootstrap-toggle to transform my input checkbox into a switch, the event is not triggered anymore event though if the switch is displayed/behaves correctly.
Any ideas?
bootstrap 4
jquery 3.2.1
angular 4

Yes, happening to me too. The change event is being blocked here too. Cannot make it work as docs read.

I think the solution to this problem is, to put the
<script src="/js/bootstrap-toggle.min.js"></script>
after the reference of the general jquery and your script

Did anyone fix this? I'm getting the same thing.  The toggle displays and works correctly, however when inspecting the document, the checkbox is not checked when it should be.
I'm loading the bootstrap-toggle-min.js at the end of the document.

I'm having this issue as well, I think the problem may be that the event listener for the div.toggle container's click isn't bubbling, and it's blocking access to the native DOM listeners.
https://github.com/minhur/bootstrap-toggle/blob/master/js/bootstrap-toggle.js#L177

@minhur any chance to get this fixed?

I think this is also why I cannot get it to work with Stimulus.js. No event shows up.

Same problem here.  I used onchange="function()" in the html code instead, and it works.


Renamed "Deserializer" to "Hydrator"

This will fix #19

Thank you for the review.
I've updated variable names and functions.

Can you rebase?

Rebased


[Held requests] status on distance requests

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Coming from the discussion here: https://github.com/Expensify/App/pull/37121#issuecomment-1965819951
Determine proper display of Hold status on distance requests given that it does not currently support (Approved/Pending/Canceled) state.

Version Number:
Reproducible in staging?:
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
Action Performed:
Break down in numbered steps
Expected Result:
Describe what you think should've happened
Actual Result:
Describe what actually happened
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub

Hey @robertjchen just want to check where we landed with this one from the original discussion in https://github.com/Expensify/App/pull/37121#issuecomment-1965819951 so we can figure out the Actual Result vs Intended Result

@robertjchen for now, • xxx is showing only after Cash (manual request)
i.e. not showing for distance requests. Is this expected (or at least out of scope)?

@situchan great catch! Let's pull this discussion to Slack to determine what's the best course of action. I think this change can go out in the interim however 👍

Was this ever pulled into Slack (I can't find anything)? Or was there general agreement that distance expenses should show the status (Hold, Approved, Pending, Canceled etc) on  distance expenses?

thanks @situchan ! Feel free to link the thread here after you've created it, looking forward to see what others think

Hey @situchan have you started this convo? I'm not seeing it in Slack -- could you link it here?

@situchan has this convo been started? Is there anything we can do to move this issue along in the meantime?

started discussion - https://expensify.slack.com/archives/C01GTK53T8Q/p1710439388757009

We got answer. It should be consistent. Let's fix this to show status on all type of requests.

Not overdue

Hey @situchan  we have a couple proposals here -- can you review when you get a chance?

No one considered all cases in their proposed solutions.
I don't understand why other types of requests (i.e. scan) cannot have status labels.
i.e. request while scanning in progress can be held.
https://github.com/Expensify/App/assets/108292595/8323ed20-adb5-4075-8813-fa41d3327cb2
I'm looking for proposal which filled all of these:

Approved
Pending
Canceled
Hold
Review required
Missing amount
Missing merchant

Manual

Distance

Scanning

Scan

Split

Card

@ShridharGoel are you sure that all labels are applicable to all types of requests? Did you test? 😄

I can still update update my proposal right? @situchan

I think @ShridharGoel is the first dib unless you have any other meaningful considerations.

Sounds good. Thanks

@ShridharGoel I also expect ✔ in all fields. Just wanted to make sure that the solution doesn't cause any regressions.
As promised, let's wait for @GandalfGwaihir a few hrs. Otherwise, we can go with your proposal.

@rayane-djouah it was discussed already. If you have any doubt, please ask in slack thread.

@ShridharGoel's proposal looks good to me.
🎀 👀 🎀 C+ reviewed

Thanks for the discussion here and on Slack, and for the updated detailed proposal. Let's move forward with this 👍

PR is undergoing QA now

Hit staging 4 days ago. Should start the regression clock on the next deploy.

Summarizing payment on this issue:

Contributor: @ShridharGoel please accept the Upwork offer so we can issue payment
Contributor+: @situchan $500 via Upwork PAID

Upwork job is here: https://www.upwork.com/jobs/~01e3e32c45e45f949f

All paid! See summary here: https://github.com/Expensify/App/issues/37476#issuecomment-2045899200


Fix a couple bugs manifesting in the test-suite and improve usability

Currently HEAD is a bit broken, mostly due to ValidatorCleanRestart and sometimes ValidatorRecurringEnrollment.
This does not fully fix the issues, but removes a few spurious error message, makes things easier to debug, and removes a race condition. There's still an issue happening in the nomination process which I am currently debugging.

Merging because at least it fails timely now.


Updated InputAction definition to include an AxisConstraint

Overview
Updated InputAction definition to include an AxisConstraint
#Depends on PR #2194 (using added ready)
Changes

InputAction now includes an AxisConstraint and optional parameter in constructor

Iol, I did change it, so I wonder why the compiler didn't notice. 2 ticks

Already complete


Watson Lite: Fix Headers Not Being Parsed Correctly

Info
This fix to the watson lite header parsing code allows for header values to contain colons.
Previously, headers values would be ignored if they had colons in them.
This will help with parsing certain headers and will make it more in line with the HTTP/1.1 spec.
Motivation
Browsers usually sent a Host header to the server, which sometimes includes a colon to delimit the port number:
Host: <IP_ADDRESS>:8080. Without this fix, the Host header is ignored. With this fix, the header is parsed correctly.

@dotnet-policy-service agree company="SamsidParty"


Partition column gets dropped from dataframe when reading parquet file list using pyarrow engine

What happened:
When using dask to read a list of parquet files in a dataset with hive partitioning using the engine="pyarrow", the partitioning column(s) gets dropped. In this case the parquet files were written using pyspark.

I have not been able to confirm the same behavior using engine="pyarrow-dataset" because it results in another error as posted here post on stackoverflow,
which was an already-known bug here original bug.
engine="pyarrow-dataset" is ultimately what I hope to use in case this issue spills over to both engines.
What you expected to happen:
The partitioning column should be a column returned in the dask dataframe.
Minimal Complete Verifiable Example:

import os
from glob import glob
import pyarrow.parquet as pq
import pyarrow.dataset as ds
import dask.dataframe as dd
from pyspark.sql import SparkSession

home_dir = "/home/xxx/" # replace with your base directory

spark = SparkSession.builder.master("local[*]").getOrCreate()
df = spark.createDataFrame([("A", 1, 4), ("B", 2, 5), ("C", 3, 6)],["id", "col1", "col2"])
df.write.format("parquet").mode("overwrite").partitionBy("id").save(home_dir + "parquet_dataset")
paths = [y for x in os.walk(home_dir + "parquet_dataset") for y in glob(os.path.join(x[0],'*.parquet'))]
dd.read_parquet(path=paths, engine="pyarrow").compute()

But the following pyarrow methods work fine:
pq.read_table(paths).to_pandas()
ds.dataset(source=paths, partitioning="hive", format="parquet").to_table().to_pandas()

Anything else we need to know?:
Let me know if I missed providing anything else useful. Thank you for taking a look.
Environment:

Dask version: 2021.4.0
Python version: 3.8
Operating System: Ubuntu
Install method (conda, pip, source): pip
pyarrow version: 3.0.0
pandas version: 1.2.4

Thanks for raising @senglehardt - Can you check that #7541 doesn't address this issue?  If not, I'll be happy to try to track down a fix.  However, I have no experience with pyspark, so it may be a bit faster if I have a pandas-based reproducer :)

Thank you @rjzamora.
Regarding issue #7512, according to this SO post, spark does not write the partitionBy columns to the parquet file. I came across this while searching for a way to not have Spark write the partitions to columns in the parquet file in order to circumvent the dask error using the pyarrow-dataset engine. Turns out, spark does not do this assuming we take the first answer at face value (although I have not checked with parquet tools).
#7541 considers when the partition is also written to the column so I am not sure if it addresses this. I think #this (#7600) has to do with passing a list of files. Only the version providing a list of paths drops the column using the ArrowLegacyEngine:
dd.read_parquet(path=home_dir + "parquet_dataset", engine="pyarrow").compute() # Works
dd.read_parquet(path=paths, engine="pyarrow").compute() # Drops the column
Now that I think about it, I am not sure how pyarrow.dataset is able to read the list of file paths unless it is extracting the partition from the filename or if this is included in the metadata of the parquet file.
I'll follow up in a little while to see if I can replicate this using a pandas-based reproducer. Thanks again!

Interestingly, writing the parquet using pandas fixed the engine=pyarrow-dataset issue but not the ArrowLegacyEngine issue for reading in a list of files:
import pandas as pd
import dask.dataframe as dd
from glob import glob

data = {"id": ["A", "B", "C"],
        "col1": [1, 2, 3],
        "col2": [4, 5, 6]}
pdf = pd.DataFrame(data)
pdf.to_parquet(path=home_dir+"parquet_dataset2", engine="pyarrow", compression="snappy", partition_cols="id")

paths = [y for x in os.walk(home_dir + "parquet_dataset2") for y in glob(os.path.join(x[0], '*.parquet'))]

dd.read_parquet(path=home_dir + "parquet_dataset2", engine="pyarrow").compute() # Still works
dd.read_parquet(path=paths, engine="pyarrow").compute() # Still does not work

dd.read_parquet(path=home_dir + "parquet_dataset2", engine="pyarrow-dataset").compute() # NOW WORKS!
dd.read_parquet(path=paths, engine="pyarrow-dataset").compute() # NOW WORKS!

So once #7512 is merged into new dask release, I should be golden (as I need to read spark written partitioned parquet file list).

Now that I think about it, I am not sure how pyarrow.dataset is able to read the list of file paths unless it is extracting the partition from the filename or if this is included in the metadata of the parquet file.

Yes - pyarrow.dataset is extracting partitioning information from the path when it processes either a list of files or when it needs to walk through the files within a root directory.  By default, pyarrow.dataset assumes that you may have "hive"-like partitioning.
The legacy API does NOT make any assumptions about dataset partitioning if you pass in a list of files.  It will only extract partition information from the paths if you pass in a root directory (so that pyarrow is forced to walk through the files itself).  For now, it seems that the only way to read a partitioned dataset "correctly" with the legacy API is to pass in the root directory, OR pass in a list of files that includes a "_metadata" file (although I still need to confirm the latter case myself).

@senglehardt just checking in here, are you still running into issues or does upgrading to a newer version of dask resolve the issue?

Closing due to lack of activity. Feel free to re-open as needed


Eagerly evaluate const functions at parse time

Ruby has literals. Simple, like Integer, Float, String, Range and aggregate like Hash, Array.
Simple literals like:
def foo
  "abc"
end

can be evaluated at parse time and stored as shared data in the interpreter.
Some functions on literals can also be computed at parse time. Such functions are called const and pre-computing values is called const evaluation.
a = 'a' * 1024
b = [nil] * 100
c = { foo: ["bar".delete_prefix('b')] }
d = 1 + 2 * 3

The preceding are all const if String#*, Array#*, String#delete_prefix, Integer#+ and Integer#* are not redefined. I consider this unlikely, so it is worthwhile to precompute these literals at parse time. Then as this code is executed in the bytecode loop, validate that the "const functions" have not been tainted since ??? (interpreter init most likely) and either return a Cow to the eagerly parsed literal or shell out to a funcall via the VM.
A further optimization that may fall out of this work is to inline functions that return const literals.

We'd probably require custom opcodes on top of MRI to accomplish this.
@jamatthews does this seem feasible?

Supporting something like this will also influence the heap design. Rather than pass around RBasics I was considering passing around usize object ID, resolving objects through the heap via the ID, and doing GC coloring of object IDs.
It'd be useful to mark object ids as Cows of some other interpreter lifetime (template) object id.

Oh cool! I'm working on constant evaluation at the moment. There's a lot to consider here. Have you read through https://github.com/ruby/ruby/pull/1419 ?

Thanks for the link to that PR 🙏


Add slim images

Add "slim" images for Ubuntu 16.04 and 18.04.  Slim images are images designed just for running an already built Swift program. Consequently, they do not contain the Swift compiler.
The normal and slim images can be combined via a multi-stage Dockerfile to produce a lighter-weight image ready for deployment. For example:
FROM swift:latest as builder
WORKDIR /root
COPY . .
RUN swift build

FROM swift:slim
WORKDIR /root
COPY --from=builder /root .
CMD [".build/x86_64-unknown-linux/debug/docker-test"]

Prior discussion: https://forums.swift.org/t/idea-runtime-slim-images/22695

@ianpartridge this is fantastic! What are the slimmed down image sizes now?

@ianpartridge this is great! What are the slimmed down image sizes now?

@ianpartridge this is really great! Very excited to give this a try 👌

16.04 slim: 249MB
18.04 slim: 226MB

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

Can you add information about the slim docker image in the readme?

@swift-ci test

@swift-ci test

@swift-ci test

@ianpartridge  isn't it better to have Dockerfile in the slim directory rather than having Dockerfile-slim, i.e
swift-docker/5.0/ubuntu/bionic/Dockerfile
swift-docker/5.0/ubuntu/bionic-slim/Dockerfile
swift-docker/5.0/alpine/Dockerfile
or in current way:
swift-docker/5.0/ubuntu/18.04/Dockerfile
swift-docker/5.0/ubuntu/18.04/slim/Dockerfile

Yes it looks like that's how Python and Ruby do it.  I'll push an update to reorganise things.

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test


Update Machinery Backgrounds to use new Lch

in docs extras backgrounds btw

Think it's just my monitor (I use a MacBook) being funky, so I'd close this.


Update scalafmt-core to 3.5.2

Updates org.scalameta:scalafmt-core from 3.5.1 to 3.5.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (3.5.1).
You might want to review and update them manually.
docs/src/adr/006rfc-unit-testing.md
project/Dependencies.scala

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequest = { frequency = "@monthly" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Codecov Report

Merging #1650 (8b24f8f) into unstable (c5137c9) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           unstable    #1650   +/-   ##
=========================================
  Coverage     74.96%   74.96%
=========================================
  Files           358      358
  Lines         11612    11612
  Branches        541      541
=========================================
  Hits           8705     8705
  Misses         2907     2907

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 c5137c9...8b24f8f. Read the comment docs.


FRW-1928 Fixed response code for OAuth token creation endpoint.

Developer @yaroslav-spryker

Ticket: https://spryker.atlassian.net/browse/FRW-1928

Release Group: https://release.spryker.com/release-groups/view/4818

merge: squash

Release Table

Module
Release Type
Constraint Updates

AuthorizationPickingAppBackendApi
minor (currently 0.x)

Module AuthorizationPickingAppBackendApi
Change log
Breaking Changes

Adjusted AuthorizeResourceController::postAction() to return a 200 OK response code when access token is created.

@yaroslav-spryker can you confirm that the on the QA http response with 200 OK is always returned?
From the code, it does not look so

Yes, this was verified during QA. It returns 200 if no error code was set before


Re-hydration from previous state

Right now, I'm accessing the "private" members directly, and assigning them manually. would be good to have an static method, like fromJSON(theOneReturnedFromtoJSON) to initialize the classes using a previous state.

This sounds reasonable. I really dislike the method name toJson that all the metrics impl, because some of them do NOT return JSON, should really be getValue() or getPayload().
Since this lib is inspired from Dropwizard see: https://github.com/dropwizard/metrics/blob/4.1-development/metrics-core/src/main/java/com/codahale/metrics/Gauge.java#L25
So maybe the static initialize could be fromValue.

recreating the internal representation is harder than I expected, there are too many moving parts (new class instances, and their internals as well)


Groups in workspace with paths relative to container: not being handled

We're seeing a project not being able to be found by the workspace parser. It looks to me like it's related to handling of the location property of Groups. Workspace snippet:
<?xml version="1.0" encoding="UTF-8"?>
<Workspace
   version = "1.0">
...etc...
   <Group
      location = "container:Modules"
      name = "SomethingElse">
      <FileRef
         location = "group:MyProject/MyProject.xcodeproj">
      </FileRef>
      <Group
         location = "container:Modules"
         name = "Modules">
         <FileRef
            location = "group:FeatureA/FeatureA.xcodeproj">
         </FileRef>
         <FileRef
            location = "group:FeatureB/FeatureB.xcodeproj">
         </FileRef>
...etc...

The first problematic reference that gen-ir can't find is MyProject.xcodeproj. Unclear if it would find FeatureA and FeatureB projects if it got that far. On disk, the project layout is:
MyWorkspace.xcworkspace/
...
Modules/MyProject/MyProject.xcodeproj/
Modules/FeatureA/FeatureA.xcodeproj/
Modules/FeatureB/FeatureB.xcodeproj/
...

If I were to guess, the suspicious part seems to be that the location attribute of the group has a part after the colon: container:Modules, which apparently sets the root path of the group to Modules/. These groups are the only groups I see it in; everything else is just container:. My reading of XcodeWorkspace.swift implies that only prefixes of group: are handled, for both Groups and FileRefs, not container:.
I'm not sure how to set this up in the most recent Xcode UI (this is a really old workspace), so you have to edit the workspace file directly. Both Xcode and the xcodeproj Ruby gem handle group locations this, so it seems like a legitimate thing. I don't have a great "sanitized" minimal reproduction, sorry. Let me know if it would be useful.

First off - thank you for such a detailed report! It really helps narrow down the issue and the amount of prep work needed. I am also unable to replicate this in the Xcode UI, but editing the workspace file manually did indeed get me there - is the Xcode Project generated by a third-party tool out of interest?
Regardless, you are correct in your suspicions. I'll improve the handling in this situation to cover this case.

Hmm, it's getting hung up somewhere else now with container: tags. Snippet:
<Workspace
   version = "1.0">
   <Group
      location = "container:"
      name = "Infrastructure">
      <Group
         location = "container:"
         name = "Widgets">
         <FileRef
            location = "group:Infrastructure/Foo/Foo.xcodeproj">
         </FileRef>
      </Group>
...

It's looking for the project at Infrastructure/Widgets/Infrastructure/Foo/Foo.xcodeproj, instead of just Infrastructure/Foo/Foo.xcodeproj.

Sorry for the late reply - end of conference season.
I've rewritten workspace project discovery to make it a little more flexible that should resolve your recent issue. It does require additional testing (including writing a test) and some research to see if I can dig out the format a little better from Xcode.

Apologies for the delay - right in the middle of my conference season. From my testing, looks like the updates in #50 do correctly process our workspaces! I will wait warmly for the release.

This was released today as an alpha build of the 0.5.0 release - you can install it with brew install<EMAIL_ADDRESS>


Expectation of operator specified as dictionary fails

Setup:
import numpy as np
import mthree
from qiskit import Aer, QuantumCircuit

backend = backend = Aer.get_backend("statevector_simulator")
mit = mthree.M3Mitigation(backend)
mit.cals_from_system(range(2))

circuit = QuantumCircuit(2)
circuit.rx(np.pi / 4, 0)
circuit.rx(np.pi / 2, 1)
circuit.measure_all()

counts = backend.run(circuit, shots=1000).result().get_counts()
quasis = mit.apply_correction(counts, range(2))

Passing strings works:
print(quasis.expval('00'))
print(quasis.expval('ZI'))

0.419
0.7020000000000001

Passing dictionary fails for second example:
print(quasis.expval({'00': 1}))
print(quasis.expval({'ZI': 1}))

0.419
0.0

Passing float fails:
print(quasis.expval({'00': 0.5}))

0.0

mthree version: 13b941f5d0517c63276970e53e08ce5a3941456f
Python version: 3.8.12
Operating system: Arch Linux

Oops, posted too early, I will edit opening post.

Dict format is only for specifying diagonal operators in terms of their bitstrings and probabilities.  It does not work with operators.
The other behavior is odd will have to see what is causing it.

Dict format is only for specifying diagonal operators in terms of their bitstrings and probabilities. It does not work with operators.

Ah, I see. This is not documented anywhere as far as I can tell, but it should be. The example at https://qiskit.org/documentation/partners/mthree/expvals.html is ambiguous as to whether the keys of the dictionary must be restricted to bitstrings, or whether Z and I are allowed. I think that in the context, one might easily assume the latter, like I did.

I guess I was not expecting to have an user use a dict of operator strings.  But I did add clarification into the docs for this.


Some providers have been discarded due to unexpected errors: opensubtitles

Hi, newbie here..
I'm on windows 10
Subliminal finds the movie.. but get the error "Some providers have been discarded due to unexpected errors: opensubtitles"
Is there something wrong with my command?
subliminal --opensubtitles MYNAME MYPASS download -p opensubtitles -l en "E:\MyMovieFile.mkv"
If i turn on debug i get:
subliminal --debug --opensubtitles MYUSERNAME MYPASSWORD download -p opensubtitles -l en
...
...
StackHD.mkvINFO:subliminal.core:Listing subtitles with provider 'opensubtitles' and languages {<Language [en]>}
INFO:subliminal.core:Initializing provider opensubtitles
INFO:subliminal.providers.opensubtitles:Logging in
ERROR:subliminal.utils:Unexpected error. Provider opensubtitles
Traceback (most recent call last):
File "C:\Python311\Lib\site-packages\subliminal\core.py", line 115, in list_subtitles_provider
return self[provider].list_subtitles(video, provider_languages)
~~~~^^^^^^^^^^
File "C:\Python311\Lib\site-packages\subliminal\core.py", line 67, in getitem
provider.initialize()
File "C:\Python311\Lib\site-packages\subliminal\providers\opensubtitles.py", line 129, in initialize
response = checked(self.server.LogIn(self.username, self.password, 'eng', self.user_agent))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python311\Lib\site-packages\subliminal\providers\opensubtitles.py", line 278, in checked
raise Unauthorized
subliminal.providers.opensubtitles.Unauthorized
INFO:subliminal.core:Discarding provider opensubtitles
Downloading subtitles  [####################################]  100%
Some providers have been discarded due to unexpected errors: opensubtitles
DEBUG:subliminal.core:Terminating initialized providers
Downloaded 0 subtitle

Your password might have reserved chars that need to be properly escaped. Try enclosing your password in double-quotes and see if that works.

Thanks for reply, my password quite simple and short, no special characters

We agree that the user / pass is simply the same as when you log into the opensubtitles as a user..  should i put in my API key somewhere?

Wow... i changed i password, and then it worked! -  But really my password was just numbers, and i just registered the account a couple of days ago!


Remove print of config dictionary from --quickstart

Printing the config dictionary could confuse users that then see it as toml. Either print the toml correctly or just remove the print.

Done with #585


Proposal: Firewalld support

These patches add a possibility to communicate with Firewalld's [1] D-Bus interface so Docker doesn't use iptables tool directly when Firewalld is running and it also adds a reaction to Firewalld's restart/reload.
Adresses #9395
[1] http://www.firewalld.org/

I updated the 'React to reload' commit due to merged #7003.
I also added some tests. But I'm not sure how to automatically test that it works with firewalld when in the container where the tests are run is no running firewalld.

@jfrazelle I see no harm too.

We would really like to get this patch in.  Currently we have to recommend disabling firewalld when running docker.  Trust me, I am not a big fan of tools requiring disablement of security features in order to run applications.

@jfrazelle How does one discover about whey the Jenkens build is failing.

I triggered a rebuild, but you click the link to see the logs ;)

Thanks @jfrazelle, I am new to jenkens.
@jpopelka Any idea why this is blowing up https://jenkins.dockerproject.com/job/Docker-PRs/63/console

Oh ya you have to click a bit in to get the real logs
https://jenkins.dockerproject.com/job/Docker-PRs/label=ubuntu-prs/63/console
On Mon, Feb 2, 2015 at 11:43 AM, rhatdan<EMAIL_ADDRESS>wrote:

Thanks @jfrazelle https://github.com/jfrazelle, I am new to jenkens.
@jpopelka https://github.com/jpopelka Any idea why this is blowing up
https://jenkins.dockerproject.com/job/Docker-PRs/63/console
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/9397#issuecomment-72523292.

No idea, I can't see anything related to my commits.
I'm also wondering why the Details of the 'drone' build say Failure now while everything was OK when I was checking it yesterday.

As having installed docker on a fedora machine myself I understand why this is necessary, but because I know you all are a lot more familiar with firewalld than myself. I fixed this on the fedora machine by changing the docker init script to load firewalld and docker in the correct order (of course i dont remeber what this is now, but one has to come before the other)
Is that not a viable fix? I am more interested in why that couldnt be a fix as a replacement to a new firewalld package.
I'm not saying we don't need this (i've felt the pain) I just want to understand better how this is a better fix than changing the init script.

Please have a look at https://github.com/docker/docker/issues/9236 and also https://github.com/docker/docker/issues/10218.
There are issues with several firewall systems out there, that are a result of firewall changes, like for example reload or restart. These can be solved by using firewalld and the patch for docker.
Benefit of using firewalld:

Most changes to the firewall are done in place without the need to restart or reload the firewall.
firewalld is sending out signals over D-Bus to make other services aware of changes also reloads or restarts. With the proposed patch, the docker rules can added again as a reaction to the signal.

The change in the boot order of docker and firewalld without adding the patch to docker is not solving the firewall reload or restart issue. The docker rules will be gone.
With firewalld it would also be possible to add the docker rules permanently. Then the rules would be added with a reload automatically again. But the removal  of old docker rules would be needed.

Ok I'm going to test this out on a fedora machine.
On Wed, Feb 4, 2015 at 12:50 AM, Thomas Woerner<EMAIL_ADDRESS>wrote:

Please have a look at #9236 https://github.com/docker/docker/issues/9236
and also #10218 https://github.com/docker/docker/issues/10218.
There are issues with several firewall systems out there, that are a
result of firewall changes, like for example reload or restart. These can
be solved by using firewalld and the patch for docker.
Benefit of using firewalld:

Most changes to the firewall are done in place without the need to
restart or reload the firewall.
firewalld is sending out signals over D-Bus to make other services
aware of changes also reloads or restarts. With the proposed patch, the
docker rules can added again as a reaction to the signal.

The change in the boot order of docker and firewalld without adding the
patch to docker is not solving the firewall reload or restart issue. The
docker rules will be gone.
With firewalld it would also be possible to add the docker rules
permanently. Then the rules would be added with a reload automatically
again. But the removal of old docker rules would be needed.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/9397#issuecomment-72811103.

@jfrazelle  If you're new to Fedora here are few tricks that might come in handy:
$ systemctl stop firewalld
Stops firewalld service.
$ systemctl start firewalld
Starts firewalld service.
$ systemctl restart firewalld
The above start & stop
$ systemctl reload firewalld
Reload firewalld configuration.
The reload/restart basically flushes all ip[6]tables rules/chains and creates new ones from permanent firewalld configuration (/var/lib/firewalld/). With my patch the docker notices it and recreates its own ip[6]tables rules/chains.
You can put 'FIREWALLD_ARGS=--debug' into /etc/sysconfig/firewalld to see debugging messages in /var/log/firewalld
With this you'll see that docker uses firewalld as a middle-man for setting up ip[6]tables rules.

Do you have an update on this?

@crosbymichael @tianon @jfrazelle Can you guys look at this, although @t-woerner @jpopelka It looks like Jenkins is failing.

I am +1 on this being added

@crosbymichael @tianon @jfrazelle Any update on this?  Would be nice to get this into docker-1.6

+1
ping @tiborvass or @crosbymichael to move to code-review

Pushing to code review

@jpopelka Unit-tests failing to compile.

@LK4D4 Should be OK now.

I'll try right now.

@jpopelka Can we have log on INFO level about using Firewalld?

@jpopelka Working like a charm, will be my default now. I'll LGTM after adding log message

FYI: I'll be offline till 13th. Once I'm back, I'll add the logging on INFO and other problems that arise till then.

LGTM
@jfrazelle @tiborvass Can we merge this and I'll add logging myself?

Would certainly be nice to finally close this one.

shoot sorry needs rebase, then feel free to ping me for the merge

@jfrazelle rebased, added logging on INFO about using firewalld, tested

Small nit but LGTM! :+1: Pushing to 3-needs-docs-review, but please @jfrazelle feel free to give it more test if you have a chance!

@icecrime there don't seem to be any docs changes in this PR.

Indeed, there is no docs changes yet, but shouldn't the advanced networking articles be updated? I think it would make sense (although it would also make sense to do it ourselves considering that PR already waited for so long).
Let me know @SvenDowideit @moxiegirl!

@icecrime @SvenDowideit  Currently, the only mention of FirewallD is in the CentOS installation.
@jpopelka  Maybe at least eye ball the CentOs stuff confirm it still applies (I realize CentOS is not RedHat)  http://docs.docker.com/installation/centos/#firewalld -- Do we need a mention of this in the Redhat or Fedora installation material?
Also, I see the code is throwing errors -- the errors don't offer any suggestions for recovery. Is there anything the user can do when these errors are thrown to toubleshoot or recover from them? If yes, we should document that possibly in Networking or the appropriate installation sections.

Well firewalld is default on RHEL7, Centos7 and supported Fedoras.

@rhatdan That was my impression. So:

Is this FirewallD section correct in light of this change? Or does it need updating or removal?
If the section remains as is or is updated, we should include it with the RedHat and Fedora install material

I'd like to include recovery information if recovery is possible from the error conditions.

@moxiegirl I would say with this patch merged these comments are no longer necessary.  This patch is all about making docker and firewalld play well together.

KK. Since this needs a rebase -- why not add in the doc change of removing the FirewallD section from docs/sources/installation/centos.md before the squash and rebase.
With that addition LGTM

@moxiegirl Yes, the code can throw some errors:

when there's no D-Bus:
in that case ip[6]tables are used directly, so nothing to worry about
when the Firewalld's Passthrough method fails:
most likely due to failing the underlying ip[6]tables call (Passthrough only passes the arguments to ip[6]tables).
Hard to say what can be wrong and how to easily investigate that.

removed needs-doc-review as the PR hasn't reached the stage where the docs are ready to review.

@SvenDowideit @icecrime added the status/3-needs-docs-review three days ago.  I think it is ready.  Why did it need another code review?

Code is reviewed already: I believe it's part of the 3-needs-docs-review to bring docs to an acceptable state (either through requesting updates to the PR or carrying)!

@icecrime not really - https://github.com/docker/docker/blob/master/MAINTAINERS#L169
when this process was being nutted out, it was indicated to me that the code and design reviews could not really be completed unless there was already documentation describing the intended change - 3-needs-docs-review comes after the facts (ie the docs) are technically clear, and are ready for the writers to polish into the larger documentation picture.

mmm, @tiborvass its your document at this point - can you please clarify? (imo, this PR can't be in docs-review just like a PR that contains no code can't be in code-review state.

@icecrime @SvenDowideit Problem comes from the dichotomy between code reviewers and doc reviewers. At some point, somebody has to read both the code and the documentation to make sure they are in sync. What the MAINTAINERS file (implicitly?) says, is that if there is an existing documentation in the PR, design reviewers should make sure it makes sense and code reviewers should make sure it matches what the code does. Then docs reviewers have to make sure it's in sync with other places throughout the docs. So what are we calling needs-docs-review ? Idk. I agree it's not clear what to do in cases where there is no documentation and it's needed. Either way let's not pollute this PR any longer with meta-issues, let's take it on IRC or ML.
@jpopelka would you mind adding documentation to the PR? If not, let us know and we'll carry the PR.

After talking with @moxiegirl let's just merge this now and she will add further modifications to other parts of the docs.
This PR doesn't need docs because as Dan said it just makes Docker work with firewalld: https://github.com/docker/docker/pull/9397#issuecomment-94944262

Thanks @jpopelka !

I still have issues with this. When I create a custom bridge network, the iptable DROP rules docker created for those networks within the DOCKER-ISOLATION chain are gone when I restart firewalld (or firewall-cmd --reload). I can't find any documentation how to configure this properly on docker side or firewalld side.
Note, I'm running:
ubuntu server: 16.04
firewalld: 0.4.0-1
docker-engine: 1.12.0
Example: I've created 2 custom bridge networks.
#systemctl restart docker

#iptables -S | grep DOCKER
-N DOCKER
-N DOCKER-ISOLATION
-A FORWARD -j DOCKER-ISOLATION
-A FORWARD -o docker0 -j DOCKER
-A FORWARD -o br-d74acac8e23a -j DOCKER
-A FORWARD -o br-5d6eab97e193 -j DOCKER
-A DOCKER-ISOLATION -i br-5d6eab97e193 -o docker0 -j DROP
-A DOCKER-ISOLATION -i docker0 -o br-5d6eab97e193 -j DROP
-A DOCKER-ISOLATION -i br-d74acac8e23a -o docker0 -j DROP
-A DOCKER-ISOLATION -i docker0 -o br-d74acac8e23a -j DROP
-A DOCKER-ISOLATION -i br-5d6eab97e193 -o br-d74acac8e23a -j DROP
-A DOCKER-ISOLATION -i br-d74acac8e23a -o br-5d6eab97e193 -j DROP
-A DOCKER-ISOLATION -j RETURN

# firewall-cmd --reload
success

# iptables -S | grep DOCKER
-N DOCKER
-N DOCKER-ISOLATION
-A FORWARD -j DOCKER-ISOLATION
-A FORWARD -o br-d74acac8e23a -j DOCKER
-A FORWARD -o br-5d6eab97e193 -j DOCKER
-A FORWARD -o docker0 -j DOCKER
-A DOCKER-ISOLATION -j RETURN


Added the community forum

Signed-off-by: sayantan1413<EMAIL_ADDRESS>Description
Added the community forum
This PR fixes #86
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@sayantan1413 : LGTM :partying_face:
@leecalcote : Sir, take this to your end!


Multiple interpretations

Haskell's diagrams allows for multiple interpretations of the domain specific language (DSL). For example, the code
mconcat . iterateN 3 (rotateBy (1/9)) $ triangle 1

can yield various outputs depending on how it is interpreted: the image on left, if interpreted as a diagram, or the third image, if interpreted as a trail.

(See their manual for more information.)
This feature might be nice to have (it will allow drawing something like the Hilbert curve), but I expect to be rather involved.

I honestly could not figure this part of the docs out... It seemed cool, but maybe a bit too cool?

Yes, I think you are right, it might be too confusing. If I really want that behavior, I guess I could have another data type that behaves differently. Specifically for paths (what is called Trails in diagrams), I could potentially introduce a data type whose composition glues the end of a path to the start of the other (instead of overlapping them at their origins).

Chalk now roughly has this. It's a bit more explicit in python since we don't have the ability of the type system to disambiguate. But that feels pretty close to what we are going to get.
If we wanted we could make the "monoid" structure a bit more explicit. (maybe having a general associative mconcat operator). But we can't do magic haskell stuff like have empty() work for all the types.


Installing yotta breaks ubuntu's pip

Following the instructions for Linux installation on http://docs.yottabuild.org/#installing results in a broken pip.
Ubuntu 14.10's version of pip is very old and relies on requests < 2.4. While installing yotta requests>=2.5 is installed.
I think using ubuntu pip to install a newer pip from PyPI before installing yotta should prevent this, but have not tested thoroughly.

This should be fixed by the updated installation instructions, which include installing a new pip.

See video here for instructions : https://www.youtube.com/watch?v=7UtCpvnP7EE