use UTC based timestamps

We use in mscolab datetime.datetime.now()
This can have a timezone issue, we should replace it by a timezone based on UTC
datetime.datetime.now(tz=datetime.timezone.utc)

I would like to work on this issue, can you please assign this to me?

sorry for the delay, you have it .

Is this issue open?

@Preetam-Das26 yes, the PR was not finished.

I would like to work on this issue, can you please assign it to me ??

@ReimarBauer Ok can you please assign this issue to me?

@ReimarBauer please assing this issue to me.

@ReimarBauer   I am getting this error during tests:
during,
invoke dockertesting image creation:
🏃 Workflow Dispatch Action v1.2.1
Error: Parameter token or opts.auth is required
Can you please help me resolve it?

@Preetam-Das26 I assume you have not sent a  pull request to this repository? What exactly did you do to get this?

There are more instances of datetime.now() remaining:
$ git grep -n '\.now()'
docs/samples/automation/retriever.py:191:        path = [(wp[0], wp[1], datetime.datetime.now()) for wp in wps]
mslib/mswms/mss_plot_driver.py:519:        d1 = datetime.now()
mslib/mswms/mss_plot_driver.py:527:        d2 = datetime.now()
mslib/mswms/mss_plot_driver.py:558:        d3 = datetime.now()
mslib/mswms/mss_plot_driver.py:646:        d1 = datetime.now()
mslib/mswms/mss_plot_driver.py:654:        d2 = datetime.now()
mslib/mswms/mss_plot_driver.py:684:        d3 = datetime.now()
mslib/mswms/mss_plot_driver.py:851:        d1 = datetime.now()
mslib/mswms/mss_plot_driver.py:858:        d2 = datetime.now()
mslib/mswms/mss_plot_driver.py:870:        d3 = datetime.now()
mslib/utils/mssautoplot.py:69:    now = datetime.now()
mslib/utils/mssautoplot.py:107:        self.path = [(wp[0], wp[1], datetime.now()) for wp in self.wps]
tutorials/tutorial_waypoints.py:125:    current_time = datetime.datetime.now().strftime('%d-%m-%Y %H-%M-%S')
tutorials/utils/screenrecorder.py:66:        current_time = datetime.datetime.now().strftime('%d-%m-%Y_%H-%M-%S')
tutorials/utils/screenrecorder.py:107:        ini_time_for_now = datetime.datetime.now()
tutorials/utils/screenrecorder.py:159:                if datetime.datetime.now() >= future_date_after_120seconds:

We have to be carefully not to create a dependency to mscolab in the mswms code parts.


Configure Flutter CI

Cirrus CI recently added Flutter support by providing Docker images with Flutter pre-installed. It allows to avoid boilerplate of setting up an environment which makes it possible to start tests execution in a few hundred milliseconds from the moment a change is pushed to GitHub.
For this repository particularly I've seen CI times from 12 to 15 minutes for the longest part of the build (building apks). It's 2 to 3 times faster comparing to the current CI setup.
Unfortunately flutter_plugin_tools can't run things in parallel and take advantage of up to 8 cores that Cirrus CI can provide.
Here is a build for that particular change:

I was testing Flutter support on this repository and decided to share the results. Feel free to close the PR if not interested in a better CI. Didn't want to waste my work 😅
In case you will merge the PR. Don't forget to install Cirrus CI application from Github Marketplace as described in Quick Start guide.

This is interesting. What version of Flutter does it test against?
cc @mravn-google

@Hixie it uses cirrusci/flutter:0.1.0 Docker container image which has Flutter 0.1.0 pre-installed. There is 0.1.1 version available so I can bump it if you want.
BTW cirrusci/flutter:0.1.0 is based on cirrusci/android-sdk:26 image which has everything for Android development so it is possible to run an Android emulator using cirrusci/flutter images for integration tests.

cc @sethladd and @yjbanov since we have an ongoing email thread about Cirrus CI for Flutter projects.

We ship new versions of Flutter multiple times a week. I am worried about using a mechanism that is tied to a particular version. We care about testing against the latest version.

Fair point. Added flutter channel master && flutter upgrade which adds a little bit less than a minute to a build. Also removed pub cache to always test against the latest versions of libraries. See https://cirrus-ci.com/build/5708110253522944

I'll be able to give this more thought after Feb 28. But I'm curious how well-trodden the Emulator/Simulator path is on Cirrus CI. This repository contains lots of Objective-C and Java code that need to be tested in either a simulated environment or real devices. Travis does not make it easy to test in emulators.

These images can run an Android emulator. Unfortunately it won't have hardware acceleration since it's a Dockerized environment but, to my knowledge, no hosted CI has hardware accelerated emulators at the moment. Internally we have a design proposal on how to securely add such accelerated emulators but there are other priorities at the moment like adding Mac OS support.
Have a great vacation and let's revisit this PR once you are back 🙌

Ok I've landed this and turned on Cirrus CI! Let's see how it goes!

Thank you, @Hixie! Please let me know about any feedback and feel free to tag me on PRs for questions.


Error re-rendering Graph when no config prop is set

I was stuck with this error for a while, until I finally found out what was causing it. Details on the error and the current workaround are listed bellow, just in case it can be of use to someone else who's struggling with the same problem.
Problem description
By the time I got the error, my Graph component was looking like this:
<Graph
    id="network-graph"
    data={this.props.data}
    onClickNode={this.props.onNodeClick || this.onNodeClick}
    onClickLink={this.props.onEdgeClick || this.onEdgeClick} />

When I run the application for the first time, the graph would render perfectly on the screen. However, when I updated its data in my component state and the graph was to be rendered for a second time, I got the following error message:
Uncaught TypeError: Cannot convert undefined or null to object
    at Function.keys (<anonymous>)
    at Object.isDeepEqual (utils.js:52)
    at Graph.componentWillReceiveProps (Graph.js:390)
    at callComponentWillReceiveProps (react-dom.development.js:11527)
    at updateClassInstance (react-dom.development.js:11719)
    at updateClassComponent (react-dom.development.js:13153)
    at beginWork (react-dom.development.js:13824)
    at performUnitOfWork (react-dom.development.js:15863)
    at workLoop (react-dom.development.js:15902)
    at HTMLUnknownElement.callCallback (react-dom.development.js:100)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:138)
    at invokeGuardedCallback (react-dom.development.js:187)
    at replayUnitOfWork (react-dom.development.js:15310)
    at renderRoot (react-dom.development.js:15962)
    at performWorkOnRoot (react-dom.development.js:16560)
    at performWork (react-dom.development.js:16482)
    at performSyncWork (react-dom.development.js:16454)
    at interactiveUpdates$1 (react-dom.development.js:16719)
    at interactiveUpdates (react-dom.development.js:2150)
    at dispatchInteractiveEvent (react-dom.development.js:4532)

Workaround
After hours trying to figure it out, I finally discovered that adding the config prop to the Graph component prevented the error from happening.
<Graph
    id="network-graph"
    data={this.props.data}
    config={{}}
    onClickNode={this.props.onNodeClick || this.onNodeClick}
    onClickLink={this.props.onEdgeClick || this.onEdgeClick} />

Since I wasn't overriding anything in the default config, I just hadn't considered adding it before. In order to solve this issue, I was forced to pass down an empty object in the config prop.
After doing so, the graph would always successfully render whenever there was a change in the data.
Steps to reproduce the issue:
This issue can be easily reproduced in the sandbox example bundled with this project.

In the Sandbox.jsx file, remove / comment the config property in the constant graphProps defined in the render method (currently, line 295).

const graphProps = {
    id: 'graph',
    data,
    //config: this.state.config,
    onClickNode: this.onClickNode,
    onClickLink: this.onClickLink,
    onMouseOverNode: this.onMouseOverNode,
    onMouseOutNode: this.onMouseOutNode,
    onMouseOverLink: this.onMouseOverLink,
    onMouseOutLink: this.onMouseOutLink
};

Open the updated sandbox example in a browser.

Click any of the buttons to dynamically add or remove a node (the + and - buttons above of the graph). This action will trigger the error immediatly.

Additional notes
The project README file mentions that only id and data are required, and that both config and any of the callback methods are entirely optional. This is true for the first time the graph is rendered, but right now it seems that the config object must be present to update the view later on.
The solution for this issue would be to initialize the config value in the component state to an empty object, which will be used by default in case no config prop has been passed down to the Graph component.
The componentWillReceiveProps handler seems to be a good place to start. If you'd like, @danielcaldas, I can take a closer look at this later and submit a pull request.
For the time being, users must explicitly set a config prop even if they're not overriding any of the default settings.

Hey there @LonelyPrincess, nice report, you identified with good precision the issue. Regarding the solution for this, componentWillReceiveProps is the proper place to handle this we need to check if the client provided a config (in nextProps.config) and simply fallback to an empty object if that isn't the case.
Fell free to submit a PR for this, nice job!
Thanks!

No prob! I'm glad to be of assistance! 😊
I've just uploaded two commits addressing this same issue in my fork repo:

The first one is pretty much what you described. If the received props don't include config, we work with an empty object instead. This approach keeps most of the code exactly as you had it before.

The second approach is slightly different, but it should be more efficient since we will avoid some variables and operations which serve no purpose if the config property is not set and we want to stick to the defaults.

Could you tell me which of them you like better, @danielcaldas? I'll submit the PR for the solution you choose. 👍

I prefer the first one, it is cleaner than the second one, the performance gain with the second option is not substantial since we check for empty object with isObjectEmpty before proceeding with the more expensive operations.
Nice one :+1:

Fixed with #84


Godot 4 Builds via Github Actions

These changes managed to get me working Godot 4 + Ecmascript builds via Github Actions. I've only done light testing with the Windows editor build and my own project, so definitely needs more testing to make sure everything works as expected, and there's more cleanup work that should be done to the Github Actions, but I figured I'd make this PR in case it helps/inspires others to take it further. :)

Also just now realized there's a script that generates the Github Actions from the main repo 🤦 Probably would've saved a few commits if I just read the README fully 🙃 Will try that out to see if gives a cleaner result than the manual edits I did here. 🤞

Good work!
Can you git squash commits ?

@animanmaster Are you able to work on this in the next two weeks?

Thanks for your work. @Geequlim @animanmaster
As https://github.com/Geequlim/ECMAScript/pull/150 is merged, this can be closed.

Thanks!

