text:
Connection closed issue

./v380 -p Secretpass1 -addr 192.168.0.31
Connection closed
Connection closed
Connection closed
Connection closed
Connection closed
i tried every option combination with the all possible params(except -retry --enable-ptz=0) but always receive connection closed
./v380 --discover
ID:  16171531
IP:  192.168.0.31
MAC: 1c:bf:df:ca:29:12
i have verified my values all are right(i have changed all given values for privacy)
It does work using the V380S Apk, but i would like to receive video for linux
OS: Debian GNU/Linux 10 (buster) x86_64 4.19.0-8-amd64 #1 SMP Debian 4.19.98-1 (2020-01-26) x86_64 GNU/Linux
There is no firewall blocking and i can also open a telnet connection to port 8800

@ihateadmins could provide wireshark data for connection sequence?

I'm having the same issue. Been trying everything I can think of. Camera was purchased on Amazon under manufacturer name Autens. It's the nicest of these bulb cams I have found and has 3 IR lights with real night vision, as well as better resolution than any others I have tried.
nmap shows only available port is 8800 and your software is the closest I have been to communicating with it, outside of my Android's V380 Pro app that can interact with it and see the video stream. I can netcat or telnet to its 8800, but it never replies, and like the other person here your software gets connection closed, although --discover somehow is able to see it with the correct ip, mac, and p2p ID.
My Wireshark dumps can see my phone connecting to it, but only shows Mac address of my phone and the camera, and doesn't show any TCP communication, and I never see anything like a handshake, port, or a URL. All the data in the packets seems like giberish to me as I have no idea how to decode it.
Any advice would be greatly appreciated.

@sbcdave @ihateadmins I believe I found out the problems, I will begin fixing now

Thank you. Now if I run any of the connection commands without piping the output anywhere I get stoi 5 times instead of the failed connection message.
I tried piping the output to ffplay, ffmpeg, and the gstreamer server, without any luck yet. Probably just some mistakes I'm making though. I'll keep trying later today and let you know. Thanks

@sbcdave ah yes, I made some mistake. You could connect with -id though instead of -addr

Ok thanks. For your information, your readme mistakenly says -ip where it
should say -addr.
Dave Goldsmith
Electrical Engineer - Astronics AES
Data Acquisition Lead - KWH
Veteran | Helicopter Pilot | Medic
+1.425.268.1938 | Dave@KiloWattsforHumanity.org
On Tue, Mar 31, 2020, 9:24 AM Syahmi Azhar notifications@github.com wrote:

@sbcdave https://github.com/sbcdave ah yes, I made some mistake. You
could connect with -id though instead of -addr
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prsyahmi/v380/issues/5#issuecomment-606732921, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADST3WK3FZSLUHXDMUXBQKDRKIKNVANCNFSM4LFEMQNQ
.

Fixed, thanks! Does it connect to cam properly now?

I haven't figured out how to get video out of it yet, but it send to be
connecting now.
I'm not familiar with ffplay and not very familiar with ffmpeg, and the
example lines you give are throwing bash errors about ) for some reason,
maybe my version of ffmpeg or I'm typing something wrong.
I got the ffmpeg line to work, and didn't see anything in my browser (error
message) with localhost:8090
If you have any advice or suggestions I can try them now, but I'm supposed
to be working so I'll have to wait until tonight to really dig into it and
try to figure it out on my own.
Dave Goldsmith
Electrical Engineer - Astronics AES
Data Acquisition Lead - KWH
Veteran | Helicopter Pilot | Medic
+1.425.268.1938 | Dave@KiloWattsforHumanity.org
On Tue, Mar 31, 2020, 10:02 AM Syahmi Azhar notifications@github.com
wrote:

Fixed, thanks! Does it connect to cam properly now?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prsyahmi/v380/issues/5#issuecomment-606752767, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADST3WI3ZEPLBJ7V5GU3X5LRKIOZXANCNFSM4LFEMQNQ
.

@sbcdave Perhaps you got error like https://github.com/prsyahmi/v380/issues/4
Since I mainly use windows, the syntax might be different on linux
You could try:
./v380 -u username -p password -id 123456 | ffplay -vf 'setpts = N / (25 * TB)\' -i -
or
./v380 -u username -p password -id 123456 | ffplay -i -

I just tried to email you with some attachments and it bounced because it
was too big. I'm going to send the text first and then try to send the
attachments individually.
Looks like it might not be working. It seems to connect and open ffplay,
but data rate shows as 0kbs
I'm very unfamiliar above IP layer, but my research suggests this is a p2p
camera that uses layer 2 (Mac to Mac, no IPs) for communication across my
network. Is it possible that the camera is bound to my smart phone's MAC,
and I should reset the camera and then try from the computer?
Ultimately I need the video stream fed to an end point so I can capture it
with zoneminder (my home surveillance software).
I've attached screenshots of running the two lines you sent me just now, as
well as the most informative output I've gotten from nmap, which suggests
there is an IP layer connection available on 8800 with a sun web server
running, but I haven't figured out anyway to get a response from the web
server.
Thanks again for your help.
Dave Goldsmith
Electrical Engineer - Astronics AES
Data Acquisition Lead - KWH
Veteran | Helicopter Pilot | Medic
+1.425.268.1938 | Dave@KiloWattsforHumanity.org
On Tue, Mar 31, 2020, 10:29 AM Syahmi Azhar notifications@github.com
wrote:

@sbcdave https://github.com/sbcdave Perhaps you got error like #4
https://github.com/prsyahmi/v380/issues/4
Since I mainly use windows, the syntax might be different on linux
You could try:
./v380 -u username -p password -id 123456 | ffplay -vf 'setpts = N / (25 *
TB)' -i -
or
./v380 -u username -p password -id 123456 | ffplay -i -
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prsyahmi/v380/issues/5#issuecomment-606767173, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADST3WLYBCYQVWIZCINIYZDRKISA5ANCNFSM4LFEMQNQ
.

@sbcdave I forgot to mention, with ffplay I always need to wait some minutes before ffplay window will appear. Maybe you can try to wait first a minute or two. Not sure why is this happen, it may be codec related. I have no knowledge how h264 packet works. Perhaps there is a corruption on packet header or something is just missing.
The camera may transmit video with variable framerate and the -vf 'setpts = N / (25 * TB)' will fix ffplay to 25fps, this can prevent image juddering due to seek head reach end of stream first before new stream comes available.
The camera is a server-client using TCP. The server on camera port 8800 is what v380 software communicate with.

the -probesize 32 may help.
./v380 -u username -p password -id 12345678 | ffplay -vcodec h264 -probesize 32 -formatprobesize 0 -avioflags direct -i -

No luck yet. I let ffplay run for an hour periodically hitting enter at the terminal and it would just create a new line that also showed all zeros.
The other bulb cameras I have seen are all lower resolution than this one, I started looking through your code and wonder if the video encoding stuff you're using is designed for a lower resolution?

Another thought, this camera advertised having better security. I wonder if it's an https connection and maybe your code is only designed for http?

I just confirmed I was able to view another bulb camera's rtsp stream with ffplay, and tried using the same ffplay settings while piping your v380 output to ffplay, but same thing happens, it never seems to get the video stream

@sbcdave if you run without piping, does it output garbage to console?

So I'm off work now and digging in. I added a bunch of debug print statements to your code and I can see that it's stuck in the recv loop and generally has an unknown type (no type case matching, I added a default case that prints unknown), but it occasionally receives type 0x16, which your comments call audio, but don't do anything with. Any ideas?

Ha, I just missed your comment as I was commenting. No, it holds the Shell and doesn't output anything. I had to print to stderr to get any output out of it

I wonder if it's working fine and I just need to redirect stderr to ffplay. I'm not sure how I can do that though. I'll meet around with it.

Can you check if the resp->version > 2

Yeah version is 3

It looks like to me you have received video frame packet, maybe try output it to file instead to check if it really working:
./v380 -u -p -id xxx > file.h264

It starts with 13 loops that get type 52, then it alternates between x16 and 53. I tried passing the unknowns and x16 all to ffplay and it just showed a bunch of errors. Maybe I should try to only pass the type 53. I'll try that now

Got it :) I had to let 52 and 53 use the same code from your x01. Thanks a million. Any ideas for audio?

@sbcdave nice! I didn't find type 52/53 on the official v380 app though, maybe packet is corrupted. I have some changes that I will push later, maybe you can give that a try.
The audio code is there but requires muxing (combine video+audio together). I have looked into matroska, but seems complicated at that time. Maybe I will take a look at it again later.
You can try to output just the audio, comment out the video part and let it play with ffplay.

Cool thanks. Right now I'm struggling to get ffmpeg http steam working and keep getting an error about requiring audio. Looks like I should be able to bipass it with proper ffserver setup, but I'll gave to dig into that tomorrow. Thanks again for the troubleshooting help :)

sry that i didn't upload my wireshark trace, i have completely forgot this thread. But ye after i downloaded the recent version everything works perfectly fine. Thanks a lot!!!

I got around the audio and got it setup as an rtsp stream with ffserver now. I'm having some blur issues that are probably related to fps, but all is totally unrelated to this issue now :) Thanks again for all the help. I'm digging into trying to get the audio data parsed through an aac codec (assuming v380 bulb cam is sending aac encoded audio with type x16). If I have any luck I'll let you know. At the moment my next attempt is going to be to have your code output the video to one file and the audio to another file, but I still need to figure out how to do something like tail on both files at the same time and feed them both into ffmpeg as the video and audio inputs.

I have "SDETER 1080p 720P IP camera" from https://www.aliexpress.com/item/4000012251071.html.
The camera works fine with V380 Pro mobile application, but haven't been able to get it working with this v380 project.
I am getting the following: types 0x1 (video) and 0x16 (audio). There are also a lot of 0x1f commands that are currently not parsed. The issue is similar to what sbcdave was having. ffplay/ffmpeg is not playing the stream no matter what I have tried. If the output of v380 is redirected to the file, I can see it is outputting something. However, haven't been able to play the output file with ffmpeg or vlc.
I have tried the following without things:
-tried all the ffmpeg and ffplay commands mentioned here
-printing the cur frame. It seems do the following sequence 0,0,0,0,1,2,3,...N,0,0,0,1,2,3,....N, where N is between 7  and 14.
-added handling for 52 and 53 types sbcdave did
-port scan reports the following ports open: 8800 and 9800
-the login seems to work, returning the version 3.
I think the problem is related 0x1f commands, not sure what to do with those. Also noticed the android app "V380_v5.4.apk" from https://github.com/brusdev/V380-SDK is not able to show the video from this camera. Perhaps there is a difference between V380 and "V380 PRO" models?
Any help is highly appreciated! Thanks.

Irs been a while since I've looked at the code or thought about this, but I was never able to get the audio working, and what did eventually work for me was to have this code output only the "correct" "type", where "correct" was which ever type had only video data. So first modify the code to build you a list of types, and let it run for a minute or so to make sure you know all the types you might want to try. Then with the list of types try sending one at a time as the only output of the code, with a catch all that does nothing with the other types. Cycle through all types one by one to see if you get one that works. Good luck

@user201534 is it possible that your camera uses x265 ?

Mine's x264 if you were asking me. Sorry I just saw this.
Aside from getting the audio working, I still have a lot of the screen all grey very often and only temporary (few seconds) instances where I can see the entire window

authors: ihateadmins prsyahmi sbcdave user201534


text:
Add summary comments to public members

