Master to Live for a number of updates including Set-CsConferencingPolicy.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/ConvertTo-JsonForPSWS.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsBusyOptions.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsConferencingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Export-SPOQueryLogs.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Convert-SPWebApplication.md
:white_check_mark:Succeeded
View (sharepoint-ps)

skype/skype-ps/skype/ConvertTo-JsonForPSWS.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsBusyOptions.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsConferencingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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


Make tab and badge styling global

Previously, the .md-tabs and .badge tweaks were only applied to the notifications page. This change is necessary for the new search page to appear correctly.

Coverage remained the same at 33.517% when pulling aa3e94f690e67ba2ef0340babb2e2a49829287f4 on thevoiceofzeke:tabs into fa8e379b7d3bfffe04f8b7f1e0191e74e7bde729 on UW-Madison-DoIT:master.


Search the hestia cycles for a cycle that has a specific output

# all referring to staging-api
node = bhb.get_hestia_node({"@type": "cycle", "@id": "cx_dtpwud1iu"})
node_output = [i for i in node["products"] if i["term"]["@id"] == "bananaFruit"][0]

# TODO: Search the hestia cycles if a cycle has node_output as an input
# Perhaps in this style?:
bhb.search_hestia({"cycle.inputs.term.@id": node_output["term"]["@id"]})

# And probably in other contexts as well, like looking for cycles with a specific product
node_input = [i for i in node["inputs"] if i["term"]["@id"] == "saplings"][0]
# Perhaps in this style?
bhb.search_hestia({"cycle.products.term.@id": node_input["term"]["@id"]})

os.environ["API_URL"] = 'https://api-staging.hestia.earth' #'https://api.hestia.earth'
os.environ["SEARCH_URL"] = 'https://search-staging.hestia.earth' #'https://search.hestia.earth
from hestia_earth.schema import SchemaType
from hestia_earth.utils.api import find_node, download_hestia, find_node_exact

node = find_node(SchemaType.CYCLE, {"products.term.name": 'Saplings'})
node = download_hestia("exqfz4a8clsx", "cycle")
node["products"]
find_node(SchemaType.CYCLE, {"products.term.@id": 'saplings'})


fix: set soft focus for property pane inputs

Description
Removes cursor focus which was added in https://github.com/appsmithorg/appsmith/commit/17dbe63ed3113babf7a6da71e26e351c792b1883 reverts back to soft focus for property pane fields.
Also fixes cypress test https://cloud.cypress.io/projects/eyxvp8/runs/27243/test-results/7afa85a1-5497-4c3c-810f-de52a9ccfa4b
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual
Cypress

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=5107d4f


RFM - thebrain

I DON'T Want To Become The Maintainer

[x] I have followed the Package Triage Process and I do NOT want to become maintainer of the package;
[x] There is no existing open maintainer request for this package;

Checklist

[x] Issue title starts with 'RFM - '

Existing Package Details

Package URL: https://chocolatey.org/packages/thebrain
Package source URL: https://github.com/dtgm/chocolatey-packages/tree/master/automatic/thebrain/

Date the maintainer was contacted (in YYYY-MM-DD): 2021-12-13
How the maintainer was contacted: email

this request covers the packages thebrain.install and thebrain.portable as well.

Hello,
This one is done and approved.
You can close it.
Thank you.


Possible bug in miog_beta.cl (generated kernel)

Hi @newling,
I've been having some problems with the miog_beta.cl kernel, which is generated by MIOpenGEMM (it is possible that #9 fixes this, I'm trying your suggestion from there now).  My tracing of the bug has led me back to something in the code I would like some clarification on.  Specifically, the part of the code I'm trying to understand is:
for (TSHORT i = 0; i < WORK_PER_THREAD; ++i){
      /* beta scaling */
      **if (beta <= 0 && beta >= 0)**{c[i] = 0;}else{c[i] *= beta;}
}

The part I'm trying to understand is starred above.  The only case where the if would be true is if beta (a float) is exactly 0.  But if the intent was to check if beta was 0, it would seem like there is a much simpler way to check for this (I realize that checking if a float is exactly 0 is frowned upon).
More broadly, I have two questions about MIOpenGEMM:

Regardless of whether the above is a bug or not, how would I go about fixing it?  Where are these kernels being generated from within MIOpenGEMM?  Maybe in basegenerator.hpp and bylinegenerator.hpp?

EDIT: I see that the specific line in question is in betacgenerator.cpp, but would still appreciate some help understanding how this single line ties together with everything else.

Is there a way to look at these OpenCL kernels directly in MIOpenGEMM?  So far the only way I've found to get the "original" source code is to force MIOpen to emit it before it compiles the kernel, but this is very manual and error prone.

Thanks,
Matt

Hi Matt,
I think the beta >=0 && beta <= 0 line here was to prevent a compiler warning generated from beta == 0.0f (comparing floats is generally bad), I should comment this.
As for viewing the generated kernels, take a look at examples/print.cpp
In GEMM, beta == 0 must be handled separately, as matrix C is allowed to contain nans and infs. For example if the user wants to set C = A*B, it should not be necessary for the user to first write valid floats to C. By having this branch in the code, the user does not need to worry about there being nans/infs in C before calling GEMM.
James

Got it, thanks.
Matt


Wallets/unhashed_description

Adds new field unhashed_description to make CLN and everyone else happy at the same time.

Codecov Report

Merging #870 (cb859e4) into main (3457ff1) will decrease coverage by 0.04%.
The diff coverage is 16.12%.

@@            Coverage Diff             @@
##             main     #870      +/-   ##
==========================================
- Coverage   39.67%   39.63%   -0.05%
==========================================
  Files         218      218
  Lines       11981    12004      +23
==========================================
+ Hits         4754     4758       +4
- Misses       7227     7246      +19

Impacted Files
Coverage Δ

lnbits/core/services.py
53.21% <ø> (ø)

lnbits/wallets/cliche.py
18.27% <0.00%> (ø)

lnbits/wallets/cln.py
21.69% <0.00%> (-0.42%)
:arrow_down:

lnbits/wallets/eclair.py
21.62% <0.00%> (-0.40%)
:arrow_down:

lnbits/wallets/fake.py
76.08% <ø> (ø)

lnbits/wallets/lnbits.py
19.27% <0.00%> (-0.48%)
:arrow_down:

lnbits/wallets/lndgrpc.py
19.11% <0.00%> (-0.29%)
:arrow_down:

lnbits/wallets/lndrest.py
17.39% <0.00%> (-0.31%)
:arrow_down:

lnbits/wallets/lnpay.py
22.35% <0.00%> (-0.54%)
:arrow_down:

lnbits/wallets/lntxbot.py
17.77% <0.00%> (-0.41%)
:arrow_down:

... and 5 more

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


Updated: alldup <IP_ADDRESS>

$notices:
- Updated `installers.host`

dfca28cb-220f-4b43-b924-96ffc2d9e5d6

Newer version of alldup (<IP_ADDRESS>) is now available in master. 5cf7aa65bf99cb856b1ac4380e9ee032d7fde74b


Gemspec files ignored if installing from path or git

When a gem is specified via path or git in the Gemfile, bundler will ignore the files field of the gemspec.
This results in certain files being loadable when they shouldn't be.

Please share the information requested in https://github.com/bundler/bundler/blob/master/ISSUES.md#reporting-unresolved-problems so we can investigate. Thanks!

I'm using the latest bundler release (1.13.6), RVM and ruby 2.3.1.
I hesitate to provide a script because there is a non-trivial chance of this being closed as Works-as-intended. Perhaps if you confirm that this is a bug?
Here is an explanation with examples:
If you have a Gemfile that depends on another gem by path, e.g.:
# Gemfile
gem 'x', path: '/tmp/x'

And the x.gemspec specifies some files:
# x.gemspec
Gem::Specification.new do |s|
  s.gemspec = ['y.rb']
end

And /tmp/x contains a file not specified by x.gemspec, e.g. x.rb, that x.rb will be available to the script using Gemfile.
This means depending on the gem via path or git is not the same as depending on the gem built from that path or git repo, which is IMO unexpected.

Thinking it over, there's no way around this -- bundler just sets the load path, for it to truly follow the files attribute, it would have to delete all other files in your gems directory

@segiddins Perhaps it can build the gem and use that, instead of using the files directly? That's kind of what I was expecting.

An RFC for that feature would be required, but I don't think this will be high-priority for anyone on the core team, sorry.

Are there any plans to have bundler "package" and install git/path gems differently? I feel like the expected behavior is for a git/path gem to be installed into the GEM_HOME just as a normal gem. And if you're bundle-packaging, a my-gem-SHA.gem to be created in (e.g.) vendor/cache

@agibralter no, there are no plans to do that. gems in git repos are frequently not able to successfully build into .gem files, and so packaging requires checking out and packaging the entire state of the repo at the sha that the bundle is locked to.


JsonWriter writes out control characters in strings

JSON spec forbids including any control characters in a string.  The JsonWriter currently doesn't do anything with them when writing, but it should replace them with unicode escape sequences.
Given the string containing the null terminator character \0, writing this out with JsonMapper should produce "\u0000"

Fixed in 079e052


Turning off report of Runnable's run() method

Hi,
May I ask how to turn off the log writing of Runnable's run() method?
I've tried some steps but couldn't turn that off:

Finding documentation at https://tinylog.org/configuration: no answer.
Calling Configurator.shutdownWritingThread(): does not work.
Searching source code: can't find relevant source.

It would be great if you could help (and update documentation).
Thank you very much,

I'm very sorry. I think I misunderstood something. So this is an invalid issue.


[Class modifiers] Implement the new class modifiers feature

New class modifiers
This issue tracks the implementation of the new class modifiers feature.
Specs

Sealed types
Base-interface-final

Subissues:

https://github.com/dart-lang/sdk/issues/50726
https://github.com/dart-lang/sdk/issues/50727
https://github.com/dart-lang/sdk/issues/50728
https://github.com/dart-lang/sdk/issues/50729
https://github.com/dart-lang/sdk/issues/50730
https://github.com/dart-lang/sdk/issues/50731
https://github.com/dart-lang/sdk/issues/50732
https://github.com/dart-lang/sdk/issues/50733
https://github.com/dart-lang/sdk/issues/50734
https://github.com/dart-lang/sdk/issues/50735
https://github.com/dart-lang/sdk/issues/50736
https://github.com/dart-lang/sdk/issues/50737
https://github.com/dart-lang/sdk/issues/50738
https://github.com/dart-lang/sdk/issues/50739
https://github.com/dart-lang/sdk/issues/50740
https://github.com/dart-lang/sdk/issues/50741
https://github.com/dart-lang/sdk/issues/50742
https://github.com/dart-lang/sdk/issues/50743
https://github.com/dart-lang/sdk/issues/50744
https://github.com/dart-lang/linter/issues/3917
https://github.com/dart-lang/sdk/issues/50746

