Connected to peer failed

I tryed to install peer call to my server, it's work but when i join a call, peer connection fail :/
Look at bellow picture

Unfortunately I don't think I can help you with this - does it work when you use peercalls.com? I noticed that sometimes corporate firewalls block connections. For example, when I'm at work I can establish a connection between my phone and a PC if they're on the same network, however it doesn't work when I try to reach out to somebody outside of the network, whereas if I'm at home everything works.

With peercalls.com it's work.
But locally peer to peer connexion failed :/
I probably forget a file to configure the HPPS, can you tel me what file i need to edit for configure the HTTPS.
I edited /src/server/app.js file for configure the HTTPS.

What did you edit in that file, MrDarkSkil?
Could you share the code?

File: src/server/app.js

#!/usr/bin/env node
'use strict';
var express = require('express');
var handleSocket = require('./socket.js');
var os = require('os');
var path = require('path');
var https = require('https');
var fs = require('fs');

// Require in express.Router Middleware.
var callRouter = require('./routes/call');
var siteRouter = require('./routes/index');

var app = express();

var hskey = fs.readFileSync('./ssl/myssl_key.key');
var hscert = fs.readFileSync('./ssl/myssl_cer.cer');
var ca = fs.readFileSync('./ssl/myssl_cer_INTERMEDIATE.cer');

var options = {
    ca: ca,
    key: hskey,
    cert: hscert
};

var server = https.createServer(options, app);

var http = server;
var io = require('socket.io')(http);

app.set('view engine', 'jade');
app.set('views', path.join(__dirname, '../views'));

app.use('/res', express.static(path.join(__dirname, '../res')));

if (__dirname.indexOf('/dist/') >= 0 || __dirname.indexOf('\\dist\\') >= 0) {
  app.use('/js', express.static(path.join(__dirname, '../client')));
  app.use('/css', express.static(path.join(__dirname, '../css')));
} else {
  const browserify = require('browserify-middleware');
  const less = require('less-middleware');
  browserify.settings({
    transform: ['babelify']
  });

  const tempDir = path.join(os.tmpDir(), 'node-peer-calls-cache');
  app.use('/js', browserify(path.join(__dirname, '../client')));
  app.use('/css', less(path.join(__dirname, '../less'), { dest: tempDir}));
  app.use('/css', express.static(tempDir));
  app.use('/css/fonts', express.static(
    path.join(__dirname, '../less/fonts')));
}

// using Express.Router Middleware
app.use('/call', callRouter);
app.use('/', siteRouter);

io.on('connection', socket => handleSocket(socket, io));

module.exports = { http, app };

I'm running into the same problem.
I had set up an HTTPS connection,
but the call always seems lost.
In peer-calls.com works. I can add as many tabs-browser as I want, and all of them work with my ugly face showing.
In my server, only I see myself; but the rest of myselves (!) are nowhere. They got lost in translation or something.
Look here: http://mrkapl.science:3000

@jeremija can you give the peer-call.com source (without ssl file of course) ?

As you can see, I have two instances running in the same call; and yet, I only see myself.

But, if you look at peercalls.com, it totally works!
Why can this be happening, @jeremija?
Maybe you missed something in the last branch that you put into peer-calls.
By the way, amazing work. Really clever.

PD: The console error throws no errors, all calls made are working perfectly.
Why would this happen?
In my localhost, and my server, the same thing.
Maybe it's something in the core, @jeremija

But yes, @jeremija
Perhaps sharing the code of peercalls.com (without the SSL, like Mr said), and we test it out! (:

@MrDarkSkil he already did, per haps we are doing something wrong. or maybe something went wrong in the last merge.
PD100: I totally disabled my firewall and all, and the same..

To enable https in node natively, you'd have to use the https module similar in the manner mrdarkskill described. Check the https part int he docs here.
In production I use nginx as a reverse proxy server for my peer calls. The app config you'd need looks something like this:
server {
        server_tokens off;

        listen                  443;
        server_name             yourdomain.com;

        ssl                     on;
        ssl_certificate         /path/to/fullchain.pem;
        ssl_certificate_key     /path/to/privkey.pem;

        # optinally, enable strong ciphers to make the connection more secure
        # ssl_protocols           TLSv1.2;
        # ssl_ciphers             ...;

        location / {
                proxy_set_header        X-Real-IP       $remote_addr;
                proxy_set_header        X-Forwarded-For $proxy_add_x_forwarded_for;
                proxy_set_header        Host            $http_host;

                # the following enables websockets
                proxy_http_version      1.1;
                proxy_set_header Upgrade        $http_upgrade;
                proxy_set_header Connection     "upgrade";

                proxy_pass      http://<IP_ADDRESS>:3000;
        }

}

I did.
I have the PeerCalls with HTTPS up and running, @jeremija : https://mrkapl.science:3000
Check it out yourself. Open more than one tab in the same link.
Why? :(

Not what I meant, @jeremija
but thanks!
What I meant it's that only ONE can see how many users are connected to the peer; but only shows the video of myself. Not others.
Try it out yourself, above I left the link

  let peer = peers[user.id] = Peer.init({
    initiator: '/#' + socket.id === initiator,
    stream,
    config: {
      iceServers: [{
        url: 'stun:<IP_ADDRESS>',
        urls: 'stun:<IP_ADDRESS>'
      }, {
        url: 'turn:numb.viagenie.ca',
        urls: 'turn:numb.viagenie.ca',
        credential: 'muazkh',
        username<EMAIL_ADDRESS>      }]
    }
  });

What does this means, @jeremija ?
What are those IP's?
Tried to access them but refused me the attempt.

What do you think, @jeremija ?
http://mrkapl.science:3000

@MrDarkSkil if you don't like this project, feel free to switch whatever whatever you feel competent using. i spent my free time to develop and it works for me, and for many other people; so you can either figure out what the problem is on your end and submit a pull request to fix it if you find out the code is to blame, or stop making inarticulate complaints.
@matikbird this is a list of stun and turn servers used to establish a peer-to-peer connection. you can read more about it here. normally i would advise against storing credentials in version control, however these credentials are publicly available, and will be available from the client side anyway so it doesn't really make sense to hide them.
the bottom line is, i cannot, nor do i have time to provide free support for your personal servers, so i'm closing this.


Publish Sign.Core as a nuget package

Are there any plans to publish Sign.Core as a nuget package so that it can be consumed from other applications? We already have a custom tool to update/sign clickonce .application files and it would be great if we could simply call Sign.Core from it rather than needing to a) call the dotnet sign CLI or b) re-implement the AKV signing code ourselves.

Currently, there are no plans to publish Sign.Core as a NuGet package.  There are a lot of issues we'd need to work through before this becomes a viable option.  We'd like to, but it's not a priority at this time.
Please upvote if this is important to you.

Have you got a list of the issues that would need to be worked on? Is it something that you'd accept contributions for?


Setting i18n::default_locale via config file has no effect.

Setting the default locale in a YAML config file has no effect on i18n::get_locale() or i18n::default_locale().
Example YAML:
i18n:
  default_locale: 'fr_FR'

The reason seems to be that i18n::default_locale() never accesses Config but directly returns the private static.
Proper behavior can be achieved by adding the following to _config.php:
i18n::set_default_locale(Config::inst()->get('i18n', 'default_locale'));

Is this intended behavior? If not, I'll gladly create a PR that fixes getter and setter for default_locale.

I think that's a good fix; Can you PR please for review?
It should probably go into the 3 branch.

PR is here: #4924


Service type override check fails when there is no overrides

Error: invalid path 'environments.master.types.cli', can't query subvalue 'types' of a leaf (leaf value is None)

This happens on deploy when followed "Lagoonize" steps from docs.

I believe this is fixed


Service Request by Emma Rogers and David Lounsbury - MONITORING Feedback/QI field relabeling

[x] e-mail acknowledging that User's Qualtrics Service Request Form was sent
[ ] Problem NOT RESOLVED
[ ] Problem PARTIALLY RESOLVED
[ ] Problem FULLY RESOLVED and Issue is Closed
[ ] Post-Service Request Survey was sent to User

Use comments feature (below) to document:

A clear and concise description of the solution.

A clear and concise description of any alternative solutions or features you've considered.

Any additional context, such as screenshots about the problem.

From: david
lounsbury
Sent: Friday, September 3, 2021 12:01 PM
To: Emma Rodgers<EMAIL_ADDRESS>Timothy Hunt
<EMAIL_ADDRESS>Cc: Nasim Sabounchi<EMAIL_ADDRESS>Mia-Cara
Christopher<EMAIL_ADDRESS>James L. David
<EMAIL_ADDRESS>Donrie Purcell
<EMAIL_ADDRESS>Subject: RE: Taking Coaching Notes in IPA Tracker

Hi
Emma:

We are
open to relabeling and reorganizing the fields in the Feedback/QI component in
the Tracker, so they are more in line with the template that you are asking the
DCs to use. Can you send that template to me? We can look to make these
changes.

You are
correct, to document Feedback/QI over time, the IPA MONITORING Feedback/QI form
is where this information should go.

Note
that Feedback/QI is documented for each named IPA in the MONITORING domain of
the Tracker. In contrast, in the SPECIFICATION domain of the Tracker,
Feedback/QI documented at the level of each Smart Strategy, but it is just,
sort of, do document insights or thoughts about how that Strategy will be
implemented.

Emma,
if you are backstopping the DCs,  it may be useful to link you to our
GitHub repository, so we you can stay informed, and even advise, on bugs and
features that we are working on with Jeff. Would you like that?

Thanks
and best,
David

From: Emma Rodgers<EMAIL_ADDRESS>Sent: Friday, September 3, 2021 11:35 AM
To: Timothy Hunt<EMAIL_ADDRESS>david lounsbury
<EMAIL_ADDRESS>Cc: Nasim Sabounchi<EMAIL_ADDRESS>Mia-Cara
Christopher<EMAIL_ADDRESS>James L. David
<EMAIL_ADDRESS>Donrie Purcell
<EMAIL_ADDRESS>Subject: Taking Coaching Notes in IPA Tracker

CAUTION:   This email comes from an external source; the attachments and/or links may   compromise our secure environment. Do not open or click on suspicious emails.   Please click on the “Phish Alert” button on the top right of the Outlook   dashboard to report any suspicious emails.
Hi Tim and David,

Hope all is well. I
met with Nasim and Erin this morning about the best place for the Research
Coordinators to add notes from coaching calls in the tracker. Currently, they
are taking notes in separate word/Google documents and emailing them to the
county teams. Ideally they would put them in the tracker. It looks like this
could happen in the Feedback/QI section or under each milestone. However, neither
seems exactly right. The Feedback/QI fields do not perfectly align with the
coaching discussion and an ongoing list of notes within the milestones could be
challenging to follow as I believe the notes will just be a long running list.
Also, I believe there isn't much that can be done for this, but I've heard the
tracker is a little slow in making updates. This makes it hard to update
information in real time.

