Thinktecture.IdentityModel and Owin.BasicAuthentication library not setting identity correctly.

I am using Thinktecture.IdentityModel and trying to use the Owin.BasicAuthentication lib with the UseBasicAuthentication with Webapi and OWIN. The Identity in my controllers has no claims and shows not authenticated.
I setup the owin config with this in Startup.Auth.cs
        app.SetDefaultSignInAsAuthenticationType("Basic");

        //app.Use(typeof (BasicAuthentication), new[] {_container.Resolve<UserAccountService>()});
        app.UseBasicAuthentication(new BasicAuthenticationOptions("realm", ValidationFunction)
        {
            AuthenticationType = "Basic",
            AuthenticationMode = AuthenticationMode.Active
        });

        var oauthServerConfig = new OAuthAuthorizationServerOptions
        {
            AllowInsecureHttp = true,
            Provider = new MembershipRebootProvider(_container.Resolve<UserAccountService>()),
            TokenEndpointPath = new PathString("/token")
        };
        app.UseOAuthAuthorizationServer(oauthServerConfig);

        var oauthConfig = new OAuthBearerAuthenticationOptions
        {
            AuthenticationMode = Microsoft.Owin.Security.AuthenticationMode.Active,
            AuthenticationType = "Bearer"
        };
        app.UseOAuthBearerAuthentication(oauthConfig);

    private static Task<IEnumerable<Claim>> ValidationFunction(string userName, string password)
    {
        IEnumerable<Claim> claims = null;
        UserAccount user;
        string tenant = "";

        if (userName.Contains("\\"))
        {
            var parts = userName.Split('\\');
            tenant = parts[0];
            userName = parts[1];
        }
        else
        {
            throw new Exception("Cannot determine tenant and username.");
        }

        var userAccountService = _container.Resolve<UserAccountService>();
        if (userAccountService.Authenticate(tenant, userName, password, out user))
        {
            claims = user.GetAllClaims();
        }

        return Task.FromResult(claims);
    }

the claims are returned from membership reboot as expected.
In the AuthenticateCoreAsync method in the BasicAuthenticationHandler class I see the AuthenticationTicket is created and returned successfully.
But when I view it in my controller method there are no claims and it says not authenticated..
var identity = (ClaimsPrincipal)Thread.CurrentPrincipal;

What am i missing?

Check the Controller's User property. Also, do you have SuppressDefaultHostAuthentication enabled in your Web API config?

I did have the suppress SuppressDefaultHostAuthentication in the config but when checking this I noticed I did not have a filter for "Basic" only Oauth BearerToken.  I added that and now it works!
Thanks for the push in right direction!
            config.SuppressDefaultHostAuthentication();
            config.Filters.Add(new HostAuthenticationFilter(OAuthDefaults.AuthenticationType));
            config.Filters.Add(new HostAuthenticationFilter("Basic"));


Fix doc building

[x] Our CI should test that the docs build correctly, but this is not the case
[x] Our docs currently don't build

The issue was revealed with the with_bokeh example. Because that example does not run on CI, the problem in session.py was never revealed.


[di] DI assumes that all structs it analyzes will have /1/ property... this is not always true =><=.

[di] DI assumes that all structs it analyzes will have /1/ property... this is not always true =><=.
The culprit here is NSManagedObject subclasses that only have @NSManaged
attributes.
This doesn't affect predictable mem opts since this issue is in the
DIMemoryUseCollector that only affects DI and that I have removed.
rdar://34589327

@swift-ci smoke test and merge

It would be good if somewhere we documented all the different kinds of initializers in the language and how they map to DI constructs. For example this special case of an NSManagedObject subclass keeps coming up, and I would wager a bet that even most of the core team does not know about this language "feature". ;-)


taxa2lca is vulnerable to different order of listed input taxon IDs

taxa2lca subcommand should be insensitive to the order in which the taxon IDs are listed in stdin; however, this is not the case in the following example
$ echo "2\n131567\n" | unipept taxa2lca
taxon_id,taxon_name,taxon_parent_id,taxon_rank
2,Bacteria,131567,superkingdom
$ echo "131567\n2\n" | unipept taxa2lca
taxon_id,taxon_name,taxon_parent_id,taxon_rank
1,root,1,no rank

Original issue by @pdawyndt on Thu May 08 2014 at 20:49.
Closed by @bmesuere on Fri May 09 2014 at 15:47.

On my machine this produces the same output (root). @pdawyndt If you provide an exact timestamp you ran these tests, I can maybe pinpoint the error with the logs on scruffy.
Original comment by ghost on Fri May 09 2014 at 15:00.

The issue here has to do echo itself. Using echo -e seems to solve this.
Original comment by ghost on Fri May 09 2014 at 15:45.

@bmesuere @tvwillem : sure, my mistake; however, I still think it's less correct to give root as LCA here, but I'll discuss this in another existing issue that deals with possible scenarios for aggregation over taxonomic hierarchies.
Original comment by @pdawyndt on Sat May 10 2014 at 09:32.


can't get entity id?

When creating an entity using the <Entity> component, how can I get the id of the created entity? With vanilla cesiumJS the viewer.add(entity) function returns the entity that's been created/added so in that case the id can be accessed. What's the resium way?

