Installing OR-Tools on Google Cloud Functions

I'm trying to run my python script as a cloud function and wish to install or-tools when I deploy my cloud function.
My requirements.txt
flask==1.1.1
google-cloud-error-reporting==0.33.0
ortools

If I run a gcloud deploy:
gcloud functions deploy optimise --project my-project --source ./src --runtime python37 --trigger-http

I get:
ERROR: (gcloud.functions.deploy) OperationError: code=3, message=Function failed on loading user code. Error message: Code in file main.py can't be loaded.
Did you list all required modules in requirements.txt?
Detailed stack trace: Traceback (most recent call last):
  File "/env/local/lib/python3.7/site-packages/google/cloud/functions/worker.py", line 305, in check_or_load_user_function
    _function_handler.load_user_function()
  File "/env/local/lib/python3.7/site-packages/google/cloud/functions/worker.py", line 184, in load_user_function
    spec.loader.exec_module(main)
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/user_code/main.py", line 1, in <module>
    from ortools.linear_solver import pywraplp
ModuleNotFoundError: No module named 'ortools'

Is it possible to install on CFs? It uses a supported Ubuntu version etc.

Following https://cloud.google.com/functions/docs/quickstart-python
I've just created a Google Function using:
Runtime
Python 3.7
requirements.txt
# Function dependencies, for example:
# package>=version
ortools

main.py
"""Vehicle Routing Problem"""
from ortools.constraint_solver import routing_enums_pb2
from ortools.constraint_solver import pywrapcp

###########################
# Problem Data Definition #
###########################
def create_data_model():
    """Stores the data for the problem."""
    data = {}
    data['distance_matrix'] = [
        [
            0, 548, 776, 696, 582, 274, 502, 194, 308, 194, 536, 502, 388, 354,
            468, 776, 662
        ],
        [
            548, 0, 684, 308, 194, 502, 730, 354, 696, 742, 1084, 594, 480, 674,
            1016, 868, 1210
        ],
        [
            776, 684, 0, 992, 878, 502, 274, 810, 468, 742, 400, 1278, 1164,
            1130, 788, 1552, 754
        ],
        [
            696, 308, 992, 0, 114, 650, 878, 502, 844, 890, 1232, 514, 628, 822,
            1164, 560, 1358
        ],
        [
            582, 194, 878, 114, 0, 536, 764, 388, 730, 776, 1118, 400, 514, 708,
            1050, 674, 1244
        ],
        [
            274, 502, 502, 650, 536, 0, 228, 308, 194, 240, 582, 776, 662, 628,
            514, 1050, 708
        ],
        [
            502, 730, 274, 878, 764, 228, 0, 536, 194, 468, 354, 1004, 890, 856,
            514, 1278, 480
        ],
        [
            194, 354, 810, 502, 388, 308, 536, 0, 342, 388, 730, 468, 354, 320,
            662, 742, 856
        ],
        [
            308, 696, 468, 844, 730, 194, 194, 342, 0, 274, 388, 810, 696, 662,
            320, 1084, 514
        ],
        [
            194, 742, 742, 890, 776, 240, 468, 388, 274, 0, 342, 536, 422, 388,
            274, 810, 468
        ],
        [
            536, 1084, 400, 1232, 1118, 582, 354, 730, 388, 342, 0, 878, 764,
            730, 388, 1152, 354
        ],
        [
            502, 594, 1278, 514, 400, 776, 1004, 468, 810, 536, 878, 0, 114,
            308, 650, 274, 844
        ],
        [
            388, 480, 1164, 628, 514, 662, 890, 354, 696, 422, 764, 114, 0, 194,
            536, 388, 730
        ],
        [
            354, 674, 1130, 822, 708, 628, 856, 320, 662, 388, 730, 308, 194, 0,
            342, 422, 536
        ],
        [
            468, 1016, 788, 1164, 1050, 514, 514, 662, 320, 274, 388, 650, 536,
            342, 0, 764, 194
        ],
        [
            776, 868, 1552, 560, 674, 1050, 1278, 742, 1084, 810, 1152, 274,
            388, 422, 764, 0, 798
        ],
        [
            662, 1210, 754, 1358, 1244, 708, 480, 856, 514, 468, 354, 844, 730,
            536, 194, 798, 0
        ],
    ]
    data['num_vehicles'] = 4
    data['depot'] = 0
    return data

###########
# Printer #
###########
def print_solution(data, manager, routing, assignment):
    """Prints assignment on console."""
    print('Objective: {}'.format(assignment.ObjectiveValue()))
    total_distance = 0
    for vehicle_id in range(data['num_vehicles']):
        index = routing.Start(vehicle_id)
        plan_output = 'Route for vehicle {}:\n'.format(vehicle_id)
        route_distance = 0
        while not routing.IsEnd(index):
            plan_output += ' {} ->'.format(manager.IndexToNode(index))
            previous_index = index
            index = assignment.Value(routing.NextVar(index))
            route_distance += routing.GetArcCostForVehicle(
                previous_index, index, vehicle_id)
        plan_output += ' {}\n'.format(manager.IndexToNode(index))
        plan_output += 'Distance of the route: {}m\n'.format(route_distance)
        print(plan_output)
        total_distance += route_distance
    print('Total Distance of all routes: {}m'.format(total_distance))
    return total_distance

########
# Main #
########
def main(request=None):
    """Entry point of the program"""
    # Instantiate the data problem.
    data = create_data_model()

    # Create the routing index manager.
    manager = pywrapcp.RoutingIndexManager(len(data['distance_matrix']),
                                           data['num_vehicles'], data['depot'])

    # Create Routing Model.
    routing = pywrapcp.RoutingModel(manager)

    # Create and register a transit callback.
    def distance_callback(from_index, to_index):
        """Returns the distance between the two nodes."""
        # Convert from routing variable Index to distance matrix NodeIndex.
        from_node = manager.IndexToNode(from_index)
        to_node = manager.IndexToNode(to_index)
        return data['distance_matrix'][from_node][to_node]

    transit_callback_index = routing.RegisterTransitCallback(distance_callback)

    # Define cost of each arc.
    routing.SetArcCostEvaluatorOfAllVehicles(transit_callback_index)

    # Add Distance constraint.
    dimension_name = 'Distance'
    routing.AddDimension(
        transit_callback_index,
        0,  # no slack
        3000,  # vehicle maximum travel distance
        True,  # start cumul to zero
        dimension_name)
    distance_dimension = routing.GetDimensionOrDie(dimension_name)
    distance_dimension.SetGlobalSpanCostCoefficient(100)

    # Setting first solution heuristic.
    search_parameters = pywrapcp.DefaultRoutingSearchParameters()
    search_parameters.first_solution_strategy = (
        routing_enums_pb2.FirstSolutionStrategy.PATH_CHEAPEST_ARC)

    # Solve the problem.
    assignment = routing.SolveWithParameters(search_parameters)

    # Print solution on console.
    if assignment:
        distance = print_solution(data, manager, routing, assignment)

    request_json = request.get_json()
    if request.args and 'message' in request.args:
        return request.args.get('message')
    elif request_json and 'message' in request_json:
        return request_json['message']
    else:
        return f'{distance}'

if __name__ == '__main__':
  main()

then using the Testing tab and clicking on Test the function, I've got:
Output
6208

as expected...
Did you try to test you function using the web interface aka https://console.cloud.google.com/functions/... ?

Do you use a 64bits OS ?

Hey @Mizux - I didn't use the web interface, was getting the errors deploying via the CLI. Did that work for you?
i.e. gcloud functions deploy optimise --project my-project --source ./src --runtime python37

didn't test the cli, need to see how i can install it on my archlinux

Thanks for the help - ideally like to deploy with github actions so the CLI command is fairly critical.

so after installing the last gcloud CLI (275.0 IIRC),  I put the two files in a src directory then run:
$> gcloud functions deploy or-tools --source ./src --runtime python37
$> gcloud functions call or-tools --data "{}"
executionId: jqpdh8pybik8
result: 'total distance: 6208'

So unfortunately I can't reproduce your issue :(
note: I add total distance:  to the output string to verify my deploy was correct.


⚠ TROJAN 🇭🇰 Hong Kong HKE 1 has degraded performance

