stitch: Provide a useful error when Node isn't installed

To compile Quilt specs, we shell out to Node. Before, if Node wasn't
installed, we displayed only a blank error message. We now check that
Node is installed before shelling out to it, and return a more helpful
error if it is not installed.

Huh, it's failing to find Node on travis for some reason. Investigating.

Ok since this is failing anyway -- can you make this slightly more helpful? like "Failed to find Node; ensure that Node.js is installed"? (helpful to have more of a suggestion for how the user should fix it)

@kayousterhout I made a couple changes, could you just review it again?

No.. This code is kind of tough to unit test because it's basically just shelling out to Node.

I'm just wondering if we should have a super simple test that reads a very minimal stitch and makes sure that there isn't an error? Since otherwise the existing test isn't that useful, since that test would pass even if the code always exited with an error.

I see your point, but I'd like the unit tests to not depend on Node. What do you think of mocking out the exec call to Node, and replacing it with a function that always returned the same JSON string during unit tests?

That makes sense re: not wanting to depend on Node.  Mocking out the call seems fine to me, or also fine to just skip this if you'd prefer.

It's nice to have more coverage:  I'll add it to my todo list.


chore: sentry improvements

Current dependencies on/for this PR:

main

PR #2649

PR #2656   👈

This comment was auto-generated by Graphite.


Switch Networks  Your wallet does not support switching networks from Rainbowkit Demo. Try switching networks from within your wallet instead.

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
^1.0.10
wagmi Version
^1.2.0
Current Behavior
When I open nextjs localhost app, or the live version app and I click on switch network I receive this:
Your wallet does not support switching networks from Rainbowkit Demo. Try switching networks from within your wallet instead.
BUT, if I refresh the website all the networks appear. Just on the first visit, and for some reason it doesn't detect the networks.
Note that I testned on both my app, and existing demo applicaiton from official repo and I get the same error
https://github.com/rainbow-me/rainbowkit/blob/main/examples/with-next/pages/_app.tsx
Expected Behavior
No response
Steps To Reproduce
No response
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

@Morgosh Which wallet(s) are you testing this flow with? Will attempt to reproduce.

@Morgosh Bumping this; can investigate with some more information about your test path and wallet.

@Morgosh Are you still seeing this issue ? Can't reproduce this on my end. It might be the same issue mentioned here: https://github.com/wevm/wagmi/discussions/343
Let me know if you can also share a small reproducible example