Can we meet next
Wednesday to discuss? I'm available from 9AM - 10AM and 11AM - 12PM.  Please
let me know if this works for your schedules.

Tim - we might also
consider getting rid of the weekly CEF form and asking the CEFs to make updates
in the tracker. This will mean all information is in one place for the coaching
teams to review.

Thanks,
Emma

--
Emma Rodgers, MS
Director of Community Engagement
HEALing Communities Study
Social Intervention Group
Columbia University School of Social Work
<EMAIL_ADDRESS>Phone:<PHONE_NUMBER>
https://sig.columbia.edu/

We at CSSW condemn
anti-Black racism in all its forms and are committed not just to making
statements, but to taking action.

From: david lounsbury
Sent: Friday, September 3, 2021 12:01 PM
To: Emma Rodgers<EMAIL_ADDRESS>Timothy Hunt<EMAIL_ADDRESS>Cc: Nasim Sabounchi<EMAIL_ADDRESS>Mia-Cara Christopher<EMAIL_ADDRESS>James L. David<EMAIL_ADDRESS>Donrie Purcell<EMAIL_ADDRESS>Subject: RE: Taking Coaching Notes in IPA Tracker
Hi Emma:
We are open to relabeling and reorganizing the fields in the Feedback/QI component in the Tracker, so they are more in line with the template that you are asking the DCs to use. Can you send that template to me? We can look to make these changes.
You are correct, to document Feedback/QI over time, the IPA MONITORING Feedback/QI form is where this information should go.
Note that Feedback/QI is documented for each named IPA in the MONITORING domain of the Tracker. In contrast, in the SPECIFICATION domain of the Tracker, Feedback/QI documented at the level of each Smart Strategy, but it is just, sort of, do document insights or thoughts about how that Strategy will be implemented.
Emma, if you are backstopping the DCs,  it may be useful to link you to our GitHub repository, so we you can stay informed, and even advise, on bugs and features that we are working on with Jeff. Would you like that?
Thanks and best,
David

@NBOVE1 : Can you get a copy of the note taking template that the DC are being asked to you, from Emma? Thanks

Sure, will do. I'll follow up.

From: Timothy Hunt<EMAIL_ADDRESS>Sent: Friday, September 3, 2021 12:51 PM
To: david lounsbury<EMAIL_ADDRESS>Cc: Emma Rodgers<EMAIL_ADDRESS>Nasim Sabounchi<EMAIL_ADDRESS>Mia-Cara Christopher<EMAIL_ADDRESS>James L. David<EMAIL_ADDRESS>Donrie Purcell<EMAIL_ADDRESS>Subject: Re: Taking Coaching Notes in IPA Tracker
These are great ideas for consolidating notes.
I would like us to think in regards to WAVE II AND beyond this study.
Currently in this model we are heavy on Coaching and Research Coordinators. In the future it may be a TA provider or supervisor making notes.
Can we make a notes section that is separate from specific STRATEGY monitoring for on-going TA consultation? The specific notes on a strategy should remain for reference I would think.
Specifying the differences between IPA notes, strategy notes and TA notes will be important.
Thanks,
Tim

Met with Emma Rogers and Tim Hunt for a brief meeting regarding this issue.
We talked about how coaches and others are using, or not using, the Tracker to document discussion.
Emma pointed out that most coaches are not using the Tracker during coaching calls with counties.
David suggested that a template that the DCs use for taking meeting minutes during this calls be amended to include sections that would collect minutes that would inform Feedback/QI sections in SPECIFICATION of Smart Strategies and MONITORING of IPAs.
Tim noted that we need to be thinking about who in the community would ultimately be using the Tracker (like a Supervisor).
We all agreed to revisit the utility of the Feedback/QI component, especially its utility relative to the CHALLENGES component.
Emma is going to organize a meeting with Mia, Tim, me to further explore.
Emma noted that the DCs complain that the Tracker is slow to use in real time, during a coaching call. We need to investigate this matter, too, if we want the Tool to be used this way.
David noted that more support for the Coaches in using the Tracker should be organized. David will follow up with @NBOVE1 about this.

Thanks David,
I can certainly support the coaches in the usability of the tracker. Would it be helpful to set up a call with all coaches to get feedback and review the use of the tracker? My hope would be in doing so that coaches are more comfortable utilizing it during coaching meetings. Regarding, DC's using the tracker in real time during coaching meetings, my thought would be to update IPA FEEDBACK/QI to mirror their meeting minutes format used.

Have we  made  a decision to include Jeff on this as part  of the conversation speaks to tardiness of the tracker, which quite honestly I have not experienced ?
I don't have the facts on this but could it be that the coaches themselves are overwhelmed as they have other job functions outside of the HCS and as such learning something (the tracker) that is not really a priority of theirs, one that is more so for the coalition members has affected the role out.
It will be instructive to see the results of Nick's investment in retraining,...

Hi Nick:
I defer to you about how to support the Coaches. I person touch may be what is needed at this point.
What if you polled the Coaches in some way to find out if and how they are using the tracker, if a one-on-one would be helpful?
Thoughts?

I'll reach out to each coach separately to see how I can further support. Many coaches work off of an agenda to keep the meeting structured although I might be able show the usability of the tool to utilize in real time during coaching meetings. I'll keep our team updated.
Best,
Nick

From: david lounsbury
Sent: Monday, September 13, 2021 3:03 PM
To: Donrie Purcell<EMAIL_ADDRESS>Emma Rodgers
<EMAIL_ADDRESS>Cc: Timothy Hunt<EMAIL_ADDRESS>James L. David<EMAIL_ADDRESS>Mia-Cara Christopher<EMAIL_ADDRESS>Samantha W. Lang
<EMAIL_ADDRESS>Subject: RE: Tracker - RC Role

Hi Emma:

In support of Donrie, there is a real need to help each
County review and revise the way they have SPECIFIED one or more of their
IPAs/Strategies. If SPECIFICATION of a given IPA/Strategy is problematic, then
MONITORING it will be, too.

What is needed is more guidance on how we can work with
the counties, the RCs and their coaches to be able to make constructive changes
to their Tracker data at this juncture.

Towards this end, we have a protocol in mind, but we have
been unsure of who to work with to implement it.

As you know, we have been responding to all requests for
Tracker TA, which is a good thing, but we need and want a more deliberative
approach that engages the PMs, the RCs and the Coaches.

Maybe this can be taken up on today’s Coaches Call.

DL

From:
Donrie Purcell
Sent: Monday, September 13, 2021 2:20 PM
To: Emma Rodgers<EMAIL_ADDRESS>Cc: david lounsbury<EMAIL_ADDRESS>Timothy Hunt<EMAIL_ADDRESS>James L. David<EMAIL_ADDRESS>Mia-Cara Christopher<EMAIL_ADDRESS>Samantha W. Lang<EMAIL_ADDRESS>Subject: Re: Tracker - RC Role

Hi Emma,

I don’t understand
why this email was directed to me yet I am happy that you’ve taken the time to
clear this up for I was a bit confused as to who does what and when.

The fact of the
matter is that at best I have been interacting with RC’s, community coalition
members, Jessica, etc on issues relating to the tracker and I was not sure who
is responsible for what.

In future, I would
argue that a clear hierarchy is established and lines of communication be made
clear before such an email is sent.

Have a great day.

Regards
Donrie Purcell, PhD, MPH

Every generation has the obligation to free men's minds for a look
at new worlds,....to look out from a higher plateau than the last generation.
Ellison S. Onizuka

CONFIDENTIALITY: This email (including any attachments) may contain
confidential, proprietary and/or privileged information. Any duplication,
copying, distribution, dissemination, transmission, disclosure or use in any
manner of this email (including any attachments) without the authorization of
the sender is strictly prohibited. If you receive this email (including any
attachments) in error, please notify the sender and delete this email
(including any attachments) from your system. Thank you.

I don’t understand
why you’ve sent me this email. I am a bit lost.
From: Emma
Rodgers<EMAIL_ADDRESS>Date: Monday, September 13, 2021 at 2:03 PM
To: Donrie Purcell<EMAIL_ADDRESS>Cc: david lounsbury<EMAIL_ADDRESS>Timothy Hunt<EMAIL_ADDRESS>James L. David<EMAIL_ADDRESS>Mia-Cara Christopher<EMAIL_ADDRESS>Samantha W. Lang<EMAIL_ADDRESS>Subject: Tracker - RC Role
CAUTION:   This email comes from an external source; the attachments and/or links may   compromise our secure environment. Do not open or click on suspicious emails.   Please click on the “Phish Alert” button on the top right of the Outlook   dashboard to report any suspicious emails.
Hi Donrie,

Hope you had a
great weekend!

Just so we're on
the same page -- my expectation is that the county staff add, delete and edit
IPAs/strategies in the tracker, not the RCs. I'll state this again during the
coaches meeting at 3:30PM, but please make sure to follow this process if you
speak with the county teams.

Thanks,
Emma

--
Emma Rodgers, MS
Director of Community Engagement
HEALing Communities Study
Social Intervention Group
Columbia University School of Social Work
<EMAIL_ADDRESS>Phone: (212)
853-7273
https://sig.columbia.edu/

We at CSSW condemn
anti-Black racism in all its forms and are committed not just to making
statements, but to taking action.

From: david lounsbury
Sent: Wednesday, September 15, 2021 4:17 PM
To: Emma Rodgers<EMAIL_ADDRESS>Cc: Mia-Cara Christopher<EMAIL_ADDRESS>Donrie Purcell<EMAIL_ADDRESS>Nasim Sabounchi<EMAIL_ADDRESS>Timothy Hunt<EMAIL_ADDRESS>Subject: Follow up RE: Tracker Feedback/QI form
Importance: High
Hi Emma, Mia and Tim:
Thanks for the call today.
So, after we got off the call with you, we found a bug IPA MONITORING Feedback/QI component. Data entered into and of the text boxes or into the ratings items cannot be saved.
Second, we appreciate more, now, why the RCs might have been so reluctant to add notes to the IPA MONITORING Feedback/QI: (1) the labels for the text boxes are too specific and somewhat sensitive and, (2) while these text boxes and the ratings make sense in the SPECIFICATION module, we agree that their overall utility is limited.
Third, even if the data were saving properly, the current design does not build a log of Feedback/QI, date stamped for a given period or meeting date.
So, I propose that we take this opportunity to streamline the form for MONITORING Feedback/QI and add a history table feature so that new entries are presented as a date-stamped log. Maybe the new form would have just one box and two variables, as follows --
Implementation progress notes: [                            ]
[date of entry]
[role/name of person who added the notes]
Mia has agreed to meet with use once more, so thank you for that. We will use the meeting to come to terms about the new design.
Best and thanks,
David

