Problem while trying to sync a remote using Irmin_mirage

Hi!
My goal was to sync a remote repository with a local in-memory Irmin_git store, but I can't make it work within mirage.
I have the following config.ml
open Mirage

let stack console = generic_stackv4 console tap0

let main =
  let libraries = ["irmin.git"; "mirage-http"; "irmin.mirage";"tls.mirage"] in
  let libraries = if get_mode () = `Xen then libraries else "irmin.unix" :: libraries in
  foreign
    ~libraries
    ~deps:[abstract nocrypto]
    ~packages:["irmin"; "mirage-http"; "mirage-flow";"tls";
               "mirage-types-lwt"; "channel"; "git"; "mirage-git"; "git-unix"]
    "Unikernel.Main" (console @-> resolver @-> conduit @-> job)

let () =
  let sv4 = stack default_console in
  let res_dns = resolver_dns sv4 in
  let conduit = conduit_direct sv4 ~tls:true in
  register "test" [
    main $ default_console $ res_dns $ conduit
  ]

and unikernel.ml
open Lwt
open V1_LWT

module Main  (C: CONSOLE) (RES: Resolver_lwt.S) (CON: Conduit_mirage.S) = struct

  let task s = Irmin.Task.create ~date:0L ~owner:"Server" s

  let start console res ctx _ =

    let module Hash = Irmin.Hash.SHA1 in
    let module Context =
      ( struct
        let v _ = Lwt.return_some (res, ctx)
      end : Irmin_mirage.CONTEXT)
    in
    let module Mirage_git_memory = Irmin_mirage.Irmin_git.Memory(Context)(Git.Inflate.None) in
    let module Store = Mirage_git_memory(Irmin.Contents.String)(Irmin.Ref.String)(Hash) in
    let module Sync = Irmin.Sync(Store) in
    let config = Irmin_mem.config () in
    let new_task () = Store.Repo.create config >>= Store.master task in

    let uri = "git://github.com/Engil/KaamelottAsAService.git" in
    let upstream = Irmin.remote_uri uri in
    new_task () >>= fun t ->
    Lwt.catch
      (fun () -> Sync.pull_exn (t "Updating") upstream `Update)
      (fun e -> Lwt_io.printlf "%s: %s"
          uri (Printexc.to_string e)) >>= fun _ ->
    Lwt.return_unit

end

And after running mirage configure --unix --net socket, make, and then ./mir-test, I got the following error:
git://github.com/Engil/KaamelottAsAService.git: Invalid_argument("invalid bounds in Cstruct.get_uint8 [43,0](40356) off=0 len=1")

I don't know where I should start to understand what's going on, I tried with various repositories and the result is still the same. I am doing something wrong with Irmin's API ?
I can provide more details if needed.
Cheers,

I think it's because of this line:
let module Mirage_git_memory = Irmin_mirage.Irmin_git.Memory(Context)(Git.Inflate.None) in

Can you try with Git_unix.Zlib instead of Git.Inflate.Context and see if it works under unix? This was functorized if someone wants to change the compression algorithm with a different server implementation. Maybe not a good idea to expose that interface directly ...

Looks like it works using Git_unix.Zlib, thank you. :)
Is there a way to accomplish the same thing when not running under unix ?

You have two options to use that under Xen:

you can either use the irmin branch on mirage-dev: https://github.com/mirage/mirage-dev/tree/irmin. It's not been updated since a long time so it's probably broken now but should be easy enough to upgrade. The branch contains a patched version of camlzip to run with Xen.

or you can adapt Decompress to work with Irmin. It's something I wanted to do for a long time and it's definitely the way forward, so if you have the time and patience, that'd be great if you can look at it and at least evaluate how hard it is to do the plumbing.

Ok,
Seems like @dinosaure is already working on making Decompress's interface similar to camlzip, so I guess I will wait for that before.
Thank you. :-)

On 03/08/2016 03:08 PM, Enguerrand Decorne wrote:

Ok,
Seems like @dinosaure https://github.com/dinosaure is already
working on making Decompress's interface similar to camlzip, so I
guess I will wait for that before.
Thank you. :-)

You may be interested in some work I did getting a MirageOS unikernel to
speak with GitHub and a remote Irmin server, which uses Decompress:
https://github.com/yomimono/example-unikernels/blob/store_in_irmin/https-client/unikernel.ml
Thanks,
Mindy

I think this is fixed with decompress 0.4...

I am closing the issue seems it seems to be fixed upstream. I really want to have Decompress be the default for ocaml-git, but this needs more work. Luckily @dinosaure is working on this!


Rename exceptions to errors

Intead of ValidationException, name the class ValidationError to follow JS conventions of naming the exceptions errors.

Fixed in c78cf8fd27f3c0028e884f74a068cc94a6bd9fe4


Added ratchet icons to build

Fixes https://github.com/ekonstantinidis/react-cordova/issues/8

Awesome! @jwulf :+1:


@vuex-orm namespace & Plugin package naming

I wanted to publish the orm-vuex-apollo plugin with the name @vuex-orm/plugin-apollo, because the search plugin uses a similar name and well it made sense for me.
I got Error: Forbidden : @vuex-orm/plugin-apollo, which actually makes sense, due to the fact that this namespace doesn't belong to me.
Should I simply rename my plugin or does it make sense to have all plugins under the same namespace?

It would be nice to add it under the vuex-orm namespace if you're OK with it! Let me email you so that I can add you to the organization and such 👍

@phortx And sorry, I didn't know your email address :p  Could you send me your NPM account at<EMAIL_ADDRESS>?


could you make me use bower?

i want to use bower for adding this library.
plz add bower.json here

I'd rather make you use npm

You can use npm to install this library in a


fix(versions): update Activiti/activiti-cloud-full-chart-security versions

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.182

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.182


Select missing opts Prop

The prop opts has been removed from Select. (#215)
It does not offer an alternative for all needed other options like disabled.

https://github.com/msaraiva/surface/issues/200#issuecomment-727577086
This is not enough because of that part of the select docs:

All other options are forwarded to the underlying HTML tag.

@maennchen you absolutely right. It shouldn't have been removed. I missed that in the review. We need to add it back.
Thanks for reporting the issue.


403 This action is unauthorized.

Version information

Laravel: v 5.8
Voyager: v 1.2
PHP:  7.2
Database: [mysql] [7.2.11] (e.g. MySQL 8.0)

Description

i have a problem with my default admin when i'm try to browse my Beard show message
say:  403 This action is unauthorized.
and  i'm can't find any solve for this problem
Steps To Reproduce
Steps to reproduce the behavior:

Go to '.create Voyager with dummy'
Click on 'create Beard '
Scroll down to 'open the beard to browse '
See error

Additional context
if and one has the same problem or has way to solved please help

Maybe you need you need to run php artisan route:clear. but please enable debug mode from .env and attach screenshot for the errors

why i'm need to run route:clear , i'm not understand

and i'm enable debug

problem solved

@mohamedadel2018 how did you solve it?

in BREAD my model path was \App[model_name], so i chang it to App[model_name] and it start working.

@mohamedadel2018  please, how did you solve it? I have this same error.

in BREAD my model path was \App[model_name], so i chang it to App[model_name] and it start working.

@josetiagobispo, in BREAD, I have same it, but not work.

in BREAD my model path was \App[model_name], so i chang it to App[model_name] and it start working.

hello please i still have same isue here

in BREAD my model path was \App[model_name], so i chang it to App[model_name] and it start working.

hello please i still have same isue here

https://github.com/the-control-group/voyager/issues/3722#issuecomment-439708646


Unable to get GCM token

Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository file a Github issue.
For general technical questions, post a question on StackOverflow tagged appropriately.

Step 2: Describe your environment

Device: iPhone 6S Plus iOS 10.2
SDK version: Latest with pod update

Step 3: Describe the problem:
I tried following exactly in the sample but I get this in my log.
2016-12-30 03:11:08.585: GCM | GCM library version 1.1.4
2016-12-30 03:11:08.669: GCM | GCM registration is not ready with auth credentials
Could not connect to GCM: The operation couldn’t be completed. (com.google.gcm error 501.)
af442b82bffdf3bc1c1a30189be3edd77c9fe73e0b486b1fad58bdb535a1ea85
2016-12-30 03:11:25.255376 WeddingLah[6326:1609423] XPC connection interrupted
Registration to GCM failed with error: Optional("The operation couldn’t be completed. (com.google.iid error 0.)")
af442b82bffdf3bc1c1a30189be3edd77c9fe73e0b486b1fad58bdb535a1ea85
Registration to GCM failed with error: Optional("The operation couldn’t be completed. (com.google.iid error 0.)")
Things that I tried
Sanbox = true
regenerate cert and upload them
specify dev team and profile
Does anyone still has any suggestion apart from the 2 closed issue?

Closing this issue due to age (it's been a while since the last update).  If you still want to discuss, feel free to re-open.


Allow ConvertOptions.timestamp_parsers for date types

Currently, the timestamp_parsers option of the CSV reader only works for timestamp datatypes.
If one wants to immediately read dates as date32 objects (in my use case csv data is read and stored as parquet files with correct types), one has to cast the table to a schema with date32 types after the fact.
This snipped shows that loading the data fails when specifying the date type:

import pyarrow as pa
from pyarrow import csv

def open_bytes(b, **kwargs):
    return csv.open_csv(pa.py_buffer(b), **kwargs)
def read_bytes(b, **kwargs):
    return open_bytes(b, **kwargs).read_all()

rows = b"a,b\n1970/01/01,1980-01-01 00\n1970/01/02,1980-01-02 00\n"
schema = pa.schema([("a", pa.timestamp("ms")), ("b", pa.string())])
opts = csv.ConvertOptions(column_types=schema, timestamp_parsers=["%Y/%m/%d"])
table = read_bytes(rows, convert_options=opts)
assert table.schema == schema # works

schema = pa.schema([("a", pa.date32()), ("b", pa.string())])
opts = csv.ConvertOptions(column_types=schema, timestamp_parsers=["%Y/%m/%d"])
table = read_bytes(rows, convert_options=opts) # error here
assert table.schema == schema
---------------------------------------------------------------------------
ArrowInvalid                              Traceback (most recent call last)
Input In [134], in <cell line: 22>()
     20 schema = pa.schema([("a", pa.date32()), ("b", pa.string())])
     21 opts = csv.ConvertOptions(column_types=schema, timestamp_parsers=["%Y/%m/%d"])
---> 22 table = read_bytes(rows, convert_options=opts)
     23 assert table.schema == schemaInput In [134], in read_bytes(b, **kwargs)
      9 def read_bytes(b, **kwargs):
---> 10     return open_bytes(b, **kwargs).read_all()Input In [134], in open_bytes(b, **kwargs)
      5 def open_bytes(b, **kwargs):
----> 6     return csv.open_csv(pa.py_buffer(b), **kwargs)File ~/.virtualenvs/ogi/lib/python3.9/site-packages/pyarrow/_csv.pyx:1273, in pyarrow._csv.open_csv()File ~/.virtualenvs/ogi/lib/python3.9/site-packages/pyarrow/_csv.pyx:1137, in pyarrow._csv.CSVStreamingReader._open()File ~/.virtualenvs/ogi/lib/python3.9/site-packages/pyarrow/error.pxi:144, in pyarrow.lib.pyarrow_internal_check_status()File ~/.virtualenvs/ogi/lib/python3.9/site-packages/pyarrow/error.pxi:100, in pyarrow.lib.check_status()ArrowInvalid: In CSV column #0: CSV conversion error to date32[day]: invalid value '1970/01/01'

It would be useful to allow the timestamp_parsers for date types as well (or add an analogous argument for dates), such that such errors don't occur and the resulting table has the required datatypes without a casting step.

A little bit more context is in the comments of https://issues.apache.org/jira/browse/ARROW-10848 (26/Oct/22).
Reporter: Tim Loderhose
Watchers: Rok Mihevc / @rok
Related issues:

[C++] Strptime issues umbrella (is a child of)

Note: This issue was originally created as ARROW-18166. Please see the migration documentation for further details.

timestamp_parsers should work on any type of date or timestamp type. Right now it only applies to timestamp types..
If you schema map a CSV column to timestamp it will get converted to a timestamp column.
If you schema map a CSV column to date you get an error that '2023-01-01' is not a date32 even if you add '%Y-%m-%d' to the timestamp_parsers list..
The only work around is to schema map a CSV column to timestamp.. Then apply a cast("date32") on it..
I think the best solution is to either:
A. have timestamp_parsers also parse date32/64 columns in a schema definition.
or
B. add a date_parsers arg which will support a list of date formats to check.. i.e. '%Y-%m-%d'
You basically can't read date text in CSV files into Arrow Date columns right now..

Probably either A or B would be useful, but would require some C++ refactoring. PRs are always welcome :).


Sort memes alphabetically in the list

Currently, the memes are ordered in the autocomplete feature, but not in the list of meme when /meme is called.

It would be handy if they were sorted.
If we implement this change, can we implement the sort directly in getAvailableMemes() (plugin.go) as we always want them sorted ?

Thanks for the suggestion @nathanaelhoun :+1:

If we implement this change, can we implement the sort directly in getAvailableMemes() (plugin.go) as we always want them sorted ?

That is totally reasonable.


Placeholder not added with Django 1.11

I've just stumbled into it locally, so not had chance to investigate.
Basically, with Django 1.11, the placeholders aren't being added to the output.
We rely on the default behaviour of adding the placeholder from the label.
A basic version of our usecase:
# forms.py

class SomeForm(forms.Form):
    some_field = forms.IntegerField(label="This is the label", help_text="this field is for things")

# page.html
{% load bootstrap3 %}

{% bootstrap_field some_form.some_field show_help=False %}

I'll investigate further in a bit, and probably write a fix.
output of what I had (the top one is just a text field):

https://github.com/cybojenix/django-bootstrap3/tree/fix-placeholder

Can you submit the fix as a Pull Request on the develop branch? Thanks!

Op 7 apr. 2017, om 10:47 heeft Anthony King<EMAIL_ADDRESS>het volgende geschreven:
https://github.com/cybojenix/django-bootstrap3/tree/fix-placeholder https://github.com/cybojenix/django-bootstrap3/tree/fix-placeholder
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/dyve/django-bootstrap3/issues/387#issuecomment-292479012, or mute the thread https://github.com/notifications/unsubscribe-auth/AAFBiwQTV577Wya1U7fe4AZqqDqyzub2ks5rtfgVgaJpZM4M1g12.

@dyve it's not a fix yet. it's just a test case, posting it here for tracking purposes is all.
Will be a PR when fixed

Ah, wishful thinking on my part, apologies! Will look before replying next time :-)

