Invalid JS in example code

The example JS has invalid object notation:
      var ruleResult = {
  	IsValid = false;
       Message = '';
       Type = 'error';
      };

Should be:
      var ruleResult = {
  	IsValid: false,
       Message: '',
       Type: 'error'
      };

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

ID: 2430af49-5aa5-f1d5-2df7-af99e529d40e
Version Independent ID: 87058b43-3478-ebd1-a32d-d601e4858ac0
Content: Set up booking rules in Dynamics 365 Field Service
Content Source: ce/field-service/set-up-booking-rules.md
Service: dynamics-365-customerservice
GitHub Login: @FieldServiceDave
Microsoft Alias: daclar

Thanks for the feedback! That JS has been updated :)


Could we get a new release?

After several PR's approved yesterday, will there be a new release created soon?

Just tagged!


Allow configuring listener addr on agent (reduce security exposure)

The agent currently binds <IP_ADDRESS>:$PORT
When running in 'onebox' mode, where hub and agent runs on the same machine, this expose an unnecessary port outside. This can be blocked by fw rules, but it would be good enhancement to allow configuring the addr using a parameter.
This allows onebox setup to listen only on localhost. (<IP_ADDRESS>:45876)
It's fairly minor change, I can open PR for that if accepted

PORT accepts a port or an address:port, so you should be able to do PORT=<IP_ADDRESS>:45876. Let me know if that doesn't work for you.
I know it's a bit confusing. Maybe we should add a separate ADDRESS env var.

That would do, but I think it really better to change this to ADDRESS. Make much more sense


Overlapping two reg arrays

Hi,
I'm trying to instantiate two or more reg arrays with strides that should not overlap each other in the following way:
`
reg {
regwidth =32;
// reg content
} FIRST__REG_ARRAY[2] @0x0 += 0x8;
reg {
regwidth =32;
// reg content
} SECOND_REG_ARRAY[2] @0x4 +=0x8 ;
`
Which means, that the first reg of the first array should be located at 0x0, the first reg of the second array at 0x4 and thus on, since each one of them is exactly 32 bits . But, I get an error of overlapping. Does it mean that this cannot be done or am I doing something wrong here?
Thanks

The compiler does not support interleaving register arrays. See a similar question #8.


Update apache-client, kinesis to 2.20.144

About this PR
📦 Updates

software.amazon.awssdk:apache-client
software.amazon.awssdk:kinesis

from 2.20.118 to 2.20.144
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #664.


Could you provide examples on the docs?

Hi,
Thank you very much for fitdecode! Really nice!
I wonder if you could provide some examples to make it easy to understand how to use it. It took me a while to understand a bit how it works - which I do now - to have typical things people might want to do, like having a personal archival system for their workouts.
But one thing I still don't understand is how to use the processors. Can you give an example on how to use them, like for example changing the distance from m to km, or the speed from m/s to km/h?
Best,
Francis

Best examples you can find remain the processor classes from the processors.py module. The docstring of DataProcessorBase explains how it is invoked by FitReader. So if you need more details than the ones already provided, you probably want to check out FitReader's code.
However do note that fitdecode features are closely related to the FIT file format itself. Therefore there is a point in your journey where you will probably want to dig more in format's official documentation if you go that far.

Hey oculos, maybe if you wanna have a little introduction into the FIT file format, you may wanna check my github repo for a FIT project I'm working one:
https://github.com/inonoob/Coxswain2Fit
I try to explain how the Garmin FIT file is build

Examples would be lovely. If I figure out more how this library works I'll try to post some.
I've gone through the FIT protocol and still a little overwhelmed.
with fitdecode.FitReader(src_file) as fit:
    for frame in fit:
        if isinstance(frame, fitdecode.FitDataMessage):
            if (frame.name == "session"):
                 # How would I hex dump the entire session data message?
                 # How would I hex dump an individual field in this message?

Hey,
have you download the Garmin SDK ? if you go into the java folder and have java installed you can decode the FIT file which results in a csv file. You can do the changes and re-encode it to a FIT file.
pi@raspberrypi:~/Prog/java $  java -jar FitCSVTool.jar -h
FIT CSV Tool - Protocol 2.0 Profile 21.40 Release Usage: java -jar FitCSVTool.jar <options> <file>       -b <FIT FILE> <CSV FILE>  FIT binary to CSV.       -c <CSV FILE> <FIT FILE>  CSV to FIT binary.
If you wanna hexdump a FIT file in LINUX:
hexdump -C xxxxx.fit
This helped me to more understand the FIT format


Bumped version to 1.1.0-dev, updated docs, added bits for conda builds

Coverage decreased (-0.006%) to 95.11% when pulling 26ac21ce56a152cc7db864fed3ff3cfbcb0b3616 on adding_conda_build into 05fe8eda90f7db63efdb141700fed9e8e26fca34 on master.

Coverage decreased (-0.06%) to 95.056% when pulling db3cd413237ed0c830a85def0c8b1593686f3b03 on adding_conda_build into 05fe8eda90f7db63efdb141700fed9e8e26fca34 on master.

Coverage decreased (-0.006%) to 95.11% when pulling 016b142d181604ec58f8cff32479e66d85d314eb on adding_conda_build into 05fe8eda90f7db63efdb141700fed9e8e26fca34 on master.


Get Started - New File not working

Issue Type: Bug
When trying on Get Started - New File tab create file, after type name file and enter, nothings is done.
The shortcut ctr+n wokring normaly.
VS Code version: Code 1.63.2 (899d46d82c4c95423fb7e10e68eba52050e30ba3, 2021-12-15T09:40:02.816Z)
OS version: Windows_NT x64 10.0.18363
Restricted Mode: No

Are you referring to this picker?

Yes, for me on this version not working.

Could you include a recording of what is happening? The expectation is that you'd click "Text File" in that picker then get an empty buffer as usual.
\gifPlease

Dam... I try do record, and was sure that when have input I should type file name and extension no just push enter, the enter creating file. Also working, just wrong intuition here for me. Issue can be close or the feature here can be as enhancement to offer in input create file name and extension to change this in next steps on tab or when saving tab/file maybe.


common statistics season display

in the output xls file, It displays some year number (e.g. 2006,2007) instead of "summer", "winter"...

Can you tell me which season option you tried? I can't reproduce the error

Sorry for the late reply. I can't reproduce the error either and I can't remember which data file generated this issue. Never mind, I will keep you posted if this occurs again.


feat(api): Adds Magdeck driver

overview
Add driver for for MagDeck. Closes #1809
changelog
Adds the following driver methods:
connect(port)
disconnect()
is_connected()
home()
probe_plate()
move(distance_mm)
get_device_info()
enter_programming_mode()

And properties:
plate_height
mag_position

review requests
Please review if the functionality of the driver is as expected.
The hardware's current response for GET_CURRENT_POSITION is Z<distance> but this driver assumes Z:<distance> (note the colon). Requested a change of device response on opentrons-modules PR#15

Also, when you make revisions, please change the scope string in the commit message from drivers to api

Made changes

🐲


New version

Advantage 360 Pro PR template
What's changed:
Why has this change been implemented:
What (if any) actions must a user take after this change:

Please dont open PRs into the main branch for personal keymap changes


Return err when search chart failed

None

In testing other package managers, I can't find any that return an error if no matching package is found. Is there a reason to change this to an error?

@technosophos Hi, thank for your review. I just want to return error if regexp.Compile() is failed.
I think we should return error when 'helm search' meet errror, and just return 'No results found' when 'helm search' is nil. Do you think so?

@technosophos Sorry, I have understanded it. I close this pr.

I agree.
Get Outlook for Androidhttps://aka.ms/ghei36

From: FengyunPan<EMAIL_ADDRESS>Sent: Tuesday, May 9, 2017 8:31:26 AM
To: kubernetes/helm
Cc: Matt Butcher; Mention
Subject: Re: [kubernetes/helm] Return err when search chart failed (#2402)
@technosophoshttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftechnosophos&data=02|01|Matt.Butcher%40microsoft.com|d35a0b3fceca49b857c408d496d75a6b|72f988bf86f141af91ab2d7cd011db47|1|0|636299299064045122&sdata=jkJyiESxV6WNRQt5egOm79MaEz1LbWvaf4mR8SQMIwo%3D&reserved=0 Sorry, I have understanded it. I close this pr.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkubernetes%2Fhelm%2Fpull%2F2402%23issuecomment-300148314&data=02|01|Matt.Butcher%40microsoft.com|d35a0b3fceca49b857c408d496d75a6b|72f988bf86f141af91ab2d7cd011db47|1|0|636299299064045122&sdata=A5TcPxh3BPET52iPQ%2BulVUUkhyhRy8a7e11ktL5x%2FmI%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAFcaYtWSM0bJzd9McwhH1iNiBef2n_vks5r4FyegaJpZM4NRsxu&data=02|01|Matt.Butcher%40microsoft.com|d35a0b3fceca49b857c408d496d75a6b|72f988bf86f141af91ab2d7cd011db47|1|0|636299299064045122&sdata=4%2FVCoyRUtuErH7bGfQ5u7AYZ9wzhxoewV3d1IcgtQhs%3D&reserved=0.


Implement os command injection test

Option to test for command injection and reverse shell, pwn.

Added command injection test in https://github.com/hansmach1ne/lfimap/commit/02708ecfc58221da6d76e606e7fcbbd4b3e18530


open map in new tab

should we open the "Daten-Karte" on the Mapathon page in a new Tab? As there is no "back" button on the maps the only way to get back to the mapathon page is via the browser back function.

We could also do it with an iFrame, so the users could still see the navgiation

For a quick fix I changed the button behavior to open a new Tab


 See https://golang.org/issue/49065.
FAIL
FAIL	runtime/pprof	5.806s

Marking as release-blocker for Go 1.18 because this test is new (as of CL 324129).
(CC @prattmic @rhysh)

The number of samples in the serial portion (cpuHog1) are spot-on: 10ms per sample, 100ms wall clock, 10 samples. The number of samples in the parallel portion (cpuHog3) are very close to the goal: 10ms per sample, 100ms wall clock, GOMAXPROCS=4 (this builder runs on a m6g.xlarge, 4 cores and no hyperthreading), 41 samples vs an expected 40.
The failure is in the comparison of the serial portion. The profile reports 100ms, but rusage(2) reports 184ms. So this looks like another source of "extra/background/non-application CPU time that the runtime expends". (Earlier flakes were from GC.)
Of the 59 total samples, taking out the 10+41 that are in the CPU hog functions the test expects, the remaining 8 are all in findrunnable. I'm surprised to see any there, and I'm not sure what to make of it.
Of those, five point at the PC right after the epollwait syscall: https://github.com/golang/go/blob/766f89b5c625f8c57492cf5645576d9e6f450cc2/src/runtime/sys_linux_arm64.s#L731
Two are at atomics: https://github.com/golang/go/blob/766f89b5c625f8c57492cf5645576d9e6f450cc2/src/runtime/proc.go#L5749 https://github.com/golang/go/blob/766f89b5c625f8c57492cf5645576d9e6f450cc2/src/runtime/proc.go#L5818
One is at a function call boundary, either before or after calling nanotime1: https://github.com/golang/go/blob/766f89b5c625f8c57492cf5645576d9e6f450cc2/src/runtime/time_nofake.go#L19

Was the change above intended to fix the test, or just improve debuggability? This is marked as a release blocker.

Just improve debuggability. Fixing this still needs more investigation, which the CL helps with.

I ran this test for several hours on a linux-arm64-aws gomote and was unable to reproduce any failures.
I think that adding a temporary skip would be OK given that as @rhysh's mentions, this appears to be unexpected background CPU, rather than an issue with profiling itself. That said, I don't have any particular reason to believe this issue would be limited to arm64, so I'm not sure what constraints to put on the skip.
@rhysh right now the test compares time in runtime/pprof.cpuHog1 samples vs rusage. What would you think about comparing the total sample time (i.e., any samples) vs rusage. It seems that should still cover the core of this test, which is whether we receive enough samples to cover all CPU usage and I believe should solve these kinds of issues with background runtime CPU.

Any updates on this, @prattmic? Checking in as it's a release-blocker for Go 1.18.


output different with IMac

Hi, First thanks for the wonderful project, I really have a great interest on this project.
When I check the result of render with my colleague, example project seems quite different with mine.

The upper one is iMac (Retina 5K, 27 inch, 2020). there are no error log in console.
Below is M1 mini Mac.
If I can get any log about this I can get a hint but there is any log for this. :)
Platform:

Device: Desktop
OS: MAC
GPU: Radeon
Browser: Chrome

Hello! This looks related to #62 which also shows artifacts on Mac with a Radeon graphics card. From this comment it looks like it could be a browser-related issue. Do other examples demonstrate the same incorrectness? Does the demo work in Safari?
Unfortunately device-specific issues like this are difficult to understand without access to the hardware and generally it's not possible to support workarounds for every scenario. Depending on whether it works in Safari it might be best to submit an issue to Apple or Chrome.


⚠ PassThePopcorn has degraded performance

In f059e24, PassThePopcorn ($PTP) experienced degraded performance:

HTTP code: 502
Response time: 5527 ms

Resolved: PassThePopcorn performance has improved in b6cb9fd after 7 minutes.


The cpp-unit-test and xctest plugins should have consistent names and ids

These plugins both basically play the same role, which is to add support for a particular kind of native test framework and apply a convention that the project has a unit test suite that uses the framework. Their class names and ids should reflect this and do so consistently.

We should resolve this before we get too deep in the documentation.

Here is the summary of our discussion during planning. Given we are doing a big push on documentation, it's a good idea to review the name for the plugins we have. Both test plugins defer by a little and should be made more consistent. Both plugins are a convention plugin for adding one test component to the project.
The cpp-unit-test adds support for C++ language using any executable-based testing framework. In fact, we can write tests using homegrown assertions, Google Test, CppUnit, Catch, etc. It's simply a dependency of the test executable.
The xctest adds support specifically for Swift language using XCTest testing framework. The plugins respect build difference between Linux and macOS. It's also an executable-based testing framework and the plugins correctly parse the outputs to generate HTML reports. It can also do test filtering.
The solution path so far is to use convention in both plugin class name given how they configure things. The extension name - unitTest for cpp-unit-test plugin and xctest for xctest plugin - are a bit inconsistent and we don't have a good answer for them. The plugin ids are also inconsistent and we don't have a good answer for them as well.

Software model plugin ids used the following convention:

Just the name of the testing framework for plugins creating a convention ready to be used. For example, cunit and google-test.
The name of the testing framework followed by -test-suite for base-like plugins. For example, cunit-test-suite and google-test-test-suite.

The XCTest plugin correctly follows the previous naming convention. However, cpp-unit-test special. It doesn't have any real test framework. It's a generic or multi-test framework plugin.

The configuration extension should avoid containing the implementation detail and focus on an agnostic name. Using test would align well with Java-land.
"What's the contract of the user when applying the test plugin?" is an important question to seek an answer with regards to using the right plugin id name. It should be obvious happen to the project when the plugin is applied.
The plugin ids should follow a convention. Something like xctest-unit-test or xctest-test-suite or just xctest. We should avoid including the language used for implementing the tests. The generic C++ test plugin is a bit special. The cpp part of cpp-unit-test should be removed. Using test-application-unit-test is a bit confusing and leak some of the implementation details. It's basically a generic unit test plugin that happens to create an executable that is used to run the tests. Some discovery to solve this issue is "how would we name JVM-based testing framework plugins if we were to split them from the java plugin?" Some example are junit-platform-unit-test, junit-legacy-unit-test, and spock-unit-test.
Scope creep of this inconsistency resolve around language mixing, multiple test components, composing support for new testing frameworks. All those questions should be resolved first through a sample to focus on the important changes and avoid building a flexible solution that ends up being over-engineered.


Nuisance parameters to rescale xsecs

Introduce a function that adds nuisance parameters that rescale the cross sections of entire LHE / H5 files.

See also #64

Implemented in 933bdc51a28d0f22f9e50ab08b78b7bfcdfdec9e


Idr solver

This PR adds the IDR(s) solver.
The solver works for general matrices, it takes factory parameters kappa (for a "maintaining the convergence" strategy, default 0.7) and subspace_dim (the s in the name, determines how many intermediate vectors are computed in each iteration, default is 2)

[x] Write documentation
[x] Optimize generation of random matrix on GPUs as at the moment, the generate step takes about an order of magnitude more time then the apply step

The checks on this will still fail as long as hiprand is not installed on the CI system. I also still need to figure out what to do for hip on NVIDIA hardware.


reactiveQuery result

I am experimenting with asteroid.
My html looks like :
<!DOCTYPE html>
<html>
<head>
	<title>test</title>

<script src="bower_components/ddp.js/src/ddp.js"></script>
<script src="bower_components/q/q.js"></script>
<script src="bower_components/asteroid/dist/asteroid.browser.js"></script>
</head>
<body>

<script type="text/javascript">
	var ceres = new Asteroid("localhost:3000");
    var tasks = ceres.getCollection("movies");
    var _t=tasks._set
    console.dir(tasks._set._items) // line 15
    console.dir(tasks.reactiveQuery({}).result) //line 16

</script>

</body>
</html>

The chrome console gives me this output

Why can't I access the result of the Query ?

reactiveQuery is asynchronous. It makes total sense the nothing is printed out.
I can use timeout .
setTimeout(function() {
	console.dir(movies.reactiveQuery({}).result)
}, 100);

Is there some event i can use that can notify when result are available ?


envoy: Bump cilium envoy to latest version v1.21.3

The images digest is coming from below build.
https://github.com/cilium/proxy/runs/6816960166?check_suite_focus=true.
Release note: https://www.envoyproxy.io/docs/envoy/v1.21.3/version_history/current
Signed-off-by: Tam Mach<EMAIL_ADDRESS>

/test

All the required CI are passed, most of reviews are in. Marking this ready to merge.


Feature Request: ReaScript function that returns dBs going in and out of focused Track/ItemFX

If possible a function that can be called from a ReaScript that returns the dB-values going in and out of a focused TrackFX/ItemFX.

As in measuring the audio before and after a given plugin? The only way to do that would be via inserting a plugin before/after the plugin to spy on. A ReaScript can do that by inserting two instances of JSFX.

Ok, thanks =)


Support more algorithms on KittenBlue.JWK.from_compact/1

KittenBlue.JWK.from_compact/1 currently supports only limited algorithms.
https://github.com/ritou/elixir-kitten-blue/blob/ad0dd94975b50de5a19ffc8419d49a74496f819d/lib/kitten_blue/jwk.ex#L198
It would be nice to support more algorithms like EC256, Ed25519, etc.

It will be changed to...

from_compact

key is map: from_map
alg in octet: from_oct
others: from_pem

to_compact

use_map in opts: to_map
alg in octet: to_oct
others: to_pem


⚠ Samsung has degraded performance

In 986a1d3, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2680 ms

Resolved: Samsung performance has improved in 46b10d9.


"Temporal Table Considerations and Limitations" missing information

The page states:

Regular queries only affect data in the current table. To query data in the history table, you must use temporal queries. These are discussed later in this document in the section entitled Querying Temporal Data.

But there is no "Querying Temporal Data" section.

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

ID: 7967cb92-4205-31da-e7a3-6ee76091c9fb
Version Independent ID: d947240d-76dd-036f-b1b2-814e85ba3e9f
Content: Temporal Table Considerations and Limitations - SQL Server
Content Source: docs/relational-databases/tables/temporal-table-considerations-and-limitations.md
Product: sql
Technology: table-view-index
GitHub Login: @rwestMSFT
Microsoft Alias: randolphwest

@IanKemp -- Ian, thank you again for your feedback.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@rwestMSFT -- Randolph, please look into this issue.

@IanKemp The article should link here. Thanks for the feedback, and I'll get that fixed.


Null model on controller route

Hi, i have generated model, view And controller by usino wpmcv command Line tool.
In The controller using $this->Model->find() method i getto fatal error: Call To A member function find() on null in ...
Where is The fault?

public function show() {

		$users_model = mvc_model('Users');

		$object = $users_model->find_by_ID($this->params['id_user']);

        if (!empty($object)) {
            $this->set('data', $object);
        }else{
			$this->set('data', $this->params);
		}
    }

This works! But $this->model is always null... when  can(/do) i use $this->model->{method} ?

Model name must be in singular form. Your model name and file must be "User" and "user.php".


WIP: BasePath

Include Basepath

What is the benefit of the basepath property comparing to this simple concatenation ?
const basepath = "demo"
// ...
<Route path={`${basepath}/topic`} render={Topic} />

I'm having a separate hyperapp app as a widget: https://github.com/kidandcat/newsuncork/blob/master/src/apps/product-creator.jsx
And I'm instancing it here: https://github.com/kidandcat/newsuncork/blob/master/src/components/CreateProduct.jsx
I'll investigate why today, but when it loads, the location of CreateProduct.jsx's router is "/" (but the URL path is /admin/product/create)

