Train on with other seeds

Currently, our mdoel is only trained for one seed , 15, would be good to test other seeds.

Yeah, this has to be done but we can't our results with ivadomed anymore if we change seeds. We can only do a comparison between nnUNet and MONAI (which could be a comparison between hard and soft labels).

Also redo a STD per subject for each seed to get more subjects in the violin plot

Just a quick note of clarification -- choosing different seeds might result in a slight overlap of test subjects coming from each seed. While we will surely have more points in the STD CSA plot, it will simply not be num_seeds x num_test_subjects (i.e. we would have to remove some repeated subjects)


fix c-ares and absl install dir on centos

Add -DCMAKE_INSTALL_LIBDIR:STRING=lib flag to c-ares and absl so it get installed in lib dir and not lib64 on CentOS

Thanks for your contribution! The changes passed on the CI testing.  If not, please do so via the instructions here: https://github.com/triton-inference-server/server/blob/main/CONTRIBUTING.md#contributor-license-agreement-cla
CC @GuanLuo

@ax-vivien LGTM. Thank you for your contribution! Please let me know once the CLA is sent so I can merge this PR.

Thanks for your contribution! The changes passed on the CI testing.  If not, please do so via the instructions here: https://github.com/triton-inference-server/server/blob/main/CONTRIBUTING.md#contributor-license-agreement-cla CC @GuanLuo

I just sent the signed CCLA, it contains all members of my team that may contribute to Triton at some point. Tell me If something is missing.


Add the pattern "You have to use sudo for that." to sudo.py.


Postgres configurations are not active

On the servers, values of the postgres configs are different than what's in the postgres conf... (Showing production here)

/var/lib/postgresql/data/postgres.conf looks nothing like what's committed to the repo.
I wonder whether these databases were ever updated with the new configs//these looks like they are being overwritten every time?

I just updated the Staging server to include this change, and looking at the active postgres configuration running there now, I can confirm that this worked. Thanks for catching and fixing this, @dchiller!


docs: Create documentation and examples for Internationalization

Our current version of templ does not feature examples for internationalization, which could help with adoption.
This PR aims to resolve that by

Adding an example for i18n using ctxi18n
Adding documentation under Integrations to use ctxi18n

For the example: I created an SSR app with a title and a few buttons. Languages could be switched based on provided path parameters (e.g. /en gives English translations, whereas /de gives German translations etc.)
For the documentation: I added the internationalization documentation as a subpage under Integrations, while renaming the existing page as "Web frameworks"
Closes #154

Thanks for that!


CI Should fail on Lint Errors

If CI throws a lint error it should fail. It currently passes. This is primarily for my PRs. 😄

Yeah, I've noticed some linting errors, but was myself under the impression they are failing. I'll take a look today, there's something in gulp that prevents that.

@enykeev I clean up those linting issues in #377

This has been fixed in the react rewrite, which should be included with 2.6.


python311Packages.flask-marshmallow: 0.15.0 -> 1.0.0

Diff: https://github.com/marshmallow-code/flask-marshmallow/compare/refs/tags/0.15.0...1.0.0
Changelog: https://github.com/marshmallow-code/flask-marshmallow/releases/tag/1.0.0
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

4 packages built:

python311Packages.flask-marshmallow
python311Packages.flask-marshmallow.dist
python312Packages.flask-marshmallow
python312Packages.flask-marshmallow.dist


px4_work_queue: rename navigation_and_controllers -> nav_and_controllers

The length exceeding the current NuttX limit appears to clobber ps output. I'd rather not find out about the other things in the system it has the potential to smash.
Now's a good time if anyone has a better name in mind.

It seems something else is going on, although it's worth getting the name under the limit regardless.

This PR appears to be breaking the SITL tests


Feature Request: RepeatExtensions allow generation for last x days

Currently, to have the repeat extensions generate the repeated task, I need to remember to run it on each day that contains a completed extended repeat task, or I need to remember to place the onEditorWillSave trigger.
This feature request is to run allow "/generate repeats" to have a number of days to look back on. for example, "/generate repeats -1w" would generate repeats for all the days of the last week (using the same time syntax as the extended repeats).
Note: having an automatic daily note template where I have inserted this trigger would eliminate the need for this request, as the trigger would be the already and allow the functionality I'm looking for. unfortunately, this is not quite possible yet, and I'm really worried about accidentally abandoning a task that I felt was important enough to make repeating. so this is a solution to minimize this risk.

v0.14.0 of Tidy Plugin now gives you this capability: either as "/Generate @repeats in recent notes" or through a template tag.
I'm going to mark this as closed, but do open it again if you find any issues.


Update api.php

User description

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[ ] No

ℹ Additional Information

Description

Refactored the API data fetching functions to improve clarity and functionality.
Introduced a new function loadData that handles the API response and error management.
Updated fetchAllGitHubPages to use loadData, enhancing code maintainability.

Changes walkthrough 📝
Relevant filesEnhancement

api.phpRefactor API data fetching functions for improved clarity and functionality

Src/api.php
Renamed function fetchAllGitHubPages to loadData for clarity.  Updated loadData to return both headers and body from the API response.  Modified fetchAllGitHubPages to utilize loadData for fetching data.  Added error handling to return null in case of a cURL error.

+24/-7

💡 Penify usage:
Comment /help on the PR to get a list of all available Penify tools and their descriptions

Summary by CodeRabbit

New Features

Enhanced data retrieval for pull requests, including additional state information.
Improved pagination handling for fetching GitHub pages.

Bug Fixes

Enhanced error handling during data loading to prevent failures.

Refactor

Split the fetchAllGitHubPages function into two distinct functions for better modularity and clarity.

@gstraccini codacy bypass


Support "cumulative" counts

While the current "COUNTER" type is possible it requires using very expensive queries. In particular, the number of new records since the last query is relatively small.
Instead of requiring a re-query of an entire table, a "cumulative" option could report "counter" metrics by summing past results with current results. In this way, each query could be for a small window of time and added to the totals from all past queries on every refresh period.

This is probably not a good idea.


corrected grammar (ENG)

What does this PR do ?
Grammar corrections in English
How should this be manually tested?
Read changes

Codecov Report

Merging #414 into 2.x will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              2.x     #414   +/-   ##
=======================================
  Coverage   85.79%   85.79%
=======================================
  Files          38       38
  Lines         725      725
=======================================
  Hits          622      622
  Misses        103      103

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 f943a87...c5045b7. Read the comment docs.


Sudden issue when trying to deploy a docs build to a preview site on Azure Storage - an action that's worked hundreds of times till now

Does this occur consistently?  YES
Repro steps:

Select Deploy Static Website
Select xtdocsprev..z22.web.core.windows.net

Made minor changes to a single .RST file, build without errors, try to deploy to the preview site and I suddenly start getting these errors!
Thank you for looking into this.
Action: azureStorage.deployStaticWebsite
Error type: EPERM
Error Message: EPERM: operation not permitted, scandir 'c:\ExperimentToolsDocs\docs\xt_build\html_static\css\fonts\Roboto-Slab-Regular.woff'
Version: 0.9.0
OS: win32
OS Release: 10.0.19041
Product: Visual Studio Code
Product Version: 1.46.0
Language: en

@rchrdgrace
Thanks for reporting.  That error seems like the extension doesn't have authorization to access the .woff file.  Did you change the directory structure/location or is that a new file that could be potentially causing issues?


Inconclusive tests should be marked as failed in VS-Test Task

When working with dynamic test data from a 3rd party it can sometime be invalid for your test scenario - This is a good scenario where the test itself can be considered inconclusive because the state is not suitable.
However, when you have inconclusive tests with VS-Test Task it marks it as Failed while there should be a different status like Inconclusive or Did not Run so that we know which ones failed and which ones had bad data.
Does this make sense?

Any ideas about this?

Let me know if you have any questions about this feature request!

@tomkerkhove Thanks for reminding on this. VSTest task reports whatever VS reports in the trx file after running the tests, we have asked for inputs internally on this. I will update the thread as soon as I get a reply on it.

@vimegh Thanks for the update!