Sorry for the delay, thanks for picking this up @fire.


test(parser): refactor appsync schema tests

Summary
As mentioned in #3305, rather than have lots of JSON events that we have to read from the filesystem, we should have one event that we extend/amend to suit each test case. This PR refactors the AppSync schema tests to do this.
Changes

Changed tests to read a single event from the file system
Added an omit function to packages/parser/tests/unit/schema/utils.ts that allow us to give a list of fields to be omitted from an object. This is useful for testing.invalid schemas so we can can test where properties that must be present are not.

Issue number:
n/a

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Hi @svozza, we'll try to review this by early next week - thank you for your patience!

No worries, I know Re:Invent has taken up most of the teams' capacity.


Light and Shadow Effect

Unity 2017.4.3f1, SabreCSG 1.7.2.
I created a new scene and created a cube with the UnityEdtior tool to give the SabreCSG default material;

After run I got this effect.

But I use the same scene and created a cube with SabreCSG, but I got this effect,

I am confused about this, because I didn't find the lighting settings in the two scenes and the material shader of the cube. However, the gap between light and shadow is so great, and scenes created using SabreCSG have not been able to restore Unity's default scene lighting effects even after removing SabreCSG-related objects. I hope to use SabreCSG to create objects but get the default lighting effects with UnityEditor, please help me.

This isn't necessarily SabreCSG exclusive, open up the "Window -> Lighting -> Settings" window and press the generate lighting button:

The problem is that it doesn't have any reflection probes so reflections don't work, so it just defaults to black and darkness.

Thank you, It works for me! But I have another question, I used the SabreCSG code to dynamically create some objects, and the reflections of those objects  doesn't work in runtime. Does the Unity engine have no way to perform the same light reflection calculation  as baking in runtime ?

Dynamic objects have their reflections calculated using Reflection Probes.
https://docs.unity3d.com/Manual/UsingReflectionProbes.html
https://docs.unity3d.com/Manual/class-ReflectionProbe.html
Objects also need to be tagged appropriately as static to render into the probes.
And their lighting often calculated with Light Probes.
https://docs.unity3d.com/Manual/LightProbes.html
https://unity3d.com/learn/tutorials/topics/graphics/probe-lighting?playlist=17102
Which bake lighting information into 3D space.
Realtime shadows can apply shadows on top of both of these.
The details of all the above can also depend on what render path or scriptable render pipeline you are using.

Can this issue be closed? 😀

Yes! 😀


Verify Weight units and Virtual size units

Verify the MultipartyTransactionParameters.SharedOverhead unit is weight and that we are not making operations using different units (apple + orange)

Closed by https://github.com/WalletWasabi/WalletWasabi/pull/13312


[WIP] Isomorphic walletconnect example

Example isomorphic app that works with this PR

[x] Store / create keys on server
[x] Login with wallet connect in browser
[ ] Sync session between server and browser
[ ] Use credential as authentication
[ ] Credential flows
[ ] Signing flow

This needs to be a separate repo to support one click deployments


Show Project Velocity in dashboard

Background
In ColoredCow some of the projects are charged hourly. So we have to capture the hours booked in the hour and based on it we generate the client invoice/bill for the period.
The velocity of the project is the ratio of hours booked in the project and the total expected hours in the project for a billing period. In an ideal scenario, it should be 1.
Task
We currently show the velocity of a project on the project listing page

As we also show the user project on the dashboard, so we want to show the project velocity on the project card shown in the dashboard

Action Items
Basics Concept - 1 pointer:

[ ] Read about Models in Laravel. Share your understanding of why we use them in the task comment in your own language
[ ] Read about Views in Laravel. Share your understanding of why we use them in the task comment in your own language
[ ] Read about Routes in Laravel. Share your understanding of why we use them in the task comment in your own language
[ ] Read about Controller in Laravel. Share your understanding of why we use them in the task comment in your own language
[ ] Read about Services files in Laravel. Share your understanding of why we use them in the task comment in your own language
[ ] Read how we can print a value in Laravel using dd()
[ ] Try printing hello world using the dd function

Velocity

[ ] Check how we can get the velocity of a project. See how we are getting it in the project listing page
[ ] Similarly, show the velocity on dashboard project card on the right hand side of the card

Good afternoon Sir
I'm working on the  task that you assign.
I will submit it as soon as possible..
On Tue, 21 Feb 2023, 12:16 pm Gaurav Gusain, @.***>
wrote:

Assigned #2823 https://github.com/ColoredCow/portal/issues/2823 to
@kajal7575 https://github.com/kajal7575.
—
Reply to this email directly, view it on GitHub
https://github.com/ColoredCow/portal/issues/2823#event-8567351027, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A43YSKF2QPIPQ5ZSOLT5UITWYRQFPANCNFSM6AAAAAAU5XK7WU
.
You are receiving this because you were assigned.Message ID:
@.***>

Breakdown-
Code review and documentation: 0.5-1 hour.
Understanding the functionality: 1-2 hours.
Making the required changes: 2-3 hours.
Testing, code, review, and documentation: 1 hour.
Expected Time =6-8 hours.

I had bathed my head but it didn't happen.
On Mon, 26 Jun 2023, 10:38 am Gaurav Gusain, @.***>
wrote:

Closed #2823 https://github.com/ColoredCow/portal/issues/2823 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/ColoredCow/portal/issues/2823#event-9633455194, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A43YSKCT74KNOBOBAAPTR53XNEKN5ANCNFSM6AAAAAAU5XK7WU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@kajal7575 , you can check this,  I have mentioned about the approach  PR-  https://github.com/ColoredCow/portal/pull/3130

Thanks, I'll check it out.
On Tue, 27 Jun 2023, 2:43 pm jaskaransingh304, @.***>
wrote:

@kajal7575 https://github.com/kajal7575 , you can check this, I have
mentioned about the approach PR- #3130
https://github.com/ColoredCow/portal/pull/3130
—
Reply to this email directly, view it on GitHub
https://github.com/ColoredCow/portal/issues/2823#issuecomment-1609107128,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A43YSKABVR23USW7HUE5SQDXNKP5DANCNFSM6AAAAAAU5XK7WU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Notary v2 / Notation test and experimental integration

Describe the feature
Notary v2 / Notation is available: https://github.com/notaryproject/notation/
Perform a first test and consider how an initial experimental support could look like...

Hi @xopham @pflaeging
I am a Notation project maintainer. Notation recently cut v1.0.0-RC.3 release and will have a stable v1.0.0 release in the mid of April. I would like to know if there is any interest or plan to prioritize this issue and create a guide for https://sse-secure-systems.github.io/connaisseur/v2.6.0/validators/notaryv2/?

@FeynmanZhou thanks for reaching out. We do believe that broad support of signature solutions is important for an admission that has to consume whatever is being used.
We plan to discuss notation integration this week and are generally interested in a timely implementation based on our availabilities. As SSE only supports Connaisseur as a contribution to the overall community without financial interest, we have to obviously balance work with our other responsibilites.

@FeynmanZhou we hope to integrate notaryv2 soon. Do you have any input on best ways to attempt integration? Python libraries? Minimal working examples?

Thank you. Notation has only Golang library so far. https://github.com/notaryproject/notation-go


Add spec for opensearch phone plugin.

Description
Describe what this change achieves.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

The approach in this PR is not ideal, waiting on https://github.com/opensearch-project/OpenSearch/pull/16219 and/or https://github.com/opensearch-project/opensearch-build/issues/5096.

I thought I was reviewing Ralph's PR, instead commented on my own 🤦


[IOTDB-2694] fix without null any bug in raw query with value filter

You can see details in JIRA-2694

Coverage decreased (-0.01%) to 67.787% when pulling 0f610b4bf6d08d6e70a7a483a55ee7af54194103 on zjx1230:fix_without_null_bug into c7887266826d8b2b20dae10b3e3cdf2d2e37a5f8 on apache:master.


Put patch notes at the top of release descriptions

Is your feature request related to a problem? Please describe.
It's unintuitive that patch notes are at the bottom of the release descriptions, as Home Assistant requires expanding it and scrolling down to see them (I genuinely believed that you were just re-using the same description for subsequent patch releases).
Describe the solution you'd like
Prepend the patch notes at the top of the release notes, possibly with a headline of "Changes in vX.X.1", that way the user can follow the trail of patches back to the major release, in case they skipped one or more versions.

I have been copying the release notes for patch releases and adding to the patch changes at the bottom. I'll start also copying the release specific patch changes to the top of the release for patch changes, and modify the last few releases to match.


Fix typo in README.md

Seems appropriate enough that my first contribution would be the "contributing" section of the readme...
Description
Reduse the numbre of typoes in the reamde file
How this patch was tested
The pull request template asks "You tested it, right?" and I can honestly say that no, I did not test this patch at all.
Documentation link and external references
"Please provide any info that may help us better understand your code."
I'm just going to leave this one as "N/A".

Sorry the PR is off sync as the README was updated before we take a look at it, so we integrate the type fix manually. Thanks for the notification.


Update katex-header.html

PR updates katex-header.html file to be the same across all repos. Some of them were broken and caused double header menu displayed on docs.rs. I update all repos regardless of whether the katex header was broken, to sync all repos.

I'll fix CI issues in another PR. They are caused by CI using a new version of Rust compiler


ENH: model pvsyst thin-film recombination for CdTe and a:Si in pvsystem.singlediode()

Problem
This request was initially made in #163 and is also related to #470. As of #504 bishop88 can accept the thin-film recombination parameters: d2mutau and vbi to output IV curve and its gradients, but there is no propagation of this ability to the proxies that use it such as pvsystem.singlediode()
Solution
Here is a proposed list of methods that should now be able to use these new terms:

singlediode_methods.bishop88_i_from_v()
singlediode_methods.bishop88_v_from_i()
singlediode_methods.bishop88_mpp()
singlediode_methods._lambertw_v_from_i()
singlediode_methods._lambertw_i_from_v()
singlediode_methods._lambertw()
pvsystem.PVSystem.singlediode()
pvsystem.PVSystem.i_from_v()
pvsystem.singlediode()
pvsystem.max_power_point()
pvsystem.v_from_i()
pvsystem.i_from_v()

Alternatives
Please suggest alternative solutions. There has been discussion of refactoring calcparams_pvsyst or even creating model specific singlediode_pvsyst methods. Is that desired?
Additional context
this is a follow on to #504

