Add support for lambda (functions as first-class types)

Yeah I was going to flail my arms around and scream "Lambda doesn't even have support for lambda !" and things like "What an insult !"
I'm bummed because this is one of the better Lisp toys on iOS and it would be great if lambdas were supported.
> ((lambda (x) x) 5)
Unknown operator ''

Expected result
5

Thanks for the feedback @naomik . Will take a go at this sometime soon.

This should be fixed now.
> (defun add-one (x) (+ x 1))
<Method: add-one>
> (defun twice (f x) (f (f x)))
<Method: twice>
> (twice add-one 2)
4

https://github.com/reddragon/lambda/commit/005e9085b9e65a12f3f9a2b97f9eccfdd7b7da57 fixes it.

A+++


Video encoding settings

Can you please give some details about what are the settings for the encoded videos?
What encoder was used?
Is the encoding constant bitrate? Is there any buffer allowed while encoding?
Is frame rate set to constant?

This project is about adapting bitrates at video streaming time -- we assume the video has already been encoded and the bitrate levels are known. This is the case for on-demand video streaming -- such as users watching videos on Youtube or Netflix.
The encoding is not constant bitrate. Each chunk of the video is encoded in different bitrates -- that's how we adapt the bitrate while the user is watching a video.
Buffer in our context refers to the playback buffer. That's a few seconds of videos that's already downloaded but not yet played (e.g., shown as the grey bar ahead of current video time in Youtube) I don't know the "buffer" you refer to when encoding a video.
Frame rate I believe is constant.
Hope these help!

Hi Hongzi,
The question is for the original encoding settings for each source video
before you convert them to DASH content(chunks). While encoding, you can
set up constant bitrate but even in that case there can be some buffer so
that the average quality is preserved for the selected bitrate. This can be
set if you use ffmpeg encoder for h264 codec. You can also set no buffer
but it may affect the quality of the encoded video.
Can you please also let me know if the video's raw file (the original file
not the dash content) is available at any public repository?
Best
On Thu, Mar 12, 2020 at 11:48 AM Hongzi Mao<EMAIL_ADDRESS>wrote:

This project is about adapting bitrates at video streaming time -- we
assume the video has already been encoded and the bitrate levels are known.
This is the case for on-demand video streaming -- such as users watching
videos on Youtube or Netflix.
The encoding is not constant bitrate. Each chunk of the video is encoded
in different bitrates -- that's how we adapt the bitrate while the user is
watching a video.
Buffer in our context refers to the playback buffer. That's a few seconds
of videos that's already downloaded but not yet played (e.g., shown as the
grey bar ahead of current video time in Youtube) I don't know the "buffer"
you refer to when encoding a video.
Frame rate I believe is constant.
Hope these help!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hongzimao/pensieve/issues/92#issuecomment-598262419,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADALXKVGQPMTVDJE2SLKCXLRHD76VANCNFSM4LEOQ6HQ
.

@bekiroguzhan,
I have seen your build issue of AMuSt-ndnSIM in the simulator issues page. I'm sorry to write to you here as the issue thread is closed there.
I have resolved the building issue if you are still interested on that you can write to me using my email address achraf_gyahoo.fr.
Regards.
Hyperonex


Certificate 'Modifications' In Windows are not detected

Describe the bug
Open the user certificate managed and change some properties after first creating a base scan.  Run another and note the changes are not detected.  It appears some properties are not captured for comparison which could be by design?  But some key ones like OCSP checking being disabled are important from a security standpoint.
To Reproduce
Steps to reproduce the behavior:

Go to a certificate and edit the properties
Change the friendly name i.e. DO_NOT_TRUST to DO_TRUST, change the cert purposes or disable the OCSP checks if enabled or change the extended validation for example.  Save and run a scan to compare to a baseline scan.
Note the changes are not detected

Expected behavior
Modification of key properties should be detected.  Might be a good idea to document if not in scope for the current release and convert this to an enhancement.
Screenshots
If applicable, add screenshots to help explain your problem.

System Configuration (please complete the following information):

OS: Windows
OS Version: Windows 10
Application Version: 2.1...see image
CLI or GUI: GUI

Additional Context
Add any other context about the problem here.

We don't do any "modified" checks for certificates right now. This is a good suggestion for enhancement for 2.1.

I'm also not sure if that data being changed actually changes the Certificate or if instead it changes how Windows handles the certificate. I'm inclined to believe it is the latter as "modifying" the certificate itself doesn't make sense, given the signing and hashing.

Good with that.

I haven't been able to make any headway on this.  Moving it to Future, but I can't find any programmatic way to get those settings.


Split add_any_resource into two distributed functions for Citus

Is your feature request related to a problem? Please describe.
Citus supports distributing functions based on one of the passed parameter values.
Describe the solution you'd like
It may be more efficient to split the add_any_resource logic into two separate functions:

add_logical_resource_ident - distributed by the logical_id parameter
add_any_resource - distributed by the logical_resource_id parameter

The add_logical_resource_ident function should manage the insert or select for update of the logical_resource_ident record, returning the logical_resource_id. When this function returns, the logical_resource_ident record must exist and the row locked for update.
The add_any_resource function now takes an additional attribute (logical_resource_id). It no longer needs to return the logical_resource_id.
Also delete_resource_parameters should be distributed by p_logical_resource_id.
Describe alternatives you've considered
Use the current single-procedure solution.
Acceptance Criteria

a new resource is correctly created
an existing resource is correctly updated
a new resource with an existing logical_resource_ident record (created by a reference parameter from another resource) is correctly created.
the system correctly handles concurrent updates of the same resource

fhir-bucket can load synthea data with better performance than the single-procedure solution when using a multi-node Citus database.
To check that citus is using two functions, describe fhirdata.add_any_resource and note that the first parameter is the logical_resource_id (which is now passed into this function instead of being generated within it):
postgres=# \df fhirdata.add_any_resource;
                                                                                Li
st of functions
  Schema  |       Name       | Result data type |
                  Argument data types
      | Type
----------+------------------+------------------+------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
------+------
 fhirdata | add_any_resource | record           | **p_logical_resource_id** bigint, p_resource_type_id integer, p_resource_type character varying, p_logical_id character varying, p_payload bytea, p_last_updated timestamp without time zone, p_is_deleted character, p_sou
rce_key character varying, p_version integer, p_parameter_hash_b64 character varying, p_if_none_match integer, p_resource_payload_key character varying, OUT o_current_parameter_hash character varying, OUT o_interaction_status integer, OUT o_if_none_match_version in
teger | func

Further, show that the add_logical_resource_ident procedure exists:
postgres=# \df fhirdata.add_logical_resource_ident;
                                                                       List of functions
  Schema  |            Name            | Result data type |                                     Argument data types                                      | Type
----------+----------------------------+------------------+----------------------------------------------------------------------------------------------+------
 fhirdata | add_logical_resource_ident | bigint           | p_resource_type_id integer, p_logical_id character varying, OUT o_logical_resource_id bigint | func

You can check that both functions are distributed by checking the log output when running the schema tool --update-proc option:
...
2022-08-12 13:17:44.366 00000001    INFO abase.utils.citus.CitusAdapter Distributing function: fhirdata.add_logical_resource_ident(integer,character varying)
...
2022-08-12 13:17:44.563 00000001    INFO abase.utils.citus.CitusAdapter Distributing function: fhirdata.add_any_resource(bigint,integer,character varying,character varying,bytea,timestamp without time zone,character,character varying,integer,character varying,integer,character varying)
...

Additional context
None.

The stored procedure has been split.  Additional testing is needed with multi-node Citus but that will be performed separately.


Added Contributing Guidelines

@PradyumnaKrishna I have added Contributing Guidelines as CONTRIBUTING.md.
Please review it.
Issue #16

Okay

@PradyumnaKrishna I have made the necessary changes.
Please review it.


Up-Button for list of acceptance points

Is your feature request related to a problem? Please describe.
There are around 5800 acceptance points in Bavaria, when I scroll down the list, the categories disappear and I have to move back up all the same way. This is annoying and unnecessary if there were either an 'up' button or the field with the categories remain visible.
Describe the solution you'd like

add a scrollToTop button that scrolls to top

@hauf-toni at the moment there is a "In meiner Nähe suchen" floating action button, so i don't know if its a good ui/ux to put another button next to it.
Maybe you can add a proposal for a design

As another alternative idea, maybe clicking "Suche" again on the bottom bar should refresh the page and thus return to the top?

You can find my design suggestion in this 📍 Figma file

Is this still in design review?

@f1sh1918 can you give us an insight into the current status?


Consider: Settings Table Component?

I've been working on what I'm calling an "Accessible Settings Table". I'm wondering if the jQuery dependency were stripped if this pattern might make sense as a "Frendly Component".

It's a fairly simple concept. You have a table grid of different "settings". Each row is a setting. There is a corresponding form to update each setting. Whenever a row receives focus the associated settings form appears below. Of course on .no-js the component is "expanded" and everything is accessible.
This concept came from re-imagining a similar experience that used Right Click > Update Setting to open a JavaScript modal window.
I really like how you can tab through the whole thing. Combined with some bypasslinks that could be really nice.
Demo and explanation in this pen:
http://codepen.io/jpdevries/pen/BKXbXg

Thanks for the write-up @jpdevries. It's an interesting pattern! Feels somewhere between an accordion and description list. Trawling the APG Design Patterns to try to place it.
I wonder how much could be separated into the "shell" mechanic (accordion headers and panels), and "content" (the form content within panels). The relationship between values and form responses is something that may be extended from existing components, but the updating of values may create the need for aria-live attributes, or similar. What do you think?

@adamduncan that is a really good question. I would have to think more on how much things could be separated. I know it still needs a sprinkle of ARIA roles and such, but I really like the concept of using colspan to create these "anti modals" for each row.
I'm doing something very similar in the Mange Users Lab here
https://github.com/jpdevries/matboard#labs


I translated data.README.md file into korean

I translated data.README.md file into korean

@taewankang Your contribution to Apollo is highly appreciated. Please create a new folder in docs and put all your korean translations of documents there. I am closing all your existing pull requests. Please create a new one after you make the changes. Thanks!


[BUG] Segfault when calling a method from a child item that releases it

Describe the bug
First thanks for the wonderful improvements Cython has seen recently. This is really game changing.
Mostly so far I've only encountered usability issues, and this is the first time I encounter what I think is really a bug.
Here is the code that has an issue (below generated cpp):
      /* "dearcygui/core.pyx":1166
 *             while (<baseItem>child)._prev_sibling is not None and \
 *                 already_attached.find((<baseItem>child)._prev_sibling.uuid) == already_attached.end():
 *                 (<baseItem>child)._prev_sibling.detach_item()             # <<<<<<<<<<<<<<
 *             already_attached.insert(uuid)
 *
 */
      ((struct __pyx_vtabstruct_9dearcygui_4core_baseItem *)((struct __pyx_obj_9dearcygui_4core_baseItem *)__pyx_v_child)->_prev_sibling->__pyx_vtab)->detach_item(((struct __pyx_obj_9dearcygui_4core_baseItem *)__pyx_v_child)->_prev_sibling, 0); if (unlikely(PyErr_Occurred())) __PYX_ERR(1, 1166, __pyx_L1_error)
    }