Hi, encoutered the same problem some time ago.
My solution was to keep a ref of the <Viewer /> component (as described in resium's docs) called viewerRef, and then the object viewerRef.current.cesiumElement had all the ID's of my entities.
Note: We're still thinking in my team about the best practice of how to keep this global ref, since many components in the tree with Viewer as root are using some raw cesium data/properties... But as a temporary solution, I believe it will do the trick for you

Hi, encoutered the same problem some time ago.
My solution was to keep a ref of the <Viewer /> component (as described in resium's docs) called viewerRef, and then the object viewerRef.current.cesiumElement had all the ID's of my entities.
Note: We're still thinking in my team about the best practice of how to keep this global ref, since many components in the tree with Viewer as root are using some raw cesium data/properties... But as a temporary solution, I believe it will do the trick for you

ID is automatically generated with no id specification on Entity component. So we refer it to use ref:
export default function Example() {
  const entityRef = useRef(null);
  useEffect(() => {
    const id = entityRef.current?.cesiumElement?.id;

    /* something to do with id */
  }, []);

  return <Viewer><Entity ref={entityRef} /></Viewer>;
}

ID is automatically generated with no id specification on Entity component. So we refer it to use ref:
export default function Example() {
  const entityRef = useRef(null);
  useEffect(() => {
    const id = entityRef.current?.cesiumElement?.id;

    /* something to do with id */
  }, []);

  return <Viewer><Entity ref={entityRef} /></Viewer>;
}


Initialize module

what
Initialize module

What does it do?
Why does it do it?

[ ] add enabled flag

@osterman I can't approve the changes since I started the PR but @Jamie-BitFlight addition looks good to me... with the ternary to include a ServiceName only when specified - this module can be used for both ecs cluster and service level metrics now. I'm tempted to rename it to terraform-aws-ecs-cloudwatch-sns-alarms and add another ternary possibly for FARGATE vs EC2 launch type (this can come later).


Fix edit command not working correctly with filtering

Fixed clear functionality not clearing cat -A.
Fixed clear functionality not removing the displayed person from taskListManager.

Codecov Report

Merging #209 (39a75a3) into master (d9bc3de) will decrease coverage by 0.00%.
The diff coverage is 72.58%.

@@             Coverage Diff              @@
##             master     #209      +/-   ##
============================================
- Coverage     68.81%   68.81%   -0.01%
- Complexity      860      866       +6
============================================
  Files           115      115
  Lines          3082     3117      +35
  Branches        393      399       +6
============================================
+ Hits           2121     2145      +24
- Misses          831      838       +7
- Partials        130      134       +4

Impacted Files
Coverage Δ

...c/main/java/seedu/address/ui/AllTaskListPanel.java
0.00% <0.00%> (ø)

...eedu/address/logic/commands/DeleteTaskCommand.java
75.32% <36.36%> (-14.39%)
:arrow_down:

...java/seedu/address/logic/commands/EditCommand.java
87.02% <79.06%> (-3.30%)
:arrow_down:

...rc/main/java/seedu/address/model/ModelManager.java
97.08% <100.00%> (+0.05%)
:arrow_up:

...java/seedu/address/model/task/TaskListManager.java
82.65% <100.00%> (+0.73%)
: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 d9bc3de...39a75a3. Read the comment docs.


Fixed  the release jade version of rplidar_ros .

why rplidar_ros Package Links: the Jenkins jobs(12) status can not be loaded. and apt-get cannot find ros-jade-rplidar-ros in the jade version. How can i check it and fixed it ?


[proposal of enhancement] slot validation, docs

Rasa Core version: 0.10.1
Python version: 3.5
Operating system (windows, osx, ...): Ubuntu 16.04
Issue:
My suggestion is to update the documentation about the possibility and example of custom field validation.
I use FormAction to collect a few pieces of information. I wanted one of the text slots to store only specified values so I needed to check if the value is valid and if it's not then set the slot to None in order to keep asking for its value.
My first attempt was to override run method of FormAction class.
    def run(self, dispatcher, tracker, domain):
        my_slot = tracker.get_slot("my_slot")
        if my_slot not in VALID_SLOTS:
            tracker.update(SlotSet("my_slot", None))
        return super().run(dispatcher, tracker, domain)

but it didn't work. Locally the slot value was set to None but in the implementation of run method from the parent class it was restored to the invalid value. Digging through the source code I found that FormField class has "validate" method with the following comment:

Check if extracted value for a requested slot is valid.
Users should override this to implement custom validation logic, returning None indicates a negative validation result, and the slot will not be set.

and this was it. I've updated list of fields in "required_fields" method from my custom FormAction class to contain a field with custom "validate" method and it works perfectly.

Oops, I've missed that. Sorry :)
Is it better for you to receive wrong issue like that or not receive it at the cost of not getting possibly valuable feedback? I'm wondering because I have another idea in my mind and I'm not sure if I should start a new issue for that.
Please close this thread.

Of course, if you have improvement suggestions/other questions you're always welcome to open a new issue!


CA + US tax updates

Docs Build status updates of commit 90d1429:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamics-gp/payroll/CAN18YETax.md
:warning:Warning
View
Details

dynamics-gp/payroll/US19JanTaxR1.md
:white_check_mark:Succeeded
View

dynamics-gp/payroll/CAN18YETax.md

Line 380, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'media/FED2022.JPG'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fix a bug in examples/shapes/shapes_following_eyes

I found this bug and a trivial fix in the shapes_following_eyes example.

@casavaca Nice! Thanks for the review!


Root path resolved incorrectly in imported files?

1. What version of Caddy are you running (caddy -version)?
0.9.0
2. What are you trying to do?
I have the following setup:
/etc/caddy/Caddyfile
import /usr/share/project-x/Caddyfile

/usr/share/project-x/Caddyfile
example.com {
    root ./build
}

ls -l /usr/share/project-x
drwxrwxr-x 4 web web 4096 Jul 30 11:51 build/
-rw-rw-r-- 1 web web  123 Jul 30 12:07 Caddyfile
-rw-rw-r-- 1 web web  497 Jul 30 11:51 config.json
drwxrwxr-x 6 web web 4096 Jul 30 11:51 contents/
-rw-rw-r-- 1 web web  121 Jul 30 11:51 package.json
drwxrwxr-x 2 web web 4096 Jul 30 11:51 plugins/
-rw-rw-r-- 1 web web   86 Jul 30 11:51 readme.md
drwxrwxr-x 2 web web 4096 Jul 30 11:51 templates/

4.
With upstart script
5. What did you expect to see?
I expect the root to resolve to the ./build folder
6. What did you see instead (give full error messages and/or log)?
 [WARNING] Root path does not exist: ./build

I think this is consistent with doc of directive root : A relative root path is relative to the current working directory.

Should root be relative to the Caddyfile path instead?
Main problem with that is if the Caddyfile is loaded some other way than from a file on disk; plugins make it possible to load Caddyfile over the network, for example. Or in the near future, an API request can POST a new Caddyfile that doesn't have a location on disk, only in memory.


Ability to adopt existing .apps as Cask artifacts

Provide a detailed description of the proposed feature
A flag to brew install --cask (say --adopt) that skips the artifact move / link steps when the target already exists and has the same checksum as the source (or doesn't; see alternatives considered).
To illustrate, if "there is already an X at '/path/to/target'"
$ brew install textmate
==> Downloading https://github.com/textmate/textmate/releases/download/v2.0.23/T
Already downloaded: /Users/ian/Library/Caches/Homebrew/downloads/51ffb61879f6a076ae32e92cd9363820ee0d909e9b03015448bc0d0728970273--TextMate_2.0.23.tbz
==> Installing Cask textmate
==> Purging files for version 2.0.23 of Cask textmate
Error: It seems there is already an App at '/Applications/TextMate.app'.
$

users currently have two options:

Use the artifact that's already installed, without it being managed by Homebrew
Use --force to overwrite the installed binary, potentially changing it to some different version

This feature would add a third option:

Use --adopt (or whatever) to have Homebrew treat the existing artifact as if it was installed from a cask

What is the motivation for the feature?
This would be a useful option to migrate an existing system to use casks, reaping the associated benefits:

Ability use to brew upgrade for apps that don't auto-update
Ability to use brew uninstall for easy or mass uninstallation
Ability to use brew uninstall --zap to nuke all files
Inclusion in brew bundle dump
etc.

without risking installing a different artifact by accident, with --force.
<selfish reason, ignore if uninterested>
My Homebrew GUI client would also benefit, as there would be less confusion about apps installed from the web showing as "Available" but then failing to install. If the failure were instead an --adopt, all of the benefits listed above involving the command line would apply inside the GUI, too.
</selfish reason, ignore if uninterested>
How will the feature be relevant to at least 90% of Homebrew users?
Every Homebrew user has installed apps manually from the web before.
It's highly likely that many of these apps exists as casks, but were not installed that way. Once users learn about casks, I believe many would be willing to use them manage all of their apps. However, there is currently no easy way to migrate without passing scary --force flags or manually trashing existing .apps.
What alternatives to the feature have been considered?

Add --adopt, but don't have it verify the checksum.

This would allow the "installed version" reported by brew info to not match the .app's real version; however, cask auto-updates already do this, so perhaps this is a moot point.
If the above point is not seen as a big deal, I would actually prefer this behaviour.

<bikeshedding> Call it --adopt-existing or something else. </bikeshedding>

I'd be willing to attempt a PR for this if sentiment is positive.

I'd be willing to attempt a PR for this if sentiment is positive.

Yeh, I'd be game for this as long as it verified the nested file contents and checksums match.
My only fear is that this will end up being pretty slow if you're gonna do a comparison between two big .app. May be better with a recursive (fail fast) diff rather than checksum validation on every file.

I'm in the middle of implementing this, and have realized the combination of --adopt and --force is somewhat ambiguous. It could:

Ignore the result of the diff, and just pretend the source and target are the same
Overwrite the target, as if --adopt wasn't even given
Just raise an error, leaving a possible future extension point

Any thoughts? I personally like option 1 again, but I can also think of reasonable arguments for options 2 and 3.

Personally, I like option 1. Adding --adopt shows that the user wants something different from just --force so I think it's fine to have it function differently in that case.

Closed by #14033. Thanks @ThatsJustCheesy!


Click here issue in index.md

Describe the bug
Click Here in index.md hasn't been assigned any link.
Expected behavior
After clicking it should redirect us to a different page.
How to fix
Either we can remove the click here or we can just attach a link to it.

Fixed:
https://github.com/trywilco/wilco-docs/pull/32
Thanks @sahajkedia !


fix: add space between value and unit symbol for file size formatting

As specified by ISO 31-0. ;-)

Hm, one could make the argument that number and unit form a token that is easier to process as terminal text. E.g., copy it by double-clicking.
Also cf.
# ls -lh | head -n 1
total 2.2G
df -h | head -n 2
Filesystem                              Size  Used Avail Use% Mounted on
udev                                     32G     0   32G   0% /dev

Further evidenc, Python's tqdm prints by default 4.38s/it

Good point, but what if I only want to copy the number part? Also, there seems to be another discussion ongoing: https://github.com/tqdm/tqdm/issues/825

If you think it's actually an improvement then please go ahead and merge it. It's not worth the time discussing/thinking about.

;-)

This is mainly a nitpick of mine which I probably would've ignored if it weren't Friday afternoon. Let's wait a bit to see if there are more comments on this pressing issue. :-)


Upgrade option - unable to resolve host

FYI, tried the script using the upgrade option and it was unable to complete the download. Let me know if you'd like me to test anything! Thanks.
xxx@xxx:/config/scripts/post-config.d$ ./wireguard.sh upgrade Upgrading WireGuard... Checking latest release for v2.0-e300... Downloading null... curl: (6) Could not resolve host: null xxx@xxx:/config/scripts/post-config.d$

It looks like you're using an old version of the script, you could try running self-update and then trying the upgrade again.

DOH! Sorry, was setting up a new pi last night at the same time and lost track of what I'd done, in which window. That's exactly it, operator error. Updated script and then updated Wireguard, and everything works as expected. As they say, move along, nothing to see here... ;-)


Fix #2393, always set alignment of cuda static memory regions

to the natural alignment of the underlying type.
See explanation of the problem at https://github.com/numba/numba/issues/2393#issuecomment-305894825

Codecov Report

Merging #2397 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2397   +/-   ##
=======================================
  Coverage   86.25%   86.25%
=======================================
  Files         315      315
  Lines       60004    60004
  Branches     6518     6518
=======================================
  Hits        51756    51756
  Misses       7199     7199
  Partials     1049     1049


Transferring ownership

Hi @silverwind. I initiated the transfer already to your account.
Regarding npm, I am not sure how to do it. Tried npm adduser but cancelled it midway because I got confused with the questions/prompt. If you know how to, can you guide me please so I can transfer the npm ownership as well. Thanks.