Add <summary> comments to public members:

classes
interfaces
methods
properties

@akravch should this issue be closed?

@pryabov It should, thanks,

source: gharchive/issue

authors: akravch pryabov

repo: pryabov/scheduler-bot

license: MIT

license_type: permissive

license_source: github-api


text:
Decidir estruturas de dados

Decidir como armazenar:

Vagas do estacionamento
Andares
Clientes
Histórico de cliente

tudo lista ou .txt

authors: lorennavictoria pserey

repo: pserey/parktech


text:
Would dnsReaper qualify?

Bit of a curve ball... We check for subdomain takeover attacks, which can be useful in CSP bypasses etc.
https://github.com/punk-security/dnsReaper

I'm kind of resisting extending this to cover all pentesting tools, as it then becomes too unweildy and hard to maintain 😁

authors: SimonGurney psiinon

repo: psiinon/open-source-web-scanners


text:
Error: connect ECONNREFUSED 127.0.0.1:33330

Hi @psiphi75 , I am looking for a solution to control someone's computer through my web browser. But I am getting the error while connecting with the port 33330. I have also tried by opening this port from the control panel but no luck.
Please have a look at the attached screenshot.

You need to be running the proxy on localhost (127.0.0.1). ECONNREFUSED means the connection has been refused, likely due to proxy not running.

license_source: bigquery


text:
Face

turn back everything to views and handle

i will create another one to update the video and chat box at the same time

Open a new PR #24

source: gharchive/pull-request

repo: psu-oss-group/ChatAppFaceID


text:
Admin email notification when percentage of open access publications changes

This depends on https://github.com/psu-stewardship/researcher-metadata/issues/123
Each time we record new stats for the total number of publications vs. the number of publications that are open access, we should compare the percentage of open access publications to some past statistics, and if the change in the percentage is greater than some threshold value, we should send an email notification to administrators so that they can investigate the reason for the change.
Questions:

To what past data should we compare the current percentage? Should we just compare to the last snapshot? What if the notification threshold is, for example, -5% change and the actual change has been -4% from each of the last 5 snapshots to the next? (Maybe this depends on how often snapshots are taken.)
What exactly is the threshold value for notification?
Should we notify about both positive change and negative change?
To what email address(es) should the notification be sent?

Some answers if we were to do this soon:

The snapshots are weekly now; I'd suggest comparing to the last four snapshots.
Looking at the data, I'd say 1 point of change in OA percentage would be an appropriate threshold. The only changes of that magnitude that I see since weekly snapshots began in April 2021 are July 23, 2021 (likely due to reminder emails sent then) and November 12, 2021 (in the middle of development).
I would notify about both positive and negative change.
Send notifications to openaccess@psu.edu. The team monitoring that (mostly me) could determine whether it requires a larger conversation.

authors: EricDurante anaelizabethenriquez

repo: psu-stewardship/researcher-metadata


text:
Evaluate the flash message coming from Sufia after upload.

The messages coming out of Sufia after upload seem a bit bogus.  We should make sure they are something the user can understand.

Your files are being processed by ScholarSphere in the background. The metadata and access controls you specified are being applied. Files will be marked Private until this process is complete (shouldn't take too long, hang in there!). You may need to refresh this page to see these updates.

We need to remove "Files will be marked Private until this process is complete" and change that to "You files will be displayed  below as they are processed"

repo: psu-stewardship/scholarsphere


text:
kills widgets in 1.x.x

after install plugin, widgets and widgets screen in ACP is not available

Can confirm.
"Internal error.
Oops! Looks like something went wrong!
/api/admin/extend/widgets
Cannot call method 'map' of null"
is displayed on widgets page

authors: Pappmann philip-ulrich

repo: psychobunny/nodebb-plugin-custom-pages


text:
onyo git commit without -m "message" crashes

onyo git commit -m "commit these changes" works as expected, but onyo git commit (should open editor for commit message) crashes.

Both with and without -m, I get an error from onyo (source code from 2022-08-, but no crashes nor commits:
I edited one of my templates, to have something to commit.
(onyo) aghisla@juvuzela in ~/source/onyo-testrepo on git:master!
❱ onyo git commit                                                           1 !
ERROR:onyo:
Onyo expects a clean onyo working tree before running commands. Please commit or .gitignore all changes and check the syntax of asset files.

The Onyo working tree is not clean.
Changes not staged for commit:
	 .onyo/templates/laptop.template
Please commit all changes or add new files to .gitignore
(onyo) aghisla@juvuzela in ~/source/onyo-testrepo on git:master!
❱ onyo git commit -m"Commented the vendor model type keys in laptop template, duplicates of the asset creation questions."
ERROR:onyo:
Onyo expects a clean onyo working tree before running commands. Please commit or .gitignore all changes and check the syntax of asset files.

The Onyo working tree is not clean.
Changes not staged for commit:
	 .onyo/templates/laptop.template
Please commit all changes or add new files to .gitignore

But I seem to get the same error with onyo git status and a few more commands. What should I check next?

You have found another bug in onyo git, that must have been introduced after Tobias filed this bug.
Onyo runs some checks before executing commands (similar to onyo fsck). One of these checks is ensuring that the git repo is clean.
You have created the template file, and thus there is an untracked file.
It seems this check is also done for onyo git, which somewhat defeats the purpose of the command.

True. I wonder if it should be just simply removed, or if there would be value to run the checks in the end to get output describing the situation after the onyo git commands.

Closing as wont-fix with the merging of #201

authors: TobiasKadelka aghisla aqw

repo: psyinfra/onyo

license: ISC


text:
Show app name before it's processed by BSI

Discussed in https://github.com/ptarmiganlabs/butler-sheet-icons/discussions/262

Originally posted by mountaindude January  1, 2023
As of version 2.1.5 the log output for each app is:
...
2023-01-01T18:04:53.304Z info: About to process app a3e0f5d2-000a-464f-998d-33d333b175d7
2023-01-01T18:04:55.534Z info: Opened app a3e0f5d2-000a-464f-998d-33d333b175d7
2023-01-01T18:04:55.547Z info: Number of sheets in app: 5
...

Add a new log line, showing what the app name is.
This will make it easier to follow along what BSI is doing, what app is causing troubles etc.

Note to self: This requires a lookup to be done against the QRS API.

repo: ptarmiganlabs/butler-sheet-icons


text:
Handle P(foo=1) | P(foo=2) as P(foo__in=[1, 2])

Right now P(foo=1) gets completely discarded during eval.

Wow, nice catch. I'm pretty amazed that the test suite passes with a bug this significant.

authors: lucaswiman staticshock

repo: ptone/django-predicate


text:
Add Airtel IP

Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[x] Your submissions are formatted according to the guidelines in the contributing guide
[ ] Your additions are ordered alphabetically
[x] Your submission has a useful description
[x] The description does not end with punctuation
[x] Each table column should be padded with one space on either side
[x] You have searched the repository for any relevant issues or pull requests
[x] Any category you are creating has the minimum requirement of 3 items
[ ] All changes have been squashed into a single commit

Thanks for contribution @ytteroy :)

authors: matheusfelipeog ytteroy

repo: public-apis/public-apis


text:
Import / export genesis state for curating and stake

Fixes #254

Codecov Report

Merging #258 (1429097) into master (56fbe3f) will decrease coverage by 0.18%.
The diff coverage is 5.57%.

@@            Coverage Diff            @@
##           master    #258      +/-   ##
=========================================
- Coverage    9.18%   9.00%   -0.19%
=========================================
  Files          59      59
  Lines        8450    8665     +215
=========================================
+ Hits          776     780       +4
- Misses       7590    7799     +209
- Partials       84      86       +2

Impacted Files
Coverage Δ

x/curating/keeper/genesis.go
13.33% <0.00%> (-31.12%)
:arrow_down:

x/curating/types/curating.pb.go
0.58% <0.00%> (-0.03%)
:arrow_down:

x/curating/types/genesis.pb.go
1.24% <0.00%> (-0.81%)
:arrow_down:

x/curating/types/post.go
75.00% <ø> (ø)

x/curating/types/upvote.go
0.00% <0.00%> (ø)

x/stake/keeper/genesis.go
0.00% <ø> (ø)

x/curating/keeper/post.go
59.37% <100.00%> (+0.64%)
:arrow_up:

x/curating/keeper/upvote.go
57.14% <100.00%> (+1.04%)
: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 56fbe3f...1429097. Read the comment docs.

Codecov Report

Merging #258 (1429097) into master (56fbe3f) will decrease coverage by 0.18%.
The diff coverage is 5.57%.

@@            Coverage Diff            @@
##           master    #258      +/-   ##
=========================================
- Coverage    9.18%   9.00%   -0.19%
=========================================
  Files          59      59
  Lines        8450    8665     +215
=========================================
+ Hits          776     780       +4
- Misses       7590    7799     +209
- Partials       84      86       +2

Impacted Files
Coverage Δ

x/curating/keeper/genesis.go
13.33% <0.00%> (-31.12%)
:arrow_down:

x/curating/types/curating.pb.go
0.58% <0.00%> (-0.03%)
:arrow_down:

x/curating/types/genesis.pb.go
1.24% <0.00%> (-0.81%)
:arrow_down:

x/curating/types/post.go
75.00% <ø> (ø)

x/curating/types/upvote.go
0.00% <0.00%> (ø)

x/stake/keeper/genesis.go
0.00% <ø> (ø)

x/curating/keeper/post.go
59.37% <100.00%> (+0.64%)
:arrow_up:

x/curating/keeper/upvote.go
57.14% <100.00%> (+1.04%)
: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 56fbe3f...1429097. Read the comment docs.

Replaced by https://github.com/public-awesome/stakebird/pull/260

Replaced by https://github.com/public-awesome/stakebird/pull/260

authors: codecov-io shanev

repo: public-awesome/stakebird


text:
match config var names, increase gas

fixes to get further into yarn run init

i was working with the old config.example.js format and didn't update it.

repo: public-awesome/stargaze-nft


text:
toStars(addr) validate coin type

coin type 118 is compatible with keplr
https://medium.com/chainapsis/keplr-explained-coin-type-118-9781d26b2c4e

make a strict whitelist of coin type 118. test and add new coins manually

repo: public-awesome/stargaze-tools


text:
Consider removing genericName

As of now, genericName is mandatory and limited to 35 characters.
However, field experience is showing that this field always contains information that is already carried by shortDescription, softwareType and categories. We should consider removing genericName if no good reason for it arises.

Posted to the forum as https://publiccode.discourse.group/t/consider-removing-genericname-issue-59/65

authors: alranel bfabio


text:
[BUG] || UAT  [Uploaded Docs not shown in Case file PDF]

Uploaded Docs not shown in Case PDF in UAT
Steps to reproduce the behavior:

Go to UAT
File a case
Check Case PDF

Expected behavior
All uploaded docs should be shown in Case PDF
Screenshots

PDF ATTACHED
case 3.pdf

Resolved in QA
CC: @rajeshcherukumalli

Working fine in UAT.

repo: pucardotorg/dristi


text:
usage to replace pug addWith

while working on https://github.com/pugjs/pug-code-gen/issues/21, I have been trying to use this plugin to replace the addWith feature of pug at the AST level.
There are 2 things that are not clear to me :
1/ If I use the plugin on a code block where no global is found, then it seems to leave the prelude intact while I would have expected the plugin to remove it
If i did not make a mistake,
// @with
{
  locals || {};
  { var a = 1; }
}

outputs
  locals || {};
  { var a = 1; }

but the leading expression should maybe be removed when it is not used (?)
2/ the generated code is
(function (console, str) {
  console.log(str);
})("console" in _ref ?....

when pug has something like
(function (console, str) {
  console.log(str);
}).call(this, "console" in _ref ?

I suspect that forcing the bind on this was done for some reason but I do not know.

locals || {} is not side-effect-free since locals was not declared in the code (as determined by Babel's isPure function)
The this binding is automatic through Babel's shadow function property, which is implemented as an internal plugin. Try writing some code with this and you'll see that a _this variable is declared outside of the function.

Thanks for the response. I feel like I enter a new land as babels internals are totally new to me and a bit ... huge ;-)
ok I understand point 2. and saw _this appear which I understand.
regarding point 1. do you mean that the expression locals could have a side-effect ? It probably can by a mechanism that I do not grasp. In that case, the old with(locals || {}) can be understood to have the same side effect so it would make sense to keep it.
I ask these questions because I try to keep the generated AST as close as I can to the original pug generated js and these 2 things appeared.

Regarding the locals || {} maybe the plugin will remove them if I create the surrounding function(locals) {} beforehand. I'll try that.

Yeah, that's how it should work. As to why this is Babel's behavior, I can't say I am too sure about that; but if a variable is global, then it could have a side effect if a getter is assigned. Consider for example the following (albeit unlikely) case:
var counter = 0;
Object.defineProperty(window, 'locals', {
  get() {
    counter++;
    return {};
  }
});

with (locals || {}) {
  var a = 0;
}

console.log(counter);

If locals || {} is just removed then counter would equal 0, instead of the expected 1.

Yes that makes sense. I'll try with the surrounding function to see what happens.
can I ask you why you wrote 2 different implementations of with as per

For the with module, you can now either use the new (not-yet-published) https://github.com/pugjs/with, which takes a Babylon AST, or https://github.com/pugjs/babel-plugin-transform-with.

what differences do you see in them since from my quick reading the babel version of with does not make use of babel-plugin-transform-with which surprises me given the non trivial implementations of both

with uses Babylon directly and is quite a bit faster (since it uses the lightweight babylon-walk instead of the heavyweight babel-traverse, and changes the source code directly instead of mutating the AST then regenerating the source). And it seems like I misspoke about with taking a Babylon AST -- it doesn't, taking a string as input instead.
babel-plugin-transform-with is more complete (handles not only returns but also breaks, continues, yields, and awaits), but is a lot slower.

@TimothyGu I managed to use babel-plugin-transform-with inside the ast based pug-code-gen but I run into 1 last issue with a test case.
The template is
mixin my-mixin(arg1, arg2, arg3, arg4)
  p= arg1
  p= arg2
  p= arg3
  p= arg4

+my-mixin('1','2','3','4')

And the generated code is
function helloWorld(locals) {
  var pug_html = "",
      pug_mixins = {},
      pug_interp;

  var _ref = locals || {};

  (function (arg2, arg3, arg4) {
    var pug_indent = [];

    pug_mixins["my-mixin"] = pug_interp = function (arg1,  arg2,  arg3,  arg4) {
      var block = this && this.block,
          attributes = this && this.attributes || {};
      pug_html = pug_html + "\n";
      pug_html = pug_html + pug_indent.join("");
      pug_html = pug_html + "\u003Cp\u003E" + pug.escape(null == (pug_interp = arg1) ? "" : pug_interp) + "\u003C\u002Fp\u003E\n";
      pug_html = pug_html + pug_indent.join("");
      pug_html = pug_html + "\u003Cp\u003E" + pug.escape(null == (pug_interp = arg2) ? "" : pug_interp) + "\u003C\u002Fp\u003E\n";
      pug_html = pug_html + pug_indent.join("");
      pug_html = pug_html + "\u003Cp\u003E" + pug.escape(null == (pug_interp = arg3) ? "" : pug_interp) + "\u003C\u002Fp\u003E\n";
      pug_html = pug_html + pug_indent.join("");
      pug_html = pug_html + "\u003Cp\u003E" + pug.escape(null == (pug_interp = arg4) ? "" : pug_interp) + "\u003C\u002Fp\u003E";
    };

    pug_indent.push("");
    pug_mixins["my-mixin"]('1', '2', '3', '4');
    pug_indent.pop();
  })("arg2" in _ref ? _ref.arg2 : typeof arg2 !== "undefined" ? arg2 : undefined, "arg3" in _ref ? _ref.arg3 : typeof arg3 !== "undefined" ? arg3 : undefined, "arg4" in _ref ? _ref.arg4 : typeof arg4 !== "undefined" ? arg4 : undefined);

  ;
  return pug_html;
}

can that be a bug with babel-plugin-transform-with or is it related to the mention that "Supports arguments (unlikely to be supported)" is not supported ?
it seems that arg2, arg3 and arg4 are wrongly considered globals.

note that it may be related to a problem in my AST. when I try to create a test case for the module (starting from source code as string) the library correcly detects the globals. I will keep you posted.

OK. Sorry for the noise here. I had a bug where I forgot to trim arg names. Leading spaces on identifiers were leading to the problem.
So it seems all pug tests pass now :-)
thanks

authors: TimothyGu jeromew

repo: pugjs/babel-plugin-transform-with


text:
Spider pug

Esse spider vai ajudar na issue https://github.com/pugpe/encontros-palestras/issues/2.
É preciso ainda atualizar o spider pra extrair data, hora e local.

@andersonberg muito obrigado!
não ficou claro se você pretende atualizar o spider ainda. vou assumir que é um wip.
Caso contrário, só dar um toque que eu já testo aqui e dou um merge.

Massa @andersonberg !
Por se tratar de uma spider bem simples, não seria melhor dispensar a arquitetura de projeto scrapy e usar apenas o spider.py para ser executado com scrapy runspider?
Acredito que assim deixaria o projeto mais enxuto e fácil de contribuir.

@heliomeiralins sim, esqueci de completar, é um WIP. Ainda precisamos incluir data, hora e local e depois gerar o readme. Eu posso fazer isso, mas deixo aberto se alguém também quiser contribuir.
@giuliocc é uma ótima ideia. Esse spider eu tinha feito para outra palestra do pug e eu precisava da estrutura do scrapy pra demonstrar outras coisas.

ótimo, @andersonberg . eu atribuí a task a você então.

Pra rodar o script para um evento específico:
$ python pugpe.py lxv
Por enquanto tem que ser o número em romano, porque tem que ser igual a URL.

Ainda vou adicionar alguns comentário no código e talvez um pequeno readme sobre os scripts

Tá ficando show! Quando tiver finalizado é só avisar que a gente prossegue com o review :)

@heliomeiralins @giuliocc Acho que já podem revisar! :)