Here is the code with a workaround that prevents the issue (below generated cpp):
    (Note: sibling is declared as cdef baseItem sibling)

      /* "dearcygui/core.pyx":1169
 *                 # Setting sibling here rather than calling detach_item directly avoids
 *                 # crash due to refcounting bug.
 *                 sibling = (<baseItem>child)._prev_sibling             # <<<<<<<<<<<<<<
 *                 sibling.detach_item()
 *             already_attached.insert(uuid)
 */
      __pyx_t_6 = ((PyObject *)((struct __pyx_obj_9dearcygui_4core_baseItem *)__pyx_v_child)->_prev_sibling);
      __Pyx_INCREF(__pyx_t_6);
      __Pyx_XDECREF_SET(__pyx_v_sibling, ((struct __pyx_obj_9dearcygui_4core_baseItem *)__pyx_t_6));
      __pyx_t_6 = 0;

      /* "dearcygui/core.pyx":1170
 *                 # crash due to refcounting bug.
 *                 sibling = (<baseItem>child)._prev_sibling
 *                 sibling.detach_item()             # <<<<<<<<<<<<<<
 *             already_attached.insert(uuid)
 *
 */
      ((struct __pyx_vtabstruct_9dearcygui_4core_baseItem *)__pyx_v_sibling->__pyx_vtab)->detach_item(__pyx_v_sibling, 0); if (unlikely(PyErr_Occurred())) __PYX_ERR(1, 1170, __pyx_L1_error)
    }

What happens here is that detach_item() will change the value of (child)._prev_sibling, and thus there is no more reference to the item during the call. This causes a SEGFAULT during the detach_item call.
The workaround does generate INCREF/DECREF calls and thus the segfault is avoided.
While I agree this is quite a corner case, I don't think I have seen any warning relative to it in the Cython doc as a known limitation. In the general case, I don't think INCREF and DECREF should be generated around method calls, and at least in the same file I use that a lot in nogil sections.
Since the issue doesn't seem to have been reported though, I decided to post it here.
Code to reproduce the behaviour:
Easiest would be to cythonize the target file and check the .cpp
Expected behaviour
One of these:

No crash
A warning during compilation
A warning in the documentation.
A readable way to indicate when we call an item's method that the object should be considered a PyObject (borrowed refenced) or an object (incref/decref)

OS
Linux
Python version
3.10.12
Cython version
3.0.11
Additional context
No response

As you observe, we call cdef and cpdef methods with a borrowed reference to self. That's obviously dodgy in your case but in a lot of cases it's an optimization. It also allows you to call these methods without the GIL (which is a nice feature that would break a lot of code of if we changed it).
The more robust workaround in your code would probably be to add self2 = self inside detach_item which would at least make sure that the caller doesn't have to remember to do it.
I'm not sure what the best overall solution is.

No crash

