[Length] Attribute isn't supported

This issue is a feature request for a support of the [Length] attribute. Judging by the example in the Contribution guide, this issue falls into the Feature Requests category.
The System.ComponentModel.DataAnnotations namespace has 3 (actually even more, but it is not within the scope of this Issue) Attributes responsible for the string length validation:

[MinLength(length)] - MS Docs
[MaxLength(length)] - MS Docs
[Length(minimumLength, maximumLength)] - MS Docs

And while the first two are supported, the latter isn't.
Suppose, we have the following DTO definition:
class Dto
{
    [MinLength(2), MaxLength(10)]
    public string Prop1 { get; set; }

    [Length(2, 10)]
    public string Prop2 { get; set; }
}

Having such model SwaggerGen produces the following OpenApi specification:
"Dto": {
  "type": "object",
  "properties": {
    "prop1": {
      "maxLength": 10,
      "minLength": 2,
      "type": "string",
      "nullable": true
    },
    "prop2": {
      "type": "string",
      "nullable": true
    }
  },
  "additionalProperties": false
}

Where the prop1 field has both maxLength and minLength values specified, but the prop2 field doesn't have them at all.

I'll also provide a PR that solves this issue, but it'll take some time (first time I opened the source-code of this library).

It turns out that the Project has not been updated for a year (at this point in time). This means that the current version for the project is .Net 7.0, while the [Length] attribute was introduced in .Net 8.0. It's impossible to add support for this attribute at the moment.

If You stumbled upon this Issue and the Project hasn't yet migrated to .Net 8, then you can make use of the following SchemaFilter in Your Solution:
internal sealed class LengthAttributeSupportSchemaFilter : ISchemaFilter
{
    public void Apply(OpenApiSchema schema, SchemaFilterContext context)
    {
        if (context.MemberInfo is null)
        {
            return;
        }

        var lengthAttributes = context.MemberInfo.GetInlineAndMetadataAttributes().OfType<LengthAttribute>();
        foreach (var lengthAttribute in lengthAttributes)
        {
            ApplyLengthAttribute(schema, lengthAttribute);
        }
    }

    private void ApplyLengthAttribute(OpenApiSchema schema, LengthAttribute lengthAttribute)
    {
        if (schema.Type == "array")
        {
            schema.MinItems = schema.MinItems is null
                ? lengthAttribute.MinimumLength
                : Math.Max(schema.MinItems.Value, lengthAttribute.MinimumLength);

            schema.MaxItems = schema.MaxItems is null
                ? lengthAttribute.MaximumLength
                : Math.Min(schema.MaxItems.Value, lengthAttribute.MaximumLength);
        }
        else
        {
            schema.MinLength = schema.MinLength is null
                ? lengthAttribute.MinimumLength
                : Math.Max(schema.MinLength.Value, lengthAttribute.MinimumLength);

            schema.MaxLength = schema.MaxLength is null
                ? lengthAttribute.MaximumLength
                : Math.Min(schema.MaxLength.Value, lengthAttribute.MaximumLength);
        }
    }
}

I'm not closing this Issue, since the solution I've provided is temporary. I hope that the Project'll migrate to the .Net 8 eventually and we'll be able to add support for the [Length] attribute on the Project's source-code side.

ITNOA
I need this feature too.

I wanted to help you out by implementing this, but I need .NET 8 support for this... Guess I will start with reviewing the .NET 8 PR instead :)

@satma0745 @dnperfors As you can see, .NET 8 added in PR #2799 and merge it.
So do you can start implementing this feature?
thanks

Yeah, I'll start working on it

@soroshsabz
Yeah, I'll start working on it

@soroshsabz Opened a PR: #2882


Workflow plugin architecture

Make workflows loadable as plugins.
See https://github.com/ArjanCodes/2021-plugin-architecture for an example.

More references:
https://flask.palletsprojects.com/en/3.0.x/blueprints/
But instead of importing a class like Blueprint,
we can define a Protocol so that the plugins don't necessarily have to be dependent on beamlime.

Have a look at https://packaging.python.org/en/latest/guides/creating-and-discovering-plugins for a way to register plugins using entry points.

Loaded plugins should be used somewhere here: https://github.com/scipp/beamlime/blob/d2bc0b6d0593ac6ec28310006c5dc1b1e20f5752/src/beamlime/stateless_workflow.py#L36-L38


Reconnect evnent on Client Disconnect

Using the default options from client side code and XHR Polling transport on the server.
when a client diconnect somehow, and its browser still alive (no refresh, or forced disconnect) it triggers the disconnect event on the server properly.
However the same client won't reconnect again even though its browser is open and connected...
This happens only on XHR Polling.
var chat = {
    socket : {},

    init : function(){
        chat.socket = io.connect();

        chat.socket.on('connect', function () {
            console.log("connected");
        });

        chat.socket.on('reconnect', function () {
            console.log("Reconnected to the server");
        });

        chat.socket.on('reconnecting', function () {
            console.log("Attempting to re-connect to the server");
        });

        chat.socket.on('error', function (e) {
            console.log(e ? e : 'A unknown error occurred');
        });

        chat.socket.on('message', function(message){
            console.log(message);
        });

        chat.socket.on('users', function (users) {
            console.log(users);
        });
    }
};

@alonronin which browser / os ?
we're seeing a similar issue in ff4

google chrome 17.0.963.56 m windows 64bit
didn't test on other browsers....


[desktop] notebook/browse/  view gives 404 response in hue 4.4

I am trying to access the hue/notebook/browse/<database>/<table> endpoint in hue 4.4 in order to browse table results.
It appears that the hue/notebook/browse/<database>/<table> view is no longer working in hue 4.4 and I receive a 404 response
From desktop/lib/notebook/urls.py I can see the url is still present
url(r'^browse/(?P<database>\w+)/(?P<table>\w+)/(?P<partition_spec>.+?)?$', notebook_views.browse, name='browse'),

Is there some way of still accessing this view with a get request?

How do you access the page?
Currently: right click on a table in the left assist and "Open in Editor" calls it under the cover

We are truing to access the the page from a direct url link, which would open the editor with a limit 100 query.
For example the url
https://demo.gethue.com/hue/notebook/browse/foo/bar
would result in an editor with the executed query
"SELECT * FROM foo.bar LIMIT 100"
We recently upgraded from version 3.12 where this endpoint was accessible and I would love to find a way to have this view accessible once again.

Indeed, it is not implemented, and probably need changes in
https://github.com/cloudera/hue/blob/master/desktop/core/src/desktop/templates/hue.mako#L521
https://github.com/cloudera/hue/blob/master/desktop/core/src/desktop/js/onePageViewModel.js#L593


Added scrollToElement steps and tests

Details
This feature is an action that lets the user scroll to a specific element  using the scrollIntoView method
Observation
This feature behaves like the resizeScreenSize feature just added. When i coded this feature without the this.page.screenshot() function or without the this.page.waitFor() function the test results where inconsistent. However since I've added the waitFor function I didn't get anymore failure.

Nice PR @valentinmagot - Few little things to tweak, but overall, looks good!  That's interesting that this had the same fails as the resizeScreenSize without the page.waitFor.    Something I'm going to look into adding is an onError screenshot feature that takes a screenshot whenever any test fails.
That should help give us some clues about what's going on with these cases.

Thanks for the review and feedback @patheard - In every programming language i used i always had a hard time with the const key word even if it should be simple, it is the kind of simple concept in programming that i cant seems to understand completely.

Not a problem @valentinmagot .  A simple way to think of it is to use const if the variable doesn't need to be reassigned after it has been created.  For example, this will fail:
const aNumber = 27;
aNumber = 42; // exception thrown

In the above case, you'd want to use let, since you're reassigning:
let aNumber = 27;
aNumber = 42; // no exception, JS is happy, life is good

One note, is for things like objects and arrays, you can use const and then update the array or object properties, as long as you don't reassign the whole thing:
// legit
const aPerson = {
  hair: "brown"
}
aPerson.hair = "blond";

// will fail since variable is assigned a new object
aPerson = {
  hair: "blond"
};

// legit, just adding an element to the array
const someArray = ['foo', 'bar'];
someArray.push('bam');

// will fail since variable is assigned a new array
someArray = ['bam'];


perf!: avoid extra copies of immediate args

The Instruction structure no longer keeps a copy of whatever value follows the tentative instruction. If it's an immediate, other fields are enough to deduce that. If the argument is missing, memory access will make that evident.
There's no need for that copy.
Because a public field of a public structure is gone, it's a breaking change.
Checklist

[ ] Linked to Github Issue
[x] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.
[x] CHANGELOG has been updated.

nice one!


[Site] Mailing list: update user and dev hyperlinks

Signed-off-by: marcocosta1618<EMAIL_ADDRESS>Description
This PR fixes  #435
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

My pleasure, really!

Haha! 😎


Add missing ) in GetPathOfFileAbove example

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/msbuild/property-functions.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/msbuild/property-functions.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


feat: enable noImplicitAny

As the name suggests, this pull request enables the "noImplicitAny" rule in the tsconfig and adds all typings missing to make this possible.
This enables the end users to use "noImplicitAny" (or not) in their own repositories. Right now the end user was required to set "noImplicitAny" to "false" in his repository due to this library not being compatible (which is my case - thus the contribution), which by many is considered a bug.

this is a massive pr - thank you for that.
i'll try to review and merge this week.

It's not that big really, the amount of lines is just package-lock.json (I believe I might have a newer version of npm and it restructured the file?). Anyways, the real amount of changes to review is very manageable 😅 Cheers!

Thank you @vladmandic !


Introductory React Router.

Styling way off.

Wanted to start introducing routes even with a crappy nav bar.
Halp @philplckthun

https://www.youtube.com/watch?v=9OyDK98w77M

Current layout (yes, styling is not adjusted):

Seems like we should submit the color prop top down to the children of the main app.

Closing in favor of #24


FIX: Beatmap id is signed

Unsubmitted beatmaps have an id of -1, which is 4 billion in uint32.

uhhhhhhhhhhhh

yes


Implement response models

Discarding this for now as we should let the user decide how he wants to process the response instead of feeding him pre-processed data.


Conflict with fancybox