com a biblioteca roman, agora dá pra rodar com:
$ python pugpe.py 65

Fiz uns testes e os únicos problemas que tive foi com instabilidade do próprio site ou formatação totalmente diferente (encontro 63).
No geral acho que ficou muito bom.
Obrigado @andersonberg ! :fireworks:

authors: andersonberg giuliocc heliomeiralins

repo: pugpe/encontros-palestras


text:
JXPagerListContainerViewController计算出来的生命周期在手指切换的时候是错误的

bug提交前阅读
1.请先仔细阅读文档，尤其是【其他注意事项】文档，里面有常见问题的解决方案；
2.请先搜索已有的Issues，尝试找到相同问题的讨论；
3.请先阅读相关类的API、源码，尝试从源码找到答案；
4.如果以上步骤都未能解决你的疑问，欢迎提交ISSUE；
bug描述
生命周期在用手指切换tab的时候全部错误了
以下两段代码计算生命周期有问题，而且是必现的，我尝试修复，短时间解决不了，麻烦作者给点思路或者给修复下

(void)scrollViewDidScroll:(UIScrollView *)scrollView {
......
[self listDidAppearOrDisappear:scrollView];
}

(void)scrollViewDidEndDecelerating:(UIScrollView *)scrollView {
//    NSLog(@"2222222");
if (self.willDisappearIndex != -1) {
[self listWillAppear:self.willDisappearIndex];
[self listWillDisappear:self.willAppearIndex];
[self listDidAppear:self.willDisappearIndex];
[self listDidDisappear:self.willAppearIndex];
self.willDisappearIndex = -1;
self.willAppearIndex = -1;
}
if (self.delegate && [self.delegate respondsToSelector:@selector(listContainerViewWDidEndScroll:)]) {
[self.delegate listContainerViewWDidEndScroll:self];
}
}

版本号
1.JXPagingView版本号：2.0.12
2.OC还是Swift版本 OC
3.手机系统版本号：13.6.1
4.手机型号：iphone xs max
复现步骤

遇到问题的相关效果、类是：JXPagerListContainerView
2.具体的操作步骤是：比如作者的Demo，第一个例子打开，正常下面三个tab，你左右切换，或者点头部切换都是没问题，生命周期都是正常的，但是如果你能力tab切换到爱好tab，切过去的时候手指一直按照不松开，也就是手指拖动整个屏幕宽度的距离，等切换到第二个tab的时候，生命周期调用的是前一个tab的，以下代码会出问题吗，整个生命周期就混乱了。

截图、GIF
提供bug出现时的截图或者GIF。

可以看到已经用手指切换到了第二个tab，但是你看下面回调的生命周期是错误的，下表是0

(void)scrollViewDidEndDecelerating:(UIScrollView *)scrollView {
if (self.willDisappearIndex != -1) {
CGFloat currentIndexPercent = scrollView.contentOffset.x/scrollView.bounds.size.width;
NSInteger disappearIndex = self.willDisappearIndex;
NSInteger appearIndex = self.willAppearIndex;
if (self.willAppearIndex > self.willDisappearIndex) {
if (currentIndexPercent >= (self.willAppearIndex)) {
NSLog(@"EEEEEEEEEEEEEE");
[self listWillAppear:appearIndex];
[self listWillDisappear:disappearIndex];
[self listDidAppear:appearIndex];
[self listDidDisappear:disappearIndex];
self.willDisappearIndex = -1;
self.willAppearIndex = -1;
}else{
[self listWillAppear:disappearIndex];
[self listWillDisappear:appearIndex];
[self listDidAppear:disappearIndex];
[self listDidDisappear:appearIndex];
self.willDisappearIndex = -1;
self.willAppearIndex = -1;
}
}else {
if (currentIndexPercent <= (self.willAppearIndex)) {
          NSLog(@"FFFFFFFFFFFFFFFFF");
          [self listWillAppear:appearIndex];
          [self listWillDisappear:disappearIndex];
          [self listDidAppear:appearIndex];
          [self listDidDisappear:disappearIndex];
          self.willDisappearIndex = -1;
          self.willAppearIndex = -1;
      }else{
          [self listWillAppear:disappearIndex];
          [self listWillDisappear:appearIndex];
          [self listDidAppear:disappearIndex];
          [self listDidDisappear:appearIndex];
          self.willDisappearIndex = -1;
          self.willAppearIndex = -1;
      }
  }

}
if (self.delegate && [self.delegate respondsToSelector:@selector(listContainerViewWDidEndScroll:)]) {
[self.delegate listContainerViewWDidEndScroll:self];
}
}

我直接在这个方法这样写了，您之前是直接不管，就回滚到之前的index，这边要做下判断，您看下是否可以，这个生命周期对我太重要了

