[Rendering] Teams rendering Dates incorrectly in Adaptive Cards on iOS

I am developing MS Teams app which with the user can search in and send cards from an internal system. The card contains Date and Time values, which are rendered incorrectly only on iOS.
I figured out that the issue only occurs if I am using a specific Date Format.
I have the following settings in iPhone:
Settings -> General -> Language & Region

Region -> select Hungary
Date Format -> select first option, for me its 2022. 08. 29

I have tried it with few other settings, but those are all working, and the card looks good on other platforms (Android, Desktop, Browser).
This is how it looks like on iPhone 12 Mini and iPhone 14 (both iOS 16 and the latest Teams):

The bot creates the following Adaptive Card:
{
  "type": "AdaptiveCard",
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "version": "1.4",
  "verticalContentAlignment": "Top",
  "body": [
    {
      "type": "ColumnSet",
      "spacing": "None",
      "horizontalAlignment": "Left",
      "columns": [
        {
          "type": "Column",
          "width": 1,
          "items": [
            {
              "type": "Image",
              "url": "data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAEAAAABACAIAAAAlC+aJAAAAAXNSR0IArs4c6QAAAARnQU1BAACxjwv8YQUAAAAJcEhZcwAADsMAAA7DAcdvqGQAAABbSURBVGhD7c8BCQAwDMCwmZjny76MMCjEQObNnlZAK6AV0ApoBbQCWgGtgFZAK6AV0ApoBbQCWgGtgFZAK6AV0ApoBbQCWgGtgFZAK6AV0ApoBbQCWgHteGD2A0Bh0IgCvu/vAAAAAElFTkSuQmCC",
              "size": "Medium",
              "isVisible": true,
              "horizontalAlignment": "Center"
            }
          ]
        },
        {
          "type": "Column",
          "width": 4,
          "items": [
            {
              "type": "TextBlock",
              "text": "Just a sample title",
              "wrap": true,
              "fontType": "Default",
              "size": "Large",
              "weight": "Bolder",
              "spacing": "None",
              "separator": true
            },
            {
              "type": "ColumnSet",
              "columns": [
                {
                  "type": "Column",
                  "width": 1,
                  "items": [
                    {
                      "type": "TextBlock",
                      "text": "Start",
                      "wrap": true,
                      "isSubtle": true,
                      "weight": "Bolder"
                    },
                    {
                      "type": "TextBlock",
                      "text": "{{DATE(2022-12-20T00:00:00Z)}} {{TIME(2022-12-20T00:00:00Z)}}",
                      "wrap": true,
                      "spacing": "None"
                    }
                  ]
                },
                {
                  "type": "Column",
                  "width": 1,
                  "items": [
                    {
                      "type": "TextBlock",
                      "text": "End",
                      "wrap": true,
                      "isSubtle": true,
                      "weight": "Bolder"
                    },
                    {
                      "type": "TextBlock",
                      "text": "{{DATE(2022-12-21T00:00:00Z)}} {{TIME(2022-12-21T00:00:00Z)}}",
                      "wrap": true,
                      "spacing": "None"
                    }
                  ]
                }
              ]
            }
          ]
        }
      ]
    }
  ]
}

It is a crucial feature of our App, so I hope you can find a solution for the bug as soon as possible.

We did try the JSON provided by you and see the card being rendered as follows:
iOS:

Android & Web :

Only difference we observed is year mentioned in 'yy' format in iOS. We checked with the default settings[English]

We did try again with Hungary, and we see the same rendering as you. We will check internally and update you.

Thank you for your quick response!

We have raised a bug for the same, we will keep you posted on the updates. Thank you.


Disable PTS for architecture tests

Architecture tests do now combine well with PTS since they check all classes. So disabling PTS and using the build cache instead makes sense.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build cancelled.

@bot-gradle test and merge

OK, I've already triggered a build for you.


[feat/aut964] Fixed set path in FilePicker

Changing path has no effect since we are assigning a temporary string.

@ahmedskhalil i know you have recently changed that maybe to fix one issue, please can you check if it breaks something


Raspberry Pi 4-image

Hi
Now is released Raspberry Pi 4 :)
When we can think that image for Raspberry Pi 4 will be available ?

Can yo release immediately hassos on Rpi4 ?
Only option now is use manual installation on linux
ResinOS it also not work on rpi4

Just wait a little bit, give the devs a chance!
On Sat, 29 Jun 2019, 20:36 homeassistantmi2<EMAIL_ADDRESS>wrote:

Can yo release immediately hassos on Rpi4 ?
Only option now is use manual installation on linux
ResinOS it also not work on rpi4
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/hassos/issues/413?email_source=notifications&email_token=AB6SOLT23YQTLP2ACXQYJXDP462TDA5CNFSM4H24MOE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODY363XQ#issuecomment-506981854,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB6SOLRTW5QKNYWU3N46DPTP462TDANCNFSM4H24MOEQ
.

And good will be add support boot from ssd on usb3. 0
This is very need.

We can first add support for RPi4 if u-boot (our bootloader they we use because we are a embedded system) have a support patch.

Would be amazing if we can just take out our sd  card from Rpi3 and stick it in Rpi4 and it just starts working like nothing happend 🥰

I think that's an impossibility due to the changes thr new hardware has.
Nothing wrong with starting again 😁
On Wed, 3 Jul 2019, 10:55 skynet01<EMAIL_ADDRESS>wrote:

Would be amazing if we can just take out our sd card from Rpi3 and stick
it in Rpi4 and it just starts working like nothing happend 🥰
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/hassos/issues/413?email_source=notifications&email_token=AB6SOLRDFI7SW577RDQKPX3P5RZR5A5CNFSM4H24MOE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZD55ZY#issuecomment-508026599,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB6SOLVFLF6REOBMEGMCGLDP5RZR5ANCNFSM4H24MOEQ
.

@Joeboyc2 have right, there are complete different bootloader and microcode to setup the CPU and start the kernel. Raspbian work a bit different, they can handle that because they but everthing into same image and support only 32bit.

Support start with 3.3

Woohoo! Thank you!

I'll be flashing it tonight, will let you know of I have any issues 😁
On Tue, 16 Jul 2019, 18:30 skynet01<EMAIL_ADDRESS>wrote:

Woohoo! Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/hassos/issues/413?email_source=notifications&email_token=AB6SOLSW35TK6KJ7PEVYU63P7YAUTA5CNFSM4H24MOE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2BSQMA#issuecomment-511911984,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB6SOLXQIGZ3LQCI7DVXWYLP7YAUTANCNFSM4H24MOEQ
.

oh.... @pvizeli i dont see the pi 4 image.... which one should i use to flash ? the pi3 one?

You just have to wait a bit while they are building, now they are up in assets

So stoked to try it. Is there going to be a migration tutorial of some sort from Pi3+? Or you just flash it and restore a snapshot?

Ah i see them now, literally only 22 assets when i check and sent the message, 24 now :)


Bots in Group PMs

Refer to https://github.com/zulip/zulip/issues/14228 for more details. This PR addresses #551 and #552.

@aero31aero What's the latest status on this?

Got sidetracked with other stuff, I'll work on this today/tomorrow.


attaching csv file results in UploadedFile with content-type of "text/comma-separated-values"

I'm attaching a file to a file field like ths:
attach_file('Csv', Rails.root.join('test', 'fixtures', 'push_notifications_valid_upload.csv'))
If I debugger in the controller, I see this
(byebug) params["push_notification"]["push_notification_member_list"]["csv"]
#<ActionDispatch::Http::UploadedFile:0x007fe8922feef0 @tempfile=#<Tempfile:/var/folders/3y/t3n5r5d56wn2zl9z9byzx2_h0000gn/T/RackMultipart20160912-11407-wdnkf7.csv>, @original_filename="push_notifications_valid_upload.csv", @content_type="text/comma-separated-values", @headers="Content-Disposition: form-data; name=\"push_notification[push_notification_member_list][csv]\"; filename=\"push_notifications_valid_upload.csv\"\r\nContent-Type: text/comma-separated-values\r\nContent-Length: 60\r\n">

(byebug) params["push_notification"]["push_notification_member_list"]["csv"].content_type
"text/comma-separated-values"

If I attach the same file in a browser and submit, the content type is "text/csv" - so it seems that the test suite is incorrect.  Is this a bug?  Configuration problem?  Is there a way to stub the content type to the correct value?
Thanks!
Additional info I found:
"text/comma-separated-value" exists but appears to be flagged as obsolute here:
https://github.com/mime-types/mime-types-data/blob/master/types/text.yaml#L38-L45

I assume you're using the rack-test driver?  The mime-types gem is used to select the first mime-type that matches the given filename.  According to the docs http://www.rubydoc.info/gems/mime-types/MIME/Types#type_for-class_method they should be sorted with non-obsolete before obsolete, however from your report it would appear that is not happening.  What version of mime-types are you suing?

2.99.2
I also tried bumping to 3.1 and that didn't help.

Seems like mime-types sorting is fully broken then

@josh-m-sharpe Please try the mime_types_broken branch and see if that fixes the issue for you.

Yep - that branch looks good!

Fixed in master


Allow trailing commas

...in function argument, shader argument, and metadata lists.  From:
https://groups.google.com/forum/#!topic/osl-dev/swRNU97n16Y

Like I said in the other thread, CLA should be all good to go.  Thanks for your patience!

Yes, that's what I hear! I will test and merge.

Awesome. Let me know if you see anything.
On Tue, Jan 27, 2015 at 5:21 PM, Larry Gritz<EMAIL_ADDRESS>wrote:

Yes, that's what I hear! I will test and merge.
Reply to this email directly or view it on GitHub:
https://github.com/imageworks/OpenShadingLanguage/pull/447#issuecomment-71764246

This is also good to go now that the CLA is all good. Thanks!

Squashed and merged! Thanks for the patch!


Allow no upper limit on the number of functions passed to a pipe

Description:
Currently, a pipe could only have 0-10 functions arguments.
It is not clear to me why this arbitrary number (maybe some kind of 'safety' for the user?)
This patch dissables this limitation.

Currently, if you have more than 10 functions for a pipe I do this:
        this.oc.list(directory).pipe(
            tap(directory => console.log(directory)),

            concatMap(() => this.oc.saveFile(newFile, fileContent)),
            tap(hasSucceded => console.log('File creation succeded: ', hasSucceded)),

            concatMap(content => this.oc.readFile(newFile)),
            tap(content => console.log('Content: ', content)),

            concatMap(content => this.oc.list(directory)),
            tap(directory => console.log(directory)),

            concatMap(content => this.oc.delete(newFile)),
            tap(hasSucceded => console.log('File deletion succeded: ', hasSucceded)),

        ).pipe(
            concatMap(content => this.oc.list(directory)),
            tap(directory => console.log(directory)),

        ).subscribe();

Is there another alternative?

