Conflict with user Python libraries

From @pareng on June 26, 2018 10:47
When I run the RNA-seq pipeline v1.4 (on the UPPMAX cluster rackham), the MultiQC step fails due to a version conflict with my local python libraries.  The problem seems to be that my local libraries are used instead of the ones in the Singularity image.
As a temporary fix, I've moved the directory containing my local libraries (~/.local), so that Python does not find it. @marcelm, who helped me understand the problem, suggested that a better fix would be to set the environment variable PYTHONNOUSERSITE in the Singluarity image. If it is set to a true value, local libraries should be ignored.
I tried setting it in my shell (export PYTHONNOUSERSITE=x) prior to running nextflow, but that did not help. I guess my environment is overridden by the Singularity image?
Error message from MultiQC (file work/.../.command.err):
[INFO   ]         multiqc : This is MultiQC v1.5
[INFO   ]         multiqc : Template    : default
[INFO   ]         multiqc : Searching '.'
[INFO   ]         multiqc : Only using modules custom_content, picard, preseq, rseqc, featureCounts, hisat2, star, cutadapt, fastqc

Traceback (most recent call last):
  File "/opt/conda/envs/ngi-rnaseq/bin/multiqc", line 767, in <module>
    multiqc()
  File "/opt/conda/envs/ngi-rnaseq/lib/python2.7/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/opt/conda/envs/ngi-rnaseq/lib/python2.7/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/opt/conda/envs/ngi-rnaseq/lib/python2.7/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/conda/envs/ngi-rnaseq/lib/python2.7/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/opt/conda/envs/ngi-rnaseq/bin/multiqc", line 413, in multiqc
    template_mod = config.avail_templates[config.template].load()
  File "/opt/conda/envs/ngi-rnaseq/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2323, in load
    self.require(*args, **kwargs)
  File "/opt/conda/envs/ngi-rnaseq/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2346, in require
    items = working_set.resolve(reqs, env, installer, extras=self.extras)
  File "/opt/conda/envs/ngi-rnaseq/lib/python2.7/site-packages/pkg_resources/__init__.py", line 783, in resolve
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.VersionConflict: (Jinja2 2.7 (/home/pareng/.local/lib/python2.7/site-packages), Requirement.parse('jinja2>=2.9'))

Copied from original issue: SciLifeLab/NGI-RNAseq#237

I believe @drpatelh told me that a user had something similar on the atacseq (?) pipeline and this was resolved (?). Is this still an issue?

Yes, if you are using an existing pipeline I think you can try to export this variable in the actual script section of the process in main.nf for it to be used properly by the Singularity image e.g. above this line:
https://github.com/nf-core/rnaseq/blob/37f260d360e59df7166cfd60e2b3c9a3999adf75/main.nf#L1103
This obviously means you would have to clone the pipeline, and make changes to main.nf in order to get it to work.

Adding this in general to the multiqc process isn't really an option or what do you think @ewels @drpatelh

cf. https://github.com/nf-core/tools/pull/444
Another option would be to add it to process.shell, but I don't love that..

This should have been fixed in https://github.com/nf-core/tools/pull/445


#491 - Stubs are registered to consul with localhost

#491 - Stubs are registered to consul with localhost

@marcingrzejszczak your build https://circleci.com/gh/spring-cloud/spring-cloud-contract/2882 caused the master to fail the build and as such fail my pull request :-( when is the expected fix?

Merged to 1.2.x

Thanks a lot :)


Can't retrieve TOKEN at webmail if signing into<EMAIL_ADDRESS>account

I'm signing into whiteout.io using a<EMAIL_ADDRESS>account. It entered the email and passcode which then takes me to the screen where it requests a 'TOKEN' or basically a 2FA but, that TOKEN is sent to the email one is signing into.
This is only an issue for me when I'm signing into a<EMAIL_ADDRESS>account because when using an existing email (e.g<EMAIL_ADDRESS>I can simply log into THAT account and get the TOKEN.
Am I missing something?
Please advise.

yes, this is still an open issue, but it will soon be obsoleted when we change the way key sync works. in the meantime: you can sign into that wmail.io account with another IMAP client, e.g. Apple Mail or Thunderbird, to retrieve the token. it will be readable in plain text

Got it!
Thanks for the response. Oh and FYI, I opted to import the keypair manually and that worked without having to go to a another IMAP client.


Angular doc: Remove unused BundlerMinifier.Core package references

Remove 2 unused references to the BundlerMinifier.Core NuGet package in the *.csproj file.

@scottaddie,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@scottaddie  Thanks!


Navbar only working in stackblitz demo

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
Was trying out 5.3.2 and the navbar. Noticed on the documentation page that it didn't appear to be rendering correctly.

Went to the stackblitz demo page for it and it is working there.

Put the same code in a single page in my project and it doesn't work (looks like the documentation page).
Then created a codepen page and copied the same code over to it. It also had the same issue.
https://codepen.io/scottlocknet/pen/poYovjJ
Appreciate any thoughts on what is going on.
Reduced test cases
https://codepen.io/scottlocknet/pen/poYovjJ
What operating system(s) are you seeing the problem on?
Windows
What browser(s) are you seeing the problem on?
Microsoft Edge
What version of Bootstrap are you using?
5.3.2

Thanks for reporting this issue @scottlocknet
Could you please provide a more precise definition of what you mean by 'that it didn't appear to be rendering correctly'? I'm not quite sure about the nature of the issue here.

Sorry for not being more precise. In the first screenshot, you can see that the hamburger menu didn't collapse up and instead spreads out to the right. The bootstrap documentation site (https://getbootstrap.com/docs/5.3/components/navbar/), CodePen example and looking locally at a generic page all show the subitems spread out to the right.
When I viewed in stackblitz, it rendered as expected, with it collapsed and then able to expand.

Ok, this is a user issue. I didn't even notice the nav class the example was generating. I've been stuck in legacy code for the past two years and just took the example code without looking through ever line... :)

Expand in there is forcing it out. Removing that fixed my issue. Odd that it worked in one testing tool and not anything else. No biggy though.


How the menu item is goes hidden

Steps to reproduce
The menu item container is hidden but the box is still on but the container still takes up space on the page and when used in a conversation with multiple messages, each message has its own menu with message interactions like edit, delete etc so all the messages I have to scroll to have a menu and hence take up space on the page and when I try to do overflow:hidden to prevent all menus from coming out of a particular container I can't do it because the menus are located directly inside the  tag and there is a solution to this problem that I have tried which is to show and hide the menus via
display: none
and not
visibility: hiddin
display: none => not reserve a place on the page
The element will not allocate space on the page and will not be visible
visibility: hiddin => reserve a place on the page
The element allocates space on the page but is not visible
Current behavior
The menu item container is hidden but the box is still on but the container still takes up space on the page and when used in a conversation with multiple messages, each message has its own menu with message interactions like edit, delete etc so all the messages I have to scroll to have a menu and hence take up space on the page and when I try to do overflow:hidden to prevent all menus from coming out of a particular container I can't do it because the menus are located directly inside the  tag and there is a solution to this problem that I have tried which is to show and hide the menus via
display: none
and not
visibility: hiddin
display: none => not reserve a place on the page
The element will not allocate space on the page and will not be visible
visibility: hiddin => reserve a place on the page
The element allocates space on the page but is not visible
Expected behavior
No response
Context
solution to this problem that I have tried which is to show and hide the menus via display: none and not visibility: hidden display: none => not reserve a place on the page The element will not allocate space on the page and will not be visible visibility: hiddin => reserve a place on the page The element allocates space on the page but is not visible
Your environment

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: visibility:hidden on menuItem

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve


gostatic: 2.34 -> 2.35

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

1 package built:

gostatic

Thank you!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1472


invalid includes/type names of fixed-width integers