`- (void)scrollViewDidEndDecelerating:(UIScrollView *)scrollView {
if (self.willDisappearIndex != -1) {
    CGFloat currentIndexPercent = scrollView.contentOffset.x/scrollView.bounds.size.width;
    NSInteger disappearIndex = self.willDisappearIndex;
    NSInteger appearIndex = self.willAppearIndex;
    if (self.willAppearIndex > self.willDisappearIndex) {
        if (currentIndexPercent >= (self.willAppearIndex)) {
            NSLog(@"EEEEEEEEEEEEEE");
            [self listWillAppear:appearIndex];
            [self listWillDisappear:disappearIndex];
            [self listDidAppear:appearIndex];
            [self listDidDisappear:disappearIndex];
            self.willDisappearIndex = -1;
            self.willAppearIndex = -1;
        }else{
            [self listWillAppear:disappearIndex];
            [self listWillDisappear:appearIndex];
            [self listDidAppear:disappearIndex];
            [self listDidDisappear:appearIndex];
            self.willDisappearIndex = -1;
            self.willAppearIndex = -1;
        }
    }else {
        if (currentIndexPercent <= (self.willAppearIndex)) {

            NSLog(@"FFFFFFFFFFFFFFFFF");
            [self listWillAppear:appearIndex];
            [self listWillDisappear:disappearIndex];
            [self listDidAppear:appearIndex];
            [self listDidDisappear:disappearIndex];
            self.willDisappearIndex = -1;
            self.willAppearIndex = -1;
        }else{
            [self listWillAppear:disappearIndex];
            [self listWillDisappear:appearIndex];
            [self listDidAppear:disappearIndex];
            [self listDidDisappear:appearIndex];
            self.willDisappearIndex = -1;
            self.willAppearIndex = -1;
        }
    }
}
if (self.delegate && [self.delegate respondsToSelector:@selector(listContainerViewWDidEndScroll:)]) {
    [self.delegate listContainerViewWDidEndScroll:self];
}

}`
我直接在这个方法这样写了，我没有看得很细，您看下是否就可以，我看你的代码是直接不管就回滚到之前的index，我这边做了判断，看下是否可以，我目前看来生命周期就正常了，看你是否可以优化下，修复这个Bug，生命周期对我来说太重要了，我可能考虑不全，麻烦您看下

我也遇到这个问题了，手指拖到到快要完全切换的时候松开就会出现，导致vc的viewDidAppear和viewDidDisappear完全乱了，非常致命

你能在iOS14.2系统复现吗？

这个必现的，生命周期完全是乱了，看来大佬没时间修复了，我自己fork下来重写逻辑了

手动滑，来回切，生命周期完全是乱的，作者大大应该没时间修复了，自己改改吧 @yautou

@DeftMKJ 方便贴一下你修改的代码吗？我还没有复现出来

authors: DeftMKJ pujiaxin33 yautou


text:
HTTP requests for unsupported content types should issue a 406 or 400 response

Backtrace
View full backtrace and more info at honeybadger.io

We'll reopen if we see this again.

authors: jrgriffiniii tpendragon

repo: pulibrary/dpul


text:
link to holding location and sort based on code

Closes #41.

Why did the coverage drop so much?

That's odd - it decided to include all of the engine-related files in the coverage this time. Can we configure coverage to ignore the files?
lib/locations/version.rb, lib/locations/engine.rb, lib/locations.rb, lib/generators/locations/install_generator.rb
The install_generator clearly works if the specs are able to run...

@tampakis: you can add filters to exclude stuff.  See https://github.com/pulibrary/plum/blob/master/spec/spec_helper.rb

Thanks @escowles! Back to 100%

authors: escowles jpstroop tampakis

repo: pulibrary/locations

license: mit


text:
Adds pagination on the events page

Adds pagination on the events page. Displays 50 records per page.
Adds will_paginate gem
FactoryBot for event,
events_controller_spec
fixes part of #690

Coverage increased (+0.007%) to 81.954% when pulling a0c678378e32855fb65b206efebe4fb2ce45c523 on 690-pagination-events-page into d9b88bdf5e0f1fd58538c4fc14694cea823aaf7c on master.

authors: christinach coveralls


text:
Display problems w/ slideshow

Which record is that? Guessing this is just a case of a quotation mark not getting escaped

Shoot, forgot the URL. I'll try to get it tomorrow.
-Js
Sent via mobile. Please excuse typos, brevity, etc.
On Feb 7, 2017 12:25 PM, "Trey Pendragon" notifications@github.com wrote:
Which record is that? Guessing this is just a case of a quotation mark not
getting escaped
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pulibrary/pomegranate/issues/224#issuecomment-278073434,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9BFYp2z2rFFqu-dcZJF5kn4ipv8zXSks5raKjtgaJpZM4L5fYC
.

That's a good guess, anyway. Here's that record:
https://dpul.princeton.edu/spotlight/pudl0127/catalog/d0586fbf8d62b5bb8c0daf990f77cb87
This might also be related:
https://dpul.princeton.edu/spotlight/pudl0127/catalog/4136399ea9b7c73c33a83a37c7f60f60
In this case there is a ' in the "Description" field. So there are escaping issues both ways.
And characters that can't be rendered by UV as well. I think we have a ticket for that already though

This is still a problem.

authors: jpstroop tpendragon

repo: pulibrary/pomegranate


text:
Project show page

Update the Show page to display the data captured in https://github.com/pulibrary/tiger-data-app/issues/170 and https://github.com/pulibrary/tiger-data-app/issues/171
See https://github.com/pulibrary/tiger-data-app/issues/113 for UI ideas
Acceptance Criteria:

[ ] Be able to show the information stored in the database about a project
[ ] It should not contact mediaflux, but may need to in the future

Hey team! Please add your planning poker estimate with Zenhub @bess @jrgriffiniii @kelynch

authors: carolyncole hectorcorrea

repo: pulibrary/tiger-data-app


text:
Not getting proper height using this.

Hi,
I am using this in custom tableViewcell but its not getting proper height.
Please provide some details.  this is hapening for some cells only. plz help me to solve this

The height must be calculated automatically. What is your exact scenario?
El 16 may. 2016 16:23, "Rameshwar Gade" notifications@github.com escribió:

Hi,
I am using this in custom tableViewcell but its not getting proper height.
Please provide some details.
[image: screen shot 2016-05-16 at 7 52 11 pm]
https://cloud.githubusercontent.com/assets/14885326/15292255/da7d87fa-1b9f-11e6-9aba-9b9b983f417a.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/pulimento/TagListView-ObjC/issues/2

authors: Ram-OAB pulimento

repo: pulimento/TagListView-ObjC


text:
UX Design Request

Design request for Pulumi AI work related to FAQ.

Closing as canceled.

repo: pulumi/docs


text:
Upgrade Sentry

Some of the errors being logged in Sentry look to be related to Sentry itself. We should upgrade the SDK as noted in the Sentry UI.

Closing as we currently fetch the latest Sentry SDK on page load.

authors: cnunciato zchase

repo: pulumi/pulumi-hugo


text:
Unable to create Warehouse: QUERY_ACCELERATION_MAX_SCALE_FACTOR error

What happened?
I am trying to create a warehouse using Pulumi Snowflake but I get an error message when I try:
warehouse = snowflake.Warehouse("warehouse",
        name=f"MY_WAREHOUSE",
        auto_resume = True,
        initially_suspended=True,
        auto_suspend=120,
        warehouse_size='X-SMALL')

The error message is the following:
error: 1 error occurred:
    	* error creating warehouse err = 001423 (22023): SQL compilation error:
    invalid property 'QUERY_ACCELERATION_MAX_SCALE_FACTOR'; feature 'Query Acceleration Service' not enabled

Steps to reproduce
Create a warehouse using Pulumi Snowflake on Snowflake Standard edition.
Expected Behavior
I expected the warehouse to be created. I also expected that Pulumi / TF would ignore the QUERY_ACCELERATION_MAX_SCALE_FACTOR parameter on Snowflake Standard edition.
Actual Behavior
Warehouse is not created, instead got an error message.
The error message is the following:
error: 1 error occurred:
    	* error creating warehouse err = 001423 (22023): SQL compilation error:
    invalid property 'QUERY_ACCELERATION_MAX_SCALE_FACTOR'; feature 'Query Acceleration Service' not enabled

Output of pulumi about
CLI
Version      3.51.0
Go Version   go1.19.4
Go Compiler  gc

Plugins
NAME       VERSION
python     unknown
random     4.8.2
snowflake  0.15.0
tls        4.6.1

Host
OS       darwin
Version  13.0
Arch     arm64

This project is written in python: executable='/Users/istvanfodor/Documents/Projects/Github/antares-idl-fix/deployment/snowflake/venv/bin/python3' version='3.11.0
'

Current Stack: starschema/antares-idl-snowflake/dev

TYPE                                         URN
pulumi:pulumi:Stack                          urn:pulumi:dev::antares-idl-snowflake::pulumi:pulumi:Stack::antares-idl-snowflake-dev
pulumi:providers:snowflake                   urn:pulumi:dev::antares-idl-snowflake::pulumi:providers:snowflake::default_0_15_0
snowflake:index/database:Database            urn:pulumi:dev::antares-idl-snowflake::snowflake:index/database:Database::database-user
snowflake:index/role:Role                    urn:pulumi:dev::antares-idl-snowflake::snowflake:index/role:Role::antares-writer-role
snowflake:index/role:Role                    urn:pulumi:dev::antares-idl-snowflake::snowflake:index/role:Role::antares-reader-role
pulumi:providers:tls                         urn:pulumi:dev::antares-idl-snowflake::pulumi:providers:tls::default_4_6_1
pulumi:providers:random                      urn:pulumi:dev::antares-idl-snowflake::pulumi:providers:random::default_4_8_2
tls:index/privateKey:PrivateKey              urn:pulumi:dev::antares-idl-snowflake::tls:index/privateKey:PrivateKey::tls-key
snowflake:index/schema:Schema                urn:pulumi:dev::antares-idl-snowflake::snowflake:index/schema:Schema::database-schema
random:index/randomPassword:RandomPassword   urn:pulumi:dev::antares-idl-snowflake::random:index/randomPassword:RandomPassword::password
snowflake:index/databaseGrant:DatabaseGrant  urn:pulumi:dev::antares-idl-snowflake::snowflake:index/databaseGrant:DatabaseGrant::db-grant
snowflake:index/schemaGrant:SchemaGrant      urn:pulumi:dev::antares-idl-snowflake::snowflake:index/schemaGrant:SchemaGrant::writer-schema-grant-USAGE
snowflake:index/schemaGrant:SchemaGrant      urn:pulumi:dev::antares-idl-snowflake::snowflake:index/schemaGrant:SchemaGrant::reader-schema-grant-USAGE
snowflake:index/schemaGrant:SchemaGrant      urn:pulumi:dev::antares-idl-snowflake::snowflake:index/schemaGrant:SchemaGrant::writer-schema-grant-CREATE_TABLE
snowflake:index/schemaGrant:SchemaGrant      urn:pulumi:dev::antares-idl-snowflake::snowflake:index/schemaGrant:SchemaGrant::writer-schema-grant-CREATE_PIPE
snowflake:index/tableGrant:TableGrant        urn:pulumi:dev::antares-idl-snowflake::snowflake:index/tableGrant:TableGrant::reader-table-grant
snowflake:index/schemaGrant:SchemaGrant      urn:pulumi:dev::antares-idl-snowflake::snowflake:index/schemaGrant:SchemaGrant::writer-schema-grant-CREATE_STAGE