Hmm, I don't see any indication of the move. IIRC, the recpient shouldn't have to confirm anything, the repo will just move after you typed its name. If it doesn't work out, I can always just fork and push to new repositories, no issue.
As for NPM, I suggest simply adding me as a collaborator on the web interface, and if all goes well, you can probably "leave" afterwards (if you like, but I wouldn't mind if you stayed).

That is strange. In the repo settings, I'm seeing status "This repository is being transferred to @silverwind".
I added you as collaborator just now. Maybe that will do the trick?

Just added you as collaborator on npm.

Thanks. Do you want to transfer the Github repos to me, so the issues/prs are not lost? You can do that in the repo settings with the Transfer button. Otherwise, I'll recreate new ones.

Have you received the notification about the transfer? It still says in progress.
I'd like to keep the issues/pr intact, but that shouldnt delete those when transferring ownership? Also, I dont see an option for that in the settings.

Ah, found two mails in my inbox for it that I missed earlier, but the links in them have expired. Can you restart?

Here it is:
https://help.github.com/articles/about-repository-transfers/
It says All of the issues and pull requests remain intact when transferring a repository".

I aborted the transfer and did a retry. Please check your email inbox.

That worked, thanks. And sorry for missing that earlier mail!
https://github.com/silverwind/cidr-regex
https://github.com/silverwind/is-cidr


Completion at bad position

TypeScript Version:  2.2.2-insiders.20170317
Code
class Foo {
    append() {
    }
    private c<cursor>
}

Expected behavior:

remove <cursor> and place your cursor there
trigger completions
:bug: global symbols like clearInterval are suggested, I'd expect no suggestions

fyi @mjbvz

should be fixed in latest.


Fix issue with links to already translated into Korean

Closes #23828
Update links to en docs which were already translated into Korean in the following files.

docs/concepts/configuration/configmap.md
docs/concepts/configuration/overview.md
docs/concepts/extend-kubernetes/api-extension/apiserver-aggregation.md
docs/concepts/extend-kubernetes/api-extension/custom-resources.md
docs/concepts/overview/kubernetes-api.md
docs/concepts/security/overview.md
docs/concepts/services-networking/connect-applications-service.md
docs/concepts/storage/volumes.md
docs/concepts/workloads/controllers/job.md
docs/concepts/workloads/pods/_index.md
docs/concepts/workloads/pods/pod-lifecycle.md
docs/contribute/participate/pr-wranglers.md
docs/reference/command-line-tools-reference/feature-gates.md
docs/setup/best-practices/certificates.md
docs/tasks/access-application-cluster/web-ui-dashboard.md
docs/tasks/configure-pod-container/pull-image-private-registry.md
docs/tasks/inject-data-application/define-command-argument-container.md
docs/tasks/run-application/run-single-instance-stateful-application.md
docs/tasks/tools/_index.md
docs/tasks/tools/install-kubectl.md
docs/tutorials/stateful-application/mysql-wordpress-persistent-volume.md

/assign @seokho-son

#23835 브랜치 일정 참고하시기 바랍니다. ^^

감사합니다!
/lgtm
/approve


viber: fixed sha256 mismatch

Motivation for this change
trying https://download.cdn.viber.com/cdn/desktop/Linux/viber.deb
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 95.0M  100 95.0M    0     0  10.7M      0  0:00:08  0:00:08 --:--:-- 11.1M
hash mismatch in fixed-output derivation '/nix/store/gb3nafqslnmmrz3853wxll4k4vdjj68b-viber.deb':
  wanted: sha256:06mp2wvqx4y6rd5gs2mh442qcykjrrvwnkhlpx0lara331i2p0lj
  got:    sha256:0rs26x0lycavybn6k1hbb5kzms0zzcmxlrmi4g8k7vyafj6s8dqh
cannot build derivation '/nix/store/5hlb0lc3bqg9zq2c5pwff4fyih67f8ip-viber-<IP_ADDRESS>5.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/j6l7p7x7azmk1ayf7j102jgv51iwl9zi-system-path.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/3rdwp8765ck93vwblwqcql73a1lbpaa8-nixos-system-mac-21.03pre253060.a322b32e9d7.drv': 1 dependencies couldn't be built
error: build of '/nix/store/3rdwp8765ck93vwblwqcql73a1lbpaa8-nixos-system-mac-21.03pre253060.a322b32e9d7.drv' failed

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

❯ nixpkgs-review pr 104880
$ git -c fetch.prune=false fetch --force https://github.com/NixOS/nixpkgs nixos-unstable:refs/nixpkgs-review/0 pull/104880/head:refs/nixpkgs-review/1
From https://github.com/NixOS/nixpkgs
 * [new branch]              nixos-unstable        -> refs/nixpkgs-review/0
 * [new ref]                 refs/pull/104880/head -> refs/nixpkgs-review/1
$ git worktree add /home/sevenfourk/.cache/nixpkgs-review/pr-104880/nixpkgs 2247d824fe07f16325596acc7faa286502faffd1
Preparing worktree (detached HEAD 2247d824fe0)
Updating files: 100% (23269/23269), done.
HEAD is now at 2247d824fe0 Merge pull request #104479 from Mic92/freeoffice
$ nix-env -f /home/sevenfourk/.cache/nixpkgs-review/pr-104880/nixpkgs -qaP --xml --out-path --show-trace
$ git merge --no-commit 9173b3e67ee1ed0516456db43af797e7c3f8ce87
Updating 2247d824fe0..9173b3e67ee
Fast-forward
 pkgs/applications/networking/instant-messengers/viber/default.nix | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
$ nix-env -f /home/sevenfourk/.cache/nixpkgs-review/pr-104880/nixpkgs -qaP --xml --out-path --show-trace --meta
1 package updated:
viber

$ nix build --no-link --keep-going --option build-use-sandbox relaxed -f /home/sevenfourk/.cache/nixpkgs-review/pr-104880/build.nix
warning: ignoring the user-specified setting 'sandbox', because it is a restricted setting and you are not a trusted user
[2 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/104880
1 package built:
viber

[0.0 MiB DL]
$ nix-shell /home/sevenfourk/.cache/nixpkgs-review/pr-104880/shell.nix
these paths will be fetched (0.05 MiB download, 0.28 MiB unpacked):
  /nix/store/fp7y3g7nwsy3hp0xlgmbh2rigz1ksjss-bash-interactive-4.4-p23-dev
copying path '/nix/store/fp7y3g7nwsy3hp0xlgmbh2rigz1ksjss-bash-interactive-4.4-p23-dev' from 'https://cache.nixos.org'...

[nix-shell:~/.cache/nixpkgs-review/pr-104880]$

running in pr nix-shell causes this:
Got keys from plugin meta data ("xcb")
QFactoryLoader::QFactoryLoader() checking directory path "/nix/store/vfynhq88b11b1lgz8imchykvcv4cli0q-viber-<IP_ADDRESS>5/opt/viber/platforms" ...
Cannot load library /nix/store/vfynhq88b11b1lgz8imchykvcv4cli0q-viber-<IP_ADDRESS>5/opt/viber/plugins/platforms/libqxcb.so: (libxkbcommon-x11.so.0: cannot open sh
ared object file: No such file or directory)
QLibraryPrivate::loadPlugin failed on "/nix/store/vfynhq88b11b1lgz8imchykvcv4cli0q-viber-<IP_ADDRESS>5/opt/viber/plugins/platforms/libqxcb.so" : "Cannot load libr
ary /nix/store/vfynhq88b11b1lgz8imchykvcv4cli0q-viber-<IP_ADDRESS>5/opt/viber/plugins/platforms/libqxcb.so: (libxkbcommon-x11.so.0: cannot open shared object file
: No such file or directory)"
qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Available platform plugins are: eglfs, linuxfb, minimal, minimalegl, offscreen, vnc, wayland-egl, wayland, wayland-xcomposite-egl, wayland-xcomposite-glx, webg
l, xcb.

Aborted (core dumped)

now it runs fine:
[nix-shell:~/.cache/nixpkgs-review/pr-104880-3]$ viber
qml: CALL_WEB_VIEW_DEBUG: callWebEngineLoader active: false
qrc:/QML/DebugMenu.qml:248: TypeError: Cannot call method 'isIdentityChangeNotificationsEnabled' of undefined
...
qrc:/QML/Feed/FeedView.qml:108:5: QML ListViewEx: Binding loop detected for property "bottomMargin"
^C

I changed the target branch to master.

@SuperSandro2000 thanks, getting used to it.

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

1 package built:

viber


Numbers.primes returns odd numbers, not primes

The logic for primes in Numbers.primes is incorrect, and is instead for odd numbers:
nums = array.array('L', (i for i in range(start, end) if i % 2))
return nums.tolist() if to_list else nums

The docstring for it is also incorrect. I'll send a PR with the fixes soon.

@faheel Thank you for find this bug. I'll wait for your PR.


NDK crash when starting Geth

https://sentry.io/organizations/celo/issues/1123965781/?project=1250733&query=is%3Aunresolved
https://sentry.io/organizations/celo/issues/1123977955/?project=1250733&query=is%3Aunresolved
https://sentry.io/organizations/celo/issues/1123977939/?project=1250733&query=is%3Aunresolved

All the three links posted by @jmrossy are for Pixel 2 (ARM64) device. So, something ARM64 specific is happening in this case as well.

@jmrossy is this still a real issue? can we close?


Update Protocol Versions for Cipher Suites

When TLS 1.1 and TLS 1.2 support was added to Vista / Server 2008 SP2 we did not update the documentation that specifies which protocol versions can be used for each cipher suite.

@poloplayerrl : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Ask for docker image prefix

It currently use the project name as prefix for docker image, but it should ask another question, something like "docker image prefix" or so.

DOCKER_DEVBOX_REGISTRY_REPOSITORY environment is now available for this purpose


Idea: allow transition to abstract state when they have a redirectTo property

I don't have a problem with this conceptually.  As long as it doesn't cause .  I think abstract states overall are confusing to a lot of users.  The term abstract makes people assume all sorts of incorrect things about what abstract states do.

Yeah. I personaly usualy use this feature for "parent" states that doesn't need to have a dedicated view. I think this is a real use case.
Eg: /admin beeing an abstract (i don't want an admin menu page on it's own), and /admin/users or /admin/stuffs actual routes. Though, I sometimes want to resirect to /admin/stuffs by defaut when I click on /admin


Global JSX to local JSX w/ generics

Initial checklist

[x] I read the support docs
[x] I read the contributing guide
[x] I agree to follow the code of conduct
[x] I searched issues and couldn’t find anything (or linked relevant results below)
[x] If applicable, I’ve added docs and tests

Description of changes
Cleaner but completely broken version of what GH-6 attempts.
This does not remove functionality (components).
But because TS limitations (and likely that jsx types are not very good), we can’t infer some IntrinsicElements registry from a jsx function or an Element type.

Closing this, I think it’s better for folks to define JSX: https://github.com/syntax-tree/hast-util-to-jsx-runtime/commit/ae638ba52762906fe094d74f82ba835ce23a2621. Similarly to how we expect a runtime to be passed in, it makes sentence that types are passed in too. And while TS doesn’t have a way to pass namespaces, a global seems good. Especially a global that is in use by millions of code examples out there.


init commit

Description
This PR adds the initial implementation of dap-secret-webhook which seeks to integrate Flyte Secret Manager with MLP as the secret storage.
This webhook replaces the Flyte default webhook server and does not support any other form of Secret Manager to be used in conjuction.
Under the hood
For a Flyte Pod with secret injected, it will have the following label and secrets in annotation
Labels:           inject-flyte-secrets=true
Annotations:      flyte.secrets/s0:....

Flyte Webhook extracts the secrets from the annotation and mount the secrets as env_var to the container with a predefined format. This is automatically loaded into FlyteContext and allow usages with FlyteKit.
dap-secret-webhook keeps the interface intact but change the implementation of how the secret value is being populated. The secret is retrieved from mlp client, created as kubernetes secret and then mounted onto to pod. The secret is deleted upon pod deletion.
.
├── client                  # MLP CLient
├── cmd                     # Entrypoint
├── config                  # Configuration
├── test                    # Test data and mocks
├── webhook                 # Webhook Server
└── README.md

Testing
The e2e test in github will check for the mounted secret values and the correct secret generation.
Future improvement to be considered

Failure detections and handling (eg when a project does not have mlp secrets created but is requesting for it. Flyte will attempt to re-run the pod)
Automating creating self-signed cert programmatically and renewing existing one if it falls within a certain timeframe

It's also useful if we could add logs for important operation such as:
receiving create pod request (within Mutate method)
receiving delete pod request (within Mutate method)
inject secret to a pod (within mutatePodAndCreateSecret )
creating secret (within mutatePodAndCreateSecret )
deleting secret (within deleteK8Secret )
These logs will facilitate easier debugging.

I thought about doing it but was afraid that it will spam the logs since there can be many tasks running. No opinion on this.


New-PartnerAccessToken command executed in parallel for multiple tenants from C# code returns multiple tokens for one tenant and empty PSObjects for others

When the New-PartnerAccessToken command is executed in parallel for multiple tenants from C# code, one of the responses contains multiple tokens, while empty PSObjects are returned for the other tenants. There are no execution errors.
To demonstrate this behaviour, I have written a simple script (using .NET7) that runs multiple New-PartnerAccessToken commands simultaneously in separate runspaces.
It's important to note that the results of the execution can vary - sometimes a single token is returned in the response for a tenant, sometimes multiple tokens are returned, one of which is issued for another tenant. However, the results for the other tenants are always empty. I have attached a screenshot where the results dict contains 2 tokens resolved for one tenant, while for other tenants empty results have been received.
using System.Management.Automation;
using System.Management.Automation.Runspaces;
using System.Security;
using Microsoft.PowerShell;

var tenants = new[]
{
    "XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXX1",
    "XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXX2",
    "XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXX3"
};

var task1 = new ExoAccessTokenGenerator().GetExoAccessTokensAsync(tenants);
Task.WaitAll(task1);

var results = task1.Result;
foreach (var res in results)
{
    Console.WriteLine($"Number of access tokens for {res.Key}: {res.Value.ToArray().Length}");
}
Console.ReadKey();

public class ExoAccessTokenGenerator
{
    public async Task<IDictionary<string, IEnumerable<PSObject>>> GetExoAccessTokensAsync(IEnumerable<string> tenantIds)
    {
        var accessTokens = new Dictionary<string, IEnumerable<PSObject>>();
        var tasks = new List<Task>();

        foreach (var tenantId in tenantIds)
        {
            var task = Task.Run(() =>

            {
                var accessToken = GetExoAccessToken(tenantId);
                accessTokens.Add(tenantId, accessToken);
            });

            tasks.Add(task);
        }

        await Task.WhenAll(tasks);

        return accessTokens;
    }

    private IEnumerable<PSObject> GetExoAccessToken(string tenantId)
    {
        var appId = "XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX";
        var appSecret = "";
        var refreshToken = "";
        var scopes = "https://outlook.office365.com/.default";
        var credential = new PSCredential(appId, ToSecureString(appSecret));

        var exoTokenCommand = new Command("New-PartnerAccessToken");
        exoTokenCommand.Parameters.Add("RefreshToken", refreshToken);
        exoTokenCommand.Parameters.Add("Credential", credential);
        exoTokenCommand.Parameters.Add("Scopes", scopes);
        exoTokenCommand.Parameters.Add("ApplicationId", appId);
        exoTokenCommand.Parameters.Add("ServicePrincipal", true);
        exoTokenCommand.Parameters.Add("Tenant", tenantId);

        using (var powerShell = PowerShell.Create())
        {
            InitialSessionState sessionState = InitialSessionState.CreateDefault();
            sessionState.ExecutionPolicy = ExecutionPolicy.Unrestricted;
            sessionState.ThrowOnRunspaceOpenError = true;
            var runspace = RunspaceFactory.CreateRunspace(sessionState);
            runspace.Open();

            powerShell.Runspace = runspace;

            powerShell.Commands.AddCommand(exoTokenCommand);

            var result = powerShell.Invoke();

            if (powerShell.HadErrors)
            {
                var errors = string.Join(Environment.NewLine, powerShell.Streams.Error);
                throw new Exception($"Failed to execute PowerShell command. Errors: {errors}");
            }

            if (result.Count == 0)
            {
                Console.WriteLine($"Token has not been received for tenant {tenantId}");
            }

            return result;
        }
    }

    private static SecureString ToSecureString(string str)
    {
        var secureString = new SecureString();

        foreach (var c in str)
        {
            secureString.AppendChar(c);
        }

        return secureString;
    }
}

Environment
PSVersion  5.1
PartnerCenter 3.0.10
Microsoft.PowerShell.SDK 7.3.3
Microsoft.IdentityModel.Token 6.27.0
The issue is reproducable on both .NET 4.8 and .NET 7

Thank you for the explanation!
Can we use the same script for .Net Core 6.0?
Also can you please help us with the nudget packages that needs to be installed.
I am getting the error "The getter method should be public, not void, static, and have one parameter of the type PSObject."
when executing the script using .Net core 6. Can you please help

Hello again,
I have somehow managed to resolve the error "The getter method should be public, not void, static, and have one parameter of the type PSObject.".
But now I get a new error upon executing the powerhell.invoke() command -:
"The term 'New-PartnerAccessToken' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again"
Can you please help.

Hello, I have the same problem using this code with The term 'New-PartnerAccessToken' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, check that the path is correct and try again'
Did you have a solution?
Thank you in advance

I am looking to reproduce in c# this process in powershell
TenantID = "xxxxxxxxxxxxxxxxxxx" #  Tenant
$ApplicationID = "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx" # Application in Tenant "Partner Center"
$ApplictionSecret = "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
$credential = New-Object PSCredential ($ApplicationID, (ConvertTo-SecureString $ApplictionSecret -AsPlainText -Force))
$token = New-PartnerAccessToken -ApplicationId $ApplicationID -Scopes 'https://api.partnercenter.microsoft.com/user_impersonation' -ServicePrincipal -Credential $credential -Tenant $TenantID -UseAuthorizationCode
$token | clip
. Someone help me a little?


AttributeError: module 'tensorflow.compat.v2.internal.distribute' has no attribute 'strategy_supports_no_merge_call'

Hello.
I am importing the following libraries while running the example that comes as the basic text classification  in the official Tensorflow documentation. But I get an error message. My version of tensorflow is 2.7.0
Running code for which the error occurred:
epochs = 10
history = model.fit( train_ds, validation_data=val_ds, epochs=epochs)

Libraries loads:
import matplotlib.pyplot as plt
import os
import re
import shutil
import string
import tensorflow as tf
from tensorflow.keras import layers
from tensorflow.keras import losses

Error message:
AttributeError: in user code:

    File "C:\Users\Octavio\anaconda3\lib\site-packages\keras\engine\training.py", line 1525, in test_function  *
        return step_function(self, iterator)
    File "C:\Users\Octavio\anaconda3\lib\site-packages\keras\engine\training.py", line 1514, in step_function  **
        outputs = model.distribute_strategy.run(run_step, args=(data,))
    File "C:\Users\Octavio\anaconda3\lib\site-packages\keras\engine\training.py", line 1507, in run_step  **
        outputs = model.test_step(data)
    File "C:\Users\Octavio\anaconda3\lib\site-packages\keras\engine\training.py", line 1474, in test_step
        return self.compute_metrics(x, y, y_pred, sample_weight)
    File "C:\Users\Octavio\anaconda3\lib\site-packages\keras\engine\training.py", line 961, in compute_metrics
        result = metric.result()
    File "C:\Users\Octavio\anaconda3\lib\site-packages\keras\utils\metrics_utils.py", line 124, in decorated
        tf.__internal__.distribute.strategy_supports_no_merge_call()):

Thank you

Hi @Fer020707 ! I was able to run above example  in Colab with 2.7 version too. Could you please share a minimal standalone code along other details through template to reproduce this issue ?Attaching relevant thread for reference. Thanks!

Of course. This would be the minimum code to reproduce the error. Will it be an installation aspect?
Imported libraries:
import matplotlib.pyplot as plt
import os
import re
import shutil
import string
import tensorflow as tf

from tensorflow.keras import layers
from tensorflow.keras import losses

Data upload
url = "https://ai.stanford.edu/~amaas/data/sentiment/aclImdb_v1.tar.gz"

dataset = tf.keras.utils.get_file("aclImdb_v1", url,
                                    untar=True, cache_dir='.',
                                    cache_subdir='')
dataset_dir = os.path.join(os.path.dirname(dataset), 'aclImdb')
train_dir = os.path.join(dataset_dir, 'train')
remove_dir = os.path.join(train_dir, 'unsup')
shutil.rmtree(remove_dir)

Data work
batch_size = 32
seed = 42
raw_train_ds = tf.keras.utils.text_dataset_from_directory(
    'aclImdb/train',
    batch_size=batch_size,
    validation_split=0.2,
    subset='training',
    seed=seed)
raw_val_ds = tf.keras.utils.text_dataset_from_directory(
    'aclImdb/train',
    batch_size=batch_size,
    validation_split=0.2,
    subset='validation',
    seed=seed)
raw_test_ds = tf.keras.utils.text_dataset_from_directory(
    'aclImdb/test',
    batch_size=batch_size)
def custom_standardization(input_data):
  lowercase = tf.strings.lower(input_data)
  stripped_html = tf.strings.regex_replace(lowercase, '<br />', ' ')
  return tf.strings.regex_replace(stripped_html,
                                  '[%s]' % re.escape(string.punctuation),
                                  '')
def vectorize_text(text, label):
  text = tf.expand_dims(text, -1)
  return vectorize_layer(text), label

max_features = 10000
sequence_length = 250

vectorize_layer = layers.TextVectorization(
    standardize=custom_standardization,
    max_tokens=max_features,
    output_mode='int',
    output_sequence_length=sequence_length)
train_text = raw_train_ds.map(lambda x, y: x)
vectorize_layer.adapt(train_text)
text_batch, label_batch = next(iter(raw_train_ds))
first_review, first_label = text_batch[0], label_batch[0]
train_ds = raw_train_ds.map(vectorize_text)
val_ds = raw_val_ds.map(vectorize_text)
test_ds = raw_test_ds.map(vectorize_text)
AUTOTUNE = tf.data.AUTOTUNE

train_ds = train_ds.cache().prefetch(buffer_size=AUTOTUNE)
val_ds = val_ds.cache().prefetch(buffer_size=AUTOTUNE)
test_ds = test_ds.cache().prefetch(buffer_size=AUTOTUNE)
embedding_dim = 16
model = tf.keras.Sequential([
  layers.Embedding(max_features + 1, embedding_dim),
  layers.Dropout(0.2),
  layers.GlobalAveragePooling1D(),
  layers.Dropout(0.2),
  layers.Dense(1)])

model.summary()
model.compile(loss=losses.BinaryCrossentropy(from_logits=True),
              optimizer='adam',
              metrics=tf.metrics.BinaryAccuracy(threshold=0.0))
epochs = 10
history = model.fit(
    train_ds,
    validation_data=val_ds,
    epochs=epochs)

Ok @Fer020707 !  it might be related to environment/installation perspective. Could not replicate this in Colab again. Attaching Gist and relevant thread for reference.
Please post this issue on keras-team/keras repo if it does not get resolved.
To know more see;
https://discuss.tensorflow.org/t/keras-project-moved-to-new-repository-in-https-github-com-keras-team-keras/1999 . Thanks you!

@Fer020707 I was able to find the source of this issue. When installing tensorflow 2.7.0, keras 2.8.0 was installed. I solved this issue by pinning keras to 2.7.0 in my pyproject.toml

Ok @Fer020707 ! Thanks for pointing out the solution . we will let the team know about it. Thank you!

Thank you very much. By correcting that, it worked perfect

Ok @Fer020707 ! Closing this issue as it seems to be resolved from above comment . Thank you!

Similar error on MacOS (ARM) with tensorflow-macos. After an pip -U ... I downgraded keras and tensorflow-estimator to 2.7.0.
When fitting the model, the following error occurs:
model.fit(x_train, y_train, epochs=5)

Epoch 1[/5]

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
/Users/wolf/Development/py/5 ml/[1]() prerequisites/07 tensorflow/01_quickstart.ipynb Cell 18' in <module>
----> 1 model.fit(x_train, y_train, epochs=5)

File ~/Development/py/5 ml/venv/lib/python3.9/site-packages/keras/utils/traceback_utils.py:67, in filter_traceback.<locals>.error_handler(*args, **kwargs)
         65 except Exception as e:  # pylint: disable=broad-except
         66   filtered_tb = _process_traceback_frames(e.__traceback__)
---> 67   raise e.with_traceback(filtered_tb) from None
         68 finally:
         69   del filtered_tb

File ~/Development/py/5 ml/venv/lib/python3.9/site-packages/tensorflow/python/framework/func_graph.py:1129, in func_graph_from_py_func.<locals>.autograph_handler(*args, **kwargs)
   1127 except Exception as e:  # pylint:disable=broad-except
   1128   if hasattr(e, "ag_error_metadata"):
-> 1129     raise e.ag_error_metadata.to_exception(e)
   1130   else:
   1131     raise

AttributeError: in user code:

    File "/Users/wolf/Development/py/5 ml/venv/lib/python3.9/site-packages/keras/engine/training.py", line 1021, in train_function  *
        return step_function(self, iterator)
    File "/Users/wolf/Development/py/5 ml/venv/lib/python3.9/site-packages/keras/engine/training.py", line 1010, in step_function  **
        - A `tf.data.Dataset`.
    File "/Users/wolf/Development/py/5 ml/venv/lib/python3.9/site-packages/keras/engine/training.py", line 1000, in run_step  **
        validation_data: Data on which to evaluate
    File "/Users/wolf/Development/py/5 ml/venv/lib/python3.9/site-packages/keras/engine/training.py", line 863, in train_step
        model._train_counter.assign_add(1)  # pylint: disable=protected-access
    File "/Users/wolf/Development/py/5 ml/venv/lib/python3.9/site-packages/keras/optimizer_v2/optimizer_v2.py", line 532, in minimize
        return self.apply_gradients(grads_and_vars, name=name)
    File "/Users/wolf/Development/py/5 ml/venv/lib/python3.9/site-packages/keras/optimizer_v2/optimizer_v2.py", line 668, in apply_gradients
        grads_and_vars = self._aggregate_gradients(grads_and_vars)
    File "/Users/wolf/Development/py/5 ml/venv/lib/python3.9/site-packages/keras/optimizer_v2/optimizer_v2.py", line 484, in _aggregate_gradients
        return self.gradient_aggregator(grads_and_vars)
    File "/Users/wolf/Development/py/5 ml/venv/lib/python3.9/site-packages/keras/optimizer_v2/utils.py", line 33, in all_reduce_sum_gradients
        if strategy_supports_no_merge_call():

    AttributeError: module 'tensorflow.compat.v2.__internal__.distribute' has no attribute 'strategy_supports_no_merge_call'

This worked before the update


[Bug]:

Before you file a bug report

[X] I have checked the issue tracker and have not found an issue that matches the one I'm filing.
[X] This issue is not a troubleshooting question. Troubleshooting questions go here: https://forum.mattermost.com/c/trouble-shoot/16.
[X] This issue is not a feature request. You can request features and make product suggestions here: https://mattermost.com/suggestions/.
[X] This issue doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[X] This issue reproduces on the most recent stable version, or the most recent prerelease version of the Mattermost Desktop App.
[X] I have read the contribution guidelines.

Mattermost Desktop Version
5.5.0
Operating System
Windows 10 x64
Mattermost Server Version
8.1.0
Steps to reproduce
Set the Desktop client window so narrow that the Channels part from the left disappears/collapses. When you do that, the responsive mode is activated.
Expected behavior
The call icon should be on the top of the backgroun circle
Observed behavior
As a result the Start call -link moves to the top as a icon BUT now it is only a button without an icon (only the background circle no icon).
Log Output
There is nothing in the logs or in diagnostics file regarding this issue.

Additional Information
We have tested this in several client and client themes.

Transferring to main repo as this is an issue with the web app's mobile view


Video Conference Mute freezing

Microsoft PowerToys version
0.67.1
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
Video Conference Mute
Steps to reproduce
PowerToysReport_2023-02-28-11-25-07.zip
Steps to reproduce:

Enable Video Conference Mute on an audio device so that the prompt looks like this:

Disconnect said device from system without unmuting

Prompt will be frozen in place and VCM will no longer work until Powertoys is outright restarted.

✔ Expected Behavior
VCM to still function properly after one device is disconnected while in use.
❌ Actual Behavior
Complete seizure of the application preventing the function from being used again before restarting the program completely.
Other Software
No response

VCM is in maintenance mode and we will only resolve a small amount of issues in it moving forward, please see #21473


add function access keys to additional contracts by using multiple key stores

Experimenting and work in progress on what's discussed here: https://github.com/near/neardevhub-widgets/issues/137
Currently implementing a test case that demonstrates adding additional function access keys for another contract that the one that was originally signed in for. This is useful when logged into near.social and posting to other contracts like e.g. DevHub in order to avoid having to confirm with the wallet each time.

Thanks for taking the time to submit a pull request!
If you haven't already, be sure to take a look at the CONTRIBUTING documentation to get up to speed with the practices in this repository.
When you're ready, switch to the "Preview" tab to select an applicable template:

General template (most common)
Wallet integration template

Hi, this PR seems to be related to the signInMulti PR https://github.com/near/wallet-selector/pull/811 I left a comment https://github.com/near/wallet-selector/pull/811#issuecomment-1683581762 to explain the current status of the signInMulti in wallet-selector.

@petersalomonsen I think this PR is irrelevant at this point since https://github.com/NearSocial/VM/pull/156, right? Should we close it?

@petersalomonsen I think this PR is irrelevant at this point since NearSocial/VM#156, right? Should we close it?

Yes let's close it. And to address the issue of providing the standard login flow for all wallet types we have this issue here: https://github.com/near/wallet-selector/issues/1045


[FLINK-31407] Bump fabric8 version to 6.5.0

This new version brings critical stability fixes for the informers. I fixed the newly introduced deprecation warnings.

The changes broke our tests that rely on the KubernetesMockClient, e.g.:
Caused by: io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: PATCH at: https://localhost:54301/apis/flink.apache.org/v1beta1/namespaces/flink-operator-test/flinkdeployments/test-cluster?fieldManager=fabric8. Message: Not Found.
	at io.fabric8.kubernetes.client.dsl.internal.OperationSupport.requestFailure(OperationSupport.java:701)

I will come back to this a bit later, any pointers are welcome. 😏

I've just had a deeper look at the test issues and here are my findings:

The originally used createOrReplace has the following semantics

Create the resource with POST operation
If response is HTTP_CONFLICT (409) then replace the resource with PUT operation (replace operation is effectively a forced update with retries)

The createOrReplace doc says the following:/**
 * Creates a provided resource in a Kubernetes Cluster. If creation
 * fails with a HTTP_CONFLICT, it tries to replace resource.
 *
 * @return created item returned in kubernetes api response
 *
 * @deprecated please user {@link ServerSideApplicable#serverSideApply()} or attempt a create and edit/patch operation.
 */
@Deprecated
T createOrReplace();

The doc is cloudy what serverSideApply really does so had an in-depth look of the source code.
In case of custom resource the code is unconditionally patch the resource with PATCH operation

My general conclusion is that createOrReplace and serverSideApply are having totally different semantics if we consider the source code. My proposal is the following:

Use create when new resource is needed
Use update when existing resource need to be upgraded (since replace is deprecated too)
If it's super hard to decide whether create or update is needed then one can re-implement to original createOrReplace functionality but I discourage to do it

I've just had a deeper look at the test issues and here are my findings:

The originally used createOrReplace has the following semantics

Create the resource with POST operation
If response is HTTP_CONFLICT (409) then replace the resource with PUT operation (replace operation is effectively a forced update with retries)

The createOrReplace doc says the following:
/**
 * Creates a provided resource in a Kubernetes Cluster. If creation
 * fails with a HTTP_CONFLICT, it tries to replace resource.
 *
 * @return created item returned in kubernetes api response
 *
 * @deprecated please user {@link ServerSideApplicable#serverSideApply()} or attempt a create and edit/patch operation.
 */
@Deprecated
T createOrReplace();

The doc is cloudy what serverSideApply really does so had an in-depth look of the source code.

In case of custom resource the code is unconditionally patch the resource with PATCH operation

My general conclusion is that createOrReplace and serverSideApply are having totally different semantics if we consider the source code. My proposal is the following:

Use create when new resource is needed
Use update when existing resource need to be upgraded (since replace is deprecated too)
If it's super hard to decide whether create or update is needed then one can re-implement to original createOrReplace functionality but I discourage to do it

I agree with @gaborgsomogyi 's assessment. If we need we can keep the serverSideApply in some tests but we should not use it in the core logic.


proposal filterMap

We often have the use-case where we want to findAll elements with a certain class and then narrowing the search result to a specific subset. We use index to find the right element, but I think those tests tend to be pretty brittle. I think it's better to narrow the results with an actual selector. I chose filterMap as the name for this function, but I'm not super happy with it. I had findAllIn before, which I think is worse.
How do you like this?

don't know what's up with travis 🤔.

Also, regarding the name: I agree. What about keep?

The build was failing on travis due to the infamous caching issue. Cleaned caches and rebuilt

I like keep. would you be okay with renaming and taking one selector?

That looks good to me 👍 One thought - how often have you needed to do the opposite? e.g exclude a given selector from a collection?

One thought - how often have you needed to do the opposite? e.g exclude a given selector from a collection?

I didn't have to do that yet, but I can see a use-case for that. We could add drop in the future.

Let's just merge this for now and see how we go 👍

published with v5.1.0


Panic! in the Build System (Use new library for Travis runner)

This PR migrates our Travis test runner to use the new travistooling library. Bonus is more testing, more reliability, and we can start acting on the Travis-ECR notifications to improve build times by patching the new library.
This is largely driven by thinking around https://github.com/HypothesisWorks/hypothesis/issues/1224 – I’d really like to have us using travistooling and getting it working properly before I try to use it in another project!

Oh, and I ditched the .travis directory, because dotfiles being hidden on macOS was just making that mildly tedious.

Niiiiiice.

In general, changes to the Travis build scripts (travistooling, .travis, .travis.yml) probably shouldn’t trigger a full rebuild – but I’m going to save that for a separate patch, as I want to check we can still do a full build with this setup.


Facebook Share

Open graph metas need to be added to allow content to be displayed when sharing pages on Facebook.
Documentation located here:
https://developers.facebook.com/tools/debug/sharing/

Added hardcoded open graph meta tags.


Fix registry operator hostname format for k8s

Which area/kind this issue is related to?

/area registry
Issue Description

Currently, the registry operator hostname setup only uses the CR name value for the hostname part, for openshift deployments and how the helm chart sets up the hostname _template.tpl#L16, the CR name is coupled with the namespace name to create the hostname.
The registry operator source should be changed to something of the following:
GetDevfileRegistryIngress
func GetDevfileRegistryIngress(cr *registryv1alpha1.DevfileRegistry) string {
	return GetHostname(cr) + "." + cr.Spec.K8s.IngressDomain
}

GetHostname
func GetHostname(cr *registryv1alpha1.DevfileRegistry) string {
	return fmt.Sprintf("%s-%s", cr.Name, cr.Namespace)
}

Parent Epic: #1274
Acceptance Criteria

[x] Create getter function to receive the hostname part of the ingress address

Create hostname using the format <DevfileRegistry.Name>-<DevfileRegistry.Namespace>

[x] Update GetDevfileRegistryIngress to couple the result of the hostname getter function and the set ingress domain DevfileRegistry.Spec.K8s.IngressDomain
[ ] Ensure integration test cases are passing

Functions have been created and updated to reflect the changes. I added some unit tests for these changes and they seem to be working great.
Currently working on adding a few more cases to the unit tests and then need to integration test. The integration tests are giving me issues as the pods don't want to run properly. Working through that issue before opening a PR.


Check for client.abort function before executing.

line is now identical to the open() method, on #435 in the same file.

Can you provide a test case that fails when this check is not present?

Closing due to lack of response, but happy to reopen and merge if we can add a test for this.

This blows up for me in a test (Jest) with an url variable that was undefined (so: $.get(undefined).then(...)), but the same goes for any other falsy url values.
  ● Test suite failed to run
    TypeError: client.abort is not a function

      at XMLHttpRequest.abort (node_modules/jsdom/lib/jsdom/living/xmlhttprequest.js:369:16)
      at Object.abort (node_modules/jsdom/lib/jsdom/living/xhr-utils.js:315:13)
      at RequestManager.close (node_modules/jsdom/lib/jsdom/living/nodes/Document-impl.js:146:21)
      at Window.close (node_modules/jsdom/lib/jsdom/browser/Window.js:362:29)
          at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:169:7)