@adriesse didn't #762 and #763 already address this? I think this can be closed right?

[x] pvsystem.max_power_point
[x] singlediode.bishop88_i_from_v
[x] singlediode.bishop88_v_from_i
[x] singlediode.bishop88_mpp

I think the other functions in the list above have a Lambert-W which I don't think has a way to handle the thin-film recombination terms yet.
Can I close this issue?

I think this issue should stay open, or, be closed and replaced by several smaller issues that cover the scope:

finish implementation in pvsystem functions (see below)
implementation for PVSystem methods
implementation for ModelChain methods

Implementation in pvlib.singlediode is complete by #763. bishop88 functions accept the arguments. The lambertw functions don't use the arguments and shouldn't; the method can't solve the single diode equation with the recombination term.
For the first item above, pvlib.pvsystem.max_power_point accepts the arguments by #763. The following functions could (and should) accept the arguments:

pvsystem.i_from_v
pvsystem.v_from_i
pvsystem.single_diode

These functions have a method kwarg which defaults to lambertw, but could be set to newton or brentq and thus d2mutau could be used.


Performance of Property Getters and Setters

Performance of Property Getters and Setters: The custom getter and setter for each property might be less efficient than native property access. If performance is a concern, consider alternative methods to observe property changes, like using Proxies or more efficient change detection strategies.

To improve the performance of property getters and setters in your component, you can adopt an alternative change detection strategy. Using JavaScript Proxies, as mentioned previously, is a highly efficient way to observe property changes. Proxies allow you to intercept and customize operations performed on objects, such as getting, setting, and enumerating properties, without the overhead of manually defining getters and setters for each property.
Here's how you can implement a Proxy-based solution to replace the individual getters and setters:

Define a Proxy for Your Data Object: Create a Proxy to wrap your data object. This Proxy will intercept all get and set operations on the object.

Handle Changes in the Set Trap: In the set trap of the Proxy, you can implement the logic to detect changes to properties and take appropriate actions, like re-rendering the component or emitting events.

Integrate the Proxy with Your Component: Replace the manual getters and setters with the Proxy, and use it to handle all property interactions.

Here's an example implementation:
export function Component(options: any): any {
  return function <T extends ComponentClass>(OriginalClass: T) {
    return class extends HTMLElement {
      // ... existing code ...

      private _data: any = {};
      private _dataProxy;

      constructor(...args: any) {
        super();
        // ... existing constructor code ...

        // Initialize the proxy for data object
        this._dataProxy = new Proxy(this._data, {
          get: (target, property) => {
            // Optional: Add any additional logic for 'get' if needed
            return target[property];
          },
          set: (target, property, value) => {
            const oldValue = target[property];
            target[property] = value;
            this.onPropertyChanged(property, oldValue, value);
            return true; // Indicate success
          }
          // Optionally, you can add other traps like deleteProperty, etc.
        });

        this._data = this._dataProxy;
      }

      private onPropertyChanged(property, oldValue, newValue) {
        // Handle property changes
        if (oldValue !== newValue) {
          // Do something when the property changes
          // e.g., re-render the component or emit a custom event
          this.updateComponent();
        }
      }

      private updateComponent() {
        // Logic to update the component, if necessary
      }

      // ... rest of your code ...
    } as unknown as T;
  };
}

The _dataProxy is a Proxy that wraps the _data object.
The set trap handles changes to properties. When a property is set, it checks if the new value is different from the old one and calls onPropertyChanged.
The onPropertyChanged method contains the logic to handle property changes, such as updating the component.
This approach removes the need for individual getters and setters, thereby improving performance.

By using a Proxy, you centralize and simplify change detection, which can lead to better performance compared to manually defining getters and setters for each property. This method is also more scalable and maintainable, especially for components with a large number of properties.


Add option to use rustls 0.20.

This a minor update of #155 to use the latest version of rustls.
I think this is useful because it allows tiny-http to serve TLS on platforms that don't have OpenSSL, like OpenBSD.

This is excellent and something I very much want to support, apologies for leaving the PR to stagnate, it seems I wasn't getting email notifications for this repo.
Could you rebase onto master and fix the build failures then I'll re-review?

Hi there @bradfier and @3xmblzj5,
I've rebased @3xmblzj5`s branch (and updated the examples and cert/key) here: https://github.com/travispaul-forks/tiny-http/tree/rustls20
Would you like me to open a separate PR? I too would like this feature!

Hi there @bradfier and @3xmblzj5,
I've rebased @3xmblzj5`s branch (and updated the examples and cert/key) here: https://github.com/travispaul-forks/tiny-http/tree/rustls20
Would you like me to open a separate PR? I too would like this feature! I also have a fork of rouille where this appears to be working after a quick smoketest and some minor changes: travispaul-forks/rouille@4d264b9

Please do @travispaul! I'm currently casting an eye over your branch at rustls20 and I have a couple of comments already, but it would be great if you opened a PR here so we can discuss them conveniently.
When you've done so I can close this PR and redirect visitors to the new one.

@bradfier I've created tiny-http/tiny-http/pull/218 please add any comments or suggesions
Any obvious build/test failures I will work on Monday (along with comments if there are any by that time.)
Thanks and have a great weekend!


Update .gitignore

Fixes #29
After you merge this with main, I think its also good to execute git rm -r --cached build to prevent tracking build/ (I think it must be done on main branch and pushed to this repository)
source: https://stackoverflow.com/questions/1470572/ignoring-any-bin-directory-on-a-git-project
P.S. if upper git rm ... indicates that nothing found, use git rm -r --cached **/build (this is correctly works on my side)

I found out that a little guide, that i explained, also can be done on anyone side, lets I'm also done things that clear track build/ and info about build/ from main branch

@dbaroncelli Check updates, please!


Overriding commands is inconsistent and partially broken

Originally reported by: schlamar (Bitbucket: schlamar, GitHub: schlamar)

I want to override the install_scripts command to ship a customized
*.manifest on Windows because my script needs elevated rights.
I noticed various inconsistencies and issues.
Basic script cli.py:
# coding: utf-8

import click

@click.command()
def cli():
    click.echo('OK')

if __name__ == '__main__':
    cli()

Basic setup:
from setuptools import setup

setup(
    name='test',
    version='0.1',
    py_modules=['cli'],
    install_requires=[
        'Click',
    ],
    entry_points='''
        [console_scripts]
        test=cli:cli
    ''',
)

Issue 1: custom install does not process install_requires.
Setup:
from setuptools.command.install import install

class custom_install(install):
    def run(self):
        install.run(self)

