Error message on directive namespace conflict

I got this error message today:
Error: [$compile:multidir] Multiple directives [pagination, pagination] asking for new/isolated scope on: <pagination>
http://errors.angularjs.org/1.4.0-rc.1/$compile/multidir?p0=pagination&p1=pagination&p2=new%2Fisolated%20scope&p3=%3Cpagination%3E

Took me a while to debug whats wrong with my pagination directive. Until I discovered another directive with the exact same name pagination from ui.bootstrap. It wasn't obvious as I didn't include ui.boostrap myself.
If the error message would include the module name the directive came from, this had helped a lot. Something like:
Error: [$compile:multidir] Multiple directives [pagination (module: foundation), pagination (module: ui.bootstrap)] asking for new/isolated scope on: <pagination>

Example code: http://plnkr.co/edit/Mgxutpvfr4ngqRD6fV1F?p=preview
Version: angular.js/1.4.0-rc.1

Feel free to provide a PR for this. Otherwise we will try to get to it in 1.5. At the moment there is no link back from injectable components to their originating modules.

Trying my hand at a PR #11793. Would love some feedback to make sure I'm taking this the right direction. A little iffy on the idea of passing the module to all the invoke queue. But I can't think of any other way to accomplish this.

@kentcdodds Probably there could be other solution to do this. I'm just not 100% sure if this is possible, cuz I'm fairly new to angular.js sources.
I assume, that each directive is an object. If so, there could probably be a way to find module in which directive is defined by using object comparison. Even if this solution is not good in terms of performance, I think it could be quite okay during exception throwing.
And again, it's only an assumption, so don't judge me strictly please :)
As well I can provide pull request, if this solution is implementable and somebody will tell me which parts of this solution are not implemented at all.

It would be better find a way to search the loaded modules for directives with a given name.

That could be reasonably done. I've done this before (the list on the left is auto-generated based on the contents of the _invokeQueue). That would also touch much less of the entire library.

Yeah, I see that. But in your example you have root module hardcoded:
var allTheThings = [];
  _.each(angular.module('kcd').requires, function(submoduleName) {
    allTheThings = _.union(allTheThings, getThings(angular.module(submoduleName)));
  });

As from what I understand, in real application this information is available only on injector level. That's why from my point of view, the only way to achieve what we want is to make injector expose list of loaded modules here: https://github.com/angular/angular.js/blob/master/src/auto/injector.js#L838
Guys, am I heading in the right direction? Can I go on with that and create a pull request?

@lugovsky if you have a solution that does not impact the runtime performance and is cleanly implemented then by all means make a pull request.

Hey guys. I finally created a pull request for this issue. Please check it out and tell, does it look good enough for you. I will be happy to do any modifications


Missing error handler on socket. TypeError: (msg || "").replace is not a function Error #1969

It seems that I have the same issue with webpacket#2. version.
It happended to me when I treid to instantiate partical class which was btw. in one .ts file along with other classes. Even this class was exported. When I put this class out into separate file the problem was gone.
Is there any particulare relation between having files in the same file and using them inside the tests?
Edit: I thought it is related as when I recreated  1 file it is gone but it happens also for others. this has probably nothing to do with it.
Any idea why I can see such error ?
Thanks,
Frank

update:
Not really understanding but I can turn it on / off this issue by commenting 1 line of code.
in the test I do instantiate this class:
new RuleX(....);
export class RuleX
{
    constructor (public _selectors: Array<Selector> , public _properties: Map<string, any> , public _rank: number = -1 , public _lineNumber: number = -1) {

    }

    apply (meta: Meta , properties: PropertyMap , declareKey: string): number
    {
        return Rule.merge(meta , this._properties , properties , declareKey);
    }
}

When I comment out this Rule.merge - which is some static method in some other file the error is gone. Whne I put it back I see this error.
 static  merge (meta: Meta , src: Map<string, any> , dest: Map<string, any> , declareKey: string): number
    {
        var updatedMask: number = 0;

        return 0;
    }

I tried to removed all the code, body everything but not sure why calling static method woudl cause this problem.

@fkolar could you debug and look at value of msg variable here TypeError: (msg || "").replace
Thanks

I am closing this issue as this one is already resolved while ago. The main problem was it could not instantiate certain class , - and it was at the time I was using barrells and try to import Class as {XX, XX, XX, XX} from 'One common root',  and it did not work propertly - when I imported every single class it started to work.


Testing in tunnel

Hi, thank you for sharing your work.
We are testing Locus in different environments with a drone (currently hand carried) with 1 Velodyne VLP-16 and a Vectornav IMU.
Where there are sufficient planes and edges all is well, now we are testing it with a rosbag made in a very monotonous railway tunnel about 50 m long.
Arriving about 9 m from the start Locus hangs for some time until the drone approaches the exit.
In this way it does not track about 25 meters.
Video here: https://www.youtube.com/watch?v=RqD9AQTF9YE
I have tried many combinations of these parameters:
leaf_size: 0.25/0.10/0.20/0.30/0.50
recompute_covariance_local_map: true/false
icp_covariance_method: 1/0
icp_max_covariance: 0.01/0.1
Iterate ICP this many times.
iterations: 30/25/20/15/10
Radius for normal search
normal_search_radius: 10/20/30
I did not get much difference.
Do you know of any other configurations to try in this environment?
If this, as with other slams, is a difficult environment does a second point cloud have a good chance of solving the problem?
And, since we need the first lidar horizontal for collision avoidance, what orientation do you think would be optimal for a second lidar?
See you soon.

Hi @wpwebrobotics - thanks for testing locus!
You have hit the nail on the head with your question - this is one of the more challenging components of lidar SLAM, especially with lower resolution lidars, such as a VLP-16. So I am super excited you are testing in such a challenging environment! A great way to try and tackle these lidar-degenerate cases. That tunnel looks to be especially challenging!
Unfortunately, there is probably a limited amount that you can do by tuning ICP parameters - there is just inherently limited information in that environment. We did have an observability module that would have been showing that your observability in the tunnel is very low. @yunzc might have some modules in LAMP you could use to test the point clouds from there (if you are interested, I can give you more info).
Hence, my advice for these environments are:

Integrate in another source of odometry - e.g. the visual signal you have shows movement, and running some kind of VIO on that would give an odometry that could be fed into LOCUS as the initial guess. This is how we got around these environments in our tests - feeding VIO or Wheel-Inertial-Odometry (WIO) as the initial guess to LOCUS. Of course, you need to have the available compute on your drone. One quicker option is to get a Realsense T265 - that gives you an odometry output.
Try a lidar with higher vertical resolution and FOV (e.g. Ouster OS0 64 or 128) - this is expensive, I know, but should give more information for ICP in these environments
A more affordable option is to use a Realsense L515 (if you can get one) and point it diagonally down to the ground  - that might help. In general for a second lidar, pointing down to see the ground can help. People tend to point L515's straight down or up to also be useful for avoiding obstacles during vertical flight.
Tilting your VPL16 to around 30 deg might help, to see the ground and ceiling, but that is tough to do on a drone, and may not help that much (you coud test with a hand-carry test keeping the drone tilted at 30 deg).

It is especially a challenge on drones because of the limited power and weight, but hopefully the thoughts above can help?
Happy to continue the discussion here.

@femust - do we still have our observability computations in the code?

Yes we have and it may be use for this purpose, though probably we shouldn't do that after update of odometry step hehe but it is to be improved yet :)

Hello, I have a L515 and I wana use it as second odometry source like you suggest.
How can I do this? Realsense package offers an example of tracking, but is not realible...can you please suggest a way to do this?

@Cristian-wp if L515 provides for example visual odometry you just need to change the ODOMETRY_TOPIC in locus.launch + in lo_setings.yaml keep
data_integration on mode 3

@wpwebrobotics can we close this issue or some additional comments are needed?

I have check online and even ask to Intel, the L515 can not provide an odometry topic, you have to use an external package to generate it. Maybe can you suggest a robust package to perform visual odometry?
Is better to generate this odometry using the camera internal IMU (low grade) or maybe use an external high grade IMU?

it doesn't have to be odometry directly from visual sensor, it can't be wheeled as well (or legged), you can use imu as well
in lo_setings.yaml keep
data_integration on mode 1 or 2 (or 0 if you want to do just purely lidar odometry without preintegration)

@femust thanks for the fast reply :)
I wana use as input the visual inertial odometry, with the standard lidar intertial slam performed by locus.
As you say "it doesn't have to be odometry directly from visual sensor", so I wana fuse the camera imu with the camera images,  can you please suggest a ROBUST ros package to perform this operation?

Hi everyone, I was away on vacation but my colleague Cristian kept this very interesting discussion open.
To summarize, to attempt navigation in our environment I would exclude in the beginning a pure lidar solution with only one lidar, so there are, in my opinion, these solutions that should be evaluated depending on the degree of complexity, weight and consumption, taking into account that a pointcloud forward is necessary for avoidance:

Pure lidar: two lidars with larger vertical amplitude than the VLP-16, and tilted at different angles. One can try for weight containment the L515 as a second lidar.
A Lidar + external odometry: give Locus a source of odometry derived from sensors other than the Lidar (eg. VI-SLAM). So one package among those you mention: VINS we know, we know ORB-SLAM2 and can evaluate ORB-SLAM3, Kimera we can try. We have also used ROVIO in the past with good robustness results.
Point 2 with 2 lidars.

We hope to do these tests soon.
Giorgio.

Good luck with this! I think that is a good summary. Yes, ROVIO is a great option as well.
Another thought - depending on the specifics of your application - is to use a UWB ranging beacon. It is light-weight
, and low-power, and if you are just moving down a tunnel, it can provide enough constraints to combat the "lidar slip" as well call the scenario you are seeing in the tunnel. This isn't plug-and-play in LOCUS though - and would need another fusion step after LOCUS (e.g. an addition to LAMP, or another approach).

Thank you and likewise.

@femust Thank you for your help, but can you reopen the issue? Last week I have try to run locus with vins_mono as odometry input, but the performance does not change, Locus fail always at the same point.
In rqt_graph I can see that the topic are correctly passed, and I have verify that are not empty.
My setup is:
data_integration:
  mode: 3