Firing up the debugger shows that client is an EventEmitter but does not have an abort fn. I did notice an abort fn under _events though.

Please open a new issue following the issue template; in particular pay attention to the instructions about a minimal example, not involving other libraries like Jest.


1.7.4

hello, after update v2b version to 1.7.4, Traffic records for TAs isn't working for me. pls help


Fix to attribute directive guide

Two small fixes to the attribute directive guide

remove extra asterisk
reword the section about prefixes to no longer mention hyphens

I signed it!

I like your suggestions.  I'm squashing to one commit.


Make sure compliance-trestle demos are running correctly - Creating an SSP using trestle author

Compliance trestle demos repo is outdated and needs refreshment and review.
It is needed to go through the Creating an SSP using trestle author review it runs correctly as the documentation specifies concretely for this one.
Compliance-trestle version used for the generation of the demos is too outdated now vs the current one, so as part of the revision, it should be ideal to run the demo with the latest version and see if anything has changed or needs refreshment.
If there is anything needed to be done there by the demo, please submit a PR against the repo to be reviewed and tracked.

Moving comments from @Ma1h01 from original repo:
The instructions in the README use odd commands. I made a pull request to correct them and refactor the demo directory.

Will provide feedback on the PR. Thanks


Provide JAM codec test vectors

Currently SCALE test vectors are provided but it is not that useful as we are not using SCALE codec in GP.
Instead, JAM codec one should be provided so we can test the codec part of the implementation.