setup(
    cmdclass={'install': custom_install},
    ...

Running this as python setup.py install does not install Click.
Issue 2: custom install_scripts is ignored
Setup:
from setuptools.command.install_scripts import install_scripts

class custom_install_scripts(install_scripts):

    def run(self):
        print 'run'

setup(
    cmdclass={'install_scripts': custom_install_scripts},

Running this does not print 'run' to console.
Additional observation: overriding install + install_scripts does work,
but suffers from the first issue (install_requires does not get processed).
Setup:
from setuptools.command.install_scripts import install_scripts
from setuptools.command.install import install

class custom_install(install):
    def run(self):
        install.run(self)

class custom_install_scripts(install_scripts):

    def run(self):
        print 'run'
        install_scripts.run(self)

setup(
    cmdclass={'install': custom_install,
              'install_scripts': custom_install_scripts},

Bitbucket: https://bitbucket.org/pypa/setuptools/issue/232

Thank you very much for pointing out at this problem!
I conducted some investigation, and my conclusions are as follow:

setuptools.commands.install have a _called_from_setup guard.
According to this guard, it will call do_egg_install (which delegates to easy_install) only if the run method is directly invoked from the setuptools.dist.run_commands functions.
When you use inheritance and call install.run(self) or super.run() you add one level of depth to your stack and this guard fails falling back to the distutils.commands.install behaviour (which is not capable of handling dependencies).

do_egg_install will not call install_scripts (instead it uses easy_install.install_egg_scripts), but distutils.commands.install will.
That is reason why creating custom_install_scripts alone doesn't do anything, but works when you also create custom_install.

Since the install command is deprecated, I would say that it no longer makes sense to allow people to customise it, and close this issue for now. If there is any other use case that cannot be accomplished without it, please feel free to ask us to re-open this issue or go ahead and create a new one.


Backspace in query box changes focus to table view instead of deleting text

Describe the bug
Backspace in query box changes focus to table view instead of deleting query text
To Reproduce
Steps to reproduce the behavior:
Write some text in query and then try to delete some text using backspace. 'Delete' works fine but 'Backspace' doesn't
Expected behavior
Text to be deleted
OS:

Redhat 8.9

I couldn't reproduce that behavior in Linux Lite 6.6, probably it's a OS related issue.

Using windows changes the focus too

I am same issue in:
dblab: 0.29
OS: Windows 10 22H2
Open SQLite DB

Same issue. Backspace never worked, did not test Delete everywhere, but failed every time I did.
Tested environment combinations:

Architecture/Distro: (rpi) aarch64 debian testing/unstable, x86_64-glibc void
Shell: Bash, Zsh (with and without completion plugin loaded)
Connection (I mean connect to remote server and run dblab natively, not connecting to remote database): tty, ssh, mosh
Terminal: qterminal
Kernel: 6.6, 6.12
dblab: 0.29.0
db: sqlite (flag, config file)

Have other distros at hand but did not continue as it's already disappointing enough, I don't think they would work either.


Updates specific to Buletin Peternakan

[ ] Create page
[ ] Green area (or whatever alternative is decided upon)
[ ] Add papers
[ ] Share on socmed

Article is under review


Add logger for missing keys

Add a logger for missing keys. The logger function can be overwritten by passing it as a prop to I18nProvider.
Close #237

@giovannigiordano I prereleased under 0.17.3-canary.5


Strawberry 0.121.0 breaks things

The new strawberry 0.121.0 update changes how default field resolvers work. The get_result function has been completely removed, and its' content moved to the from_resolver function (https://github.com/strawberry-graphql/strawberry/compare/0.120.0...0.121.0#diff-00af9d43c9b59404ba170e72e470939f2a6dd50e2eae24d0e24ef105431e5414L284).

https://github.com/strawberry-graphql/strawberry/pull/2038#issuecomment-1193388591


prime factorise a natural number into its prime factors.

add in maths folder

I would like to work on it. Can you assign it to me?

ok

If the issue is still open, can you assign it to me?

If the issue is still open, can you assign it to me?


Installation gotchas to be covered in the README

Here are a few things I stumbled over during the installation of Fractal at the FMI which we should cover in the updated READMEs.

The fractal user needs to have read & write access to the log & env folders. The users running jobs also need write access to the log folder & at least read access to the env folder.
Easiest way: Have the fractal-demos folder in a folder with broad permissions that is accessible to both fractal and the users, start the server from there with default log & env locations within that server folder

To go towards production, logs & envs will need to be in new places. Let's document how to set the necessary env variables (I think those are FRACTAL_ROOT and RUNNER_ROOT_DIR?)

Set the RUNNER_CONFIG correctly before running Fractal
For FMI, that means: export RUNNER_CONFIG='fmi'

For the future: How easy will it be for people that want to test Fractal to add RUNNER_CONFIG settings for their own cluster? Basically means modifying this part of the server, right? https://github.com/fractal-analytics-platform/fractal-server/blob/06676fe74bd514d3eaa58e3a84f8d7adae3f0195/fractal_server/app/runner/_parsl/_setup.py

Installing the envs does require significant space, ensure the space & tmp space is available
server + task env added up to 4.3 GB in this installation. But it also required > 1GB in tmp space for pypi. The default /tmp location on our VM had only 1GB of /tmp space. Thus, setting TMPDIR explicitly as the fractal user before starting the server was necessary. This can be done by:
export TMPDIR=/PATH/TO/SOMEWHERE/WITH/SOME/GIGS/OF/SPACE

We may want to set the HOME variable in the run_example.sh file explicitly
It defaults to $HOME. Setting it to an explicit path on the share (e.g. on tungsten at FMI) may be more stable. I think only the user needs to be able to write there, is that correct?

Actually, on 2: Would it make sense to be able to import future configs from a file? Then, a new place could "just" create the correct config file instead of having to change any Fractal server code

Actually, on 2: Would it make sense to be able to import future configs from a file? Then, a new place could "just" create the correct config file instead of having to change any Fractal server code

That would make sense.
Note that the new SLURM backend will be simpler to configure.
We may still want to have "executors", which will only be defined by the set of (partition name, cpu requirements, memory requirements) -- see https://github.com/fractal-analytics-platform/fractal-server/issues/278. For those executors, reading from a file is trivial.

Here are a few things I stumbled over during the installation of Fractal at the FMI which we should cover in the updated READMEs.
1. The fractal user needs to have read & write access to the log & env folders. The users running jobs also need write access to the log folder & at least read access to the env folder.
   Easiest way: Have the fractal-demos folder in a folder with broad permissions that is accessible to both `fractal` and the users, start the server from there with default log & env locations within that server folder

To go towards production, logs & envs will need to be in new places. Let's document how to set the necessary env variables (I think those are FRACTAL_ROOT and RUNNER_ROOT_DIR?)
2. Set the `RUNNER_CONFIG` correctly before running Fractal
   For FMI, that means: `export RUNNER_CONFIG='fmi'`

For the future: How easy will it be for people that want to test Fractal to add RUNNER_CONFIG settings for their own cluster? Basically means modifying this part of the server, right? https://github.com/fractal-analytics-platform/fractal-server/blob/06676fe74bd514d3eaa58e3a84f8d7adae3f0195/fractal_server/app/runner/_parsl/_setup.py
3. Installing the envs does require significant space, ensure the space & tmp space is available
   server + task env added up to 4.3 GB in this installation. But it also required > 1GB in tmp space for pypi. The default /tmp location on our VM had only 1GB of /tmp space. Thus, setting `TMPDIR` explicitly as the `fractal` user before starting the server was necessary. This can be done by:
   `export TMPDIR=/PATH/TO/SOMEWHERE/WITH/SOME/GIGS/OF/SPACE`

4. We may want to set the `HOME` variable in the run_example.sh file explicitly
   It defaults to $HOME. Setting it to an explicit path on the share (e.g. on tungsten at FMI) may be more stable. I think only the user needs to be able to write there, is that correct?

I think (to be tested) that this was only to have a place where the parsl bash scripts are written (by the user), and cd-ing into HOME is a simple way to reach a folder where the user can write. This will most likely go away with the new SLURM executor.

Note that the new SLURM backend will be simpler to configure.
We may still want to have "executors", which will only be defined by the set of (partition name, cpu requirements, memory requirements) -- see https://github.com/fractal-analytics-platform/fractal-server/issues/278. For those executors, reading from a file is trivial.

Very good point. If we're anyway opening a slurm job per task & parallelization object (e.g. for each well-image that we apply illum_corr to), then we can even consider making this more granular, i.e. exposing CPU & mem options directly. But a set of predetermined executors may be an easy start :)

I think these comments should all be covered now and the relevant environment variables are documented well now. Closing this issue, feel free to reopen.


GATT server as a central

It doesn't seem possible to expose GATT services when using noble (i.e as a BLE Central). That functionality seems to only exist in bleno (i.e as a BLE Peripheral).
That asymmetry isn't required by the BLE/GATT standard, and it is fairly common to find cases where a BLE Central needs to expose GATT services (ex: an iPhone exposes a GATT service for notifications). Would unifying noble and bleno make sense for a future release of the library?

I'm not sure why they are separate either.


Can not clear/ override the input value in v 1.0.3

There is currently no way to clear the input value before the email is transformed into the tile.
The value is set while someone writes inside an input before the email address is finished and converted to the tile.
At this moment, when I want to create for e.g an action button, that clears input from unfinished emails like this:
const input: HTMLInputElement | null = document.querySelector('.react-multi-email input')

input.value = '' // this will not work

This will not work, because the package overrides this process with the value. Please provide an option to clear the input from value.

same here. need provision to clear the input.

@ojaspatharkar-qure , @olgaplaga Ddi you find the solution?

@ojaspatharkar-qure , @olgaplaga Ddi you find the solution?

have you tried using onChangeInput props?

@ojaspatharkar-qure , @olgaplaga Ddi you find the solution?

have you tried using onChangeInput props?

onChangeInput gives what ever we type(letter by letter). Can we remove input by using this prop?

@LoveWithCoding hello,
Can you confirm if I understand this correctly?
Is this the issue where it doesn't work when I create a button externally and try to control the input value?

@LoveWithCoding hello,
Can you confirm if I understand this correctly?
Is this the issue where it doesn't work when I create a button externally and try to control the input value?

@LoveWithCoding hello,
Can you confirm if I understand this correctly?
Is this the issue where it doesn't work when I create a button externally and try to control the input value?

let say if i type correct email and press tab then it will be shown as chip(with some background color). but some times user might leave the filed with half typed value which is any way not a valid email. In such a cases i want to remove the half typed value on OnBlur event. I do not want fields contain unfinished emails.
Is there a way to achieve this?

@hovelopin
I just made a PR. PLZ review my code.
I thought a new PROP was needed to control INPUT values in the component. It can be used as in the following example.
const [input, setInput] = useState<string>('');

return (
        <ReactMultiEmail
          inputValue={input}
          onChangeInput={value => {
            console.log(value);
            setInput(value);
          }}
        />
);

@hovelopin
I just made a PR. PLZ review my code. I thought a new PROP was needed to control INPUT values in the component. It can be used as in the following example.
const [input, setInput] = useState<string>('');

return (
        <ReactMultiEmail
          inputValue={input}
          onChangeInput={value => {
            console.log(value);
            setInput(value);
          }}
          onBlur={() => {
            setInput('');
          }}
        />
);

Exactly. this what I am looking for.

Now, you can use 1.0.24
t can be used as in the following example.
const [input, setInput] = useState<string>('');

return (
        <ReactMultiEmail
          inputValue={input}
          onChangeInput={value => {
            console.log(value);
            setInput(value);
          }}
          onBlur={() => {
            setInput('');
          }}
        />
);

@thomasJang Thank you for this quick response. This solved the problem discussed here, but got new issue.
https://github.com/user-attachments/assets/1a710d08-adc0-46cb-aeb8-8f7d953a1d3c
when email is validated(When tapped on tab bar), email field renders redundant emails because of new prop inputValue. Is it possible to prevent rendering the inputValue once email is validated.

@Yandamuri
Could you share your code?

@thomasJang
const MultiEmail = () => {
    const [tempInput, setTempInput] = useState('');
    const [emails, setEmails] = useState([]);

    return (
        <ReactMultiEmail
            placeholder="Type here"
            emails={emails}
            inputValue={tempInput}
            onChange={(emails) => setEmails(emails)}
            onChangeInput={(value) => setTempInput(value)}
            onBlur={() => setTempInput('')}
            getLabel={(email, index, removeEmail) => {
                return (
                    <div data-tag key={index}>
                        {email}
                        <span data-tag-handle onClick={() => removeEmail(index)}>×</span>
                    </div>
                );
            }}
        />
    );
}

@Yandamuri
https://codesandbox.io/p/sandbox/react-multi-email-test-f79zn2
I don't think there's a problem when I check it on the link. Check it out.

@thomasJang
Still issue persists in the link you provided. I am not sure if understood the problem from my previous comment. Problem is redundant emails and it still exists.
[
https://github.com/user-attachments/assets/35d1467c-882c-4341-9a60-d076a634a891
](url)
OR Please correct me if i am missing something.

Are you using Windows?
And did you enter the tab key after entering the email?

@Yandamuri can you press space?

@Yandamuri
I've fixed the code. I'll let you know when it's distributed.
https://github.com/axisj/react-multi-email/pull/178

Are you using Windows? And did you enter the tab key after entering the email?

I am using MacBook and of course I pressed space bar(not the tab key).

Now, you can use 1.0.25 @Yandamuri

@thomasJang Yes, It's working as expected. Thank you so much for your effort and prompt response.
It would be great if the README.md and version change logs are updated regarding these new changes. Please be notified that package version is still showing as 1.0.17 only


Investigate if we should use mt19937_64 for 64 bit numbers

There exists a slower mersenne twister that computes 64 bits of random values instead of 32.
Maybe we should document this fact for users that are concerned with higher quality random numbers?

Seemingly related discussion: https://github.com/numpy/numpy/issues/3137


feat: support decode json-encoded base64

fuzz test
fuzz: elapsed: 15m42s, execs: 3718301 (5088/sec), new interesting: 9 (total: 59)
fuzz: elapsed: 15m45s, execs: 3732737 (4813/sec), new interesting: 9 (total: 59)
fuzz: elapsed: 15m48s, execs: 3748861 (5375/sec), new interesting: 9 (total: 59)
fuzz: elapsed: 15m51s, execs: 3762779 (4635/sec), new interesting: 9 (total: 59)
fuzz: elapsed: 15m54s, execs: 3777984 (5072/sec), new interesting: 9 (total: 59)
fuzz: elapsed: 15m57s, execs: 3793799 (5272/sec), new interesting: 9 (total: 59)
fuzz: elapsed: 16m0s, execs: 3807634 (4612/sec), new interesting: 9 (total: 59)
fuzz: elapsed: 16m3s, execs: 3822623 (4996/sec), new interesting: 9 (total: 59)
fuzz: elapsed: 16m6s, execs: 3837584 (4987/sec), new interesting: 9 (total: 59)
fuzz: elapsed: 16m9s, execs: 3851757 (4724/sec), new interesting: 9 (total: 59)
fuzz: elapsed: 16m12s, execs: 3866220 (4821/sec), new interesting: 9 (total: 59)
fuzz: elapsed: 16m15s, execs: 3879421 (4400/sec), new interesting: 9 (total: 59)

This fixes https://github.com/bytedance/sonic/issues/119 and https://github.com/bytedance/sonic/issues/327.


Remove "node" from "node ./bin/www"

./bin/www is already a node executable so it uses node automatically. https://github.com/expressjs/generator/issues/25 doesn't mention this.
Is there some edge case where node is needed?

Hi! Sorry if that issue is long. I mentioned why the node is necessary in this comment: https://github.com/expressjs/generator/issues/25#issuecomment-43613128
You can see that reflected in the CI build failing now that the node was removed as well


VMSS Automatic OS Image Upgrade

As per the documentation - the automatic OS upgrade will run in batches with no more than 20% of the instances so does this mean that the minimum number of instances in the vmss should be 5 for this feature to work?

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

ID: 51c75ce7-2144-d81a-693c-fb51cc00f3e5
Version Independent ID: cba654ea-ee84-fca4-a76d-717bfb9d9bf5
Content: Automatic OS image upgrades with Azure virtual machine scale sets - Azure Virtual Machine Scale Sets
Content Source: articles/virtual-machine-scale-sets/virtual-machine-scale-sets-automatic-upgrade.md
Service: virtual-machine-scale-sets
Sub-service: management
GitHub Login: @avirishuv
Microsoft Alias: avverma

@arseyam, Thank you for reaching out. We are looking into this and will get back to you soon.

If there are fewer than 5 instances, then the upgrade will be done one instance at a time. cc @mayanknayar

Thanks for the answer, appreciate if you made it clear in the documentation.

This is not clear in the documentation, can you at least update the FAQ ?

@avirishuv Can you please check and add your comments on this doc update request as applicable

@psmanhas the query is addressed in the documentation. Two points to note:
In the intro section we have called out "Automatic OS image upgrade can be enabled on a scale set of any size". In addition, the upgrade process section of the doc mentions "The upgrade orchestrator identifies the batch of VM instances to upgrade, with any one batch having a maximum of 20% of the total instance count, subject to a minimum batch size of one virtual machine".
This should hopefully address the query raised by @arseyam

@mayanknayar Thank you for the clarification.
@arseyam Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


Bug with cyrillic symbol

Type: Bug
Create and save PS Script with cyrillic letter "ф":
$a = "ф"
and Run script.
the script will exit with an error
the error can be different, depending on the location of this symbol in the code
VS Code version: Code 1.76.2 (ee2b180d582a7f601fa6ecfdad8d9fd269ab1884, 2023-03-14T17:55:54.936Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
AMD Ryzen 7 5800X3D 8-Core Processor            (16 x 3400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
63.91GB (53.36GB free)

Process Argv
--crash-reporter-id 15d3f56b-ea65-4db4-a3ef-682bc15a388d

Screen Reader
no

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

isort
ms-
2022.8.0

python
ms-
2023.4.1

vscode-pylance
ms-
2023.3.40

jupyter
ms-
2023.2.1200692131

jupyter-keymap
ms-
1.1.0

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

powershell
ms-
2023.3.3

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
nodejswelcome1:30587005
3biah626:30602489
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
2i9eh265:30646982
vsctsb:30677850
functionswalk:30687959
pythonms35:30701012

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.77.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

script file should be in UTF-8 with BOM encoding if your content has cyrillic chars.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.77.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

After updating to the latest version, the problem did not disappear.

Yes, but the problem is only with one character, the error does not occur with others

anyways, if your code has even just any one cyrillic char, the encoding must be UTF-8 with BOM to avoid runtime errors.

I agree, but now the problem is definitely not this, it's clearly a bug)

It does reproduce when saving as utf-8 but not when saving with BOM:

This seems a bug in PowerShell then.


FileChooser.accept not selecting the files

#5473 # Steps to reproduce
Tell us about your environment:

Puppeteer version: 2.1.1
Node.js version: 12.14.1

What steps will reproduce the problem?
    public static async uploadFile(page: puppeteer.Page, selector: string, path: string) {
        const [fileChooser] = await Promise.all([
            page.waitForFileChooser(),
            this.click(page, selector)
        ]);
        console.info(`Using the Path "${path}" on the current working directory "${process.cwd()}"`);
        await fileChooser.accept([path]);
    }

What is the expected result?

Select the file and triggering the file upload.

What happens instead?

Nothing.
Works properly on the version 2.0.0.

I got the same issue
Puppeteer version 2.1.1
NodeJS version 12.14.0

Same issue here with the method uploadFile Pippeteer v2.1.1
Version 2.0.0 works fine

same issue

same issue
NodeJS v12.16.1
puppeteer v2.1.1

The same problem here, the issue occurs in v2.1.0 and v2.1.1 as well.
NodeJS v13.6.0

same issue
NodeJS v13.12.0
OS : Debian Stretch
puppeteer v2.1.1
I'm using puppeteer inside of a docker container with a base of this image: node:13.12.0-stretch

This should be fixed as of 532ae573d29063ffd081ed9ee340d71b25320dec, so closing this 👍


numactl: 2.0.13 -> 2.0.14

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/numactl/numactl/releases.
meta.description for numactl is: "Library and tools for non-uniform memory access (NUMA) machines"
meta.homepage for numactl is: "https://github.com/numactl/numactl"
meta.changelog for numactl is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.0.14 with grep in /nix/store/3nijk82kyddk0929ndhrmis3vcakqy0q-numactl-2.0.14

directory tree listing: https://gist.github.com/1bdf9537999c5ae370260b370d4a0867

du listing: https://gist.github.com/1c423c65b2137d908bd44dfc0be613f4

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

4262 total rebuild path(s)

1717 package rebuild(s)

1637 x86_64-linux rebuild(s)
1402 i686-linux rebuild(s)
7 x86_64-darwin rebuild(s)
1216 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
DisnixWebService
MIDIVisualizer
abiword
acoustidFingerprinter
adapta-gtk-theme
adoptopenjdk-icedtea-web
aegisub
ahoviewer
airtame
akonadi
akregator
alchemy
alda
aldor
alephone
almanah
alpine-make-vm-image
amarok
amarok-kf5
ammonite
ammonite_2_12
ammonite_2_13
anki
ankisyncd
apache-directory-studio
apache-flex-sdk
apktool
apostrophe
aqemu
aravis
arc-theme
ardour
arduino
arduino-core
areca
arx-libertatis
asc-key-to-qr-code-gif
asciidoc-full
asciidoc-full-with-plugins
asciidoctorj
aspectj
astroid
astrolabe-generator
asymptote
atlassian-cli
attract-mode
audacious
audaciousQt5
audacity
audio-recorder

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3nijk82kyddk0929ndhrmis3vcakqy0q-numactl-2.0.14
ls -la /nix/store/3nijk82kyddk0929ndhrmis3vcakqy0q-numactl-2.0.14/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings

@GrahamcOfBorg build openmpi openmolcas

@GrahamcOfBorg build numatop slurm


Show songs in albums in correct order

Right now, when opening an album it shows the songs in alphabetical order instead of the track order.

that'd be awesome, big step toward becoming a full time player.

I think we can ultilize the "Track Number" metadata. You can get it with lofty by this key. The metadata also needs to be added and indexed in the database so we can sort it in album view.
I'll try to implement that when I have some time, but PR for this is welcome.

Support for album artist and track number has been added in this commit: https://github.com/tranxuanthang/lrcget/commit/71fa8c0aa84d8a71eeb823a8b06226eee06fb426. This means compilation albums with various artists are now grouped together nicely and displayed in the correct order, too!


Support building index with online mode

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: bright-starry-sky:x: darionyaphetYou have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: darionyaphet:x: bright-starry-skyYou have signed the CLA already but the status is still pending? Let us recheck it.

Generally LGTM


undefined symbols ALAssetsLibrary after update to Xcode8.3 from 8.2.1

Hi - had a project that would build last night - then I upgraded to XCode 8.3, now I get this error:

Any idea how to resolve this?

Resolved by adding AssetsLibrary in Link with Binaries

This link help me fix this error
What it does is include 3 more libraries.
https://stackoverflow.com/questions/23634940/opencv2-framework-not-compile-with-linker-flag-objc


pip install django-constance[database] shows no matches

Describe the problem
Trying to install database version of constance by following the docs
Steps to reproduce

pip install django-constance[database]
outputs no matches found: django-constance[database]

System configuration

Django version: 3.0
Python version: 3.6

You need to use quotes
pip install 'django-constance[database]'


Grammy Docs Bot should have an independent repository

Description
As any other plugin or bot, Grammy Docs Bot should be outside of website repository. This repository is only for the website that preserve the documentation.

That's true, we were just lazy. Moving the bot means having to change some config about the automatic deployment, but this is doable and I'll fix it when I have time :)

Check out https://github.com/grammyjs/docs-bot


Open source?

Can we edit the source code and publish the edited version again? By changing the name. We can credit. Is that possible?

Can you tell me what do you want to improve or change?

Not actually what I mean. I'm a mobile developer and want to change the software a lot from the base and republish it. I'm sure that you'll not want this for your own application. There are a lot changes to do. Please can I? I can credit the original program as I've said.

Not actually what I mean. I'm a mobile developer and want to change the software a lot from the base and republish it. I'm sure that you'll not want this for your own application. There are a lot changes to do. Please can I? I can credit the original program as I've said.

And you will use what of my app if you will recode it? The engine is the webview

No, I understood that part. Just the pre-designed UI. I just need to edit like a brand new application.

Yeah ok, and why taking my app if you want to redit it? Tell me which classes you want

Thanks for the permission. I'm actually a beginner I mean not too beginner but still started 1 year ago and I don't have much information so I needed a template and I was searching and found this. Also, your browser is a nice browser. Congratulations on the work.

I'll be also notifieing you from your email. What is your email address so we can get in touch easily?

My email is<EMAIL_ADDRESS>when you publish the app tell me and insert the name of creator

If you will steal my work prepare for a bad situation :)

But i know you will not so good luck :)

Thanks, check your email address that you've provided to me for further details.

Thanks, check your email address that you've provided to me for further details.


pull request for e2e test at 12/20/2016 13:15:20

:white_check_mark: Validation status: passed
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.


Putting chart version in metadata.labels in a Deployment v1beta1 without selector prevents upgrade

BUG REPORT
Any Chart containing Deployment/StatefulSet/DaemonSet apps/v1beta1 or extensions/v1beta1 without spec.selector using a chart version in metadata.labels can't be upgraded.
Note:
This is the first bug among 3 regarding "I can't upgrade my Release".
For the apps/v1beta2 problem, see https://github.com/helm/charts/issues/7680.
For the immutable volumeClaimTemplate in StatefulSet (any version), see #7803
From the k8s documentation

in Deployment/StatefulSet/DaemonSet apps/v1beta1 or extensions/v1beta1, spec.selector is not mandatory.

If it is not specified, it is generated from spec.metadata.labels

Source: Official documentation: https://v1-7.docs.kubernetes.io/docs/concepts/workloads/controllers/deployment/

If .spec.selector is unspecified, .spec.selector.matchLabels defaults to .spec.template.metadata.labels

Impacted charts
Probably all of https://github.com/helm/charts/pulls?utf8=✓&q=is%3Apr+"Add+chart+and+release+labels+in+pods"+
How to reproduce (using stable/dokuwiki as an example)
Let's install the stable/dokuwiki chart using git, change the version in chart.yaml (example: 1.2.3 -> 1.2.4) without changing anything else, and try to upgrade. See it fail:
cd stable/dokuwiki
helm install --name test .
vim Chart.yaml
helm upgrade test .

Because of metadata.labels in templates/deployment.yaml containing:
chart: {{ template "dokuwiki" . }}
It will fail with:
Error: UPGRADE FAILED: Deployment.apps "test-dokuwiki" is invalid: spec.template.metadata.labels: Invalid value: map[string]string{"release":"test", "app":"dokuwiki", "chart":"dokuwiki-2.0.7"}: `selector` does not match template `labels`

I think it comes from the fact that during the initial installation, k8s generated the selector, but during upgrade, both helm and k8s ignore it.
But putting an explicit selector, even if it contains a chart label, will make it upgradable:
  selector:
    matchLabels:
      app: {{ template "dokuwiki.name" . }}
      chart: {{ template "dokuwiki.chart" . }}
      release: {{ .Release.Name | quote }}

helm upgrade test .

It works!
Conclusion

We must always set selector in Deployments/StatefulSets/DaemonSets.
Selectors can technically contain a chart label although it would be not wise as it would break compatibility for apps/v1beta2 and later (see #7680 )
Ideally, Helm should be aware of selector (explicit or generated)
Note that charts using v1beta1, not having explicit selector, and don't have mutable fields (i.e no chart or version) in spec.template.metadata.labels are not impacted.

Thankfully, this can be solved using a minor version change.

cc @mattfarina @davidkarlsen @scottrigby @javsalgar @carrodher

Hey @scottrigby, just helping!
You are correct, except that, in my understanding, the immutability comes with apps/v1beta2 and later, and not only apps/v1. Any confirmation or invalidation would be appreciated! :)
Let me clarify the conclusion by clearly separating the two problems, i.e what should be done now (apps/v1beta1 or extensions/v1beta1) and what is nice for the future (>=v1beta2).
Regarding #7692, I didn't want to get into the details of "why" but let's try, let's continue there for this part of the discussion.

Have all charts been updated? Can we close this?

Charts owned by Bitnami were updated

I've also encountered this issue in the stable/mongodb chart.  It doesn't appear to be fixed yet (https://github.com/helm/charts/tree/master/stable/mongodb).

It is related to https://github.com/helm/charts/issues/7680 to be exact, thanks for reporting.

MongoDB chart was fixed on this PR: https://github.com/helm/charts/pull/10121

We can close this issue because we have the correct way to handle this documented in the Names and Labels section of the REVIEW_GUIDELINES.
If we want to automate checking for this, we should add to https://github.com/helm/chart-testing 😄


Return the created component from BrowserRenderer.AddComponent

I have a sceanrio that I need to render a RenderFragment into a given <div> element, provided by javascript. For this, I create a component to wrap my RenderFragment and render it via an instance of the BrowserRenderer. The problem is that I cannot pass the concrete RenderFragment to the component rendered by the BrowserRenderer. The easiest solution is to return the created component from the AddComponent method of the BrowserRenderer.
The concrete scenario is to draw arbitrary content into a leaflet div-icon.
This could also be used to pass the entry assembly into the router component without the workaround of the App component.
var router = app.AddComponent<Router>("app");
router.EntryAssembly = Assembly.GetEntryAssembly();

I think a preferred solution would be for BrowserRenderer's AddComponent method to get an extra overload that lets you pass in a ParameterCollection for the new root component you're adding. I'm adding this feature request as a note on #1.


I noticed this thing after picking mastermind

I got Arcanist Spellbook with 0 levels in it.

The arcanist's consume spells feature I patched references both the arcanist's spellbook and the mastermind spellbook which causes this to happen. If you play the arcanist class it would also show mastermind 0.
Instead of patching the original arcanist feature, I'll need to create a duplicate consume spells feature that would reference the mastermind spellbook.


[Fix-4289][*] Flink name with disappeared and unescaped problem

Tips

Thanks very much for contributing to Apache DolphinScheduler.
Please review https://dolphinscheduler.apache.org/en-us/community/index.html before opening a pull request.

What is the purpose of the pull request
this closes #4289
Brief change log

fix flink name not display in process definition editor
fix flink name not escape problem

Verify this pull request
(Please pick either of the following options)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Manually verified the change by testing locally.

Codecov Report

Merging #4290 (54415e3) into dev (140679e) will decrease coverage by 0.00%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##                dev    #4290      +/-   ##
============================================
- Coverage     43.16%   43.16%   -0.01%
  Complexity     3168     3168
============================================
  Files           467      468       +1
  Lines         21668    21670       +2
  Branches       2621     2621
============================================
- Hits           9354     9353       -1
- Misses        11414    11415       +1
- Partials        900      902       +2

Impacted Files
Coverage Δ
Complexity Δ

...pache/dolphinscheduler/server/utils/ArgsUtils.java
50.00% <50.00%> (ø)
1.00 <1.00> (?)

.../dolphinscheduler/server/utils/FlinkArgsUtils.java
86.53% <100.00%> (ø)
11.00 <0.00> (ø)

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.00% <0.00%> (-2.78%)
9.00% <0.00%> (-2.00%)

...inscheduler/service/zk/CuratorZookeeperClient.java
65.85% <0.00%> (+4.87%)
8.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

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


Not imported classes / NameResolver handles unknown namespace

Actual when a class, that is used within a method, is not imported by a use-statement, this classes are resolved as if they belong to the global namespace (see OOPExtractorTest#40-60 and f4.php and also OOPExtractorTest#62-75 and f5.php. But instead they should be resolved as if they belong to the current namespace.
The PR changes this behavior.
Therefor I needed to change the NameResolver. Actual when the dependencies are fetched from a class, the NameResolver will resolve the dependencies within the class (knowing the namespace of the class, see ReflectedClass#129) and before that the class will also be resolved within the method (without knowing the namespace, see ReflectedMethod#223).
This is how an unimported class will be resolved actual:

Namespace of the method
Class name used within the method
Class name after resolving within ReflectedMethod
Class name after resolving within ReflectedClass

My
Example
\Example
\Example

And this is how an unimported class will be resolved with this pr:

Namespace of the method
Class name used within the method
Class name after resolving within ReflectedMethod
Class name after resolving within ReflectedClass

My
Example
Example
My\Example

Yes, that's a big change, but that is logical... Thanks !


Update dr.sh with better error message

Note below the improved error message under the following if statement
if [ $CONTAINER = "none" ]; then
#!/bin/sh

CMD=$1
CONTAINER="none"
ACTION="pass"
RUN_OPTS="-it"
YARN_DIR=themes/ptv-mpw

case "$CMD"
in
    php)
        CONTAINER="phpcli"
        ACTION="php"
        ;;
    composer)
        CONTAINER="phpcli"
        ACTION="composer"
        ;;
    phing)
        CONTAINER="phpcli"
        ACTION="phing"
        ;;
    codecept)
        CONTAINER="phpcli"
        ACTION="vendor/bin/codecept"
        ;;
    fpm)
        CONTAINER="php"
        ACTION="bash"
        ;;
    mysql)
        CONTAINER="mysql56"
        ACTION="mysql"
        ;;
    mysqlimport)
        CONTAINER="mysql56"
        ACTION="mysql"
        RUN_OPTS="-i"
        ;;
    sel)
        CONTAINER="selenium"
        ;;
    node)
        CONTAINER="node"
        ACTION="bash"
        ;;
    yarn)
        CONTAINER="node"
        ACTION="yarn"
        ;;