Found no pending operations associated with dev

Backend
Name           pulumi.com
URL            https://app.pulumi.com/ifodor
User           ifodor
Organizations  ifodor, starschema

Dependencies:
NAME              VERSION
nob               0.8.2
pip               22.3.1
pulumi-random     4.8.2
pulumi-snowflake  0.15.0
pulumi-tls        4.6.1
setuptools        65.6.3
wheel             0.38.4

Pulumi locates its logs in /var/folders/0k/5cb46pm90dv17f7675zv7xlm0000gn/T/ by default

Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

This might be caused by this issue: https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/1343

Seeing this too after upgrading to 0.15 and making no changes to my Warehouse configuration.
error: 1 error occurred:
	* updating urn:pulumi:***************::wsai:SnowflakeResources$snowflake:index/warehouse:Warehouse::TRANSFORMING_WAREHOUSE: 1 error occurred:
	* error altering warehouse err = 001423 (22023): SQL compilation error:
invalid property 'QUERY_ACCELERATION_MAX_SCALE_FACTOR'; feature 'Query Acceleration Service' not enabled

Thank you for finding the upstream issue @istvan-fodor.
We'll pull in the resolution once it ships upstream!

Hey @istvan-fodor, our latest release contains a fix for this! Please try v0.16.0 out and let us know.

authors: AaronFriel guineveresaenger istvan-fodor mattotodd

repo: pulumi/pulumi-snowflake


text:
Expose GUID generation in scope

Expose method to generate GUIDs via scope. Something like scope.$id()

scope.$id() method will return a GUID

repo: pumlhorse/pumlhorse


text:
[UI] Closes #55: Grey out news if it's already read in news list

@timotiusnc
Please critique the color.
Also please advise the invalidation strategy for the shared preferences.

The color is appropriate, IMO.
As for the invalidation strategy, as we have discussed, we will take the cheap approach first: no invalidation; save all read/unread status. As one key is approximately 10 bytes in size, a million read/unread status will cost us "only" 10 MB.
So, LGTM.
Merged!

authors: timotiusnc wiseodd

repo: pumpkinz/pumpkin-reader

license: WTFPL


text:
url parameter checking

@Snugug
Adding in new url parameter checking. Co-exists in this PR with existing parameter checking because this is the foundation for a new route middleware. These new checks cannot supersede the existing checks because it would require a large amount of re-writing to incorporate the promises - which would all be throw-away code as this is all bound for a middleware function.

Resolves #331
DCO 1.1 Signed-off-by: Scott Nath <github@scottnath.com>

@Snugug function/file locations have changed as per our conversation.
Any other changes needed?

@Snugug gtg

repo: punchcard-cms/punchcard


text:
[Feature]: Ability to get the original page source code

Feature description
It is possible to get the page content which is a snapshot of the rendered DOM after executing all the scripts. In our use case we need to get the HTML source as returned by the server, then render the page and get the page content.
I wasn't able to find a way of getting the page source as returned by the server and we most likely have to send a separate request to obtain that.

That should be possible already:
import puppeteer from "puppeteer";

const browser = await puppeteer.launch();

const page = await browser.newPage();

const response = await page.goto('https://example.com');

const source = await response.buffer();

console.log(source.toString('utf-8'));

await browser.close();

@OrKoN, this is awesome, thank you! I tried to find out how to do this, but no luck.

authors: OrKoN heaven

repo: puppeteer/puppeteer


text:
(FACT-1448) Add acceptance test to ensure json handles bools

On macOS (at least), json output converts all boolean values to true.

I'm not sure how to actually fix this, nor am I sure how to run the acceptance tests without all of puppet inc's infrastructure, but I think this is a test that will prove the bug in any case.

CLA signed by all contributors.

https://github.com/puppetlabs/facter/pull/1374 looks to fix the underlying issue - yay! This is a reasonable addition to this test. Once we get the fix merged I can give this a run and merge. Thanks for the contribution!

Someone more clever than I am should also probably add lib/test/ruby/ruby_value.cc

Also, it would be worth checking that this test actually fails with the previous code and passes with the fix before relying on it.

:+1: this looks right eyeballing it. We're gonna go ahead and merge so it runs through our CI system, as that's the easiest way to verify test changes. We can fix it up if anything breaks. Thanks for taking the time to write this test! We really appreciate test/doc fixes when somebody finds a bug. It makes our lives so much easier :D

authors: branan ccaviness puppetcla whopper

repo: puppetlabs/facter


text:
(FACT-3204) Revert "(maint) Skip vlans test for Ubuntu2204"

This reverts commit fb4d13de157c7961e27d874f16affe390fc2707f.

The test passes now in our jenkins CI without any alteration when run in isolation from my laptop; I think it's worth re-enabling in ci to see if it still fails.


text:
(PUP-1085) Be more specific about expected exception

Previously, the pacman unit test "should raise a Puppet::Error when
trying to install a group and allow_virtual is false" assumed
allow_virtual was false, and tried to install a package, assuming the
provider would raise a Puppet::Error.
However, in Puppet 4, allow_virtual is true, so the test was not
failing early as expected, and instead would actually try to execute:
/usr/bin/pacman --noconfirm --needed --noprogressbar -Sy package'

Puppet raises Puppet::ExecutionFailure on systems without the pacman
executable. However, this went unnoticed on *nix, because the test was
only expecting Puppet::Error, which is the superclass for
Puppet::ExecutionFailure. This did not go unnoticed on Windows, because
puppet raises a different exception:
Errno::ENOENT: No such file or directory - CreateProcess

This commit modifies the unit test to expect specific messages. It also
sets state directly on the resource object, e.g.
resource[:allow_virtual] = false. The resource is defined in a
let, so a new instance is created for each example as needed. Also
note, since :allow_virtual specifies :boolean => true, puppet
automatically creates a predicate method of the same name, e.g.
allow_virtual?.

CLA signed by all contributors.

Thanks for fixing this up!

authors: joshcooper kylog puppetcla

repo: puppetlabs/puppet


text:
(PUP-4890) Add code_id to the catalog

Adds a code_id accessor to the catalog, updates the api docs, schema, and tests.
Updates the compiler terminus' find method to add the code_id from the request to the catalog it returns.

@kylog @cprice404 @lutter @senior I hear you want a code_id?

:+1:

authors: joshcooper kylog


text:
(PUP-6629) Respect the passed log level when log_exception is passed a custom message

The first time we tried to fix PUP-6629, we did not fix all the code
paths in log_exception - specifically, when a custom message was
passed (which insync_values? does), we still logged at error level.
This now handles that case, and also adds a test for insync_values? to
ensure it does not log at error level if insync? fails.

Ugh, looks like there are some places that expected log_exception to call err directly :( Once again the levels at which we choose to mock make me sad.
Those are easy enough to clear up, though.

CLA signed by all contributors.

I'm not really happy with the way I changed the mocks here - I had issues trying to add expects for Puppet::Util::Logging::log_exception, and sorta gave up in the interests of getting it working. I do want to spend more time trying to make those less dumb if I can.

running the ACL tests against this PR passes! I still want to figure out if I can make the test changes less terrible, but this should get us to where we want to be.

I have pushed up #5241 as an alternative implementation, that requires fewer code changes (and no spec changes).  I'd be more comfortable with just changing the offending caller than the behavior of the logging code, considering this is targeted at a bugfix z release that we're shooting to turn around quickly.
If we want to change logging behavior, I suggest we slate that for a future release.

authors: Iristyle branan puppetcla


text:
(PUP-2172) Don't use an http proxy for localhost

Previously if an http proxy was configured, then puppet would use it when trying
to connect to localhost. HTTPS connections to localhost would most likely fail
due to certname mismatch. However, HTTP connections would unnecessarily connect
via the proxy. In addition to being inefficient, many network environments don't
allow internal users to route through a proxy to reach other internal hosts. So
trying to retrieve a file from an internal file server could fail.

Blocked on PR #7661

This is a little tricky to test, but if you have a manifest like:
file { '/tmp/build.yaml':
  ensure => file,
  source => 'http://localhost:8888/build.yaml',
}

and you configure a local webserver to listen on 8888, then an agent run like will use the proxy to connect to the puppetserver, but not use the proxy to download the file content from localhost webserver:
$ bundle exec puppet agent -t --http_proxy_host localhost --http_proxy_port 8080
Info: Using configured environment 'production'
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Retrieving locales
Info: Caching catalog for ...
Info: Applying configuration version '1565225324'
Notice: /Stage[main]/Foo/File[/tmp/build.yaml]/ensure: defined content as '{mtime}2019-08-08 00:47:58 UTC' (corrective)
Notice: Applied catalog in 0.06 seconds

CLA signed by all contributors.

Do we know if no_proxy is space-character-insensitive?
The documentation and examples I've found never use spaces.

Yeah good question. it's strips spaces due to https://github.com/puppetlabs/puppet/blob/master/lib/puppet/util/http_proxy.rb#L49, which is covered in the existing tests: https://github.com/puppetlabs/puppet/blob/master/spec/unit/util/http_proxy_spec.rb#L133

Ahh. Sorry I missed that!

authors: joshcooper puppetcla tkishel


text:
(maint) Merge 6.x to main

commit 'a13074652077766c8dfd535d6f6765d6b6005a21':
(packaging) Updating manpage file for 6.x
(PUP-11321) Allow loading of safe classes from the persistence store
Revert "Merge pull request #8699 from Dorin-Pleava/PUP-10820"
(maint) ruby-openssl now sets store_context.error

@ciprianbadescu Could you rebase this on upstream/main to pick up the windows-2019 change?

Superseded by https://github.com/puppetlabs/puppet/pull/8832

authors: ciprianbadescu joshcooper mhashizume


text:
Test merge master

[facebones-ignore]

Test FAILed

authors: austb puppetlabs-jenkins

repo: puppetlabs/puppetdb


text:
MODULES-1476: Fix noisy facter if update-notifier is not installed

Moves file executable test block around the entire script so that we avoid errors if update-notifier is not installed.
https://tickets.puppetlabs.com/browse/MODULES-1476

I think the downside to reimplementing with the basic commands would be parsing to figure out if the updates are security updates or not.  While we could reimplement that, you would would have to account for different Debian distros and how they may have extended that list.

authors: WolverineFan teran-mckinney

repo: puppetlabs/puppetlabs-apt


text:
(FM-2564) Ensure rake task copies DSC resources to vendor folder

A bug in the rake task meant the cloned xDscResources did not copy over to the lib/puppet_x/dsc_resources folder. This fixes it

We don't use (bug) that I am aware of. Please adjust the commit message to (FM-2564) and the rest. Otherwise :+1:

authors: ferventcoder jpogran

repo: puppetlabs/puppetlabs-dsc


text:
Change to use webmock for tests

This enables tests to be run without needing a httpbin server running
locally.

Updated to remove the helper file from the PR now. That means that the tests are now dependent on cloning the task helper.
No worries about me being a guinea pig. Making a CONTRIBUTING.md will be a great addition! :+1:

Thank you!!

authors: barskern lucywyman


text:
Add shell.nix mirroring the CI configuration

The only difference is that purs-tidy 0.9.0 is installed instead of 0.9.2 due to easy-purescript-nix not providing the newer version yet.

The only difference is that purs-tidy 0.9.0 is installed instead of 0.9.2 due to easy-purescript-nix not providing the newer version yet.

If there isn't already one, why not submit a PR updating that to 0.9.2 first then?

I submitted easy-purescript-nix/pull/219.

Now we should be on par with the CI configuration.

authors: JordanMartinez turlando

repo: purescm/purescm


text:
User and Admin add

Describe the bug
1). Add the user sign-in for server-side code.
2). Add the admin sign-in and signup for server-side code.