From: david lounsbury
Sent: Wednesday, September 15, 2021 4:17 PM
To: Emma Rodgers<EMAIL_ADDRESS>Cc: Mia-Cara Christopher<EMAIL_ADDRESS>Donrie Purcell<EMAIL_ADDRESS>Nasim Sabounchi<EMAIL_ADDRESS>Timothy Hunt<EMAIL_ADDRESS>Subject: Follow up RE: Tracker Feedback/QI form
Importance: High
Hi Emma, Mia and Tim:
Thanks for the call today.
So, after we got off the call with you, we found a bug IPA MONITORING Feedback/QI component. Data entered into and of the text boxes or into the ratings items cannot be saved.
Second, we appreciate more, now, why the RCs might have been so reluctant to add notes to the IPA MONITORING Feedback/QI: (1) the labels for the text boxes are too specific and somewhat sensitive and, (2) while these text boxes and the ratings make sense in the SPECIFICATION module, we agree that their overall utility is limited.
Third, even if the data were saving properly, the current design does not build a log of Feedback/QI, date stamped for a given period or meeting date.
So, I propose that we take this opportunity to streamline the form for MONITORING Feedback/QI and add a history table feature so that new entries are presented as a date-stamped log. Maybe the new form would have just one box and two variables, as follows --
Implementation progress notes: [ ]
[date of entry]
[role/name of person who added the notes]
Mia has agreed to meet with use once more, so thank you for that. We will use the meeting to come to terms about the new design.
Best and thanks,
David

@jbazinet1111
I am looping you in on this issue. We are not ready to recommend changes yet, but we think we will want them soon.
Best and thanks,
DL

I am going to redesign this and present to Jeff ASAP this week. I want this feature in place soon.


How to implement Siamese network architecture using keras

HI all
How to implement Siamese network architecture using keras,  could u give a sample code or  a hint ?
Thank u very much

Its still in progress.#928

Thank u .
BTW, Dose the graph() in keras can solve the Siamese ?

After #928 is merged, yes

Closed by #928

Hi,
I saw the implementation of Siamese network as mentioned on this link:- https://github.com/NVIDIA/keras/blob/master/examples/mnist_siamese_graph.py.
If I understand correctly then it is a network of three layers  where the last one is the output layer. Now, I want to extract feature vector learnt by an intermediate layer which I tried to do using this code snippet as mentioned in the Keras documentation:-  val = model.layers[1].output . This gave me the output as:-
<tf.Tensor 'input_36:0' shape=(?, 850) dtype=float32>
So, my questions/doubts are as follows:
1)How to get the feature vector from this tensorflow "val instance" as obtained above?
2)Since the Siamese network is nothing but the twin network where two networks share the weights so if I am extracting output of this intermediate layer will be referring to either of the layer at the same level in this twin network?
3)Now, the next step I want to do is to use this extracted feature vector and add some noise and do some manipulation with this so further this feature vector can be used as an input for classification. So, would I need to create another Siamese network (if in case I need to use Siamese network only for classification) with the existing model weights or  the existing Siamese model with this modified feature vector itself as an input?. Then in either of the case how can I do implementation.
Any guidance/direction for this implementation would be really helpful for me.
Really sorry for such a long question, but I need to clarify these doubts. Looking forward to getting the answers.
Regards,
Priya Arora


[5434] - new guidance for outside sign-off period

Context
Some of the guidance and reports content only makes sense inside the census or performance profiles sign off periods. For example, the performance profiles guidance mentions a deadline which is now in the past.
Changes proposed in this pull request

defines new copy in views
adds Setting for in_sign_off_period? set to false for now

The content of the report and guidance pages looks good to me.
The title of the report needs to change to match the h1 heading. At the moment it's "Sign off your list of trainees from  for the performance profiles publication - Register trainee teachers - GOV.UK".
It should be "Signing off your list of trainees for the performance profiles publication - Register trainee teachers - GOV.UK"


Add Support for Extracted Comments

gulp-wp-pot does not support extracted comments e.g. using the "translators:" (or other) keyword at the start of comments just prior to a translation function call, e.g.
your-translated-file.php:
<?php
/* translators: Limit to 32 characters or less */
$short_variable = __('Limited Length Phrase', 'sample-textdomain');

might produce a pot entry like so:
sample-textdomain.pot: (excerpt)
#. translators: Limit to 32 characters or less
#: your-translated-file.php:3
msgid "Limited Length Phrase"
msgstr ""

Thank you,
I think this would be good, but it requires a lot of work to make the script find it so I will have it in mind but I don't know when I can implement it. If someone can help implement it, it's always appreciated.

I think the added value is high enough that I will consider implementing it -- I'd like to make the keyword(s) configurable and let the parsing be tolerant cases like encountering opening/ending PHP tags, end of line comments the preceding line, inline comments, and either multi-line or multiple adjacent single line comments. It's enough complexity that perhaps forcing some limitations (e.g. only single-line comments) for an initial attempt would eliminate bugs.

I'm working on a major rewrite right now that will fix this when it's released.

Excited to see this. I'll try to make use of the new support in my project once it makes it into a usable release.

Fixed in 2.0. Defaults to translators: but can be changed with commentKeyword option


How to implement R-FCN using ZF or VGG base model

I tried to implement R-FCN for my own dataset. But my dataset contains lots of tiny objects and I think a relative shallow base network could be better for tiny objects. So I want to train R-FCN using ZF or VGG as base model but I could not find prototxt file for ZF in R-FCN and only faster rcnn. Could anyone help me to implement it?

@liuliu66 Have you finish it ? I use VGG_M model to train, but I find that it can't save model. Maybe the prototxt from faster-rcnn is not correct. The error message show below:
File "/home/yan/disk1/phone_rfcn/py-R-FCN-phone_v1.2/tools/train_net.py", line 119, in <module>     max_iters=args.max_iters)   File "/home/yan/disk1/phone_rfcn/py-R-FCN-phone_v1.2/tools/../lib/fast_rcnn/train.py", line 184, in train_net     model_paths = sw.train_model(max_iters)   File "/home/yan/disk1/phone_rfcn/py-R-FCN-phone_v1.2/tools/../lib/fast_rcnn/train.py", line 132, in train_model     model_paths.append(self.snapshot())   File "/home/yan/disk1/phone_rfcn/py-R-FCN-phone_v1.2/tools/../lib/fast_rcnn/train.py", line 77, in snapshot     self.bbox_stds[:, np.newaxis]) ValueError: operands could not be broadcast together with shapes (20,1024) (8,1)
Do you have any idea? Thanks!


clarify behavior of memory request vs limit parameters

https://kubernetes.io/docs/tasks/configure-pod-container/assign-memory-resource/#exceed-a-container-s-memory-limit says:

A Container can exceed its memory request if the Node has memory available. But a Container is not allowed to use more than its memory limit. If a Container allocates more memory than its limit, the Container becomes a candidate for termination. If the Container continues to consume memory beyond its limit, the Container is terminated.

This paragraph is very confusing. Is this a correct reading?

the container may rely on at least 'request' bytes of memory.
the container is immediately terminated if it ever exceeds 'limit' bytes of memory.
in between these values, the container may be terminated at any time due to memory exhaustion.

It would be helpful to explain what system mechanism is used to limit memory, and whether it interacts with Linux overcommit. Also, whether garbage-collected language runtimes have any particular expectations or preferences. The latter two points could be addressed by links to other articles.

/sig node

/language en

The doc says "[a] Container can exceed its memory request if the Node has memory available", which implies (though it is not unambiguously stated) that if memory is not available, then a container that allocates more than its request is liable to be terminated. If this is true, and it is also true that, as you say, "a container that allocates more than its memory limit is liable to termination" but "might be able to continue", then why have the concept of limit at all? It appears to add nothing.
Assuming limit >= request, there are three situations of interest:

ram <= request
request < ram <= limit
ram > limit
What can the user assume when operating in these three regimes?

/remove-lifecycle stale


Idea: AVS Landing Zone Accelerator to be made available from inside of Azure Portal

It is observed that AVS private cloud is deployed from "inside" of Azure portal. When deployed this way, AVS Landing Zone Accelerator doesn't appear as an option. This results in AVS deployment that is not aligned with AVS best practices recommended through AVS Landing Zone Accelerator.

Hello Mahesh,
Let me know if we can close this.

@Mahesh-MSFT  This is something we will track internally and looking for help

closing this as its accepted as backlog now!


Support for vector data types

Thanks again for your hard work, amazing library.
I may be missing something, but does the library support vector data types, such as float4? I'm working on a project that could really benefit from being able to pack float4, but cannot find docs or support for it.

Seems I answered my question, so I'll write what I did here for future reference and close.
const ARRAY_SIZE: usize = 8;
let value_array = [std::simd::u64x4::default(); ARRAY_SIZE];

// Create an OpenCL SVM vectora
let mut test_values =
    SvmVec::<std::simd::u64x4>::allocate(&self.context, ARRAY_SIZE).expect("SVM allocation failed");

The above is taken from the example and will use long4.


Replace appsody init

The init step currently utilizes the appsody binary and custom stacks to initialize new Service Functions.  This should be replaced by either built-in templates or a git-repo-based solution.  Architecture TBD.

I think git clone -q into a temp directory, then mv node-template <project-dir> (or whatever) is pretty straightforward, not using the git binary though.
Let's iterate/brainstorm on potential extensibility here too

What about packaging template as tar.gz or zip and then download in via plain http(s)?

@matejvasek that works too. I think keeping it simple is important and that's even simpler :). If we go with the tar.gz route, we'd still want to have the templates under RCS of course. Maybe it could be as simple as keeping the templates in github and tagging/releasing them. The CLI could just download the .tgz release files.


compatibility with flexbox and transform

I'm using this in conjunction with a carousel that uses flexbox and transform to display/animate the slides, and the zoom only works on the first slide. After a bit of digging I've found that newLeft needs to be set differently depending on the left value, so something like this:
if(left > 0){ newLeft = Math.max(Math.min(left, this.thumbWidth), 0)     }else{             if(this.thumbWidth + left > 0){                 newLeft = Math.max(Math.min(this.thumbWidth + left, this.thumbWidth), 0);             }else{                 newLeft = Math.max(Math.min(this.thumbWidth-(Math.abs(left)-this.thumbWidth), this.thumbWidth), 0);             }         }
This is the example that worked in my case.

Thanks for the report. Still looking for more use-cases to test the plugin in to figure out where things might break down. Unfortunately I'm down sick at the moment so it may take a little while before I can look into it, but I'll do it as soon as I get a chance.

I did a few more test cases on this, and found the better solution is to actually use getBoundingClientRect() instead of offsetLeft to calculate the left position, especially in cases where the image might initially be outside the viewport (ie. sliders).

Hi @geekyposh - I just released a beta version 0.6.0 to npm where I rewrote a big chunk of how I calculate the zooming position, using getBoundingClientRect() as you suggested. This has resolved some issues I've seen with big pages and several ngx-image-zoom elements. Would you mind giving it a test and see if it resolves your issues as well?

Closing this as fixed in 0.6.0, feel free to reopen or comment if there's still issues with some more details.


Update device-update-raspberry-pi.md