esac

case "$2"
in
    cli)
        ACTION="cli"
    ;;
esac

if [ $CONTAINER = "none" ]; then
    echo "Please provide a container or a command to be run - ./dr.sh {container|command} [arguments] "
    echo "Available containers are as follows; any arguments will be used as docker exec parameters : "
    echo "  php, fpm, mysql, node"
    echo "Available commands are below; any arguments are passed to these commands in the container "
    echo "  composer, phing, codecept, mysqlimport, yarn"

    exit 1;
fi

if [ $ACTION = "cli" ]; then
    echo "Dropping to shell in $CONTAINER"
    docker exec -it -u `id -u`:`id -g` $(basename $(pwd))_${CONTAINER}_1 /bin/bash
else
    shift
    echo "Running command $ACTION in $CONTAINER"
    if [ "mysqlimport" = $CMD ]; then
        docker exec ${RUN_OPTS} -u `id -u`:`id -g` $(basename $(pwd))_${CONTAINER}_1 ${ACTION} "$@" < /proc/$$/fd/0
    elif [ "yarn" = $CMD ]; then
        docker exec ${RUN_OPTS} -u `id -u`:`id -g` $(basename $(pwd))_${CONTAINER}_1 bash -c "cd ${YARN_DIR}; yarn $@"
    else
        docker exec ${RUN_OPTS} -u `id -u`:`id -g` $(basename $(pwd))_${CONTAINER}_1 ${ACTION} "$@"
    fi