By default, inconclusive tests are reported as Skipped (Not Run). If you are using MSTest framework based tests, the way to change that is by using a setting in the runsettings file (see MapInconclusiveToFailed here: https://msdn.microsoft.com/en-us/library/jj635153.aspx). Can you check if such a .runsettings file is in effect?

That would be great to have as well - Unfortunately, when you run them in VSTS with the Test-task it marks them as failures instead of Skipped.
Would it be possible to change this to result in Skipped tests?

@tomkerkhove As far as I know, if there is a inconclusive/notexecuted test in the trx file generated after running the tests, VsTest task marks it as Others/NotExecuted (doesn't mark is as failed). Can you please share the trx file and run settings file by using which Inconclusive/NotExecuted tests are being marked as failed by vstest task. Thanks.

We are using NUnit instead of MSTest but when we run them locally they are marked as Others/NotExecuted while on VSTS they are marked as Failed.

@tomkerkhove Can you share the results file being generated by Vstest task? You can email us the resulted trx file generated by VSTest Task at devops_tools AT microsoft DOT com
Also are you using OnPrem TFS, if yes can you share with us the version. Thanks.

@vimegh Thank you very much, mail is on the way!

I got a failure delivery for certain people so please let me know if you didn't get it.

Hi @tomkerkhove we have received the mail. @vimegh will check and reply back

Hi, we are probably facing similar problem. I reported it here: https://github.com/Microsoft/vsts-agent/issues/713 It could well be I picked the wrong component when raising the issue, as pointed out in the comments. Could you please check it and let me know whether I should somehow move the issue here (or merge it with this one?). Thanks a lot!

I am able to reproduce the issue with Nunit 3.5 and Nunit Test Adapter 3.6 and VS 2015 U3
@tomkerkhove Are you using the same nunit version?
@zelenp You can follow this issue thread only, instead of creating a new one, as it seems you are facing the same issue.
Also can you please update the thread with following:

VS Version you are using
TFS version being used (if on premises)

thanks.

@vimegh We are still on 3.4.1 but I think it's applicable for a few/all versions. Thank you for looking into this!

@vimegh We are using the hosted build agent/pipeline, our account is https://bssai.visualstudio.com
Individual developers use either VS 2015 U3 or Rider, depending on their preference.
The behaviour we experience is following (copied from the original report for your convenience):
When NUnit tests are executed on Visual Studio Team Services (VSTS) build server, tests marked as Explicit are correctly skipped as per console output:
2016-12-06T10:00:47.2789724Z Information: NUnit Adapter <IP_ADDRESS>: Test execution complete
2016-12-06T10:00:47.2789724Z
2016-12-06T10:00:47.5128672Z Results File: C:\a\1\s\TestResults\buildguest_TASKAGENT5-0027 2016-12-06 09_59_48.trx
2016-12-06T10:00:47.5158684Z
2016-12-06T10:00:47.5158684Z Attachments:
2016-12-06T10:00:47.5168676Z   C:\a\1\s\TestResults\f3d40ef8-75d5-4756-b1c5-e6338daf9b3c\buildguest_TASKAGENT5-0027 2016-12-06 09_59_37.coverage
2016-12-06T10:00:47.5168676Z
2016-12-06T10:00:47.5168676Z Total tests: 76. Passed: 62. Failed: 0. Skipped: 0.
2016-12-06T10:00:47.5168676Z Test Run Successful.
2016-12-06T10:00:47.5178694Z Test execution time: 1.1636 Minutes

However, all explicit tests are marked as Failed in test record trx file like this:
<UnitTestResult executionId="a5898e64-e18b-497c-aaa3-6f0b3c188154" testId="34822bf0-4fc5-43dd-9d33-5979bab83594" testName="TestInspectorConstraint" computerName="TASKAGENT5-0027" duration="00:00:00.0010000" startTime="2016-12-06T09:59:50.0000000+00:00" endTime="2016-12-06T09:59:50.0000000+00:00" testType="13cdc9d9-ddb5-4fa4-a97d-d965ccfc6d4b" outcome="Failed" testListId="8c84fa94-04c1-424b-9868-57a2d4851a1d" relativeResultsDirectory="a5898e64-e18b-497c-aaa3-6f0b3c188154" />

We are using NUnit 3.5 and 3.6.0 NUnitAdapter and hosted build agent.

Hi @tomkerkhove  @zelenp
It looks like the nunit adapter today sets the inconclusive outcome results to None in the test result object returned to vstest.console, which due to being indeterminate is set Failed. We have contacted the owner of the nunit adapter code to resolve the issue. Will update the thread soon.

@prawalagarwal Thank you for the update and looking into this. I'm looking forward to having this supported

@prawalagarwal Can you provide an update on this status or a link to an issue if one was filed so I can stay up to date?

Hi @tomkerkhove
Sorry for the delays in updating. There is an on going discussion going on this topic on the nunit adapter issues page :. Once it reaches a conclusion we should be able to update you what is the way forward. Currently, this seems like a design level blockade in how nunit adapter is used by the VSTest.

@prawalagarwal   The NUnit3 adapter issue is clear on needing a change in the vstest task.  See @charliepoole 's comment
"On January 8, @prawalagarwal noted on that issue that it was being discussed here. As you can see, that's not the case. There has been no further discussion since I outlined the problem as we see it on January 6."
So you can reopen this, and also remove the question tag and mark it as an enhancement.

@nigurr Did you close the issue because it was moved to https://github.com/Microsoft/vstest/issues/525 a dedicated issue to get this fixed or what is the reason?

@tomkerkhove I moved it to vstest repo so that right set of people can address this issue. Since vsts-task reads TRX and publishes the information, and this issue is more closely related to vstest.console and Nunit adapter.
We already notified the product team about this issue.

@nigurr Thanks for the update!


add configurable vars to main.go

When merged, this PR will:

[x] make the log level configurable by the use of a command-line-flag

Closes #39.
cc: @btmorr

@btmorr where would you like to document this env var?

@btmorr in the latest commit. Those corrections were because of the markdown linter present in my editor. I hope that it isn't having a negative effect.


CSS added with the extraHead option is not always respected

I am adding custom css via the "extraHead" option in my gulp task to make the generated styleguide align more with the product it is documenting. Everything works when loading the "Overview" section but if I go to any other section the added styles are no longer respected. The same thing happens if I navigate from the overview section, highlight the url and hit enter.

For anyone else having this issue. I got the desired result by using "customColors" rather than "extraHead"


Update maintainers

Removed the whole checklist as this is only for updating maintainers

We'll need to switch this to a fork branch

Sorry for the inconvenience, see PR #16 instead. Closing this one.


Ballerina thows java.lang.OutOfMemoryError: Java heap space

Description:
ballerina throws this error during run time
java.lang.OutOfMemoryError: Java heap space
Dumping heap to /usr/lib/ballerina/ballerina-0.981.0/heap-dump.hprof ...
Unable to create /usr/lib/ballerina/ballerina-0.981.0/heap-dump.hprof: Permission denied
error: error, message: java heap space
	at main(hello_word.bal:12)

Steps to reproduce:
Simply run this
 string s = "ssssssssssssssssssssssssssssssss";
   while(true) {
           s += s;
   }

Affected Versions:
Ballerina 0.981.0
OS, DB, other environment details and versions:
Ubuntu 16.06 LTS

Can throw internal bvm runtime error instead ?

Hi,
We cannot avoid the initial part of the heap-dump related error message since it is getting printed by Java when the heap dumps are enabled.
We do catch the throwable errors that include OutOfMemoryError. I have change the wording to replace Java with runtime.
Thanks


Update dependency byteorder from 0.3.x to 0.5.x

The rationale of this pull request is similar to that of pull request #40 of byteorder. Before byteorder 0.5.x, we need to handle two (syntactically) different yet (conceptually) equivalent versions of errors (i.e., std::io::Error and byteorder::Error). With byteorder 0.5.x, we can write much neater code.

Brilliant! Thanks!


Got exception when access to luci-app-shadowsocks-libev web page

Got below exception when access to luci-app-shadowsocks-libev page:
Failed to execute arcombine dispatcher target for entry '/admin/services/shadowsocks-libev/instances'.
The called action terminated with an exception:

...b/lua/luci/model/cbi/shadowsocks-libev/instances.lua:19: bad argument #1 to 'pairs' (table expected, got boolean)
stack traceback:
	[C]: in function 'pairs'
	...b/lua/luci/model/cbi/shadowsocks-libev/instances.lua:19: in function 'func'
	/usr/lib/lua/luci/cbi.lua:66: in function 'load'
	/usr/lib/lua/luci/dispatcher.lua:838: in function 'target'
	/usr/lib/lua/luci/dispatcher.lua:956: in function </usr/lib/lua/luci/dispatcher.lua:952>

Following are system informaion
Model	Linksys WRT1900ACS
Architecture	ARMv7 Processor rev 1 (v7l)
Firmware Version	OpenWrt 18.06.1 r7258-5eb055306f / LuCI openwrt-18.06 branch (git-18.228.31946-f64b152)

luci app version is
luci-app-shadowsocks-libev - git-19.127.24963-84692e5-1

Any suggestion on quick fix? Can I modify any file like "/usr/lib/lua/luci/model/cbi/shadowsocks-libev/instances.lua" on target to fix it?

bad argument #1 to 'pairs' (table expected, got boolean)

The error is likely about missing/wrong config. The page logic expects contents for some dropdown list and there is nothing in the config file. Or something like that.

Any suggestion on quick fix?

Fix your shadowsocks config manually by editing the config file?
Revert to the default shadowsocks config file?
There have been lots of fixes/enhancements from @yousong in master, which have not been backported to 18.06. But it is really hard to say if any of those would fix your problem.

Was /etc/config/shadowsocks-libev present?  If yes, please share the content of it with password and server name redacted.

It's my fault. The configuration file was broken after editing by vi. I fixed it and the issue is gone.
Thanks for all the help!

It's my fault. The configuration file was broken after editing by vi. I fixed it and the issue is gone.
Thanks for all the help!

I had the exactly the  same problem ,  but I did not change the configuration ,Please tell me how did you fix it ?


config: harmonize ip-mask -> ip-masq

- Description for the changelog
Harmonize Docker Engine daemon parameter --ip-masq, replacing --ip-mask in daemon.json.
Closes #22807.

Hi! Thanks for opening this PR, but it looks like this is trying to resolve the same issue as https://github.com/docker/docker/pull/22756. Also check the discussion on that PR.
I'll close this PR, but thanks again for contributing!


docs(contributing-md): ledger connector plugin test automation

Also talks about the all in one docker images and
the classes that they use them.
Fixes #233
Signed-off-by: Peter Somogyvari<EMAIL_ADDRESS>

Thanks for describing the requirements of the test cases.
Can you consider to rewrite some parts regarding my comments?

Just added this: https://github.com/hyperledger/cactus/pull/234/commits/1f64b7f8518fe9e538ca71a26d6747826f45339b


Updated CONTRIBUTORS.md

Description
Updated CONTRIBUTORS.md
Changes

[x] Added @gigasquid to list of committers
[x] Moved @marcoabreu to committers section
[x] Added myself to list of contributors

Build is failing due to https://github.com/apache/incubator-mxnet/issues/12767. Restarted.


[Feature]: be able to use iframes without padding

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Iframes are currently useless for small components because padding is added and is not related to external component.
This makes iframes very difficult to use since it is not possible to align smaller iframes as close as I want.  Making height smaller adds an annoying scroll bar on overflow.

I want to have zero padding for iframe or possibility to remove it.
Custom widget needs a lot of copy and pasting to be able to use at large scale as component library, maintainace of these components are unclear and performance is really poor compared to iframes. It is not possible to solve by putting iframe inside Custom component.
Why should this be worked on?
We want to replace small components with components of our own design, hosted externally.
Iframes works for larger widgets, but is useless for smaller widgets because of padding (left, right, top and bottom).
The padding sometimes makes it difficult to implement larger widgets as well.

@toxicflux have you tried using the custom widget for this?

Yes, I've tried Custom widget for this (as screenshot show). Custom widget do not work for reusable code, it has severe performance issues  and it doesn't work putting an iframe inside a custom widget as I described above.

Ok I'm not sure iFrame is the best idea either because many iFrames on a single page will likely lead to a performance issue as well. Could you tell us why you are looking to customise the components? We are working on reusable widgets as well but it doesn't seem like that will suit your use case either.

We need to style our applications to meet customer requirements, meaning we want to override the style of basic components such as buttons, inputs, selects. The styling of appsmith components are limited, the best would be if we could provide our own component library.

@toxicflux, thank you for raising this topic. We’d like to understand more about the use cases and limitations, which will help us identify what's missing in the Appsmith product and potential improvements.
Please feel free to book a time that works for you. I look forward to speaking with you.

This is also something I have had issues with. I am trying to display an external website within an iframe, but it is zoomed in too much to fully fit the page in view. This is especially a problem on the mobile view. If I had the ability to fit content to width on the iframe at least this would be much more usable. I tried doing a custom iframe, but I was not able to interact with my content, only display it.


Attribute name typo

https://github.com/cb80/add2project/blob/231dcbfe53e268e2a1f75a19352f927fb449e5af/src/main.ts#L87
Shouldn't .lenght  be .length?

Thanks @rdietrick, 119c53d100b8c3c64c51fad81ea1014fe7581917 should fix this.


Content Creator: repeat attributes

As a Content Creator I want a way to repeat an attribute during content entry so that I may input more than one items or items
Feature: repeat Input Plugins
  As a Content Creator
  I want a way to repeat an instance of an attribute during content entry
  So that I may input more than one items or items.

  Scenario: Add attribute instance
    Given that I need to add another instance of an attribute
    When I want to duplicate an attribute
    Then I should have a button which duplicates that attribute

  Scenario: Remove attribute instance
    Given that I remove an instance of an attribute
    When I want to remove an attribute in a repeatable list of attributes
    Then I should have a button which removes an attribute(s)

  Scenario: Persistent validation
    Given that some attribute have validation
    When duplicating an attribute
    Then the validation should validate the new attribute instance.

  Scenario: Persistent UX functionality
    Given that some attribute have UX scripts connected to them
    When duplicating an attribute
    Then the ux should function on the new attribute instance.

  Scenario: Editing content
    Given that repeatable attributes will need to be edited
    When I open an edit screen for content with repeated attributes
    Then all of the attribute instances that I need to edit will be showing in the form.

[x] repeat attributes / add
[x] repeat attributes / remove
[x] repeat attributes / validation
[x] repeat attributes / ux
[x] repeat attributes / editing

repeatables completed


refactor: use chain/network speciffic block explorers

Description:
use chain/network speciffic block explorers
Motivation and Context:
Remove hardcoded assumptions about what chain/network we are connecting to.
How Has This Been Tested?
Manually
Types of changes:
Refactor
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage increased (+0.01%) to 21.372% when pulling 3eac65b998738398a0222cccc3a2ad645763de2d on mrfelton:refactor/multi-network-sync into 786c704682799ba0403077c3eb0daed401e7ea44 on LN-Zap:next.


Upgraded dependencies

Breaking changes.

Set minimum php version to >= 7.3.
Halite tests checks if sodium is loaded instead of libsodium (as it's included by default)
Upgraded all dependencies
Moved from Zend/Crypt to Laminas/Crypt
Added breaking change notification to readme
Updated travis.yml to check 7.3/4 and 8

Looks great!


Able to process TPC-H q1, q9, q12, q15, q17

Q1
select
  l_returnflag,
  l_linestatus,
  sum(l_quantity) as sum_qty,
  sum(l_extendedprice) as sum_base_price,
  sum(l_extendedprice * (1 - l_discount)) as sum_disc_price,
  sum(l_extendedprice * (1 - l_discount) * (1 + l_tax)) as sum_charge,
  avg(l_quantity) as avg_qty,
  avg(l_extendedprice) as avg_price,
  avg(l_discount) as avg_disc,
  count(*) as count_order
from lineitem
  where l_shipdate <= '1997-12-01'
  group by l_returnflag, l_linestatus
  order by l_returnflag, l_linestatus;

Q9
select nation, o_year, sum(amount) as sum_profit
from (
  select
  n_name as nation,
  year(o_orderdate) as o_year,
  l_extendedprice * (1 - l_discount) - ps_supplycost * l_quantity as amount
  from lineitem
       inner join orders on o_orderkey = l_orderkey
       inner join partsupp on ps_suppkey = l_suppkey
       inner join part on p_partkey = ps_partkey
       inner join supplier on s_suppkey = ps_suppkey
       inner join nation on s_nationkey = n_nationkey
  where p_name like '%green%') as profit
group by nation, o_year
order by nation, o_year desc;

Q12
select l_shipmode,
       sum(case when o_orderpriority = '1-URGENT' or o_orderpriority = '2-HIGH'
                then 1 else 0 end) as high_line_count,
       sum(case when o_orderpriority <> '1-URGENT' and o_orderpriority <> '2-HIGH'
                then 1 else 0 end) as low_line_count
from orders, lineitem
where o_orderkey = l_orderkey
    and l_shipmode in ('MAIL', 'SHIP')
    and l_commitdate < l_receiptdate
    and l_shipdate < l_commitdate
    and l_receiptdate >= '1994-01-01'
    and l_receiptdate < '1995-01-01'
group by l_shipmode
order by l_shipmode;

Q15
select s_suppkey, s_name, s_address, s_phone, total_revenue
from (
    select l_suppkey as supplier_no,
           sum(l_extendedprice * (1 - l_discount)) as total_revenue
    from lineitem
    where l_shipdate >= '1996-01-01' and l_shipdate < '1996-04-01'
    group by l_suppkey) revenue_temp
    inner join supplier
    on s_suppkey = supplier_no
where total_revenue = (
        select max(total_revenue)
        from (
            select l_suppkey as supplier_no,
                   sum(l_extendedprice * (1 - l_discount)) as total_revenue
            from lineitem
            where l_shipdate >= '1996-01-01' and l_shipdate < '1996-04-01'
            group by l_suppkey) revenue_temp)
order by s_suppkey;

Q17
select sum(l_extendedprice) / 7.0 as avg_yearly
from lineitem
     inner join (
       select l_partkey as partkey, 0.2 * avg(l_quantity) as small_quantity
       from lineitem inner join part on l_partkey = p_partkey
       group by l_partkey) t
     on l_partkey = partkey
     inner join part
     on l_partkey = p_partkey
where p_brand = 'Brand#23'
  and p_container = 'MED BOX'
  and l_quantity < small_quantity;

Q15
create view revenue_temp as
select l_suppkey as supplier_no,
       sum(l_extendedprice * (1 - l_discount)) as total_revenue
from lineitem
where l_shipdate >= '1996-01-01' and l_shipdate < '1996-04-01'
group by l_suppkey;

select s_suppkey, s_name, s_address, s_phone, total_revenue
from supplier, revenue_temp
where s_suppkey = supplier_no
  and total_revenue = (
        select max(total_revenue)
        from revenue_temp)
order by s_suppkey;

drop view revenue_temp;

Sample creation statements
create 1% uniform sample of lineitem;

create 1% uniform sample of orders;

create 1% universe sample of lineitem on l_partkey;

create 1% universe sample of orders on o_partkey;

create 1% universe sample of lineitem on l_orderkey;

create 1% universe sample of orders on o_orderkey;

create 1% stratified sample of lineitem on l_suppkey;


Support launching ddp job as module python -m ...

🚀 Feature

Motivation
Some users wish to launch their training program as a module with python -m some.module.py
Pitch
We should evalute whether this is possible for ddp and support this option when possible.
We need to strip the -m argument and append it to the command with which we launch the child processes.
Alternatives

Additional context
This feature was orginally reported as a bug: #3600

torchelastic could be a good option here: https://pytorch.org/elastic/0.2.0/quickstart.html

Hey @awaelchli, I would like to take this

that's good with me! give it a shot!

Hey @awaelchli , sorry for not being clear about this earlier, will this require gpus for testing!

will this require gpus for testing!

the standard ddp with gpus yes. But if you don't have gpus, you can probably implement the feature for the ddp_cpu backend first (this kind of simulates ddp on the cpu).

To add to the discussion: I'm able to use DDP backend when running my training program as a module, eg: python -m seg_lapa.train, so long as I use absolute imports. See #4243.


Analysis on peroxisome gradients

Problem
Adds new analyses for peroxisome gradients
Solution
Updated the analysis configuration to create more modular options. Added new gradient options including surface distance based gradients.
Type of change

New feature (non-breaking change which adds functionality)

Change summary:

Improvements to calculation of similarity score
Improvements to analysis config schema
Improvements to parametrized representation calculations
New analyses for surface distance gradients

Steps to Verify:

conda activate autopack
Run the following:
pack -r  examples/recipes/v2/peroxisomes_surface_gradient.json -c examples/packing-configs/peroxisome_packing_config.json
for the desired gradient weight modes: "linear", "square", "cubic", "None" by changing the version in peroxisomes_surface_gradient and changing the "weight_mode" for nucleus_surface_gradient.
To run the analyses:
analyze -a examples/analysis-configs/peroxisomes_analysis_config.json

Screenshots (optional):

Keyfiles (delete if not relevant):

cellpack/autopack/Analysis.py
cellpack/autopack/Gradient.py

I tried running the peroxisomes_surface_gradient recipe and it was pretty cool! However, I got an error ValueError: invalid literal for int() with base 10: 'linear' when I ran the analyses, so I'm guessing I need to make some adjustments before running the analyses?
The doc strings you added really helped me with understanding! Thanks!

I tried running the peroxisomes_surface_gradient recipe and it was pretty cool! However, I got an error ValueError: invalid literal for int() with base 10: 'linear' when I ran the analyses, so I'm guessing I need to make some adjustments before running the analyses?

Good catch! The issue was caused by how the dataframe was being saved. It should be fixed now.


Better support mediatype suffixes and parameters

Implements #5273
To resolve the ambiguity about whether SupportedMediaTypes like application/custom should match against [Produces("application/custom;v=2")] or not, I've implemented support for the application/custom;* syntax as previously discussed. This is not a standard media type syntax, but then neither is application/*+json or indeed anything besides */* or x/* (for some x). Such strings are only used for configuring the server (they never get sent to the client) so no HTTP spec would be applicable anyway.
In terms of performance, this shouldn't cause any extra allocations on normal HTTP requests. There is some extra CPU work involved in parsing media types in extra places to determine whether they contain wildcards, but the parsing in general is very cheap, and we already parse them in a bunch of places so I don't expect this to make any meaningful difference.

@SteveSandersonMS,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

The only issue I think left to resolve of that of patterns with the form foo/bar+*. I don't think we should support this because I think it's a bit antithetical to what something like foo/bar+json means.

OK, agreed. I only put it in as a matter of completeness (as in, consistently interpreting what looks like intent-to-wildcard), but I agree that there's no known mainstream use case for wildcarding suffixes alone. Have now remove this.

Sounds like this is ready to go in then.
My only remaining question is about the type/subtype; * syntax that this PR adds support for. To recap, this is needed so we know to interpret it as a pattern (matching all additional params), versus type/subtype which is interpreted as a specific concrete media type (not matching anything that specifies extra params). The latter is needed for back-compatibility; the former is needed to satisfy the feature requests we're doing this for. type/subtype; * is a  nonstandard syntax (not in any HTTP spec), but I don't think that alone makes it problematic, because type/*+suffix is equally nonstandard and made up by us.
So my question is: Are the rest of you positively acknowledging that this is what we want? .... because nobody has explicitly commented on it yet. If there was an alternative, it would probably be to tell people to use type/subtype; *=*, which is no more standard, but conveys the same intention.

@SteveSandersonMS if your last question was just about the syntax we should use, I think what you have is fine. I don't think there's an expectation that it be a spec-compliant-syntax because that's not what we're even trying to do here. Just like with files, a "glob" is not a valid "file name," neither is this meant to be a valid pattern for MIME types.

My only remaining question is about the type/subtype; * syntax that this PR adds support for. To recap, this is needed so we know to interpret it as a pattern (matching all additional params), versus type/subtype which is interpreted as a specific concrete media type (not matching anything that specifies extra params). The latter is needed for back-compatibility; the former is needed to satisfy the feature requests we're doing this for. type/subtype; * is a nonstandard syntax (not in any HTTP spec), but I don't think that alone makes it problematic, because type/*+suffix is equally nonstandard and made up by us.

So my question is: Are the rest of you positively acknowledging that this is what we want? .... because nobody has explicitly commented on it yet. If there was an alternative, it would probably be to tell people to use type/subtype; =, which is no more standard, but conveys the same intention.

I think this is good. I'd even go so far as to declare it dope.
/cc @khellang and @tuespetre - who have had opinions on this sort of thing.

I think this looks good! In short; if this lets match application/*+json; version=* without needing to add a type for each new version (I have result filters for mapping versioned representations) or subtype, I'm good with this. I guess I technically also can live with a broader wildcard on media type parameters, i.e. just accept whatever.

@kichalla Great point - thanks for raising this. I will change the order of registration.
@khellang To be sure we're talking about the same thing, could you clarify how exactly you would be registering and using these content types? When you say lets me match, what is doing the matching, and against what input? For choosing the response content type, the way it works is roughly like this:

A client can send a request saying something like Accept: application/vnd.example+json; v=3
Your action method can either declare [Produces("application/vnd.example+json; v=3")], or it can use custom code to populate your ObjectResult's ContentTypes (or Response.ContentType) with anything you like, which could for example be "application/vnd.example+json; v=3". If you use [Produces], you have to specify explicit concrete content types (not with wildcards in them), as described below.
By default, JsonOutputFormatter will agree to write the result for this (because by default it matches application/*+json, which includes any additional params), so the HTTP response will have Content-Type: application/vnd.example+json; v=3; charset=utf-8 or similar, and the body will be JSON-serialized

An important point to understand is that, for security, we do not reflect back arbitrary content type strings from Accept headers to the response. The only response content-types are those you have declared in your own code (e.g., via [Produces] or on an objectresult's ContentTypes). This includes parameters.
Therefore, when you say "lets me match application/+json; version=* without needing to add a type for each new version"*, the answer is that yes, the output formatters will happily do that, matching against whatever is on your objectresult's ContentTypes (which your code put there). But the answer is also no, it doesn't that [Produces] will agree to set the response content-type to an arbitrary, undeclared version parameter value that came from an Accept header. If you really trust that it's safe to reflect back arbitrary values, or even to ignore unknown content type parameters demanded by clients, you can write custom code in your action or a filter to automatically populate ContentTypes based on an incoming Accept header according to your own rules (and don't use [Produces] at all), but that's your own risk.

An important point to understand is that, for security, we do not reflect back arbitrary content type strings from Accept headers to the response.

I don't know that we'd necessarily say that this is for security. I see it more as for sanity.
The server has a finite set of media types that it can serve. Allowing the client to specify the media type without the server applying some sort of verification totally defeats the purpose of using media types.

@SteveSandersonMS Alright, I think I get what you're saying:
So I have to, in addition to add the wildcard media types to the formatter, also annotate my actions with a Produces attribute with a concrete media type. Or I could add the concrete media type to the result.
This is fine because, as I mentioned, I have a global result filter that inspects headers, route- and/or query parameters for a version, then maps the returned model (from the action) into a versioned representation. This means that the result filter can simply add a media type with the correct version parameter and the output formatter will happily pick it up :smile:
I think this is a nice soution 👍
Also, I think not echoing back whatever media type the client asked for is perfectly fine :smile:

🚢🡺✅

Thanks, @SteveSandersonMS! I'll buy you a beer next time we're in Redmond 🍻


Fixup Actions, building more images + adding release tooling

More work towards #393
Remove kernel-ver from bytecode image spec
Add a job which

Creates a release with the bpfd and bpfctl binaries
publishes bpfd-api and bpfctl crates to crates.io

Also update our image building to include the example images + to build with a tagged version whenever a tag is pushed

I'll merge this then push a fake tag to make sure it's working as expected (fingers crossed)


Implement Lazy Loading Pattern For Commit Statuses

Implement Lazy Loading Pattern For Commit Statuses

This tech debt item will be addressed as part of new feature development item #18.


Add latch mechanism to pause/resume in inmemory source

Purpose

Fixes #1603

Codecov Report

Merging #1605 into master will decrease coverage by 0.01%.
The diff coverage is 86.66%.

@@             Coverage Diff              @@
##             master    #1605      +/-   ##
============================================
- Coverage     67.87%   67.85%   -0.02%
  Complexity       45       45
============================================
  Files           654      654
  Lines         27982    27995      +13
  Branches       4403     4405       +2
============================================
+ Hits          18993    18997       +4
- Misses         7158     7167       +9
  Partials       1831     1831

Impacted Files
Coverage Δ
Complexity Δ

...iddhi/core/stream/input/source/InMemorySource.java
93.93% <86.66%> (+13.93%)
0 <0> (ø)
:arrow_down:

.../siddhi/core/util/event/handler/StreamHandler.java
62% <0%> (-12%)
0% <0%> (ø)

...siddhi/core/util/IncrementalTimeConverterUtil.java
52.77% <0%> (-5.56%)
0% <0%> (ø)

.../processor/stream/window/DelayWindowProcessor.java
78.18% <0%> (-5.46%)
0% <0%> (ø)

...ut/stream/state/AbsentStreamPreStateProcessor.java
82.11% <0%> (-1.33%)
0% <0%> (ø)

.../io/siddhi/core/util/snapshot/SnapshotService.java
70.8% <0%> (+0.2%)
0% <0%> (ø)
:arrow_down:

...o/siddhi/core/aggregation/IncrementalExecutor.java
81.25% <0%> (+0.78%)
0% <0%> (ø)
:arrow_down:

...ore/util/persistence/InMemoryPersistenceStore.java
58.97% <0%> (+5.12%)
0% <0%> (ø)
:arrow_down:

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 859d665...9063380. Read the comment docs.


Why set posembed=False?

Hi! I'm very curious about the parameter posembed. Previous methods such as ViTs use positional embedding. But in your method, you set posembed=False. Have you done ablation study about this?

I did not.
I did not use it because I think the positional embedding may prompt the classification performance, but may also affect the ability to accept images with different size. Simply interpolating the positional embedding is clearly a suboptimal solution, so I decided to avoid this problem by omitting it.

Thanks for your reply. I get it.


Can't send table information

I can't seem to send table information through it. Am I doing something wrong?

Convert the table into json then send it.

or you can just make it a string then send it afterwards convert it back into a json


Fake Fullscreen patch misbehaving with chromium.

I found your git while looking for a proper fakefullscreen toggle and your "fullscreen compilation" patch seems to be working well with mpv, firefox and a few other apps.
However on chromium, it misbehaves. So if I toggle fake fullscreen on and then turn fullscreen on (in chrome) it goes offscreen. Basically its showing a zoomed in version.
A workaround I found is to toggle (in app) fullscreen first and then toggle fake fullscreen. But the problem with this is it goes back to the misbehaving state after turning fullscreeen (in app) off and on.
Anyclue on why this is happening or what I can do to fix this? I also installed your full dwm build (instead of just the patch) just to make sure I wasn't missing anything and chromium still misbehaves there as well.

@MaxTendency Hi! Apologies for the tardiness.
Looks like this is a chromium specific issue and has nothing to do with dwm per se. However, I'll let you investigate and let me know if anything comes up, which we could add to the README. (Chrome does behave weirdly at times. For example, there's this issue on chromium which affects pulseaudio output and which really isn't a pulseaudio bug).


AzureRM

Hi!
It says Install AzureRM but your examples uses the new AZ commands.

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

ID: 5f901056-c225-b086-cdaa-c8bd978db64c
Version Independent ID: 9c067eb4-54cb-e141-d82f-e136ff3c671e
Content: Link an Azure account to a partner ID
Content Source: articles/billing/billing-partner-admin-link-started.md
Service: billing
GitHub Login: @dhirajgandhi
Microsoft Alias: banders

@PatLaPoint Thanks for your comment. The feedback has been shared with the content owner for further review. Thanks for your patience.

@azurerm Hi. This article was updated to replace AzureRM commands with new AZ commands. The note you're referring to at the top of the article was added to all Azure articles that had the older Azure RM commands that were replaced.
#please-close

Hi
Ah ok thats why i didnt find them in az,
Rm it is then
Mvh
Patric Sten

From: Bill Anderson<EMAIL_ADDRESS>Sent: Friday, May 10, 2019 4:41:59 PM
To: MicrosoftDocs/azure-docs
Cc: Patric; Mention
Subject: Re: [MicrosoftDocs/azure-docs] AzureRM (#30936)
@azurerm Hi. This article was updated to replace AzureRM commands with new AZ commands. The note you're referring to at the top of the article was added to all Azure articles that had the older Azure RM commands that were replaced.
#please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F30936%23issuecomment-491313451&data=02|01||42e46c309d3e43b9d6f208d6d555a8cf|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636930961205804461&sdata=2il%2FTKLfJo%2F8Mnuwc0RzUcXHABfa8cj5vN3Het9kpdo%3D&reserved=0, or mute the threadhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJLRL7BDXKXNJFO7YMUYC6LPUWCTPANCNFSM4HL44JGA&data=02|01||42e46c309d3e43b9d6f208d6d555a8cf|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636930961205814472&sdata=f8iWc6sg9xkDWkihSV51JYdI4yRAbjMgBgVYwJhBJv4%3D&reserved=0.

Hi
still a bug in the module reference I take.
the Install-Module should be:
Install-Module Az.ManagementPartner

Personally, I use Login-AzAccount so that I can loop through all tenants I have access to, have my MPN ID push to all.


Loop when trying Instructions

Description
When I write 'test' into instructions (searx search enabled)
get an error message
with Azure as Provider
Steps to Reproduce the Bug

go to instructions
write 'test'
get no answer in frontend
get loop in backend

Expected Behavior
get response
Actual Behavior
2023-05-21 15:52:34.596 Using embedded DuckDB with persistence: data will be stored in: agents/Researcher-Azure/memories
2023-05-21 15:52:34.604 loaded in 0 embeddings
2023-05-21 15:52:34.606 loaded in 1 collections
Running Command Execution Validation...
Response: No
2023-05-21 15:52:37.770 error_code=DeploymentNotFound error_message='The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.' error_param=None error_type=None message='OpenAI API error received' stream_error=False
Failed to store memory: The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.
Response: {
"commands": {
"search_searx": {
"query": "how to test code",
"category": "general"
}
}
}
2023-05-21 15:52:38.980 error_code=DeploymentNotFound error_message='The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.' error_param=None error_type=None message='OpenAI API error received' stream_error=False
Failed to store memory: The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.
Running Command Execution Validation...
Response: No
2023-05-21 15:52:34.596 Using embedded DuckDB with persistence: data will be stored in: agents/Researcher-Azure/memories
2023-05-21 15:52:34.604 loaded in 0 embeddings
2023-05-21 15:52:34.606 loaded in 1 collections
Running Command Execution Validation...
Response: No
2023-05-21 15:52:37.770 error_code=DeploymentNotFound error_message='The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.' error_param=None error_type=None message='OpenAI API error received' stream_error=False
Failed to store memory: The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.
Response: {
"commands": {
"search_searx": {
"query": "how to test code",
"category": "general"
}
}
}
2023-05-21 15:52:38.980 error_code=DeploymentNotFound error_message='The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.' error_param=None error_type=None message='OpenAI API error received' stream_error=False
Failed to store memory: The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.
Running Command Execution Validation...
Response: No
Additional Context / Screenshots
No response
Operating System

[ ] Microsoft Windows
[ ] Apple MacOS
[X] Linux
[ ] Android
[ ] iOS
[ ] Other

Python Version

[ ] Python <= 3.9
[X] Python 3.10
[ ] Python 3.11

Environment Type - Connection

[X] Local
[ ] Remote

Environment Type - Container

[ ] Using Docker
[X] Not Using Docker

Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of AGiXT.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

This error is related to your Azure deployment ID being either incorrect or missing, can you confirm the deployment ID in Azure is correct?

:hand:


TST: make sure Incremental works with dataframes

What does this PR implement?
This PR makes sure that dataframes work with Incremental. For tests, I modify the basic incremental test and pass both Dask arrays and dataframes.
Reference issues/PRs

#220: "Verify Incremental works with dask.dataframe"
#454: I found this issue while writing https://github.com/dask/dask-ml/issues/454#issuecomment-463258586
#456, which is on setting minimum dependency versions for CI jobs

This is no longer a work in progress, and ready for review (and maybe merge).
I had to rebase off #455 to make the tests work.

Maybe one more merge of master? I think CI should be passing again.

The CI tests for this PR are passing.

Thanks!


Adds GSOC 2022 Alumni Summit data

add speakers.json
map initial data

 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.


insync: <IP_ADDRESS>925 -> <IP_ADDRESS>499

Description of changes
The insync-v3 package is currently broken in both master and 22.11.

Resolution

Deleting insync-v3 from top-level, superseded by insync.
Using buildFHSEnv to simplify packaging process.
Drop backport attempt for 22.11 for this package going forward.

Related Issues

[BLOCKED] #207829 - Because it has dependencies on python 3.7 packages pyside2 and shiboken2, it also depends on pull request.
[RELATED] closes #186805
[RELATED] closes #139728

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Current the binary crashes, will find the root cause of it.

Update: I managed to make it work, but I still need to make shiboken2 and pyside2 to compile for python 2.7, I am merging upstream code then pushing to this pull request soon.

Pulled out the python package pull requests to 207829

When you are eventually ready to merge this, would you like to take over maintainership of the derivation in nixpkgs?  I'm going to remove myself from the meta.maintainers line since I haven't been using insync for a few years now.

When you are eventually ready to merge this, would you like to take over maintainership of the derivation in nixpkgs? I'm going to remove myself from the meta.maintainers line since I haven't been using insync for a few years now.

Happy to do so, since I am still using it as part of the workflow. I don't know how to add myself to maintainers list yet, I will look into it.

For some reason, the system tray icon is not visible. I am using gnomeExtensions.appindicator.
It seems a chronic bug in upstream insync, I will consider it a known bug and mark this PR ready for now.

@benley do you want me to update the maintainer-list.nix and change the meta.maintainers in the same pull request.

doing some testing now!

I am still having trouble getting insync to start with this PR, both as-is and rebased onto nixpkgs master.  Might I be missing something?
$ nix-build -A insync-v3
... 15 minutes of building pyside2 later ...
$ result/bin/insync start --no-daemon
Warning: Ignoring XDG_SESSION_TYPE=wayland on Gnome. Use QT_QPA_PLATFORM=wayland to run on Wayland anyway.
/nix/store/xch0pgrcpmzs4pvx2iabd603hg8xmpp8-insync-<IP_ADDRESS>481/lib/insync/insync: symbol lookup error: /nix/store/xch0pgrcpmzs4pvx2iabd603hg8xmpp8-insync-<IP_ADDRESS>481/lib/insync/PySide2/plugins/platforminputcontexts/libibusplatforminputcontextplugin.so: undefined symbol: _ZdlPvm, version Qt_5

I am still having trouble getting insync to start with this PR, both as-is and rebased onto nixpkgs master. Might I be missing something?
$ nix-build -A insync-v3
... 15 minutes of building pyside2 later ...
$ result/bin/insync start --no-daemon
Warning: Ignoring XDG_SESSION_TYPE=wayland on Gnome. Use QT_QPA_PLATFORM=wayland to run on Wayland anyway.
/nix/store/xch0pgrcpmzs4pvx2iabd603hg8xmpp8-insync-<IP_ADDRESS>481/lib/insync/insync: symbol lookup error: /nix/store/xch0pgrcpmzs4pvx2iabd603hg8xmpp8-insync-<IP_ADDRESS>481/lib/insync/PySide2/plugins/platforminputcontexts/libibusplatforminputcontextplugin.so: undefined symbol: _ZdlPvm, version Qt_5

That's interesting, which nixpkgs channel are you on?

That's interesting, which nixpkgs channel are you on?
I'm running nixos-unstable (probably a week or two out of date), but I built and ran insync-v3 from your PR branch, and also tried rebasing it onto master.

That's interesting, which nixpkgs channel are you on?

I'm running nixos-unstable (probably a week or two out of date), but I built and ran insync-v3 from your PR branch, and also tried rebasing it onto master.
(edit: formatting oops)

that's a pitty...
I found a similar issue in https://github.com/NixOS/nixpkgs/pull/194779
I will look into more.

That's interesting, which nixpkgs channel are you on?

I'm running nixos-unstable (probably a week or two out of date), but I built and ran insync-v3 from your PR branch, and also tried rebasing it onto master.
(edit: formatting oops)

fwiw, mine looks like:
$ ldd result/lib/insync/PySide2/plugins/platforminputcontexts/libibusplatforminputcontextplugin.so
ldd: warning: you do not have execution permission for `result/lib/insync/PySide2/plugins/platforminputcontexts/libibusplatforminputcontextplugin.so'
	linux-vdso.so.1 (0x00007fff22bfd000)
	libQt5Gui.so.5 => /nix/store/9lcgwnc70f4wj1czklczql7akwcv24mi-qtbase-5.15.7/lib/libQt5Gui.so.5 (0x00007ff2ffd90000)
	libQt5DBus.so.5 => /nix/store/9lcgwnc70f4wj1czklczql7akwcv24mi-qtbase-5.15.7/lib/libQt5DBus.so.5 (0x00007ff30067d000)
	libQt5Core.so.5 => /nix/store/9lcgwnc70f4wj1czklczql7akwcv24mi-qtbase-5.15.7/lib/libQt5Core.so.5 (0x00007ff2ff7e0000)
	libGL.so.1 => /nix/store/ykjcsxdh9c1w664g6v38d86gph8m6mq7-libglvnd-1.5.0/lib/libGL.so.1 (0x00007ff2ff752000)
	libpthread.so.0 => /nix/store/4nlgxhb09sdr51nc9hdm8az5b08vzkgx-glibc-2.35-163/lib/libpthread.so.0 (0x00007ff300678000)
	libstdc++.so.6 => /nix/store/wprxx5zkkk13hpj6k1v6qadjylh3vq9m-gcc-11.3.0-lib/lib/libstdc++.so.6 (0x00007ff2ff53c000)
	libm.so.6 => /nix/store/4nlgxhb09sdr51nc9hdm8az5b08vzkgx-glibc-2.35-163/lib/libm.so.6 (0x00007ff2ff45c000)
	libgcc_s.so.1 => /nix/store/wprxx5zkkk13hpj6k1v6qadjylh3vq9m-gcc-11.3.0-lib/lib/libgcc_s.so.1 (0x00007ff30065c000)
	libc.so.6 => /nix/store/4nlgxhb09sdr51nc9hdm8az5b08vzkgx-glibc-2.35-163/lib/libc.so.6 (0x00007ff2ff253000)
	libpng16.so.16 => /nix/store/8gmgb7m65nafhhqksc71bsfykggsx2ps-libpng-apng-1.6.37/lib/libpng16.so.16 (0x00007ff2ff219000)
	libz.so.1 => /nix/store/zaflwh2nwzj1f0wngd7hqm3nvlf3yhsx-zlib-1.2.13/lib/libz.so.1 (0x00007ff30063c000)
	libharfbuzz.so.0 => /nix/store/9q6s2jbnaikjj7lhxpgy5zaz9biw68bz-harfbuzz-5.3.1/lib/libharfbuzz.so.0 (0x00007ff2ff111000)
	libdbus-1.so.3 => /nix/store/lyvgw6wxb61l2c1krpr2hzprd34cms4v-dbus-1.14.4-lib/lib/libdbus-1.so.3 (0x00007ff2ff0b9000)
	libicui18n.so.72 => /nix/store/wkmc51s1nyyic7ym9c44k57wk8aqlpp9-icu4c-72.1/lib/libicui18n.so.72 (0x00007ff2fed7a000)
	libicuuc.so.72 => /nix/store/wkmc51s1nyyic7ym9c44k57wk8aqlpp9-icu4c-72.1/lib/libicuuc.so.72 (0x00007ff2feb75000)
	libicudata.so.72 => /nix/store/wkmc51s1nyyic7ym9c44k57wk8aqlpp9-icu4c-72.1/lib/libicudata.so.72 (0x00007ff2fcda5000)
	libpcre2-16.so.0 => /nix/store/b33ikpmz4b1gd654v5s62phykmpd58fp-pcre2-10.40/lib/libpcre2-16.so.0 (0x00007ff2fcd15000)
	libzstd.so.1 => /nix/store/9vgv9wx5wf0311j8mxb8zkzc873mnicb-zstd-1.5.2/lib/libzstd.so.1 (0x00007ff2fcc51000)
	libgthread-2.0.so.0 => /nix/store/61qy7d70070djqcg135l22v18a5py758-glib-2.74.3/lib/libgthread-2.0.so.0 (0x00007ff300635000)
	libglib-2.0.so.0 => /nix/store/61qy7d70070djqcg135l22v18a5py758-glib-2.74.3/lib/libglib-2.0.so.0 (0x00007ff2fcb14000)
	/nix/store/4nlgxhb09sdr51nc9hdm8az5b08vzkgx-glibc-2.35-163/lib64/ld-linux-x86-64.so.2 (0x00007ff300711000)
	libGLX.so.0 => /nix/store/ykjcsxdh9c1w664g6v38d86gph8m6mq7-libglvnd-1.5.0/lib/libGLX.so.0 (0x00007ff2fcae0000)
	libX11.so.6 => /nix/store/3zk3527zglm18ykrvgjdhgz3a4zrfapr-libX11-1.8.1/lib/libX11.so.6 (0x00007ff2fc99b000)
	libXext.so.6 => /nix/store/z1zr4xyiaxvdkxc0f1a5cswkvgmix3dw-libXext-1.3.4/lib/libXext.so.6 (0x00007ff2fc986000)
	libGLdispatch.so.0 => /nix/store/ykjcsxdh9c1w664g6v38d86gph8m6mq7-libglvnd-1.5.0/lib/libGLdispatch.so.0 (0x00007ff2fc8ce000)
	libfreetype.so.6 => /nix/store/f1lgywhbb9frvkjkjkbgw2yp63hc462p-freetype-2.12.1/lib/libfreetype.so.6 (0x00007ff2fc806000)
	libgraphite2.so.3 => /nix/store/f0acn4l3g0yiavxjx4c68wad03ah67iq-graphite2-1.3.14/lib/libgraphite2.so.3 (0x00007ff2fc7de000)
	libsystemd.so.0 => /nix/store/iap1if4pm6h6sacydm344p38j182c189-systemd-minimal-252.1/lib/libsystemd.so.0 (0x00007ff2fc6f4000)
	libpcre2-8.so.0 => /nix/store/b33ikpmz4b1gd654v5s62phykmpd58fp-pcre2-10.40/lib/libpcre2-8.so.0 (0x00007ff2fc656000)
	libxcb.so.1 => /nix/store/6031yyvcgc64qm40s5vj7rblz7fwbvnj-libxcb-1.14/lib/libxcb.so.1 (0x00007ff2fc62b000)
	libbz2.so.1 => /nix/store/kxdyx0dqy8gpx8yp61wybn2dr1va0vn8-bzip2-1.0.8/lib/libbz2.so.1 (0x00007ff2fc618000)
	libcap.so.2 => /nix/store/9vfggw8qmvdxhb1yj9yf8crqndg5xc08-libcap-2.66-lib/lib/libcap.so.2 (0x00007ff300623000)
	libXau.so.6 => /nix/store/fh0ih93xv6ifvgidjknydvc950mlch3r-libXau-1.0.9/lib/libXau.so.6 (0x00007ff2fc611000)
	libXdmcp.so.6 => /nix/store/iwb5zaid6mxkivhl5p0xykgmsdj0mzjv-libXdmcp-1.1.3/lib/libXdmcp.so.6 (0x00007ff2fc609000)

$ nix-build $PWD -A insync-v3
/nix/store/xch0pgrcpmzs4pvx2iabd603hg8xmpp8-insync-<IP_ADDRESS>481

But I guess we should trust reproduciable builds...

I am still having trouble getting insync to start with this PR, both as-is and rebased onto nixpkgs master. Might I be missing something?
$ nix-build -A insync-v3
... 15 minutes of building pyside2 later ...
$ result/bin/insync start --no-daemon
Warning: Ignoring XDG_SESSION_TYPE=wayland on Gnome. Use QT_QPA_PLATFORM=wayland to run on Wayland anyway.
/nix/store/xch0pgrcpmzs4pvx2iabd603hg8xmpp8-insync-<IP_ADDRESS>481/lib/insync/insync: symbol lookup error: /nix/store/xch0pgrcpmzs4pvx2iabd603hg8xmpp8-insync-<IP_ADDRESS>481/lib/insync/PySide2/plugins/platforminputcontexts/libibusplatforminputcontextplugin.so: undefined symbol: _ZdlPvm, version Qt_5

Actually I got that during exiting of the insync instead:
$ ./result/bin/insync start --no-daemon
QXcbIntegration: Cannot create platform OpenGL context, neither GLX nor EGL are enabled
QSystemTrayIcon::setVisible: No Icon set
INFO     2022-12-29 16:40:36,692 [mainlogs:_log_run:138] Core(app_version=<IP_ADDRESS>481, platform=Linux-x86_64-nixos/22.11) initialized
INFO     2022-12-29 16:40:36,739 [mainsettings:_load_settings:236] Loaded settings, machine fingerprint = b29xkLEt4XijophExAsrl-hTWys=
INFO     2022-12-29 16:40:36,839 [inotify_manager:__init__:132] INotifyManager FS_ENCODING: utf-8
INFO     2022-12-29 16:40:36,840 [fswatcher:_start:57] LinuxFSWatcher._start
INFO     2022-12-29 16:40:36,840 [inotify_manager:_pull_loop:315] Inotify loop enter
INFO     2022-12-29 16:40:37,032 [fswatcher:watch:77] watch origin: /home/hellwolf/Documents/Personal/Google
INFO     2022-12-29 16:40:37,035 [app:start_core:65] core started
INFO     2022-12-29 16:40:37,035 [unix_socket_server:start:106] unix socket server thread start
js: Access to XMLHttpRequest at 'https://app.posthog.com/decide/?v=2&ip=1&_=1672324837152&ver=1.26.0' from origin 'file://' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, chrome-untrusted, https.
js: Bad HTTP status: 0
js: Enqueued failed request for retry in 6000
js: Access to XMLHttpRequest at 'https://app.posthog.com/e/?ip=1&_=1672324837336&ver=1.26.0' from origin 'file://' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, chrome-untrusted, https.
js: Bad HTTP status: 0
js: Enqueued failed request for retry in 6000
js: Access to XMLHttpRequest at 'https://app.posthog.com/decide/?v=2&ip=1&_=1672324838052&ver=1.26.0' from origin 'file://' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, chrome-untrusted, https.
js: Bad HTTP status: 0
js: Enqueued failed request for retry in 6000
js: Access to XMLHttpRequest at 'https://app.posthog.com/e/?ip=1&_=1672324841053&ver=1.26.0' from origin 'file://' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, chrome-untrusted, https.
js: Bad HTTP status: 0
js: Enqueued failed request for retry in 6000
INFO     2022-12-29 16:40:44,629 [app:quit:80] quitting
INFO     2022-12-29 16:40:44,630 [unix_socket_server:stop:126] unix socket server thread stop
INFO     2022-12-29 16:40:44,636 [fswatcher:_stop:70] LinuxFSWatcher._stop
INFO     2022-12-29 16:40:44,659 [app:stop:75] core stopped
INFO     2022-12-29 16:40:44,660 [app:stop:77] clean exit
/nix/store/xch0pgrcpmzs4pvx2iabd603hg8xmpp8-insync-<IP_ADDRESS>481/lib/insync/insync: symbol lookup error: /nix/store/xch0pgrcpmzs4pvx2iabd603hg8xmpp8-insync-<IP_ADDRESS>481/lib/insync/PySide2/plugins/platforminputcontexts/libibusplatforminputcontextplugin.so: undefined symbol: _ZdlPvm, version Qt_5

Here you go, I got lots of great help from @lilyinstarlight  in NixOS matrix dev channel. This new version should work more reliably now.
@benley please test again!

Hey there !!
I don't want to bother but what is the actual state of insync?
I tried to install it from the unstable channel but I encountered several problems.
Thanks
~S

I added insync from this PR to my config and it works perfectly fine. The only issue I see is that the tray icon has no image.

Hey all, yes I have been using, happy to push to merge if more people tested it.
And yes, a known issue is that the insync tray icon doesn't show image correctly. I have fought this issue with  @lilyinstarlight for a few hours, and the conclusion was it was it's very likely due to insync packaging decided to sanitizing the input required XDG_ environment variable and causing not reading icons correctly on Nix.
If you all okay with this known bug before we can find a new workaround, I will push to the right force to help us merge this!

just noticed that python37 has been removed from nixpkgs

Silly me, I synced with master branch today, and I didn't realize python37 was removed. fwiw, you can apply this patch to 22.11 branch if you need it urgently.
Unfortunately insync has silly packaging, currently this one relies on python 3.7. The other package from insync against the latest ubuntu though relies on glibc 2.36, so it can't be used neither... I will watch the situation and see what I can do to fix it..

For issue/pr cross-referencing, the glibc 2.36 update is #188492

Taking a note re libpython3 libpython3.x. https://stackoverflow.com/questions/39863827/what-is-libpython3-so-compared-with-libpython3-5m-so-built-from-python-3-5-2-sou for future reference and people who are also curious about it.

@Artturin would it still make sense to back port it to 22.11 where python37 is available, while hope insync's new binary moving to newer python?

Hey folks, would it be a terrible idea if we simply wrap this with run-steam?

Okay, actually it may be a good time to ask everyone in this thread a question, considering where we are:

Upstream insync-v3 does not seem to move on from python37 anytime soon.
As a result, packaging is quite a challenge since all python37 dependencies must be rebuilt for this package, especially pyside for Qt.
Currently only on 22.11 this package can still be built, though some fundamental issue still persists re notification try icon.

Instead of giving up and abandoning this package, I have a proposal, how about we change it to using steam-run as runner instead? Downside is that whoever needs this package is damned to have the entire steam-run dependency; but upside is it becomes much easier to package.
I will give two vote options, please signal your preference:

:+1: yes steam-run
:-1:  noooooo

Good news folks! It works now! Without using full steam-run, instead with a very minimum FHS! Please give it a try!
Although the system tray icon still not working:
$ result/bin/insync-<IP_ADDRESS>499 start --no-daemon
QApplication: invalid style override passed, ignoring it.
    Available styles: Windows, Fusion
QSystemTrayIcon::setVisible: No Icon set

I will give a few more debugging of this before call it known bug and consider we are good to go for this package again!

@ofborg build insync

Everyone this is ready now!!

Removed insync-v3 from top-level, with only insync  superseding it.
insync: <IP_ADDRESS>925 -> <IP_ADDRESS>499

Simplified packaging process:

Use as much vendor provided binaries as possible.
Only replace glibc with system provided ones (2.36+ required, hence
nixos 22.11 is out).
Use buildFHSEnv to create FHS environment to run insync in.

Known bug(s):

Currently the system try icon does not render correctly.
libqtvirtualkeyboardplugin does not have necessary Qt library shipped from vendor.

Please don't use steam-run as it is giant and has a proprietary dependencies on steam, use a plain fhs instead and add the libraries that are required to it.

Yep, using buildFHSEnv now after other recommending, works like a charm.

Anything more I can do to help to get this merged?

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

Good day everyone! I have fixed the alias issue as suggested by @SuperSandro2000 , can this still make to the 2023-05-15 timeline?

That timeline is not relevant for unimportant leave packages.


Remove 24.10 deprecated commands

Description
This PR removes the deprecated functions rapids_cpm_thrust,  rapids_cpm_libcudacxx and rapids_cuda_patch_toolkit.
Merges together https://github.com/rapidsai/rapids-cmake/pull/664 and https://github.com/rapidsai/rapids-cmake/pull/661 together to have passing CI
Checklist

[ ] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[ ] The documentation is up to date with these changes.
[ ] The cmake-format.json is up to date with these changes.
[ ] I have added new files under rapids-cmake/

[ ] I have added include guards (include_guard(GLOBAL))
[ ] I have added the associated docs/ rst file and update the api.rst

Thanks for the assistance here @robertmaynard! Hope the starting point was helpful.

You did all the hard work, was just one more test that needed an update

/merge


Nested scrollview, parent has paging, child does not. Both scrollviews result in paging enabled

https://rnplay.org/apps/U-k6oA
Can see the example here. The parent scrollview has pagingEnabled={true}, the child has it explicitly set to false. However, the child one has the page-snap to it when you try to scroll. The rnplay example is using 0.11, but i have this in my own app in version 0.12 as well.
Sample code is here as well:
'use strict';

var React = require('react-native');
var {
  AppRegistry,
  Dimensions,
  StyleSheet,
  Text,
  ScrollView,
  View,
} = React;

var SampleApp = React.createClass({
  render: function() {
    var screen = Dimensions.get("window");
    return (
      <View style={styles.container}>
        <ScrollView style={{width: screen.width, flex: 1}} horizontal={true} bounces={false} showsHorizontalScrollIndicator={false} pagingEnabled={true} contentInset={{top:-20}}>
        	<ScrollView bounces={true} style={{width: screen.width}} pagingEnabled={false}>
            <Text style={styles.welcome}>
              Welcome to the React Native Playground! This is the sample iOS app.
            </Text>
            <Text style={styles.instructions}>
              Just edit and save with Cmd+S. Changed will be reflected immediately in the simulator.
            </Text>

            <Text style={styles.instructions}>
              To try out Android, just flip the switch and edit index.android.js!
            </Text>
            <Text style={styles.welcome}>
              Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.
            </Text>
          </ScrollView>
          <ScrollView style={{width: screen.width}} pagingEnabled={false}>
            <Text style={styles.welcome}>
              Welcome to the React Native Playground! This is the sample iOS app.
            </Text>
            <Text style={styles.instructions}>
              Just edit and save with Cmd+S. Changed will be reflected immediately in the simulator.
            </Text>

            <Text style={styles.instructions}>
              To try out Android, just flip the switch and edit index.android.js!
            </Text>
            <Text style={styles.welcome}>
              Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.
            </Text>
          </ScrollView>
		    </ScrollView>
      </View>
    );
  }
});

var styles = StyleSheet.create({
  container: {
    flex: 1,
    flexDirection: 'column',
    justifyContent: 'center',
    alignItems: 'center',
    backgroundColor: '#F5FCFF',
  },
  welcome: {
    fontSize: 28,
    textAlign: 'center',
    margin: 10,
  },
  instructions: {
    textAlign: 'center',
    color: '#333333',
    fontSize: 19,
    marginBottom: 5,
  },
});

AppRegistry.registerComponent('SampleApp', () => SampleApp);

What platform is this?

@astreet ios

Additionally I have a similar case on iOS where pagingEnabled supported at.
A horizontal ScrollView is parent of a few ListViews and pagingEnabled on the ScrollView.
This ScrollView does not let the child ListViews scroll continuously, that is without paging.
It can be accepted that paging direction is deduced from the ScrollView direction, however better it is a explicit property. Currently ReactNative sets it for both directions.
It is certain that paging enabling should have explicit direction values like:
pagingEnabled enum(horizontal, 'vertical', 'both')

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/nested-scrollview-parent-has-paging-child-does-not-both-scrollviews-result-in-paging-enabled
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub.
Also, if this issue is a bug, please consider sending a PR with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.


[Feature request] quick interface opening

This would be opening the interfaces of things like crafting tables and stonecutters (maybe even anvils, with the normal change to damage the anvil in item form).

Actually, yes, I like this suggestion. That's exactly what this mod was made for ;)


MM-25694 Disallow profile picture update is set by LDAP

Summary
Do not allow the user to update their profile picture if the account is using LDAP or SAML to login and the profile picture attribute is set
Ticket Link
https://mattermost.atlassian.net/browse/MM-25694

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/6531
https://circleci.com/gh/mattermost/mattermost-mobile/6530

Artifact links:
https://6531-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa
https://6530-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk


requirements: Upgrade sphinx to 1.6.4.

This fix was merged to sphinx 1.6.4:
https://github.com/sphinx-doc/sphinx/commit/9f889932815dea46f8e97c732aaa8216e4b37c08
That was previously causing problems building the zulip docs.

Nice, merged, thanks for tracking that bug @rodrigc!  The test failure is an unrelated flake that we really need to track down.
@hackerkid FYI :)


chore: update typo

I believe this part means older projects that aren’t actively maintained (?)

Ah I see the confusion now. No I think I understand that these refer to the replacements too. I think it comes to this:
When I read the sentence, I assume  which are actively maintained (unlike the source module) refers to the older packages, not these. So I'm confused why older packages are still actively maintained.
Maybe we should add a comma before the which? Or I think if others read the way I read too, this PR also works, because it implies these to be more actively maintained.

Sometimes these may also be actively maintained forks of older, unmaintained source packages.

maybe this will do better?


Broken template and links

I forked the repo and have edited the navigation.yml and config.yml files but when I go to the deployed website none of my links work and the website looks like the picture below. Any solution for this?

This looks like a style sheet wasn't loaded by your web browser and can happen to an error in the configuration that prevents it from building correctly. Can you link to the repository that you are deploying from? Also, what web browser are you using to view the site?

I am using firefox. Here is the link to the repository https://github.com/Jvogie/website. Thanks for the help

It looks like there might be a misconfiguration in the [_config.yml](https://github.com/Jvogie/website/blob/master/_config.yml) - where are you deploying the website to?

I am just using github pages to deploy to https://jvogie.github.io/website/

That's likely the problem since the template is fairly strict about pathing due to how GitHub Pages works - if you update GitHub Pages to deploy to jvogie.github.io then it should work.


Remove section definition in index.html

Discussed in https://github.com/h-enk/hyas/discussions/294

Originally posted by ruslanbes May 29, 2022
Hi,
I'm new to Hugo so sorry if the question is obvious.
I'm confused about this code in the https://github.com/h-enk/hyas/blob/master/layouts/index.html
{{ define "section" }}
<section class="section container-fluid">
  {{- .Content -}}
</section>
{{ end }}

This code affects the homepage as I understand, but If I build the site I don't see this block at all.
I wonder how is it used and why the content is included as {{- .Content -}} and not {{ .Content }} ?

236278aceea3e3021abd1ab09300081f283e4027


[ISSUE #5748] Fix slave's store.log has many invalid log outputs in master-slave deploy mode

Make sure set the target branch to develop
What is the purpose of the change
Close #5748
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

In my opinion, errors should not be ignored when writing messages, which will increase the complexity of troubleshooting. May temporarily keep consistent with the original implementation works well.

Rease to make Bazel CI pass

Codecov Report

Merging #5749 (f8e0be3) into develop (e986534) will increase coverage by 0.02%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #5749      +/-   ##
=============================================
+ Coverage      42.40%   42.43%   +0.02%
  Complexity      8226     8226
=============================================
  Files           1060     1060
  Lines          73347    73346       -1
  Branches        9595     9595
=============================================
+ Hits           31105    31125      +20
+ Misses         38332    38307      -25
- Partials        3910     3914       +4

Impacted Files
Coverage Δ

...main/java/org/apache/rocketmq/store/CommitLog.java
65.37% <ø> (+0.06%)
:arrow_up:

...org/apache/rocketmq/store/ha/WaitNotifyObject.java
80.35% <0.00%> (-5.36%)
:arrow_down:

...tmq/remoting/protocol/body/ConsumerConnection.java
95.83% <0.00%> (-4.17%)
:arrow_down:

...ache/rocketmq/broker/client/ConsumerGroupInfo.java
69.29% <0.00%> (-3.51%)
:arrow_down:

...pache/rocketmq/remoting/common/RemotingHelper.java
15.38% <0.00%> (-2.31%)
:arrow_down:

...pl/consumer/ConsumeMessageConcurrentlyService.java
47.05% <0.00%> (-2.27%)
:arrow_down:

...ava/org/apache/rocketmq/filter/util/BitsArray.java
58.11% <0.00%> (-1.71%)
:arrow_down:

...he/rocketmq/client/trace/AsyncTraceDispatcher.java
82.21% <0.00%> (-1.45%)
:arrow_down:

...he/rocketmq/client/impl/consumer/ProcessQueue.java
60.61% <0.00%> (-1.33%)
:arrow_down:

...cketmq/broker/schedule/ScheduleMessageService.java
56.91% <0.00%> (-1.14%)
:arrow_down:

... and 9 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Duplicated bottom bar in some cases

@stephanegigandet any specific steps to reproduce ?

scan a product -> edit product -> press back


Closure not recognizing constructors/methods on namespaced globals

(repro-able in http://codemirror.net/demo/tern.html)
If I try to autocomplete at the | in
var Bar = function() {};
Bar.prototype.a = function() {
  this.|
};
Bar.prototype.b = function() {};

I get a and b as options, as expected. But when I try it in
foo.Bar = function() {};
foo.Bar.prototype.a = function() {
  this.|
};
foo.Bar.prototype.b = function() {};

I get nothing, because tern doesn't recognize foo.bar to be the constructor. This makes Tern not really work for files written to work with the Closure library's/compiler's standard namespacing system, as in https://github.com/google/closure-library/blob/master/closure/goog/array/array.js.
The example would work if I had a var foo = {} line above, but that normally happens as part of a call above like goog.provide('foo.Bar') (cf https://github.com/google/closure-library/blob/master/closure/goog/base.js#L268). Since that happens via a lot of indirection, tern unsurprisingly can't pick it up even when the whole base.js is pasted into the file...

Tern needs to have an object type for (in your case) foo to even be able to represent the path foo.Bar. Have you tried using the plugin at https://github.com/google/tern-closure ? That supposedly implements goog.provide and might solve your issue.

I'm aware of that plugin and we're working on getting it up and running in a web browser (since it was built to run on node.js). But that's still a ways out, so I was hoping you'd be able to suggest some interim solution, such as some way that I'd be able to tell tern to (say) treat all ? objects as {} or something like that.

This problem has come up before and in other contexts, and after some thinking, I think I found a rather elegant solution. Attached patch should help with this.


Use InfoCardVariants on custom cards variant attribute

Resolves #10951
Signed-off-by: Damien Vitrac<EMAIL_ADDRESS>

Thanks for this @oodamien!


[Bug]: Border outline color not working again

What happened?
I'm having the same problem as described at https://github.com/199ocero/radio-deck/issues/2.
How to reproduce the bug
Trying with any color reproduces the bug. Maybe some breaking change with Filament v3.2?
Package Version
1.2.1
PHP Version
8.2
Laravel Version
10
Which operating systems does with happen with?
Linux
Notes
No response

Not working on both light and dark theme:

Hi @miguilimzero I can confirm that this is working with latest filament

Maybe you miss a configuration. Please see this issue: #8

It seems I was missing the filament preset on tailwind.config.js. The strange thing is that only the outline color was not working; everything else was fine. Thanks for the support @199ocero!


fix: Installation error when using --no-sudo flag due to lack of privileges

When using the --no-sudo flag, the installation will fail as non-root
user have no privileges for /usr/local/bin directory. Changed the
default $HELM_INSTALL_DIR to $HOME/.local/bin when using --no-sudo flag, and allow user to specify install path using --install-dir or -i flag.
This issue was opened in (refs #6040) but was not resolved and closed due to inactivity (there was a proposed fix later in the issue but not commited).
CHANGES

Added if-else statement to check if using root/non-root user, and run respective cp command. (scripts/get-helm-3#L172-176)
Added $HOME/.local/bin as default installation path if using --no-sudo flag.
Added --install-dir path for user to specify install path.

Signed-off-by: Daniel Tan<EMAIL_ADDRESS>

Why the use of ~/.local/bin as the location? Is that consistently the location people will use? I ask because there are situations, such as a mac homebrew user, who may have access to /usr/local/bin without being root. While I didn't write the original code, I would not be surprised if it was tested there.
There is an environment variable where you can set the location for the binary.

Why the use of ~/.local/bin as the location? Is that consistently the location people will use?
I'm not a Mac user, but for other Linux-based OS, the user might not have access to /usr/local/bin if they lack sudo privileges. I'm not sure if most developers would prefer the path ~/.local/bin. That being said, the installation guide for kubectl suggests ~/.local/bin for non-root users (ref. Therefore I would assume that ~/.local/bin would work well as a default path for non-sudoers.

There is an environment variable where you can set the location for the binary.
Sorry for my ignorance, but is the environment variable $HELM_INSTALL_DIR? Following the installation instructions from https://helm.sh/docs/intro/install/, I fail to install the binary to the desired path even when I've set the variable to ~/.local/bin.

I'm new to helm, so I would be happy to update the installation manual (documentation) if that's a better solution for helping new developers like me to have a better installation experience.
Cheers.

@mattfarina Any thoughts?


Multiple open to the side brings layout in a broken state

Refs: https://github.com/microsoft/vscode/issues/79845

Have a smaller vscode window
Open explorer and start clicking on the Open to the Side context menu command
Every editor is opened in a new group reducing the available horizontal space
When there is no more space the explorer is no longer visible and the layout is in a broken state since I can no longer click on any of the Sashes to resize :bug:

Notice on the gif that I am trying to find the sashes but am not able

Note that I can more easily getting into the sashes are broken state by having more than 3 groups side by side. Somehow the sashes are always off then. Adding label bug.

Sashes now don't show, that's good, but I still want what terminal split does:

@octref what makes this tricky today is that the editor can increase its size beyond the currently allotted space and should expand, but agreed that this would be an improvement.


Modern Fortran in Science and Technology by Lindemann and Dahlblom

Modern Fortran in Science and Technology could be added to the Online Courses section. I skimmed it, and it seems pretty thorough.

Yes, we should. Although the first place I randomly clicked: https://modern-fortran-in-science-and-technology.readthedocs.io/en/latest/chapter_fortran.html#arrays-and-matrices and it has a mistake, they assign 5.0 to a double precision number, which will only assign the first 7 digits or so instead of 15.

They illustrate this issue with a sample program
program constants
    implicit none
    integer, parameter :: dp = selected_real_kind(15,300)
    real(dp) :: pi1, pi2
    pi1 = 3.141592653589793
    pi2 = 3.141592653589793_dp
    write(*,*) 'pi1 = ', pi1
    write(*,*) 'pi2 = ', pi2
    stop
end program constants

Ok, I submitted a PR: https://github.com/jonaslindemann/modern_fortran_book/pull/1

I will look into the issues.

I hope I have fixed it in most places.
Thanks for the input.

Thanks @jonaslindemann!
Yes, we should link it.
Btw, currently when I open the page: https://modern-fortran-in-science-and-technology.readthedocs.io/, I am getting "Warning: Potential Security Risk Ahead" in Firefox and Chrome and it does not open for me. It could be an issue on my end.

I just tried with FireFox myself and had no such problem.
Op di 29 jun. 2021 om 15:48 schreef Ondřej Čertík @.***

:

Thanks @jonaslindemann https://github.com/jonaslindemann!
Yes, we should link it.
Btw, currently when I open the page:
https://modern-fortran-in-science-and-technology.readthedocs.io/, I am
getting "Warning: Potential Security Risk Ahead" in Firefox and Chrome and
it does not open for me. It could be an issue on my end.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fortran-lang/fortran-lang.org/issues/285#issuecomment-870616154,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN6YR4A3SI3OA55X5675VLTVHFKFANCNFSM47PCT32A
.

I don't have any problems myself. It is built and hosted automatically by readthedocs using Sphinx. Thanks for linking it by the way!
Jonas

No problem with MS Edge either, so it seems indeed to be on your side (or
it is a temporary thing)
Op di 29 jun. 2021 om 15:50 schreef Arjen Markus @.***

:

I just tried with FireFox myself and had no such problem.
Op di 29 jun. 2021 om 15:48 schreef Ondřej Čertík <
@.***>:

Thanks @jonaslindemann https://github.com/jonaslindemann!
Yes, we should link it.
Btw, currently when I open the page:
https://modern-fortran-in-science-and-technology.readthedocs.io/, I am
getting "Warning: Potential Security Risk Ahead" in Firefox and Chrome and
it does not open for me. It could be an issue on my end.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fortran-lang/fortran-lang.org/issues/285#issuecomment-870616154,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN6YR4A3SI3OA55X5675VLTVHFKFANCNFSM47PCT32A
.

Perfect, I am glad it is only on my end. I'll try again in a few days.


Refactor loading powerplant data in add_electricity.py

I think we should refactor the loading of powerplants in add_electricity.py to make it easier to add new datasets in the future.
For example there is quite a bit of overlap in this conditional:
https://github.com/PyPSA/pypsa-usa/blob/7575652bbb717ef1e7b543d7bf42b29afba1e57e/workflow/scripts/add_electricity.py#L1382-L1553
We should be able to generalize all the attach_batteries(..), attach_conventional(..), add_missing_fuel_costs(...), ect. if we set up some standard data structures. That way the only unique logic will be the read in and formatting of the base dataset :)

Agree'd. Though, depending on how we integrate with pypsa-earth the changes here would be different. PyPSA earth has an  option to replace plant data with csv format. We would use that function, but then would need to create alternate functions for non-atlite profiles etc.
I started thinking more about the integration last week, and will have some discussion points for next week's meeting.


Explain how to setup Erlang distribution

Add explanation to setup erlang distribution with Elixir on GRiSP
Aims to solve https://github.com/grisp/mix_grisp/issues/15

@Rashidwi Hi, with these settings I was able to get a nodename, can you try it too?
I'll wait your feedback.
u[ERL] GRiSP mountpoint from the FDT: /media/mmcsd-1-0/
hub1: <Freescale EHCI root HUB, class 9/0, rev 2.00/1.00, addr 1> on usbus1
[ERL] Reading /media/mmcsd-1-0/grisp.ini
[ERL] Booting with arg: erl.rtems -C multi_time_warp -- -noshell -noinput -mode embedded -home . -pa . -root myapp -bindir myapp/erts-15.0.1/bin -boot myapp/releases/0.1.0/start -boot_var RELEASE_LIB myapp/lib  -config myapp/releases/0.1.0/sys.config -s elixir start_iex -kernel inetrc "./erl_inetrc" -internal_epmd epmd_sup -sname bob -setcookie cocky -extra --no-halt
[ERL] hostname: grisp-001159
[ERL] Setting up /etc from /media/mmcsd-1-0/etc
[ERL] Copying directory /media/mmcsd-1-0/etc to /etc
[ERL] Copying file /media/mmcsd-1-0/etc/dhcpcd.conf to /etc/dhcpcd.conf
[ERL] Copying file /media/mmcsd-1-0/etc/hosts to /etc/hosts
[ERL] Copying file /media/mmcsd-1-0/etc/resolv.conf to /etc/resolv.conf
[ERL] Starting DHCP
[ERL] mkdir /tmp
info: ffec0: link state changed to DOWN
[ERL] mkdir /tmp/log
[ERL] mkdir /home
[ERL] Setting environment
[ERL] chdir(/media/mmcsd-1-0/)
[ERL] getcwd: /media/mmcsd-1-0
[ERL] Starting BEAM
[ERL] No shell mode
uhub0: 1 port with 1 removable, self powered
uhub1: 1 port with 1 removable, self powered
ugen1.2: <Realtek 802.11n NIC> at usbus1
rtwn0: <Realtek 802.11n NIC, class 0/0, rev 2.00/0.00, addr 2> on usbus1
rtwn0: MAC/BB RTL8188EU, RF 6052 1T1R
warning: the VM is running with native name encoding of latin1 which may cause Elixir to malfunction as it expects utf8. Please ensure your locale is set to UTF-8 (which can be verified by running "locale" in your shell) or set the ELIXIR_ERL_OPTIONS="+fnu" environment variable
Erlang/OTP 27 [erts-15.0.1] [source] [32-bit] [smp:1:1] [ds:1:1:10] [async-threads:1]

Interactive Elixir (1.16.3) - press Ctrl+C to exit (type h() ENTER for help)
iex(bob@grisp-001159)1>

Hi Ziopio, many thanks, that works fine, and remsh works as expected.
Given the somewhat extensive length of the boot arg, is there anyway these settings could be automated, could sname and cookie come from a file?

You can write -sname{{release_name}} to fill it automatically with the app name, but you cannot escape the fact that we require these options to be put in the grisp.ini
I can tell you that if you omit -setcookie, Elixir will search for ./.erlang.cookie to read the cookie from such file. The problem is that we would need to move such file in the right location before VM initialization. I am not sure this can be done.
We recently developed a rebar3 grisp configure command for rebar3_grisp that makes it easy to assemble the correct grisp.ini and all other project settings. Unfortunatly this is only available in rebar3 for Erlang projects, and done mainly to easely enable users to connect their board with https://grisp.io/.
Some of the logic is present in grisp_tools so we support an adoption in mix_grisp, but that is some work and we are not planning it right now.


Show output of cpc dpm-import command

If the API call returns with a 200 code, additional details are included in the response.
This is an example how the output will look like in this case:
Partially imported DPM configuration from DPM config file ./export.yaml into CPC '1X091404'. The following parts were not restored:
+---------------------+------------+
| category            | items      |
|---------------------+------------|
| boot-password-reset | partition2 |
|                     | partition1 |
+---------------------+------------+

Fixes #342

Coverage: 31.475% (-0.04%) from 31.511% when pulling a6952701083ca65ead19fc3adb42b426d8238f9a on simon/dpmimportoutput into ae916ca4937e2a92fbd295e66d0fdf3134cb26b3 on master.