<param name="b_integrate_interpolated_odom" value="true"/> # 10Hz odom frequency
 <param name="b_pub_odom_on_timer" value="false"/>
 <param name="compute_icp_covariance" value="false"/>

Can you suggest me a way to fix the problem?
Maybe I have to increase the "weight" of the odometry during data fusion, how can I do it?
There is a way to check that the odometry topic is fuse correctly?(I do not have warning or error in the terminal)
The odometry topic is correct respect to the ground truth (distance hand measured in the test field)

If the tunnel is very very long (as in your case) the locus may not act nicely and it's a widely known issue with lidar odometry system and ICP, even with preintegration from odometry sources (probably you would spend a lot of time for tunning params to make it work which does not make sense I think if you want to do something generic).
What I would suggest is to turn on flag compute_icp_observability: true in point_cloud_localization/parameters.yaml
and write in PointCloudLocalization.cc some logic-based method with a threshold that switches to a pure-visual-inertial-based odometry system (vins) -> we have for this flag is_healthy and the whole diagnostic_msgs::DiagnosticStatus infrastructure which is not currently used frankly speaking.

Hi @BenjaminMorrell, about your offer to test our pointcloud with the LAMP modules "@yunzc might have some modules in LAMP you could use to test the point clouds from there (if you are interested, I can give you more info)." @yunzc
Can you please explain how to do it?

@femust thanks I will try this way with the velodyne. Do you have an example or maybe a paper that explain how to switch the slam method in a safe way? Maybe I am wrong, but is not dangerous to switch the slam method in air?
Now we have even a Ouster so we try to repeat the first suggested tests with this new sensor.

locus is an odometry system so it works on the incremental changes. So you can just detect the slip of lidar odometry and then from start using incremental measurements from vio and then if you want use SLAM system, just feed this to LAMP

@Cristian-wp - have you looked at how well Vins-Mono runs? By itself is it tracking the motion correctly in the areas where raw LOCUS is not?
That will be helpful to get an idea of how much we can expect the VIO to assist.
From what @femust is saying just above - that would be the way to get an observability parameter (I believe that is the same method @yunzc has implemented). If you read this metric, I would test if it gives you a good signal when you are in the challenging parts of the tunnel (i.e. tells you that your observability is low).
You could switch the input to LAMP based on this signal (e.g. switch from LOCUS to VIO) - however you have to be careful when doing this switching not to introduce incorrect jumps. See this paper for details on how we have done this in the past: https://arxiv.org/pdf/2008.09679.pdf

@BenjaminMorrell VINS runs well in our datasets were the LOCUS fail. @femust thanks a lot for your reply, I will try in these day to implement your solutions.
I let you know my progress and problems :)


created a new object to hold results

Hey, I created a new object to hold the results that were returned based on a TODO I found in the code. It allows for some of the results operations to be pulled out of the fetcher.lib and put into a Result class. This class can currently generate a pandas dataframe and write to a csv file. I feel like there is a lot of opportunity to expand/refactor this object but it was mainly just copy and pasted from the fetcher.lib for now until I checked to see if it's something you are interested in moving forward with.

not sure why flake8 is complaining about fetcher/extras/backfill.py. I initially didn't change that file. After there error I modified the docstring trying to fix the flake8 error with no success. Locally flake8 does not complain about this file.

not sure why flake8 is complaining about fetcher/extras/backfill.py. I initially didn't change that file. After there error I modified the docstring trying to fix the flake8 error with no success. Locally flake8 does not complain about this file.

I pushed backfill.py last night accidentally. Shouldn't have

Hi, thanks!
If you don't mind to wait a bit longer, I'll check this tomorrow morning

I was actually thinking about this some more. It might even make sense to create a directory under fetcher called release. This folder could hold the release object and things like push_to_sheets or other functions.

Re: push_to_spreadsheet it's used for other files too right now, not just the fetches, so tools fits better for now. I have a couple of other spreadsheet helpers that will go with it soon.
I think we can definitely extract all the DataFrame handling into its own class.
The class can be simplified and it should encapsulate all the options regarding the data frame.
Specifically:

It doesn't need to extend dict, and the aggregation of items can happen at append time
All the configuration that's passed to write_to_csv and get_dataframe can be part of the class
Currently, because of the structure of write_to_csv and get_dataframe the dataframe is built twice, once for each method. This should be compacted.

I want to introduce a small config change that can make part of the current control flow and parameters redundant.

Thanks for the feedback that makes perfect sense.

@space-buzzer here is my latest based on your feedback. Let me know what you think. A couple of items to note

When you append to the Result object it adds the data to the internal dictionary and sets a flag that the dataframe needs to be regenerated. The thought here is reading happens much less than writing so only generate the dataframe if it isn't already generated when the dataframe property is accessed.
-In the fetcher I changed the method signature of the fetch_all method to fetch_states and added a states parameter. The thought here was that it made one main entry point into the fetcher to get states instead of two. I also marked fetch state as private since you can go through fetch_states and just pass a state now. This made it easier to maintain the results in one place of the fetcher.

Let me know what your thoughts are.

@space-buzzer hey I noticed several changes that had been made that put this PR in conflict now. Would you like me to rework this to incorporate your changes to lib.py or do you want to just leave the results implementation in lib.py the way it is now. Either way works for me just trying to figure out where I could help.

This PR is in conflict now. Closing.


Error on already removed object patch

Expected Behavior
When I create and quickly remove object, that's an odd behavior, but I expect that kopf will handle it as non-error.
Actual Behavior
Insetad, I get errors in log and in error tracker:
ApiException: (404)
Reason: Not Found
HTTP response headers: HTTPHeaderDict({'Cache-Control': 'no-store', 'Content-Type': 'application/json', 'Date': 'Wed, 10 Jul 2019 10:03:20 GMT', 'Content-Length': '222'})
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"increments.tutu.ru \"inc-e5c1\" not found","reason":"NotFound","details":{"name":"inc-e5c1","group":"tutu.ru","kind":"increments"},"code":404}

  File "kopf/reactor/queueing.py", line 145, in worker
    await handler(event=event)
  File "kopf/reactor/handling.py", line 139, in custom_object_handler
    await patching.patch_obj(resource=resource, patch=patch, body=body)
  File "kopf/clients/patching.py", line 41, in patch_obj
    await loop.run_in_executor(config.WorkersConfig.get_syn_executor(), functools.partial(patch_func, **request_kwargs))
  File "concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "kubernetes/client/apis/custom_objects_api.py", line 1951, in patch_namespaced_custom_object
    (data) = self.patch_namespaced_custom_object_with_http_info(group, version, namespace, plural, name, body, **kwargs)
  File "kubernetes/client/apis/custom_objects_api.py", line 2057, in patch_namespaced_custom_object_with_http_info
    collection_formats=collection_formats)
  File "kubernetes/client/api_client.py", line 334, in call_api
    _return_http_data_only, collection_formats, _preload_content, _request_timeout)
  File "kubernetes/client/api_client.py", line 168, in __call_api
    _request_timeout=_request_timeout)
  File "kubernetes/client/api_client.py", line 393, in request
    body=body)
  File "kubernetes/client/rest.py", line 286, in PATCH
    body=body)
  File "kubernetes/client/rest.py", line 222, in request
    raise ApiException(http_resp=r)

I'm not sure what behavior will be correct in such situation, but maybe passing such situations will be ok?
except ApiException as e:
    if e.reason != 'NotFound':
        raise

Steps to Reproduce the Problem

Empty handler with enough sleep on create
Create and remove object faster than sleep

Specifications
kopf==0.17

@dbazhal Thanks for reporting.
Side-notes: it was introduced in #24 — when the finalizers were made optional in case there are no deletion handlers. Now, the k8s-side deletion process does not wait for the operator's "approval" (i.e. removing its finalizers). Kopf should be ready for a sudden disappearance of the individual resources during the handling cycle (especially when the handling is not instant). Though, only when a DELETED event is received on the watch-stream (presumably, it is always the latest event for each resource); otherwise, it should continue treating such cases as regular errors.

Hi, perhaps the issue I'm facing during a delete is related to this one here:
[2019-07-30 21:46:53,387] kopf.reactor.queuein [ERROR   ] functools.partial(<function custom_object_handler at 0x10a0c1a60>, lifecycle=<function asap at 0x10a884510>, registry=<kopf.reactor.registries.GlobalRegistry object at 0x109c2c5c0>, resource=Resource(group='vsphere.vmware.com', version='v1alpha1', plural='vmgroups'), event_queue=<Queue at 0x10bac1550 maxsize=0 _getters[1]>, freeze=<asyncio.locks.Event object at 0x10a0e1400 [unset]>) failed with an exception. Ignoring the event.
Traceback (most recent call last):
  File "/Users/mgasch/GO/src/github.com/embano1/kopf-operator-vmworld/lib/python3.7/site-packages/pykube/http.py", line 241, in raise_for_status
    resp.raise_for_status()
  File "/Users/mgasch/GO/src/github.com/embano1/kopf-operator-vmworld/lib/python3.7/site-packages/requests/models.py", line 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://localhost:59982/apis/vsphere.vmware.com/v1alpha1/namespaces/default/vmgroups/kopf-vmworld-example-1

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/mgasch/GO/src/github.com/embano1/kopf-operator-vmworld/lib/python3.7/site-packages/kopf/reactor/queueing.py", line 148, in worker
    await handler(event=event)
  File "/Users/mgasch/GO/src/github.com/embano1/kopf-operator-vmworld/lib/python3.7/site-packages/kopf/reactor/handling.py", line 131, in custom_object_handler
    await patching.patch_obj(resource=resource, patch=patch, body=body)
  File "/Users/mgasch/GO/src/github.com/embano1/kopf-operator-vmworld/lib/python3.7/site-packages/kopf/clients/patching.py", line 35, in patch_obj
    await loop.run_in_executor(config.WorkersConfig.get_syn_executor(), obj.patch, patch)
  File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/Users/mgasch/GO/src/github.com/embano1/kopf-operator-vmworld/lib/python3.7/site-packages/pykube/objects.py", line 142, in patch
    self.api.raise_for_status(r)
  File "/Users/mgasch/GO/src/github.com/embano1/kopf-operator-vmworld/lib/python3.7/site-packages/pykube/http.py", line 248, in raise_for_status
    raise HTTPError(resp.status_code, payload["message"])