If switch codec is trivial, can you just switch scale to codec to jam codec?
JSON serialization is not part of the protocol and requires extra work and jam codec is part of the protocol and requires no extra work.

If switch codec is trivial, can you just switch scale to codec to jam codec?

Offcourse we will, when codec implementation is ready. Once ready, switching codec is trivial.

JSON serialization is not part of the protocol and requires extra work and jam codec is part of the protocol and requires no extra work.

JSON has been mainained to offer a human readable version of the binary file for quick inspection. It is basically free to generate.

In that case can I request someone to prioritize the jam codec implementation and test vectors? Or let me know if it won't be done in next few weeks then in that case I will spend some hours to support the JSON format.

https://github.com/w3f/jamtestvectors/pull/12


Sync with system python broken in 0.4.13

For some use cases, I had been using UV_PROJECT_ENVIRONMENT=/usr/local to sync with the system python based on this comment https://github.com/astral-sh/uv/pull/6834#issuecomment-2319253359. With 0.4.13, this now breaks with

Project virtual environment directory /usr/local/ cannot be used because it is not a virtual environment and is non-empty

Which was added in https://github.com/astral-sh/uv/pull/7527.
I've looked through the docs, but can't find any official recommendation for what path to use to sync with the system python /usr/local/bin/python.

Sorry about this. The problem is #7522
I have a fix up in #7585