Hello. This jQuery plugin is very useful (I'm using it along with Superslides script) but on the other hand it causes a conflict with Fancybox because it changes the way that script calculates top and left positioning animation. Is there any way to prevent jQuery-Animate-Enhanced from working on divs where Fancybox is expected to be activated? I red the instructions coming with the plugin but I didn't fix it. Any help will be appreciated. Thanx.

Tried your method Ipshal, when scrolling the page via mousewheel, the fancybox window still jumps back and forth between top left and center. Any other solution?


Tooltips and axis label positioning

Hello,

Question for @Globegitter and @gionkunz - I can't seem to get the tooltips to work on this graph -
http://dev.klp.org.in/stories/
The first graph you'll see type of respondents. I've had to rename the tooltip class to chartist-tooltip because of the conflict with the main css. Any help would be appreciated. I tried an example separately and that worked. I can't tell what's going on here.

Question for @gionkunz - when the screen size goes < 750 px, I rotate the axis labels, you'll see the 45 degree rotation of the label eats into the graph area - under the heading preschool summary. for the same respondent graph. Is this because text-anchor:middle is not working, I have a y translate of 30 px, But i was wondering if there's an elegant way to display this label?

Hi @meghaklp
Nice to see that you're using Chartist on the school site! :-)

jQuery 1.9.1 does not support adding event listeners to SVG elements with delegation. Seems that this was a bug :-( This support started with 1.10.0, can you upgrade to something above 1.10.0 ?

This does not work on SVG with jQuery < 1.10
$chart.on('mouseenter', '.ct-bar', function() {});

However, this does:
$chart.find('.ct-bar').on('mouseenter', function() {});

As the labels are horizontally centered, you need to set the transform origin to 50% also for the X axis. Try this style for your style below 750px:

transform-origin: 50% 50%;
transform: translateX(0px) translateY(20px) rotate(-45deg);

Thanks so much @gionkunz ! Both these worked. We hope to use a lot more chartist all over the website :)

@gionkunz - I still have one issue though -
http://dev.klp.org.in/stories/
There are two vertical bar  charts on this page, the tool tips on the first one Types of Respondents is working fine.
The tool tips on the Call Volume chart (if you collapse the IVRS section/accordion) has gone astray. I tried changing absolute to relative, but there's a lot of flicker as you mouse over. I'm seeing this on Chrome 42.0

@meghaklp the label is positioned absolute and therefore the chart needs to be positioned relative to form a new containing block where absolute positioned elements will be laid out. Therefore just put a position: relative on your chart elements if you're using tooltips.
Now it's questionable if we should't include a position: relative in the default styles of chartist. I guess as there's nothing the chart element need to be laid out against that is outside of the chart container it would be safe to add a position: relative on the container in the Chartist default styles.
@Globegitter It would be nice if you could ensure a position: relative on the Chartist container when your plugin gets initialized as it relies on that somehow.

I'm closing this issue for now. Feel free to comment further. Cheers Gion


:seedling: Fix pinsDependencies outcomes

What kind of change does this PR introduce?
probe cleanup

[X] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?

OutcomeNotAvailable was used for cases where there are no dependencies.

The description mentions this is for API issues which isn't the case here.

OutcomeNotApplicable was used for cases where our data is inconsistent?
Probe outcome descriptions are wrong

What is the new behavior (if this is a feature change)?**

Inconsistent data states are now OutcomeNotSupported, as OutcomeError is already used for processing errors (#3515)

I'm not sure this is a perfect match, open to suggestions

Situations with no supported dependencies now use OutcomeNotApplicable
Probe documentation updated to fix copy/paste mistake
Tests were added for processing errors and no dependencies
Finding locations were simplified (see 6439b056c306bed2429acf5304fa21338f8dfd52 for details)

[X] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes
Fixes #3855

Special notes for your reviewer
Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the release-note
(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

NONE

/scdiff generate Pinned-Dependencies


How to avoid using Poolable Component?

public class AnimationComponent implements Component {
       IntMap<Animation> map ...
}

if AnimationComponent is removed from the PooledEngine, the animation map value is not automatically cleared.
Ex. The character animation will remain in the map, if you create another entity, let say a coin then add animation. The coin animation will be character animation.
Entity character  = engine.createEntity();
AnimationComponent animation = engine.createComponent(AnimationComponent.class);
animation.map.put(0, CHARACTER_WALKING );
character.add(animation);
engine.add(character);

 engine.removeEntity(character);

Entity coin = engine.createEntity();
AnimationComponent animation = engine.createComponent(AnimationComponent.class);
animation.map.put(0, COIN);
coin.add(animation);
engine.add(coin);

The solution. But I'm afraid because I always forgot to add Poolable to my newly created component, that's why I want to avoid using Poolable.
AnimationComponent implements Poolable {
     @override
      public void reset() {
          map.clear();
      }
}

Perhaps using an abstract clas BaseComponent which implements Poolable and thant let all your components extend BaseComponent? Or something similar.

As the wiki states, you should implement the Poolable interface if your components need to be cleaned up when returned to the pool.
Example:
public class PositionComponent implements Component, Poolable {
    public float x = 0.0f;
    public float y = 0.0f;

    @Override
    public void reset() {
        x = 0.0f;
        y = 0.0f;
    }
}


How to reduce cpu and memory utilization

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
None
OS Platform and Distribution
window 10
MediaPipe version
<IP_ADDRESS>
Bazel version
No response
Solution
mediapipe hands
Programming Language and version
python
Describe the actual behavior
the cpu utilization is 16% and the memory utilization is 162mb

Describe the expected behaviour
the cpu utilization the memory utilization can decrease

Standalone code/steps you may have used to try to get what you need
import cv2
import mediapipe as mp
mp_drawing = mp.solutions.drawing_utils
mp_drawing_styles = mp.solutions.drawing_styles
mp_hands = mp.solutions.hands

# For webcam input:
cap = cv2.VideoCapture(0)
with mp_hands.Hands(
    model_complexity=0,
    min_detection_confidence=0.5,
    min_tracking_confidence=0.5) as hands:
  while cap.isOpened():
    success, image = cap.read()
    if not success:
      print("Ignoring empty camera frame.")
      # If loading a video, use 'break' instead of 'continue'.
      continue

    # To improve performance, optionally mark the image as not writeable to
    # pass by reference.
    image.flags.writeable = False
    image = cv2.cvtColor(image, cv2.COLOR_BGR2RGB)
    results = hands.process(image)

    # Draw the hand annotations on the image.
    image.flags.writeable = True
    image = cv2.cvtColor(image, cv2.COLOR_RGB2BGR)
    if results.multi_hand_landmarks:
      for hand_landmarks in results.multi_hand_landmarks:
        mp_drawing.draw_landmarks(
            image,
            hand_landmarks,
            mp_hands.HAND_CONNECTIONS,
            mp_drawing_styles.get_default_hand_landmarks_style(),
            mp_drawing_styles.get_default_hand_connections_style())
    # Flip the image horizontally for a selfie-view display.
    cv2.imshow('MediaPipe Hands', cv2.flip(image, 1))
    if cv2.waitKey(5) & 0xFF == 27:
      break
cap.release()

Other info / Complete Logs
No response

Hello @xiaolinpeter Would you please try this with the latest release of v0.9.1 since we have optimized our solutions?
Also, we would request you to elaborate upon

Which parts of the solution can be optimized?
Who will benefit with this feature?
The use cases for this feature?

thanks your answer, I will try v0.9.1

<EMAIL_ADDRESS>试试一下吗？ 另外，我们会要求您详细说明

解决方案的哪些部分可以优化？
谁将从这一变化中受益？
改变这个用例？

i try the same code with mediapipe (v0.9.1), but I am sorry to tell you the cpu utilization is 16%  and the memory utilization is 321mb, the cpu  utilization have no difference, the memory utilization get bigger

Hello @xiaolinpeter
Would you please provide hardware spec of the device you are using? Because the CPU utilisation of 16% is relative metric with respect to the device config. We are able to reproduce the issue of around 170Mb memory utilisation. However, we need hardware details as well to exactly replicate this issue at our end.
Also, just to provide context, we are trying to isolate the problem between the graph and the model inference.


Document that topology charges are ignored

Description
Resolves #1059
Checklist

[x] Document details relevant to #1059

I read the updated doc strings, and I'm not sure the changes make the implications sufficiently clear for a use case like Irfan's. Should it explicitly say somewhere that in no case can isomorphic molecules end up with different charge sets?

Whoops, yes, this is meant to address #1059!

Should it explicitly say somewhere that in no case can isomorphic molecules end up with different charge sets?

Separate issue (due to my mistake) but in #1070 I'm adding an error when charge_from_molecules includes any isomorphic molecules


V19.0.0 triggers unwanted AFK when playing VRChat

It seems like at v19.0.0, ALVR will turn on AFK mode when the headset is not moving. However, its not ideal when you are lying down (like when watching videos) in VRChat because your avatar will suddenly turn to afk mode and block your view. It didn’t happen at v18.2.3 and I can’t find any settings to turn it off. This doesn’t happen at AirLink either.
Edit: added environment just in case.
Environment
Hardware
CPU: i7 11800H
GPU: RTX 3060 Laptop (MUX on dGPU mode)
Installation
ALVR Version: v19.0.0
SteamVR Version: 1.24.6
Install Type: Packaged (exe)
OS Name and Version Windows 11 (22H2) Build 22621.963

Try disabling "Pause VR when headset is inactive" in SteamVR settings in Video. Advanced Settings need to be enabled.

Thanks for the reply, but it doesn’t work. This happens only when using V19.0.0 version of ALVR, V18.2.3 or AirLink doesn’t have this behavior

@TamakiRuri can you please try nightly releases? do a bisection testing.

I see what's happening. my "Input rewrite" PR forgot to handle the proximity sensor (nightly.2022.06.26), while later I removed the input completely from the steamvr driver on steamvr (nightly.2022.07.01). I think the solution is to add back the sensor input and force it to "true".

Thats funny, i have been using v19 for months now. I probably never had the afk issue because i turned off vrc afk detection. The only thing i did noticed however was when no game is running and by not moving the HMD, SteamVR will go standby and stop transmitting data to ALVR client resulting in a cyan screen on my oculus. Never happened in games however.

@TamakiRuri Ok, I pushed a commit with the fix, but's it's untested (i'm in a hurry). let me know how it goes with the next nightly.

@zarik5 I tried it and it works just fine. Is it okay to close this issue now?

yes


feat: add JARPATTERN env var to java runtimes to identify binary

Fixes #169

/cc @jponge

To be documented too under the README. @metacosm

Can you tweak the pattern from the resource manifest?

Not everybody prepares a shaded jar name *-all.jar, some people use *-fat.jar, etc.

I don't think that it should be documented here actually. This is documented at the image level which is where it needs to be, in my opinion.

As the operator owns the process to create the pods from the list of the images mapped with the runtime property and that could also certainly overrided this property using an ENV VAR, then that should be documented here.

LGTM. Could be merged


How can I make the istio-ingressgateway internal in gcloud

Istio ingress gateway uses an external load balancer in GKE.
I don't wish to expose Kubernetes applications to the world, instead only applications which are running on Compute instances inside my project cluster should be able to access them.
How can I configure google internal load balancer for the ingress gateway?
istio version: 1.0.6
GKE version: 1.11.7-gke.4

Istio can be configured to use an internal load balancer in GKE by configuring ingress gateway value in helm chart. You can find the details in following URL: https://github.com/salrashid123/gcegrpc/tree/master/istio
However, I am unable to find a way to find a way to do this in GKE provided Istio.


msngopay api is not working

when i am creating a legal user on mango pay api then api si not working and throw exception
like this
Warning: Creating default object from empty value in /home/leadconcept/public_html/clients/abrar/freelancer/wp-content/themes/superlist-child/vpb_create_mangopay.php on line 204
Warning: Creating default object from empty value in /home/leadconcept/public_html/clients/abrar/freelancer/wp-content/themes/superlist-child/vpb_create_mangopay.php on line 211
MangoPay\ResponseException Code: 404
Message: Not found
Details:

Hi,
Could you share some more code ? Helping you with so few information is going to be difficult...

Hello @Mfaisal541 ,
Are you using our Wordpress plugin ? (your website seems to be based on WP)
Feel free to share more information about your issue on our Wordpress support forum here
Have a nice day !
Mickaël


Teensy + K20D50M - GCC ARM - blinky demo does not run

Using ARM to build a blinky - ok, but using GCC ARM - not functional. The led does not blink.

GCC ARM is not finished yet only some preliminary changes at the moment.

Ok thanks for sharing, let me know if there's any progress. I might find time to look at it, as most of users of teensy are expecting gcc arm to be functional :)

Should be fixed via #1052


GNU tar without gzip support makes backup config fail

Steps to reproduce:
Compile image with GNU tar and no gzip support
Go to http://openwrt//cgi-bin/luci/admin/system/flash
Download backup and note how the file is broken.
Actual behavior:
Broken compressed file.
Expected behavior:
Working compressed file
Recently ran into this with a custom image.
The issue is here: https://github.com/openwrt/luci/blob/master/modules/luci-mod-system/root/usr/share/rpcd/acl.d/luci-mod-system.json#L152
The solution is to pipe tar's output to gzip. No idea if this stuff supports piping.

Not going to fix this, any sane system should provide a tar command that understands the z flag.


ZEPPELIN-157: Adding Map Visualization for Zeppelin

[x] Listing Map charting Libraries
[x] Checking compatible license
[x] Adding chart library (leafletjs)
[x] Adding new button for chart mapping
[x] Loading GIS Mapp in Zeppelin
[x] Removing the setting for mapping chart
[x] Adding sample data set (loading)

[x] Get the sample GIS data and read it
[x] Add it as in tutorail, where it can be run from the clean environment

[x] Generalization data validation model

[x] Add 'validator' for the Map graph input format as service

[x] Test for this

Map Visualization for Zeppelin

Hi @Madhuka....
when you try to resize the paragraph the map does not update to the new size ... I have seen that you have:

It is possible to do dynamic
thanks for you contributions

@vgmartinez
Thanks, Yes, I will make it dynamic

Add it as in tutorial, where it can be run from the clean environment

Loading sample GIS Data for tutorial

Makers looks like

Due to -len it is breaking

Loading sample data from internet

In Data validatorservices will return error status in boolean true or false. And also it will return error msg with explanation which data record is not validated under data schema.

Data validator message will contain {'error':true/false,'msg':'error msg'}.
Data schema can be set as config file.
Data validator is expose as service and it is extendable (factory model).
Here is current screen shoot of PR.

Test are added on data validation.
PR  covers all the chart type data models in zeppelin.

Hi,
I tried testing your PR (152) but I could not see the map icon on the output page.

Did I miss something?

I don't think you missed anything if you have built it correctly. Please clear the browser cache.

Cool that did the trick.
Now that I can run the code I found this behavior :
Try the following code:
%sql
select Name  from myMap
where Country = "${Country='United States'}"
The data output of this one have only one column - name but when I press
the map icon I get it to show the data as before. (other view respect the
select and present it as expected)
Eran
On Sun, Aug 16, 2015 at 1:36 PM madhuka<EMAIL_ADDRESS>wrote:

I don't think you missed anything if you have built it correctly. Please
clear the browser cache.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/incubator-zeppelin/pull/152#issuecomment-131522919
.

The Build runs multiple time on the CI

yes, Build has run multiple times and final build was successful according to log files. But there is still one failing check.

The Travis CI is only check first Build only, isn't it? First build test failure which is bind to CI and In next build It was fine. According to log second build (final build)  says "BUILD SUCCESS"  in the log.
[INFO] Reactor Summary:
[INFO]
[INFO] Zeppelin ........................................... SUCCESS [  6.055 s]
[INFO] Zeppelin: Interpreter .............................. SUCCESS [  3.867 s]
[INFO] Zeppelin: Zengine .................................. SUCCESS [  2.659 s]
[INFO] Zeppelin: Spark .................................... SUCCESS [ 35.690 s]
[INFO] Zeppelin: Markdown interpreter ..................... SUCCESS [  0.699 s]
[INFO] Zeppelin: Angular interpreter ...................... SUCCESS [  0.316 s]
[INFO] Zeppelin: Shell interpreter ........................ SUCCESS [  0.315 s]
[INFO] Zeppelin: Hive interpreter ......................... SUCCESS [  3.282 s]
[INFO] Zeppelin: Apache Geode interpreter ................. SUCCESS [  7.297 s]
[INFO] Zeppelin: PostgreSQL interpreter ................... SUCCESS [  0.413 s]
[INFO] Zeppelin: Tajo interpreter ......................... SUCCESS [  1.088 s]
[INFO] Zeppelin: Flink .................................... SUCCESS [  4.955 s]
[INFO] Zeppelin: Apache Ignite interpreter ................ SUCCESS [  0.680 s]
[INFO] Zeppelin: Lens interpreter ......................... SUCCESS [  2.181 s]
[INFO] Zeppelin: Cassandra ................................ SUCCESS [  6.649 s]
[INFO] Zeppelin: web Application .......................... SUCCESS [ 37.351 s]
[INFO] Zeppelin: Server ................................... SUCCESS [ 18.031 s]
[INFO] Zeppelin: Packaging distribution ................... SUCCESS [ 26.737 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 02:39 min
[INFO] Finished at: 2015-08-17T05:10:28+00:00
[INFO] Final Memory: 56M/360M
[INFO] ------------------------------------------------------------------------
travis_time:end:2f3eec76:start=1439788067730573331,finish=1439788229232148013,duration=161501574682

CI Build is test with PR[1], It contain data validations (fully) and leaflet support with map and tutorial with squashing commits
[1] https://github.com/apache/incubator-zeppelin/pull/238

Did you see my comment from 5 days ago?
Now that I can run the code I found this behavior :
Try the following code:
%sql
select Name  from myMap
where Country = "${Country='United States'}"
The data output of this one have only one column - name but when I press
the map icon I get it to show the data as before. (other view respect the
select and present it as expected)
Eran
בתאריך יום ו׳, 21 באוג׳ 2015, 21:17 מאת madhuka<EMAIL_ADDRESS>
CI Build is test with PR[1], It contain data validations (fully) and
leaflet support with map and tutorial with squashing commits
[1] #238 https://github.com/apache/incubator-zeppelin/pull/238
—
Reply to this email directly or view it on GitHub
https://github.com/apache/incubator-zeppelin/pull/152#issuecomment-133519842
.

@eranwitkon It is been fixed few days ago. Please update your copy.
Now there is data validator. So when you pass 1 column test will fail. Since Zeppelin currently doesn't have notifications you will not see any messages in UI as notification. But the error messages will be printed on the console. Please see that. (There is a separate PR for notifications.)
Further if you are interested about data validator visit the documentation. #203

Shall we rebase it on latest master?

@bzz : PR is re-based and local build fine and test also fine.

@bzz : Even after rebase CI is giving fail and CI build has run for two times last one is successful and first one is fail in zeppelin server test. (where is my commits  are in zeppelin web and I have post my finding bout test in this PR)
any idea?

This branch has conflicts that must be resolved signifies that it is not aligned with the latest master, you need to make sure both checks are green here.

@buzz I will try to make both checks in green

@bzz : I have updated the PR to master. Build has run for multiple times, and any idea or reason to run build for multiple times.

anything we can do to figure out the failing CI build issue? This feature is incredibly useful- will stop quite  a lot of HTML and JS hacking for users

@randerzander thank you for bringing it to our attention!
So, it looks like selenium in-browser Integration Tests of Zeppelin Server submodule are failing here:
-------------------------------------------------------
 T E S T S
-------------------------------------------------------
Running org.apache.zeppelin.ZeppelinIT
Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 68.171 sec <<< FAILURE! - in org.apache.zeppelin.ZeppelinIT
testAngularDisplay(org.apache.zeppelin.ZeppelinIT)  Time elapsed: 67.963 sec  <<< ERROR!
org.openqa.selenium.NoSuchElementException: Unable to locate element: {"method":"xpath","selector":"//div[@ng-controller=\"ParagraphCtrl\"][1]//div[@id=\"angularTestButton\"]"}
Command duration or timeout: 14 milliseconds

@Madhuka can you verify that it actually passes well please? (at first on your local env, by something like mvn -am -pl zeppelin-server integration-test)

@bzz : I wil verify in my local env.

@bzz : ZeppelinIT which is failed in CI run fine in local environment. I have re-base (update) the PR. is there any more things to do for this PR?

T E S T S
Running org.apache.zeppelin.ZeppelinIT
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.075 sec - in org.apache.zeppelin.ZeppelinIT
Results :
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
[INFO]
[INFO] --- scalatest-maven-plugin:1.0:test (test) @ zeppelin-server ---

Map Visualization for Zeppelin would be a great addition!
I've tried the code and ran into several problems.
I don't think it's good to select the lat and lon statically like so:
https://github.com/Madhuka/incubator-zeppelin/blob/milestone05-map/zeppelin-web/src/app/notebook/paragraph/paragraph.controller.js#L866
It would be nice if they were detected automatically. Maybe if there are columns lat, latitude, lon, long or longitude, pick them?
The same goes for the key:
https://github.com/Madhuka/incubator-zeppelin/blob/milestone05-map/zeppelin-web/src/app/notebook/paragraph/paragraph.controller.js#L863
If a key exists twice, it obviously breaks. Key and message are the same though, but I may be interested in the same message at different locations.
There should be a settings drop down like for the other chart types where this can be configured. With the current version, that doesn't seem to work. (Only for me?)

@datagiraffe : thanks for the interest

Current zeppelin is written on order_of_key level. It is not written to detected automatically at the moment.

key is local variable and it is not twice. And all the variable name can have issue. That is the reason 'jshint' in zeppelin. It is not reason to break.

breaking happen in build due to CI binding test. you can find the points in this thread.

@bzz : I have build PR for data validator. At the same time I will create new PR for leaflet mapping as lot of users are interested.

@Madhuka @bzz what's holding this one up? Is there something someone else can do to get this across the goal line?

@randerzander: Thanks for ping!
@bzz : I will create new PR for adding leaflet.

@randerzander @Madhuka
I'm reviewing #262 first, already took a look at it.
I'm planning on finishing review on those 3 PR this week.
But I think this will need a new PR without data validation and rebased at least.

@corneadoug : Thanks, get  #262, it contain data validation.
So new PR will have only Map Vis.

@Madhuka Are you working on a new PR?
Tell me if you need some help

any update on this? @Madhuka this would be very cool to have

Hi @Madhuka any update on this? really appreciate your effort for this.
Thanks

After discussion with @Madhuka, this should be rebased and finished by 1st week of March. (since he is busy)
There has been a new PR opened recently (#728), however I'd like Madhuka to finish his contribution for a few reasons:

This PR was part of Google Summer of Code, and is all thats left of all the work Madhuka did, so it would be nice for him to get his contribution
This PR has been around for quite some time, and I took way too long to properly review it

However if it is not done by then, we should get things moving and go with #728, @eddies fine with that?

@corneadoug absolutely. As I mentioned in #728, I only made my PR out of expediency because I didn't see any movement here. If @Madhuka will be revisiting this PR soon, that would be ideal--he definitely deserves the credit for all his work.

@Madhuka Any update on this?

When will this be in the main stream？It is a pity to see some great function missing while the solution is already available for more than half a year. Thanks.

Based on the earlier comment, are we moving to #728 then?

Referee #765

Great - will you have time to get that ready then?
_____________________________

Nice work ! When we will be able to use these nice maps?

@Madhuka Let's close this one since you opened another one

@corneadoug As we have #765 We can close this, I am willing to finish that #765 as lot of people need this feature. Please let me know when I can continue to finish that from your end?


MudTextField value overlaps the placeholder

Bug type
Component
Component name
MudTextFiled
What happened?
If I use password chrome storage to autocomplete form with Username and Password MudTextBox fields, both of them overlap the placeholders.

Everithing is going well if I click on any place on page (like placeholder appears as a label under field when it's filled).
Expected behavior
MudTextField has expected behavior when I use autocomplete passwords.
Reproduction link
https://mudblazor.com/components/form#simple-form-validation
Reproduction steps

Use chrome browser.
Open https://mudblazor.com/components/form#simple-form-validation
Go to Simple Form Validation section.
Fill Email and Password fields.
Reload page.
Save credentials for this site.
P.S. The same behavior for my application.

Relevant log output
No response
Version (bug)
6.0.16
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Duplicate of #3486


Updated docs for MS MARCO

Replicated @rodrigonogueira4 's results. Some documentation tweaks along the way...

LGTM

LGTM


New DigitalClock.py

Please Review my new type of Digital Clock and merge it.

also pulled Python Calculator


Cache random purge when dir is overflowing

As @oknet point out, It might always purge the same url when dir is overflowing. It should be random to purge dir's in freelist_clean.

As @scw00 point out, there is a Lagain infinite loop within dir_insert and dir_overwrite if the bucket is full and the freelist is empty.
int
dir_insert(const CacheKey *key, Vol *d, Dir *to_part)
{
...snip...
Lagain:
  // get from this row first
  e = b;
  if (dir_is_empty(e)) {
    goto Lfill;
  }
  for (l = 1; l < DIR_DEPTH; l++) {
    e = dir_bucket_row(b, l);
    if (dir_is_empty(e)) {
      unlink_from_freelist(e, s, d);
      goto Llink;
    }
  }
  // get one from the freelist
  e = freelist_pop(s, d);
  if (!e) {
    goto Lagain;
  }
...snip...
}

When the freelist is empty, freelist_pop calls freelist_clean to release at least one entry and return nullptr.
inline Dir *
freelist_pop(int s, Vol *d)
{
  Dir *seg = d->dir_segment(s);
  Dir *e   = dir_from_offset(d->header->freelist[s], seg);
  if (!e) {
    freelist_clean(s, d);
    return nullptr;
  }
...snip...
}

The freelist_clean traverses all entries in the segment to find head and earliest Dir.
The 1st, 11th, 21th, 31th, etc...(every 10) Dirs will be erased.
For a URL,

It has a unique CacheKey,
And the Bucket is selected by CacheKey to store a Dir object.
Therefore it always stores its Dir to a specific Bucket.

The URL may have a high cache miss rate if the Dir stores in the 1st Entry.
IMO, It is better to randomize the starting position.


Output directory option

Hello,
another small one, while testing importing another project, I found that it would be handy to be able to set output directory for generated code from the command line, so here it is.
using -od -outputdir allows this (omitting argument uses default behaviour)
Thanks

How does this interact with config files with dependencies as used in SharpDX? Does this cause output files to be overwritten or placed in subdirectories to avoid that?

It only sets the base folder to a custom location, rest is standard behaviour (which still creates folder hierarchy as before)

I'm all fine to add an option like this.


Any plan to incorporate a native param.bicep file type?

Not an issue per se, but is there a plan in the future to incorporate a native param file based on bicep syntax?  Thank you friends

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

ID: cdf73225-d2e6-b23e-b355-fbb9d3fcdca3
Version Independent ID: fa832878-9821-f222-5f15-ad8646737a04
Content: Create parameter file for Bicep - Azure Resource Manager
Content Source: articles/azure-resource-manager/bicep/parameter-files.md
Service: azure-resource-manager
Sub-service: bicep
GitHub Login: @mumian
Microsoft Alias: jgao

Thanks for bringing this to our attention. Your question has been shared with the content owner for further review.

@helplessJ - thanks for your question. Yes, that is being considered. The best place to track that information and provide feedback to the product team is through the project repo. Here's a GitHub issue for that subject.

Given that there is no documentation to update currently, we'll close.
#please-close


TypeLoadException is thrown in complicated cases with inheritance if renPublic=true and several assemblies are involved

Consider the main assembly SignatureMismatch2:
using System;
using SignatureMismatch2Helper;

namespace SignatureMismatch {
	public interface IInterface
	{
		void Method(External obj);
	}

	public class Class : IInterface
	{
		public void Method(External obj) => Console.WriteLine(obj.Name);
	}

	public class Program {
		static int Main(string[] args) {
			Console.WriteLine("START");
			new Class().Method(new External());
			Console.WriteLine("END");

			return 42;
		}
	}
}

And the helper assembly SignatureMismatch2Helper:
namespace SignatureMismatch2Helper {
	public class External
	{
		public virtual string Name => "External";
	}
}

When I'm trying to apply renaming obfusctation with renPublic=true the following exception is being thrown:
Unhandled Exception: System.TypeLoadException: Signature of the body and declaration in a method implementation do not match.  Type: '?????????????????????????????????????????'.  Assembly: 'SignatureMismatch2, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
   at ?????????????????????????????????????????.?????????????????????????????????????????(String[] )

Instead of correct output.
It seems to me this is because of missing obfuscation of External type in IInterface.Method(External) signature:
void \u202A\u206D\u206E\u206E\u202A\u206B\u200D\u206F\u200C\u200B\u206F\u202D\u200B\u206F\u200E\u206E\u206C\u206F\u206B\u202C\u202A\u206F\u200D\u200E\u202E\u200B\u206D\u206E\u202A\u206E\u206E\u206A\u200B\u200B\u206F\u200D\u206E\u206D\u202B\u202E\u202E(External);

The signature of the implementation class Class.Method(External) is being transformed correctly:
public void \u202A\u206D\u206E\u206E\u202A\u206B\u200D\u206F\u200C\u200B\u206F\u202D\u200B\u206F\u200E\u206E\u206C\u206F\u206B\u202C\u202A\u206F\u200D\u200E\u202E\u200B\u206D\u206E\u202A\u206E\u206E\u206A\u200B\u200B\u206F\u200D\u206E\u206D\u202B\u202E\u202E(\u200D\u206B\u206B\u202D\u202E\u206D\u206E\u200E\u206B\u206E\u202D\u206F\u200F\u202C\u200E\u206A\u202B\u200C\u206B\u202B\u206A\u202E\u200E\u200D\u202E\u206E\u200B\u200E\u200C\u206A\u200E\u206D\u206F\u200D\u202D\u202C\u206F\u202A\u206F\u200C\u202E A_1)
{
	Console.WriteLine(A_1.\u206F\u202C\u202B\u200F\u200C\u206A\u200F\u206E\u206E\u206B\u202C\u200D\u206D\u200F\u200E\u200E\u202D\u206B\u202D\u206D\u206F\u200B\u202A\u200C\u200C\u206B\u200E\u200F\u206D\u206D\u206D\u206F\u206D\u202B\u200B\u202E\u200F\u200E\u202E\u202B\u202E);
}

I found the cause if this issue. The problem is that the type reference in the overwrite signature is not fixed during the renaming. The only time this happens is in case an method in another assembly is overwritten that is still being renamed.

Any change to fix it?

Yes. There is a single line that this causing the issue. This one. If I disable it or only execute it if the modules do not match everything works fine. But I am not sure why this line is required at all. I need to think about the implications of removing it (conditionally).
The reason this is a problem is that it creates a duplicated TypeRef entry for the External type, when it processes the overwrite directives for the type.

 If the import is required, it's already done at this point.
The bigger issue here is that the VTableAnalyzer modifies that assembly during the analysis phase. This causes all sorts of issues, because it requires that this analyzer is executed first to cause no issues. This is not the case currently.
To avoid any further issues in the future, this analyzer must not alter the assembly during the analysis phase.

Now everything related to obfuscation works as expected in my job project, big thanks!


bug(MatMenuItem): If a  or any other node is used in the Item label, getLabel() doesn't work

Reproduction
https://stackblitz.com/edit/components-issue-vpdbyy?embed=1&file=src/app/example-component.html
Steps to reproduce:

Create a list of mat-menu-items
Use a custom node in one of them ( for example) to style the item
Use MatMenuItem.getLabel()

Expected Behavior
What behavior were you expecting to see?
item.getLabel() would work to get the label text contents of the mat-menu-item, regardless if they're in nested nodes.
Actual Behavior
What behavior did you actually see?
item.getLabel() only works if the component is specifically a text node and does not recurse into nested nodes:
https://github.com/angular/components/blob/master/src/material/menu/menu-item.ts#L177
Environment

Angular: 10.0.3
CDK/Material: 10.0.1
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): macOS

We do have a solution for this in the experimental cdk menu:
https://github.com/angular/components/blob/cdbf2c18103ef71077f87ae9e86b74e50cc2fd2c/src/cdk-experimental/menu/menu-item.ts#L134-L142


Date Format Improvements

Per @yschimke suggestion, we can move away from dev.johnoreilly.confetti.utils.DateService in a favor of simple methods to handle date.
The idea is:

Introduce common functions inside shared to format date.
Provide an implementation per platform.
Replace DateService in favor of direct calls.

It's worth noting as well that kotlinx datetime library should have functionality soon for formatting dates.....might cover at least some of want we need anyway.

Discussion for kotlinx datetime formatting is over there if anyone wants to chip in https://github.com/Kotlin/kotlinx-datetime/discussions/253. Looking forward to it although it might take a bit of time to land something.

Took this approach for now https://github.com/joreilly/Confetti/pull/372


Quick start demo doesnt work on Windows with WSL 2 Ubuntu

I tried to run the quick start demo, but the ruby and python based images error out on exec.
I don't think this is something that can be worked around locally?
$ docker ps
CONTAINER ID   IMAGE                                  COMMAND                  CREATED          STATUS                         PORTS                                                                                NAMES
714efec629c8   timescale/promscale-demo-load          "python3 load.py"        26 seconds ago   Restarting (1) 1 second ago                                                                                promscale-demo-load-1
a5888d0c126c   timescale/promscale-demo-generator     "flask run --host=0.…"   26 seconds ago   Restarting (1) 1 second ago                                                                                promscale-demo-generator-1
a8a86bdcdf76   timescale/promscale-demo-lower         "bundle exec ruby lo…"   26 seconds ago   Restarting (1) 2 seconds ago                                                                                promscale-demo-lower-1
8abcb8759d74   timescale/promscale-demo-special       "flask run --host=0.…"   26 seconds ago   Restarting (1) 4 seconds ago                                                                                promscale-demo-special-1
c984a108aa65   timescale/promscale-demo-digit         "flask run --host=0.…"   26 seconds ago   Restarting (1) 3 seconds ago                                                                                promscale-demo-digit-1
d272298bde24   timescale/promscale-demo-upper         "flask run --host=0.…"   26 seconds ago   Restarting (1) 2 seconds ago                                                                                promscale-demo-upper-1
1b28137ef3e2   vineeth97/promscale-demo-grafana       "/run.sh"                26 seconds ago   Up 22 seconds                  <IP_ADDRESS>:3000->3000/tcp, :::3000->3000/tcp                                                                                promscale-demo-grafana-1
b79f042859c4   otel/opentelemetry-collector:0.55.0    "/otelcol --config=/…"   26 seconds ago   Up 23 seconds                  <IP_ADDRESS>:4317-4318->4317-4318/tcp, :::4317-4318->4317-4318/tcp, <IP_ADDRESS>:14268->14268/tcp, :::14268->14268/tcp, 55678-55679/tcp   promscale-demo-collector-1
5c16d64a22ba   prom/prometheus:latest                 "/bin/prometheus --c…"   26 seconds ago   Up 24 seconds                  <IP_ADDRESS>:9090->9090/tcp, :::9090->9090/tcp                                                                                promscale-demo-prometheus-1
b3b41640c361   jaegertracing/jaeger-query:1.36.0      "/go/bin/query-linux…"   26 seconds ago   Up 6 seconds                   <IP_ADDRESS>:16686->16686/tcp, :::16686->16686/tcp                                                                                promscale-demo-jaeger-1
4711057370f8   timescale/timescaledb-ha:pg14-latest   "/docker-entrypoint.…"   26 seconds ago   Up 25 seconds                  8008/tcp, <IP_ADDRESS>:5432->5432/tcp, :::5432->5432/tcp, 8081/tcp                                                                   promscale-demo-timescaledb-1
0b3371656b3f   prom/alertmanager:latest               "/bin/alertmanager -…"   26 seconds ago   Up 25 seconds                  <IP_ADDRESS>:9093->9093/tcp, :::9093->9093/tcp                                                                                promscale-demo-alertmanager-1
072e34c59955   quay.io/prometheus/node-exporter       "/bin/node_exporter"     26 seconds ago   Up 25 seconds                  <IP_ADDRESS>:9100->9100/tcp, :::9100->9100/tcp                                                                                promscale-demo-node_exporter-1
$ docker logs promscale-demo-load-1
exec /usr/local/bin/python3: exec format error
exec /usr/local/bin/python3: exec format error
exec /usr/local/bin/python3: exec format error
exec /usr/local/bin/python3: exec format error
exec /usr/local/bin/python3: exec format error
exec /usr/local/bin/python3: exec format error
exec /usr/local/bin/python3: exec format error
exec /usr/local/bin/python3: exec format error

I'm able to reproduce the issue. Working on fixing the issue...

@martinothamar I managed to fix the issue before publishing the images to the timescale repo. Do you mind verifying it on your end by using images published in my repo?
To verify all you have to do is change the below image repo names in docker-compose.yaml:

timescale/promscale-demo-load -> vineeth97/promscale-demo-load
timescale/promscale-demo-generator -> vineeth97/promscale-demo-generator
timescale/promscale-demo-lower -> vineeth97/promscale-demo-lower
timescale/promscale-demo-special  ->  vineeth97/promscale-demo-special
timescale/promscale-demo-digit -> vineeth97/promscale-demo-digit
timescale/promscale-demo-upper -> vineeth97/promscale-demo-upper

Sure! Will give it a go this evening

The updated images seem to run fine, I'm having a different issue now
Docker DNS resolution doesnt seem to work in my setup. Not sure what's wrong...

Jaeger is crashlooping on this error, presumably because of DNS issues
{"level":"fatal","ts":1659648363.3808448,"caller":"./main.go:106","msg":"Failed to init storage factory","error":"grpc-plugin builder failed to create a store: error connecting to remote storage: context deadline exceeded","stacktrace":"main.main.func1\n\t./main.go:106\ngithub.com/spf13/cobra.(*Command).execute\n\tgithub.com/spf13/cobra@v1.5.0/command.go:872\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\tgithub.com/spf13/cobra@v1.5.0/command.go:990\ngithub.com/spf13/cobra.(*Command).Execute\n\tgithub.com/spf13/cobra@v1.5.0/command.go:918\nmain.main\n\t./main.go:166\nruntime.main\n\truntime/proc.go:250"}

I don't think this is related to the docker-compose config, but rather something fishy with my Docker install?
When I try to run in from powershell however it doesn't work
PS > docker-compose up -d
time="2022-08-04T23:16:07+02:00" level=warning msg="The \"PWD\" variable is not set. Defaulting to a blank string."
time="2022-08-04T23:16:07+02:00" level=warning msg="The \"PWD\" variable is not set. Defaulting to a blank string."
time="2022-08-04T23:16:07+02:00" level=warning msg="The \"PWD\" variable is not set. Defaulting to a blank string."
time="2022-08-04T23:16:07+02:00" level=warning msg="The \"PWD\" variable is not set. Defaulting to a blank string."
time="2022-08-04T23:16:07+02:00" level=warning msg="The \"PWD\" variable is not set. Defaulting to a blank string."
...
Error response from daemon: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/prometheus.yml" to rootfs at "/etc/prometheus/prometheus.yml": mount /prometheus.yml:/etc/prometheus/prometheus.yml (via /proc/self/fd/6), flags: 0x5000: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type

I'm running the setup in a digital ocean ubuntu machine. I couldn't reproduce the DNS issues that you shared above. The connection to Promscale and other components just works fine for me.
Can you share more details on what's happening on your end?

Yeah I'm pretty sure its just my Docker setup that is bugged atm. I'll confirm tonight by trying this on another computer

Hi @martinothamar
Did you manage to re-run the quick start from your end? :)

I verified it on a ubuntu machine and it works. So closing the issue. Feel free to create a new issue, If you are seeing a similar issue.


TAJO-1972: Invalid sort order with NULLS FIRST|LAST

+1 LGTM!
Ship it!

@jinossy, thank you for quick review!


fix(e2e): stabilise E2E tests

#3234

Release Note
NONE

I think we should backport in 1.9 release branch as well.


Error handling for location API

Would like to have you review this approach first.

This try catch location didn't work today as the API did return something, just an error.  I will try a different approach.

Updated to throw an exception if data is missing.   Works currently but my API for location is working no issue.  Considering number of reboots to test this I suspect my API will max later today to test the failure.

Would you be open to a 15-30 minute call at some point?  It may be easier to walk you through what I am seeing from the Canadian API.

Hey, it is late in EU now, but what about tomorrow? Please drop me an email over<EMAIL_ADDRESS>and we can have a google meet session.

Changed approach on this.  The only APIs I hit limits on are items I pass the PIN for, maintenance isn't one of them.   So I am using odometer as something to compare with.  If this hasn't increased I don't call location.   Now I am not sure holding this data in the API class is the best spot for it.  Still open a call to discuss.  I am no pro just messing around. :)

I will monitor this new change tomorrow morning as my location API is maxed out.    Now everything else works great without errors which is how it should be.  Ideally tomorrow location works fine which means we only call this occasionally and can also handle errors if it ever is maxed out.

Closing this pull to create a cleaner final one.


Missing C# 8 documentation

C# 8 adds an additional return type: IAsyncEnumerable<T> as long as you're using yield return instead of just return.
See the "What's new in C# 8" documentation under Asynchronous Streams: https://docs.microsoft.com/en-us/dotnet/csharp/whats-new/csharp-8#asynchronous-streams

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

ID: d055727b-c97c-4d25-39b8-a4d3f21d0444
Version Independent ID: c8a7c6f1-8ee1-5030-db12-3db65a968501
Content: Async Return Types (C#)
Content Source: docs/csharp/programming-guide/concepts/async/async-return-types.md
Product: dotnet-csharp
Technology: csharp-async
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

thanks for pointing this out @powerlord
I've added this to our next sprint, as we continue to update our content for all the C# 8 features.


Fixes light API for operators with two outputs

Codecov Report
Attention: 8 lines in your changes are missing coverage. Please review.

Comparison is base (70d6f64) 81.69% compared to head (56914b6) 81.63%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
- Coverage   81.69%   81.63%   -0.07%
==========================================
  Files          45       45
  Lines        6032     6055      +23
==========================================
+ Hits         4928     4943      +15
- Misses       1104     1112       +8

Files
Coverage Δ

onnx_array_api/light_api/_op_var.py
51.42% <100.00%> (ø)

onnx_array_api/light_api/_op_vars.py
44.02% <100.00%> (+0.62%)
:arrow_up:

onnx_array_api/light_api/var.py
88.32% <73.33%> (-2.91%)
:arrow_down:

... and 1 file with indirect coverage changes

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


[TwigComponent] Fix twig:lint bug with anonymous component tag

Q
A

Bug fix?
yes

New feature?
no

Tickets
Fix #1089

License
MIT

Sincerely i'm not sure how to handle this properly "today"..
There is a lot of things i'd like to refactor in a future version, and the fact than auto-closing tags and classic ones are handled by two entirely different processes is at the top of my todo :)
But right now, i'm not sure we can do better than a quick patch, allowing the "lint:twig" to not crash for anonymous twig:Foo</twig:Foo> tags.
Timeline of what "bugs":

the lint:twig command find all templates, and for each one create an empty template loader, then parse & compile the template
during the parsing, the ComponentParser calls ComponentFactory::metadataFor to find the template matching the component name
if no class-based component matches, the ComponentFactory calls the the ComponentTemplateFinder
the ComponentTemplateFinder then asks the Loader if a matching template exists
but as the Loader is temporary "empty".... 🐞

So we must ether:

A) stop resolving template during the parser/compiler work
B) ensure the TemplateFinder always contains the real "full" loader
C) change the way the lint:twig command works

A is my long-term objective, but clearly not "today".
C is not so easy, as the corresponding code is hidden between 3 private methods
So....  first suggestion (dirty but working)

Thank you Simon - beautiful PR


add easy-coding-standard

Might need to rerun tests. I'm not sure if I can do anything.

Might need to rerun tests. I'm not sure if I can do anything.

Yeah I reran it some time ago and it failed again, not sure why. Let's try again


fix error link under preview

You can still see the wrong link error under preview .

You can still see the wrong link error under preview .

What does this suppose to mean?

Not very useful in fact..
But you can view the link while you are previewing the file you are editing and click it

The Bug #3561 ,Sorry I forget that

...thanks


docker hub fails to build jewel tag

The jewel tag is not building in docker hub.
https://hub.docker.com/r/ceph/daemon/builds/bybfburnh92az44kffaafjk/

This is expected, this branch should not be used anymore, I'll remove it at some point.
Please use tag tag-build-master-jewel-ubuntu-14.04 instead.


Update regression tests

The links now work here and failed for lack of a report file ?
https://ci.runtimeverification.com/jenkins/job/Haskell Backend/job/PR-3130/1/console


Emoji and CDN

Wordpress enqueues wp-emoji-release.min.js via inline feature detection.
So suppose my website is on the subdomain www.example.com and the CDN cname is cloud.example.com
The file path in browser should be http://cloud.example.com/wp-includes/js/wp-emoji-release.min.js?ver=4.6.1
But that is not the case.
It's probably because the URLs are JSONencoded in the inline JS in the HTML.

Sorry for the delay.  Backlogged.  I'll have to do a test but i notice, at least in "Auto" mode, that wp-emojo-release.min.js file is included late (i think it is requested via javascript on the client side).  And so w3tc doesnt know about it at all -- hence, no CDN name used.  I havent checked yet but just going based on memory from way back.  I'll do some digging tonight and update you on a solution.

Hi @vijayaraghavanramanan
Ok i did some digging and yea because, as you even noticed, the url is JSON encoded within an inlined JS w3tc has no chance to modify the url to CDN.
However, a solution would be to add the following lines into your theme's functions.php file (don't forget to modify the cloud.example.com part:
function my_script_location( $script_url, $script_name ) {
        global $wp_version;
	if ( $script_name == 'concatemoji') {
		$script_url = "http://cloud.example.com/wp-includes/js/wp-emoji-release.min.js?ver=$wp_version";
	}

	return $script_url;
}
add_filter( 'script_loader_src', 'my_script_location', 10, 2 );

Cheers
Kimberly

Hi @amiga-500
Great. Yeah that code should go in functions.php.
A more general code would affect w3tc so that it can take care of such issues, not just emoji.
Which lines did you modify in lib/W3/Plugin/TotalCache.php?

Ok yea...i will not touch w3tc.  But, since you asked, here were the new lines for TotalCache.php:

First, you'd modify lines 80-85 of lib/W3/Plugin/TotalCache.php from this:

if ($this->can_ob()) {
    ob_start(array(
              &$this,
              'ob_callback'
    ));
}

to this:
if ($this->can_ob()) {
    ob_start(array(
              &$this,
              'ob_callback'
    ));
    add_filter( 'script_loader_src', array($this, 'my_script_location'), 10, 2 );
}

And secondly, adding the following function anywhere within lib/W3/Plugin/TotalCache.php (but still inside the W3_Plugin_TotalCache class):

function my_script_location( $script_url, $script_name ) {
    if ( $script_name == 'concatemoji') {
        if ($this->_config->get_boolean('cdn.enabled')) {
            $w3_plugin_cdn = w3_instance('W3_Plugin_CdnCommon');
            $cdn = $w3_plugin_cdn->get_cdn();
            $script_url = str_replace(w3_get_site_url(), '', $script_url);
            $remote_path = $w3_plugin_cdn->uri_to_cdn_uri($script_url);
            $script_url = $cdn->format_url($remote_path);
        }
    }
    return $script_url;
}


Juma Emmanuel

Name
Juma Emmanuel
Profile Image

Country
South Sudan
Region
Juba-Central Equatoria State
Organisation
No response
Languages
English: Good!   Arabic: Good!  Kakwa: fluent
Skill 1
Media & Art
Qualifications for Skill 1

Multimedia journalist. Diploma and certificate in journalism and mass communication from United Media Consultants And Trainers (UMCAT) school in Kampala Uganda.
Worked in radio for over three years.
I also have a skill in podcasting, media content creation such as text, audio and video editing
Trained by the media challenge initiative
Worked with MCI radio as host of a youth program called the youthful regional talk in Uganda Kampala.
Voice artist skills

Skill 2
Web & Software
Qualifications for Skill 2

Fact checking skills(debunking skills)
Basic cyber security knowledge.
Able to connect and use the internet for research and communication

Skill 3
Organizational Development
Qualifications for Skill 3

Leadership skills. I managed to conduct trainings and outreach programs for radio one juba.
I also served  as a minster of internal affairs  while at United Media Consultants And Trainers (UMCAT) school of journalism and mass communication
I organized school meetings.

Skill 4
No response
Qualifications for Skill 4
No response
describe yourself
I am Juma Emmanuel a multimedia journalist. I tell stories through audio, visual video and I can also use text to tell stories that's writing. I also connect people through the use of media. I use the media to create a peaceful environment, connect and educate my society. I'm also passionate about youth empowerment and one of my biggest skills include advocating for the youth to be useful and that's through the media.
I also have a skill in podcasting, media content creation such as text, audio and video editing
And basic cyber security knowledge.
E-Mail
<EMAIL_ADDRESS>Phone
+211922300013
WhatsApp
+256782721101
Website
No response
Telegram
No response
GitHub
No response
LinkedIn
https://www.linkedin.com/in/juma-lo-sunday-emmanuel-95174a213
Twitter
@JumaloSunday
Facebook
juma lo Sunday
Instagram
No response
Mastodon
No response
WikiFab
No response
Other
No response

profile created


Please add typing.Literal result type for key in TypedDict class

search existing issues to avoid creating duplicates
None found
Requested Feature
I would like vscode-python to be able to see that

a key, required or optional exists in the TypedDict class
Here is an example:

import typing
import decimal
DictAnnotation = typing.TypedDict('Annotations', {'some_str': str, 'some_int': int}, total=False)

class SomeDict(dict):
    annotation = DictAnnotation

    @overload
    def get_item(self, name: typing.Literal['some_str'], in_annotation: typing.Literal[True]) -> str: ...

    @overload
    def get_item(self, name: typing.Literal['some_int'], in_annotation: typing.Literal[True]) -> int: ...

    @overload
    def get_item(self, name: str, in_annotation: typing.Literal[False]) -> decimal.Decimal: ...

    def get_item(self, name: str, property_property: bool):
        # if an attribute does not exist
        return self[name]

some_dict = SomeDict(SomeDict.annotation({'some_str': 'a', 'some_int': 1}))
my_bool = 'some_str' in SomeDict.annotation # key definition exists in TypedDict class, seen as bool but should also be typing.Literal[True]
# I want the type hint to work on some_str
some_str = some_dict.get_item('some_str', my_bool)

my_bool = 'str_of_any_type' in SomeDict.annotation
str_of_any_type = some_dict.get_item('str_of_any_type', my_bool)
# I want the type hint to work on str_of_any_type

This is generally applicable to classes that inherit from dict, and have properties defined with a TypedDict.
This extra in_annotation argument is needed to ensure that the type hint for some_str does not include the type hint for get_item(self, name: str, in_annotation: typing.Literal[False]) -> decimal.Decimal: ...
I understand that using a TypeDict for SomeDict can accomplish this, but I am not able to use TypeDict because I need SomeDict to be able to be mixed in to other classes.

You're using TypedDict incorrectly here. Please refer to PEP 589 for details about how TypedDict is intended to be used.
When pyright (the type checker upon which pylance is built) sees a TypedDict declaration, it synthesizes overloaded get methods with literals.
class SomeDict(TypedDict):
    some_str: str
    some_int: int

some_dict = SomeDict(some_str = 'a', some_int = 1)
x = some_dict.get('some_str') # Type is 'x' is 'str'

As I said, TypedDict won't work in my use case. Thank you for pointing out that vscode can correctly see types using the get method on an instance of TypedDict.
My ultimate goal here is to store a dict where some type hints are know for certain keys and for all unspecified keys, there is a different type.
That'w why I showed the decimal.Decimal type as the default type.
This may be accomplishable with the PEP 675. I will open a separate ticket on that.

My ultimate goal here is to store a dict where some type hints are know for certain keys and for all unspecified keys, there is a different type.

There has been a recent discussion in the Python typing community about extending TypedDict to support this notion. (TypeScript supports a similar capability.) Nothing concrete has happened yet on this front, but you are not alone in wanting this functionality.

No need for another ticket.
Wow, actually vscode supports this today:
    @typing.overload
    def __getitem__(self, name: typing.Literal["some_str"]) -> str: ...

    @typing.overload
    def __getitem__(self, name: typing.Literal["some_bool"]) -> bool: ...

    @typing.overload
    def __getitem__(self, name: str) -> int: ...

...
# results in
some_val = aa["some_str"]  # str
some_val = aa["some_bool"]  # bool
some_val = aa["hi there"]  # int


stats: Change time ranges for bar graph and pie chart.

Change templates/analytics/stats.html to use 'Last
Week', 'Last Month', 'Last Year' time ranges instead
of 'Last 10 days', 'Last 30 days'.
Change static/styles/stats.css to not set background
color for default time option, for messages sent by
client and message by recipient type.
Change static/js/stats/stats.js to show only available
time range options, and set background color for the
default. The default is Last Month if it exists, and
otherwise All Time.

Fixes: #3856

Automated message from Dropbox CLA bot
@PhoeniXkrypT, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

Automated message from Dropbox CLA bot
@PhoeniXkrypT, thanks for signing the CLA!

This looks great, thanks @PhoeniXkrypT! @timabbott ready for review/merge.

@PhoeniXkrypT the code looks good, merged!
Just a quick note -- you had a few independent changes in this branch, which ideally would have been in separate commits.  Ideally, next time this would be a separate commit for each of the bullets in this commit (all in a single PR).
Check out the Zulip commit guidelines for more details: https://zulip.readthedocs.io/en/latest/version-control.html.
Thanks @PhoeniXkrypT!!  If you want to do #3630 next, you're well prepared for it :).  There's also a lot more quick UI things that we can do to make that page nicer, especially on the second two graphs (e.g. making them align vertically unconditionally, placing the buttons nicely, styling the buttons to be prettier, etc.).

Actually, about to close Issue 3630; but here are a few other things on that page that would be great to work on:

Hovering over the green bars in "Messages Sent by Client" should show the count in the lower left corner, similar to what hovering over the bars in "Messages Sent Over Time" does. E.g. "Website: 3456" when hovering over the Website bar.
Changing "Humans" and "Bots" on the "Messages Sent Over Time" graph to be checkboxes instead of colored squares. (Ping me on chat.zulip.org before starting this if you take it up.)
#3865

@timabbott @rishig Thanks! It would be nice to work on javascript code however it would be more interesting for me if it involves backend also (Python/django).

@rishig do you think the count of sent messages in the last week for the subscriptions page would be a good project that involves both and the analytics codebase?  We've been wanting to do that for a while...

yeah, I think that would be great. Opened #3989!


multimodal conversation support

Is your feature request related to a problem? Please describe.
A real assistant would not only converse by text but can speak and use video / images.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Support text, images, audio and vidoe.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Use OpenAI's chatgpt.
Additional context
Add any other context or screenshots about the feature request here
I know that multimodal ais are still a challenge for FOSS tooling.

See https://huggingface.co/vonjack/Hermes-2-Pro-BakLLaVA-Mistral-7B

Did you check https://github.com/Chainlit/cookbook/tree/main/audio-assistant ?

Hello, check the Multi-modality on the documentation to include sound and files


Non wear storage in part 4 reports (new branch)

I'm setting up this new branch to investigate why the checks are not passing in PR #681

Close this PR since I have found the problem with the checks and it is now fixed in the PR #681


[NOMERGE] vNext

Faster type inference (@schani can you quantify or add any details?)
Allow recursive maps in JSON Schema
Separate type inference and code rendering, allowing caching
TypeScript enums
GraphQL
Simpler main module

I think we should distinguish between stuff that affects users and stuff that's just infrastructure when we write release notes/changelogs.
Stuff that affects users:

Faster type inference - what it says on the box.  I'll blog about it at some point, but that's not interesting to users.
TypeScript enums
GraphQL

Stuff that doesn't affect users:

Recursive maps in JSON Schema.  Doesn't affect users because no target language supports it yet.  See #302
Caching.  Doesn't affect users yet because parsing is slow part, and we don't cache that.
Simpler main module


Update to new dagger dependencies.

Signed-off-by: Erik Sipsma<EMAIL_ADDRESS>Haven't tested this yet as I don't have terraform creds or anything set up, do you have a way of testing this quickly @kpenfound ?

@sipsma yeah I can test this with a demo repo that uses it. Will follow up


Workbench reloading intermittently fails on Windows

Testing #120431
Repro steps
On Windows:

Enable process reuse
Open Devtools in OSS
Start reloading the workbench multiple times
🐛 Sometimes, only the script files seem to load

Example

To figure out:

Does this issue still occur when process reuse is disabled?

Edit: It seems this issue goes away when process reuse is disabled.

Even weirder: sometimes the css files load, but not the js ones.
This still results in a blank workbench.
@alexdima I tried looking through some code in https://github.com/microsoft/vscode-loader. I'm wondering whether you have any idea what could be happening here? Is there some callback that's not being called?

I'm sorry, I have no idea what could be wrong. 🤷

@deepak1556 the part where the file is read still ends up hitting this line, which changes the format of the path URL: https://github.com/nodejs/node/blob/09c97b813890f6f1465ebd149831bed07682d496/lib/path.js#L647
Context
I noticed that on Windows, with process reuse enabled and while debugging, that the following callback body is sometimes not being run during a reload:
https://github.com/microsoft/vscode-loader/blob/main/src/loader.js#L832
However, the line before that is run, and that line calls into https://github.com/microsoft/vscode-loader/blob/cbe8f639e7e2ae8a0ede5da411555815158bd13e/src/core/scriptLoader.ts#L538, which reads the file.
In more detail:

The NodeScriptLoader calls _readSourceAndCachedData
_readSourceAndCachedData calls into fs-wrapper https://github.com/electron/electron/blob/master/lib/asar/fs-wrapper.ts#L496
That line calls https://github.com/nodejs/node/blob/master/lib/fs.js#L329, passing the arguments, including the callback, into the fs call
The callback isn't run

CC @jrieken who wrote _readSourceAndCachedData. I've also looked at _readSourceAndCachedData and the function appears to me to be fine.

Given its age and importance I'd be surprised to find major issues with _readSourceAndCachedData. Tho, never say never. It this happen with rapid reloading only? Is the callback function still from a previous (before reload) state?
Also, one thing to consider is that all of it will be away when switching to node-free renderers. Tho, also the performance it guarantess 😢

I haven't tried to repro this yet, additional to above when this issue happens can you run some generic fs.readFile calls and see if they succeed. If they don't, then the problem is deep in the event loop layer and not related to the vscode-loader

@bpasero can you try the async version of fs apis.

@deepak1556 oh yeah, the async variant never returns!

Thanks Ben, that indicates a problem in the event loop after reload. Will look into it.

Need to verify whether this still applies now that ext host etc. moved to shared process.

Need to verify whether this still applies now that ext host etc. moved to shared process.

It does not apply!


surface range camera support

It seems like Microsoft Surface Camera Support is not working. Surface Pro 5 (2017)
Linux Surface Camera Support
I don't know how to apply those for NixOS
Installed libcamera from nix packages, did not make any difference.
If anyone knows a workaround please let me know.

Duplicate of https://github.com/NixOS/nixos-hardware/issues/523#issuecomment-1404714880 I think


ndt7: refactor code to make adding upload easy

This commit completes the yak shaving required to restructure
the code such that adding support for upload is easy.
We create a new ndt7/handler package where we do most of the
common work required by both download and upload.
This common code has been moved out of downlad, which now
looks very simple and understandable (IMHO).
27b2504

This change is

Pull Request Test Coverage Report for Build 541

20 of 36 (55.56%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 54.512%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ndt7/download/download.go
0
2
0.0%

ndt7/handler/handler.go
16
30
53.33%

Files with Coverage Reduction
New Missed Lines
%

ndt7/download/download.go
1
0.0%

Totals

Change from base Build 538:
-0.03%

Covered Lines:
737

Relevant Lines:
1352

💛  - Coveralls


ECMPS 1.0 2024 Q1 Check Changes for ECMPS 2.0

MATS 2.0 Check Changes to Migrate:

COMPON-14: Addition of Component Type Code "PM" Basis Code check.
FORMULA-13: Addition of checking where Formula Parameter And Component Type And Basis To Formula Code crosscheck where Component Type And Basis equals "PMD" or "PMW".
METHOD-9: Addition of "PMRE" and "PMRH" to null Substitute Data Code required check.
MONPLAN-11: Addition of "PMRE" and "PMRH" to Parameter Code filter list.
SYSTEM-12: Addition to prevent checking of inactive "PM" systems for required methods.

GNP Check Changes:

HOUROP-7: Addition of "CALC" to Active NOx Mass Method Code list.

Need to add a link to the corresponding jira tickets for acceptance testing.


[rust-1.16.0] Update OPENSSL_CC_powerpc64-unknown-linux-gnu

With the new rust-slave-linux-cross:2017-02-02, powerpc64 no longer
uses gcc-5.  I updated the Dockerfile at the time to reflect this, but
missed OPENSSL_CC_powerpc64-unknown-linux-gnu in Makefile.in.

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

 Warning

Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.16.0. Please double check that you specified the right target!

Same as #3648, just on the beta branch.

:pushpin: Commit 0b89f8f has been approved by alexcrichton

@bors: r+

:hourglass: Testing commit 0b89f8f with merge 5f5cdd9...

:broken_heart: Test failed - status-travis

https://github.com/rust-lang/cargo/pull/3652 should fix those errors, I'll r+ after it lands.

Thanks!

@bors: retry

:hourglass: Testing commit 0b89f8f with merge ec1e036...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing ec1e036140f771ec8f62579172f77ebe4738f331 to rust-1.16.0...


Add workflow or tool language to search

Feature Request
Desired behaviour
A simple oversight, we should add a facet for searching on tools or workflows by a particular language (WDL or CWL currently)
Encountered by @briandoconnor

Kind of related issue, clicking on labels for workflows and tools should bring up a search for that label

Working on staging.dockstore.org


Add required field to fixture

Missed this earlier because functional tests are flaky on Travis.
Sorry!

There are still some failing functional tests, but will raise another PR to fix those.


Meteor <IP_ADDRESS> TypeError: Property 'extend' of object #

Hi,
Im trying to get tap:i18n on a simple project but I keep getting this:
W20150507-20:58:14.614(-3)? (STDERR) ~/.meteor/packages/meteor-tool/.1.1.3.j3dwdo++os.linux.x86_64+web.browser+web.cordova/mt-os.linux.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
W20150507-20:58:14.614(-3)? (STDERR)                                            throw(ex);
W20150507-20:58:14.614(-3)? (STDERR)                                                  ^
W20150507-20:58:14.653(-3)? (STDERR) TypeError: Property 'extend' of object #<Object> is not a function
W20150507-20:58:14.653(-3)? (STDERR)     at Object.init (packages/tap:i18n/lib/tap_i18next/tap_i18next-1.7.3.js:611:1)
W20150507-20:58:14.653(-3)? (STDERR)     at __coffeescriptShare (packages/tap:i18n/lib/tap_i18next/tap_i18next_init.js:1:1)
W20150507-20:58:14.653(-3)? (STDERR)     at ~/Dropbox/Webs/project/.meteor/local/build/programs/server/packages/tap_i18n.js:2817:4
W20150507-20:58:14.653(-3)? (STDERR)     at packages/tap:i18n/lib/tap_i18n/tap_i18n-server.coffee:1:1
W20150507-20:58:14.654(-3)? (STDERR)     at ~/Dropbox/Webs/project/.meteor/local/build/programs/server/boot.js:222:10
W20150507-20:58:14.654(-3)? (STDERR)     at Array.forEach (native)
W20150507-20:58:14.654(-3)? (STDERR)     at Function._.each._.forEach (~/.meteor/packages/meteor-tool/.1.1.3.j3dwdo++os.linux.x86_64+web.browser+web.cordova/mt-os.linux.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
W20150507-20:58:14.654(-3)? (STDERR)     at ~/Dropbox/Webs/project/.meteor/local/build/programs/server/boot.js:117:5

Couldnt find anything related anywhere, so I'm asking that here.
Thanks for your time :+1:

Replicate on an example repo and send to me.
Thanks,
-Daniel Meteor Developer

Seems that I could solve the issue after all, I did a new meteor app, added all packages, wasnt able to replicate the issue, I added even android platform to it, nothing, copied all my code over, nothing, I just replaced the packages file of my app for the one of the new app (same packages, diff order), and the error went away... but I couldnt replicate the error on a brand new app...


it doesn't work in IDEA

it disable when i tapping in IDEA.Looking forward to your reply.thanks.

yes, i code in viscode it do not work

start it with administrator


NetTruyen URL changes

Source information
NetTruyen 1.4.27
Source language
Vietnamese
Source new URL
https://www.nettruyenbb.com/
Other details
https://nettruyenco.vn/ is a fake site
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[ ] I have admitted that I am a clown by having checked this box, as I have not read these acknowledgements.
[X] I will fill out all of the requested information in this form.

It's true, the old Nettruyen source has been changed "https://www.nettruyenbb.com/".


Standard Shader error on Android

Describe the bug
I have a project running with the MRTK and for some reason, whenever I try to build for Android, I receive this error message:
Shader error in 'Mixed Reality Toolkit/Standard': Non system-generated input signature parameter () cannot appear after a system generated value. at line 775 (on gles3)
To reproduce
Steps to reproduce the behavior:

Create a project (MRTK Guide)
Build Android

Screenshots

Your setup (please complete the following information)

Unity Version [2021.2.14f1c1]
MRTK Version [<IP_ADDRESS>]

Target platform (please complete the following information)

Oculus Quest2

Same error here, I cannot use any material using this shader.

I solved my problem with the upgrade process to URP

Closing as duplicate of #10449

I solved my problem with the upgrade process to URP

Thank you for this!

I am actually having this issue, and the upgrade option is currently greyed out..... could something else be causing this issue?

Did you manage to find the solution?

@interactive3d You'll have to follow the instructions in<EMAIL_ADDRESS>to install and setup URP for this menu item to be enabled

I tried by adding URP but it still doesn't work. This issue seem to have started when upgrading unity to 2021. It works fine on 2019 LTS.

This still does not work is there an actual real fix for this!

This still does not work is there an actual real fix for this!

Here is the fix tht works for me in ehs hader code change:
fixed facing : VFACE" to "bool facing : SV_IsFrontFace" at line 775

facing to "* (facing ? 1.0 : -1.0)" at lines 956 & 959

This still does not work is there an actual real fix for this!

Here is the fix tht works for me in ehs hader code change:
fixed facing : VFACE" to "bool facing : SV_IsFrontFace" at line 775

facing to "* (facing ? 1.0 : -1.0)" at lines 956 & 959

You legend! This saved me hours of troubleshooting.

possible fix :             fixed4 frag(v2f i, fixed facing : VFACE) : SV_Target
//      fixed4 frag(v2f i, bool facing : SV_IsFrontFace : SV_Target


Discrimination filter

@tongshuangwu, I'm creating a pull request on the main branch, as you requested. No conflicts detected.

Earlier PR: https://github.com/GEM-benchmark/NL-Augmenter/pull/189

@sebastianGehrmann reminder.

Yes, I agree, "group inequity" is a better title for this PR - I'll change it.

The filter has been renamed.

@asnota maybe you forgot to push the rename change?

Thanks @tongshuangwu and @sebastianGehrmann. I've merged https://github.com/GEM-benchmark/NL-Augmenter/pull/280

@kaustubhdhole I guess you've found out already that the changes related to renaming were pulled to #280. Thank you for merging!


fix: Nested Sortable issues

This PR fixes

Issues with nested Sortables, particularly incorrect value of oldIndex, and console errors while dragging items around.

This PR closes the following issues

[BUG] Incorrect oldIndex when there are nested draggable containers #427

Does this PR require the Docs to be updated?
No
Does this PR require new tests?
Not sure
This branch been tested on
Browsers:

[ ] Chrome 84
[ ] IE / Edge 84

Hi @davidaik , Everything ok now 🎉 .
But we need a minor change, based on (CONTRIBUTING.md)[https://github.com/Shopify/draggable/blob/master/CONTRIBUTING.md#opening-pull-requests], please use rebase instead of merge when updating the latest code. It will make the codebase more cleaner.
So sorry for bothering you, but may you use an interactive rebase to remove the merge commit above and then force push to your branch again?
Thank you so much

@bahung1221 Done. Hope it's all good now.

Hi @davidaik , I rebased your branch to sync up with the latest code of the master branch. Thank you so much.

@tsov , this PR can be merge, please help to merge it when you have rest time 🎉
This PR will add support for Nested Sortable, that will resolve #427 , #129 😄


Updating copy per blog post feedback

@mlettini Updated the copy on our notable page per the blog post feedback. Let me know if this looks good. Thanks!

two comments

Updated! How about now @mlettini?

+1


Update SLURM client and job scheduler to v0.0.40 of REST api

I don't know enough to review slurm_rest.py properly but the rest looks good to me!

slurm_rest.py is generated as described in ParProcCo/slurm/README.md then fixed manually

I think this is now ready for review and merging before a release


creepsFromRoomAmount stat

Describe what you want the bot to do
A commune-only room stat for creepsFromRoomAmount, to track how many creeps each commune is spawning and utilizing

Change will be removing remote creep count and creating an new stat called total creep count which uses creepsFromRoomAmount

https://github.com/The-International-Screeps-Bot/The-International-Open-Source/commit/9a76eeadd671f5da42c7c33bc1d07fb9f6114d72


How to prevent abi_json_to_bin from Man-In-The-Middle attack ?

When sending asset, we have to call the nodeos api abi_json_to_bin to get the hex to sign. Thus, leave the opportunity for Man-In-The-Middle attacks in which one can just return a maliciously crafted hex. How to avoid this from happening? 4 approaches came to my mind :

abi_json_to_bin implementation in local code (most preferred, but I cannot find any)
use https calls to nodeos and check its certificate ( tedious )
call this api to a local running nodeos, no need for syncing.( is it possible?)
there is way to decode the bin and check the params before signing. ( I didn't find any )

Any recommendation ?

https://github.com/EOSIO/eosjs or https://github.com/EOSIO/abieos

Option 3. is also possible if you create an account matching your account name on the network you wish to submit your transaction to and load your contract and its ABI into the account.  It would be your responsibility to update your local node's copy of your ABI any time you change it on the network you are using.


chore(main): release 1.0.3

:robot: I have created a release beep boop
1.0.3 (2022-10-03)
Bug Fixes

neo4j env, cli (b3c125f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/socialgene/sgpy/releases/tag/v1.0.3 :sunflower:


e2e test for istioctl rule CRUD, mixer rule CRUD, version, etc

For #215 .  e2e test for istioctl rule CRUD, mixer rule CRUD, version, invalid rule reporting, bash completion.
This tests istioctl create and istioctl mixer rule create with invalid rules to verify  error reporting.
This tests CRUD to create a manager rule, verify it us created, update it, and delete it via istioctl.
This tests CRUD to create a mixer rules and verify it us created.
This tests istioctl version and istioctl completion to verify they don't crash

This change is

Jenkins job istio/presubmit passed

Jenkins job istio/presubmit passed

jenkins rebuild istio/presubmit

Jenkins job istio/presubmit passed

Jenkins job istio/presubmit passed

Closing this PR because our Jenkins setup provides no way to deal with changes in error messages because those changes span repos and we can't test against branches of other repos.

I don t think it is related to our Jenkins setup. Jenkins is just a job scheduler. I think you should do error message checking in manager repo, but still have some basic coverage in here. Like, create a rule, make sure it is listed, expected failure, etc...

@sebastienvas I discovered a bug in istioctl last week.  It did not print the failure message if an invalid mixer rule was created.  It did return a non-0 exit status
I wrote a test for it that tried to match a little piece of the error message to verify that it was printed to stdout.  @ayj pointed out, correctly, that if the error message verbiage was changed in the istio/mixer repo it would cause builds of the istio/manager to break.
On a previous project we used Jenkins "promotions" for this.  Each project would test it's own stuff, and then to promote an image we would have to test a group of things together.  I was just trying to say that I don't know how to solve this problem without adding a lot of complexity to how we test.  I don't feel that complexity is helpful during the alpha release.

Jenkins job istio/presubmit passed

Jenkins job istio/presubmit passed

Jenkins job istio/presubmit passed

Jenkins job istio/presubmit passed

Jenkins job istio/presubmit passed

Jenkins job istio/presubmit passed

I think what you talk about would actually be captured when we run the e2e suite on Manager postsubmit. (today It will actually by discovered when you do a PR in manager master as we run the same tests for pr and postsubmit). So that means that this PR (the one changing istioctl messaging) would require a change istio/istio in order to get commited to manager/master.
I understand that error message checking should be done on Manager directly though.


[mtl] override argument buffer sets

Makes argument buffers compatible with push constants again. Also reduce the binding of argument buffers to only the affected stages.
PR checklist:

[ ] make succeeds (on *nix)
[ ] make reftests succeeds
[ ] tested examples with the following backends:
[ ] rustfmt run on changed code

I think this brings us to the end of the second push for argument buffers. They seem pretty solid and operational as of today, vkQuake is a good test case. There are serious limitations though that ask for big changes:

Storage images and dynamic-offset buffers can't be a part of the argument buffer - #2866 . This requires shader translation changes.
We need to know the dimensionality and msaa-ness of textures at the descriptor set layout creation time. This would mean another derivation from Vulkan proper.

Once we have these implemented, we'll be able to benchmark Dota and Dolphin again. But until then, argument buffer support will be kept as an experimental feature disabled by default.

bors r=grovesNL

bors retry

bors r=grovesNL


Corrected grammatical error in README

What type of PR is this? (check all applicable)

[ ] 🚀 Added Details
[ ] 🌟 stared the repo
[x] 🐛 Grammatical Error
[x] 📝 Documentation Update
[ ] 🚩 Other

Description
Add Link of GitHub Profile

@keshavsingh4522  kindly review my PR #529


Comparison against TF-IDF Vectorizer (from scratch)

This PR attempts to fix issue #8

@2bit-hack please could you make the PR again ? Actually the PR Template hadn't come in. It is now active. Please make the PR again so that it follows the PR template.
Thanks


⚠ Jenkins Plugins Artifactory has degraded performance

In 60b1825, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1343 ms

Resolved: Jenkins Plugins Artifactory performance has improved in e72e325 after 12 minutes.


Update to Jekyll 4.3.1

https://github.com/jekyll/jekyll/releases/tag/v4.3.0
https://github.com/jekyll/jekyll/releases/tag/v4.3.1

Jekyll 4.3.0 added webrick as a dependency, so this update should also fix #335.

Have you been able to test this? I created an issue because I don't think you can use an Alpine base image for 4.3
https://github.com/envygeeks/jekyll-docker/issues/353

I haven't tested this, because I can't run the pipeline.

Hi 👋
While waiting for @envygeeks to update this repo, I've pushed to Docker Hub an updated version for jekyll/builder image:
https://hub.docker.com/r/jvconseil/jekyll-docker
See also PR https://github.com/envygeeks/jekyll-docker/pull/369
Best,
@JV-conseil


Negative battle bug?

On the join room page, it can sometimes display that there are negative battles in progress.
Checking the global room battleCount and it is also negative, so it's server side and not client side.
Further investigation of this bug to come later, hopefully.

Fixed