pykube.exceptions.HTTPError: vmgroups.vsphere.vmware.com "kopf-vmworld-example-1" not found
[2019-07-30 21:46:53,496] kopf.objects         [DEBUG   ] [default/kopf-vmworld-example-1] Invoking handler 'vm_operator'.
deleting vm group "kopf-vmworld-example-1"
[2019-07-30 21:47:02,621] kopf.objects         [INFO    ] [default/kopf-vmworld-example-1] Handler 'vm_operator' succeeded.

Note: I am using @kopf.on.event raw event handling and switching between event types with:
event_type = event['type']
    if event_type == "DELETED":

This should be OK imho...


Event Position Adding

The problem of adding a position to an already created event still causes the center positions to be wiped. Also, the ability to press enter to add a position isn't working right. Just by typing a single letter in it add that as a position.

Fixed by be3bf1e.


Added link for AWS Community Day in Turkey

1. Issue, if available:
2. Description of changes:
3. How was this change tested?
4. Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: link to issue
[ ] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

maybe we should put these on a page in the website soon? The list is growing!


pretrained weights

trained with LogoDet-3K.

Hi,
To train with LogoDet-3K, I need to have higher specifications. Using my personal computer, it would take too much time. So, I went ahead and shared the training codes here.


Add missing Entity Type definitions (supported by Amundsen)

I do not know the type definitions so please create PR and add the types you want here: typedef-ingestion-source.json and the rest will be done during atlas init by atlas-setup.sh

Fixed after merging this one: https://github.com/ing-bank/rokku-dev-apache-atlas/pull/20


Helper line indentation is wrong

monaco-editor version: 0.16.1
Browser: Chrome
OS: Windows 10
Steps or JS usage snippet reproducing the issue:
monaco.languages.typescript.javascriptDefaults.setDiagnosticsOptions({
    noSemanticValidation: true,
    noSyntaxValidation: false
});

// compiler options
monaco.languages.typescript.javascriptDefaults.setCompilerOptions({
    target: monaco.languages.typescript.ScriptTarget.ES6,
    allowNonTsExtensions: true
});

scriptEditor = monaco.editor.create($($element).find('#scriptEditor')[0], {
    value: $scope.operation.testCode,
    fontFamily: 'monospace',
    fontSize: '13px',
    theme: "vs-dark",
    language: "javascript"
});

Screenshot:

There was a css rule that was messing with monaco, even though I had specified a font explicitly. When I removed the following, indentation worked fine:
* { font-family: 'Noto Sans', sans-serif; }

I had to replace it with:
html, body { font-family: 'Noto Sans', sans-serif; }

Calling monaco.editor.remeasureFonts() did not help.
I hope this helps someone.


Issue when copying from a file share locally onto another file share

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
10.0.1
Which platform are you using? (ex: Windows, Mac, Linux)
Linux
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
./azcopy_linux_amd64 cp 'https://<fileshare:.file.core.windows.net//?<redacted_sas>&sharesnapshot=' '<different_azure_file_share>' --recursive=true
What problem was encountered?
000 : File Creation Error operation not supported
How can we reproduce the problem in the simplest way?
If I change the path from the CIFS share to the local directory the command works (i.e. saving onto the VM itself). But if I make the path something on a new empty windows cifs fileshare (i.e. restoring my snapshot onto a fresh share) I get errors
Have you found a mitigation/solution?
No. But I believe it to be related to allocating memory for the file here https://github.com/Azure/azure-storage-azcopy/blob/500f2eec4b658a5294eef523b5802b19ba80f5b7/common/mmf_linux.go#L99

Hi, @wilsonge
Thanks for reaching us and report the issue.
This issue specifically happens in linux systems, where fallocate is not supported by cifs/smb3. We'll provide a fix to this issue, and will update this issue when related fix is released.
Thanks,
Jiachen

Awesome. Thanks!

Thanks! Sorry I've been taken off the part that involves this right now. But will try when I'm next able to and let you know!


session cookie not properly cleared in del/destroy

in https://github.com/npm/redsess/blob/master/redsess.js#L62
session cookie is set using
var copt = util._extend({
expires: expireDate,
signed: !!this.keys
}, cookieOptions)
in  https://github.com/npm/redsess/blob/master/redsess.js#L117
session cookie is cleared by
this.cookies.set(this.cookieName, '', {
expires: new Date(0),
signed: !!this.keys
})
if I have domain, path, secure etc in cookieOptions, it's not properly cleared after destroy

It looks like it's properly "cleared" as far as the browser is concerned -- by setting the response header.
However, doing cookies.get() again will grab the first one set in the request header.
Let's say I send a request header: Cookie: foo:bar
cookies.get('foo') // "bar"
cookies.set('foo', 'something'); // res._headers['Set-Cookie'] = [..., ..., "foo=something; ..."]
cookies.get('foo') // "bar" :(

I think pillarjs/cookies package only relies on request cookies when doing get, making sets useless in the same request context:
https://github.com/pillarjs/cookies/blob/master/lib/cookies.js#L26-L29


Selecting label files for Waymo dataset

Hello,
After converting Waymo data to KITTI format, there are image_* and label_* folders. When using this format for training a network configured for kitti, does the image_2 folder correspond only with the label_2 folder? Should the label_all folder not be used in this case?
I am trying to train PointRCNN https://github.com/sshaoshuai/PointRCNN using the Waymo data converted to KITTI but I think there is a label mapping error since the losses are very noisy and not reducing at all

Image_x should only correspond with label_x folder, and label_all contains all the surrounding boxes.
The main difference between KITTI and Waymo is that we only make predictions in the frontal view on KITTI. If you want to get a good performance on Waymo, you need to change the detection range in the corresponding model config.
For your case, I guess there maybe some inconsistency in terms of the generated info between our codebase and the implementation of PointRCNN. Have you ever tried to use the infos generated from mmdet3d to train the PointRCNN on KITTI?

The implementation of PointRCNN I am using does not use info files and instead loads the files directly. I have changed the point cloud ranges accordingly, but still am running into trouble. The OpenPCDet implementation seems to generate infos differently, since attempting to use these generated infos in the place of kitti infos results in key errors.
Alternatively, generating the infos from the Waymo-KITTI converted files using the OpenPCDet create_data function results in similar noisy loss curves that do not converge.
Any help would be appreciated!!

Actually, there is no guarantee that any model can be smoothly transferred to new settings and datasets, especially for LiDAR-based 3D detection. Both the pointpillars and SECOND provided by our codebase also need much finetuning. I think you may need to find some experiences about tuning PointRCNN on Waymo, or at least on new datasets, like how to set the learning rate/batch size/anchors, etc.

Yes, have been experimenting with those parameters. I will update here in case I have any luck.

Update: https://github.com/cxy1997/3D_adapt_auto_driving provides a useful format conversion method.


cleanup(tf): #0 remove unused options

Non of these options appears to be used anywhere, anymore.

out of date


add reset-submodule script

So when switching branch/tag, we can cleanup workspace conveniently.

does the script have the executive mode?

Yes.

LGTM

PTAL @BusyJay

PTAL @overvenus


for scenes/actors/transform/scale insert menu shows 3 samples

I think it should be one sample.

Fixed

Fixed


Closes #2638: Copy Button for Secrets