Given state is unexpected, but proposed PR is not desired change. when param number exceeds overload, it should fall back to original signature instead. I'm closing this PR, filing https://github.com/ReactiveX/rxjs/issues/3346 instead to track correct operator behavior.
Thanks for finding out issue, I usually haven't had cases of more than 10 operators at once.


Session not saving when returning a redirect response

Basically the issue is that I have an endpoint (lets call it auth endpoint) that fetches some data from a 3rd party application and based on the response from that app if it's successful it inserts key signed_in into a session, then the client is redirected to another endpoint (lets call it data endpoint). and from inside that other endpoint when i try to read the session it says that the key signed_in is None, this is the issue. but when the auth endpoint sends a response back without redirecting.. it saves the session successfully and when i navigate to the data endpoint it reads the session correctly.
Excuse me for my rusty English and my terrible explanation but I hope you get the idea.
Edit:
Here is a better explanation
the issue
fn auth_endpoint(mut session: WriteableSession) {
    do_some_logic();
    session.insert("signed_in", true).unwrap();
    return Redirect::to("/data-endpoint");
}

fn data_endpoint(session: ReadableSession) {
    let signed_in = session.get::<bool>("signed_in").unwrap_or(false);
    // signed_in is None (false)
}

The walk-around
fn auth_endpoint(mut session: WriteableSession) {
    do_some_logic();
    session.insert("signed_in", true).unwrap();
    return "OK!"
}
// then when a user manually navigates to this /data-endpoint
fn data_endpoint(session: ReadableSession) {
    let signed_in = session.get::<bool>("signed_in").unwrap_or(false);
    // signed_in is true
}

Is there a way i can fix this or am i doing something wrong?

Thanks for raising this. Would you mind putting together an example following the format of our examples directory? (The easiest way is probably to fork this repo and create a new example in that directory.) This will allow me to inspect what you're seeing more directly.

Hi again, I've tried to reproduce this issue but it turns out to work, I've tried to debug and do everything I thought of in the project I am working on but nothing has fixed the issue, I still do not know whats causing it.. Even though everything in the reproduced repo is pretty much the same as the project I'm working on but it's really messed up..

Sorry you're running into that. That's frustrating.
Have you checked that the crates are all the same versions between the two?

I've sent you an invite to the repository, thanks much!

Require .with_same_site_policy(SameSite::Lax) and .with_secure(false)

@vkill in production, I would advise setting secure to true.


Match rules for signals are too broad

I recently tried to replace python-dbus with python-dbus-next in one of my projects and realized that it is not possible to register a signal handler for the specific combination of signal arguments.
For example, if one wants to get a notification when the Spotify app is closed (that is, when its well-known bus name loses an owner), they can do this using python-dbus:
bus.get_object(
    bus_name='org.freedesktop.DBus',
    object_path='/org/freedesktop/DBus',
).connect_to_signal(
    signal_name='NameOwnerChanged',
    handler_function=on_name_owner_changed,
    dbus_interface='org.freedesktop.DBus',
    arg0='org.mpris.MediaPlayer2.spotify',
    arg2='',
)

The code above does to the following method call:
org.freedesktop.DBus.AddMatch("\
type='signal',\
sender='org.freedesktop.DBus',\
path='/org/freedesktop/DBus',\
interface='org.freedesktop.DBus',\
member='NameOwnerChanged',\
arg0='org.mpris.MediaPlayer2.spotify',\
arg2=''\
")

