max and min validators behaves inconsistently

The max and min validators used to ensure that a number is within range behave inconsistently as the following snippet (using Assert.js)  demonstrates:
    assertEquals("Test must be less than or equal to 123",
            Backbone.Validation.validators.max("54,03",'test',123,{}));  // OK
    assertEquals("Test must be less than or equal to 123",
            Backbone.Validation.validators.max("54,003",'test',123,{}));  // gives undefined

Note the use of comma instead of a punctuation separator in the 54,03 strings.
In summary, 54,03 is correctly rejected as a number but 54,003 is incorrectly accepted as a number.

What's the best way to fix this, anyway? Seems like the proper answer to anything involving number punctuation would require having some internationalization support. (Original poster asserts that 54,03 is not a valid number, which is true in the U.S. and other places, but is false at least in parts of Europe. In some places, comma and decimal/period/full-stop work exactly the opposite from how they do here, so 54,03 is how one would write fifty-four and three-hundredths.)
Personally, I think the smarter thing would be to change the parsing to somehow avoid any punctuation (for now) until some decision is made about internationalization support. But it's not my call.

This library checks if a string is number using defaultPatterns.number.
"54,003" is interpreted as fifty four thousands which is in accord to US convention.
To not accept such values is a matter of changing the pattern: https://github.com/thedersen/backbone.validation#adding-custom-patterns
So i think this can be closed as the user already can change the behavior to matches its use case

@blikblum I forgot about custom patterns. Yes, I agree, the user should just override the pattern here. (I think a case could still be made that punctuation should be ignored in the default pattern, but that's a separate issue.)

@platinumazure the "category" this comes close is a "buggy", but @blikblum pointed well a sweet solution.
I will close since a custom pattern resolves the case. Thank you guys. :smile:


crda: fix build with gcc6

Motivation for this change
fixes build with gcc6
related to #28643
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

back ported here: 39099cd9aefb731afb3cfb059cbd3c2321caaca4


client should preserve better state around idleness

Whenever a client enters/leaves a state of idleness (when configured to track such events via idleTimeout), it would be useful to record a timestamp.  This would aid postmortem debugging.

👋
On February 22, 2023, we released version 3 of this library. As a result, we are closing this issue/pull request.
Please see issue #839 for more information, including how to proceed if you feel this closure is in error.


Error: only root can modify 'usb0' config for real devices

Hello,
I'm getting this error when trying to add an USB HDD passthrough to an existing VM by adding this block to the resource:
  usb {
    host = "1058:0730"
    usb3 = true
  }

While the error is self-evident, I still don't understand why it happens since my API token id (root@pam!terraform) is associated with the root user and is configured with NO privilege separation.
Please advise.
Thank you.

I'm now running Proxmox 8 with provider version 2.9.14 and this is the full log output:
  Error: only root can set 'usb0' config for real devices

    with proxmox_vm_qemu.docker[0],
    on vm-1000-docker.monte.nunogodinho.com.tf line 1, in resource "proxmox_vm_qemu" "docker":
     1: resource "proxmox_vm_qemu" "docker" {

I don't know what to do since my API token is already associated with the root user.

I had a similar issue with PCI passthrough and I solved it by authenticating with the root username and password instead of with an API token.
I think the Proxmox team added a hard check in Proxmox 8 and now only the 'root@pam' user can pass through non-mapped PCI/USB devices.
In your case 'root@pam!terraform' is different from  'root@pam' and that's why you can't pass through the non-mapped USB device.

Thank you @n-asenov. I will try your suggestion.

It worked @n-asenov. Thank you!


[fix] Frida snippet empty method arguments

A problem that happens when copying a frida snippet, where no arguments appear in the snippet.
This problem occurs almost always after renaming methods which add comments above them, or adding comments yourself.
problem example:

after fix:

I found out why this happen after reading the code and debugging it using logs, and I saw that in the collectMethodArgNames function the getDefPos is used the get the function start, and starting from this position you try to find the function arguments, the problem is that this position dont get changed when the user renames functions above for example which adds comments and change the actual start position of the function (or when the user adds comments).
Instead of using the getPos I changed the logic a bit to just scan the class for this method variables until you have an amount equivlent to the method arugments amount.
I added the PR for this fix of course.

@Ran-Naor thanks for this workaround, but root cause of this issue need to be fixed anyway, because it can cause similar issues in other places. I will try to investigate and resolve it, so please wait couple days :thinking:

No problem, If you can fix the getDefPos it will be better :)

Ok, I commit a fix to resolve caching which cause usage of outdated method instance after code reload. Instead of using maps, I moved cached object into node it references, so it always will be point on correct one.
Anyway, just to be sure, @Ran-Naor please check latest unstable build :slightly_smiling_face:

From my tests it works great! thanks 😄


Forum: subscriptions for topic not working - notifications aren't sent

Or maybe they aren't needed here at all

@antaratma there are a lot of duplicate notifications already, it will make situation even worse. Also it isn't fitting in our system very well, since user can subscribe to profile based modules, but forum is just text post, which can't have subscribers by the design.

I agree that it's not in-line with standard "Following" that we have for profiles, but it's nonetheless significant, if not critical for most interest-based communities, where people don't care to follow or even recognise specific authors, but focus entirely on topics of conversations. Same applies to most messenger-style sites, where people join/follow conversation channels because they're interested in the topic.
Even on Facebook, lack of such functionality results in the problem that users add "F" comment to the post they want to follow because they are interested in comments or updates, and not interested in following participants. I think it's the kind of feature which was useful and we need to fix, debug and make it work right.


Keccak assembler can give a linker error in (obscure) situations

Remove the .globl directive addresses the issue.
A relocation of type R_X86_64_PC32 was generated which causes a linker warning in some (obscure) circumstances.
edit: updated based on comments from @dot-asm below.

Thanks for the clarification, it's been a long time since I delved into the linker's inner workings.
I believe the issue arose when OpenSSL was included inside another shared library.

@dot-asm your explanation makes perfect sense.  I agree the linker should be more forgiving of what isn't an error.

While we're at it, could you remove corresponding artefact from s390x module? There is local label (so that it wouldn't actually suffer from corresponding problem), yet it would be appropriate to not expose a symbol that is not exposed elsewhere.

I've removed the artifact.  I've not tested it.

The S390 code builds.

I've not tested it.

I did before suggesting the change.

Editorial note. I'd suggest to squash the two under "remove development atfefacts" subject. Message can of course clarify that specifically x86_64 was problematic.


core(tests): fix the db_equal assertion to include encrypted docs

As a result of https://github.com/lockbook/lockbook/pull/1175, @parth realized we weren't comparing documents when asserting db equality.

As per this discussion over Slack


AttributeError: 'NoneType' object has no attribute 'get_pipeline_params'

how solve it?

Do you use  model.get_pipeline_params() or model.get_pipeline_params ?

I used model.get_pipeline_params().
I ran print(model) and it returned 0, I don't understand why it's not getting the value in experiment.run()

Did you run experiment.run() successfully?


关于few shots的问题

非常感谢您杰出的工作和及时高效的代码分享。
我有一个小问题想请教您。
在openai处理prompt的代码中：
for i, paragraph in enumerate(filtered_paragraphs):
if constants.IMAGES_INPUT_TAG in paragraph:
image_introduction_paragraph_index = i
image_introduction_paragraph = paragraph
break
paragraph_input = params.get(constants.IMAGES_INPUT_TAG_NAME, None)
看起来似乎提示词的few shots部分并没有当成图像来处理。
请问few shots部分是直接作为文本输入的，还是我在理解代码时出现了问题。
因为没有购买游戏，所以我暂时无法运行代码来验证，如果您可以给我解答，我将非常感激。
再次感谢您非常精彩的工作。

Hi, a message that we call GPT-4V API is divided into 4 parts:

system message: Provide a comprehensive introduction to GPT-4V, mainly describing the games GPT-4V is currently playing and its role positioning.
user message part1: This is the text description part before the few-shot examples that include images, such as the current task definition and description. Clearly, it cannot be used as an instruction for the image. Therefore, following logical order, we place this preceding description of the few-shot example within a user message.
image introduction message：This includes some few-shot examples, images, and their instructions. Because few-shot examples may contain replies from GPT-4V as assistant messages, we combine the few shots and the new images and their prompts into this part of the message.
user message part2: This part is the specific prompt, for example, some observable information, historical information about reflection, including the constraints and formats of the output.

Finally, all the message items will be combined into a complete message to call the GPT-4V API to get response.
You can refer to the template of the prompts: https://github.com/BAAI-Agents/Cradle/tree/main/res/prompts
A minor note: To ensure everyone can participate in the discussion, it would be better if communication could be in English. Thank you very much for your attention and support.

Thank you very much for your timely reply and patient explanation. Fortunately, your explanation of the prompt handling process closely aligns with my previous understanding. However, there is still one issue that perplexes me:
In the section of your code related to the image introduction message:
paragraph_input = params.get(constants.IMAGES_INPUT_TAG_NAME, None)
if I understand correctly, it exclusively extracts the content tagged as IMAGES_INPUT_TAG_NAME (i.e., 'image_introduction') from params. Yet, I have not found the code responsible for extracting the "few shot" sections, such as those within decision_making.json.

The few_shots field is reserved for updating extensions. In the current version, this field is not used.
In the current version few shot examples are included in the image_introduction. So only the image_instruction field is parsed.
Thank you.


Outdated Screenshot for Secrets

The Security screenshot on this article does not match what the ARM template is asking for.
The template is also asking for:
Server Certificate Url
Server Certificate Thumbprint
Admin Client Certificate Thumbprint
Non-Admin Client Certificate Thumbprint

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