Fixes: #2638
Added a copy button after unlocking secret.
As mentioned in #2638, the clipboard cannot be modified while running an asynchronous operation (like AJAX call) since the user-triggered event would have been processed before the response is received.
There are hacky solutions, but I'd rather keep things simple. There is a draft API that addresses this challenge, but its support is not wide enough, yet.
Note: I have purposefully avoided adding the copy button to the secret_edit.html page as I see it as unnecessary (you wouldn't normally go to edit to unlock then copy the secret). However, if you'd like it there for consistency, you can copy the button from secret.html (that's the only change needed).

Looks like the changes from #2849 got caught in this PR too. Can you separate the origin branches so we avoid conflating the two in a single PR please?

Oh sorry. My bad. I'll create a new pull request and reference it here.

#2860 created with a fixed origin.


Upgrade pinecone-client to v2?

@rileytomasek is there a reason it's still on v1? Happy to create a PR.

@transitive-bullshit nope, it should be on v2. The v2 release was mostly just upgrading min Node version and dropping CJS support so it should be straightforward.

Fixed in https://github.com/dexaai/dexter/pull/7


required doesn't work in iOS Safari

Versions:

VueJs: 2
Vee-Validate: Most recent as of Sept. 6

Description:
The "required" rule does not work in Safari browser in iOS. Other browsers work great.
Steps To Reproduce:
Navigate to main examples page at http://vee-validate.logaretm.com and try the first example with the Email field. The "required" rule is ignored.

In my own application the required rule works fine under iOS Safari, I regularly test that. The required examples on the website also work fine with Safari, I just tested that too. So, not sure what you're experiencing. Can you explain exactly what you do, or post a screenshot or something?

This is what I see in SafarI:
https://i.gyazo.com/7663f194a0cd9f2bab9aad34cdd0c9f9.png
Essentially, it's as if the required directive is not even there.

Both on iOS and macOS I get the correct error for that example, it says "The email field is required.". I am using Safari 10.1.2 though, you're on Safari 9.1.1. Can you replicate the issue with that browser if you recreate the example independently? It may be a problem with the website itself, because the menubar on the left is also acting weird for me on Safari. Maybe it's a faulty polyfill or other library interfering or something, not sure.

I am facing he same problems here on an iPad with Safari 601.1 and Chrome 60.0.3112.89

I face this issue as well, the browser says the field is required as soon as the focus leave the field, even if it's already filled. Only on my old iPad2 and iOS9 on both Safari  and Chrome.

Well this is pretty hard to debug for me, I will try to get hold of an ipad to test this. any help debugging is welcome.

I'm unable to reproduce this issue on:

Safari iPhone6s
Safari 11.0 MacOS 10.12.6
Safari iOs 9.0 iPad-air-2 (simulator)

What are the exact steps to reproduce this? I tried on the first example of http://vee-validate.logaretm.com by placing the input in focus and then inserting some text. After removing all the text the error returns to The email field is required. as expected

It works just fine for me too on

MacOS 10.12.6 Safari 11.0
iPad 3 iOS 9.3.5 Safari

So I think this problem just happens on old browsers like Safari 9.1 and bellow.


Enable code coverage reporting for integration tests

Closes: https://github.com/onflow/cadence-tools/issues/94
Depends on: https://github.com/onflow/flow-emulator/pull/332
Description
Inject a CoverageReport object to the underlying EmulatorBackend's Blockchain instance, in order to generate coverage report/metrics for integration tests.

[x] Targeted PR against master branch
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[x] Code follows the standards mentioned here
[x] Updated relevant documentation
[x] Re-reviewed Files changed in the Github PR explorer
[x] Added appropriate labels

I have updated the necessary versions, and fixed the breaking changes that occurred. Please have a look, as I can't say I am very familiar with it :pray:


Unable to start new assessment CSET Stand Alone <IP_ADDRESS>

🐛 Bug Report
Unable to start a new assessment in version <IP_ADDRESS>.
Tried using multiple browsers and all had same results.  Screen would flicker but would return to the selection screen with Start New Assessment or Import Assessment.  Import Assessment button does open the import dialog.
To Reproduce
Steps to reproduce the behavior:

Start Stand Alone CSET <IP_ADDRESS>.
Open browser (Edge, IE, Chrome, Firefox)
Navigate to local CSET website
Select Start New Assessment button

Expected behavior
Expect new assessment dialog to open.
Any helpful log output
Paste the results here:

I concur. Latest version is broken. Who tested this before doing a "release"?

Ditto.  The error has been posted by others.

Same here as well.  I have reverted back to the original 10.0 and no issues at all so far.  I will test other versions, to see which ones are affected

Same here as well.  I have reverted back to the original 10.0 and no issues at all so far.  I will test other versions, to see which ones are affected


Welch PSD Example Problem

Hello Pedro,
When I try to run the frequency domain Welch PSD example with SampleECG.txt file I got a different plot figure

Example code:
`
import biosppy
import pyhrv.frequency_domain as fd
from opensignalsreader import OpenSignalsReader
signal = OpenSignalsReader('SampleECG.txt').signal('ECG')
rpeaks = biosppy.signals.ecg.ecg(signal)[2]
result = fd.welch_psd(rpeaks=rpeaks)`
When the result should be:

Hi @tgterra,
The issue is due to the current SampleECG.txt containing a signal of too short duration for a proper PSD computation. It might happen that I have replaced the SampleECG.txt over the last months and didn't update the output results in the pyHRV documentation.
The new update will come with a feature load sample NNI series of 5min


Parse macro conditional for modifiers and skip that line if the modifier doesn't apply

Example for sequence:
/cast [nomod] firestorm
/cast [nomod] facebop
/cast [mod:alt] ubersmash
If user is holding alt down this sequence will execute ubersmash in one click.
If user is not holding alt down, this sequence will execute firestorm on click 1, facebop on click 2, and firestorm on click 3

Can’t be done in the way you are thinking. GSE can’t tell if the alt key is pressed or not.  It can only pass the line onto WoW for it to interpret.
TimothyLuke

On 10 Jan 2021, at 2:41 pm, Furyswipes<EMAIL_ADDRESS>wrote:

Example for sequence:
/cast [nomod] firestorm
/cast [nomod] facebop
/cast [mod:alt] ubersmash
If user is holding alt down this sequence will execute ubersmash in one click.
If user is not holding alt down, this sequence will execute firestorm on click 1, facebop on click 2, and firestorm on click 3
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Can’t be done in the way you are thinking. GSE can’t tell if the alt key is pressed or not.  It can only pass the line onto WoW for it to interpret.
TimothyLuke

On 10 Jan 2021, at 2:41 pm, Furyswipes<EMAIL_ADDRESS>wrote:

Example for sequence:
/cast [nomod] firestorm
/cast [nomod] facebop
/cast [mod:alt] ubersmash
If user is holding alt down this sequence will execute ubersmash in one click.
If user is not holding alt down, this sequence will execute firestorm on click 1, facebop on click 2, and firestorm on click 3
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

What you can do instead is
cast [mod:alt] ubersmash; firestorm
cast [mod:alt] ubersmash; facebop

What you can do instead is
cast [mod:alt] ubersmash; firestorm
cast [mod:alt] ubersmash; facebop

Or even better put
cast [mod:alt] ubersmash
In KeyPress and put
/cast firestorm
/cast facebop
in the normal part of the macro.

Or even better put
cast [mod:alt] ubersmash
In KeyPress and put
/cast firestorm
/cast facebop
in the normal part of the macro.

IsAltKeyDown()? IsControlKeyDown()? Not allowed in insecure code?

IsAltKeyDown()? IsControlKeyDown()? Not allowed in insecure code?

I use IsAltKeyDown() in combat--it's not protected. I use it for follow.
function follow()
--This is meant to be in your alt-4 macro, and gets everyone to follow and assist the focus (meant to be your current window toon)
if FSMB_game == "shadow" or FSMB_game=="classic" then
if not IsAltKeyDown() and not IsControlKeyDown() and not IsShiftKeyDown() then
FollowUnit(unitname(FSMB_raidleader))
end
elseif not IAmFocus() and FSMB_raidleader and IsAltKeyDown() then
FollowUnit(unitname(FSMB_raidleader))
end
end

I use IsAltKeyDown() in combat--it's not protected. I use it for follow.
function follow()
--This is meant to be in your alt-4 macro, and gets everyone to follow and assist the focus (meant to be your current window toon)
if FSMB_game == "shadow" or FSMB_game=="classic" then
if not IsAltKeyDown() and not IsControlKeyDown() and not IsShiftKeyDown() then
FollowUnit(unitname(FSMB_raidleader))
end
elseif not IAmFocus() and FSMB_raidleader and IsAltKeyDown() then
FollowUnit(unitname(FSMB_raidleader))
end
end

But the way GSE is currently architected it only knows about the current line when you click.  It doesn’t know that 4 clicks later there is an alt line

But the way GSE is currently architected it only knows about the current line when you click.  It doesn’t know that 4 clicks later there is an alt line


Update README.md

adding -d ( -d, --detach  Detached mode: Run containers in the background) to execute next command in the same console.

@girardengo
I prefer to avoid usage of -d parameters in a non-production behaviour. This makes that closing the console you close the app and this is easier for newbies.
For dev purposes, it's better to keep a volatile instance of docker and a second console for interacting with container. This is also a good option to continue watching logs.
You can use the tab on vscode for switching between the docker-compose stream and interactive console


Add help button to stan editor - open user guide

Close #192

Looks good to me. Only thing to consider is making the help button the same blue color as the other icons in the toolbar--wasn't sure if the difference was intentional (to make it stand out) or not.

I was just being consistent with the help buttons on the data script editors. If others have a preference it doesn't matter to me.

I think we can consider colors as part of #186, for now I agree matching the others is good


Please update the settings file, it kills a lot of time

in the readme file,
'rest_framework_jwt.authentication.JSONWebAuthentication',
will be now
'rest_framework_jwt.authentication.JSONWebTokenAuthentication', .
I have spent a lot of time figuring out why the project was showing errors.

I found that it was already in read me


Make the config of $VIMSIDIAN_PATH_PATTERN optional

Make the setting of $VIMSIDIAN_PATH_PATTERN optional. The timing of executing commands and mappings is left to the user.
In order to avoid mapping duplication, we had an example in README.md that only mapped when $VIMSIDIAN_PATH_PATTERN was used. However, this was difficult for users to understand and also affected performance.
For example, if you only want the mapping to be enabled for markdown files, you can write the mapping in ~/.vim/ftplugin/markdown.vim, and as a more specific option, we provide $VIMSIDIAN_PATH_PATTERN to users.
related issue:

https://github.com/kis9a/vimsidian/issues/4

done


Block for conditional field is always called

Example
relationship :user, unless: -> { @object.user.nil? } do
  link :related do
    @url_helpers.v1_user_path @object.user
  end
end

Expected outcome
The relationship block is not called when the conditional fails.
Actual outcome
The relationship block is called regardless of the conditional failing; only the display of the serialized relationship is changed. This requires additional checks within the block, which makes the code less readable (especially for nested resource routes) and duplicates the condition:
relationship :user, unless: -> { @object.user.nil? } do
  link :related do
    @url_helpers.v1_user_path @object.user if @object.user.present?
  end
end

Solution
Do not call the block if the condition fails.

Would you be open to a pull request?

@ezekg You are right – at the moment links are built eagerly (only the related objects are constructed lazily). However, a simple fix for your use-case (that would be in-line with the spirit of the spec) would be to expose the related user via some nested route (like /posts/3/user), rather than the canonical resource url (/users/:id).
Regarding the former, I am currently refactoring that part to avoid eagerly building stuff, so it should be fixed for the next release.

@ezekg See https://github.com/json-api/json-api/issues/1134 and related issues for an overview of why exposing the related link as /posts/3/user is the most flexible solution.

As much as I'd like to implement that, at the moment it brings in too much overhead into developing the API. I'd need to create new routes/controllers for each relationship, and write tests for them.
Appreciate the guidance though. I'm going to create a ticket as a reminder to come back to this.

@ezekg Would you mind writing a regression test for this?

Definitely. I'll try and do that this weekend.

Out of curiosity, does the library currently support specifying any conditional rendering of attributes in the serializer via the exposed instance variables?
For example:
class SerializableBar < JSONAPI::Serializable::Resource
  type 'bars'

  attribute :always_show_me
  attribute :conditionally_show_me, unless: -> { @authenticated }
end

This requested feature was the closest I could find to this and just wanted to see if I was missing anything obvious that already existed in the library.

@dylanlewis89 Yes it does, you just have to call extend JSONAPI::Serializable::Resource::KeyFormat in your serializer in order to enable it.

@dylanlewis89 Sorry, I obviously meant JSONAPI::Serializable::Resource::ConditionalFields rather than JSONAPI::Serializable::Resource::KeyFormat.

Thanks @beauby for the quick and helpful response!
For anyone tackling this issue in the future, the 0.2.1 release introduces the extend JSONAPI::Serializable::Resource::ConditionalFields syntax. Earlier versions use prepend instead of extend

@ezekg So if you're still interested in fixing this, it's just a matter of delaying the instance_evaling here until either as_jsonapi or related_resources are called.

Hey, I just updated my API to use the latest version of jsonapi-rb/rails so I can dig into fixing this now! I was trying to hold off until I could get around to updating, as I was still on v0.1.1. 🙂


[notification-hubs] Fix FCMV1 bugs for notification creation and list registrations

Packages impacted by this PR

@azure/notification-hubs

Issues associated with this PR

#29404
#29371
#29372

Describe the problem that is addressed by this PR

Fixes the issue with creating a wrapper around the entire FCM V1 notification body to include message.
Changes the data types of the data properties from Record<string, any> to Record<string, string>.
Fixes the listRegistrationsByChannel to include fcmv1 as a parameter to query for devices.

What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Added unit tests for the helper for listRegistrationsByChannel to validate the query string returned.
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[x] Added a changelog (if necessary)

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/notification-hubs


Allow change of file permission

When running Kibana as a different user the configuration file alos
need to be owned by that user.

Pull request acceptance prerequisites:

[ ] Signed the CLA (if not already signed)
[ ] Rebased/up-to-date with base branch
[ ] Updated CHANGELOG.md with patch notes (if necessary)
[ ] Any relevant docs (README.markdown or inline documentation) updated (if necessary)
[ ] Updated CONTRIBUTORS (if you would like attribution)
[ ] Tests pass CI

made in https://github.com/voxpupuli/puppet-kibana/pull/64
this PR could be closed

Thanks for the head up @phaedriel


[Vulnerability][DOS] WitchServer panic with crafted packet

DOS attack with crafted packet
You don't check the HTTP request validity that cause a Denial Of Service vulnerability with crafted packet. You should check the array length before using the second element.
To Reproduce
Steps to reproduce the behavior:

Use the following python code on your WitchServer

Expected behavior
The WitchServer will crash with a panic (runtime error).
Send crafted packet using python script:
from urllib.request import urlopen
from socket import socket, setdefaulttimeout, AF_INET, SOCK_STREAM

response = urlopen("http://localhost:8000/", timeout=2) # check server is up
response.status                                         # 200
print(response.read().decode())                         # <title>Witch -> Cant Find file</title><img style='border-top: groove;' src='https://i.imgur.com/GJIUMVC.png'

setdefaulttimeout(2)
address = ("localhost", 8000)

sock = socket(AF_INET, SOCK_STREAM)
sock.connect(address)

sock.send(b"GET\r\n")
response = sock.recv(1024)
response                                                # b''

sock.close()

response = urlopen("http://localhost:8000/", timeout=2) # check server is down

Output:
# Traceback (most recent call last):
#     ...
# ConnectionRefusedError: [Errno 111] Connection refused
# Traceback (most recent call last):
#     ...
# urllib.error.URLError: <urlopen error [Errno 111] Connection refused>

WitchServer output:
# ~# docker run --rm -it -p 8000:8000 mauricelambert/witch:latest bash
#  _       ____________________  __
# | |     / /  _/_  __/ ____/ / / /
# | | /| / // /  / / / /   / /_/ /
# | |/ |/ // /  / / / /___/ __  /
# |__/|__/___/ /_/  /____/_/ /_/

# Witch Started
# Now Listening For new Connections On Port 8000
# Acccess The Server by navigateing to http://localhost:8000 In Your Web Browser
# No certificate was loaded
# ----------------------------------------------------------------------------
# A Web Web Browser has connected -> <IP_ADDRESS>:36596
# No Index.html File Found
# A Web Web Browser has connected -> <IP_ADDRESS>:36598
# No Index.html File Found
# A Web Web Browser has connected -> <IP_ADDRESS>:36600
# No Index.html File Found
# A Web Web Browser has connected -> <IP_ADDRESS>:36602
# No Index.html File Found
# A Web Web Browser has connected -> <IP_ADDRESS>:36604
# panic: runtime error: index out of range [1] with length 1

# goroutine 10 [running]:
# main.handleConnection({0x6a1188?, 0xc0000100b8})
#         /go/WitchServer/listener.go:46 +0xa07
# created by main.main
#         /go/WitchServer/witch.go:147 +0x8c7

This is a important issue. I going to start work on it now

Fixed


How can I use eddymotion for my own dMRI data?

What would you like changed/added and why?
How can I use eddymotion to analyze my own dMRI data? Is there a pipeline, a tutorial or example? Please help me!
Do you have any interest in helping improve the documentation?
Yes
Do you have any suggestions for the new documents?
No response

In addition to #169, there's nothing we can do today. Hopefully, with the new energy of the past few months, a version that is amenable to third-party users will be able to see the light. Stay tuned for that!

您的邮件已收到。


Not able to connect to internet from within vscode (terminal/install extentions) under VPN

Issue Type: Bug
The strange thing is that I have no issue with non-vscode environment - chrome/Iterm can access DNS/internet without an issue.
On vscode start I have this in Log (window):
[2021-03-03 22:02:21.146] [renderer5] [error] Network Error: Error: Network Error
    at e.exports (/Applications/Visual Studio Code.app/Contents/Resources/app/node_modules.asar/tas-client-umd/lib/tas-client-umd.js:1:7511)
    at XMLHttpRequest.p.onerror (/Applications/Visual Studio Code.app/Contents/Resources/app/node_modules.asar/tas-client-umd/lib/tas-client-umd.js:1:6478)
[2021-03-03 22:02:21.263] [renderer5] [error] net::ERR_NAME_NOT_RESOLVED: Error: net::ERR_NAME_NOT_RESOLVED
    at SimpleURLLoaderWrapper.<anonymous> (electron/js2c/browser_init.js:109:6508)
    at SimpleURLLoaderWrapper.emit (events.js:315:20)

when do
Developer Tools -> Network -> after execute "Install Extensions" it says "Cannot connect to Extensions marketplace" and there is an error in Developer Tools which shows net::ERR_NAME_NOT_RESOLVED
when do (as example marketplace.visualstudio.com, happens to any other URL)
terminal -> curl -I https://marketplace.visualstudio.com -> curl: (6) Could not resolve host: marketplace.visualstudio.com
when do same by IP
curl -v <IP_ADDRESS> got
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Immediate connect fail for <IP_ADDRESS>: Operation not permitted
* Closing connection 0
curl: (7) Couldn't connect to server

As I said I have no issue with accessing the mentioned FQDN/IP from non-vscode environment, also have no issue accessing any of this list of hostnames from chrome/ non-vscode terminal https://code.visualstudio.com/docs/setup/network.
So I'm a bit lost and don't know how to troubleshoot further.
VS Code version: Code 1.53.2 (622cb03f7e070a9670c94bae1a45d78d7181fbd4, 2021-02-11T11:45:54.515Z)
OS version: Darwin x64 19.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 x 2200)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 3, 2

Memory (System)
16.00GB (0.32GB free)

Process Argv
--crash-reporter-id e9f182b5-69a9-4cab-a789-bfca3e15c5ef

Screen Reader
no

VM
0%

Extensions: none

guys please any thoughts?

@deepak1556 please any guidance with troubleshooting this?

Sorry for the delayed response, couple of things to try

Does the issue happen when you start with code --user-data-dir /tmp/vscode
If above still repros the issue, start with code --user-data-dir /tmp/vscode --log-net-log=/tmp/netlog.json
2.1) Install extensions that shows the host resolver error
2.2) Quit the app and attach the netlog file.