Op 7 apr. 2017, om 10:49 heeft Anthony King<EMAIL_ADDRESS>het volgende geschreven:
@dyve https://github.com/dyve it's not a fix yet. it's just a test case, posting it here for tracking purposes is all.
Will be a PR when fixed
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dyve/django-bootstrap3/issues/387#issuecomment-292479573, or mute the thread https://github.com/notifications/unsubscribe-auth/AAFBi30lx0FGElh3IFscd-FyFzShztAaks5rtfi0gaJpZM4M1g12.

Thanks to everyone that helped with this!


Add synchronous interface

It can be easily implemented via async interface, just with Result instead of TelegramFuture.

What would be involved in getting this done? Would this be a new method on the Api, something like send_sync? Would an additional Connector trait also be required that returns a Result instead of a TelegramFuture?
If so, the a synchronous Connector implementation would also be required?

Maybe it can be implemented via custom API type — SyncApi or something with the same methods except for the return type. Something like:
impl SyncApi {
    pub fn send<Req: Request>(&self, request: Req) -> Result<<Req::Response as Response>::Type> {
        self.core.run(self.async_api.send(request))
    }
}

What do you think?

Ok, pretty straightforward. When I was trying to picture how this would hang together, I imagined that it would just use a standard hyper request (without futures), as it seemed strange to create and use an event loop for running one synchronous request at a time.
But I can see that using using the asynchronous api call within the synchronous call would required less code and reduce code duplication.
So I'm not sure, it's your call.

Also this implementation provides one more benefit — request timeouts, I think it's hard to do it without async backend.
Another question is how to implement sync stream of updates.


ROUGE-SU4 is actually ROUGE-S4

An issue coming from pythonrouge, which calls ROUGE-SU4 what is actually a ROUGE-S4 score.
https://github.com/tagucci/pythonrouge/issues/7

Fixed in github.com/pltrdy/files2rouge/commit/b04afd7709a1819c33aa6f55632dab0c13e3a1b0


Can't work now?

I tagged a stared github repo, but the tags missing when I refresh or restart Chrome.
what's wrong?
Chrome: 版本 50.0.2661.102 m
OS: Win 10 14342

Hi,
I cannot reproduce the issue.
Does this happen consistently?

Yes, everytime.

Have you tried to remove the extension and re-install it?

@artisologic
Oh, It working when reinstall this amazing plugin.
But, My tags had lost, so, how to restore them. :sob:

I am not sure how to restore them to be honest. The sync is handled by google so not much I can do unfortunately, I will try and look into it.

can't work in
Chrome: 57.0.2987.133 (64-bit)

@sinchang I am running the exact same version of Chrome as you and everything still works fine for me. Can you give a bit more info about what's not working? Are you definitely looking at the stars page? (https://github.com/stars)

@artisologic sorry, can't work because I visit https://github.com/sinchang?tab=stars, working in https://github.com/stars. thanks.

Cool! 🙂


allow trailing comman in type arguments list

Fixes #21984

seems not my fault

@weswigham any objections to this change?

It seems like this, #21986, and #21984 are stagnating without comment/objection or merge/completion. What's the deal? If there's a reason not to allow this, and therefore not to merge this or #21986, and to close #21984, can it be posted somewhere?

Closed why?

@bensaufley


Create branch if missing in hugginface-cli upload

Solves https://github.com/huggingface/huggingface_hub/issues/1841 and https://github.com/huggingface/huggingface_hub/issues/1657 cc @AlpinDale
With this PR, running huggingface-cli upload ... ... --revision=my-branch will automatically create the branch if it does not exist yet. I added some tests and adapted the docs.
Note:

it is similar to repo creation if repo doesn't exist yet
branch is created from the latest main
does not apply if --create-pr

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks for the quick review @LysandreJik!

API makes sense to me too


send valid JSON in error reponse

use string type for object names
see p. 3 in Ecma JSON standard
https://www.ecma-international.org/publications/files/ECMA-ST/ECMA-404.pdf

thanks for the drive by fixup! :)


Possible memory leak when monitoring large number of prefixes

Hello,
Before anything, thanks for this project. I've been watching it for a while, and now finally deployed it... I hope I'll be able to contribute to at some point, for now, at least, with some reports.
Describe the bug
Well, it is not exactly a bug (or maybe it is), but I've noticed the memory is steadily increasing. In 24 hours after deploying the app, the memory footprint grew from 150MB to over 800MB, and it looks like it's only going to be getting worse - see screenshot below:

The CPU usage seems pretty stable however.
I'm yet to play with Node Js, but in general a pattern like that means there's a possibility of a memory leak somewhere.
Provide an example
Provide an example in terms of prefixes and BGP messages. Possibly provide a snippet of config.yml and prefixes.yml.
Nothing really specific, pretty much all the prefixes from https://bgp.he.net/AS14061#_prefixes and https://bgp.he.net/AS14061#_prefixes6, e.g.,
"<IP_ADDRESS>/21":
  description: "Some description"
  asn: 14061
  ignoreMorespecifics: false
  ignore: false