ID: 80a333ac-8208-4146-2d99-6b6258b64a3f
Version Independent ID: 30341430-ba3a-ae9e-5599-5ed618e9213a
Content: Deploy a secured Service Fabric cluster in Azure Stack
Content Source: azure-stack/user/azure-stack-solution-template-service-fabric-cluster.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@easyveazie Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@easyveazie  Thanks for your feedback. I've updated the security screen shot.
#please-close


html: support coverage diffs with deleted sources

When a previous result mentions a non-existent file, just ignore the
error; it has probably just been deleted since the last run (different
branch, reworked source tree, etc.).
Fixes #382

Looks good to me just update the changelog section under unreleased and I'll merge it :+1:


GCP routers, subnets, buckets, and objects

@nuwang @afgane @baizhang @mbookman
Please take a look.

Changes Unknown when pulling ba70df6a5f238f927ee7c59ad383771a539be823 on chiniforooshan:mods into ** on gvlproject:gce**.

Changes Unknown when pulling ba70df6a5f238f927ee7c59ad383771a539be823 on chiniforooshan:mods into ** on gvlproject:gce**.

Codecov Report

:exclamation: No coverage uploaded for pull request base (gce@34ae3bd). Click here to learn what that means.
The diff coverage is 27.76%.

@@          Coverage Diff           @@
##             gce      #50   +/-   ##
======================================
  Coverage       ?   51.32%
======================================
  Files          ?       21
  Lines          ?     4156
  Branches       ?      546
======================================
  Hits           ?     2133
  Misses         ?     1974
  Partials       ?       49

Impacted Files
Coverage Δ

cloudbridge/cloud/providers/gce/helpers.py
50% <100%> (ø)

cloudbridge/cloud/providers/gce/services.py
23.69% <18.79%> (ø)

cloudbridge/cloud/providers/gce/resources.py
27.33% <32.57%> (ø)

cloudbridge/cloud/providers/gce/provider.py
27.85% <62.5%> (ø)

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 34ae3bd...dad1f84. Read the comment docs.

Changes Unknown when pulling dad1f84f0db9cba1c79ddf8e732e93718dd00e44 on chiniforooshan:mods into ** on gvlproject:gce**.

Changes Unknown when pulling dad1f84f0db9cba1c79ddf8e732e93718dd00e44 on chiniforooshan:mods into ** on gvlproject:gce**.

Changes Unknown when pulling ff9da1e85d138ce0916d038f1ce3bbaa686538c8 on chiniforooshan:mods into ** on gvlproject:gce**.

Changes Unknown when pulling ff9da1e85d138ce0916d038f1ce3bbaa686538c8 on chiniforooshan:mods into ** on gvlproject:gce**.

Changes Unknown when pulling 67a39c3764c97a46c3cab121fdbd0d79cd5935ee on chiniforooshan:mods into ** on gvlproject:gce**.

Changes Unknown when pulling 67a39c3764c97a46c3cab121fdbd0d79cd5935ee on chiniforooshan:mods into ** on gvlproject:gce**.

@chiniforooshan That was really fast :-) Awesome.


Has anyone tried using this with gogoinflight?

Lol I'll try in a few days and get back to you all :)

It's been a year and no reply :)

@rsgrinko flights have the free wifi with T-Mobile these days so I never ended up needing to try this again :p

Google in bed 😆

This project is long abandonded. I will mark it as archived to prevent further issues from being created.


Image processing error: Error: Request failed with status code 500

if the address doesn't have a street view, for example "38 Galshoyan Street, Yerevan, Armenia"
or it is a wrong address, we are getting an error
Image processing error: Error: Request failed with status code 500
also, if previous request was successful, it keeps the title and street view of the previous one. Should be empty in case of the error, to do not confuse the user

When the address isn't supported by GSV, the API will now return a 200 OK with the following response:
{
  "error": "Image not found in response from Google Street View"
}


Bivariant method check fails for index-signature generic type

Bug Report
When instantiating a type with a broad index signature, and assigning it to another type with a more specific index signature whose properties are still assignable, bivariant method checking fails under a misleading error. This seems to only happen when the method under question uses a deferred keyof type query:
🔎 Search Terms
bivariant method function index signature keyof
🕗 Version & Regression Information
At least version 3.5

This is the behavior in every version I tried, and I reviewed the FAQ for entries about method bivariance checks

⏯ Playground Link
Playground link with relevant code
💻 Code
type KeysOfType<TTarget, TConstraint, TKey extends keyof TTarget = keyof TTarget> = TKey extends string
    ? string extends TKey
        ? string
        : TTarget[TKey] extends TConstraint
            ? TKey
            : never
    : never

declare class Component<T extends Record<string, boolean | string>> {
    constructor(options: T)
    toggle(key: KeysOfType<T, boolean>): void
    getText(key: KeysOfType<T, string>): string
}

declare var genericComponentWithConstraintAsDefault: Component<Record<string, boolean | string>>
declare var genericComponentWithAnyAsDefault: Component<any>
const instantiatedComponent = new Component({
    isHidden: false,
    isBig: false,
    message: 'Hello world'
})
genericComponentWithConstraintAsDefault = instantiatedComponent; // type error 2322
// Type 'Component<{ isHidden: false; isBig: false; message: string; }>' is not assignable to type 'Component<Record<string, string | boolean>>'.
//   Type 'Record<string, string | boolean>' is missing the following properties from type '{ isHidden: false; isBig: false; message: string; }': isHidden, isBig, message

genericComponentWithAnyAsDefault = instantiatedComponent; // works

🙁 Actual behavior
The assignment of genericComponentWithConstraintAsDefault fails with the following error:
Type 'Component<{ isHidden: false; isBig: false; message: string; }>' is not assignable to type 'Component<Record<string, string | boolean>>'.
  Type 'Record<string, string | boolean>' is missing the following properties from type '{ isHidden: false; isBig: false; message: string; }': isHidden, isBig, message

🙂 Expected behavior
The assignment of genericComponentWithConstraintAsDefault does not fail. Note that when using any as the generic parameter, the assignment works. Also when removing the string extends TKey ? clause (which checks for wide type instantiations) in KeysOfType, it also removes the error, though I kind of need it to prevent the method parameters from being typed as never.

What's going on here is that we measure Component to be invariant, thus there's no need to check the structural relation between the two definitions of toggle since bivariance can't fix an invariant situation.
Similar demonstration + proof of should-be-assignable
// Appears invariant, actually covariant
type Inv<T> = {
    someValue: T[keyof T];
};

declare class Component<T> {
    constructor(options: T)
    toggle(key: Inv<T>): void
}

declare var genericComponentWithConstraintAsDefault: Component<Record<string, boolean | string>>
const instantiatedComponent = new Component({
    isHidden: false,
    isBig: false
});

instantiatedComponent.toggle({ someValue: true })
genericComponentWithConstraintAsDefault = instantiatedComponent; // type error 2322

// This assignment is OK -- methods are bivariantly related
const p: Parameters<(typeof genericComponentWithConstraintAsDefault)["toggle"]>[0] = null as any as Parameters<(typeof genericComponentWithConstraintAsDefault)["toggle"]>[0];

However, I think the example as given is actually a correct error, at least given the reasonable limitations of our analysis. KeysOfType really is invariant, because a covariant substitution of TTarget could produce a wider result set. By construction this situation can't happen with Component because of its constraint, but we have no way to reason about that fact.

When does the compiler determine a type to be invariant for assignability checks? I only know so far of enums and private fields but this one is new to me.

This is a different thing. Generic types are always "measured" in terms of their variance relationship to their type parameter(s). Component is invariant in T because it has both co- and contra-variant usages of T. This enables much faster relational checks since you can check T<S> => T<U> by checking S => U, U => S, or U == S depending on what that measurement produced.

I see, hence why the any substitution works. Thanks for the explanation!


Buffer allocation pool for parsing protobuf

Reduces allocations significantly.

Please note the TODO in the comment. Maybe, let's have that on the project board otherwise.

bors r+


Added session aware interface and trait

Hi @rjgamer  Thanks for the PR.
I'm not sure for which purpose this trait is needed if you use (constructor) dependency injection.
Can you tell more about it and give an example?

AwareInterfaces are needed, if you cannot define the constructor arguments, because the parent class has it already set. And I don't like to change the constructor arguments of child classes (not best practise).
Example:
abstract class BaseController {

   protected $fooBar;

   public function _constructor(FooBar $fooBar) {
       $this->fooBar = $fooBar;
   }
}

class MyController implements SessionAwareInterface {
    use SessionAwareTrait;
}

$container->add(MyController::class, function($container) {
   $fooBar = $container->get(FooBar::class);
   $session = $container->get(SessionInterface::class);

   $controller = new MyController($fooBar);
   $controller->setSession($session);

   return $controller;
}

In combination with Inflectors it is really powerful. Check the examples here:
https://container.thephpleague.com/3.x/inflectors/
Psr/Log has also LoggerAwareInterface and LoggerAwareTrait implemented:
https://github.com/php-fig/log/tree/master/Psr/Log
What you think? If your lib don't offer these classes, I've to write them myself.

Ok this is fine. I have never used this pattern because I always use composition (with constructor dependency injection) and I also try to avoid "extending" classes whenever possible. ;-)


Implemented prototype Rest API for modifying domains and credentials

Hi,
I'm interested in having REST API support for managing domains and credentials, which seems to have been discussed previously:
https://issues.jenkins-ci.org/browse/JENKINS-28407
https://groups.google.com/forum/#!topic/jenkinsci-dev/BCZr7Bn9sKY
Unfortunately I'm a newbie when it comes to Jenkins development, so I'm pretty sure what I'm proposing in the pull request is wrong.
I hope someone could point me in the right direction for implementing this feature properly.
What I've attempted to do is:

Add some notes on how to use the API using jelly views
Respond to HTTP requests with a status code which should be meaningful alone, instead of a 302 Redirect
Respond with the ID of the created credential
Take the incoming data directly from the payload as a JSON object, instead of an element from an x-www-form-urlencoded payload

The open questions for me currently are:

The urls that I chose are probably strange. E.g.: apiCreateCredentials. How could I make these better?
There is some code duplication between the API methods and their regular counterparts. How could I reduce this duplication?

I was able to call the existing endpoints without too much hastle, so closing this for now as the pull request appears to be redundant.

@roccivic, as a matter of fact, how did you finally proceed?

@dcoraboeuf Well, I had a few issues to work around, but nothing major:

Existing endpoints only reply with a 302 redirect

Solved with a fire-and-forget approach, with a second request to see if the object was created/deleted.

The ID of the created object is not returned

Solved by providing a GUID as the new ID from my side.

I wanted the request payload to go up as application/json

Just used the standard Jenkins way of adding the JSON payload to a url-encoded form.

All that I really needed was to know the exact payload to send in the request for the various actions. I just used the network tab of the developer tools in chrome to watch the request as I performed the action in Jenkins.
Authenticating is pretty much the same as for other endpoints: Basic authentication using the user's API token.
I can't share the code as it's from a closed source application, but let me know if I can help any further.

It seems that this python script does the job: https://github.com/wunzeco/ansible-jenkins-extra/blob/master/files/create-credentials-config.py as called in the corresponding Ansible script https://github.com/wunzeco/ansible-jenkins-extra/blob/master/tasks/job-creds.yml Going to check it out right now.

I can confirm that the ansible-jenkins-extra role sets credentials nicely, even though I've only tested the Jenkins job credentials. The repo could use some love in the little details, so please give it a look and give it a go.


Add pmorie and font to the OWNERS file.

/sig multicluster

/retest

/lgtm

/lgtm
On Thu, Dec 21, 2017 at 9:08 PM k8s-ci-robot<EMAIL_ADDRESS>wrote:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: perotinus
https://github.com/kubernetes/cluster-registry/pull/158#issuecomment-353504725
The full list of commands accepted by this bot can be found here
https://github.com/kubernetes/test-infra/blob/master/commands.md.
Needs approval from an approver in each of these OWNERS Files:

OWNERS
https://github.com/kubernetes/cluster-registry/blob/master/OWNERS
[perotinus]

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/cluster-registry/pull/158#issuecomment-353504736,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWXmHMKtw5Tp9YawlpNgU9TtWAH2fdNks5tCw83gaJpZM4RI_NK
.


Enh - Enumerate Local User Accounts

Collect and report information about local security accounts.

No interest / disregard


Input loses caret position when typing, if event is delegated to form