In 6d76d99, TROJAN 🇭🇰 Hong Kong HKE 1 (https://hkt-1.opensvr.net/) experienced degraded performance:

HTTP code: 200
Response time: 14350 ms

Resolved: TROJAN 🇭🇰 Hong Kong HKE 1 performance has improved in e287276.


Don't delete a DataPackage's loaded zip contents until the DP is garbage collected

When creating a DataPackage from a ZIP file, we extract its contents into a temporary folder. This folder was deleted as soon as we finish loading the DataPackage, which is fine for most cases. But on https://github.com/ckan/ckanext-datapackager, we need the original files to upload the resources to CKAN. This means we need that temporary folder with the ZIP contents a bit longer.

The broken test is related to a bug on httpretty 0.8.12 (https://github.com/gabrielfalcao/HTTPretty/issues/275).


ite: drivers/adc: enable keeping ADC data buffer

it8xxx2's ADC data is 10-bit and saving in two registers
(VCHxDATL/bit7-0 and VCHxDATM/bit9-8).
The CL enables the function of keeping ADC data buffer to prevent
reporting incorrect ADC raw data due to the following race condition:

adc data valided: 0 0xf8 (voltage is 3x248/1024=0.72v)
ec read VCHxDATM(0x0)
next adc conversion done: 0x1 0x08 (voltage is 3x264/1024=0.77v)
ec read VCHxDATL(0x8)
EC will return incorrect voltage (3x8/1024=0.02v) to caller for above
conditions.

Signed-off-by: Tim Lin<EMAIL_ADDRESS>

Abandoned this PR, Fixed it in other PR.
https://github.com/zephyrproject-rtos/zephyr/pull/34901


Johtaylo/adaptivedialogbot

A IBot implementation that manages AdaptiveDialogs with minimal and explicit dependencies internally and externally.

This seems like a PR that should wait for R14, and the potential V5 versioning. The layering - is this runtime, or is this SDK - should be discussed.

This seems like a PR that should wait for R14, and the potential V5 versioning. The layering - is this runtime, or is this SDK - should be discussed.

Other than making an assumption about the names of some properties in IConfiguration this doesn't change any layering.
Regarding configuration, in other places we have:
class SomeClass;
class ConfigurationSomeClass : SomeClass;
where ConfigurationSomeClass provides an additional constructor that takes IConfiguration
The reason we did that was because we wanted to first class both dotnet core and webapi with each having different configuration systems.
So I could add this, in the spirit of sticking with that pattern:
class ConfigurationAdapterDialogBot : AdaptiveDialogBot;
Otherwise this is DialogManager implementing IBot. Because you would only do that if you were using AdaptiveDialogs it also takes the extra simplification of tying that to AdaptiveDialog and because you would only create AdaptiveDialogs from JSON it takes the next simplification of typing that to ResourceExplorer.
The class is in Dialogs.Adaptive whereas DialogManager was in Dialogs. Although technically DialogManager could be used to run a Dialog other than AdaptiveDialog it never was used to do that, there was no point. The only real scenario for using DialogManager was running AdaptiveDialogs. DialogManager even pulled in Telemetry.
So anyhow, I don't think this muddies the waters at all in terms of layering or coupling between SDK and Composer/Adaptive in fact it slightly improves the situation by moving the Adaptive specific Dialog management tasks into the Adaptive Dialog package.
I guess what I'm saying is the coupling exists within the SDK, this class doesn't introduce it, it just makes it clear. The exception to that statement might be including the configuration: how about I add that ConfigurationAdaptiveDialogBot class in one of the integration packages?

Pending review by @gabog , @stevengum, @joshgummersall.


[FEATURE] Adding preloader to child therapy page

Description
I would like to add to  preloader child therapy page
Screenshots
No response
Additional information
No response

Hello there! I followed a quick yt tutorial for this and have the following changes for this ->
childTherapy.html -
<body>
+ <div class="preloader">
+   </div>

<!-- Navbar Section -->
    <nav>

  </footer>
  <script src="childTherapy.js"></script>

+ <script>
+    var loader = document.querySelector(".preloader");

+   window.addEventListener("load", function(){
+     loader.style.display = "none";
+    });
+ </script>

  </body>

childTherapy.css (all additions)
.preloader {
  height: 100vh;
  width: 100%;
  background: #1C1B1B url(../images/comp_4.gif) no-repeat center center;
  position: fixed;
  z-index: 100;
  background-size: 25%;
}

Used a simple pre loader gif and tailored the background with respect to the gif.
Does this meet the specifications / demands for the feature? Should i open a PR with these changes


Cis v2 scheduling, alerting  and also custom benchmark docs

This is an initial draft of the following CIS v2 enhancements:

scheduling
alerting
custom benchmark

https://github.com/rancher/docs/issues/2848

I have made some edits for organization and put information about 2.5.0 and 2.5.4 in separate tabs. I'll put it in staging so it's a little easier to see the result.

@prachidamle @sowmyav27 do you have an example alertmanager config? I think it might help to see the alert route configuration in context.

While configuring the routes for rancher-cis-benchmark alerts, you can specify the matching using the key-value pair job: rancher-cis-scan.

@catherineluse A route configured - (A receiver using "Slack" has been configured)
spec:
  receiver: test-cis
  group_by:
#    - string
  group_wait: 30s
  group_interval: 30s
  repeat_interval: 30s
  match:
    job: rancher-cis-scan
#    key: string
  match_re:
    {}
#    key: string

@sowmyav27 I have made the updates here https://github.com/rancher/docs/commit/ccceb987d49e651bce1760caf5d838e2c4dde7f1

LGTM


fix worker entry

the allows variable has no value on the worker entry.

Ah thanks for this one @cometkim super good!


Add docs for Local x Existing deployments - WIP

Reference Issues or PRs
closes #108
closes #104
What does this implement/fix?

Local installation notes for Development and Testing Nebari locally
Copied old installation notes for deploying Nebari on existing k8s infrastructure
Refactored definition of local installation
merged cloud options into deployment options
more to come

Put an x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features to not work as expected)
[x] Documentation Update
[x] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Documentation

Access-centered content checklist
Text styling

[ ] The content is written with plain language (where relevant).
[ ] If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
[ ] All links describe where they link to (for example, check the Nebari website).
[ ] This content adheres to the Nebari style guides.

Non-text content

[ ] All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
[ ] If there are emojis, there are not more than three in a row.
[ ] Don't use flashing GIFs or videos.
[ ] If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

I am currently working on the existing part, as this needs to be tested for each cloud provider (to validate we have up-to-date information) this takes some time. I am writing the GCP part and will update this until EoW.
The local section can be reviewed already. I need to update the MacOS part once we get that working, though...

Closing this in favor of creating a new How To guide for "existing" kubernetes clusters.


"Enumerable not implemented for nil" on subscription creation

Following the docs and trying to set up elixometer with influxDB, I get the following error on the first call to Elixometer.update_counter/2. I've not found any configuration changes that get the subscription step to succeed.
I've set up a small example repo displaying the problem: https://bitbucket.org/kmeehl/exotest
** (Protocol.UndefinedError) protocol Enumerable not implemented for nil. This protocol is implemented for: Date.Range, File.Stream, Function, GenEvent.Stream, HashDict, HashSet, IO.Stream, List, Map, MapSet, Range, Stream
            (elixir) lib/enum.ex:1: Enumerable.impl_for!/1
            (elixir) lib/enum.ex:116: Enumerable.reduce/3
            (elixir) lib/enum.ex:1832: Enum.map/2
            (elixometer) lib/elixometer.ex:352: Elixometer.create_subscription/1
            (elixometer) lib/elixometer.ex:309: Elixometer.handle_call/3
            (stdlib) gen_server.erl:636: :gen_server.try_handle_call/4
            (stdlib) gen_server.erl:665: :gen_server.handle_msg/6
            (stdlib) proc_lib.erl:247: :proc_lib.init_p_do_apply/3
    (elixir) lib/gen_server.ex:774: GenServer.call/3
    (elixometer) lib/elixometer.ex:293: Elixometer.ensure_registered/2
    (elixometer) lib/updater.ex:79: Elixometer.Updater.do_update/1
    (elixir) lib/enum.ex:675: Enum."-each/2-lists^foreach/1-0-"/2
    (elixir) lib/enum.ex:675: Enum.each/2
    (elixometer) lib/updater.ex:50: Elixometer.Updater.handle_info/2
    (stdlib) gen_server.erl:616: :gen_server.try_dispatch/4
    (stdlib) gen_server.erl:686: :gen_server.handle_msg/6
Last message: {:mail, #PID<0.279.0>, [{:counter, "Exoteststartcount", 1, nil}], 1, 0}

Hi @kmeehl, unfortunately there is backwards incompatible change in exometer core 1.5. You'll need 1.4 for now.

Thanks! I'll try 1.4. Is this documented somewhere, and I just missed it?

Hi @fishcakez, I've tried changing my deps to use exometer ~> 1.4, but I'm seeing the same problem. Can you please describe what my deps should look like in full to get this working? Here's what I have currently:
{:elixometer, "~> 1.2"},
{:exometer_core, "~> 1.4"},
{:exometer_influxdb, "~> 0.5.7"},
{:setup, github: "uwiger/setup", manager: :rebar, override: true},
{:pobox, "~> 1.0"},
{:edown, "0.8.1", override: true},

i too stumbled across this; unfortunately, this means that currently there is no way to use elixometer with erlang 20 as exometer_core 1.4 is incompatible with that erlang version or am i mistaken?

@kmeehl @mzdrahal as far as I know elixometer 1.2.1 should work with Elixir 1.4.5 and OTP 20 if you have lager as mix dep override entry:
 {:lager, ">= 3.2.1", override: true}

Note I made a mistake previously and meant ~> 1.4.0 for exometer_core but that is the version requirement of elixometer 1.2.1 so a dep entry is not required.
Unfortunately Elixometer does not have a release that is compatible with Elixir 1.5 but master is.

This is fixed by #94.


[Plot] Matplotlib deprecation warning

Seen in continuous integration.

This has been fixed in PR #2890, and it is available in v0.13.0b0 so you should get it with pip install --pre.


Practically removes OldDoFn from SDK

After this change, the only usages of OldDoFn in the SDK are in combine context stuff, which itself should be moved to runners-core (outside the scope of this change).

(don't review this yet, until I split into commits)

:champagne: :clap:

Ready to review! (modulo tests - should pass, but ready for review regardless)

R: @kennknowles

Some changes have gone in that will require a rebase to resolve conflicts, and there are some checkstyle issues that failed Jenkins early. Given the nature of this change, the signal from integration tests is important.
I'll begin review anyhow, since I expect it to be easy and the results to be stable across rebase & any test fixing.

Please tag the entire PR with [BEAM-498]

Thanks, PTAL.

Jenkins, retest this please.

retest this please

Ugh fixed the checkstyle issue during rebase, but the rest of the rebase reintroduced it. Apparently my autosquash-foo is not good enough. Pushed fix.

Apex failure is https://issues.apache.org/jira/browse/BEAM-1140. I didn't notice these checkstyle failures, but fixed them now, sorry for the churn.

Jenkins passed, Travis failure looks flaky.

LGTM on Spark runner changes, they would be mostly unnecessary if we merge #1578, but I can merge this later as well.

Since #1578 is functionality and this is refactor, I will favor the former.

I've merged #1578 so now if you rebase and fix up this should be good. We have resolved all (known) precommit breakage, too.

LGTM on Spark runner changes.

LGTM. All green. Merging!


/etc/resolv.conf: Read-only file system

Getting this error on docker-compose build
Dockerfile-cmdline has been updated with jasper reports version 7.8.0


Add BibleNLP dataset

Checklist for adding MMTEB dataset

This dataset has an extremely large coverage of 829 languages, and includes numerous low-resource languages. For this reason I only considered English-centric directions, resulting in a reasonable 1656 pairs. I will add performance results later, but initial testing suggests that while quality is not good for most (low-resource) languages, it is far better than random guesses, so IMO a good addition to the benchmark.

[x] I have tested that the dataset runs with the mteb package.
[x] I have run the following models on the task (adding the results to the pr). These can be run using the mteb run -m {model_name} -t {task_name} command.

[x] sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
[x] intfloat/multilingual-e5-small

[x] I have checked that the performance is neither trivial (both models gain close to perfect scores) nor random (both models gain close to random scores).
[x] If the dataset is too big (e.g. >2048 examples), considering using self.stratified_subsampling() under dataset_transform()
[x] I have filled out the metadata object in the dataset file (find documentation on it here).
[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.
[ ] I have added points for my submission to the points folder using the PR number as the filename (e.g. 438.jsonl).

Could you take a look at this PR  @KennethEnevoldsen @imenelydiaker? I'm personally quite excited by the large language coverage.

I have addressed your comments @isaac-chung @dokato, thanks for your suggestions!
Results are included now as well. intfloat/multilingual-e5-small performs best, but like I pointed out the performance is bad for most low-resource languages. However, almost all directions are way better than random chance (1/256), so this seems like a valuable addition to evaluate (future) multilingual models.
I think this is ready to merge now, do you agree? Let me know, then I'll calculate the points. Thanks!

Thanks for your quick response! I calculated the number of new languages using this script: 756 new languages.
I'll update the points and merge.

Ready to merge now @isaac-chung (I don't have write access)


Dropdowns a11y [done]

Description
I have taken work of @ekrasnopolskaia that was made a month ago and made Select component accessible in the most basic case (without multiselect).
Motivation and Context
Dropdowns&co. are the most inaccessible components if Intergalactic components. Making them accessible will significantly increase accessibility of all library.
How has this been tested?
Currently I have not added tests but planning to add playwright voice over tests.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Planned todo

[ ] Voice over automated tests
[ ] Dropdown component accessibility
[ ] DropdownMenu component accessibility
[ ] AutoSuggest component accessibility
[ ] BaseTrigger component warnings if needed aria attributes are not provided

LGTM! only one thing - is it ok that some components don't have changes in CHANGELOG file? I mean accordion, checkbox and color-picker

Thank you for review! Yes, that was a mistake (except color-picker, it's changes don't affect UX and DX and targeted to more stable unit tests.
I have added changelogs, will merge as soon checks will be green


Interface with caesar service

Design a UI panel to enable users to:

upload a selected 2D image file in FITS format to the service storage. The service returns the file uid to be used for later operations (e.g. job submission). The image can be one of the image currently present in VLVA (taken from local filesystem or from VLKB). At present, the service accepts only an image file, so maybe we need to foresee the possibility to send encoded image data, with format to be agreed.
submit source finding jobs on the uploaded image, using one of the selected algorithm (or apps), for example caesar, and job options configurable from the UI.
query job status, e.g. periodically, and update status in UI
retrieve job outputs (a tar file, or individual data files) and import them in the UI for later operations (e.g. source visualization and analysis). For example, one of the outputs is an ascii (or json) catalog file with list of extracted sources. Another one is a DS9 region with the list of extracted sources defined as DS9 polygons.

Service API are available at: https://github.com/SKA-INAF/caesar-rest
An interface example is the caesar-ui developed by Cristobal.
Maybe worth to organize a joint meeting with cristobal to show the features.

When AAI is enabled, the caesar service requires from the UI a valid OpenID connect token. We are currently using the NEANIAS AAI service so guess this is very similar to what has been done for the VLKB integration. We can ask Cristobal some details here.
For the future we need to discuss also to have a unique AAI service (in case VLKB and caesar service differs) and which one (the NEANIAS, EOSC, IA2 or something implemented by us).


Collectd http write plugin

I would like to request development of a plugin compatible with Write_HTTP.
https://collectd.org/wiki/index.php/Plugin:Write_HTTP

This would be fantastic.


(PDB-2360) Fix aliasing in aggregates and group-by

This commit fixes a bug with aggregates and group-by where the fields
used as arguments to those operators (function and group_by) weren't
being converted to our fields (honeysql) constructs which have the
table disambiguation and correct names for certain query API fields
(e.g. queries to /nodes grouping by certname now work instead of
erroring with an ambiguous column message, and queries to /reports
with ["function","min","report_timestamp"] will now properly rename
report_timestamp to reports.end_time in the SQL).

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-stable/135/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-stable/137/
Test PASSed.

+1


Use mssql/server:2022-latest for MsSql container

Motivation
The MsSql image mcr.microsoft.com/azure-sql-edge:latest is very flaky at startuo, probably because of this issue : microsoft/mssql-docker#868
This image may not support kernel 6.7 and above, and we're using 6.8 kernel on github action large runners.
Changes
Use mcr.microsoft.com/mssql/server:2022-latest image
Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team

just FYI, this change seems to make our tests pass: https://github.com/DataDog/dd-trace-js/pull/4710 (inspired by https://github.com/microsoft/mssql-docker/issues/868#issuecomment-2080122626)

Yes, the ticket says that the 2019 fix has been ported to 2022. Let;'s try with 2022.


Fix build

Android Ant build system is no longer supported.
Move to gladle is deal, However javadoc tasks themselves doesn't depends on Android Ant build system so that we can still use them.

LGTM


All traces are named RunnableSequence - Langchain + JS

Bug description
Currently all the traces are named "RunnableSequence", is there a way to name the trace?

How to reproduce
I'm currently using it with langchain.js like this:
import * as dotenv from "dotenv";
import { PromptTemplate } from "@langchain/core/prompts";
import { ChatOpenAI } from "@langchain/openai";
import { JsonOutputParser } from "@langchain/core/output_parsers";
import { CallbackHandler } from "langfuse-langchain";

dotenv.config();

const prompt = PromptTemplate.fromTemplate(`Tell me a joke about {param}`);

const llm = new ChatOpenAI({modelName: "gpt-4-turbo", verbose: true});

const parser = new JsonOutputParser();
const chain = prompt.pipe(llm).pipe(parser).withConfig({ runName: "getRelatedQuestions" });

chain.name = "getRelatedQuestions";

await chain.invoke({ param: "a black cat"}, { callbacks: [new CallbackHandler({flushAt: 1})] });

I've created a project that shows the issues: https://github.com/guidev/langfuse-bug-traces
Related sub-libraries

[ ] All of them
[x] langfuse
[ ] langfuse-node

Additional context
Thank you for your bug report – we love squashing them!

Interesting that the runName is not picked up, thanks for reporting this!
moving this to our main repo as we centralize bug reports there


html5lib: full (or at least common) selector support

only type (i.e. tag name) selectors are supported right now, e.g. "div"
most useful ones, IMHO

ids
classes
descendants
children
attributes

basically everything in simple selectors, other than psuedos:
http://www.w3.org/TR/css3-selectors/#simple-selectors
and the combinators, except sibling:
http://www.w3.org/TR/css3-selectors/#combinators

This issue has been moved to dart-lang/html#35.


Fix calibration perfect model, add observations tutorial, fix ClimaLandSimulations

This PR (will) contains three commits:

edits to perfect model calibration
fix ClimaLandSimulations run_fluxnet()
add latent heat flux calibration tutorial

https://clima.github.io/ClimaLand.jl/previews/PR816/

@kmdeck @Sbozzolo
I set nensemble and niterations to 3 each, and it gives nice results still, but documentation build went from 18 min to 37 min... is it okay?

Can't we just have a small use case and paste the picture of the completed calibration, as you suggested?

Can't we just have a small use case and paste the picture of the completed calibration, as you suggested?

it is already the small use case. otherwise it takes 2 hours.
it will be faster with the suggestions - almost done!


Added schema for MCA LSA service station specialist finder

https://trello.com/c/um5lWlkX/2586-mca-specialist-finder-request-life-saving-appliances

Jeez I hate reviewing these

Those dashes had to be changes everywhere :)


Automate Windows setup

New Batch scripts, Makefile step to install all languages + dependencies on a Windows machine

Testing:

Works on my machine 🤷🏻

I'd prefer we instead add this to our internal DevEx repo as it's more tooling for maintainers than it is for users. No need to bloat the examples repo with something only we'd use (since most users would already have this setup or are typically only programming in one environment/language.)


Option to keep selection on multiDrag and clone behaviour

Is there a way to keep the selection of the items you have multiDragged?

@JPernerstorfer This doesn't make sense to me. If you have multidrag enabled, all selected elements will move together.
Could you be more specific?

Hi @waynevanson , yeah sorry, it was not really clear. What I meant was to keep the selection after droping the elements somewhere. When you drop the items, you loose the selection of the multidragged items.
E.g.: I have a list of dishes, and a list for each day of a week, I would like and option to select 3 dishes, drag them to monday, and still have them selected on the original list, so I would be able to drag them again to another list/day.
Right now I am using a hack using what is mentioned here. But it is not the exact behaviour I need.
I was thinking something along the lines of "option.keepSelectionAfterDrop".
Also, this way, if you accidentally drop the items in an empty space (like missing the list or letting go of the mouse button to soon) you do not lose the selection.
Best Regards

Hi,
I would like to have the opposite behaviour. For me, after dropping the multidrag items, they remain selected. I would like to get them all deselected after dropping. :)
Also it would be great to have manual control over select and deselect events. There seems to be a gap somewhere in the code...
I can catch the select event and execute a deselect(item), but only once. When the code hits the onSelect(item) method on the sortable for a second time, Sortable.utils.deselect(item) misbehaves. The sortable options.isMultidrag option is set to undefined on the second pass, on the first pass this option is 'true' (as it should be).
This works only on the first call:
Sortable.utils.deselect($(event.target).closest('.list-group-item')[0])
there seem to be quite a few issues reports related to select/deselect events.


Usernames and passwords randomly generated and stored in vault / or use AD where applicable

Potentially https://github.com/sethvargo/vault-secrets-gen

Learn feature and usage
Implement feature
Tests

Test access with and without

Documentation

Vault supports from box several API, including consul, nomad and databases.
All credential management will be delegated to vault.


Avoid reflection on java calls in s-unquote

I have no idea why I didn't get notified of this pull request. I'm sorry I didn't notice it until now. Thank you for contributing and I see no reason not to get this in. I'll try and merge it in this week.


Missing rosdep key for ament_cmake_clang_tidy

Hello.
I had a package relying on ament_cmake_clang_tidy but that key is not present in rosdistro. Looking around to see which packages use it, the only recent example was here.
The error:
Cannot locate rosdep definition for [ament_cmake_clang_tidy]
ROS Version: Galactic
Is there a reason this is not present as a rosdep key  in this repo or other reason it cannot be found?

If I understand the index correctly, the rosdep package would be ament_lint, as per https://github.com/ros/rosdistro/blob/master/galactic/distribution.yaml#L156
Note that it pulls in multiple other packages in tow, but I think it's a [possibly intentional] limitation of rosdep when dealing with multiple packages originating from the same repository.

ament_cmake_clang_tidy is a ROS package, not a rosdep key. It looks like it is present in ROS Galactic. https://github.com/ament/ament_lint/tree/galactic/ament_cmake_clang_tidy
The error message you posted looks like rosdep was unable to find the package ament_cmake_clang_tidy, and then looked it up as a rosdep key and was unable to find it that way too. There's not enough context for me to reproduce it. Mind providing a minimal reproducible example?
Does the issue persist after rosdep update?


Body of PUT request is encoded into UTF-8

Hello!
There is an issue with add_data method now - the requests module breaks when you try to send request body with cyrillics or other non-ASCII symbols without encoding it preliminarily. So I fixed it.
BTW, thanks for the module!

Awesome! thanks so much! I haven't had much time to work on this so this is very very helpful 🙏


Campaigns endpoint returns all campaigns twice

/campaigns returns sad double data
[
 {
  title: "Comeback Clothes",
  nid: 362,
  is_staff_pick: true
 },
 {
  title: "Comeback Clothes",
  nid: 362,
  is_staff_pick: true
 },
....

Duplicate of #4110. The endpoint currently is using the apache solr drupal view, about to look into #4110 now


Fixed bug in test_smc.py

Python 2.7 and 3.5 require numpy slice bounds to have integer type, but np.ceil outputs a float; this PR adds an explicit cast.

Coverage remained the same at 67.713% when pulling b7441705e2f0be87b4aa57953ea11b340f7b3214 on ihincks:fix-test_smc-slice-bug into ddac8218f5367f303a7e98e191fbf4b7731e2af5 on QInfer:master.

I suspect this was probably a recent upgrade on NumPy's end, they've issued DeprecationWarnings on non-integer slicing for a while, but those warnings tend to get lost in the noise. Thanks for taking point on this!


Microsoft.Azure.CognitiveServices.Vision.Face. Select version 2.7.0-preview.1 doesn't work with current quickstart code

"Microsoft.Azure.CognitiveServices.Vision.Face. Select version 2.7.0-preview.1" doesn't work with the quick start code that's displayed.
Unrecognized types:
DetectedFace.FaceAttributes.QualityForRecognition
QualityForRecognition

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

ID: 24f2953b-2900-322d-dbfd-a53846e75940
Version Independent ID: 25af9875-58cf-69d5-478d-08fb888d12dc
Content: Quickstart: Use the Face service - Azure Cognitive Services
Content Source: articles/cognitive-services/Computer-vision/quickstarts-sdk/identity-client-library.md
Service: cognitive-services
Sub-service: face-api
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@scottlwalker
Thanks for your feedback! We will investigate and update as appropriate.

@scottlwalker
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.
@PatrickFarley
Could you Please add comments on this and update as appropriate.

@scottlwalker I've updated the instructions to use the latest SDK version by default.

#please-close


Set up basic auth for the staging envrionment

Description:
We need to set up basic authentication for our staging environment. This is to ensure that only authorized users can access the staging environment, where we plan to present some of our newest features to elect users. Implementing basic authentication will help secure the environment and prevent unauthorized access or exploitation.
Objective:

Secure the staging environment with basic authentication.
Ensure that only authorized users with the correct credentials can access the staging environment.

Requirements:

Basic Authentication Setup:

Implement basic auth on the staging server.
Create a set of credentials (username and password) to be shared with authorized users.

Documentation:

Provide documentation on how to access the staging environment with the basic auth credentials.
Include steps for updating or changing the credentials as needed.

Testing:

Verify that the basic auth is correctly implemented and that unauthorized users cannot access the staging environment.
Ensure that the staging environment remains functional and accessible to those with the correct credentials.

To be checked on Stage env


refactor(stopFlag): completly remove stopFlag for context

context is Go package used mainly to signal cancelation.
stopFlag package inside of gemini was used the same way.
It was just adding overhead and maintainability issues in the long
run, as it was only a wrapper for context.Context and all at once
calling the cancel funcs at the end of the program. This removal
makes code easier to follow for everybody working on it. context
package is well known in Go community and can serve many purposes
beyond just canceling the background workers (jobs in gemini
terminology).
Hard kill in gemini was used to signal the immadiet stoppage
for gemini, without pritting the results, and soft on the contrary
could be triggers in a few ways:

gemini validation failed
gemini mutation failed
SIG(INT,TERM) was sent

There is no reason to have HARD kill in the application, if we need hard
kill, SIGKILL can be sent and everything would be stopped, there will be
no need to the the cleanup as kernel will ensure that happens.
context.Context works as a soft kill, and if something happens bad
in gemini (validation fails or mutation fail) globalStatus.HasError()
would stop all other goroutines from continuing if failFast CLI flag
is passed, so the set of softKill is not needed.

@CodeLieutenant what's the haste to get this merged ?


vue-components: create TextInput component

TextInput component :sparkles:

[x] initial styling
[x] width prop
[x] responsiveness
[x] label
[x] placeholder
[x] disabled
[x] states (hover, active/focus, ...)
[x] add warning/error icons

Phabricator task: https://phabricator.wikimedia.org/T263734
Figma spec: https://www.figma.com/file/gi3JYwSUQYhKS0mHwK25cQwi/WiKit?node-id=3975%3A1
Most of the code here was written in pair programming by @guergana and @jakobw.

The latest update is only a rebase on top of the latest (small) changes that happened in #197 .


Writing dice tests.

Goals

[ ] Write tests for each context.
[ ] Parse integers out so that the rolls can be asserted to be within range and an Int.

Closing because of upstream :rage: delete


Cashbook (signed in) FAQ

The 'Get help' tab doesn't take the user directly to the page to contact
us anymore, instead an FAQ is presented to the user with useful/more
appropriate actions whenever possible.
The user can still reach the 'Contact us' (a.k.a cashbook_submit_ticket)
page from the FAQ page.
⚠ NOTE: This PR is currently based on govuk-design-system branch/PR.
Ticket: https://dsdmoj.atlassian.net/browse/MTP-1833

I've rebased and made those tweaks in this commit so it should be hopefully not too annoying to re-review ☺


Split Qry into Qry and Expr

While Qry still provides a monadic interface, most of the actual calculation happens in the interpreter for Expr, and any past-horizon exceptions come from the evaluation of an Expr. Unlike Qry, Expr uses PHOAS, which means that we can actually Show expressions, despite them containing functions. This means that we can include the offending expression in the exception.
This is also a strict generalization: previously, an entire Qry was always evaluated against a single era; with the split, every Expr is evaluated against a single era, but every Expr embedded in a Qry can be evaluated against a different era.
Closes #2463.

Marking this as a draft request, not because it's not ready, but because by rights it should have been opened against #2446 . This is open against master instead so that it's easier for the wallet folks to cherry-pick it on top of their checkout of ouroboros-consensus.

Closes https://github.com/input-output-hk/ouroboros-network/issues/2463 .


Remote repo

Add ability to use remote repos.
ftp for the prototype, then add nexus and maven.

will add artifactory

https://github.com/comtihon/coon/commit/cb5a8cae1d12474a5b905fc7b84c44ec89545e70


Add test to dhcp_collector and refactor

This refactors dchp collector to be easier to test:

Wrap constructor that can pass interface
Change signature of metrics generator to not return prometheus.Metric
Add tests for dhcp_collector

Signed-off-by: Giri Kuncoro<EMAIL_ADDRESS>Co-authored-by: William Albertus Dembo<EMAIL_ADDRESS>

address #2


pthread constants and functions for VxWorks

Changes made are supporting changes for https://github.com/iddm/thread-priority/pull/45

Changes name of sched_param in VxWorks so that it matches unix platforms
Add pthread constants and functions required for thread-priority.

Requesting for stable nomination.
r? @tgross35
Thank you,
B I Mohammed Abbas

Thanks for the PR, could you provide links to headers or manpages? Or at least copy+paste the relevant bits of the headers you are testing against to the PR description.

Do you have links to headers / docs to check against?
Also FYI the commit summary only says "VxWorks"

Do you have links to headers / docs to check against?
Also FYI the commit summary only says "VxWorks"

I am using original sources to check these against. Unfortunately, I can't share them.

Alright, they match linux so I guess that is straightforward enough. It would be good if we could find somebody who can crosscheck these in the future if sources aren't available.
Thanks for updating the commit message.


EXCEPTION: No value accessor for ''

Hi, I am close-minded and I am not interested in Typescript. I've been trying to get started on this simple example.
(function(app) {
  app.comp = ng.core.Component({
    selector: 'div[ng]',
    template: '
Hello {{obj.a}}
' +
      '
<input type="text" [(ngModel)]="obj.a"/>
',
  }).Class({
    constructor: function() {
      this.obj = {a: "World"};
    }
  });
})(window.app || (window.app={}));

It works with 2.0.0-beta.0. When I run it against beta.1 to beta.11, the code breaks and I get EXCEPTION: No value accessor for '' in [obj.a in class0@0:48].
If I want to use simple map values instead of the prototyped objects that Typescript enforces, how do I do that with the latest versions?

Please only file documentation issues here. Yours is a support question which is better placed on StackOverflow.  That's why I'm closing it.
But ... because I was curious ... I modified the QuickStart plunker to align with your code and this works:
(function(app) {
  app.AppComponent =  ng.core.Component({
      selector: 'my-app',
      template: '
Hello {{obj.a}}
'+
      '
<input type="text" [(ngModel)]="obj.a"/>
'
    })
    .Class({
      constructor: function() {
         this.obj = {a: "World"};
      }
    });
})(window.app || (window.app = {}));

The only differences I see are (a) the selector and (b) the name of the component.
(a) Why did you write: 'div[ng]?  You didn't show your index.html so it's difficult to see how this does or does not match the application start tag there.
(b) You don't show your main.ts so can't see how you bootstrap. I suspect that you aren't importing and bootstrapping with app.comp.  I reproduce your error when I misalign the component name.

Hi, thank you for your response. I admit perhaps it should have been on Stack Overflow instead, but I suspected it to be an error because it gets produced by changing code version. I followed your change and I'm still producing the same error. Below is the entire code. Changing the scripts to 2.0.0-beta.0 would fix the error.
<!doctype html>

<my-app></my-app>

```javascript


Post-OBR TODO and issues

[x] Have a build file in the root of the project (or just move dist contents to the root)
[x] Consolidate the lib folders to the root
[ ] Remove all the complicated "ant setup" stuff
[ ] Get rid of "publish" (if possible)

I'm going to close this given that we'll probably move to Gradle at some point.


Deployment Fails

Deployment fails with an object not defined error when using the deploy button method.
So I tried using the command line method for deploying works.

Hi @kentstroker
I did a minor edit to the readme which was using an https link for the deploy button service which seems to not have SSL anymore. I switched to http and was able to deploy using the button.
If you have some time, do you mind verifying it works for you as well? And if not, can you share some more details?
Thanks

I will close the issue assuming everything is now fine. Feel free to submit a new one if that's not the case. thanks


Move::Move() is not properly initializing bool trigger[MaxTriggerKeys];

Just a silly thing...They are assuming MaxTriggerKeys == 6
Actual code:
   trigger[0] = false;
   trigger[1] = false;
   trigger[2] = false;
   trigger[3] = false;
   trigger[4] = false;
   trigger[5] = false;

Proposed code:
for (S32 i = 0; i< MaxTriggerKeys; i++)
trigger[i] = false;

Seems totally reasonable. I'm sure a PR for that would get accepted in a flash :).

Looks like the fix for this was merged in last year. Should this be closed?

Ah! Indeed it should, thanks!


How to obtain the processed data like S3DIS?

Hi @guochengqian
If I use my own dataset, how to obtain the processed data s3dis_val_area5_0.040.pkl like S3DIS dataset? Could you provide the code? Thanks.

https://github.com/guochengqian/PointNeXt/issues/22


Upgrade rules_foreign_cc

This upgrades rules_foreign_cc to a version including
https://github.com/bazelbuild/rules_foreign_cc/pull/938, which fixes
a build failure when the requested Apple SDK from --xcode_version does
not match the system's default Xcode's SDKs.
Example output:
rules_foreign_cc: Build failed!
rules_foreign_cc: Keeping temp build directory and dependencies directory for debug.
rules_foreign_cc: Please note that the directories inside a sandbox are still cleaned unless you specify --sandbox_debug Bazel command line flag.
rules_foreign_cc: Printing build logs:
_____ BEGIN BUILD LOGS _____
xcrun: error: SDK "macosx12.1" cannot be located
xcrun: error: SDK "macosx12.1" cannot be located
xcrun: error: unable to lookup item 'Path' in SDK 'macosx12.1'
+ XCODE_VERSION_OVERRIDE=<IP_ADDRESS>C100
+ APPLE_SDK_VERSION_OVERRIDE=12.1
+ APPLE_SDK_PLATFORM=MacOSX

This fixes a hermeticity problem in the build and is a prerequisite for
upgradting the macOS RE cluster to macOS 12, which in turn is a
requirement for upgrading to Xcode 13.4.
Progress on https://github.com/envoyproxy/envoy-mobile/issues/2100

cc @lfpino @drwx

This is great - thank you. I've definitely hit this error in the past.

If the upstream PR can't be merged for any reason, we may need to revert this if it prevents us from staying up to date with the official rules repo.


adding scroll to top feature

FIXES #87
Scroll bar to top has been added

@Gopal2021025, what's the progress??

@EshanTrivedi21 I will update it

@EshanTrivedi21 I have made the required changes Kindly check

Can you please attach a screenshot? @Gopal2021025

@EshanTrivedi21 Kindly check I have attached the screenshots


Add capability of a custom free/destroy function to objects in MCB

In case of packet_t pointers, deallocation doesn't free the inner pointers allocated space, which becomes unreacheable. The solution is to allow a custom function to take care of that, in this case, packet_dealloc.

@fmontoto may you do this?

Sure, could someone  (@emilio-aburto ?) explain how to reproduce the problem or point me to the code?

The problem is exactly here, that free function may be not enough.
I've no problem adding this feature, but given that it seemed out of place in my current branch, I created a new issue.

I see, you're right. I think the function must be pass at mcb_init_buffer and store the pointer in the multiconsumer_buffer struct and use it from there, instead of passing the function each time mcb_unref is called.
No sure which one is your branch, but seems like this should have its own branch.

I was thinking instead of changing the existing "constructor" (which may
break some code elsewhere), adding some "set_custom_free" function that
change the default value (free) to another function in a given mcb.
El abr. 7, 2016 16:21, "fmontoto"<EMAIL_ADDRESS>escribió:
I see, you're right. I think the function must be pass at mcb_init_buffer
and store the pointer in the multiconsumer_buffer struct and use it from
there, instead of passing the function each time mcb_unref is called.
No sure which one is your branch, but seems like this should have its own
branch.
—
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub
https://github.com/niclabs/ratadns-fievel/issues/30#issuecomment-207056004

This is not an open API, I don't see a problem changing the interface.
I prefer the constructor because it seems more simple to me, so my argument would be that including it in the constructor keeps the API smaller and simpler.
But you'll implement it, it's your call.

I think using the constructor way is better with maintainability in mind. With the parameter in the constructor is harder to forget to free something, as you are force to choose a destroy function.
So, @fmontoto won.


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2423

Questions ?
Please refer to the Backport tool documentation


Missing steps for configuring multiple Reply URLs

This guide does not address nor define the domain and URLs section.

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

ID: 2136c8d4-927f-1253-9c97-5cbb13f95edf
Version Independent ID: dbe2d463-6770-d93f-e365-f0da154494fa
Content: How to configure federated single sign-on for an Azure AD Gallery application
Content Source: articles/active-directory/application-config-sso-how-to-configure-federated-sso-gallery.md
Service: active-directory
GitHub Login: @ajamess
Microsoft Alias: asteen

@grizzljt Thanks for your feedback! We will investigate and update as appropriate.

Thank you!  If it helps, I am trying to setup Azure AD as the IdP for SharePoint on prem and I am struggling through setting up multiple web apps.

@grizzljt Please refer two below two documents and see if any of them helps with your requirement.
Using Azure AD for SharePoint Server Authentication
Enable remote access to SharePoint with Azure AD Application Proxy

Thank you for the links!  I truly appreciate your quick response.  My problem is with trying to enable multiple reply URLs.
I cannot seem to apply multiple reply URLs or wildcards through the GUI or through Set-AzureADServicePrincipal. When I try to add additional Reply URLs via Set-AzureADServicePrincipal I get the error

Set-AzureADServicePrincipal : Error occurred while executing SetServicePrincipal Code: Request_BadRequest Message: One or more properties on the service principal does not match the application object.

I actually followed the guide in the first link to set up SharePoint with Azure AD.  Unfortunately the guide only covers one web application, not multiples.
Azure AD Application Proxy would only expose SharePoint to the web via a Proxy, which does not give me the benefits of using Azure AD as the IdP for SharePoint.

@grizzljt you need to update the ReplyURLs in the App object first and then in the Service principal. This is because AAD check if the ReplyURL you're modifying in the SP matches any of the ReplyURLs in the app object. You can use this PS if you have access to the app object:
$sp = Get-AzureADServicePrincipal -SearchString "<Exact App name>"
$app = Get-AzureADApplication -SearchString "<Exact app name>"
Set-AzureADApplication -ObjectId $app.ObjectId -ReplyUrls "<ReplyURL>"
Set-AzureADServicePrincipal -ObjectId $sp.ObjectId -ReplyUrls "<ReplyURL>"

@grizzljt We will now proceed to close this thread. If there are further questions regarding this matter, please open a new issue and we will gladly continue the discussion.

Thank you @luleonpla! This worked but the article was not updated to reflect the direction of this thread thus it doesn't make sense to close the issue @MohitGargMSFT before updating the article.

Hi @grizzljt - We'll get the article updated - it takes a bit for it to go through our staging system, so it might be a few days :). @luleonpla as FYI


use correct manifest list digest for worker build

Signed-off-by: Robert Cerven<EMAIL_ADDRESS>

Updated release notes draft:

use correct manifest list digest for worker build image


Release Google.Cloud.Gaming.V1 version 1.2.0

Changes in this release:

Commit 1577824: feat: support version reporting API
Commit ac367e2: feat: Regenerate all APIs to support self-signed JWTs

Created release for Google.Cloud.Gaming.V1-1.2.0


How to run pantalimon always

There is no docs on how to setup pantaliamon to run via systemd, it is not possible to leave the console for linux users, please help

Untested, but systemd has a pretty standard template so this should work:
Create the systemd config file (and set permissions):
sudo touch /etc/systemd/system/pantalimon-daemon.service
sudo chmod 664 /etc/systemd/system/pantalimon-daemon.service

Content of the file:
[Unit]
Description=Pantalimon systemd service

[Service]
ExecStart=<point to your pantalimon binary>

[Install]
WantedBy=multi-user.target

Run this to reload your daemon config (so systemctl start pantalimon-daemon, stop, restart, etc work):
sudo systemctl daemon-reload

Run this to make it start automatically on boot:
sudo systemctl enable pantalimon-daemon

Run this to see the service logs:
journalctl -u f pantalimon-daemon

Untested, but systemd has a pretty standard template so this should work:
Create the systemd config file (and set permissions):
sudo touch /etc/systemd/system/pantalimon-daemon.service
sudo chmod 664 /etc/systemd/system/pantalimon-daemon.service

Content of the file:
[Unit]
Description=Pantalimon systemd service

[Service]
ExecStart=<point to your pantalimon binary>

[Install]
WantedBy=multi-user.target

Run this to reload your daemon config (so systemctl start pantalimon-daemon, stop, restart, etc work):
sudo systemctl daemon-reload

Run this to make it start automatically on boot:
sudo systemctl enable pantalimon-daemon

Run this to see the service logs:
journalctl -u f pantalimon-daemon

I followed the setup but i am getting this error "gi.repository.GLib.GError: g-io-error-quark: Cannot autolaunch D-Bus without X11 $DISPLAY (0)"

I've been meaning to ask if anyone was able to get pantalaimon working with a service file. I've tried several configurations and have only managed to get it to work for about 3 hours max before it crashes. I've tried running it with pm2 as well, with the same result. Here's some logs from when I last tried it.
Oct 17 11:11:08 myvps pantalaimon[3550]: [2022-10-17 16:11:08.526138] INFO: pantalaimon: Trying to decrypt sync
Oct 17 11:11:08 myvps pantalaimon[3550]: [2022-10-17 16:11:08.526349] INFO: pantalaimon: Decrypting sync
Oct 17 11:11:18 myvps pantalaimon[3550]: [2022-10-17 16:11:18.568507] INFO: pantalaimon: Trying to decrypt sync
Oct 17 11:11:18 myvps pantalaimon[3550]: [2022-10-17 16:11:18.568886] INFO: pantalaimon: Decrypting sync
Oct 17 11:11:20 myvps pantalaimon[3550]: [2022-10-17 16:11:20.342103] INFO: pantalaimon: Trying to decrypt sync
Oct 17 11:11:20 myvps pantalaimon[3550]: [2022-10-17 16:11:20.342278] INFO: pantalaimon: Decrypting sync
Oct 17 11:11:22 myvps systemd[1833]: Stopping Pantalaimon...
Oct 17 11:11:22 myvps systemd[1833]: pantalaimon.service: Main process exited, code=exited, status=1/FAILURE
Oct 17 11:11:22 myvps systemd[1833]: pantalaimon.service: Failed with result 'exit-code'.
Oct 17 11:11:22 myvps systemd[1833]: Stopped Pantalaimon.

and
## Oct 17 16:00:38 myvps pantalaimon\[60660\]: \[2022-10-17 21:00:38.976889\] INFO: pantalaimon: Decrypting sync
Oct 17 16:00:39 myvps pantalaimon\[60660\]: \[2022-10-17 21:00:39.068572\] INFO: pantalaimon: Trying to decrypt sync
Oct 17 16:00:39 myvps pantalaimon\[60660\]: \[2022-10-17 21:00:39.068748\] INFO: pantalaimon: Decrypting sync
Oct 17 16:00:42 myvps pantalaimon\[60660\]: \[2022-10-17 21:00:42.372092\] INFO: pantalaimon: Trying to decrypt sync
Oct 17 16:00:42 myvps pantalaimon\[60660\]: \[2022-10-17 21:00:42.372265\] INFO: pantalaimon: Decrypting sync
Oct 17 16:00:46 myvps systemd\[57693\]: Stopped target Main User Target.
-- Subject: A stop job for unit UNIT has finished
-- Defined-By: systemd
-- Support: http://www.ubuntu.com/support

## -- A stop job for unit UNIT has finished.

## -- The job identifier is 56 and the job result is done.
Oct 17 16:00:46 myvps systemd\[57693\]: Stopping D-Bus User Message Bus...
-- Subject: A stop job for unit UNIT has begun execution
-- Defined-By: systemd
-- Support: http://www.ubuntu.com/support

## -- A stop job for unit UNIT has begun execution.

## -- The job identifier is 52.
Oct 17 16:00:46 myvps systemd\[57693\]: Stopping Pantalaimon...
-- Subject: A stop job for unit UNIT has begun execution
-- Defined-By: systemd
-- Support: http://www.ubuntu.com/support

## -- A stop job for unit UNIT has begun execution.

## -- The job identifier is 57.
Oct 17 16:00:46 myvps systemd\[57693\]: dbus.service: Succeeded.
-- Subject: Unit succeeded
-- Defined-By: systemd
-- Support: http://www.ubuntu.com/support

## -- The unit UNIT has successfully entered the 'dead' state.
Oct 17 16:00:46 myvps systemd\[57693\]: Stopped D-Bus User Message Bus.
-- Subject: A stop job for unit UNIT has finished
-- Defined-By: systemd
-- Support: http://www.ubuntu.com/support

## -- A stop job for unit UNIT has finished.

## -- The job identifier is 52 and the job result is done.
Oct 17 16:00:46 myvps systemd\[57693\]: pantalaimon.service: Main process exited, code=exited, status=1/FAILURE
-- Subject: Unit process exited
-- Defined-By: systemd
-- Support: http://www.ubuntu.com/support

## -- An ExecStart= process belonging to unit UNIT has exited.

## -- The process' exit code is 'exited' and its exit status is 1.
Oct 17 16:00:46 myvps systemd\[57693\]: pantalaimon.service: Failed with result 'exit-code'.
-- Subject: Unit failed
-- Defined-By: systemd
-- Support: http://www.ubuntu.com/support

-- The unit UNIT has entered the 'failed' state with result 'exit-code'.

I've ended up running it as a command inside of tmux after each reboot and it runs endlessly without a problem. I don't know what the difference is, but if anyone is running it successfully inside a systemd .service file I'd be interested in seeing the contents of it. Or if anyone knows of any tricks to get it to run properly, I'd be interested in hearing them.  Thanks.

I was able to get this to work, but I manually edited pantalaimon/ui.py to set UI_ENABLED = False
It would be really great if there was something like a command-line switch to allow panatlaimon to run in a daemon mode that doesn't init all the stuff in ui.py

Posting my setup here in the odd case it might help someone. I've got a setup that starts a Docker container via systemd:
Description=Matrix TLS Proxy
Requires=docker.service
After=docker.service
DefaultDependencies=no

[Service]
Type=simple
Environment="HOME=/root"
ExecStartPre=-/usr/bin/env sh -c '/usr/bin/env docker kill pantalaimon 2>/dev/null'
ExecStartPre=-/usr/bin/env sh -c '/usr/bin/env docker rm pantalaimon 2>/dev/null'

ExecStart=/usr/bin/env docker run --rm --name pantalaimon \
                        --log-driver=none \
                        -p <IP_ADDRESS>:18080:8008 \
                        --mount type=bind,src=/pantalaimon,dst=/data \
                        pantalaimon:latest

ExecStop=-/usr/bin/env sh -c '/usr/bin/env docker kill pantalaimon 2>/dev/null'
ExecStop=-/usr/bin/env sh -c '/usr/bin/env docker rm pantalaimon 2>/dev/null'
Restart=always
RestartSec=10
SyslogIdentifier=pantalaimon

[Install]
WantedBy=multi-user.target

The service mounts the /pantalaimon path in the container. /pantalaimon contains the pantalaimon.conf file, which in my case looks like this:
[Default]
LogLevel = Debug
SSL = True

[local-matrix]
Homeserver = <https://matrix.example.com>
ListenAddress = <IP_ADDRESS>
ListenPort = 8008
SSL = False
UseKeyring = False
IgnoreVerification = True

However, this doesn't result in a working panctl. Logs are only available via journalctl, though this can be set to output to the docker container logs by removing the SyslogIdentifier and --log-driver=none lines.


Devices without batteries break even with monit

This is because Battery::new() returns Result<Battery, Error> but this can be fixed with returning just a Battery with maybe a value exists = false;.


Sggreen 471 UI changes

Summary
NIFI-00000
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[ ] Apache NiFi Jira issue created

Pull Request Tracking

[ ] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[ ] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[ ] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 11
[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Please check the above list of things to do before opening a pull request. The very first thing is to have a NIFI JIRA created to describe the work (bug fix, feature, etc).


Make GitHub recognize the LICENSE

GitHub should display MIT badge in top right corner if license is recognized.

Current page shows this.

Even if this PR doesn't change license text much, it is still able to kick the GitHub license match job.
As a benefit, this project could be then filtered by MIT License.

All good now.


Add init sizes

Addresses https://github.com/ItsNickBarry/hardhat-contract-sizer/issues/33

@ItsNickBarry addressed most comments, see unresolved thread above

@ItsNickBarry comments addresses

@RyanRHall Made a few more changes in #37.  Take a look if you like.

Published in v2.10.0.


Deploy action doesn't set VPC connector

To connect to MongoDB securely, the Cloud Run services need to use a VPC connector and be routed out of a gateway to get a static outbound IP. It seems like the VPC connector config is lost every time a new revision is deployed by GitHub Actions. Is that something I can configure in the workflow file?
As a workaround, I can go into the GCP console for Cloud Run, choose the just-deployed revision, edit to use VPC connector for all outbound connections, then move the revision URL to the new revision. (I guess revision URL is part of the manifest for the FE?)

No longer using MongoDB, so this is obsolete.


[feat] Support set execute type to pydolphinscheduler

Up to now, we can only submit workflow with parallel mode. this patch give users ability specific execute type to workflow

Codecov Report

Merging #12871 (9a3ec58) into dev (229c554) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev   #12871      +/-   ##
============================================
- Coverage     39.25%   39.24%   -0.01%
+ Complexity     4231     4227       -4
============================================
  Files          1051     1051
  Lines         39849    39818      -31
  Branches       4581     4567      -14
============================================
- Hits          15642    15627      -15
+ Misses        22433    22422      -11
+ Partials       1774     1769       -5

Impacted Files
Coverage Δ

...che/dolphinscheduler/api/python/PythonGateway.java
18.01% <0.00%> (-0.09%)
:arrow_down:

...org/apache/dolphinscheduler/remote/utils/Host.java
42.55% <0.00%> (-2.13%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...hinscheduler/plugin/alert/script/ScriptSender.java
70.58% <0.00%> (-1.64%)
:arrow_down:

...cheduler/plugin/alert/dingtalk/DingTalkSender.java
34.13% <0.00%> (-0.78%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
51.38% <0.00%> (-0.70%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.86% <0.00%> (-0.55%)
:arrow_down:

...inscheduler/plugin/registry/etcd/EtcdRegistry.java
50.69% <0.00%> (-0.35%)
:arrow_down:

.../dolphinscheduler/plugin/task/datax/DataxTask.java
36.62% <0.00%> (-0.26%)
:arrow_down:

.../dolphinscheduler/plugin/task/spark/SparkTask.java
72.56% <0.00%> (-0.25%)
:arrow_down:

... and 19 more

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


Reference data set & Generator

I have a issue regarding the default.md & numbering convention of the events themselves.
For example TCS7 , TCS6 & TCS5 all occur in time before TCS4 , TCS3 & TCS1.
Another example is EDef2 occurs in time before EDef 1.
It appears the correct way to read the links, is from the bottom of the page up to the top. (start at the last event and work backwards).
This creates a bit of conflict with the numbering if the flow of time is taken into consideration.
(e.g EDef2 occurs before EDef1).
In addition , it make it difficult to scale the link model.
Maybe all that is needed to resolve the conflict is to clarify how to read the link flows with respect to time.

@d-stahl-ericsson
I have a proposal to remedy the time versus the event numerical value issue.
What if we removed the numerical value itself altogether (e.g TCS7 becomes just TCS) and simply state in the diagram that each event in the diagram represents an individual event in time.
This would allow us to scale and would result in little work to resolve the issue in terms of editing etc,,.

There's a point to being able to uniquely identify each event, though, and that point would be lost in that case. But of course you could use something other than an integer, which is indicative of an ordered series, and use a randomized scheme, e.g. "TCS HX" and "TCS SB", but then chances are people would scratch their heads and try to figure out what those acronyms mean...

The event names are also part of the actual reference data set (in the zip file) and there they are specified as customData.name. I assume we have them there merely for documentation purposes? And since json lacks a good way to write comments we added it as customData instead? I see a risk that people will start using customData.name in that way which I assume we didn't intend?
Additionaly - some (a lot of?) links are missing between events in the actual data set.
I propose that we

renumber the events in the data set to follow the intended order
update the gliffy accordingly
break out those names to C-style comments in the json file and then make use of a JSON Minifier to easily remove them before actually using the json file
add the missing links in between the events in the actual data set

sounds like a plan .......
who will do what ?
If you don't mind I could renumber the events and update the gliffy (I need the practice here) ?

Please do! :)

renumber the events in the data set to follow the intended order

Ok!

update the gliffy accordingly

Ok!

break out those names to C-style comments in the json file and then make use of a JSON Minifier to easily remove them before actually using the json file

I have a problem with that. If you run a development project with centralized control of how the data is used, e.g. in a build script or something, that's fine. But that's not what we're doing - we're providing a data set for unknown use cases by unknown users. Then if we say "Oh, it's not really JSON, by the way, it's just sort of like JSON-ish" that's not good. To clarify that adding names in customData isn't encouraged as general practice, we can actually add a disclaimer as a property: "The use of customData properties in this data set is solely for event identification and clarity in the context of usage as a reference data set. It is not encouraged as general practice."

add the missing links in between the events in the actual data set

Are we talking about event types which are omitted from the data set because they are not mandatory, but would be nice to include, or are we talking about individual links which have been overlooked?

I have submitted a Pull request for your review.

This was PR #140, correct? That was closed - any updates on this? Should we keep this issue?

Doing a bit of housekeeping here. I would like to close this issue unless anyone is active working on it. If we want to address this again, we can always create a new issue. Thoughts?


Handle Cognito Sync Count Exception

Addresses unhandled mismatch sync count exception per issue #430

@soapyigu can we get this looked at as well?

In my opinion, this pull request is unnecessary as currently this kind of error falls into data storage exception, which is the base class that data conflict exception extends from.

@soapyigu Per the issue #430, the problem is that the consumer of this exception is specifically expecting a data conflict exception, not a data storage exception.  See https://github.com/aws/aws-sdk-net/blob/master/sdk/src/Services/CognitoSync/Custom/SyncManager/Dataset.cs#L619

Thanks for the pull request. Version <IP_ADDRESS> of CognitoSync contains this change.


Adds a couple of integration tests.

Before adding a few more, I'd like to get some feedback on these to get a grasp of what you're after. If you listed the kind of tests that you want that'd be great.
Also, I just went for the fastest I could write them; as I mentioned in #77 I had some written in my application and I'm using xunit there. If you'd like to keep nunit or have the tests in a different project structure let me know. I made it its own project as these take quite a bit to run but it's your call.
Regards

BTW, I don't know why this happens but I'm getting an error when trying to run your tests:
System.TypeInitializationException : The type initializer for 'RocksDbSharp.Native' threw an exception.
  ----> NativeImport.NativeLoadException : Unable to locate rocksdb native library, either install it, or use RocksDbNative nuget package

Have you ran into this?

BTW, I don't know why this happens but I'm getting an error when trying to run your tests:

System.TypeInitializationException : The type initializer for 'RocksDbSharp.Native' threw an exception.

  ----> NativeImport.NativeLoadException : Unable to locate rocksdb native library, either install it, or use RocksDbNative nuget package

Have you ran into this?

If you run on Linux (maybe in macOS too), you need to install some dependencies for RocksDb support.
You can check my CI script to install these :
https://github.com/LGouellec/kafka-streams-dotnet/blob/develop/.circleci/config.yml

I'm starting your PR.
First thanks for your contribution.
Some feedbacks :

I prefer keeping NUnit for maintenance even if Your work with Xunit is great. If you can switch to NUnit with same structure will be wonderful.

Great idea to create a dedicated project for integration. All right ! As I triggered this project for PR, and merge to master for example.

So just switch to NUnit and you can continue integration test with join, aggragate, some state full tests.
Perfect !! 😀🚀


replaced nprogress package with accessible-nprogress

Related Issue
Resolves #50
Description
Replaced outdated nprogress module with newer accessible-nprogress module.
No changes made to the UI.
npm package: https://www.npmjs.com/package/accessible-nprogress
v1 seems to work fine, and here is the info on the changes in each version: https://github.com/nmackey/accessible-nprogress/releases
There is a v2 of the accessible-nprogress module, however I was running into this error:

Impacted Areas in Application
_app.tsx or the loading bar on the main page.
Steps to Test or Reproduce

No changes to setting up project, no changes in UI.
Submit a search query to view the accessible-nprogress loading bar

Related PRs

PR Checklist:

[x] My code follows the style guidelines of this project and I have double check my own code
[x] I have made corresponding changes to the documentation, if any
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have run npm run test and be sure all test pass
[x] I have run npm run build:dev and be sure no error blovk the build
[x] I have test locally that everything run as expected
[x] I have properly fill in the PR template
[x] I have ask for reviews

Thank you for working on this @karlrez !! The new package seems good :)

@karlrez thank you very much for this PR!! :clap: :clap:


Downgrading typedoc version so that it plays well with the @sastan/typedoc-plugin-pages package

Correct the documentation package errors like this
https://app.circleci.com/pipelines/github/raiden-network/light-client/9988/workflows/f94ef6f8-18eb-48cb-92c8-ef3838386361/jobs/63812
Thank you for submitting this pull request :)
Fixes #
Short description
Definition of Done

[ ] Steps to manually test the change have been documented
[ ] Acceptance criteria are met
[ ] Change has been manually tested by the reviewer (dApp)

Steps to manually test the change (dApp)

@andrevmatos you might have to force merge this as well as I dont have the priviledge to force merge without everything passing.


Fix commonmark syntax

@XhmikosR
related

https://github.com/twbs/bootstrap/pull/29750
https://github.com/gohugoio/hugo/issues/6553

What does this fix exactly? The previous solution worked fine too.

Actually, this leads to no change of your site's view
It is just about styling of code

The previous solution worked fine too.

Of course, it is ok as it is.
If so, please close
I would like to share another solution to you and close gohugoio/hugo#6553

I have already removed the newlines to work around the issue. I'm not against this change, but I'll wait for someone else to chime in since it's just a cosmetic change :)

@MartijnCuppens what's your take on this? There's basically no functional change apart from the indentation in the source and output files.


Request for an addition to the documentation about the required minimum version of PostgreSQL

I am currently running PostgreSQL 9.1, and I was not sure if PostgREST requires the JSON datatype as a minimum.
Anyway, I am going to upgrade in any event, but I thought it would be a welcome addtion to have a "Minimum Requirements" page on the wiki.
Thanks for what looks like a wonderful product.

Good idea, I'll update the readme and the new docs.


A variety of small bugfixes

Fixing issues #608, #585, #551, #545, #541, #515, #513, and a number of other issues that don't (I think) have issue trackers since I noticed them as I was fixing known issues.

Merge conflict needs resolving, apparently


sap.m.ListSelection: partially hidden list item doesn't get selected on first click in Chrome

OpenUI5 version: latest
Browser/version (+device/version): Chrome (latest)
Any other tested browsers/devices(OK/FAIL): Firefox, Edge OK
URL (minimal example if possible): https://openui5.hana.ondemand.com/#/sample/sap.m.sample.ListSelection/preview
User/password (if required and possible - do not post any confidential information here):
Steps to reproduce the problem:

Open up the ListSelection sample and resize the browser height as necessary so that the bottom list item is only partially visible on screen (bottom part of it cut off)
Make sure the list item does not have focus
Click on the bottom list item near the bottom of the item (not near the top of it)
Note that the list scrolls up to show the entire bottom item, but the item does not get selected

What is the expected result? The bottom list item should be selected when clicked on, similar to how it behaves on other browsers
What happens instead? The bottom list item is not selected, and needs an additional click in order to select it.
Any other information? (attach screenshot if possible) From our analysis:

Mouse down on an element inside partially obscured item inside a scrolling container.
ItemNavigation set a focus on the item.
Chrome auto-scrolls that item element into view.
The simulated tap event somehow gets canceled. Then ListItemBase's ontap handler is never called, which does not carry out the selection toggle routine.

We suspect one of the two things:
a) Like moving your cursor out of the target element area after mouse down and then release a mouse, the mouse up is not called. In this case, it scrolled up, then mouse up is not called because the target element is no longer in the same area from the mouse down. This could be what cancelled the simulated tap event.
b) As part of a "move" touch, if you move like dragging an element, then there should be no tap event.  Did jquery mistakenly use this browser auto scroll as a move to cancel the simulated tap event?

When the item is not fully into view it is just focused. Once the item is into view it can be selected.
To me it seems correctly, as to not select item that is not clear.
The case will be checked by my colleagues via #1880360586


[GODSEND Agenda] v.1.2.0

Changes / Comments

repeating weapons, powers, armor
add/remove some secondary attributes
update looks
rolltemplate shows degree of success

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[x] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Update README.md

Coverage remained the same at 87.113% when pulling 6790c90addf31b2894b3945a0e590744ec90d7f6 on kulakowka:patch-2 into 6dc74912e2a6b2ac1c65326830e7528afd047474 on gregnb:master.


image domains not loaded from next.config.js

🐛 Bug Report
I created the nextjs project with npx create-next-app. I added Jest and next-page-tester.
When trying to render a Page that uses next/image I get following Error:
Error: Uncaught [Error: Invalid src prop (https://picsum.photos/id/238/200/300) on `next/image`, hostname "picsum.photos" is not configured under images in your `next.config.js`

I don't get this error when using yarn dev or yarn build && yarn start
My next.config.js looks like this
module.exports = {
  reactStrictMode: true,
  images: {
    domains: ['picsum.photos']
  }
}

Reproduction
Create a next.js project. Add Jest + next-page-tester. Use the next/image component with an image from an external Domain and add this domain to next.config.js.
Expected behavior
next-page-tester should load the next.config.js and accept the image domains.
Jest patch
Have you patched Jest as described here?

[ ] yes
[x] no

Run npx envinfo --preset jest
Paste the results here:
  System:
    OS: macOS 10.15.4
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
  Binaries:
    Node: 16.4.0 - ~/.nvm/versions/node/v16.4.0/bin/node
    Yarn: 1.22.10 - ~/.nvm/versions/node/v14.17.0/bin/yarn
    npm: 7.18.1 - ~/.nvm/versions/node/v16.4.0/bin/npm
  npmPackages:
    jest: ^27.0.5 => 27.0.5

Hi @derweili, thanks for reporting.
It definitely seems a missing feature probably introduced with v11. We should reproduce the issue in tests and then see how to extend the existing logic to cover this external domain scenario.
I can't tell when I'll be able to work on this, though. As usual we're open to contributions and glad to provide reviews/guidance.

I've reproduced the issue in #236.


Fix ssl check in image store browser

Description
This PR fixes the ssl in image store browser.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@blueorangutan ui

@shwstppr a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/8430 (QA-JID-250)

@wido , can you test this?

@wido , can you test this?

Yes. Will do so tomorrow (04-04-2024)


Test ./320_claim_3_test.sh is still flaky

e.g. https://circleci.com/gh/weaveworks/weave/3031
previous attempted "fix" is #1268.

Note test 160 was renamed to 320

Latest failure: https://circleci.com/gh/weaveworks/weave/5745
(this seems different to the earlier report)

Going to close this as we haven't seen the original failure in four weeks.


istio.io/common is now istio.io/pkg

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [geeknoid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@geeknoid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-framework-local-presubmit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test integ-framework-local-presubmit-tests-master

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.

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

Test name
Commit
Details
Rerun command

prow/integ-framework-local-presubmit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test integ-framework-local-presubmit-tests-master

prow/istio-presubmit.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test istio-presubmit-master

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.

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

Test name
Commit
Details
Rerun command

prow/integ-framework-local-presubmit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test integ-framework-local-presubmit-tests-master

prow/istio-presubmit.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test istio-presubmit-master

prow/istio-unit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test istio-unit-tests-master

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.

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

Test name
Commit
Details
Rerun command

prow/integ-framework-local-presubmit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test integ-framework-local-presubmit-tests-master

prow/istio-presubmit.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test istio-presubmit-master

prow/istio-unit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test istio-unit-tests-master

prow/integ-galley-local-presubmit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test integ-galley-local-presubmit-tests-master

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.

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

Test name
Commit
Details
Rerun command

prow/integ-framework-local-presubmit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test integ-framework-local-presubmit-tests-master

prow/istio-presubmit.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test istio-presubmit-master

prow/istio-unit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test istio-unit-tests-master

prow/integ-galley-local-presubmit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test integ-galley-local-presubmit-tests-master

prow/integ-pilot-local-presubmit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test integ-pilot-local-presubmit-tests-master

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.

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

Test name
Commit
Details
Rerun command

prow/integ-framework-local-presubmit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test integ-framework-local-presubmit-tests-master

prow/istio-presubmit.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test istio-presubmit-master

prow/istio-unit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test istio-unit-tests-master

prow/integ-galley-local-presubmit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test integ-galley-local-presubmit-tests-master

prow/integ-pilot-local-presubmit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test integ-pilot-local-presubmit-tests-master

prow/integ-security-local-presubmit-tests.sh
17e610a9f72aa20f4ca409c81492b721021dce6d
link
/test integ-security-local-presubmit-tests-master

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.

@geeknoid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
d289765d90c63b526d3b232bb0723cc8df5f72a9
link
/test istio-unit-tests-master

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.

@geeknoid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
3ae7d514de66b9b772ec7f99fd795cf5180692ac
link
/test istio-unit-tests-master

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.

@geeknoid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e1557ca773de77d40453c58b5c773a55e194a4a0
link
/test istio-unit-tests-master

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.

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e1557ca773de77d40453c58b5c773a55e194a4a0
link
/test istio-unit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
e1557ca773de77d40453c58b5c773a55e194a4a0
link
/test integ-pilot-k8s-presubmit-tests-master

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.

@geeknoid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
a08ee7d28b147de79e78c817207b445d2a9b88dc
link
/test e2e-mixer-no_auth-master

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.

/test e2e-mixer-no_auth-master

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

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
a08ee7d28b147de79e78c817207b445d2a9b88dc
link
/test e2e-mixer-no_auth-master

prow/integ-security-k8s-presubmit-tests.sh
a08ee7d28b147de79e78c817207b445d2a9b88dc
link
/test integ-security-k8s-presubmit-tests-master

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.


Inconsistency around use of Client Secret

This section says:

Under Manage, select Certificates & secrets > Client secrets > New client secret

And here it says:

Replace Enter_the_Client_Secret_Here with the Client secret that was created and recorded in an earlier step

However, the appsettings snippet the previous instruction refers to does not contain Enter_the_Client_Secret_Here so it's impossible to follow the instruction.

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

ID: 8e3d9337-8e8d-8cd0-3298-06e61ceab21d
Version Independent ID: d979e41b-017e-abcf-fd5e-0fc1629320bc
Content: Quickstart: Sign in users and call the Microsoft Graph API from an ASP.NET Core web app - Microsoft Entra
Content Source: articles/active-directory/develop/quickstart-web-app-aspnet-core-sign-in.md
Service: active-directory
Sub-service: develop
GitHub Login: @cilwerner
Microsoft Alias: cwerner

@Neutrino-Sunset
Thanks for your feedback! We will investigate and update as appropriate.

@cilwerner
Can you please check and add your comments on this doc update request as applicable.

@Neutrino-Sunset
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

Hi @Neutrino-Sunset, thanks for pointing this out! This quickstart will be rewritten in the coming couple of weeks to align with new code samples and new tutorial series. The client secret value in the .json file is addressed there and can help you get the app working while we work on the quickstart.

As we are revamping the quickstart, and our tutorial is the source of correct information here, I will close this issue. If there is a problem with the tutorial series, please raise a new issue.
#please-close


Can this be done using CPC? Printing from any application to a GCP-printer.

I have a GCP-connected HP printer. Printing from https://www.google.com/cloudprint using Chrome and uploading PDFs to this printer works fine. Now I would like to print from any application to this printer. CPC seems to offer that functionality but I don't know how to set it up. This is on OS X.
I have successfully compiled the code and then tried running gcp-connector-util
and gcp-cups-connector according to (the very brief) instructions in the wiki and that also seemed to work as expected but now what? How do I add this printer to CUPS? Do I need to be in the same network as the printer when configuring this?

Since you're on OSX, I'd just use this app:
https://itunes.apple.com/us/app/cloud-printer/id463448876?mt=12
I use it personally and it works pretty well.
This connector really isn't for what you're trying to do.
Thanks.
On Wed, Mar 14, 2018 at 9:04 AM, l-g<EMAIL_ADDRESS>wrote:

I have a GCP-connected HP printer. Printing from https://www.google.com/
cloudprint using Chrome and uploading PDFs to this printer works fine.
Now I would like to print from any application to this printer. CPC seems
to offer that functionality but I don't know how to set it up. This is on
OS X.
I have successfully compiled the code and then tried running
gcp-connector-util
and gcp-cups-connector according to (the very brief) instructions in the
wiki and that also seemed to work as expected but now what? How do I add
this printer to CUPS? Do I need to be in the same network as the printer
when configuring this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/cloud-print-connector/issues/402, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ALu0KbpyqgFXQFji26bQP91mMEW85gLuks5teT-ggaJpZM4Sqt70
.

--
ANDREW BALINGIT
nerd.
T<PHONE_NUMBER>  |  M<PHONE_NUMBER>  | go/allquestionsanswered
5808 W. Sunset Blvd | Los Angeles CA 90028

Thanks for you reply.
I also need a Linux solution and it would be nice if I could use the same software on both Linux and OS X.
Exactly what is the purpose with this connector? Can it be used with some "workarounds" for my purpose?

The connector is of no use for you. It just makes printers available through CUPS available locally and/or in the cloud as google cloud print enabled printer. What you're looking for is more or less a printer driver which prints to google cloud