fi

@silbinarywolf if you're across the solution, could you please put a PR through?

@nglasl This was already put in.
https://github.com/symbiote/docker-project/blob/master/dr.sh#L69

Sweet, cheers for the quick response @silbinarywolf.


Fatal error LiDAR.h No such file or directory

I have update from agml=0.4.6 to agml=0.5.0 and encounter "fatal error: LiDAR.h: No such file or directory" error when running "agml.synthetic.reinstall_helios()".
I have tried to use "pip install lidar" and that doesn't solve the error.
I am using wsl2 with win11

Hey @Vincent-WangCH, you need to use the lidar_enabled option: e.g., agml.synthetic.reinstall_helios(lidar_enabled = True). The lidar PyPi package is not used in this project, so pip install lidar won't do anything. Let me know if this helps.

@Vincent-WangCH, are you still having trouble with this issue?

Hey @amogh7joshi I have tried the mentioned step above but I'm getting the below error. Could you please help me ?
TypeError                                 Traceback (most recent call last)
Cell In[3], line 1
----> 1 agml.synthetic.reinstall_helios(lidar_enabled = True)
TypeError: reinstall_helios() got an unexpected keyword argument 'lidar_enabled'

@dariojavo Any thoughts on this?

Hi @sahith-vancha,
I imagine you are having the LiDAR.h: No such file or directory error prior to this one. Could you please try to remove your Helios installation folder manually? You can find it in the corresponding environment that you are using. For example:
/home/user/anaconda3/envs/example/lib/python3.8/site-packages/agml/_helios/Helios/
you can remove the 'Helios' folder and run your code again. Now, the latest version of Helios will be download with the updates that we have made to run it from AgML, including the LiDAR plugin.