This will be fixed in the next release (sometime today)
New integration test in #7591

Thank you!


[Doc] Remove podman/docker dependency to access internal registry

This patch uses oc mirror command to push image from remote container
registry to openshift internal registry and make it available to
imagestream. Recently user observed if ubi images are pulled using
podman then pushing it to internal registry have signature verification
error (https://github.com/containers/podman/issues/14558) for
mac/windows. This issue is recently fixed on podman remote clients but
still not released so better not to use podman/docker steps just to show
user around internal registry usage.

This also switches from alpine to ubi8, this is not mentioned at all in the commit log.

Mentioned in the commit log.

There was one good thing about the previous example based on podman: this was showing that the openshift registry and regular podman/docker were interoperable, something we lost here. However, not sure crc's doc are thebest place to showcase/document this :)

I am also not sure may be a different section in the doc around this interaction of podman/docker with cluster?

The changes look good to me. I assume they've been tested across the supported platforms, correct?

Yes, they are tested on all the platform.

Also, agreed, we could use a more dedicated section to document interaction between the cluster and Podman.

@robin-owen created https://github.com/code-ready/crc/issues/3227 if you want to look at this


Update core-modules.zh.md

fix typo

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@bshy522  thanks for your contribution, please sign the CLA


Give locations a major/capital city field

Update locations.html and jurisdiction.html to order them like:
Capital City Event
A-Z other events

If they are an official event, I'd just leave the location name only.
Also to enable consumers of Jabberwocky info to do the same.

Resolved by fc10776c8f0902af36d9dc8764cbd5d1c4bf3ba8.


Unsupported Packet socket and Socket Filtering(when trying to run dhcpd)

A brief description
Trying to start isc-dhpc-server results in messages about unsupported Packet socket and Socket Filtering
Expected results
Everything working fine and dhcp-server giving ips to clients
Actual results (with terminal output if applicable)
Internet Systems Consortium DHCP Server 4.3.3
Copyright 2004-2015 Internet Systems Consortium.
All rights reserved.
For info, please visit https://www.isc.org/software/dhcp/
Config file: /etc/dhcp/dhcpd.conf
Database file: /var/lib/dhcp/dhcpd.leases
PID file: /var/run/dhcpd.pid
Wrote 0 leases to leases file.
socket: Address family not supported by protocol - make sure
CONFIG_PACKET (Packet socket) and CONFIG_FILTER
(Socket Filtering) are enabled in your kernel
configuration!
Your Windows build number
15014
Steps / All commands required to reproduce the error from a brand new installation
-- run bash as Administrator
-- create subnet in dhcpd.conf for your adapter
-- run dhcpd eth0
Strace of the failing command
http://pastebin.com/DmiTxGAx

...
socket(PF_PACKET, SOCK_RAW, 768)        = -1
    EAFNOSUPPORT (Address family not supported by protocol)
...

#1515 (message)

Thanks, @crashtua for the post and @therealkenc for the diagnosis. Yes, currently, WSL does not has support for AF_PACKET address family. If you can open a user voice ticket for this, it will help us prioritize it.

This feature would be great if implemented ! https://wpdev.uservoice.com/forums/266908-command-prompt-console-bash-on-ubuntu-on-windo/suggestions/17817211-support-af-packet-address-family-and-raw-sockets

@gpotter2 - Thanks for opening the user voice ticket. Just FYI that SOCK_RAW is currently supported for RAW, UDP and ICMP(V6) protocols. TCP can follow without much problem. So, I would restrict the user voice ticket for AF_PACKET and also add what applications are dependent on it. Now, let's get some votes behind it.

@sunilmut Edited :)

@sunilmut socket(AF_INET,SOCK_DGRAM,IPPROTO_ICMP); also shows Permission denied with root user.

Why ppl like Linux because Linux has fewer restrictions, people can do more what they want to do.
Please don't bring the limit from windows to the WSL.if only can do the windows can do,why we should be use WSL?
Soo,when could be the problem soloved?
https://wpdev.uservoice.com/forums/266908-command-prompt-console-bash-on-ubuntu-on-windo/suggestions/17817685-support-af-packet-address-family-tcpdump-wireshar
that has 1426 votes now ?
how many votes need ?

dupe #1515


drivers: spi: fix async slave API documentation

It is not possible to return the number of received frames for an asynchronous transfer in slave mode. The number of received frames is only known once the transfer got completed.
Return 0 if the transfer was started successful and a negative error code if the transfer could not start.

Even if this is an asynchronous transfer, this functions wait for a transfer to occur. When that happens, the number of frames received is known and the value can be returned. The problem I see is that the function is not currently returning the frames in slave mode.

I have observed the same issue. Frames will always return 0 as this sets an asynchronous task. I wonder how to get the real length of the transceive function. Should the rx and tx buffer len field return the received length?


BufferUpdate event should inform buffer duration properly

Goal
BufferUpdate event was not informing buffer duration properly. We should send the information in the userInfo dictionary with duration key.

Can’t we write some tests to avoid this problem on the BufferUpdate event?


Update and rename new to System_Text_Encoding_UTF8.md

Add a comment from the MSDN documentation about the behavior of the returned encoding object.

Is that better?

I think what you wanted is to add remarks @ellismg. Then it should be remarks: not summary:

Thanks @mairaw.  I've added another commit.

It is nice of @ellismg to contribute. Are we taking ref updates at this time?