I'm in the process of moving the base-interface-final proposal over to "accepted" (https://github.com/dart-lang/language/pull/2722). Part of that is also giving it a better name since it now includes "mixin" and "base-interface-final-mixin" is a truly horrible name. We also need a name for the corresponding experiment flag. After some discussion on the PR, we settled on "class modifiers". I went ahead and updated all of the issues I could find from "NCM" to "class modifiers". Does that sound OK?

Sounds good!


ICMP requests are allowed for IPv4 but denied for IPv6

test failed for IPv6 but it was allowed for IPv4
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/1587/testReport/junit/(root)/runtime/RuntimeValidatedPolicies_L4Policy_Checks/
time="2018-03-23T21:12:10Z" level=debug msg="running command: docker exec -i  app1 ping -W 2 -c 5 <IP_ADDRESS>"
cmd: "docker exec -i  app1 ping -W 2 -c 5 <IP_ADDRESS>" exitCode: 0
 PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=63 time=0.134 ms
64 bytes from <IP_ADDRESS>: seq=1 ttl=63 time=1.170 ms
64 bytes from <IP_ADDRESS>: seq=2 ttl=63 time=1.152 ms
64 bytes from <IP_ADDRESS>: seq=3 ttl=63 time=0.473 ms
64 bytes from <IP_ADDRESS>: seq=4 ttl=63 time=0.493 ms

--- <IP_ADDRESS> ping statistics ---
5 packets transmitted, 5 packets received, 0% packet loss
round-trip min/avg/max = 0.134/0.684/1.170 ms

time="2018-03-23T21:12:14Z" level=debug msg="running command: docker exec -i  app1 ping6 -c 5 f00d::a0f:0:0:1150"
cmd: "docker exec -i  app1 ping6 -c 5 f00d::a0f:0:0:1150" exitCode: 1
 PING f00d::a0f:0:0:1150 (f00d::a0f:0:0:1150): 56 data bytes

--- f00d::a0f:0:0:1150 ping statistics ---
5 packets transmitted, 0 packets received, 100% packet loss

