random_walk() does not act as expectation (find the new peer and add it into DHT)

The random_walk() function should walk around on DHT to find the peer. After new peer found, it should be add to the DHT to let other nodes know this new peer join the p2p network. However, the new peer won't be added into DHT base on current code base, even we set up Network behaviour "Identify" already. In this situation, if a node connect to network by a bootstrap node, it only be awared by the bootstrap node, but cannot be awared by other node (the Identify behaviour of libp2p won't be triggered). Therefore, the new peer node's info won't be added into map 'peer_lookup_in_networking', and issue#3 happens.

It seems like kad is running in client mode. Ref. client and server mode.

Nodes, both those operating in client and server mode, add another node to their routing table if and only if that node operates in server mode.

I suppose your findings was somehow related to the recent change in libp2p Kad.

By default, a node is in client-mode and only allows outbound requests.

If upgraded version is being used in this case, the issue can be solved by calling swarm.set_external_address to configure kad to be server mode.


Add capability to load symbols from current executable

If PKCS#11 symbols are available inside the current executable as it can be necessary for iOS development.

I agree with @hug-dev that this can be simplified. Btw, I'm on mobile, isn't the libloading dep missing from cargo.toml?
The idea behind the change is good though!

Thanks for the PR!
I am curious, in which context would the library be contained in the executable? In case it has been statically linked 🤔?
Also wondering if instead of changing the interface and introducing the new LibLoadingType we could create a new constructor, like new_from_self which would do the same :) What do you think?

Yes when it's statically linked, which is mandatory on iOS when following the AppStore rules :)
I was hesitating between different approch when searching on how to add this feature and the duplication of the new function was one of them, so I've changed it to that.

I agree with @hug-dev that this can be simplified. Btw, I'm on mobile, isn't the libloading dep missing from cargo.toml?
The idea behind the change is good though!

We were already using this on iOS and Android with the old rust-pkcs11 crate (but never merged on upstream), so it's working well. And I've tested it on a real device ;)

This is shaping really nicely 👌
I won't be able to look at it deeply before Monday, though :(

Could you tell me what's happening next ? Or more precisely, what could be the ETA of an official release with this version ? :)
Thanks a lot !

what could be the ETA of an official release with this version

We have no schedule set! It's all based on demand. Since the last release was in June and we got quite a lot of contributions recently, I guess we could make a new version if you wanted!

I guess we could make a new version if you wanted!

I think it's a good idea. As you've said we have quite a lot of good contributions piled-up in main that people may wait for :+1:

We can maybe announce in the community channel that a new release will be done in like two weeks, so that people have time to push in more changes before the release?

We can maybe announce in the community channel that a new release will be done in like two weeks, so that people have time to push in more changes before the release?

Yes, excellent idea! :)


Please Publish 0.17

@Krzysztof-Cieslak I would like to take advantage of https://github.com/Krzysztof-Cieslak/vscode-elm/pull/196 However, this PR was merged after the latest published release of this plugin. Please can you publish 0.17 that includes https://github.com/Krzysztof-Cieslak/vscode-elm/pull/196 Thanks

New version released, sorry for troubles!

Many thanks @Krzysztof-Cieslak Does exactly what I needed. Keep up the great work on this project 👍


alignchecker: add code to also check values

In the context of #12415, we wanted to add a test to compare values in
Go with ones in BPF. The first approach used cgo, which is not ideal.
This add some initial code to alignchecker that allows to check values
between BPF and go by reading the ELF .rodata section.
If this makes sense, it can be used for to add testing for #12415.
Signed-off-by: Kornilios Kourtis<EMAIL_ADDRESS>

Coverage increased (+0.009%) to 36.946% when pulling e0c5241937576354efeb4d3edcb1193858e678cb on kkourt:valuetester into f4ea3f67145eb5c8c6d14a6087058577e9885322 on cilium:master.

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

I thought it would be a good to move the code to a new package called constchecker to avoid confusion. Other than some name changes, the code is the same, so I thought there is no need for a new PR.

test-me-please

Something I just remembered is it'd be nice to split out the rename and the actual implementation changes into separate commits. This will help with bisecting, backporting fixes, etc.

Something I just remembered is it'd be nice to split out the rename and the actual implementation changes into separate commits. This will help with bisecting, backporting fixes, etc.

I just forced pushed the new commits, so there shouldn't be any renaming (unless I'm missing something). That is, the changes to alignchecker are as if they never happened.

retest-runtime (failure seems to be on a DNS request, which seems like unrelated https://jenkins.cilium.io/job/Cilium-PR-Runtime-4.9/1105/)

Something I just remembered is it'd be nice to split out the rename and the actual implementation changes into separate commits. This will help with bisecting, backporting fixes, etc.

I just forced pushed the new commits, so there shouldn't be any renaming (unless I'm missing something). That is, the changes to alignchecker are as if they never happened.

Ah, my mistake. I thought the previous code was added on to, not new code :)

test-me-please

Changed the PR to draft until a use of constchecker is added.

Latest version adds a check for the loadbalancer, a minor checkpatch fix, and removes the object file for the constchecker self-test.

@qmonnet  I added a small checkpatch change here so that it does not complain about using inline comments for SPDX, since using SPDX seems to be the preferred way (https://github.com/cilium/cilium/pull/12436#discussion_r451125813).
I later noticed that you had sent a (more complete) patch upstream for this issue (https://lore.kernel.org/patchwork/patch/1265784/#1461076).
Not sure what the proper thing to do here, but it's a bit confusing to get conflicting checkpatch errors, so maybe we should carry the SPDX change locally regardless of whether it's upstreamed?

Not sure what the proper thing to do here, but it's a bit confusing to get conflicting checkpatch errors, so maybe we should carry the SPDX change locally regardless of whether it's upstreamed?

(Thanks for asking!) I agree this is confusing. I was hoping the patch would be merged upstream, but it didn't get a warm reception and now this seems unlikely to happen. We discussed that during a datapath meeting (last week maybe), and we agreed on the following: There is some follow-up I intend to do on checkpatch, to package it as a Docker image instead of keeping it in the Cilium repo (on Ilya's suggestion). Once we have that, we can apply local patches (patch -p1 < ...) when building the image: this way we keep the mainline version from Linux repo, making it easy to update, but apply the desired local changes before using it.
In the meantime, I would as well keep the current version untouched. Unless you feel this is too much confusing and really prefer to patch, in which case we can start changing checkpatch.pl now and then report the changes later when packaging it as an image.
I opened #12467 about packaging checkpatch.

In the meantime, I would as well keep the current version untouched. Unless you feel this is too much confusing and really prefer to patch, in which case we can start changing checkpatch.pl now and then report the changes later when packaging it as an image.

As long as people don't mind the red checkmark in BPF checks, it's fine by me :)
I will remove the patch from this PR.
As a side note, there is also:
+++ b/contrib/checkpatch/checkpatch.sh
@@ -23,6 +23,8 @@ ignore_list=(
     LONG_LINE_COMMENT
     # Ignore tolerance that comes by default
     C99_COMMENT_TOLERANCE
+    # We do not have a MAINTAINERS file, so we can ignore this
+    FILE_PATH_CHANGES
 )

This seems like something that can be applied directly (or maybe not?)

We could either ignore the FILE_PATH_CHANGES reports as you suggest or patch it in the future to say CODEOWNERS (that we do have) instead of MAINTAINERS. Either is fine by me.

test-me-please

rebased

test-me-please

test-me-please

This seems like a reasonable way to perform such checks. At a high level, I wonder whether we would benefit from moving more towards a model where the Golang implementation generates all of these constants from the Golang definition into a header file instead, then the BPF code simply includes those headers. That way, the constants would be ensured to be the same without having to perform additional checks afterwards.

I do think that the approach in this PR is problematic in the sense that it's too much work for a developer to add new checks. They would have to edit two (go test, bpf file with constants) or more (test script) files, and probably would end up just adding a comment instead. And, if people are not going to use it, I'm not sure if it's worth merging.
Generating constants from Go would be better in that sense, but I'm also not sure if it's worth doing it, since it adds some level of complexity, and I do like the fact that the bpf code is self-contained.
Also, I'm not sure if constants checks solve a real problem, i.e., having a chance to catch actual bugs. At the end of the day, maybe just a comment in Go and BPF side that these constants should match is the best solution at this point.

test-me-please

retest-net-next

Generating constants from Go would be better in that sense, but I'm also not sure if it's worth doing it, since it adds some level of complexity, and I do like the fact that the bpf code is self-contained.

In practice we're already doing this via the headerfiles for ep_config.h, netdev_config and node_config.h so it's very much in line with our existing templating approaches.

Also, I'm not sure if constants checks solve a real problem, i.e., having a chance to catch actual bugs. Maybe a comment in Go and BPF side that these constants should match is the best solution at this point.

It's always possible, but given that both (Go/C) constants are usually introduced at the same time it's typically fairly easy to catch mistakes. I can't specifically recall hitting an issue like this in the past with Cilium.