I agree. Good question @richlander. We haven't accepted any so far and I'm not 100% sure of the right format (need to test this one with docfx since their spec doesn't mention the remarks format, but I think you need quotes surrounding that).

Thanks @ellismg. As we're still trying to bring more of the existing content from MSDN for .NET Core, I'll close this for now (see https://dotnet.github.io/api/System.Text.Encoding.html#System_Text_Encoding_UTF8 for the latest). I'll revisit this when we're done making our changes.


✨ [New Demo Request]: Notifications API

What is the Example and Demo Topic?
Notifications API
Please explain why do you need it?
I It would be useful to demonstrate the Barcode detection
Any Links, Reads?
https://developer.mozilla.org/en-US/docs/Web/API/Notifications_API
Do you want to contribute to the source code for this example?
Yes

i would like to work on it. can you help me with it?


Make CDROM disk attach errors more descriptive

Info

Which version of Pi are you using: Raspberry Pi 4 4GB Model B
Which github revision of software: 524f1c7
Which board version: 2.3 FULLSPEC
Which computer is the RaSCSI connected to: Quadra 700 and PowerMac 8500

This behavior is consistent via the web interface or command line.
When attaching a disk image (or anything strange that doesn't resemble a disk) as a CDROM, the error "File open error" is ambiguous and implies that it's possibly an issue with the underlying file system (permissions, locking, etc) rather than the ability to parse the file format of the accessed image.
Error message: Error : File open error [/home/pi/images/cdstuff1995.img]
Consider changing "File open error" to "CD format unreadable" or similar.  Consider a warning that states you can mount raw/ISO style CD images but not DiskCopy disk images (dmg/img format would be a WONDERFUL FEATURE THOUGH - Requires a little bit of hacking though!)

@kraptv Please check whether PR https://github.com/akuker/RASCSI/pull/175 resolves this issue.

I can confirm that where you in master get a generic error when mounting an invalid CD image, I now get this for the same CD images:

Error: Tried to open an invalid file '/home/pi/images/71A.iso': ISO CD-ROM file size must be a multiple of 2048 bytes

uweseimet_develop b32f324


[PHPStanExtensions] Add SplFileInfoTolerantDynamicMethodReturnTypeExtension

SplFileInfo->getRealPath() always exists, since it comes from Finder, that finds only existing files
This removes many false positives that have to be excluded manually otherwise.

Why? Finder provides only existing files and ignoring or re-check already string value makes both config and code ureadable with no added value.

Yep, you can create SplFileInfo of a nonexistent file. To quote PHP docs:
Returns the path to the file, or FALSE if the file does not exist.
On Fri, 21 Sep 2018 at 07:31, Gabriel Caruso<EMAIL_ADDRESS>wrote:

@carusogabriel commented on this pull request.
I don't think is a good idea. Maybe @ondrejmirtes
https://github.com/ondrejmirtes can opine better?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Symplify/Symplify/pull/1123#pullrequestreview-157550063,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGZuJTWjZAD-s1dRF_yzH6B8sHCnxyJks5udHm_gaJpZM4Wzd3I
.

--
Ondřej Mirtes

I know. Finder doesn't find non-existing files.

But you can’t limit the extension only to Finder instances.
The right solution would be to have Finder return something else than
SplFileInfo, with your own typehints.
Also, the file can probably be deleted between it is found by Finder and
asked about in getRealPath().
On Fri, 21 Sep 2018 at 07:40, Ondřej Mirtes<EMAIL_ADDRESS>wrote:

Yep, you can create SplFileInfo of a nonexistent file. To quote PHP docs:
Returns the path to the file, or FALSE if the file does not exist.
On Fri, 21 Sep 2018 at 07:31, Gabriel Caruso<EMAIL_ADDRESS>wrote:

@carusogabriel commented on this pull request.
I don't think is a good idea. Maybe @ondrejmirtes
https://github.com/ondrejmirtes can opine better?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Symplify/Symplify/pull/1123#pullrequestreview-157550063,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGZuJTWjZAD-s1dRF_yzH6B8sHCnxyJks5udHm_gaJpZM4Wzd3I
.

--
Ondřej Mirtes

--
Ondřej Mirtes

I don't see added value in copying Symfony\Finder package with 1 own line.

I've noticed there is SplFileInfo incorrecl. I've narrowed it to the Symfony created one and eliminated the rest of issues.
Thanks for review :+1:

Yes, this looks better, you should probably submit it to phpstan-symfony cc @lookyman


Light Level HUD Display Crashes Game

Mod Version:
DynamicSurroundings-1.16.4-<IP_ADDRESS>
Forge Version:
forge-35.1.36
Link to client log:
https://pastebin.com/GBpDB5pX
Link to crash log:
https://pastebin.com/upwYESX8
Description:
Attempting to enable the light level display will occasionally result in it not displaying at all. In a particular spot in our base, having it on and walking by a specific location will cause the game to crash.
We've been using this mod since we spun up this server. It's always crashed, occasionally in various areas (but now we have a reproducible location).
The pack being used is maintained by me, and is located at https://www.curseforge.com/minecraft/modpacks/logicraft_. However CurseForge is slow to publish updates, so we stopped updating the pack on CurseForge and I distribute the pack from my server directly to my friends.

Wanted to report we upgraded to 1.16.4-<IP_ADDRESS> and now the light level HUD is no longer togglable at all. The option to configure the keybind is totally removed from settings as well. Not sure if this was intentional so wanted to report as such. Thanks!

Yes, intentional.  These HUDs were added for debugging purposes.  I may activate in the future - needs polishing.  If you want to enable set "Debug Flag Mask" to 3 under the Dynamic Surroundings config section, or dsurround.toml file.

Alrighty, thanks! A semi-related question: are there any plans to bring Dynamic Surroundings: HUDs to 1.16+? I presume that's where that feature would end up in the end.

I haven't decided.  The Light Level HUD that is buried in the 1.16.4 version is the one from HUDs.  Chunk fencing is a short cut to the built in chunk fence debug feature of Minecraft.  I am not sure about implementing the compass or the Seasonal Clock from Serene Seasons.  If anything I would put in item frame support for it.


The scraper does not show that HR Metrics Quality Expert job was duplicated and posted twice

ENVIRONMENT:
https://admin.peviitor.ro/
Browser: Chrome
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:
Enter url in browser
In go to "https://admin.peviitor.ro/" and click to jobs for "Allianz"
Click on "HR Metrics Quality Expert" tab on "Catre Job "option
EXPECTED RESULTS:
The page shows the full description of the job
ACTUAL RESULTS:
The page shows the same full description of the previous job announcement
See below the screenshots
Attachment:
Page from Validator for Allianz

Joburile sunt prezente și pe site-ul lor de cariere. Uită-te atent la link ul jobului ca au id uri unice. Nu sunt duplicate.


Apply Profiles and their contained Bundles to multiple clusters simultaneously

As I have many existing clusters it would be great to be able to bulk apply Profiles to multiple clusters in a single command/action.
Aha! Link: https://pf9.aha.io/features/ARLON-160

One basic way to do this would be to have a comma separated list of clusters.
However, we can also define a dynamic cluster set or cloudlet from the orchestrator proposal - and have a way to define one using some kind of query/ expression logic.
e.g. some match expression in YAML or a SQL query like SELECT * FROM CLUSTERS WHERE LABEL IN ('dev', 'staging')

We'll need more details, before sizing this issue. If the requirement is simply a comma separated list, then it could be simpler.

Assuming we are talking about AppProfiles (and not the older gen1 profile), then applying one or more profiles to a cluster is done by annotating the cluster. The recommended way is to use kubectl annotate, see https://github.com/arlonproj/arlon/blob/main/docs/appprofiles.md#targeting-appprofiles-to-workload-clusters
Arlon does not currently offer a CLI command to do it, as it would be redundant. We could add a command, and support annotating multiple clusters in one shot. Frankly, I think the value add of this command is low, as it'll be used rarely. Annotating a Kubernetes object via k8s API is a little messy, it's not hard but you'd have to look at how kubectl does it. Adding an estimate based on the fact that we'd have to add a new command from scratch.


typo: remove value from no value context SUSER_ID()

@jong82 : Thanks for your contribution! The author, @edmacauley, has been notified to review your proposed change.

@jong82 closing and reopening the PR.

@jong82 Thank you for this fix. Reviewed and signing off. #sign-off

@rothja: I'm sorry - only the author of this article, @edmacauley, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


[Question] Parameter "sequence_length" of tf.nn.ctc_loss.

Hi,
According to the code, I found there is a parameter called “pad_to” that make sure the length can be divided by an integer (default is 8).
https://github.com/NVIDIA/OpenSeq2Seq/blob/e1f56a5c4acd9a82c2156f1d39eca36d049cc6f1/open_seq2seq/data/speech2text/speech_utils.py#L254-L255
Then I am confused about the calculation of “tf.nn.ctc_loss”.
https://github.com/NVIDIA/OpenSeq2Seq/blob/0a872ffdf1519047f42d69661293bc33dba4077c/open_seq2seq/losses/ctc_loss.py#L77-L82
Is the parameter “src_length” the original length of dataset (before padding) or after padding?
Thanks for answering!

In the current implementation, it is the length of the spectrogram after the pad_to operation.


URLSessionConfiguration timeoutIntervalForRequest not supported

I've noticed that if I create a new sessionconfiguration with a specific timeoutIntervalForRequest, the request does not time out when I use OHHTTPStubs:
let config = URLSessionConfiguration.default
config.timeoutIntervalForRequest = 2
stub(condition: isHost("httpbin.org") && isPath("/status/200"))
{ _ in
let stubData = "hello".data(using: String.Encoding.utf8)
return OHHTTPStubsResponse(data: stubData!, statusCode:200, headers:nil)
.responseTime(4)
}
When I make a request using the above session configuration, it should timeout in 2 seconds but it does not.  It also does not work if I use the timeoutInterval on URLRequest.
However, if I set the session configuration's timeoutIntervalForResource value, the timeout happens as expected.
I'm using Alamofire to make the requests by the way. Is this a known issue?

I have to admit I never tested that. Will have to investigate to know if it's an existing bug of OHHTTPStubs itself, of Alamofire, or if it's even possible at all to intercept that when using a custom NSURLProtocol (like this pod does to intercept requests)…

I used a custom UrlProtocol to get around this issue. I set this in the .protocolClasses property of my sessionconfiguration. The custom protocol basically uses an internal session to forward the request  (which incidentally is to my  OHHTTPStubs stub ).

Hey @CalvHobbes
Thanks for your feedback.
If you managed to make it work, I'd be very interested to see your solution.
You speak about using a custom NSURLProtocol and setting it in the protocolClasses property, but basically that's also what OHHTTPStubs does internally, so I'm intrigued.
Maybe your workaround could be integrated as a fix in OHHTTPStubs itself (feel free to do a PR) so others can profit?

Hello, I took another look at this. Turns out, the framework is fine. I was using .responseTime instead of .requestTime to delay the return of the response from the stub, which is why it was returning the response earlier than expected (and my request was therefore not timing out) :) Think we're ok wiht the current code. FYI though, for the custom UrlProtocol, I just used the standard code from one of Alamofire test UrlProtocol classes.

Cool that's reassuring then 👍 thanks!


Android annotations: implement automatic grouping

Android annotations API proposal implementation PR as a part of https://github.com/maplibre/maplibre-native/issues/1491. Closes #1494.

NDD fields have been added.
The "Add Markers in Bulk" activity in the test app has been modified to allow manual testing: the symbol number 2 always stands upright, and the last digit indicates the z layer (i.e. 89 comes above 98 comes above 7).
Unchanges, to avoid conflicts, the following classes still have temporary names that differ from their intended names:

Annotation → KAnnotation
AnnotationContainer → KAnnotationContainer

Grouping only uses default .groupBy; there's still lots of room for improvements to also support the use case where users might add lots of different z layers.
Unused bitmaps should now be discarded.

@fynngodau Suggestion: could you add some (unit) tests?
Right now, when I add a throw inside the getOrCreate method of KAnnotationContainer the unit tests still pass.
Maybe also add some tests for the IllegalArgumentExceptions?

@fynngodau CI is failing for some reason.

@louwers Thanks for the heads up. The problem was that with the changes from two commits ago, DraggableAnnotationControllers were being spawned in tests where they should be mocked.

Not sure if you wanted to add the exception tests or not. But looks good to me. :)

Yeah I still want to, I'll do it in a later PR.


keyerror in training mm-grounding-dino

I install  mmegine/mmdetection step by step , but when i begin training, it shows that there is something wrong when mmegine loading pth,
mmengine - INFO - Loads checkpoint by local backend from path: grounding_dino_swin-t_pretrain_obj365_goldg_grit9m_v3det_20231204_095047-b448804b.pth Traceback (most recent call last):   File "tools/train.py", line 121, in <module>     main() KeyError: 'backbone.stages.0.blocks.0.attn.w_msa.relative_position_bias_table'

I met the same problem, could you share how u solve this isssue, thanks！


Eclipse logger deadlock

Because of a potential deadlock in the Eclipse internals, the log action needs to be executed in the UI thread.
This is a temporary (and not really ideal) fix for #1000914.
I'll open a ticket in the Eclipse issue tracker as soon as I can log in again...

That would be nice to have the Eclipse ticket number as a comment in the code, for future reference.

Yep, that is planned. As said in the commit message: "I'll open a ticket in the Eclipse issue tracker as soon as I can log in again..." ;)