@Vincent-WangCH Did this resolve your issue?


Update "conda" CI workflow

TBA.

How to review
Note the CI checks all pass.
PR checklist

[ ] Continuous integration checks all ✅

[ ] Add or expand tests; coverage checks both ✅

[ ] Add, expand, or update documentation.

[ ] Update release notes.

@glatterf42 at a glance, we are getting the same 2 jobs failing with the updated versions. Will continue to poke at this.


[Loc] Alex update to xlfs

This PR fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.124%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
28179

Relevant Lines:
62412

💛  - Coveralls


Dev add embedding functionality

Embedding module added; data/processed directory populated with samples.

Before looking at anything, this PR has there are 28 file changes, which gave me a mini heart-attack :) Typically, you don't git track data (your jsons). These are usually stored on disk, s3, drive, etc. I would untrack them now (can be a separate PR).


limitedEntry[key].slice is not a function

I just installed this plugin
Strapi Version:  "@strapi/strapi": "4.20.3",
"strapi-plugin-fuzzy-search": "^2.2.0",
plugins.js
 "fuzzy-search": {
    enabled: true,
    config: {
      contentTypes: [
        {
          uid: "api::order.order",
          modelName: "orders",
          fuzzysortOptions: {
            characterLimit: 300,
            keys: [
              {
                name: "orderNum",
                weight: 100,
              },
              {
                name: "poNum",
                weight: 100,
              },
            ],
          },
        },
      ],
    },
  }

[2024-02-29 16:44:10.116] error: limitedEntry[key].slice is not a function
TypeError: limitedEntry[key].slice is not a function
at C:\Users\hey\coding\sapele_server\node_modules\strapi-plugin-fuzzy-search\dist\server\services\fuzzySearch-service.js:25:33
at Array.forEach ()
at C:\Users\hey\coding\sapele_server\node_modules\strapi-plugin-fuzzy-search\dist\server\services\fuzzySearch-service.js:21:15
at Array.map ()
at limitCharacters (C:\Users\hey\coding\sapele_server\node_modules\strapi-plugin-fuzzy-search\dist\server\services\fuzzySearch-service.js:18:68)
at buildResult (C:\Users\hey\coding\sapele_server\node_modules\strapi-plugin-fuzzy-search\dist\server\services\fuzzySearch-service.js:32:11)
at getResult (C:\Users\hey\coding\sapele_server\node_modules\strapi-plugin-fuzzy-search\dist\server\services\fuzzySearch-service.js:94:42)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async C:\Users\hey\coding\sapele_server\node_modules\strapi-plugin-fuzzy-search\dist\server\controllers\search-controller.js:36:20
at async Promise.all (index 0)
I think it has to do with this function

const limitCharacters = (entries, characterLimit, keys) => entries.map((entry) => {
    const limitedEntry = { ...entry };
    const entryKeys = Object.keys(limitedEntry);
    entryKeys.forEach((key) => {
        if (!keys.includes(key))
            return;
        limitedEntry[key] = limitedEntry[key]
            ? limitedEntry[key].slice(0, characterLimit)
            : limitedEntry[key];
    });
    return limitedEntry;
});

The errors leaves when i remove characterLimit from the plugins.js
fuzzysortOptions: {
    // characterLimit: 300,
    threshold: -100,
    keys: [
      {
        name: "poNum",
        weight: -100,
      },
      {
        name: "orderNum",
        weight: 200,
      },
    ],
  },

Hey @heyjaypray ,
I could reproduce your error. It is due to my oversight, as I always assumed that one would search only in strings and not in numbers. I will fix this asap.
Might I ask you what your specific use case is with fuzzy searching numbers though? Just out of curiosity and getting a grasp of more potential use cases...

Hey @heyjaypray rolling out the fix asap, it seems like GitHub actions and Pull Requests seem to have issues at the moment now which means I can't run my release workflow 😅

Hey @heyjaypray ,
I could reproduce your error. It is due to my oversight, as I always assumed that one would search only in strings and not in numbers. I will fix this asap.
Might I ask you what your specific use case is with fuzzy searching numbers though? Just out of curiosity and getting a grasp of more potential use cases...

I'm using numbers to search in my case because I need to look up order numbers
Thanks for the quick update!!


Fix typo to get proper error message

Otherwise you get: error is not defined
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #175 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #175   +/-   ##
=======================================
  Coverage   80.85%   80.85%
=======================================
  Files          85       85
  Lines        5700     5700
  Branches      914      914
=======================================
  Hits         4609     4609
  Misses        932      932
  Partials      159      159

Impacted Files
Coverage Δ

...ons/appsync-operations/helpers/helper-resolvers.js
91.89% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Directly calling predict function in prediction endpoint is failing

(train_X, train_y), (test_X, test_y) = read_input('ames_dataset/train.csv')
endpoint.predict(test_X)

This fails as it tries to pass a numpy array directly to requests object. @abhi-g and I found it during our manual e2e testing.

Will this interface allow for batch prediction, i.e., looks like test_X is a collection of data points to make predictions on?

it depends on the predict function impl. if the user handles batch of examples then we can do it.


Fix linking problem of protobuf-3.2.

Fix #46

And, please create an issue?

And, please create an issue?

Done.


Added extra setup information to CONTRIBUTING.md

Hi, I added important information to solve errors I ran into when setting up Pipeliner. Hopefully these new additions will make the setup process smoother for future Pipeliner contributors! 😀

🎉 Great job🤩


Whatsapp Integration [10h]

Issue
Nurses and CHWs need a way to share a patient's progress report with them. To do this, we are doing a WhatsApp integration. At the bottom of the progress reports, we are including a new button where they can "send via whatsapp".
Flow:
Hit "Send Via WhatsApp" Button --> consent Screen --> validate phone number (branches off if new number is needed)--> choose which portions of the report to send --> verify send
Video Overview:
You can watch a long winded but helpful video here.
Design:
You can find the design files in Figma here.  This flow is located to the right of the screen under the heading "Whatsapp Send"
Report sections for selection:
Source: https://docs.google.com/spreadsheets/d/1gQU_-YdyHKZKDz-3cIpF725aU3QSIxnkw8Hxe7zFx4U/edit#gid=0

To Do:

Create backend logic to send to whatapp.
Style screens per design

@jeanette-pelizzon @adamhstewart
Can't open Figma link. It says file does not exist.
Points after reviewing the video:

As I see it, country code needs to be reconsidered. If we really need it, there are over 200 countries. Does it mean that we need to add all their flag icons.
Also, what about phone validation? Right now we don't validate anything, beside number characters. Do we need to change that? And, if we need country code, do we need to have validation logic for over 200 countries?

I don't think these are the right button labels:

At entering phone number flow, I don't see an option of canceling the action. What if we want to get back to previous step, or cancel sending message completly?

Selection of what part of report to send need to be well defined for all progress reports we need to support.
Technical implementation of this is taking a screenshot of what we want to send, meaning that we will have to view that section on tablet (lots of effort hiding here), so we need clear definition on all variations.
Also, I don't see how this should be supported for for ANC Labs section, for example.

Note: Besides making the screenshot themselves, we'll need to define a way of storing these messages with screenshots, and interaction with WhatsApp API for sending them. I can't estimate before we have API details, but I'd say that roughly this will reqiure at least 40h.

@LudmilaGoichman please test on Dashboards env

NOT FINISHED!
Questions:

Should there be spaces between numbers like in Figma?

app
Figma

Typo - y is missing in Immunization History:

Maybe add the word Charts like in Figma?

app
Figma

When the phone number does not exist, there should be 5 delivery attempts. But in all nodes (with existing phone number and non-existing) Delivery attempts = 0:

existing phone
non-existing phone

Checked and Approved:
Flow:

1
2
3
4

5
6 (if the answer yes)
7
8

9
10
node created in the backend

@LudmilaGoichman

When the phone number does not exist, there should be 5 delivery attempts. But in all nodes (with existing phone number and non-existing) Delivery attempts = 0:

The thing is that we deliver message to vendor, we don't know if number is correct or not.
Delivery attempts counter is for retrying message delivery to vendor.
Once vendor got the message, it's their responsibility. We don't get feedback from vendor.
We consider message as delivered.

@LudmilaGoichman  @adamhstewart

Should there be spaces between numbers like in Figma?

No, we show numbers as they are. If we get feedback from customer, we may reconsider

Typo - y is missing in Immunization History:

Fixed

Maybe add the word Charts like in Figma?

No, we show the name of report pane, and pane is called "Growth'.

~4. When the phone number does not exist, there should be 5 delivery attempts. But in all nodes (with existing phone number and non-existing) Delivery attempts = 0:~

Answered at https://github.com/TIP-Global-Health/eheza-app/issues/292#issuecomment-1630626599.

Photos are not appear in the whatsapp image (in all reports except progress photos from ANC report //made just before sending the message).

This is limitation. html2canvas loads photos, by issuing fetch requests to image urls.
On our APP however, we have SW layer that catched photos fetch requests and maps then to locations at device cache, where real images are stored. html2canvas can't use SW, and therefore, it can't get the image.
This affects the image we show at person details, and Photos pane at ANC report.
Seems to me we can not suggest to send the photos pane, and hide person image to avoid issuing invalid requests to backend. @adamhstewart what do you think?

Standard Pediatric report. The section "Acute Illness History" in the list (in the flow), in the report called "Active Diagnosis"

We show the name of report pane.

NCD Progress Report. The section "Active Diagnosis" in the list (in the flow), in the Excel file called "Acute Illness"

We show the name of report pane.

ANC report. "Medical History" section is mentioned in the Excel file, but it doesn't exist in the report and in the screen:

Excel issue - we don't have such pane at report, so obviously, we can't send it.

The word "diagnosis" is used in the Excel file, but in NCD Progress Report list (in app) it's "diagnosis" and in ANC report list it's "diagnoses" it's not consistent.

Fixed.

Standard Pediatric report, NCD Progress Report, ANC Report. Dots are not displayed in the charts in the whatsapp image. I think it's more critical in the charts from the NCD Progress Report and the ANC Report

This happens because htm2canvas have trouble rendering certain SVG elements and their associated styles - SVG markers which we use to mark the points.
Solution for this is to replace SVG markers usage with basic SVG circle shape.
This is time consuming as we have multiple charts to handle, but since charts are important elements of the report, we'll have to implement the fix. //cc @adamhstewart

In the group encounters the report cannot be sent, right?

There are no actions there, but maybe we do need it. @adamhstewart ?

The first pop-up doesn't appear for a long time after clicking the "Send via WhatsApp" button. And when clicking the button, it doesn't react for a long time (doesn't change color to a darker). And all buttons in the pop-ups also don't react for a long time

Taking screenshot can take up to a minute to complete.
The problem here is that this is actual screenshot, so if we put a popup on screen, it will cover area of the report.
We do disable actions and make screen a little darker till actions is completed.

AI. The icons from "Action Taken" are not displayed in the message:

Fixed

When I save nutrition signs, they appear in the report (before sync!), but after sync - they disappear.

Known issue that was handled already.

@adamhstewart Opened a follow up issue - #872


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_a2yhaui_cj/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_a2yhaui_cj/clone/recipe/meta.yaml:19: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_a2yhaui_cj/clone/recipe/meta.yaml:42: http_url: http://tweepy.readthedocs.io is not https
WARNING: /tmp/abs_a2yhaui_cj/clone/recipe/meta.yaml:42: missing_description: The recipe is missing a description
ERROR: /tmp/abs_a2yhaui_cj/clone/recipe/meta.yaml:42: http_url: http://www.tweepy.org is not https
Errors were found


out of date syntax

You can also use .[] to index strings in all versions. The new syntax is preferred.

Should be “use [] to index”

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

ID: 752405db-4da8-6c06-f58b-22dcc9caa7a7
Version Independent ID: 78362807-847f-b947-2c75-5036f7765b32
Content: Strings - F#
Content Source: docs/fsharp/language-reference/strings.md
Product: dotnet-fsharp
GitHub Login: @cartermp
Microsoft Alias: dotnetcontent

@daveyostcom I'm not sure what change you'd like. We know that folks in newer F# versions will only use s[] but s.[] appears in older versions and older code. Would you just like it to be more clear how strong this preference is, or is there something else you are seeing.  The paragraph is:

You can access individual characters in a string by using array-like syntax. The following examples use [] to index strings. This syntax was introduced in F# 6.0. You can also use .[] to index strings in all versions. The new syntax is preferred.

And thanks for helping make the F# docs better!


Sequence of data in REQUEST_RECORD_LOAD

The response format of REQUEST_RECORD_LOAD makes it impossible for a client to load the entire response sequentially:
[(payload-status:byte)[(record-content:bytes)(record-version:int)(record-type:byte)]*]+

In order to read sequentially over the entire response, the client needs to know the record-type in advance, so the order of this data appears to be backwards currently.
I believe the record format of each payload chunk would need to change, placing the payload metadata record-type and record-version ahead of record-content, e.g.:
[(payload-status:byte)[(record-type:byte)(record-version:int)(record-content:bytes)]*]+

With the new binary serialization format, which doesn't require parsing, client implementations could then be made simpler and more memory-efficient, if they didn't need to load/buffer record-content in-memory before interpreting the data.
I would be extremely glad to see this change made to the protocol before the final 2.0 release, as I am implementing a client that targets 2.0+ exclusively.

On a related note, while working on a binary driver, I discovered this change will not in fact enable a binary client to read through the data stream sequentially.
The issue here, is the binary serialization format, for which the documentation is incomplete - I had to discover (by reading the source code) what negative values for length means, that it is not in fact a length, but a property in the schema.
The concept of "global properties" isn't defined in the documentation at all, and there is no explanation on how to find them - I had to discover (also by reading the source code) that you need to do another round-trip doing an SQL select from schema:metadata, another undocumented feature.
The problem is, there is no way a client can know about the necessary schema in advance - hence, still no way to read sequentially through the serialized document. You have to buffer it in-memory, then do another round-trip to obtain the schema, before you can deserialize.
Possible solution: every change to the schema could result in an updated hash of the whole schema. Clients could then cache the schema, and specify the hash of the schema they have cached locally, when they connect - so that the server would know if the client has the latest schema or not, and could supply it up front, at connection-time. It makes sense, I think? Seeing as how the client cannot deserialize records without the schema information - in fact, it probably doesn't make much sense to connect without this information? Every client currently needs to send the query for the scheme metadata upon connecting anyway - there's no reason this needs to be optional, is there?
If you're concerned about large schema with frequent changes, I would suggest including both a hash and a serial number for schema changes - every change to a class/cluster/property would increase the serial number, and the serial number would be attached to the schema entity that was updated/added. This would enable the server to send incremental schema changes to the clients.
Thoughts?

@mindplay-dk
One point is that we are seriusly thinking to remove the global ploperty id from network serialization, this will have a small overhead in terms of bandwidth, but for some client implemetation(like php) would be much easier.
Sorry for the not updated documentation that's my fault going to fix that now.
for the schema push we have a plan for do it but not design done till now, your idea si not bad.

we are seriusly thinking to remove the global ploperty id from network serialization

If I'm frequently fetching small records with (say) lots of integer fields, the metadata could outweigh the actual data in volume.
I would strongly advice against that.
Adding schema-awareness to the client/server protocol sounds like a more natural and scalable (for all scenarios) approach, but it sounds like you like that idea as well.

(I would rather wait for a solid solution than have an interim or "patch" solution arriving sooner...)


fix(sbb-autocomplete): highlight option when options change

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

:exclamation: No coverage uploaded for pull request base (main@33dfe8e). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2317   +/-   ##
=======================================
  Coverage        ?   93.76%
=======================================
  Files           ?      238
  Lines           ?    24306
  Branches        ?     2146
=======================================
  Hits            ?    22790
  Misses          ?     1473
  Partials        ?       43

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


Estimize Custom Data Implementation Does Not Set Symbol Property

Expected Behavior

We can access the symbol for a given piece of Estimize data
Actual Behavior

The symbol is blank (default)
Potential Solution

Set Symbol = config.Symbol in Reader calls
Reproducing the Problem

public class EstimizeDataAlgorithm : QCAlgorithm
{
    public override void Initialize()
    {
        SetStartDate(2017, 1, 1);
        SetEndDate(2017, 12, 31);

        AddData<EstimizeRelease>("AAPL.R");
    }

    public void OnData(EstimizeRelease data)
    {
        Log($"{Time} - {data.Symbol.Value}");
    }
}

prints:
20190801 10:26:20.596 Trace:: Log: 1/31/2017 9:00:00 PM -
20190801 10:26:20.596 Trace:: Log: 5/2/2017 8:00:00 PM -
20190801 10:26:20.596 Trace:: Log: 8/1/2017 8:00:00 PM -
20190801 10:26:20.596 Trace:: Log: 11/2/2017 8:00:00 PM -

System Information

Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

Close by https://github.com/QuantConnect/Lean/pull/3467


Add source files to android-ddp-0.0.1-bundle.jar

It would help in development of apps that use Android-DDP, if android-ddp-0.0.1-bundle.jar contained also source files of Android-ddp library. This way one could easily peek how given library class is implemented.
If you prefer to keep sources outside of bundle jar, consider providing separate android-ddp-0.0.1-bundle-src.jar with sources only.

Thanks! You're right, that would be useful.
Another JAR that includes the sources should have been introduced with https://github.com/delight-im/Android-DDP/commit/fa7822d7c2a0c948b768ccbb7b609afa7cc1aa3b now.
Please try again with the *-src JAR and confirm whether it works or not.

Thanks for so quick fix!
However for now I cannot use the lib as I realize it doesn't support wss as autobahn library used by it internally doesn't have that support as described here https://github.com/tavendo/AutobahnAndroid/issues/8

Thank you!
Tracking the SSL issue here from now on :)


dashboard "see more events" bug

Subject: dashboard "see more events" bug
Description:
if i click on "see more events" on the dashboard, it keeps replicating the first two preview events as often as i click it.
Results:
see more events ;)
Backend version: develop-b3cee755
Frontend version:
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:101.0) Gecko/20100101 Firefox/101.0
Page: https://couchers.org/
User: moe012 (2769)

sure sounds like a bug