@KosmosKey
I'm having the same problem.
It can be reproduced on the demo dapp (https://codesandbox.io/p/devbox/with-create-react-app-qqp2xl?file=%2Fsrc%2FApp.tsx) in the following order.

open the demo dapp
connect wallet
in the wallet, change the chain to one that is not supported by the dapp (e.g. linea)
open a new tab (several times)
(wallet will be connected automatically)
click on wrong network button
You will see Your wallet does not support switching networks from this app. Try switching networks from within your wallet instead message

https://github.com/rainbow-me/rainbowkit/assets/24788756/7998e081-15ff-4c8e-aca3-16e97548e77a

@ene5135 That's super useful. Thank you for the reproducible example will have look into this and report back once this is fixed 👍

@ene5135 This will be fixed in RainbowKit v2. Issue seems to be that wagmi won't return switchChain if you haven't entered the app with the correct one or at least one of the chains which your dApp supports.
All this will be fixed in this PR: https://github.com/rainbow-me/rainbowkit/pull/1709


Initial import of ReLAPACK

Add Peise's recursive implementation of LAPACK functions from https://github.com/HPAC/ReLAPACK for issue #788 , enabled by default in Makefile.rule (mostly for the automated checks now) . Cmake integration still needs to be done.

It looks like travis-ci failed because of multiple definitions. e.g.
multiple definition of slauum_'`

Yes, but at the moment I do not see where they are coming from - on Linux both compiling natively and cross-compiling for Windows with mxe the relevant netlib definitions get replaced by their relapack wrappers as they should. Is there any way to access the travis build tree after a failure to see  where things ended up ?

Did not notice that it is mxe x86_64-w64 that is giving the trouble, my local test was i686-w64 only. Seems the behaviour of ar is different in the 64bit version (though the only hint is a message about
option "-u" being ignored that is only generated by x86_64-w64-mingw32)

Got it now - the optimized LAPACK functions (such as slauum) from interface/ are expressly built with varying suffixes on the object file (.o,.obj) depending on the target operating system, while the pure netlib parts are compiled to .o files throughout. This needs to be reflected in the relapack Makefile for the substitution by ar -r to work. And I had failed to notice that my local cross-compiles had errored out before even building the shared library as they did not skip the "make test" step (which would have windows binaries incompatible with the Linux host) for some reason. Not sure yet if that is a separate issue or just an oversight on my part.
Now that all tests are passing, the remaining question is whether ReLAPACK should be enabled by default in Makefile.rule (as I needed it to be for the travis/appveyor runs)

@martin-frbg Great work!

@martin-frbg I suggest ReLAPACK is disable by default for 0.2.20 version.

Ah, sorry, probably not. Good catch. I was working on cpuset support at around the same time so it seems some unwanted cross-pollination may have happened. :(


A way to initialize matrix values based on a callable

#43 actually depends on this because we need a way to generate a non-singular matrix for inverse benchmark.

Generalized this to allow generations of values from a callable, and then a random generator function can be implemented as a callable.

Generalized this to allow generations of values from a callable, and then a random generator function can be implemented as a callable.


chore: Escape markdown for dynamic values

Related Issues
Fixes https://github.com/descope/etc/issues/8016
Related PRs

branch
PR

service a PR
Link to PR

service b PR
Link to PR

Description
A few sentences describing the overall goals of the pull request's commits.
Must

[ ] Tests
[ ] Documentation (if applicable)

@nirgur do you know how can I resolve the license check issue?

@tomerlichtash Please make sure that the new package is being published

@nirgur

Make sure that the new package is being published - as simple as npm publish it manually for the first time?
Do you think we should add "private": "true" to package.json?


MB-15408: Text updates for backup address fields

Jira ticket for this change
Summary
This pull request updates the header and helper text for the Backup Address form fields, in both the customer onboarding flow, and the edit customer profile page; specifically, it removes the word "mailing" from both headers and standardizes the helper text.
Setup to Run Your Code

💻 You will need to use two separate terminals to test this locally.
Terminal 1
Start the UI locally.
make client_run

Terminal 2
Start the Go server locally.
make server_run

Additional steps

Log in locally to the client application as a new customer user.
Proceed through the onboarding flow normally until you get to the onboarding step that formerly contained the header text "Backup mailing address."
Verify that the header text has changed (removing the word "mailing"), and that help text exists between the header and the form fields.
Finish the onboarding flow and progress to the customer's home page.
Click the "Edit" button on the right side of the first progression step.
Click "Edit" on the customer details section.
Scroll down and verify that the header text for the Backup address section also no longer includes the word "mailing," and that the help text between it and the form fields matches the help text in step 3.

Verification Steps for Author
These are to be checked by the author.

[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Screenshots

Onboarding flow, desktop width
Onboarding flow, mobile width
Edit profile, desktop width
Edit profile, mobile width

Messages

:book:
:link: MB-15408

  Generated by :no_entry_sign: dangerJS against 4712c79f19b073d39a4d0dab88a835a47a4ed5b0

I found a few places that still reference "backup mailing address" that I think likely should be updated

I don't think these were explicitly in the A/C, but I should fix them. (Will rename the component while I'm at it.)


Avoid filesystem limits using a trie

Implement storage trie based on idea from @pcarranza in discussion for #18.
All tests are passing with no additional changes... should I write a test that fails if the storage isn't using a trie though?

Let me get the weird cases we will find down the line to add tests for those and see what happens there.

I can already think of a case that will fail... - seems fine on my machine, but I bet . would do bad things.
However, a simple regex gsub I think would fix it... maybe gsub anything not in a set of acceptable characters? Like:
trie_parents = @name[0...3].gsub(/[^0-9a-zA-Z]/, "_").split("")

hmmm, now that I think of it, the . probably wouldn't really pose a problem... it would essentially get ignored in the File.join, but it wouldn't be added in a trie manner, so I think gsubbing everything outside a certain range might still be the best option

This is the whole data sample of what we can find with current specs:
["_", "-", "0", "1", "2", "3", "4", "5", "6", "7", "8", "9", "a", "b", "c", "d", "e", "f", "g", "h", "i", "j", "k", "l", "m", "n", "o", "p", "q", "r", "s", "t", "u", "v", "w", "x", "y", "z", "."]

So, yeah, maybe replace the . with something else that is not so full of meaning at the filesystem level.


Subtitles are not displaying (ERR: impossible to display subtitles.)

Description
stpp subtitles embedded in dash manifest are not displayed. This is reproducible in sample video player.
Steps to reproduce
Play URL https://livesim.dashif.org/dash/vod/testpic_2s/multi_subs.mpd using videojs-contrib-dash
Results
Expected
Subtitles should display
Actual
Subtitles are not displaying
Error output
[257263][TextTracks] impossible to display subtitles.
Additional Information
Same URL plays fine in reference dash player.
versions
videojs
7.14.3
browsers
Tested on Chrome, Edge (Chromium) and Firefox and issue exists in all
OSes
Tested on Windows 10
plugins
No

Can confirm it's an issue. Not really sure what's up.

I'm seeing the same error with our stpp captions. Note: the stream / captions display fine in the 4.0.1 DASH reference player

potential "fix" mentioned here: https://github.com/videojs/videojs-contrib-dash/issues/236#issuecomment-484205633

Fixed via #319.

Just published videojs-contrib-dash 5.1.0 which allows for TTML captions if set up via useTTML option
videojs('video', {
  html5: {
    dash: {
      useTTML: true
    }
  }
});


Problem with ES 1.2.3

I ran into problem while trying to download and benchmark ES v.1.2.3. Apparently, Python script in Rally 0.6.1 is complaining about not being able to extract ES bundle for this version. Tests never start, race always fail on this step.
Here is the relevant part of log:
2017-06-21 12:15:45,592 rally.provisioner INFO Preparing candidate locally in [/home/elasticsearch/.rally/benchmarks/races/2017-06-21-12-15-44/jatheontest/install].
2017-06-21 12:15:45,592 rally.provisioner INFO Unzipping /home/elasticsearch/.rally/benchmarks/distributions/elasticsearch-1.2.3.tar.gz to /home/elasticsearch/.rally/benchmarks/races/2017-06-21-12-15-44/jatheontest/install
2017-06-21 12:15:45,593 rally.mechanic ERROR Cannot process message [<esrally.mechanic.mechanic.StartEngine object at 0x7f19988eb748>]
Traceback (most recent call last):
File "/usr/local/lib/python3.4/dist-packages/esrally/mechanic/mechanic.py", line 244, in receiveMessage
cluster = self.mechanic.start_engine()
File "/usr/local/lib/python3.4/dist-packages/esrally/mechanic/mechanic.py", line 345, in start_engine
self.provisioner.prepare(binary)
File "/usr/local/lib/python3.4/dist-packages/esrally/mechanic/provisioner.py", line 49, in prepare
self._install_binary(binary)
File "/usr/local/lib/python3.4/dist-packages/esrally/mechanic/provisioner.py", line 72, in _install_binary
io.decompress(binary, self.install_dir)
File "/usr/local/lib/python3.4/dist-packages/esrally/utils/io.py", line 172, in decompress
_do_decompress(target_directory, tarfile.open(zip_name))
File "/usr/lib/python3.4/tarfile.py", line 1553, in open
raise ReadError("file could not be opened successfully")
tarfile.ReadError: file could not be opened successfully
2017-06-21 12:15:45,597 rally.racecontrol INFO Starting engine has failed. Reason [file could not be opened successfully].
2017-06-21 12:15:45,597 rally.main INFO Attempting to shutdown internal actor system.
2017-06-21 12:15:45,699 rally.main INFO Shutdown completed.
2017-06-21 12:15:45,699 root ERROR Cannot run subcommand [race].
Traceback (most recent call last):
File "/usr/local/lib/python3.4/dist-packages/esrally/rally.py", line 429, in dispatch_sub_command
race(cfg)
File "/usr/local/lib/python3.4/dist-packages/esrally/rally.py", line 375, in race
with_actor_system(lambda c: racecontrol.run(c), cfg)
File "/usr/local/lib/python3.4/dist-packages/esrally/rally.py", line 395, in with_actor_system
runnable(cfg)
File "/usr/local/lib/python3.4/dist-packages/esrally/rally.py", line 375, in
with_actor_system(lambda c: racecontrol.run(c), cfg)
File "/usr/local/lib/python3.4/dist-packages/esrally/racecontrol.py", line 323, in run
raise e
File "/usr/local/lib/python3.4/dist-packages/esrally/racecontrol.py", line 320, in run
pipeline(cfg)
File "/usr/local/lib/python3.4/dist-packages/esrally/racecontrol.py", line 42, in call
self.target(cfg)
File "/usr/local/lib/python3.4/dist-packages/esrally/racecontrol.py", line 262, in from_distribution
return race(Benchmark(cfg, distribution=True))
File "/usr/local/lib/python3.4/dist-packages/esrally/racecontrol.py", line 218, in race
benchmark.setup()
File "/usr/local/lib/python3.4/dist-packages/esrally/racecontrol.py", line 107, in setup
raise exceptions.RallyError(result.message)
esrally.exceptions.RallyError: file could not be opened successfully
2017-06-21 12:15:45,706 root INFO ---- Actor System shutdown

I successfully ran esrally --distribution-version=1.2.3 with Rally 0.6.1. just now.
Your problem may be caused by a corrupt download of Elasticsearch 1.2.3. You could try to wipe ~/.rally/benchmarks/distributions` and retry again.


set-filter 25.1.0

Type: Missing
Summary:
set-filter 25.1.0
Details:
ClearlyDefined license indicates OTHER
NPM indicates OTHER
GitHub license is MIT AND OTHER: https://github.com/ag-grid/ag-grid/blob/v25.1.0/LICENSE.txt
Resolution:
MIT AND OTHER
Affected definitions:

set-filter 25.1.0

@LandOfBliss - the package.json says "commercial" and the LICENSE in the package is a commercial license, so I think the declared license for the package is just "OTHER."


Fix sending files through a rest proxy

When the rest manager detects that it has a proxy to call, it converts all file blobs to data uris. When the proxy receives this request, the data uri isnt converted back to a blob, so an error is thrown by undici in a node environment.
This PR adds a conversion step before any files are added to a request to ensure their blob property is a blob instance.
The code for converting from a string to arraybuffer was taken from the same gist as the already existing code to convert an arraybuffer to a string, I just moved them to their own file (/util/base64.ts)

Codecov Report

Merging #2593 (5edd0ed) into main (5a18874) will decrease coverage by 0.07%.
The diff coverage is 51.61%.

@@            Coverage Diff             @@
##             main    #2593      +/-   ##
==========================================
- Coverage   24.36%   24.28%   -0.08%
==========================================
  Files         386      387       +1
  Lines       10159    10171      +12
  Branches       27       36       +9
==========================================
- Hits         2475     2470       -5
- Misses       7683     7700      +17
  Partials        1        1

Flag
Coverage Δ

unit
24.28% <51.61%> (-0.08%)
:arrow_down:

Impacted Files
Coverage Δ

helpers/channels/editChannel.ts
2.67% <0.00%> (-0.03%)
:arrow_down:

rest/createRequestBody.ts
2.63% <3.12%> (+0.84%)
:arrow_up:

util/base64.ts
68.53% <68.53%> (ø)

rest/runMethod.ts
2.59% <100.00%> (ø)

util/urlToBase64.ts
12.50% <100.00%> (-51.66%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@danny-may Please run deno fmt

I missed that, fixed!


Create AKA links for LUDown and LUIS CLI tools

Install the ludown CLI tool here to help describe language understanding components for your bot.
Install the luis CLI tool here to create and manage your LUIS applications.

LUDown: https://aka.ms/using-ludown
LUIS CLI: https://aka.ms/using-luis-cli

aka.ms links created ..
https://aka.ms/botbuilder-ludown
https://aka.ms/botbuilder-luis-cli


safari startup

It seems on some safari versions ( mobile ?) automatic media startup fails and block whole startup process
The actual process is :

load an empty.ogg sound and try to play it, catching exception to set a lock.
repeat until unlock media play success while displaying the prompt in loading screen ( template default.tmpl )
continue and give hand to python code (main.py).

for safari it probably should be:

detect safari and do nothing if matching.
Display a prompt in loading screen, wait for a mouse click (same event for one finger touch )
continue and give hand to python code (main.py).

This would not work if navigator user agent is faked, but unlikely to happen on iOS.

please report iOS version status

ume fixed for Safari 15, tested on iphone/ipad
keeping it open for safari 13/14 as it may just be a wasm compilation flag problem. But don't except much.


Exception on start on mobile

(Windows.UI.Xaml.Markup.XamlParseException)
Cannot find a Resource with the Name/Key DefaultMenuFlyoutPresenterStyle [Line: 49 Position: 21]
Phone Version         : 10.0.15254
Windows Version     : 10.0.15254.600
OS Build                  : 15254.feature2_rs3svc(winpbld).20191127-1832
Quarrel                   : 19.12.27

What triggered this exception, is this when you opened Quarrel?

Yep, it crashes after app start whilst that big Quarrel splash logo is rotating (even before some first page could appear)


Jump to latest added timeline whenever a new timeline is added

Otherwise we're showing log_time all the time and this is unlikely of interest once there is a new one.
Also, this is another thing that should go to blueprint configuration!

Some caveats:

log_time has all the logged data. Any other timeline could be missing a lot of the logged data, making users wonder "why isn't my logged data showing up!?
Switching timeline at runtime (when a new timeline is added) can be quite annoying/distruptive ("why did my UI change!?")

To make this good I think we should separate the two cases:

Selecting a default timeline
Automatically switching to a new timeline

For the second case, we should re_log::info! the switch, which will show a little toast notification to the user, making it less surprising.
We should also make sure that any Space View which lacks data because there is none on the current timeline should communicate that by showing some text "No data for this view on the current timeline ${name}. Click here to switch to the log_time timeline instead." or similar.

seems at some point we implemented this without any extra logging


App completely unuseable

Platform:
macOS Sierra 10.12.3

Kap Version:
1.0.1
Steps to reproduce
Opening App. Nothing works. First attempted with a second display, but does not work with single display either.
Current behaviour
It seems that no buttons are clickable. Nothing happens when I click buttons on the app. I cannot even quit the app without using the activity monitor.

Expected behaviour
I expect to be able to use the app.
Workaround
I have found no workaround. I even uninstalled the app and installed the latest update a month apart. Still have the same issue.

I have the exact same issue

This is really strange. A couple of questions, are you able to drag the app off the menubar and/or access preferences? If you are able to access preferences, does it allow you to change any settings?
Also wondering if you have or have had issues like this in other Electron apps? Anything about your system you think could be of interest?

Please try Command + Option + I – the DevTools should pop up. Then, please go to the console and take a screenshot if there's any relevant info there 👌

@skllcrn I can drag it off the menu bar, but can't access preferences.
@matheuss Here's a screenshot of the console log.

Thank you, @iamtirado! That's very useful – we'll release a patch this weekend.
Could you also try to go to a terminal and run cat ~/Library/Application\ Support/Kap/Settings? If such file doesn't exist, please try ls ~/Library/Application\ Support/Kap

I get the same console.log

Amazing, thank you too @andreleon. Definitely an easy fix and will definitely be out this weekend 🙌

How about that patch on this
 issue? 😏

We haven't had a lot of time recently @torkiljohnsen. Contributions welcome 🙂

@CodeTheory I wouldn't know where to start 🙂  I just heard good things about Kap and wanted to try it out, but it is not working at all. The program window opens underneath the icon, but no buttons are clickable and the controls panel won't open at all.

@torkiljohnsen I'll try and find some time on the weekend to resolve this for you. ❤

Just pushed a fix 🙏 Could you guys please try it? https://kap-artifacts.now.sh/master @torkiljohnsen @andreleon @iamtirado 🙌

Just pushed a fix 🙏 Could you guys please try it? https://kap-artifacts.now.sh/master @torkiljohnsen @andreleon @iamtirado 🙌

I have no idea where it's saving the videos or gifs if it's actually saving them. And when I go to preference I have no response when clicking [Choose]

When I choose Save to WebM, it stays stuck on "Processing..."

When choosing GIF or mp4, it seems to cancel the recording when I hit stop. It doesn't notify me of any further activity.

Exactly the same issues as @iamtirado above.
Console shows this:

Recording looks like it's working, but I have no way of setting the "Save to…" location, and I am unable to locate any new gif file on my computer after doing a 10 second test recording.

@matheuss I noticed a potential problem. You only apply the default settings if getting audio sources succeed, but if it fails, it will fail silently and not apply those.
Reply to this email directly, view it on GitHub, or mute the thread.

No. Mid 2014 MacBook Pro, 15-inch, retina
macOS Sierra 10.12.3


EOFError with Gzip read

I created an issue on the Pandas repo, but it looks like it might be an s3fs, versions 0.3.0 and greater, error.
import pandas as pd
data = pd.read_csv("s3://bucketname/file.csv.gz")

Gives the following error:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/ubuntu/miniconda3/envs/test/lib/python3.7/site-packages/pandas/io/parsers.py", line 685, in parser_f
    return _read(filepath_or_buffer, kwds)
  File "/home/ubuntu/miniconda3/envs/test/lib/python3.7/site-packages/pandas/io/parsers.py", line 463, in _read
    data = parser.read(nrows)
  File "/home/ubuntu/miniconda3/envs/test/lib/python3.7/site-packages/pandas/io/parsers.py", line 1154, in read
    ret = self._engine.read(nrows)
  File "/home/ubuntu/miniconda3/envs/test/lib/python3.7/site-packages/pandas/io/parsers.py", line 2059, in read
    data = self._reader.read(nrows)
  File "pandas/_libs/parsers.pyx", line 881, in pandas._libs.parsers.TextReader.read
  File "pandas/_libs/parsers.pyx", line 896, in pandas._libs.parsers.TextReader._read_low_memory
  File "pandas/_libs/parsers.pyx", line 950, in pandas._libs.parsers.TextReader._read_rows
  File "pandas/_libs/parsers.pyx", line 937, in pandas._libs.parsers.TextReader._tokenize_rows
  File "pandas/_libs/parsers.pyx", line 2124, in pandas._libs.parsers.raise_parser_error
  File "/home/ubuntu/miniconda3/envs/test/lib/python3.7/_compression.py", line 68, in readinto
    data = self.read(len(byte_view))
  File "/home/ubuntu/miniconda3/envs/test/lib/python3.7/gzip.py", line 482, in read
    raise EOFError("Compressed file ended before the "
EOFError: Compressed file ended before the end-of-stream marker was reached

Setup
Installed Pandas and s3fs via pip:
pip install pandas s3fs

Pandas version: 0.25.1
s3fs version: 0.3.3

Looks like the problem is unrelated to pandas:
import gzip
import s3fs
fs = s3fs.S3FileSystem()
s3_fh = fs.open('s3://bucketname/file.csv.gz')
fh = gzip.open(s3_fh)
data = fh.read()

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ubuntu/miniconda3/envs/test/lib/python3.7/gzip.py", line 276, in read
    return self._buffer.read(size)
  File "/home/ubuntu/miniconda3/envs/test/lib/python3.7/gzip.py", line 482, in read
    raise EOFError("Compressed file ended before the "
EOFError: Compressed file ended before the end-of-stream marker was reached

Again, the s3fs+gzip read works in versions < 0.3.0.

Weird timing,  but I am experiencing the same issue with s3fs==0.3.3.  Files that previously work on 0.2.0 now fail with EOFError

I am hitting the same error with s3fs==0.3.3 on gzipped files.

Since s3fs hasn't been released in almost a month, I think this is due to one of the three dependencies pushing a break change. I tested boto3's latest version and botocores latest version and had no problem accessing gzipped files so I would think that clears both boto3 and botocore.
It looks to me that filesystem_spec pushed some code yesterday  in this commit and this commit  in two releases that may be the culprit. I don't know this space quite well enough to be able to debug their work, but I hope this might help in some heuristic sense.

After another quick check, I can confirm that downgrading by using:
pip install fsspec==0.4.1
and pulling files using s3fs works fine.

Thanks for the report. Taking a look.

I'm able to reproduce, but only on a real request against s3, not with moto. I'll look into this today though, and should be able to do a release of fsspec once it's fixed.

I tried fsspec==0.4.3 still got the same error as that from https://github.com/dask/s3fs/issues/225#issue-486662802
So while pending on a fix from fsspec, May I ask for a minor release to include :point_up: workaround so s3fs users like me won't need to dig into this issue to find a workaround?

I think you need fsspec==0.4.1

I think you need fsspec==0.4.1

Yes, that's what I tried to apply via #226

Well that was a fun diversion. I've kinda figured out why we couldn't reproduce with moto.
When we get to the end of the file, we make a request to S3 with byte range request like range=3-2. According to https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.35, that's invalid

If the last-byte-pos value is present, it MUST be greater than or equal to the first-byte-pos in that byte-range-spec, or the byte- range-spec is syntactically invalid. The recipient of a byte-range- set that includes one or more syntactically invalid byte-range-spec values MUST ignore the header field that includes that byte-range- set.

So IIUC, the server is suppose to ignore that header and return the full body. S3 does indeed do that. Moto (which I think uses httpretty) doesn't. Instead, there's an InvalidRangeRequest exception, which s3fs catches.

This should be fixed with #229. If anyone wants to test that out quickly, I'd appreciate it, since we can't actually write a unit test for this (that doesn't hit S3). But with my local testing, things are OK.
Otherwise, I'll do a release later this afternoon.

I can confirm code in master (v0.3.4) branch works for me w/ fsspec==0.4.3.
@TomAugspurger Thanks! when we can see this from pypi.org? the latest release over there still shows 0.3.3 (Aug 8, 2019).

@TomAugspurger works for me! Thanks for the quick work on getting a fix!

0.3.4 is on PyPI. Will show up on Conda-forge later today.

@TomAugspurger Thanks for quick fix and release!


Enable cf push without .git

When a project (like mdw-demo includes their .git directory in .cfignore, the get the stack trace below.  This needs analysis to figure out how we can handle this.      for the property mdw.git.local.path

This is a good workaround if you want to take your assets from Git and don't actually want to push from local.  The reason is that the buildpack is wired to push to ".".  If I cf push with these settings:
MDW_ASSET_LOCATION: './git/assets'
"mdw.git.local.path": "./git"

Then mdw clones into ./git and loads assets from ./git/assets.  This works, but meanwhile the assets I pushed are ignored.  Also there's an exception message (which doesn't seem to affect anything).  May need a fix in mdw.
2017-12-08T12:31:46.02-0700 [App/0]      OUT ** Please import the MDW base and hub packages
2017-12-08T12:31:46.02-0700 [App/0]      OUT ****************************************
2017-12-08T12:31:46.02-0700 [App/0]      OUT ** WARNING: Asset location ./git/assets does not exist.
2017-12-08T12:31:46.02-0700 [App/0]      OUT ******************************************
2017-12-08T12:31:46.05-0700 [App/0]      ERR     SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
2017-12-08T12:31:46.05-0700 [App/0]      ERR SLF4J: Defaulting to no-operation (NOP) logger implementation
2017-12-08T12:31:46.05-0700 [App/0]      ERR SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
2017-12-08T12:31:46.18-0700 [App/0]      OUT [(i)20171208.19:31:46.185 ~13] Loading assets from path: assets
2017-12-08T12:31:46.18-0700 [App/0]      OUT [(s)20171208.19:31:46.185 ~13] **** WARNING: Git location ./git does not contain a repository.     If .git was ignored during cf push, would those pushed assets be replaced during startup when MDW clones repo?

That's exactly the point of this issue.  We want to be able to ignore .git.  Currently if we ignore .git and keep mdw.git.local.path as ".", then we get the stack trace in the first comment above.  If we change mdw.git.local.path to something other than ".", then that problem is fixed, but then whatever assets are being pushed in local are ignored and only imported assets can be used.  This works fine for non-dev purposes, but it effectively prevents you from trying out changes via cf push without committing them to Git.  We need to enhance mdw-buildpack to handle non-"." locations because it's currently wired to push relative to the project directory (".").


feat: add footer sparkling effect

https://github.com/user-attachments/assets/6f8a07f7-a3f3-45a1-8e87-cd0af581aa77

F for firetruck?


Add a way to prefer narrowing array to tuple if possible

Search Terms
tuple narrow, prefer tuple
Suggestion
This is a meta-suggestion over the top of some of the cases in #16896 and #26113, basically at current it's quite repetitive to specify overloads for tuple types and we tend to wind up things like this:
// etc up to how ever many overloads might be needed
function zipLongest<T1, T2, T3, T4>(
  iterables: [Iterable<T1>, Iterable<T2>, Iterable<T3>, Iterable<T4>]
): Iterable<[T1, T2, T3, T4]>

function zipLongest([]): []
function zipLongest<T>(iterables: Array<Iterable<T>>): Iterable<Array<T>>

We can already specify the type if we know only tuples are going to be input e.g.:
type Unwrap<T> = T extends Iterable<infer R> ? R : never
type ZipUnwrapped<T> = { [P in keyof T]: Unwrap<T[P]> }

declare function zipLongest<Ts extends Array<Iterable<any>>>(iterables: Ts): Iterable<ZipUnwrapped<Ts>>

const strs = ['foo', 'bar', 'baz', 'boz']
const nums = [1, 2, 3, 4]
const zipPair: [string[], number[]] = [strs, nums]
const z = zipLongest(zipPair)

In this case the type of z is correctly inferred to be Iterable<[string, number]>, but if we remove the type declaration on zipPair then we get that the inferred type of z is Iterable<(number | string)[]>.
Basically what I propose is adding some way to specify that an array should be narrowed to the most precise tuple possible if we can.
There's a couple approaches that might be used:
Approach 1
Use the [...TupleParam] spread syntax already mentioned in #26113 and when this syntax is seen narrow to the best tuple we could get. So the above declaration would become:
// The only difference is iterables: Ts has become iterables: [...Ts]
declare function zipLongest<Ts extends Array<Iterable<any>>>(iterables: Ts): Iterable<ZipUnwrapped<Ts>>

This has the downside that it might for any Ts in [Some, Tuple, Params, ...Ts] to be narrowed even when the wider type was what was intended.
Approach 2
Add some new syntax that specifies that if a certain value can be narrowed into a tuple then we should do so:
// Arbitrary syntax 1
declare function zipLongest(
  iterables: [...tuple Ts]
): Iterable<ZipUnwrapped<Ts>>

// Arbitrary syntax 2
declare function zipLongest<Ts extends [!...Iterable<any>]>(
  iterables: Ts
): Iterable<ZipUnwrapped<Ts>>

The syntax isn't really all that important. The main downside of this approach is that we need two overloads, one for the narrowed tuple and one for the non-narrowed tuple. Additionally extra syntax would need to be supported and maintained.
Examples
Other than the above example another example that would benefit is the builtin lib.promise.d.ts which specifies overloads in this repetitive overloading pattern.
We could write something like Promise.all like so:
type UnwrapMaybePromiseLike<T>
= T extends PromiseLike<infer R> ? R : T

type UnwrappedArray<T extends Array<any>> = {
    [P in keyof T]: UnwrapMaybePromiseLike<T[P]>
}

interface PromiseConstructor {
    all<Ts extends Array<any>>(
        promises: [...Ts]
    ): UnwrappedArray<Ts>
}

And inference would just work.
// a would inferred to be [number, number]
const a = Promise.all([1, 2])

// b would be inferred to be [string, number]
const b = Promise.all(['foo', Promise.resolve(12)])

Checklist
My suggestion meets these guidelines:

[?] This wouldn't be a breaking change in existing TypeScript / JavaScript code
[✔] This wouldn't change the runtime behavior of existing JavaScript code
[✔] This could be implemented without emitting different JS based on the types of the expressions
[✔] This isn't a runtime feature (e.g. new expression-level syntax)

EDIT: Realized Promise.race was supposed to be Promise.all oops.

You can force an array to be treated as a tuple by creating a function
function tuple<Args extends any[]>(...args: Args): Args { return Args; }

const pair = tuple(1, "string");
// pair is inferred as [number, string]

The rest of it seems a bit more tricky. Mapped types don't work properly with tuples so it might need language level support to convert one tuples type to another (sort of like array.map but for tuples)
An even more general approach might be to allow recursion in types so we can use conditional types to write our own mapping logic.

I think you can already do this with features currently in typescript@next. In particular, the work in #26063 enables your scenario.
type Iterablized<T> = { [K in keyof T]: Iterable<T[K]> };

declare function zipLongest<T extends any[]>(iterables: Iterablized<T>): Iterable<T>;

declare let triple: [Iterable<string>, Iterable<number>, Iterable<string[]>];

let zippedTriple = zipLongest(triple);  // Iterable<[string, number, string[]>

With respect to explicit syntax for creating tuples from array literals, we're tracking and debating that in #16656.
Meanwhile, with #24897 (in TS 3.0) you can easily add the following convenience function:
function tuple<T extends any[]>(...elements: T) {
    return elements;
}

and you then have a very low cost general way of creating tuples. For example:
const t1 = tuple(1, "hello");      // [number, string]
const t2 = tuple(1, tuple(2, 3));  // [number, [number, number]]
const t3 = tuple(1, [2, 3]);       // [number, number[]]
const t4 = tuple();                // []

I've been trying it with typescript@next and the mapped types isn't quite the problem, strictly speaking the definition above works, it's just not particularly useful when doing it inline with inferred types e.g.:
for (const [a!, b!] of zipLongest([strings, numbers]) {
  // Inferred type of both a and b are string | number
}

Which is what I'm proposing improving. Basically someway to say that if [strings, numbers] can be inferred as a tuple, it should be.
This already works (in typescript@next) when the argument is variadic rather than as an array directly e.g.:
type Iterablized<T> = { [K in keyof T]: Iterable<T[K] | undefined> };

declare function zipLongest<Ts extends Array<any>>(...iterables: Ts): Iterablized<Ts>

const strs = ['foo', 'bar', 'baz']
const nums = [1, 2, 3]
// Correctly inferred to be Iterable<[string | undefined, number | undefined]>
const z = zipLongest(strs, nums)

However this can't be used as I'm extending the zipLongest with additional arguments that can't be disambiguated from iterables. So it'll look something like this soon:
const fillers = [() => '', () => 0]
for (const [a, b] of zipLongest([strs, nums], fillers)) {

}

@Jamesernator One trick that does work is to ensure that the contextual type for the array literal includes a tuple type. Once that is the case we'll infer tuple types:
declare function zipLongest<Ts extends Iterable<any>[] | [Iterable<any>]>(iterables: Ts): Iterable<ZipUnwrapped<Ts>>;

Oh that's useful, that should probably be added into the handbook somewhere as I don't think I would've ever figured that out.

@DanielRosenwasser @weswigham FYI since we were discussing this the other day. Note that if we put this in the handbook, we should recommend using T extends XXX[] | [XXX] as the constraint pattern (versus T extends XXX[] | []) because this pattern ensures that members on T are just what you would see on XXX[]. (With an empty tuple in the pattern some never types creep in.)

The original comment suggested that Promise.all would be able to be written without N overloads, i.e.:
interface PromiseConstructor {
    all<Ts extends Array<any>>(
        promises: [...Ts]
    ): UnwrappedArray<Ts>
}

Is that possible, or is there a separate issue tracking that specific capability?
FWIW it looks like the Promise.all overloads are still in the standard library so I generally assume not.
Also, I'm trying to get an example working like this:
interface AstNode<T> {
  value: T;
}

const n1: AstNode<number> = { value: 1 };
const n2: AstNode<string> = { value: "asdf" };

// https://github.com/microsoft/TypeScript/pull/26063 says that if T is a tuple
// type, the mapped type will map each specific type in the tuple.
type Values<T> = { [P in keyof T]: T[P] extends AstNode<infer V> ? V : never };

function node<
  T,
  // https://github.com/microsoft/TypeScript/pull/24897 allows open ended tuple types, and
  // I want D to be open-ended here.
  D extends [...AstNode<any>[]],
  // But then here I want to convert it back to fixed-length to get [string, number] instead
  // of [string | number, string | number]
  A extends Values<D>
>(deps: D, fn: (...args: A) => T): AstNode<T> {
  const values = deps.map(d => d.value) as A;
  return { value: fn(...values) };
}

const n3: AstNode<string> = node([n1, n2], (v1, v2) => {
  const v1n: number = v1; // v1 is string | number
  const v2s: string = v2; // v2 is string | number
  return v1n + v2s;
});

Where I want to use the syntax of open-ended tuples to say I take N deps, but then I want to map those onto a fixed-length tuple so that my node(...) caller function gets the benefit of fixed-length tuple type inference on its v1, v2 parameters.
My naive guess is that what I want to accomplish is basically the same problem that Promise.all has, of jumping from open-length tuples in declarations to fixed-length tuples in usage sites. Is that somewhat right/coherent? Thanks!


BUGFIX extension was incorrectly merged into config as higher priorit…

…y than class config
Attempted fix for https://github.com/silverstripe/silverstripe-comments/issues/242
See comment: https://github.com/silverstripe/silverstripe-comments/issues/242#issuecomment-378471568

More discussion about this here https://github.com/silverstripe/silverstripe-framework/issues/7970

yeah, duplicate of https://github.com/silverstripe/silverstripe-framework/pull/7971 and it was rejected. sozzles


本机起链，一次多节点，不出块

Issue Template
本机起链，一次多节点，不出块
Description
[Description of the issue]
Steps to Reproduce
场景：起链时，启动多个节点

一次启动10个节点，成功出块；
一次启动 大于等于20节点，不出块。

Expected behavior: [What you expect to happen]
Actual behavior: [What actually happens]
Reproduce how often: [What percentage of the time does it reproduce?]
Versions
[The versions of the CITA, operating systems or VM software you use]
Additional Information
[Any other information which would be helpful to reproduce the issue].

For a single machine, start 20 nodes at the same time takes up too much resources, which can cause problem.


SSO Garmin error:  Access denied

Hi,
I am getting this error:

We’re sorry.

Something went wrong, and our site is temporarily unavailable.
We’re already working to get things back on course.

Access denied
You cannot access sso.garmin.com. Refresh the page or contact the site owner to request access.

Ray ID: 750c0e9a6f3495e5
Timestamp: 2022-09-26 12:38:30 UTC
Your IP address: <IP_ADDRESS>
Requested URL: sso.garmin.com/sso/login?service=https%3A%2F%2Fconnect.garmin.com%2Fmodern&clientId=GarminConnect&gauthHost=https%3A%2F%2Fsso.garmin.com%2Fsso&consumeServiceTicket=false
Error reference number: 1020
Server ID: FL_128F174
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.95 Safari/537.36

Is this temporary or am I doing something wrong?
Best regards,
Arturo

I'm getting the same error since Friday:
We’re sorry.

Something went wrong, and our site is temporarily unavailable.
We’re already working to get things back on course.

Access denied

You cannot access sso.garmin.com. Refresh the page or contact the site owner to request access.

  • Ray ID: 750c2f33784dbb67
  • Timestamp: 2022-09-26 13:00:45 UTC
  • Your IP address: <IP_ADDRESS>
  • Requested URL: sso.garmin.com/sso/login?service=https%3A%2F%2Fconnect.garmin.com%2Fmodern&clientId=GarminConnect&gauthHost=https%3A%2F%2Fsso.garmin.com%2Fsso&consumeServiceTicket=false
  • Error reference number: 1020
  • Server ID: FL_470F240
  • User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.95 Safari/537.36

I only run the job twice a day and am getting the same failure with my last successful run being on the 23rs (Friday).  Sounds like maybe something upstream changed.

I also have the same failure. I wrote garmin_upload.txt, a little python/selenium/chrome automation to upload a fit file automatically to garmin connect.
As it is, the script uploads the file withings-sync-log.fit that resides in the same directory as the script itself. You can modify line 16 to point to another file. Username and password are hardcoded in lines 18-19.
As with all selenium browser automations, the script might fail at any point because the page takes to long to load, or garmin has made a change somewhere.
If you set debug=True on line 183, the script will stop right before clicking on "import data", so nothing gets sent to garmin.
Have fun.

Hi, i am having the same as well as

I switched to https://github.com/La0/garmin-uploader for uploading for now. Flexible and scriptable.
Sadly I don't know enough about python to have a closer look at that implementation and get an idea why this still works. Possibly someone else can jump in?

I switched to https://github.com/La0/garmin-uploader for uploading for now. Flexible and scriptable. Sadly I don't know enough about python to have a closer look at that implementation and get an idea why this still works. Possibly someone else can jump in?

from a first glance, it seems to check more things. It is difficult to say for sure what makes difference.
I can see apart from user and pass also uses a csrf_token. I will try to do some tests the following days.

I did some test and it looks like Cloudflare caught up on http2 connections.
When reverting to the earlier CloudScraper solution in this commit things seems to work again for me.
@longstone I can't remember why we had to switch to http2 instead of this so I am not sure if this is a valid fix.

If someone wants to test, this version of garmin.py is working for me.

If someone wants to test, this version of garmin.py is working for me.

just tried - this version works for me, thanks!

I did some test and it looks like Cloudflare caught up on http2 connections.
When reverting to the earlier CloudScraper solution in this commit things seems to work again for me.
@longstone I can't remember why we had to switch to http2 instead of this so I am not sure if this is a valid fix.

I have no clue..

I did some test and it looks like Cloudflare caught up on http2 connections. When reverting to the earlier CloudScraper solution in this commit things seems to work again for me. @longstone I can't remember why we had to switch to http2 instead of this so I am not sure if this is a valid fix.

It was a solution by @ciancim to some Garmin-Connect problems in this PR: https://github.com/jaroslawhartman/withings-sync/pull/49

If someone wants to test, this version of garmin.py is working for me.

For me it didn;t work

For me it didn;t work
Can you share your (redacted) debug logs?

For me it didn;t work
Can you share your (redacted) debug logs?
I have slightly modified the sync.py(+ using a bit older version) but i do not think that makes difference:

File "/home/pi/withings-garmin-v2/garmin.py", line 183, in login
session = GarminConnect.get_session(email=username, password=password)
File "/home/pi/withings-garmin-v2/garmin.py", line 103, in get_session
raise APIException(f"SSO error {ssoresp.status_code} {ssoresp.text}")
garmin.APIException: SSO error 401
I remember in the past that was not working for me as well as.
I trued to manually inspect the post of https://sso.garmin.com/sso/login but in my browser, it logging works.
I tried to include all the parameters, but that didn;t make any difference.

If someone wants to test, this version of garmin.py is working for me.

Thanks for the solution. I need to install Cloudscraper (pip3 install cloudscraper). After the installation the import to garmin works.

If someone wants to test, this version of garmin.py is working for me.

Didn't work for me.
`    We’re sorry.
Something went wrong, and our site is temporarily unavailable.We’re already working to get things back on course.

Access denied
You cannot access sso.garmin.com. Refresh the page or contact the site owner to request access.

Ray ID: 754f5dbbfa6182b1
Timestamp: 2022-10-04 16:41:37 UTC
Your IP address: <IP_ADDRESS>
Requested URL: sso.garmin.com/sso/login?service=https%3A%2F%2Fconnect.garmin.com%2Fmodern&clientId=GarminConnect&gauthHost=https%3A%2F%2Fsso.garmin.com%2Fsso&consumeServiceTicket=false
Error reference number: 1020
Server ID: FL_413F394
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.95 Safari/537.36
`

Solution from @stynoo worked for me only after I changed my IP. Initially I got rejected multiple times, then I tested solution from @stynoo and got rejected as well. It seems that Cloudflare or Garmin is very sensitive on number of tries to log-in from particular IP. So I powered on my NordVPN, connected to VPN in my country and it worked. So you might get lucky if you try it again after some time you have tried that the last time. Or you can try to use your mobile phone internet ;)

Solution from @stynoo worked for me only after I changed my IP. Initially I got rejected multiple times, then I tested solution from @stynoo and got rejected as well. It seems that Cloudflare or Garmin is very sensitive on number of tries to log-in from particular IP. So I powered on my NordVPN, connected to VPN in my country and it worked. So you might get lucky if you try it again after some time you have tried that the last time. Or you can try to use your mobile phone internet ;)

Similar behavior with me. At the beginning it didn;t work, but i tried again and now it works . (but i did not tried any further tests)

If someone wants to test, this version of garmin.py is working for me.

Thanks! Works for me too!

If someone wants to test, this version of garmin.py is working for me.

That is working for me.  I checked out your modded branch, built it with podman, pushed to my private registry, and re-pointed my kubernetes manifests to that image and tag.  I had to authenticate the withings API, but after that it is working as expected.

If someone wants to test, this version of garmin.py is working for me.

works for me. Thanks a lot

yes, replaced that file, and it works for me, too. Thanks!

Is there an easy way to replace this file in docker image that was installed using docker pull command as shown on the main page?

Is there an easy way to replace this file in docker image that was installed using docker pull command as shown on the main page?

Well the only way I see is to run the container with mounted folder from host (-v <host_path>:<container_path>) and copy the modified python file from host to the container. The image then stays modified. However it will be mounting your folder every time so think carefully which path you mount.

Well the only way I see is to run the container with mounted folder from host (-v <host_path>:<container_path>) and copy the modified python file from host to the container. The image then stays modified. However it will be mounting your folder every time so think carefully which path you mount.
Thanks for the idea, I replaced all the garmin.py files I could find, I also had to install cloudscraper (pip install cloudscraper) and it seems to be working again.
Also in my case (being docker beginner user running it on a Synology NAS) I had to rename .withing_user.json temporarily to keep my container running (waiting for the token), otherwise it terminated before I could "docker exec" sh on it and run the copy/pip commands.


Upgrade to normalize 7.0.0

Normalize 7.0.0 is out. https://github.com/necolas/normalize.css/releases

Do you have any thoughts/feelings about whether opinionated rules should be included or not?

I do, actually!
I prefer to fork normalize.css alter the rulesets to match what the particular project needs. That means having a scaffold for rules needed in my project, but not needed in a drop-in-and-forget version of normalize.css. For example, p { margin: 1.5rem 0 }
Fortunately, this project a mixin-based version of normalize and a fork-ready version of normalize. So I can keep my opinionated rules in the forks and, in the mixin-based normalize, follow the parent normalize.css project's direction of removing opinionated rules.

What I'm referring to is the rules that version 6 removed and that version 7 is bringing back, due to the difference in vision of the two prior maintainers.
(And on that note, I have to ask: is that debacle the reason for your recent "code of conduct" commits?)

What I'm referring to is the rules that version 6 removed and that version 7 is bringing back

I'm just following whatever is in their code when deciding what goes in the mixin version of normalize-scss. Other contributers and users have expressed the desire that the default CSS output of the mixin should match the CSS in normalize.css.
And I will keep the opinionated rules that I like in the fork versions.

And on that note, I have to ask: is that debacle the reason for your recent "code of conduct" commits?

@garrettw Lol. No. It's unrelated to their disagreements. I've been meaning to add one to all my projects and I am finally getting around to adding them.


Blog and about page mockups

Blog Page

About Page

No longer using this design, it felt to artificial. It didn't have enough "personality". It seemed like a weird mix of Googleish design and random shapes combined together to make some sort of "minimal and moden" UI.
All existing issues are being closed.


fixed overlapping of text in mobile view

fixes: #1006
The overlapping of text in examples and documentation pages in mobile view has been fixed.

Thank you for your work @Soumi150 but the text is still overlapping when the width is <= 790px

Awesome thanks!
@all-contributors please add @Soumi150 for code and bug


Fix autocorrectable Cops

Needs some changes to the cops enabled:

We have a convention of commas in the last element of a literal, let's just keep that
I dont understand the changes from alias_method and alias
The parallel assignment changes are silly
Array %w should be delimited by brackets, not parens

Closing until I move forward on #442


fixed cpanfile syntax

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
Review the contributing guidelines for this repository; remember in particular:

do not modify code without testing for regression
provide simple unit tests to test the changes
if you change the schema you must patch the test databases as well, see Updating the schema
the PR must not fail unit testing

Description
One or more sentences describing in detail the proposed changes.
Correct syntax in cpanfile
Use case
Describe the problem. Please provide an example representing the motivation behind the need for having these changes in place.
The cpan file was updated in release/92 branch but the changes were not ported to master and are also missing from release/93 branch.
Benefits
If applicable, describe the advantages the changes will have.
File correction is not lost.
Possible Drawbacks
If applicable, describe any possible undesirable consequence of the changes.
can't think of any
Testing
Have you added/modified unit tests to test the changes?
NA
If so, do the tests pass/fail?
NA
Have you run the entire test suite and no regression was detected?
NA

Coverage remained the same at 82.445% when pulling 590c3240bc82226ec528510e0bc6ee9fcd57c5c4 on bugfix/cpanfile_update into 71dab88b9fc7b25da994668360ab29102e4a15b2 on master.

Necessary fix, approved.


Update request

I just noticed my domain angelosada.com is on your list and I can't get why. Can you explain me the reason?

Need to introduce a whitelist here for PR's

HI @angelosada, It's because of https://github.com/mitchellkrogza/Phishing.Database/blob/7f962127e689c228636c2f5aa27f97b04ee90714/input-source/mitchellkrog-feed.list#L30
Cheers,
Nissar

Ok, thank you so much for your help.

@angelosada Removed from my feed list - should be pulled in on next build and eventually clear itself


CSV parsing error

the NaN extra row is being caused by an empty line being parsed - with the length-1 hotfix you then lose a line of data if there is no empty line at the end.

Parsing should be all good now


feat: update vulnerable libs in docker

Before

After

Nice work! Regarding the cross-spawn dependency - you probably need to override every single occurance in the package.json for it to work. I think thats preferable over hacking the dockerfile?


Test-UcmO365ServicePlan unable to return known service state

Attempting to pull a 365 service state for a user known to have the service enabled fails with an unknown state error.
Need to investigate this.
`PS C:\Users\Atreidae\OneDrive - Telstra Purple\Customers\yyy\Scripts> Test-UcmO365ServicePlan -UPN<EMAIL_ADDRESS>-ServiceName MCOSTANDARD
WARNING: 13:48:12 MCOSTANDARD is in unknown state , you may need to run '(Get-MsolUser -UserPrincipalName john.doe@yyy.com.au).Licenses' for more information
INFO: MCOSTANDARD is in unknown state
Name                           Value

Function                       Test-UcmO365ServicePlan
Status                         ERROR
Message                        MCOSTANDARD is in unknown state, you may need to run '(Get-MsolUser -UserPrincipalName John.doe@yyy.com.au).Licenses' for more information  `


Vaccinations history has decimal values

The endpoint "GET https://api.corona-zahlen.org/vaccinations/history" returns a decimal value for the number of second vaccinations on the days 03/07 and 03/08 as you can see here:
        {
            "date": "2021-03-07T00:00:00.000Z",
            "vaccinated": 114893,
            "firstVaccination": 114893,
            "secondVaccination": 34600.5
        },
        {
            "date": "2021-03-08T00:00:00.000Z",
            "vaccinated": 178973,
            "firstVaccination": 178973,
            "secondVaccination": 51629.5
        }

The RKI .xlsx-file includes the following:

07.03.2021
114.893
34.601
149.494

08.03.2021
178.973
51.630
230.603

So it seems there is a bug in the API.
Quick fix is to just to round up the values but would be better to have a look in to it.

The RKI .xlsx-file includes the following:
07.03.2021 	114.893 	34.601 	149.494
08.03.2021 	178.973 	51.630 	230.603
So it seems there is a bug in the API.
Quick fix is to just to round up the values but would be better to have a look in to it.

You are wrong!
The cells in the RKI .xlsx file are reduced to 0 decimal values!

if you extend the values from 7.3. and 8.3. to 1 decimal value this shows up

so what the API reads out is exactly what RKI transmit! Okay, ,5 vacciation makes no sence but there is no bug in the API!

Well, my fault, sorry.
Very weird. Didn't expect them to record .5 vaccinations 😂

a few weeks ago there was the problem too. after a few days RKI corrected the values ..................
if round up or cut , i think thats make not a big difference! Excel rounds it up if the number of decimal values are set to 0!
and that is what RKI published!

The .5 Values are still there. I have written a PR, Marlon have to merge it if he agreed.

In the meenwhile RKI corrected the data! I think the Issue can be closed!


gmp-bug-13420-13421 fails to compile

A Makefile input seems to be missing from the ManyBugs archive?
[vagrant@localhost gmp]$ ./configure --build=i686-linux-gnu "CFLAGS=-m32" "CXXFLAGS=-m32" "LDFLAGS=-m32"
checking build system type... i686-pc-linux-gnu
checking host system type... i686-pc-linux-gnu
...
config.status: error: cannot find input file: `tests/mpbsd/Makefile.in'

There appears to be no tests/mpbsd directory, although there is a mpbsd/Makefile.in within the gmp directory.
Is the result of losing hard symbolic links during the extraction of the ManyBugs archive? No errors were thrown during extraction on Cygwin, but F22 threw plenty. If so, I might be able to extract the symlinks by tricking F22 into believing it has the same directory structure as the original ManyBugs machine (via chroot); even then, files would have to be extracted in the correct order.

I've stumbled upon this same problem inside the chroot.
I've managed to make this work removing all the references of *mpbsd/Makefile* from the configure file.
Although this is a hack, I think is plausible for us because we're not running the scenario in a BSD environment.
btw The sanity is working as expected.

Nice! I'll give that a go on the Docker container.

Okay!

Incorporated fix into many-bugs/gmp/Dockerfile.bug.


change the Venafi URL for cert-manager support

Can we replace the current URL on the cert-manager support page with this link to direct users to the Venafi LTS cert-manager product
https://venafi.com/lts-for-cert-manager/

Can we replace the current graphic with the attached image and change the accompanying text to say:
Venafi is the principal maintainer of the cert-manager project and works directly with the CNCF. Commercial support for cert-manager is available as part of Venafi's Long Term Support (LTS) offering which includes access to expertise to help organisations scale effectively and securely

with cert-manager.


Organized Properties

Problem
In the Dev Protocol, many projects exist as Properties. However, there are so many of them that it can be challenging to choose a project.
Solution
Rather than staking on a project directly, you stake on a creator which has all of his projects within their profile. This makes choosing staking easier for stakers and vastly lowers the amount of projects a staker has to look for.

This DIP was drafted by @calaber24p

I put forward this DIP and I would like to talk about it a bit and why I believe it is a solution to a current problem.
Right now we have over 158 pages worth of properties and more being added every day. Despite the amount of properties, there are a small amount of developers currently. New stakers and veteran stakers alike are finding it not only hard to find projects, but new people to support.
My solution, rather than having single projects staked, will put all of a single developer's projects under a profile that allows the staking of all at the same time. Essentially you support a developer rather than a single project. This makes staking the person you want to support much easier while also getting exposure to all that they are doing. This will come in handy as we start on boarding new developers, especially ones with many projects under their belt.
If a solution is not taken before many developers sign up in large amounts, we will have potentially thousands of pages to sort through for new projects, most of which will be buried and never seen. If you have any questions feel free to ask and I will respond!

I would like to add to this DIP.
There are two other major instance not discussed where "packaged assets" makes sense: teams and projects
Teams: OSS teams, for example cryptocurrency teams, should be able to stake their OSS assets and have the community stake in a pool with them. This creates sustainable income for the team rather than selling tokens or compromising on their vision due to having VCs.
Furthermore, with a lot of crypto teams reaching the end of their roadmap they will be handing off the project to Layer 2's and OSS devs who will update and expand product. Dev Protocol is an amazing way to make sure teams have a treasury to support these projects. The protocol can have a Dev Pool, when the team is gone, which allows the protocol's DAO to vote on how minted DEV are allocated. The Layer2 can have a Dev Pool where only they decide how to use minted DEV tokens.
Projects: Imagine if ETH 2.0, or other core development projects, can stake their project for the ecosystem to support. This would achieve the following:

Core team has more resources to build faster
Stakers receive APY
Everyone receive the benefits of ETH 2.0 being launch sooner

If packaged OSS assets are supported then I see no reason for teams not to join!

I'll share some assumptions:

Property is an ERC20 token (fixed supply) and can transfer the balance at will.
Creators or contributors etc. receive compensation based on their share of the Property.
Multiple assets (e.g. OSS) can be linked and unlinked to a Property.
This means that one Property can be shared by multiple people for a project team or a creator with multiple OSS.

Those are the specs that are already publicly available.
However, existing npm packages are based on old specs and therefore cannot be unlink.
If you can't unlink assets, the organization will not work properly.
But we have multiple ways to resolve this issue. (I'll add that later.)

Ok, looking forward to hearing about them

Here's an overview of Wrapped Property as an idea of mine.
Wrapped Property is an ERC20 token that can be pegged with multiple Properties(on Dev Protocol). The total supply of Wrapped Property starts from 0, and when a property is transferred to a Wrapped Property, the same amount of Wrapped Property will be minted.

The initial total supply of all Wrapped Properties is 0.
Transferring a Property to a Wrapped Property will mint the same amount of Wrapped Property. (Eg, if you transfer 100 Property "Avalon" to Wrapped Property "Mamoru," you can get 100 "Mamoru-Avalon.")
Wrapped Property holders get rewards for the original Property, depending on their share. (Eg, when you have 100 Mamoru-Avalon, you get the same reward as someone who has 100 Avalon)
You can stake into a Wrapped Property contract. (Eg, staking to "Mamoru")
Wrapped Property's creator rewards will be received according to a Wrapped Property's share. (Eg, if creator reward of Mamoru is 100 and the total supply of Mamoru is 1000, who hold 500 Mamoru-Avalon will get 50 creator rewards)
An author of Wrapped Property must approve property that can be pegged to Wrapped Property. This is a way to prevent unintended pegs. (Eg, Mamoru's author, pre-approves Property "Avalon")

Wrapped Properties support flexible organization. A Wrapped Property can create multiple Properties and pegs, and a Property can create multiple Wrapped Properties and pegs.
Creators can express themselves by their Wrapped Property. In addition, OSS or each works Property will also exist.
Creators can also create a Wrapped Property for their team instead of the creator itself. The presence of creators, teams, or properties for each work allows the staker to find the intended support.


Camtasia is at version 3.0.6

I don't understand how to update the casks myself, so here's an update to an existing cask. The version number on https://github.com/caskroom/homebrew-cask/blob/cbd7c946c5ed9282e57118faa17eaaab2f317427/Casks/camtasia.rb is 3.0.6 now.
Sorry I am unable to provide more information.

https://github.com/caskroom/homebrew-cask/blob/master/CONTRIBUTING.md#updating-a-cask

# install and setup script - only needed once
brew install vitorgalvao/tiny-scripts/cask-repair
cask-repair --help

# use to update <outdated_cask>
cask-repair <outdated_cask>

https://github.com/caskroom/homebrew-cask/pull/34829


How do you enforce non-contract calls?

Your docs state that

The account which locks the tokens cannot be a smart contract (because can be tradable and/or tokenized), unless it is one of whitelisted smart contracts (for example, widely used multi-signature wallets).

How do you want to enforce that though in a way that cannot be cheated by technical means (e.g. constructor calls)?

The current logic around restricting contracts is found here:
https://github.com/curvefi/curve-dao-contracts/blob/25a0c25e4e77ff493b977fb95a3cddd42df735e3/contracts/VotingEscrow.vy#L171-L182
smart_wallet_checker will compare the hash of the code at addr against a mapping of approved hashes, and revert if the contract is not approved.  Use of CREATE2/SELFDESTRUCT to make all interactions via constructor will still trigger this logic due to addr != tx.origin and ultimately revert because 0x00 won't be an approved hash.

tx.origin might not return meaningful values past Serenity. Depending on the timeline of your project and Serenity that might be an issue or not.

:+1: thanks for mentioning this - we've looked at it some more and ultimately decided that it's still the approach we want to take.
Closing this issue for now, if you want to continue discussion feel free to join us on Telegram or Discord.


[DOCS] List of available HF_TASK and default inference scripts

Hi, if for example I deploy the following:
from sagemaker.huggingface import HuggingFaceModel

# Hub Model configuration. https://huggingface.co/models
hub = {
  'HF_MODEL_ID':'distilbert-base-uncased-distilled-squad', # model_id from hf.co/models
  'HF_TASK':'question-answering' # NLP task you want to use for predictions
}

# create Hugging Face Model Class
huggingface_model = HuggingFaceModel(
   env=hub,
   role=role, # iam role with permissions to create an Endpoint
   transformers_version="4.26", # transformers version used
   pytorch_version="1.13", # pytorch version used
   py_version="py39", # python version of the DLC
)

Where can I find the list of available HF_TASK, and the default inference script for each task (if I don't provide a custom one)?

I ask because I'd like to deploy a text embedding model, which I believe is under the task feature-extraction, but I'm not quite sure, and also don't know what the inference script looks like for this.
For more context, I've deployed the hkunlp/instructor-large model with a custom inference script, but am wondering if that is needed or if I could have deployed it without a custom script.

The snippet always use the task the model is fine-tuned on. The distilbert-base-uncased-distilled-squad cannot be used for any other task.
For your example to run instructor models you need their pypi package.

Do I need custom inference code to run the instructor model? I got it to work using the below code in the style of your sagemaker/17_custom_inference_script/sagemaker-notebook.ipynb example:
from InstructorEmbedding import INSTRUCTOR
import torch
import torch.nn.functional as F

def model_fn(model_dir):
  model = INSTRUCTOR(
    model_name_or_path=model_dir,
    #device: Optional[str] = None,
    #cache_folder: Optional[str] = None,
    #use_auth_token: Union[bool, str, NoneType] = None,
  )
  print(f"model: {model}")
  print(f"model.device: {model.device}")
  return model

def predict_fn(data, model):
    # Get instruction
    instruction = data.pop("instruction", "Represent the query for retrieval: ")
    print(f"instruction: {instruction}")

    # Get sentences
    sentences = data.pop("sentences")
    print(f"sentences: {sentences}")

    # Pair instruction with each input sentence and encode
    with torch.no_grad():
        embeddings = model.encode([[instruction, sentence] for sentence in sentences])
    print(f"embeddings.shape: {embeddings.shape}")

    # Return dictonary, which will be json serializable
    return {"vectors": embeddings.tolist()}

However I'm curious if I needed to do that, or if there's a way to deploy it without custom code?

Thanks


can I improve the quality of the resulting video? And how to do this?

I want to receive videos in higher quality and resolution, what steps do I need to take to get the desired result?

Change the train_width, train_height, and fps in configs/train/stage2.yaml to your desired augments. Then finetune the model.

@DBDXSS For fine tuning, do I need to run only train stage 2?

I am not a developer of this project, so I won't be able to give you a definite conclusion. The results of the two methods cannot indicate which one is better. You can also use weights for initialization of the training stage 1, in fact. Since the facial features of your fine-tuned data and the author's training data may be different, I think it will be better to fine-tune from stage1. However, this training process will take a lot of time. You can also fine-tune directly from stage2. If the effect is not good enough, then try starting from stage1 again.


Streaming stopped working

Hi,
I'm using a FilteredStream with more than 900 twitter accounts, and it stopped working yesterday.
Now my MatchingTweetReceived event handler is never called... but I don't have any exceptions (SwallowWebExceptions is set to false).
How can I find what is the problem?

Have you verified that the stream is actually running?
Are you listening to the Stream.StreamStopped event to check if it is actually invoked.

Thanks! Got an exception in StreamStopped!
error details
"Malformed escape pair at index 516", do you know what it means?

Strange, now it is working again...

Hmmm, has the number of people you follow been reduced?

I did some tests by reducing the number of accounts to follow: 100 (did not work), 10 (dnw), 1 (it worked), then I reverted my code back to before, and it works again with 925 accounts now...

I know that there is a limit to the number of accounts you can add because C# HttpRequest length is limited. But I am pretty sure you should be able to ask for up to 1000 without any problem.
Please report back to me if you still encounter this problem.
Looking at the URL that you sent the problem is obvious and it seems that the query sent is not complete, as you can see it finishes with %2 which is the beginning of %2C which is the separator between the ids.
Please make sure you report here so that I can fix any problem.

My stream stopped working unexpectedly today, with this error in StreamStopped:

System.IO.IOException: Received an unexpected EOF or 0 bytes from the transport stream.
at System.Net.ConnectStream.Read(Byte[] buffer, Int32 offset, Int32 size)
at System.Net.Http.HttpClientHandler.WebExceptionWrapperStream.Read(Byte[] buffer, Int32 offset, Int32 count)
at System.IO.StreamReader.ReadBuffer()
at System.IO.StreamReader.ReadLine()
at System.Threading.Tasks.Task`1.InnerInvoke()
at System.Threading.Tasks.Task.Execute()

What could be the reason?

It is an expected issue that should be handled by Tweetinvi and not throw. A StreamStopped event should be raised. If bot this is a bug.


Fix looping style

with_items were all changed to loop. Amended less than @showalrm :D.
Closes #203

I can't believe I missed this behavior of apt, especially after we spent so much time trying to understand it initially. I don't think the loop | flatten syntax will help, as that seems to just flatten nested lists into a single-level list.
While we could probably change it to:
- name: Install packages
  apt:
    state: present
    name:
      - package1
      - package2

this might be the time to look at which of these should be moved to a named list in vars/main.yml and if we need to adopt a naming convention to prevent them from overwriting each other, as I believe they're all global. Perhaps just prefixing them with the role name.

@coledn please comment that you're finished and ready for review when you're finished and ready for review.

I believe I am done. Please look it over in case I made some mistakes.

LPT: hit Submit Review, don't just leave it as pending for an hour :(

I think that takes care of everything I've been able to spot. Unless somebody else confirms they've done this, I do want to let Ansible run the entire playbook before approving this for merging, especially since it touches almost everything.
Great job though, I can't believe you accepted our bait to take on this project. :)

I can confirm that the playbook ran without any issues for me. Definitely doesn't hurt to run it on another VM, though. I also didn't test the OEM role, but it looks fine.
@coledn this is fantastic work! Thanks for your patience and for doing such a great job. I looked through all our pull requests and this had the highest number of comments and discussion of any of our PRs so far :1st_place_medal:
This was a much more significant change than @ripleymj or I expected and it's awesome you got it knocked out so quickly. Thank you!

I ran a test on the VM myself as well and it had no issues.

I've hit an error twice now running the CS354 content: https://gist.github.com/ripleymj/f0da484c92165fcc2f736a54f2e5f3a4
I doubt it's related to your changes, but I'd like to try to understand it better before merging.

Just fixed it, try to see if you get errors now.

@ripleymj

I was able to run all class targets here, and the robot error didn't show up.
@showalrm, you added yourself as a reviewer - don't disappoint @coledn now.

I think that was an unintended consequence of joining the conversation earlier.
The changes to the code look good and when I did testing everything went smoothly. Great job @coledn


improving the speed of  to_csv

Hello,
I dont know if that is possible, but it would great to find a way to speed up the to_csv method in Pandas.
In my admittedly large dataframe with 20 million observations and 50 variables, it takes literally hours to export the data to a csv file.
Reading the csv  in Pandas is much faster though. I wonder what is the bottleneck here and what can be done to improve the data transfer.
Csv files are ubiquitous, and a great way to share data (without being too nerdy with hdf5 and other subtleties). What do you think?

duplicate of #3186
Well using 1/10 the rows, about 800MB in memory
In [5]: df = DataFrame(np.random.randn(2000000,50))

In [6]: df.memory_usage().sum()
Out[6]: 800000072

In [7]: df.memory_usage().sum()/1000000
Out[7]: 800

In [8]: %time df.to_csv('test.csv')
CPU times: user 2min 53s, sys: 3.71 s, total: 2min 56s
Wall time: 2min 57s

about 8.5MB/sec in raw throughput, way below IO speeds, so obviously quite some room to improve.
you might be interested in this blog here
Of course there IS really no reason at all to use CSV unless you are forced.
In [11]: %time df.to_hdf('test.hdf','df')
CPU times: user 10.5 ms, sys: 588 ms, total: 599 ms
Wall time: 2.22 s

This is also part of the documenation: http://pandas.pydata.org/pandas-docs/stable/io.html#performance-considerations

hi @jreback thanks for your help!
unfortunately

yes i am forced
i have mixed types in my columns and somehow to hdf fails

what do you mean by mixed types?
you should NEVER have mixed types (e.g. python objects). that is a big big no-no.

show a df.info(); you can pass format='table' which is somewhat slower but much more flexible format for writing HDF5

I will run the to hdf line tomorrow and tell you my error message. it may be due to the fact that i have string columns and or some miscoded observations...

yes thanks i will run the df info. keep in touch. Thanks jeff

good morning jeff (@jreback))
Please find my df.info
df.info()
<class 'pandas.core.frame.DataFrame'>
Int64Index: 10762587 entries, 0 to 12864511
Columns: 275 entries, bagent_name to index_month
dtypes: bool(1), datetime64[ns](16), float64(30), int32(1), int64(172), object(53), timedelta64[ns](2)
memory usage: 22.0+ GB

Do you see anything wrong? I have many string columns here

and this is what I get when I try
df.to_hdf(path + 'main.h5','stored',format='table',mode='w')
TypeError: Cannot serialize the column [zip_dw] because
its data contents are [mixed-integer-float] object dtype

So you probably have something like this. Its actually pretty hard to construct this. If you are not doing this explicity then pls show how its contructed as this should always be converted to a concrete dtype.
You NEVER want object dtypes, except if they are strings. Yes you can use them, but they are completely non-performant and generally cannot be serialized. These are by definition python objects.
In [7]: df = DataFrame({'A' : [1,2,3], 'B' : Series([1,2.0,3],dtype=object)})

In [8]: df
Out[8]:
   A  B
0  1  1
1  2  2
2  3  3

In [9]: df.dtypes
Out[9]:
A     int64
B    object
dtype: object

In [10]: df.to_hdf('test.h5','df',format='fixed')
pandas/core/generic.py:1096: PerformanceWarning:
your performance may suffer as PyTables will pickle object types that it cannot
map directly to c-types [inferred_type->mixed-integer-float,key->block1_values] [items->['B']]

  return pytables.to_hdf(path_or_buf, key, self, **kwargs)

In [11]: df.to_hdf('test.h5','df',format='table')
TypeError: Cannot serialize the column [B] because
its data contents are [mixed-integer-float] object dtype

Hi Jeff, Yes I think you are right. zip_dw is a variable that contains a zipcode.
In the data, it looks like 95966 but the dtype is object.
The question is:

is there something in Pandas to force a conversion over these object columns?
i thought hdf5 could handle strings and fast-searching them. Am I wrong?

Thanks again @jreback

of course it can handle strings, but it makes them fixed width. But that's not your issue. you have actual Python objects (an actual integer and a float object), that are NOT in numpy (e.g. that's why the column is object).
 So you need you either need to stringify them, or you can leave them as object and use only the fixed format (it will work, but it pickles the data).
@randomgambit you have to understand and be cognizant of your data and types. Pandas provides lots of tools and mostly does not allow you to shoot yourself, but it cannot do 'everything' even though it will try very hard to infer things.

thanks @jreback  and sorry if I bother you with such basic questions. I come from a language (Stata) that is much less flexible and where everything is either a float or a string.
To recap, the problem here is that I have a column that contains some numbers (floats and integers) and some strings. This is why pandas treat them as objects. You are saying that this mixed-type column generates performance issues and cannot be well stored in hdf5. To fix this, I should probably either use to_numeric or astype('str') on this column, in order to have a single dtype for every value in that column.
Is this correct?

Following our chat, I have bought a book about hdf5 and python. that will help me understand this storage better. @jreback , if you can just tell me if my reasoning above is correct that would help. Thanks and keep up the great work with Pandas!

so you would typically do something like to this. pandas support many dtypes; you want to type as much as possible. All of these types are supported when serialized to HDF5 (table format), though for example strings become fixed width and nans are replaced with a string.
``
In [31]: s = Series(['a',1,2.0])
In [32]: s
Out[32]:
0    a
1    1
2    2
dtype: object
In [33]: pd.to_numeric(s, errors='coerce')
Out[33]:
0    NaN
1    1.0
2    2.0
dtype: float64

So how about an answer to the initial question rather than getting off track.
I need cvs format
Can pandas improve speed of writing that file format?

I've managed to reduce write time by 90% using pyarrow to write the pandas data frame:
out = pa.Table.from_pandas(out_pd)
del out_pd
csv.write_csv(out, out_file)


Polymorph doesn't affect players

Polymorph doesnt polymorph players but triggers morphing enemies very often while they are standing in it.

Has been implemented in 0.12.0


How to search for specific tags within TEI? Struggling with starting point for encoding.

What is the best angle of approach to take to find the correct tags I need to begin encoding my information.
 for example, I want to take each question, and separate them from the answer. First I tried to wrap them in a division and it flagged wrong (although the error text had div included within the "Tags must be one of:,,") So I tried to search for which to use, Author doesn't seem correct, and now I'm stuck sifting through endless pages of tags with no real goal in mind, it's severely hurting my workflow. I have no problem with XML, I can make my own schema, I can make some clean XML mark, but when it comes to trying to follow someone else's rules I'm falling flat.

@AtomicOlsen TEI is long and slow work, and sometimes you stumble into a pretty good combination.
You can definitely work with <div> elements: I don't know what error you were seeing, but if you're stuck like that, it helps to document the error and show that specific code you tried. (Paste the into the message, so we can try it ourselves and try to fix it!) You can try making a code block here in the issue, too. The screen capture here doesn't show the error.
Here is one that you might want to try out, the <ab> element:
https://tei-c.org/release/doc/tei-p5-doc/en/html/ref-ab.html
Notice that this can include an <add> element that could be helpful.


⚠ Bofrost.es has degraded performance

In c1eac3a, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1705 ms

Resolved: Bofrost.es performance has improved in de71b3f after 6 minutes.


Replace mktemp with mkdtemp

Fix build warning complaining about dangerous use of mktemp.

Fixed the merge conflict.


Bitwise Operator to Check Odd, Regular Expression to Check Even

Before jumping into Pr first comment for assign.All Problem from Edabit. Link is mandatory to add
**Problem statement: Create two functions:

The first is is_odd() to check if a given number is odd using a bitwise operator.

The second is is_even() to check if a given input is even using regular expressions.**

**Problem Link: https://edabit.com/challenge/QEL3QAY9ZhSkZcEih **

Before submitting a PR please Check some details.Check Edabit for problem task

[x] fileName formet should be in camelcase
example- returnTheNextNumber.py
[x]  Add label for Problem
[x]  In file there should be problem statement and link to probelm
example -

'''
    Probem Task : This program will add two numbers
    Problem Link : https://edabit.com/challenge/SFzHtm63XT6EYNHWY
'''
*/
def main():
    return a + b

This repository is valid for HacktoberFest2020
Read before making PR - How to contribute on Github
Good Luck,

Assign me this issue @aayushi-droid

Can i do it . Can u assign it to me

Can u assign me?


memory_usage_test fail under mac

D0711 10:54:11.677362000<PHONE_NUMBER>50816 ev_posix.cc:174]              Using polling engine: poll
D0711 10:54:11.678205000<PHONE_NUMBER>50816 dns_resolver_ares.cc:485]     Using ares dns resolver
D0711 10:54:11.692659000<PHONE_NUMBER>50816 test_config.cc:384]           test slowdown factor: sanitizer=1, fixture=1, poller=1, total=1
D0711 10:54:11.692816000<PHONE_NUMBER>50816 test_config.cc:384]           test slowdown factor: sanitizer=1, fixture=1, poller=1, total=1
D0711 10:54:11.693971000<PHONE_NUMBER>50816 ev_posix.cc:174]              Using polling engine: poll
D0711 10:54:11.693988000<PHONE_NUMBER>50816 ev_posix.cc:174]              Using polling engine: poll
D0711 10:54:11.694074000<PHONE_NUMBER>50816 dns_resolver_ares.cc:485]     Using ares dns resolver
D0711 10:54:11.694077000<PHONE_NUMBER>50816 dns_resolver_ares.cc:485]     Using ares dns resolver
I0711 10:54:11.694176000<PHONE_NUMBER>50816 server.cc:180]                creating server on: [::]:27749
I0711 10:54:11.696370000<PHONE_NUMBER>50816 subchannel.cc:1076]           New connected subchannel at 0x7ff5afe06160 for subchannel 0x7ff5afe04760
memory_usage_client(23863,0x7fff96ef4380) malloc: *** error for object 0x7ff5afe02150: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug

Fixed memory_usage_test failure under Linux by #19811. Not sure if it fixed the situation under Mac since there is another test failed #19819 .


Scale to a given window instance

Allowed to pass in a specific window instance to scale to.

Coverage remained the same at 92.03% when pulling e93e58c46309ddaf878c27da358314a1465ba32d on jon/scale-to-specified-window into 6af26c599ca43e8dc6432495ff2981fa620e140d on master.


Per bug 1377566, OpenRaster format improvements, and OpenDocument standard.

includes fix for opacity/visibilty;
enhancements: selected layer; mergedimage; OpenDocument compatibility;

I don't expect I'll have time to look at this again before the weekend (definitely not time to rewrite anything, maybe time to read and comment here, and read the code maybe).
Thanks for your comments.

OK, no problem. I think I've reviewed everything except for the .ora format changes, so I'll try to look at those by the weekend.

Is this merged anytime?

I've merged the .ora bug fixes in dcb5c7af and 632c5660


Reload when exiting resource pack selection screen

When opening and then closing the resource pack selection screen (ESC -> Options -> Resource Packs...) after a server has (un-)applied a resource pack the resources are reloaded.

fixed in 1.20.5 rc1


Various Improvement Suggestions

Just thought I'd dump out a list, I know this is not how issues usually works, but just for posterity.

Introduce a faster game mode similar to blitz where turns are fast, but units move regular time or ~1.2s (the fast move is quite jarring, I assume it's 2x) with a better feedback mechanism of time remaining (a reducing progress bar). Initial gut feeling is like 10-30s turns, without losing if you miss a turn.
With this, the games will be more interactive.

Or improve blitz a bit, the timing is a bit odd, it's a bit too fast where people lose turns and games from not moving fast enough. The result should be to prompt high interactivity, not unnecessarily punish the user. Example, give extra time once they move to attack. Don't make them forfeit after missing turn, allow more missed turns before forfeit. (I could be wrong about it forcing forfeit here).

Better turn indicator. Example, having a stronger indicator by not having the golden banner for the player who's turn it is not, but IMO from a UX Perspective, it should be even stronger of an indicator -- I would say my gut feeling is that the action panel on the top right should have an indicator there.
Some better navigation -- example after finishing a game, you have to use browser navigations, but adding a back button or go to lobby.
Also when clicking profile, after finishing, the user navigation state resets -- example if you were browsing lobby.
Maybe when you go to tactics homepage, if you're logged in, go straight to lobby without having to press enter, or show the lobby for new players.
HotKeys: press tab/shift+tab to cycle through the attack/move/position, and maybe 1/2/3/4 or some letters corresponding as well. Pressing something like / to initiate chat.
Separate mode where you have to earn units, opening them with some kind of points, and RNG for getting units to add some gamification, and stickiness for new users.
Public chat on screen / maybe chat tab, integrate discord general channel?
Full screen - can use try to integrate with browser zoom or css transform()
Figuring out some kind of cashflow, even if it's super small and simple, to start advertising for a higher population of player-base. Example can be purchasing additional unit credits from the gamification idea. Or simply donations to start, giving some sort of OG / Founder badge to players and or meaningfully special avatar.

Many more ideas :)

Sample for indicator improvements:


strange outlines appear around logos on page footer

I've tried fixing this in the browser, but it seems to be a subtle issue - maybe a combination of an error in the sprite viewport definitions or some inherited CSS attributes? The fact that the outlines aren't rectangular is particularly strange.

I won't say anything, guys, hahaha


ui: backoffice full height sidebar

backoffice: re-layout to have sidebar full height + removed not used top header

@kprzerwa @topless Since I have changed some css of the layout, I hope I did not break anything :) Everything looked ok to me... Otherwise, I am sorry! :)


[TypeScript] Required properties should be on top of suggest

interface Inter {
	a: string;
	b?: string;
	c: string;
	d?: string;
}

const obj: Inter = {
	|
};

Invoke suggest at |.
Actual (Insiders):
a
b?
c
d?

Expected (Stable):
a
c
b?
d?

VS Code version: Code - Insiders 1.31.0-insider (ece3a90fb58986570d8afdfb16530c4dd8c743af, 2018-12-14T06:19:42.976Z)
OS version: Windows_NT x64 10.0.17763

Actually the same issue exists in Stable if they are in multiple files:
1.ts
export interface Inter {
	a: string;
	b?: string;
	c: string;
	d?: string;
}

2.ts
import { Inter } from '1';

const obj: Inter = {
	|
}

@mjbvz, not sure if this issue slipped inbox or not since it doesn't have labels...

Closing this as out of scope since it hasn't had additional feedback since being opened.
I personally think that the current ordering is more consistent and better matches how intellisense works for other contexts / langauges


LL3633 black friday banner

add back black friday banner
Type
UI
Context
https://ledgerhq.atlassian.net/browse/LL-3633
Parts of the app affected / Test plan
check black friday banner. You can change the local date to check the conditional display

@machard why is it in HODL?
(also: conflicts)


FEAT: Anthropic Claude support

Any interest in supporting Anthropic's Claude model? The 100k token context window opens a lot of possibilities.
If so I might take a stab at it and send a PR.
Also, love how configurable your extension is.

yes! I absolutely plan to support Claude, just not quite there yet. if you get to it before I do, that's totally cool!

I stubbed out support for Claude using the new Provider API, and the OpenAIProvider can serve as an example for how to implement a new provider (if you are interested):

Provider: https://github.com/nvms/wingman/blob/8817a200a0fa674e066ec2519e96bdb639cd10a2/src/providers/index.ts
OpenAIProvider: https://github.com/nvms/wingman/blob/8817a200a0fa674e066ec2519e96bdb639cd10a2/src/providers/openai.ts

I still don't have access to Claude yet, so apart from the above, I haven't been able to really dig into this yet

@nvms I've started taking an initial look at this. Full disclosure: I have exactly zero experience with JS/TS or vscode extensions so I'm kind of learning it and the relevant tooling on the go.
The main thing that's blocking me is the fact that the anthropic/claude API has no counterpart to the OpenAI conversation tracking stuff. It's much simpler/more limited. Each request exists on its own and is basically just a single blob of text with alternating User: ... and Assistant: ... blocks. So, for a conversation I'd basically need to manually build the next prompt by keeping the current state somewhere and appending new questions and responses at each step.
It's not clear to me where this state should live, or when/how to clear it.

no counterpart to the OpenAI conversation tracking stuff

ah, I wasn't aware of this. I think your method of preserving state is exactly what you'll want to do, then.
maybe something like this?:

in create, set state to an empty string (e.g. this.conversationState = "")
when a complete response is received, concatenate everything onto this string (e.g. this.conversationState = this.conversationState + "\n\nHuman: <last question>\n\nAssistant: <last reply>"
the next time you send a prompt, prefix it with conversationState.
the place to clear this state would be the Provider's destroy method, which is called before every new conversation: https://github.com/nvms/wingman/blob/b93e22b2a431e5f3196926a814d5feceb6a225c1/src/views.ts#L202-L204


pypi install of v0.10.0 results in "No module named 'tensorrt'"

System Info
Debian 11
2 L4 GPUS
CUDA 12.4
Who can help?
No response
Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
pip install tensorrt-llm==0.9.0 -U --extra-index-url https://pypi.nvidia.com
trtllm-build --version
Results in printing: "[TensorRT-LLM] TensorRT-LLM version: 0.9.0"
pip install tensorrt-llm==0.10.0 -U --extra-index-url https://pypi.nvidia.com
trtllm-build --version
Results in:
Traceback (most recent call last):
  File "~/bin/trtllm-build", line 5, in <module>
    from tensorrt_llm.commands.build import main
  File "~/lib/python3.10/site-packages/tensorrt_llm/__init__.py", line 32, in <module>
    import tensorrt_llm.functional as functional
  File ".../python3.10/site-packages/tensorrt_llm/functional.py", line 25, in <module>
    import tensorrt as trt
ModuleNotFoundError: No module named 'tensorrt'

Expected behavior
I expect that the install of tensorrt-llm installs the required dependencies of tensorrt
actual behavior
import of tensorrt within v0.10.0 leads to an exception in the CLI
additional notes
NA

Please try the container: nvcr.io/nvidia/tritonserver:24.05-trtllm-python-py3
Or build the container using  make -C docker release_build

you can use pip install tensorrt==10.0.1 --force-reinstall


chore: make IDEA support IssueNavigationLink and add icon

Purpose of the PR
as title, similar: https://github.com/apache/incubator-hugegraph-toolchain/pull/600
before:

after:

Main Changes

Verifying these changes

[x] Trivial rework / code cleanup without any test coverage. (No Need)
[ ] Already covered by existing tests, such as (please modify tests here).
[ ] Need tests and can be verified as follows:

xxx

Does this PR potentially affect the following parts?

[ ]  Dependencies (add/update license info & regenerate_dependencies)
[ ]  Modify configurations
[ ]  The public API
[ ]  Other affects (typed here)
[x]  Nope

Documentation Status

[ ]  Doc - TODO
[ ]  Doc - Done
[x]  Doc - No Need

Perhaps the license header comment in XML files should not include ~ (to maintain consistency with other XML files), does https://github.com/apache/incubator-hugegraph-toolchain/pull/600 also need to be modified for that?

@liuxiaocs7 Perhaps the license header comment in XML files should not include ~ (to maintain consistency with other XML files), does apache/incubator-hugegraph-toolchain#600 also need to be modified for that?

Thanks, Done. yes, i'll change it later.


User::loginRequired with subdomain

I've got a Module with some UrlRule's to work with a subdomain.
e.g.
site.com     -> DefaultController
sub.site.com -> Module/DefaultController

When going to sub.site.com it will redirect to site.com/login which works fine.
The issue is the call to User::loginRequired() only sets the relative part of the url so after logging in you get redirected back to site.com and not sub.site.com
This could be fixed by changing this line User.php#L446 to use the absolute url, not sure if this would cause any BC issues.

Q
A

Yii version
2.0.22

Should not cause any issues. Redirecting not back but to main domain does not make any sense.

@alex-code do you want to do a pull request fixing it?

Yeh, I'll do a PR.
Switching from $request->getUrl() to $request->getAbsoluteUrl() should be all it needs.