@matsujirushi : Thanks for your contribution! The author(s) have been notified to review your proposed change. @valls

Thank you for your recommendation! I will add it as part of my current PR for the same page.

@ValOlson Do you want us to merge the pull request or close it if you'll be adding the content change in a different pull request?
Thanks.
-PR review team

Feel free to close it, my changes will be in next weeks update.
From: Kristine Toliver @.>
Sent: Friday, March 26, 2021 9:33 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Val Olson @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Update device-update-raspberry-pi.md (#72814)
@ValOlsonhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FValOlson&data=04|01|valls%40microsoft.com|730dbdd8604144b1656f08d8f074c7ce|72f988bf86f141af91ab2d7cd011db47|1|0|637523731662846429|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Ihh75qYQzMDr5gdfI0SEo%2FifF6c6Ane8E%2BY2TLv3nmI%3D&reserved=0 Do you want us to merge the pull request or close it if you'll be adding the content change in a different pull request?
Thanks.
-PR review team

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F72814%23issuecomment-808356682&data=04|01|valls%40microsoft.com|730dbdd8604144b1656f08d8f074c7ce|72f988bf86f141af91ab2d7cd011db47|1|0|637523731662856426|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=a1TNwPUdroj0yEu%2FQ8AWaRoeMjGwaNQJjJqmvtCPZIs%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOMKDHT245ADPLXCFMECNITTFSZKVANCNFSM4Z2U2W5Q&data=04|01|valls%40microsoft.com|730dbdd8604144b1656f08d8f074c7ce|72f988bf86f141af91ab2d7cd011db47|1|0|637523731662866422|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7BFHnrSJussFVSiRUf38Gf%2FrjUeejsIBy80qmqSGjvw%3D&reserved=0.


Defaults options of plugins (legend, tooltips and title) are undefined

Expected Behavior
I'm expecting to have from Chart.defaults.legend the defaults options for legend element.
The same for Chart.defaults.tooltips and Chart.defaults.title.
Current Behavior
Using dist/master, the Chart.defaults.legend is undefined. The Chart.defaults.tooltips and Chart.defaults.title are undefined as well.

Steps to Reproduce
https://codepen.io/stockinail/pen/NWxLZNM

Environment

Chart.js version: dist/master
Browser name and version: FF 78.0.2

Those were moved to Chart.defaults.plugins to avoid special handling of the core plugin defaults (same registration with external plugins)
I guess that info is missing from the migration guide.

Thank you!
Nevertheless it's quite strange to have the options structure different from the defaults. The expectation is to have the save structure.
It sounds as a lack of consistency comparing with all other options.
Anyway I'm gonna change my implementation to adopt this new structure.

In my opinion, to be consistent you should remove tooltips, legend and title nodes from options and force to use the plugin node, everywhere.
Otherwise the tooltips, legend and title nodes should be reimplemented into Chart.defaults.
That's my personal opinion, of course.

@kurkle FYI I've change my impl to use defaults.plugins and it works.
Thank you very much.


Wrong hour of DST end for Europe TZ

In Europe DST ends by 2 am (this year on Sunday 2015-10-25), when clock is moved one hour back. However in Timestamp this transition is one hour earlier ( 1 am - which is hour, when transition happens in US I think).   pytz works as expected.
See code ( tested in pandas 0.17.0 and 0.16.2):
In [1]: import pandas as pd
In [2]: import pytz
In [3]: from pandas import Timestamp

In [7]: pd.__version__
Out[7]: u'0.17.0'

In [8]: cz_tz=pytz.timezone('Europe/Prague')

In [9]: Timestamp('2015-10-25 00:00', tz=cz_tz)
Out[9]: Timestamp('2015-10-25 00:00:00+0200', tz='Europe/Prague')

In [10]: Timestamp('2015-10-25 01:00', tz=cz_tz)
Out[10]: Timestamp('2015-10-25 02:00:00+0200', tz='Europe/Prague')

In [11]: Timestamp('2015-10-25 02:00', tz=cz_tz)
Out[11]: Timestamp('2015-10-25 02:00:00+0100', tz='Europe/Prague')

In [13]: from pandas.tseries.offsets import Hour

In [14]: Timestamp('2015-10-25 01:00', tz=cz_tz) + Hour(1)
Out[14]: Timestamp('2015-10-25 02:00:00+0100', tz='Europe/Prague')

In [16]: from datetime import datetime

In [17]: cz_tz.dst(datetime(2015,10,25, 1, 0))
Out[17]: datetime.timedelta(0, 3600)

In [18]: cz_tz.dst(datetime(2015,10,25, 2, 0))
---------------------------------------------------------------------------
AmbiguousTimeError                        Traceback (most recent call last)
<ipython-input-18-4b6d0ae06c09> in <module>()
----> 1 cz_tz.dst(datetime(2015,10,25, 2, 0))

/home/ivan/tmp/pandas-test/local/lib/python2.7/site-packages/pytz/tzinfo.pyc in dst(self, dt, is_dst)
    445             return None
    446         elif dt.tzinfo is not self:
--> 447             dt = self.localize(dt, is_dst)
    448             return dt.tzinfo._dst
    449         else:

/home/ivan/tmp/pandas-test/local/lib/python2.7/site-packages/pytz/tzinfo.pyc in localize(self, dt, is_dst)
    347         # ambiguous case
    348         if is_dst is None:
--> 349             raise AmbiguousTimeError(dt)
    350
    351         # Filter out the possiblilities that don't match the requested

AmbiguousTimeError: 2015-10-25 02:00:00

In [19]: cz_tz.dst(datetime(2015,10,25, 2, 0), is_dst=True)
Out[19]: datetime.timedelta(0, 3600)

In [20]: cz_tz.dst(datetime(2015,10,25, 2, 0), is_dst=False)
Out[20]: datetime.timedelta(0)

In [21]: cz_tz.dst(datetime(2015,10,25, 3, 0))
Out[21]: datetime.timedelta(0)

I think it is an issue in the datetime string parsing, instead of the underlying datetime type itself:
In [18]: tz=pytz.timezone('Europe/Brussels')

In [19]: pd.Timestamp('2015-10-25 00:00', tz=tz)
Out[19]: Timestamp('2015-10-25 00:00:00+0200', tz='Europe/Brussels')

In [20]: pd.Timestamp('2015-10-25 01:00', tz=tz)   # <-- this should still be unambiguous and returns just a plain wrong timestamp
Out[20]: Timestamp('2015-10-25 02:00:00+0200', tz='Europe/Brussels')

In [21]: pd.Timestamp('2015-10-25 03:00', tz=tz)
Out[21]: Timestamp('2015-10-25 03:00:00+0100', tz='Europe/Brussels')

In [22]: pd.Timestamp('2015-10-25 00:00', tz=tz) + pd.Timedelta('1 hour')  # <-- manually adding one hour however gives the correct value
Out[22]: Timestamp('2015-10-25 01:00:00+0200', tz='Europe/Brussels')

In [23]: pd.Timestamp('2015-10-25 00:00', tz=tz) + pd.Timedelta('2 hour')
Out[23]: Timestamp('2015-10-25 02:00:00+0200', tz='Europe/Brussels')

In [24]: pd.Timestamp('2015-10-25 00:00', tz=tz) + pd.Timedelta('3 hour')
Out[24]: Timestamp('2015-10-25 02:00:00+0100', tz='Europe/Brussels')

In [25]: pd.Timestamp('2015-10-25 00:00', tz=tz) + pd.Timedelta('4 hour')
Out[25]: Timestamp('2015-10-25 03:00:00+0100', tz='Europe/Brussels')

not a bug a all. you are just using this in an incorrect manner. you need to localize a time, simply constructing is exactly that, it just takes what you give it.
In [16]: Timestamp('2015-10-25').tz_localize(cz_tz)
Out[16]: Timestamp('2015-10-25 00:00:00+0200', tz='Europe/Prague')

In [17]: Timestamp('2015-10-25 02:00:00').tz_localize(cz_tz)
AmbiguousTimeError: Cannot infer dst time from Timestamp('2015-10-25 02:00:00'), try using the 'ambiguous' argument

In [18]: Timestamp('2015-10-25 01:00:00').tz_localize(cz_tz)
Out[18]: Timestamp('2015-10-25 01:00:00+0200', tz='Europe/Prague')

In [19]: Timestamp('2015-10-25 03:00:00').tz_localize(cz_tz)
Out[19]: Timestamp('2015-10-25 03:00:00+0100', tz='Europe/Prague')

In [21]: Timestamp('2015-10-25 02:00:00').tz_localize(cz_tz, ambiguous=False)
Out[21]: Timestamp('2015-10-25 02:00:00+0100', tz='Europe/Prague')

In [22]: Timestamp('2015-10-25 02:00:00').tz_localize(cz_tz, ambiguous=True)
Out[22]: Timestamp('2015-10-25 02:00:00+0200', tz='Europe/Prague')

@jreback I agree that localizing is the better option, but I don't really see how the below can be correct:
In [3]: pd.Timestamp('2015-10-25 01:00', tz='Europe/Brussels')
Out[3]: Timestamp('2015-10-25 02:00:00+0200', tz='Europe/Brussels')

1 o'clock should not become 2 o'clock as it is both in the same DST part?

you misunderstand what is happening.
you are taking a naive time, and simply SETTING it to a tz. pytz then sort of 'picks' what it should do w/o any control whatsover. It NEVER should be done this way. I suppose may you could raise but this check is not cheap at all.
The suppose a doc note. But this is just incorrect code.

In any case, I agree this is not good documented. It is not really clear how the tz parameter is handled in Timestamp (no docs on that). I would expect it actually does a tz_localize afterwards?
BTW, DatetimeIndex handles this fine:
`In [15]: pd.DatetimeIndex(['2015-10-25 01:00'], tz='Europe/Brussels')
Out[15]: DatetimeIndex(['2015-10-25 01:00:00+02:00'], dtype='datetime64[ns]', freq=None, tz='Europe/Brussels')

and pytz itself also gives another result:
In [26]: tz = pytz.timezone('Europe/Brussels')

In [27]: print tz.localize(datetime.datetime(2015,10,25,1))
2015-10-25 01:00:00+02:00

In [28]: print pd.Timestamp('2015-10-25 01:00', tz=tz)
2015-10-25 02:00:00+02:00

(there is actually nothing ambiguous to pick in this case, 1am can only be 1am)
DatetimeIndex also raised an AmbiguousTimeError for '2015-10-25 02:00' (while Timestamp picks the second occurence), which seems the more correct behaviour.
@jreback It is true I don't really understand what is happening under the hood, but IMO this is not 'incorrect code' from a user perspective. This is perfectly allowed in our API (why is there otherwise a tz argument in a public class?), and is handled correctly in a related class (DatetimeIndex).

@jorisvandenbossche
I think this is using a different path for the localization when it is passing thru the Timestamp constructor. It should first create it as naive, THEN localize.
In [1]: pd.DatetimeIndex(['2015-10-25 01:00']).tz_localize(tz='Europe/Brussels')
Out[1]: DatetimeIndex(['2015-10-25 01:00:00+02:00'], dtype='datetime64[ns, Europe/Brussels]', freq=None)

In [2]: pd.DatetimeIndex(['2015-10-25 01:00'], tz='Europe/Brussels')
Out[2]: DatetimeIndex(['2015-10-25 01:00:00+02:00'], dtype='datetime64[ns, Europe/Brussels]', freq=None)

In [3]: Timestamp('2015-10-25 01:00').tz_localize('Europe/Brussels')
Out[3]: Timestamp('2015-10-25 01:00:00+0200', tz='Europe/Brussels')

In [4]: Timestamp('2015-10-25 01:00',tz='Europe/Brussels')
Out[4]: Timestamp('2015-10-25 02:00:00+0200', tz='Europe/Brussels')

I suspect the path for a dateutil tz actually works, but this does look incorrect (e.g. [3] == [4] should be True)

our DST experts
cc @rockg
cc @ischwabacher
cc @adamgreenhall
cc @sinhrks

I think this might be the same bug as @ischwabacher reported in #8225 (the America/Havana example)

Thanks for all comments -   just from user perspective - it would be quite weird if Timestamp(x, tz=tz) behave differently from Timestamp(x).tz_localize(tz)  -   the result e in both cases is timestamp in same tz, right?  And it should behave correctly - e.g. DST end  is is 2 am  in this time zone.

I agree that this is a bug. This is a fall transition, not a spring one, so unless there's political hoo-hah going on, this is an ambiguous time rather than a nonexistent one. I'm not sure which offset the result should have, but it should have the original local time in that offset. This is regardless of whether you follow the Timestamp(x, tz=tz) or Timestamp(x).tz_localize(tz) route.
Also I'm not sure whether I deserve the designation of "expert". I just have loud opinions. ;D

@ischwabacher hahah, ok, let's revise to 'interested' parties!

Also, I agree that this is a dupe of #8225.

@ischwabacher hmm, isn't #8225 about parsing though? (or is it really just that the parsing is ok, but its locazling incorrectly?)

@ischwabacher note that the initial example in this thread is not an ambiguous time. '2015-10-25 01:00' is unambiguously defined, as the transition happens from 03:00 -> 02:00, so only the timestamps between 02:00 and 03:00 are ambiguous.
(but anyway, it has of course something to do with the code that handles this fall transition, regardless of it being an ambiguous time or not)
As a side note, I would also prefer that pd.Timestamp('2015-10-25 02:00', tz='Europe/Brussels') would raise an AmbiguousTimeError (as tz_localize does) instead of using the second occurence. But this is indeed maybe costly to check (and it is the default of pytz's localize, is_dst=False)

@jorisvandenbossche I'm not sure about the default handling of an ambiguous time. The latest news from python-dev is PEP 495. One of the themes in the discussion of that PEP was avoiding raising AmbiguousTimeError/NonexistentTimeError. If python-dev wants to avoid exceptions in its environment of individual datetime instances, I think it's definitely worth avoiding in our vectorized context. (FWIW, I think they're wrong, but "they" here refers to Tim and Guido, so the degree of certainty I have in that belief is low.)
@jreback It's definitely a dupe:
In [1]: import pandas as pd

In [2]: pd.__version__
Out[2]: u'0.17.0'

In [3]: pd.Timestamp('2015-10-25 01:00', tz='Europe/Prague')
Out[3]: Timestamp('2015-10-25 02:00:00+0200', tz='Europe/Prague')

In [4]: pd.Timestamp('2015-10-25 1:00', tz='Europe/Prague')
Out[4]: Timestamp('2015-10-25 01:00:00+0200', tz='Europe/Prague')

In [5]: pd.Timestamp('2015-10-25 01:00').tz_localize('Europe/Prague')
Out[5]: Timestamp('2015-10-25 01:00:00+0200', tz='Europe/Prague')

In [6]: pd.Timestamp('2015-10-25 1:00').tz_localize('Europe/Prague')
Out[6]: Timestamp('2015-10-25 01:00:00+0200', tz='Europe/Prague')

Relevant:

The problem is in 67323069fb5e8f1de3000d7606d26172a2e18a9f, presumably somewhere in parse_iso_8601_datetime.

@ischwabacher right, so the parsing is a red-herring. ok then!

actually I don't think the problem is parsing at all. It is a naive time when parsed. It is when assigning it is localized incorrectly I think.

But given that the result is correct/incorrect depending on slight changes in the string format, it seems that it has something to do with the parsing (or triggering another code route):
In [58]:  pd.Timestamp('2015-10-25 01:00', tz='Europe/Prague')
Out[58]: Timestamp('2015-10-25 02:00:00+0200', tz='Europe/Prague')   #  <---- incorrect

In [59]:  pd.Timestamp('2015-10-25 1:00', tz='Europe/Prague')
Out[59]: Timestamp('2015-10-25 01:00:00+0200', tz='Europe/Prague')   #  <---- correct

In [60]:  pd.Timestamp('20151025 01:00', tz='Europe/Prague')
Out[60]: Timestamp('2015-10-25 01:00:00+0200', tz='Europe/Prague')   #  <---- correct

Exactly. If you make the ISO 8601 parser give up, it passes it on to dateutil, which parses it correctly. But you can't get rid of the ISO 8601 parser, because you need a fast path for reading in large CSV files.
But that third example baffles me. I hadn't noticed that before— it's exactly the /YYYY-MM-DD[ T]hh(:mm(:ss)?)?/ format that fails AFAICT.

Further note that the route taken by DatetimeIndex for parsing these ISO 8601 string is also correctly (but don't know if this is a fast path):
In [61]:  pd.DatetimeIndex(['2015-10-25 01:00'], tz='Europe/Prague')
Out[61]: DatetimeIndex(['2015-10-25 01:00:00+02:00'], dtype='datetime64[ns]', fr
eq=None, tz='Europe/Prague')

hmm, these use eactly the same parser. the difference is that in the array processing (e.g. [61]), the tz is handled after, while in a Timestamp it is handled as the result of the out_tz_local IIRC variable (which is where the issue is).

WTB: debugger that runs same code in parallel with different inputs, breaks when code paths diverge.

I don't think that's it, despite the fact that it definitely looks like a bug, since Pacific/Chatham doesn't show this behavior:
In [15]: pd.Timestamp('2015-9-27 03:00:00', tz='Pacific/Chatham')
Out[15]: Timestamp('2015-09-27 03:00:00+1245', tz='Pacific/Chatham')

In [16]: pd.Timestamp('2015-9-27 3:00:00', tz='Pacific/Chatham')
Out[16]: Timestamp('2015-09-27 03:00:00+1245', tz='Pacific/Chatham')

In [17]: pd.Timestamp('2015-9-27 04:00:00', tz='Pacific/Chatham')
Out[17]: Timestamp('2015-09-27 04:00:00+1345', tz='Pacific/Chatham')

In [18]: pd.Timestamp('2015-9-27 4:00:00', tz='Pacific/Chatham')
Out[18]: Timestamp('2015-09-27 04:00:00+1345', tz='Pacific/Chatham')

As @jreback says,  tz localization logic after _string_to_dts (ISO 8601 parser) looks incorrect though Timestamp and DatetimeIndex uses the same parser. Currently _string_to_dts doesn't care DST and regards tz-like string as pytz.FixedOffset.


feat: update to eye v10.13.2

feat: update to eye v10.13.2

:tada: This PR is included in version 16.12.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v16.12.0

Your semantic-release bot :package::rocket:


[carbon dashboard] Content Changes - Tokenized vs Digital

As part of the move from talking about Digital Carbon vs Tokenized Carbon we are doing a scrub of the Carbon Dashboard and looking for content that needs to change.
Each change is being left as a "to-do" in a comment.

Digital Carbon page

[ ] Change "KlimaDAO is holding 16,727,444 carbon assets, making them the largest single holder of tokenized carbon assets." to "KlimaDAO is holding 16,727,443 carbon assets, making them the largest single holder of digital carbon assets."

ToC/Menu

[ ] Change page title link from "Digital Carbon" to "Digital Carbon Overview"

Digital Carbon page

[ ] Change "To date, iFood has retired 115,089 tokenized carbon credits, making them the current leader in tokenized carbon retirements." TO "To date, iFood has retired 115,089 digital carbon credits, making them the current leader in digital carbon retirements."

Carbon Pricing page

[ ] Change page title from "Carbon Pricing" to "Digital Carbon Pricing"

Carbon Supply page

[ ] Change page title from "Carbon Supply" to "Digital Carbon Supply"

@originalpkbims I think this issue is good to go.

PR to review: https://github.com/KlimaDAO/dash-apps/pull/98

Looks good!


RBAC info is way out of date

RBAC details on this pg is very old as they don't even appear to exist anymore.  What roles should we leverage?  thx
#https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-health-operationsocumentation issue guidance

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

ID: 35b52f01-3dee-6d41-f9f6-7710e461ffeb
Version Independent ID: c8ef08a8-d503-b52f-9de8-21b13cfa3d72
Content: Azure Active Directory Connect Health operations
Content Source: articles/active-directory/hybrid/how-to-connect-health-operations.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@dje-git
Thanks for your feedback! We will investigate and update as appropriate.

@dje-git
Thanks for your feedback! We will investigate and update as appropriate.

The screen snaps on this page are way out of date, don't match current experience integrated into main Azure Management portal. Please review and update all snaps and step by step procedures to ensure they match to current portal UX. Thanks

The screen snaps on this page are way out of date, don't match current experience integrated into main Azure Management portal. Please review and update all snaps and step by step procedures to ensure they match to current portal UX. Thanks

Thank for submitting this. I have created a work item for it and will send this to engineering so that they can review it and provide me with updated details. I am not sure what the timeline will be, so I am going to close this for now. But once I have an update I will drop it in here and let you know.
Thank you!
Bill
#please-close


Provide a way to donate to help fund this project.

Most open source projects have a way to donate money to keep them active and having the ability to donate to the author is also a nice way to say thanks for the hard work. I think this project will benefit from having a way to donate financially and I also think the project is big enough to warrant accepting donations.

Thanks for thinking this project is big enough, really appreciate that. I started this project as a way to learn and contribute to the community. I have limited time but do try to release new features every week (Delayed this week, trying to add a feature to select a web server Kestrel/IIS/Nginx and am learning Nginx in-depth to do it). Donating would not speed up my development but I suppose would let people do the equivelant of buying a virtual coffee.

Added to the TODO.


Support '?' to match single character in globbing pattern

From @natemcmaster on Monday, September 26, 2016 12:56:46 PM
Syntax: The ? wildcard character matches a single character.
This would allow us to use globbing to match MSBuild wildcard syntax (see https://msdn.microsoft.com/en-us/library/ms171453.aspx#Anchor_3).
Example scenario: creating a file watcher for an MSBuild file....maybe.
Copied from original issue: aspnet/FileSystem#236

From @dazinator on Wednesday, December 7, 2016 2:22:42 PM
I ended up writing a small globbing library in the end, as the pattern matching stuff currently available in asp.net core didn't implement enough of the globbing spec for my needs.
It handles the following syntax (examples sourced
from wikipedia:

Wildcard
Description
Example
Matches
Does not match

*
matches any number of any characters including none
Law*
Law, Laws, or Lawyer

?
matches any single character
?at
Cat, cat, Bat or bat
at

[abc]
matches one character given in the bracket
[CB]at
Cat or Bat
cat or bat

[a-z]
matches one character from the range given in the bracket
Letter[0-9]
Letter0, Letter1, Letter2 up to Letter9
Letters, Letter or Letter10

[!abc]
matches one character that is not given in the bracket
[!C]at
Bat, bat, or cat
Cat

[!a-z]
matches one character that is not from the range given in the bracket
Letter[!3-5]
Letter1, Letter2, Letter6 up to Letter9 and Letterx etc.
Letter3, Letter4, Letter5 or Letterxx

Just posting on the off chance someone will find that useful.

From @s-trooper on Monday, February 20, 2017 1:11:21 AM
are there plans to support standard globbing patterns?

From @muratg on Monday, March 20, 2017 2:40:17 PM
We'd accept PRs for this.


Set up test version of Airflow upgraded to 2.latest

Set up an instance of Airflow that is based on our existing configuration, upgraded to 2.latest.

Puppet setup of Postgres and Airflow, getting it up and running, successful.
Starting testing on hivevm198

Waiting for input from Systems folk (Ryan?) about existing Puppet configuration to prevent collision during testing

Remaining:

Set up testing duplicate of prod's setup on hivevm198
Instrument upgrade steps

Intermediate snapshot captured in this TOI doc.


Run tests with -swift-version 4

@airspeedswift A few tests still fail, but it's mostly ready.

@swift-ci please test

(just to get the list)

Build failed
Swift Test Linux Platform
Git Sha - e745f449732858f20079c680e745a60364687971

Build failed
Swift Test OS X Platform
Git Sha - e745f449732858f20079c680e745a60364687971

@airspeedswift I'm fixing the last test now.

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - e0767fc71ac5a326f9d6a78b027b3059f4a54901


qemu: 4.0.0 -> 4.1.0

Motivation for this change
Update qemu to version 4.1.0
Changelog - https://wiki.qemu.org/ChangeLog/4.1
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@GrahamcOfBorg build qemu

@GrahamcOfBorg build pkgsMusl.qemu.override{ hostCpuOnly=true; spiceSupport=false; gtkSupport=false; sdlSupport=false; }

@GrahamcOfBorg test virtualbox

@edolstra do you want to review this, or can someone merge?

I'm not sure if its worth a separate issue, so I'll just mention it here.
Note that you'll get error: unsupported configuration: host doesn't support hyperv 'vpindex' feature error if you have vpindex enabled in libvirt vm. This is due to some incompatibility with libvirt <4.6 and qemu>4.0. Upgrading to libvirt >4.6 (nixpkgs contains 4.4) or downgrading qemu to 4.0 should solve the problem.

With libvrit 4.6- 4.7 OS loading freezes- https://github.com/NixOS/nixpkgs/pull/66584#issuecomment-524780748
No ideas how to fix it. I delete this script on my server.


Readme.md cpp syntax highlighting doesn't works on start

VSCode Version: 1.46.1
OS Version: Ubuntu 18

Steps to Reproduce:

Create a Readme.md file with CPP code syntax highlighting like this:

``` cpp
#include <iostream>

int main() {
    std::cout << "Hello World!";
    return 0;
}
```

Save the file and Close VSCode and open again, syntax highlighting doesn't work and you have to close that file and open it again for it to work.

Does this issue occur when all extensions are disabled?: Yes

I can't repo this. Do you see it in VS Code 1.47+?

I updated to 1.47 and its fixed there.


Add explaining-signet to bitcoin-magazine/bitcoin-explained

This PR adds Explaining Signet transcript to the bitcoin-magazine/bitcoin-explained directory.

Thank you for your submission. It appears that the transcript was submitted without any edits or changes made.
Please ensure that you review and make necessary edits before submitting your work in the future. I've closed this submission to allow others the opportunity to review it or for you to revisit.
We hope to see your work in the future. Thank you!

Thank you for your submission. It appears that the transcript was submitted without any edits or changes made.
Please ensure that you review and make necessary edits before submitting your work in the future. I've closed this submission to allow others the opportunity to review it or for you to revisit.
We hope to see your work in the future. Thank you!


Guide maybe? lol

Hey bro, huge Jellyfin fan here annd your git repo is exciting! But I need a little bit of help. It's installed andn everything but I need a bit of help on what to input in Crunchyroll, Jellyfin base URL, API, etc. I know you can get an API from the dashboard and I know how to get the user ID, but I nneed more details, sort of like, is there a certain plugin to install? How is it going to appear in my medias or display?
Please assist bro!

It is not necessary to install any plugin in Jellyfin, simply add the directory where you are generating the strms to your library, everything will appear in your Jellyfin and it is capable of playing the strms natively.
Then the configuration for the integration between the Crunchyroll plugin for ytdlp2STRM is very basic,
jellyfin_base_url: the address of your Jellyfin, if you have it locally then something like http://192.168.1.X:8096, it also accepts domains, in my case I have a no-ip one with type http://xxxx.ddns. net:8096
jellyfin_user_id and jellyfin_api_key you already mentioned that you know the values, just put them.
There is another related parameter in the general configuration and it is ytdlp2strm_temp_file_duration, here you specify in seconds how long you want the temporary files that are necessarily downloaded to last. Default is 86400 seconds (24 hours)
I hope that with this and the README.md I have resolved your doubts.

Thank you, where exactly do I add those files, I've seen them on my disk drive under C:\Media\Twitch. I was wondering where to add them in Jellyfin, as a movie? A show? Live TV?
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: David Grau @.>
Sent: Monday, June 17, 2024 1:36:45 PM
To: fe80Grau/ytdlp2STRM @.>
Cc: Stephane Mallette @.>; Author @.>
Subject: Re: [fe80Grau/ytdlp2STRM] Guide maybe? lol (Issue #51)
It's not necessary to install any plugin in Jellyfin, simply add the directory where you are generating the strms to your library, everything will appear in your Jellyfin and it is capable of playing the strms natively.
Then the configuration for the integration between the Crunchyroll plugin for ytdlp2STRM is very basic,
jellyfin_base_url: the address of your Jellyfin, if you have it locally then something like http://192.168.1.X:8096, it also accepts domains, in my case I have a no-ip one with type http://xxxx.ddns. net:8096
jellyfin_user_id and jellyfin_api_key you already mentioned that you know the values, just put them.
There is another related parameter in the general configuration and it is ytdlp2strm_temp_file_duration, here you specify in seconds how long you want the temporary files that are necessarily downloaded to last. Default is 86400 seconds (24 hours)
I hope that with this and the README.md I have resolved your doubts.
—
Reply to this email directly, view it on GitHubhttps://github.com/fe80Grau/ytdlp2STRM/issues/51#issuecomment-2173967414, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AG5EXGRURM45UHBUR2XAYTTZH4NC3AVCNFSM6AAAAABJNOKQNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZTHE3DONBRGQ.
You are receiving this because you authored the thread.Message ID: @.***>

Ieeeeep.
You can manage the strm folder in plugin settings with strm_output_folder option.
In Jellyfin I use TV Show.


Document that the RSA e value is mandatory when importing.

The lab tried doing a RSA decryption primitive using just n (using p, q) and d.
This failed for 2 reasons:
(1) e is required when importing
(2) Internally e is used for blinding.
Note n and e can be calculated using:
n = pq
e = (1/d) mod (p-1)(q-1)

Checklist

[x] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

Merged to master, 3.1, and 3.0 branches. Thank you.


Make unit tests more valuable

Make some unit tests more valuable by actually check what they should check to verify it the tested functionality works.
This checks if rfc2822() and iso8601() actually return the date in the promised format.

thanks!


new style build doesn't handle using hspec-discover in test suite

https://travis-ci.org/hopper-lang/hopper/jobs/125604461#L1413 for the offending piece in question

derp, seem to not have had hspec-discover in the explicit deps for the test suite

looks like its still an issue :( https://travis-ci.org/hopper-lang/hopper/jobs/125623180#L1413

would adding a cabal install hspec-discover fix it?

@hvr  where would cabal install hspec-discover place the binary (ie so i can include it in my path?)
https://travis-ci.org/hopper-lang/hopper/jobs/126538367#L1455

i currently have the following set in before_install
before_install:
 - unset CC
 - export PATH=$HOME/.cabal/bin:/opt/ghc/$GHCVER/bin:/opt/cabal/$CABALVER/bin:$PATH

idea being ... hspec discover should work / be installed by then?

err, i mean, should be in $HOME/.cabal/bin right?

solve! looks like i needed to do cabal install hspec-discover before the newbuild invocation or it wouldn't land in my path :)
see https://travis-ci.org/hopper-lang/hopper/builds/126541753  for an example (and the associated shift in the travis file :) )

*solved :)


ci(gui-client/linux): see if we can build the GUI Client for ARM

This would make it a little easier to replicate prod issues on old releases
### Tasks
- [x] Add comment to changelog
- [x] Check Vercel preview
- [x] Request review
- [x] Update arches link
- [x] `apt-get update`
- [x] Re-request review

The user guide will have a broken link to the arm64 download after this is merged and before the next GUI Client release is cut. Will that be a problem?

The user guide will have a broken link to the arm64 download after this is merged and before the next GUI Client release is cut. Will that be a problem?

That should be fine. I'll open a new Pr to bump that in a bit.


[FEA] multi-threaded shuffle above 200 partitions

Is your feature request related to a problem? Please describe.
The current Spark shuffle has two modes. For tasks with less than 200 partitions, configurable. It will write the shuffle data out to a temp file per partition, and then in a second pass it will merge the files together in sorted order. For larger numbers of shuffle partitions the data is sorted by the shuffle partition and written out in sorted order. This might involve spilling if the data is too large. The problem is that it is not a graceful transition between the two. If I run NDS scale factor 3k on dataproc with 256 shuffle partitions, not multi-threaded shuffle, I see a 4.7% total run time difference between the two approaches. Going from 200 to 201 partitions is likely to cause a large performance regression.
Also we cannot just configure the shuffle to always use the merge-bypass shuffle. It uses much more system resources when the number of partitions is high. For example the same NDS 3k runs show a 5.88% performance loss when using merge-bypass shuffle at 768 partitions. Going to 896 partitions caused NDS to fail.
Our current multi-threaded shuffle implementation only deals with the 200 partitions or smaller use case. It is really a multi-threaded merge-bypass shuffle. This is likely to compound the performance issues, and it is not clear to an end user that there is a lot more performance to be had if they need a few more shuffle partitions. It is also not clear to the end user that the multi-threaded shuffle is not used when the number of partitions gets too high. It would really be nice to have a way for the performance to transition gracefully, and to have an accelerated version of the sort based shuffle as well.
In addition to this for larger jobs and larger clusters we are likely to want to have more partitions.
It would really be even better if we could optimize for some common cases when a single "batch" of data is output by a task.
Describe the solution you'd like
I am not 100% sure what we want to do. I have a few ideas, but we need to do some experiments to find out what might and what might not work.
When evaluation a shuffle implementation we need to think about a few factors.

write amplification - how much more data do we end up writing than was handed to us. But we are going to measure it in total amount of data written to disk.
read amplification - how much more data do we end up reading than was handed to us. But we are going to measure it in total amount of data read from "disk" (could include just the page cache)
I/O access pattern - is this sequential I/O or random I/O, and how many IOPs can the hardware support.
compression parallelism - can we split the compression/decompression up between multiple threads.
Memory usage

So if we evaluate the various shuffles it can help to show what is happening here.

shuffle
write amplification
read amplification
I/O pattern
comp parallel
memory usage

merge-bypass
2 * compressed size
2 * compressed size
random write + sequential read
task parallelism
(Compression codec + file handle + buffer) * partition * task

multi-threaded
2 * compressed size
2 * compressed size
random write + sequential read
up to partition parallelism
(Compression codec + file handle + buffer) * partition * task

sort-shuffle
spill + compressed size
spill + compressed size
sequential write & read
task parallelism
Compression codec + sort buffer size

The buffer size is managed by Spark and can get rather complicated, but it is by default at most (heap - 300 MiB) * 0.6, but it is shared by all tasks.
From this I have a few ideas that would be nice to play with and test out.
The sort in Spark is really designed around a lot of small rows. It is also designed to be generic for all types of RDDs where they key may not be the partition. But we typically have a few number of large-ish sub-batches. Also the number of partitions is going to be something we can reasonably handle in memory (not the data, but the partitions). We also have a clear separation between the partition key and the data. It would be interesting to see if we could take a hybrid approach here.
We keep the data in memory, but we "sort" it by keeping a List<LinkedList<SubShuffleBatch>> or something like that. The key to the top level list is the partition and then we can tack more sub-batches onto the each partition as we see them. This is to keep the data in the same order it was output by the task. That is not strictly a requirement but would be nice. When we run out of memory and need to spill, we can start by compressing different partitions in parallel and writing them out to disk.  When we write them out to disk we can compress partitions in parallel (up to N Codecs and file handles where N is configurable, but probably 200 by default) and we would replace them with pointers to where they were written out. So for example if I had 400 shuffle partitions to spill and 200 codecs/files to use I would compress the even ones first and write them out to 200 files at the beginning of the files, then I would create new compression codecs and do the odd partitions writing them after the even ones. When all of the data is added we can do a final pass and decide how to build up the final output file.
For any partition where all of the data for it is fully in memory we can compress them in parallel and write them out at the appropriate time to the final location in the file. This would hopefully help with the common case of a small shuffle by not having any read or write amplification.
For any partition where all of its data was written out to a file in a single chunk, we can move it just like we do today for the multi-threaded shuffle.  We might not be able to send an entire file, but we can send a large chunk of it.
For any partition where the data is not all in a single location we are going to have to read in back in and compress it in order.  We can do some compression in multiple threads, but we are going to have to pay attention to memory usage and buffer sizes. If we are okay with some write amplification we can write some number of them in parallel to different files and we try to get the final size and them move them to the final location.
We would have to do some profiling and testing to see what is best and especially on what kind of disks.

Actually I have some new information, which might help us a lot in all of the cases, but especially the case when we had to spill data to disk and there were too many partitions to keep the file handles open.
All of the compression codecs that Spark supports can be concatenated after the fact.
https://github.com/apache/spark/blob/46acedb3842484cb4eadb02f1c1e69e71c334748/core/src/main/scala/org/apache/spark/io/CompressionCodec.scala#L62-L71
Also the default checksum used by Spark adler32  or NONE depending on the version of Spark we are on have a way to calculate the checksum in a distributed way. (The checksum is used as a part of the message handed back to the driver with the shuffle data is committed and written out to the metadata file as well)
For adler32 I need to do a bit more research but CERN did it so we should be able to do it too. http://cds.cern.ch/record/2673802/files/ATL-DAQ-PROC-2019-002.pdf
The only problem shows up if encryption is enabled or if the checksum is CRC32.  So we will need to support the algorithm I described originally for when encryption is enabled but we might be able to have a very fast common case for shuffle.  If creating a compression output stream is cheap enough we should be able to compress each batch as we see it come in, in parallel in a thread pool along with a checksum for the data. We can decide if that should stay in memory or go out to disk directly, but I think keeping it in memory might be the best. Then if we need to spill we can write it out to disk and keep a pointer to where it is along with the checksum.
If the checksum is CRC32 we can still do the compression in parallel, but we would have to calculate the checksum serially on the resulting data.
When we need to write out the final data we can move the compressed data to the proper location in the final file and start combining checksums together, if needed.
The details of the output stream used are.
SerializationStream( // Converts the objects into bytes
  CompressionOutputStream( // Optional compresses the data
    ErrorHandlingOutputStream(CryptoOutputStream( // Optional encrypts the data
      ManualCloseOutputStream( // Does a flush instead of a close and the close is separate
        BufferedOutputStream(
          ChecksumOutputStream( // Optional if empty none is set
            TimeTrackinOutputStream( // used for metrics
              FileOutputStream
            )
          )
        )
      )
    ))
  )
)

A few things to keep in mind.
First if spark.file.transferTo is enabled, and the there is no encryption and the compression codec can be concatenated, then spark will use NIO (file channels) to transfer the data to the final location.  Apparently there are some bugs in old versions of Linux that make this not work. I don't think we support any version of linux with this bug in it (2.6.32 according to the error message), so we might be able to always to the faster transfer.
Second spark.shuffle.sync is off by default, but if it is enabled then we need to make sure that the data hits the disk before we go on with shuffle (or we need to fall back to the old shuffle code)
The reason I am bringing this up is that in order to make this work we are likely going to have to copy over a lot more code from Spark at a lower level for shuffle.  We also are going to have to be much more careful about auditing any shuffle changes while we are at it.


Remember grid view column widths across app launches

Is your feature request related to a problem? Please describe.
It's possible to resize the column widths on the Outputs and Inputs tab, but those widths aren't persisted across app launch. When you get them sized the way you like you'll lose it when you quit Mobiflight then re-launch.
Describe the solution you'd like
Remember the column widths across app launch.
Additional context
Requested by Glenn in Discord
I asked ChatGPT and this is totally possible to do without too much effort.

What's the motivation behind this?
What information is not displayed correctly and how does the resizing make it better?
How does the resizing work across different config? Will it require adjustment for specific configs or is it really working well once adjusted?

What's the motivation behind this?
What information is not displayed correctly and how does the resizing make it better?

Here's an example from my Outputs tab:

With the default widths I can't see all the output information even though there's plenty of overall width in the app.
Here's examples from my Input tab, same issue:

How does the resizing work across different config? Will it require adjustment for specific configs or is it really working well once adjusted?

I was thinking the column widths could get stored in the config file directly, so each file maintains its own widths and just sets the columns correctly when opened. Maybe?


TypeError when used with xusenlinzy/api-for-open-llm

xusenlinzy/api-for-open-llm is another backend similar to LocalAI that can provide OpenAI-like API for local LLMs. However it is wierd that its generated completions end with a None, then shell_gpt would show a TypeError.

Maybe better to add type checks before concatenating?
Places are:

sgpt/cache.py:40
sgpt/handlers/handler.py:37

I'm seeing the exact same thing when using BerriAI/litellm as the backend.  Litellm would be particularly useful in combination with shell_gpt because it would allow switching between a local model and OpenAI using just the --model switch.

I didn't test sgpt with any other backends except OpenAI and LocalAI, but as I can see the error you are getting is related to cache. Try to disable caching sgpt --no-cache "test".

ShellGPT with LiteLLM was released as 1.3.0. This should fix the issue, given that I'm closing this issue, but feel free to reopen it.


Unauthenticated firebase functions?

Hi,
Currently, I have deployed the algolia-firestore extension along with a transform function as explained in the README. With my current deployment, I get this error:
Error when performing Algolia index { FetchError: invalid json response body at https://accounts.google.com/ServiceLogin?passive=true&continue=https://el.appengine.google.com/_ah/conflogin%3Fstate%3D~< some long key >
reason: Unexpected token < in JSON at position 1

So looks like Algolia app is not properly authenticated before calling the firebase transform function. This results in the app skipping the transform and just writing the firebase doc as it is.
The README Mentions:
The Transform Firebase Function should be set up to unauthenticated users at this time
But there doesn't seem to be any option to set https callable functions to be unauthenticated in firebase. Can you please help understand what exactly I need to do for the transformation to work?

I found the issue. I was using a different location for the firebase function that what I had deployed Algolia on.


[Frontend] Manage/Add Testimonial on Admin Panel Added

Issue that this pull request solves
Issue Link resolve #1049
Closes: #1049
Brief description of what is fixed or changed

[x] Introduce a new card called Testimonial on admin dashboard, Clicking on this should take us to a page having 2 cards Create Testimonial and Manage Testimonial
[x] The Create Testimonial should open a form which admin will use to save the testimonials in DB
[x] The Manage Testimonial should have list of testimonials fetched and each of the fetched testimonial card should have a - delete button, clicking on which the testimonial should get removed from the DB.
[x] Backend also Integrated.

Types of changes
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] My changes does not break the current system and it passes all the current test cases.

@Kajol-Kumari  can you please check and merge it.

@Kajol-Kumari  updated can you please check.


adds a customization for how much of the filename gets truncated

The length of the filename in the go guru output was hardcoded to 20,
and I wanted it longer, so I added a customization.

Can you rebase or squash this PR to get rid of the extra merge commit?

I think I did this right. Let me know :)


feat(utils): add signRecoverable method to the ECPair class

add signRecoverable method to the ECPair class, and use it to sign the transactions.
BREAKING CHANGE: use signRecoverable instead of sign method to sign the transactions.

Codecov Report

Merging #260 into develop will increase coverage by 0.09%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #260      +/-   ##
===========================================
+ Coverage    95.84%   95.94%   +0.09%
===========================================
  Files            9        9
  Lines          289      296       +7
  Branches        76       90      +14
===========================================
+ Hits           277      284       +7
+ Misses          11        8       -3
- Partials         1        4       +3

Impacted Files
Coverage Δ

packages/ckb-sdk-utils/src/ecpair.ts
96.29% <100%> (+1.29%)
:arrow_up:

packages/ckb-sdk-rpc/src/resultFormatter.ts
91.3% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


tweaked config

*Issue:*https://github.com/bcgov/entity/issues/21296
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

/gcbrun

Temporary Url for review: https://business-dashboard-dev--pr-3-bl1h0zf4.web.app


Does this support receiving messages?

e.g. creating a bot etc
if not, any reason why it couldn't be added? i believe the webhook url supports this

It does not -- and reasons for it not to would be complexity I guess.
Maybe you could describe your use case a bit more though? To support receiving the messages we'd need to have a small web stack (at minimum a little rack app, if not Roda or Sinatra) to take in the messages and then hand off to whatever is handling the commands. By that point I would think that it'd be better to defer to the user's existing stack or suggest using a dedicated bot project like Lita.

Ok. So I made something to actually accept a url (using this gem).
It isn't generalized yet, it only works for github labels on issues on pull requests, but it will expand. Also, it's open source so if you want to add something please do! https://github.com/Reliefwatch/slack-reconciler

@barmstrong going to close, I think this is answered. Feel free to open again.


Got stuck after executing the auto-instrumentation command

Describe the bug
I tried to instrument my application on the same host according to this doc, but after I execute the cmd sudo OTEL_GO_AUTO_TARGET_EXE=/data/home/server OTEL_SERVICE_NAME=my_service OTEL_EXPORTER_OTLP_ENDPOINT=xxxxxxxx  ./otel-go-instrumentation, it got stuck and only show one line message: {"level":"info","ts":1725605497.1578906,"logger":"go.opentelemetry.io/auto","caller":"cli/main.go:86","msg":"building OpenTelemetry Go instrumentation ...","globalImpl":false}. I tried on several machines but the error is the same.
The application is only a simple web app using net/http.
Environment

OS: Linux
Go Version: go1.23.0 linux/amd64
Version: v0.14.0-alpha

To Reproduce
Steps to reproduce the behavior:

Run the targert application: /data/home/server;
Run the OpenTelemetry Go Automatic Instrumentation: sudo OTEL_GO_AUTO_TARGET_EXE=/data/home/server OTEL_SERVICE_NAME=my_service OTEL_EXPORTER_OTLP_ENDPOINT=xxxxxxx  ./otel-go-instrumentation. My endpoint is normal.

Expected behavior
Show messages or logs about instrumented successfully.
Additional context
The output is only one line msg: {"level":"info","ts":1725605497.1578906,"logger":"go.opentelemetry.io/auto","caller":"cli/main.go:86","msg":"building OpenTelemetry Go instrumentation ...","globalImpl":false}.
I also tried instrument my app using docker compose, and everything is ok.

I've solved the problem with changing OTEL_EXPORTER_OTLP_ENDPOINT to OTEL_EXPORTER_OTLP_TRACES_ENDPOINT, maybe there is some issues of my endpoint.
By the way, I want to know the difference between OTEL_EXPORTER_OTLP_TRACES_ENDPOINT and OTEL_EXPORTER_OTLP_ENDPOINT?


Attribute of Interactive widget is not changed.

I've updated the Coverflow patch. #314
Coverflow normally creates a default image.
The selected images are saved in the local images directory.
Finally, select the Effect option and it will be applied to the Container.
But! The problem doesn't apply to the Container again if you choose the Effect option differently.
Of course, applying the File Size option does not apply.
However, it seems to be applied normally in Preview.
Pressing the save button will save the normal code logic to the html.
The point of the problem I thought was that adding the widget and applying the first attribute worked fine. Since then, it doesn't seem to be working properly.
Please check.

@DohyungLim could you list up reproduction steps so it will be easier to analyse your issue?

Patch #314  must be applied first.
But Tomasz says that the Attributes Tab breaks.
I and Ms.Hwang (Cookie) confirmed on the local PC that the issue was resolved.
I need a review to see if patch 314 is the problem.
If patch #314  is applied, issue #315  is guided below.
1.D & D the Coverflow or i3d widget in the Design-Editor.
2.Select multiple (CoverFlow) images.
3. Apply the effect. (example: wheel)
4. Apply different effects. (wheel-> coverflow)
The effect is not applied and appears in the container as a list. -> The problem is.


Cant use curl with video titles having special characters

Checklist

[ ] I'm reporting a broken site support issue
[ ] I've verified that I'm running youtube-dl version 2019.10.22
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

[debug] System config: []
[debug] User config: ['-f', 'bestvideo[height>2160]+bestaudio[acodec=opus]/(bestvideo[height=2160][fps>30]+251)/bestvideo[height=2160]+251/bestvideo[height=2160]+140/(bestvideo[height=1440][fps>30]+251)/bestvideo[height=1440]+251/bestvideo[height=1440]+140/(bestvideo[height=1080][fps>30]+251)/bestvideo[height=1080]+251/bestvideo[height=1080]+140/(bestvideo[height=720][fps>30]+251)/bestvideo+251/bestvideo+bestaudio/best', '--write-sub', '--sub-lang', 'en,en-US,en-CA,en-GB', '--convert-subs', 'ass', '--merge-output-format', 'mkv', '-o', '%(title)s']
[debug] Custom config: []
[debug] Command-line args: ['-v', '-f', '244+250', '--external-downloader', 'curl', 'https://www.youtube.com/watch?v=3BpnB47Fxts']
[debug] Encodings: locale cp932, fs mbcs, out cp932, pref cp932
[debug] youtube-dl version 2019.09.28
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: ffmpeg N-94953-gef50cf7b32
[debug] Proxy map: {}

The complete verbose log with curl is too big, so here is the end part
100 19.5M  100 19.5M    0     0  81226      0  0:04:12  0:04:12 --:--:--  122k
* Connection #1 to host r4---sn-cvh76nes.googlevideo.com left intact
ERROR: unable to download video data: [WinError 2] The system cannot find the file specified: '[MV] FIESTAR(피에스타) _ APPLE PIE(애플파이).f244.part'
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpyi91grvc\build\youtube_dl\YoutubeDL.py", line 1919, in process_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpyi91grvc\build\youtube_dl\YoutubeDL.py", line 1864, in dl
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpyi91grvc\build\youtube_dl\downloader\common.py", line 366, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpyi91grvc\build\youtube_dl\downloader\external.py", line 52, in real_download
  File "C:\Python\Python34\lib\genericpath.py", line 50, in getsize
FileNotFoundError: [WinError 2] The system cannot find the file specified: '[MV] FIESTAR(피에스타) _ APPLE PIE(애플파이).f244.part'

Description

It seems like youtube-dl is unable to pass the filename with special characters to curl, but it has no problems doing it on its own without an external download manager. In this case the file '[MV] FIESTAR(피에스타) _ APPLE PIE(애플파이).f244.part' actually has been downloaded and renamed as '[MV] FIESTAR() _ APPLE PIE().f244.part' instead(so youtube-dl thinks the file does not exist). The subtitles downloaded by youtube-dl on its own has been properly named '[MV] FIESTAR(피에스타) _ APPLE PIE(애플파이).en.vtt'. Is there no work around for this ?

And Github is formatting the filename. Its actually this

> py -3.7 .\youtube_dl\__main__.py https://www.youtube.com/watch?v=3BpnB47Fxts --external-downloader curl
[youtube] 3BpnB47Fxts: Downloading webpage
[youtube] 3BpnB47Fxts: Downloading video info webpage
[youtube] 3BpnB47Fxts: Downloading js player vflsEMaQv
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
[download] Destination: [MV] FIESTAR(피에스타) _ APPLE PIE(애플파이)-3BpnB47Fxts.f137.mp4
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
 13 85.0M   13 11.1M    0     0  4130k      0  0:00:21  0:00:02  0:00:19 5353k
ERROR: Interrupted by user

Ofcourse, you cant reproduce it, since you interrupted the download. Try completing it.
 youtube-dl -f 244+250 --external-downloader curl https://www.youtube.com/watch?v=3BpnB47Fxts
[youtube] 3BpnB47Fxts: Downloading webpage
[youtube] 3BpnB47Fxts: Downloading video info webpage
WARNING: en-US subtitles not available for 3BpnB47Fxts
WARNING: en-CA subtitles not available for 3BpnB47Fxts
WARNING: en-GB subtitles not available for 3BpnB47Fxts
[info] Writing video subtitles to: [MV] FIESTAR(피에스타) _ APPLE PIE(애플파이).en.vtt
[download] Destination: [MV] FIESTAR(피에스타) _ APPLE PIE(애플파이).f244
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 19.5M  100 19.5M    0     0   166k      0  0:02:00  0:02:00 --:--:--  195k
ERROR: unable to download video data: [WinError 2] The system cannot find the file specified: '[MV] FIESTAR(피에스타) _ APPLE PIE(애플파이).f244.part'

It renames it as mentioned in my previous comment after downloading the whole track and then says the system cannot find the file specified.
Try finding the part file you interrupted download for, you will see it has been renamed.


refactor: rename SortedSetPut to SortedSetPutElements

Adjusts request and response types appropriately.

this matches node.js so it looks right to me, but will defer to @poppoerika and @poppoerika to make sure that's how we're naming stuff for list requests etc.

This is in line with the API spec and other SDKs. As far as lists go, the equivalent operations would be ListPushFront/Back so no mention of Elements. We do have ListRemoveValue as opposed to SetRemoveElement, but that is consistent with the spec and other SDKs. This LGTM.


[TPC-DS] Use "nonpartitioned" input for Jenkins jobs

[TPC-DS] Use "nonpartitioned" input for Jenkins jobs

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Spark Runner Tpcds Tests

Run Dataflow Runner Tpcds Tests

R: @mosche

I'd suggest to create a ticket describing the problem with the partitioned dataset and reference it in the PR / commit


Support Referrals

Referrals reward members when they invite others to expand the community. How should this mechanic work in Convey?
Options;

Fixed coin reward for each referral that signs up - the downside is it encourages account creation, but not content contribution or discussion engagement.
Coin reward based on referral's first coin purchase - encourages account creation and coin purchases, but again not contribution or engagement.
Coin reward based on referral's first conversation's yield - encourages member to invite people the believe will do well on and generate a high yield
Coin reward based on referral's first digest publication's reward - similar to 3, but slightly more forgiving (a new user's first post might not be as successful as a later post)

Any other ideas?

What do you want to incent?  I'd argue it should be some combination of "bring in users who pay" and "bring in users who play the content game well".  I'd say that for today, the content is way more valuable than the money, so I'd just reduce it to some function of referred users' earnings.

I agree the priority for now is to incentivize content and engagement, so basing referrer's reward on the referral's earnings makes sense.

Do you have a preference between these two approaches?

You enter a friend's email, platform send them an email invitation
Platform issues you a code/link, you give that to your friend

I'd go with 2-it's strictly more capable, although it is less amenable to people just spamming all their email contacts.

Referrals are now supported! Head over to https://convey.earth/account to find your referral link and invite your friends - you'll get a reward whenever any of your referrals appear in the digest