Unless someone suggests differently, I would opt to drop this one. (I know it's my PR and I'm the one supposed to argue for it, but as I said I think it's too much hassle for people to use it.) We can always resurrect it if we think it makes sense.


Dark Theme Added

Light Mode :

Dark Mode :

@Swadhin2001 Please shift the toggle button to the top right corner of the header.

Fixed

@Swadhin2001 Great!! Your PR have been merged. 💯


Update taffy requirement from 0.5 to 0.6

Objective

Supersedes #16127.

Solution

Added missing fields in taffy::style::Style struct.

Testing

TBD

Wondering if I should just do ..default() to fill in the new fields or if it's better to have all the fields explicitly in case there are future breaking changes that might affect the display of UI elements. Feedback will be much appreciated.

Oh! Thanks for the heads up @rparrett, I'll close this and the dependabot PR and reference that one.


No Access-Control-Allow-Origin header

Chrome and Firefox have both implemented Cross Origin Read Blocking now.  This results in the following:
Chrome:

Access to XMLHttpRequest at 'https://secure.screeenly.com/api/v1/fullsize' from origin 'https://localhost:5001' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource.

Firefox:

Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at https://secure.screeenly.com/api/v1/fullsize. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing).

Please consider adding the Access-Control-Allow-Origin header.

Thanks, will look into this.

Just for clarification: This only happens if you're calling the API through JavaScript in a Browser?

Yes, using Angular 8 HttpClient.
import { Component, OnInit, Input, OnDestroy } from '@angular/core';
import { HttpClient, HttpErrorResponse, HttpHeaders } from '@angular/common/http';
import { Domain } from '../models/domain.model';
import { catchError, tap } from 'rxjs/operators';
import { Observable, ObservableInput, throwError, Subscription } from 'rxjs';

const SCREEENLY_URL = 'https://secure.screeenly.com/api/v1/fullsize';

interface Request {
  key: string;
  url: string;
  height: number;
  width: number;
}

interface Response {

}

@Component({
  selector: 'investigator-domain-screenshot',
  templateUrl: './screenshot.component.html',
  styleUrls: ['./screenshot.component.scss']
})
export class ScreenshotComponent implements OnInit, OnDestroy {
  private subscriptions = new Subscription();

  @Input() domain: Domain;
  imageUrl: Response;

  constructor(
    private http: HttpClient
  ) { }

  ngOnInit() {

    const data: Request = {
      key: 'h1cH4L6DJORSAkeRnPyJmBO4ugPwc3S9HqaKJshrVWBhWRKaDr',
      url: `http://${this.domain.title}/`,
      height: 600,
      width: 600
    }

    const httpOptions = {
      headers: new HttpHeaders({
        'Content-Type':  'application/json',
        'Authorization': 'my-auth-token'
      })
    };

    this.subscriptions.add(
      this.http.post<Response>(SCREEENLY_URL, data, httpOptions).pipe(
        tap(response => console.log(response)),
        catchError(this.handleError)
      ).subscribe( response => {
        console.log(`received: ${response}`);
      })
    );
  }

  ngOnDestroy(): void {
    this.subscriptions.unsubscribe();
  }

  private handleError(error: HttpErrorResponse) {
    if (error.error instanceof ErrorEvent) {
      console.error('An error occurred:', error.error.message);
    } else {
      console.error(
        `Backend returned code ${error.status}, ` +
        `body was: ${error.error}`);
    }
    // return an observable with a user-facing error message
    return throwError('ERROR: Probably CORB/CORS related.');
  };
}

CORS-Headers are now correctly returned. Here's the JS code I used to test this.
fetch("https://secure.screeenly.com/api/v1/fullsize", {
    method: "POST",
    headers: {
        "Accept": "application/json",
        "Content-Type": "application/json"
    },
    body: JSON.stringify({
        url: "https://en.wikipedia.org/wiki/Special:Random",
        key: "API_KEY"
    })
})

.then(response => response.json())
.then(response => console.log(response.path))
.catch(error => console.error(error));


[TW#26654] conn_id mismatch in esp_ble_gatts_cb_param_t

Hi @Weijian-Espressif
I think there is conn_id mismatch in esp_ble_gatts_cb_param_t when we get events. As you can see BT_GATT is logging conn_id=3 and app callback is reporting conn_id=0. Test performed with esp-idf gatt_secure example.
I (12517) BT_GATT: GATT_GetConnectionInfor conn_id=3
I (12517) BT_L2CAP: L2CA_SendFixedChnlData()  CID: 0x0004  BDA: 5000f4ca1265
I (12520) SEC_GATTS_DEMO: [conn_id: 0, trans_id: 1]

@chegewara ,
 #define GATT_GET_TCB_IDX(conn_id)  ((UINT8) (((UINT16) (conn_id)) >> 8)) #define GATT_GET_GATT_IF(conn_id)  ((tGATT_IF)((UINT8) (conn_id)))
in application layer , we will translate conn_id to gatt_if.

Ok, so its only debug problem.


Unable to build from source

Describe the bug
I am unable to build phanpy from source by dowloading an artifact (tar/zip) from tags.
To Reproduce
Steps to reproduce the behavior:

Go to https://github.com/cheeaun/phanpy/tags
Click on any tar.gz link
Unpack the downloaded file
Change into phanpy-$version folder
Try to npm install, npm run build

Expected behavior
You get a clean, distributable version of phanpy as described in Self Hosting
Error message
$ npm run build

><EMAIL_ADDRESS>build
> vite build

fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
failed to load config from /tmp/tmp.bxIPUvnwZt/phanpy-2<IP_ADDRESS>b1800d/vite.config.js
error during build:
Error: Command failed: git rev-parse --short HEAD
fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).

    at checkExecSyncError (node:child_process:890:11)
    at execSync (node:child_process:962:15)
    at file:///tmp/tmp.bxIPUvnwZt/phanpy-2<IP_ADDRESS>b1800d/vite.config.js.timestamp-1696719649450-96852b13653e5.mjs:19:18
    at ModuleJob.run (node:internal/modules/esm/module_job:194:25)

Tags now have Releases. It's pre-built, without the need to run npm run build on your end. Let me know if this works

Perfect! That's even better then buildung it myself. Thank you! :+1:

Ah, one remark: would it be possible to ship a .tar or .tar.gz additionally/exclusively?
This would streamline my Dockerfile since nginx comes with curl and tar, but not with unzip.
Thanks for considering!

Awesome @cheeaun, thank you!
FROM docker.io/library/nginx:latest
WORKDIR /usr/share/nginx/html
RUN \
curl -sSOL https://github.com/cheeaun/phanpy/releases/latest/download/phanpy-dist.tar.gz \
&& tar -xf phanpy-dist.tar.gz \
&& rm phanpy-dist.tar.gz


[Request] Tabs

Something like this: https://getbootstrap.com/docs/5.0/components/navs-tabs/#javascript-behavior, it's great for Information boxes.

For note, tabs can be done without JS, I've done so with a set of radio inputs and labels to act as the switchers and tags for the CSS to show the right section, easily overall.  Even possible to be generic to the number of tabs as well.

@OvermindDL1 Would you illustrate us?


MEC Proceedings in Bibliography?

What is our policy on entering items from the MEC in the new bibliography? Specifically exclude? Include only the MEI ones (which could be quite a few)?

I was thinking of the published (and soon-to-be-published) talks.


Allow for special chars in password

When using special-ish characters, ><-, etc, in your password, the string would get garbled and base64 decoding breaks on the test server with the following err: illegal base64 data at input byte...

Do you have a test case? I can't make this fail: http://play.golang.org/p/3DEIuNdvnU

I wrote this test case and can't get it to fail using code from lfs-test-server. Based on my understanding of base64 encoding, changing base64.URLEncoding shouldn't have any effect on what characters it can take

This was my error.  I was trying to decode a string encoded with base64.URLEncoding with base64.StdEncoding, which is why I was getting an error.  This seems to work with a standard string like asdf but not with asdf-><.  See this: http://play.golang.org/p/D2FmyaZrs8
The bottom line is that I should use the same method to decode as was used to encode.  Sorry to waste your time.

Interesting. My test case decodes with base64.StdEncoding, so I didn't think it'd be an issue. We may have to change the lfs test server.  Thanks for the test case! I'll investigate further.


Put together parts list for building our own camera

Create a parts list for Raspberry Pi based trailcam.

[Arducam Day-Night Vision for Raspberry Pi Camera, Automatic IR-Cut Switching All-Day Image All-Model Support, IR LED for Low Light and Night Vision, M12 Lens Interchangeable, OV5647 5MP 1080Phttps://
www.amazon.com/dp/B07X1VGQSL?ref=ppx_yo2ov_dt_b_fed_asin_title
CQRobot 10.525GHz Doppler Effect Microwave Motion Sensor Compatible with Raspberry Pi and Arduino Board. for Industrial, Measurement, Automatic Doors, Sensor Lights, Reversing Radar, etc.
https://www.amazon.com/dp/B089NKGWQQ?ref=ppx_yo2ov_dt_b_fed_asin_title
[Gikfun 2" 4Ohm 3W Full Range Audio Speaker Stereo Woofer Loudspeaker for Arduino (Pack of 2pcs) EK1725]
https://www.amazon.com/dp/B01CHYIU26?ref=ppx_yo2ov_dt_b_fed_asin_title
[Teyleten Robot AMG8833 8 * 8 IR Infrared Temperature Thermal Imager Array Camera Imaging Sensors Module I2C 3-5V for Raspberry Pi Arduino]
https://www.amazon.com/dp/B0C7KXHXTP?ref=ppx_yo2ov_dt_b_fed_asin_title

I'll reopen this issue so we can copy it into the webpage. @seblkoglin do you want to do that?

And thanks for all of the links! Makes this much easier :smile:


added applet solution sets

closes #57 #56
Lines (#56)
https://github.com/user-attachments/assets/2f3739ef-6f30-4ebf-9a8a-41253571eb41

[!TIP]
See applet: https://deploy-preview-248--playful-otter-9e0500.netlify.app/applet/solution_sets/two_lines_in_r3

Planes (#57)
https://github.com/user-attachments/assets/286c8151-a278-4323-8156-1471ffb2bfc7

[!TIP]
See applet: https://deploy-preview-248--playful-otter-9e0500.netlify.app/applet/solution_sets/two_planes_in_r3

For two lines

[ ] Make lines thicker
[ ] Add labels $L_1$ $L_2$
[ ] Add $L_2$:  in formula
[ ] Mathcal{L} for lines

For two plane

[ ] r_0 is unlabeled
[ ] P_1 P_2 for planes
[ ] And add P_2 to formula
[ ] P needs to be mathcal
[ ] Make a = 7
[ ] Move camera to something like this:


[Bug] Navigator doesn't save the locator position when closing the app in background

Describe the bug
I am using the navigator.currentLocator to get an update of where the position in the book is.
When the app is in the foreground and I'm using TTS, it all works fine - the currentLocator is updated as the TTS engine reads the text.
However, when I put the app in background, all the callbacks from the EpubNavigatorFragment.Listener and EpubNavigatorFragment.PaginationListener stop being called when the app is in the background (there is a service for the TTS to keep reading the book).
So, even though the text keeps being read with the app in the background, the navigator doesn't update the position in the book.
I checked and it seems that's because the navigator is tied to the lifecycle of the view, so when I put the app in the background, the fragment goes to the paused state and so does the navigator.
It seems that the position stored in the navigator is the position when the app was put in the background. So, if the book was on position 5 and I put the app in the background and let the book be read until page 7 when I killed the app, when I reopened it the book would still open on page 5.
I would like to know if there's a way to keep updating the navigator so that I can get the updated locator even if the app is in the background.
How to reproduce?

Open any book
Activate TTS
Let the TTS start reading the book
Put the app in the background while the text is still being read
Let the book be read for some time (making sure it has changed to the next page)
Kill the app
Reopen the app, go back to the book
The book will reopen in the position when the app is put in the background, not in the position where the TTS engine stops.

Readium version
2.4.0
Android API version
13
Additional context
No response

This is the expected behavior. When using the TTS, you should take the current locator from the TtsNavigator.
See the test-app.

@qnga I understand. However, I still haven't been able to find a way to save this location while the app is in the background. I see that the test app is in Readium 3.0.0, I'm currently using 2.3.0, is there any example for this in 2.3.0? Thanks

I'm not sure. The TTS engine available in 2.3 does not properly support background playing anyway (Android is likely to kill your app). Upgrade to 2.4.x, at least if this is something you care about.


Update exome release code to account for changes in schema

Changed in https://github.com/broadinstitute/gnomad_qc/pull/477/


ember install ember-router-scroll not installing 0.3.0

I feel like a total nube here, but this has never happened to me before. I just moments ago ran ember install for this add-on. I am in need of the new feature "scrollElement". The code that npm is pulling to my machine does not have this feature. The addon/index.js file that is on my machine does not look like what is in master for 0.3.0. It seems that I am getting the previous version.

Ah, this was an oversight on my part.
Can you try grabbing 0.3.1 which was just published?

No worries. I was kind of hoping to get schooled on the finer details of npm dependency specifics. The add-on works well for us, by the way. Thanks for publishing it.
Get Outlook for Androidhttps://aka.ms/ghei36
On Mon, Sep 11, 2017 at 4:13 PM -0500, "Brian Gonzalez" <notifications<EMAIL_ADDRESS>wrote:
WARNING E-MAIL FROM OUTSIDE ULTRAX NETWORK
Closed #71https://github.com/dollarshaveclub/ember-router-scroll/issues/71.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/dollarshaveclub/ember-router-scroll/issues/71#event-1244130903, or mute the threadhttps://github.com/notifications/unsubscribe-auth/APzBH2HjSQIV1cTIhARCrcpvhuOQzjx8ks5shaJZgaJpZM4PTvAG.
This e-mail (including any attachments) may contain material that (1) is confidential and for the sole use of the intended recipient, (2) is proprietary and may be protected by a Non-Disclosure Agreement (NDA), mutual NDA or other agreement, and (3) is restricted from export by either the International Traffic in Arms Regulations (ITAR) or the Export Administration Regulations (EAR). Violations of these export laws are subject to severe civil and criminal penalties. Any review, use, dissemination, distribution, or copying by an unintended recipient, including access by any U.S. foreign national regardless of their location without prior approval from the appropriate U.S. export control authority, is strictly prohibited. If you are not the intended recipient, please notify sender with reply email and destroy all copies of the original message and enclosed attachments.

Thanks, and sorry for the troubles!


Content and Implementation for Energy Saving Tips

It'd be great if someone could take the general design pattern that I put in https://github.com/codeforboston/cambridge_energy_app/commit/6939c00aa45acb807032cc09acbd86161426545d and expand it so that a user receives a random tip in this box when they login. If you are feeling even more ambitious it'd be great if we could keep track of what tips a user says they have taken when they click a button so that we do not show them that tip anymore. And/or maybe it should be a voting tool where they can say "this worked for me" or "this didn't work for me" so we get a sense of what is working. I've attached sample content from the City of Cambridge we can use.
Energy House Handout Final 8-25-15-1.pdf

Pull #185 puts in some implementation for tips. Still missing: sharing, periodic reminders. Also, how "daily" are these "tips of the day"? The current setup I have just grabs a random one each time because we get desync issues with the next-tip and don't-show-again options.
Possible future feature: currently, liked tips are still shown again; maybe don't show again only after sharing? Or should it simply not be shown again either way?


sync squid from current stable - Lincoln's pod resource requests updates

@fengpinghu, feel free to squash and merge.


Closed tabs still showing in Sidewise

When I close a tab in Chrome, Sidewise still lists it as an open tab.  Happening consistently.
Chrome Version 39.0.2171.95 (64-bit) on Mac OS 10.10.1.

I'm not sure if this is a feature or a bug, but not being able to choose the behaviour is annoying.

Agreed, very annoying.

SideWise might as well be abandonware FFS.

Have you activated the setting enable-fast-unload in chrome? I found that this setting was incompatible with Sidewise. See #70


~💚<2024>💚LaTeSt!!**Free tiktok Follower Generator 2024 🎮 Up to Likes& Follower in UNDER 5 minutes!

25 secs ago - Tik Tok Followers Free - Free Tiktok Likes app fast fans followers Tik Tok Followers Free - Free Tiktok Likes app fast fans followers TIK-TOK can help you to get the very first fans for the TIK-TOK videos, you will get to the key page and attract more followers.
Latest Update 13 Jun 2024
CLICK HERE TO GET FREE NOW
CLICK HERE TO GET FREE NOW
The TikTok Followers generator for TikTok Followers will generate 6 different squares. Each square will show its own TikTok Followers item. As .. Free TikTok followers are a great way to grow your account on TikTok. Try our free followers service and increase your visibility.
Free Tiktok Follower (** 99000 free tiktok followers) - Tik Tok Followers Free - TikTok Followers Generator (2024) You'll reach NursingCenter in much less time. Free Tiktok Likes app fast fans followers TIK-TOK can help you to get the very first fans for the TIK-TOK videos, you will get to the key page and attract more followers.
Free TikTok Followers and Likes Now
Free Tiktok Followers Generator 2024 No Human Verification. Tiktok Free Followers Generator Unlimited Free Tiktok Followers.Free Tiktok Likes 2024.foundation/motion-ui#1 TikTok Followers Generator! Grow your TikTok account with FREE and REAL followers NOW!Free TikTok Followers, Fans and Likes. Get up to 20k TikTok Followers for Free daily, all you need is your NursingCentername. Join us and be famous Get Real Followers for more than 3 TikTok accounts.
hit 10,000 TikTok followers today. Normally, I don’t make a big deal out of this stuff. But, this feels like quite an accomplishment given the dramatic shift I’ve made.
It would be safe to say that six months ago, this would have been laughable. I wasn’t active on TikTok, and I detested everything about it (and short-form video generally). But, by the end of September, I put my old man shoes away and realized that this was something I needed to do.
When I committed to it, I made a conscious decision to become the most followed account related to the topic of Facebook ads on TikTok. I’m obviously not there yet, but this is a nice milestone.
What I did was simple and complicated at the same time. It really comes down to two things…
Have questions about short-form videos? Ask my AI-powered bot!
Some Ads Early
might take more and charge more and that is tiktok ads you should try this tip once having a popular video of yours use tiktok ads to make the video spreads much more and reach millions it might cost some money but it is 100 worthwhile besides getting 1000 free tiktok followers like what has been explained in promotion and make money you already knew that the more tiktok followers you have the more money you will make this section will give you a clue on how to get promotions on tiktok and how much you will make normally tiktok followers generator 2023 you ll reach fame in much less time free tiktok followers generator 2023 no human verification tiktok free followers generator unlimited free tiktok followers free tiktok likes 2023 tiktok followers generator grow your tiktok account with free and real followers now free tiktok followers fans and likes get up to 20k tiktok followers for free daily all you need is your username join us and be famous get real followers for more than 3 tiktok accounts rejos UPDATED FREE TIKTOK FOLLOWERS GENERATOR 2023 of external sources to get tiktok views at the start once the tiktok views start coming because of the uptake of the external sourced views the profile will soon get organic views hits and likes which will put the user on top free tiktok follower 99000 free tiktok followers
25 secs ago - Tik Tok Followers Free - Free Tiktok Likes app fast fans followers Tik Tok Followers Free - Free Tiktok Likes app fast fans followers TIK-TOK can help you to get the very first fans for the TIK-TOK videos, you will get to the key page and attract more followers.
I hit 10,000 TikTok followers today. Normally, I don’t make a big deal out of this stuff. But, this feels like quite an accomplishment given the dramatic shift I’ve made.
It would be safe to say that six months ago, this would have been laughable. I wasn’t active on TikTok, and I detested everything about it (and short-form video generally). But, by the end of September, I put my old man shoes away and realized that this was something I needed to do.
When I committed to it, I made a conscious decision to become the most followed account related to the topic of Facebook ads on TikTok. I’m obviously not there yet, but this is a nice milestone.
What I did was simple and complicated at the same time. It really comes down to two things…
Have questions about short-form videos? Ask my AI-powered bot!
Some Ads Early
might take more and charge more and that is tiktok ads you should try this tip once having a popular video of yours use tiktok ads to make the video spreads much more and reach millions it might cost some money but it is 100 worthwhile besides getting 1000 free tiktok followers like what has been explained in promotion and make money you already knew that the more tiktok followers you have the more money you will make this section will give you a clue on how to get promotions on tiktok and how much you will make normally tiktok followers generator 2023 you ll reach fame in much less time free tiktok followers generator 2023 no human verification tiktok free followers generator unlimited free tiktok followers free tiktok likes 2023 tiktok followers generator grow your tiktok account with free and real followers now free tiktok followers fans and likes get up to 20k tiktok followers for free daily all you need is your username join us and be famous get real followers for more than 3 tiktok accounts rejos UPDATED FREE TIKTOK FOLLOWERS GENERATOR 2023 of external sources to get tiktok views at the start once the tiktok views start coming because of the uptake of the external sourced views the profile will soon get organic views hits and likes which will put the user on top free tiktok follower 99000 free tiktok followers
The toughest part of getting started is publishing videos into the void. Maybe it was to give me a psychological boost or to make my profile look a bit more desirable in the early going, but I ran a low level of TikTok ads during the first couple of months.
From the time I committed to creating videos on TikTok in October through the middle of December, I collected 3,253 followers via ads. It was a nice foundation.
This was good and bad.
The bad: I tried really hard to target precisely to build my audience with relevant people. All indications are that this wasn’t successful. It was mostly numbers padding (the relevant followers would come organically).
The good: We can knock vanity metrics all we want, but it does mean something on TikTok if someone has a following. It surely helped convince some relevant people that I was worth following.
Did this lead to a significant spike in engagement during those first few months? Not really. But I do think that it helped me reach the next step.
A Dedicated Commitment to Create
Look, I could run ads forever on TikTok, but unless I was actually creating good content, it would not matter. I dedicated myself early.
I would create bad videos in the beginning, there was no avoiding that. But the goal was to get myself out of that phase as quickly as possible. I created 70 videos in October. It was a difficult, challenging, and inefficient month of video creation.
But it was worth it. I started to figure out my process. I found a workflow. Needs like software and lighting became clearer to me.
I dedicated myself to publishing at least one video per day in 2023. Truthfully, I missed very few days once I started in October, but I wanted to avoid any days off going forward.
I’ve kept to that commitment and I’ve published more than 260 videos on TikTok. Consistency, volume, and quality. More than anything else, I’d say that these were the keys to my growth.
What I Didn’t Do
I didn’t try to go viral. I didn’t worry about trends or popular sounds. Trust me, you’re going to hear a lot of advice around that stuff.
But it’s wrong.
If all you care about is being popular, sure. Chase that stuff. But you’re going to attract a very general, random audience. And that’s not how you build an engaged following around a specialty.
While I haven’t completely niched down on TikTok (I talk about Facebook ads, video creation, and entrepreneurship), it’s at least a similar category. I don’t want to attract randos who don’t care about any of these things.
The result: Engagement is growing with the audience size.
Tracking Growth
TikTok only offers analytics back 60 days, as far as I can tell. But even within the past two months, the improvement really jumps out.
Those spikes are when some videos took off. I have three recent videos that received more than 30,000 views (all organic).
Those three videos alone resulted in 2,942 new followers.
But it’s not just those three. Here’s a look at the trend of views during the past 60 days…
Even during the end of January, most days I’d receive very few views. But then something started to change. The floor raised.
Engagement is no longer focused on a few videos. I can now expect that my recent videos will almost always get at least 1,000 views. That was definitely not the case before.
That doesn’t mean that this new engagement level will continue forever, but it’s certainly a good sign.!

Does this issue occur when all extensions are disabled?:


Fix various small issues revealed by staticcheck

This covers most of #2843, with the exception of the obsolete http client used in the Docker proxy.
I sense that is either done that way on purpose or needs extra thought to update.

LGTM: good improvements to have merged in IMO.

This PR aside, we already do go vet in lint, but would it be worth adding staticcheck too/instead?

Moved discussion to https://github.com/weaveworks/build-tools/issues/82


bump error-chain to 0.11.0

derive-error-chain: proc macro error_chain has been renamed to ErrorChain.

Thank you for your contribution, Paul! Merging 💯


Kotlin Boolean values are serialised to 0 and 1 (Int)

Describe the bug
We are building a Kotlin multiplatform library and include the iOS tracker via cocoapods. Whenever we send Boolean values they are serialised into 0 and 1 as Integer instead of true or false.
To Reproduce
Embed Snowplow as Cocoapod:
 cocoapods {
        pod("SnowplowTracker") {
            version = "~> 1.7.1"
        }
    }

And track an event:
val event = SPUnstructured.build { builderProtocol ->
    builderProtocol?.setEventData(eventData.selfDescribing)
}
tracker?.trackUnstructuredEvent(event)

Expected behavior
Serialised values should be true or false
Additional context
This problem occurs only for the iOS tracker, Android tracker does serialise properly.

Hi @tassilo-posegga
v1.7.1 is an old major version, you could try with the last v2 but I don't think it would change the behaviour.
The reason is that in ObjC the boolean objects are represented by NSNumber, which represents also numbers.
NSJSONSerialization will treat as boolean the NSNumber created as [NSNumber numberWithBool:YES], and will treat as number all the other NSNumber objects.
In this case, Kotlin multiplatform converts the boolean to NSNumber and ObjC can't distinguish between the two.
Unfortunately, the issue fall between the two platforms at a very low level that makes a fix in the tracker more complex.
I noticed that the problem has been already discussed in some other repositories.
Here an example with a workaround: https://youtrack.jetbrains.com/issue/KT-45781

This should be addressed now that we rewrote the tracker in Swift in version 5 – the tracker no longer uses NSNumber but actual Swift types.


Improve aircraft list

Codecov Report

Merging #726 (d1c8367) into main (6397117) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##               main     #726   +/-   ##
=========================================
  Coverage     92.89%   92.89%
  Complexity     2580     2580
=========================================
  Files           281      281
  Lines          5110     5110
  Branches        528      528
=========================================
  Hits           4747     4747
  Misses          234      234
  Partials        129      129

Impacted Files
Coverage Δ

...ain/java/net/datafaker/idnumbers/KoKrIdNumber.java
81.25% <0.00%> (-6.25%)
:arrow_down:

...in/java/net/datafaker/providers/base/Locality.java
64.81% <0.00%> (+1.85%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Hovering mouse on Venmo button highlights both buttons

We should only highlight the button which is being hovered

Fixed and pushed.


Support usage in Deno with ESM

Is your feature request related to a problem? Please describe.
I want to use this module in Deno however due to the package.json listing "browser": "src/browser.js", which is expected to be an ESM module and is unavailable Deno fails to import.
Describe the solution you'd like
Support ESM bundles properly
Describe alternatives you've considered
I have tried importing the compiled module from UNPKG at https://www.unpkg.com/browse/<EMAIL_ADDRESS>however Deno fails to resolve the exports here as well.

Hi @jthegedus,
Unfortunately it won't work with Deno right now (neither @grpc/grpc-js nor protobufjs are currently compatible with Deno, and both of them are our dependencies). We are thinking of how we can make it work but I cannot give you any ETA at this moment.


Unable to render course

Attempting to run this using the WSL on Windows 10.
2020-03-27T20:37:37.533Z [info]: Image already exists. Skip downloading /mnt/c/Users/Trevor.Rydalch/Videos/Udacity/React v5.0.0/img/react-classroom.jpg
2020-03-27T20:37:37.568Z [info]: Completed rendering Nanodegree summary file /mnt/c/Users/Trevor.Rydalch/Videos/Udacity/React v5.0.0/index.html
2020-03-27T20:37:37.571Z [info]: ____________________

2020-03-27T20:37:37.608Z [info]: Completed rendering lesson summary file /mnt/c/Users/Trevor.Rydalch/Videos/Udacity/React v5.0.0/Part 01-Module 01-Lesson 01_Welcome to the React Nanodegree Program/index.html
2020-03-27T20:37:37.609Z [info]: ____________________

2020-03-27T20:37:37.639Z [info]: Completed rendering lesson file /mnt/c/Users/Trevor.Rydalch/Videos/Udacity/React v5.0.0/Part 01-Module 01-Lesson 01_Welcome to the React Nanodegree Program/01. Rating Your Career Confidence.html
2020-03-27T20:37:37.640Z [info]: ____________________

✖ Getting Youtube video id GONrgnKbuxg information
2020-03-27T20:37:37.677Z [error]: ERROR: Command failed with exit code 127: /root/.nvm/versions/node/v13.5.0/lib/node_modules/udacimak/node_modules/youtube-dl/bin/youtube-dl --dump-json -f best -f=22/18/17 http://www.youtube.com/watch?v=GONrgnKbuxg
STACK: Error: Command failed with exit code 127: /root/.nvm/versions/node/v13.5.0/lib/node_modules/udacimak/node_modules/youtube-dl/bin/youtube-dl --dump-json -f best -f=22/18/17 http://www.youtube.com/watch?v=GONrgnKbuxg
    at makeError (/root/.nvm/versions/node/v13.5.0/lib/node_modules/udacimak/node_modules/execa/lib/error.js:56:11)
    at handlePromise (/root/.nvm/versions/node/v13.5.0/lib/node_modules/udacimak/node_modules/execa/index.js:114:26)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)```

download youtube-dl.exe from here http://ytdl-org.github.io/youtube-dl/
then replace it

I ended up doing the download on my mac instead and the renders are running.
It times out quite often, but then I just restart it and it keeps going.
From: Ahmed Hany<EMAIL_ADDRESS>Reply-To: udacimak/udacimak<EMAIL_ADDRESS>Date: Friday, March 27, 2020 at 3:49 PM
To: udacimak/udacimak<EMAIL_ADDRESS>Cc: trydalch<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [udacimak/udacimak] Unable to render course (#93)
download youtube-dl.exe from here http://ytdl-org.github.io/youtube-dl/
then replace it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/udacimak/udacimak/issues/93#issuecomment-605329680, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEDOJAUBWV5TPAJI75X3YRDRJUNPRANCNFSM4LVI2YYA.

Hi, I have the same issue in macos 10.15.4. I think is related to youtube-dl version.

My issue was with OpenSSL. I manage to fix it following this: https://stackoverflow.com/questions/59269208/errorrootcode-for-hash-md5-was-not-found-when-using-any-hg-mercurial-command

https://github.com/udacimak/udacimak/issues/100#issuecomment-606465183

There's an update v1.6.0 with fix for this issue you can follow the steps to get it working for you.

Uninstall
npm uninstall -g udacimak
Install
npm install -g udacimak


vector serialization does not take into account endianness

The gRPC protocol is defined as sending over the bytes of arrays directly, but does not take into account endianness. This means the protocol will fail spectacularly (type 3 error, bad data with no error) if the client and server machines have different native endianness.
https://github.com/ansys/api-eigen-example/blob/108085fcaa860a9d1e27abbdc561c160a98e09e3/src/ansys/eigen/cpp/grpc/client/src/GRPCClient.cpp#L499
https://github.com/ansys/api-eigen-example/blob/108085fcaa860a9d1e27abbdc561c160a98e09e3/src/ansys/eigen/python/grpc/client.py#L280
Unless there is another reason, I recommend using gRPC arrays instead of bytes. gRPC can handle this for us.

Yes, I've had to deal with this kind of problems in the past... machine endianness is a good point to take into account. We wanted to test serialization/deserialization of data in this example. But I guess we didn't take that into account. We could probably send the machine's endianness as part of the metadata.
The use of bytes was purposefully intended, since what we usually transmit is chunks of bytes from one machine to another. Using arrays would have been the best solution indeed for this case-scenario, but we wanted to show an example of serialization/deserialization.
@akaszynski do you think we should look into changing the nature of the data from bytes to an array? If so, I'll take care of the implementation.
Your feedback is much appreciated @phxnsharp! The machine endianness was a very good point.


Login on failure with Nexus with basic auth

Hello,
I have a Nexus repository with basic auth enabled (not anonymous pull). I see two issues when pulling:

Login fails even with docker/config.json having credentials properly set
oras-py expects realm of WWW-authenticate to be an URL which is not what Nexus does

Note that pulling the same artifact with Go oras works fine.
I have a doubt on how basic auth is handled in provider.py when using config.json:

In _load_auth if a matching token is found, it sets _basic_auth but does not change the headers like set_basic_auth does. Shouldn't it be:

            # Case 2: no auth there (wonky file)
            elif not auth:
                return False
            self._basic_auth = auth
            self.set_header("Authorization", "Basic %s" % self._basic_auth)# <<< add
            return True

get_manifest does not load self.header when doing the query while a lot of other methods do. Shouldn't it be something like:

        headers = {"Accept": ";".join(allowed_media_type)}
        headers.update(self.headers) # <<< add
        url = f"{self.prefix}://{container.get_manifest_url()}"  # type: ignore
        response = self.do_request(url, "GET", headers=headers)

If I make these two changes, it works but I'm not sure whether this is entirely correct. If it is, I can submit an MR.
*** .venv/lib/python3.10/site-packages/oras/provider.py     2023-03-24 18:54:38.717625157 +0100
--- provider.py 2023-03-24 19:00:13.727639413 +0100
***************
*** 110,115 ****
--- 110,116 ----
              elif not auth:
                  return False
              self._basic_auth = auth
+             self.set_header("Authorization", "Basic %s" % self._basic_auth)
              return True
          return False

***************
*** 694,699 ****
--- 695,701 ----
          if not allowed_media_type:
              allowed_media_type = [oras.defaults.default_manifest_media_type]
          headers = {"Accept": ";".join(allowed_media_type)}
+         headers.update(self.headers)
          url = f"{self.prefix}://{container.get_manifest_url()}"  # type: ignore
          response = self.do_request(url, "GET", headers=headers)
          self._check_200_response(response)

If you'd like to debug against nexus and submit a pull request with changes, this would be appreciated! I don't have access so I wouldn't be able to make changes and then test them. Thanks!

Done


Package request: betterbird

Project description
betterbird is a soft fork of thunderbird that includes bug fixes for bugs that mozilla refuses to fix.
Metadata

homepage URL: https://www.betterbird.eu/
source URL: https://github.com/Betterbird/thunderbird-patches
license: MPL2
platforms: unix, linux, darwin

hmm, it didn't show up when i searched for it. Maybe i did something wrong, sorry.


feat: Add nix flake

Add a nix flake build to this repo

Is there any update to this? The lean-toolchain file has stabilized and we can probably use something like
    lean = {
      url = "github:leanprover/lean4?ref=v4.4.0";
    };

as a flake input

Update: Lean's official flake build is deprecated https://github.com/leanprover/lean4/pull/4895
I might create a Lean4 nix repo sometimes later to replace this, but I don't think having the user handle the build themselves is very difficult, e.g.
      test = leanPkgs.buildLeanPackage {
        name = "Test";
        roots = [ "Test.Main" ];
        deps = [ lspecLib repl ];
        src = pkgs.lib.cleanSource (pkgs.lib.cleanSourceWith {
          src = ./.;
          filter = path: type:
            !(pkgs.lib.hasInfix "Pantograph" path);
        });
      };


Remove organisations from Rummager payload

Why?:
Rummager has recently been updated to fetch links data directly from the
publishing API when indexing a document. Setting links data in the
publishing app is therefore redundant.

👍
let's not merge until we're actually indexing links from publishing-api though.


too slow when pop from the scanner page

Hi Thanks for providing this awesome component,  I already implemented this into our project, it is working! however we met a problem when running the app on android device, it is too slow when call " this.props.navigator.pop();"  on the scanner page. is there any method to improve the performance? thanks very much. will attach some screenshots for your information. the fps is very low.

Best Regards & Thanks
Emily

@andreaskeller can you please help to have a look at this problem? thanks very much!

@222xiaohuan you can use a view to fix it. like this
let content   if (this.state.backing) {      content = <View style={{flex:1,backgroundColor:'rgba(0,0,0,0.6)'}}/>  } else {      content = <BarcodeScanner             onBarCodeRead={this._onBarCodeRead}             style={{height: screenHeight,flex:1}}             torchMode={'off'}             cameraType={'back'}         />  }
onPress back you can change state on backing and then setTimeout to pop this scene.

Does anyone have a better solution?

I am doing almost the same… displaying a black screen with a loading animation. And then I replace it with the actual camera with
InteractionManager.runAfterInteractions(() => {
  // Start scanner preview as soon as page transitions have finished
  this.enable();
});

I'll give it a try, thanks @namxam.

@222xiaohuan 我也遇到了同样的问题，你现在使用什么解决方法，可以告诉我吗？

@222xiaohuan  @Cocoon-break 当此问题出现之后，我同时再使用其他camera相关的插件，例如react-native-image-picker，这时候react-native-image-picker也会变得很慢，我就得这可能和资源没有释放干净有关。

@liuyimx 你解决了吗？我还没有解决，上面的方法你试了没，我一会试试看。这个性能老大完全不能接受T_T

@222xiaohuan 上面的方法我试了，先给个loading页面然后pop，视觉上pop没有卡顿，但是使用了此插件还是会导致react-native-image-picker及整个app变慢，另外 #47 问题依然存在。最近有个新插件叫 react-native-barcode-scanner-google，我试过貌似还不能用。现在我们正在改这个插件希望可以解决android的扫码问题。

@liuyimx 是在该现在这个插件，还是在改react-native-barcode-scanner-google这个？方便的话加下我qq435477171

@Cocoon-break your method works, thanks :)

@Cocoon-break Your method will crash the app, and throw exception:

java.lang.RuntimeException: Camera is being used after Camera.release() was called

BarcodeScannerManager add stopCamera method expose to js:
@Nullable
    @Override
    public Map<String, Integer> getCommandsMap() {
        return MapBuilder.of("stopCamera", COMMAND_STOP_CAMERA);
    }

    @Override
    public void receiveCommand(BarcodeScannerView root, int commandId, @Nullable ReadableArray args) {
        if (commandId == COMMAND_STOP_CAMERA) {
            root.stopCamera();
        }
    }

BarcodeScannerView JS:
stopCamera() {
        UIManager.dispatchViewManagerCommand(
            findNodeHandle(this),
            UIManager.RNBarcodeScannerView.Commands.stopCamera,
            [],
        );
    }

We can use it like this:
componentWillMount() {
        InteractionManager.runAfterInteractions(() => {
            this.setState({
                showOverlay: false,
                showBarcodeScanner: true,
            });
        });
    }

_onBackHandler = () => {
        this.setState({
            showOverlay: true
        }, () => {
            this.scanner.stopCamera();
            this.timeout = setTimeout(() => {
                this.props.router.pop();
            }, 1000);
        });

        return true;
    };

@someok thanks, nice job, I finally added some release and start methods in module to solve this problem, will try your method later.

@222xiaohuan 你解决了这个问题么？我也遇到了同样的问题，我尝试使用了someok的方法但是没作用。 如果你解决了可以告知一下吗？谢谢！

@someok could you please help us figure out where this code should go.
    @Override
    public Map<String, Integer> getCommandsMap() {
        return MapBuilder.of("stopCamera", COMMAND_STOP_CAMERA);
    }

    @Override
    public void receiveCommand(BarcodeScannerView root, int commandId, @Nullable ReadableArray args) {
        if (commandId == COMMAND_STOP_CAMERA) {
            root.stopCamera();
        }
    }```

@Cocoon-break I have do it like you on some device(Galaxy S7 Edge) will get error, have a black screen in the bottom, all animation is slow down.

@someok where the variable ReadableArray?


add check befor fencing

Resolve the issur : Should not shutting down a server which is already shut. Thanks for DaanHoogland which provide code.
Description
This PR...

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI
[ ] test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@blueorangutan package

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✖ el8 ✖ el9 ✖ debian ✖ suse15. SL-JID 11743

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11748


[BUG] Match_phrase_prefix query on a text field with setting index_prefixes works not well

Describe the bug
When executing match_pharse_prefix query on a text field with multiple values and with setting index_prefixed, the documents which should be hit are not returned, but if the search keyword hits the first element in the text field, the document will be returned.
To Reproduce
Steps to reproduce the behavior:

Create a index with a text field which set index_prefixes using default properties.

PUT test11
{
  "mappings": {
    "properties": {
      "platform_name": {
        "type": "text",
        "index_prefixes": {}
      },
      "applications": {
        "properties": {
          "name": {
            "type": "text",
            "index_prefixes": {}
          },
          "app_version": {
            "type": "keyword"
          },
          "publisher": {
            "type": "text"
          }
        }
      }
    }
  }
}

Add a doc

POST test11/_doc/3?refresh
{
  "platform_name": "Red Hat",
	"applications.name": ["python-libs","bind-libs"]
}

Search the document, nothing will be returned

GET test11/_search
{
  "query": {
    "match_phrase_prefix": {
      "applications.name": "bind-libs"
    }
  }
}

Add another doc, change the order of elements in the field applications.name, execute the query in step 3, this document will be returned.

POST test11/_doc/3?refresh
{
  "platform_name": "Red Hat",
	"applications.name": ["bind-libs","python-libs"]
}

Expected behavior
Return the document not matter the order of the elements in the text field when executing match_phrase_prefix query.
In addition, match_phrase query works well in this case.
Host/Environment (please complete the following information):

OS: [macOS]
Version [2.9]

Duplicate of: https://github.com/opensearch-project/OpenSearch/issues/9203


Android setupLocalVideo SurfaceView导致黑屏 本地Video无效

"react": "16.6.3"
"react-native": "0.57.8"
agora SDK 2.3.1
安卓8.1
安卓7.0
测试

Object created on the libs of Agoraio is conflicing with RN #react-native delhi

@Rohan5696 any details?


Create 3. Longest Substring Without Repeating Characters.cpp

You can follow me on github and give ⭐ to this repo.


laravel11支持

我看文档中没有写laravel版本限制，但是实际安装过程中发现laravel11里面无法安装.
Do not run Composer as root/super user! See https://getcomposer.org/root for details
Continue as root/super user [yes]? yes
./composer.json has been updated
Running composer update mckue/laravel-excel
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - mckue/laravel-excel[v1.0.0, ..., v1.1.0] require illuminate/support 5.8.*|^6.0|^7.0|^8.0|^9.0|^10.0 -> found illuminate/support[v5.8.0, ..., v5.8.36, v6.0.0, ..., v6.20.44, v7.0.0, ..., v7.30.6, v8.0.0, ..., v8.83.27, v9.0.0, ..., v9.52.16, v10.0.0, ..., v10.48.14] but these were not loaded, likely because it conflicts with another require.
    - Root composer.json requires mckue/laravel-excel * -> satisfiable by mckue/laravel-excel[v1.0.0, ..., v1.1.0].

You can also try re-running composer require with an explicit version constraint, e.g. "composer require mckue/laravel-excel:*" to figure out if any version is installable, or "composer require mckue/laravel-excel:^2.1" if you know which you need.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

这个问题等我抽时间处理一下


kics: 2.1.2 -> 2.1.3

Changelog: https://github.com/Checkmarx/kics/releases/tag/v2.1.3

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

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

1 package built:

kics


[LabelRadio] radio button circle appears at  the top of select popup

Material-UI: 1.0.0-alpha.19
React: 15.5.4
Browser: Chrome
I usу react-select together with LabelRadio.

But when select is opened svg element of radio button appears at the top of the select popup

Would removing the zIndex help? I don't know why it was added in the first place.


Add the batch_norm op

The main reason for adding this op is to resolve the torch memory issue.
This allows torch to use the fused op for batch norm, which saves memory.
Benchmarking
Measured by averaging 100 steps.

backend
batch size
fit (ms/step)
predict (ms/step)
link

JAX  (before)
128
459
178
link

JAX  (after)
128
456
171
link

Torch (before)
64
397
141
link

Torch (after)
64
321
126
link

Torch (before)
98
OOM

link

Torch (after)
98
Passed

link

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

Comparison is base (66bb075) 78.97% compared to head (d264733) 64.18%.
 Consider uploading reports for the commit f14876e to get more accurate results

Files
Patch %
Lines

keras/backend/jax/nn.py
7.14%
13 Missing :warning:

keras/backend/tensorflow/nn.py
9.09%
10 Missing :warning:

keras/ops/nn.py
43.75%
8 Missing and 1 partial :warning:

keras/backend/torch/nn.py
94.73%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #18793       +/-   ##
===========================================
- Coverage   78.97%   64.18%   -14.79%
===========================================
  Files         336      336
  Lines       34040    34234      +194
  Branches     6668     6703       +35
===========================================
- Hits        26883    21974     -4909
- Misses       5580    10799     +5219
+ Partials     1577     1461      -116

Flag
Coverage Δ

keras
64.18% <49.23%> (-14.68%)
:arrow_down:

keras-jax
?

keras-numpy
?

keras-tensorflow
?

keras-torch
64.18% <49.23%> (-0.16%)
:arrow_down:

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

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

Note that we should validate the final code via a benchmark on a convnet (with all backends). I'm not worried about the TF side, but the JAX and torch side need some numbers.

Note that we should validate the final code via a benchmark on a convnet (with all backends). I'm not worried about the TF side, but the JAX and torch side need some numbers.

Sure. I can benchmark different backends with a convnet using a commit before this change and a commit after this change to compare the numbers.

Numbers updated in the description.
We have speedups on torch and jax.
Memory saved on torch.


Link for 1.5.0 Linux binaries is broken

Linux binaries for 1.5.0 were available here
https://github.com/hyperledger/fabric-ca/releases/download/v1.5.0/hyperledger-fabric-ca-linux-amd64-1.5.0.tar.gz
This link is broken now

I think there was an issue with the binaries.  You should use the 1.5.1 binaries which are now available.


Not Able To Authenticate

When you click authenticate it just takes you to the tumblr home page

Nevermind...  Needed reblog checked


[FLINK-5884] [table] Integrate time indicators for Table API & SQL

This PR introduces time indicators in the Table & SQL API. The PR splits the current row type into a logical and physical row type. Time indicators are part of every logical row and stay logical if they are just forwarded and not accessed for calculation. If a time attribute is passed to a function or arithmetic expression it will be materialized. A ProcessFunction can access the meta timestamp for materialization (code generation for this has to be implemented). Every logical plan includes time attributes now and looks more consistent:
val table = util.addTable[(Long, Int, String)]('long.rowtime, 'int, 'string)

val windowedTable = table
  .window(Session withGap 7.milli on 'long as 'w)
  .groupBy('w, 'string)
  .select('string, 'int.count)

val expected = unaryNode(
  "DataStreamAggregate",
  streamTableNode(0),
  term("groupBy", "string"),
  term(
    "window",
    SessionGroupWindow(
      WindowReference("w"),
      'long,
      7.milli)),
  term("select", "string", "COUNT(int) AS TMP_0")
)

Main changes of the current solution:

Processing and rowtime time indicators can be named arbitrarily
They can be defined as follows: stream.toTable(tEnv, 'long, 'int, 'string, 'proctime.proctime) or stream.toTable(tEnv, 'long.rowtime, 'int, 'string)
In a streaming environment: if the "long" field is already defined in the record, it will not be read by the runtime. "long" always represents the timestamp of the row.
In batch environment: "long" must be present in the record and will be read by the runtime.
The table definition looks equivalent in both batch and streaming (better unification than current state)
Internally row types are split up in a logical and a physical row type.
The logical row type contains time indicators, the physical rowtime never contains time indicators (the pure "long" will never be in a record)
After validation and query decorrelation, a special time indicator converter traverses the RelNodes and analyzes if the a time indicator is accessed or only forwarded.
An access to a time indicator means that we need to materialize the rowtime using a ProcessFunction (not yet implemented). The timestamp (not an indicator anymore) becomes part of the physical row. E.g. long.cast(STRING) would require a materialization
Forwarding of time indicators does not materialize the rowtime. It remains a logical attribute. E.g. .select('long)
Windows are only valid if they work on time indicators.
A new RowSchema unifies logical and physical rows and tries to make type handling throughout the classes easier and consistent (it is not used everywhere yet).

Missing so far:

Java API tests for defining time attributes
Tests for TableSources that define time attributes
Multiwindow tests
ProcessFunction for accessing the timestamp
More tests in general

I didn't look at RelTimeIndicatorConverter yet.
Will do so tomorrow.

Hi @twalthr, I addressed my comments, rebased to current master, and will merge this PR given that all tests pass.
Thanks, Fabian


Readability enhancements

If you like the PR, feel free to adjust the colors / line height. Just trying to add some readability enhancements.

Add indents for bullet points in props documentation.
Add a line height to text so multi-line text is not squished together
Changed the code color for prop definitions and usage of the code tag throughout

Looks great. I like it very much.


Exclude spam tokens from Treasury page

Closes #1403
Only able to "excludeSpamToken" in this PR, not "only include trusted tokens".
For more context, see

https://github.com/decentdao/decent-interface/issues/1403#issuecomment-1976885673
https://github.com/decentdao/decent-interface/issues/1403#issuecomment-1976970672
https://github.com/decentdao/decent-interface/issues/1403#issuecomment-1976970672

@decentdao/engineering, please take a look at https://github.com/decentdao/decent-interface/issues/1267 and let me know if you have any ideas...

@tomstuart123 i'm going to merge this, considering your OOO for the rest of the week. it won't hurt anything, it just doesn't quite filter out all of the shitty tokens.

makes sense as a fix for now. Intrigued on getting caught up on 1267 ramifications


all variables are initally uncommented in _settings.scss?

The foundation doc says in order to make a change, variables must first be uncommented (http://foundation.zurb.com/apps/docs/#!/sass - "The Settings File" section Paragraph 3) Yet when I create a project, all of the variables inside the _settings.scss file are already uncommented?
If I were to comment any of them, that component becomes unusable. No default styling is available. Any ideas why?

Hi @yucelm,
I've commented the whole variables in scss/settings/settings.scss to be matched what is written in the documentation.
However, you should not face any problem either it's commented or uncommented, because the default variables should work instead!

Thanks. I am just checking it out.
I just wanna ask you another quick question if you don't mind.
In the app.scss file, the main foundation file is already imported at the top "@import 'foundation';"
When I comment out one of the includes however (i.e @include foundation-grid;) it breaks that component.
Aren't all components already included in the main foundation file anyways? Should it not work regardless even though I comment out the individual components?
Thanks a lot.

I understood that you are trying to @import the Foundation's component, but NOT @include its mixin, if you did so, that component will NOT work, because you only @import its FILE but didn't call its mixin, and this is a major difference between F5 and F6. In F5, you only need to @import the component's file but in F6 you need to @include its mixin too.

Hello,
Here is my app.scss and settings.scss files.

I see that is something I wasn't aware of. It is clear now.

I hope everything does well..
Good luck,

@yucelm So that when using a component the classes are not automatically added into the CSS and to remove the need for a variable to check if it should include the CSS at a global level and on a per component level (I think to clean up fragmentation).
If you want to include all the Foundation classes use @include foundation-everything();

@Knight-Yoshi It is super clear now. I haven't had enough time fully review all the changes in F6. Many thanks.

We made a last-minute call to uncomment every variable in the settings file, but the documentation wasn't updated to match. I updated it yesterday: 2d0daf9703a660679ef0c291a6380d8077add67b
The reason we're uncommenting every variable is to avoid reference errors. For example, let's say you left all the colors the same, so they all have comment marks. Now you try to reference $primary-color farther down, and you get an error, because $primary-color hasn't been defined yet. It is in the settings file, but it's not uncommented. Uncommenting every value gets around this issue.

Hi @gakimball,
I've just noticed the major difference between my own Build Flow and the other one you assumed, and before I go further in explaining that difference, I am going to share you few things of my projects that are based on Foundation.
I usually use Foundation as a base of my projects, then add my code and other third-parties. Therefore, that collection leads me to make my own custom build steps.
When I came to Foundation -F6 in particular-, I saw that its sass build walks through the following steps:
@import 'settings';
@import 'foundation';
@include foundation-everything;
Thus, that flow might causes a "reference errors" when one of the commented variable is called within the settings.scss.
It's great idea that the settings.scss works as a secondary configuration layer -current situation-, but when we use it in the previous order -before @import 'foundation'; - that will lead us to some challenges, "reference errors" one of them. In my opinion, that secondary configuration layer settings.scss should come after calling the initial components' settings @import 'foundation';, so all the variables that are written in the settings.scss should be already initialized in @import 'foundation'; step, thus, will not be any dependency inside the settings.scss itself.
The following picture abstracts the build steps that I do:

@import 'foundation';
@import 'settings';
@include foundation-everything;
Finally, I would really appreciate it if anyone has any concern or improvement regarding to the suggested build steps.
Enjoy F6 :)

@abdullahsalem The @import 'foundation'; loads all of Foundation's mixins. The _settings.scss file uses some of the mixins, such as rem-calc(). This would also be necessary if you used any of the Foundation mixins yourself in the _settings.scss file.
I don't see any point in loading the settings file before the main foundation file.

Thank you for sharing @Knight-Yoshi,
I am not attempting to use any of the components' mixins in settings.scss, and I strongly believe that settings.scss should be for configuration.
But as I previously mentioned:

when we use it in the previous order -before @import 'foundation'; - that will lead us to some challenges, "reference errors" one of them. In my opinion, that secondary configuration layer settings.scss should come after calling the initial components' settings @import 'foundation';, so all the variables that are written in the settings.scss should be already initialized in @import 'foundation'; step, thus, will not be any dependency inside the settings.scss itself.
To touch that issue, try to test @gakimball's example:

For example, let's say you left all the colors the same, so they all have comment marks. Now you try to reference $primary-color farther down, and you get an error, because $primary-color hasn't been defined yet. It is in the settings file, but it's not uncommented.
If you test that example with the default order:
@import 'settings';
@import 'foundation';
@include foundation-everything;

You will get this error:

But if you change the order to this:
@import 'foundation';
@import 'settings';
@include foundation-everything;

You will avoid that issue.
In fact, my concern goes beyond that ordering, I summarized it previously:

In my opinion, that secondary configuration layer settings.scss should come after calling the initial components' settings @import 'foundation';, so all the variables that are written in the settings.scss should be already initialized in @import 'foundation'; step, thus, will not be any dependency inside the settings.scss itself.

I've just noticed that some setting variables depend on each other, for instance
/// Default pip width for tooltips.
/// @type Number
$tooltip-pip-width: 0.75rem !default;

/// Default pip height for tooltips. This is helpful for calculating the distance of the tooltip from the tooltip word.
/// @type Number
$tooltip-pip-height: $tooltip-pip-width * 0.866 !default;

And if you would like to change the $tooltip-pip-width value in the settings.scss, according to my suggested order:
@import 'foundation';
@import 'settings';
@include foundation-everything;

You will need to re-configure all variables that use $tooltip-pip-width to take the new value, such as $tooltip-pip-height in the previous example.
Thanks for sharing guys..

The reason we imported the settings file before the framework in Foundation 5 was because the act of @importing something would print the CSS. So by the time you've called @import 'settings', the framework has already done its thing and printed the CSS with the default settings.
The new model in Foundation 6 was supposed to fix this, but we still ran into some issues and ended up keeping it how it was.

Thanks @gakimball , I do appreciate all what you have been doing on F6, it's great work.. :+1:

Hi. I have a quick question and any response is appreciated.
In my app.scss file there is  @import 'foundation'; at the top and then all other @includes are beneath that. When I comment out the @import 'foundation'; everything still works however, if I was to comment out one of the includes beneath that (@include foundation-button; for example) it stops working.
What I don't understand is that @include foundation-button; is also included in the @import 'foundation'; which is at the very top.
So should it not work anyways because it is still available in the file anyways? Why does is stop working?
Thanks in advance...


Fix the yaml panic

Fixes https://github.com/pulumi/pulumi-pulumiservice/issues/418

@komalali I tested this locally, confirmed fixes the panic and working with both computed/non-computed


[Bug] Does not work with Nx

Describe the bug
In Nx, you have a root .storybook directory but the actual storybook projects are nexted in "apps/" or "libs/". As soon as you run Storybook in an Nx repo, it looks for root/.storybook/tsconfig.json instead of, e.g. libs/my-lib/.storybook/tsconfig.json
Workaround
Create a .storybook/tsconfig.json or use the existing one as a proxy config
{
  "extends": "../packages/storybook/.storybook/tsconfig.json"
}

Currently the path the tsconfig which should be used is hard-coded in the constants.
I think the solution would be to replace the hard-coded constant by a plugin option
https://github.com/storybookjs/webpack-angular-types-plugin/blob/main/packages/storybook-webpack-angular-types-plugin/src/lib/webpack-angular-types-plugin/plugin.ts#L19
Would this resolve your problem?


The most allowing way to load external js library in web part

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

Expected or Desired Behavior
I have a js app which is quite big and has many dependencies and the loading is complicated, it injects dependencies into head tag.
Observed Behavior
The library itself non-AMD and i checked it here https://rencore.com/sharepoint-framework/script-check/
I tried to follow this article https://docs.microsoft.com/en-us/sharepoint/dev/spfx/web-parts/basics/add-an-external-library
which also didn't help much
Is there any way to add this script and let it to do the job without getting into its way?

What isn't working exactly?

On a ordinary web page library injects multiple dependencies scripts in head tag. When it's on a SPFx WebPart it breaks while doing this.

How are you trying to add scripts to the head tag exactly and what errors are you getting?

Closing due to no response.

@waldekmastykarz the script wants to interact with the script tag, developer of that script built some logic inside based on that there is  on the page. Loading script this wayhttps://docs.microsoft.com/en-us/sharepoint/dev/spfx/web-parts/guidance/use-existing-javascript-libraries doesn't seem to add script as script tag.

Have you tried adding the script tag to the page yourself?

Yes, finally I've injected the into the page and waited until it's loaded. Worry was about it could be unsupported way.

It's not recommended. Depending on what your script does, it might not work correctly when navigating between pages. Modern SharePoint UI uses partial page loading which might not work as expected with your script, so it's definitely something you should double check.


How do you use this cookbook?

Hey Noah,
Big fan of your blog, man.
I know this cookbook is in the middle of a rewrite.  I've been following posting and github issues for both this project and the application_ruby cookbook over the last month or so, and I'm still not seeing much about how to USE these new cookbooks.
Any ETA on when the project will be complete?   Do you need any help writing docs?
Regards,
Joe Reid

I wrote up a guide for poise-service when I announced it on the mailing list that will work here too (applies to all Halite cookbooks).
In your Gemfile:
gem 'halite', github: 'coderanger/halite'
gem 'poise', github: 'poise/poise'
gem 'poise-application', github: 'poise/application'
gem 'poise-boiler', github: 'poise/poise-boiler'
gem 'poise-service, github: 'poise/poise-service'

And then in your Berksfile:
extension 'halite'
cookbook 'application', gem: 'poise-application'
cookbook 'poise-service', gem: 'poise-service'

I've not put docs in each of the cookbooks as this will only be needed before I cut a formal release, at that point you will get them from Supermarket like normal. As for the actual resources, either check out the code or take a look at the integration tests. So far the only plugins that are done are Git and Ruby (sans Rails and Passenger).

Right on, thanks.  I'll start playing around with that locally in Test Kitchen.
Quick follow up:  Any rough ETA on the formal release?  I know it's hard to estimate work in progress.  But are you thinking it will be done by end of April?  End of June?  Just looking for a guesstimate.

I've got a storm of conferences for basically all of April (ChefConf and PyCon) so give or take reduced output during those I've got probably another 4 weeks of dev work left on Kickstarter-related stuff.


Remove cache

Please remove cache from this module. This is useless:
https://github.com/PrestaShop/ps_sharebuttons/blob/master/ps_sharebuttons.php#L145
There is nothing to cache, and because of complexity of cache key we have thousands more files of cache worth nothing...

@kpodemski Please create issue on the repo "PrestaShop/PrestaShop".
@PrestaShop/prestashop-product-team Can we close issues on this repository ?

Hi @Progi1984,
Once the 2 issues in the repo are transferred to /recreated in the PrestaShop/PrestaShop repo, you can close issues.


Check that replies feed is empty before calling getReplies

Fixes #1715.
Changes:

Check that the replies feed is empty before calling getReplies

Good catch thank you for this PR @jrawsthorne . CircleCi is failing, are you able to solve the issues: https://circleci.com/gh/busyorg/busy/4967 if not @jm90m can take a look on these

The problem is a blank username in the test so getFeedFromState gives an undefined error.
Could check with
const content = !_.isEmpty(username) && getFeedFromState('replies', username, feed);

Thanks for contributing.
It doesn't make much sense anymore to test this component as it became pretty connected.
Can you:

rename it from IReplies to just Replies (and don't expect unconnected class)
delete related test

We probably should add integration tests for those in the future.


feat(primitives): add Arrow icons and change PlusIcon

Purpose
feat(primitives): add Arrow icons and change PlusIcon
Related Issues

https://github.com/wso2/oxygen-ui/issues/253
https://github.com/wso2/oxygen-ui/issues/254

Checklist

[ ] UX/UI review done on the final implementation.
[ ] Story provided. (Add screenshots)
[ ] Manual test round performed and verified.
[ ] Unit tests provided. (Add links if there are any)
[ ] Documentation provided. (Add links if there are any)

Security checks

[ ] Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines?
[ ] Ran FindSecurityBugs plugin and verified report?
[ ] Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets?

We are following the guideline of adding icons with the line height as 1px. shall we make it consistent.


Add a configurable scale factor to scale in/out

This adds a scale-in and scale-out factor that can be used to speed up or slow down scale outs and ins.
For instance, setting SCALE_IN_FACTOR=0.5 will reduce the change applied in each scale in by 50%.  Setting SCALE_OUT_FACTOR=2.0 will increase each scale out change by 200%.
This is a simpler take on #12.

@etaoins would love to hear your thoughts!


Update README.md

Description of the bug
I want to work on updating readme.md according to codebase i will do it.
Update README.md Description: Review and update the project's README.md file to provide comprehensive documentation and instructions for developers and contributors for making contribution to the project.
Please add screenshots (if applicable)
No response
Add any other context about the problem here
No response
Code of Conduct

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

@Swapnil-2503 How would you improve existing Readme file? can you make some points?

@Swapnil-2503 How would you improve existing Readme file? can you make some points?

@Deepu178 sorry ,  i didnt find any ideas other than folder structure that i can include, thank you.


script: added seed-file

@lrt98802 I'm fine with having it here or putting it in a folder like seeds. If we choose the former, we can always move it into a new seeds folder when we need to do so at a later time.

I think the train of thought was that the seed scripts would be in the controllers folder, so it didnt make as much sense to have a seed folder with just the 1 file of the seed (unless we had the 3 seeds separately, but even then, since the scripts would be elsewhere, it would be extra folders)

I am fine with keeping it here as long as we move it together with the script in the future. Merging it now 😃


(events) Remove AnsibleFest Top Banner

The AnsibleFest is now over, so the top banner has been removed.
Work done via choco-theme:
https://github.com/chocolatey/choco-theme/pull/109/files

@designxsteph thanks for getting this fixed up!


Health checks with SERVICE_CHECK_SCRIPT doesn't work

Hi there,

What version of docker are you running?
Docker version 18.09.7, build 2d0083d
What version of registrator are you running?
v7
What version of Consul are you running?
Consul v1.6.1
Did you build a custom version of registrator? If so, what is that image?
No
What is the exact command you are running registrator with?
-internal=true -deregister=on-success consul-tls://consul:8501
What is the exact command you are running your container with?

environment:
  - SERVICE_CHECK_SCRIPT=curl --noproxy '*' example.com

A log capture of all the docker events before, during, and after the issue.

2020/10/02 14:53:41 [ERR] agent: Check "service:xxxxxxxxxxxxxxxxx" failed to invoke: exec: "curl --noproxy '*' example.com": executable file not found in $PATH

Description of the problem:
I want to use health checks with Registrator and Consul, but when I add the env var SERVICE_CHECK_SCRIPT, Consul is unable to register the health check.
I have tried to change the command by this one :
environment:
  - SERVICE_CHECK_SCRIPT=/usr/bin/curl --noproxy '*' example.com

But I have this error message :
fork/exec /usr/bin/curl --noproxy '*' example.com: no such file or directory

/usr/bin/curl exists in the Consul container :
ls -lah /usr/bin
-rwxr-xr-x    1 root     root      214.0K Jun  5  2019 curl

It's really sad that a useful project like this is left in an abandoned state.
I made a Docker image for my personal use that solves this problem and works in swarm with overlay networks.
docker pull lettore/registrator:8.0.1-3.11, where 3.11 is the Alpine version used, it's also multi-arch.
Hopes this can be useful.


feat: Svelte extra

I was about to open a PR when I thought to search first and found yours. My implementation for svelte support is basically identical except for a few additions. Is there a way I can add commits to someone else's PR on GitHub?

return {
  {
    "nvim-treesitter/nvim-treesitter",
    opts = function(_, opts)
      if type(opts.ensure_installed) == "table" then
        vim.list_extend(opts.ensure_installed, { "svelte" })
      end
    end,
  },

  {
    "neovim/nvim-lspconfig",
    opts = {
      servers = {
        svelte = {
          keys = {
            {
              "<leader>co",
              function()
                vim.lsp.buf.code_action({
                  apply = true,
                  context = {
                    only = { "source.organizeImports" },
                    diagnostics = {},
                  },
                })
              end,
              desc = "Organize Imports",
            },
          },
          capabilities = vim.fn.has("nvim-0.10") == 0 and {
            workspace = {
              didChangeWatchedFiles = {
                dynamicRegistration = true,
              },
            },
          } or nil,
        },
      },
    },
  },

  {
    "stevearc/conform.nvim",
    optional = true,
    opts = {
      formatters_by_ft = {
        ["svelte"] = { "prettier" },
      },
    },
  },
}

Dynamic registration solves sveltejs/language-tools#2008.
I'm not sure if prettier should be set up for the svelte filetype here in the svelte extra or in the formatting.prettier extra.

svelte syntax relies on HTML, javascript and CSS. These syntaxes need to be added to ensure_installed in tree-sitter

I have tried the changes in the commit and the changes by @matadaniel in LazyVim with NeoVim 0.10 on MacOS but svelte files still do not pick up changes in TS files.
Manually calling LspRestart is a poor workaround for the moment.
Any ideas what I can do to fix this? Thanks!


orders: add pick-lists

Fund, location, format

http://stackoverflow.com/questions/6400010/rails-select-drop-down-for-states


微调时报错：IndexError: index 0 is out of bounds for dimension 0 with size 0

报错位置：
File "/root/OneChart/OneChart_code/vary/model/vary_opt_math.py", line 272, in forward
det_patch_token = torch.where(input_ids == self.config.number_token)[1][0]
IndexError: index 0 is out of bounds for dimension 0 with size 0
训练集格式：

我更新了readme中的train部分，修改json格式后再试试

It looks like the "" token was not implemented in author's provided dataset preparation. However, the decoder tried to locate the token through its id, which leads to a result of 0 in torch.where when it was going to calculate num loss.
Adding ""(or "" in author's code) to the assistant part of the conversation will bring you on your way.

我更新了readme中的train部分，修改json格式后再试试

我看您更新了数据集的格式，有两个问题请教一下：
1、Readme里面说，If the auxiliary head is Not used and trained, the format example is xxx，请问如果我想按照示例1的数据格式进行训练，auxiliary head开关在哪控制？
2、示例2中的 "Numbers": [6.12, 5.32, 5.11, 4.5, 1.45]，这类数据内容是必须要写的吗？只把value的数据加上前缀是否可行？

关掉auxiliary head，我没有写开关，你可以通过注释 这行代码 实现这个目的
如果你不要auxiliary head，可以不写"Numbers": [6.12, 5.32, 5.11, 4.5, 1.45]


fix: Bump server deps in client release group and stop skipping regression tests

Description
Update deps on server packages in the client release group from ^1.0.0 to ^1.0.1 by running flub bump deps server -g client -t patch from the root of the repo. While a lockfile update could have sufficed, we preferred to explicitly indicate that we want to use at least the version of server packages that includes this fix.
Also stops skipping the regression tests added with the linked fix, now that the fix is included in the resolved dependencies and the tests won't fail.
Reviewer Guidance
The review process is outlined on this wiki page.
AB#5389

should also do this for the tinylicious package

should also do this for the tinylicious package

@anthony-murphy you mean a test with synthetic requests? Since the tinylicious package doesn't depend on any client packages we don't have all the Fluid concepts or utils so not sure how we would test this there other than synthetic requests. I can create tracking items for that if that'd be the idea, but I think I'll move on to my other sprint work once this is merged.


Workaround for missing Parquet page indexes in ParquetMetadaReader

Which issue does this PR close?
Part of #6447
Rationale for this change
There is an inconsistency in how the various Parquet metadata parsing APIs handle files that lack page indexes. This PR temporarily modifies ParquetMetaDataReader to match the current behavior in parquet::file::metadata::page_index::index_reader (return empty vectors rather than None).
What changes are included in this PR?

Are there any user-facing changes?
No

Presumably this will make replacing the existing APIs without test changes possble, so I think it is ok for now that it isn't covered by tests.

Yes, that's the hope. As deprecated methods are replaced by this API, their tests should cover the new API.

Pushed a small commit to make cargo fmt happy, then I will merge this PR so we can get https://github.com/apache/arrow-rs/pull/6451 in