@deepak1556 perfect!

fixed the issue for me, so easy..
So strange cause I was sure I already wiped everything about vsCode and reinstalled it cleanly, but it seems some user data stayed there.
so now after I did rm -rf $HOME/Library/Application\ Support/Code it started to work back again with just code. thanks!


vcpkg ports, add repo semver tags

vcpkg insists on having actual semver repository tags in the upstream.

https://github.com/microsoft/vcpkg/pull/37402

so "v2.0.2" in-stead of "v2.02"

Sure, can do that. I take it the reason is that you want to include plf_queue as a downloadable library within vcpkg?

Fixed in the most recent version, and if you'd like me to adjust it for any of the other projects, let me know.


Fix a typo in SelectionSort for C++

I created https://github.com/hadshirt/sorting-algorithms/pull/89


-transparent

Hi,
I have been using export_fig for a long time and now that I changed to R2017a the transparency option for pdf exporting (-transparent) does not work. Have you seen this already? (it does work for png).
Regards
Alfredo

Perhaps it is important to mention that I am using the last version of export_fig. The old version I used to have did not work with R2017a.

Please paste a minimal reproducible example

Maybe this can be a minimal example of code:
figure(51);
colormap(cool)
surf(peaks)
export_fig('output.eps','-eps','-pdf','-opengl','-m3','-q101','-transparent','-nocrop');

The output still has a grey background in my case. Feel free to tune the parameters.
The only way of making it work is to have set(gcf, 'Color',[1 1 1]); before plots, but I hope the export_fig works with the -transparent parameter. I also found the quality of exported figures are really bad on Matlab 2017*. Hopefully, this can be confirmed and fixed soon with eveyone's insights.

This is apparently due to an internal Matlab bug with the opengl renderer output in the built-in print function (which is used by export_fig). The workaround is to use -painters instead of -opengl.
I added a warning notification about this in export_fig, alerting users when they try to use -transparent together with -opengl.