3581 prefixes in total.
Expected behavior
Given the high number of prefixes, I'd expect to see a pretty important memory footprint, but I find an always growing figure more worrying because it's not easily predictable how much resources you're going to need on the long term.
Are you using the binary or the source code?
A custom Docker image based on nttgin/BGPalerter, so source code.
Your information
Provide your name and your AS/company (see Bert Hubert's post https://berthub.eu/articles/posts/anonymous-help/).
AS14061, DigitalOcean.
Cheers!

Hello Mircea,
Thanks for the report!
I am really interested in understanding this so I will ask you more questions. I hope you can assist me because I am (both privately and as NTT) monitoring a greater amount of prefixes but I cannot see the same memory pattern.

Could you please post here your config.yml? I would like to check what monitors and reporting you have in place (be careful to don't post passwords or webhooks contained in the configuration).
Also, high values for notificationIntervalSeconds increases the memory footprint during such interval. A alertOnlyOnce set to true is known to create a memory leak (described also in the config). The reason is described here.

Are you monitoring something that generates a lot of BGP updates (not necessarily alerts)?

Could you please let it run longer and let me know if the memory consumption stabilise after some time?

Is there any warning/error in logs/error-*.log?

Thanks

Hi @massimocandela,
Thanks for looking into this. Yes, it's quite possible there's a misconfiguration on my side. Please find below the details.

Could you please post here your config.yml?

It's simply https://github.com/nttgin/BGPalerter/blob/master/config.yml.example, with the only difference that the reportSlack section under reports is uncommented (and provided with the webhook URL).
Perhaps it also worth mentioning that in prefixes.yaml, besides the configuration I've mentioned, I also have:
options:
  monitorASns:
    14061:
      group: default

Are you monitoring something that generates a lot of BGP updates (not necessarily alerts)?

Normally, no.

Could you please let it run longer and let me know if the memory consumption stabilise after some time?

I let it run over the weekend, but due to memory limits (2GB), the application has been restarted (two times within 72h) - so it continues to grow linearly till 2BG at least. I can increase the limits if you'd like me to keep watching beyond 2GB.

Is there any warning/error in logs/error-*.log?

Yes, quite a few:
/opt/bgpalerter/logs # cat error-2019-12-21.log
2019-12-21T11:47:56.885Z [production] info: ris connector connected
2019-12-21T11:55:20.530Z [production] error: Error: Client network socket disconnected before secure TLS connection was established
2019-12-21T12:20:20.095Z [production] error: 1006
2019-12-21T12:20:25.567Z [production] info: ris connector connected
2019-12-21T14:04:01.480Z [production] error: Error: Client network socket disconnected before secure TLS connection was established
2019-12-21T16:01:37.115Z [production] error: Error: Client network socket disconnected before secure TLS connection was established
2019-12-21T18:03:11.869Z [production] error: Error: Client network socket disconnected before secure TLS connection was established
2019-12-21T19:00:30.542Z [production] error: 1006
2019-12-21T19:00:36.049Z [production] info: ris connector connected
2019-12-21T20:31:13.935Z [production] error: Error: Client network socket disconnected before secure TLS connection was established
2019-12-21T21:24:53.946Z [production] error: 1006
2019-12-21T21:24:59.385Z [production] info: ris connector connected
2019-12-21T22:04:46.337Z [production] error: Error: Client network socket disconnected before secure TLS connection was established
/opt/bgpalerter/logs #
/opt/bgpalerter/logs # cat reports-2019-12-21.log
2019-12-21T11:53:01.282Z [production] verbose: AS14061 is announcing 2400:6180::/48 but this prefix is not in the configured list of announced prefixes
2019-12-21T13:58:07.011Z [production] verbose: AS14061 is announcing some prefixes which are not in the configured list of announced prefixes: 2400:6180::/48,2a03:b0c0::/48,2a03:b0c0:1::/48,2a03:b0c0:2::/48,2a03:b0c0:3::/48,<IP_ADDRESS>/17,<IP_ADDRESS>/19,<IP_ADDRESS>/19,<IP_ADDRESS>/19,<IP_ADDRESS>/20,<IP_ADDRESS>/19,<IP_ADDRESS>/22,<IP_ADDRESS>/21,<IP_ADDRESS>/20,<IP_ADDRESS>/22,<IP_ADDRESS>/21,<IP_ADDRESS>/23,<IP_ADDRESS>/22,<IP_ADDRESS>/18,<IP_ADDRESS>/17
2019-12-21T15:58:16.476Z [production] verbose: AS14061 is announcing some prefixes which are not in the configured list of announced prefixes: 2400:6180::/48,2a03:b0c0::/48,2a03:b0c0:1::/48,2a03:b0c0:2::/48,2a03:b0c0:3::/48,<IP_ADDRESS>/17,<IP_ADDRESS>/19,<IP_ADDRESS>/19,2604:a880::/48,2604:a880:1::/48,2604:a880:2::/48,2604:a880:400::/48,2604:a880:800::/48,2604:a880:801::/48,2604:a880:cad::/48,<IP_ADDRESS>/19,<IP_ADDRESS>/20,<IP_ADDRESS>/19,<IP_ADDRESS>/22,<IP_ADDRESS>/21,<IP_ADDRESS>/20,<IP_ADDRESS>/22,<IP_ADDRESS>/21,<IP_ADDRESS>/23,<IP_ADDRESS>/22,<IP_ADDRESS>/18,<IP_ADDRESS>/17
2019-12-21T18:00:00.393Z [production] verbose: AS14061 is announcing some prefixes which are not in the configured list of announced prefixes: 2400:6180::/48,2a03:b0c0::/48,2a03:b0c0:1::/48,2a03:b0c0:2::/48,2a03:b0c0:3::/48,<IP_ADDRESS>/17,<IP_ADDRESS>/19,<IP_ADDRESS>/19,2604:a880::/48,2604:a880:1::/48,2604:a880:2::/48,2604:a880:400::/48,2604:a880:800::/48,2604:a880:801::/48,2604:a880:cad::/48,<IP_ADDRESS>/19,<IP_ADDRESS>/20,<IP_ADDRESS>/19,<IP_ADDRESS>/22,<IP_ADDRESS>/21,<IP_ADDRESS>/20,<IP_ADDRESS>/22,<IP_ADDRESS>/21,<IP_ADDRESS>/23,<IP_ADDRESS>/22,<IP_ADDRESS>/18,<IP_ADDRESS>/17
2019-12-21T20:11:04.472Z [production] verbose: AS14061 is announcing some prefixes which are not in the configured list of announced prefixes: 2400:6180::/48,2a03:b0c0::/48,2a03:b0c0:1::/48,2a03:b0c0:2::/48,2a03:b0c0:3::/48,<IP_ADDRESS>/17,<IP_ADDRESS>/19,<IP_ADDRESS>/19,2604:a880::/48,2604:a880:1::/48,2604:a880:2::/48,2604:a880:400::/48,2604:a880:800::/48,2604:a880:801::/48,2604:a880:cad::/48,<IP_ADDRESS>/19,<IP_ADDRESS>/20,<IP_ADDRESS>/19,<IP_ADDRESS>/22,<IP_ADDRESS>/21,<IP_ADDRESS>/20,<IP_ADDRESS>/22,<IP_ADDRESS>/21,<IP_ADDRESS>/23,<IP_ADDRESS>/22,<IP_ADDRESS>/18,<IP_ADDRESS>/17,<IP_ADDRESS>/23,<IP_ADDRESS>/24
2019-12-21T22:02:34.660Z [production] verbose: AS14061 is announcing some prefixes which are not in the configured list of announced prefixes: 2400:6180::/48,2a03:b0c0::/48,2a03:b0c0:1::/48,2a03:b0c0:2::/48,2a03:b0c0:3::/48,<IP_ADDRESS>/17,<IP_ADDRESS>/19,<IP_ADDRESS>/19,2604:a880::/48,2604:a880:1::/48,2604:a880:2::/48,2604:a880:400::/48,2604:a880:800::/48,2604:a880:801::/48,2604:a880:cad::/48,<IP_ADDRESS>/19,<IP_ADDRESS>/20,<IP_ADDRESS>/19,<IP_ADDRESS>/22,<IP_ADDRESS>/21,<IP_ADDRESS>/20,<IP_ADDRESS>/22,<IP_ADDRESS>/21,<IP_ADDRESS>/23,<IP_ADDRESS>/22,<IP_ADDRESS>/18,<IP_ADDRESS>/17,<IP_ADDRESS>/22,<IP_ADDRESS>/23,<IP_ADDRESS>/24
/opt/bgpalerter/logs #

Looking at those logs, my first reaction would be to add the prefixes it is complaining about that are not configured in the list of prefixes. I'm going to add them and see if there's any improvement.

Fyi, I'm still working/investigating on this (in the meanwhile, happy new year).

Thanks @massimocandela, and happy new year!
In the meanwhile I have changed the configuration of the prefixes.yml file to start monitoring aggregates (yup, we do announce both aggregates and more specifics altogether :roll_eyes:), however I'm seeing the same pattern:

Naturally, as we now monitor the aggregates as well, I'm seeing info logs such as:
2020-01-01T06:42:37.263Z [production] verbose: Possible change of configuration. A new prefix 2400:6180::/48 is announced by AS14061. It is a more specific of 2400:6180::/32 (APNIC).
2020-01-01T07:02:13.684Z [production] verbose: Possible change of configuration. A new prefix <IP_ADDRESS>/22 is announced by AS14061. It is a more specific of <IP_ADDRESS>/18 (ARIN).
2020-01-01T07:02:13.686Z [production] verbose: Possible change of configuration. A new prefix <IP_ADDRESS>/21 is announced by AS14061. It is a more specific of <IP_ADDRESS>/19 (RIPE).

I did so to eliminate a possible cause, although I incline to believe this might be rather related to this error:
2020-01-02T12:27:48.334Z [production] error: Error: Client network socket disconnected before secure TLS connection was established
2020-01-02T13:22:14.418Z [production] error: Error: Client network socket disconnected before secure TLS connection was established
2020-01-02T13:27:44.249Z [production] error: Error: Client network socket disconnected before secure TLS connection was established
2020-01-02T11:39:58.606Z [production] error: 1006

In 24 hours, there were over 400 occurrences:
/opt/bgpalerter/logs # zcat error-2020-01-01.log.gz | grep "Client network socket disconnected" | wc -l
411

Any idea what could be the cause of this?

Hi @mirceaulinic,
BGPalerter is in many cases able to automatically optimise the monitoring of aggregates instead of more specifics. That doesn't impact the memory usage.
Anyway, I believe I managed to identify and solve the memory leak in your case. It refers to a specific case, which I don't think impacted anybody else before.
Please, can you run the latest version of BGPalerter available in dev and let me know if this solves the memory leak as soon as possible?
You should see a memory usage between ~25Mb and ~200Mb
The error "Client network socket disconnected before secure TLS connection was established" I don't know what is related to. Never seen it before. Could you please try to debug it in your configuration? In case open another ticket with instructions on how to replicate it.

Hi Massimo,
Fantastic, thanks a lot (and sorry!) for this amount of work. BTW: I looked over the PR, I'm almost following, but I'd greatly appreciate if you'd take a minute to explain me what's the issue you identified, and it's particular to my use case?
Yes, I have deployed the app with the new basecode, let's see how it goes.
Regarding the "Client network socket disconnected before secure TLS connection was established" error: BGPalerter is deployed behind a firewall that normally drops the connection after 20 minutes. Was thinking it may be related to that. :thinking: Anyway, what's the method you'd recommend to enable the detailed logging of full traces? Thanks!


added ExportString and ImportString function

Hi,
I've added a function to export and import the Instagram object as base64 encoded string to be easily stored in databases.

Hi,
Let me say thank you for what you wrote.

@AlexanderProd We can create a folder named utils and add these functions to it.

@ahmdrz but there's already the utils.go file though :'D ?

I mean create another package. Not a file.
I'll do it later.

Alright,
Now you can add your function to utilities package. I'd suggest naming your functions like ImportFromBase64String and ExportAsBase64String.


Tutorial doesn't work

I just tried to walk through this tutorial, and stumbled across many issues.

The MyGet feed for one of the packages changed, so restore failed. The Microsoft.CodeAnalysis.CSharp.CodeFix.Testing.MSTest package is now on  https://dotnet.myget.org/F/roslyn-analyzers/api/v3/index.json instead of  https://dotnet.myget.org/F/roslyn/api/v3/index.json
The AssemblyName of the VSIX MakeConst.Vsix project is the same as the MakeConst project, which causes you to run into this issue.
The using statement on line 9 in the MakeConstUnitTest.cs file is incorrect and should be removed.
The sample tells you to call the VerifyCSharpCodeFix overload method, but doesn't tell you to create a base class that implements this method and then have your unit test derive from it.

The actual sample is located here.
Here is a GitHub project that is the output of the new analyzer project that highlights all these issues: https://github.com/MadEste/MkConst

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

ID: 6e3fc905-7aa1-7f5e-9bfe-fbcef461ad5e
Version Independent ID: a286e20b-1e43-dc57-e0fb-5a9dee7ec3f6
Content: Tutorial: Write your first analyzer and code fix
Content Source: docs/csharp/roslyn-sdk/tutorials/how-to-write-csharp-analyzer-code-fix.md
Product: dotnet-csharp
Technology: csharp-roslyn
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Yes, I am facing the same issue. I don't know when it gets fixed.
@stevenbrix do you have the correct project that at least builds and run? because I need to create a sample analyzer but I am not able to do that because of these errors.

@MCCshreyas the tutorial on GitHub has been updated: https://github.com/dotnet/samples/blob/master/csharp/roslyn-sdk/Tutorials/MakeConst
You can use that and everything should work

@stevenbrix Thanks.


Provide method to write data to csv file

Issue implemented.

the implemented code appears to be corrupted if organization name contains commma or double quote.

since character encoding is not specified when outputting CSV, it seems that an error will occur when outputting Japanese data in Windows English environment

@toshihiro-shiino  Thanks for remarks. There is plan to replace simple toCSV conversion using apache csv just not to reinventing wheel again;)


allow deploy action to work on PRs from external forks

See https://github.com/leanprover-community/doc-gen/pull/115#issuecomment-779953415 , https://github.com/leanprover-community/doc-gen/pull/131
I think this test is unnecessary, since the action triggered is in leanprover-community/doc-gen:master and not in the PR. It should have access to the repo secrets.

I have no idea if it will work, but I don't think this can do any harm.


Package MobileCoin Wallet

Project description
Wallet application for the crypto currency MobileCoin
Metadata

homepage URL: https://github.com/mobilecoinofficial/desktop-wallet
source URL: https://github.com/mobilecoinofficial/desktop-wallet
license: gpl3
platforms: linux

Was able to package it from the AppImage. Tried from the tarball, but the application is too dependent of AppImage mechanisms that it was impossible to execute.

Merged


[STORM-139] Correctly hash byte array tuple values

Unit test for correctness is included
The following tests showed no discernible difference in latency or throughput:

word_count, last 10min stats after being running > 10min
word_count modified to send byte[] words instead of Strings, same conditions

This should handle the kinds of Objects we care about: reference types, all primitive array types, and reference array types.
Using Java's .hashCode is a problem for distributed systems, as was pointed out in the JIRA.
However, the behavior of String#hashCode, and the implementation of java.util.Arrays#hashCode for all primitive array types is defined and should be consistent across JVMs.  This leaves Objects and Object[] that are not Strings.  In these cases, this change still trusts the hashCode method provided in those classes.
In practice, bad cases should be rare.  In the cases when hashing is inconsistent and some sort of partitioning is used, it could be very difficult to debug.

+1

+1

Ok, addressed comments.
@HeartSaVioR, I made your suggested change as well.

LGTM, +1.


Support for Rotary Embeddings for Llama

It looks like Llama uses an unsupported embedding scheme:
https://nn.labml.ai/transformers/rope/index.html
I'm opening this thread so we can have a conversation about how to support these embeddings within langchain. I'm happy to help, but my knowledge is limited.

I got the recently merged embeddings pr of llama working. Outputting a 5121 element long vector. What would be the best way to go about plugging this output to langchain? Take one of the existing embedding classes and base it of that or am I missing something?

How to convert Llama structure into Faster transformer sturcture ?
it seem has 32 layers with LlamaRotaryEmbedding ?


please help me-silly or is there a typo?

New to multihash (sounds sound ;) ) so pardon my ignorance and possibly RTFM question.
https://github.com/multiformats/multihash/blob/master/README.md#idea-self-describe-the-values-to-distinguish
has an example for sha3 256 which starts with 1420caad, but according to the multihash table (https://github.com/multiformats/multihash/blob/master/README.md#table-for-multihash-v100-rc-semver) 0x16 sha3-256.  So why example doesn't start with 16? (I guessed that those examples are autogenerated so there must be no mistakes/bug, thus I guess it is all my misunderstanding)

sha3 used to be 14 but has been deprecated, so i believe the readme is out of date in those pictures.
also in that picture sha2-512 should have length: 0x40 not 0x20
but only the captain @jbenet can clarify ;)