In this case, the client will only receive signal messages which have the following signature: NameOwnerChanged('org.mpris.MediaPlayer2.spotify', <anystring>, '') (see https://dbus.freedesktop.org/doc/dbus-specification.html#bus-messages-name-owner-changed), other messages will not be routed to the client at all.
I dug into the library code and found out that the current implementation uses very broad match rule: https://github.com/altdesktop/python-dbus-next/blob/6dee14cc43ca3494e9a206eeba137f81a0c0fe4e/dbus_next/proxy_object.py#L47
That is, any signal with the specified combination of a bus_name, interface, and object path will be delivered to the client regardless of whether or not there is a registered handler. If there is no registered handler, the message will be just ignored (if ... msg.member not in self._signal_handlers): https://github.com/altdesktop/python-dbus-next/blob/6dee14cc43ca3494e9a206eeba137f81a0c0fe4e/dbus_next/proxy_object.py#L76
It could be inefficient. If there are lots of signals and one is only interested in a small part of them, they will still receive all of them. Moveover, mattern matching must be implemented in the signal handler code. For example, if one registered a signal handler using python-dbus as it described in the example above, they would not have to check the name and new_onwer arguments because the check would be already done on the server side (nonmatching signals would not be sent to the client in the first place):
def on_name_owner_changed(name: str, old_owner: str, new_owner: str):
    # this check is not required
    if name != 'org.mpris.MediaPlayer2.spotify' or new_owner != '':
        return
    do_something()

However, it isn't that simple. Pattern matching (dispatching) is still required because there can be more than one signal handler for the same signal. That is, we still need to dispatch a signal message to an appropriate handler function, otherwise all handlers will be called on all signals regardless of signal arguments. The point is that pattern matching should be a part of a dispatching mechanism of the library, not be implemented ad hoc by a library user. This is how it is implemented in dbus-python:

https://github.com/freedesktop/dbus-python/blob/dbus-python-1.2.14/dbus/connection.py#L523
https://github.com/freedesktop/dbus-python/blob/dbus-python-1.2.14/dbus/connection.py#L182

In my opinion, it could work like this:

BaseProxyInterface._signal_match_rule is not used anymore.
Instead, When and only when interface.on_<signal>(fn, [arg0=..., [arg1=..., [argN=...]]]) is called, the org.freedesktop.DBus.AddMatch method is called with more specific matching rule. The rule should be constructed using <signal> as a mandatory member= field and argN arguments as optional argN= fields (if any). The rule should have a Python representation for dispatching purposes (see https://github.com/freedesktop/dbus-python/blob/dbus-python-1.2.14/dbus/connection.py#L55 for example). The Python representation of the rule and the handler should be registered in some sort of mapping or tree.
When a signal is received, the corresponding handler(s) are found using registered matching rules and called.
It must be possible to unregister a specific handler with a specific set of arguments.

Yeah sounds reasonable. Are you available to work on it?

It must be possible to unregister a specific handler with a specific set of arguments.

Yeah kind of tricky. This is an edge case that needs to be worked out.
iface.off_signal(fn) # remove all handlers regardless of match rules?
iface.off_signal(fn, arg1='foo') # remove all handlers with `arg1='foo'` even if they specified an `arg2='bar'`?

Unfortunately, I'm not ready to work on it, at least in the near future.

Ok well that workaround should work fine until someone gets around to it.


Scraper: Use already saved images, just scrape metadata

Hi,
I have 750 NES roms with images named the same (e.g.  1942.nes and 1942.jpg). I've just run the scraper and it's creating a new image 1942-image.jpg in a folder called downloaded_images.

how do I tell the scraper to not download an image if one already exists
if by chance I didn't have a corresponding image, how do I tell the scraper not to save it in the downloaded_images folder and to save it alongside the rom

Is either of the above possible?

You can't. The scraper downloads the image every time you are scraping a platform
The destination folder for the meta data box art is fixed. It will stay like that to maintain a clear structure.

The thing is, I don't want it to skip downloading the description etc. , I literally only want it not to download the image - I've spent absolutely ages sourcing decent-quality boxart images.
There's no way of having an option so 'if image found, check metadata - if missing, only download metadata' ?

As you can see by yourself, no. Sorry.
An option to only download the missing part of meta data would be helpful in this case.
@Aloshi, what do you think? Is this possible with a kind of checking routine which information to download?

I'm actually in the same situation as @robertybob right now. Can't appear to find any type of workaround.
Consider this a +1 from me at the very least I suppose. Subscribing to this issue, and willing to test/cherry-pick any new commits!

@scar45 I have a long-winded workaround if you're interested. I run ES on my RasPi but use my Windows PC to generate the XML file:

In the folder where my NES (for example) ROMs are, make an 'images' folder
Put all the boxart I've downloaded into that 'images' folder
Use Bulk Rename Utility to suffix '-image' to the artwork
Using github.com/sselph/scraper , put Scraper.exe into my ROM's folder and run Scraper.exe
If the scraper finds a game and an image exists in the 'images', it'll download the metadata and create an image suffixed -thumb
Once the scraper has finished, put images suffixed -thumb into ROM folder and delete all that have corresponding ROM file. This is the easiest way to work out whether the scraper has missed any game - i.e. if no -thumb image exists for a ROM, the game wasn't scraped and you'll have to add it manually.
Delete -thumb images
Using WinSCP or Filezilla, transfer ROMs to Pi and artwork from 'images' folder to correct folder on Pi
Put the generated gamelist.xml file in correct folder on Pi
Open the gamelist.xml file and using Find & Replace, edit it to link images to correct folder

and I've done this for my NES, SNES, MegaDrive, Master System, Sega 32x ROMs.

Thanks a bunch @robertybob! I have already batch renamed my images to append '-image' to the filename, so I'm set there, but I was unaware that there exists a Windows scraper which will generate ES-friendly gamelist.xmls. #6 is a nice tip as well for optimizing this process.
I will be giving it a shot as soon as I am home from work. Cheers and thanks again!

@robertybob I just came across this thread again and realized I hadn't confirmed that your method above worked great for me. Cheers!
@changetip grab him a beer, will ya?

Hi @robertybob, @scar45 sent you a Bitcoin tip worth a beer (22,151 bits/$5.00), and I'm here to deliver it ➔ collect your tip.
Learn more about ChangeTip

@scar45 Dude, that Bitcoin's now worth over $300 , I love you!!


Does not recognize hooks in mutators

As of version 6.9.5 (with the previous ones it works fine), it is not correctly recognizing the hooks.
If in the file https://github.com/anymaniax/orval/blob/master/samples/react-query/hook-mutator/use-custom-instance.ts I modify and add the line (const queryClient = useQueryClient();):
import Axios, { AxiosRequestConfig } from 'axios';
import { useQueryClient } from 'react-query';

export const AXIOS_INSTANCE = Axios.create({ baseURL: '' });

export const useCustomInstance = <T>(): ((
  config: AxiosRequestConfig,
) => Promise<T>) => {
  const queryClient = useQueryClient();

  return (config: AxiosRequestConfig) => {
    const source = Axios.CancelToken.source();
    const promise = AXIOS_INSTANCE({
      ...config,
      cancelToken: source.token,
    }).then(({ data }) => data);

    // @ts-ignore
    promise.cancel = () => {
      source.cancel('Query was cancelled by React Query');
    };

    return promise;
  };
};

export default useCustomInstance;

The generator does not recognize it as a hook.
Version 6.9.5

Previus version:

Sorry for that should be fixed with 6.9.6

@anymaniax still the case with 6.9.6.
Here i have base useCustomInstance
import Axios, { AxiosInstance, AxiosRequestConfig } from 'axios'

import { useCredentials } from '../store/useCredentials'

export const useCustomInstance = <T>(
  AXIOS_INSTANCE: AxiosInstance
): ((config: AxiosRequestConfig) => Promise<T>) => {
  const [credentials] = useCredentials()
  return (config: AxiosRequestConfig) => {
    const source = Axios.CancelToken.source()
    const promise = AXIOS_INSTANCE({
      ...config,
      cancelToken: source.token,
      headers: {
        Authorization: `Bearer ${credentials}`,
        ...config.headers
      }
    })
      .then(({ data }) => data)
      .catch((err) => {
        if (err.response.status !== 401 && err.response.status !== 403) {
          // todo error
        }
        return Promise.reject(err)
      })

    // @ts-ignore
    promise.cancel = () => {
      source.cancel('Query was cancelled by React Query')
    }

    return promise
  }
}

export default useCustomInstance

And then creating a wrapper useCustomInstanceAuth over it
import Axios, { AxiosRequestConfig } from 'axios'
import useCustomInstance from './use-custom-instance'

export const AXIOS_INSTANCE = Axios.create({
  baseURL: process.env.NODE_ENV === 'development' ? '/auth/api' : '/auth'
})

export const useCustomInstanceAuth = <T>(): ((config: AxiosRequestConfig) => Promise<T>) => {
  return useCustomInstance(AXIOS_INSTANCE)
}

export default useCustomInstanceAuth

Using just plane useCustomInstance results in correct hook generation. But using useCustomInstanceAuth results in problem that was explained above.
Interesting, though, this pattern worked on 6.9.0


createConnection timezone not working

Hi guys,
in below code timezone setting does not seem to have any effort , i think its the issue with the driver could anyone help ?
my sql DB timezone : "-08:00"
application time : "+05:30"
yes this needs to be handled at app level. DB cant change.
var mysql = require('mysql');

//connect to db
var dbCon = mysql.createConnection({
    host: "xxxx",
    user: "xxxxx",
    password: "xxxxx",
    database:"xxxx",
    **timezone:"+05:30"**
  });

  //db connection
  dbCon.connect(function(err) {
    if (err) throw err;
    console.log("DB Connected!");
  });

var dbCon = db.dbCon;
//query functions
function get_total_sales(startDate, endDate, callback) {
    dbCon.query("SELECT sum(Paid) as total_sales, count(Paid) as item_soled FROM `orders`  WHERE DateCreated >= " + mysql.escape(startDate) + " AND DateCreated <= " + mysql.escape(endDate) + "  ",
        function (err, result, fields) {
            if (err) {
                throw err;
            } else {

                finalWriteDataObject["total_sales"] = numeral(result[0].total_sales).format();
                finalWriteDataObject["item_soled"] = numeral(result[0].total_saitem_soledles).format();
                callback();
            }

        });
}

get_total_sales(startDate, endDate, writeToFile);

Maybe can you elaborate what you're expecting the timezone setting to do? What it does it just fix conversation,  not change the timezone. You must set it to the timezone your mysql server is set to.

Hi Dougwilson,
thanks for reply, im coming from php and in there in db config when we give the timezone parameter it passes that info into the DB so DB returns correct dataset.
leaving that just take mysql it supports below
"SET time_zone='+5:30';"
example of running with above and not without above.
SELECT NOW(); return 2018-11-24 15:28:24
but  SET time_zone='+5:30';SELECT NOW() returns 2018-11-25 04:59:10.
basically when the correct timezone information is not passed to DB , select queries based on say  mysql data type timestamp will give you wrong results.
SELECT sum(paid) as Totsales FROM fm_cusinvoLink WHERE paid=1 and date(DateCreated) = '2018-11-23' returns 65.
SET time_zone='+5:30';SELECT sum(paid) as Totsales FROM fm_cusinvoLink WHERE paid=1 and date(DateCreated) = '2018-11-23' returns 71
if your table uses timestamp type than  your screwed, so the driver must pass the timezone data to mysql server

Gotcha. If you want to set the timezone on the server, then you don't need to pass in the timezone parameter at all. Just run that SET command as the first query on your connection.

Gotcha. If you want to set the timezone on the server, then you don't need to pass in the timezone parameter at all. Just run that SET command as the first query on your connection.

actually thats what we are doing in php pdo .
but when we do the same in node.js its showing error

What is the error you're getting in node.js?

What is the error you're getting in node.js?

dbCon.query("SET time_zone='+5:30';select now()",
function (err, result, fields) {
if (err) {
throw err;
} else {
console.log(result);
// res.json(result);
}
});

for above function
    throw err; // Rethrow non-MySQL errors
    ^

Error: ER_PARSE_ERROR: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'select now()' at line 1
in php pdo we fun as 2 queries in same connection , here dont know how to

You have to enable that feature. See the docs here: https://github.com/mysqljs/mysql#multiple-statement-queries

thanks a lot but i dont understand the purpose of below
From the docs https://github.com/mysqljs/mysql/blob/master/Readme.md#connection-options
timezone: The timezone configured on the MySQL server. This is used to type cast server date/time values to JavaScript Date object and vice versa. This can be 'local', 'Z', or an offset in the form +HH:MM or -HH:MM. (Default: 'local')

The setting is used to type cast server date/time values to JavaScript Date object and vice versa. It is to be set to the timezone configured on the MySQL server. In your case, this would be the value '-08:00', but if you run the query SET time_zone='+5:30', you instead need to set that to '+5:30' to match. But since it is just used to type cast server date/time values to JavaScript Date objects, if you're not actually using Date objects, the setting would not be meaningful to your code.


fix axisbg

Remove axisbg and replace with facecolor, since this is deprecated starting 2.0+ of matplotlib.

I guess this is a small one, @havogt are you gatekeeper?


BUG: Release the Python Global Interpreter Lock (GIL) during execution

Release the Python Global Interpreter Lock (GIL) so multi-threaded
Python applications, like Dask, can run in parallel while ITK does heavy
operations.
For more information, see section 32.13 Support for Multithreaded
Applications at
http://www.swig.org/Doc4.0/Python.html#Python
Also, set nothreadallow as suggested in the comments
on https://github.com/swig/swig/issues/927. This avoids calls to
PyEval_SaveThread() and PyEval_RestoreThread(), which can negatively
effect performance. This change only affects module C++ code -- code
that can call Python code like ITKPyUtils does not have threads
enabled.

The GIL should be reacquired when calling Python call backs. See SimpleITK for an example:
https://github.com/SimpleITK/SimpleITK/blob/master/Wrapping/Python/sitkPyCommand.cxx

@blowekamp thanks for the pointer! Added in https://github.com/InsightSoftwareConsortium/ITK/pull/1065/commits/3eb1f6a6e59344a756dece628833b0214145057b

LGTM


Support configurable quality of service controls

When running a DHT relay as a service, the service is currently susceptible to DoS attacks as a bad actor can easily saturate the relay. We should provide some sort of configurable QoS controls to allow relay operators to mitigate this.
Some initial ideas:

Configurable per-connection usage limits, such as limits on the number of open servers and connections, active queries, etc.

Hooks to allow/deny all actions that allocate resources, such as opening a server or a connection, running a query, etc.

can this be a modular?
when it is used on localhost any overhead seems unnecessary, because the user is the only one interacting with the relay


Act like an eval

With these updates, sval can really act like eval, returning the value of the passed code, but with the context isolation sval offers.
I think that could be optional, but I think that don't damages anything just acting always as an eval.
let res1 = eval("10 * 2");
let res2 = sval.run("10 * 2");
console.log("Eval test 1", res1 === res2);

let res3 = eval("let aj = 123;const aj2 = aj*3;function pepe(asd, asd2) {return asd + asd2;} var cosas = 12; var a = 44; var b = cosas * 2 + a;b");
let res4 = sval.run("let aj = 123;const aj2 = aj*3;function pepe(asd, asd2) {return asd + asd2;} var cosas = 12; var a = 44; var b = cosas * 2 + a;b");
console.log("Eval test 2", res3 === res4);

Damn, created the PR with GitHub and fu*kd it up

Pull Request Test Coverage Report for Build 201

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 73.745%

Totals

Change from base Build 179:
0.0%

Covered Lines:
1024

Relevant Lines:
1337

💛  - Coveralls


Ansi alternative

Possible alternative to #348.
This is using Verbatim instead of sphinxVerbatim.
This is not yet complete.
I still have to set the background color on stderr cells.
The spacing inside the plain-text output cells is probably not quite correct.
But most importantly, the plain text lines are too far apart!
This can be e.g. be seen in the ANSI outputs: https://49-210404706-gh.circle-artifacts.com/0/nbsphinx.pdf#subsection.4.4

To fix the extra whitespace:
diff --git a/src/nbsphinx.py b/src/nbsphinx.py
index 9695816..5c1453b 100644
--- a/src/nbsphinx.py
+++ b/src/nbsphinx.py
@@ -139,8 +139,7 @@ RST_TEMPLATE = """

     .. raw:: latex

-        \\kern-\\sphinxverbatimsmallskipamount
-        \\kern+\\FrameHeightAdjust\\kern-\\fboxrule
+        \\lineskip0pt\\relax
         \\begin{Verbatim}[commandchars=\\\\\\{\\}]
 {{ output.data[datatype] | escape_latex | ansi2latex | indent | indent }}
         \\end{Verbatim}

But I think you are losing things by dropping sphinxVerbatim for Verbatim: long code lines will not wrap, the mechanism for highlighted lines is lost and (but does it apply here?) you lose the caption.

Thanks, I've added the suggested fix: bd41d5c2add1737dab053d3e7a84985861c41cf8
Here's a preview: https://52-210404706-gh.circle-artifacts.com/0/nbsphinx.pdf#subsection.4.4

long code lines will not wrap

Oh, that's bad, I like the line wrapping ...

the mechanism for highlighted lines is lost

This is not needed here.

you lose the caption

This isn't needed either

also, Sphinx adds a \strut in each codeline to help keep a consistent baseline distance

Indeed, that looks bad.
It looks like this isn't the right way to proceed ... therefore I've tried again: #351
This avoids the nested framed environments and AFAICT it works well!

Superseded by #351.


docs(firebase_auth): fix two typos.

Description
Just fixed typo.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (melos run analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

@russellwheatley thank you for your approval! cant wait to be merged~


Problem with useNames = NA

When I launch the SPIDER_predict() command on the test example, I get the following error:
Erreur : [matrixStats (>= 1.2.0)] useNames = NA is defunct. Instead, specify either useNames = TRUE or useNames = FALSE. See also ?matrixStats::matrixStats.options
Thank's for your help

Thank you for using SPIDER.
The error you got is likely caused by the version incompatibility between matrixStats and Seurat (your current matrixStats has the version of >= 1.2.0 which may be too high for Seurat). Would you try re-install the matrixStats package with a lower version of 1.0.0 or 1.1.0?
(related issue on Seurat: https://github.com/satijalab/seurat/issues/7501#issuecomment-1854571904)


Stop TypeScript compiler from emitting helper functions in every file - use ts-helper instead (to fix code coverage numbers)

This patch is to fix the code coverage numbers being generated by Istanbul. The source maps do not appear to be working 100% correctly in Istanbul (the code you see is mangled TypeScript), but this at least fixes the coverage numbers by stopping the compiler from emitting __decorate / __extend / etc functions in every file.

Actually, hang on a moment before merging this. I think a previous commit I did broke code coverage source maps. I will have it fixed in a moment

OK - source maps work in both the debugger and in the code coverage report now. Sweet! My apologies. I think the commit I did yesterday broke istanbul source maps. @JaKXz
@SethDavenport Let me know if there are any other issues with this PR - but from what I can tell, the code coverage reports and the debugging source maps both work correctly. And the production bundle size hasn't increased.

:shipit:


Is there any way to use C++17 on MSBuild?

Node Version:10.15.0
Platform:Windows 10 x64
Compiler:MSBuild ,toolset v150
Module:my SVLauncherDLL

Verbose output (from npm or node-gyp):
e:\visual studio项目\dsted\source\cpp-addon\svlaunchdll_jswrap.cc(135): error C7525: 内联变量至少需要 "/std:c++17" (编译源文件 ..\cpp-
addon\svlaunchdll_jswrap.cc) [E:\Visual Studio项目\DSTed\Source\build\SVLaunchDLL.vcxproj]

I need C++17, seriously.
cpp-addon.zip

Add the following to your binding.gyp:
  'msvs_settings': {
    'VCCLCompilerTool': {
      'AdditionalOptions': [ '-std:c++17', ],
    },
  },

Failed
 Traceback (most recent call last):   File "E:\NPM_PREFIX\node_modules\node-gyp\gyp\gyp_main.py", line 16, in <module>     sys.exit(gyp.script_main())   File "E:\NPM_PREFIX\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 545, in script_main     return main(sys.argv[1:])   File "E:\NPM_PREFIX\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 538, in main     return gyp_main(args)   File "E:\NPM_PREFIX\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 514, in gyp_main     options.duplicate_basename_check)   File "E:\NPM_PREFIX\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 130, in Load     params['parallel'], params['root_targets'])   File "E:\NPM_PREFIX\node_modules\node-gyp\gyp\pylib\gyp\input.py", line 2783, in Load     variables, includes, depth, check, True)   File "E:\NPM_PREFIX\node_modules\node-gyp\gyp\pylib\gyp\input.py", line 399, in LoadTargetBuildFile     includes, True, check)   File "E:\NPM_PREFIX\node_modules\node-gyp\gyp\pylib\gyp\input.py", line 251, in LoadOneBuildFile     None)   File "binding.gyp", line 23     'msvs-settings':                   ^ SyntaxError: invalid syntax
binding.gyp.txt

replace [ with { for 'msvs_settings' and 'VCCLCompilerTool'
binding.gyp.txt

Verbose output (from npm or node-gyp):
Traceback (most recent call last):
  File "E:\NPM_PREFIX\node_modules\prebuild\node_modules\node-gyp\gyp\gyp_main.py", line 16, in
    sys.exit(gyp.script_main())
  File "E:\NPM_PREFIX\node_modules\prebuild\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 545, in script_main
    return main(sys.argv[1:])
  File "E:\NPM_PREFIX\node_modules\prebuild\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 538, in main
    return gyp_main(args)
  File "E:\NPM_PREFIX\node_modules\prebuild\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 514, in gyp_main
    options.duplicate_basename_check)
  File "E:\NPM_PREFIX\node_modules\prebuild\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 130, in Load
    params['parallel'], params['root_targets'])
  File "E:\NPM_PREFIX\node_modules\prebuild\node_modules\node-gyp\gyp\pylib\gyp\input.py", line 2783, in Load
    variables, includes, depth, check, True)
  File "E:\NPM_PREFIX\node_modules\prebuild\node_modules\node-gyp\gyp\pylib\gyp\input.py", line 425, in LoadTargetBuildFile
    build_file_data, PHASE_EARLY, variables, build_file_path)
  File "E:\NPM_PREFIX\node_modules\prebuild\node_modules\node-gyp\gyp\pylib\gyp\input.py", line 1293, in ProcessVariablesAndConditionsInDict
    build_file, key)
  File "E:\NPM_PREFIX\node_modules\prebuild\node_modules\node-gyp\gyp\pylib\gyp\input.py", line 1303, in ProcessVariablesAndConditionsInDict
    ' for ' + key)
TypeError: Unknown type set for AdditionalOptions while trying to load binding.gyp

binding.gyp.txt
i had changed binding.gyp as you said, but it does not work

please try and use Refael's exact syntax:
			'msvs-settings':
			{
				'VCCLCompilerTool':
				{
					'AdditionalOptions':
						[
						'-std:c++17',
						]
				}
			}

note the placement of {' and ['s

@rvagg still not working

This is a full working example for anyone who also has this problem:
{
  "targets": [
    {
      "target_name": "proj",
      "sources": [ "src/proj.cc"],
      "cflags!": [ "-fno-exceptions" ],
      "cflags_cc!": [ "-fno-exceptions"],
      "include_dirs": [
        "<!@(node -p \"require('node-addon-api').include\")"
      ],
      'defines': [ 'NAPI_DISABLE_CPP_EXCEPTIONS' ],
    }
  ],
  "target_defaults": {
    "cflags_cc": ["-std=c++17"],
    "conditions": [
      ['OS=="mac"', {
          "xcode_settings": {
            "CLANG_CXX_LIBRARY": "libc++",
            "CLANG_CXX_LANGUAGE_STANDARD":"c++17",
          }
      }],
      ['OS=="win"', {
        "msvs_settings": {
          "VCCLCompilerTool": {
            "AdditionalOptions": [ "-std:c++17", ],
          },
        },
      }]
    ]
  }
}

The solutions above didn't work for me - the only way I could get it to work  was using
"msbuild_settings": {
    "ClCompile": {
        "LanguageStandard": "stdcpp17"
    }
}

It thows off some warnings, but it configures the project as it should and compiles

The solutions above didn't work for me - the only way I could get it to work  was using
"msbuild_settings": {
    "ClCompile": {
        "LanguageStandard": "stdcpp17"
    }
}

It thows off some warnings, but it configures the project as it should and compiles

The solutions above didn't work for me - the only way I could get it to work was using
"msbuild_settings": {
    "ClCompile": {
        "LanguageStandard": "stdcpp17"
    }
}

It thows off some warnings, but it configures the project as it should and compiles

Yes. ClCompile MSBuild item controls how compiler treats source file

The solutions above didn't work for me - the only way I could get it to work was using
"msbuild_settings": {
    "ClCompile": {
        "LanguageStandard": "stdcpp17"
    }
}

It thows off some warnings, but it configures the project as it should and compiles

Yes. ClCompile MSBuild item controls how compiler treats source file

The solutions above didn't work for me

That's strange. None of the given solutions sets the standard in the sln file for the analysis. But it works for the build.

The solutions above didn't work for me

That's strange. None of the given solutions sets the standard in the sln file for the analysis. But it works for the build.

The solutions above didn't work for me

That's strange. None of the given solutions sets the standard in the sln file for the analysis. But it works for the build.

see .vcxproj file?

The solutions above didn't work for me

That's strange. None of the given solutions sets the standard in the sln file for the analysis. But it works for the build.

see .vcxproj file?

If anyone need solution irrespective of node-gyp use can use cmake...
It easily handle these kind of things. Here is a example for you: https://www.github.com/AtiqGauri/Patternscape/tree/master/Patternscape_Native_Addon%2FCMakeLists.txt

If anyone need solution irrespective of node-gyp use can use cmake...
It easily handle these kind of things. Here is a example for you: https://www.github.com/AtiqGauri/Patternscape/tree/master/Patternscape_Native_Addon%2FCMakeLists.txt

The solutions above didn't work for me

That's strange. None of the given solutions sets the standard in the sln file for the analysis. But it works for the build.

see .vcxproj file?

Node-gyp does should be able to do this automatically, but it does not. Cmake easily does it by using
target_compile_features(project_options INTERFACE cxx_std_17)

The solutions above didn't work for me

That's strange. None of the given solutions sets the standard in the sln file for the analysis. But it works for the build.

see .vcxproj file?

Node-gyp does should be able to do this automatically, but it does not. Cmake easily does it by using
target_compile_features(project_options INTERFACE cxx_std_17)

Ran into this issue myself. Setting it through "msbuild_settings" worked but I think doing it that way messes up the node.exe hooking stuff since the compiled module would not load in electron. It also changes some other flags as well since the module no longer built as a static binary.
@refack's original suggestion worked for me. That said, the "VCCLCompilerTool" option should probably be updated to allow setting "LanguageStandard" directly instead of adding it through an additional command line option.

Ran into this issue myself. Setting it through "msbuild_settings" worked but I think doing it that way messes up the node.exe hooking stuff since the compiled module would not load in electron. It also changes some other flags as well since the module no longer built as a static binary.
@refack's original suggestion worked for me. That said, the "VCCLCompilerTool" option should probably be updated to allow setting "LanguageStandard" directly instead of adding it through an additional command line option.

Add the following to your binding.gyp:
  'msvs_settings': {
    'VCCLCompilerTool': {
      'AdditionalOptions': [ '-std:c++17', ],
    },
  },

nice, it works!!

For those coming here trying to get C++ 20 working with node addons relying on either Node directly or a derivative (e.g. Electron), don't forget that on Windows you will need to specify /Zc:__cplusplus. It took a while for me to discover this sharp windows specific edge...
https://stackoverflow.com/questions/63057156/language-supported-in-visual-studio-for-c-cplusplus-is-showing-as-c98/63057345#63057345

         "msvs_settings": {
            "VCCLCompilerTool": {
              "AdditionalOptions": [
                "/std:c++17"
              ]
            }
          },

Thank you for this. In my case, I was trying to sort out how to compile a projection that used the older C++/CX which isn't c++20 compatible, and that did it.
    "msvs_settings": {
      "VCCLCompilerTool": {
        "AdditionalOptions": ["/ZW", "-std:c++17"],
        "DisableSpecificWarnings": [4609],
      }
    },

I was really banging my head against the wall trying to figure out where the C++20 requirement was coming from when it was in the generated vcproj file. None of the other options worked with Visual Studio 2022 and its MS Build version.

In my case, the -std:c++17 didn't work with VS 2022 and MS Build. I had to use the CLCompile version mentioned above.
I was banging my head up against the wall trying to find out how to override the C++20 that node-gyp was putting into the vcxprj but never could locate the source.
This is all because NodeRT uses C++ CX to create the projections, and that is not compatible with C++ 20.
    "msbuild_settings":{
        "ClCompile":{
            "LanguageStandard": "stdcpp17",
            "AdditionalOptions": "/ZW",
            "AdditionalUsingDirectories": [
              "%ProgramFiles(x86)%/Microsoft Visual Studio 14.0/VC/lib/store/references",
              "%ProgramFiles%/Microsoft Visual Studio 14.0/VC/lib/store/references",
              "%ProgramFiles(x86)%/Windows Kits/10/UnionMetadata/10.0.20348.0",
              "$(VCToolsInstallDir)/lib/x86/store/references",
              "../../",
            ]

        }
    }

I still have this in the binding.gyp, but it is overridden by the msbuild settings above
    "msvs_settings": {
      "VCCLCompilerTool": {
        "AdditionalOptions": ["/ZW", "-std:c++17"],
        "DisableSpecificWarnings": [4609],
      }
    },

For those coming here trying to get C++ 20 working with node addons relying on either Node directly or a derivative (e.g. Electron), don't forget that on Windows you will need to specify /Zc:__cplusplus. It took a while for me to discover this sharp windows specific edge... https://stackoverflow.com/questions/63057156/language-supported-in-visual-studio-for-c-cplusplus-is-showing-as-c98/63057345#63057345

In my case, node-gyp was adding both that and the std:c++20 to the project file by default, with no obvious way to override.

This is a full working example for anyone who also has this problem:
{
  "targets": [
    {
      "target_name": "proj",
      "sources": [ "src/proj.cc" ],
      "include_dirs": [
        "<!@(node -p \"require('node-addon-api').include\")"
      ],
      'defines': [ 'NAPI_DISABLE_CPP_EXCEPTIONS' ],
      "cflags": [ "-fno-exceptions" ],
      "cflags_cc": [ "-fno-exceptions", "-std=c++17" ],
      "conditions": [
        ['OS=="mac"', {
            "xcode_settings": {
              'GCC_ENABLE_CPP_EXCEPTIONS': 'YES',
              "CLANG_CXX_LIBRARY": "libc++",
              "CLANG_CXX_LANGUAGE_STANDARD":"c++17",
              'MACOSX_DEPLOYMENT_TARGET': '10.14'
            }
        }],
        ['OS=="win"', {
          "msvs_settings": {
            "VCCLCompilerTool": {
              "AdditionalOptions": [ "-std:c++17", ],
            },
          },
        }]
      ]
    }
  ]
}

Worked with this config !!


Working Directory as Default Directory

Hi there!
I was searching why running brunch watch command failed until I saw :
Brunch will look for a directory called 'brunch' in the current working directory, if the path is omitted.

So, I was asking myself why this choice. Why ''brunch''? I'm not used to have a huge subdirectories tree. Is it possible to think about considerate current working directory as the default/first directory to look for if the path is omitted?
Bye!

Hi. Next release of brunch (0.9) is doing exactly the same thing — it compiles files in current working directory. It's almost ready.


Unknown flag --replicas

This fails
⧑ kubectl resize --replicas 2 rc cloud-api-stream
unknown flag: --replicas

This works
⧑ kubectl resize --replicas=2 rc cloud-api-stream
resized

Both should work, no?

Version:
⧑ k version
Client Version: version.Info{Major:"0", Minor:"9+", GitVersion:"v0.9.0-389-gaf25c3c671d592", GitCommit:"af25c3c671d592bf8643d65d24fb8b37b3aec994", GitTreeState:"clean"}
Server Version: version.Info{Major:"0", Minor:"9+", GitVersion:"v0.9.2-dirty", GitCommit:"013a97ea6e2a143f763035defdbcbcdc1b6d3411", GitTreeState:"dirty"}

I don't think so:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/docs/kubectl.md
/cc @jlowdermilk @brendandburns

I believe pflags requires the format --flag-name=value.

Ok, happy to close then. Its a bit surprising as most-all other CLIs I use do not enforce this restrict, or if they do, I haven't noticed.

I second the notion that this is surprising and unfortunate.
On Sun, Feb 1, 2015 at 3:41 PM, Brian Grant<EMAIL_ADDRESS>wrote:

Closed #4014
https://github.com/GoogleCloudPlatform/kubernetes/issues/4014.
--
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/4014#event-228220430
.

Its surprising even within the context of kubectl itself:
This works, surprisingly surprisingly:
$ kubectl get pods -l name=cloud-api-stream

: (

This isn't the first time I've been surprised by behavior of the pflag parser. It looks like for long arguments, the parser expects and requires --flag-name=value, but short flags (-f, -a, -s) are parsed as -c arg.
I agree this behavior is surprising and unfortunate. If we are sufficiently unhappy with it, we can fork pflag and/or submit patches.

+1 to patches
On Mon, Feb 2, 2015 at 11:39 AM, Jeff Lowdermilk<EMAIL_ADDRESS>wrote:

This isn't the first time I've been surprised by behavior of the pflag
parser https://github.com/ogier/pflag/blob/master/flag.go#L406. It
looks like for long arguments, the parser expects and requires
--flag-name=value, but short flags (-f, -a, -s) are parsed as -c arg.
I agree this behavior is surprising and unfortunate. If we are
sufficiently unhappy with it, we can fork pflag and/or submit patches.
--
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/4014#issuecomment-72522559
.

jbeda actually has committer rights on the pflag library, so send him
patches!
On Mon, Feb 2, 2015 at 3:53 PM, Tim Hockin<EMAIL_ADDRESS>wrote:

+1 to patches
On Mon, Feb 2, 2015 at 11:39 AM, Jeff Lowdermilk<EMAIL_ADDRESS>
wrote:

This isn't the first time I've been surprised by behavior of the pflag
parser https://github.com/ogier/pflag/blob/master/flag.go#L406. It
looks like for long arguments, the parser expects and requires
--flag-name=value, but short flags (-f, -a, -s) are parsed as -c arg.
I agree this behavior is surprising and unfortunate. If we are
sufficiently unhappy with it, we can fork pflag and/or submit patches.
--
Reply to this email directly or view it on GitHub
<
https://github.com/GoogleCloudPlatform/kubernetes/issues/4014#issuecomment-72522559

.

—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/4014#issuecomment-72564417
.


add support for the key_updated event for TLS key updates

Codecov Report

Merging #2355 into master will decrease coverage by 0.13%.
The diff coverage is 71.79%.

@@            Coverage Diff             @@
##           master    #2355      +/-   ##
==========================================
- Coverage   86.22%   86.09%   -0.13%
==========================================
  Files         117      117
  Lines        8648     8726      +78
==========================================
+ Hits         7456     7512      +56
- Misses        900      917      +17
- Partials      292      297       +5

Impacted Files
Coverage Δ

session.go
73.26% <ø> (ø)
:arrow_up:

qlog/event.go
100% <100%> (ø)
:arrow_up:

qlog/qlog.go
96.63% <100%> (+0.33%)
:arrow_up:

internal/handshake/crypto_setup.go
66.02% <15.79%> (-2.14%)
:arrow_down:

qlog/types.go
82.98% <86.36%> (+2.98%)
:arrow_up:

Continue to review full report at Codecov.

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


Fix typo in method name

Just fixing a small typo in method name of a sample class.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Removing files

Is there a way for bloxstrap to delete files, instead of modifying them? Thanks.

What use would this have?


Add Bottom Face Recess Option for Spur Gear Generation

Summary
This PR extends the spur gear script to support a bottom face recess, addressing the limitation of only having a top face recess.
Key Changes

Enhanced _make_recess Function:

Added bottom_recess parameter to allow recesses on the bottom face.
Parameters bottom_hub_d and bottom_recess_d are also added; if not specified, they default to the corresponding top recess values.
Added validation for scenario where only bottom_recess is being used.

Updated _build Function:

New arguments: bottom_recess, bottom_recess_d, bottom_hub_d for full control over bottom recesses.

Design Example:
Included a 5-second clip demonstrating a gear with both top and bottom recesses.
https://github.com/user-attachments/assets/7562d866-08d5-40e1-8b65-6e1fc4b7d917
Why This Change?
Adding support for a bottom recess makes the gear design more versatile, for common cases requiring symmetrical recesses on both faces for weight reduction or fit purposes.

Thanks, it works and looks fine

Thanks, it works and looks fine

Thank you! It goes without saying but amazing work on the repository. It has been incredibly helpful.


Make header images/avatars "clickable"

Header images alone are shown well, but when they are turned into links, some of image styling gets lost. See the example: https://jsfiddle.net/d7xyhmew/1/

A guess/workaround that seemingly work is like this:

  Patrick

If this guess is correct, an example should be documented.

I'm surprised this is the first report of it. I'm seeing the same issue here. While adding the style to the link helps, all padding related behaviors are lost.

@stale Ping.

Currently facing the same issue. It would be nice to be able to wrap the header image in a hyperlink and maintain the styling.


IPRoyal coupon.

The discount code youtubers80 already not working. any new valid 80% discount code?

The discount code youtubers80 already not working. any new valid 80% discount code?

Please use these poxy's they are best and i have been using it for the last few months without any error, they have good customer care as well.
https://app.privateproxy.me/from/41487

The coupon is valid, but it is only valid for ROY.
If you don't know what ROY is, read their webpage carefully.

its not valid on ROY either though.

@starverer
Seems working just fine

@starverer Seems working just fine

Hey, I made a new account and now it's working fine. Not working on my original account for some reason.
Thanks for the project :)


Code editor for full app

While the visual editor is fantastic, I find that I prefer doing it both the backend and the frontend on my code editor.
From your first article about Streamsync, I can see that this was the initial way you chose. However, you went with the visual editor. Maybe there are drawbacks to the first approach, would really like to have your input.
If not, do you plan on having this interface back? Thanks

The big issue that I didn't mention on the first post is making that we can't really have the visual and code approaches at the same time.
Maybe it is possible to put the frontend code in ui.py, and if this file is detected, then disable the possibility to use the visual editor? And there could be an option to use visual or code when creating the app as well.

This is a great question and it's made me think quite a bit.
As a general guideline on what to expect from Streamsync, I'm trying really hard to encourage best practices with minimum effort. It's of course an ongoing effort and there's a lot yet to be done.
The visual editor is that it enables separation of concerns in a rather intuitive way and lowers reliance on documentation.
Separation of concerns can of course be achieved with code, but it becomes harder to manage and it requires a lot of discipline to keep the code clean. Plus, you'd need to remember/check each of the properties for all components. I didn't even remember that Column had an option named isSticky, so I don't think others would.
This would be rather straightforward to implement, but building a whole application like this I think would be a pain.
with ssc.columns():
  with ssc.column({"width": "1", "isSticky": "yes"})
    ssc.text({"text": "Hello"})
  ssc.column({"width": "1", "title": "@{my_column_name}"})
    ssc.text({"text": "World"})

However, I'll probably experiment with it at some point, as I think it could work as a complement to ui.json. You could have "static" components (in ui.json) and "dynamic" components (rendered on-the-fly via Python calls).


Extra Metadata Loader logos overlap details section when aligned to bottom

Extra Metadata Loader logos overlap details section when aligned to bottom

fixed via 6617da00d65ae064cddb9c45cce272c4a2dfa3b7


WARN  [react-native-gesture-handler] None of the callbacks in the gesture are worklets.

Describe the bug
I am getting the following message when I try to open a modal from within a useEffect, is this allowed?
WARN  [react-native-gesture-handler] None of the callbacks in the gesture are worklets. If you wish to run them on the JS thread use '.runOnJS(true)' modifier on the gesture to make this explicit. Otherwise, mark the callbacks as 'worklet' to run them on the UI thread.
I'm using Expo 51 with "react-native-gesture-handler": "^2.17.1" and "react-native-reanimated": "^3.12.1".
Here are some code snippets:
// App.js
<Provider store={store}>
  <PersistGate loading={null} persistor={persistor}>
    <GestureHandlerRootView>
      <SafeAreaProvider>
        <ThemedApp />
      </SafeAreaProvider>
      <MagicModalPortal />
    </GestureHandlerRootView>
  </PersistGate>
</Provider>

// ThemedApp.js
const QuickModal = ({ text }) => {
  return (
    <View>
      <Text>Hey! I'm going to be closed imperatively</Text>
    </View>
  );
};
// ...
React.useEffect(() => {
  magicModal.show(() => <QuickModal />, {
    onBackButtonPress: () => {},
    onBackdropPress: () => {},
    swipeDirection: undefined,
  });
}, []);

+1

While I could not reproduce the warnings, I just merged a PR marking the functions as workouts.
This should close the issue. Let me know if it still happens on the latest version, and we can reopen it.


remove many box creations in the nested callbacks

Feature Request
Is your feature request related to a problem? Please describe:
Now for an operation, we create many boxes to wrap the callbacks to ensure these callbacks can be passed to other places and called. E.g, for the raw put API
At first, we use let (cb, f) = paired_future_callback(); to create a callback to notify the gRPC thread. Then we pass this callback to Storage async_raw_put.
In the async_raw_put, we will create another box to pass to Engine async_write
self.engine.async_write(
            &ctx,
            vec![Modify::Put(
                Self::rawkv_cf(&cf)?,
                Key::from_encoded(key),
                value,
            )],
            box |(_, res): (_, engine::Result<_>)| callback(res.map_err(Error::from)),
        )?;

Then in the async_write, we will create another box to pass to exec_write_requests
self.exec_write_requests(ctx, reqs, box move |(cb_ctx, res)| match res {

In the exec_write_requests, we finally create a box to pass to Raftstore
self.router
            .send_command(
                cmd,
                StoreCallback::Write(box move |resp| {
                    let (cb_ctx, res) = on_write_result(resp, len);
                    cb((cb_ctx, res.map_err(Error::into)));
                }),
            )
            .map_err(From::from)

As you can see, too many boxes here and these definitely can hurt the performance.
Describe the feature you'd like:
Mostly, we only need the last box because we will pass the callback to another thread, so we can remove all other boxes. We can pass Fn instead of FnBox.
Before for paired_future_callback, it is hard for us to return a Fn to pass the Rust compiling, but now the Rust has impl feature, so we can write
pub fn paired_future_callback_fn<T>() -> (impl FnOnce(T), oneshot::Receiver<T>)
where
    T: Send + 'static,
{
    let (tx, future) = oneshot::channel::<T>();
    let callback = move |result| {
        let r = tx.send(result);
        if r.is_err() {
            warn!("paired_future_callback: Failed to send result to the future rx, discarded.");
        }
    };
    (callback, future)
}

For other places, we can use FnOnce instead,
// fn async_write(&self, ctx: &Context, batch: Vec<Modify>, callback: Callback<()>) -> Result<()>;

fn async_write_fn<T: FnOnce((CbContext, Result<()>)) + Send + 'static>(
        &self,
        _ctx: &Context,
        _batch: Vec<Modify>,
        _callback: T,
    ) -> Result<()>;

I write a simple benchmark, https://gist.github.com/siddontang/fecbbfbb39be306f831623b6f05f6f50
The benchmark result:
test tests::bench_box    ... bench:      64,939 ns/iter (+/- 2,728)
test tests::bench_fnonce ... bench:      40,622 ns/iter (+/- 2,022)

As you can see, using FnOnce can improve the performance.
This just a simple test. In TiKV, we may gain less increment, but it still needs to try because we can also remove lots of memory allocation.
Describe alternatives you've considered:
At first, to reduce the cost of the box, we considered using channel directly, not passing box, this can work for the final step. For other steps, we can just remove the box and use FnOnce.
Teachability, Documentation, Adoption, Migration Strategy:
This is a big change, but luckily it is compatibility, but we still need to write an RFC at first. /cc @overvenus

To reduce nested FnBox, impl FnOnce is a good choose, it is easy to implement.
We may consider Future which reduces FnBox too. Also it can be integrated to coprocessor and storage seamlessly.


additions to screen_daily_data function

This is a reminder for myself mostly to add a line in the screen_daily_data function to remove estimated as well as provisional data and also to add the site number to the output table in this function.  Should be a quick fix but I just don't want to forget over the weekend.

Has been fixed and merged.

Did you fix this in your copy of the repo?

oh crud - yes, its in my fork but not in this version. What step did I miss? Why is this so confusing to me?!

ok - I forgot to merge it after you reviewed it.  I think it should be updated now.

Thanks! I think GitHub will close these issues automatically after the merge when the issue is listed in the PR, so that's why I caught it. I just realized that we didn't add "closes #15" to that PR, so I added that to the comments

Actually, it's in the "Linked issues" option on the right panel of the PR


Missing Fitness Access popup

Hi I have used your widget in other apps, but this particular app using the newest widget is not popping up the authorization screen, I am only seeing the account selection.

Hey @troyzonzon could you be more specific. Android or iOS? Which version?

Hi, I have the same problem with my app. I'm using the Android version. Every time I start the app the account selector pops up, when I click on my account nothing happens. When I try to read data using this:
final results = await FitKit.read(
      DataType.DISTANCE,
      dateFrom: DateTime.now().subtract(Duration(days: 5)),
      dateTo: DateTime.now(),
   );
I get this error:
PlatformException (PlatformException(FitKit, User denied permission access, null))

@ltonn02 can you please verify this: https://github.com/krokyze/FitKit/issues/45#issuecomment-643667079

@krokyze thanks for your advice, I think I fixed the issue. You need to add the release SHA-1 to the debug configuration as well.
My build.gradle now looks like this:


Pinot UI doesn't set correct Auth header for queries

Try with 0.8.0 release(https://downloads.apache.org/pinot/apache-pinot-0.8.0/apache-pinot-0.8.0-bin.tar.gz) quickstart:
bin/quick-start-auth.sh

Once it's up, go to http://localhost:9000/
Try out user credential: user/secret
Then click on table name baseballStats:

This also happens when click RUN QUERY button.
Schema call has the Authorization header, but seems not verified:

Query call has no header:

Tried with admin permission: admin/verysecret
Then click on table name baseballStats:
Schema call is ok, but the query is still missing auth header.

Schema call has the Authorization header, and verified:

Query call has no header:

@shahsank3t

@apucher

@xiangfu0
/sql call issue has been fixed on master branch by @joshigaurava in https://github.com/apache/pinot/pull/7230
We need to cherry-pick in 0.8.0 release or will be part of the next release.
/schema call issue with user credentials that doesn't verify the user needs to be checked by @apucher.

I fixed the user permission validation for /schema calls
So it should be ok now.
Thanks @shahsank3t !


Etcd service is abnormal after deleting one node

Rancher versions:v1.6.17
rancher/server:v1.6.17
rancher/agent:v1.2.10
Infrastructure Stack versions:
network-services:vxlan
kubernetes (if applicable):rancher/k8s:v1.10.0-rancher1-2
Docker version: 17.03.2-ce
**Operating system and kernel:ubuntu16.04,4.4.0-87-generic,,
Type/provider of hosts: XenServer7.2
Environment Template: Kubernetes
Steps to Reproduce:
When I add two nodes to rancher and everything is ok.
Then I delete one node from Rancher UI
Results:
the etced service  create containers constantly.

Thanks for your report!
With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.
If you think we should keep this issue open, please let me know.


Bookmarks missing from latest TestFlight

Some have disappeared - should test possible regressions like this in the future

Bug Report Dump (Auto-generated)

Version 1.12.0 (1137)
Device: iPhone 7 Plus (iOS 11.1)
TestFlight: true

All of mine have disappeared bit annoying, is this because they're now namespaced per-account? Likely had no migration.
I have (using FLEX) a "bookmarks" and a "com.whois.freetime.bookmarks" directory with a file within. Without looking at the code I assume the directory is the new one, with each file being an account. We could fairly easily just check for the "bookmarks" file and migrate it over? (or just tell people to deal with it 🤷♂)

Namespaced bookmarks were in TF version 1127. Ideally, we should not get this missing bookmarks in TF 1137 🤔

I don't think I used TF 1127 tbh, so moving straight to 1137 likely caused the issue. Bas said "some" have disappeared, the ones he had before 1127 would've been deleted and any new ones will have stayed?
A possibility anyway

Think the new ones are gone; not sure though.

@Sherlouk Yup.
TF 1127 has this change for namespaced bookmarks. Previously all bookmarks saved under documents folder with directory named Bookmarks. Now with namespaced changes, we created a new directory com.whois.freetime.bookmarks and save each user bookmarks under file named by user_token

Maybe it is related to #822
if you are missing bookmarks.

I think since it is just on TF, we can deal with not having migration between the versions. Had it been on app store version I'd say the opposite

Yup they are flushed from the user namespace changes. Beta tester sacrifices 🙏
Sent with GitHawk


Error running "Build Image in Azure..." command

🚨 Looks like you arrived from the internal review site
🚨 Do not enter Microsoft confidential information here
Following the instructions as written results in an error when attempting to run the "Build Image in Azure..." command.
Error:
Step 4/10 : COPY ["package.json", "package-lock.json*", "npm-shrinkwrap.json*", "./"]
COPY failed: file not found in build context or excluded by .dockerignore: stat package.json: file does not exist
Container failed during run: build. No retries remaining.
failed to run step ID: build: exit status 1

It appears that there's an issue with vscode when the building a Dockerfile that is not in the root directory. As a workaround, eother:

Open the cloned repo's src directory, then build
Modify the Dockerfile as follows:

FROM node:lts-alpine
ENV NODE_ENV=production
WORKDIR /usr/src/app
#COPY ["package.json", "package-lock.json*", "npm-shrinkwrap.json*", "./"]
COPY src/ .
RUN npm install --production --silent && mv node_modules ../
#COPY . .
EXPOSE 3500
RUN chown -R node /usr/src/app
USER node
CMD ["npm", "start"]

Also note that the default image name, in either workaround, becomes src, which may not be expected. The expectation would be that it's called containerapps-albumapi-javascript.

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

ID: 61801b87-f37b-1025-c41e-73689e49b9da
Version Independent ID: f66e1382-6bd2-1b08-bb15-1e1868bf0062
Content: Deploy to Azure Container Apps using Visual Studio Code
Content Source: articles/container-apps/deploy-visual-studio-code.md
Service: container-apps
GitHub Login: @alexwolfmsft
Microsoft Alias: alexwolf

@petertuton
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for your feedback and your contribution to Azure docs.
Feedback for this repository is moving away from GitHub to a system specific to the Microsoft Learn platform. Issues for this repository will soon be disabled, and additional comments from GitHub will no longer be possible. However, we are now tracking and triaging this issue in the new feedback system.
To learn more about our feedback systems, please see Provide feedback for Microsoft Learn content.
#please-close


Openhab config

Added a python script for parsing the params.txt file into a .yaml file. Also added a pre-generated .yaml file which can be used directly in an OpenHab thing configuration.
Added some missing quotes to the params.txt file.

thanks for contribution.
I would try to use this on my test setup in the next days to see if this would work for me.

sorry, bit for some reasons I decided not to implement this


Documentation is incomplete.

Issue in ./help/segmentation/tutorials/using-external-audiences.md

The documentation says that "With the dataset enabled, data can now be sent into Platform either through the UI ". This is not true for streaming ingestion.
Internal Wiki document gives an example of a call which refers EntityUpdate within the POST request, when one has to mark a segmentMap on the profile. This is not explained or even indicated.
The page also fails to indicate what must be done (SegmentationMap) on the profile for it to work. It would have been nice to pick an example with data structures and show how it lands.
The document does not indicate how will exits and entries of segment behave. This is a standard need of the use cases and should be indicated.

Internal reference: https://wiki.corp.adobe.com/display/DMSArchitecture/Importing+and+using+External+Audiences

Thank you for submitting this issue. We will review the documentation and make the necessary updates as quickly as possible. Tracking internally in PLAT-114873.

@ankurmehrotra The documentation was reviewed and multiple updates were done by the team. Closing this issue.


decoratedModel

Allows the usage of the lib without decorators
Example
  const Point = decoratedModel(
    "decoratedModel/Point",
    // although class could be anonymous, by giving it a name
    // type declarations are shorter
    // and the IDE shows better typings if we do
    class Point extends Model({
      x: prop<number>(),
      y: prop<number>(),
    }) {
      setX(x: number) {
        this.x = x
      }

      setY = (y: number) => {
        this.y = y
      }

      setXY(x: number, y: number) {
        this.setX(x)
        this.setY(y)
      }

      get length() {
        return this.x + this.y
      }

      volatile = "volatile"
    },
    {
      setX: modelAction,
      setY: modelAction,
      setXY: modelAction,
      length: computed,
      // multiple decorators could be applied with xxx: [deco1, deco2...]
    }
  )
  type Point = InstanceType<typeof Point> // needed to be able to do SnapshotInOf<Point>, etc

  const p = new Point<number>({ x: 10, y: 20 })

missing: docs

Coverage decreased (-0.05%) to 89.483% when pulling f291a06f27c9f0c66d8c9c13c384242c4eee30d6 on decoratedModel into 3d13fa0bc2330883bef19e649f0189ae9eb1b70e on master.


Bookmarklet and speed

Imagine Jetzt lives on Firefox’s bookmarks toolbar. Upon activation it sets speed to 400 words per minute. How to change this behavior, e.g. make default speed slower? Perhaps there is (could be) a way to pass speed parameter within code of bookmarklet (so everybody could customize according to his needs)?

Err… Hello?

Don't be rude, we're all volunteers.
Please see issues like:

https://github.com/ds300/jetzt/issues/145
https://github.com/ds300/jetzt/pull/22

#145
In jetzt.solid.min.js search for the substring target_wpm:400 and change 400 to whatever WPM you like.
It implies rehosting custom version of script somewhere. Not flexible at all.
Hence the question whether author could alter current script to pass this parameter via bookmarklet.

Hey @sergeevabc , sorry I don't actively work on this project any more, but would be happy to review a PR if you have the time.

Had user been able to polish reported roughness, he would be called a developer.
The latter role is yours, hence the badge, yet passion to carry on seems to be lost.
Consider marking project as abandoned to strangle illusory hopes for changes at birth.

It is often unrealistic to expect the maintainers of an open source project
to implement one's feature requests. It is always uncouth to become pissy
when they do not.
I will not mark the project as abandoned. If bugs are reported I will
attempt to fix them. But I am not currently inclined to implement new
features for a thing I don't even use. Unless somebody is willing to pay.
You still want a customisable bookmarklet? My rate is 600€ a day.
On Thu, Dec 15, 2016 at 8:54 PM, Aleksandr<EMAIL_ADDRESS>wrote:

Had user been able to polish reported roughness, he would be called a
developer.
The latter role is yours, hence the badge, yet passion to carry on seems
to be lost.
Consider marking project as abandoned to strangle illusory hopes for
changes at birth.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ds300/jetzt/issues/159#issuecomment-267426950, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABL1qQOB2V3UKuQu5Lr2l5rL0IFaWkTuks5rIZsFgaJpZM4K4WIH
.

Core value of open source philosophy besides freedom to look inside is to make life easier voluntarily.
And making life easier means doing, even with loose schedules and vague roles, not procrastination.
Whereas you, @ds300, have been neglecting the community for years with no courage to be honest.
Instead you feed us with unfeasible promises and extort money, thus disgracing your name.

The core value of open source philosophy besides freedom is to share solutions that make one's life easier.
By releasing his software under the Apache License, @ds300 has granted everyone possibility to modify and use freely his code at one's intention. That's something huge.
Similarly, all the other things, like whether the creator will merge or update or not, is up to his intention as well.

@gywn, we’re at the public issue tracker and not within his private walls, hence the public expectations.
Would you like to share with no further responsibility? Have a courage to state this in the first place.

@sergeevabc That's exactly what the 8th term (Limitation of Liability) in the Apache License is for.

@gywn, software license might help you to talk a way out of trouble within the walls of court, outside there is always another Vitaly Kaloyev waiting to counterbalance unfair selfish recreant behaviour.
You gain people’s trust, they rely on you, and then gone fishing without notice? Unfair, selfish, recreant.

I feel like @sergeevabc is trolling us

Did he just imply that I deserve to be murdered? For releasing some free software and then not wanting to work on it anymore? Daaamn. Brother has high standards.

@sergeevabc You failed to the creator's selflessness in the first place, and turn moralism into threats of legal illiteracy. We might just end the conversations here.

Did he just imply that I deserve to be murdered?

@ds300, it means you deserve all this public censure since you offended the community (or betrayed the brotherhood, if you will) by committing the sin of sloth for years (i.e. laziness, carelessness, unwillingness to act, half-hearted effort).

…not wanting to work on it anymore…

Have the guts to inform the public a new maintainer is wanted so no improvement should be expected until then and go through opened issues and repent for unfulfilled promises and ignored calls.

ds300 took the time just a few months ago to merge a couple of pull requests: https://github.com/ds300/jetzt/pulls
Referencing Vitaly Kaloyev  is really uncalled for. @sergeevabc I've reported you with Github.

@peteruithoven, merging pull requests is interaction between developers, while dozens of issues posted by ordinary people are neglected for years, some of them stays without even a basic reply out of decency. And don’t forget to send screenshots to police to make them laugh about referencing Wikipedia.


ChatDeepInfra does not implement bindTools

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain.js documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain.js rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
Following the quickstart tutorial here, but replacing ChatOpenAI with ChatDeepInfra like this:
const model = new ChatDeepInfra({
  model: 'meta-llama/Meta-Llama-3-70B-Instruct',
  temperature: 0,
  apiKey: 'XXXXX',
}).bindTools(tools);

Error Message and Stack Trace (if applicable)
TypeError: (intermediate value).bindTools is not a function
Description

I'm trying to to build an agent using ChatDeepInfra with tools
I would expect it to implement bindTools, like the python version
Instead I get an error

System Info
npm info langchain

<EMAIL_ADDRESS>| MIT | deps: 12 | versions: 297
Typescript bindings for langchain
https://github.com/langchain-ai/langchainjs/tree/main/langchain/

keywords: llm, ai, gpt3, chain, prompt, prompt engineering, chatgpt, machine learning, ml, openai, embeddings, vectorstores

dist
.tarball: https://registry.npmjs.org/langchain/-/langchain-0.3.2.tgz
.shasum: aec3e679d3d6c36f469448380affa475c92fbd86
.integrity: sha512-kd2kz1cS/PIVrLEDFlrZsAasQfPLbY1UqCZbRKa3/QcpB33/n6xPDvXSMfBuKhvNj0bjW6MXDR9HZTduXjJBgg==
.unpackedSize: 2.9 MB

dependencies:
@langchain/openai: >=0.1.0 <0.4.0        openapi-types: ^12.1.3
@langchain/textsplitters: >=0.0.0 <0.2.0 p-retry: 4
js-tiktoken: ^1.0.12                     uuid: ^10.0.0
js-yaml: ^4.1.0                          yaml: ^2.2.1
jsonpointer: ^5.0.1                      zod-to-json-schema: ^3.22.3
langsmith: ^0.1.56-rc.1                  zod: ^3.22.4

maintainers:
- nfcampos<EMAIL_ADDRESS>- jacoblee93<EMAIL_ADDRESS>- andrewnguonly<EMAIL_ADDRESS>- davidduong<EMAIL_ADDRESS>- hwchase17<EMAIL_ADDRESS>- basproul<EMAIL_ADDRESS>
dist-tags:
latest: 0.3.2                              tag-for-publishing-older-releases: 0.2.20
next: 0.3.2-rc.0

published a month ago by jacoblee93<EMAIL_ADDRESS>
node --version
v20.10.0

Platform: Linux

Hey there @ovuruska. Would you be able to help here from the Deep Infra side?

Hi, we're a group of students from the University of Toronto and we can take a look at this and see what we can do?


fix path so files are saved into directories

I've noticed that when files are saved into S3 storage, they're saved with the url path encoded because of path-to-regexp over here .
Example:
> const pathToRegexp = require('path-to-regexp')
> const path = 'spacechop/:preset/:image';
> const toPath = pathToRegexp.compile(path)
> const params = { preset: 'keyart-png', image: 'movies/media/browser/my_movie.png'}
> toPath(params)
'spacechop/keyart-png/movies%2Fmedia%2Fbrowser%2Fmy_movie.png'

What I would prefer is that when these filenames are saved, they keep the pseudo directory structure in the url. Looking over the options for path-to-regexp, I can't see anything that let's me turn this encoding off so I've used decodeURIComponent here instead.
Example:
> decodeURIComponent(toPath(params))
'spacechop/keyart-png/movies/media/browser/my_movie.png'

No longer required as this has been handled by #263.


Test comments

Description

Type of change

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

References

How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change
[ ] example: I check if my changes works with both PySide and PyQt backends
as there are small differences between the two Qt bindings.

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

The initial link in the comment was https://nightly.link/melissawm/napari/actions/artifacts/379806765.zip
For commit f3a3be7 the link should be updated in the same "sticky" comment.


bug: unable to change scope via  in scopes window

Did you check docs and existing issues?

[X] I have read the Grapple docs
[X] I have searched the existing issues of Grapple

Neovim version (nvim -v)
NVIM v0.9.5
Operating system/version
linux
Describe the bug
Not sure if it's just me:
I am unable to change scope in the scopes window via <S-CR>; however, scope can be changed via use_scope api
Steps To Reproduce

open scopes window
hit <S-CR>
hit <-> to move back to the scopes window
current scope is unchanged

Expected Behavior

the current scope should be changed accordingly

Repro
No response

I'm also experiencing this exact same issue.

Fixed by adding the below to alacritty
[[keyboard.bindings]]
chars = "\u001B[13;2u"
key = "Return"
mods = "Shift"

explanation


generate_mil_features for extracting last layer activations in TransMIL and AttentionMIL models

Added new function generate_mil_features that returns a MILFeatures object generated from extracting activation weights from the last layer of a TransMIL, AttentionMIL, CLAM_sb, or CLAM_mb model.
Required dependencies: None
Args:
weights (str): Path to model weights to load.
config (:class:slideflow.mil.TrainerConfigFastAI or :class:slideflow.mil.TrainerConfigCLAM): Configuration for building model. If weights is a path to a model directory, will attempt to read mil_params.json from this location and load saved configuration. Defaults to None.
dataset (:class:slideflow.Dataset): Dataset.
outcomes (str, list(str)): Outcomes.
bags (str, list(str)): Path to bags, or list of bag file paths. Each bag should contain PyTorch array of features from all tiles in a slide, with the shape (n_tiles, n_features).
get_last_layer_activations in the model classes is a copy of the forward method, but simply returns the layer weights before they are converted to scores.

Thanks, Matt. I've completed my first pass changes. Could you confirm that functionality continues to work as intended on your end? I will work on more rigorous testing in the next few days.

Yes, absolutely. I will comment here once I'm done with my testing.

@CostanzaSiniscalchi tested code and pushed some changes. Everything looks good on our end.


Markdown content styling in a Markdown Layout does not match spec in documentation

What version of astro are you using?
2.6.2
Are you using an SSR adapter? If so, which one?
None
What package manager are you using?
npm
What operating system are you using?
Linux
What browser are you using?
Firefox
Describe the Bug
When using a Markdown frontmatter layout, the documentation describes being able to use <style> tags OR imported stylesheets to style markdown content.  However, the only way the markdown content is actually styled is through the use of imported stylesheets.
This could be mitigated by explaining the difference more clearly in the documentation or enabling styling markdown content in a layout using <style> tags as described in the documentation.
Link to Minimal Reproducible Example
https://stackblitz.com/edit/github-aks7dp?file=src%2Flayouts%2FWorkLayout.astro
Participation

[X] I am willing to submit a pull request for this issue.

Thanks!  I'll post below some of the confusion in the phrasing for the consideration of updating docs:
Problem
While the documentation for Markdown Styling does specify "global styles", I would argue the preceding sentence muddles up the meaning:

Any Astro styling methods are available to a Markdown layout component.

Scoped styles (e.g. creating a CSS rules for p tags in the layout component) do not work since rendered Markdown content does not have the class for the parent layout component, so I don't think "any" really applies here.
On a general note, global rules do work and are encouraged for CMS style content that lives outside of Astro, however I wouldn't categorize Markdown content from a Content Collection as living outside of Astro, since it's part of the Astro ecosystem, though that could be a misinterpretation on my part. Nonetheless, it's another part of the confusion.
Imported stylesheets seem to be the best and simplest way of applying stylings to rendered content within a Markdown Layout without having to resort to global styles. I think this is key to call out because having to understand why the layout classes aren't applied to rendered markdown content should not be something the typical user needs to understand.

All in all, to me, content rendered within the layout should include the stylings applied to the layout. I understand this is not the case, but I imagine others might similarly intuit my interpretation.
At issue is in part how markdown content is rendered on the page and partly the cascading order of stylings within Astro.
Suggestion
I would suggest rephrasing the Markdown Styling section to something along these lines:

Any Astro styling methods are available to a Markdown layout component. Please note that layout styling is not the same as the rendered Markdown content styling.

You can apply global stylings to your rendered Markdown content by adding

Very good, I'll create a PR for this soon.

@delucis and @sarah11918 Apologies for the delay, haven't had a lot of bandwidth to sit down and get this done, but I just opened up the small PR with the changes described.


chore: license update on package.json and minor style change at webpack.config.js

Not much, really. Just some minor details I noticed.

Maybe the other branch's license should be updated as well.

thanks for the PR but I updated repo in another PR


feat(ContinuousFunctionalCalculus): Show that integration commutes with the CFC

This PR shows that cfc (fun r => ∫ x, f x r ∂μ) a = ∫ x, cfc (f x) a ∂μ under appropriate conditions.
The continuity condition given here is enough for most applications (I think) and reasonably easy to apply for the integral representations this is meant for, but I would be open to suggestions.

bors r+


Skype Channel Video Cards don't play anything

The Microsoft Bot Framework team prefers that "How To" questions be submitted on Stack Overflow. The official Bot Framework GitHub repository  is the preferred platform for submitting bug fixes and feature requests.
Bot Info

SDK Platform: .NET
SDK Version: V4 Preview
Active Channels: Skype, Web Chat
Deployment Environment: Azure Bot Service

Issue Description
Skype does shows a empty video card box with no content instead of
Code Example
        private async Task MessageReceivedAsync(IDialogContext context, IAwaitable<object> result)
        {
            Activity activity = (Activity)await result;

            IMessageActivity msg = context.MakeMessage();
            msg.Attachments.Add(new VideoCard()
            {
                Autostart = true,
                Autoloop = true,
                Media = new List<MediaUrl>
                {
                    new MediaUrl()
                    {
                        Url = "http://localhost:3978/Videos/playing.mp4",
                    }
                },
            }.ToAttachment());

            msg.Text = $"Hello World!";

            await context.PostAsync(msg);
            context.Wait(this.MessageReceivedAsync);
        }

Reproduction Steps

Create a video card Echo bot
Register to Skype Channel
Message Skype Bot
Observe Failing Video Card Response from Bot

Expected Behavior
For the Video Cards to play like they did in the web test
Actual Results
What happens on Skype

What happens on Web Test

This may just simply be because you are using an http endpoint instead of an https endpoint.

@Jagreaper Did adding those work for you?


minor modification to sample/download_files.sh

The extracted files of SETIMES2.ro-en.txt.zip are named as SETIMES.en-ro . Not as SETIMES2.en-ro.

thanks! This renaming seems to be due to a new release of SETIMES, but with the same amount of data.


Whitespace deltas in wiki.js patch

Hello, Joshua!  Working on a project (MIDI 2.0 guidebook) that needs to depict JSON schemas with pseudocode examples, and I'm investigating your plugin here.  It looks well-suited to my uses.
The wiki.js patch seems to include numerous whitespace-only changes (probably an overzealous text editor converting tabs to spaces or vice versa).  I worry these might increase the likelihood of a conflict with future versions of the TiddlyWiki core.  (Admittedly ignoring whitespace-only diffs might be a good option for the modloader in the future.)

Hi Evan!
That has been a very strange bug I have not been able to track down. I thought it was due to line-ending settings during git checkouts, and fixing that to match the TW core repo did fix lot, there was always still whitespace chunks I couldn't figure out. Thanks for any help you can provide on that front.


crash when disconnect after commit in windows

tcp_client::disconnect whill call io_service::untrack to erase sock_info,
but this sock_info may used in io_service::process_io
auto redis_sub = new cpp_redis::redis_subscriber();
redis_sub->psubscribe(xxx);
redis_sub->commit();
delete redis_sub;

i think sock_info should delay delete to all io_context is return back

Mmmh, I'm going to look into that as soon as I have some time.
I'm not the developer who wrote the windows port of the library, so might take me a bit more time to go through windows debug.
Thanks again for reporting several issues on windows!

Closing as it is not relevant anymore after the new release v3.0.0 as explained in #39.


Upstart - Mysql error

Hi,
I have an issue with Mysql.

sudo service mysql start
initctl: Impossible de se connecter à Upstart: Failed to connect to socket /com/ubuntu/upstart: Connexion refusée
/lib/lsb/init-functions.d/50-ubuntu-logging : ligne 5 :   214 Abandon                 (core dumped) plymouth --ping > /dev/null 2>&1
Starting MySQL database server mysqld                                                                                /lib/lsb/init-functions.d/50-ubuntu-logging : ligne 5 :   671 Abandon                 (core dumped) plymouth --ping > /dev/null 2>&1
[ OK ]
Checking for tables which need an upgrade, are corrupt or were
not closed cleanly.

With this

mysql --host=localhost --user=root -p
Enter password:
ERROR 2002 (HY000): Can't connect to local MySQL server through socket '/var/run/mysqld/mysqld.sock' (2)

Not exactly the same error with this

mysql --host=<IP_ADDRESS> --user=root -p
Enter password:
ERROR 2003 (HY000): Can't connect to MySQL server on '<IP_ADDRESS>' (111)

i try this

mysql -uroot -p --host=localhost --port=3308 --protocol=TCP
Enter password:
ERROR 2003 (HY000): Can't connect to MySQL server on 'localhost' (111)

Build 15063.138
Need some help please... thank you! :-)

The service command might not be working (yet)?
Service mainly runs a bash script, usually in /etc/init.d. You can try just running the bash script and see what happens. There should also be a mysqld_safe script that should also work. If you start the server that way, and it doesn't keep running, you know you are dealing with a server problem, not a service problem.
You can use ps -ef or some variation, to see if the mysqld daemon is running. If it is not running, there's no chance the mysql client can connect to it. So that is a different issue from when the daemon is running but the socket or port doesn't work.

Ref #1611 (comment).


[C] Fixed small typo in CHeader.cs

I'm really sorry for that.

Hello! I'm the build bot for the Mono project.
I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done.
Contributors can ignore this message.

whitelist