The issue was indeed caused by an internal Matlab bug, which caused the transparency setting to be ignored when the figure's InvertHardcopy property was 'off' (which it is by default in export_fig, since this solves a lot of other unrelated issues).
The workaround was to automatically imply noinvert (which prevents updating the figure's InvertHardcopy) when -transparent is specified in the export_fig command.

Looks like this broke the transparent option for PNG files. I get a file with 1x1 pixels only. When I comment out line 1100 in export_fig.m, it works fine.

Fixed, sorry about that. Please retest.
-transparent output should now be ok in all formats.


replace flask link

Hi. I couldn't find anything related to any policies on replacing a link (except for https://github.com/sindresorhus/awesome/issues/1063), but before I go through the process of getting my list ready for a PR, I'd like to know if it would be possible to add another Flask list or replace the current list altogether.

My awesome Flask list: https://github.com/mjhea0/awesome-flask
The current list thats listed (https://github.com/humiaozuzu/awesome-flask) has not been updated in over 15 months. I created my list back in Sep 2019 as I was tired of PRs not being addressed. For more, review https://github.com/mjhea0/awesome-flask/issues/3. I update my list regularly.
Also, if it helps with my request, I do maintain another awesome list (https://github.com/mjhea0/awesome-fastapi) that's currently listed, so I do understand the work required to maintain a list.

Thanks!

Sure. Feel free to submit yours as a replacement.


Add fonctionnality to install automatically the plugin via plugin pack info

maybe use this : https://github.com/centreon/centreon/blob/develop/centreon-gorgone/contrib/gorgone_install_plugins.pl


Account for flatpak and snap

I added a second Array APP_SUBPATHS which holds relative paths to the sockets for flatpak and snap installs. Seems you are already on it, but didn't see that until after coding it up. I'm not a professional, so better look over it carefully. See #27

Amazing, this is so much cleaner than my initial implementation I did before submitting the issue. Works for me using Discord on Flatpak, thanks!


Generate page with Table of Contents without content of articles

Is it possible to generate only page with contents but without content of articles to use it as sitemap?
Such feature can be very useful
My suggestion is to add some option like show_content
plugins:
  - print-site:
      add_to_navigation: true
      print_page_title: 'Site map'
      add_table_of_contents: true
      toc_title: 'Table of Contents'
      toc_depth: 6
      add_full_urls: true
      show_content: false

Thank you for your answer.
I also haven't found any plugin that can create only ToC for whole site... RSS plugin is not suitable for it - I want to generate and place ToC on some page

I think the only approach for your specific use-case is to build a customized plugin, see https://www.mkdocs.org/user-guide/plugins/#developing-plugins. Please feel free to re-use any code from this project.
It's not something I want to add to this plugin, so I'm closing the issue.


Add appstream metainfo and .desktop file

Related to #86, Flathub maintainers would like the metainfo and .desktop files to be upstreamed.

Thanks for putting this forward!
Are the .github elements of the filenames necessary for flathub? In theory, there's no github attachment. What does the io part refer to?
Also, could these be moved to a "flatpak" subfolder?

We can name the files whatever we want in this repo. Moving them elsewhere is also not an issue.
To explain the current name - it follows the application ID. The application ID spec is here https://docs.flathub.org/docs/for-app-authors/requirements#control-over-domain-or-repository

Okay, cool! If they don't mind how they're named right now, I don't care.
Thanks for moving them!


[evaluation] ci,tests,fix: Ensure pf service isolation across parallel pytest sessions

Description
This pull request is a follow up to #37411, which attempts to make the cleanup of the promptflow service more robust by isolating the service across test sessions (when run through tox).
Background
Our test suite spawns a separate process (promptflow service) that gets orphaned. This is an issue for our windows tests since that orphaned process holds handles to files that our CI tries to delete, which fails and causes the CI step to fail (See #37411 for more detail).
#37411 provided an initial attempt at a fix. However, the issue persisted intermittently and rerunning the failed job eventually fixes the issue.
The non-determinism suggested that this was potentially a race condition stemming from the fact that our CI runs 3 test sessions concurrently. So this PR attempts to rectify this by ensuring that each session manages its own promptflow service.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

/check-enforcer override


Task status & trello board mapping schema

Add task status enum to replace "completed" boolean.
Add trellomappings table to store mappings for an integration from trello column names to task statuses
Add tokenId column to integrations to reference a token that is to be used during imports using said integration
Add nullable trelloTableId column to integrations to reference a table name that is to be used during imports from trello

Converted taskStatus to be an integer instead, and modified places wherever it was used to accommodate this


ci: Workflows security improvements

Regular update of security scorecard,
tighten workflow token permissions
not persisting git credentials during workflow runs
auto dependancy reviews using https://github.com/actions/dependency-review-action

Thank you in this year also !! :)


More consistent config/deps output in Foxx API (devel)

Having the option to get the same response format from the get/put/patch routes reduces the overhead for tools like foxx-cli. This should be backported to 3.2 and 3.1 after merging.

@jsteemann Can we merge this? The sibling PR for 3.2 has been merged already. I think you wanted to wait for the extended jenkins test results.


Update Import-CsOnlineAudioFile.md

Match file name

Docs Build status updates of commit fd157aa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Import-CsOnlineAudioFile.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@tseward

@tiburd Please merge this PR. Thanks

Docs Build status updates of commit 07ed5a3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Import-CsOnlineAudioFile.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Please Tag for 3.04.00 release

Thanks, Zdenko.

Any news? Can someone say where's the tarballs?

Done. See https://github.com/tesseract-ocr/langdata/releases/tag/3.04.00


doctl: 1.108.0 -> 1.109.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/doctl/versions.
meta.description for doctl is: Command line tool for DigitalOcean services
meta.homepage for doctl is: https://github.com/digitalocean/doctl
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.109.1 in filename of file in /nix/store/2a4jbviadwlq25whcfvjw0cbiw1fb4hi-doctl-1.109.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

doctl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2a4jbviadwlq25whcfvjw0cbiw1fb4hi-doctl-1.109.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A doctl https://github.com/r-ryantm/nixpkgs/archive/09a3539b37b5f72b273096a04fc1b7b6d8e18250.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/09a3539b37b5f72b273096a04fc1b7b6d8e18250#doctl

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2a4jbviadwlq25whcfvjw0cbiw1fb4hi-doctl-1.109.1
ls -la /nix/store/2a4jbviadwlq25whcfvjw0cbiw1fb4hi-doctl-1.109.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

doctl

Maintainer pings
cc @langston-barrett for testing.

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

1 package built:

doctl


Not working again

Again empty results when trying to download pdf files. Looks like it is related to the new Web-UI.

I investigated a bit and it looks like the url changed to "sso_url": "https://www.flatex.at/login.at/sso", and "sso_url": "https://www.flatex.de/login.de/sso",

Updated the URLs but did not verify.


Add Tunnelmole as an open source alternative to ngrok to some more docs

Previously, Tunnelmole was added as an open source alternative to ngrok to the Cognitive Search docs in https://github.com/MicrosoftDocs/azure-docs/pull/117399
This PR adds it to

Various Azure Functions docs
Call automation
Event Grid
Communication Services

@ggailey777 @tophpalmer -
IMPORTANT: This PR has developed merge conflicts that need to be addressed before the PR can be reviewed. When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

PR review team emailed author @ggailey777 today

@tophpalmer @ggailey777
Can you review this old PR and determine whether it needs to be closed or merged?
@MicrosoftDocs/public-repo-pr-review-team


autop() is inserting paragraph tags on spaces when  wrapped in

Initially reported by: @lildude as #TRAC1231
The following pseudo code inserted into a post:
{{{

}}}
... results in the following after it's been through autop():
{{{

}}}
Note the location of the , it's after the first  tag.
Placing a space immediately after the  works around this issue and results in:
{{{

}}}

The actual text of the imported ticket:
The following pseudo code inserted into a post:

... results in the following after it's been through autop():

Note the location of the
, it's after the first  tag.
Placing a space immediately after the  works around this issue and results in:


chore: remove temporary eslint rules

:tada: This PR is included in version 4.9.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Change readme example so it compiles

The example code in the readme didn't compile for me with 0.9.6.

Closing and reopening to see if I can actions to run

Might also be good to automate this by using something like skeptic or doc-comment. Should be pretty easy to do.


feat: Support put.sync to wait the effect to resolve

Add put.sync effect
put.sync is a native effect API of redux-saga. This PR will patch this behavior to the effect of dva.

In order to wait for the returned Promise, use the variant put.sync(...) which will wait until the Promise resolve (or rejects) before resuming.

Use cases

Dispatch other effects in order

const model = {
  namespace: '@user',
  ...,
  effects: {
    *fetchList({ payload }, { call }) {
      const list = yield call(service.fetchList, payload)
      yield put({ type: 'dumpList', payload: list })
    },
    *delete({ payload }, { call }) {
      yield call(service.delete, payload)
    },
    *deleteAndReload({ payload }, { put }) {
      yield put.sync({ type: 'delete', payload })
      yield put({ type: 'fetchList' })
    }
  }
}

Coverage increased (+0.3%) to 70.796% when pulling ed509d85e8a5b3c9710905371b6ea4d89b3d5fa8 on macroxing:feat/support-put-sync into 79d4cfec1640e2b67cbdc13c59075e8793544b20 on dvajs:master.

put.sync is deprecated, using put.resolve instead.

我知道为啥可以了，因为 dva@2 dispatch 一个 effect 时会返回 Promise 。


Feature Request: RecordingRule (POC code inside)

Hello
I want to share simple JUnit rule to handle recording\playback for a test. Here the code.
RecordingWireMock do stubbing if no stub files are found or playback if stub dir isn't empty.
Please feel free to close the issue if my attempt isn't in a line with wiremock strategy.

I think this looks useful, but I'd suggest making it a separate project as I don't think there's need for it to be integral. I'll happily link to it from the website once it's up, if that's the route you want to go down.

I hope it can be helpful. And thanks for the great product!

Can be closed


fix: mapping to the request id

Adding better logging
Adding a tracer request id to the response

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.35%. Comparing base (550342f) to head (26d7534).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
+ Coverage   97.33%   97.35%   +0.02%
==========================================
  Files          17       17
  Lines         789      795       +6
==========================================
+ Hits          768      774       +6
  Misses         21       21

Flag
Coverage Δ

unittests
97.35% <100.00%> (+0.02%)
:arrow_up:

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

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


Microsoft Team's incoming webhook not working if used in Google Cloud Monitoring service

We are having issues sending webhook to MT channel from Google Cloud Monitoring service. The webhook URL works when tested locally on my laptop. So this is not an issue with the webhook either. According to Google Cloud support, it's possible that Microsoft blocked the request from Google Cloud Monitoring that contains a header User-Agent: Google Alerts that might not be recognized or not allowed.

Unable to repro the issue with UA set to “Google Alerts”.
Also, Incoming Webhooks do not have any filtering on User-Agent for Webhook messaging on Connectors side and so that should not be an issue.
We would need the following details from the user to troubleshoot this further:
Failure response content from the connector along with response headers when sending message to the below webhook URL, or Network capture of the failed request made to the webhook URL.
Please note that Incoming Webhooks currently support only the message card format:
https://docs.microsoft.com/en-us/outlook/actionable-messages/message-card-reference

@Nikitha-MSFT as per MT dev Pradeep Anantharaman's response, this might not be possible at the moment:

I see that the webhook URL is set up to receive directly from this external app, and the payload schema from them would not match message card format and this is why it can fail.
The payload schema needs to be of message card format: https://docs.microsoft.com/en-us/outlook/actionable-messages/message-card-reference
So, for solving this, you can set up an intermediate service (webhook API hosted at your end) that transforms the payload from this external app “Google Cloud monitoring” into message card format and then send this payload to Incoming Webhook.
Hope this helps!


String interpolation broken

I'm on v1.23.9 of the C# extension for vscode. And string interpolation shows everything as as string, I'm expecting the stuff part of "{stuff}" to be showed a different color.

This is OmniSharp server repo, of you have a problem with C# extension please open an issue here https://github.com/OmniSharp/omnisharp-vscode and provide the necessary details (OmniSharp log, repro steps and so on). Thanks.


Yaml implementaion

Currently the go commons uses an external dependency for yaml support in codec.
I would like to get away with this dependency with an implementation of yaml Reader and Writer within go commons

Not Planned


Visualization error

Hello, I used your code, but in the image display, I found that it did not achieve the effect presented in your paper. Instead, I had terrible results.
Can you tell me where the problem is?
The picture below is the result after training for 70k steps

Hi, I was wondering whether this is caused by this transformation, mentioned by another user: here. Could you try to set it to all 1 to see?
Basically, make sure that the camera poses are right for the nerf rendering.

Hello author, after I modified the first point you mentioned and checked the camera parameters for nerf rendering.
But I still get the wrong rendering result below

Could you please tell me what else I should check?

Hi, how many camera views do you use?

And could you try to set BC loss to be 0, and check whether only rendering loss could work?

Well，I have set the BCE loss to 0.0001. And I only used one camera. At first, I thought it was a coordinate system mapping issue, but I carefully checked the code to ensure the coordinate systems are consistent, yet the problem still occurs.

By the way I also want to ask why the camera's focus to be negative. For example,-175. My is  75. But I thconvergingink it doesn't matter. And it seems that my rendering loss does not converging

Hi,
When I set _coord_trans  all to 1, I can render some in one scene, but the rendering results in other scenes are still very poor. Can you tell me what else I need to check?


[Bug]: Millibars missing?

What happened?
Millibars are listed as an available unit in the readme, but are not found in the UnitsOfPressure enum.
How to reproduce the bug
Measures::from('100mbar') returns null
Package Version
0.1.8
PHP Version
8.3
Laravel Version
11
Which operating systems does with happen with?
No response
Notes
No response

Hi Ben,
thanks!
I will fix it as soon as possible.

Closed with release 0.1.9


use requests instead of urllib{,2,3} directly where possible

This gets rid of the necessity for most of the 'if PYTHON_VERSION...'
cruft, as well as cleaning up the code around the requests themselves.
I'm certain that there is more tidying to do here, but for now, this
seems to work for all the endpoints I've tried, on both python3.4.3 and
python2.7.9 against my v4 controller.

The requests package is nice, but it's also a new external dependency. Is this fine everywhere unifi-api may be used?

My python experience is fairly limited, so it's hard for me to argue a technical reason for or against. That being said, it's hard to imagine a python installation that didn't cater for even a local install of requests but did allow for this library to be installed.
I mostly did this because I was trying to integrate my Unifi AP into an installation of home-assistant and had difficulty trying to get it to work out of the box. It was as much a matter of personal taste and ignorance as anything else.
Either way, if and when you're happy with this, it might be worth tagging beforehand; or, if you'd rather not take the dependency I can just maintain my own fork.

For whatever it's worth, my changes to home-assistant are here.

I'd vote for requests, myself. Along with the benefits @emu-signal mentioned, here are a few more to convince you!

+1 on the requests implementation. It's becoming the defacto standard for all kinds of web based communications. Very well designed, easy to implement and read after the fact.
I just had to pull config and statistics data from a set of ToughSwitches and even though they lack an official API, requests made it very easy to hook into the existing Ajax end points.
https://github.com/fxstein/SentientHome/blob/master/feed/feed.home.ubnt.toughswitch.py

+1

Ok, lets get the conflicts resolved and this merged. I'm convinced, sorry for dragging my feet.

Awesome!


Remove kernel security path dependecy for kubearmor-controller

Feature Request
Short Description
KubeArmor Controller make use of kernel security path /sys/kernel/security/lsm to detect the enforcer and this dependency can be removed safely now with KubeArmor Operator, as it deploys snitch on the cluster which detects the enforcer and attach this information to node using the labels. KubeArmor Controller can make use of this therefore it's no longer will required kernel security path to be mounted.
Is your feature request related to a problem? Please describe the use case.
It will remove the dependency of a hostpath to be mounted and will improve the securityContext of the deployment.
remove dependency of kernel security path for KubeArmor Controller, get information of the enforcer using:

node labels, or
Add label to the KubeArmor Controller itself with the operator and use that.

hi @rksharma95 as far as looked it up I assume the snitch also uses this dependency to detect enforcer https://github.com/kubearmor/KubeArmor/blob/d350b0e9d269940596af7ec28d12c3408b4754ec/pkg/KubeArmorOperator/cmd/snitch-cmd/main.go#L100 , correct me if I am wrong

@swastik959 yes you're right! Snitch make use of security path to detect the enforcer and adds that information to the node labels.
Now Controller no longer needed access to the kernel security path to get the same information, instead it can look for the enforcer detail using node labels.

Hey @swastik959 , are you still working on this issue?

Hey @swastik959 , are you still working on this issue?

Actually it will be implemented later on you can see the reviews in the pr

Resolved in PR #1335


Moving feed to different folder resets custom name

When moving a feed between folders, your custom set feed name gets reset.
https://user-images.githubusercontent.com/1580388/117591282-728b2900-b0f9-11eb-9658-55f2167b42fc.MP4

@kielgillard If we add the feed name to FeedlyAddFeedToCollectionOperation in the FeedlyAddExistingFeedOperation initializer, do you think it will fix this bug?

@vincode-io I think it will, too. I'll implement, test and submit a PR in a day or so with this fix.

@vincode-io I haven't forgotten just been busy. Within the next week for sure.

Moving this to 6.1 milestone. If it gets in for 6.0, then that’s awesome, but it’s not required.

Moving this to the 6.0 milestone because I have a PR #3139 for the iOS fix.

PR #3140 for the macOS fix.

Thanks for reporting this @petebocken! It should be fixed in the next iOS and macOS releases.


[TASK] Update to PHPUnit 8.5.40

https://github.com/sebastianbergmann/phpunit/releases/tag/8.5.39
https://github.com/sebastianbergmann/phpunit/releases/tag/8.5.40

coverage: 38.622%. remained the same
when pulling 3742f3ff3b0c9215ab5159b853996f0c0271a000 on task/phpunit-update
into 13b077930e4635019d973684bda2c642bb976559 on main.


Be able to specify one or more 'required' constraints for successful application launch

Since the default placement when NO constraints are provided is to place it anywhere, sometimes we want to make sure somebody doesn't do just that because they forgot to provide constraints.
In my case, I have all Mesos slaves tagged with an attribute called service_cluster (but the name doesn't matter -- this is just to explain the use case).  I'd like to be able to reject applications if I've configured by cluster to require at least a service_cluster constraint.
The default behavior of today (no constraint => place anywhere) should remain, but if I set one or more 'required' constraints in my marathon configuration, then I can reject any applications from starting if it was accidentally left off.
This can be used to stop accidental placement when your mesos cluster is running more than one logical environment w/o resorting to OS level isolation.
Thoughts?

Hi @bacoboy,
in the upcoming 0.9.0 release we will include code that will allow you to do something similar with Mesos roles.
You can assign a role to some of your Mesos resources (e.g. service_cluster) and make sure that these resources are not used by default.
Look at --default_accepted_resource_roles in the command line config and "acceptedResourceRoles" in the REST API.
I still think that we might want something like you are suggesting for constraints. We have discussed something like this but haven't come up with a intuitive approach yet.

My idea doesn't require any mesos integration.  You just reject the application POST/PUT if missing one or all of the constraints specified in the marathon configuration that are deemed 'required'.  Since the default would skip this, the change would be backward compatible with existing functionality.
If you give me a pointer into the code where the requests are processed, I can attempt to make a PR.

Hi @bacoboy,
can you outline the exact syntax of the configuration in mind with examples of app definitions which are rejected/accepted? I guess most of the work is actually in that. Thanks!

Is this what you mean?

Add an optional command line argument --required-constraints=key[,key]*.

If BOTH service_cluster AND environment were required the command line argument would be:
--required-constraints=service_cluster,environment

If only environment was required the command line argument would be:
--required-constraints=environment

For packages, I could have a file /etc/marathon/required-constraints which contains these values:
% cat /etc/marathon/required-constraints
service_cluster,environment

Reject API POST/PUT if required constraint is missing with a 400 (bad request) and a json payload saying like Missing constraint: service_cluster

By example, let's say I used the command line argument --required-constraint=service_cluster then this request would return success:
	{
	    "constraints": [
	        [
	            "service_cluster",
	            "CLUSTER",
	            "test-cluster"
	        ],
	        ["hostname", "GROUP_BY"]
	    ],
	    "container": {
	        "docker": {
	            "image": "nginx:latest",
	            "network": "BRIDGE",
	            "portMappings": [
	                {
	                    "containerPort": 80,
	                    "protocol": "tcp"
	                }
	            ]
	        },
	        "type": "DOCKER"
	    },
	    "cpus": 0.5,
	    "id": "/nginx",
	    "instances": 1,
	    "mem": 512
	}

This request would fail:
	{
	    "constraints": [
	        ["hostname", "GROUP_BY"]
	    ],
	    "container": {
	        "docker": {
	            "image": "nginx:latest",
	            "network": "BRIDGE",
	            "portMappings": [
	                {
	                    "containerPort": 80,
	                    "protocol": "tcp"
	                }
	            ]
	        },
	        "type": "DOCKER"
	    },
	    "cpus": 0.5,
	    "id": "/nginx",
	    "instances": 1,
	    "mem": 512
	}

If --required-constraints is not passed, then normal placement occurs.

Thanks for your clarification! I thought about doing something similar with "policies". I would love to hear your feedback about this.
That is the basic idea (read this first): https://gist.github.com/kolloch/0d435af5e68a4febcb42
And this is an expanded/improved version: https://gist.github.com/kolloch/7705593539ae8a675fae

Are these policies meant to replace or sit along side constraints?

They would sit along constraints. So basically, they in effect only add constraints.

You are absolutely correct in that all slave nodes are not created equal and having something to manage those differences is important.
I like the ability in mesos to apply arbitrary tags to servers and use constraints in marathon (positive or negative) to decide where things should or shouldn't run.  But your policy approach shifts the decision making from the user/launcher to a server-side decision engine.  In effect, you apply constraints when none are given via the policies.  I for one don't like this approach, but I'm sure there are many people who would.
For me, my launcher already has to decide what to launch, where (multiple mesos clusters), how big (ram/cpu), etc so adding the constraints to further specify additional constraints is a no brainer.  For simpler environments, I can see the lure of using policy to apply constraints where none are give, I'm not debating that.
Case in point, while people are testing, they often launch manually and forget to add constraints.  It could wind up everywhere and mess things up.  Your policy system would pick a default location and that might be fine for your environment.
However, my request is to say the opposite.  I'm saying there is no default.  If you forget to add it, your launch should fail.  This idea can co-exist with policies.  You can have a policy to make sure your app lands on a non-ssd box by default, but you MUST tell me the environment (production, staging, etc).
Ideally, I wouldn't have to seed this information into the policy API but could a pass command line option as outlined above.  If somebody didn't pass the required constraints for this cluster, marathon would reject the deployment attempt.

Been some time...  Any thoughts?

Is there a way to define optional constrains?
I.e. I want to consume some local datacenters hosts resources (if available) before placing on third party clouds.

Probably my use case could be covered by this long term issue https://github.com/mesosphere/marathon/issues/1174

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-3248. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Sprig and flatpickr calendar

Support Request
I don't really get how to combine properly sprig and js functionality.
I managed to filter item with using of sprig and flatpickr.
But if I want to add clear button, how can I refer to initial calendar?
For example, I have
const myInput = document.querySelector(".myInput");
const fp = flatpickr(myInput, {});
const clearButton = document.querySelector(".clear");
clearButton.addEventListener('click', e => {
fp.clear();
});
somehow it reacts on the button, but after first sprig reload doesn't know anything about calendar (fp)
Is there a work around to make it work?
Plugin Version
2.8.1

The component refresh will wipe out the HTML elements that are tied to your calendar. You can either specify a partial to replace using s-replace, or re-initialise the calendar using JS on each Sprig refresh, for example:
{% if sprig.isRequest %}
    <script>
        flatpickr(myInput, {});
    </script>
{% endif %}


Replace bespoke DID dereference implementation with web5/dids after update

Context: https://github.com/TBD54566975/tbdex-js/pull/141#discussion_r1454398823

i see that did-resolver in @web5/dids have resolve() as well, but in tbdex we are currently using resolveDids() in our own did-resolvers.ts.
should we try to replace tbdex's resolveDid() with web5's resolve() in this ticket?


Convert to Simple Darwin Core

Continued from https://github.com/niconoe/pyinaturalist/issues/143
Tasks
Things left to do:

[ ] Add remaining observation fields (about 11 fields noted in code comments)
[ ] Add remaining photo fields (for eol:dataObject; about 7 fields noted in code comments)
[ ] Formatting functions (for values derived from JSON fields but in a different format)
[x] Set observation fields that map to multiple DwC fields (like uri --> dcterms:references, dwc:occurrenceDetails)
[x] Include metadata for all observation photos (currently just using the first photo)
[x] Add XML namespaces
[x] Support converting a list of observations to a single SimpleDarwinRecordSet with multiple SimpleDarwinRecord objects
[ ] Unit tests
[ ] Documentation (just a simple usage example is fine for now)

Usage
When this is complete, I expect the usage will look something like:
from pyinaturalist import get_observations
from pyinaturalist_convert import to_dwc

observations = get_observations(user_id='my_username')
to_dwc(observations, 'my_observations.dwc')

Example observations
For reference:

Observation JSON (from node_api.get_observations()): obs_45524803.json
The same observation in DwC format (from rest_api.get_observations()): obs_45524803.dwc
Current draft of pyinaturalist-convert output: test/sample_data/observations.dwc

@FelipeSBarros I made a bit more progress on this, and added some notes for remaining tasks. I'll keep working on this when I have time, and you can pick up any of these tasks too if you'd like. Here's an example of what the output currently looks like.

@JWCook that's great. I will definitely take a look on those tasks to see how can I contribute.
This week and the next will be a littlebit hard todo it, but it is already in my agenda to get back on this colaboration.

Hey, @JWCook ! Hope everything is fine with you...
I am finishing the system I have told you and I just realized that  have to consider the pagination of the results from get_observations. Looking the documentation I found the pagination module: `pyinaturalist.pagination.paginate_all(api_func, *args, method='page', **params) .
I am wondering if this is the correct approach and if it is, how to use it?
Thanks in advance
Felipe

All you need to do is pass page='all', for example:
get_observations(project_id=1234 page='all')

There is some more info in the GeneralUsage section of the docs.

get_observations(project_id=1234, page='all')

I am getting a 400 response error. Should I be loged in to do such request?
all_pages = get_observations(project_id=105820, page='all')
 raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://api.inaturalist.org/v1/observations?project_id=105820&page=all&only_id=false

Which version of pyinaturalist are you using? That feature was added in the latest release (0.13).

Which version of pyinaturalist are you using? That feature was added in the latest release (0.13).
Thanks, @JWCook ! ...and I am sorry for such dumb questions I have made.
I was using 0.12. All working fine with latest version.

Good!
 I want to make these things easy to use, so it's useful to know which parts are potentially confusing.

Hey @JWCook, sorry leaving you alone on this issue. I am using the hacktoberfest to get back on collaborations.
Let me know if there is any task you think I should prioritize.
Best regards


ROS2 support

Hi
Your roboteq_control is awesome. I have tried it out and find the it's really awesome. I am curious do you have ROS2 version of roboteq_control that can integrate with diff_drive_controller of ros2_controllers ?
Best,
Samuel

Hi @BRoboticsSHADO ,
It's something that I wish to do soon, but at this period I am really busy, and I don't know when I will start with this process.


CAMEL-9916 Allow configuration of connectionFactory& co in SjmsEndpoint

Allow sjms endpoints to be used without requiring prior initialization of the component with a connection factory to make this endpoint xml friendly. The new endpoint usage is also friendly to context restart

Can you rebase this PR as it has merge conflicts now

This pull request had rebase issues. Created a new branch and pull request for the same purpose


python312Packages.xml2rfc: 3.23.0 -> 3.23.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.xml2rfc is: Tool generating IETF RFCs and drafts from XML sources
meta.homepage for python312Packages.xml2rfc is: https://github.com/ietf-tools/xml2rfc
meta.changelog for python312Packages.xml2rfc is: https://github.com/ietf-tools/xml2rfc/blob/v3.23.1/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.23.1 in filename of file in /nix/store/932r28493vs5ya5s297rjkbp5rjslfbc-python3.12-xml2rfc-3.23.1

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

python311Packages.xml2rfc
python312Packages.xml2rfc
xml2rfc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/932r28493vs5ya5s297rjkbp5rjslfbc-python3.12-xml2rfc-3.23.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.xml2rfc https://github.com/r-ryantm/nixpkgs/archive/a88f53332f2ce5e54c4e68684755b15dcfa28abd.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a88f53332f2ce5e54c4e68684755b15dcfa28abd#python312Packages.xml2rfc

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/932r28493vs5ya5s297rjkbp5rjslfbc-python3.12-xml2rfc-3.23.1
ls -la /nix/store/932r28493vs5ya5s297rjkbp5rjslfbc-python3.12-xml2rfc-3.23.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

python311Packages.xml2rfc
python311Packages.xml2rfc.dist
xml2rfc (python312Packages.xml2rfc)
xml2rfc.dist (python312Packages.xml2rfc.dist)

Maintainer pings
cc @vcunat @yrashk for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

Duplicate of https://github.com/NixOS/nixpkgs/pull/343196


Gitignore vscode & jetbrains IDE folders

Pull Request
Related issue
There is no issue for it, and i couldn't find an appropriate category to make an issue for it.
What does this PR do?

Its just a gitignore edit so people who use vscode and jetbrains IDEs (like IntelliJ) dont have to deal with committing the folder the IDE generates to store local project configs by mistake. (I honestly wanted to fork the repo to add something else but this bothered me enough to make a PR for it first)

PR checklist
Please check if your PR fulfills the following requirements:

[ ✔ ] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[✔  ] Have you read the contributing guidelines?
[✔  ] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

bors merge


 Which version are you running?

kerb-sts==0.1.3

On Sep 27, 2018, at 3:05 PM, Nick Brown<EMAIL_ADDRESS>wrote:
@ndmikkelsen https://github.com/ndmikkelsen Thanks for the report. Which version are you running?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/commercehub-oss/kerb-sts/issues/30#issuecomment-425225117, or mute the thread https://github.com/notifications/unsubscribe-auth/AbaHgg-uuGHU_tXPcBf1MTCgQTgemvNiks5ufS-OgaJpZM4W9F6c.