The 0x20 byte sha2-512 multihash is a valid multihash. It means the resulting 512-bit hash has been truncated to 256-bits and only those first 256-bits of the hash should be checked when validating the hash (see https://github.com/multiformats/multihash/issues/1#issuecomment-91783612).
Although, that reference is still just a comment in an issue, it really should be added to the actual spec if it is going to be the standard.

@Nemo157 Want to make a PR? :)


Duration timer never stops after 1st run

Tried a basic pipeline to check how return values are used in the GUI.
The first run of the following simple code seems to generate unstable behaviour:
@task
def test():
    return 10

@task
def test1():
    return 20

@task
def test2():
    return 30

register_pipeline(
    id="test",
    description="simple pipeline fails",
    tasks=[test, test1, test2],
)

Here  the output log:
INFO:     Will watch for changes in these directories: ['/Users/luca/Documents/JobScraper']
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
INFO:     Started reloader process [33667] using StatReload
INFO:     Started server process [33669]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     <IP_ADDRESS>:60912 - "GET / HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:60912 - "GET /assets/index-86ef8bb9.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:60912 - "GET /assets/index-4fe84deb.css HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:60913 - "GET /assets/vendor-3fad8192.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:60913 - "GET /api/auth/whoami HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:60913 - "GET /api/pipelines/ HTTP/1.1" 200 OK
INFO:     ('<IP_ADDRESS>', 60914) - "WebSocket /api/ws/" [accepted]
INFO:     connection open
INFO:     <IP_ADDRESS>:60913 - "GET /api/runs/?pipeline_id=&trigger_id= HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:60912 - "GET /api/pipelines/ HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:60913 - "GET /api/runs/?pipeline_id=&trigger_id= HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:60916 - "GET /api/pipelines/ HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:60915 - "GET /api/runs/?pipeline_id=&trigger_id= HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:60915 - "GET /api/pipelines/test/input-schema HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:60915 - "POST /api/runs/ HTTP/1.1" 200 OK
Executing pipeline `test` #1 via trigger `_manual`
Executing task test
Executing task test1
Executing task test2
INFO:     <IP_ADDRESS>:60915 - "GET /api/pipelines/test HTTP/1.1" 200 OK
INFO:     ('<IP_ADDRESS>', 60917) - "WebSocket /api/ws/" [accepted]
Task exception was never retrieved
future: <Task finished name='Task-33' coro=<WebSocket.send_json() done, defined at /Users/luca/Documents/JobScraper/venv/lib/python3.10/site-packages/starlette/websockets.py:168> exception=ConnectionClosedOK(Close(code=1005, reason=''), Close(code=1005, reason=''), True)>
Traceback (most recent call last):
  File "/Users/luca/Documents/JobScraper/venv/lib/python3.10/site-packages/starlette/websockets.py", line 173, in send_json
    await self.send({"type": "websocket.send", "text": text})
  File "/Users/luca/Documents/JobScraper/venv/lib/python3.10/site-packages/starlette/websockets.py", line 85, in send
    await self._send(message)
  File "/Users/luca/Documents/JobScraper/venv/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 65, in sender
    await send(message)
  File "/Users/luca/Documents/JobScraper/venv/lib/python3.10/site-packages/uvicorn/protocols/websockets/websockets_impl.py", line 320, in asgi_send
    await self.send(data)  # type: ignore[arg-type]
  File "/Users/luca/Documents/JobScraper/venv/lib/python3.10/site-packages/websockets/legacy/protocol.py", line 635, in send
    await self.ensure_open()
  File "/Users/luca/Documents/JobScraper/venv/lib/python3.10/site-packages/websockets/legacy/protocol.py", line 935, in ensure_open
    raise self.connection_closed_exc()
websockets.exceptions.ConnectionClosedOK: received 1005 (no status code [internal]); then sent 1005 (no status code [internal])
INFO:     connection open
INFO:     connection closed
INFO:     <IP_ADDRESS>:60916 - "GET /api/runs/1 HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:60916 - "GET /api/runs/1/logs HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:60918 - "GET /api/pipelines/test HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:60920 - "GET /api/runs/1/logs HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:60919 - "GET /api/runs/1 HTTP/1.1" 200 OK

UPDATE: it seems that adding a little delay into each task seems to fix problem, something like
time.sleep(0.2)

hi, thanks for reporting this, I'll check what's going on.

After some investigations on different OS and browsers I wasn't able to reproduce this issue, though it seems bound to the websocket and not to the pipeline executor, I'll try to handle those websocket connection issues!


Facet filter improvements

Fixes #31

facet options are sorted by count
facet options filter now translates words to intersection regexes
facet properly filters before limiting (so filter works properly)

Warnings

:warning:

You should add at least 1 more reviewer to the PR

:warning:

The README has not been updated. Please update the README.

Messages

:book:

Awesome! Thanks for the well-formatted PR!

  Generated by :no_entry_sign: dangerJS


cookies.set and cookies.get