the policy loaded is under test/runtime/manifests/Policies-l4-policy.json and is with the v3 changes.
you can download the PR https://github.com/cilium/cilium/pull/3164, build cilium and import test/runtime/manifests/Policies-l4-policy.json. It will be automatically translated to v3.
the test failing is this one (pingRequests is a slice of tests to do ping and ping6)
	It("L4Policy Checks", func() {
		_, err := vm.PolicyImportAndWait(vm.GetFullPath("Policies-l4-policy.json"), helpers.HelperTimeout)
		Expect(err).Should(BeNil())

		for _, app := range []string{helpers.App1, helpers.App2} {
			connectivityTest(pingRequests, app, helpers.Httpd1, true)
			connectivityTest(httpRequestsPublic, app, helpers.Httpd1, true)
			connectivityTest(pingRequests, app, helpers.Httpd2, true)
			connectivityTest(httpRequestsPublic, app, helpers.Httpd2, true)
		}

FWIW this requires kernel < 4.11 to reproduce, like the default Ginkgo VMs. I didn't manage to get to the bottom of why this is the case, but it seems to suggest some issue with the IPv6 CIDR ingress policy datapath code. I double-checked the actual masking operations, which seem to be fine, but the hashmap lookup in the CIDR6 ingress map is not matching the ::/0 CIDR for some reason.

I suspect that #3366 will fix this issue.


perf: stable row id prefilter

Cache the row id mask prefilter and make it faster to construct.

Codecov Report
Attention: Patch coverage is 83.78378% with 18 lines in your changes missing coverage. Please review.

Project coverage is 79.50%. Comparing base (448ecbd) to head (be8be9b).
Report is 13 commits behind head on main.

Files
Patch %
Lines

rust/lance-core/src/utils/mask.rs
0.00%
7 Missing :warning:

rust/lance-table/src/rowids/segment.rs
91.30%
4 Missing :warning:

rust/lance/src/index/prefilter.rs
90.90%
0 Missing and 3 partials :warning:

rust/lance-core/src/utils/deletion.rs
66.66%
2 Missing :warning:

rust/lance-table/src/rowids.rs
89.47%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2706      +/-   ##
==========================================
- Coverage   79.57%   79.50%   -0.08%
==========================================
  Files         226      226
  Lines       66523    66759     +236
  Branches    66523    66759     +236
==========================================
+ Hits        52938    53078     +140
- Misses      10474    10566      +92
- Partials     3111     3115       +4

Flag
Coverage Δ

unittests
79.50% <83.78%> (-0.08%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


fix: add color onto 'Gotta restart' messages

Owo now it's pretty and yellow!

Why not make it red since it’s a bad thing? We could even decide the color of the server returned based on http statuses

Well red is an error, but that isnt an error

Red just means bad in general, we could make it orange by using the gchalk library

ooooof

it's ok, I'll do the gchalk thing!


OS X El Capitan > Run > Symbol not found: _clock_gettime

stderr: Traceback (most recent call last):
  File "/Users/alexander/Documents/project/get-me-through/server/routes/../py/run.py", line 3, in <module>
    import cv2
  File "/usr/local/lib/python3.6/site-packages/cv2/__init__.py", line 4, in <module>
    from .cv2 import *
ImportError: dlopen(/usr/local/lib/python3.6/site-packages/cv2/cv2.cpython-36m-darwin.so, 2): Symbol not found: _clock_gettime
  Referenced from: /usr/local/lib/python3.6/site-packages/cv2/.dylibs/libavutil.55.78.100.dylib (which was built for Mac OS X 10.12)
  Expected in: /usr/lib/libSystem.B.dylib
 in /usr/local/lib/python3.6/site-packages/cv2/.dylibs/libavutil.55.78.100.dylib

 byyee
run.js -> success code:1 null

Thanks @zolotykh for pointing out the issue.
The problem is that OpenCV3 a.k.a cv2 needs libjpeg 8d version instead of 9b(latest at this time). With homebrew the version got upgraded to 9b and thus this incompatibility error.
You can switch to version 8d using
$ brew switch libjpeg 8d
which I came to know from here.
Please try it out and check if it works, it works for me.


Add member object accessors to With<>

Currently the With<> template constructs an object, but gives no access to it, so it's only applicable to situations where we rely on the side-effects of creating the object.

I'm not sure if the intent of With was to only rely on side-effects, and use the extra syntax to emphasize that.  If so, then please reject this PR.


Readme added

If a PR is about README, then let there only be README related changes in it
Make other changes separately

@championswimmer  , do we need this now #274

@aayusharora You can add the description of the project and what are the purpose of each folder etc in this PR. The current README I added does not have those things.


Chia_plotter causes a strange problem of network packet loss.

https://www.youtube.com/watch?v=hcRX-G7ytWY
I recorded the process of the problem and put it on youtube
Today I added a new machine and encountered a very, very strange problem.
After running chia_plotter on <IP_ADDRESS>, the machine immediately began to lose packets severely. Using <IP_ADDRESS> to ping the gateway <IP_ADDRESS> will also cause serious packet loss.
All network problems occurred at the moment of running chia_plotter.
I don't have any idea to solve this problem, everything is normal without running chia_plotter.
update:

I restored this situation through a cpu stress test. Is this a hardware problem?

The network packet loss is very, very serious, but everything is normal with chia_plotter.

my windows machine had this same problem(cpu 100% and ping drop or up to 4000+ after a stress).
for me after installing the motherboard chipset drivers they worked fine

This might explain why my chia client seems to lag behind or get disconnected while plotting with madmax

This probably belongs in "discussions" not "issues" as I don't think it's a bug with MadMax. This is very likely just resource contention that is not being well managed for your system. Check whether you are running irqbalance. You may also need to tune MadMax threads down.


Electrum wallet not working on Mojave

Hi,
I updated my Mac software to Mojave and now electrum-xzc wallet does not work. When I click open it does not open.

This is a fix that worked for me.
Always make sure you have a back up of your wallet.
1st try the following: go to System Preference- Security&Privacy- Privacy. Add the Electrum-XZC
Run the wallet.
If this does not work try the following
Go to ~/.electrum and delete the files blockchain_headers and config. then run it. it should start up now.
Hope it resolve it for you.


Default implementation for all provider operations

Verified the new changes locally with ibm-csi-common, ibm-volume-vpc and ibm-vpc-block-csi-driver


Submit to TCIA?

Note that TCIA, which holds the PROSTATEx collection, has a process that allows users to submit analysis results, see https://www.cancerimagingarchive.net/analysis-results/. This way it would be easier to discover your dataset, you would get a DOI that can be cited, and masks will be easy to access and visualize along with the original images. Would be great if you could consider submitting this dataset to TCIA.

Thank you for the suggestion. I will try submitting the dataset to TCIA as soon as possible, even though there are some unsolved issues with the ADC masks for the DICOM images. Maybe I could upload the masks for the NiFTI-converted exams as well, as done for this site.

Unfortunately the issue (#9) persists. I would like to submit masks that are compatible with the DICOM files on TCIA rather than requiring conversion to NIFTI. As soon as I am able to solve this issue I will proceed to the upload.
Do you know if TCIA allows updating or editing the files easily or does the product need to be in a finished state?

I proposed the submission today. If it is approved I will upload the same files to the TCIA as presented here.

If #9 persists, and it is not possible to overlay those segmentations with the DICOM images using tools like 3D Slicer, then I don't think it will be accepted by TCIA.

I imagine the issue is solvable, but would require too much time for me to tackle it at the time. Providing NIFTI images and masks effectively avoids the problem altogether and allows for the correct use of the masks (checked both on 3D Slicer and ITKSnap). Transposing masks for use on the original DICOM files is not part of my usual workflow and I underestimated this aspect at the time of first processing.
In my submission to TCIA I specified all files will be in NIFTI format, let's see what they answer. Anyhow, the most important thing for me is having made a usable dataset available. I am only sorry for early cloners/users who might have to reimplement the dataset in its current form. Hopefully they have been made aware of the issues by the subsequent release version updates.


bump k8s deps and mco

Bump k8s deps - just bumps the dependencies in a single patch as opposed to renovate patches that didn't pass tests individually
Bump MCO to a version that no longer requires github.com/Masterminds/goutils - this MCO version gets rid of github.com/Masterminds/goutils
which recently had an important CVE. Because MCO is a CO dependency, the CVE was tracked against CO as well.

no PX, docs or QE checks needed.

To verify that the goutils dependency is gone, call:
go mod graph | grep goutils

need to bump builder to 1.18..

         --- FAIL: TestE2E/Serial_tests/TestKubeletConfigRemediation (100.12s)

this test has been very flaky recently.
/retest

another test failed, we really need to look at the flaky tests one of these days
/retest

last try before I debug the CI just to get the patch through...
/retest

rebased on recent master


Bookstore: add reducers and actions

Hi there 👋
In this banch, I implemente Redux Toolkit inside of bookstore application.  Following the requirements:
Project Requirements

Add Redux Toolkit (npm install react-redux @reduxjs/toolkit).
Create a directory that will contain all your Redux logic (/src/redux).
Configure a Redux store (/src/redux/store.js)
Define a slice of state for books that:

Includes an array of books (initial state: empty array).
Includes a reducer that adds a book.
Includes a reducer that removes a book.

Define a slice of state for categories that:

Includes an array of categories (initial state: empty array)
Includes a reducer that checks the status and always returns "Under construction" (the initial state should check to that string).

NOTE: No changes in React components are required in this project.
Thanks for your kind feedback!

Thanks @thecodechaser !!


Conditional field not removed from model

Hi, I'm having a problem with the "condition" parameter. It's not removing the value from the model. This is the case for both 'select' and 'date' types . Below example is user selects something from select dropdown, then the date field is shown/hidden - but the date value in the model doesn't disappear. (And same for another select field if this condition is applied)
Schema:
type: "object"
properties: Object
    settlementdate: Object
        condition: "model.settlementtype==='0'"
        fixtag: 64
        format: "date"
        name: "settlementdate"
        required: false
        title: "Settlement Date"
        type: "string"
        uiType: "date"
settlementtype: Object
    default: "0"
    fixtag: 63
    name: "settlementtype"
    onChange: "dropdownChanged()"
    required: true
    title: "Settlement Type*"
    type: "string"
    uiType: "select"

Part of form:
condition: "model.settlementtype==='0'"
format: "yyyymmdd"
key: "settlementdate"
maxDate: Fri Oct 02 2015 13:29:03 GMT+0100 (GMT Daylight Time)
onChange: undefined

Any advice please?

@PhilBroadbent did you figure out how to deal with it? If not, can you make a gist (from the demo app)?

I think this is the same as #769 which I believe is fixed in the latest alpha so given the age of this I will close it for now.


No module named moe_kernel in Flash Attention Installation while compiling TGI2.3.1

Task - Flash Attention Installation from Source. [Completed]
Run- TGI2.3.1 with models that support for Flash attention enabled models.  [Issue does not occur in TGI2.2.0]
Error -
2024-10-08T09:26:27.562016Z  INFO text_generation_launcher: Using prefix caching = True
2024-10-08T09:26:27.562042Z  INFO text_generation_launcher: Using Attention = flashinfer
2024-10-08T09:26:28.345909Z  WARN text_generation_launcher: Could not import Flash Attention enabled models: No module named 'moe_kernels'
2024-10-08T09:26:28.555288Z  WARN text_generation_launcher: Could not import Mamba: No module named 'causal_conv1d'
2024-10-08T09:26:29.238808Z ERROR text_generation_launcher: Error when initializing model
Traceback (most recent call last):
File "tgi_new_env/bin/text-generation-server", line 8, in
sys.exit(app())
File "tgi_new_env/lib/python3.9/site-packages/typer/main.py", line 311, in call
return get_command(self)(*args, **kwargs)
File "tgi_new_env/lib/python3.9/site-packages/click/core.py", line 1157, in call
return self.main(*args, **kwargs)
File "tgi_new_env/lib/python3.9/site-packages/typer/core.py", line 778, in main
return _main(
File "tgi_new_env/lib/python3.9/site-packages/typer/core.py", line 216, in _main
rv = self.invoke(ctx)
File "tgi_new_env/lib/python3.9/site-packages/click/core.py", line 1688, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/remote/vg_llm/anmolb/tgi/tgi_new_env/lib/python3.9/site-packages/click/core.py", line 1434, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "tgi_new_env/lib/python3.9/site-packages/click/core.py", line 783, in invoke
return __callback(*args, **kwargs)
File "tgi_new_env/lib/python3.9/site-packages/typer/main.py", line 683, in wrapper
return callback(**use_params)  # type: ignore
File "text-generation-inference-2.3.1/server/text_generation_server/cli.py", line 109, in serve
server.serve(
File "/text-generation-inference-2.3.1/server/text_generation_server/server.py", line 280, in serve
asyncio.run(
File "/usr/lib64/python3.9/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "/usr/lib64/python3.9/asyncio/base_events.py", line 634, in run_until_complete
self.run_forever()
File "/usr/lib64/python3.9/asyncio/base_events.py", line 601, in run_forever
self._run_once()
File "/usr/lib64/python3.9/asyncio/base_events.py", line 1905, in _run_once
handle._run()
File "/usr/lib64/python3.9/asyncio/events.py", line 80, in _run
self._context.run(self._callback, *self._args)

File "text-generation-inference-2.3.1/server/text_generation_server/server.py", line 235, in serve_inner
model = get_model_with_lora_adapters(
File "text-generation-inference-2.3.1/server/text_generation_server/models/init.py", line 1277, in get_model_with_lora_adapters
model = get_model(
File "text-generation-inference-2.3.1/server/text_generation_server/models/init.py", line 843, in get_model
raise NotImplementedError(FLASH_ATT_ERROR_MESSAGE.format("Sharded Llama"))
NotImplementedError: Sharded Llama requires Flash Attention enabled models.

The result of pip show flash-attn to show Flash Attention:
Name: flash_attn
Version: 2.6.3
Summary: Flash Attention: Fast and Memory-Efficient Exact Attention
Home-page: https://github.com/Dao-AILab/flash-attention
Author: Tri Dao
Author-email<EMAIL_ADDRESS>License:
Location: tgi_new_env/lib/python3.9/site-packages/flash_attn-2.6.3-py3.9-linux-x86_64.egg
Requires: einops, torch
Required-by:

moe-kernels is an optional install, so we should indeed import the module conditionally. Will make a PR to fix this. Thanks for reporting!

Made mandatory and installed through a make install in #2632, so should fixed in the next release. Feel free to reopen if the issue occurs after the next release.


Added a shortcut for faster _iter* calls supplied by modules.

Referenced in Issue #7218
This commit adds the possibility to implement faster iterators over Objects in C++.
This will be handy for custom data structures (hint* linked list #7194) and modules providing anything iterable (AI, Simulation related stuff, ...) .

Little Benchmark (release mode):
C++ impl iterator: Needed 67 ms for 1000000. 0.000067 ms per item.
C++ impl bound functions: Needed 855 ms for 1000000. 0.000855 ms per item.
GDscript iterator: Needed 1100 ms for 1000000. 0.001100 ms per item.
Array: Needed 61 ms for 1000000. 0.000061 ms per item.
Test module and gdscript

Looks good to me, but assigning reduz to review it (as it touches some really core stuff :smile: )

Sorry, it's way too hackish, the decision for GDScript is that simple code is prefered over performance, you can most likely find another way around the problem you are trying to solve


Add docs on indexing dataobjects

Copied from CWP docs at https://www.cwp.govt.nz/guides/core-technical-documentation/common-web-platform-core/en/cwp-features/solr_search.
It's not specific to this platform, so should be part of the general docs.

@tractorcow I've removed the CWP specific docs, thanks for catching that!

Sorry to always add problems. :D I hope you don't mind me spotting a few other things.

Ping @chillu :D

@chillu - do you have any desire to revisit this and push it over the line?

Meh, merged by hand and made the ammends


Azure: remove use of affinity groups.

Affinity groups are deprecated, replaced by regional virtual networks.
Also switches Endpoints to be PKB resources, which makes creating them a bit more resilient.

Could you make sure that this doesn't break remote ssd on Azure? I think we rely on affinity groups to make it work, but I could be wrong.

@ehankland - will do, thanks.

@cmccoy thanks. Specifically, I don't think we specify which storage account to use when creating disks/vms - in the affinity groups we set up there's only one to choose from, but without it I'm not sure what will happen (if there are multiple storage accounts in the same location). I suspect everything will seem to work even if it breaks, but you'll get the wrong disk type.

Has the status of this changed?

No, I haven't looked more.

@ehankland - Can you work with Connor to complete this.  Thanks!

#833 has a superset of these features, so I'll close this in favor of that PR.


Feature request: Make privacy policy less vague

Could you list out exactly what data is collected? Thanks!

Hey, I will add a Privacy Policy page.
On Page Assist:

We do not collect any data. Everything happens locally and stays in your browser.
Permissions we use in the manifest are:

Notifications.
Website content access (for the chat-with-webpage feature).
Unlimited storage access (to save chats locally).

If you use the Page Share feature for sharing chats:

Data is shared only when connecting with an external source.
Page Share also supports self-hosting.
Shared chats can be deleted, and once deleted, they are permanently removed from the server.

Thanks!

Could you update this page? https://n4ze3m.com/privacy

I will also update my site. Thanks for the suggestion! I have added the privacy.md file to the repository: https://github.com/n4ze3m/page-assist/blob/main/PRIVACY.md.


Not going to next map

Challenge Waypoint: Learn how Script Tags and Document Ready Work has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; rv:42.0) Gecko/20100101 Firefox/42.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

jQuery Playground

#left-well

        <button class="btn btn-default target" id="target1">#target1</button>
        <button class="btn btn-default target" id="target2">#target2</button>
        <button class="btn btn-default target" id="target3">#target3</button>

#right-well

        <button class="btn btn-default target" id="target4">#target4</button>
        <button class="btn btn-default target" id="target5">#target5</button>
        <button class="btn btn-default target" id="target6">#target6</button>

Is your issue that it won't take you to the next challenge after you complete this? I am not able to repro this if that's the case.

I'm having the same issue. Using Chrome, it won't advance to the next challenge. However, it works fine in Firefox.

Move your jQuery code below the last , I don't know why but that's what fixed it.  Tested in chrome and firefox.


Aggregate stratifier renders some queries unpleasant

When authoring a unit testing framework example I ran into an obnoxious query. Filtering by the result of an aggregate by any value naturally joined with one of it's inputs is impossible to do in the same block. The stratifier deems this to be a cycle since the aggregate's output may filter its inputs. The obvious intended result in this example is for the aggregate to run on its inputs and post-filter.
This isn't a show-stopping issue because it can be rewritten into two blocks, but it is likely to surprise any users who run across this.
Commit a test case to run.
commit @test
  [#test input:
    [#add a: 7, b: 3]
    [#add a: 6, b: 4]
    [#add a: 1, b: 1]
   expected:
     [#result value: 10]
     [#result value: 2]]

Kick off the test case.
search @test
  [#test input]
  lookup[record: input, attribute, value]

bind
  lookup[record: input, attribute, value]

Derive some metadata about the test cases.
search @test
  test = [#test expected]
  lookup[record: expected, attribute, value]
  av-count = count[given: (attribute, value), per: expected]
  test-count = count[given: expected, per: test]

bind @test
  expected.av-count := av-count
  test.count := test-count

Confirm the test passes.
search @test
  test = [#test expected]
  av-count = expected.av-count
  test-count = test.count
search
  not(actual = [tag: expected.tag]
               lookup[record: expected, attribute, value]
               lookup[record: actual, attribute, value]
               av-count != count[given: (attribute, value), per: actual])
  test-count = count[given: expected, per: test] // The problem is here

bind
  test.passed := true

The block to test.
search
  [#add a b]

bind
  [#result value: a + b]

@ibdknox


Fix refactoring error which changed html, update gradle

Accidentally modified the start page while refactoring a java class

@jtmcn Can you merge this? Somehow I managed to refactor the html file


ARM64 brew install ${formulae}, always show Error: SHA256 mismatch

brew config output
HOMEBREW_VERSION: 3.0.7-11-g53f34af-dirty
ORIGIN: https://github.com/Homebrew/brew
HEAD: 53f34af1c53c5716a77255d3227a5ba2cdc58ea9
Last commit: 20 hours ago
Core tap ORIGIN: https://mirrors.ustc.edu.cn/homebrew-core.git
Core tap HEAD: 3ad9891001294e7583e61fa6cfb7db68435abdca
Core tap last commit: 14 hours ago
Core tap branch: master
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_BOTTLE_DOMAIN: https://mirrors.aliyun.com/homebrew/homebrew-bottles
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 8
HOMEBREW_NO_AUTO_UPDATE: set
Homebrew Ruby: 2.6.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: octa-core 64-bit arm_firestorm_icestorm
Clang: 12.0 build 1200
Git: 2.24.3 => /Library/Developer/CommandLineTools/usr/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 11.1-arm64
CLT: <IP_ADDRESS>.1.1610135815
Xcode: N/A
Rosetta 2: false
brew doctor output
Much info but no waning.

[X] The brew doctor above contains no "Warning" lines.

What were you trying to do (and why)?
ARM64 brew....
Install some Formulae such as<EMAIL_ADDRESS>kafka.
What happened (include all command output)?
Always shows:
brewa install<EMAIL_ADDRESS>==> Downloading https://mirrors.aliyun.com/homebrew/homebrew-bottles/bottles/apr-1.7.0_2.arm64_big_sur.bottle.tar.gz
Already downloaded: /Users/linyida/Library/Caches/Homebrew/downloads/88d0983312cd82ff8b457d6646677fd9d447b15e235c7941e5d6e321647a000d--apr-1.7.0_2.arm64_big_sur.bottle.tar.gz
==> Downloading https://mirrors.aliyun.com/homebrew/homebrew-bottles/bottles/apr-util-1.6.1_3.arm64_big_sur.bottle.tar.gz
Already downloaded: /Users/linyida/Library/Caches/Homebrew/downloads/1e5f752c21a2e00680ae2875987d66f4b5ebf20921bc08fb0af0d3aabf83f981--apr-util-1.6.1_3.arm64_big_sur.bottle.tar.gz
Error: SHA256 mismatch
Expected: 29ffec15c99813b879d9410e2040044b02ee8e219492d065043b4bb65cf30741
Actual: 79b9dc1a57ac27e3ba8677023ab5eed214db2c9ae4af72dea2aa4ac91fafabc3
File: /Users/linyida/Library/Caches/Homebrew/downloads/1e5f752c21a2e00680ae2875987d66f4b5ebf20921bc08fb0af0d3aabf83f981--apr-util-1.6.1_3.arm64_big_sur.bottle.tar.gz
To retry an incomplete download, remove the file above.
What did you expect to happen?
install succeed
Step-by-step reproduction instructions (by running brew commands)
brew cleanup && brew update
brew install<EMAIL_ADDRESS>

One of these two might be causing the problem:
Core tap ORIGIN: https://mirrors.ustc.edu.cn/homebrew-core.git

or
HOMEBREW_BOTTLE_DOMAIN: https://mirrors.aliyun.com/homebrew/homebrew-bottles

Try setting these to the defaults (i.e. unset HOMEBREW_BOTTLE_DOMAIN and/or set your core tap origin to https://github.com/Homebrew/homebrew-core.git) and see if that helps.

I think the issue is with your bottle mirror:
❯ curl -sSL https://homebrew.bintray.com/bottles/apr-util-1.6.1_3.arm64_big_sur.bottle.tar.gz | sha256sum
29ffec15c99813b879d9410e2040044b02ee8e219492d065043b4bb65cf30741  -
❯ curl -sSL https://mirrors.aliyun.com/homebrew/homebrew-bottles/bottles/apr-util-1.6.1_3.arm64_big_sur.bottle.tar.gz | sha256sum
79b9dc1a57ac27e3ba8677023ab5eed214db2c9ae4af72dea2aa4ac91fafabc3  -

The bottles in your mirror appear to be compromised.

If Bintray is too slow, you might want to try https://mirrors.sustech.edu.cn/homebrew-bottles. This seems to work:
❯ curl -sSL https://mirrors.sustech.edu.cn/homebrew-bottles/bottles/apr-util-1.6.1_3.arm64_big_sur.bottle.tar.gz | sha256sum
29ffec15c99813b879d9410e2040044b02ee8e219492d065043b4bb65cf30741  -


Dropping a file on the Dropzone does not work

Describe the bug
Dropping a file to the Dropzone does not work. The file is opened in the Browser instead.

I am experiencing the same issue

The component has the following event forwarding, so it is up to you how you want to handle it.
  on:focus
  on:blur
  on:mouseenter
  on:mouseleave
  on:mouseover
  on:dragenter
  on:dragleave
  on:dragover
  on:drop

Please note that Flowbite-Svelte it a UI library.

Thank you for your contribution! We are closing it for now. If you encounter any other problems in the future, please feel free to open a new issue. We appreciate your help in making our project better!


RFC: Named arguments should not make explicit arguments implicit except for structure parameters

Summary
Remove the anyNamedArgDependsOnCurrent feature from the app elaborator that was added in https://github.com/leanprover/lean4/commit/a5ab59a413df6b2fca2c6909aea878d19c3ad8af and instead address #1851 by making structure projection notation elaborate as if all the structure parameters are implicit.
Supersedes RFC #5386.
Motivation
The app elaborator has a feature (added in https://github.com/leanprover/lean4/commit/a5ab59a413df6b2fca2c6909aea878d19c3ad8af) where every explicit parameter that is depended upon by a named argument becomes an implicit parameter. We will call this feature argument suppression via named arguments for the purpose of this RFC.
The motivation for it is to make projection notation for structures more uniform. For background, recall that for a structure such as
structure S (α β : Type) where
  f : α → β

the field projection S.f has a type of the form {α β : Type} → (self : S α β) → α → β. The parameters before self are the structure parameters, which are always fully determined by the type of self. Similarly, for a class such as
class C (α β : Type) where
  f : β

the field projection C.f has a type of the form (α : Type) → {β : Type} → [self : C α β] → β. Notice that in this case the first structure parameter is explicit since it cannot be inferred from any other explicit arguments or from the return type, and it is needed for typeclass inference of the self parameter.
For x : S, the projection notation x.f is elaborated as if it were S.f (self := x), using named arguments. Similarly, when inst : C, then inst.f is elaborated as if it were C.f (self := inst). Named arguments are the mechanism by which the elaborator provides a value for self even when it is an implicit argument.
In issue #1851 it was noticed that elaborating inst.f as C.f (self := inst) alone led to confusing behavior, since the structure parameters needed to still be supplied. This was fixed by implementing the argument suppression via named arguments feature, which addresses the issue because the structure parameters are always depended upon by the self parameter, and so they become implicit. The result is that for inst : C α β the projection inst.f elaborates as @C.f α β inst.
However, this leads to other confusions. Named arguments in general suppress all explicit parameters depending on them, not just in the structure projection case. For example, in issue #1867 there is unintuitive behavior with using a named argument for an instance parameter. The general issue is that it is unclear to the user which parameters depend on implicit parameters — after all, they do not appear in the source text! — so it can be hard to predict what will happen. Furthermore when it does occur, the only mitigation when it is unwanted is to use named arguments for the suppressed explicit parameters.
The argument suppression via named arguments feature also has a counterintuitive interaction with explicit mode ("@ mode"), pointed out on Zulip. The assumption of explicit mode is that each argument should be provided explicitly. Argument suppression via named arguments is however still enabled in explicit mode since the implementation of explicit mode is, essentially, that each parameter becomes explicit (with a special case for _ arguments for instance implicit parameters). Since they are explicit, argument suppression via named arguments applies. For an example counterintuitive behavior, in @C.f (self := inst) the α parameter becomes implicit since the self parameter depends on it. We believe users would expect to need to write @C.f (self := inst) α in explicit mode.
Guide-level explanation
(Rough draft with key information) Projection notation (also known as field notation or dot notation) is a convenient object-oriented-like way to apply projection functions to a term. (... examples ...)
In the simplest case, x.f resolves to S.f x, where S is a structure and f is a field of S. If f is located in a parent structure, then Lean will automatically insert parent projections. (... examples ...)
In particular, the notation x.f resolves as S.f (self := p x) where p (optional) is the projection to the parent structure S.
Classes are a kind of structure as well, and projection notation can be used on class instances. However, classes are different from normal structures in that the self parameter of field projections is instance implicit rather than explicit. In this case, inst.f resolves to C.f _ ... _ (self := inst), where there is one placeholder _ per explicit class parameter.
In a section on named arguments:
Migration: In Lean pre-v4.XX.0, using named arguments would make every explicit argument it depends on become implicit, even in explicit mode. This feature was removed, and you may need to insert some number of positional _ or (_) arguments to adapt to the change. Using _ should usually work, but (_) may be necessary in explicit mode for instance implicit arguments that are non-canonical.
Reference-level explanation
(... explanation of elaboration of normal applications ...)
(... in section on projections ...) Projection notation x.f for structure projections resolves as S.f _ ... _ (self := x) with one placeholder _ per structure parameter that appears as an explicit parameter in the type of S.f.
Drawbacks
Users may be making use of this feature for named arguments. Kevin Buzzard for example was using it in explicit mode to avoid needing to supply "obvious" _s. It should be said that Kevin said he was making progress with which arguments to fill in next with guess-and-check, not intentionally.
Floris van Doorn thinks that making class structure parameters implicit in projections can still lead to confusion.
Rationale

As issue #1867 shows, users have been surprised by the argument suppression via named arguments feature, and often users feel like it is an elaboration bug.
The argument suppression via named arguments feature is motivated by the specific use case of not supplying structure parameters in instance projections. We can easily implement this precise feature.
Since there is no such thing as explicit mode projection notation, implementing it only for projection notation would immediately improve named argument handling for explicit mode, making explicit mode more predictable. It keeps the invariant that each argument (be it positional or named) corresponds to one parameter. It is hard to predict which arguments are suppressed, especially ones that are originally implicit arguments as they do not generally appear in the syntax.
Rather than using the implementation where it auto-inserts the correct number of _s, we may consider making argument suppression via named arguments apply to any named argument for a self parameter. However, users can write C.f (self := inst) (using the above example C), which would surprisingly omit a parameter.
Users can specify the exact expression they want for every argument. Argument suppression via named arguments leads to terms that might not definitionally be in the correct form — there are some mathlib examples where this causes typeclass inference failure.
Generalized field notation uses a separate "lval resolution" procedure, and it would either use positional or named arguments, preferring positional when possible. This meant that argument suppression via named arguments only applied for underapplied generalized field notation. Removing argument suppression via named arguments makes generalized field notation elaboration less confusing.
The pretty printer is not aware of argument suppression via named arguments, and the pp.analyze option can cause round-tripping issues because of this. Removing the feature fixes this. Note that projection notation notation is disabled for instances, so this RFC has no effect on pretty printing instance projections.
Lean 3 resolved argument-free structure projections in the way described by this RFC.

In the issue #1867, Mario Carniero suggests another implementation that tries to preserve argument suppression via named arguments. That would entail a relatively large refactor since (1) the elaborator does not keep track of the actual order of named vs positional arguments, just an array for each and (2) the elaborator processes in parameter order, not argument order.
Unresolved questions and future possibilities

There is some disagreement about whether #1851 is an issue. Floris van Doorn suggests the fix is for the user to use _s for the explicit structure parameters.
Generalized field notation will not omit any arguments, meaning user-defined "fields" will elaborate with different behavior. Should users be expected to use only implicit arguments for structure parameters? Should they be willing to accept that _s can be necessary for generalized projections? Or should generalized field notation compute what the "structure parameters" are and omit them for consistency?

Community Feedback
This RFC comes from discussion on the Lean Zulip pointing out a bug in the interaction between how _ is handled in explicit mode for instance implicit arguments and argument suppression via named arguments. The PR #5283 fixes it with the assumption that we want the feature to be enabled in explicit mode, but there was feedback to disable argument suppression via named arguments in explicit mode. This led to RFC #5386, and there was more feedback supporting disabling the feature more widely. There is some support to disable the feature entirely.
Impact
Add :+1: to issues you consider important. If others benefit from the changes in this proposal being added, please ask them to add :+1: to it.

It turns out the implementation I suggested here has a big drawback, which is that inserting placeholders for explicit structure parameters means that those structure parameters can't be provided using named arguments. Also, using named arguments for them would lead to those placeholders being shifted to the next explicit arguments!
I see three options:

Add a flag to NamedArg that turns on argument suppression for just that named argument.
Change the interface to elabAppArgs to accept an (ftype? : Option Expr) argument, so that way the structure parameters can be made implicit. (Users of this interface may also want to use this to change parameter names.)
Revert the feature entirely.

Completed by #5283.


Spelling

Just making sure we're paying attention.
Spelling on "our documentation" needs to be updated.
"Help us improve out docuementaiton by sending us a pull-request or opening a GitHub Issue"
Located https://github.com/Microsoft/cordova-docs

:) good catch, thanks for this!


Chrome DevTools for full-day workshop

On our full-day skulk in Zurich we got the feedback that it can be very confusing for people who just start to develop when they are asked to install Chrome Devtools for the full-day workshop, when we use Codesandbox to do our workshop and on Codesandbox the Devtools won't work.
Wouldn't it be better to remove that suggestion from the workshop content itself and maybe add a new section after the full-day workshop where we propose Devtools to be installed if they develop locally?
I could do a PR with those changes for the english & german version of the full-day workshop.

Funny, I was just thinking this. Yes, please PR to remove that recommendation; CodeSandbox works fine in terms of having a console available.


Add consumption production limitation to site area chart

Adding changes in mongodb pipeline to fetch cumulated data in a single object instead of as multiple arrays.

Opened a second PR with proper and merged changes


discovery: Make kubectl.py able to find kubectl

Description
I was trying to get kube-hunter to display something about the remote k3s cluster with only port 6443 accessible.
While doing so I saw in the debug log output that it couldn't find kubectl, even though it's in $PATH
Doing so I eventually made conclusion that the arguments in subprocess check_output function in python 3.8.10 needs to be a
list. I do not know if this has changed in python. There might be better ways to change this. At least this change makes the code find the kubectl version.
Contribution Guidelines
Please Read through the Contribution Guidelines.
Fixed Issues
Fixes #(issue)
"BEFORE" and "AFTER" output
BEFORE
Before the change the error in "--log debug" would look like:
2022-06-03 13:54:56,497 DEBUG kube_hunter.modules.discovery.kubectl Could not find kubectl client

AFTER
And if I add small code change to also print the exception error it
prints:
2022-06-03 13:54:56,497 DEBUG kube_hunter.modules.discovery.kubectl [Errno 2] No such file or directory: 'kubectl version --client'
2022-06-03 13:54:56,497 DEBUG kube_hunter.modules.discovery.kubectl Could not find kubectl client

Contribution checklist

[x] I have read the Contributing Guidelines.
[ ] The commits refer to an active issue in the repository.
[ ] I have added automated testing to cover this case.

Notes
Please mention if you have not checked any of the above boxes.

 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.

And shlex seems to have been in python3 for a long time. Looks good to me :)


Update detector model used for CI and add tests for LCFIJet and --help

BEGINRELEASENOTES

Updated detector model used for CI to CLD_o2_v06 and added tests for LCFIJet and k4run --help

ENDRELEASENOTES

We should wait for https://github.com/key4hep/key4hep-spack/issues/605


Terms to add

[x] data contract
[ ] semantic layer <- simon writing an article about this

[x] maybe also semantic warehouse

[ ] Software-Defined Assets
[ ] Protobuf
[ ] Apache Kafka
[ ] GraphQL

re: protobuf - this is new https://news.ycombinator.com/item?id=32812873


feature: enforce across zone hard anti-affinity scheduling

v0.8.0 only supports soft anti-affinity scheduling. #929

Test skeleton for zone anti-affinity:
@pytest.mark.node  # NOQA
def test_replica_zone_anti_affinity(client):  # NOQA
"""
Test replica scheduler with zone anti-affinity
1. Set zone anti-affinity to hard, and replica count to 2.
2. Create 2 nodes and add the same zone label to them
3. Create a volume.
4. Wait for volume condition `scheduled` to be false.
5. Change the zone label to be different on one node.
6. Create a volume.
7. Wait for volume condition `scheduled` to be success. And clear the volume
8. Set zone anti-affinity to soft.
9. Change the zone labels to be the same on both nodes.
10. Create a volume.
11. Wait for volume condition `scheduled` to be success.
12. Clean up the replica count, the zone labels and the volume.
"""

@meldafrawi , @yasker , please take a look.

Pre-merged Checklist

[ ] Does the PR include the explanation for the fix or the feature?
[ ] Is the backend code merged?
[ ] Is the reproduce steps/test steps documented?
[ ] If labeled: area/ui Has the UI issue filed or ready to be merged?
[ ] If labeled: require-automation-e2e Has the end-to-end test plan been merged? Have QAs agreed on the automation test case?
[ ] if labeled: require-automation-engine Has the engine integration test been merged?
[ ] if labeled: require-doc Has the necessary document PR submitted or merged?


100% cpu on Linux

Even starting very simple applications, kivy (from master) makes one cpu core 100% busy on my linux machine (ubuntu 14.04). Application works properly. If I do something, for example, moving slider cpu goes down, but while being idle goes back to 100%. Is this a bug or a feature?

Definitely a feature. 😛

Cannot reproduce this on Ubuntu 14.04 with py2 or py3, testing touchtracer, pictures or showcase on master. @pyfrid can you please verify that you see this behavior with these examples?

I can reproduce this on my PC with touchtracer both with master and with daily build.
The output is the following:
[INFO   ] [Kivy        ] v1.9.1-dev
[INFO   ] [Python      ] v2.7.6 (default, Jun 22 2015, 17:58:13)
[GCC 4.8.2]
[INFO   ] [Factory     ] 177 symbols loaded
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_gif, img_sdl2, img_pil (img_ffpyplayer ignored)
[INFO   ] [Text        ] Provider: sdl2
[INFO   ] [OSC         ] using  for socket
[INFO   ] [Window      ] Provider: sdl2(['window_egl_rpi'] ignored)
[INFO   ] [GL          ] OpenGL version <3.0 Mesa 10.1.3>
[INFO   ] [GL          ] OpenGL vendor
[INFO   ] [GL          ] OpenGL renderer <Mesa DRI Intel(R) Sandybridge Desktop >
[INFO   ] [GL          ] OpenGL parsed version: 3, 0
[INFO   ] [GL          ] Shading version <1.30>
[INFO   ] [GL          ] Texture max size <8192>
[INFO   ] [GL          ] Texture max units <16>
[INFO   ] [Window      ] auto add sdl2 input provider
[INFO   ] [Window      ] virtual keyboard not allowed, single mode, not docked
[INFO   ] [GL          ] NPOT texture support is available
[INFO   ] [Base        ] Start application main loop
[INFO   ] [Base        ] Leaving application in progress...

@pyfrid could you profile touchtracer? Please run the command below, leave the app open for a bit while the cpu is at 100%, then close it. touchtracer.prof will be saved in the same folder and you will have to attach that file here.
python -m cProfile -o touchtracer.prof main.py

Here it is
https://drive.google.com/open?id=0B47IF5KCd6h0Mjc4V19QSVljbWs

It asks for a login, please make it public.

sorry
https://drive.google.com/file/d/0B47IF5KCd6h0Mjc4V19QSVljbWs/view?usp=sharing

There seems to be a very high number of clock calls, some of the core devs looked into it, but i'm not sure if a possible cause was found.
@kived, @tshirtman, @Kovak: what could cause this?

Can you post the .config for your computer? This could be that fps has been set to a very large number forcing way too many updates. For comparison in a 13 second run on my computer we see ~1000 clock iterations, you are seeing ~80,000. Even if we were executing a clock schedule at 60 fps we would only see 36,000 calls, and touchtracer does not try to do so much work as to even need that rate of update.

My FPS limit was set to 0, i.e. unlimited. When I set it to 60 fps, CPU usage is normal. I think it must be better clarified in settings widget, that unlimited FPS can cause high CPU load. I change it couple of months ago just for trying and forgot immediately because there was no performance change.


added a launchTargetDirectory command

Changes

add a new cmake.launchTargetDirectory command.

This allows to use variable substitution to get the active launch target's directory in launch configurations. It works like cmake.launchTargetPath but instead of the path to the file, it's the path to the directory containing it.
Why
In almost any complex project, lots of additional files are built and usually output next to the main executable (configuration files, data, etc.) When debugging this kind of projects, we have to set the launch working directory to the directory containing all those files. And if we configured CMake build directory using ${builtType} we're screwed : we have no way of automatically knowing what working directory we should use.
Until this patch, I used the ${input:...} feature, but it's error prone, and a bit tedious. With this command, we can easily access the directory and the launch configurations are a lot simpler. Also debugging no longer requires any user interaction (well, except for actually hitting F5 :) )
Notes
I updated the doc and added a unit test. It basically just uses the existing launchTargetPath command and just does a path.dirname on the result, so the changes are pretty trivial.
I hope you'll accept this request, it would really make our life easier :)
Regards,
Damien.

Pull request of hell, just noticed its ID number :p
As a side note: is there any contributor there ??

@dcourtois Hm, seems like something went wrong with travis. Can't merge the PR like this.
Maybe try pushing a new commit :man_shrugging: ?

@vector-of-bool Something seems to be wrong with the travis-ci check.

Hi, thanks for checking this ! It seems to be a problem with the Travic infrastructure right ? Is there something I can do ?

No, I don't have the needed permissions for changing the CI stuff, so I (we) can't really do anything.
You could try opening a new PR though.

Ok, will do that to see if it changes anything.


Alpha Fold crash

After running AF Colab multimer for weeks, it started to get unstable this week. Recently it has crashed multiple times. Each when starting task 5 (Run AlphaFold and download prediction) the system crashes within no time although I am Prolab+ user.
The following or similar error message occurs, which is similar to what other users feedback:
/opt/conda/lib/python3.7/site-packages/haiku/_src/data_structures.py:144: FutureWarning: jax.tree_flatten is deprecated, and will be removed in a future release. Use jax.tree_util.tree_flatten instead.
  leaves, treedef = jax.tree_flatten(tree)
/opt/conda/lib/python3.7/site-packages/haiku/_src/data_structures.py:145: FutureWarning: jax.tree_unflatten is deprecated, and will be removed in a future release. Use jax.tree_util.tree_unflatten instead.
  return jax.tree_unflatten(treedef, leaves)
---------------------------------------------------------------------------
UnfilteredStackTrace                      Traceback (most recent call last)
[<ipython-input-5-ca4ee2dc266d>](https://localhost:8080/) in <module>
     44     processed_feature_dict = model_runner.process_features(np_example, random_seed=0)
---> 45     prediction = model_runner.predict(processed_feature_dict, random_seed=random.randrange(sys.maxsize))
     46

________________________________________
23 frames
________________________________________
UnfilteredStackTrace: AttributeError: 'Config' object has no attribute 'jax_experimental_name_stack'

The stack trace below excludes JAX-internal frames.
The preceding is the original exception that occurred, unmodified.

--------------------

The above exception was the direct cause of the following exception:

AttributeError                            Traceback (most recent call last)
[/opt/conda/lib/python3.7/site-packages/haiku/_src/module.py](https://localhost:8080/) in wrapped(self, *args, **kwargs)
    406       f = functools.partial(unbound_method, self)
    407       f = functools.partial(run_interceptors, f, method_name, self)
--> 408       if jax.config.jax_experimental_name_stack and module_name:
    409         local_module_name = module_name.split("/")[-1]
    410         f = jax.named_call(f, name=local_module_name)

AttributeError: 'Config' object has no attribute 'jax_experimental_name_stack'

Same error here but unsure how to solve it

This is an upgrade from google, the newest version of colab by default takes up jax 0.3.25 (https://colab.research.google.com/notebooks/relnotes.ipynb#scrollTo=iOWrh41ocOca) which has no support for 'jax_experimental_name_stack', I have solved this issue by downgrading colab jax to 0.3.17, you can do it by adding this command: !pip install --upgrade "jax[cuda]==0.3.17" -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html and then restarting the runtime

This is an upgrade from google, the newest version of colab by default takes up jax 0.3.25 (https://colab.research.google.com/notebooks/relnotes.ipynb#scrollTo=iOWrh41ocOca) which has no support for 'jax_experimental_name_stack', I have solved this issue by downgrading colab jax to 0.3.17, you can do it by adding this command: !pip install --upgrade "jax[cuda]==0.3.17" -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html and then restarting the runtime

Hey, I have been struggeling to apply your solution. Could you elaborate further on when and where to insert this command?

I solved this problem, but there was another problem with the procedure

It seems because after reconnecting, all steps need to be restarted

Thanks for raising this. Closing as this is a duplicate of https://github.com/deepmind/alphafold/issues/635.


Fixed #6130 -- Set overwrite_url initial value on advanced settings form

Fixed #6130

Coverage remained the same at 79.182% when pulling 73afc4f5b1d6eb00ff05dd8aa359ef52b8357344 on czpython:fixes/3.5.x/6130 into 70b88a24b24c67071e4940f0c18b4246e3d08e40 on divio:develop.


Removing executable file attribute from LICENSE file. chmod -x LICENSE.

Seems like LICENSE file is marked as an executable.

@creativepsyco thanks for the heads up. Looks like there were multiple +x files so I had to make additional changes beyond your PR. I've added your contribution to the CHANGELOG. Thanks!

Awesome cool !


bug: How to shutdown a openllm service?

Describe the bug
I didn't find a way to stop the LLM service, and I used the KILL command to forcefully kill the process, resulting in the process being restarted and causing the server process to hang.
To reproduce
No response
Logs
No response
Environment
transformers:3.33.0
System information (Optional)
No response

You can kill the circusd process underlying


Sample notebook incorrectly states that you can edit a cell by clicking on it (editing requires double click)

Environment data

VS Code version: 1.49.1
Extension version (available under the Extensions sidebar): 2020.9.111407
OS and version: macOS 10.13.6
Python version (& distribution if applicable, e.g. Anaconda): 3.7.7
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: N/A
Relevant/affected Python-related VS Code extensions and their versions: N/A
Value of the python.languageServer setting: Jedi

[NOTE: If you suspect that your issue is related to the Microsoft Python Language Server (python.languageServer: 'Microsoft'), please download our new language server Pylance from the VS Code marketplace to see if that fixes your issue]
Expected behaviour
I should be able to edit the cell by clicking on it, per the sample notebook.
Actual behaviour
I must double click the cell to edit it.
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Open the sample notebook.
Attempt to edit the "Getting Started" cell by single clicking it.
Experience disappointment.

Thanks for calling that out, just to check, you'd like the notebook to say the following?

"To edit the text in this cell, simply double click on the cell to change it into edit mode."


How To Connect Quantum Instance IBM in Qiskit Machine Learning

Environment

Qiskit Machine Learning version:0.6.1
Python version: 3.11.4
Qiskit: 0.44.1

Issue
i have qiskit machine learning model file, i try to load this qiskit machine learning model file with quantum instance
provider = IBMProvider(token="YOUR_API")
backend = provider.get_backend("ibmq_qasm_simulator")
quantum_instance = qiskit.utils.QuantumInstance(backend, shots=1024)

but i didn't found how to load qiskit machine learning model file with quantum instance in qiskit_machine_learning documentation. i try to embed quantum instance in predict method but i found this error(i think it will be work in this version)
a = vqc.predict([[0.21412, 0.5125214124]],quantum_instance=quantum_instance)

TypeError: NeuralNetworkClassifier.predict() got an unexpected keyword argument "quantum_instance"

QuantumInstance is deprecated. Please take a look at either Sampler or Estimator.
A few posts:

https://medium.com/qiskit/what-are-qiskit-primitives-9bf63c1eacc7
https://qiskit.org/documentation/migration_guides/algorithms_migration.html
https://qiskit.org/ecosystem/machine-learning/migration/01_migration_guide_0.5.html

QuantumInstance is deprecated. Please take a look at either Sampler or Estimator.
A few posts:

https://medium.com/qiskit/what-are-qiskit-primitives-9bf63c1eacc7
https://qiskit.org/documentation/migration_guides/algorithms_migration.html
https://qiskit.org/ecosystem/machine-learning/migration/01_migration_guide_0.5.html

I see, i will try it


Update README file to reflect recent changes

Koushik tried building the utilities on a fresh system.
When building grpc, we provide build options as SSL_PROVIDER=package so we should include installing openssl-devel as part of ### Install basic utilities in ipdk-dpdk.md.
For Fedora :openssl-devel
For Ubuntu: libssl-dev

Crikey. There’s more to it than this, isn’t there? Aren’t we now logging to a file instead of stderr by default?
Derek
From: Nupur Uttarwar @.>
Sent: Thursday, December 8, 2022 12:56 PM
To: ipdk-io/networking-recipe @.>
Cc: Foster, Derek Graham @.>; Author @.>
Subject: Re: [ipdk-io/networking-recipe] Update README file to reflect recent changes (PR #83)
@nupuruttarwar commented on this pull request.
I just realized we should specify the new log location - /var/log/stratum. Can you include this as well in the Readme? Rest looks good
—
Reply to this email directly, view it on GitHubhttps://github.com/ipdk-io/networking-recipe/pull/83#pullrequestreview-1210766410, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADB5QKJBPSSON722WNLOTUDWMJDNJANCNFSM6AAAAAASYSRA7I.
You are receiving this because you authored the thread.Message ID: @.***>


Some UI elements do not scale with window.zoomLevel

VSCode Version: 1.11.2
OS Version: Windows 10 (150% system scale) or macOS 10.12

Steps to Reproduce:

Set window.zoomLevel to any value (e.g. 1.4)
See screenshots

The same happen when zooming out (-1)
Markdown and previewHtml overflow is hidden, and not scrollable

Duplicate of #5745. Should be fixed in current insiders


Issue with Directory Exclusion in Scan

Task Version : SynopsysDetectTask@6
DetectVersion : 7.1.0
We are trying to exclude some folders from Detect scanning and reporting in Blackduck. Below is our project structure.

Root
          |-> Folder A
              | -> Solution 1
          |-> Folder B
              | -> Solution 2
          |-> Folder C
          |-> Tools
             | -> ToolProject
          |-> Extensions
             | -> ExtensionProject

We are trying to exclude both the "Tools" and "Extensions" folder.

Option 1
--detect.source.path= "Root"  --detect.excluded.directories=Tools,Extensions
Result : This does not skip any of the folders

Option 2
--detect.source.path= "Root"  --detect.excluded.directories="\Tools,\Extensions"
Result : This does not skip any of the folders

Option 3
--detect.source.path= "Root"  --detect.excluded.directories="Tools"
Result : This does skip only the Tools folder.

Based on the documentation at here and here , the property should be accepting a comma seperated values . However that is not reflecting in the action.
Looking for options to achieve the goal of excluding multiple folders

Hello,
I tried to add quotes around the comma separated list for issue #45 and that apparently fixed our issue.
Something like:
--detect.excluded.directories="Tools,Extensions"
I hope this helps,


Remove dead code

From @benjaminwood on April 10, 2018 17:19

Remove portal based authoring: remove controllers, views & assets. Authoring functionality is now contained in LARA.
Remove portal wide researcher or guest roles (no longer supported).
Not all Embeddables and Saveables are used. We plan to remove the unused ones before upgrading Rails.

Potentially dead:

config/refactoring/em.rb

Copied from original issue: hintmedia/rigse#8

For future reference.
Use coverband to help measure production code usage.


ci: added dependency review action

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


Confusing text at homepage

When you enter in caskroom website, you see:
"Install with ease" with a chrome installation example. But this is confusing because, you don't have cask yet.
I suggest move the "get cask" box to the top, before the examples.

Please do not open issues if you’re going to submit a PR right after.

Sorry about that. It was not my intend to send the pull request. But I found some free time and decide to do it myself :)


Implement try_mutate for storage value and storage double map

This adds the try_mutate API to Storage Value and Storage Double Map.
try_mutate allows you to execute a closure using the value of a storage item, and only update the value if the closure returns Ok.
Also includes a test!

@bkchr your proposal makes sense to me. I also don't see a reason to duplicate code if we can do it ur way.


Clarify that all processing is local

Changed wording and icons on buttons and emphasize the readme that all processing is done in-browser. Files are not uploaded to and downloaded from remote servers.

Thank you!


Timestamp

Time of messages is always year 1980..

My network server is Chirpstack
This is an example of decoded packet:
{"tst":"2023-09-27T10:57:15.161510+0200","topic":"application/9e548d1d-37cb-43fc-8231-25bfe45508f3/device/685b00ff4c1f9511/event/up","qos":0,"retain":0,"payloadlen":1436,"payload":{"deduplicationId":"a7fed1c4-6243-44af-9310-f3f4e3e3eb00","time":"1980-01-21T23:17:20.234+00:00","deviceInfo":{"tenantId":"52f14cd4-c6f1-4fbd-8f87-4025e1d49242","tenantName":"ChirpStack","applicationId":"9e548d1d-37cb-43fc-8231-25bfe45508f3","applicationName":"counter","deviceProfileId":"408f84b7-f359-4995-8fb7-1a70f0f12f85","deviceProfileName":"device_profile","deviceName":"dev3","devEui":"685b00ff4c1f9511","deviceClassEnabled":"CLASS_A","tags":{}},"devAddr":"010da236","adr":true,"dr":0,"fCnt":183,"fPort":1,"confirmed":false,"data":"dGVzdDMgMjAgc2Vjb25kaQ==","object":{"cmd":"test3 20 secondi"},"rxInfo":[{"gatewayId":"285bd82c7d76e7e1","uplinkId":50925,"time":"2023-09-27T08:57:14+00:00","timeSinceGpsEpoch":"1379840.234s","rssi":-60,"snr":7,"channel":1,"location":{"latitude":43.735555660125584,"longitude":10.95216751098633,"source":"GPS"},"context":"ZRPuag==","metadata":{"region_config_id":"eu868","region_common_name":"EU868"},"crcStatus":"CRC_OK"},{"gatewayId":"4645c2a5de7dfb99","uplinkId":58674,"time":"2023-09-27T08:57:14+00:00","timeSinceGpsEpoch":"1379840.234s","rssi":-60,"snr":7,"channel":1,"location":{"latitude":43.724019581115016,"longitude":10.949935913085939,"source":"GPS"},"context":"ZRPuag==","metadata":{"region_config_id":"eu868","region_common_name":"EU868"},"crcStatus":"CRC_OK"}],"txInfo":{"frequency":868300000,"modulation":{"lora":{"bandwidth":125000,"spreadingFactor":12,"codeRate":"CR_4_5"}}}}}

ok the problem could be the forwarder because the new protocol requires a valid GPS to bring the timestamp, I will investigate on this

I can confirm this issue.
The field timeSinceGpsEpoch contains a bad value, not corresponding to the time field


fix: add a code sample for CreateInstance

@googlebot I consent

@googlebot I fixed it

@googlebot I fixed it.

PTAL
Thanks for the review feedback. It's much cleaner now :)

Hmmm UpdateBackup timed out in the test. I will take a look at these tests hopefully tomorrow as a part of https://github.com/GoogleCloudPlatform/dotnet-docs-samples/issues/1065.

@amanda-tarafa if you can force merge, that would be great. I will skip some more tests in a separate PR.


Issue Implementing Video Calling

Describe the bug
A clear and concise description of what the bug is.
I'm trying to use the SDK to implement video calling. From our backend, I receive a Teams URL which I can join from within our iOS app. When I join the call, I see the camera of the application in the Teams app on my laptop. Thus the SDK is sending the camera. But sadly enough I do not see the preview of the camera in the app. It feels like the problem is with the
self.localVideoView = try self.renderer!.createView() line. I checked to see what the size of Frame is (self.localVideoView.frame) but this resulted in (0, 0, 0, 0). I tried adding the created view as a subview but with no success:
let localView = try self.renderer!.createView()

self.localVideoView.addSubview(localView)

Exception or Stack Trace
Add the exception log and stack trace if available
Device manager is set -> ACS device manager is set
Set audio session -> Created audio session
Set local camera -> Called before the let camera = self.deviceManager!.getCameraList()!.first line
Localview frame: Optional((0.0, 0.0, 0.0, 0.0)) -> Printed the frame of the local view after the try self.renderer!.createView()
Join meeting
Start video success
State changed: Connecting
State changed: Unknown -> I don't know if this is a bug as well, but to me it looks like the user is in the Lobby, but the SDK says that the state is unknown. After I let the user join the call in the Teams app, the state changes to Connected.
To Reproduce
Steps to reproduce the behavior:
I guess you can reproduce it by using the code I added.
Code Snippet
Add the code snippet that causes the issue.
    let locator = TeamsMeetingLinkLocator(meetingLink: "<TEAMS URL>")

    let callOptions = JoinCallOptions()

    let camera = self.deviceManager!.getCameraList()!.first
    let localVideoStream = LocalVideoStream(camera: camera)
    let videoOptions = VideoOptions(localVideoStream: localVideoStream)
    callOptions?.videoOptions = videoOptions

    do {
        self.renderer = try Renderer(localVideoStream: localVideoStream!)

        do {
            self.localVideoView = try self.renderer!.createView()

            guard let call =  self.callAgent?.join(with: locator, joinCallOptions: callOptions) else {
                print("Error: failed to join call")
                return
            }

            self.call = call
            self.call?.delegate = self
            self.call?.startVideo(stream: localVideoStream, completionHandler: { error in
                if let err = error {
                    print("Start video error: \(err)")
                } else {
                    print("Start video success")
                }
            })
        } catch let error {
            print("Failed to create local view: \(error)")
        }
    } catch let error {
        print("Failed to create local renderer: \(error)")
    }

Expected behavior
A clear and concise description of what you expected to happen.
I expect the try self.renderer!.createView() to create a view that shows the video stream of the user.
Screenshots
If applicable, add screenshots to help explain your problem.
The blue View is the self.localVideoView which I made blue to show the View.

Setup (please complete the following information):

OS: MacOS [e.g. iOS]
IDE : Xcode [e.g. IntelliJ]
Version of the Library used: 1.0.0-beta.8

Additional context
Add any other context about the problem here.
The CallClient and CallAgent are all setup fine. There is no problem there as far as I can see.
What might be good to know is the the localVideoView is send from the UIViewController to a class via a protocol. In that class, the view is used. Example code:
protocol TempProtocol {
 func sendLocalVideoView(_ view: UIView)
}

class VideoVC: UIViewController {
   @IBOutlet weak var localVideoView: UIView!

   var delegate: TempProtocol?

   self.delegate.sendLocalVideoView(self.localVideoView)
}

class TempImplementation: TempProtocol {
   var localVideoView: UIView?

   func sendLocalVideoView(_ view: UIView) {
      self.localVideoView = view
   }

  // The code in Code Snippet is used here.

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ X] Bug Description Added
[ X] Repro Steps Added
[ X] Setup information Added

@jimmy555 which module are you importing in your code?
@RezaJooyandeh please tag the appropriate developer.

@tjprescott I'm using these modules:
pod 'AzureCommunicationCalling', '~> 1.0.0-beta.8'
pod 'AzureCommunication', '~> 1.0.0-beta.8'
pod 'AzureCore', '~> 1.0.0-beta.8'

Issue moved to Azure/Communication #212 via ZenHub


single line import of multiple modules not recognized

Below modules are not recognized by the scan.
import <module 1>,  <module 2>, <module 3>

PR #11 fixed this issue


Feature/#25 types definitions

Description

Added task to generate TS types definitions from JSDocs

Any update on when we can expect typescript support?


cni-plugin: fix plugin execution broken by ambient

Please provide a description of this PR:
cni-plugin: fix plugin execution broken by ambient
cni cmdADD will not take any effect for legacy sidecar mode while cni configuration enabling ambient.
Fortunately/unfortunately, redirection still works in sidecar because we have the repair/reconcile mechanism for sidecar mode.

This is a miss on my part from merging the internally-separate containers back into one CNI daemonset - ty @PlatformLC!

@PlatformLC: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
547656762b97c891ee921745b0b92d0115a50d14
link
true
/test lint

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

In response to a cherrypick label: new pull request created: #49230


DataRobot Integration

Goal
As a developer, I want to integrate Datarobot, an automated ML solution, so that I can quickly explore various ML models.
Inspiration
The DataRobot automated machine learning platform helps data scientists and business analysts discover the best predictive models for every situation, and then deploy them so they can consistently make smarter and faster business decisions that impact their company's bottom line. (DataBricks, 2018)
Here is the standard machine learning process at a high level (DataRobot, 2018)

Important

DataRobot requires a purchase of a license. Contact DataRobot for pricing options.

Closing issue to focus on open-source alternatives to DataRobot


C: clamping of budget in elements_simplicity_execSimplicity is incorrect

In elements_simplicity_execSimplicity we clamp the passed budget value to ensure it does not exceed BUDGET_MAX. But we do not check for negative values. If you pass a negative value, it will be casted to a ubounded that exceeds BUDGET_MAX, eventually triggering an assertion failure in eval.c.
There are three ways we might handle negative values:

clamping to 0
returning an error
asserting in execSimplicity

I have no preference about which.

I'm leaning towards making the budget non-negative a precondition and adding an assertion in execSimplicity.

Fixed in 'staging'.


"platformchecktable" doesn't apply to "trackplatform"

There should be a "trackplatformchecktable" or "platformchecktable" should apply to track platforms

#268

OH


Add placeholder if parts of author information are missing

For example: https://staging.dracor.org/ita/beccari-il-sacrificio-1555

"b. , Ferrara"

In this case, year of birth is missing. If either years or places of birth/death are missing, we should put a placeholder to make transparent that this info is not available in Wikidata (or not known at all). Could be a question mark "?", or something like "[n/a]".

Couldn't we just omit the missing part? So in the example we would display "b. Ferrara". When the birth or death place is missing it would look like "b. 1234". This was the reason I didn't put a comma between year and place in the original implementation. But of course we could omit the comma if only one part of information is available.

Good solution, but what if we have neither date nor location, would this yield an empty "b." line?

When there is no information, we'd just omit the b. or d. line.


Yosys does not support $changed

$changed(X) is defined in SVA to be equivalent to (A != $past(A));
Steps to reproduce the issue
The following code,
always @(posedge i_clk)
if (f_past_valid)
    assert($changed(X) == (X != $past(X));

will cause Yosys to fail with an error, "Can't resolve function name '$changed'."
Expected behavior
The example above should compile and pass without problems.
Actual behavior
See above.
Proposed solution
See the attached patch for a solution to this issue.
patch.txt
Does this mean I now (finally) qualify for guru status?

See the attached patch for a solution to this issue.

Applied in 62424ef.

Does this mean I now (finally) qualify for guru status?

You already have that status. See e060375.  :)


Styleguide font vars

Open to suggestions and will likely change as we see more pages finalized
Trying out set styles for headings and subheadings
headings -> display-1:6, adelle, aleo, serif
subheadings -> h1:6, proxima nova, open sans, san serif

Updated so that it is up to each page to set font-sizes and styling for now (until we get a more unified approach)
I will (in another PR when I have time) continue to look at what things we can reduce to variables for a more consistent base to design from in the future


[Bug]: The 'video.api.livestream/events' channel sent a message from native to Flutter on a non-platform thread

Version
v1.1.1
Which operating systems have you used?

[ ] Android
[X] iOS

Environment that reproduces the issue

iOS simulator iPhone 15 Pro max

Is it reproducible in the example application?
Yes
RTMP Server
api.video
Reproduction steps

start the streaming in ios
stop it
you can see the error message in the console

Expected result
There should not be any error message in the console and the package should have sent the Platform channel messages on the platform thread
Actual result
The app give the following error message in console
[ERROR:flutter/shell/common/shell.cc(1015)] The 'video.api.livestream/events' channel sent a message from native to Flutter on a non-platform thread. Platform channel messages must be sent on the platform thread. Failure to do so may result in data loss or crashes, and must be fixed in the plugin or application code creating that channel.
See https://docs.flutter.dev/platform-integration/platform-channels#channels-and-platform-threading for more information.
Additional context
Example:
ApiVideoLiveStreamController createLiveStreamController() {
    return ApiVideoLiveStreamController(
      initialAudioConfig: rtmsConfigaudioConfig,
      initialVideoConfig: rtmsConfig.videoConfig,
      onConnectionSuccess: () {
        log('VideoStream: Connection succeeded');
      },
      onConnectionFailed: (error) {
        log('VideoStream: Connection failed: $error');
      },
      onDisconnection: () {
        log('VideoStream: Connection Disconnected');
      },
      onError: (err) {
        log('VideoStream: Connection Error: $err');
      },
    );
  }

Relevant logs output
[log] VideoStream: Stream Started
[ERROR:flutter/shell/common/shell.cc(1015)] The 'video.api.livestream/events' channel sent a message from native to Flutter on a non-platform thread. Platform channel messages must be sent on the platform thread. Failure to do so may result in data loss or crashes, and must be fixed in the plugin or application code creating that channel.
See https://docs.flutter.dev/platform-integration/platform-channels#channels-and-platform-threading for more information.
[log] VideoStream: Connection succeeded

Could you test branch with fix https://github.com/apivideo/api.video-flutter-live-stream/tree/fix/ios_event_thread?

But now it takes several seconds to to get the onConnectionSuccess callback.
Previously, it happens instantly

Hi,
I made a fix in the iOS native RTMP lib: https://github.com/apivideo/api.video-swift-live-stream
I will update this dependency before releasing a new version of apivideo_live_stream.

Thank you
I will keep an eye on the update

Could you test on the main branch?

yeah but does it include the changes made for Issue 48 for Android too?

yes

i tried it using the main branch, it is still taking several seconds for the stream connection to succeed.

https://github.com/apivideo/api.video-flutter-live-stream/assets/11173809/c980fab2-719b-4a50-a473-f35992de4177
You can see in the video, it takes around 10 seconds for the connection to succeed.

🤔

Do you have the same issue with the example?
If yes, do you have the example from https://github.com/apivideo/api.video-swift-live-stream?

Yes i have the same issue while running the example in iOS too
NO, I don't have an example from that URL

Sorry, could you test with the example from https://github.com/apivideo/api.video-swift-live-stream?

I think that repo is for native iOS app. I am using flutter so not sure how

Do you have xcode? I guess yet because you need it for Flutter.
Run open Examples/ExampleUIKit/ExampleUIKit.xcodeproj/
If I ask so, it just to know if the issue comes from the Flutter iOS part or the native iOS part.

It worked really fast. No issues

I did have a look at Flutter iOS code but could not find anything.
Are you sure to be up to date? (last version of the internal library with pod update,...)

Yes.
I did pull the latest branch from git and run pod update, pod install --repo-update.
Installed and run correctly. just giving me that 10 seconds delay to succeed the connection

The original issue is solved in v1.2.0. Please reopen it if you feel something is missing.
About your connection issue, as it is not related to the opened issue. Please open a new issue.


Including the key in the mapper function

Great utility library! Ran into this exact scenario and was about to implement it myself until I stumbled across this.
One thing that would be great would be to add support for getting the key in the mapper function in createObjectSelector. For instance I might have an example like:
import { createObjectSelector } from 'reselect-map'

const exampleState = {
  numbers: { a: 1, b: 2 },
}

const mul = createObjectSelector(
  state => state.numbers,
  state => state.multiplier,
  (number, key, multiplier) => `${key}-${number * multiplier}`
)

console.log(mul(exampleState)) // { a: 'a-5', b: 'b-10' }

Lodash and other utility libraries typically provide the key also, typically as the second argument. Seems like it would be easy to change https://github.com/HeyImAlex/reselect-map/blob/master/src/memoize.js#L80 from
const result = fn(value, ...args)

to
const result = fn(value, key, ...args)

Obviously this is a breaking change and not every mapper would necessarily need the key, so thought I'd see what approach you'd take before submitting a PR

Hm, I would be hesitant to add this only because it's a breaking change, but tentatively I think you're right that it's a better api. I'm about to head home so I'll mull over this tonight.

I supposed you could throw it as the last argument to the mapper and it wouldn't be breaking. Seems a little clunky back there but better than nothing!

I'm not completely opposed to a breaking change, and it looks like a good choice. I'm going to sit on this issue for a little while though, until I have time to think about it some more.

Hi @mjrussell, can use use Array selector instead of Object selector? Something like this.
`import { createArraySelector } from 'reselect-map'
const exampleState = {
numbers: [ {key: 'a', value: 1}, {key: 'b', value: '2' }],
}
const mul = createArraySelector(
state => state.numbers,
state => state.multiplier,
(number, multiplier) => ${number.key}-${number.value * multiplier}
)
console.log(mul(exampleState)) // [ a: 'a-5', b: 'b-10' ]`
It could solve your problem.
Anyway, I agree the API with key on object selector would be nicer. But if you do this change, to be consistent, there should be also index on array selector API ("Why is the key on object selector but not the index on array selector?"). But for array selector its performance killer (you will have to reevaluate array items because of index change - this will break the main benefit of array selector).

@LiborOl if I was going to walk the data set to create an array as you describe, I might as well just _.map across the object and put the key inside the object values. However the whole point of this library is to prevent unnecessarily walking through the map each time.

But for array selector its performance killer (you will have to reevaluate array items because of index change - this will break the main benefit of array selector).

This would only be true if you are not inserting at the end of your array.

I think, it wouldn't be so easy. The _.map will create new objects so all the items will be reevaluated. You should reuse the old ones. Or I am missing something?

This would only be true if you are not inserting at the end of your array.

Of course just for items with changed indices (and not only inserting but also deleting). E.g. we are inserting in one big array at the beginning (the newest items on the top - not so rare I think). So we will not be happy at all with the index in the API :-)

I think, it wouldn't be so easy. The _.map will create new objects so all the items will be reevaluated. You should reuse the old ones. Or I am missing something?

Well sure, but how do you supposed I get from
numbers: { a: 1, b: 2 } to numbers: [{ key: 'a', value: 1 }, { key: 'b', value: 2}] without mapping if this is the response from the server? I'd have to do it somewhere.
Fair point about the array, seems like a good reason to keep that API as is, but doesn't prevent changing the object one.

You should create new array and merge there existing items from old array and insert new items from the response (and remove items that are not in the response anymore).
I don't know, if you have really only numbers(simple values) in the response object are there are some complex objects. If there are some complex objects, you should do the merge with old array anyway.

Yeah, I think adding an index to the array selectors would be a mistake. Arrays are actually treated more like sets internally, so to add indexes we could:

Make each element memoized by index (like maps and objects are memoized by keys). That would be terrible for performance as you've said; deleting the first element of an array would lead to every remaining element being recomputed.
Use the same memoizing system but still pass the index. This makes the result of the selector impure, which would lead to confusing cache invalidation issues.
Add a separate function for doing it this way. But I don't like having two separate nuanced ways to map over arrays. If it's really needed, the lower level objectMemoize makes it easy enough to write yourself.

That none of these options are particularly good is why I didn't include the key in the first place. However, that was kind of shortsighted because obviously it doesn't cause any issues with key-based memoization. Though it might be a little confusing that they're not perfectly symetric, I think that adding keys to key-based memoization functions and leaving set based functions alone is the way forward.
tl;dr: ya

Bikeshedding on how the key should be passed:
Mapping apis usually look like (value, key, collection), and that's probably because key often isn't used and callbacks can just leave out that argument. I would imagine that the common case will be ignoring the key, so I think we should optimize for that case.
With (value, key, ...args), you need to include the key argument even if you don't want it. If we were going to force the key, I would actually prefer (key, value, ...args) since I usually think of keys and values in that order. But right now I'm leaning toward the more clunky (value, ...args, key) that you suggested earlier. Key is optional, it kind of mirrors the mapping convention, and it's not breaking.
If there aren't any objections, I can write up this change pretty quick, or if someone else wants to submit a PR that's cool tool.

I agree that while (value, ...args, key) looks clunky at first, it does seem more similar to other libraries and is not breaking. Im for it, and happy to submit a PR if we are agreed. Should be a 1 line change and maybe a new test.

Alright, should be good to go. Thanks to everyone involved! Hopefully I didn't break anything. It's been a somber morning...