Various headers include <cstdint> which is guuarantees only std::uint64_t and other std::*_t types. To use uint64_t without risk of compilation errors, one must inlude (deprecated) header <stdint.h>.
Since the library includes new (C++11) <cstdint>, it should use std::*_t type names (or make aliases for them in it's namespace).

Thanks for pointing it out. I fixed in latest commit, e5d884b. I am closing the issue, as in case I missed to fix any of them, it should be considered an isolated occurrence, and not a general issue like this one.


Sample/datasets api

Closes https://github.com/Woosmap/developers.woosmap.com/issues/1053
Describe your changes
Added sample for Datasets Api using DatasetsOverlay & DatasetsService service classes.
How to test
Test on PR-76
Checklist:

[x] My code follows the style guidelines for this repo
[x] I have performed a self-review of my code
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings/errors
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I don't require ops changes for this PR to go to prod
[x] This change does not include a migration

Comments

:tada: This PR is included in version 1.38.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Cache various lookups with Dry::Core::Cache

This PR uses Dry::Core::Cache to cache a few lookups that could otherwise be costly:

Templates within paths
Part classes within namespaces
Scope classes within namespaces

@GustavoCaso I hope you don't mind me going ahead with this, I wanted to play around with what we could cache. And I figured we should use our existing caching helper from dry-core.
Before this change, we were ~5.5x slower than ActionController.
Warming up --------------------------------------
   action_controller     1.000  i/100ms
            dry-view     1.000  i/100ms
Calculating -------------------------------------
   action_controller      6.519  (± 0.0%) i/s -     33.000
            dry-view      1.171  (± 0.0%) i/s -      6.000

Comparison:
   action_controller:        6.5 i/s
            dry-view:        1.2 i/s - 5.56x slower

After this change, we're ~1.8x slower:
Warming up --------------------------------------
   action_controller     1.000  i/100ms
            dry-view     1.000  i/100ms
Calculating -------------------------------------
   action_controller      5.995  (±16.7%) i/s -     30.000
            dry-view      3.317  (± 0.0%) i/s -     17.000

Comparison:
   action_controller:        6.0 i/s
            dry-view:        3.3 i/s - 1.81x slower

Here's the latest profile diagram (run using Hotch and a new script I checked into the repo): bundle exec ruby benchmarks/profile_controller.rb

So no more low hanging fruit, as far as I can tell? Would definitely appreciate any thoughts on taking things from here. 👋 @GustavoCaso @solnic

@solnic I decided not to use dry/core/cache because according to the docs:

Allows you to cache call results that are solely determined by arguments.

In my opinion the template? method cannot be solely cache by there arguments:
def template?(name, format)
name and format could be the same for multiple directories so that is why I need to create a unique key using dir as well. But seing @timriley changes looks like we can pass multiple arguments to dry/core/cache fetch_or_store(root, dir, name, format) do
I guess would be nice to update the docs as well 😄

@timriley LGTM, it's already a great start. We should be able to make it at least equally fast eventually. I'll take another look at perf improvements once I get a chance.

@solnic cool, will do. FWIW I just moved some of the caching a little earlier, with
https://github.com/dry-rb/dry-view/pull/97/commits/e178d41110498ebba7b7e4e9feb61f5e23097a64, and we're now just ~1.3-1.4x slower.
It's worth recognising that we're certainly doing a lot more in the dry-view side of this benchmark versus action view, at least in how we're preparing the scope for the template, e.g. checking for exposure dependencies, wrapping up values in parts, finding classes, etc., so this isn't truly an apples-for-apples comparison. Perhaps a better one would be e.g. using Draper to decorate every ivar in the Rails controller before it's passed to the template.

One more thing, MRI 2.6.0 has various optimizations that we benefit from, so here are the results running MRI 2.6.0.rc2:
∞ [ruby-2.6.0-rc2][node-v8.12.0] dry-view git(caching-with-dry-core) be ruby benchmarks/benchmark_controller.rb
Warming up --------------------------------------
   action_controller     1.000  i/100ms
            dry-view     1.000  i/100ms
Calculating -------------------------------------
   action_controller      7.135  (± 0.0%) i/s -     36.000  in   5.054691s
            dry-view      6.115  (± 0.0%) i/s -     31.000  in   5.077587s

Comparison:
   action_controller:        7.1 i/s
            dry-view:        6.1 i/s - 1.17x  slower

As you can see, the difference is meaningless, this probably means dry-view can easily become faster on future MRIs.

It's worth recognising that we're certainly doing a lot more in the dry-view side of this benchmark versus action view, at least in how we're preparing the scope for the template, e.g. checking for exposure dependencies, wrapping up values in parts, finding classes, etc., so this isn't truly an apples-for-apples comparison. Perhaps a better one would be e.g. using Draper to decorate every ivar in the Rails controller before it's passed to the template.

That is correct. OTOH it is a legit scenario, and a very common one, so it's probably worth having such a basic benchmark. To balance this better adding a more advanced scenario would be great too!


Removed ThisCouldBeUsButYouPlaying

In May 29, the project stopped being maintained

          1 Warning

:warning:
Found 2 link issues, a project collaborator will take care of these, thanks: )

Link issues by awesome_bot

Line
Status
Link

313
Error
http://oclint.org/  getaddrinfo: Name or service not known

2446
Error
https://iconresizer.com/  Connection refused - connect(2) for "iconresizer.com" port 443

  Generated by :no_entry_sign: Danger


Unable to detect a Project Id in the current environment.

Hi I'm getting this annoying error: How do I add a project ID in the current environment. How do I fix it?

Hi @floyergilmour , looks like you are missing the step 2 of requirements which is setting up default application credentials for GCP.
You can follow the following guide to set it up. Once that is setup, the error should go away. You then can verify if you gcp project is correctly setup using
gcloud config get-value project

This will show you your current gcp project.  Please let me know if this works for you

@floyergilmour please following the 'Google' component of the Development Requirements here, if you still have issues, please feel free to reach out!


Delete function for Dictionary, Set, and List

Implement del function for Dictionary, Set, and List types.
Documented here.

#26 took over this.


I got "Unable to connect to your device with this details" when trying to connect with my Socket

Describe the bug
All info is right, but I can't connect my Socket to Tuya Local. I got the "Unable to connect to your device with this details" message and I can't connect!
To Reproduce
Trying to connect first time.
Additional context
Add any other context about the problem here.
If the bug involves a device, then please include device diagnostics from
Settings / Devices & Services / Tuya Local (your device) / "1 device"
Please check if there are messages from Tuya Local in the Home
Assistant log (Settings / System / Logs), and include those too.
If applicable, add screenshots to help explain your problem.

Please see #522 for possible causes of connection issues.


feature request: Make GET channels/{target}/messages use request body instead of query parameters

What do you want to see?
At the moment, Search for Messages (POST channels/{target}/search) utilizes nearly identical parameters to Fetch Messages (GET channels/{target}/messages), but the former uses a JSON request body whereas the latter uses query parameters, which in my personal opinion is a worse way to build requests especially with as many parameters as the methods allow.
Would it be possible to make both endpoints use the same request body type (specifically a JSON request body instead of query parameters) for consistency? This would obviously be a breaking change for clients and API wrappers which depend on this functionality, but I think it would be better to get that over with sooner rather than later.

I've just remembered that the reason Fetch Messages uses query parameters is that GET methods cannot have a body, which is obviously something that has to be worked around.
Might I instead propose a sort of "merging" of the two endpoints into one single Search for Messages endpoint with an optional query field on the object which determines whether to do a normal fetch versus an actual search?


Teensy41 support SDHC - Storage init Error

Describe the bug
Working on the Teensy41 and trying to get the SD Card functionality to work. For this using the application developed by @bdkrae / @ufechner7. See https://github.com/bdkrae/zephyr_teensy4_test. Which is based on the zephyr/samples/subsys/fs/fat_fs.
Please also mention any information which could help others to understand
the problem you're facing:

Using a Teensy41
SanDisk Edge 4GB Micro SD, both tried with a formatted (MS-DOS and ExFAT partition)

I have also tried to follow the instructions
Card voltage switch
Teensy 4.1 has no hardware to switch card voltage. As a workaround in the file zephyr/drivers/disk/usdhc.c the line 502 needs to be changed:

#define SDMMCHOST_NOT_SUPPORT USDHC_HOST_CTRL_CAP_VS18(1)

This prevents the driver from trying to switch over to 1,8V supply for the card. Also, a dummy pin is defined in board defs.

I cannot find this Macro where that is defined. I already tried using the FLAG directly USDHC_HOST_CTRL_CAP_VS18_MASK. Still the same error.
To Reproduce
Using Zephyr 2.7.1
Expected behavior
SD Card should have been mounted
Logs and console output
[00:00:00.501,000] <inf> usdhc: USDHC detection other than GPIO
[00:00:00.501,000] <err> sdhc: Storage init ERROR!

Environment:

OS: MacOS
Toolchain GNU  10.3.1
Zephyr 2.7.1

I tested the interface with Arduino and the same SD card. Which works and shows both information and files on the SD card.

When the function usdhc_board_access_init is ran in zephyr/driver/disk/usdhc.c
if (!priv->detect_gpio) {
		LOG_INF("USDHC detection other than GPIO");
		priv->inserted = true;
		/* DATA3 does not monitor card insertion */
		base->PROT_CTRL &= ~USDHC_PROT_CTRL_D3CD_MASK;
		if ((base->PRES_STATE & USDHC_PRES_STATE_CINST_MASK) != 0) {
			priv->inserted = true;
		} else {
			priv->inserted = false;
			return -ENODEV;
		}
	} else {
		ret = usdhc_cd_gpio_init(priv->detect_gpio,
				config->detect_pin,
				config->detect_flags,
				&priv->detect_cb);
		if (ret) {
			return ret;
		}
		ret = gpio_pin_get(priv->detect_gpio, config->detect_pin);
		if (ret < 0) {
			return ret;
		}

		gpio_level = ret;

		if (gpio_level == 0) {
			priv->inserted = false;
			LOG_ERR("NO SD inserted!");

			return -ENODEV;
		}

		priv->inserted = true;
		LOG_INF("SD inserted!");
	}