Describe the bug
I'm trying to set a cookie the following way inside of my action aka +page.server.ts:
cookies.set('state', state, { path: '/', secure: false })
Back in another route via a +server.ts i try the following:
const expectedState = cookies.get('state');
If i console.log(expectedState) i get undefined
Reproduction
In my +page.server.ts
OAuth2: async ({ locals, cookies, url }) => {         const authMethods = await locals.pb.collection('users').listAuthMethods();         if (!authMethods) {             return { authMethods: '' }         }         const redirectURL = ${url.origin}/oauth        const googleAuthProvider = authMethods.authProviders[0]         const authProviderRedirect =${googleAuthProvider.authUrl}${redirectURL}`;
const state = googleAuthProvider.state;
const verifier = googleAuthProvider.codeVerifier;
cookies.set('state', state, { path: '/', secure: false })
cookies.set('verifier', verifier, { path: '/', secure: false })
    throw redirect(302, authProviderRedirect)`

in my +server.ts
export const GET = async ({ locals, url, cookies }) => {
	const redirectURL = `${url.origin}/oauth`;
	const expectedState = cookies.get('state');
	const expectedVerifier = cookies.get('verifier');
        console.log(expectedState)

Tried with a  +page.server.ts to fetch the cookies back and they where nowhere to be found.
Logs
No response
System Info
System:
    OS: macOS 10.15.7
    CPU: (8) x64 Intel(R) Core(TM) i7-3740QM CPU @ 2.70GHz
    Memory: 326.43 MB / 16.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 18.14.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.3.1 - /usr/local/bin/npm
    pnpm: 8.6.0 - /usr/local/bin/pnpm
    bun: 1.0.7 - ~/.bun/bin/bun
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 120.0.6099.129
    Safari: 15.6.1
  npmPackages:
    @sveltejs/adapter-auto: ^3.0.0 => 3.0.0
    @sveltejs/kit: ^2.0.0 => 2.0.0
    @sveltejs/vite-plugin-svelte: ^3.0.0 => 3.0.1
    svelte: ^5.0.0-next.1 => 5.0.0-next.25
    vite: ^5.0.3 => 5.0.10

Severity
blocking an upgrade
Additional Information
Thanks for your time!

Please provide a proper, minimal reproduction, in the form of a repo or a StackBlitz link, as directed in the issue template.
Have you checked whether the cookie appeared to be set in the browser and whether the issue seemed to be on the set side or the get side?

Did tests and seems to work just fine. The problem with my code is the following line in my hooks file, which for some reason blocks my ability to set a cookie inside my actions.
response.headers.set('set-cookie', event.locals.pb.authStore.exportToCookie({ secure: false }))

I had this problem because response.headers.set discards the cookie your trying to set in the form action, but you can have multiple set-cookie headers. Change your +hooks.server.ts to be
response.headers.append('set-cookie', ...


Fix Broken Links in README

Found some dead links in the README. This PR fixes them.
PS: Just stumbled upon this project. This looks really great! Can't wait to use it for a project. Keep up the good work ❤.

Thanks for the contribution! ❤


FPS - DynamicFPS++ - No Launch

When enabled FPS - DynamicFPS++ - Includes 60fps and DynamicFPS does not allow TOTK to launch, I tested by manually disabling every mod and re launching the game. I am running TOTK 1.1.2 and downloaded the correct folder for this version of the game yet still face this issue as well as the black screen when switching weapons or abilities even with the Fix and Xbox controllwe UI enabled

Well, sometimes we need to nuke the Yuzu config...
Remove all mods except the update 1.1.2.
Reset Yuzu settings in Configure > "Reset All Settings" button.
Clear the cache by right click on the game name > Remove > "Remove All Pipeline Caches".
Clear the cache storage by right click on the game name > Remove > "Remove Cache Storage".
Restart your PC.
Start the game and wait a couple minutes to rebuild the shaders.
Only after that start installing mods. Redownload the mods and starting installing by the guide: https://github.com/HolographicWings/TOTK-Mods-collection/blob/main/SETUP.md
IMPORTANT: Make a backup of your save folder!
Note: I'm on Yuzu mainline.

Uninstall TOTK updates and reinstall it, works for me when I have issues of TOTK not launching


Editor subwindows displayed on the wrong screen when using WMR with SteamVR causing freezes

Godot version
4.1.1.stable.mono
System information
Godot v4.1.1.stable.mono - Windows 10.0.22621 - Vulkan (Mobile) - dedicated NVIDIA GeForce RTX 3060 (NVIDIA; <IP_ADDRESS>13) - AMD Ryzen 9 7900 12-Core Processor (24 Threads)
Issue description
When using Window Mixed Reality HMD with SteamVR enabled, Godot editor will most of the time open subwindows (Add Node, Add script, Project settings...) in inaccessible screens, causing freezes as you cannot close them anymore.
Steps to reproduce

Plug a Window Mixed Reality headset.
Define SteamVR as OpenXR executive environment
Start Mixed Reality portal along with SteamVR.

Now the Windows Screen settings panel should display multiple screens:

Start Godot
Open any project
Add a node to the scene or open Project settings
7.1 Subwindow is not displayed on the main screen, causing the editor freeze
7.2 Subwindow is displayed on the main screen at a random place

Minimal reproduction project
N/A

As a workaround, enable Single Window Mode in the Editor Settings then restart the editor.


Fix README markdown formatting

That's another checkbox to mark
... down
Sorry I could not resist

I fixed the typo in the markdown, but I wish to preserve the link being directly in the body of the page.


Demographic Transition Africa

CentricCracker
The demographic transition modifier works fine everywhere else, but Africa, it's doesn't work well. The UN projects Africa's population to quadruple by the end of the century. But this doesn't happen in game at all. Look at Nigeria for example. They're expected to have a population of ~800 million by 2100, but this doesn't happen in game because it can't.

Solve as part of Southeast Asia Population Growth Not Great Enough #291


How to disable deletion of expired jobs?

If I set
MongoStorageOptions.JobExpirationCheckInterval = TimeSpan.FromMilliseconds(0);

will this disable deletion of expired jobs?

We use a WaitHandle when checking for expired jobs. You can use settings below to wait indefinetly.
This will however also stop deletion of associated job data.
var migrationOptions = new MongoStorageOptions             {                 JobExpirationCheckInterval = TimeSpan.FromMilliseconds(-1)             };

Thanks @gottscj , if I would like to deleted job related data that by job id, which collection I should consider?

I think I have found a solution, which I want to test:
    public class MyCustomFiler : IApplyStateFilter
    {
        public void OnStateApplied(ApplyStateContext context, IWriteOnlyTransaction transaction)
        {
            if (context.NewState is DeletedState)
            {
                context.JobExpirationTimeout = TimeSpan.FromMinutes(1);
            }

            if (context.NewState is SucceededState)
            {
                context.JobExpirationTimeout = TimeSpan.MaxValue;
            }
        }

        public void OnStateUnapplied(ApplyStateContext context, IWriteOnlyTransaction transaction)
        {
        }
    }

@kkorus ,
glad to hear it. Let us know if it works out. :)


Trying to set the timeout when using the HTTP.GET and HTTP.POST methods

I am trying to determine how to set the timeoutInterval using the HTTP.GET and HTTP.POST methods in the Swift2 APIs.
It looks like (from looking at other issues on GitHub) that you are suggesting that users recreate the functionality of HTTP.New().
Can I suggest that the verb-based APIs be extended to include a timeout value as an option?  Or, perhaps a class-based default value to use to set this value?
Is this something you'd consider supporting?

I can see how the might be a benefit, but I believe I came up with a solution that won't require me to forward on every property of NSMutableURLRequest and still gives the behavior desired. Check out:
https://github.com/daltoniam/SwiftHTTP#global-handlers

Tag 1.0.2 released.

This looks great and thank you for making this happen.  I am in the midst of re-working my code from ABAddressBook to CNContact and I'm slogging through that process.  Once I get to the other side, I will check this out and move over to this approach.  Looks great!


Externalize the maxItems value

It would be nice to have the maxItems value option. There are sometimes more than 10 files with similar names.
The maxItems value is currently hardcoded in the
https://github.com/atom/fuzzy-finder/blob/master/lib/fuzzy-finder-view.coffee#L16

+1 This is a showstopper for me as it is..

I have an alternative (and maybe more general) suggestion. Drop the maxItems concept at all and lazy-load items 10-15 at a time while scrolling.

Can I +∞ this?
Right now my workaround is in my init.coffee script (idea given to me by someone else):
FFV = require '/Applications/Atom.app/Contents/Resources/app.asar/node_modules/fuzzy-finder/lib/fuzzy-finder-view.js'

init = FFV.prototype.initialize;
FFV.prototype.initialize = ->
  init.apply this, arguments
  this.setMaxItems 50


TestMultiVersionRevisionTest: add grpc calls

Adds another traffic type to the multi-version revision test, refactors cleanup logic to be more idiomatic
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[x] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[x] Does not have any changes that may affect Istio users.

cc @ericvn for review (upgrade testing related)


provide support for light client (lcd)

A lite client is a process that connects to Tendermint full nodes and then tries to verify application data using the Merkle proofs.
More on Light Client

update from upstream, tendermint team is working on updates to lcd.

Implemented with sdk-migration


can't create two intents with the same name in different NLP applications

Steps to reproduce
1.Start the NLP stack with docker
2.Login and create a new NLP application from the "configuration" tab
3.Create a new Intent
4.Create a second NLP application
5.Create a new intent with the same name a the first one in the new application
It should create an intent with the name entered but instead it doesn't create it and display an error saying "Intent YOUR_INTENT_NAME already exists"
I'm using the compose file from the open-data-bot example with the latest tag

Thanks for reporting.
We need to decide if the intent should be rebuilt from the sentences of the two applications (ie it's a "shared" intent between the two apps) or if two intents of the same name are in fact completely distinct.
The intent was designed to be shared, but I'm not so sure now, so temporarily it's forbidden to have an intent of same name.
What do you think? What is the behaviour you expect when you try to create twice an intent with same name in two apps?

I would expect them to be different, but if I'm informed that shared intents are a thing, then I'd use them for general purpose intents (ie "greetings" that don't really change between to apps). I think i could help save a lot of time just copying the same sentences for the exact same purpose.

We have decided to keep the "share intent" facility - they are shared in a given namespace (ie an account), so you still can have two different intents with same name in two different namespaces.
I will commit soon the changes that allow intent reuse for the same namespace in the admin interface


Add delete_date attribute

API spec draft for https://github.com/giantswarm/giantswarm/issues/7516
The idea is to have deleted clusters appear

in the GET /v4/clusters/ response
in the GET /v4/clusters/{cluster_id}/ response
in the GET /v5/clusters/{cluster_id}/ response

as long as they are not fully deleted. The delete_date value should represent the date when deletion of the cluster has been requested.

Note that only the lister is implemented yet.


Why example code is wrong (not up-to-date)?

In other there is ->setApiKey that doesn't exist also https://github.com/cmfcmf/OpenWeatherMap-PHP-Api/blob/master/Examples/CurrentWeather.php#L41

Hi @intarstudents,
the example code actually is up to date. It refers to the current master version. You are probably (as you should!) using the latest released version which worked slightly different. Please refer to the README of version 2.0.x: https://github.com/cmfcmf/OpenWeatherMap-PHP-Api/tree/v2.0.11
I am going to release version 2.1 soon. It will be fully backwards compatible to version 2.0.11.

Ooo right. roger that.

I have now released version 2.1.0 which means you can now follow the master README and pass the API key as first parameter. :smile:
Sorry for the inconvenience though :confused:


feat: use any IDataReader as a data source

Codecov Report
Attention: Patch coverage is 38.98305% with 36 lines in your changes missing coverage. Please review.

Project coverage is 82.15%. Comparing base (0551710) to head (86eaf34).

Files with missing lines
Patch %
Lines

Streamistry.Core/Pipes/Sources/DataReaderSource.cs
38.98%
30 Missing and 6 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #97      +/-   ##
==========================================
- Coverage   84.97%   82.15%   -2.82%
==========================================
  Files          70       71       +1
  Lines         905      964      +59
  Branches      140      157      +17
==========================================
+ Hits          769      792      +23
- Misses         89      119      +30
- Partials       47       53       +6

Flag
Coverage Δ

82.15% <38.98%> (-2.82%)
:arrow_down:

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

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


Vault Token in ENV var

Neat implementation!  Suggest that you move the KUBERNETES_VAULT_TOKEN from a static definition on the ENV of the deployment to a separate secret file that the deployment mounts in with secretKeyRef.  Anyone with the ability to get/list deployments will be able to steal the vault token and get all the secrets it has access to.  Fewer roles/clusterroles allow get/list of secrets than deployments, so this improves the access control surrounding that sensitive token.
        env:
        - name: KUBERNETES_VAULT_TOKEN
          valueFrom:
            secretKeyRef:
              name: vault-crd-secret-name
              key: "token"

Hi @bgeesaman,
Thank you for your response. Let's continue our discussion here ;-)
You are completely right! I'll update the deployment.yaml.
I think the way with a static Vault Token is also not the preferred, I made a change in the past to also support Service Account Authentication. So for me the Vault Token as static one is not so secure.
I created also a helm chart, but unfortunately I had no time so far to deploy it :-)
Together with an envoy proxy you have I think a good way to enable tls communication between microservices and up to date tls that refreshes automatically.

Sure.  Ideally, you're using Kubernetes SA Auth to fetch this client token.  But for now, this small change at least improves the current implementation a bit.


Added getIssuePicker method

Added support for getIssuePickerResource API call - returns suggested issues which match an auto-completion query
https://docs.atlassian.com/software/jira/docs/api/REST/7.6.1/#api/2/issue-getIssuePickerResource

Please add this to the "Supported API Calls" section of the README and we'll get it pulled in.

Sorry for the delay. Added to Supported API Calls.


ci: enable asan and ubs

Interesting. You can do a GoogleTest vendorpull patch, but I believe there are also ways to instruct the sanitizers to ignore certain files/symbols, i.e. passing an exclude

Unfortunately, I did not manage to find a way to exclude it for a few reasons:

The file that has an issue is /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX13.3.sdk/usr/include/c++/v1/__functional/hash.h:107:40  I don't know how to exclude it, should I use hash.h?

Which rule should I disable? According to the documentation, I tried shift-base and shift-exponent: https://releases.llvm.org/3.8.0/tools/clang/docs/UndefinedBehaviorSanitizer.html#availablle-checks

The doc is thin. I also searched on github.
I open an issue on LLVM https://github.com/llvm/llvm-project/issues/82500

To be honest, not sure from the top of my head, but I think you are on the right track. Let's see if LLVM responds and I can try to help if it remains a problem

Hey @jviotti
Finally solved it :)


angular.module(...).info is not a function in angular-aria

I encountered the following bug - after upgrading angular-aria to recently released v1.6.3 I'm getting the following error:
Uncaught TypeError: angular.module(...).info is not a function
    at angular-aria.js:62
    at angular-aria.js:403

When I downgrade to v1.6.2, the app works normally.

You need to update angular to 1.6.3, too. The .info fn is defined in angular.js itself. It's always been the case that the core file and the module files have to have the same version.

Found out it's a problem with angular-material dependency, not angular itself. Leaving this here should anyone have the same problem https://github.com/angular/material/issues/10475.
Thanks for help!

Thank U so much Narretz :)
I spent ONE day debugging my codes.

I have the same problem with npm and webpack, the problem was that angular-messages was after angular-material so it get installed by angular-material before proper version was taken from package.json. Moving angular-messages before angular-material fixed the issue.

Hi I'm running into this problem as well. Not using Aria, just angular-cookies. I upgraded to angular 1.6.4 and noticed that there is no info function in angular.js
http://stackoverflow.com/questions/43188708/installed-angular-cookies-getting-angular-module-error-on-info-function

@leongaban, you should update all AngularJS modules to be on the same version.

@gkalpak yes, it was my mistake, I had all the angular dependencies manage by bower, not npm. Npm was only for Gulp deps, but I had angularjs in there as well. Removed from npm, installed via bower and ran my build scripts and it all works now!

Hmm, I'm still getting it:
bower list
meanjs /data/mean
├── angular#1.6.4 (1.6.5-build.5353+sha.136a42a available)
├─┬ angular-animate#1.6.4 (1.6.5-build.5353+sha.136a42a available)
│ └── angular#1.6.4 (latest is 1.6.5-build.5353+sha.136a42a)
├─┬ angular-bootstrap#1.2.5 (latest is 2.5.0)
│ └── angular#1.6.4 (1.6.5-build.5353+sha.136a42a available)
├─┬ angular-material#1.1.4
│ ├── angular#1.6.4 (1.6.5-build.5353+sha.136a42a available)
│ ├── angular-animate#1.6.4 (1.6.5-build.5353+sha.136a42a available)
│ ├─┬ angular-aria#1.6.4 (1.6.5-build.5353+sha.136a42a available)
│ │ └── angular#1.6.4
│ └── angular-messages#1.6.4 (1.6.5-build.5353+sha.136a42a available)
├─┬ angular-messages#1.6.4 (1.6.5-build.5353+sha.136a42a available)
│ └── angular#1.6.4
├─┬ angular-mocks#1.6.4 (1.6.5-build.5353+sha.136a42a available)
│ └── angular#1.6.4
├─┬ angular-resource#1.6.4 (1.6.5-build.5353+sha.136a42a available)
│ └── angular#1.6.4
├── angular-ui-notification#0.2.0 (latest is 0.3.6)
├─┬ angular-ui-router#0.2.18 (latest is 1.0.0)
│ └── angular#1.6.4 (1.6.5-build.5353+sha.136a42a available)
├─┬ bootstrap#3.3.7 (latest is 4.0.0-alpha.6)
│ └── jquery#3.2.1
├─┬ ng-file-upload#12.1.0 (latest is 12.2.13)
│ └── angular#1.6.4 (1.6.5-build.5353+sha.136a42a available)
└── owasp-password-strength-test#1.3.0

Hi I get the same error with
  <script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.6.4/angular-route.js"></script>
  <script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.6.4/angular.min.js"></script>

@ileathan, maybe try loading angular.min.js first.

just replace ^ sign of the before version to ~ sign and apply following command
bower install --force
it worked for me..  :)