In React + JS, I tend to not put "onChange" handlers on all of my inputs (it gets tedious). Instead, I put them on my form:
But in re-frame, this causes the caret to move to the end of any input being typed into:
This doesn't work:
[:form {:on-change #(rf/dispatch-sync [:field-change (-> % .-target .-name keyword) (-> % .-target .-value)])}
  [:div.field-group
    [:label.field-label "Name"]
    [:input.field {:value name
                          :name "name"
                          :placeholder "Name"}]]

This does work:
[:input.field {:value name
                      :on-change #(rf/dispatch-sync [:field-change (-> % .-target .-name keyword) (-> % .-target .-value)])
                      :name "name"
                      :placeholder "Name"}]]

Is it simply not possible to do this using event delegation?
Edit: not sure why GitHub is screwing up the formatting in those examples... sorry about that!
Thanks! (re-frame rocks, by the way)

Er. I should note that it's much less tedious in re-frame than in React to put an on-change on each input, since re-frame's architecture doesn't require me to create/bind a function within my render, so worst-case workaround, I can do something like this:

(defn- sync-field [e]
  (rf/dispatch-sync [:field-change (-> e .-target .-name keyword) (-> e .-target .-value)]))

(defn main-panel []
  (let [name @(rf/subscribe [:name])
        email @(rf/subscribe [:email])]
    [:div.card
      [:header.card-header
        [:h1.card-title (str "Name: " name)]
        [:span.card-subtitle email]]
      [:form
        [:div.field-group
          [:label.field-label "Name"]
          [:input.field {:value name
                         :name "name"
                         :on-change sync-field
                         :placeholder "Name"}]]
        [:div.field-group
          [:label.field-label "Email"]
          [:input.field {:value email
                         :name "email"
                         :on-change sync-field
                         :placeholder "Email"}]]]]))

A few notes:

This issue should really go against the reagent repo. It isn't a re-frame specific.
But I wouldn't bother raising that issue because I doubt very much you'll get a fix.  I happen to know something about this area because I provided https://github.com/reagent-project/reagent/pull/126


timeline: Only draw InvisibleButton if size greater than zero

InvisibleButton size must be greater than zero in both x and y so only draw Effect button if this is the case. As discussed in #85
I now notice there are several other InvisibleButtons in timeline, is it worth adding the same check to all of them or is there a more elegant approach?

I think you're right actually yes. On line 362 we use item_duration.to_seconds() to set the width and that is where the 0 comes from in this case, so maybe it's better to go through and make sure any item with a duration of 0 is not drawn? Something in a timeline with length zero is essentially not there right?

In OTIO it is valid to have a clip with zero duration. Sometimes that's meaningful when it occurs before/after a transition. So that case will come up in practice. Does Raven need to draw it? No. But It would be nice if Raven did let you find those and inspect them. We don't yet have #73 so there isn't a way to select something with zero duration at the moment. I'm open to suggestions about how to do that - but your fix to avoid rendering something with zero pixel width seems totally fine.

Updated based on your feedback, should I include markers too? I know they often have a duration of zero and their width is already based of off the track height

If the markers also cause a crash we should fix that too. The rest looks great.

Hi, sorry to bump this, I just wanted to check if you are waiting on changes from me with?

@ThomasWilshaw I was hoping the duration test would allow drawing items of zero length, and also that the width would be clamped to a minimum size to avoid drawing zero area markers (maybe that's already in place, I'm not sure).

@meshula I possibly misunderstanding what you mean but how would you display something on the timeline if it's duration was zero? I've only come across it in one file so far but is it actually fairly common?
When you say a zero area marker do you mean one that only marks a single frame rather than a range of frames?

Markers have a duration, and a marked frame may have a duration of one frame as you say. That's not an issue, that works with the code already.
However, markers may be instantaneous with a zero duration, and appear at any time, not necessarily aligned on a frame.
A typical method of displaying zero length markers is to use a hair line of a certain number of pixels centered on the time. Typically a visual affordance can be provided, such as an inverted triangle or a circle filled with the marker's color, atop the hair line, giving the appearance of a traffic sign or lollipop in the time line, and also providing an easy handle for hovering, selection, and dragging.
Adding a "lollipop" and interaction affordances is extra candy though. I'm not proposing that you take that part on, just proposing that you draw a hair line with a small bounding box for hovering and selection.

I think all that is already supported. Here we have a selected marker (the white one) that has a duration of zero and starts on an integer frame:

Here we have a multi frame marker that starts part way through a frame (again in white as it's selected):

Single marker that starts part way through a frame (although that's not reflected in the inspector so I'm displaying the JSON):

Got it, I thought this change was going to not render those because of the zero length.


Error in ios

"react": "16.8.6",
"react-native": "0.60.5",
"react-native-image-crop-picker": "^0.25.2",
Podfile ios 9.0
libsystem_kernel.dylib`__abort_with_payload:
0x10e09700c <+0>:  movl   $0x2000209, %eax          ; imm = 0x2000209
0x10e097011 <+5>:  movq   %rcx, %r10
0x10e097014 <+8>:  syscall
->  0x10e097016 <+10>: jae    0x10e097020               ; <+20>
0x10e097018 <+12>: movq   %rax, %rdi
0x10e09701b <+15>: jmp    0x10e079457               ; cerror_nocancel
0x10e097020 <+20>: retq
0x10e097021 <+21>: nop
0x10e097022 <+22>: nop
0x10e097023 <+23>: nop

platform :ios, '9.0'
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'
target 'BuildingNetworks' do
Pods for BuildingNetworks
pod 'React', :path => '../node_modules/react-native/'
pod 'React-Core', :path => '../node_modules/react-native/React'
pod 'React-DevSupport', :path => '../node_modules/react-native/React'
pod 'React-RCTActionSheet', :path => '../node_modules/react-native/Libraries/ActionSheetIOS'
pod 'React-RCTAnimation', :path => '../node_modules/react-native/Libraries/NativeAnimation'
pod 'React-RCTBlob', :path => '../node_modules/react-native/Libraries/Blob'
pod 'React-RCTImage', :path => '../node_modules/react-native/Libraries/Image'
pod 'React-RCTLinking', :path => '../node_modules/react-native/Libraries/LinkingIOS'
pod 'React-RCTNetwork', :path => '../node_modules/react-native/Libraries/Network'
pod 'React-RCTSettings', :path => '../node_modules/react-native/Libraries/Settings'
pod 'React-RCTText', :path => '../node_modules/react-native/Libraries/Text'
pod 'React-RCTVibration', :path => '../node_modules/react-native/Libraries/Vibration'
pod 'React-RCTWebSocket', :path => '../node_modules/react-native/Libraries/WebSocket'
pod 'React-cxxreact', :path => '../node_modules/react-native/ReactCommon/cxxreact'
pod 'React-jsi', :path => '../node_modules/react-native/ReactCommon/jsi'
pod 'React-jsiexecutor', :path => '../node_modules/react-native/ReactCommon/jsiexecutor'
pod 'React-jsinspector', :path => '../node_modules/react-native/ReactCommon/jsinspector'
pod 'yoga', :path => '../node_modules/react-native/ReactCommon/yoga'
pod 'DoubleConversion', :podspec => '../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec'
pod 'glog', :podspec => '../node_modules/react-native/third-party-podspecs/glog.podspec'
pod 'Folly', :podspec => '../node_modules/react-native/third-party-podspecs/Folly.podspec'
pod 'RNImageCropPicker', :path =>  '../node_modules/react-native-image-crop-picker'
target 'BuildingNetworksTests' do
inherit! :search_paths
# Pods for testing
end
use_native_modules!
end
target 'BuildingNetworks-tvOS' do
Pods for BuildingNetworks-tvOS
target 'BuildingNetworks-tvOSTests' do
inherit! :search_paths
# Pods for testing
end
end

pod 'React', :path => '../node_modules/react-native/'
pod 'React-Core', :path => '../node_modules/react-native/React'
pod 'React-DevSupport', :path => '../node_modules/react-native/React'
pod 'React-RCTActionSheet', :path => '../node_modules/react-native/Libraries/ActionSheetIOS'
pod 'React-RCTAnimation', :path => '../node_modules/react-native/Libraries/NativeAnimation'
pod 'React-RCTBlob', :path => '../node_modules/react-native/Libraries/Blob'
pod 'React-RCTImage', :path => '../node_modules/react-native/Libraries/Image'
pod 'React-RCTLinking', :path => '../node_modules/react-native/Libraries/LinkingIOS'
pod 'React-RCTNetwork', :path => '../node_modules/react-native/Libraries/Network'
pod 'React-RCTSettings', :path => '../node_modules/react-native/Libraries/Settings'
pod 'React-RCTText', :path => '../node_modules/react-native/Libraries/Text'
pod 'React-RCTVibration', :path => '../node_modules/react-native/Libraries/Vibration'
pod 'React-RCTWebSocket', :path => '../node_modules/react-native/Libraries/WebSocket'
pod 'React-cxxreact', :path => '../node_modules/react-native/ReactCommon/cxxreact'
pod 'React-jsi', :path => '../node_modules/react-native/ReactCommon/jsi'
pod 'React-jsiexecutor', :path => '../node_modules/react-native/ReactCommon/jsiexecutor'
pod 'React-jsinspector', :path => '../node_modules/react-native/ReactCommon/jsinspector'
pod 'yoga', :path => '../node_modules/react-native/ReactCommon/yoga'
pod 'DoubleConversion', :podspec => '../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec'
pod 'glog', :podspec => '../node_modules/react-native/third-party-podspecs/glog.podspec'
pod 'Folly', :podspec => '../node_modules/react-native/third-party-podspecs/Folly.podspec'
pod 'RNImageCropPicker', :path =>  '../node_modules/react-native-image-crop-picker'
target '..Tests' do
inherit! :search_paths
# Pods for testing
end
use_native_modules!
end
target '...-tvOS' do
Pods for ...-tvOS
target '...-tvOSTests' do
inherit! :search_paths
# Pods for testing
end
end


DatePicker not updating ngModel value

Bug?
I have this Date Picker example
<mat-form-field>
  <input matInput [(ngModel)]="from" [matDatepicker]="from" (change)="onChangeEvent($event)" placeholder="From">

  <mat-datepicker-toggle matSuffix [for]="from"></mat-datepicker-toggle>
  <mat-datepicker #from></mat-datepicker>
</mat-form-field>

import {Component} from '@angular/core';

@Component({
  selector: 'date-filter',
  templateUrl: 'date.filter.html',
})
export class DateFilter {
  from: any;

  onChangeEvent(e): void {
    console.log(e.target.value);
    console.log('this.from = ', this.from);
  }
}

What is the expected behavior?
On date change i should have the value in this.form.
What is the current behavior?
It doesn't update the model and i get this error: Uncaught Error: Cannot assign to a reference or variable!.

please ask this question in StackOverflow

I just found the solution myself, thank you for your awesome support.
<mat-form-field>
  <input matInput [(ngModel)]="fromDate" [matDatepicker]="from" (change)="onChangeEvent($event)" placeholder="From">

  <mat-datepicker-toggle matSuffix [for]="from"></mat-datepicker-toggle>
  <mat-datepicker #from></mat-datepicker>
</mat-form-field>

I had to update the [(ngModel)] to [(ngModel)]="fromDate" differently from [matDatepicker]="from".
Completely unintuitive.

please use (dateChange)="onValueChange($event) instead of change

@dhrn it was just for debugging purposes


next: Handler error

encode error to json
handle error recovered from panic

@carmenlau updated

@rickmak updated

The last update makes me think of should we also move RequestIDMiddleware to NewServer, and RequestIDMiddleware should be added before RecoverMiddleware. So in case we want to add log to RecoverMiddleware, we can have the request id to trace.

Again I think the Middleware should be bound to HTTP semantic, hence the golang specific recover should not belong to that stage of the request processing.

I think for production code, we need a big try catch to at the entry point of the request, hence I put the recovery code in the middleware.

I dropped 374be2a and merge via e40a588


fix: sort assets by balance in trade to and from

Description
While we search assets in trade to/from, we want to display the assets by balance in the descending order
I added a new selector doing exactly this and reusing the sort by name selector
It might not be enough as we supercharge with the custom assets but it should be at least better when typing the full ticker of the asset (USDC for example)

Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #7513
Risk

High Risk PRs Require 2 approvals

What protocols, transaction types, wallets or contract interactions might be affected by this PR?

Testing

Go to the trade to or from asset modal
Try to search for any asset with some balances, USDC for example, it should be sort by balance un the descending order
Do this for multiple assets, no search.. play with it!

Engineering
n/a

Operations
n/a

[ ] :checkered_flag: My feature is behind a flag and doesn't require operations testing (yet)

Screenshots (if applicable)
Prod

This branch

@gomesalexandre seems like filtering at selector level can't be done because SearchTermAssetList is filtering in an effect killing the selector ordering
as a fix, I ordered by balance at component level in b3db2b2
Final result:


Users management with authentication

What will be delivered ?

Login Signup pages
Firebase connectivity
User Authentication
Documentation

Finished!


[RN 0.60+] How to make it work on iOS?

Installation instructions does not work on RN 0.60+ because of autolinking (README should be updated).
Although, I was able to successfully install this package on android, based on this steps.
But on iOS I was unable to install it. I made the following steps:

npm i --save react-native-keyboard-input
cd iOS && pod install

But it does not work:

Also I tried the steps described in README, but it fails to build (because of RN0.60+ autolinking).
So my question is: how this package should be installed on iOS for RN0.60+?
  "dependencies": {
    "react": "16.11.0",
    "react-native": "0.62.0",
    "react-native-autogrow-textinput": "^5.3.0",
    "react-native-keyboard-input": "^6.0.0",
  },

I got it working with cocoapods. I ignored all the instructions listed for installing on iOS and added
pod 'ReactNativeKeyboardInput', :path => '../node_modules/react-native-keyboard-input'
pod 'ReactNativeKeyboardTrackingView', :path => '../node_modules/react-native-keyboard-tracking-view'

to my podfile

@samgermain Thank you!
I added
pod 'ReactNativeKeyboardInput', :path => '../node_modules/react-native-keyboard-input'
pod 'ReactNativeKeyboardTrackingView', :path => '../node_modules/react-native-keyboard-tracking-view'

as you suggested and then I ran pod install inside ios directory.
I get an error message regarding autolinking when I run react-native run-ios, but it works.


it dosen't work on mobilephone

The microphone dosen't work when I used ipad or andorid to test the demo.Could it be the browser's problem?We hope not.😱😱😱😱😱

For the iPad, it was definitely the browser's problem, Safari on IOS doesn't give microphone access, and Apple doesn't allow for other browsers (except ones that are really just Safari wrappers). I believe that some Android browsers do, but definitely not all - can you specify what browser/Android version you tried?.

Hello @Camixxx , I have tried on a number of Android devices, using Chrome, and the microphone component does not capture the audio right, it comes out corrupted, so the STT demo wont work.
We have released mobile SDKs (Android and iOS) to interact with both STT and TTS services, you can find them here:
https://github.com/watson-developer-cloud/speech-ios-sdk
https://github.com/watson-developer-cloud/speech-android-sdk
Dani

BTW @daniel-bolanos & @Camixxx, I just tested in Chrome and Firefox on my Android phone and had success in each - it correctly transcribed all of my test phrases.
The only issue I saw is that Firefox keeps the "Microphone is on" notification even after I stop recording. In Firefox, the notification only went away when I closed the tab, whereas Chrome correctly removes the notification as soon as I stop the recording.
My phone is a Verizon HTC One M8 running:
Android 5.1.1 (CyanogenMod 12.1-20150925-NIGHTLY-m8)
Firefox 41.0.2
Chrome 46.0.2490.76

hi @nfriedly , really? Hmm in the past every time I tried from Chrome in my Android the recognition results were very poor, then I would listen to the audio file recorded and was always corrupt. What phone are you using? is it last generation?
Regarding that notification issue, I think I have seen that even in my ubuntu... will need to look into that. Maybe we are not releasing something.
thanks for feedback :)

@daniel-bolanos this phone: http://pdadb.net/index.php?m=specs&id=5890&c=verizon_htc_one_m8_2014_lte-a_htc6525lvw_htc_m8 - about 1.5 years old, so previous generation.
And I should clarify that I didn't get perfect transcription, but generally pretty good, on par with what I get from my macbook.


Segmentation fault in R3.3.1

An irrecoverable exception occurred. R is aborting now ...
/usr/local/lib/R/bin/INSTALL: line 34:  1549 Done                    echo 'tools:::.install_packages()'
      1550 Segmentation fault: 11  | R_DEFAULT_PACKAGES= LC_COLLATE=C "${R_HOME}/bin/R" $myArgs --slave --args ${args}

installation of package ‘devtools’ had non-zero exit status

wrong repo, sorry


Used the AT_DISPATCH_ALL_TYPES macro for the CPU version too.

Right now, the CPU version and the CUDA version take different input arguments - the CPU version only accepts a tensor of floats while the CUDA version uses templates and the AT_DISPATCH_ALL_TYPES macro to accept other scalar types.

thanks a lot I'll merge that shortly


bugfix : build SVGFigure create a wrong  'no viewbox'

The create of
SVGFigure(width="16cm",height="16cm")
raise a catched exception
the viewbox is not create in the
"it will nice to have unit test"

I am agree with the merge block
my pull request is not relevant , I didn't read the tests
SVGFigure(width="16cm",height="16cm")
should be
SVGFigure(width=Unit("16cm"),height=Unit("16cm"))


why does parent sortable drop and drag,child sortable can't drop and drag with angular

why does parent sortable can drop and drag,child sortable can't drop and drag with angular,when the scope data changed ,without ng-sortable.js . clike the change button, parent sortable can drop and drag,but child sortable can't drop and drag.
code like this :
http://jsbin.com/kalojosuka/1/edit?html,css,js,output

Try dev-branch and use only ng-sortable.js
Example: http://jsbin.com/huyipuniwo/1/edit?html,output

thanks a lot

This seems to still have a bug, if you have more than 2 child items they can't be reordered properly:
http://jsbin.com/qudizihodo/2/edit?html,output
All of the children objects always have an oldIndex of 0 when onUpdate is called.
From my initial investigation it looks like _index(el) is returning the parent's draggable index.

http://jsbin.com/qudizihodo/2/edit?html,output — example does not work.

Sorry about that, can you try this example:
http://jsbin.com/denozivupe/1/edit?html,output
Try to reorder Item 5 in group A and you should see the issue.

in this case,_onTapStart() function was fired 2 times, firsrt time is child scope and second is parrent scope, so oldIndex was set parrent index. so, i added evt.stopPropagation(); at the end of this function, it works. hope fixed this issue for next version :)

I can confirm that @tplaner 's solution works great.
Please add it to official

@tplaner Thanks Tom, you saved us time as well!

Dear all! ng-sortable.js is moved to the separate repository. If this issue is still actual, please create it there once again and this rojects needs a maintainer:

2+: https://github.com/SortableJS/angular-sortablejs 🆘
1.*: https://github.com/SortableJS/angular-legacy-sortablejs


Joshua s brown plugin globus

Adds functionality for interacting with Globus using plugin

@rafaelfsilva I have the Globus plugin now at least tested where it can send two files between two different endpoints. Still needs to be tested more but in it's current state I can transfer and detect globus endpoints on the local machine and the globus cloud.


You exceeded your current quota

Hello.
I subscribed to the AICommit plugin, activated the license for the plugin in phpStorm 2023.3.2, but when I try to generate commit text, an error appears:
You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors
No commits could be generated successfully.

Hello.
I subscribed to the AICommit plugin, activated the license for the plugin in phpStorm 2023.3.2, but when I try to generate commit text, an error appears:
You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors
No commits could be generated successfully.

This prompt comes from OpenAI and has nothing to do with the AICommit plug-in itself.
Your OpenAI request hit the limit. Check the official documentation here.


Remediate High Zookeeper vulnerability

Snyk reports the following High severity vulnerability in HumanCellAtlas/data-consumer-vignettes. Please remediate by the end of Q1 Milestone 1.
Description
org.apache.zookeeper:zookeeper
Suggested Remediation
Upgrade org.apache.zookeeper:zookeeper to version 3.4.10, 3.5.4-beta or higher.
Details
org.apache.zookeeper:zookeeper is a centralized service for maintaining configuration information, naming, providing distributed synchronization, and providing group services. Affected versions of this package are vulnerable to Authentication Bypass. No authentication/authorization is enforced when a server attempts to join a quorum, as a result an arbitrary end point could join the cluster and begin propagating counterfeit changes to the leader.

Same question as #80 - where is zookeeper being used in this repo? https://github.com/HumanCellAtlas/data-consumer-vignettes/search?q=zookeeper&unscoped_q=zookeeper

PR #83 should remediate this vulnerability, please re-open if not


How about letting the users to pick videos and create new bots

This is a POC of a similar approach done for google drive files and folders
https://github.com/dinukadesilva/reactriot2017-vibrato
So in this case,
Users might pick videos as URLs or via youtube video picker
Relates to https://github.com/99xt/youtube-queuing-bot/issues/9

Interesting. picking new videos is fine and will work about well. how do you propose to create new bots? since if we are to create new bots and associate them with the same set of API's that we are building it will ultimately perform the same task.
what if we allow users to query trending videos based on the similarity of videos in the queue ?


Miscellaneous improvements to integration tests

This PR is a grab bag of little fixes/improvements to the integration tests:

some responses were not correctly validated
some errors were checked twice
some typos in test names and comments
some inconsistent formatting

Part of the work for #561.


[Script]: Fetch contributions

Aim
It's a Python script to fetch the pull requests made by a github user in an Organization. It helps organization members to keep record of a particular github user requests and also helps the github user to record the pull requests made to the organization.
Details

Script can be executed as a CLI.
Takes arguments username and organization.
Returns the pull requests made by user to the organization into a Markdown file.
Markdown file consists a table having - "Title of PR" , "Link of PR" , "Status(Merged/Closed/Open)"

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Go Ahead @sabhisharma-ise


is there function for drawing a grid (cf matlab, matplotlib)

I assume you mean a "background grid" that highlights the values. Currently this is not supported, but it is a good suggestion.

I also searched for a background grid to make one able to read plots more accurately.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/519 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit 2dce38d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/sql-database-engine.png
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

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


Login: AAD login (server flow) failing with 401

AAD login fails with 401 after redirecting 3 times to the login endpoint

Related issue: https://bugs.webkit.org/show_bug.cgi?id=140188

AAD server flow started failing after moving to WKWebView: https://github.com/Azure/azure-mobile-apps-ios-client/pull/47
WKNavigationDelegate does not stream http body request. This resulted in making request to auth endpoint with empty body even after the user  logs in with aad.


settings.js showAppSettingsMenu() expects .info file for every settings file

Hi,
in showAppSettingsMenu() an *.info-file is read for every *.settings file:
      const a=storage.readJSON(id+'.info',1);
      return {id:id,name:a.name,sortorder:a.sortorder};

When developing the .info-file is not yet present. While readJSON() will not throw an exception the next line will because it can not access "name" on "a" which is undefined. To avoid this situation can we provide a default value? e.g.
const a=storage.readJSON(id+'.info',1) || {name: id};

Alternative: Can we make readJSON() throw the exception? This should provide a more readable error.

Just done - thanks! I think just carrying on regardless is the best plan - if there's an error for a non-developer user we don't want it to just silently stop working


Ref transform intellisense not working

hii @johnsoncodehk while working with the new ref sugar, I found an issue. Not sure if it's a bug or volar doesn't have support for this yet. Please check

as you can see above, the intellisense should be 'sink' | 'index', but it's something different. is it a bug ?

This is $ref types issue, view can't auto-complete as possible value because it is not basic union string type.

thx

@soulsam480 Better solution here: https://staging.vuejs.org/guide/extras/reactivity-transform.html#typescript-integration-ts
env.d.ts
/// <reference types="vue/macros-global" />


Dark Theme

Currently the default theme is Light. But it would be nice if we give option to change the theme of the cards to the users.

It will have an "data-theme" attribute which will take "dark" or "default" as value.
If the dark theme is chosen, the card css will change accordingly.

This issue is open for anyone who is interested. ( make sure you have a basic knowledge of working with web components)

Hey!
If you want I made a dark version of your project, but it is only dark. So you cannot switch between light and dark. If you like it feel free to use it!
https://github.com/macedonga/github-card

@macedonga I like what you did. But we need to make it available as an option not as default. I will start working on this feature after a few days. Currently I'm busy with my internship. That's why I made this issue. But keep an eye for more issues :)

@macedonga There are some conventional life cycle callbacks  in a web component that we need to follow and use. you didn't use that properly. but thanks for showing interest :)


Dropping support for Python 3.7

Description
Python 3.7 is very safely EOL now, having been out of security support for 5+ months. While there's been no great need to break compatibility, upstream dependencies are dropping support and becoming harder to keep in-line. By having to constrain our dependencies to versions which still support 3.7, we're making it difficult to install PSR in an environment alongside other packages when do not support 3.7 and have pinned to versions higher than we can accommodate, and this problem obviously only worsens with time.
The download stats for the last 120 days also indicate that the majority of users are running more modern Python versions:

As a result of this, the intention is to release a breaking change on February 3rd, 2024, after which new versions of Python Semantic Release will no longer support Python 3.7.

Hello 👋
Will there be other breaking change for PSR 9.0 ?

We should make a separate ticket that summarizes the plan for the next major release, and it would reference this ticket and/or others.
I have two in mind but I'm not sure I'll have them done by 24 Feb and that's of course if we are willing to change the defaults to what I would like them to be. Was planning to add them as a not default feature first, that would then become the default with the next major release.


Do not assert if FL calculation has multiple algorithms

Instead print a warning and statically select the first possible algo in
the list.

Codecov Report

Merging #95 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #95      +/-   ##
=========================================
+ Coverage   97.09%   97.1%   +<.01%
=========================================
  Files          10      10
  Lines        1688    1693       +5
=========================================
+ Hits         1639    1644       +5
  Misses         49      49

Impacted Files
Coverage Δ

p4c_bm/gen_json.py
98.05% <100%> (ø)
:arrow_up:

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 f59d441...82453c4. Read the comment docs.


42Intra API authentication

The API key has expired,it should be regenerated fromhere
Here

Thanks you @Gab-182 , The new  API key was been updated ✅


Improve status spec to conditions

This improves the status spec to use conditions
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)
[x] A PR is opened to update the documentation on our docs repo

Fixes #9
Related to #15

@arschles if you could please add your review :)


fix(coral): ConnectorApprovals, SchemaApprovals: actions in RequestDetailsModal should be disabled

What happened?
In ConnectorApprovals and SchemaApprovals, when viewing a request that cannot be approved or declined (because already approved, declined or deleted), the actions in the RequestDetailsModal are still enabled:
https://github.com/Aiven-Open/klaw/assets/20607294/4b1fae38-8842-47cc-8746-d0a0f649dd2c
What did you expect to happen?
The actions in this modal should be disabled when their action cannot be successful.  This check is present in TopicApprovals and AclApprovals., and should be added to ConnectorApprovals and SchemaApprovals : https://github.com/Aiven-Open/klaw/blob/2c33aedbff0bf6b7bc0c9b56dca4baa74577ab5b/coral/src/app/features/approvals/topics/TopicApprovals.tsx#L255

hey I'd like to take this

@mathieu-anderson Hey

Resolved by https://github.com/Aiven-Open/klaw/pull/1974


Amplify mock doesn't run lambda triggers on GraphQL mutations

Describe the bug
When amplify mock is running, it doesn't run lambda triggers associated to a GraphQL model, no matter the updates I do to items of that model type.
Amplify CLI Version
4.17.1
To Reproduce

Create a Lambda Trigger connected to the Todo @model through a DynamoDB Stream.
To do so, run amplify add function and choose the following options:

Using service: Lambda, provided by: awscloudformation
? Provide a friendly name for your resource to be used as a label for this category in the project: lambdaTriggerTest76
? Provide the AWS Lambda function name: lambdaTriggerTest76
NodeJS found for selected function configuration.
? Choose the function template that you want to use: Lambda trigger
? What event source do you want to associate with Lambda trigger? Amazon DynamoDB Stream
? Choose a DynamoDB event source option Use API category graphql @model backed DynamoDB table(s) in the current Amplify project
Selected resource lambdatestapp
Selected @model Todo
? Do you want to access other resources created in this project from your Lambda function? No
? Do you want to edit the local lambda function now? No
Successfully added resource lambdaTriggerTest76 locally.

Run amplify mock.

On the local GraphQL Explorer (at http://localhost:20002/ in my case), run some mutations on Todo objects.

Expected behavior
The mock console should show that the lambda trigger and its corresponding function has run. The function includes some console.logs, so it is expected to see them.
Desktop (please complete the following information):

OS: Ubuntu 18.04
Node Version: v12.16.1

Additional context
I tried all this with a simple app created following the Getting Started tutorial and this https://aws-amplify.github.io/docs/cli-toolchain/quickstart#as-a-part-of-the-graphql-api-types-with-model-annotation.
My GraphQL schema:
type Todo @model {
  id: ID!
  name: String!
  description: String
}

Probably related to #2128. @kaustavghosh06 responded there saying that this is supported by amplify mock. Is there anything I'm missing that should be done before mock can run the triggers?

Hi
any tips how we can test the triggers currently?

@ofpau did you find a solution to this?

I also have this issue on 4.24.3

@ammarkarachi I am having this issue on 4.29.3. Any update?

@ammarkarachi @yuth I am having this issue on 4.29.3. Any update?

Mock uses DynamoDB local to store the records locally and does not support triggers. This has been added to our backlog, and I don't have any timelines when this will be picked up.

@yuth is there some way I can simulate this trigger locally by manually invoking the lambda function?

+1

This would be very useful

My workaround is to use another lambda to simulate the trigger event, but I have to manually invoke the lambda

+1

+1


add htmx to the list

-Added List Item : HTMX
-Description :  htmx is a JavaScript library that makes it easier to create dynamic web pages. It does this by adding a number of new features to HTML, such as the ability to send and receive data with AJAX, update the DOM without a full page reload, and create custom event handlers.

@paradite hey could you please check if it follows the standards and tell me if there's an issue?
Thank you

Looks fine. Maybe the author_url need to be updated to be more specific, but it is okay for now.


[Discussion] Clean shutdown of streams and sockets

@vasyaxparfenov
In the discussion in issue #996 you started a topic that needs to be explored.
You wrote:
I think logic that we have on top of calling Socket.ConnectAsync which handles cancellation after timeout is redundant. Consider using Socker.ConnectAsync overload that accepts CancellationToken and it will handle it.

So we might even consider removing extra bits added on top of logic and just pass cancellation token which fires after timeout value.

I will need to research and analyze to fully understand that sentence*, but I have an inkling what you are getting at. It is just important to keep all edge cases in mind that have been there in the past and that led to some of the pieces of code that became redundant with later improvements in other areas.
*) Probably not the highest priority right now.

@vasyaxparfenov
Could you provide some example?

@FanDjango, sorry got lost a little bit. I'll send a code snippet with an example tommorrow.

@FanDjango, actually it turned out not that easy after all hah. Could you please tell me which version of dotnet were you using when experiencing timeout exception issues? Dotnet 6 and later has ConenctAsync overload which accepts cancellation token so we could avoid creating custom cancellation logic for latest version, but older version don't have such overload. We might consider checking whether socket was connected before trying to dispose it in case timeout occurs. Currently, our logic might break due to a fact that we're trying to close never opened socket.

I am closing this for now. I have been looking at the code and although there are still some places that don't look nice, we can leave them like that for now.


Modify delay in inventory sends in mempool

This PR is related to issues discussed in #1250
It includes the following changes to mempool behavior:

Removal of the delay implemented in CanSend (there is already a delay in asyncloop for SendTrickleAsync)
Change to delay in asyncloop to be 5 seconds instead of 10 seconds as per bitcoin core.

@mikedennis please fix conflicts


Keyboard problems

Bug summary
windowSoftInputMode="adjustResize" not working after connecting react-native-bars
Keyboard always opens on top of the screen
The default state of the example from the repository:
https://user-images.githubusercontent.com/39951413/191948054-ca4898fe-7a67-4183-9096-842064f363b5.mov
After removing this line RNBars.init(this, "dark-content");
android/app/src/main/java/com/rnbootsplashexample/MainActivity.java:26
https://user-images.githubusercontent.com/39951413/191948078-de28b720-6c7f-48cc-ab04-3f4346ae9912.mov
In both cases I have android:windowSoftInputMode="adjustResize" in AndroidManifest.
And I expect the first case to have the same behavior. But it's not.
Library version
1.2.2
Environment info
"react-native": "0.70.0",
"react-native-bars": "^1.2.2",
"react-native-bootsplash": "^4.3.2",

Steps to reproduce
This is an example with react-native-bootsplash template
Reproducible sample code
import * as React from "react";
import {
  ScrollView,
  TextInput,
  View,
} from "react-native";
import * as BootSplash from "react-native-bootsplash";

export const App = () => {
  React.useEffect(() => {
    BootSplash.hide();
  }, []);

  return (
    <ScrollView>
      <View style={{height: 500, borderWidth: 1, marginBottom: 10}} />
      <TextInput style={{borderWidth: 1, borderRadius: 10, padding: 10}} />
    </ScrollView>
  );
};

@exzos28 I dug a bit on this.

mikepenz/MaterialDrawer used KeyboardUtil.java
The same "fix" is available on Google issue tracker
In React Native issues
And in this gist

But as you can see in the comments, this is not ideal since it creates a flickering.
Dont' mind the navigation bar here, just focus on the status bar:
https://user-images.githubusercontent.com/1902323/192281350-14336428-30c3-4e9e-8982-616e28f86948.mp4
The "best" option for me seems to switch windowSoftInputMode="adjustResize" to windowSoftInputMode="adjustPan":
https://user-images.githubusercontent.com/1902323/192281525-1daf02ed-d5a8-400b-b28b-eb290d1656b8.mp4
I will update the README accordingly.

I published a new version with the README update: https://github.com/zoontek/react-native-bars/releases/tag/1.2.3

Yes, but in this case, the page is panned, and the header moves up. This is undesirable in 99% of cases, in my opinion.
https://user-images.githubusercontent.com/39951413/192526854-77a490d9-5e38-40c1-9182-5fb96b2eedb9.mov

@exzos28 I agree, but checked current solutions, and there's none viable.
Google does not seems to care about its OS.

@zoontek I think we need to implement an api in your library to be able to turn transparent status and navigation bars on and off. It would be great.

@exzos28 Sure, your company can hire me as a freelancer to take time to add this 🙂

@zoontek
I understand 🙂 I'll try to find some time and send MR. Does that make sense?

I'm onto something: https://github.com/zoontek/react-native-bars/pull/16
Unfortunately, an issue seems to exists with react-native-safe-area-context.
Using React Native built-in SafeAreaView:
https://user-images.githubusercontent.com/1902323/192560319-f667562e-ca56-4bec-a91e-674715ac5328.mp4
Using react-native-safe-area-context one:
https://user-images.githubusercontent.com/1902323/192560395-bf2c9e9a-0b7f-4693-a00f-4d2b284b4b53.mp4
I will reach them soon to see what can cause this.


Tap loss between two Stacked GestureDetectors.

With the following code you lose the next tap after tapping the top gesture detector.
tap 1: top gesture detector triggers
rebuild: top gesture detector is disabled
tap 2: no one gets it
tap 3: bottom gesture detector triggers
rebuild: top gesture detector is enabled
GOTO tap1
Future main() async {
  runApp(new Foo());
}

class Foo extends StatefulWidget {
  @override
  FooState createState() => new FooState();
}

class FooState extends State<Foo> {
  bool _disableTopTapDetector = false;

  @override
  Widget build(BuildContext context) {
    print('build: _disableTopTapDetector $_disableTopTapDetector');
    return new Stack(children: [
      new GestureDetector(
          behavior: HitTestBehavior.translucent,
          onTap: () => setState(() {
                print('bottom tapped!');
                _disableTopTapDetector = false;
              })),
      new GestureDetector(
          behavior: HitTestBehavior.translucent,
          onTap: _disableTopTapDetector
              ? null
              : () => setState(() {
                    print('top tapped!');
                    _disableTopTapDetector = true;
                  }))
    ]);
  }
}

@dragostis mentioned to me that when looking at https://github.com/flutter/flutter/issues/5201 the other day he thought taps were being dropped.  I doubt these are related, but maybe.


no user role with Azure AD authentication

Although authentication works, with the below I couldn't find any info on user roles that are assigned to users in the App Registration.
def oauth_callback(provider_id: str, token: str, raw_user_data: dict, default_user: cl.User):
print(f"{provider_id=}")
print(f"{token=}")
print(f"{raw_user_data=}")
print(f"{default_user=}")
return default_user

You need azure-ad-hybrid authentication for that. It was just added a few weeks ago.

that's great news @hayescode, but how/where? I'm on the latest 1.1.400r1

ah, the env vars. Got it!

having troubles after replacing the OAUTH_AZURE_AD_* keys with OAUTH_AZURE_AD_HYBRID_*. Get double login buttons, one works, the other seems to have no call_back uri. But that button shouldn't be there in the first place, i guess.

AADSTS700016: Application with identifier 'None' was not found in the directory 'Signify'. This can happen if the application has not been installed by the administrator of the tenant or consented to by any user in the tenant. You may have sent your authentication request to the wrong tenant.

@hayescode I looked at the raw user data (provider_id='azure-ad-hybrid'), but can't find the roles. These are the keys (no nested values):
<EMAIL_ADDRESS>'businessPhones', 'displayName', 'givenName', 'jobTitle', 'mail', 'mobilePhone', 'officeLocation', 'preferredLanguage', 'surname', 'userPrincipalName', 'id', 'image'])

Add id_token to your params in the decorated login params, then decide it using jwt

Not very familiar with this. When not using msal, it gets tricky very quickly. This what I used:
from msal.oauth2cli.oidc import decode_id_token
decode_id_token(id_token=token, client_id=os.environ['OAUTH_AZURE_AD_HYBRID_CLIENT_ID'])

this is what it returns:

msal.oauth2cli.oidc.IdTokenAudienceError: 3. The aud (audience) claim must contain this client's client_id "---GUID---", case-sensitively. Was your client_id in wrong casing? Current epoch = 2024-07-19 18:51:19.  The id_token was approximately: {
"aud": "https://graph.microsoft.com",
...

it's https://graph.microsoft.com instead. I can fill that in and it will not complain, but that doesn't make any sense. Furthermore, no roles.

Add id_token params at the end.
@cl.oauth_callback
def oauth_callback(provider_id: str, token: str, raw_user_data: dict, default_user: cl.User, id_token: str):

Then decode the id_token. You don't need any other API calls you already have the token with app roles here.

Yes, that does it @hayescode! Last issue, do you know why there are 2 auth "flows"/buttons presented? The one not working calls a microsoft authorize with:

client_id: None
tenant: None
redirect_uri=http://localhost:8000/auth/oauth/azure-ad/callback

It seems it is the "azure-ad" flow, for which I didn't provide the env vars of course. I guess I need a way to disable it somehow?

For those with the same problem, I have a temporary work around. Add this to stylesheet.css:
/* Temporary patch for "duplicated" continue with Microsoft */
.MuiStack-root .MuiButtonBase-root:first-of-type:has(span > svg[data-testid="MicrosoftIcon"]) {
    display: none;
}


DIP ON STAKE

my wallet got dip 500, 300, 1343, 2546. what should i do bro?

Wow that's good.  immediately post 400$.  I'm sure 8 hours later your profit will come out bro.

Wow that's good. immediately post 400$. I'm sure 8 hours later your profit will come out bro.

why 400$?

Wow that's good. immediately post 400$. I'm sure 8 hours later your profit will come out bro.

why 400$?

because there are 4 dip transactions.

does 1 dip transaction cost 100$?

does 1 dip transaction cost 100$?

yes, because blockchain algorithm standard is 100$ usd.

before this has happened.  and I supply it to the server as a reserve fund to take the dip license.  but the bonus hasn't come out yet.

You are wrong.  supply for dip license is different from posting.  It's only natural that the bonus hasn't come out yet.  because you do sync dip not do post

do as per blockchain algorithm.  do not be careless in making steps.  Just do as I tell you.  otherwise, your server will crash.

You have a little more work to do.  if 8 hours are passed you will always be faced with the same problem.

so should i post or take Dip license bro?

so should i post or take Dip license bro?

do not synchronize DIP if the bonus has not been distributed smoothly.  with you posting, dip license will sync automatically.

so should i post or take Dip license bro?

do not synchronize DIP if the bonus has not been distributed smoothly. with you posting, dip license will sync automatically.

so I have to post before 8 hours for 400$.  and the bonus will come out after it?

remember, you only have 8 hours to do it.  otherwise, your algorithm will start over.

thanks for the info bro

This is my server bro

 This is my server bro
Wow..

Hello bro are you online

bro, I've posted 400$ but the profit hasn't worked yet.
the details are: 1 user 100$, 1 user again 275$, and 1 creator 1 30$.
I have 2 creator wallets bro.

Yes, iam.
do you have 2 creators?
then when will each creator post?

creator 1 last post last month.  then last night posted again to make up the 400$ shortfall.
while creator 2 for the past few weeks has posted 100,200,500 posts.

what?  Your main wallet hasn't posted for more than 20 days?
user posts should suffice.
You must have a reserve fund.

what? Your main wallet hasn't posted for more than 20 days?
user posts should suffice.  You must have a reserve fund.

if I have enough budget for the machine, which is better between posting and buying a machine?

what? Your main wallet hasn't posted for more than 20 days?
user posts should suffice.  You must have a reserve fund.

if I have enough budget for the machine, which is better between posting and buying a machine?

My advice is to buy a more mainstream machine.  because after the engine, you don't have to post.  everything will work itself out.
but your main wallet must post once a month

so wallet creator 1 must post 500$ every month?

Yes

Only your wallet


bgpv2: relax mandatory PeerASN field in BGP peer configuration

Users can decide to omit PeerASN field, in which case underlying implementation will not do ASN validation in BGP OPEN state. Once peering is established and runtime state like BGP CLI or CiliumBGPNodeConfig CRD status will reflect peer ASN.
Now if user does not specify PeerASN, 0 will be passed to Cilium BGP Control Plane.

/test

/test


ci: unit test coverage

Detailed description:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Checklist

[ ] Documentation added
[ ] Tests updated

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f95dac1). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #53   +/-   ##
=========================================
  Coverage          ?   82.58%
=========================================
  Files             ?       13
  Lines             ?      735
  Branches          ?        0
=========================================
  Hits              ?      607
  Misses            ?      117
  Partials          ?       11

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add "which message was seen" feature in onMessageSeen()

I can't find any way to check which message was seen. It would be greater to add a "mid" like object or property (not too familiar with OPP ;-p ) so that actually which message was read can be found.
If there is already a way, please tell me.

It's been a while since I tinkered with this, but I think the method is wrongly named, perhaps it should have been called onThreadSeen, or something 😕.
Arguably, the documentation is very lacking on this point, but perhaps onMessageDelivered is what you're searching for?

Just checked, we don't receive data about which message is marked as seen, so I'll close this, since we can't fix this


What is behavior of Startegy OnDemand validation

In validationStrategy has 2 strategy

OnDemand
OnChange

What is behaviour expected in OnDemand?
Because the OnChange has validate when change input.

OnChange - triggered when happens any changes on the field.
OnDemand - you have to trigger the validation manually by calling form.validateForm or form.validateFields.

I'll close the issue since we answered the question.
We're working on a new version of the documentation that will cover the details of the form strategy.


FTFunction maps to "functional" in the JSON schema

Version 3.3.0 of the Vega-Lite spec says that the fieldTitle takes "verbal", "functional", or "plain"
https://github.com/vega/schema/blob/master/vega-lite/v3.3.0.json#L3139
(I noted this when I was updating hvega)

Thanks very much Doug - well spotted.


What version of OR-Tools and what language are you using?
Version: v9.10
Language: C++
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
CP-SAT
What operating system (Linux, Windows, ...) and version?
Linux
What did you do?

Search codebase for diversify_lns_params with git grep:$ git grep -l diversify_lns_params
ortools/julia/ORToolsGenerated.jl/src/genproto/operations_research/sat/sat_parameters_pb.jl
ortools/sat/sat_parameters.proto

Search git history using pickaxe to find relevant commits:$ git log --oneline -S diversify_lns_params
7574421541 export julia wrappers
2264cb9059 [CP-SAT] more aggregate presolve; fix bug with no_overlap and duplicated intervals; add parameter for the probing time limit; use all threads to find the first solution while the lns workers are idle; tweak lb_tree_search by allowing more glop iterations at the root node
...

Inspect commit 2264cb9059 to see that the relevant code was removed from ortools/sat/cp_model_solver.cc

What did you expect to see
Code implementing the documented behavior ("registers more lns subsolvers with different parameters")
What did you see instead?
Only the sat_parameters.proto file and a generated Julia version of the same contain the string diversify_lns_params.
Anything else we should know about your project / environment
This parameter looked promising to me, and I would have benefited from a comment mentioning it no longer has an effect.

clearly not an issue. It may come back at some point.


remove(getEnv): unnecessary utility

also you shouldnt throw errors if DISCORD_TOKEN is empty because people could name it something else!!!

also you shouldnt throw errors if DISCORD_TOKEN is empty because people could name it something else!!!

great point, we should try to avoid forcing conventions as much as possible


Screenshots need update

[x] Any screenshots related to role should be updated since the word "Authority Settings" is changed to "Authorization Settings".  This one is low priority

[x] https://kubesphere.com.cn/docs/v2.1/zh-CN/installation/intro/  and the English one need to update the first screenshot since the supported  k8s versions changed to 1.17.x

For "Authorization Settings":
Getting Started with Multi-tenant Management:

App Store

For Installation screenshot
https://kubesphere.com.cn/docs/v2.1/en/installation/intro/
https://kubesphere.com.cn/docs/v2.1/zh-CN/installation/intro/

verified


Improve virtual weave Pairing test round 1

--Remove duplicate src/test-apps/happy/lib/WeaveDeviceMgr.py, use the
default WeaveDeviceMgr.py in weave
--Remove src/test-apps/happy/test-templates/WeaveBleCentral.py, make
weave ble test use apis from default WeaveDeviceMgr.py instead of CLI
tool
--Fix customized port for Service Directory in ToolCommonOptions.cpp and
update virtual weave pairing test.
--Add scalable support for weave pairing service test
--Initially merge ble test into happy/lib/WeaveDeviceManager.py, further refactor
needed.

Codecov Report

Merging #236 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #236      +/-   ##
=========================================
+ Coverage    54.2%   54.2%   +<.01%
=========================================
  Files         332     332
  Lines       56424   56424
=========================================
+ Hits        30583   30584       +1
+ Misses      25841   25840       -1

Impacted Files
Coverage Δ

src/test-apps/ToolCommonOptions.cpp
21.11% <0%> (ø)
:arrow_up:

src/lwip/standalone/sys_arch.c
69.48% <0%> (+0.46%)
:arrow_up:

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 a2aa275...c2967ac. Read the comment docs.


Implement METcalcpy as the statistical package in METexpress

Replace italics below with details for this issue.
Describe the Enhancement
METexpress should use the METcalcpy package to perform calculations, to ensure that it produces the same results as METviewer.
Time Estimate
3-5 days
Sub-Issues

[ ] Add METcalcpy repo as a sub-module in METexpress.
[ ] Utilize METcalcpy code for calculations.

Relevant Deadlines
List relevant project deadlines here or state NONE.
Funding Source
Define the source of funding and account keys here or state NONE.
Define the Metadata
Assignee

[ ] Select engineer(s) or no engineer required
[ ] Select scientist(s) or no scientist required

Labels

[ ] Select component(s)
[ ] Select priority
[ ] Select requestor(s)

Projects and Milestone

[ ] Review projects and select relevant Repository and Organization ones or add "alert:NEED PROJECT ASSIGNMENT" label
[ ] Select milestone to next major version milestone or "Future Versions"

Define Related Issue(s)
Consider the impact to the other METplus components.

[ ] METplus, MET, METdatadb, METviewer, METexpress, METcalcpy, METplotpy

Enhancement Checklist
See the METplus Workflow for details.

[ ] Complete the issue definition above, including the Time Estimate and Funding Source.
[ ] Fork this repository or create a branch of development.
Branch name: feature_<Issue Number>_<Description>
[ ] Complete the development and test your changes.
[ ] Add/update log messages for easier debugging.
[ ] Add/update unit tests.
[ ] Add/update documentation.
[ ] Push local changes to GitHub.
[ ] Submit a pull request to merge into development.
Pull request: feature <Issue Number> <Description>
[ ] Define the pull request metadata, as permissions allow.
Select: Reviewer(s), Project(s), Milestone, and Linked issues
[ ] Iterate until the reviewer(s) accept and merge your changes.
[ ] Delete your fork or branch.
[ ] Close this issue.

@bonnystrong @ksearight As of now I think there are two obstacles we'll need to overcome to make this work:

METexpress updates are released at a much greater frequency than METcalcpy updates, so if we need to change something in METexpress that also requires a change in METcalcpy, we'll either have to wait months for a METcalcpy update to be released, or we'll have to maintain our own branch of METcalcpy that we use for app builds (not best practice).
METcalcpy is tailored to how METviewer produces plots, but there are a few deliberate differences in our methodology. For example, METexpress excludes duplicate data while METviewer does not, and METexpress matches on both time and level while I believe METviewer just matches on time.

Everything in METexpress is now using METcalcpy except for ensemble and MODE statistics. The former will be trivial to add, the latter not so much. The matching algorithm also needs to be updated to use the new data structures, but that shouldn't be too big a deal.

This work has been largely done and I am creating a preliminary pull request. There are a few sub-tasks for future work, but I will create new, separate issues for those.


LineEdit: Now double click to select a word, and triple click to sele…

…ct all the content
Fix #46259
I used the code from TextEdit to make this one.

Thanks!

@kuruk-mm Thanks for the contribution!
I noticed that due to commit 06ae77a320ba7b83616e4a30b1f40152bd1d3a82 the method "shaped_text_get_word_breaks" now breaks on underscores (aka _ ), which means that a doubleclick on the editor window on a variable with_underscores_like_this will only select part of it (word by word split by the underscores) instead of the whole variable. Is this intended with your commit? I did some patch on my local godot to be able to select the entire variable name for myself and just want to know if this is something I should open an issue or PR about.. Thanks!

I noticed that due to commit 06ae77a the method "shaped_text_get_word_breaks" now breaks on underscores (aka _ ), which means that a doubleclick on the editor window on a variable with_underscores_like_this will only select part of it (word by word split by the underscores) instead of the whole variable.

Ah yeah, I had the same issue. This is quite annoying.


[FEATURE] Smart Contracts and Conditional Contracts

According to https://lucentgold.com, it was said that we would have "smart contracts and conditional contracts."
At the time of writing, we still do not have such a feature, and therefore must be met.
(This is currently being worked on the fork I made on the patch-smart-contracts where smart contracts and conditional contracts are being made)

If anyone can help with linking the QT design to C++, feel free to reply back to this thread! Much appreciated!


Support for golang version 1.13.X

As Golang version 1.13.X is released since 3rd September, is this version already supported?

Go is pretty good with its backwards compatibility so unless you're a big project like Kubernetes it is fairly safe to assume that libraries like this one don't break with minor releases.
Or in other words: I haven't seen any bug report yet saying it doesn't work, so it's supported.


[BUG] - inconsistent generated owl ontology property value type of fse:hasMultiplicity "1"

Description
The generated owl is inconsistent because of type mismatch of value property
Steps to Reproduce
use oml2owl to generate owl
load owl into some resoner (I have used protege and pellet)
The reasoner gives the error:
InconsistentOntologyException
Propellant 'has mulitplicyti' "1" expect integer
The problem seems to be this generated owl part in file
/example-firesat/src/opencaesar.io/example/firesat/wp6.4/spacecraft.oml
@rdfs:label "Propellant"
ci Propellant : fse:ConsumableMass [
	fse:hasMultiplicity "1"
]

Will be translated to
<owl:NamedIndividual rdf:about="http://opencaesar.io/example/firesat/wp6.4/spacecraft#Propellant">
<rdf:type rdf:resource="http://imce.jpl.nasa.gov/discipline/fse#ConsumableMass"/>
fse:hasMultiplicity1</fse:hasMultiplicity>
rdfs:labelPropellant</rdfs:label>
</owl:NamedIndividual>
But the 1 should be int type.
After correcting it to
@rdfs:label "Propellant"
ci Propellant : fse:ConsumableMass [
	fse:hasMultiplicity 1
]

It will be explicitly typed integer
<owl:NamedIndividual rdf:about="http://opencaesar.io/example/firesat/wp6.4/spacecraft#Propellant">
    <rdf:type rdf:resource="http://imce.jpl.nasa.gov/discipline/fse#ConsumableMass"/>
    <fse:hasMultiplicity rdf:datatype="http://www.w3.org/2001/XMLSchema#integer">1</fse:hasMultiplicity>
    <rdfs:label>Propellant</rdfs:label>
</owl:NamedIndividual>

this works for reasoner
I am not sure if it is the problem of wrong "oml" or it is issue for oml2owl
Anyway Eclipse rosetta plugin does not complain about it.
Perhaps problem of rosetta Eclipse Tooling
Expected Behavior

Produce consistent owl or show error during editing

The problem here is that the OML model invalidly specified a string literal ("1") when the literal should have been of type int, which you can get when you use the literal (1) or ("1"^^xsd:int).
The problem would have been reported by the OwlReason tool (which runs the OWL reasoner) as a reasoning problem.
Whether the OML validator should have also reported this error (in the WB or any where oml-validate is run) is a different issue. I will convert this bug to a user story to consider supporting it.


nbtapi outdated version

[22:20:09 INFO]: [NBTAPI] Found Spigot: v1_8_R3! Trying to find NMS support
[22:20:09 INFO]: [NBTAPI] NMS support 'MC1_8_R3' loaded!
[22:20:09 WARN]: [NBTAPI] The NBT-API at 'package net.leonardo_dgs.interactivebooks.nbtapi' seems to be outdated!
[22:20:09 WARN]: [NBTAPI] Current Version: '2.3.1' Newest Version: 2.6.0'
[22:20:09 WARN]: [NBTAPI] Please update the nbt-api or the plugin that contains the api!
[22:20:09 WARN]: java.lang.NoSuchFieldException: ACTIONBAR
[22:20:09 WARN]:        at co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:78)
everytime i try to open a book

You are using an outdated version of the plugin.

You are using an outdated version of the plugin.