A clear and concise description of what the bug is.

Possible solution

Describe the solution you thought of.

Screenshots

If applicable, add screenshots to help explain your problem.

@VishalRajai41 is it necessary to add admin?

Yes of course. Without admin how can you handle website?

How about adding a secret code input in the login and signup form to check whether the user is the admin?

authors: VishalRajai41 aniketlodh himanshujaidka


text:
Unable to get Socket id

I have written code like this. Can you please help me out that, why I can not get socket id?
`import { Pusher } from "@pusher/pusher-websocket-react-native";
import { PUSHER_KEY, PUSHER_CLUSTER } from "@env";
export const PusherConfig = async () => {
try {
const pusher = Pusher.getInstance();
await pusher.init({
apiKey: PUSHER_KEY,
cluster: PUSHER_CLUSTER
});
await pusher.connect();
const id = await pusher.getSocketId();
console.log("id", id);
return pusher;
} catch (err) {
return err;
}
};`

Hi, you've called both connect() and .getSocketId() next to eachother as asynchronous awaits, and your getSocketId() is completing faster before the connection is established, hence it will return a null.
If you put getSocketId() after the the connect() has finished, are you able to get it?

I will close this ticket due to lack of response.

authors: benjamin-tang-pusher subhrajit-akrabhi

repo: pusher/pusher-websocket-react-native


text:
gradle migration

migrated from ant to gradle.
moved all source from src/ to src/main/java and src/test/java
added 4 test classes which demonstrate how to run this from command line via gradle
updated README with compile and test instructions.
added logback support

May be checks failed, because the new structure is based on gradle

repo: pushkar/ABAGAIL


text:
Fix emission checker behavior in case of unexpected value

Before:
If value that was received by emission checker was incorrect -> it'll throw exception which could be handled by Observable's subscriber and it brakes internal checks of emission checker, it may or may not handle it as timeout error.
After:
Now I handle all exceptions manually and threat them as Emission Checker's problem, so Observable won't handle it for us.
And finally, it may help with our flaky tests on CI problem.

Bonus -> build time -= 2 minutes!

Hm shouldReceiveChangeAfterEachUpdate failed=(

Uh… I hope, we will see Genymotion for CI soon, *ing slow Android emulator.

=)
There are problems again.
Shell I merge it?

Passed!

Yeah)

Have you change something?

No, just relaunched build on Travis 3-4 times… I am thinking about Jenkins or TeamCity setup with powerful slaves hosted on AWS EC2 .

Or, we can switch to Robolectric, but as you know, I am not fan of it…

Yes, I know, but why?

Oh, with Robolectric we will open whole new world of dealing with Robolectric's bugs.

authors: artem-zinnatullin nikitin-da

repo: pushtorefresh/storio


text:
🛑 eClass is down

In 2eb3974, eClass (https://eclass.unmer.ac.id/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: eClass is back up in 3d4fb25 after 3 minutes.

repo: pusimgit/upptime


text:
🛑 SISTER is down

In 6877566, SISTER (https://sister.unmer.ac.id/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SISTER is back up in 2ca2610.


text:
Map: add way to provide initial size hints

Since the difference in performance caused by map growth is noticeble, it would be great if we could initialize the map with size hints to try to avoid growths on add.
The fact that we spread keys across buckets makes this non-trivial, but maybe it would be enough to assume normally distributed keys and just grow the buckets proportionally to the initial hint?
I've just glossed over the code, but could try a PR if there's no obvious blocker I missed and nobody else beats me to it!

Yes, it would be a good addition to both Map and MapOf.

just grow the buckets proportionally to the initial hint

Yes, I'd simply use nextPowOf2(size / entriesPerMapBucket) as the initial table size. That's the usual way to initialize the hash table based on the hint.

damn, that was fast! I intended to work on this tonight! 😅
Thanks! 🎉

Thanks for opening this issue.

I intended to work on this tonight!

If you see any issues or potential improvements around #86, please submit a PR. I won't be releasing the next version for a few days.

Thanks, will do! 👍🏻

authors: costela puzpuzpuz

repo: puzpuzpuz/xsync


text:
코드 개선

Busan_Senior_Center(BSC)의_시급한_개선을_위한_입지_분석_모델.ipynb에서 fig.ax = center(G_Jeonggwan) ->
fig,ax = center(G_Jeonggwan)로 고치면 오류수정이 가능한가요?

네 가능합니다  fig.ax => fig, ax

authors: Sagongmin pwjdgus


text:
docker node image version

It's better to pin a specific version and update it regularly rather than something generic like lts that will probably break when the next node LTS version will be released.
If validated I'll reflect that to all other docker files.

@Zeecka do you want me to open a PR for .dev and .test as well?

Fixed with https://github.com/pwndoc-ng/pwndoc-ng/pull/101

authors: Zeecka noraj

repo: pwndoc-ng/pwndoc-ng


text:
Refactoring steps-nowcast

I started with the steps nowcast because that seemed more straightforward to tackle first. Since both methods should look alike, they must both be changed. I can't figure out how to run the test locally, so I will have to wait and see what the pull request test gives me as feedback.
This is a first suggestion how to split up the code but look forward to suggestions and improvments

I have a question about the Codacy Static Code Analysis. It sais the method _check_inputs is to complex while this has barely been touched since the last version, I moved a few extra lines here because that just makes more sense. Should I split up that part of the code to remove the error or not?

hi @sidekock , first of all, thanks for taking up this heroic task :-)

can't figure out how to run the test locally

which part? to run the test locally you need to setup the test data correctly, see https://pysteps.readthedocs.io/en/stable/user_guide/example_data.html#example-data and https://pysteps.readthedocs.io/en/stable/developer_guide/test_pysteps.html

I have a question about the Codacy Static Code Analysis. It sais the method _check_inputs is to complex while this has barely been touched since the last version, I moved a few extra lines here because that just makes more sense. Should I split up that part of the code to remove the error or not?

I agree with you, _check_inputs seems well in line with the complexity of the other methods, and feels like breaking it down into smaller pieces would somewhat overcrow your class. I silenced this issue on codacy.

@sidekock, I will try to do so asap.

@sidekock Should this be merged?

I am sorry for only commenting now. I have been busy with some other stuff. I will go over the changes and leave some input where I have it. If you don't want to hold off merging, that's fine since Ruben already reviewed this. Maybe you can then take my input along in the next steps of the refactoring

@sidekock Should this be merged?

Let me use the feedback of Mats. I will contact you once it is ready to be merged.

@mats-knmi could you look through the changes to give some feedback? I will look at the 'private' variables next week if I have time. In general, I think this approach is much cleaner compared to what we had before. We see what is changed and what type it is (state or param) and what is loaded (config, state or param) for each method.
For the 'private' discussion, I have taken a small look at the StackOverflow page but do not yet see how I would use this at this moment. Most variables in the code are part of a dataclass so can I still make then private then?

Looks good, I added 2 small comments on the type hinting, but other than that I think this makes it a lot nicer indeed. Regarding the private discussion:
Every method that currently starts with an underscore could start with 2 underscores and then they could still be called from within the class, but not as easily from outside. (e.g. _nowcast_main could be renamed to __nowcast_main and then be called normally as self.__nowcast_main()).
Every variable that is defined in __init__ could be defined with 2 underscores and then still accessed normally within the class, but not as easily from outside. (e.g. self.__state = StepsNowcasterState() and then acccess it later like self.__state.precip_forecast)

Looks good, I added 2 small comments on the type hinting, but other than that I think this makes it a lot nicer indeed. Regarding the private discussion: Every method that currently starts with an underscore could start with 2 underscores and then they could still be called from within the class, but not as easily from outside. (e.g. _nowcast_main could be renamed to __nowcast_main and then be called normally as self.__nowcast_main()). Every variable that is defined in __init__ could be defined with 2 underscores and then still accessed normally within the class, but not as easily from outside. (e.g. self.__state = StepsNowcasterState() and then acccess it later like self.__state.precip_forecast)

Thanks for all the feedback @mats-knmi and @RubenImhoff. The new version of the code feels much more readable now! I made the last changes and will soon start on the real challenge: the blending code :)

@dnerini the Static code analysis is complaining again because I changed the name of the method "__check_inputs". Would it be possible to disable this once more? That way, all tests should pass now

@sidekock I resolved all my comments that are fully resolved now, I left 2 open, since those are not fully resolved yet.
I think the docstring comment is important to still address, but the one about passing precip and velocity and/or timesteps not at the __init__ but at the compute_forecast is more of a stylistic preference.

@mats-knmi if all the requested changes are present, would you please approve the changes? Then I know I can start working on the blending part after the merge

Yes they are approved. I wanted to approve them in github, but I can't find the button to do this. Not sure if I am just blind or if I am not authorized to approve stuff...

@dnerini I think this is ready to be merged with the master branch

Fantastic work! 🙌

authors: RubenImhoff dnerini mats-knmi sidekock

repo: pySTEPS/pysteps


text:
Portable Gasolene generator name error

Mod source
Factorio Mod Portal
Which mod are you having an issue with?

[ ] pyalienlife
[ ] pyalternativeenergy
[ ] pycoalprocessing
[ ] pyfusionenergy
[ ] pyhightech
[ ] pyindustry
[ ] pypetroleumhandling
[ ] pypostprocessing
[ ] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[X] Locale (names, descriptions, unknown keys)
[ ] Graphical
[ ] Crash
[ ] Progression
[ ] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
I believe that there is never a reason to spell that "Gasolene" instead of "Gasoline", although I could be wrong. Although maybe it should be called "Portable Liquid Fueled Generator" (US) or "Liquid Fuelled Generator" (UK) instead (or just "Portable Liquid Fuel Generator")
Steps to reproduce
No response
Additional context
No response
Log file
No response

Fixed. Thank you.

authors: KhonaB notnotmelon

repo: pyanodon/pybugreports

license: Unlicense


text:
Add latexify notebook

Description
Add latexify notebook in examples
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[ ] No style issues: $ flake8
[ ] All tests pass: $ python run-tests.py --unit
[ ] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

All done, it is ready to be reviewed now

repo: pybamm-team/PyBaMM


text:
Path File Raises ModuleNotFound Error When Imported

When I import something form the path file, it gives the error that a module named "error" is not found
Steps to reproduce the behavior:

from py_everything import path
Run the code
Error occurs
The "error" module is not found as when the path file is imported the location of the call changes and hence no file named "error" is found

Device:

OS: Windows
Python Version: 3.9.5
Package Version: 2.1.0

Fixed in latest commit to bug_fixes branch.


text:
Workaround for setuptools install_requires bug with pyasn1

When installing service_identity with setup.py install the order of
the install_requires directive causes setuptools to fail to install
pyasn1 with the following error message:
error: The 'pyasn1' distribution was not found and is required by service-identity

The workaround is to switch the order of pyasn1 and pyasn1-modules in
the install_requires directive.
Setuptools issue: https://github.com/pypa/setuptools/issues/498

😮
Could you add a comment to the section that stresses, that the order matters? Thanks!

Done

thanks!

authors: cas-- hynek


text:
Revise and improve docs/developing-locally.rst