just use same version of angular.min.js and angular-sanitize.min.js.
it will fixed 100% ;)

I had this problem too and solved that by using lower version of angular-aria


google cloud compute engine tf setup for gpu size 4

This includes:

the configs for experiments on Compute Engine, Google Cloud GPU 4-32 nodes
new docker builds across that setup to fix the CUDA_VISIBLE_DEVICES bug
(mostly) complete experimental data for the size 4 runs (all except for quicksilver)
updated build script to install drivers and nccl fast socket to the VM
adding base modules, with us-central1-c selected as the zone (how I got the GPU)

I was originally just going to test tonight, and felt fairly confident in the configs and decided to do full runs (5 iters, up to 15 mins each) for each confidently working. The entire experiment was just under 2 hours. We have a custom terraform setup and VM image that spins up in minutes (granted there is availability) and provides a fully working flux cluster with singularity, oras, and our application containers. The bugs (that I think were present across any experiments that use Singularity and drivers) were hard to figure out, but I'm separating that from the setup itself, which I've very happy with. I'll state again that these custom terraform deploys are the fastest and easiest ways to deploy infrastructure on Google Cloud, and probably on AWS and Azure if we had decided to use them. With all my testing and debugging of different VMs, it has been so easy to bring up and down entire new clusters! Points of discussion before moving on to size 8:

if the configs look ok, generally. We can discuss them in this first consolidated file (making comparison easy) and I'll move into subsequent sized directories.
I think we should remove any applications that cannot use GPU (e.g., amg - I don't see we are getting improved data from the CPU setup, so it's just burning money. The one variation might be that this setup has the nccl fast socket installed, but if it's only for GPU (and we aren't using them) no point.
For OSU, the cuda worked for the point to point benchmarks. I ran D D and H H, and H H was much better. I have results for both but I'd like to suggest we do H H.
I checked GPU utilization for all apps - we hit all 8, consistently, and (it appears) well! :partying_face:
The only two that didn't work perfectly were OSU (expected, because I had issue with all reduce before) and quicksilver. Quicksilver, akin to all reduce, had the error that "smelled like MPI not cuda aware." It's such a slow running app I didn't want to wait minutes to see if it would continue running.
Lammps didn't run for the larger problem size (pooped out toward the end) but was ok for the smaller one. So (I think) we should be OK moving forward.
I ran the single-node benchmarks with and without sudo. I think we should run with sudo (and I'll nix the without).
For mixbench, I suggest we leave out the little -n. The wrapper seems to be working as expected, and when I remember testing this on larger clusters having -n doesn't scale well (it starts to take a really long time).

Let's discuss the above. When we are happy with decisions I'll move on to the other 3 cluster sizes.

Answering some of my comments above:

if the configs look ok, generally. We can discuss them in this first consolidated file (making comparison easy) and I'll move into subsequent sized directories.

These were derived from the equivalent GPU setups on other clouds, and should be OK. Someone said my amg params were off, but I confirmed this is what was used on cyclecloud.

I think we should remove any applications that cannot use GPU (e.g., amg - I don't see we are getting improved data from the CPU setup, so it's just burning money. The one variation might be that this setup has the nccl fast socket installed, but if it's only for GPU (and we aren't using them) no point.

AMG is now using GPU, and didn't work at the larger problem size on 4 nodes. I'll test the larger problem size on the larger cluster, and worst case do the slightly smaller size.

For OSU, the cuda worked for the point to point benchmarks. I ran D D and H H, and H H was much better. I have results for both but I'd like to suggest we do H H.

We are doing all three. D D is the worst, H H is slightly less offensive, and without GPU is the best.

I checked GPU utilization for all apps - we hit all 8, consistently, and (it appears) well! 🥳

:muscle:

The only two that didn't work perfectly were OSU (expected, because I had issue with all reduce before) and quicksilver. Quicksilver, akin to all reduce, had the error that "smelled like MPI not cuda aware." It's such a slow running app I didn't want to wait minutes to see if it would continue running.

I waited over 10 minutes and didn't see progress, saved the output file and moved on. I won't be running this for larger sizes.

I ran the single-node benchmarks with and without sudo. I think we should run with sudo (and I'll nix the without).

We will be running them with sudo

For mixbench, I suggest we leave out the little -n. The wrapper seems to be working as expected, and when I remember testing this on larger clusters having -n doesn't scale well (it starts to take a really long time).

I added back in the multi-gpu-models, which appear to actually work and output final results (and have great GPU utilization) despite some error messages at the end.
I'm also removing empty data directories and .gitkeep files for this PR (chore/cleanup).


why i use vllm inference deepseek v2 ,speed is low

i use vllm to inference deepspeed, use flask to deploy model. When the problem enters the model, it always gets stuck for a long time in the processd prompt step，the code i use is your example code

https://huggingface.co/deepseek-ai/DeepSeek-V2/discussions/1
@ZzzybEric

whats your gpu type？


zlog: add Context helper

This should make using the new baggage API easier.
Signed-off-by: Hank Donnay<EMAIL_ADDRESS>

@misberner Also interested in your thoughts. This is basically pulling out the helper you're suggesting in quay/claircore#501.


Return value from Key.toUrlSafe() is inconsistent with the URL-safe key displayed in Google Cloud Datastore Console

The Key.toUrlSafe() returns a URL encoded (similar to URLEncoder.encode) version of the key like the one below:
partition_id+%7B%0A++project_id%3A+%22my-project%22%0A++namespace_id%3A+%22MyNamespace%22%0A%7D%0Apath+%7B%0A++kind%3A+%22MyEntity%22%0A++name%3A+%22MyKey%22%0A%7D%0A
The Google Cloud Datastore Console displays a Base64 encoded version of the key, which looks something like -
agVoZWxsb3IPCxIHQWNjb3VudBiZiwIM
The Console allows filtering on key by URL-safe key, which works fine if I use the URL-safe value displayed by the GCD Console, but does not work when using the value returned from Key.toUrlSafe().
The Console Documentation states that -

URL-safe key
This is a base64-encoded, serialized version of your entity key. The encoding method is available in any Cloud Datastore client library
Example: agVoZWxsb3IPCxIHQWNjb3VudBiZiwIM

So, the client libraries are expected to match with the Base64 version? Nonetheless, it would be nice to have the client libraries and Console in sync to help with crosschecking/debugging purposes.

@sai-pullabhotla This is interesting. @aozarov any reason our toUrlSafe is different from the one used in the cloud console? Do you think it's possible this changed over time and we failed to keep up-to-date?

I think that at the time that we provided the toUrlSafe it only worked for this library (there was no "standard"). I am glad to see that now there is and we should change the code to adopt it.

Then @sai-pullabhotla thanks for bringing this up. I'll work on a fix soon.

@eddavisson @pcostell can you guys help me understand how the base64 URL-safe key shown in the Cloud Console is computed?
I expected it to match the following code:
BaseEncoding.base64Url().omitPadding().encode(toPb().toByteArray());

But that is not the case.

The base64 URL-safe key shown in the console only works with the App Engine client libraries. If you think it would be useful for it to provide a URL-safe key usable in the Google Cloud client libraries, please send feedback in the Cloud Console so it can get triaged by the right team.

@pcostell Can't we generate this base64 URL-safe in Google Cloud? I think it would make sense for us to generate that value as it is used in AE libraries as well as in the Cloud console.

Parsing and generating URL-Safe keys from the Google App Engine API   is essential for us to be able to use  the Google Cloud API.

@JoshuaFox Are you asking for compatibility within the library, or between GAE Standard API's / Cloud Console and this library?
Also, how would it affect you if we changed the format generated by this library to be compatible with GAE Standard / Cloud Console?

I'm not sure what "compatibility within the library" means, but of course I'd like all kinds of compatibility. Specifically, we have URL-Safe keys generated in the GAE API which need to be parsed into Key objects in Google Cloud API; and the Google Cloud API code has to be able to generate keys in the GAE URL-Safe string format.
The Cloud Console is less important.
In any case, we have since Jan 12 written code that digs into this format and does the above conversions, so for us it is less urgent; though still, this compatibility  seems like an obvious requirement.

@JoshuaFox What I'm trying to ask is, if we fix (ie. change the current behavior of the google-cloud API) to match what the GAE API does, would it break your app?

I don't think it would break my app.

@garrettjonesgoogle I think we should fix it, and the sooner, the better. (ie. before GA)

@jabubake PTAL - you might wish to inquire / make sure that that thing your working on is compatible in this.

UrlSafe keys are allowed to be used in URL's (ie. bookmark-able).  Developers who move / augment their GAE Standard apps with GAE Flex apps will want to easily move between them.  Not having this will block the adoption of GAE Flex for many developers.

Note - we should rename the existing toSafeUrl() to toOldSafeUrl() and fromUrlSafe() to fromOldUrlSafe as well, so that we don't mess up too many users.
Alternatively, but equally bad, we should create toGAESafeUrl() and fromGAEUrlSafe().

Maybe: create toBase64SafeUrl() and fromBase64SafeUrl() so that the reasons to choose one vs the other is clearer, and not so tied to a specific technology that is incidental.

Since we will be adding methods instead of changing them, this isn't a breaking change any more.

There has been some offline discussion and it doesn't seem like this can be fixed for GA, and might not be able to be fixed at all.

It turns out that replicating the base 64 format of the cloud console in open source isn't possible right now; it requires a proto that isn't open source, and an app id that isn't available in the client library context. I'm going to remove the "release blocking" label.
The solution is probably for cloud console to produce an alternative base 64 format that the client library can reproduce.

My requirement was for comparability between Datastore Cloud API and Datastore GAE API; the Cloud Console (the GUI) is less relevant.

@JoshuaFox ok, but the issue is the same anyway, right?

I don't necessarily know the implementation details, but if the Cloud Console  value is generated by the GAE API; in contrast to the Cloud API; then yes.

Is there any plan to add implementation comparable to GAE API?
FWIW, in cloud golang library, implementation is at https://github.com/GoogleCloudPlatform/google-cloud-go/blob/master/datastore/key.go#L198.
Here is GAE API at https://github.com/golang/appengine/blob/master/datastore/key.go#L211.
It seems they are comparable. I would hope that implementation is also added in Java library.

@garrettjonesgoogle have you had a chance to check if Go can achieve this compatibility why Java cannot?

Theoretically Java could, it would just require duplicating code instead of reusing it.

This has been added to our feature backlog: https://github.com/GoogleCloudPlatform/google-cloud-java/wiki/Feature-backlog . This issue will be closed but is linked in the backlog and can continue to be used for comment and discussion.

How is the status on this?
This issue seems to persist for quite some time now and we are unable to use the url safe key spanning services built in various languages due to this.


Add source registry

Use the sourceRegistry for the test app

Why do we want to rename that to sourceRegistry and move it to the top level?

/lgtm


Add --version

Most commands provide a --version.
Any chance cargo-zigbuild would be interested in doing that?

Already supported.
$ cargo-zigbuild --version
cargo-zigbuild 0.19.1

Ah, it is cargo zigbuild --version which doesnt work, but cargo-zigbuild --version is good enough for me.

It's just how cargo external subcommand works, cargo zigbuild is actually calling cargo-zigbuild zigbuild, adding a --version to that command doesn't feel right, like you won't expect a --version option in cargo build command.


[4.2][string] Only bridge tagged NSStrings to small string form.

Non-tagged NSStrings carry identity separate from their
value. Continue to bridge them lazily, even if they could fit in small
form, to respect this and avoid potential information loss.

4.2 cherry pick of https://github.com/apple/swift/pull/16320

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - a9fe583252b3f4f39dd761004aff20590b910d7c

@swift-ci please test

@swift-ci please test

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - a9fe583252b3f4f39dd761004aff20590b910d7c

Build failed
Swift Test Linux Platform
Git Sha - a9fe583252b3f4f39dd761004aff20590b910d7c


🐞 Log buffering with secrets mounted prevents seeing output during execution

What is the issue?
Observed with a long running gradle task in the airbyte repo.
Workaround described in more detail here (not mounting secrets locally): https://github.com/airbytehq/airbyte/pull/30438
To summarize:
When secrets are mounted, a custom buffer is introduced for stdout/stderr: https://github.com/dagger/dagger/blob/v0.6.4/cmd/shim/main.go#L294

The real problem is that the buffering is not configurable: dagger relies on a golang library called transform to perform the regexp matching on a stream and this library hard-codes a buffer size of 4096 bytes for each regex

In practice, this means that for this long running gradle task, which requires secrets to be mounted, no output is observed during the execution. Because of this, it's not possible to know if the process has a problem or hung, or if everything is operating normally. Additionally, if the process is killed, the user will never see the logs.
Log output
No response
Steps to reproduce
No response
SDK version
Python SDK 0.6.4
OS version
Ubuntu

Confirmed duplicate


Chromebook hardware too behind

I was intent on getting a fanless pixel i5 or i7, but after doing quite a bit of research, the 6th gen cpus, and overall design just didn't cut it.
Ended up with an i5 surface pro, but would love to see Google come back with an ultralight 7th or later i series non throttling fanless device.

Sounds good.  Do you have any other hardware requests for the crouton bugtracker?


Missing licence information

I didn't find any information about the "licence" of this great "invoice2data" lib. Did I miss something ?
I think it's important to choose a free software licence and display it clearly.

@manuelRiel We need your decision on this. You are the author, so you should decide under which licence you publish the code.

On it. I'll read up on the topic. Will be BSD or something else that's easy to work with in business environments.

OK, great !


How to get youtube-dl.exe for win32 of pac-support?

how to get youtube-dl.exe of pac-support?
I have already built pacparser.dll and got pacparser pymod.(I can import pacparser and use it's member functions)
I run setup.py within IDLE but got nothing and no any exception except some user warnings.(with VC++2008 installed)
It is to say that try to change py to exe,isnt it?
try:
    # This will create an exe that needs Microsoft Visual C++ 2008
    # Redistributable Package
    import py2exe
except ImportError:
    if len(sys.argv) >= 2 and sys.argv[1] == 'py2exe':
        print("Cannot import py2exe", file=sys.stderr)
        exit(1)

Could you give me a tip?Thanks a lot.

Use this command to build a binary:
python setup.py py2exe


Different results with the "Repeat" method

Hi, I tried to reproduce the result of the Repeat method and I did not find the same result as those displayed in your article for the ILI dataset. Your code seems to provide the same result as mine, is that an error in the paper?
Repeat:
illness
horizon 24 {'mae': 1.621, 'mse': 6.204}
horizon 36 {'mae': 1.904, 'mse': 7.702}
horizon 48 {'mae': 1.951, 'mse': 7.84}
horizon 60 {'mae': 1.787, 'mse': 6.875}

@etidav Hi! Sorry to bother you, but I'm trying to reproduce the results in this paper, but I have some problems to even obtain the results you provided with the Repeat model.
Can I ask, pricesely, how you use the model? There are more ways to compute forecasts, and I cannot understand which one is used in this paper.
For instance, I can think of these two methods:

start from the first point in the test dataset, use the last known point to predict the next 24, and move ahead of 24:

start from the first point in the test dataset - 24, and predict the point at 24 steps ahead; then, move ahead of 1, and repeat:

Can I ask which method did you use, and how did you obtain your results? Thanks!

@AlexMV12 Hi, If I'm right, the Naive method used in this paper is the method 1) of your comment. (repeat the last known point x times).
However, you do not have to move ahead of x, just move ahead of 1.
In the case where horizon=24:

Start at the beginning of the test set and compute the naive prediction (repeat 24 times the last accessible value)
Compute the MAE/MSE of the prediction compared to the ground truth (the 24 first points of the test set)
move ahead of 1
repeat the process until you reach the end of the test set.

If you want, here is the function that I used for one of my paper to reproduce the naive results (except on the ILI dataset ^^')

Hello @etidav ,
First, sorry for the late response and thanks for your answer! I found your code usefully and, basically, I am able to reproduce the same results as yours using Darts.
Let me ask a question: I found that in your code, for the ETTx datasets you do this:
dataset = dataset.loc[:"2018-02-20"]

Doing so, the results are the same reported in this paper. But, can I ask you where you found this particular timestamp to cut the time-series? Did the authors of this paper say this somewhere, or in the Autoformer paper?
Thanks a lot and good luck at ICLR!


feat: Implement Python code generation for simple ASTs (arithmetics, procedures)

We are going to implement python code generation for simple functions like arithmetics, if-else conditions and loops.
The main idea is to create a rule data base, so we recursively generate 1 line of code for each node in the tree.

Can be considered done since the implemenation already works for a subset of the re2 domain


Query execution bug with groups

When executing the query
test lang:python on the demo, this produces 15 results:
https://sourcebot.dev/search?query=test lang%3Apython
When this is changed to
test (lang:python) this drops to 0.
https://sourcebot.dev/search?query=test (lang%3Apython)
this should be equivalent, so something is getting mangled.

May be because the () are not being escaped correctly in the URL?
%28 could be used instead?
I have not looked at how the querying is passed down yet, but this should be fixed too.

I need to dig more into how the zoekt parser works, but for reference test lang:python and (test lang:python) are equivalent. (1, 2)
Another data point is that lang:python works, but (lang:python) does not (1, 2).
Ultimately, these feel like edge-case within zoekt's parser. One UX thing we can do is add error underlying (aka "red squiggles") that highlight syntax errors in the user's query.

I tried this with sourcegraph, and it seems to work there. Is there any chance that the sourcebot backend is doing anything?
Alternatively I can open a PR in the zoekt repo adding this as a test case (and fix it there in the parser if it's actually broken)


Make max websocket connection configurable

[x] Make it a server config option. Add it to clap for fedimintd.
[ ] Increase the default from 100.

Isn't that here https://github.com/fedimint/fedimint/blob/master/fedimint-server/src/config/mod.rs#L152-L153

Looks like it.
Default seems to be 1000. Should we raise it a little higher? @elsirion @dpc

I think the default is ok, wouldn't mind increasing though. But more importantly @dpc mentioned that we should be able to configure that limit via the environment.

Note: we need to increase ulimit  I guess, to have more open file descriptor. Defaults might be quite low (1k).

https://users.rust-lang.org/t/ballpark-number-of-mostly-idle-tcp-connections-that-async-server-can-handle/92910 I asked the community what to expect.


Update defender-vulnerability-management-capabilities.md

Made changes to the tabular column to make it easy to identify which feature is included with the license type

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/8588

@siosulli Please merge this PR. Thanks


[RESOLVED] High ICMP ping response time

Problem started at 16:24:33 on 2023.06.19
Problem name: High ICMP ping response time
Host: S100-Santiago
Severity: Warning
Operational data: Value: 965ms
Original problem ID: 28930
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23518&eventid=28930

Problem has been resolved in 5m 0s at 16:29:33 on 2023.06.19
Problem name: High ICMP ping response time
Host: S100-Santiago
Severity: Warning
Original problem ID: 28930


Oculus library

The latest version alvaible on Oculus store is 2.3.1 but the latest client alvaible here is 2.4.0 alpha5 not compatible with 2.3.1.
I tried to sign 2.4.0. with Apk Editor opening the apk using the setting Full Decode, I add into asset folder the file Osig downloaded from Oculus then I tap to Build to create the new apk.
I installed the new file using adb install newpack.apk and tapping to Install from Apk Editor but when I try to launch ALVR an error appears with the message "apk unsigned". I installed also Android Studio but don't know how to use, somebody can help me?
Thank you

I tried to do and follow tips found from other issues and on youtube but the problem persists. When I boot alvr installed by tapping on the apk modified with apk Editor or adb install appears this message:
Thread priority security exception. Please make sure the Apk is signed.

FIXED!! In the Osig generator you must use the DEVICE ID not the IMEI ID.
You can see the Device ID by using this app:
https://play.google.com/store/apps/details?id=com.Viewport.GearVRDeviceID&hl=en_US
or via adb devices


codecov is "semi working": doesn't provide status to PRs, reports "Unable to find commit in Github"

realized while looking at https://github.com/con/tinuous/pull/89/files that there is no codecov report.
https://app.codecov.io/gh/con/tinuous/branch/gh-88/commits?page=1 says "Unable to find commit in Github"  and
https://app.codecov.io/gh/con/tinuous/pulls?page=1&state=open&order=-pullid  has the last good one for "#66 Updated 9 days ago by jwodder" and then there is only 1 without title  and saying "Missing base report" .
Smells like codecov issue, but may be you feel that something on our end managed to break it about 9 days ago @jwodder ?

hm, codecov commenter reemerged on https://github.com/con/tinuous/pull/89 but no status withing the matrix of CI reports

nope, no codecov report for recent https://github.com/con/tinuous/pull/134
codecov seems to have coverage: https://app.codecov.io/gh/con/tinuous  --> https://codecov.io/gh/con/tinuous/commit/8799d5ce48a23f94ca31d82a9776725dd17c6bb7/  and some banner suggests to install Codecov App!  installed for con (needs Read to admin part, which kinda strange), will accept my invaluable suggestion in that PR and see what happens

I think it is "working" now


[Delver|Bug] section.dat overwriting settings

Summary
For generator/Dungeon/section.dat and generator/Undead/section.dat the properties fogStart and fogEnd are defined multiple times instead of once.
Example (Dungeon):
levelTemplates: [
    {
        fogStart:2,
        fogEnd:18,
        theme:"DUNGEON",
        fogStart:1.925,
        fogEnd:16.039,
    }
],

I'm pretty sure we are guilty of this in many places. 🙃


Guardian Approval through Slack

Problem statement
As an Approver for Guardian appeals, Users should be able to approve/reject appeal requests from slack itself.
Proposed solution
There has to be an action button (Approve/Reject) along with the approval notification sent to slack, so the approver can take necessary action directly from slack. This will enable the approver to be able to approve/reject appeals even on mobile when they are not near a laptop.
Screenshots of existing and new notification
Existing Notification:
You have an appeal created by<EMAIL_ADDRESS>requesting access to play (bigquery: project:play) with role viewer. Appeal ID: 9890-ae8e421d5640
New Sample Notification with action Buttons:

After Either approve or Reject:

Changes recommended

Slack notifier client should now support sending attachments/ blocks and buttons along with text in the notification messages.The block, button and metadata templates can be configured in env just like text is
configured right now with the default value in Guardian.
Example configuration:

blocks = [
   {
   	"type":"section",
   	"text" : {
   		"Type":"mrkdwn",
   		"Text" : "You have a appeal created {{.appeal_id}}"
   	}
   },
   {
   	"type":"section",
   	"fields" : [
   		{
   			"Type":"mrkdwn",
   			"Text" : "*Provider:*\n{{.provider_type}}"
   		},
   		{
   			"Type":"mrkdwn",
   			"Text" : "**Resource Type:*\n{{.resource_type}}"
   		},
   	]
   },
   {
   	"type":"actions",
   	"elements" : [
   		{
   			"Type":"button",
   			"Text" : "Approve",
   			"Value": "approved"
   		},
   		{
   			"Type":"button",
   			"Text" : "Reject",
   			"Value": "rejected"
   		}
   	]
   }
]

metadata_fields : ["appeal_id","resource_name","approval_step","role","account_id"]

Guardian should be able to handle actions made into slack i.e the Request Url configured in slack for actions has to be developed in Guardian. Following are the Some Proto/Handler changes recommended in Guardian:

Proto for Slack Action :
message  SlackActionRequest {
	message User {
		string Id = 1
		string Username = 2
	}
	message Action {
		string value = 1
	}
	message Message {
		map<string, string> EventPayload = 1
	}
	User user = 1
	Repeated Action actions = 2
	string responseUrl = 3
	Message message = 4
}

message  SlackActionResponse {
	string text = 1
	string responseType = 2
	string replaceOriginal = 3
}

Rpc HandleSlackAction(SlackActionRequest) returns (SlackActionResponse) {
	option(google.api.http) = {
		post: “v1beta1/fire_action”
		body: “*”
	}
}

New Handler Path :  api/handler/vibeta1/notifier.go
@bsushmith

@utsav14nov Let's have the endpoint in this way - v1beta1/notifier/<notifier-type>/execute
where <notifier-type>  = slack in this case
@rahmatrhd @mabdh @haveiss requesting for your comments.

@bsushmith I am trying to understand, so what the new endpoint responsibility is?
Is it being called when approve or reject button is clicked?
If yes, why not just using some appeal approval api ?

how will the user authenticate to access the APIs?
if slack can handle the authentication, we might as well treat the slack bot as a frontend client. And as @mabdh suggested, we can use the same approval API

@mabdh @rahmatrhd

We can configure only one request url in slack for an app where it post data on every button action in slack. It post data regarding what action is pressed, which user has clicked , from which channel it is requested and other metadata etc.
Approve and Reject endpoints need appeal_id and approval_step in url which needs to be build on runtime and slack doesn't supports it.
Also only one request url is configured for the one app, we cannot configure post url for each button in slack.
Slack Request is authenticated by our application through X-Slack-Request-Timestamp header sent through slack. There is a logic provided by slack to create this secret at backend and compare. This make sure that the request is coming from the authenticated slack app.
If above step is authenticated well, user slack id  is provided in payload sent from slack which can be used to get email id of the user performed the action.

+1 on using same approval APIs, we should treat these as clients, very similar to CLI.

@utsav14nov can you also please share the link to the slack API that you refer to?

As discussed over the call, we all agreed on only on change in guardian i.e. adding support to notification messages to also send blocks/attachments to format notification better (Point 1 of changes recommended in above issue).
Rest flow will be taken care by separately other services.
cc: @mabdh @rahmatrhd @bsushmith

@utsav14nov Can we update the issue with final approach we are taking. Which also summarises the decisions taken.

slack API on Approve/Reject button/action could capture account_id and action, account_id is an alphanumeric value and not exists in IAM users nor in appeal tables. If we add a flag into the post body of existing approval like source=slack then we can change appeal Api to resolve the email id for a given account_id using slack Api and continue with the rest of the approval flow given email id exists in appeal tables.

@singhvikash11 one of the primary problems with the initial approach or using slack api directly was with respect to authentication.
Currently, guardian service is built in such a way that it expects authentication is already done before it receives the API call(primarily through shield), and it expects a specific header with authenticated user email which slack is unable to send. so instead of mingling the responsibilities here, the following approach can be taken -

slack is just another client which has buttons configured - approve / reject for example. These buttons will be configured with a endpoint(UI)
The UI will have an endpoint to take the slack call, and then through authentication for example with google IAP, can call guardian API's with appropriate json body.

The UI is out of scope for guardian changes and it will depend on how it will be implemented by different users. With this approach, it will be extensible for other notifier providers also if needed later.
@rahmatrhd @AkarshSatija @haveiss

@bsushmith @utsav14nov let's update the main thread's description with that approach


@cavvia => Revert "add support for new homepage rails and update rails order"

Reverts artsy/force#2528
In prep for MP being rolled back: these new rails SIMILAR_TO_RECENTLY_VIEWED, etc. wouldn't exist and MP would throw an error on that.

I've already rolled back gravity to not include them in the me_modules endpoint response... so i'm not sure if Force would even request MP for these.


MWPW-139487 - Fix for Gnav scroll issue with Landscape Orientation on devices

Description
On the landscape orientation for mobile, the user cannot access (scroll to) the lower part of the mega menu.
Resolves: MWPW-139487
Screenshots:
Before:

After:

Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/devashish/wam?martech=off
After: https://mwpw-139487--milo--deva309.hlx.page/drafts/devashish/wam?martech=off

I'm not sure this is the right direction. The aim is to adapt the height, yet width and layout properties are affected instead, leading to additional issues. I added the highlighted properties from the screenshot to the feds-popup styles starting at 900px and things seem to work as expected. Will need a lot more testing, but this just impacts the height of the relevant element.

@overmyheadandbody
flex-direction does not have any impact now because we updated display property from flex to block.
I have verified all the viewports with this change.Also, height impact only on feds-popup as we are calculating based on viewport height and this will only applied when viewport height is less than mega-menu height.
I have verified all the viewports with this change. But yeah this change needs to be tested properly.

I still don't agree with this approach. Again, you want to adapt behavior on the Y axis, yet this PR makes changes to the X axis. The screenshot I provided yesterday with another way of achieving scroll inside popups only should be reconsidered. That only affects the Y axis and makes changes to a single element.

@overmyheadandbody
I understand your point about the unintended problems caused by changing the width and layout. This affected other sections, as you pointed out in footer. So, I've undone those changes and now I'm only fixing the height of the megamenu and adding scrolling just for that element. However, we still need horizontal scrolling (scroll to X axis) because we have
promo that get cut off on the right side, similar to what you might see on a Photoshop page below:

That's why I included overflow: auto in this scenario.

Validation done on the below url :
https://main--cc--adobecom.hlx.live/creativecloud/file-types/image?milolibs=mwpw-139487--milo--deva309
https://main--cc--adobecom.hlx.live/creativecloud?milolibs=mwpw-139487--milo--deva309
https://main--cc--adobecom.hlx.live/creativecloud/file-types/image?milolibs=mwpw-139487--milo--deva309&akamaiLocale=tr
vertical scrolling to the lower part of Mega menu

horizontal scrolling to the right part of Mega menu


Track is undefined

https://github.com/alex3165/fluorine-orchestra/blob/master/src/util/createReducerForStore.js#L63

#eslint is free 💃