@kidandcat We should add features to the Router which are impossible to add from the outside. Couldn't you solve this problem in userland?

I couldn't do it in the userland, but I managed to do it in a much more cleaner way:
https://github.com/hyperapp/router/pull/72
@jorgebucaran


Use Element.closest() instead of findParent

The findParent util must be replaced with the native API.
https://developer.mozilla.org/en-US/docs/Web/API/Element/closest

Done in https://github.com/Tahul/vue-catch-hrefs/commit/da4cb78d6dbabc1ada1a098b8e93f4f5b22344ef


DT-2365 Improve error for recall/transfer-in where cell is full

LGTM, prob could of got away with just a unit test in the service for this.

I would have liked to but the service has 32 dependencies injected into it and so the unit test would have been more complicated than the integration test!  (I'm hoping that service this will be broken up when it is used in anger on Receive a Prisoner).


lib_windfile.cpp: Rewrite trim() and locate2()

I have rewritten these two functions to use C++ strings and vector to make the code much simpler and clearer. There is no now arbitrary limitation of 128 columns in locate2().

I merged this in 07a40f6790e2dc06328e9586800a9aca22eae121 .  I added some exception handling, as the examples I was running were failing due to invalid stoi arguments.  Essentially, it appears as if stoi returns that exception in the event that it cannot convert the input string to an integer.


Using edge midpoints as input features

Hi, not quite an issue, but thought appropriate to write here.
You mention in your paper that you ran some experiments using edge midpoints as input features.  Is the option to do this implemented somewhere in the code?  If so, how to turn it on.  Thanks!

Hi @LiZimo ,
We did not include it (tried to keep the code as simple as possible), but you can easily add it to the code. Here is the function that did it :
    def mid_point(mesh, features):
        for edge_id, edge in enumerate(mesh.edges):
            features[edge_id].extend(list(mesh.vs[edge[0]] + mesh.vs[edge[1]] / 2))

You can add that to mesh_prepare.py
Then also, you should add "mid_point" to the list of feature extractors in this line:
https://github.com/ranahanocka/MeshCNN/blob/5af7b9c39d3e871b052f9c4c4315fb03f5f92992/models/layers/mesh_prepare.py#L316
I think that should be it.


I am having problem where no table is rendered after I click on the insert table button.

I am having the problem where no table is rendered after I click on the insert table button.
I have to click on the placeholder first, after the cursor is displayed, then the insert table button can work.
https://user-images.githubusercontent.com/130023653/230266253-c45d7c07-5d54-46dc-a8d1-6500a96609f1.mov
Originally posted by @yingning-00 in https://github.com/facebook/lexical/issues/3981#issuecomment-1498447058

The problem you're having is that the editor does not have selection. You can make it work by clicking the editor first, then inserting the table, or something similar via the API. The fix (via insertNodes) is merged #4180 but will have to wait until 0.9.2.


when switching browser tab or minimizing,it will pause.

I just learned react, Where can I change it,can you help me？ @rodri042

Hey, I'm not sure if I understand your problem. Currently, minimizing the window or selecting another tab will make the emulator stop, since requestAnimationFrame won't be called anymore.
It's not possible to keep running the game because the browser prevents that kind of behaviors to optimize computer's resources, and also there's no reason to do so.
The master node (the one who uploads the rom) will always handle the timing and send the state to the slave node. If what you want is to pause the master node when the slave node is minimized, you want to block the responses of the SlaveSyncer when the window is minimized, but there's no React code involved there.

Hey, I'm not sure if I understand your problem. Currently, minimizing the window or selecting another tab will make the emulator stop, since requestAnimationFrame won't be called anymore.
It's not possible to keep running the game because the browser prevents that kind of behaviors to optimize computer's resources, and also there's no reason to do so.
The master node (the one who uploads the rom) will always handle the timing and send the state to the slave node. If what you want is to pause the master node when the slave node is minimized, you want to block the responses of the SlaveSyncer when the window is minimized, but there's no React code involved there.

but why the jsnes website can keep running the game
I tried it on https://jsnes.org

jsnes drives the emulator by audio when the tab is minimized. Though it's a
clever hack, it's not so accurate and causes several bugs (e.g. it doesn't
work with 120hz monitors) so I've removed that part.
El 29 jul. 2019 3:37 a. m., "king2015"<EMAIL_ADDRESS>escribió:
Hey, I'm not sure if I understand your problem. Currently, minimizing the
window or selecting another tab will make the emulator stop, since
requestAnimationFrame won't be called anymore.
It's not possible to keep running the game because the browser prevents
that kind of behaviors to optimize computer's resources, and also there's
no reason to do so.
The master node (the one who uploads the rom) will always handle the
timing and send the state to the slave node. If what you want is to pause
the master node when the slave node is minimized, you want to block the
responses of the SlaveSyncer
https://github.com/rodri042/nestation/blob/master/src/networking/SlaveSyncer.js#L112
when the window is minimized
https://stackoverflow.com/questions/7389328/detect-if-browser-tab-has-focus,
but there's no React code involved there.
but why the jsnes website can keep running the game
I tried it on https://jsnes.org
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rodri042/nestation/issues/1?email_source=notifications&email_token=AAMOMCBPVJS2BRHWVWWVEPDQB2F4DA5CNFSM4IHMHA6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD27XQOY#issuecomment-515864635,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMOMCHQF4ALW7AYMSYWOJDQB2F4DANCNFSM4IHMHA6A
.

ok,I know .thanks.

it doesn't work with 120hz monitors

Is there any way to solve it?


_posts/0000-01-02-rsprague81.md

created branch, created file, made commit and opened pull request

fixed file name

fixed file name


Bug: When developing with <React.StrictMode> in React 18, will get an error because the same auth code is used twice

With Strict Mode starting in React 18, whenever a component mounts in development, React will simulate immediately unmounting and remounting the component.
As a result, the same auth code will be sent to the auth provider twice, and the second time will get an error.
As React document mentions, it requires components to be resilient to effects being mounted and destroyed multiple times.
Steps To Reproduce

root.render( <React.StrictMode>   <AuthProvider authConfig={authConfig}>         <LoginInfo />       </AuthProvider>   </React.StrictMode> );

The error will appear

The current behavior
The useEffect() in AuthProvider is fired twice, and the same auth code is sent to the auth provider twice.
The expected behavior
The same auth code should not be sent to the auth provider twice.
Btw, I am not a front-end developer, so maybe my understanding of the error is not correct.

Hi,
Thanks for reporting this.
The fact that the useEffect is triggered twice in React 18 w/strict mode is expected behaviour. As there is no support for fine grained control of where it should render twice or not.
Is this causing any issues for you, or is there only a failed network request in the background?
Because if it's only a error in dev you can safely ignore this.

Hi @soofstad , thank you for your reply.
I agree with you that it is only an error in dev so we can safely ignore this.
The motivation that I create the issue. As I understand, in the future, React would like to add a feature that allows React to add and remove sections of the UI while preserving the state. And this requires components to be resilient to effects being mounted and destroyed multiple times. (refer)
However, the component AuthProvider is not resilient to effects being mounted and destroyed multiple times. Since the AuthProvider will send two token requests to the auth provider with the same auth code, the second request will get an error response.
So, my two cents is that logic could be added to AuthProvider to prevent more token requests to be sent with the same code.
Since I am not a frontend developer and just started learning recently, I am not sure my understanding is correct. My apologies if I am wrong.

Thanks, it's a bit more nuanced that I first though.
I think you are right and on to something. I will need to read a bit more into this too see how best to handle it.
If anyone comes across this comment and have some insight I'd be happy too hear it :)
Not sure when I will get the time to fix this yet. Might take a while.


color "brightWhite" and its usage in WSL man pages

Possibly this issue is not an issue of Windows Terminal but of WSL or of one specific distribution within WSL.
Environment
Microsoft Windows [Version 10.0.18362.356]
Windows Terminal (Preview) [Version 0.4.2382.0]
WSL 1 with Ubuntu 18.04 (not sure how to find out the build number)

Steps to reproduce

start Windows Terminal
open a tab with WSL Ubuntu 18.04 (maybe also with other distributions)
choose a light theme (like One Half Light) in the profiles.json file
view a man page (e.g. man grep)
play with values for brightWhite in the choosen color schema

Expected behavior
General text should be colored in generic colors such as foreground or explicit colors like brightWhite should change their color.
Actual behavior
The WSL man page seems to use brightWhite for highlighted text in man pages. brightWhite remains white in the light themes. The remaining text has the color foreground.
grep manpage with default color schema One Half Light:

grep manpage with modified color schema One Half Light (brightWhite set to #FFFFFF):

This sounds a lot like #2638 to me

@zadjii-msft Thanks for linking. Looks similar.
brightWhite might have the same value than the background color. Does the algorithm work for equal colors?

#2638 may mitigate the issue, but it doesn't solve the underlying problem. And that is that the bold/bright attribute doesn't work correctly when the default foreground color isn't "white" (more specifically, index color 7).
In the One Half Light color scheme, the default colors are #383A42 on #FAFAFA . So selecting black on white, and then the bold attribute, like this:
echo -e "\e[30;47m NORMAL \e[1m BOLD \e[m"

should be the equivalent of selecting the default colors, and then the bold attribute, like this:
echo -e "\e[m NORMAL \e[1m BOLD \e[m"

But in the latter sequence, the "bold" text becomes bright white instead of bright black. I'm a fairly certain that is not what we want.

On further investigation, I believe the fault may be in the way the Windows Terminal applies the color scheme. If you set the palette using escape sequences, then it works correctly.
For example, this is the equivalent set of escape sequences to initialize the palette with the One Half Light color scheme:
echo -e "\e]4;0;rgb:38/3A/42\e\\"
echo -e "\e]4;1;rgb:E4/56/49\e\\"
echo -e "\e]4;2;rgb:50/A1/4F\e\\"
echo -e "\e]4;3;rgb:C1/83/01\e\\"
echo -e "\e]4;4;rgb:01/84/BC\e\\"
echo -e "\e]4;5;rgb:A6/26/A4\e\\"
echo -e "\e]4;6;rgb:09/97/B3\e\\"
echo -e "\e]4;7;rgb:FA/FA/FA\e\\"
echo -e "\e]4;8;rgb:4F/52/5D\e\\"
echo -e "\e]4;9;rgb:DF/6C/75\e\\"
echo -e "\e]4;10;rgb:98/C3/79\e\\"
echo -e "\e]4;11;rgb:E4/C0/7A\e\\"
echo -e "\e]4;12;rgb:61/AF/EF\e\\"
echo -e "\e]4;13;rgb:C5/77/DD\e\\"
echo -e "\e]4;14;rgb:56/B5/C1\e\\"
echo -e "\e]4;15;rgb:FF/FF/FF\e\\"
echo -e "\e]10;rgb:38/3A/42\e\\"
echo -e "\e]11;rgb:FA/FA/FA\e\\"

If you run man grep again after executing those sequences, you should find that it displays the "bold" text correctly.

@j4james Thanks for investigating the issues.
The bold text in the man pages is properly colored after executing those lines.
As workaround, I could copy your code into my .bashrc ... ;-) .
What exactly does \e]... and \e[... do? It is a bit hard to search for these terms.

What exactly does \e]... and \e[... do? It is a bit hard to search for these terms.

\e[ is known as CSI, the Control Sequence Introducer, and marks the start of most standard escape sequences. You can read more about them on Wikipedia.
\e] is known as OSC, for Operating System Command, and is typically used for private escape sequences that might differ from one terminal emulator to the next. The ones used here - OSC 4, 10, and 11 - are color palette commands that were originally defined by XTerm. You can read more about them here.

@zadjii-msft I'm fairly certain this isn't a duplicate, or at least not a duplicate of #2638.

I'm worried about this issue. I know that we have a duplicate somewhere, but I just can't find it. This is actually partially related to #293 (we transform 39 into 37, and technically we transform 1;39 into 97) and partially related to the fact that we don't support a separate "boldForeground" which should be used when the foreground color is bold. Where terminals usually have 19-color palettes (indexed 16 plus fg/bg/bold), we have 18 narrowed in certain instances to 16.
Ref #2661, plus the bug report I just can't find for "foreground but bold".

(or, equally likely, we transform 37 into 39, but we don't transform 97 into 1;39 or 99. There's a bit of nuance. :))