No example how to Create channel as migration mode with membershipType private

I write tool for migration Teams to Teams in same Tenant or New Tennant on Golang.
But no example how to create private Channel in migration mode.
Here my log:
Create Teams with migration mode without owner as visibility private:
Endpoint url: "https://graph.microsoft.com/v1.0/teams"
   {
      "@microsoft.graph.teamCreationMode":"migration",
      "createdDateTime":"2015-03-14T11:22:17.043Z",
      "description":"Some describtion",
      "discoverySettings":{
         "showInTeamsSearchAndSuggestions":true
      },
      "displayName":"MigrationTestSvetek-07",
      "funSettings":{
         "allowCustomMemes":true,
         "allowGiphy":true,
         "allowStickersAndMemes":true,
         "giphyContentRating":"Moderate"
      },
      "guestSettings":{
         "allowCreateUpdateChannels":false,
         "allowDeleteChannels":false
      },
      "memberSettings":{
         "allowAddRemoveApps":true,
         "allowCreateUpdateChannels":true,
         "allowCreateUpdateRemoveConnectors":true,
         "allowCreateUpdateRemoveTabs":true,
         "allowDeleteChannels":true
      },
      "messagingSettings":{
         "allowChannelMentions":true,
         "allowOwnerDeleteMessages":true,
         "allowTeamMentions":true,
         "allowUserDeleteMessages":true,
         "allowUserEditMessages":true
      },
      "visibility":"Private"
   }

Responce Team Create:
&{202 b1fb2f29-10ed-49f7-948b-57b50a357a40 b1fb2f29-10ed-49f7-948b-57b50a357a40 /teams('8db461d8-8b86-4b6e-8122-5ef47acba421') /
teams('8db461d8-8b86-4b6e-8122-5ef47acba421')/operations('00000000-0000-0000-0000-000000000000')}
Create Channel with migration mode as membershipType private
Endpoint url<EMAIL_ADDRESS>   {
      "@microsoft.graph.channelCreationMode":"migration",
      "createdDateTime":"2023-10-06T06:25:14.203Z",
      "description":"",
      "displayName":"SharedTest",
      "members":[
         {
            "@odata.type":"#microsoft.graph.aadUserConversationMember",
            "roles":[
               "owner"
            ],
            "user@odata.bind":"https://graph.microsoft.com/v1.0/users('5a0c4a45-007f-4d84-b620-4719d91006f1')"
         }
      ],
      "membershipType":"private"
   }

Responce Channel Create: Response code: 201  CannelID<EMAIL_ADDRESS>But channel is created as Standart.
How I can make migration Private to Private Channels?
Thank you!

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: c06a8a7a-b0fa-8304-84f5-9ada3b3c2ea5
Version Independent ID: 2c33f569-87dd-68be-2048-770788eae940
Content: Create channel - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/channel-post.md
Product: microsoft-teams
Technology: microsoft-graph
GitHub Login: @nkramer
Microsoft Alias: MSGraphDocsVteam

@dogonovm - Thanks for raising your issue. We will check this with engineering team and will update you accordingly.

@dogonovm - We are able to repro this issue. We have raised a bug for the same. We will inform you once we get any further update from engineering team.
Thanks


date / currency value formats not working; xmlns:number missing in root elment

how to reproduce

speadsheet-ods 0.5.2
Try to format a NaveDateTime value as date only

use std::path::Path;
use std::error::Error;

use chrono::{NaiveDateTime, NaiveDate, NaiveTime};
use spreadsheet_ods::*;

pub fn write() -> Result<(), impl Error> {

    let mut work_book = WorkBook::new();

    let date_format = work_book.add_format(format::create_date_dmy_format("date_format"));
    let date_style = work_book.add_cellstyle(CellStyle::new("date", &date_format));

    let mut sheet = Sheet::new_with_name("some sheet");
    let value = NaiveDateTime::new(NaiveDate::from_ymd(2021, 05, 02), NaiveTime::from_hms(12,23,24));
    sheet.set_styled_value(0, 0, value, &date_style);

    work_book.push_sheet(sheet);
    write_ods(&work_book, Path::new("out.ods"))
}

what happened

In the resulting spread sheet the date and time are visible
out.ods

what should happen

In the resulting spread sheet only the date should be displayed

analysis
The XML namespace  xmlns:number="urn:oasis:names:tc:opendocument:xmlns:datastyle:1.0" seams missing from content.xml. This is what xmllint says when formatting the file.
content.xml:2: namespace error : Namespace prefix number for style on day is not defined
tyle><number:date-style style:name="date_format"><number:day number:style="long"

Manually adding  xmlns:number="urn:oasis:names:tc:opendocument:xmlns:datastyle:1.0" to content.xml and putting it back into out.ods seams to fix the issue.

Yes, I notized this too, just forgot to publish a patch. Done.


drop user created publications

What kind of change does this PR introduce?
fix https://github.com/supabase/cli/issues/2640
What is the current behavior?
Failed to delete user-created publications, causing the migration to create publications to fail.
What is the new behavior?
Add a function to drop user-created publications, excluding supabase_realtime.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

1 of 3 (33.33%)  changed or added relevant lines in 1 file are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.04%) to 59.741%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/db/reset/reset.go
1
3
33.33%

Files with Coverage Reduction
New Missed Lines
%

internal/storage/rm/rm.go
2
89.53%

internal/gen/keys/keys.go
5
12.9%

Totals

Change from base Build<PHONE_NUMBER>3:
-0.04%

Covered Lines:
6357

Relevant Lines:
10641

💛  - Coveralls

Hey, thanks for the contribution. I've made refactored your query to drop.sql to simplify the code.


Per Instance Time Limit

When the instance time limit is once increased to 5 minutes and the time decreases to less than 4 min, it is no longer possible to increase the limit by even a single minute.

As of now, for testing purposes, instances can have a maximum time of 5 min

can you attach a screenshot?

@e4stw1nd


arch: riscv: per-core init function

Spinned-off from #65824, with an in-tree example of how it can be used.

I think calling z_riscv_pmp_init()) deep from SOC code is odd and wrong. That ought to remain out of the SOC code's concern. Similarly for IRQs.

Reverted the implementation to align with #65824, is it better now?

What, no tests? 😁

Thank you apply per core init hook on ae350 PMA feature , but I got some error when I tested it on adp_xc7k/ae350 with CONFIG_SOC_ANDES_V5_PMA=y.
/home/jimmyzhe/zephyrproject-upstream/zephyr/arch/riscv/include/kernel_arch_func.h: In function 'arch_kernel_init':
/home/jimmyzhe/zephyrproject-upstream/zephyr/arch/riscv/include/kernel_arch_func.h:57:9: error: implicit declaration of function 'soc_per_core_init_hook' [-Werror=implicit-function-declaration]
   57 |         soc_per_core_init_hook();
      |         ^~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

I think all file in 053112fe9b3ffbc426898f3d8f50b7bed58eab33 need to include zephyr/platform/hooks.h.

Thank you for applying per core init hook on ae350 PMA feature , but I got error when I tested it on adp_xc7k/ae350 with CONFIG_SOC_ANDES_V5_PMA=y.
/home/jimmyzhe/zephyrproject-upstream/zephyr/arch/riscv/include/kernel_arch_func.h: In function 'arch_kernel_init':
/home/jimmyzhe/zephyrproject-upstream/zephyr/arch/riscv/include/kernel_arch_func.h:57:9: error: implicit declaration of function 'soc_per_core_init_hook' [-Werror=implicit-function-declaration]
   57 |         soc_per_core_init_hook();
      |         ^~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

I think all file in 053112f need to include zephyr/platform/hooks.h.

Oops, good catch. I also refactored the soc_*_init_hook()functions frompma.ctosoc.c`