The "Getting Up and Running Locally" document needs revision from someone with great attention to detail and writing skills. Basically it needs to be checked for usability and modified as appropriate.
Perhaps this could be a way for a Cookiecutter Django newbie to get comfortable with the project while contributing.

I will get started on this.

Being worked on @dot2dotseurat and Emily Cain.

I am writing a draft of a new tutorial at http://rst.ninjs.org/?n=60d2e7683bf517906ba3b447384c8d14&theme=basic -- please comment here instead of editing the document if you have questions.

Also, I will condense the "setup" section of the new tutorial into a "quickstart" section for our readme page, and include a link to the tutorial for anyone who has additional questions.

authors: EMCain audreyr purplediane pydanny

repo: pydanny/cookiecutter-django


text:
Can we use Tabler with FastUI?

Tabler is a free and open source web application UI kit based on Bootstrap 5, with hundreds responsive components and multiple layouts.

https://tabler.io/admin-template

I believe you can but I think you must build your own implementation through NPM build.

authors: hasansezertasan smagafurov

repo: pydantic/FastUI


text:
Remove unused bool error

It seems the Bool error was replaced with BoolParsing and BoolType, but never removed. This PR removes it

please review

repo: pydantic/pydantic-core


text:
Settings are loaded in a docker but not in Kubernetes

I have a FastAPI implementation with Pydrantic settings to load Environmental Variables.
I have made a docker image and run it in docker and all worked fine, but when I load the same image in a k8s cluster, I get the following error:
 Traceback (most recent call last):
   File "/tmp/api/./src/main.py", line 3, in <module>
     from core.dependencies.settings import settings
   File "/tmp/api/src/core/dependencies/settings.py", line 70, in <module>
     settings = Settings()
                ^^^^^^^^^^
   File "/usr/local/lib/python3.11/site-packages/pydantic_settings/main.py", line 72, in __init__
     **__pydantic_self__._settings_build_values(
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   File "/usr/local/lib/python3.11/site-packages/pydantic_settings/main.py", line 160, in _settings_build_values
     return deep_update(*reversed([source() for source in sources]))
                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   File "/usr/local/lib/python3.11/site-packages/pydantic_settings/main.py", line 160, in <listcomp>
     return deep_update(*reversed([source() for source in sources]))
                                   ^^^^^^^^
   File "/usr/local/lib/python3.11/site-packages/pydantic_settings/sources.py", line 252, in __call__
     field_value = self.prepare_field_value(field_name, field, field_value, value_is_complex)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   File "/usr/local/lib/python3.11/site-packages/pydantic_settings/sources.py", line 426, in prepare_field_value
     env_val_built = self.explode_env_vars(field_name, field, self.env_vars)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   File "/usr/local/lib/python3.11/site-packages/pydantic_settings/sources.py", line 527, in explode_env_vars
     env_var = env_var.setdefault(key, {})
               ^^^^^^^^^^^^^^^^^^
 AttributeError: 'str' object has no attribute 'setdefault'

I searched, but I haven't really found any solution on this.
I use python 3.11
code:
from dotenv import find_dotenv
from pydantic import BaseModel
from pydantic_settings import BaseSettings, SettingsConfigDict

class PostgreSQL(BaseModel):
    host: str | None = None
    port: int = 5432
    user: str | None = None
    password: str | None = None
    database: str | None = None

class JWT(BaseModel):
    key: str = ""
    algorithm: str = "HS256"

class Authorization(BaseModel):
    basic: bool = False
    bearer: bool = False
    jwt: JWT = JWT()

class Api(BaseModel):
    host: str = "127.0.0.1"
    port: int = 3000
    sessionTimeout: int = 1800
    authorization: Authorization = Authorization()

class Settings(BaseSettings):
    model_config = SettingsConfigDict(
        env_nested_delimiter="_",
        env_file=find_dotenv(),
        env_file_encoding="utf-8",
        secrets_dir="/run/secrets",
        extra="ignore",
    )

    api: Api = Api()
    postgresql: PostgreSQL = PostgreSQL()

settings = Settings()

Thanks @vkaramanis for reporting this.
Please let us know your current pydantic-settings version. ( I think it is the latest version 2.1.0)
Could you provide your environment variables values? It would be great to fetch them by connecting to the container and running env command there.

@hramezani thanks for the reply, yes I am using 2.1.0
After your comment, I found the problem. k8s was rewriting some of my env vars because they were present in other services on the same namespace.
For example: POSTGRESQL_PORT=tcp://10.152.183.174:5432 (I cannot share all my vars)
Although I checked some of my vars I didn't check all of them, and it was slipped, sorry for that (stupid).
So I added a prefix to all of them from SettingsConfigDict and now it runs smoothly.
However, as a side note, I still do not understand why it raises this error.

Thanks @vkaramanis for updating.
Yeah, that's weird for me. that's why I asked you to provide your env variables.
I need a way to reproduce the problem. So, I don't know what is the reason.
I close the issue for now. but feel free to reopen it or report a new issue if you face the problem again or find a way to reproduce the problem.

authors: hramezani vkaramanis

repo: pydantic/pydantic-settings


text:
TST: Can't Seem to Run Individual Test Suites in pandas/io

$ python /home/user/Repositories/pandas/pandas/io/tests/test_data.py
Traceback (most recent call last):
  File "/home/user/Repositories/pandas/pandas/io/tests/test_data.py", line 4, in <module>
    from pandas import compat
  File "/home/user/miniconda2/envs/pandas-dev/lib/python2.7/site-packages/pandas/__init__.py", line 39, in <module>
    from pandas.core.api import *
  File "/home/user/miniconda2/envs/pandas-dev/lib/python2.7/site-packages/pandas/core/api.py", line 10, in <module>
    from pandas.core.groupby import Grouper
  File "/home/user/miniconda2/envs/pandas-dev/lib/python2.7/site-packages/pandas/core/groupby.py", line 18, in <module>
    from pandas.core.frame import DataFrame
  File "/home/user/miniconda2/envs/pandas-dev/lib/python2.7/site-packages/pandas/core/frame.py", line 39, in <module>
    from pandas.core.series import Series
  File "/home/user/miniconda2/envs/pandas-dev/lib/python2.7/site-packages/pandas/core/series.py", line 2929, in <module>
    import pandas.tools.plotting as _gfx  # noqa
  File "/home/user/miniconda2/envs/pandas-dev/lib/python2.7/site-packages/pandas/tools/plotting.py", line 25, in <module>
    import pandas.tseries.converter as conv
  File "/home/user/miniconda2/envs/pandas-dev/lib/python2.7/site-packages/pandas/tseries/converter.py", line 7, in <module>
    import matplotlib.units as units
  File "/home/user/miniconda2/envs/pandas-dev/lib/python2.7/site-packages/matplotlib/__init__.py", line 135, in <module>
    __version__ = str(get_versions()['version'])
  File "/home/user/miniconda2/envs/pandas-dev/lib/python2.7/site-packages/matplotlib/_version.py", line 21, in get_versions
    return json.loads(version_json)
AttributeError: 'module' object has no attribute 'loads'

INSTALLED VERSIONS
pandas: 0.18.0+174.g65ed3af
nose: 1.3.7
pip: 8.1.1
setuptools: 20.0
Cython: 0.23.4
numpy: 1.11.0
scipy: 0.17.0
statsmodels: 0.6.1
xarray: 0.7.2
IPython: None
sphinx: None
patsy: 0.4.1
dateutil: 2.5.3
pytz: 2016.4
blosc: None
bottleneck: 1.0.0
tables: 3.2.2
numexpr: 2.5
matplotlib: 1.5.1
openpyxl: 2.3.3
xlrd: 0.9.4
xlwt: 1.0.0
xlsxwriter: 0.8.4
lxml: 3.6.0
bs4: 4.4.1
html5lib: 0.9999999
httplib2: None
apiclient: None
sqlalchemy: 1.0.12
pymysql: None
psycopg2: None
jinja2: 2.8
boto: 2.39.0
pandas_datareader: None
Somewhere in the last 32 commits or so to the master branch this broke because my branch for refactoring test_parsers was that many commits behind when it last worked.  I also confirmed the breakage on master too.

we don't support that, though I guess it would be nice (and the reason things have __main__ in them). But its not documented / tested anywhere.
[jreback-~/pandas] nosetests  pandas/io/tests/test_data.py
.S...SS...S...^C
----------------------------------------------------------------------
Ran 15 tests in 18.535s

OK (SKIP=4)
[jreback-~/pandas]

The thing was I was able to run it before just fine.  Also, you can run other test suites (outside of pandas/io) just fine using the command above.

Hmmm...well at least nosetests works, so I'll close this then.

Hmmm...well at least nosetests works, so I'll close this then.

this happens on other files as well
not really sure why

Yeah...well it's not documented as being supported (I didn't know that until you mentioned it), so technically not an issue, and at least I still have means of running individual test suites.

authors: gfyoung jreback

repo: pydata/pandas


text:
Inconsistent Handling of na_values and converters in read_csv

On master (commit 40b4bb4):
>>> data = """A
1
CAT
3"""
>>> f = lambda x: x
>>> read_csv(StringIO(data), na_values='CAT', converters={'A': f}, engine='c')
     A
0    1
1  CAT
2    3
>>> read_csv(StringIO(data), na_values='CAT', converters={'A': f}, engine='python')
     A
0    1
1  NaN
2    3

I expect both to give the same output, though I believe the Python output is more correct because it respects na_values unlike the C engine.   I thought the simple fix would be to remove the continue statement here, but that causes test failures, so probably a more involved refactoring might be needed to align the order of converter application, NaN value conversion, and dtype conversion.
IMO this should be added to #12686, as this is a difference in behaviour between the two engines.
xref #5232

I have to say that decision to continue after applying a converter in the C-engine is really bugged.  It causes a lot of other post-processing to be skipped.  Another example (adapted from the parser tests):
>>> """A B
-999 1.200
2 -999.000
3 4.500"""
>>> f = lambda x: x
>>> read_csv(StringIO(data), sep=' ', header=0, na_values=[-999.0, -999], converters={'B':f})
     A         B
0  NaN     1.200
1  2.0  -999.000
2  3.0     4.500

The Python engine fails for a different reason, which is why I don't show any example for it.

xref #15144 (closed):
>>> data = 'A,B,C\n10,-1,10\n10,10,10'
>>> read_fwf(StringIO(data), converters={'B': lambda x: x / 10.}, na_values={'B': [-1]})
 A    B  C
10 -0.1 10
10    1 10

Expected:
 A    B  C
10  NaN 10
10    1 10

The current behavior is due to the fact that converters are applied BEFORE na_values are, but consensus appears to be that we should apply na_values first.

This is an insidious problem with the C engine! In 0.24.2, I observe implicit conversions also cause the parser to ignore the na_values parameter:
import io, pandas as pd
data = """\
Time,ConcRT(ug/m3),ConcHR(ug/m3),Flow(lpm),WS(m/s),WD(Deg),AT(C),RH(%),BP(mmHg),FT(C),FRH(%),BV(V),PM,Status
2018-06-27 21:00:00,+000012,+000007,+16.7,00.0,000,+022.6,025,704,+030.1,012,12.5,0,00000
2018-06-27 22:00:00,+000003,+000009,+16.7,00.0,000,+021.2,033,705,+028.6,015,12.5,0,00000
2018-06-27 23:00:00,+099999,+099999,+00.0,00.0,000,+019.6,038,705,+024.0,015,12.5,0,00001

"""