I don't think it's a narrowing bug, or anything along those lines, because why then would it work when the palette is set via escape sequences? It's the exact same colors being used, so surely they should fail in the same way if that were the case?
My theory (which I haven't had a chance to investigate) is that the Windows Terminal might be trying to handle the palette mapping exclusively on the terminal side, and not passing that info through to conhost. So conhost still thinks it's working with the default palette (or whatever settings it has), and when you've got a derived color (from the bold attribute), there's no longer a direct link back to the correct color on the Terminal side.
Btw, I can accept this might still be a dup. I just didn't want it to get lost if the dup closure was based on #2638, which definitely isn't the issue here.

handle the palette mapping exclusively

It definitely is, and it should be. #2661 is one of the things that's caused by it caring too much.
Setting the color palette actually leverages the issue in 2661 to fix this one. If you do this:
`e]4;6;rgb:FA/AF/AA`e/`e]10;rgb:FA/AF/AA`e/

(powershell syntax, sorry)
and then just use \e[1m, you actually get back \e[96m from ConPTY. Because the default matches one of the indexed colors, it gets narrowed to the index.
If we stop querying the color table in conhost when we need to figure out which index to use to set the color we think we want, we'll start passing through color indices unharmed.
You're right that the derived color is being matched against the wrong index, but really we shouldn't be using color index derivations at all. :smile:

(sorry, i realized i just said the same thing three times there at the end.)

OK, I think I get it now. So if the color was simply passed through from conhost as default + bold, without trying to map it to an index or COLORREF, then the Terminal side would actually have been able to map that to the correct palette values? In that I'm assuming #2661 might fix this.

would have been able

Indeed. :smile:

Just FYI -- confirmed that this is a case that #2661 will help with (plus/minus #293) :smile:

Once we stop color mapping, that 1 will stop coming through as 97 (circled in blue)


Docker Selfhosted Setup

I want to host an Instance of weserv/images on my internal Kubernetes Cluster.
How can I setup an instance without any domain limitations and DNS Server from device usage, for example on local device based development.
docker run -d -p 8081:80 --shm-size=1gb --name=weserv ghcr.io/weserv/images:5.x

curl -v http://localhost:8081/\?url=http://host.docker.internal:8080/api/files/D8270076110FCEAE93FAC573CFBCC031F7B6A0FD

The curl doesn`t work with error.
{"status":"error","code":404,"message":"The hostname of the origin is unresolvable (DNS) or blocked by policy."}

If i connect to docker container this curl is working.
curl http://host.docker.internal:8080/api/files/D8270076110FCEAE93FAC573CFBCC031F7B6A0FD

An other question is how to disable caching on my self hosted instance?
Thanks for your help!

By default, Google's open DNS server (<IP_ADDRESS>) is used to resolve DNS queries. However, to resolve host.docker.internal, you may need to use Docker's embedded DNS server within the nginx configuration, see:
https://github.com/weserv/images/issues/206#issuecomment-1752740666
To disable any caching, you can use the the imagesweserv-no-cache.conf config, see:
https://github.com/weserv/images/issues/354#issuecomment-1221544455
The following Dockerfile combines these approaches:
# Usage: docker build --build-arg DNS_SERVER=<IP_ADDRESS> -t weserv/images .
FROM ghcr.io/weserv/images:5.x

ARG DNS_SERVER=<IP_ADDRESS>

RUN cp ngx_conf/imagesweserv-no-cache.conf /etc/nginx/imagesweserv.conf \
    && sed -i "/resolver /s/[0-9].*/$DNS_SERVER;/" /etc/nginx/imagesweserv.conf

Thanks! 👍🏻

Just for Documentation and if someone else has the same Issue here my Dockerfile:
FROM ghcr.io/weserv/images:5.x

RUN rm -rf /etc/nginx/imagesweserv.conf \
    && cp ngx_conf/imagesweserv-no-cache.conf /etc/nginx/imagesweserv.conf

CMD DNS_SERVER=$(cat /etc/resolv.conf | grep nameserver | awk '{print $2}' | head) ;\
    sed -i "/resolver /s/[0-9].*/$DNS_SERVER;/" /etc/nginx/imagesweserv.conf; \
    nginx -g "daemon off;"


fix: increased yAdjustment for axis labels

This PR:

This PR increase yAdjustment for axis Titles, like this they stop overlapping
This PR resolves #1878

How to Test

Go to this link
Search Photo and Select (Einmalvergütung für Photovoltaikanlagen) Dataset
Select a Combo Chart type
Select the longest titles for but indicators
Publish the Chart resize your screen up to 350px
See how the Titles never overlap ✅

CC: @sosiology

I'd also consider adding a CHANGELOG entry along the lines of improving overlapping logic for combo charts 👍

I'd also consider adding a CHANGELOG entry along the lines of improving overlapping logic for combo charts 👍

Alright I was not sure if this is too minor for the CHANGELOG


Filtring with url params

The JSON API specification states this:

Note: JSON API is agnostic about the strategies supported by a server. The filter query parameter can be used as the basis for any number of filtering strategies.

Therefore, it is possible to filter results using the URL params like shown here: http://jsonapi.org/recommendations/#filtering
In #9, association filtering is used to filter the include part of the response. Is there a way to change this default behavior to filter the returned array instead of the include's content?
I know that I can use allow_filter with a block but I can't use the name of the relationship, I need to use another name.
Are you planning to implement this behavior? What are your thoughts about such behavior and where should I look to if I want to implement this?

I know that I can use allow_filter with a block but I can't use the name of the relationship, I need to use another name.

I think the best practice here is to be more explicit with the filter name. If we're hitting /positions you aren't filtering department - you're filtering department_id or department_name, etc.
That said, I believe Graphiti (the 1.0 version of Suite that is in alpha and will be beta shortly) would support either.

The problem with allow_filter approach is that I need to repeat the same code in a lot of places. How can I DRY things up? Would we be able to migrate from current version to 1.0 version?

I'm not sure what the repeated code would be?
Will be releasing a stable beta of Graphiti today, and yes I recommend upgrading if you can.

We have a big number of resources and each resource have numerous relationships. Writing allow_filter for each relationships of each resource would be cumbersome and error prone.
The code would be roughly like this:
allow_filter :relationship do |scope, value|
  # Split value if it's and array
  scope.where([relationship_id = value])
end

I'll be glad to work on a PR (maybe in graphiti?), if this behavior is desired. Where should I look into?

I'm not sure why you couldn't do allow_filter :relationship_id?
This is a first-class pattern in Graphiti:
class PositionResource < ApplicationResource
  attribute :employee_id, :integer, only: [:filterable]
end

I'm sorry, I didn't understand the first time that this behavior was already supported by using :relationship_id without defining the allow_filter block.
Thank you very much for your help.

No worries, glad that works!


Initial schema service implementation

Contains initial schema service implementation with the following features:

builds a schema service command line app that is run via schemas serve
cli app can statically pre-load schemas when given files or directories, e.g. schemas serve ./jhu/*.json

This eliminates the need to look up schemas from the internet

Implements schema $ref dereferencing, deduplication, and ordering of result schemas based on dependencies (e.g. so the common schema will appear before any schemas that depend on it)

The PASS model has not been updated yet, so it's not currently possible to see a round trip with a real Fedora yet.
Resolves #6

I'm not familiar with this language - what's the best way to test? I installed go and can see it on the command line but haven't figured out what to do with this code yet. Is it testable at this point? Some pointers in the README would be great.

@karenhanson there's not much that you can do at this point, other than see how it handles failure to communicate with a repository that doesn't yet support the data it needs.  But you're right about a README, I forgot about that!


notebooks to illustrate the difference between accumulated and averaged data

Just to share, it is up to you to merge it or not.

good idea to show this in a notebook. more text and comments would be helpful to understand.


What are gapseq cpds (example cpd90014)

Should we mark that those are no real cpds?

Add gapseq as a new database

Closed because superseded by new issue


Strip literal freshness of contextually typed literals

I thought this would be way too heavy-handed, but surprisingly the baselines don’t look that bad?
Fixes #49514

@typescript-bot user test this inline
@typescript-bot run dt
@typescript-bot perf test this faster

The puppeteer failure shows that Object.freeze is a fairly easy way to observe this change. I think the new behavior makes more sense, but it is certainly a breaking change:
const PUPPETEER_REVISIONS = Object.freeze({
  chromium: '1011831',
  firefox: 'latest',
});

let preferredRevision = PUPPETEER_REVISIONS.chromium; // Previously string, now "1011931"
preferredRevision = PUPPETEER_REVISIONS.firefox;      // Previously ok, now error

Would it be possible to get a test build with this? I'd like to test if I can get rid of basically a ton of as const assertions with this.

@typescript-bot pack this

Doesn't look like that worked @andrewbranch

@typescript-bot pack this

@andrewbranch I assume this is 4.9-bound, right?

@sandersn yep.

The puppeteer failure shows that Object.freeze is a fairly easy way to observe this change. I think the new behavior makes more sense, but it is certainly a breaking change:
const PUPPETEER_REVISIONS = Object.freeze({
  chromium: '1011831',
  firefox: 'latest',
});

let preferredRevision = PUPPETEER_REVISIONS.chromium; // Previously string, now "1011931"
preferredRevision = PUPPETEER_REVISIONS.firefox;      // Previously ok, now error

Is there a proposed workaround for this? I am having the same issue here. https://github.com/microsoft/TypeScript/issues/52125
I think this new behaviour goes against JavaScript's syntax. In the example you gave above, I think it is fine to do the re-assignment but now we have TypeScript telling us that it is an error.
In my use case, creating an array from an immutable object and running includes on that array should not throw a type error.
With this breaking change, does this mean that to get around this, we need to do [] as string[]?


Added Dutch locale

Added dutch translations

Hey, thanks for your contribution! Please, do not change pubspec.yaml and CHANGELOG.md when submitting PRs. Can you modify it please? Thanks

@tomasweigenast Yes! done :)

Nice, thank you!


Expected response code 220 but code "", with message ""

I am receiving this error since few days. It was working for an year but now it suddenly pop ups this error.

version of php / ssl?
usually it's an ssl related issue
On Tue, Jul 7, 2015 at 7:13 AM, ShrutiShah2990<EMAIL_ADDRESS>wrote:

I am receiving this error since few days. It was working for an year but
now it suddenly it pop ups this error.
—
Reply to this email directly or view it on GitHub
https://github.com/swiftmailer/swiftmailer/issues/633.

--
Xavier De Cock
GPG Fingerprint: 93CA EE3F 9F57 5BE1 AE4A  794D 3C74 CA9E E7A5 0C1B
GPG Id: 0xE7A50C1B

PHP version 5.5.9 and we are not using SSL

then you're probably expecting a connection reset, but without a stack
trace or a code example, it's quite complex to be sure.
PS : the ssl part is for the connection to the smtp server, not the
connection to the webserver
On Tue, Jul 7, 2015 at 7:44 AM, ShrutiShah2990<EMAIL_ADDRESS>wrote:

PHP version 5.5.9 and we are not using SSL
—
Reply to this email directly or view it on GitHub
https://github.com/swiftmailer/swiftmailer/issues/633#issuecomment-119079785
.

--
Xavier De Cock
GPG Fingerprint: 93CA EE3F 9F57 5BE1 AE4A  794D 3C74 CA9E E7A5 0C1B
GPG Id: 0xE7A50C1B

The same piece of code is working on other server,
In config I have set encryption to ssl, port 25.

I have the same fail, and like @xdecock my code works on dev. I use sendmail without SSL/TLS. Anyone hasn't solution for this problem ?

Most probably this issue arises when you face a connection time out. Please check if you have a proper internet and there is no connection time out.

can you connect to the :25 port via telnet?
might happen if the :25 port is filtered by your hosting provider.
On Mon, Aug 10, 2015 at 2:30 PM, Baptiste Donaux<EMAIL_ADDRESS>wrote:

I have the same fail, and like @xdecock https://github.com/xdecock my
code works on dev. I use sendmail without SSL/TLS. Anyone hasn't solution
for this problem ?
—
Reply to this email directly or view it on GitHub
https://github.com/swiftmailer/swiftmailer/issues/633#issuecomment-129427144
.

--
Xavier De Cock
GPG Fingerprint: 93CA EE3F 9F57 5BE1 AE4A  794D 3C74 CA9E E7A5 0C1B
GPG Id: 0xE7A50C1B

The hosting has specific configuration and I don't check the configuration. In my case, I think it's a configuration problem. I solved my problem using "mail" transport and not "sendmail". (s**t solution).

Same problem, does not seems related to sendmail / ssl from what i can see.
Because it can connect fine and send first command 'MAIL FROM: ...' then get no answer, at this stage the connection is already done ? Maybe some write buffer issues ?

Expected response code 220 but got code "", with message ""
.env file
MAIL_DRIVER=smtp
MAIL_HOST=smtp.gmail.com
MAIL_PORT=25
<EMAIL_ADDRESS>MAIL_PASSWORD=6868055a
MAIL_ENCRYPTION=null
mail.php
 env('MAIL_DRIVER',' smtp'),
'host' => env('MAIL_HOST', 'smtp.gmail.com'),
'port' => env('MAIL_PORT', 25),
'from' => ['address'<EMAIL_ADDRESS>, 'name' => "amjad"],

'username' => env('MAIL_USERNAME'),
'password' => env('MAIL_PASSWORD'),
'sendmail' => '/usr/sbin/sendmail -bs',
'pretend' => false,
];

mail code
  Mail::send('email.verify' , ['confirmation_code' => $confirmation_code],  function($message) use ($abc){
            $message->to(Input::get('email'), Input::get('username'))
                ->subject('Verify your email address');
        });

you were using a gmail account in sending you should change your port to 587 or 465 and your encryption to tls or ssl

@m-junaid You properly want to change your password!! As I got the non-edited version in my inbox. And you can't use encryption when using sendmail.
Please don't post Framework specific code, try to make the smallest possible piece of code that produces the problem. Any framework specific code can obscure the actual details and actually be the cause of the problem.

@m-junaid Try to run php artisan config:clear command.
It may be looking at cache.

'class' => 'yii\swiftmailer\Mailer',
'useFileTransport' => false,
'transport' => [
'class' => 'Swift_SmtpTransport',
'host' => 'mail.xxxxx.com',
'username' =><EMAIL_ADDRESS>'password' => 'xxxxx',
'port' => '26',
'encryption' => 'tls',
I am receiving a error."Expected response code 220 but got code "", with message "" ".But Gmail is working Fine for me.I am changing our webmail facing a problem.Anyone give a solution for me.thanks in advance.

'mail' => [
    'class' => 'yii\swiftmailer\Mailer',
    'viewPath' => '@frontend/mail',
    'useFileTransport' => false,//set this property to false to send

mails to real email addresses
//comment the following array to send mail using php's mail function
'transport' => [
'class' => 'Swift_SmtpTransport',
'host' => 'smtp.gmail.com',
'username' =><EMAIL_ADDRESS>'password' => '**************',
'port' => '587',
'encryption' => 'tls',
],
],
On Mon, Oct 24, 2016 at 1:09 PM, Amjad Sarwar<EMAIL_ADDRESS>wrote:

plaease move this config commen to backend or frontend

plaease move this config commen to backend or frontend

bonjour j'ai un problème avec mon envoie de mail d'après un formulaire. L'envoie de mail se fait bien, mais lorsque je reçois dans ma boite mail je ne reçois pas l'adresse de l'expéditeur entré dans le formulaire mais je reçois mon adresse même:
Exemple
from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>Parameters:
parameters:
database_host: <IP_ADDRESS>
database_port: null
database_name: ***
database_user: root
database_password: null
mailer_transport: smtp
mailer_host: smtp.gmail.com
mailer_user<EMAIL_ADDRESS>mailer_password: ***
mailer_encryption:  ssl
mailer_auth_mode: login
mailer_port: 465
secret: 1f55a0254de4a7699bfd3e5b3296665371ba2033
et ma config:
swiftmailer:
transport: "%mailer_transport%"
host:      "%mailer_host%"
username:  "%mailer_user%"
password:  "%mailer_password%"
port:      "%mailer_port%"
encryption: %mailer_encryption%
auth_mode: "%mailer_auth_mode%"
spool:     { type: memory }

@lorenzadamour their message, translated from French to English by Google Translate:

hello i have a problem with my sending mail from a form. The sending of mail is done well, but when I get in my mailbox I do not get the address of the sender entered in the form but I get my address same:
Example
from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>Parameters:
parameters:
database_host: <IP_ADDRESS>
database_port: null
database_name: ***
database_user: root
database_password: null
mailer_transport: smtp
mailer_host: smtp.gmail.com
mailer_user<EMAIL_ADDRESS>mailer_password: ***
mailer_encryption: ssl
mailer_auth_mode: login
mailer_port: 465
secret: 1f55a0254de4a7699bfd3e5b3296665371ba2033
and my config:
swiftmailer:
transport: "%mailer_transport%"
host: "%mailer_host%"
username: "%mailer_user%"
password: "%mailer_password%"
port: "%mailer_port%"
encryption: %mailer_encryption%
auth_mode: "%mailer_auth_mode%"
spool: { type: memory }

what should be fill in these requirments. iam facing this error "Expected response code 220 but got code "", with message ""

@HarisButt1
1: Either you must allow less secure apps or use app password by enabling 2 step verification on your gmail acc.
2: Disable any antivirus on your machine.
3: Don't forget to clear your cache (sometimes it don't take updated content from .env file)
4: You don't need to change anything in Mailer inside config directory(not recommended), you should use env variables

@ShrutiShah2990  Thank you i had this problem for one week it solved by replacing the tls encryption with
ssl


[Feature] : Navbar add close icon

Hello @Priyaaa1
closes #568

I will finish it..
please merge it

Before
Screencast from 2024-06-30 17-20-50.webm
After
Screencast from 2024-07-09 12-37-42.webm

@sivaprasath2004 the hamburger menu isn't working. Also, it's visible in larger screens too.

Screencast from 2024-07-09 21-18-15.webm
@Priyaaa1 fixed ..
please merge this..

@sivaprasath2004 the hamburger icon is not working on clicking.

@sivaprasath2004 the issue has been resolved now by someone else.


No handlers could be found for logger "roomba.main"

I am getting this error in testing the simple program example with Pyhton 2.7.16 on Debian Buster
No handlers could be found for logger "roomba.main"
Here is the test
from roomba import Roomba
address = "<IP_ADDRESS>.."
blid = "my blid"
roombaPassword = "my password"
myroomba = Roomba(address, blid, roombaPassword)
import json, time
for i in range(5):
print json.dumps(myroomba.master_state, indent=2)
time.sleep(1)
myroomba.disconnect()`

You need to create a logger called 'roomba', or at least a basic logger, eg:
import logging
log = logging.basicConfig(level=logging.DEBUG,
        format='%(asctime)s - %(name)s - %(levelname)s - %(message)s')

before you create the Roomba instance.
This won't stop it working, it just means it can't log output as you haven't created a logging instance. Then instead of using print you uselog.info().


Sitecrafting feature/get current item

Ticket: #1716
Some tweaks to @acobster's PR ...

Remove get_ from public methods
Use null as the default to return all values rather than -1

Coverage decreased (-0.3%) to 93.794% when pulling f8debd94adf29bb69b1a1d554edabfa2fa1da72c on sitecrafting-feature/get-current-item into 7a73a5d11ffd6f05358aed9e0138752512a8399c on master.


Address Form translation

I want to merge this change because...
Address Form was not translated (as mentioned in #1917). This pull request fix that.
Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[X] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] Python code quality checks pass: pycodestyle, pydocstyle, pylint.
[ ] JavaScript code quality checks pass: eslint.

@RemyDCF please keep us updated about the changes to be done :+1: If you cannot, feel free to tell us, if so, I will take a look ASAP for a better way. This fix is quite important.

Agree with you, @NyanKiyoshi. I edited the file as you suggested, but I don't really have the knowledge to create a proxy of the Address's Meta.

We create all the forms on initialization anyway so assigning the correct labels is not a big issue in my opinion. We could think of some optimization but that's a separate issue. For now I'd like to merge it as it fixes the original problem.


added Document.querySelectorAll() method

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Guys, pardon me, I messed up a bit on rebasing this PR.

@lynxlynxlynx Is there an easy way to "fix" this mess or should we just close it and have the user try again with a freshly synced fork?

@RandellDawson I fixed it. Never mind

Thanks for contributing to the freeCodeCamp repository. Congratulations on your first PR! 🎊
Look forward to more contributions from you! ✈


CTRL+C

I would suggest to manage the CTRL+C signal inside your code since it is one of the two ways to exit. When I use ctrl+c I got this:
Please enter your input message:
^CTraceback (most recent call last):
File "wctool.py", line 238, in
converse(parms.workspaceID)
File "wctool.py", line 165, in converse
minput = raw_input("\nPlease enter your input message:\n")
KeyboardInterrupt
which is an error message that should not appear when you use one of the method suggested to exit from the application.
Best,
M

Yes, nice to have to improve it but lower priority.


Sollet wallet does not work

Describe the bug
I'm unable to connect with Sollet wallet in starter projects or in the example: https://solana-labs.github.io/wallet-adapter/example/
To Reproduce
Change the wallet to Sollet and try to connect. It gets stuck in a "Connecting" state. There are no errors in the browser console.
Desktop (please complete the following information):

OS: Windows 10
Browser: Tried Chrome and Firefox

Interesting, I'm seeing the same issue with no error. I'll work on it, thanks for the report.

Ok it seems to be the provider url in getSolletWallet function, change to https://www.sollet.io works for me. Don't know why...

Good catch, thanks for finding that. I'll fix and release

This is fixed in e73d6275c29b96d01a9ad18282c61687d3cfd7c4 /<EMAIL_ADDRESS>Please let me know if it's not working after updating!

It's working !

It still not working what's going on?

The sollet.io is not responding. The Chrome extension also is not working (infinite loader is shown)

Sollet is deprecated, and no longer works because it relies on Serum RPCs which are no longer active.
Check out this thread for how to export your key from Sollet, which you can use with a different wallet:
https://twitter.com/steveluscher/status/1593075105042944000


Note about move to open-telemetry

Part of #37.

This is waiting on https://github.com/open-telemetry/community/issues/575.

I created the new repository under OTel and have updated the README to indicate our plans. I would like to finish #22 before we release 0.2, archive this repo, and begin replaying changes in the new repo.

https://github.com/open-telemetry/prometheus-sidecar


Do not assume that stdlib.mli contains a (*MODULE_ALIASES*) comment

Resolves #257.

This PR has been superseded by #306, which was merged to main, so closing.


Kadane's Algorithm || C++

Kadane algorithm for finding the maximum sum of contiguous subarray of an array.

Please assign me this issue to add the code for the algorithm @pranavgoel29

I am sorry, but I will not be adding a particular algorithm, please read the Readme.md and create a program which will help a beginner.

Ok !!


Why do we need do read file uri from message context to read a file

Version: 0.93
According to [1] a file location is read after obtaining it from the message context. Why do we need to do that when we have a configuration called "dirURI" that specifies the read location?
[1] https://ballerinalang.org/docs/by-example/ftp-server-connector

This is to track: https://github.com/ballerinalang/ballerina/issues/3526


"Flow Description" Mapping

Current Camara QoD API does not have ”Flow Information” or “Flow Description” concepts that are in 3GPP NEF AsSessionWithQoS API. Instead, a parameterization approach is used, i.e., putting attributes such as “ueAddr”, “uePorts” etc. at the root level of the request data model, and expect the transformation layer to map them to “Flow Information”/”Flow Description”.
After evaluation, we see some limitations on this approach:

This “flat” data model may be difficult to evolve to other features already supported by 3GPP NEF AsSessionWithQoS API, such as Ethernet flow and external app ID.
Lose the flexibility to define multiple flows in an AS session.
Confusing mapping of “protocolIn”/“protocolOut” to 3GPP Flow Description, since IP-based protocols are usually bi-directional.

I'm supportive of extending the API definition to allow the API consumer to request enhanced QoS levels for multiple flows. However, I would add the following comments:

It should be possible to request different QoS levels for the different flows, as this may be a requirement for the service the developer is trying to create. If multiple calls are required to the NEF to action this, then so be it.
To prevent the API consumer bundling unrelated flows into a single request, the API should either grant all or refuse all requests within a given API call. In the event that the request is refused, any undoing of intermediate configuration steps is up to the network operator.
The proposed changes should not be incorporated into the API until the mechanism for identifying the target UE by GPSI is defined and also incorporated (camaraproject/WorkingGroups/issues/4).
If the UE is identified by source IP address, no restriction should be placed within the API (e.g. in the documentation or any pattern/regex) on whether this is the public (NATed) or private address of the UE. It should be possible for the API consumer to specify either depending on their knowledge, though mobile operators may chose whether to support both or only one of these possibilities depending on their implementation.

There are different points addressed in this issue and additional ones in Eric's comment. My proposal would be to split the points up into different issues which can be discussed and addressed independently. I see the following different issues in the discussion:

Allow multiple flow descriptions:
We deliberately have choosen the "flat" approach and to allow the request of one flow (description) per service request. Our target was to reduce the complexity of the 3GPP API definition and provide an easy-to-use API for the use cases we are aware of. We have not yet seen use cases were developers have the need to request multiple flows in one (transactional) service request. Therefore we don't recommend to start with the complexity of an array of flow descriptions. The proposal is to start with the simplified "flat" API and discuss on additional API for concurrent multiple flows separately later.

Incorporating port options explicitly into the JSON schema:
These parameter options are currently only mentioned in the API documentation (single port, range of ports, or even "*"). We would welcome a solution proposal. This could be addressed independent of the topic of single or multiple flows per service request.
BTW: with these options the "flat" description of a single flow has already a lot of flexibility which covers a lot of use cases we are aware of with a single service request.

Resolving the confusion of “protocolIn”/“protocolOut” with a better parameter design:
I agree that the current parameters are confusing and that we need a better parameter design here, e.g. the bidirectional character of TCP in one parameter. Probably we need also to cover the case where UDP flows use different port combinations per direction. Open for proposals.

Identifying the target device/UE by GPSI:
As mentioned by Eric this is another issue which need to be addressed for the QoD API after the mechanism is agreed and it will be supported by the south-bound interfaces and/or there is a non-proprietary translation function available.

Identifying a device (UE) via public and/or private IP address:
That's as well an independent issue to discuss (unfortunatley a complex one). On the one side I tend to agree (as on the device the developer sees the private, on the backend the public IP address). But I'm concerned if this will lead to differences between operators, and developers would need to find out if an operator has implemented the one or the other option or both.

@shilpa-padgaonkar i dont see clarification of below two questions. Pasting the ques in this thread

Are ueAddr and asAddr subnet ranges and not  individual IP addresses? What is the reason behind subnet range?
uePorts and asPorts are not singular. It will become complicated to create QoS flows for each of ueAddr/uePorts+asAddr/asPorts combination.How do we address that?

@hdamker
The issue of identifying the UE by public and/or private IP address is indeed important. If identifying the UE by source IP address is to be retained as an option once identification by GPSI is supported, then the only use case that makes sense to me is when that is the public IP address, as any API consumer that knows the private IP address should also know the MSISDN. A public server, however, would be unlikely to know the MSISDN.
But if private IP addresses can still be specified, there is an additional issue not yet addressed which is that the private IP address may not be unique, and several UEs may have the same private address. How do you propose to resolve this ambiguity? The 3GPP solution is to specify an ipDomain parameter along with the private UE address, this parameter identifying which PGW allocated the UE private address. But the API consumer would not know this information, so that solution would not work here.
@SyeddR
I agree that IP subnet ranges are unnecessary here, and Vodafone would not implement support for these (i.e. if the API consumer specified a subnet range other than /32, then we would return an error message). There is no obligation on MNOs to fully implement all capabilities that can be requested through a valid API call, but only to give sensible error messages for those capabilities that can be requested but have not been implemented. Other MNOs may choose to support subnet ranges if they see some commercial advantage in doing that.
Similar arguments apply to port ranges, but here I think that there are valid use cases where port ranges may be specified. For example, a UE may have multiple active TLS sessions with the target server, and each will have a separate source port. Indeed, specifying a port range is optional, and Vodafone would treat the absence of any specified port range to mean whatever ports were in use between the UE and the specified server, which could be multiple. Again, it is up to each MNO to decide what capabilities to implement, and which API calls to return an error for.

@SyeddR @eric-murray Thanks for your questions and comments.
Regarding identifying UE: that is an common topic not only affecting QoD APIs, therefore the discussion in Commonatilies working group. I also see the ambiguity of IPv4 addresses in mobile networks, thanks for creating issue #34 for that. For all identifiers (including IPv6 private or public addresses, MSISDN of GPSI as proposed in Commonalities) the main question is how the API user will get this identifier. E.g. even for an application on the UE the dominating device operating systems aren't allowing to read the MSISDN.
Regarding IP subnet ranges: Fully agree that on UE side a range does not make sense and a sensible requirement is to have /32 here. On the application server side we got already requests from developers to allow broader ranges, e.g. to cover redundant backend servers. I suppose that the operator policy will here depend on the requested quality profile. For some profiles it might be ok to allow any application server in the internet (<IP_ADDRESS>/0), others might even need to be restricted to known edge servers. In short: I agree with operator policy and sensible error messages here.
Also agree with the proposal of @eric-murray regarding the handling of port ranges as optional and with "*" as default. Until now we have seen port ranges for UDP connections in use cases.
@SyeddR IP address and port ranges are supported by the flow description parameter we are handing over to the south-bound network API, so they are not adding complexity. Nevertheless the policy checks have to be done.

@emil-cheung : Could you kindly share the sample JSON you had shared during this topic presentation?

@emil-cheung : Thank you for sharing the material. It is uploaded here for future reference - QoD enhanced flow info

The IPv4Addr supports subnets and should be labelled as IPv4AddrMask or the mask should be dropped in order to not mislead the dev.

@emil-cheung we have now new #51 which should superseding this one. Can we close here?


"No selection" dropdown value not translated when testing form

When testing a form and the page is in English, the default value of a dropdown list, "No selection", is not translated to "Pas de sélection" when toggling the testing to French.
Similarly, when testing a form and the page is in French, the default value of a dropdown list, "Pas de sélection", is not translated to "No Selection" when toggling the testing to English.
Private Zenhub Video

Once a form is published, this is not an issue anymore. The proper language appears in the dropdown list.

Reported in https://cds-snc.freshdesk.com/a/tickets/16256

The French string exists dropdown-initial-option-text.
However it is set to match the form builder interface language rather than the preview/test language:
Private Zenhub Video

@srtalbot To check if needs to be reopened.

@srtalbot The issue only applies to the new date fields:


Vulnerable to Cross-site Scripting (XSS)

Describe the bug
Unsanitized input from data from a remote resource flows into innerHTML, where it is used to dynamically construct the HTML page on client side. This may result in a DOM Based Cross-Site Scripting attack (DOMXSS).
client/src/components/account.component.js Line 73
    const BackupCodes = () => {
        const codes = [...valid, ...invalid]
        // function validateToken(token) {
        //     for (let a=0; valid.length; a++){
        //         if(token === valid[a]) return true
        //         else if(a === valid.length-1 && token !== valid[a].toLowerCase()) return false
        //     }
        // }
        let table = document.createElement('table')
        let row = document.createElement('tr')
        let column = document.createElement('td')
        table.classList.add('isCentered', 'full-width', 'no-border')
        for (let x=0; x<codes.length; x++) {
            if(x%2 === 0 && x !== 0) {table.innerHTML += row.outerHTML; column.innerHTML = codes[x]; row.innerHTML = column.outerHTML}
            else if(x === codes.length - 1) {column.innerHTML = codes[x]; row.innerHTML += column.outerHTML; table.innerHTML += row.outerHTML}
            else{column.innerHTML = codes[x]; row.innerHTML += column.outerHTML}
        } return table.outerHTML
    }

Expected behavior
This issue should be fixed as soon as possible to prevent Cross-site Scripting (XSS) Attack.
Desktop (please complete the following information):

OS: Windows 10 OS Version 2009 (Build 19042.985)
Browser Chrome
Version 90.0.4430.212

After reading some references, may these articles maybe we will use dompurify dependency which sanitize html into a cleaner code and prevents DOM Based Cross-Site Scripting attack (DOMXSS).


Feature/import hadoop

import hadoop 2.2.1 cookbook and deps

import only... merging.  travis failures unrelated, will fix in a separate PR


Override new Image, and createElement("img") with src

Often somebody is using this API to send metrics.
It would be useful to have them here, what do you think?
I have 50% ready solution about this section, so I can help you with.

Oh. Sorry, this has been unnoticed for quite some time. I think this is a fantastic idea as a matter of fact. Would love to see a PR :)


How can update the entity if the primary key is GUID

I'm still using the version 3 of OData and the result was of my debugging was shown on this

https://www.dropbox.com/s/k9z6ujqiogc7tst/OdataDelete.png?dl=0

I'm sorry, this library doesn't support ODatav3


Unexpected Keyword 'feature_defaults' in 'add_points'

🐛 Bug
Calling add_points with the keyword argument feature_defaults does not work. It throws a TypeError because feature_defaults is not a valid keyword.
However, according to the doc it should be a valid keyword argument: https://napari.org/stable/api/napari.Viewer.html#napari.Viewer.feature_defaults
To Reproduce
Can be reproduced with this sinippet:
import napari
v = napari.Viewer()
v.add_points(data=[[0, 0], [1, 1]], feature_defaults=["True", "False"])
napari.run()

which throws:
Traceback (most recent call last):
  File "/home/pape/Work/playground/napari/check_points.py", line 3, in <module>
    v.add_points(data=[[0, 0], [1, 1]], feature_defaults=["True", "False"])
TypeError: add_points() got an unexpected keyword argument 'feature_defaults'

Expected behavior
It opens the viewer with a point layer and the passed feature default values.
Environment
This happens in a fresh installation from conda forge, using the latest napari version.
Here's the napari info:
napari: 0.4.17
Platform: Linux-5.15.0-71-generic-x86_64-with-glibc2.31
System: Ubuntu 20.04.6 LTS
Python: 3.10.10 | packaged by conda-forge | (main, Mar 24 2023, 20:08:06) [GCC 11.3.0]
Qt: 5.15.6
PyQt5: 5.15.7
NumPy: 1.24.3
SciPy: 1.10.1
Dask: 2023.4.1
VisPy: 0.11.0
magicgui: 0.7.2
superqt: unknown
in-n-out: 0.1.7
app-model: 0.1.4
npe2: 0.7.0

OpenGL:
- GL version: 4.6 (Compatibility Profile) Mesa 21.2.6
- MAX_TEXTURE_SIZE: 16384

Screens:
- screen 1: resolution 1920x1200, scale 1.0

Additional context
I ran into this while working on https://github.com/napari/docs/pull/145

However, according to the doc it should be a valid keyword argument: napari.org/stable/api/napari.Viewer.html#napari.Viewer.feature_defaults

This links to attribute documentation, not constructor keywords arguments.
However, the feature_defaults  keyword argument is added to the current main. (in #5646)
On 0.4.17 the feature_defaults are calculated based on feature values.
@andy-sweet Did we want to backport it to 0.4.18, or is it somehow breaking?

Thanks for reporting! Indeed it appears that this is merged, but not release, see:
https://github.com/napari/napari/pull/5646
I'm not 100% sure why the stable docs would suggest this would work—that part is auto-generated, as is the add_points method, so I guess something went sideways.
Please note that our docs are released/updated along with napari releases, so the example you are working on will become public on napari.org (stable) along side the next napari release. I tagged that bugfix PR with the 0.4.18 milestone, so it should be part of the next release, so everything should be good to go—note that feature_defaults expects a dict or dataframe, see:
https://napari.org/dev/api/napari.Viewer.html#napari.Viewer.add_points
Also, 😬 but the stable docs for contributing are not current, because of changes to the build system. So you need to look here (latest in the version selector at the top).
https://napari.org/dev/developers/documentation/index.html
You'll see that we now recommend using development napari install.

Thanks for the replies. I have updated my set-up now to the current main and there feature_defaults is available as a keyword argument of the constructor.


Update container BC version

Is it posible to update the BC version of the container, instead of creating a new one?
Thanks!

The short answer is probably not, as it does not makes sense to do that in most cases. The longer answer is yes if you do a lot of powershell-commands where you download the newer version,, upgrade the database etc etc. then it SHOULD be possible - I guess that no-one has really tried....
The better answer is - If you really need to do stuff like this, then use Hyper-V manager or something similar, as you then have a full GUI and can see what is happening when it does not work.
Why do you want to be able to do that?

Hello Palle,
I guess my short answer would be that I don't want to deal with all the Windows overhead!
It would be nice to preserve all data, settings, and deployed extensions by having a command to do an in-place upgrade.
Alain

Hi Alain
I fully understand. I am only using Hyper-V when absolutely necessary (normallay when I am doing upgrades NAV —> BC.
When we talk abort Docker and Containers the motto is always “Backup your powershell scripts”.  So in your case you need to:

Save the powershell script that creates the database with a backup of your data.
Save the powershell script that installs the extension

You can spin up a new container in less than 10 minutes, and you are up and running on a new version -compared to what? Hours of uninstall/download new version (as a ZIP-file), install a new version and running multiple configuration powershell commands with almost no option of finding out what is wrong if it afterwards does not work.

Regards
Palle

Agree with Palle.
You should not try to update the version of BC inside a container.
You should instead run a container using SQL Server on the host or SQL Server in another container.
Then you can kill a container and start a new container pointing to the database from the other container.

Thanks, Palle and Freddy!
I get your point!
I like the idea of a separate container for the SQL Server and the BC instances.
I would still need a Hyper-V VM to test out in-place upgrades.
Thanks!
Alain


Assigning user defined properties

Is there a way to assign user defined properties on the initial value (createFormGroupState or something similar?)
On a nested form structure, what is the most efficient way to assign different values across the nested form structure but they all have the same property. For example your user def property is 'placeHolder' and each of your fields has an unique value.  I  can do it currently with an updateGroup() per object/form  but it doesn't seem to be very efficient.

Basically I have a complex form that needs to be extended, and I'm using user def properties to do that.  I'm not user the best way to get that extended data into the form object.  I can do it with a ton of updateGroup calls but it just seems messy.
I did figure out if i recreate the entire initial_value object by hand that would be generated by the updateGroup/createFormGroupState solution I can do it but that seems like a very brittle solution.

The best way to achieve this would indeed be to simply perform an updateGroup after creating the initial form state via createFormGroupState. To get rid of repeating the 'placeholder' string you can of course create your own custom update function like this:
const setPlaceholderProperty = (v: any) => setUserDefinedProperty('placeholder', v);

However, if you really do not want to use the updateGroup way you can use updateRecursive for a somewhat generic approach. updateRecursive allows you to apply an update function to every state in the form state tree. By using the fact that ngrx-forms builds the ID of a state according to the "path" it takes in the form value you can then assign placeholder values from an object (using an implementation detail like this is something that I don't usually recommend, but this one should really never change). The solution would look something like this (untested, so it may contain bugs):
interface FormValue {
  someString: string;
  nested: {
    nestedString: string;
    nestedNumber: number;
  };
}

const INITIAL_FORM_VALUE: FormValue = {
  someString: '',
  nested: {
    nestedString: '',
    nestedNumber: 0,
  },
};

// you could use a mapped type here to get some type safety;
// also, the placeholders object can be partial if not all fields should have one
const PLACEHOLDERS = {
  someString: 'some string',
  nested: {
    nestedNumber: 'nested number',
  },
};

const FORM_ID = 'myForm';

const INITIAL_FORM_STATE = updateRecursive(
  createFormGroupState(FORM_ID, INITIAL_FORM_VALUE),
  s => {
    // this returns the "path" of the state in the form value shape, i.e. the ID
    // of "nestedNumber" is "myForm.nested.nestedNumber"; we take everything but
    // the first part of the path and use the rest to navigate through the
    // placeholders object to get the correct placeholder for the state
    const pathParts = s.id.split('.').slice(1);
    const placeholder = pathParts.reduce((o, p) => o && o[p], PLACEHOLDERS as any);
    return !placeholder ? s : setUserDefinedProperty('placeholder', placeholder, s);
  },
);

Thanks for the help!  I like the idea of doing the updateRecursive to handle these type of larger merges.


Add model versions to the python Fuel Utility.

Issue #, if available:
https://github.com/aws-robotics/aws-robomaker-hospital-world/issues/1
Description of changes:

Updated fuel_utility.py to support model versions with the Ignition Fuel API. Now, customers can set model versions with the script using this format: "Model=Version".
Set the latest model versions for OpScrubs, VisitorChair, PatientWheelChair and CGMClassic.
Cleaned up the code in fuel_utility.py.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The changes look good. If possible, can you add a short description of the additional changes in the PR description for future references? I think the PR currently only describes the fuel-utility related changes.


Updating Advanced Audit events (E5 events) description

@sujitnaray PR merged. Thanks for making these updates. Next time, please submit PRs by first going to the staged version of the topic, which are located at https://review.docs.microsoft.com (just add "review" to the beginning of the URL in the live version). Doing this submits your changes and the PR to the private repo. The advantage of doing this is so I can use spellcheck and other quality control tools which aren't available in the public version. Thanks in advance.


Release Google.Cloud.DocumentAI.V1Beta3 version 2.0.0-beta05

Changes in this release:
New features

Added sharding_config field in DocumentOutputConfig.GcsOutputConfig in document_io.proto (commit 25f4d4d)
Added process_options field in ProcessRequest in document_processor_service.proto (commit 25f4d4d)
Added sample_document_uris field in ProcessorType in processor_type.proto (commit 25f4d4d)

Created release for Google.Cloud.DocumentAI.V1Beta3-2.0.0-beta05


jettywrapper gem causes RubyMine/Intellij line debugging to fail if rake gem is not loaded

This is a bug report that covers a perhaps unusual spread of cases.  I first encountered it when attempting to line debug an rspec test in a Blacklight project using Intellij (aka RubyMine).  The version of jettywrapper being used is 1.8.3, but the 2.0.0 version has the same issue.
I had created the Run/Debug Configuration, which was simply the running of a spec script.  The script failed with this error:

.../gems/jettywrapper-1.8.3/lib/tasks/jettywrapper.rake:4:in <top (required)>': undefined method namespace' for main:Object (NoMethodError)

I investigated, and found in jettywrapper.rb:
Dir[File.expand_path(File.join(File.dirname(__FILE__),"tasks/*.rake"))].each { |ext| load ext } if (defined?(Rake))

This code was attempting to load the jettywrapper rake tasks even though the rake gem was not loaded.  This turned out to be due to JetBrains code for TeamCity integration, which monkey patches the Rake module.  Example on Mac OS X:
In file ~/Library/Application Support/IntelliJIdea12016.1/ruby/rb/testing/patch/common/teamcity/utils/logger_util.rb:
...
module Rake
  module TeamCity
    module Utils
      class FileLogger
...

...and there are many other files that apply their own patches as well.
The problem is that if the rake gem is not actually loaded, the JetBrains code will cause the Rake constant to be defined regardless, and thus jettywrapper will attempt to load its rake tasks, which will of course error out because the core rake functionality is absent.
RubyMine/Intellij is a fairly popular development environment, and its line debugging is one of its strongest features.  Aside from this IDE-related issue, though, it would be more robust to test for defined?(Rake::Version) instead of just defined?(Rake), as other libraries/gems might also be monkey patching rake.
I will submit a pull request right after creating this ticket.  I haven't included a test because for my case it's an Intellij line debugging issue and that would be a difficult situation to simulate.  There are probably ways to package a spec though, merely by adding fixture that includes a monkey patch of rake and then verifying that the jettywrapper rake tasks were not loaded.  If this is considered absolutely necessary I can attempt it.
However, I think this issue and the related fix is straightforward enough that no test is needed, and I believe the switch to defined?(Rake::Version) is a reasonably stable and more conceptually sound duck-typing approach to dependency testing.

Just wondering if you know when you'll be updating the rubygems.org gem version, so as to get this fix?

I'll ping the Hydra slack channel and see if anyone has cycles to cut a release.

Actually, I've discovered that our Hydra version keeps us from upgrading jettywrapper beyond ~> 1.8.2:
$ bundle update jettywrapper
Fetching gem metadata from https://rubygems.org/............
Fetching version metadata from https://rubygems.org/...
Fetching dependency metadata from https://rubygems.org/..
Resolving dependencies.......
Bundler could not find compatible versions for gem "jettywrapper":
  In Gemfile:
    jettywrapper (~> 2.0.3)

    hydra (~> 7.1.0) was resolved to 7.1.1, which depends on
      jettywrapper (~> 1.8.2)

    hydra (~> 7.1.0) was resolved to 7.1.1, which depends on
      hydra-head (~> 7.2.0) was resolved to 7.2.2, which depends on
        hydra-core (= 7.2.2) was resolved to 7.2.2, which depends on
          jettywrapper (>= 1.4.1)

So even if a new release is made, we wouldn't be able to get it into our project without upgrading our hydra.  Is there any way the fix could be backported to 1.8.3 or a newer 1.8.x version to get it through ~> 1.8.2?

@da70 you can remove the hydra gem from your gemfile and instead pin to the version of hydra-head that you want to use.

@jcoyne, unfortunately we can't upgrade Hydra right now but we definitely plan to at some point.
If jettywrapper 1.8.3 is no longer under active development or support we can fork for the time being.  I just checked the [LICENSE|https://github.com/projecthydra/jettywrapper/blob/master/LICENSE] and it seems that forking is fine once we fulfill the requisite conditions?

@da70 I'm not advocating for you to upgrade hydra (although you should), just saying that you do not need the hydra gem.  It just imposes a strict set of dependencies that is currently hobbling you. You don't need it.


Request for an example to geolocate object/mesh/etc

Would you consider adding an example that anchors a mesh or gltf model to a fixed lat/long location on the map and scales it accordingly as the camera is rotated and moved toward/away?  That would be tremendously helpful.  Sort of like the data layer for the maps/webGL.  Thanks in advance.

Request

Would you consider adding an example that anchors a mesh or gltf model to a fixed lat/long location on the map and scales it accordingly as the camera is rotated and moved toward/away?  That would be tremendously helpful.  Sort of like the data layer for the maps/webGL.  Thanks in advance.

Would you consider adding an example that anchors a mesh or gltf model to a fixed lat/long location on the map and scales it accordingly as the camera is rotated and moved toward/away?  That would be tremendously helpful.  Sort of like the data layer for the maps/webGL.  Thanks in advance.


Adds recursive validation. Fixes display of validation errors.

Looks good; smoke testing works as expected. So an edge case oddity, using the BIBFRAME Instance fixture, after loading and clicking the Save and Publish button the expected validation errors appear:

The Item Information Barcode is displaying the Required label as expect:

Clicking the Add button for the Barcode adds and expands the Barcode but the disabled field is highlighted with the Required error message even though by clicking and adding the Barcode resource template I added a value.

This isn't a blocker because clicking on the Save and Publish again now clears out the barcode error

from the list of errors:


Translatable property with the same value gives an empty result

Hi guys,
I have a problem with the translatable extension and here is the code to test it: https://gist.github.com/1278099
This test fails at the line 37 with this error:

Hart\DssCoreBundle\Tests\Repository\PoiRepositoryTest::testLocaleNameNotNull
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-Valley

Which means that $valleyToTest->getName() return an empty string. It seems that same value in different languages are not stored in the db.
I'm using Symfony 2.0.4 and DoctrineExtensionsBundle 1.0.0

Please report in to the library. The bundle is not responsible of the logic of the extensions


memory leak

Hi.
I seem to have a memory leak in the stream function
(node) warning: possible EventEmitter memory leak detected. 11 data listeners added. Use emitter.setMaxListeners() to increase limit.  -- Called from events.js:235:17(function EventEmitter.addListener)logMsg @ client.js:63(anonymous function) @ client.js:97c.emit @ socket.io.js:2c.onPacket @ socket.io.js:2d.onPacket @ socket.io.js:2c.onPacket @ socket.io.js:2c.onData @ socket.io.js:2websocket.onmessage @ socket.io.js:2
client.js:63 Trace
at EventEmitter.addListener (events.js:239:17)
at Namespace. (/Applications/MAMP/htdocs/wit.ai/project/tweetplugin.js:59:16)
at emitOne (events.js:77:13)
at Namespace.emit (events.js:169:7)
at Namespace.emit (/Applications/MAMP/htdocs/wit.ai/project/node_modules/socket.io/lib/namespace.js:206:10)
at /Applications/MAMP/htdocs/wit.ai/project/node_modules/socket.io/lib/namespace.js:174:14
at doNTCallback0 (node.js:407:9)
at process._tickCallback (node.js:336:13)  -- Called from console.js:80:8(function Console.trace)
tweetplugin.js:59:
'stream.on('data', function(tweet)'
What does this mean?

Yeah my apps crashing and freezing up. Am using the stream function

This is more likely that your need to unlisten from the stream. You can set the max amount of listeners with emitter.getMaxListeners() and emitter.setMaxListeners()
However I would suggest when your done with an event listener you remove it and init it again when needed.

:wave: I'm getting this issue as well. We could set emitter.setMaxListeners but I believe this is happening within the node module and not within our personally written code, righT?


Add support for dependency injection to setup

This PR adds the ability to inject a setup into handlers (actually updating the handlers will be done in a follow-up PR).
It also adds support for test doubles in the setuptesting package and uses one of the doubles in the fee utils test

https://github.com/openpredictionmarkets/socialpredict/blob/96b2f441099365fff66fd26cce949f4d9a2123dc/backend/handlers/bets/betutils/feeutils.go#L47
BuySharesFee needs to be changed to EachBetFee, looks like this didn't carry over when you were separating the changes?