Could probably only be implemented on regular, gil-held calls. (Note that it's still possible to crash the nogil version by acquiring the gil in the function). Would be an option.

A warning during compilation

Would almost certainly be too noisy to be useful.

A warning in the documentation.

Could definitely add this, but I suspect it wouldn't get read

A readable way to indicate when we call an item's method that the object should be considered a PyObject (borrowed refenced) or an object (incref/decref)

Possible in principle.

Thanks for the replay.
I understand your suggestion for more robust workaround. This raises one worry is that both this one and mine depend on the current behaviour of Cython of not releasing variables before the end of the function, even if they are unused.
I also have in mind the move to free-threaded python, and that it will likely have an impact on how best to write Cython code. In a world where the gil is gone, would the first option be favored ? If in a gil-free world, refcounting is still not cheap, is an api to indicate borrowed referenced needed ?

This raises one worry is that both this one and mine depend on the current behaviour of Cython of not releasing variables before the end of the function, even if they are unused.

That's very unlikely to change I think. I think we'd assume that people might be keeping a variable around to keep something alive.

I also have in mind the move to free-threaded python, and that it will likely have an impact on how best to write Cython code.

Probably not a lot. There's some open debate about how we should handle internal thread-safety in Cython and how much effort we should spend saving people from themselves. I don't think it changes too much in this case. Certainly not immediately (where it's "experimental" at least for the next few years).

https://github.com/cython/cython/blob/a3086ff75f5b33639b867ab15922d928e14e99bf/Cython/Compiler/ExprNodes.py#L6247-L6249
This is the logic that controls it. My argument about nogil doesn't hold up - we skip creating the temps for nogil so a change wouldn't affect that.
I'm slightly less sure why we've special cased self now and slightly less convinced that we should.


Tag "Fix error message of missing value with prefix" release

Hey @sethvargo,
could you tag the fix I did here https://github.com/sethvargo/go-envconfig/pull/29 as a new version?
Thanks

v0.3.3 done


how can i import it into vue.js

please give me some advice

Duplicate of https://github.com/artf/grapesjs/issues/1082


improve policy verification

This PR supersedes https://github.com/kubewarden/policy-server/pull/214

feat: Add keyless verification by supporting verification-config.yml
feat: fetch sigstore data from remote TUF repo

Fixes https://github.com/kubewarden/policy-server/issues/211

I've updated the PR to reflect the changes you requested. I've converted the PR to be a draft because the policy-fetcher and policy-evalutator dependencies need to be updated. This is currently blocked by https://github.com/kubewarden/policy-fetcher/pull/69

Are we going to allow users to use their own Fulcio certificate as we do in kwctl? I see we always fetch the code from TUF FulcioAndRekorData::FromTufRepository , but there is no way to use your own certificate FulcioAndRekorData::FromCustomData

Yes, this isn't supported yet. I would add that via another PR.  This is something I don't see as super important right now for Policy Server.
Moreover, there's something that needs to be sorted out: we must provide a way to specify multiple fulcio certs via the cli flag. However, I have no idea about how this could be done via environment flags. Right now our controller uses environment variables as a way to configure the behaviour of policy-server. I don't know how we could expose that...

This can be reviewed now. I'm working on adding some unit tests for the downloader code

The unit tests are ready

This is now building fine, because all the dependencies are now available :partying_face:


#78 local environment setup with solr and dispatcher on Docker

is PR title up to date? solr ?

is PR title up to date? solr ?

Solr is there but unconfigured. I will probably remove it.


In k8s v1.21 Pod is not able to schedule on nodes

With k8s 1.21 and device-localpv -- waitforfirstconsumer pvc's are not getting bound, while the same is working in k8s 1.20
Events:
  Type    Reason                Age                    From                         Message
  ----    ------                ----                   ----                         -------
  Normal  WaitForFirstConsumer  3m34s (x5 over 4m23s)  persistentvolume-controller  waiting for first consumer to be created before binding
  Normal  WaitForPodScheduled   4s (x14 over 3m19s)    persistentvolume-controller  waiting for pod app-busybox-55569b5cc8-nwkxn to be scheduled

Pod keeps in pending with this error:
Events:
  Type     Reason            Age   From               Message
  ----     ------            ----  ----               -------
  Warning  FailedScheduling  16s   default-scheduler  0/4 nodes are available: 1 node(s) had taint {node-role.kubernetes.io/master: }, that the pod didn't tolerate, 3 node(s) did not have enough free storage.
  Warning  FailedScheduling  14s   default-scheduler  0/4 nodes are available: 1 node(s) had taint {node-role.kubernetes.io/master: }, that the pod didn't tolerate, 3 node(s) did not have enough free storage.

on nodes we can see, we have 50 G disk (/dev/sde) , while pvc is only for 4 G. There are no logs also in controller
k8s@lvm-node1:~$ lsblk | grep sd
sda      8:0    0  100G  0 disk
├─sda1   8:1    0  512M  0 part /boot/efi
├─sda2   8:2    0    1K  0 part
└─sda5   8:5    0 99.5G  0 part /
sdb      8:16   0   50G  0 disk
sdc      8:32   0   50G  0 disk
sdd      8:48   0   50G  0 disk
sde      8:64   0   50G  0 disk
└─sde1   8:65   0    9M  0 part
sdf      8:80   0   50G  0 disk

+1
Seeing the same on openshift:  v1.21.1+9807387

@davidkarlsen After this PR https://github.com/openebs/device-localpv/pull/37/files, it worked for me.
 Has a release been made which contains it (I'm on a phone right now 😂)

@davidkarlsen how did you install the lvm driver?

@davidkarlsen how did you install the device driver (helm or operator yaml)? Which release version you are using?

Via the parent chart which installs the full openebs-stack.

But I use lvm-local-pv (for filsystem) - same bug there?

We have fixed the helm chart for lvm localpv recently in the local repo for k8s 1.21+. You can try the helm chart from the repo and see.

I see that we have fixed the master helm chart also https://github.com/openebs/charts/pull/264. You can install the latest one and see if it is working.

I see that we have fixed the master helm chart also openebs/charts#264. You can install the latest one and see if it is working.

I see that we have fixed the master helm chart also openebs/charts#264. You can install the latest one and see if it is working.

2.12.2 worked fine! Thanks 👍

This happened to me when I installed it using the operator.
➜ kubectl apply -f https://raw.githubusercontent.com/openebs/device-localpv/develop/deploy/device-operator.yaml

I'm using device-local-pv - same bug there?


[FEATURE] - Add support for an Advanced Search

Summary
Currently next-admin supports searching against a single string input across numerous fields, which is great.  But often times our admin panels require search across multiple conditions with multiple data types.  For example:
const result = await prisma.post.findMany({
  where: {
    AND: [
      {
        content: {
          contains: 'Prisma',
        },
      },
      {
        published: {
          equals: false,
        },
      },
    ],
  },
})

Could we add support for an Advanced Search that allows users to define search fields (i.e. "content" and "published") as well as data types (i.e. "string" and "boolean") and perhaps even a comparator (i.e. lt, gt, lte, gte, etc ...).
Thank you for considering this.  Next-Admin is outstanding!!
Basic Example
As a user I'd like to be able to narrow the Posts results down to entries by author "Morrison" published after "2016".
Drawbacks
Can't think of any real drawbacks.  But the advanced search would most likely need to appear in a new modal and that may be tricky for mobile first designs.
Unresolved questions
No response

Hello
Thank you for the issue, this is something that is in the pipeline, we are thinking about how to implement it in the best possible way. A first step to this was to add the custom filters property. Next step is to add a more complex query builder which should fit any use case for the queries

This is actually related to #326

Closing in favor of #326


Contributor Code of Conduct

Is it possible to add language that the complainee will be given a right to defend himself with regards to complaints? Seems like the below is one sided, what about fake complaints ?
Enforcement
Instances of abusive, harassing, or otherwise unacceptable behavior may be reported to the community leaders responsible for enforcement at<EMAIL_ADDRESS>All complaints will be reviewed and investigated promptly and fairly.
All community leaders are obligated to respect the privacy and security of the reporter of any incident.

Code of Conduct is adopted from GitHub and will not be changed at this time.


Should there be an AbstractMPS type?

Hi,
I think it could be useful to have an AbstractMPS type and have some of the MPS functions accept AbstractMPS.
The use case I have in mind is that it will allow users to define MPS wrappers that could represent for example a purification or a vectorized density matrix , for which different algorithms and operations are exactly the same as a regular MPS, but measurement of observables is done differently. Another use case could be for an iMPS.
As a starting point it will be already useful to just define AbstractMPS and make MPS <: AbstractMPS without making any other changes to the code.
The question is what is the most minimal but still useful interface that defines anAbstractMPS.  I'll have to think about it a bit more, possibly after trying to actually implement some MPS wrapper.

Good question, but I think this sort of thing exposes an unresolved issue with generic programming in Julia and programming languages in general.
Should an MPO be considered an AbstractMPS? Or if not, should there be some concept like 1DTensorNetwork which they have in common? Or rather, is an MPO an AbstractOperator or is an MPS an AbstractWavefunction?
As you can see, depending on context, types are sometimes one thing and sometimes other things. Yet inheritance is a concept that can only be deployed once, before all use cases have been properly explored.
For now we have taken the approach to generalizing certain methods which can apply to either MPS or MPO by restricting arguments to be ::Union{MPS,MPO}. A good thing about this approach is that it's "future proof" in the sense that we could add more things to the Union or just remove the type restriction altogether. Matt may have more to add here, and I think he's been finding more and more that it can be good to leave off types altogether in more high-level generic code for reasons like this.
So let's keep this in mind but for now I'm hesistant to define a supertype for MPS unless it enables us to do something we could not do before that.

thanks for the reply Miles. I see your point regarding how it can become a liability and cause unnecessary complications.

Matt may have more to add here, and I think he's been finding more and more that it can be good to leave off types altogether in more high-level generic code for reasons like this.

This sounds good (thinking about it  now, I recall reading somewhere that it's actually the recommended way to do thing in Julia instead of getting overexcited about type hierarchies like I was suggesting). There is still the question of which functions you consider high-level enough to not have the type restriction for MPS/MPO in them. For sure it would be nice in the long run to remove the type annotation from dmrg but I guess I should bring up a concrete use case first instead of opening vague issues :) .

Thanks for understanding, Ori. I think in an ideal programming language, the way to constrain function arguments would be based around which minimum properties the arguments must have for the code inside the function to work. So a function like sort would require getindex and setindex! and length, and this would be checked statically. The D programming language has gone quite a ways in realizing this idea & more - see talks by Alexandrescu on "Design by Introspection" or his most recent CppCon talk for more about this. But it's not a widely adopted language and also has its own drawbacks. I still like Julia more overall, especially for ITensor's use case and target audience.


Refactor key derivation validation

ACK!

Coverage decreased (-0.01%) to 94.954% when pulling d518d127379e90d10606e837ab17922d0ad6a4d8 on isocolsky:ref/validation into 0e9b891859d74db9de5999a150ec098cf29d6c18 on bitpay:master.


[BUG] azure-servicebus-jms-spring-boot-starter listener: The messaging entity ... could not be found

Describe the bug
When I'm trying to create a JmsListener it fails to connect, stating that the messaging entity could not be found
Exception or Stack Trace
WARN 3090 --- [enerContainer-5] o.s.j.l.DefaultMessageListenerContainer  : Setup of JMS message listener invoker failed for destination 'user' - trying to recover. Cause: The messaging entity 'pass-servicebus-d-euwe-10c6dc:Topic:user|qpid-jms:receiver:ID:dc2dd5da-88d3-4529-90ae-d3a3b1b3d2d5:1:1:5:user' could not be found. To know more visit https://aka.ms/sbResourceMgrExceptions.  TrackingId:bbda7699-1769-44a3-aef6-d3f365984e93_B6, SystemTracker:pass-servicebus-d-euwe-10c6dc:Topic:user|qpid-jms:receiver:ID:dc2dd5da-88d3-4529-90ae-d3a3b1b3d2d5:1:1:5:user, Timestamp:2020-10-07T11:37:19 TrackingId:a691965af0ac422caf58471b5d518802_G9, SystemTracker:gateway7, Timestamp:2020-10-07T11:37:19 [condition = amqp:not-found]

To Reproduce

Integrate dependencies:

api(group = "com.microsoft.azure", name = "azure-servicebus-jms-spring-boot-starter", version = "2.3.5")
api(group = "com.microsoft.azure", name = "azure-servicebus-jms", version = "0.0.5")

add properties to application.properties:
The connection-string is of a Shared access policy from the topic:

spring.jms.servicebus.connection-string=Endpoint=sb://pass-servicebus-d-euwe-10c6dc.servicebus.windows.net/;SharedAccessKeyName=user-read-write-rule;SharedAccessKey=***=;EntityPath=user
spring.jms.servicebus.topic-client-id=test

add a Spring class:

@Component
class ServiceBusHandler(
     private val jmsTemplate: JmsTemplate
) {
    val logger: Logger by LoggerDelegate()

    @PostConstruct
    fun test() {
        //jmsTemplate.convertAndSend("user", UserUser())
        println("This actually works")
    }

    @JmsListener(
        destination = "user",
        subscription = "user-read-write",
        containerFactory = "jmsListenerContainerFactory"
    )
    fun onUser(user: User) {

    }
}

The error is visible in the logs upon startup

Expected behavior
Connection works and messages are received
Setup (please complete the following information):

OS: MacOS
IDE : IntelliJ

Additional context
Azure Configuration:

I double and triple checked the configuration. Sending messages to the topic works fine.

@jialindai could you please assist in routing this azure-spring issue

Hi, @jonasbark
I follow your configuration, When I'm trying to send a message, JmsListener it fails to connect.
When I change the configuration spring.jms.servicebus.topic-client-id to my Service Bus subscription ID, it works.
spring.jms.servicebus.topic-client-id=[Your Subscription ID]

Hope it can help.

Hi @han-gao - thanks for replying!
What exactly do you mean with Subscription ID? The Azure Subscription ID? Or the actual subscription name (in my case "user" - see screenshot)?
I tried all different combinations but it still shows the same error message.

Hi @jonasbark
Not an Azure Subscription ID,
When you create a Service Bus in the portal, there is a Subscription ID on the Service Bus overview, such as the picture below,

Can you solve your problem?

Thank you - I put it in but still no luck for me. Same message I'm afraid.
Now that we're sure the spring.jms.servicebus.topic-client-id part is correct, can we do the same with the other parameters?
spring.jms.servicebus.connection-string:

JmsListener:
@JmsListener(
        destination = "user",
        subscription = "user-read-write",
        containerFactory = "jmsListenerContainerFactory"
    )

Setup of JMS message listener invoker failed for destination 'user' - trying to recover. Cause: The messaging entity 'pass-servicebus-d-euwe-10c6dc:Topic:user|qpid-jms:receiver:ID:5e7c84f5-37dd-4b1a-8f69-ac4a8030f65d:1:1:3:user' could not be found. To know more visit https://aka.ms/sbResourceMgrExceptions.  TrackingId:6b10c354-db1b-40c7-9bcb-ff175a435ce8_B16, SystemTracker:pass-servicebus-d-euwe-10c6dc:Topic:user|qpid-jms:receiver:ID:5e7c84f5-37dd-4b1a-8f69-ac4a8030f65d:1:1:3:user, Timestamp:2020-10-23T06:03:42 TrackingId:1f34b6844b894efcbdcb6e2fd864d6a7_G8, SystemTracker:gateway7, Timestamp:2020-10-23T06:03:42 [condition = amqp:not-found]

@jonasbark
Hi, Maybe your JmsListener  is not right.
@JmsListener(
        destination = "user",
        subscription = "user-read-write",
        containerFactory = "jmsListenerContainerFactory" //This is the containerFactory for jms queue
    )

You are using jms topic, so your JmsListener should look like the following:
@JmsListener(
        destination = "user",
        subscription = "user-read-write",
        containerFactory = "topicJmsListenerContainerFactory" //This is the containerFactory for jms topic
    )

Can it work?

Yes... damn, I didn't catch that.
Thanks a lot for your help!
Feel free to close.
I would suggest the Documentation to be updated because https://docs.microsoft.com/de-de/azure/developer/java/spring-framework/configure-spring-boot-starter-java-app-with-azure-service-bus currently states:

spring.jms.servicebus.topic-client-id | Specify the JMS client ID if you're using an Azure Service Bus topic with a durable subscription.

It's not clear that the "Service Bus Subscription ID" is the JMS client ID.
Have a nice day! :)

@jafreck
I'm sorry I didn't read the news on the weekend. I used your configuration to reproduce your problem ,found that your @JmsListener is incorrect.
As below:

I also found that the document is not very clear, the document will be updated immediately.
Have a nice day! :)


Add an indicator of PR activity

Maybe time since last comment, or something more complex that "devalues" comments over time.
Maybe colour the PR number's cell and desaturate it for inactivity.
CC @bstrie

How about a banana next to each row whose ripeness corresponds to the freshness of activity on the PR. After a while a banana will turn black and begin attracting flies, whose buzzing will be heard over the user's speakers. The intensity of the buzzing will correspond to the number of rotting bananas in the queue.
Any user who successfully unripens a black banana by commenting on the PR shall receive the "Banana Bread" achievement, worth 50 gamerscore.


Add shop link to minnestar.org website

I tagged @jennapederson so we could try to have a conversation about the best way to do this. Should this go through Brave New Media, or is this something we can add directly to the WP theme?

@caseyhelbling I should be able to do this, but probably won't be til Thursday/Friday.

Do you have the link handy?

@caseyhelbling Staging environment for review: https://minnestarstg.wpengine.com/

Added main nav menu item
Added secondary sub nav menu item
Added to footer

I'll also add a redirect from minnestar.org/shop to whatever shop URL we settle on. Let me know when to update production and what the real URL will be.

Thank you! shop.minnestar.org works!


How to set working directory for unittest in neotest setup?

Case: I have tests in repo, but iI need to run it in different directory.
It seems as cwd in adapter setup does not work. I also have tried to put cwd to adapters = dap = {cwd = 'path_here'} (it is also not worked)
local neotest = require("neotest")
neotest.setup({
  adapters = {
    require("neotest-python")({
        python = "/opt/python3.7/bin/python3.7",
        runner = "unittest",
        cwd = '/tmp/tests/mega_project/tests/stateless/run',
    })
  },
})

local envs = {}
envs['LD_LIBRARY_PATH'] = '.'
envs['PYTHONPATH'] = '$PYTHONPATH:/home/user/repos/some_project'

local run_test = function()
    require("neotest").run.run({
        strategy = "dap",
        env = envs
    })
end

vim.keymap.set('n', '<F4>', run_test)

There is no cwd setting for neotest-python, it's agnostic to the project location. All paths are passed in from the core plugin.
You can pass cwd to the run function: require("neotest").run.run({ cwd = ... })

@rcarriga The cwd option to require("neotest").run.run() no longer seems work ! Let's say I am inside some deeply nested sub-directory inside my project root, the require("neotest").run.run({ cwd = ... }) didn't work for me. For example if I have the following directory structure
project/
├── project-code/
│  ├── src/
│  ├── test/
│  └── pyproject.toml
└── config/

So, here the root detected by the lsp is project (due to pyproject.toml file inside it) which has the test directory. So when I run require("neotest").run.run(vim.uv.cwd()) from inside the project-code directory all tests are detected and run, however if I go one directory up(i.e. inside the main project directory) and try to do require("neotest").run.run({cwd = vim.uv.cwd() .. "project-code") , I get No tests found. I even tried providing the hard-coded path but even that didn't work.
I was also looking another thread where you mentioned passing suite = true when you are not in the directory that contains tests, so I ran require("neotest").run.run({suite = true}) but I got the following errors
E5108: Error executing lua ...bhanot/.local/share/nvim/lazy/nvim-nio/lua/nio/tasks.lua:100: Async task failed without callback: The coroutine failed with this message:
...ocal/share/nvim/lazy/neotest/lua/neotest/client/init.lua:337: assertion failed!
stack traceback:
	[C]: in function 'assert'
	...ocal/share/nvim/lazy/neotest/lua/neotest/client/init.lua:337: in function '_get_adapter'
	...ocal/share/nvim/lazy/neotest/lua/neotest/client/init.lua:203: in function <...ocal/share/nvim/lazy/neotest/lua/neotest/client/init.lua:197>
	...al/share/nvim/lazy/neotest/lua/neotest/consumers/run.lua:39: in function 'get_tree_from_args'
	...al/share/nvim/lazy/neotest/lua/neotest/consumers/run.lua:80: in function 'func'
	...bhanot/.local/share/nvim/lazy/nvim-nio/lua/nio/tasks.lua:173: in function <...bhanot/.local/share/nvim/lazy/nvim-nio/lua/nio/tasks.lua:172>
stack traceback:
	[C]: in function 'error'
	...bhanot/.local/share/nvim/lazy/nvim-nio/lua/nio/tasks.lua:100: in function 'close_task'
	...bhanot/.local/share/nvim/lazy/nvim-nio/lua/nio/tasks.lua:122: in function 'step'
	...bhanot/.local/share/nvim/lazy/nvim-nio/lua/nio/tasks.lua:150: in function 'run'
	[string ":lua"]:1: in main chunk

This is how my config looks like
 {
  "nvim-neotest/neotest",
  optional = true,
  dependencies = {
    -- "nvim-neotest/neotest-python",
    "nvim-neotest/nvim-nio",
    "nvim-lua/plenary.nvim",
    "antoinemadec/FixCursorHold.nvim",
    "nvim-treesitter/nvim-treesitter",
  },
  keys = {
    {
      "<leader>tT",
      function()
        require("neotest").run.run(LazyVim.root())
      end,
      desc = "Run All Test Files",
    },
  },

  opts = {
    adapters = {
      ["neotest-python"] = {
        dap = { justMyCode = true },
        pytest_discover_instances = true,
      },
    },
  },
}

Am I doing something incorrect here ?


Update Documentation regarding HHVM

To prevent sinking into a rabbit-hole as I had:
https://discourse.roots.io/t/cannot-connect-to-database/3299/11
I suggest letting users know that this is experimental, because I am new to this I assumed it was something that was under control.
This was my first time using ansible and I had no idea HHVM was a new thing, I switched it to true after hearing so much good about it and completely forgot that I had.
This lead to database errors on vagrant.
One of which is the recreation of issue #73 whereby changing mysql_root_password in group_vars - development lead to errors.
The other being that even if the db settings were left at default, the vagrant box would still fail to connect to the db socket: Warning: mysql_connect(): Can't connect to local MySQL server through socket '/tmp/mysql.sock' (2)

Apologies for the trouble.
#73 is a known bug obviously which should just be fixed. Basically I never wanted to document the bug because that's just silly when I told myself I'd fix it long ago... :(
The HHVM bug was a perfect storm of timing. When I developed and tested the feature it worked perfectly. In the few days between that and actually merging it in, HHVM released a new version which has a bug. See https://github.com/roots/bedrock-ansible/pull/145 for details.

Ah thanks for that! It's that looming shadow over our heads :+1:
I don't think any of us realise that a developer/designer is acually a modern cleaner, just in the digital world hehe. I think I do more cleaning things than I do designing. I keep hoping the ratio will drop with experience.


Return entire CA PEM if unable to match host/domain for export

Signed-off-by: Alexander Wels<EMAIL_ADDRESS>
What this PR does / why we need it:
If the export code cannot match on the route/Ingress host or domain with wild card, return the entire contents of the config map. Filter out expired or not yet valid certificates. This happens if the kube-root-ca.crt only contains the root CA for instance.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/retest-required

/test pull-kubevirt-e2e-k8s-1.25-sig-compute-migrations-root

/lgtm
/approve
/retest-required

/test pull-kubevirt-e2e-k8s-1.25-sig-compute-migrations-root

/hold
Helping tide to merge https://github.com/kubevirt/kubevirt/pull/9124 first

/hold cancel
#9124 was merged


feat: add reasoning effort to agent config model + support in core

Description

add reasoningEffort on the agent_configurations table (optional)
add support for reasoning_effort config on core LLM / Chat blocks for OAI provider (as extras config).

Risk
critical path for pretty much everything + database migration
Deploy Plan

apply DB schema migration (prodbox)
deploy core (adds support for the config) and deploy front (functionally a no-op)

@tdraier it's the amount of test time compute you want to use for o1 models: https://platform.openai.com/docs/api-reference/chat/create


Spelling Checker

Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
Fixes #(issue_no)
Replace issue_no with the issue number which is fixed in this PR
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Documentation Update

Checklist:

[ ] My code follows the style guidelines(Clean Code) of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have created a helpful and easy to understand README.md
[ ] My documentation follows Template for README.md
[ ] My changes generate no new warnings
[ ] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

#205 @avinashkranjan sir this is issue plss review it

#205 @avinashkranjan sir this is issue plss review it

@avinashkranjan plss sir review it

@avinashkranjan plss sir review it

@all-contributors please add @Ayush7614  for documentation

@all-contributors please add @Ayush7614  for documentation


36 use real data in classification task

Description
Prepares a sample of labelled data, embeds this using jobbert, and trains a logistic regression model.
Fixes #36
Instructions for Reviewer
In order to test the code in this PR you need to run

[ ] dap_job_quality/pipeline/sentence_classifier/prep_training_data.py
[ ] dap_job_quality/pipeline/sentence_classifier/baseline_classifier.py
[ ] dap_job_quality/pipeline/sentence_classifier/log_reg.py

Please pay special attention to ...
Checklist:

[x] I have refactored my code out from notebooks/
[x] I have checked the code runs
[ ] I have tested the code
[x] I have run pre-commit and addressed any issues not automatically fixed
[x] I have merged any new changes from dev
[x] I have documented the code

[x] Major functions have docstrings
[x] Appropriate information has been added to READMEs

[ ] I have explained this PR above
[ ] I have requested a code review

Hey, it's looking really good! I ran all the files, and they worked. Since this is the second PR on the same code, I've done a quick read - but let me know if there's areas it would be helpful to be more thorough.
I did have one question (and it might be my confusion). When we prepare the training/ testing data, are we retaining somewhere the job id and sentence or span order attached to each phrase? (or will we have that for the inference data)? Sorry if I've missed that in the code.

Hey, it's looking really good! I ran all the files, and they worked. Since this is the second PR on the same code, I've done a quick read - but let me know if there's areas it would be helpful to be more thorough.
I did have one question (and it might be my confusion). When we prepare the training/ testing data, are we retaining somewhere the job id and sentence or span order attached to each phrase? (or will we have that for the inference data)? Sorry if I've missed that in the code.

You haven't missed that and it's a good point - we should preserve the job ID alongside the sentences/spans. I have added a new ticket so that this doesn't get missed.


getCurrentPosition on  Android12 and some Huawei models report an error TimeoutException

🐛 Bug Report
The "getCurrentPosition" method is normal for models below Android 12. Android 12 and some Huawei models report an error "TimeoutException after 0:00:03.000000: Future not completed", but the "getLastKnownPosition" method is normal.
Expected behavior
I/flutter (15995): 获取当前位置出错===TimeoutException after 0:00:03.000000: Future not completed
Reproduction steps
 try {
      ....
      LocationPermission permission = await Geolocator.checkPermission();
      permission = await Geolocator.checkPermission();
      if (permission == LocationPermission.denied ||
          permission == LocationPermission.deniedForever) {
        EasyLoading.showToast('定位失败！请确认授权或开启手机定位');
        throw null;
      }
      Position _currentPosition;
      _currentPosition = await Geolocator.getCurrentPosition(
          desiredAccuracy: LocationAccuracy.medium,
          timeLimit: Duration(seconds: Platform.isIOS ? 5 : 3),
          forceAndroidLocationManager: true);
      ....
    } catch (e) {
      print('获取当前位置出错===$e');
      return catchErr();
    }

Configuration
Version: 8.2.0
Platform:

[ ] :iphone: iOS
[x] :robot: Android12

same error
The problem only appears when the forceAndroidLocationManager is true
Flutter 2.10.3
Geolocator 8.2.0

Possibly related to #1027 and #1114.

same error
The problem only appears when the LocationAccuracy.medium
Only a few phones won't get the location
It has nothing to do with #1027 or #1114

Same error here as well. The only thing is that it happens in debug builds for me. If I compile a release build, it will work on same device. I am not using forceAndroidLocationManager and the precise location is not off.
i have tried using time-limit finally to limit the call otherwise the getCurrentPosition keeps on checking.
    Position position;
    String latitude = '';
    String longitude = '';
    try {
      position = await Geolocator.getCurrentPosition(
        desiredAccuracy: LocationAccuracy.best,
        timeLimit: Duration(seconds: 30),
      );
      latitude = position.latitude.toStringAsFixed(3);
      longitude = position.longitude.toStringAsFixed(3);
    } catch (e) {
      logger.e(e);
      return ToastMessage.showEmptyPlaceToast();
    }


fix: do not pass metadata_options if variable is null

Setting the metadata_options explicitly in the AWS launch template is not supported in certain regions. The error we get is:
You must use a valid fully formed launch template; specifing HttpProtocol metadata options for an instance is not supported in this region.

This fix sets the metadata_options only if the value passed in is not null.
As described in https://github.com/hashicorp/terraform-provider-aws/issues/25227, version 5.0.0 of the terraform-provider-aws fixed the issue of removing default values from the launch template. Hence this fix also upgrades the AWS terraform provider to that new version.

I'm probably going to incorporate this idea into a more general rework branch. This has an issue right now, but it's also an issue in the original code. You can't set just one part of the metadata options. We really need to be doing a lookup call with a default for each individual field and not assuming all of the keys always exist in the map passed to the variable, at least not if we want the type constraint to be map(any) as is currently the case.

I'm probably going to incorporate this idea into a more general rework branch. This has an issue right now, but it's also an issue in the original code. You can't set just one part of the metadata options. We really need to be doing a lookup call with a default for each individual field and not assuming all of the keys always exist in the map passed to the variable, at least not if we want the type constraint to be map(any) as is currently the case.

Sounds good. Should I close this pull request then with a reference to the new rework issue once you create it?

You can leave it. I'll take care of the linking and closing when it happens.


Pre and Pos methods

First of all, The idea is awesome!
I just wanted to know about how to implement 'pre' and 'pos' save methods in typegoose way.
Thanks!

Thank you for your interest in Typegoose! This is currently not available however this would be a great addition to the library.
If you have a working solution feel free to open a pull request, otherwise it may be implemented in the future.

Awesome! Thanks!


Is Faker integration easy/possible? Any tips on how?

I see a older thread in the php edition that talked about adding Faker. Did that ever happen?
Quick research says Faker can be autoloaded in Symfony projects… might make a good addition to the Twig version of PL. I don't have the experience or knowledge to know how.
If it can be added into PL-twig easily or someone could give me the quick rundown of how to add it myself, I'd appreciate it.

Learned some things and found a way. Documenting here for others.
Add the Faker to your project with:
composer require fzaninotto/faker
Create a new function in source/_twig_components/functions/ called faker.function.php
Copy this code into it:
<?php

$function = new Twig_SimpleFunction('faker', function ($string, $options="") {
  $faker = Faker\Factory::create();
  if ($options != "") {
    return $faker->$string($options);
  } else {
    return $faker->$string;
  }
});

?>

Call it in your Twig files with:
{{ faker('PROPERTY YOU WANT', 'options') }} {# `options` are optional #}
{{ faker('name') }} {# returns 'Neal Considine' #}
{{ faker('firstname') }} {# returns 'Jaron' #}
{{ faker('lastname') }} {# returns 'Will' #}
{{ faker('paragraph') }} {# returns paragraph #}
{{ faker('text', '500') }} {# returns 500 words #}
{{ faker('words', '5')|join(' ') }} {# returns array of words. use `join` filter #}

I haven't tested with all the various properties, but the ones I have tried work.
Improvements and advice humbly accepted.
-Dave

Wow. That's cool :) Faker support has been on the list for a very long time. This is a really smart way of adding support. I'll try to find a more permanent home to document this.

I am really glad I found this post, thanks for all the info. Faker is amazing and I've been looking for a way to streamline our workflow. Faker seems to fit the bill. I discovered I can create a loop of data like:
 {% for i in range(0, 8) | slice(1, 4) %}

{{ faker('firstname') }} {{ faker('lastname') }}

{{ faker('paragraph') }}

    {% endfor %}

In addition, I added placholdit as the default image provider as Faker now has an API for things like that. So after installing that, imageUrl defaults to placeholdit which is what we were using / manually writing in our JSON files before Faker.

Closed as "no change required". Alternatives are available.


az vm format-secret does not work

Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build) : MSI
CLI version (az --version)
chi@Azure:~$ az --version
azure-cli (2.0.27)
OS version : Windows 10
Shell Type (e.g. bash, cmd.exe, Bash on Windows)  : bash on windows
I am following these steps to try to build a LINUX VM with SSL certificate.

https://docs.microsoft.com/en-us/azure/virtual-machines/linux/tutorial-secure-web-server

But I am failing at following step.

$ secret=$(az keyvault secret list-versions --vault-name WebSvrCert --name WebCert --query "[?attributes.enabled].id" --output tsv)

$ echo $secret
https://websvrcert.vault.azure.net/secrets/WebCert/d79e98486c5f42a6a12c863608b3a6e8

$ vm_secret=$(az vm secret format --secret "$secret")
Parameter 'resource_group_name' can not be None.
Traceback (most recent call last):
File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 194, in invoke
cmd_result = self.invocation.execute(args)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 331, in execute
six.reraise(*sys.exc_info())
File "/opt/az/lib/python3.6/site-packages/six.py", line 693, in reraise
raise value
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 304, in execute
result = cmd(params)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 168, in call
return super(AzCliCommand, self).call(*args, **kwargs)
File "/opt/az/lib/python3.6/site-packages/knack/commands.py", line 109, in call
return self.handler(*args, **kwargs)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/init.py", line 348, in default_command_handler
result = op(**command_args)
File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/vm/custom.py", line 1429, in get_vm_format_secret
'id': _get_vault_id_from_name(cmd.cli_ctx, client, vault_name)
File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/vm/custom.py", line 1406, in _get_vault_id_from_name
vault = client.get(group_name, vault_name)
File "/opt/az/lib/python3.6/site-packages/azure/mgmt/keyvault/operations/vaults_operations.py", line 193, in get
'resourceGroupName': self._serialize.url("resource_group_name", resource_group_name, 'str'),
File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 488, in url
data = self.validate(data, name, required=True, **kwargs)
File "/opt/az/lib/python3.6/site-packages/msrest/serialization.py", line 558, in validate
raise ValidationError("required", name, True)
msrest.exceptions.ValidationError: Parameter 'resource_group_name' can not be None.
chi@Azure:~/clouddrive$

My suspicion is that the keyvault is not visible to the subscription you are logged into. Regardless, it should not throw up a stack trace.

Thank you for the advice,
But I am creating the key vault and run this command against the same subscription…
Do you have any other suggestions?
Thank you.
-chiyao-
From: Travis Prescott<EMAIL_ADDRESS>Sent: Saturday, March 3, 2018 1:46 AM
To: Azure/azure-cli<EMAIL_ADDRESS>Cc: Chi Yao<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/azure-cli] az vm format-secret does not work (#5718)
My suspicion is that the keyvault is not visible to the subscription you are logged into. Regardless, it should not throw up a stack trace.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli%2Fissues%2F5718%23issuecomment-369978598&data=04|01|chiyao%40microsoft.com|233eeca4833a49dd630808d5805d0dfe|72f988bf86f141af91ab2d7cd011db47|1|0|636556059511671323|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=LlHcbQXoipvBgAq3lzeKh1t5ouCnIgffWZ5KQfc4Jzs%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAa5ywcAZ_nw6in-XO0SNXj7uqUgKcJqSks5taXc7gaJpZM4SZZO0&data=04|01|chiyao%40microsoft.com|233eeca4833a49dd630808d5805d0dfe|72f988bf86f141af91ab2d7cd011db47|1|0|636556059511671323|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=Ly4fA%2F7hshI3uJfxDoS4LkIpmjV%2BZqvGtt6kbdwaMwo%3D&reserved=0.

I get the same stack trace using your secret ID, but that's because the vault isn't in a subscription I can access. I'll take a look.

Hello,
Is there an update for this issue?
Thank you.
-chiyao-
From: Travis Prescott<EMAIL_ADDRESS>Sent: Wednesday, March 14, 2018 1:19 AM
To: Azure/azure-cli<EMAIL_ADDRESS>Cc: Chi Yao<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/azure-cli] az vm format-secret does not work (#5718)
I get the same stack trace using your secret ID, but that's because the vault isn't in a subscription I can access. I'll take a look.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli%2Fissues%2F5718%23issuecomment-372725132&data=04|01|chiyao%40microsoft.com|af203875cfe14d14713a08d588fe1f14|72f988bf86f141af91ab2d7cd011db47|1|0|636565547381064115|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=25Kzy9pgXk%2FTiMrsDkt7p38VAimXNslEmzLbJFVumEc%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAa5ywWRofDwHZ-wVUMrwCbO8xAoDaCUCks5td_FvgaJpZM4SZZO0&data=04|01|chiyao%40microsoft.com|af203875cfe14d14713a08d588fe1f14|72f988bf86f141af91ab2d7cd011db47|1|0|636565547381064115|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=LA5Fq6CvA8a6CAF%2Bl3O3%2FwXPZFK4Md7O4WXN6WwI36o%3D&reserved=0.

It's scheduled to be addressed next sprint.


Negative number of test results

Hi ,
Thanks for you great m-lsd  work!
I use it to detect my own pictures,and save the Endpoint coordinates.There  are  small amount of Endpoint coordinates is Negative number .I find the mistake only happened  at  x_start.Can you give me some information of it .Thanks a lot
a result file
1403638127445096960.txt

Hi.
I guess it's due to a wrongly predicted center point.
A displacement vector for the center point is predicted by reasoning both ends of a line segment relative from the center point, but if the predicted center is located far from one side and the other endpoint is out of an image, it tries to find a virtual point outside the image.

Thank you for your prompt reply before the Lunar New Year！
I'd be happy to provide pictures for you to test .   I use mlsd  on EuRoC dataset （MH03  MH04） score_thr=0.4, dist_thr=10;
If you have an idea to solve this problem, please let me know.Thanks a lot.

Happy lunar new year too!
I've tested your examples and found it as I guessed before.
You can easily solve this problem.
If you'd like to fix a predicted line segment outside of an input image, just find an intersection between the predicted line segment and an edge of the image in the image coordinate system.
Note the edge of the image is one of these lines; (0,0)~(0,h), (0,h)~(h,w), (h,w)~(0,w), (0,w)~(0,0)
You can refer to https://github.com/navervision/mlsd/blob/master/utils.py#L176-L195 for this.


Some misconception in implementations

Hi,
currently I am learning TensorFlow ( I left Torch for a while because some stuff is hard to evaluate) and I'm looking closer to this project. I have find some misconception I think in the training code:

DeConv layer should be attach to any other layer but not the outputs from the last one (exactly like the DropOut). In the code it is attached to 'logits', so it is a 'bad way' of using it. It should be attached to 'prelogits' to decorrelation features, not class predictions.

In the Inception-ResNet there a dropout. It is wrong place to use it when we use "CenterLoss". Why? Because with DropOut the CenterLoss get features with missing fields. As we learn one 'center' per class, this could harm the performance.  The DropOut could be applied between prelogits and logits, named ex. prelogits_dropout (where still prelogits go to CenterLoss or DeConvLoss)

I have a plan to reimplement and get ~same results like Caffe implementation of Center-Loss (~98.6 %). But first I need to make some changes in training code. But I must say that TensorFlow with Tf.Slim is a really nice library. And implementing the loss function is really easy :)

Hi @melgor,
Thanks for sharing your ideas!! Looking forward to hear about your progress...
Regarding the Deconv loss the implementation in the repo is not very good, and I don't think it's possible to use on a bit larger datasets due to its memory inefficiency. Although I like the concept of penalizing correlated features I should probably remove the implementation from the repo for now, but I haven't gotten that far. ;-)
Do you mean that the estimate of the center gets noisy due to dropout? Could very well be like that, and it would be interesting to try out the solution that you propose. Currently my GPU is occupied with training on the MsCeleb-1M dataset (looks promising so far) but there is quite some parameter tuning to be done...
It will be interesting to hear about your findings!! Lately I have seen that it's possible to improve performance from 0.984 to 0.988 by training on a selected subset of the CASIA-WebFace dataset (and no facescrub). So even a relatively clean dataset as CASIA still may have enough label noise to drag down the performance a bit. That was a surprise to me at least. :-)
Yes, Tensorflow is a fantastic tool and seems to improve for every release...

I like the idea of DeConv too because when you are learning the features using CenterLoss or TripletLoss, you can not use DropOut. DeConv resolve that problem so it would be nice to check if this works ok.

Do you mean that the estimate of the center gets noisy due to dropout?
Yes, exactly this what I'm taking about. I think that it may hurt the performance as CenterLoss penatialize  fields which was masked by DropOut earlier. You may try it later with DropOut after 'prelogits'.

! Lately I have seen that it's possible to improve performance from 0.984 to 0.988 by training on a selected subset of the CASIA-WebFace dataset (and no facescrub).

This is a scenario which I want to reimplement. The people of Caffe repo with CenerLoss claim that with clean dataset (and merged features from original image and flipped) they get nearly 99%. Using Caffe, I get 98.6% (original + fliped images), so at least I have a right database. If everything will be working, then I will try to learn model with cleaned version of CASIA.
Again, thanks for fantastic code. It is much easier to learn new stuff when you have such nice code to read!

Also felt strange when I read the DropOut and DeConv part, but I wasn't confident due to the unfamiliarity with TensorFlow. Thanks @melgor for pointing them out and making good discussion here.
I have a similar question here:
In the original FaceNet paper, the embedded features are L2-normalized. In facenet_train_classifier.py, the features are normalized in the evaluation phase, but not in the training phase. Is it also some kind of consideration to the dropout layer?
Two related experiments had been made:

I removed the L2-normalization op in evaluation phase and the performance crashed.
I added a L2-normalization op in the training phase and the performance is slightly worse (Acc: 0.980=>0.978) after 100k iterations on VGGFace.

@davidsandberg is working on selecting subset of CASIA-WebFace to get better performance. From #48 I could understand that the process was done by filtering out the images far from centers (correct me if I'm wrong). But what's the intuition behind? Is it to remove correct samples with rare appearance or to remove the wrong-labeled images?
If the answer is the former one, wouldn't this process remove the semi-hard examples for triplet-loss training?
If the answer is the latter one, I found a "cleanlist" (haven't found the details how they produce it) for CASIA-WebFace in this webpage. Maybe you can run an experiment based on this list and compare the resulting performance.

@ugtony  About your experiments, it was right behavior based on theory:

At evaulation stage we want to have all features to be normalized because we calculate the distance between them. If the normalize the using L2-Norm, the maximum distance is 4. When you remove the normalization, your maximum distance between features is Unknown (it could be 100k). So, L2-Norm during evaluation is useful because we can test a range of threshold between 0-4.
When you apply the normalization during training, the input features to 'Classifier' are normalized. I think that this should not hurt the performance a lot.

About CASIA: The clean version removed the  wrong-labeled images.  I think that correct-labeled should stay even they are rare. I will be using the clean_list which you send. But maybe a data-filtering done by @davidsandberg  works better, I will see this later.

Thanks for melgor's reply. I'm curious about why normalization during training didn't make the performance better.
The question about filtering and suggestion to test on cleaned CASIA dataset is actually for davidsandberg, sorry for misleading.

@ugtony The intention is to filter out the wrong-labeled examples, but unless the model is perfect already it will not only filter out wrong labeled examples but also correctly labeled examples that can be considered more difficult. And if too many of the difficult examples are removed the training will suffer, so the question is if this procedure can result in a better model. I think the answer is yes.
It should be noted that the final goal is not to clean the CASIA dataset (I'm even using the maxpy cleaned version of casia, so it should be quite clean already) but to apply it to the MsCeleb dataset. This dataset is "intentionally" very dirty and sifting through 8M+ images manually is not anything that I would like to do. ;-)
I tried two ways of cleaning the dataset. The first one assumed that there are some identities that are more dirty than others and removing the classes with the largest intra-class variance we get a "cleaned" dataset. I removed the 10% "worst" classes but got the same performance as the full dataset.
The second method instead removed the 10% of the images that was furthest away from the respective class center. This approach seemed to work well on casia and I'm now trying it for cleaning the MsCeleb dataset.

Thanks for the explanation. Looking forward to seeing your result on the MsCeleb dataset.
Do you know where does maxpy casia come from? I googled it and found some download links. But none of them mentioned how it is made or made by whom.

I will continue my finding in code here:

The Implementation of CenterLoss in not proper and I know why:):

def center_loss(features, label, alfa, nrof_classes):
    """Center loss based on the paper "A Discriminative Feature Learning Approach for Deep Face Recognition"
       (http://ydwen.github.io/papers/WenECCV16.pdf)
    """
    with tf.variable_scope('center_loss'):
        batch_size    = tf.cast(tf.shape(features)[0], tf.float32)
        nrof_features = features.get_shape()[1]
        centers       = tf.get_variable('centers', [nrof_classes, nrof_features], dtype=tf.float32,
                            initializer = tf.contrib.layers.xavier_initializer())
        label         = tf.reshape(label, [-1])
        centers_batch = tf.gather(centers, label)
        loss          = tf.nn.l2_loss(features - centers_batch)
        loss          = tf.truediv(loss, batch_size)

    return loss, centers

I added two lines:
batch_size    = tf.cast(tf.shape(features)[0], tf.float32)
loss          = tf.truediv(loss, batch_size)

Why? -> I look like  tf.nn.l2_loss does not use reduce_mean function but reduce_sum. This cause that it sum all L2_Loss from each example and not divide by the batch_size. I checked this implementation with my using Numpy and it is correct. Additional, I remove code for updating the centers and I treat is a learnable variable (like in caffe-face repo).
This changes enable learning the model with CenterLoss with same parameters like in Caffe.
If @davidsandberg  you agree with this, I can send a PR.

I was investigating the speed of TF is in all scenario it was much slower than Torch. Based on some reading, NCHW is faster than default NHWC. And it is (faster by ~30%). I do not need a lot of changes to use it. Just:

image_batch = tf.transpose(image_batch, [0, 3, 1, 2])

after getting the image_batch variable.
When creating the mode adding data_format = "NCHW":
with slim.arg_scope([slim.conv2d, slim.max_pool2d, slim.avg_pool2d],
                                stride=1, padding='SAME', data_format = "NCHW"):

And for Inception layers, concat must be done by 1 channel
mixed = tf.concat(1, [tower_conv, tower_conv1_1, tower_conv2_2])

TensorFlow is very slow for not-optimized operation. I was trying to learn the model using PReLU:

'''Parametric activtion function'''
def PReLu(x, idx, name = "PReLu"):
    # If incoming Tensor has a scope, this op is defined inside it
    i_scope = ""
    if hasattr(x, 'scope'):
        if x.scope: i_scope = x.scope

    with tf.variable_scope(i_scope + str(idx) + name) as scope:
        alphas = tf.get_variable('alpha', x.get_shape()[-1],
                           initializer=tf.constant_initializer(0.0),
                            dtype=tf.float32)

        x = tf.nn.relu(x) +  tf.mul(alphas, (x - tf.abs(x))) * 0.5

    return x

The network with PReLU is ~30% slower and take 10-20% memory. So testing my desire architecture is really painful. Currently I decide to use ReLU, maybe I will take a single test with 'ELU'. Any suggestion?

I make a single test using Face-Res architecture with "PReLU" to validate the training with Torch. And in Torch I get ~81% of accuracy, but in TensorFlow 75%.  I'm do not know why it happen but it look like in TensorFlow the network overfitt much more.  Maybe I miss some regularizarion? Some suggestion?

Hi @melgor
I agree with the scaling of the center loss. But I guess it shouldn't affect the results but just result in another "optimal" center loss coefficient. If you send a PR I will merge it.
For the data format this is very interesting. I didn't think it would have that big impact. I assume that it has to do with CUDNNs native data format. I will try the transpose when my GPU is free for something else. ;-) Thanks!

Hi again @melgor,
I tried to change the data_format to 'NCHW', but I didn't see any speed-up.
I'm running on a Titan X (Pascal) using Cuda 8 and CuDNN 5.1.5, and training takes ~0.38 seconds for one batch (90 examples).
Which setup did you use when you saw the speed-up?

I have GTX 980 (Cuda 8 and CuDNN 5.1.5). Maybe you could check the speed based on benchmark here : I changed the batch-size to 64.
This is the results from this benchmark:
NCHW:
Forward across 100 steps, 0.091 +/- 0.009 sec / batch
Forward-backward across 100 steps, 0.281 +/- 0.028 sec / batch

NHWC
Forward across 100 steps, 0.103 +/- 0.010 sec / batch
Forward-backward across 100 steps, 0.341 +/- 0.034 sec / batch

So it is the difference. I have pretty much same difference when using "ResNet" architecture. But I did not try the running "inception_resnet_v2", so maybe it is not true for all architecutures.

Hi @davidsandberg, just to confirm that the "label" in Centerloss function is dense label, not one hot vector type , right?

@heimanba89 yes it's dense label. tf.gather function requires indices


Adding mount with similar machine name causes error

When we have for example two machines with similar names:
- machine30
- machine30trans

And we want to mount these two machines with docker-machine-nfs there is a problem when:

firstly we try to mount machine30trans - OK
secondly we try to mount machine30 - FAIL with message:

exports:2: got host/group with no directory?: trans

after running above instructions /etc/exports looks like:
trans
# docker-machine-nfs-begin machine30
/Users <IP_ADDRESS> -alldirs -mapall=1440676671:1381159979
# docker-machine-nfs-end machine30

IMO it's because there is a bug somewhere in code responsible for replacing content in /etc/exports.
For example when we want to mount machine30 - script makes search and replace texts which begins with machine30 word and when there is already mounted machine30trans - script removes only machine30 text and left unnecessary trans word.
The problem doesn't occurs when we firstly mount machine30 and secondly machine30trans

I'll try to have a look at this as soon as I can. If you have a fix don't feel free to provide a PR.

Can you try #67

Works great, problem solved :)


Fixs web interactions

Fixs to prevent the navigation engine from attempting interactions with non interactible web elements
set a windows head-mode user agent, to circumvent websites basic bots countermeasures

answer (but maybe not solve) #380 #243 #330


Remove 'lodash' dependency

Lodash is a considerable part of our build right now but we only use it for its "find" function.
We should either replace it for native javascript or just depend on that individual lodash function.
This would increase performance, especially if we're trying to start targeting mobile users more.

Hi !
I just started contributing to open source project and I stumbled upon this issue.
Here's the PR : #79

Hi !
I just started contributing to open source project and I stumbled upon this issue.
Here's the PR : #79

@sirbeefalot this has been addressed at https://github.com/beefyfinance/beefy-app/pull/79 and merged, lets close this~

@sirbeefalot this has been addressed at https://github.com/beefyfinance/beefy-app/pull/79 and merged, lets close this~


aws-otel-collector container stopping due to deprecation errors

The workshop CONFIGURE AMAZON MANAGED SERVICE FOR PROMETHEUS / GRAFANA FOR YOUR ECS CLUSTER is not showing metrics under Grafana dashboard.
Checking the ECS Task, I realised that the aws-otel-collector container was being stopped with exit code 1. Looking at the logs, I've seen these 2 errors below:
Error: failed to get config: cannot unmarshal the configuration: unknown exporters type "awsprometheusremotewrite" for "awsprometheusremotewrite" (valid values: [awsxray awsemf signalfx datadog otlphttp logzio logging otlp prometheusremotewrite prometheus file dynatrace sapm])
2022/11/02 11:59:43 application run finished with error: failed to get config: cannot unmarshal the configuration: error reading processors configuration for "metricstransform": 10 error(s) decoding:  'transforms[*]' has invalid keys: metric_name
Both erros are related to Open-telemetry deprecations as per below:

Remove deprecated metric_name setting #12737 - It was replaced by include as per the docs
awsprometheusremotewriteexporter deprecation notice - Users who want to send metrics to Amazon Managed Service for Prometheus will need to instead use the Prometheus Remote Write Exporter along with the Sigv4 Authenticator Extension to achieve the same result.

That all said, I'm opening a PR with the fixes for the above.

PR opened on ECS Demo Amp repository.

Fixed with https://github.com/aws-samples/ecsdemo-amp/pull/4


Curious about status / outdated dependencies

This looks like a really nice API for building REST services. Unfortunately, it appears that some dependencies are pretty outdated, and I'm having trouble getting it installed via Stack as a result. (This is probably really easy to work around for some, but I'm a total Haskell rookie.) Is there still any interest in this project, or should I be looking at some alternative instead?

Thanks very much for the response. I will check out Yesod for now.


Feedback button on Azure REST API doc browser and Azure template doc browser

Is your feature request related to a problem? Please describe.
The Azure REST API and template reference docs are great
But sometimes, there are issues. For example, right now, at https://docs.microsoft.com/en-us/rest/api/compute/disks/get there is no mention of the 'diskState' property, which is being returned by the API.
What is frustrating is that there's no feedback button on these pages for us to flag the issue.
Describe the solution you'd like
Add feedback buttons to the Azure REST API reference docs and Azure template reference docs
Describe alternatives you've considered
n/a
Additional context
n/a

@jtuliani thanks for your feedback. We have on-going plan to enable feedback button to reference page (not only for REST), but it would be transitioned step by step and need content writer's collaboration. I'm connecting @rloutlaw on this. Before we enable that, welcome to open issue here at the general feedback.

Thank you @hellosnow, I'll do that :)

@rloutlaw Any updates about this feedback?

@jtuliani @Powerhelmsman I have opened up an internal tracking item to better provide user feedback when looking through our reference documentation. Thanks for the feedback and bringing this need to our attention.


Unique annotation

It would be nice to have a unique annotation. It could possibly enforce unique constraints, but also it could open up the get query to allow querying by fields besides just id. It would also allow us to update entities by the unique fields.

Very good idea! We can do that for Postgres with low effort.


rosidl_tutorial

connects to https://github.com/ros2/tutorials/issues/1

@mikaelarguedas any idea why has this not been merged in? It seems to be something that goes with https://github.com/ros2/ros2/wiki/Rosidl-Tutorial, no?

@dejanpan this hasn't been merged because we decided to not release or test the tutorials repository until we had a documentation platform able to consume it. Another reason for not merging it being that we want to properly separate what should go in "demos", in "examples" and in "tutorials" to avoid untested duplicated code.
You are right the Rosidl-Tutorial wiki page assumes that this PR has been merged and result in pointing to a 404 :/. I'll try to iterate on this soon by either getting a simple tutorial merged or updating the wiki page accordingly. Thanks for pointing it out!

I am just trying to understand how the whole rosidl_* machinery works. Tutorial above is the best documentation so far, right?
If there is anything I can do to help, let me know.

@dejanpan While this branch is pretty outdated and the code may not compile as is anymore, it gives a good idea of how to create and use messages. Though it doesn't dive into how the rosidl_* generators work or how they are chained if that's the information you are looking for. I don't think we have a resource explaining this in detail.
the rmw design document goes over how data structures are converted to DDS and the notion of type support. The generators are mostly here to create ROS data structures that are convenient to use for ROS users and can be converted leveraging the typesupports mentioned above. The ROS types are defined in this design document. Their mapping from ROS types to DDS type is explained here. Finally the mapping from ROS types to language specific types are described for C++ and Python
Hope this helps

Thx a lot, this is wonderful. Code compiles with a current beta2.
On Tue, Sep 12, 2017, 10:13 AM Mikael Arguedas<EMAIL_ADDRESS>wrote:

@dejanpan https://github.com/dejanpan While this branch is pretty
outdated and the code may not compile as is anymore, it gives a good idea
of how to create and use messages. Though it doesn't dive into how the
rosidl_* generators work or how they are chained if that's the information
you are looking for. I don't think we have a resource explaining this in
detail.
the rmw design document
http://design.ros2.org/articles/ros_middleware_interface.html goes over
how data structures are converted to DDS and the notion of type support.
The generators are mostly here to create ROS data structures that are
convenient to use for ROS users and can be converted leveraging the
typesupports mentioned above. The ROS types are defined in this design
document http://design.ros2.org/articles/interface_definition.html.
Their mapping from ROS types to DDS type is explained here
http://design.ros2.org/articles/mapping_dds_types.html. Finally the
mapping from ROS types to language specific types are described for C++
http://design.ros2.org/articles/generated_interfaces_cpp.html and Python
http://design.ros2.org/articles/generated_interfaces_python.html
Hope this helps
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ros2/tutorials/pull/10#issuecomment-328920999, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AABD4F8T0nGpFccQ7z3GhNNnouAM60KFks5shrvAgaJpZM4LPdbZ
.


Properly return in examples

See https://github.com/strydercorp/handshake/pull/4399#discussion_r64497092

Wrong base. Sorry about that.

@bmchrist --  did you want me to review this?

@jasonfb No, sorry about that - meant to open that on our fork - most of the stuff in here is pretty specific to how we use it - maybe we'll push some it upstream one of these days.
Sorry!


Error on folded scalar in code mode

YS errors on block sequences and flow collections when in "code mode".
The reason is twofold:

They aren't really valuable in code mode
When you see them, you must be in data mode. Improves recognition when reading YS code.

Folded scalars arguably fall into the same are (although they aren't very useful in data mode either).
Goal: Error on folded scalars in code mode.

Done in 3041f2778


Any reference to scaleModes is done via using the enums setup

Description of change
https://github.com/pixijs/pixi.js/issues/5831
Pre-Merge Checklist

[x] Documentation is changed or added
[x] Lint process passed (npm run lint)
[x] Tests passed (npm run test)

Codecov Report

Merging #6489 into dev will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #6489   +/-   ##
=======================================
  Coverage   80.14%   80.14%
=======================================
  Files          57       57
  Lines        2825     2825
=======================================
  Hits         2264     2264
  Misses        561      561

Impacted Files
Coverage Δ

packages/canvas/canvas-graphics/src/Graphics.js
95.45% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fb16e7b...af9e0d9. Read the comment docs.


[img] transfer the img to the two pointers notes

make sure the file name is preserved @gw-billy

DONE - NO PIC


Problem with get metadata from SciCat

Get metadata from scicat using ingestor login and the password is not functioning after scicat was migrated to the new backend.
With the help of IT guys, I could adopt /sed/sed/loader/flash/metadata.py file to use my own token generated by SciCat.
Also config.yaml file has to be changed with metadata:scicat_token:"eyJhbGciOi...." #copied from the SciCat front-end webpage (User/Settings/SciCat Token).
Thus:

pros - no need to make public credentials of ingestor;
cons - token is not a long leaving thing (probably just a few hours) and has to be recopied from SciCat
Somehow I didn't manage to push changes to github directly, so metadata file is attached.
metadata.txt

This is probably something to discuss with the admins of that server. How is it handled for automatic igestion, e.g. from the DAQ? What account is being used there? A r/o accound would be sufficient for our use case. If there is no permanent solution, then maybe a cell that sets the respective config item in a template notebook would be a good idea, where users can copy paste their Token.

I was trying to make the local branch get-metadata-from-scicat-using-token and push it on github, but got 403 error :)
Concerning lifetime - let's see what will be an answer from scicat guys, but I could not find any easy set for this via the web interface.

I was trying to make the local branch get-metadata-from-scicat-using-token and push it on github, but got 403 error :)

It seems you weren't having write permissions. I have added you now. But otherwise, I also made a PR #347 to address what you put

Now the token can be added as a keyword argument when calling the sed processor.

After discussing with ITs, it seems that we can also use the build conda package for SciCat
conda install -c https://hasfcpkg.desy.de/doocsconda scicat
which already exists as the default package inside of the "flash" kernel on Maxwell.
It still uses ingestor credentials to get metadata, but it also has the power to ingest/update metadata inside SciCat.
This way we can do:

from scicat import ScicatClient
scicat_ingestor = ScicatClient(proposal_id, source_folder)
run_info = scicat_ingestor.get_dataset(runNumber)

This way we don't need to worry about refreshing the token every time you want to read metadata and also no danger of spreading credentials around, but for sure it opens a way to mess up something within SciCat if someone decides to ingest/update existing data - in this case, there is still version tracking and history available from SciCat side.
Another good side of that package is that everyone will be able to create a run summary table at any time:
scicat_ingestor.create_csv_run_table(filepath)

Preferably, ingesting should be seperated from fetching the metadata. As you said, with the Scicat package, the users can ingest data and update something. I'd suggest the ingestor is used during beamtimes but for later, the users should use their own credentials to fetch metadata (The branch is merged to main now)
The run summary table sounds interesting. Probably a fix to the question we had about fetching metadata for multiple runs at once?


Please try with snapshot

Howdy, there is related issue: https://issues.apache.org/jira/browse/MGPG-136
And PR: https://github.com/apache/maven-gpg-plugin/pull/120
And deployed version to apache-snapshots https://repository.apache.org/content/repositories/snapshots/ is 3.2.7-20240923.142458-3
Please try with this version:

with version bump all should happen again, PR makes plugin "behave as before" by default.
The try to add to <passphraseLf>false</passphraseLf> to NOT append anything to passphrase (see MGPG-99 for context)
Then try to add (if needed, like gpg spawned process hangs) <passphraseLf>true</passphraseLf> that will append LF ONLY (unlike before on Windows CRLF, where GPG expects LF only, and CR most probably caused the pw corruption).

Thanks!

Fixed with #30


UTF-8 problem with (french) accented characters

Hi
Just found that when a Autobahn.js client (HTML page charset is UTF-8) make a RPC call on an PHP client procedure, the accented characters are "encoded" like this :
Autobahn client send : é
PHP Client receive : Ã©
A Deserialization exception occurs in RatchetTransportProvider and I must do an utf8_decode on the string.
The problem is in Thruway\Serializer\JsonSerializer::deserialize line 37
Perhaps it's normal, but it seems weird.
Thanks

I am not an expert on UTF-8.
I did give your example character a try and it appears to work fine in my setup. The Ã© maps to ISO-8859 characters for the UTF-8 encoded é. (In my observation, you could see Ã© being sent in the network traffic in Chrome and then Thruway sending it back as \u00e9). Both, according to my limited research, are valid encodings in JSON (I am assuming the network tab in Chrome just doesn't display UTF-8). I do not experience the serialization errors you are experiencing.
A couple of ideas:

Make sure your character set in PHP is set to UTF-8 (you can check on your unix-based machine with php -i | grep default_charset
I am not sure where you are seeing Ã© - if it is in a terminal, check to make sure the terminal supports UTF-8

Please let me know if this helps.

Hi
My blacklist class was the problem. mb_strtolower needs the the 2nd arg (encoding) or the string is breaked.
I am really sorry...


insufficient c5n.18xlarge capacity

Environment:

AWS ParallelCluster / CfnCluster version 2.4.0
OS: centos7
Scheduler: SGE
Master instance type: c5n.xlarge
Compute instance type: c5n.18xlarge

Bug description and how to reproduce:
When I try to create a parallelcluster using c5n.18xlarge compute nodes (config file attached) I get an error saying that:
usr/bin/pcluster create meredithk-test3
Beginning cluster creation for cluster: meredithk-test3
Creating stack named: parallelcluster-meredithk-test3
Status: parallelcluster-meredithk-test3 - ROLLBACK_IN_PROGRESS
Cluster creation failed.  Failed events:
  - AWS::AutoScaling::AutoScalingGroup ComputeFleet You must use a valid fully-formed launch template. We currently do not have sufficient c5n.18xlarge capacity in the Availability Zone you requested (us-east-1a). Our system will be working on provisioning additional capacity. You can currently get c5n.18xlarge capacity by not specifying an Availability Zone in your request or choosing us-east-1b, us-east-1d, us-east-1f, us-east-1c. (Service: AmazonAutoScaling; Status Code: 400; Error Code: ValidationError; Request ID: 2996fd43-acc0-11e9-bf48-6b6dbfbd66b4)

I don't know how to switch to us-east-1b or other Availability Zones.  Also, it appears that there should be some c5n.18xlarge nodes available, as the recent spot pricing indicates that there are nodes available.
config.txt

Hi @karlvirgil,
You can switch the AZ by switching the master_subnet_id, go to VPC Console > Subnets.

This is also dependent on your placement group, so even if there's sufficient c5n capacity there may not be sufficient capacity in a placement group. Try switching the master_subnet_id to another az.

Okay, I need to add these other subnets then, because in my setup I only have a subnet for us-east-1a.  I need to get my networking guys to give me a new block of IP addresses for the new subnets I guess.

Unfortunately yes, c5n.18xlarge are very capacity constrained at the moment.

I don't understand why the spot price for each AZ doesn't reflect the fact that they are capacity constrained.  I would have expected the spot price to spike up once there are no more nodes available...

Hi @karlvirgil
were you able to solve the issue by selecting a different az?

I am in the process of having my IT department give me new blocks of IP
addresses so I can create new subnets associated with the other AZ.  Once
they create these (the process seems to be slow) then I'll give it a try.
On Mon, Jul 29, 2019 at 11:14 AM Francesco De Martino <
<EMAIL_ADDRESS>wrote:

Hi @karlvirgil https://github.com/karlvirgil
were you able to solve the issue by selecting a different az?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws/aws-parallelcluster/issues/1216?email_source=notifications&email_token=ABT2AWPUJ7SNAW7DVEPBQGDQB4CM5A5CNFSM4IF4W7C2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3BA6TI#issuecomment-516034381,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABT2AWPP4DTV5DFUVTOOWATQB4CM5ANCNFSM4IF4W7CQ
.

--

梅豪偉

Thanks for the update. I'm going to close this one since there is no further action we can take at the moment but feel free to reopen in case using a different AZ does not solve your problem.


server passwd output is misaligned

$ nats server passwd
? Enter password [? for help] **********************
                              ? Reenter password [? for help] **********************

$2a$11$DAFPlebnj555dIvBLDk9D.Zi48QcXbdRd8JhsfpApgG4Hn9FO4cjC

This is what I use using Bash 5.1.4 and Gnome Terminal 3.38.2.

Hmm. Not much I can do to fix that it’s all the survey package. Hmm. Maybe I will put a Println() between on other terminals it would be a extra blank.
Can you try running “reset” and see if it still does it?

Okay, just tried running reset before nats server passwd, but the result is the same.

https://github.com/nats-io/natscli/pull/64

$ nats server passwd
? Enter password [? for help] **************************************
? Reenter password [? for help] **************************************

$2a$11$qP8.dpLZRLH8uzlnTdmXlOi.2p26byo.RZSv/1CRO7MUMYr7lpHlW

Nice, works for me! 🎉


Added import image, get import image task status, is_image_import_completed and copy image

This all works with boto3 ec2 connection.
Tested on many AWS regions.
Maybe image_import should return tuple? ImageID and TaskID?

Lint report for commit d40a77fa60c05f6b333a9d1a38380bc15ac605e6:
mgmtsystem/ec2.py:

:red_circle: Line 562:29: F841 local variable 'ex' is assigned to but never used
:red_circle: Line 587:29: F841 local variable 'ex' is assigned to but never used

Please, rectify these issues :smirk: .
No commit flaws detected.
CFME QE Bot


“eval” can delete variables from namespaces

Here is how I can use eval to delete a variable from an existing namespace:
⬥ var tt: = (ns [&foo=bar])
⬥ echo $tt:foo
bar
⬥ eval 'del foo' &ns=$tt:
⬥ keys $tt:
⮕ foo
⬥ echo $tt:foo
Exception: variable $tt:foo not found
[tty 5], line 1: echo $tt:foo
⬥ echo $tt:[foo]
Exception: no such key: foo
[tty 6], line 1: echo $tt:[foo]

This does not seem desirable. Especially the inconsistency, with the variable still listed as a key of the namespace.
This trick also lets me delete a variable from a module namespace introduced by use.

Ugh! Unsurprisingly this can be used to delete variables defined in builtin namespaces but it also no longer shows up in the output of keys:
> keys $edit: | to-lines | grep command-duration
command-duration
> echo $edit:command-duration
0.000345857
> eval 'del command-duration' &ns=$edit:
> echo $edit:command-duration
Exception: variable $edit:command-duration not found
> keys $edit: | to-lines | grep command-duration
Exception: grep exited with 1

Note that fixing this implies that @hanche's attempt to use eval as a substitute for use won't work. This example by @hanche on IRC earlier today will stop working if eval is changed to conform to its documentation with respect to "namespace specified by &ns is never modified; it will not be affected by the creation or deletion of variables by $code":
var bar:
eval (slurp <foo.elv) &ns=(ns [&]) &on-end={|ns| set bar: = $ns }

@krader1961 I don't think that's right. Let me explain why.
But first: I incidentally discovered that if the eval'd code creates a new variable, a del does not affect the original namespace! Here is a working example, which acts just like I had expected it to:
⬥ var orig: = (ns [&x=xxx &y=yyy])
⬥ eval 'del x; set y = new; var z = zzz' &ns=$orig: &on-end={|ns| set new: = $ns }
⬥ keys $orig:
⮕ x
⮕ y
⬥ put $orig:x $orig:y
⮕ xxx
⮕ new
⬥ keys $new:
⮕ y
⮕ z
⬥ put $new:y $new:z
⮕ new
⮕ zzz

Indeed, the way I thought it was supposed to work, and which the above experiment seems to confirm, was as follows:

The eval'd code operates not exactly in the provided namespace, but in a clone of that namespace.
Hence any alterations to that namespace will not be seen in the original (but altering the value of a variable will produce visible effects in the original)
Finally, the altered namespace is provided as an argument to the on-end function

And this is exactly how I imagine the loading of a module works, except the initial provided namespace is always empty. And instead of the on-end mechanism, the language caches the namespace and assigns it to a suitable variable in the namespace of the caller. In particular, my “eval trick” should still work.
I think today's discovery offers some insight into what is happening behind the curtain, but I won't speculate as to exactly what is going on.

But first: I incidentally discovered that if the eval'd code creates a new variable, a del does not affect the original namespace!

Yes, that is my point. The documentation (as I write this) states that a namespace cannot be modified by adding or deleting a variable in the namespace. That is not true. A namespace can be modified but doing so creates a new namespace. Which is (or should be) only useful in the context of something like eval where you can capture the new namespace. My prior comment(s) were meant to point out that the documentation is misleading or the current implementation of namespaces does not conform to the documentation.


PropertyUnits not linting?

The PropertyUnits linter in 0.37.0 doesn't seem to be linting for me...
# scss-lint.yml

PropertyUnits:
    enabled: true
    global: ['em', 'rem', '%'] # Allow relative units globally

// test.scss

.widget {
   width: 10px;
}

See #422


LoRA layers not present in UNet

There is an exampe for lora but the layers are not present in the Unet class.

There are two ways to implement LoRA. One is to have aux multiplications done at runtime along with the main one (a.k.a. LoRA layers). Another is to merge the weights first. We opt to merge the weight to keep the network the same so we can reuse things such as CoreML too.

Okay thanks