Additional information:
The card present switch is not connected to the i.mxrt of the Teensy 4.1 .
So the cd-gpios in the dts should not be there.
The statement if ((base->PRES_STATE & USDHC_PRES_STATE_CINST_MASK) != 0) {  evaluates false and therefore does not detect the SD card. I could not find where this flag should be set.
Result is the card is never detected
A dirty workaround is to add this to the dts or overlay and then it works:
&usdhc1 {
	status = "okay";
	no-1-8-v;
	cd-gpios = < &gpio3 21 GPIO_ACTIVE_HIGH>;
};

Note that I also add no-1-8-v;
Which applicable to the Teensy4.1

@EmbedWise , would you like to open a PR to fix it in boards/arm/teensy4/teensy41.dts please?

@EmbedWise I don't have a teensy board to test this fix on, but looking at the schematic for the board it looks like the chip detect line for the SD card is not routed. The USDHC driver supports detecting a card via the internal pull up resistor on the DAT3 line. I've opened a PR (#42379) to switch the teensy41 board to using the DAT3 method of detecting SD card presence, instead of the dummy GPIO that was in use. Can you test that PR and verify it works for you? I would use the tests/drivers/disk test.

The PR is in main now, so you should be able to verify the fix with the latest version of Zephyr. I tested the fix on an RT1060 EVK, which has the same SoC as the teensy 4.1, but I would still like to get your feedback if possible to ensure the fix was valid @EmbedWise

@danieldegrasse I will test it and revert back to you.

The PR is in main now, so you should be able to verify the fix with the latest version of Zephyr. I tested the fix on an RT1060 EVK, which has the same SoC as the teensy 4.1, but I would still like to get your feedback if possible to ensure the fix was valid @EmbedWise

I verified it on the Teensy board but it is not detecting the SD card. Code that I used can be found on Github https://github.com/EmbedWise/zephyr_teensy4_app
If I change it to a dummy GPIO it works and detects the card.

@EmbedWise, taking another look at this, there is no power control circuitry for the SD card, so that may be causing the SD card detection with DAT3 to fail. What specific error do you see when using DAT3 card detection, as opposed to the dummy GPIO?

@danieldegrasse there is indeed no power control circuit. The error I see it the following:
[00:00:02.001,000] <inf> main: Started zephyr 3.0.0 on board arm/teensy41.
[00:00:05.000,000] <inf> usdhc: USDHC detection other than GPIO
[00:00:05.001,000] <inf> usdhc: USDHC detection using DAT3 pull
[00:00:05.001,000] <err> sdhc: Storage init ERROR!

I just tried the samples/subsys/usb/mass example and it still does not work out of the box...
maybe this issue should get reopened.

I just tried the samples/subsys/usb/mass example and it still does not work out of the box...

maybe this issue should get reopened.

Probably so, I'll go ahead and do that. This board doesn't have any card detect circuitry, so I think the best solution may be to modify the driver to assume a card is present if no card detect method is configured.

@bearsh can you try #77301 on your board?

~@bearsh can you try #77301 on your board?~ or just try the latest main, as that PR has merged

yes, works! thanks!


Clean and Reorganize Testing

Data unification #34 broke most tests and those need to be fixed.

Removed tests along with the old persistence managers. Need to add new test for the unified manager (CSV).
Can probably base on the tests built in test main.

Reorganized enough. I'd like to do some reorganization of the data classes in the next update and then upgrade the testing to JUnit 5. I also need to get refresh my grip on the best way to test all this.


make the display of process LTS of input models optional

Currently, LTS of the input models is shown along with the result of comparison. Instead, make it optional so that users can view the LTS of the inputs only if required.
Approach: Add a checkbox or button to show/hide LTS of input models.

Fixed as part of commit f835cf1a5b40b003dc3d4de8bcedbe3dfae47177


Update text.dart

Semantic identifier added because not visible on appium platform
Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
List which issues are fixed by this PR. You must list at least one issue. An issue is not required if the PR fixes something trivial like a typo.
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] I followed the breaking change policy and added Data Driven Fixes where supported.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Hello @kaganzdmr58! Looks like this change is failing some checks. Can you please take a look at those? It is also missing a test (see the bot message above). Can you please add one?

(triage) I am going to close this one since there was no follow up. If you find the time to work on this again feel free to reopen this PR. Thank you.


Update list item REST API not working as expected

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
The Update list item REST API should work as described in the documentation or the documentation should be updated.
https://docs.microsoft.com/en-us/sharepoint/dev/sp-add-ins/working-with-lists-and-list-items-with-rest#update-list-item
Observed Behavior
When using the Update list item REST API, it is not possible to make it work as described in the documentation.  The following error is thrown

The property __metadata does not exist on type...

Suggested update
In the documentation it is explicitly mentioned to add the metadata field in the body.
body: { '__metadata': { 'type': 'SP.Data.TestListItem' }, 'Title': 'TestUpdated'}
However this triggers an error. After removing the metadata from the body I was able to make the code work. So please update the documentation accordingly.

I have successfully updated an item using this REST documentation. If you are able to share your request and the headers being sent, we may be able to find the issue in your code.

@EagleWizard28 @lucabandMSFT  I don't think this is a bug. The note on the page clearly mentions getting ListItemEntityTypeFullName

To do this operation, you must know the ListItemEntityTypeFullName property of the list and pass that as the value of type in the HTTP request body.

Here's how you can get that:
http://site url/_api/web/lists/GetByTitle('Test')?$select=ListItemEntityTypeFullName

@unowiz it seems it doesn't matter which value I pass, I get the following error:

The property __metadata does not exist on type

So yes I was able to fetch the "ListItemEntityTypeFullName", but I'm unable to pass it. When I just drop the "_metadata" it works. Here some example code that works for me, which updates the list item. Take into account that the list has a custom content type.
    public async setItemComplete(itemId: number) {
        const itemUrl = `${this._normalizedWebUrl}/_api/web/lists/getByTitle('${this._ListTitle}')/items(${itemId})`;
        const client = new SPHttpClient();

        const updateData = {
            ResolutionDate: new Date().toISOString()
        };

        const httpClientOptions: FetchOptions = {
            headers: new Headers({
                 Accept: "application/json",
                "Content-Type": "application/json",
                "X-HTTP-Method": "MERGE",
                "IF-MATCH": "*",
            }),
            body: JSON.stringify(updateData)
        };

        return client.post(itemUrl, httpClientOptions);
    }

Am I overlooking something?

Hi
I am facing same issue.
I am using Fetch api  post request to create an Item, if I remove '__metadata' property then only item gets created otherwise it throws error.
Is there anyone still facing this issue?


Proposal: Repository v2 parser choice

Part of the discussion for Helm 3 at the moment is over the introduction of several approaches to make indexing-related tasks more efficient (heirarchical index files, optional server-side search, etc). My current focus is optimizing the index file parsing step. To this end, I've benchmarked a number of parsers, following existing work by @mattfarina. My current results are available here.
I've created this issue to share with the community my progress and my intended path forward, and to ask for feedback. I'd like to make sure my approach is on the right track, and that I'm not unknowingly duplicating someone else's work, or vice versa.
Approach
My approach is derived from three core expectations:

Indexes should be saved as JSON files, not YAML files

YAML is complicated, and parsing it is naturally expensive.
JSON is a very simple format, and yet it's still more than sufficient to properly encode indexing data, with no change to the structure of the data.
YAML has no advantage over JSON in this case. The primary reason to use YAML over JSON is increased ability for humans to read and edit the data, but humans don't directly read or edit index files.
Since JSON is a subset of YAML, a newer JSON index could technically be backwards-compatible with older YAML-based versions of Helm.

Index files should be streamed from disk

Currently, index files are loaded to memory in their entirety before being parsed. Streaming the text data into the parser a block at a time is much more space-efficient.
Much more often than not, index files are parsed simply to search for an attribute or check the existence of a particular chart. These simple lookups can have O(1) space efficiency if streaming is used.
As YAML supports references, this kind of efficient streaming isn't possible. If JSON is used instead, that problem disappears.

All else being equal, allocations should be minimized where possible

Parsers tend to use a lot of temporary intermediate data during conversion. If this data is allocated to the heap, it can potentially create a lot of garbage.
Go's GC is non-generational, meaning it's less likely to immediately collect short-lived allocations. These allocations can build up quickly, which can cause both time and space efficiency to decrease.
Minimizing allocations (for example, by allocating intermediate data to the stack instead of the heap) can reduce the GC workload, increasing efficiency.

At this point, my goal was to compare and judge a number of existing parsers based on these three expectations.
Results
This is a breakdown of the benchmarks, which can be found here. Index files contain 500,000 charts, and are 200 - 350 MB in size.

ghodss/yaml and yaml.v2 are the only YAML parsers being tested, and they have the worst performance by far, with 20 - 40 seconds and 4 - 8 GB to parse an index.
encoding/json (the standard library JSON parser) fares a little better, with 6 seconds and 1 GB to parse an index. It's interesting to note that the streaming version has worse time and space efficiency than the non-streaming version.
json-iterator/go and a8m/djson improve time, but not space, with 2 seconds and 1 GB for an index.
buger/jsonparser seems very promising, with 1 - 1.5 seconds and 300 - 800 MB for an index. However, it doesn't support streaming.
darthfennec/jsonparser is a customized fork of buger/jsonparser, modified to support streaming. I hacked this together as a PoC for this benchmark, because I wanted to see what kind of practical improvements streaming could make for space efficiency. The results are 2 - 3 seconds and 120 MB for an index. This is twice as slow as the original buger/jsonparser, but it has the best space efficiency by far.

Allocation count appears to be roughly proportional to cpu and ram usage, as expected. buger/jsonparser claims that zero allocations are made in the common case, although this depends on the lengths of the JSON strings being parsed. In practice, it does appear to allocate less than other parsers.
Next Steps
darthfennec/jsonparser is unstable and will not function as-is outside of the benchmark environment. I suppose that's what happens when I try to retrofit streaming into a library that really isn't designed to support it. I expect the drop in speed would have been mostly avoided if I had been more careful with the modifications; I don't think it's inherent to the addition of streaming capabilities. Also, I do think the clear space advantage of this approach makes it worth looking further into.
I don't think fixing darthfennec/jsonparser is the right move. Streaming is very much in conflict with the design of buger/jsonparser, so I don't believe this hack could ever be particularly effective. Instead, I've started to write a new JSON parser from scratch, which I expect will benchmark at least as well as darthfennec/jsonparser, without sacrificing flexibility and consistency.
When the new parser is far enough in development, I'll add it to the benchmark to see where it stands.

I think this issue should be opened at
https://github.com/helm/community/

@rimusz Thanks, moved the issue to https://github.com/helm/community/issues/49


Disruptive actions like Stop and Restart should require end-user confirmation

Detailed Description
Clicking on Terminate Action within an Application requires an end-user confirmation  Are you sure you want to terminate instance {{index}}? to avoid any accidentally termination of an active instance.
Performing more-disruptive actions like Stop and Restart on the other hand do not require end-user confirmation and immediately perform the activity.
The request would be to implement the end-user confirmation for Stop and Restart action, similar to Terminate. Ideally this also includes the application name.
Stop: Are you sure you want to stop '{{appName}}'?
Restart: Are you sure you want to restart '{{appName}}'?
Terminate of instance: Are you sure you want to terminate instance {{index}} of '{{appName}}'?

Context

Avoid interruptions of application by a single click in a GUI, especially as the Restart icon looks like a refresh button and could be misleading to an end-user.
At least it would be consistent with the current behavior of the Terminate button which requires end-user confirmation.

Thanks @berchtos for raising this issue. We recently merged a contribution that resolves this in our master branch and it will be included in the next release.


stuck at rbac issues...

DON'T MERGE YET
I'm hitting this issue.

the trick was- --serviceaccount seldon on argo


BUG: binom.stats with frequency of 0

Describe your issue.
binomial interval should be 0,0 when population frequency is 0.
e.g.
binom.interval(0.95,2000,0)
(1999.0, 0.0)
should give the same answer as with a tiny frequency.
binom.interval(0.95,2000,1e-6)
(0.0,0.0)
Reproducing Code Example
binom.interval(0.95,2000,0)

Error message
No error message, this is a bug in functionality.

SciPy/NumPy/Python version information
1.5.4 1.19.5 sys.version_info(major=3, minor=6, micro=5, releaselevel='final', serial=0)

Please upgrade your version of SciPy; it is a few years out of date and the implementation of binom has been replaced in the meantime. I cannot reproduce this behavior in main.
from scipy import stats
stats.binom.interval(0.95,2000,0)  # (0.0, 0.0)


Incorrect docstring in EH checkpoint

https://github.com/Azure/azure-sdk-for-python/blob/3fdbaa75e5ef2cd9f0151a71992d50d0c7820739/sdk/eventhub/azure-eventhub-checkpointstoretable/azure/eventhub/extensions/checkpointstoretable/_tablestoragecs.py#L228-L238
rtype must be a single type, this should be:
        :rtype: iterable[dict[str,Any]]
        :return: Iterable of dictionaries containing partition ownership information:
                - `fully_qualified_namespace` (str): The fully qualified namespace that the Event Hub belongs to.
                  The format is like "<namespace>.servicebus.windows.net".
                - `eventhub_name` (str): The name of the specific Event Hub the checkpoint is associated with,
                  relative to the Event Hubs namespace that contains it.
                - `consumer_group` (str): The name of the consumer group the ownership are associated with.
                - `partition_id` (str): The partition ID which the checkpoint is created for.
                - `owner_id` (str): A UUID representing the current owner of this partition.
                - `last_modified_time` (float): The last time this ownership was claimed.
                - `etag` (str): The Etag value for the last time this ownership was modified. Optional depending
                  on storage implementation.

fixed with recent typing PRs


Eureka Integration karyon

i want  Eureka Integration karyon,but  I don't know how to integrate it? Hope to have more examples,to achieve health check and service registration.can you help me?

Could you explain a little bit more? The current Karyon2 examples have eureka integrated,  you just need to uncomment a line


Send ContentLauncher commands from tv-casting-app to tv-app

Problem

tv-casting-app needs to be able to send ContentLaucher commands to the tv-app after commissioning completes.

Change overview
What's in this PR

Updates to the tv-casting-app.zap and zzz generated files (Most of this PR is generated code. All the generated code is separated into the first commit: https://github.com/project-chip/connectedhomeip/pull/12913/commits/66fcad31bdb6875642a05cbf5e2ca34638a0c90c)
Updates to tv-casting-app main to actually send LaunchURL commands to the tv-app

Testing
Tested that the tv-casting-app is able to send ContentLauncher.LaunchURL with test data to the tv-app. See tv-casting-app logs below:

[1639166929.632388][94996:94996] CHIP:DMG: ICR moving to [AddingComm]
[1639166929.632402][94996:94996] CHIP:DMG: ICR moving to [AddedComma]
[1639166929.632448][94996:94996] CHIP:IN: Prepared encrypted message 0x55ee2575ebb0 to 0xFFFFFFFEFFFFFFFF (1)  of type 0x8 and protocolId (0, 1) on exchange 1129i with MessageCounter:983433.
[1639166929.632481][94996:94996] CHIP:IN: Sending encrypted msg 0x55ee2575ebb0 with MessageCounter:983433 to 0xFFFFFFFEFFFFFFFF (1) at monotonic time: 703187783 msec
[1639166929.632553][94996:94996] CHIP:DMG: ICR moving to [CommandSen]
[1639166929.632751][94996:94996] CHIP:EM: Received message of type 0x10 with protocolId (0, 0) and MessageCounter:13510908 on exchange 6229r
[1639166929.632765][94996:94996] CHIP:EM: Found matching exchange: 6229r, Delegate: (nil)
[1639166929.632791][94996:94996] CHIP:EM: Rxd Ack; Removing MessageCounter:983432 from Retrans Table on exchange 6229r
[1639166929.632803][94996:94996] CHIP:EM: Removed CHIP MessageCounter:983432 from RetransTable on exchange 6229r
[1639166929.698344][94996:94996] CHIP:EM: Received message of type 0x9 with protocolId (0, 1) and MessageCounter:13510909 on exchange 1129i
[1639166929.698481][94996:94996] CHIP:EM: Found matching exchange: 1129i, Delegate: 0x55ee271b1640
[1639166929.698708][94996:94996] CHIP:EM: Rxd Ack; Removing MessageCounter:983433 from Retrans Table on exchange 1129i
[1639166929.698726][94996:94996] CHIP:EM: Removed CHIP MessageCounter:983433 from RetransTable on exchange 1129i
[1639166929.698796][94996:94996] CHIP:DMG: ICR moving to [ResponseRe]
[1639166929.698831][94996:94996] CHIP:DMG: InvokeResponseMessage =
[1639166929.698847][94996:94996] CHIP:DMG: {
[1639166929.698863][94996:94996] CHIP:DMG:      suppressResponse = false,
[1639166929.698878][94996:94996] CHIP:DMG:      InvokeResponseIBs =
[1639166929.698897][94996:94996] CHIP:DMG:      [
[1639166929.698912][94996:94996] CHIP:DMG:              InvokeResponseIB =
[1639166929.698931][94996:94996] CHIP:DMG:              {
[1639166929.698946][94996:94996] CHIP:DMG:                      CommandDataIB =
[1639166929.698963][94996:94996] CHIP:DMG:                      {
[1639166929.698979][94996:94996] CHIP:DMG:                              CommandPathIB =
[1639166929.698999][94996:94996] CHIP:DMG:                              {
[1639166929.699017][94996:94996] CHIP:DMG:                                      EndpointId = 0x1,
[1639166929.699034][94996:94996] CHIP:DMG:                                      ClusterId = 0x50a,
[1639166929.699051][94996:94996] CHIP:DMG:                                      CommandId = 0x1,
[1639166929.699109][94996:94996] CHIP:DMG:                              },
[1639166929.704464][94996:94996] CHIP:DMG:
[1639166929.704482][94996:94996] CHIP:DMG:                              CommandData =
[1639166929.704495][94996:94996] CHIP:DMG:                              {
[1639166929.704502][94996:94996] CHIP:DMG:                                      0x0 = "Example data",
[1639166929.705344][94996:94996] CHIP:DMG:                                      0x1 = 0,
[1639166929.705409][94996:94996] CHIP:DMG:                              },
[1639166929.705422][94996:94996] CHIP:DMG:                      },
[1639166929.705438][94996:94996] CHIP:DMG:
[1639166929.705450][94996:94996] CHIP:DMG:              },
[1639166929.705465][94996:94996] CHIP:DMG:
[1639166929.705477][94996:94996] CHIP:DMG:      ],
[1639166929.705491][94996:94996] CHIP:DMG:
[1639166929.705502][94996:94996] CHIP:DMG: },
[1639166929.705529][94996:94996] CHIP:DMG: Received Command Response Data, Endpoint=1 Cluster=0x0000_050A Command=0x0000_0001
[1639166929.705547][94996:94996] CHIP:ZCL: LaunchURLResponse:
[1639166929.705559][94996:94996] CHIP:ZCL:   data: Example data
[1639166929.705570][94996:94996] CHIP:ZCL:   contentLaunchStatus: 0
[1639166929.705583][94996:94996] CHIP:SVR: ContentLauncher: Default Success Response
[1639166929.705600][94996:94996] CHIP:DMG: ICR moving to [AwaitingDe]
[1639166929.705614][94996:94996] CHIP:EM: Sending Standalone Ack for MessageCounter:13510909 on exchange 1129i
[1639166929.705647][94996:94996] CHIP:IN: Prepared encrypted message 0x7fff993d2460 to 0xFFFFFFFEFFFFFFFF (1)  of type 0x10 and protocolId (0, 0) on exchange 1129i with MessageCounter:983434.
[1639166929.705682][94996:94996] CHIP:IN: Sending encrypted msg 0x7fff993d2460 with MessageCounter:983434 to 0xFFFFFFFEFFFFFFFF (1) at monotonic time: 703187856 msec
[1639166929.705746][94996:94996] CHIP:EM: Flushed pending ack for MessageCounter:13510909 on exchange 1129i

/rebase


Implement Josh Yardley's statistics

https://ipastorage.box.com/s/0eagp9vr0ttkwxpf8w5t
The Research Rap is here:
https://ipastorage.box.com/s/ze8sri9iano5gm7nan4u

Just want to re-iterate the importance of this. Josh's important statistics are the Simple Response Variance and Reliability Ratio, which can be estimated from backcheck/ reinterview data under certain assumptions. Integrating into bcstats would definitely facilitate measurement studies within IPA based on these (and possibly other) statistics.
Note that a more extensive review of the theory, ideally in consultation with a statistician, should be done before implementing this.

I definitely like the idea of making bcstats more analytically rigorous. However, I wonder how our method of directed back check questioning would affect this if at all? Maybe we can do some research and come up with a systematic analysis plan for back checks after GST. With that in hand, I am confident that I could update bcstats in the second quarter of the year.
Chris


Are CHEBI:2719 and CHEBI:131170 the same?

Hi we have noticed that the inchi for CHEBI:2719 & CHEBI:131170 are the same - do they refer to the same entity or is there suppose to be a difference between them? If they are suppose to be separate entities, should there be a relationship between them? Thank you
tagging @tommycarstensen too

@amalik01 Thanks so much responding to this so quickly. Could you let us know which was the "winning" term ID please so we can use that? Thanks

The primary ID is CHEBI:2719


Add to porn list

Please add the following to porn list. Thank you.
https://javeng.com

Thank you for this @wjang1.
Ping @sinfonietta for evaluation and inclusion in the porn list.


[Table] Demo for table should derive the sorted data

[X] This is a v1.x issue (v0.x is no longer maintained).
[X] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
The Table "Sorting & Selecting" component demo should make it easy to follow React best practices. So, if I quickly change data for this.props.data, it should just work:
class EnhancedTable extends React.Component {
  constructor(props) {
    super(props);

    this.state = {
      order: 'asc',
      orderBy: 'calories',
      selected: [],
//  ⚠ NOTE: data is stored in the state
      data: props.data,
      ...

Current Behavior
Used as is, the EnhancedTable component will not rerender when props change. The React documentation warns against copying props into the state (see here). Yet with this example it is very easy to replace the static data with props.data and expect it to work as is.
Steps to Reproduce (for bugs)
I'd be happy to provide steps to reproduce, not sure that's necessary here since there's a better pattern for the example that is fairly easy to implement.
The problem can be found in the EnhancedTable constructor:
class EnhancedTable extends React.Component {
  constructor(props) {
    super(props);

    this.state = {
      order: 'asc',
      orderBy: 'calories',
      selected: [],
//  ⚠ NOTE: data is stored in the state
      data: [
        createData('Cupcake', 305, 3.7, 67, 4.3),
        createData('Donut', 452, 25.0, 51, 4.9),
        createData('Eclair', 262, 16.0, 24, 6.0),
        createData('Frozen yoghurt', 159, 6.0, 24, 4.0),
        createData('Gingerbread', 356, 16.0, 49, 3.9),
        createData('Honeycomb', 408, 3.2, 87, 6.5),
        createData('Ice cream sandwich', 237, 9.0, 37, 4.3),
        createData('Jelly Bean', 375, 0.0, 94, 0.0),
        createData('KitKat', 518, 26.0, 65, 7.0),
        createData('Lollipop', 392, 0.2, 98, 0.0),
        createData('Marshmallow', 318, 0, 81, 2.0),
        createData('Nougat', 360, 19.0, 9, 37.0),
        createData('Oreo', 437, 18.0, 63, 4.0),
      ].sort((a, b) => (a.calories < b.calories ? -1 : 1)),
      page: 0,
      rowsPerPage: 5,
    };
  }

handleRequestSort = (event, property) => {
    const orderBy = property;
    let order = 'desc';

    if (this.state.orderBy === property && this.state.order === 'desc') {
      order = 'asc';
    }

//  ⚠ NOTE: on sorting change, `state.data` is reset
    const data =
      order === 'desc'
        ? this.state.data.sort((a, b) => (b[orderBy] < a[orderBy] ? -1 : 1))
        : this.state.data.sort((a, b) => (a[orderBy] < b[orderBy] ? -1 : 1));

    this.setState({ data, order, orderBy });
  };

I think the example would be simpler and more correct if the sorted data was computed when rendering. I will be providing a PR with a potential solution.

I think that storing the sorted data into the state is done in order to improve performance. I think that it's something I would keep for an application running in production, at least, if I had a significant amount of data an repetitive renderings.
However, for the sack of this demo, I agree, we can move toward a more "pure" logic (with less state).


Feat: add api 'WithExternalPkg'.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

kclvm-go/pkg/kcl/api.go
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

add api 'WithExternalPkg'.

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 29 (44.83%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 58.153%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/kcl/opt.go
12
28
42.86%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
1719

Relevant Lines:
2956

💛  - Coveralls


Closures with complex arguments

In #35 the discussion was mainly about how we format the body of closures. Currently rustfmt uses visual indent when we need to use multiple lines for closure's arguments:
fn examples() {
    let commented = |// first
                     a, // argument
                     // second
                     b: WithType, // argument
                     // ignored
                     _| {
         (a, b)
     };

    let block_body = move |xxxxxxxxxxxxxxxxxxxxxxxxxxxxx,
                           ref yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy| {
        xxxxxxxxxxxxxxxxxxxxxxxxxxxxx + yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy
    };

    let y = x.into_iter().map(
        |AaaaaaAaaaaaaaaaa {
             bbbbbb_bbbb,
             ccc_ccccccc,
         }| (),
    );
}

Since the default indent style in other places is block indent, I was wondering where rustfmt should use it in closure's arguments as well:
fn examples() {
    let y = x.into_iter().map(
        |
            AaaaaaAaaaaaaaaaa {
                bbbbbb_bbbb,
                ccc_ccccccc,
            },
        | (),
    );

    let commented = |
        // first
        a, // argument
        // second
        b: WithType, // argument
        // ignored
        _,
    | {
        (
            aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa,
            bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb,
        )
    };

    let block_body = move |
        xxxxxxxxxxxxxxxxxxxxxxxxxxxxx,
        ref yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy,
    | {
        xxxxxxxxxxxxxxxxxxxxxxxxxxxxx + yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy
    };
}

Any thoughts?

Figuring out how to format closures has been driving me slightly bonkers. And vim doesn't have any appreciation for visually indenting the arguments. Treating them as a fn item declaration may be the sanest approach, would be consistent with the rest of Rust. It'd be nice if they just had a more compliant syntax, but that ship has already sailed, sunken off the coast of Costa Rica, and been salvaged as a source for metal free from the radioactive contamination of nuclear testing.

With my format team hat on: I do think we should use block style for closures. I also think we should treat the leading | and trailing | -> Type the same way we treat other grouping, so that we can put the | on the same line as the start of a function accepting the closure as an argument.

The current style can have a disgusting result:
let read = move |mut conn: DbConn<'a>,
                 (post_id, my_person_id, is_mod_or_admin): (
  PostId,
  Option<PersonId>,
  Option<bool>,
)| async move {

I don't think that's consistent with the style guide, or at least I don't know what rule would produce that result.
cc @rust-lang/rustfmt

The current style can have a disgusting result:
let read = move |mut conn: DbConn<'a>,
                 (post_id, my_person_id, is_mod_or_admin): (
  PostId,
  Option<PersonId>,
  Option<bool>,
)| async move {

This is due to a very long standing bug in rustfmt and as Josh noted, does not adhere to the style guide prescriptions for closures
https://github.com/rust-lang/rustfmt/issues/3865

This issue has been bugging me too. Anyone working on it?
I wonder if a separate option should be added (like imports_indent) or should indent_style be extended for closures too? The former might be preferable for backwards-compatibility.


[Announce] HW Accelerated rendering now works also on XP

Hello, XP users.
my alternative dll, called "XP API Adapter" for Thorium/Supermium browsers,
has this feature recently added.
And you can test/use it.
https://github.com/IDA-RE-things/Chrome-xp-api-adapter/releases

@IDA-RE-things Nice! Would you be willing to share the source with me privately, I.E. without publishing it or sharing it with anyone else?
I will include this in my next Thorium release

Hello, Alex. What do you think, why the author of original lib dont shares his sources with you ? And you just use his binary DLL for this... Why he dont want to do this ? But he shares sources of the installer.


Fix manually setting octicons as iconset

Very simple fix to #193

Fixed my eslinter so hopefully no more failed checks :p


Add links to connected services docs

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/dotnet-tutorials-services-security.md
:white_check_mark:Succeeded
View

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.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/dotnet-tutorials-services-security.md
:white_check_mark:Succeeded
View

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.

@CamSoper This just adds a couple of links giving the tutorials for adding these features using VS Connected Services.


dlib.shape_predictor on the whole image

Hi, shape_predictor() takes 2 input parameters: the input image and the face detection results.
How can I just run shape_predictor() on the whole image without face detection. Can someone provide me a Python example, Thanks.
Maybe construct a bounding box wrapper for the whole image? How can I construct such a mmod_rectangle object in Python?

Yes, make a rectangle for the whole image.  Read the documentation: http://dlib.net/python/index.html. It tells you what to do.

Hmm, some of the functions are missing from http://dlib.net/python/index.html.  Like the dlib.rectangle constructor, which has a docstring. I'm not sure why sphinx didn't include it in the HTML docs.
Anyway, make a dlib.rectangle(left,top,right,bottom).  Use that.

@davisking Thanks for your reply. Yep, I have read the api and have also tried to construct an mmod_rectangle object in the following ways, but all fail, rasing an error: dlib.mmod_rectangle: No constructor defined!
det=mmod_rectangle()
det=mmod_rectangle(rect=None, confidence=1)

Can you show me an example on how to construct such an object to pass to shape_predictor()? Thanks.

Look at what I just said.  dlib.rectangle

OK, I will try that later, Thanks


MSML Deliverable Listing

What type of deliverables/outputs does MSML currently support? in what format? who do they serve? the client or our internal process?

Maybe a table?

The short answer is the main deliverables can be seen in the MSML template and they are:
Obsidian vault creation for internal development in the beginning and then eventually easily interfacing with clients
Notebooks and paradigms for internal dev of testing and playgrounds for testing out bound code on specs
Single simulation and simulation sweep notebooks for creating interfaces that clients can easily modify for running their own experiments as well as setting up experiment notebooks for client deliverables


Merge changelog and version bump back to master

What was the end-user problem that led to this PR?
The problem was that version and changelog need to be cherry-picked from the stable branch.
What is your fix for the problem, implemented in this PR?
My fix is to do just that.

@bundlerbot r=hsbt


The requested URL can't be reached

Hi
I am trying to post a request in ARC and there's a problem with the request body.
When putting the json in Raw Headers I get "Header Value should not be empty" and "Content-Type header not defined".
And When I am removing the "{}" marks from the json object, and adding custom content type, I am getting :
The requested URL can't be reached
The service might be temporarily down or it may have moved permanently to a new web address.
The request object could not be initialized
(GET requests work perfectly)
Thanks,
Gili

You must provide both content type and body at the same time. The it will not show error.


Do a 64-bit build since winget supports multiple architectures now.

That should make things faster.

Not sure how much it'll help as it appears that it uses about 100 MB of RAM when updating the cache as opposed to roughly 32 MB, but it should help anyway. The 32-bit version will still be available.

Not going to do this for now, but other people can do this if they want to manage it.


Update README.md

Added french translation of the README.md file.
I have seen that there was a request for adding translations of the README.md file.

@goldfishnour Did you create a README_FR.md? Or overwrite our current one?

@goldfishnour Did you create a README_FR.md? Or overwrite our current one?

Yes I wanted to create a french readme document, not overwriting the current one.

README.md
@jquense  Kindly find the readme.md file in french, I saw from issue 2511 that translating README document was encouraged.


Fix start selenium session for new firefox and selenium bindings

Firefox 57.0.1 & Selenium Server 3.8.1 & Python Bindings 3.8.0

Codecov Report

Merging #329 into master will increase coverage by 0.07%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #329      +/-   ##
==========================================
+ Coverage   58.26%   58.33%   +0.07%
==========================================
  Files          41       41
  Lines        3577     3550      -27
  Branches      418      416       -2
==========================================
- Hits         2084     2071      -13
+ Misses       1391     1376      -15
- Partials      102      103       +1

Impacted Files
Coverage Δ

vmmaster/webdriver/commands.py
64.92% <33.33%> (-1.21%)
:arrow_down:

core/db/__init__.py
29.41% <0%> (-5.03%)
:arrow_down:

vmpool/server.py
42.42% <0%> (-1.7%)
:arrow_down:

vmmaster/app.py
87.5% <0%> (-0.91%)
:arrow_down:

vmmaster/webdriver/helpers.py
64.08% <0%> (+0.44%)
:arrow_up:

vmpool/endpoint.py
68.99% <0%> (+0.48%)
:arrow_up:

core/db/models.py
61.96% <0%> (+0.69%)
:arrow_up:

core/sessions.py
77.08% <0%> (+1.57%)
:arrow_up:

vmmaster/webdriver/__init__.py
91.66% <0%> (+4.03%)
:arrow_up:

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 7740fe1...d6c38ee. Read the comment docs.

!test

!test

!test

!test

!test

!test

!test

!test

!test

!test

!test

!test

!test

@test

!test


Modify ui context url path - rancher 2.0

I read rancher 2.0 documentation and didn't find any way how to change the rancher ui context path url.
I would like to work with nginx in front of rancher in ha mode and be able to work with sub path like:  example.mydomain.com/rancher
Is this feature will be available in next close releases?

No; use a different subdomain.  The UI is entirely static files so supporting this is difficult, and running more than one sensitive application on the same domain is a bad idea in general.  A compromise of any one of them can allow attack on all of them since you are defeating protection against XSS, CSRF, reading cookies across domains, etc.

Hi @vincent99,
we as the company invested quite a few months to incorporate rancher to our on-prem solution. The final step was integrating Rancher web ui with Apache. We have no other choice except using subpaths. Would you please consider to develop this feature ?

As I've described, this is a bad idea and we have no plans to support it.
(How have you had it running for months if you have no other choice?)
You can possibly make it work on your own with a custom build of the UI for the right base URL and probably a bunch of rewrites to map the API call URLs.

Well, @vincent99 we did not use apache for development cause this is the one of the final steps of course. I would not even dream that it is not possible to run rancher at some context path. I understand that a subdomain would solve the situation however, it is not acceptable for our on-prem customers. And we got like several thousands installations by now.


[Courses] No error message upon attempt to add cource with name that already exists

Environment: Windows 10 (x64), Google Chrome v.93.0.4577.82
Reproducible: always
Pre-conditions:

WHAT project launched locally using npm.
User has an account.
User has been assigned the role of Mentor
'Sign in to WHAT' page is opened.

Steps to reproduce:

Click 'Add a course' button on Course page
Add course name that already exists
Click 'Save' button

Actual result:

No error message

Expected result:

Error. The course already exsist

According to endpoint accessibility, mentor is not allowed to add course.
The problem is related to Secretary and Admin's roles


Support Tag selection in Edit macro as we support layer selection

"I would like to use tags in DrawIO diagram, but it is only useful if the tags can be turned on and off on the different views of the same diagram displayed by the “embed draw.io diagram” plugin in the confluence space. Just as it can be configured for the diagram layers."
Asked in DID 6046

But please, instead generating a preview for each TAG as is the case for the layers, it would be more efficient to provide a list only (or preview only as an option). It would be good to fix this also for the layers. If you have more than 30 layers or TAG, it takes a long time to generate the previews, and selecting them is difficult.

Please do not close this issue. It is still relevant, it would be a major feature in confluence to support tags in diagrams.


Shop is broken

I was looking through the files to find more ways to use the format and found that there is a shop feature. When I tried to use it, it broke the story to the point of not loading. Is this normal?

Do you have an example of the code you're using to implement the feature?

I went into the src folder and then opened the shop.ts file using notepad++ and found the functions that calls on the shop. I put some of the functions in a story and it would not load the story.

Interesting. Did you maybe incorrectly put in commas and semicolons?


fix: enforce light mode

System dark mode was causing issues with newer Tailwind versions. This PR ensures light mode regardless of the user system setting.

Related to https://github.com/medusajs/nextjs-starter-medusa/issues/253


DNNClassifier and GridSearchCV example

Hi ,
I am trying to run an gridsearch example in https://github.com/petewarden/tensorflow_makefile/blob/master/tensorflow/contrib/learn/python/learn/tests/test_grid_search.py
If example inside testIrisDNN intend to be working ?
I am getting

TypeError: init() got an unexpected keyword argument 'params'

This bug is against a temporary fork of the TensorFlow source code I did for development purposes unfortunately, so it's not going to be possible to debug what's going wrong here. Closing for now, but please reopen if you are able to reproduce this on the main https://github.com/tensorflow/tensorflow source code.

Hi, I ran same code in grid_search_test.py and got the same error in TF 1.3.0, scikit-learn 0.19
Have you any idea?
import tensorflow as tf
from tensorflow.contrib import learn
from sklearn.model_selection import GridSearchCV
from sklearn import datasets
iris = datasets.load_iris()
feature_columns = learn.infer_real_valued_columns_from_input(iris.data)
classifier = learn.DNNClassifier(feature_columns=feature_columns, hidden_units=[10, 20, 10], n_classes=3)
grid_search = GridSearchCV(classifier, {'hidden_units': [[5, 5], [10, 10]]}, scoring='accuracy', fit_params={'steps': [50]})
grid_search.fit(iris.data, iris.target)
/Users/rickypark/anaconda/envs/handson-ml/lib/python3.6/site-packages/sklearn/model_selection/_search.py:583: DeprecationWarning: "fit_params" as a constructor argument was deprecated in version 0.19 and will be removed in version 0.21. Pass fit parameters to the "fit" method instead.
'"fit" method instead.', DeprecationWarning)
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
 in ()
----> 1 grid_search.fit(iris.data, iris.target)
~/anaconda/envs/handson-ml/lib/python3.6/site-packages/sklearn/model_selection/_search.py in fit(self, X, y, groups, **fit_params)
622                                      n_candidates * n_splits))
623
--> 624         base_estimator = clone(self.estimator)
625         pre_dispatch = self.pre_dispatch
626
~/anaconda/envs/handson-ml/lib/python3.6/site-packages/sklearn/base.py in clone(estimator, safe)
60     for name, param in six.iteritems(new_object_params):
61         new_object_params[name] = clone(param, safe=False)
---> 62     new_object = klass(**new_object_params)
63     params_set = new_object.get_params(deep=False)
64
TypeError: init() got an unexpected keyword argument 'params'


Revert "README: disables goreportcard due to gojp/goreportcard/issues/90"

Reverts osrg/namazu#144
Goreportcard works now

Coverage remained the same at 41.873% when pulling 5ee6371621016eabd4214ba79164f0da7e7055af on revert-144-AkihiroSuda-patch-1 into a8f804d406b5dcf115cdff9e512eb5c64d6a7748 on master.


[Responsive Navigation] First example fails on Safari 9.1.1

How can we reproduce this bug?

Step one - view this page in Safari 9.1.1.
Step two - resize browser window smaller than larger.
Step three - see the problem.

What did you expect to happen?
Expected menu to change to vertical on small (it did) and then back to horizontal on large.
What happened instead?
Small window correctly has vertical menu, large window shows item 1 and 2 stacked vertically, while item 3 is to the right of item 2 horizontally, rather than all 3 items being in a horizontal line.
Test case:
Test case is this page of documentation:
http://foundation.zurb.com/sites/docs/responsive-navigation.html

@cxj Thanks for the bug report. Could you confirm which example it is you are seeing this on - is it the very first example below here? (the one without any dropdowns etc).

Yes, that's the one (first example on page you linked).

I have the same issue. Do you have any updates?

Safari 9 is not supported in V6 anymore. This does not appear to be an issue in Safari 10.


cargo rustc subcommand

Thoughts?
Compile a package and all of its dependencies

Usage:
    cargo rustc [options] [<pkgid>] [--] [<opts>...]

Options:
    -h, --help              Print this message
    -j N, --jobs N          The number of jobs to run in parallel
    --features FEATURES     Features to compile for the package
    --no-default-features   Do not compile default features for the package
    -p, --profile PROFILE   The profile to compile for
    --target TRIPLE         Target triple which compiles will be for
    --manifest-path PATH    Path to the manifest to fetch depednencies for
    -v, --verbose           Use verbose output

The <pkgid> specified (defaults to the current package) will have all of its
dependencies compiled, and then the package itself will be compiled. This
command requires that a lockfile is available and dependencies have been
fetched.

All of the trailing arguments are passed through to the *final* rustc
invocation, not any of the dependencies.

Dependencies will not be recompiled if they do not need to be, but the package
specified will always be compiled. The compiler will receive a number of
arguments unconditionally such as --extern, -L, etc. Note that dependencies are
recompiled when the flags they're compiled with change, so it is not allowed to
manually compile a package's dependencies and then compile the package against
the artifacts just generated.

I think this or something like it should happen.
Personally, I'm still waiting on a way to pass a linker script to the linker without writing a custom Makefile (and defeating a lot of the things that make cargo so nice to use). I'm still fairly new to rust/cargo, so I may be overlooking something. The problem I'm running into is that cargo is almost too high level for such a low level language.

This seems like what we want. So, cargo rustc --foo will internally call rustc -lbar -lbaz --extern quux=.... -L target/foobar ....... --foo?
As long as the usual link args are available I think we can make this work :)

We have worked around this for now by instead setting the linker to be a shell script that manually invokes everything we need.

Such a function would be really helpful. Otherwise it is almost impossible to use any rustc flag that is not exposed by cargo.

I've already found myself using the verbose flag of cargo to obtain the rustc call, and then alter its flags to obtain, e.g., the LLVM bytecode or similar. A command like this would make that much easier.

@RalfJung Rust team members have told me that they replace the rustc binary with a shell script that calls out to the binary itself in order to add extra arguments, as a workaround for this lack of functionality when debugging.

Thanks :) . A trick I already used on other occasions, but it's
certainly worth remembering that it also helps here.

Attempting to get better rust support into Emacs's Flycheck has really shown me the usefulness of this -- this would make my life much, much easier.

I concur with @GBGamer - this is something really useful for editors and IDE's, as well as some refactoring and analysis tools.
I'd like to see this implemented as it would enable Emacs' Flycheck to use cargo and properly handle dependencies.

+1, the sooner this gets implemented the better.

Added in https://github.com/rust-lang/cargo/pull/1568, thanks @sondrele!


Use MainCommand for Package in Add/Remove

Fixes #84

Thanks for this contribution! I'll merge this and push the env var support later today expect a new release soonish


fix: add missing bootstrap variables

Bootstrap has continued to add and use new SCSS variables since Bootstrap 4.4.1. Since we no longer pull in all of Bootstraps variables (in order for us to colocate them with component styles) this is causing SASS compilation errors.
This PR pulls in all new variables as of bootstrap 4.6.0 and pins the version.

Coverage remained the same at 87.223% when pulling 6ce48c3cf595a78f3e2536769a698139df30eb2d on abutterworth/add-new-bs-vars into bec2481781ef960842416f986acaa11f6c14fe4f on master.

Coverage remained the same at 87.223% when pulling 6ce48c3cf595a78f3e2536769a698139df30eb2d on abutterworth/add-new-bs-vars into bec2481781ef960842416f986acaa11f6c14fe4f on master.

:tada: This PR is included in version 13.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 13.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


non-deterministic build caused error

We are using CI to build the package, we find that packages built by two of our runners are not working, while other runners produces expected results. Here is the error with bad ones:
Uncaught TypeError: Cannot set property 'ProcedureUtils' of undefined
    at Object.module.exports.$jscomp (vertical.js:779)
    at __webpack_require__ (vertical.js:21)
    at Object.<anonymous> (vertical.js:759)
    at __webpack_require__ (vertical.js:21)
    at Object.<anonymous> (vertical.js:103)
    at __webpack_require__ (vertical.js:21)
    at Object.<anonymous> (vertical.js:95)
    at __webpack_require__ (vertical.js:21)
    at vertical.js:64
    at Object.defineProperty.value (vertical.js:67)

After digg into build.py, we find that gen_blocks() didn't sort filenames outputted by glob.glob(). And the two problematic runners have the following order of filenames:
['blocks_vertical/procedures.js', 'blocks_vertical/looks.js', 'blocks_vertical/sound.js', 'blocks_vertical/extensions.js', 'blocks_vertical/control.js', 'blocks_vertical/operators.js', 'blocks_vertical/vertical_extensions.js', 'blocks_vertical/event.js', 'blocks_vertical/pen.js', 'blocks_vertical/motion.js', 'blocks_vertical/default_toolbox.js', 'blocks_vertical/sensing.js', 'blocks_vertical/data.js']

blocks_vertical/procedures.js comes first. When it creates namespace Blockly.ScratchBlocks.ProcedureUtils, Blockly.ScratchBlocks was still undefined, which causes the error.
Suggesting:

Sort filenames in build.py:gen_blocks() , to ensure deterministic build.
Create namespace properly in blocks_vertical/procedures.js

can you tell me how to solve the problem

Hey, I just dropped a separate PR from @tansly's—my fix attempts to use the closure compiler system to create namespaces and dependencies, and it also addresses the glob.glob ordering issue.

This PR should be able to be closed now, with #1896 merged! :tada:


Word and Character Count

Is it possible to display a word and non-space character count? I know that a text editor does not work nor "think" in the same way as Microsoft Word, and for these applications, it's almost as easy to open Word and type or copy in the text. Still, if it isn't a difficult feature, you'd have at least one user.

+1 to this, It be very nice to see a word count especially for using this as a writing program


add role for runnerreplicasets to fix error:

Failed to list *v1alpha1.RunnerReplicaSet: runnerreplicasets.actions.summerwind.dev is forbidden: User "system:serviceaccount:actions-runner-system:default" cannot list resource "runnerreplicasets" in API group "actions.summerwind.dev" at the cluster scope

Thank you for your controbution!
The RBAC is generated from the controller code comments, so the code needs to be modified.
I fixed it in a897eee .
Please let me know if you have any problems.
Thanks!

ah, good to know, thx


PBF

Plane of Best Fit (https://github.com/rdkit/rdkit/tree/master/Contrib/PBF, https://pubs.acs.org/doi/abs/10.1021/ci300293f)

Why don't you wrap rdMolDescriptors.CalcPBF?

Sorry for the long answer. I just can not make Travis work.

Sorry for the long answer.
Not at all 😄

I just can not make Travis work.
Why do you use the private key in the Travis environment variables?
The private key is used in upload to anaconda cloud.
And it sould be fixed in #36 (it already merged in latest develop branch).
I'm sorry for your inconvenience but could you please rebase to latest develop?

Thanks,

I made a rebase, but Travis still does not work.
Throws error "iv undefined"

Thank you.
It seems that tests are executed and some tests are failed.

Please use file attribute instead of relative path.
Please add since and __slots__ attribute (it is required in develop branch).
Please update mordred/descriptors/__init__.py doccomment

Please see review comments for details.

Use the descriptor from rdkit

I merged your PR after few modification (d65d3fa451aca3f32adf4124a83532978ae57e46).
thanks!


Alliance Side Poses

In order to drive from the controller, the heading will always be set to zero, which is rendered in WPILib poses as facing away from the blue alliance operator consoles. This orientation is not how the gyroscope will respond when on the red alliance, making the field orientation incorrect when transitioning from autonomous to teleoperated. With this issue, pose updates with AprilTags will also not work when on the red alliance.

Upon recreating the issue in simulation, some work was done in YAGSL/allianceOriented to try finding a solution by inverting the direction of translation and gyroscoping zeroing when on the red alliance. So far, this has seemed to work for both autonomous and teleoperated simulation, but will be physically tested later today.


The Documentation link in README is broken. Please fix it.

As the title says.

Hi! There is no documentation for the project currently. I should probably just remove the reference from README. I'm going to add it as soon as the project is ready to be published.


AssertionError: pipeline_idr scripts/modules - Exception

pipeline_idr can not be imported

This has been fixed - test_import passes.


Add Kudu bindings

Currently doesn't include unit tests since we don't have mock objects for that.

Thanks! One more question, otherwise looks good.

Thanks, @jdcryans!


use chainid instead of netid and avoid missing geth api - fix #54

But this doesn't address if/why geth removed net_version, and chainId is deprecated according to https://web3py.readthedocs.io/en/stable/web3.net.html?highlight=net_version#web3.net.chainId
Also, typechecking in CI fails because of unreachable code.

This issue tracks the missing net_version RPC endpoint: https://github.com/ethereum/go-ethereum/issues/22034

The endpoint was mistakenly removed from geth, and was reintroduced in https://github.com/ethereum/go-ethereum/issues/22034. Closing.


Add option to generate a list of files that would be removed

This is already listed at DEBUG log level, so this option would be enabling reporting of eligible files in some sort of format that is easily read and shared (e.g., logged to a ticket of some kind).
Example/concept usage:
sudo ./elbow --paths /var/log/tomcat7/ --recurse --age 120 --generate-report

Regarding an output format, I'm not sure what would work best. Hopefully something concise for the screen and something else to be saved in a file for later review.

Note: I need to think of the scope as being over a thousand files.
When I ran this recently against a box with a logrotate misconfiguration (thanks Nagios) a little over 1500 files were removed. I would like to see that list, but not directly on the console, at least not one file per line and certainly not twice as the v0.5.0 release is doing.

I saw an example earlier today of using tabwriter to build a formatted table. I could try the same approach for listing pruned files.
https://golang.org/pkg/text/tabwriter/


Types are not published to NPM

The package doesn't include its types when installing the latest version (18.11.0) from NPM.

Oh. You are right. Sorry.
Fixed now with 18.11.1.


Chinese translation error in issues page

Gogs version (<IP_ADDRESS>7):
Git version: Not relevant
Operating system: Not relevant
Database:

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite
[x] Not relevant

Can you reproduce the bug at https://try.gogs.io:

[x] Yes (https://try.gogs.io/ZeroNet/ZeroNet-brokenmirror/issues/1)
[ ] No
[ ] Not relevant

Log gist (usually found in log/gogs.log):

Description
Sign in to join this conversation.

登陆 并参与到对话中。
should be
登录 并参与到对话中。
like at the top right

哈哈


get() returned more than one UserSocialAuth -- it returned 2!

I'm trying to implement Steam login into my project, and I get the error get() returned more than one UserSocialAuth -- it returned 2! when I try to do user.social_auth.get(provider='steam'). This seems to happen when someone has the same username—the rows in social_auth_usersocialauth are associated to the same auth.User. Attached an image to show what I mean:

As a side note, is it a good idea to store the Steam name as username field in auth.User? Steam names can be changed and are not unique. I think it's better to just return something unique for the username field, like UID.

Anyone?

The steam username is used as a base to the user username, if it's duplicated, then a extra hash will be added (so they are never repeated), also, if the user changes his username in steam, it won't change the username in your site (username is defined only once).
You are getting that error because you have two steam accounts associated to a single user, python-social-auth doesn't put any constraint to that, you can have several accounts from the same provider associated to a given user.
You should do user.social_auth.filter(provider='steam')[0] (or select one of them).

if it's duplicated, then a extra hash will be added (so they are never repeated)

That's the problem here, it doesn't seem to add an extra hash when there are collisions. I logged in using two different Steam accounts with the same username and this happened.
Anyway, I solved the problem by writing a custom pipeline to replace usernames with Steam UIDs, it's good enough for my needs.
Thanks!

Has this been solved?  I am facing the same issue. @CrimsonRay: Could you provide your pipeline code?


Updated TypingSpeed csproj and packages.config for VS 15.0

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: egoughnour sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for your contribution @egoughnour!


Add ipv6 testing

xref: https://github.com/istio/istio/issues/1756#

Hi @aojea. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/cc @howardjohn

@aojea: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
77403801b82d5e86d6cccd93300ef4647169ca37
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@aojea: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
77403801b82d5e86d6cccd93300ef4647169ca37
link
/test lint_istio

integ-local-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-local-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@aojea: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
77403801b82d5e86d6cccd93300ef4647169ca37
link
/test lint_istio

integ-local-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-local-tests_istio

integ-galley-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-galley-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@aojea: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
77403801b82d5e86d6cccd93300ef4647169ca37
link
/test lint_istio

integ-local-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-local-tests_istio

integ-galley-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-galley-k8s-tests_istio

integ-telemetry-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@aojea: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
77403801b82d5e86d6cccd93300ef4647169ca37
link
/test lint_istio

integ-local-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-local-tests_istio

integ-galley-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-galley-k8s-tests_istio

integ-telemetry-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-telemetry-k8s-tests_istio

integ-security-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@aojea: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
77403801b82d5e86d6cccd93300ef4647169ca37
link
/test lint_istio

integ-local-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-local-tests_istio

integ-galley-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-galley-k8s-tests_istio

integ-telemetry-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-telemetry-k8s-tests_istio

integ-security-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-security-k8s-tests_istio

integ-pilot-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@aojea: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-local-tests_istio

integ-galley-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-galley-k8s-tests_istio

integ-telemetry-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-telemetry-k8s-tests_istio

integ-security-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-security-k8s-tests_istio

integ-pilot-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-pilot-k8s-tests_istio

lint_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@aojea: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-local-tests_istio

integ-galley-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-galley-k8s-tests_istio

integ-telemetry-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-telemetry-k8s-tests_istio

integ-security-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-security-k8s-tests_istio

integ-pilot-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-pilot-k8s-tests_istio

lint_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test lint_istio

integ-mixer-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-mixer-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@aojea: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-local-tests_istio

integ-galley-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-galley-k8s-tests_istio

integ-telemetry-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-telemetry-k8s-tests_istio

integ-security-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-security-k8s-tests_istio

integ-pilot-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-pilot-k8s-tests_istio

lint_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test lint_istio

integ-mixer-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-mixer-k8s-tests_istio

release-test_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test release-test_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@aojea: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-local-tests_istio

integ-galley-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-galley-k8s-tests_istio

integ-telemetry-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-telemetry-k8s-tests_istio

integ-security-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-security-k8s-tests_istio

integ-pilot-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-pilot-k8s-tests_istio

lint_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test lint_istio

integ-mixer-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-mixer-k8s-tests_istio

release-test_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test release-test_istio

integ-conformance-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-conformance-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@aojea: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-local-tests_istio

integ-galley-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-galley-k8s-tests_istio

integ-telemetry-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-telemetry-k8s-tests_istio

integ-security-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-security-k8s-tests_istio

integ-pilot-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-pilot-k8s-tests_istio

lint_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test lint_istio

integ-mixer-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-mixer-k8s-tests_istio

release-test_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test release-test_istio

integ-conformance-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-conformance-k8s-tests_istio

integ-distroless-k8s-tests_istio
64b2be552aaf50687550517a954689ef1bd7e765
link
/test integ-distroless-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Add tinycss2 and cssselect2 dependencies

Also add more tests

Checklist

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

fixes #5

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.
I do have some suggestions for making it better though...
For recipe:

Whenever possible python packages should use noarch. See https://conda-forge.org/docs/meta.html#building-noarch-packages

@conda-forge-admin, please rerender

Travis failed due to a timeout, restarted the failed jobs


Investigate if it's feasible to use local Docker or tar base image when building multi-platform images

About local Docker daemon images:
Note it is possible to pull an image to a Docker daemon even if the image platform doesn't match the platform running the daemon. (You need to pull using a digest. Of course, you can't run an image that doesn't match the machine platform.)
However, I doubted a local Docker daemon could host something like a manifest list. Tried buildx by following the doc to build multi-arch images but without pushing to a registry (i.e., without --push). As I expected, it shows a warning "no output specified".
$ docker buildx build --platform linux/amd64,linux/arm64,linux/arm/v7 .
WARN[0000] No output specified for docker-container driver. Build result will only remain in the build cache. To push result image into registry use --push or to load image into docker use --load
[+] Building 24.0s (9/9) FINISHED
 => [internal] booting buildkit                                                                                16.7s
 => => pulling image moby/buildkit:buildx-stable-1                                                                                15.5s
 => => creating container buildx_buildkit_mybuilder0                                                                                1.2s
...

It's not feasible to use a base image from a Docker daemon when configuring multiple <platform>s at the moment. Potentially we could change our <from><image> config so that the user can configure multiple local Docker images with different architectures. However, I am not sure if this is worth considering. For now, I think we can just fail when given multiple platforms.

About local tar images:
Haven't looked into it, but I speculate that a tar image (either OCI or Docker) can easily contain multiple images. Worth taking a look.

But for now, I think these are a low priority and we can just fail when attempting to use a local base image with multiple platforms configured. Should be done before we can enable multi-platform image building.

added some local tar buildx image info on issue #2751 if relevant

@tom-haines thanks for the info!

Using an OCI image tarball or even directory as an input would make a lot of sense. Related: #2173

It would also be of great help to output the built image into an oci directory structure instead of an oci tar, amending any existing index.