Note these are numeric values, but their representation probably requires an implicit converter be called.
The C engine will not locate NaNs unless the verbatim per-character value is presented to na_values:
pd.read_csv(io.StringIO(data), na_values=['99999'])

| Time | ConcRT(ug/m3) | ConcHR(ug/m3) | Flow(lpm) | WS(m/s) | WD(Deg) | AT(C) | RH(%) | BP(mmHg) | FT(C) | FRH(%) | BV(V) | PM | Status
-- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | --
2018-06-27 21:00:00 | 12 | 7 | 16.7 | 0.0 | 0 | 22.6 | 25 | 704 | 30.1 | 12 | 12.5 | 0 | 0
2018-06-27 22:00:00 | 3 | 9 | 16.7 | 0.0 | 0 | 21.2 | 33 | 705 | 28.6 | 15 | 12.5 | 0 | 0
2018-06-27 23:00:00 | 99999 | 99999 | 0.0 | 0.0 | 0 | 19.6 | 38 | 705 | 24.0 | 15 | 12.5 | 0 | 1
pd.read_csv(io.StringIO(data), na_values=[99999])

Time
ConcRT(ug/m3)
ConcHR(ug/m3)
Flow(lpm)
WS(m/s)
WD(Deg)
AT(C)
RH(%)
BP(mmHg)
FT(C)
FRH(%)
BV(V)
PM
Status

2018-06-27 21:00:00
12
7
16.7
0.0
0
22.6
25
704
30.1
12
12.5
0
0

2018-06-27 22:00:00
3
9
16.7
0.0
0
21.2
33
705
28.6
15
12.5
0
0

2018-06-27 23:00:00
99999
99999
0.0
0.0
0
19.6
38
705
24.0
15
12.5
0
1

pd.read_csv(io.StringIO(data), na_values=['+099999'])

Time
ConcRT(ug/m3)
ConcHR(ug/m3)
Flow(lpm)
WS(m/s)
WD(Deg)
AT(C)
RH(%)
BP(mmHg)
FT(C)
FRH(%)
BV(V)
PM
Status

2018-06-27 21:00:00
12.0
7.0
16.7
0.0
0
22.6
25
704
30.1
12
12.5
0
0

2018-06-27 22:00:00
3.0
9.0
16.7
0.0
0
21.2
33
705
28.6
15
12.5
0
0

2018-06-27 23:00:00
NaN
NaN
0.0
0.0
0
19.6
38
705
24.0
15
12.5
0
1

This situation highlights the shortcomings of picking a simple 'before' or 'after' for converters to be applied. Consensus so far is to apply na_values before converters but I suspect certain converters should always be applied first.

Note the Python engine handles the situation identically (and "correctly") in all scenarios:
pd.read_csv(io.StringIO(data), na_values=['99999'], engine='python')

Time
ConcRT(ug/m3)
ConcHR(ug/m3)
Flow(lpm)
WS(m/s)
WD(Deg)
AT(C)
RH(%)
BP(mmHg)
FT(C)
FRH(%)
BV(V)
PM
Status

2018-06-27 21:00:00
12.0
7.0
16.7
0.0
0
22.6
25
704
30.1
12
12.5
0
0

2018-06-27 22:00:00
3.0
9.0
16.7
0.0
0
21.2
33
705
28.6
15
12.5
0
0

2018-06-27 23:00:00
NaN
NaN
0.0
0.0
0
19.6
38
705
24.0
15
12.5
0
1

pd.read_csv(io.StringIO(data), na_values=[99999], engine='python')

Time
ConcRT(ug/m3)
ConcHR(ug/m3)
Flow(lpm)
WS(m/s)
WD(Deg)
AT(C)
RH(%)
BP(mmHg)
FT(C)
FRH(%)
BV(V)
PM
Status

2018-06-27 21:00:00
12.0
7.0
16.7
0.0
0
22.6
25
704
30.1
12
12.5
0
0

2018-06-27 22:00:00
3.0
9.0
16.7
0.0
0
21.2
33
705
28.6
15
12.5
0
0

2018-06-27 23:00:00
NaN
NaN
0.0
0.0
0
19.6
38
705
24.0
15
12.5
0
1

pd.read_csv(io.StringIO(data), na_values=['+099999'], engine='python')

Time
ConcRT(ug/m3)
ConcHR(ug/m3)
Flow(lpm)
WS(m/s)
WD(Deg)
AT(C)
RH(%)
BP(mmHg)
FT(C)
FRH(%)
BV(V)
PM
Status

2018-06-27 21:00:00
12.0
7.0
16.7
0.0
0
22.6
25
704
30.1
12
12.5
0
0

2018-06-27 22:00:00
3.0
9.0
16.7
0.0
0
21.2
33
705
28.6
15
12.5
0
0

2018-06-27 23:00:00
NaN
NaN
0.0
0.0
0
19.6
38
705
24.0
15
12.5
0
1

@patricktokeeffe I agree, this bug is insidious, very difficult to find and highly confusing. Here is a snippet to trace it down. It took me a couple of hours.
`import requests
import zipfile
from io import BytesIO
import pandas as pd
from datetime import datetime
zip_url = "https://opendata.dwd.de/climate_environment/CDC/observations_germany/climate/10_minutes/solar/historical/10minutenwerte_SOLAR_00704_20020801_20091231_hist.zip"
response = requests.get(zip_url)
response.raise_for_status()
Open the .zip file in memory
with zipfile.ZipFile(BytesIO(response.content)) as z:
# Extract the .txt file containing the data
txt_file_name = z.namelist()[0]
with z.open(txt_file_name) as file:
df = pd.read_csv(file, sep=";", na_values="-999", engine="c")
print(df.loc[df['MESS_DATUM'] == 200709010900]['GS_10'])
print(df.loc[df['MESS_DATUM'] == 200709010910]['GS_10'])`
This is my output:
262143   -999.0 Name: GS_10, dtype: float64 262144   NaN Name: GS_10, dtype: float64
Some values of my data are correctly interpreted as NAN and some are not. Apperently at the two dates I selected above, is where the switch actually happens. Before "200709010900" every value of -999 is not interpreted as NAN. After "200709010910" all -999 values are correctly interpreted as NAN. I checked the csv-file manually and couldn't find any inconsistencies. However, once I change engine="python", it all works as expected. Am I missing something or is this indeed a weird bug?

authors: fynnwz gfyoung patricktokeeffe


text:
0.15.2 to_datetime broken

It looks like the support for the new 'exact' keyword has borked to_datetime() for format strings other than '%Y%m%d' as it is passed to array_strptime which doesn't seem to support it:
ts = 201502031753

pd.to_datetime(ts, format='%Y%m%d%H%M')
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-2-685333561e5e> in <module>()
----> 1 pd.to_datetime(ts, format='%Y%m%d%H%M')

C:\app\python\lib\site-packages\pandas\tseries\tools.py in to_datetime(arg, errors, dayfirst, utc, box, format, exact, coerce, unit, infer_datetime_format)
    316         return _convert_listlike(arg, box, format)
    317
--> 318     return _convert_listlike(np.array([ arg ]), box, format)[0]
    319
    320 class DateParseError(ValueError):

C:\app\python\lib\site-packages\pandas\tseries\tools.py in _convert_listlike(arg, box, format)
    278                     try:
    279                         result = tslib.array_strptime(
--> 280                             arg, format, exact=exact, coerce=coerce
    281                         )
    282                     except (tslib.OutOfBoundsDatetime):

C:\app\python\lib\site-packages\pandas\tslib.pyd in pandas.tslib.array_strptime (pandas\tslib.c:35070)()

TypeError: array_strptime() got an unexpected keyword argument 'exact'

Are you sure you are running 0.15.2? But in any case it is a strange error message (that you as a user never should see).
For me it works as expected:
In [8]: ts = 201502031753

In [9]: pd.to_datetime(ts, format='%Y%m%d%H%M')
Out[9]: Timestamp('2015-02-03 17:53:00')

In [10]: pd.__version__
Out[10]: '0.15.2'

it looks like you don't have the correct compiled code - pls clarify how you installed

Installed via Continuum Anaconda python 3.4 on Windows 64bit.  Not the first time I've had problems with their python3 support.  Version is definitely 0.15.2.  I will open an issue with them.

FWIW, I have the same setup and it parses correctly for me. I did have an issue when I upgraded to .15.1 where I had to clear pandas from the cache in the Pkgs directory under the Conda home, and then reinstall.

I have now established it was a problem with  'conda update' which silently fails to copy the pyd files if they are locked by a running process!  Thanks for the feedback.
-------- Original message --------
From: Tom Augspurger
Date:04/02/2015 13:07 (GMT+00:00)
To: pydata/pandas
Cc: Bruce MacDonald
Subject: Re: [pandas] 0.15.2 to_datetime broken (#9405)
FWIW, I have the same setup and it parses correctly for me. I did have an issue when I upgraded to .15.1 where I had to clear pandas from the cache in the Pkgs directory under the Conda home, and then reinstall.

Reply to this email directly or view it on GitHubhttps://github.com/pydata/pandas/issues/9405#issuecomment-72851120.

Ah, I guess that's why the display the warning about installing packages with other python instances open. I should probably stop ignoring it :)

specify -f on update/install to force the update

authors: TomAugspurger bjmacdonald jorisvandenbossche jreback


text:
CleanWebpackPlugin is never used

I'm currently working on the webpack.config.js and it seems CleanWebpackPlugin is never called. Looking at the documentation it should be added in the Plugins list.
Should it come back, should it not ?
https://github.com/pydata/pydata-sphinx-theme/blob/5133e4a27541091bddfaaf877c1859af2d3b9a7d/webpack.config.js#L13

I have no idea. @choldgraf ?

No idea either. I suggest we:

Check whether furo also uses this
if not, just remove it's use here 🤷

authors: 12rambau choldgraf drammock

repo: pydata/pydata-sphinx-theme


text:
Wrong pre block width in field lists

Discussed in https://github.com/pydata/pydata-sphinx-theme/discussions/1230
When a pre block that contain long lines is placed in a field list, it crush the field labels and expand to the infinite on the right instead on creating an horizontal scroll bar
Example here: https://bookshelf.docs.gunivers.net/en/latest/modules/math.html#basis-rotation-3d

Originally posted by Leirof March  3, 2023
Hi!
I just installed the Pydata theme (and it's awsome !), but I have an issue with field list that are totally crushed as you can see here: https://bookshelf.docs.gunivers.net/en/latest/modules/math.html#rounded-division
("Inputs:", "Outputs" and "Examples:" are not supposed to be vertical 😅)
Do I need to use a custom CSS rule to fix that or is there a simpler way? If I have to user CSS, how can I put a variable size depending of the larget field label?

Thanks @Leirof think we have two components issues going on here:

The x scrollbar not appearing in the code blocks
The labels (inside dd and dt elements - this seems to be a problem with the word-break property)

I will have a quick look at these issues.

we release new versions since you opened this PR and when clicking on the examples I don't see any issue. My guess is that the documentation has been rebuild since then. Do you still face the problem somewhere in your doc ?

We haven't gotten a response to our questions in our comment https://github.com/pydata/pydata-sphinx-theme/issues/1242#issuecomment-1557183571. With only the current information on the issue, we don't have enough information to take action. I will close this but don't hesitate to reach out if you have or find the answers we need. We'll be happy to reopen the issue.

authors: 12rambau Leirof trallard