CVE-2022-29256 - moderate severity - Possible vulnerability at 'npm install' time if an attacker has control over build environment

There's a possible vulnerability in logic that is run only at npm install time when installing versions of sharp prior to the latest v0.30.5.
An attacker would need to be able to set environment variables in your build environment to take advantage, so this is unlikely to affect most people. This does not affect sharp at runtime. Please see https://github.com/lovell/sharp/security/advisories/GHSA-gp95-ppv5-3jc5 for full details.
Please upgrade to the latest v0.30.5 of sharp where possible. You might be able to run npm audit fix to do this.
If you are using another package which depends on a version of sharp that is not the latest, please open an issue against that package.
If all the above fails, and as a temporary workaround, add the following to your package.json file and run npm install or yarn again.
{
  "overrides": {
    "sharp": "^0.30.5"
  },
  "resolutions": {
    "sharp": "^0.30.5"
  }
}

I don't know what 0.30.5 updated, but I can't install 0.30.5

| gyp info it worked if it ends with ok
│ gyp info using<EMAIL_ADDRESS>│ gyp info using<EMAIL_ADDRESS>| win32 | x64
│ gyp ERR! find Python
│ gyp ERR! find Python Python is not set from command line or npm configuration
│ gyp ERR! find Python Python is not set from environment variable PYTHON
│ gyp ERR! find Python checking if "python3" can be used
...

So I still use 0.30.4 now

If you have a problem at install time, please open a new installation issue and answer all of the questions.
https://github.com/lovell/sharp/issues/new?labels=installation&template=installation.md

Closing as this is now 90 days old. Hopefully everyone that had to deal with alerts about this has now had a chance to upgrade.
Remember: if an attacker has control over environment variables in your build environment then you have a bigger problem to deal with than this issue.


Kafka connect headings and nav entry updates

What changed, and why it matters
This request came from Developer Marketing to improve the consistency and discoverability of our excellent connector documentation

@argkastan could you also take a look? You don't seem to be a member of the github org so I can't add you as a reviewer.

Does somebody need to invite me for that?

@petralylander @argkastan if you're both happy I can merge the change, could you please confirm?

@petralylander @argkastan if you're both happy I can merge the change, could you please confirm?

Everything seems ok @ftisiot !
Thank you for the changes

Thanks @lornajane for the changes!

Confirmed, approved and ready to go, yes!


Content centering problem

Hello there,
I use the dynamic filtering network structure. However, I cannot center the content when the page loads. It appears on the top left as in the screenshot below. Can you help me put this in the middle of the screen?
I did it using the example in this link.

I have no idea what your doing. The example you provided works just fine but doesn't look any similar to the screenshots.


Compositing could be much faster for examples with lots of layers

We've seen @abarth's flow demo run slowly on some hardware. Rasterizer thread long pole being ~7ms in LayerTree::Paint, and Engine thread being 1ms Paint, but 5-10ms in Compositing.  @abarth says compositing could be 2x - 4x faster if made incremental.

Some progress in https://github.com/flutter/flutter/commit/0f3bd3ae842e8b287152140aca8d641aca0c035a
Also experimenting in https://github.com/flutter/engine/compare/master...abarth:encoded_buffer?expand=1

@abarth Is this still something you're looking at, or should we close this until we run into an actual issue?

Let's close this issue.  We did culling instead, and that seems to have addressed the cases that were causing us trouble.


Chronograph exits with "Unable to open boltdb; is there a chronograf already running?"

System: Mac OS X Sierra (10.12.6)
Docker version:

Version 17.06.0-ce-mac19 (18663)
Channel: stable
c98c1c25e0

Steps to reproduce:

Clone the repo (SHA: de769deaa506bf959b7c56a553cb1f0b83b2d0e6)
./sandbox up
./sandbox logs chronograf

This is the error message in the logs:
chronograf_1     | time="2017-08-05T03:37:55Z" level=error msg="Unable to open boltdb; is there a chronograf already running?  invalid argument" component=boltstore
sandbox_chronograf_1 exited with code 1

I can see that the other containers started successfully, but chronograf is missing:
CONTAINER ID        IMAGE                   COMMAND                  CREATED             STATUS              PORTS                          NAMES
fe71b439589f        telegraf:1.3.1          "/entrypoint.sh te..."   2 minutes ago       Up 2 minutes        8092/udp, 8125/udp, 8094/tcp   sandbox_telegraf_1
a8a506fe50ea        influxdb:1.2.4          "/entrypoint.sh in..."   2 minutes ago       Up 2 minutes        <IP_ADDRESS>:8086->8086/tcp         sandbox_influxdb_1
8e44e24b6935        sandbox_documentation   "/documentation/do..."   2 minutes ago       Up 2 minutes        <IP_ADDRESS>:3000->3000/tcp         sandbox_documentation_1

Never mind, I see that this is the same issue as https://github.com/influxdata/sandbox/issues/1
After I installed the latest version of Docker, I tried the above commands in a terminal tab opened before the install, so apparently it was using the older version of Docker.  Everything works after I open a new tab 🤦♂

Hi,
we upgraded to chronograf 1.8.0 and when we bring our chronograf pod up, we get an error "Unable to create bolt clientUnable to open boltdb; is there a chronograf already running ? timeout" error.  I would appreciate any help in resolving this..
Thanks,
-SR

@rsreeni
Hi, in my case the previous version 1.7.14 was actually still running, after upgrading to 1.8
Killing it and restarting the service fixed the issue :)


Paste on new line when "vim.useSystemClipboard" has "paste"

When "vim.useSystemClipboard" has "paste" option, when you copy/paste a string (without end of line), this is pasted on a new line. This behaviour differs with classic vim.

i believe that's a duplicate of #660

Indeed. Thanks @pkariz!


Fix coping module-chart dir

Description
Changes proposed in this pull request:

Fix coping module-chart dir

Related issue(s)
https://github.com/kyma-project/serverless-manager/issues/344

/test pre-serverless-operator-build


Add deprecation notice to readme

Simple change, but let's hold to merge this until we get a first round of customer reaction to the new tool.


not predicting trajectory for every agent; validation data

I trained the model and have checkpoints. I make prediction by running test.py in tools folder. I checked the config file and it looks like prediction is made on validation data. Running test.py creates output folder which contains result.pkl file. But the issue I have is, this result.pkl file does not contain predicted trajectory for all the agents that are in validation data for a particular scenario. In one scenario of validation data, I have 83 agents but for the same scenario, in result.pkl file there are only 3 agents with trajectory. How can I get trajectory for all agents?

Have you pull the newest commit? Previous codes seem to have this issue, but the newest commit changes the data structure of result.pkl, which should have solve this.

I also encountered this problem and it seems still not been solved.

I figured it out. At the end of tfrecord file, there is field called tracks_to_predict which specifies the index of tracks to make prediction on. In my case, tracks_to_predict field had index of only 3 agents. I inserted more index in this field to do more predictions.


Immortal 2 icon is wider than normal

The small icon for immortal 2 is (69x64) while all others are (64x64).
this also affects the large icon (275×256) instead of (256×256).

At first glance what I can see is that the image is wider than it is tall due to the shadow of the side stars.
Could you please provide me some other URLs of images with a nominal size?

At first glance what I can see is that the image is wider than it is tall due to the shadow of the side stars.
Could you please provide me some other URLs of images with a nominal size?

https://media.valorant-api.com/competitivetiers/03621f52-342b-cf4e-4f86-9350a49c6d04/21/smallicon.png Here is Ascendant 1 small icon for example


[minor feature] compile option to deactivate autoupdate

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.07.18 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
autoupdate should be deactivated when yt-dlp is distributed with package managers and app stores. The likes of, linux distros, pip,, chocolety, brew etc... Currently, packagers either just leave it in, or they faf around with patches that periodically break.
A compile option should deactivate the option, and also remove it from the manual.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Feel free to make a PR for this

Related https://github.com/yt-dlp/yt-dlp/issues/4134

https://github.com/yt-dlp/yt-dlp/issues/4134#issuecomment-1162532840 is the currently recommended solution. While I think it is a sufficient solution, I can understand concerns of merge-conflicts for such a patch.
Are you a package maintainer? While I could implement something like this, I don't know what kind of implementation someone packaging actually needs - and rather than guessing, would like to hear from people who actually build the ports (cc @yurivict)
My initial idea is to implement a VARIANT variable in version.py which can be used to override yt-dlp's internal variant detection. So for example, someone can set VARIANT = 'apt' and the auto-updater will say:
[debug] Command-line config: ['--ignore-config', '-vU']
[debug] Encodings: locale cp65001, fs utf-8, pref cp65001, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.01.01 [f640e42ff] (apt*)
[debug] Lazy loading extractors is disabled
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Python 3.10.5 (CPython 64bit) - Windows-10-10.0.22000-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg N-106550-g072101bd52-20220410 (fdk,setts), ffprobe N-106624-g391ce570c8-20220415, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.14.1, brotli-1.0.9, certifi-2021.10.08, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.1
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.07.18, Current version: 2022.01.01
[debug] Downloading _update_spec from https://github.com/yt-dlp/yt-dlp/releases/download/2022.07.18/_update_spec
ERROR: It looks like you installed yt-dlp with a package manager or setup.py; Use that to update

Is this sufficient? I can provide the ability to change this variant as part of devscripts/update-version.py. Or, you can always manually change it in version.py by any means you see fit
With an appropriate PR, we could also improve the Makefile to set the variant from the environment
PS: Once an implementation is finalized, I would be very reluctant to change it even if better ideas pop up

Does it block rclone selfupdate somehow, assuming rclone was installed with any package manager? If not, I don't see the point

I added the recommended patch and it is fine IMO.
When a new version is found it recommends to use the package manager to update it.

i'm not a maintainer, i just noticed that packagers are chasing their tails with autoupdaters

Read https://github.com/yt-dlp/yt-dlp/blob/master/CONTRIBUTING.md#is-anyone-going-to-need-the-feature. If this was an actual issue for package maintainers, I expect them to request this. The only person who has ever asked for something like this (yurivict) is satisfied by the current solution. There is no point in working on features that nobody will use!
I too use scoop and apt for packages and both of them handle yt-dlp's self-update gracefully. In case of scoop, the version number scoop shows will go out of sync when using -U, but this is not a big deal.

(this is especially bad with chocolatey).

As I remember, chocolatey just uses our pre-built binaries. In that case, there is absolutely nothing I can do to for this issue. Any solution I can provide require that the maintainer be willing to build the executables themselves - which I doubt will happen for choco. If any maintainer has shown interest in doing this, pls give me link to any relevant conversation (or, you can invite them to discuss under this issue)

you should talk to packagers of several systems. What they are doing can vary.

I am not going to go around and DM package maintainers. If there are people you know are facing this issue that you claim exist, feel free to point them here so that we can discuss the matter

IMO you should add a build-time option that would do the same that this patch from the FreeBSD port does.

That is basically what I proposed in https://github.com/yt-dlp/yt-dlp/issues/4471#issuecomment-1198266075

That package how ever is on the responsibility of the user to update it.

On FreeBSD packages aren't a responsibility of users. They are rebuilt centrally, and the user only needs to run pkg upgrade to upgrade all of them every once in a while.

Wdym "removal"? You want -U to do nothing, or to completely remove that option? For former, read my above comment again. The latter will not be done

Re-read https://github.com/yt-dlp/yt-dlp/issues/4471#issuecomment-1198266075

NO! Why don't read what I actually wrote in the above comments? I am not going to implement this just because you think it may be helpful when you don't plan to use the feature yourself. https://github.com/yt-dlp/yt-dlp/issues/4471#issuecomment-1198443486
If you actually value this issue, instead of asking me to "just do that", try and bring people who'd actually benefit from this into the conversation. I didn't immediately close the issue because I thought this could be used to start a discussion on the matter. But if it is turning into a useless back and forth, I will close and lock it

I'm a little late to the party and only happened upon this bug report by chance, but I'm the Debian maintainer of yt-dlp.  Until recently the patch included in Debian also removed the update feature and updated the help text, now it simply does the latter.  I'd looked into the feature and noticed the update check bails if it detects yt-dlp was installed by a package manager, so dropped the removal of that but left in where the patch disables the help text so we don't advertise the feature since it doesn't actually update.
If you were  to include the VARIANT style of thing as mentioned above, it'd be neat if the help text could be updated to reflect that.  Informing the user of backport channels could be neat as well, but this doesn't sound like the job of upstream to me. :)

If you were to include the VARIANT style of thing as mentioned above, it'd be neat if the help text could be updated to reflect that.

-U still checks if the user is in the latest yt-dlp version. So instead of removing the helptext, I could modify it to reflect that. Something like:
-U, --update    Report the latest version of this program

Yep!  Sounds great to me.  Noting it'll update if in standalone mode or not installed via a package manager would be fine too.  Thanks for considering it, would love to drop that patch entirely.

Done. Example usage: https://github.com/yt-dlp/yt-dlp/blob/70b2340909d8d917f71d20181614fd7392d3f7f0/.github/workflows/build.yml#L92
The variant name will appear in --verbose head and the message (-M) will appear in both the helptext of -U and in the error message. Like this:
❯ yt-dlp -vU
[debug] Command-line config: ['--ignore-config', '-vU']
[debug] Encodings: locale cp65001, fs utf-8, pref cp65001, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.07.18 [135f05ef6] (pip)
...
ERROR: You installed yt-dlp with pip or using the wheel from PyPi; Use that to update

-U, --update                                        Check if updates are available. You installed yt-dlp with pip or using the wheel from PyPi; Use that to update

Setting a custom message is optional. As long as a variant is set, the update will be blocked and a generic error message is shown

Thanks!  Updated, using that option too.


docs: confirmCardPayment with split card elements needs a docs update

The documentation on pretty much all the react cases assume that you have a single CardElement and therefore specify the confirmCardPayment() call as follows:
    const result = await stripe.confirmCardPayment(clientSecret, {
      payment_method: {
        card: elements.getElement(CardElement),
        billing_details: {
          name: `${firstName} ${lastName}`
        }
      }
    })

However there's the possibility to have them separated and the docs aren't clear what to do in this case as you have no CardElement in this case.
In my opinion there should be some mention (probably in confirm_card_payment docs) on how to do this, which is:
    const result = await stripe.confirmCardPayment(clientSecret, {
      payment_method: {
        card: elements.getElement(CardNumberElement),
        billing_details: {
          name: `${firstName} ${lastName}`
        }
      }
    })

Which is cool - the library seems to be get the other 2-3 elements on it's own, which is great 👍

Hi @mweibel and thanks for reporting this confusion! We've added a note to the reference docs for the card field:

Uses the provided card or cardNumber Element to use for confirmation.


How to use jekyll with staticman on a server?

What if do not want to use Github Pages, then to make staticman work with Jekyll on a server?

You don't need to use GitHub Pages at all, the only requirement for Staticman to work is that you have a GitHub repository. The service receives form requests and pushes them as flat files to your repo. What you then do with the contents of that repo — whether you serve it with GitHub Pages, Netlify or any other way —  is entirely up to you.

If the repo is on GitHub Pages, then how we can access it on any other server?


drivers: sensor: add INA700 power and current sensor driver

Texas Instruments INA700 power and current sensor driver.
Based on existing drivers for INA230 and others.
Currently supports both trigger and non-trigger configurations, but trigger configuration only enabled for conversion ready interrupt and only using system workqueue.
Tested on out-of-tree custom board.

@jilaypandya thanks for the review, I appreciate it. I believe I have addressed your comments. Also had to rebase because of a conflict with a PR that was merged in the meantime.


Drop Shadow on an image

Hello,
I'm loving using the library, it's really awesome.
I was just wondering if there was a simple way to access the shadow attribute for a picture.
I've tried:
document.add_picture('images/image.png', width=Inches(5.25),shadow='Drop Shadow Rectangle')
and
document.add_picture('images/image.png', width=Inches(5.25)).shadow(True)
and
parag_exa = document.add_paragraph('Shadow ')
run = parag_exa.add_run()
inline_shape = run.add_picture('images/image.png')
inline_shape.shadow = 'Offset Diagonal Bottom Right'
Attribute errors for all the above.
Just wondering if there was something i was missing here that would let me access the shadow attribute or if this isn't possible.
Cheers in advance.

 They just want to code for loops and print statements all day. I suggest you find another language as soon as possible. Most python developers are obese and rather lazy..... Good luck finding nothing :sweat_smile:


qr login is not working on last line update

i've try bumped version on header, its irrelevant
email authentication is working properly
looks like they updated qr, would be nice if you have a look on this one

@Auttapakorn i dont know about that, lemme try later
i think line use second qr for now, i have't that service right now

I know  this may be a little dated but experiencing a similar issue with the qr code being incorrect as of the last update to line.

i'll fix this issue as soon as i can


add missing translation tags in portal templates

add missing translation tags in portal templates

I haven't merged this PR due to the o issue.  If you'd like to push it out, and circle back later once TrueNTH weighs in, that's fine with me.


wip: improve snap installation checks

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #5287   👈

This comment was auto-generated by Graphite.


[W5][T12-4]Prithivi Raj

Added sort Command : Displays list of persons sorted by their name.
Updated Test cases to perform regression testing: Pass
Updated User Guide to reflect instructions to use of new command.

Very detailed in the documentation, including userguide. Nice job in making the sort command, very neat looking. My documentation need to learn from you bro
#LATB


Underlines of  position error

When there is a picture in <a>, the underline is in the wrong position.

Fixed in https://github.com/YunYouJun/valaxy/commit/96e1d84b09ce55585cf3a1c5b7f3596290e2c2e2.


moment plot: bin to 3 days, plot along creeping section, seuliemum fault, include all plate interface events (?)

also while you're at it replot the vertical profiles to show the vertical stripes

want to figure out what is contributing to the seismic moment
also plot against station uptime so there's some reference
and plot the no. of events


LastToFinish: derive Foldable via Ap

The derived Foldable instance via Ap and Alt are the same, but it
makes more sense to use Ap in LastToFinish.

bors merge


Add Team Drive Support

Added Team Drive Support. Tested and working on Mac OS 10.14 and Ubuntu 16.04.
Need to specify --root-node-id for the team drive. If not specified, it works normally with the root folder of the drive.

Sorry for the sluggish feedback, I'm very busy this week. I'll hopefully habe some time to test and merge this on Sunday.
I looked at the CI test failures on Go 1.7 and they don't seem to be related to your changes.

@rtkg12 Can you descript how to get the --root-node-id for the teamdrive?

Please rebase on current master to fix the CI build.

Any update on these changes? Team Drive support would be a game changer IMO.

Would like to see an update on this if possible!

So... Is this greenlighted?

It needs to be rebased and it lacks information on how to find the correct root node id, so I couldn't verify it actually works.

Hi there,
I managed to get this PR working on my setup.
In order to get the Root Node ID of a Team drive, you just open the Team Drive in your browser and you take the last part of the URL, example:

URL: https://drive.google.com/drive/u/0/folders/ABC123qwerty987
Root Node ID: ABC123qwerty987

Also the CI fails on Go 1.7 because the code uses GetBody which has been introduced only in Go 1.8 so a fix for this is needed as well (as Go < 1.10 is unsupported as per the Release policy)

@herver Thanks for testing, please open a new PR for the documentation changes.

hi there
awesome that teamdrive is added but I can't get it to work
I only get cannot obtain root node: Could not get root object
this is my mount command
root@us-iowa-1:/# /usr/lib/plexdrive/plexdrive mount -v 2 /mnt/plexdrive --root-node-id=<removed>
Full Debug:
[USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] DEBUG  : Opening cache connection [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] DEBUG  : Authorizing against Google Drive API [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] DEBUG  : Loading token from cache [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] INFO   : Mounting path /mnt/plexdrive [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] DEBUG  : Checking for changes [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] DEBUG  : Getting start page token from cache [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] DEBUG  : Last change id found, continuing getting changes (60834) [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] INFO   : First cache build process started... [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] DEBUG  : Getting root from API [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] DEBUG  : Storing page token 60834 in cache [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] INFO   : First cache build process finished! [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] DEBUG  : googleapi: Error 404: File not found: <removed>., notFound [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] WARNING: Could not get object <removed> from API [USR/LIB/PLEXDRIVE/PLEXDRIVE] [2019-07-16 01:41] DEBUG  : cannot obtain root node: Could not get root object
thanks

Hi, can you release ?


Added Rzeszow and Gdansk GDCR 2018 events in Poland

Adding two events in Rzeszow and Gdansk (Poland) organised by PGS Software company.

Welcome to GDCR 2018! :smile:


preserve and send cookies only for multistatement transactions

Many load balancers set session cookies.
The API should throw away all session cookies returned in a response (so subsequent requests can be balanced) except when opening a multi-statement transaction.  In this case, the API should cache the cookie in a transaction object and send it in subsequent requests for the same transaction so the load balancer can ensure host affinity (which is necessary for multi-statement transactions).
Maintaining a transaction object requires an interface change:

The db.transactions.open() function will take an optional parameters object that can specify {loadBalancer:true}
When load balancer is true, the open() function will return an opaque transaction object instead of a string transaction id
Functions that accept a string transaction id parameter today will also accept a transaction object in the same position

On the next major semver boundary, loadBalancer will become the default.  On the following major semver boundary, the string transaction id will be removed completely in favour of the transaction object.

db.transactions.open() can take a new withState boolean parameter.
If withState is true,  db.transactions.open() returns a Transaction object instead of a string transaction id.
The Transaction object can be passed instead of the string transaction id on any call that accepts a transaction id.
Internally, the Transaction object keeps all cookies sent in the response to the transaction open request.  The Node.js API sends the same cookies back to the server on any request that specifies the transaction by means of the Transaction object.
In particular, if a load balancer sets cookies in the HTTP response to REST API requests, the  Node.js API captures those cookies only for the transaction open response and resends the cookies only for other requests using the transaction.

Added test on nodejs-documents-transaction-withstate.js


there are no data come back

hello,i have set the api key ,and using the right code,and the demo get the "{'status': 'ok', 'totalResults': 0, 'articles': []}",this is no new_data come back ,why

Same here.

Please see issue here: https://github.com/mattlisiv/newsapi-python/issues/77


Metamodel 192/datatype conversion functions

Do not merge yet
This is my first attempt at implementing METAMODEL-192, which thereby is also the first time we make pluggable ScalarFunctions available. I wanted to share it early on since I think some people will find the approach interesting ( @ardlema ?) as inspiration for similar scalar functions (SUBSTRING etc.) ... Please let me know if you think the approach is good or not.
At this stage it is still not made in a way that works with the JDBC module. Will have to add that.

Really nice approach!
I woudl add some unit tests for TO_STRING, TO_DATE and TO_BOOLEAN functions.

Great, then I'll continue down this line ...
A few TODOs that I can think of:

[ ] More rigoruous testing - does it also work in filter items, does it work in subqueries and all that.
[ ] Add some hooks to the query parser to understand it.
[ ] Maybe also add aliases in the query parser? I notice that for scalar functions there are often different names for the same thing in many SQL dialects.
[ ] Add JDBC support by querying the IQueryRewriter interface whether or not particular scalar functions are supported.

As you can see on my todo-list above, actually now it works in JDBC also and I have improved testing and all.
The only thing that doesn't work (that I know of) is having scalar functions in e.g. the WHERE clause of a JDBC query. I couldn't yet figure out a good way to decompose that problem. I can think of three solution scenarios:

Simply not support it, but throw a MetaModel exception when it is attempted. For me this is actually OK to begin with.
Have a failover strategy when this happens which uses a QueryPostprocessDataContext and thus eventually one or more full-table-scans to execute the query.
Do something really clever to decompose the query and figure out how much of it can be natively executed, do that, and postprocess the rest. This seems very complex so I wouldn't recommend it for now.

What do you guys think? I prefer 1 or 2.

I'd go for the first option. As a first approach I think that is fine to just throwing an exception. I always like to start with the simplest solution and then try to improve it afterwards (if it is needed)
So..my +1 to the first one

I agree. Would rather finish this quickly, maybe create a JIRA issue for improving it, but have something that works for a limited scope. And I do think that having scalar functions "just" for SELECT clause is already a big deal.

OK as far as I am concerned, this story is now complete (assuming Travis builds alright). Could someone give it a code review and then let's merge it if everybody still likes it? :-)
Side-note: I would think we then consider MetaModel 4.4.0 kinda done? Anyone else has pending stuff to add before we do a release?

Looks good to me, but it would be nice to maybe also allow something like SQL's CAST <expression> as <type> later.
+1

Let's create a separate issue in JIRA for that. I could imagine a little parser trick to do it, but I guess for consistency that format is an anomaly compared to other function literals that normally say function(argument) :-)

Reported your idea in METAMODEL-193 - https://issues.apache.org/jira/browse/METAMODEL-193


Feature request: GPU Selector

Noticed some hard frame drops in some games so I decided to investigate, and it seems like DDrawCompat isn't picking up my nvidia GPU whatsoever, seems to be always rendering from the iGPU despite having the game's .exe selected to high performance in both the nvidia control panel and windows's gpu selector

I can't do much about it, it's a documented restriction:
https://nvidia.custhelp.com/app/answers/detail/a_id/3733/~/games-based-on-directx-8-and-older-versions-of-directx-will-only-run-on
Maybe ForceD3D9On12 can work around it, but I don't have an Optimus laptop to try it. Also compatibility with D3D9On12 is not the best at the moment, and performance is generally worse than native.

But does DDC not wrap most API calls to dx9's though? in theory that should work to just get the GPU to detect the game

No. This is covered at the very beginning of the README:

There is no API conversion involved, most of the rendering is still done by the native DirectDraw/Direct3D 1-7 and GDI libraries.

I see. There is not a single day that passes where I don't regret not going with the AMD variant of my laptop. Thanks for the info though


fix: Updated TensorBoard profiler notebook

REQUIRED: Add a summary of your PR here, typically including why the change is needed and what was changed. Include any design alternatives for discussion purposes.

- 1. Used docker image instead of Python package
- 2. Updated service account permissions
- 3. Handled cloud_profiler.init() error
- 4. Used the latest template

REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[ ] Use the notebook template as a starting point.
[ ] Follow the style and grammar rules outlined in the above notebook template.
[ ] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[ ] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[ ] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[ ] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[ ] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

If you are opening a PR for Community Notebooks under the notebooks/community folder:

[ ] This notebook has been added to the CODEOWNERS file under the Community Notebooks section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

If you are opening a PR for Community Content under the community-content folder:

[ ] Make sure your main Content Directory Name is descriptive, informative, and includes some of the key products and attributes of your content, so that it is differentiable from other content
[ ] The main content directory has been added to the CODEOWNERS file under the Community Content section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

This has been covered by another pull request. Closed.


Clean up repository management

Refactored getCourseGroupRepos to getCourseGroup and more
Minor cleanup of GetRepositories calls
Renamed DeleteRepositoryByRemoveID to DeleteRepository
Migrated some tests to separate file
Added TestGormDeleteRepo
Moved GetRepositories tests to gormdb_repositories_test.go
Replace GetRepositoryByRemoteID with GetRepositories
Replaced get{User|Group}Repo(s) methods with getRepo
Simplified enrollStudent to use getRepo
Minor cleanup of deleteGroupRepoAndTeam
Replace GetRepositories with getRepo in rejectEnrollment
Replace getRepositoryURL with getRepo in GetRepositories

Fixes #618

Codecov Report

Merging #619 (2ed900d) into master (2ea10a4) will increase coverage by 0.19%.
The diff coverage is 50.52%.

@@            Coverage Diff             @@
##           master     #619      +/-   ##
==========================================
+ Coverage   22.78%   22.97%   +0.19%
==========================================
  Files          53       53
  Lines        7585     7565      -20
==========================================
+ Hits         1728     1738      +10
+ Misses       5570     5537      -33
- Partials      287      290       +3

Flag
Coverage Δ

unittests
22.97% <50.52%> (+0.19%)
:arrow_up:

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

Impacted Files
Coverage Δ

web/hooks/github.go
7.82% <0.00%> (-0.29%)
:arrow_down:

web/repositories.go
44.82% <42.30%> (+5.93%)
:arrow_up:

web/courses.go
41.25% <47.82%> (+2.74%)
:arrow_up:

web/rebuild.go
67.14% <50.00%> (ø)

web/groups.go
55.72% <52.38%> (-0.77%)
:arrow_down:

database/gormdb_repository.go
55.00% <75.00%> (+5.00%)
:arrow_up:

web/autograder_service.go
32.42% <76.92%> (+2.11%)
:arrow_up:

web/scm_helpers.go
60.26% <100.00%> (ø)

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

65 of 115 (56.52%)  changed or added relevant lines in 8 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.2%) to 25.071%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

database/gormdb_repository.go
3
4
75.0%

web/rebuild.go
1
2
50.0%

web/autograder_service.go
13
16
81.25%

web/hooks/github.go
0
6
0.0%

web/groups.go
18
29
62.07%

web/courses.go
17
29
58.62%

web/repositories.go
12
28
42.86%

Files with Coverage Reduction
New Missed Lines
%

web/groups.go
1
64.16%

web/scm_helpers.go
1
65.97%

web/courses.go
2
45.63%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
2371

Relevant Lines:
9457

💛  - Coveralls


fix permission between nginx and php-fpm

I have got "403 Forbidden" error with a webdav mount. Change the user id to resolve the problem.
Inspired by https://stackoverflow.com/questions/36120679/wrong-permissions-in-volume-in-docker-container/36130772#36130772

Do not approve, I have do 💩. I have retry my fix, and now I have a white page.
It's late and I'm tired. I verify that tomorow when I have free time.

The problem is not between the nginx and php container, nginx will communicate via the socket to the php-fpm container. Nginx only "loads and serves" the php files to the php engine, the php engine then loads the markdown files, this is the reason why the nginx container don't need to have you notes volume. The problem what you have is probably between your local user and the user of the php container. But if you set your local vault folder to world readable the php engine should be able to load the files, we can check this further in discord if you want but I don't think that a permission change in the docker containers is necessary

The problem is not between the nginx and php container, nginx will communicate via the socket to the php-fpm container. Nginx only "loads and serves" the php files to the php engine, the php engine then loads the markdown files, this is the reason why the nginx container don't need to have you notes volume. The problem what you have is probably between your local user and the user of the php container. But if you set your local vault folder to world readable the php engine should be able to load the files, we can check this further in discord if you want but I don't think that a permission change in the docker containers is necessary

So When I tried you demo, that worked fine. When I mount a web-dav directory and when I have picture inside this webdav, I have a 403 error.
You can see my branch web-dav : https://github.com/adriy-be/Perlite/tree/WebDav
With that, I don't have the error.
The white page is just due to I have put an "_" at the nginx config mount point 🙄
So, thanks for your reply. Have a good night 😴

Well, what I wrote was wrong, the nginx container indeed need also the vault volume, I just overlooked the
      - perlite

in the docker compose file.
However I checked again some permissions.

The php-fpm deamon runs as the user id 82 (which is the user www-data in the container)
The nginx deamon runs as the user id 101 (which is the nginx user in the container)
The vault (volume) gets mounted to the containers with the permissions it has on your local host

However if the vault folder and all files have the a+r and a+x flag, the user of the container should not matter and it should work without any changes. Take care that the vault folder (and content) musst be world readable AND world executable.  The reason why it is working for you after changing the php user to id 101 is probably because you have this id / user as owner or group set to your vault in your host.
So im pretty sure the user id change to 101 is not necessary when your vault folder has the necessary permissions :)

Instead of relying on vault folder permissions from the host how about mounting the vault directory and copying the contents from that directory to a directory inside of the container and changing permissions there? This would solve this issue while also not interfering with directory permissions on the host system needlessly.

Instead of relying on vault folder permissions from the host how about mounting the vault directory and copying the contents from that directory to a directory inside of the container and changing permissions there? This would solve this issue while also not interfering with directory permissions on the host system needlessly.

true, the alternative way would be to use docker volumes, but then you always need to restart docker compose to get the content copied to the docker volume. Also there are vaults which are gigabytes of data, which will take time to copy and also take disc space.
There is also a way to set the user ids of the container to the current user for example via an .env file but this is also not optimal and needs to be adjusted for every system. However, if setting the permissions is really a problem then I would go this way and manually change the ids for the php-fpm and nginx service in the containers to someone who has permissions to the vault


helm chart fails to run

I'm seeing this when I install the helm chart with crash loop back-off
helm install powerdns-release -f values.yaml aecharts/powerdns
using the values file from your repo and just changing the domain
❯ kgl powerdns-release-6c85b7898f-cpwwp powerdns-powerdnsadmin
===> Waiting for <IP_ADDRESS> MySQL service
Connection to <IP_ADDRESS> 3306 port [tcp/mysql] succeeded!
Using base prefix '/usr'
New python executable in /opt/pdnsadmin/flask/bin/python3
Not overwriting existing python script /opt/pdnsadmin/flask/bin/python (you must use /opt/pdnsadmin/flask/bin/python3)
Installing wheel...done.
===> DB management
---> Running DB Migration
Usage: flask db migrate [OPTIONS]
Error: Could not locate a Flask application. You did not provide the "FLASK_APP" environment variable, and a "wsgi.py" or "app.py" module was not found in the current directory.
Usage: flask db upgrade [OPTIONS] [REVISION]
Error: Could not locate a Flask application. You did not provide the "FLASK_APP" environment variable, and a "wsgi.py" or "app.py" module was not found in the current directory.
===> Update PDNS API connection info
Connection problem (1146, "Table 'powerdns.setting' doesn't exist")
Connection problem (1146, "Table 'powerdns.setting' doesn't exist")
[2020-06-19 08:25:17 +0000] [23] [INFO] Starting gunicorn 19.7.1
[2020-06-19 08:25:17 +0000] [23] [INFO] Listening at: http://<IP_ADDRESS>:9191 (23)
[2020-06-19 08:25:17 +0000] [23] [INFO] Using worker: sync
[2020-06-19 08:25:17 +0000] [25] [INFO] Booting worker with pid: 25
[2020-06-19 08:25:17 +0000] [25] [ERROR] Exception in worker process
Traceback (most recent call last):
File "/usr/lib/python3.7/site-packages/gunicorn/arbiter.py", line 578, in spawn_worker
worker.init_process()
File "/usr/lib/python3.7/site-packages/gunicorn/workers/base.py", line 126, in init_process
self.load_wsgi()
File "/usr/lib/python3.7/site-packages/gunicorn/workers/base.py", line 135, in load_wsgi
self.wsgi = self.app.wsgi()
File "/usr/lib/python3.7/site-packages/gunicorn/app/base.py", line 67, in wsgi
self.callable = self.load()
File "/usr/lib/python3.7/site-packages/gunicorn/app/wsgiapp.py", line 65, in load
return self.load_wsgiapp()
File "/usr/lib/python3.7/site-packages/gunicorn/app/wsgiapp.py", line 52, in load_wsgiapp
return util.import_app(self.app_uri)
File "/usr/lib/python3.7/site-packages/gunicorn/util.py", line 352, in import_app
import(module)
ModuleNotFoundError: No module named 'app'

The container was updated, please check again

Chart upgrade to 0.1.7 and tested in EKS

Works on bare metal cluster too now @aescanero.
thank you


position servo parameters ?

Hi,
I was wondering whether the <position> actuator was supported in v2, and whether we could change its gain parameters (in MJCF files, the gains can be specified).
Thanks

Yes it's supported. We don't use gain, just gear, in a somewhat simplified version of the position actuator (still subject to change)
the torque for a position actuator is $gear * (ctrl - q)$

I see, thank you very much ! So there is only a proportional term, and no derivative term (w.r.t. $\dot{q}$) ?

the derivative term is added via damping of the joint


no more globals

Love the script but leaking a lot of global variables is frowned upon by a lot of JS devs these days. I wrapped the whole script in a revealing module pattern. The notfication calls now need to be amended (as done in the demo here) but other than that, it works.

Thanks very much for your input/advice and making the changes. I learned something new today, I appreciate it. Take care!


Use echo instead of echm when write message

when write a file via command :w, the filename information will be print to the cmd line, I do not know if it is printed via :echom. it makes the :message too mess.

I think it can be good to know, that a file has been written when looking at the :mess output.

is it possible make it configable, because we want to use :message to show only the debug infomations. thing like:
xxx.tt written
8 lines yarked
etc.

can be hide.

@wsdejeg asked:

is it possible make it configable, because we want to use :message to show
only the debug infomations.

Not sure exactly what you want. But would this help you?
:filter /your_regexp/ message


Ensure Dependencies are Complete Before OpsCenter Provision

We need to ensure that the nodes are all started, have java installed, etc before OpsCenter tries to provision them.  It's unclear how to do that today.

Currently using a sleep statement to work around this.  The google guys are investigating a better solution as of 10/29/15.

Noticed that the console exits before the opscenter startup script has completed running.  This behavior may not be desirable.

One way we talked about doing this is through references. So you would need a method that generates the reference strings of the low level GCP resources based on your inputs. I believe that something like this could do the job (I haven't tested this though):
def GenerateReferencesList(context):
reference_list = []
n_of_copies = context.properties['nodesPerZone']
dep_name = context.env['deployment']
for zone in context.properties['zones']:
for idx in range(1, n_of_copies + 1)
node_name = '$(ref.%s.selfLink)' % dep_name + '-service-' + zone + '-' + idx + '-vm'
reference_list.append(node_name)
return ' '.join(reference_list)
Then you can call this method to return the concatenated string as a value in your metadata

I made a couple minor changes and just pushed the code up to github.  I'm testing it now...

We currently have references which ensure the nodes are physically provisioned prior to running OpsCenter provision.  We also have a sleep statement.  This attempts to ensure the nodes are given enough time to install Java prior to the provision.  This isn't ideal, but is apparently the best we can do in the platform currently.
When a new version of OpsCenter provisions Java itself, this issue will be fully resolved.

Moving away from OpsCenter provisioning to shell scripts instead.  There are still going to be dependencies to manage, but that should be simpler.
Those scripts are here: https://github.com/DSPN/install-dse


Only zoom out to the number of markers if the number of markers is less than the default target of 3

If a parking map has 2 or fewer parking options, the map will zoom to infinity
Resolves #25

This is great! Thanks @jarretth Sorry for the terrible delay in merging.


Updates GitHub Actions setup-python

Proposed Changes

Updates setuo-python action for CI/CD pipeline

Awesome work on this repo! I thought it'd be cool to help try to get the CI/CD pipelines working again. The last failed one is too old to show the log anymore, so I opened this PR to get a new run of the CI/CD pipeline that then would show an error to further troubleshoot.

LGTM thanks!

Cool! Looks like "this workflow requires approval from a maintainer" - can you kick off the CI/CD?

Hmm, I was expecting the test workflow to run. Maybe I can get it to run in PRs.

Thanks, that ran the test workflow and showed the syntax for updating pip needed a fix. Can you approve the workflow again?

The last workflow error is a known issue with old versions of poetry, and is resolved with: https://github.com/python-poetry/poetry/issues/6300#issuecomment-1233211156.

We're still getting workflow errors, but at least a new error now. This one might get resolved by updating nox-poetry (https://github.com/python-poetry/poetry/issues/3472#issuecomment-754853589).

Wow, the tests ran!

Seems like there's an issue within numpy used by pandas during test_e2e, which can be resolved by updating numpy (https://stackoverflow.com/a/66138833/6068036).


[Snyk] Upgrade html-snapshots from 3.4.0 to 3.4.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade html-snapshots from 3.4.0 to 3.4.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 2 days ago, on 2024-03-09.

Release notes

Package name: html-snapshots

3.4.1 -<EMAIL_ADDRESS>

3.4.0 -<EMAIL_ADDRESS>

    from html-snapshots GitHub release notes

Commit messages

Package name: html-snapshots

5c834a9 Merge pull request #659 from localnerve/rc-3.4.1
0ac69ed @ 3.4.1 -<EMAIL_ADDRESS>2d4fc34 Merge pull request #658 from localnerve/snyk-upgrade-223db7a40efe105847f8fb9a921cd45e
a8cb198 fix: upgrade html-snapshots from 3.3.0 to 3.4.0
54548a3 Merge pull request #656 from localnerve/snyk-upgrade-d52ced295d2f213f2df3641ae00ae65d
9d38ef2 Merge pull request #657 from localnerve/snyk-upgrade-3857b645d1cedb938f0f50adf52ab34e
89d5a9f fix: upgrade html-snapshots from 3.3.0 to 3.4.0
2245dea fix: upgrade html-snapshots from 3.3.0 to 3.4.0
5522f39 Merge pull request #655 from localnerve/snyk-upgrade-4e1329d37a8758813a87ffa5d9a75163
dec0a8c fix: upgrade html-snapshots from 3.3.0 to 3.4.0
ace9f45 Merge pull request #654 from localnerve/snyk-upgrade-781cd6b032974c89572c2bf6d4cf5588
8e575f5 fix: upgrade html-snapshots from 3.3.0 to 3.4.0
b2b2887 Merge pull request #653 from localnerve/snyk-upgrade-1374ead47705e9989f7ef410a89f0dce
ed5205c fix: upgrade html-snapshots from 3.3.0 to 3.4.0

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

coverage: 99.245%. remained the same
when pulling b3b45a26897ad461d4f1b1428e1708c609fdac34 on snyk-upgrade-fbb000a0b968c344eeeb8efd2f42fbd6
into a846276ec2f5876ff48dbf663cfc4cc13cd62d8a on master.


Way to store key in settings file

I think it would be nice for ease-of-use to have a key available as a settings-variable. Maybe the SECRET_KEY could be the default encrypter, but you could also have an ENCRYPTION_KEY variable as well.
That would be helpful for deployment, because you could then load these keys through environment variables. I'd rather not check my keys into version control, and deploying to something like AWS lambda is tough when it relies on your filesystem.
Less of an issue and more of a feature request. It goes without saying, but it's a great library.

Using a filesystem makes django-encrypted-fields unusable in platforms like Heroku where file system access is not possible.
Please consider using an environment variable instead. It's the standard way of handling this kind of thing in Django and web-apps in general these days.
More info: https://12factor.net/config


How-to remove illegal circle zone

I just want the circle zone for legal hunting to show. Leave them look for the illegal one. How would I do that? Thanks!

Hi, to disable the illegal zone entirely just remove the zone from Config.HuntingArea
but if you want to just hide it on the map well it's on the Test branch as an extra field named "showBlip".

You a champ. Exactly what I needed.
A little thing, the hunting shop is called "Rival" for me which is the name of the blip itself.

they were missing showBlip inside their config file.
and I change the script so their names can be changed inside the config file too.

Amazing!

You also need to tell me where to tip you :)

I disabled the shooting protection until when I'm sure it's working as intended!
(you can activate it and give me some feedback it's under "Config.ShootingProtection" )
in my game, this caused some problems within the script.

Just tested with true and false and no diff. You can now shoot anyone/everything.

it should only skip other players.

Have mine set at:
Config.ShootingProtection = true
Config.ProtectedWeapons = {'weapon_huntingrifle'}
And I can shoot players

Swkeep#7049 this is my discord I might need some debugging data.


Countdown values animation

I have progress percentage numbers  that increase in value faster than once per second, which seems to confuse the countdown control.  The control tries it's best to keep up, but the number increases much faster than it takes to spin to the next number, so then the control slows down and starts to spin in reverse to catch up to the current number. Said another way, if the actual number gets to the end and resets before the control can render that number, then it starts spinning backward.
Is there a way to force the control to reset itself to start over with the new sequence rather than rapidly spinning backward toward the lower number?

current workaround is to hide the countdown element and show a different static element if the value is zero or greater than 99. That way you don't see the countdown element spin back to zero. But if there was an additional style value, perhaps optional, that could be applied to say "don't spin to this initial value when this initial value is set". Just a thought.
I must say I really like how the countdown will spin up and back though. So, for instance, if you are monitoring a number that increases and decreases, it will spin up and down when the number changes. That is an awesome feature!
I also must say your controls are outstanding!! pretty much everything I need! (the radial progress that was requested elsewhere would be a great addition)
Thank you so much for creating and maintaining this project.
With that thought, I will go ahead and close this issue with the workaround I'm using.

thank you for the new radial progress element!!!


it wont worck

Challenge Use CSS Selectors to Style Elements has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.71 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

CatPhotoApp

Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

Thank you for closing this yourself 😁 Just so you know, if you need coding help, please use the "Help" button in the challenge. Happy coding!


List: row order does not match visual order which makes keyboard accessibility hard

The list widget puts the rows not in the visual order into the DOM which makes keyboard accessibility using tab hard because for that elements have to be in order as they appear visually.
Maybe we could have a setting to disable this and have elements in order? The notifications list would benefit from this.

Yeah we can attempt to solve this.

Repro steps: in the extension viewlet, collapse the installed extensions view to a minimum, scroll all the way down, then slowly all the way up. If you Tab into an action, it should always pick the first one.
Here's before:

And after:


Ethereum Dapps Support (MVP)

We would like to be able to integrate SH Wallet with Ethereum dapps through Wallet Connect.
Sample https://app.uniswap.org/swap
We want to have the same UI as it is with inapp browser existing capability.

Connection is working fine for me , except FF ext/web (but this is known issue with scan qr and present on dev). The rest LGTM


Support GitHub integration pt. 1 (#29)

Add avatars.githubusercontent.com to supported image domains

Closes https://github.com/slashkudos/kudos-web/issues/24

Use latest slashkudos npm package version

Refactor ListKudosResponse

Add GitHub logo and links

Closes https://github.com/slashkudos/kudos-web/issues/25
Closes #27

Do not merge, I broke pagination


Set-TeamArchiveState Parameter -SetSpoSiteReadOnlyForMembers result in BadGateway

Hello Team,
I'm having an issue  to use
Set-TeamArchivedState -GroupId $GroupId -Archived:$true -SetSpoSiteReadOnlyForMembers:$true
Result is:

If I'm removing -SetSpoSiteReadOnlyForMembers everything works again.
So,
did some tests to find out what was going on because the script is working on some teams and not on others.
I think I found the origin but might need confirmation and help on a workaround.
My firm is working with a Multigeo Tenant.
If the teams was created in our "main" tenant it's working flowlessly because I think the command is able to connect to the SPO admin setup.
Example: firm.contonso.com being the main tenant
If the team is created in another Geo Tenant,
example: firmgbr.contonso.com, the Sharepoint site will be on another "tenant" and so the command will fail.
Do you mind having a look and let me know if this is a known issue

Managed to double check and that issue is not related to the GEO tenant. Not sure what is causing this but the team in question is not archived using the WebGui either...


meson: add support for android, ios, macos

Adds support for arm and arm64 in the meson build.
Tested for android, ios, macos by https://gitlab.freedesktop.org/ystreet/cerbero/pipelines/117966

Could we please get these patches merged? I have more patches on top of this that add support for Windows ARM/ARM64 :)


Add jetcam package

I've added the jetcam package from https://github.com/NVIDIA-AI-IOT/jetcam

Thanks @danainschool ! 👍


Release version 1-0.9

Released https://search.maven.org/#artifactdetails|com.google.cloud.genomics|google-genomics-utils|v1-0.9|jar


Redisson is not reading the values set by external redis clients

I am creating a set through Medis, a Redis client or FastoRedis. When I try to read the same using redissonClient.getSet(key), I see an exception being thrown saying "Unable to evaluate the expression Method threw 'org.redisson.client.RedisException' exception." while debugging the set in IntelliJ. Why can't redisson is reading the value for the key? I noticed one thing when the redisson sets the value, it has quotes. But when you set a string from external clients, it doesn't have any quotes and redisson can't seems to be recognizing it.

Redisson by default uses JacksonJson as data codec, meaning it will try to use Jackson's ObjectMapper to deserialise the data from Redis. In your case those data were set by Medis client, which is likely not using the same encoding method.
This is the same reason why you see quotes around strings in Redis from Redisson. When Jackson serialise an object to a VALID string type in JSON format, it wraps the content around with " to comply with the format standard.

If you don't like to use JSON as the encoding scheme. Try StringCodec instead.

I did use and I was able to make progress but ran into another issue.  I modified the key/value pair in the 3rd party client and when Redisson is trying to read the same,  it is throwing the following exception.
org.redisson.client.RedisException: ERR Error running script (call to f_41086849b5c119624298370915555b51b921688a): @user_script:1: user_script:1: bad argument #2 to 'unpack' (data string too short) . channel: [id: 0x53f43883, L:/<IP_ADDRESS>:53224 - R:/<IP_ADDRESS>:6379] command: (EVAL), params: [local value = redis.call('hget', KEYS[1], ARGV[2]); if value == false then return nil; end; local t,..., 5, HeartBeat, redisson__timeout__set:{HeartBeat}, redisson__idle__set:{HeartBeat}, redisson__map_cache__last_access__set:{HeartBeat}, {HeartBeat}:redisson_options,<PHONE_NUMBER>149, PooledUnsafeDirectByteBuf(ridx: 0, widx: 16, cap: 256)]

MapCache object is entirely a Redisson's own creation which offers you the ability of individual element eviction and other features that is not available in other clients. If you want to use MapCache related features, you need to use Redisson as the only client. If you want to pass data between different clients, then you need to stay with standard Redis objects such as List, Set, Map, etc.


Add support for stable HTTP semconv

Adds support for stable HTTP semconv, while maintaining backcompat support for the pre-stable HTTP semconv.

API change check
API changes are not detected in this pull request.


Turbine drops my FPS

Issue description:
Built a 11x11x18 turbine and my FPS dropped from 200 to 20.
Here is a video example with Mekanism only: https://youtu.be/_mpVu5Ke0D0
Note: I had a smallest sized turbine before that and my FPS weren't nearly as bad.
Steps to reproduce:

Build a large Turbine

Version (make sure you are on the latest version before reporting):
Forge: 2562
Mekanism: <IP_ADDRESS>9-thiakil
Mods: list.txt

Also what happened to the connected textures?

Hmm, seems like #4829 again.

Is it thought? Thiakill said it's fixed and my turbine is not opaque.

Same here! tested in a creative world with just mekanism mod installed (and its required ones) and MC simply die (I got kicked to the main menu with an error about server not responding...not crash... but render the game unplayable)
MekanismGenerators-1.12.2-<IP_ADDRESS>8
Mekanism-1.12.2-<IP_ADDRESS>8
forge-1.12.2-<IP_ADDRESS>62
It does not matter the size of the turbine (at least in my creative test) and yes, the turbine does not show the animation just opaque blocks inside.

@jonabrv1965 328 should contain a fix to address this issue, but, sadly, @Xiaminou reported the bug with version 329 which indicates it isn't fixed yet..
I'll take a look if I can reproduce it with the latest version.

Tried to reproduce it.. I sadly cannot reproduce it. There might be two reasons, @thiakil made a wrong version, not containing the fix, or you've got another setup than I have..
My setup:

Well then can someone host the files for the latest version please?

Compiled from development commit 7c14b3d520232c379dce3855d0403c6f8c40b5b8
This is exactly the version I'm currently using
Mekanism-1.12.2-<IP_ADDRESS>c14b3d520232c379dce3855d0403c6f8c40b5b8.jar.zip

Hi Thommy, when I tried to run the version you linked above, I got a message about wrong mekanism generators and tools versions... can you send those links also? thanks in advance!

My version is an all in one if I made the correct one.

@jonabrv1965 My bad, the compilation has changed a bit since I last build a version..
Here's a good upload, with 3 jar files.
Mekanism-1.12.2-<IP_ADDRESS>c14b3d520232c379dce3855d0403c6f8c40b5b8.zip

Fixed in 230.


Add more time scale options

Hi, great package. Would it be possible to add more time scale options: daily, hourly and minutely please?

If you use the custom scale, you can customize the X axis.
On Sat, Jun 15, 2019, 10:17 PM synw<EMAIL_ADDRESS>wrote:

Hi, great package. Would it be possible to add more time scale options:
daily, hourly and minutely please?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aeyrium/bezier-chart/issues/3?email_source=notifications&email_token=ABFL3UC63NHMH2KNGXJHSJTP2WWE5A5CNFSM4HYQIF32YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GZXDVLA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABFL3UFPNSTLNNT43ETDEUTP2WWE5ANCNFSM4HYQIF3Q
.

This is not just about the axis labels. I have long series of daily data and CUSTOM does not zoom and nicely reshape the data and labels like with the other predefined scales. Example screenshot:

The CUSTOM scale works with xAxisCustomValues and footerValueBuilder but when I try to use fromDate and toDate I get an assertion error:

Unhandled Exception: 'package:bezier_chart/src/bezier_chart_widget.dart': Failed assertion: line 58 pos 11

This one:
assert(
  (bezierChartScale == BezierChartScale.CUSTOM &&
          xAxisCustomValues != null &&
          series != null) ||
      bezierChartScale != BezierChartScale.CUSTOM,
  "The xAxisCustomValues and series must not be null",
),

I don't get it. Did I miss something here? I would like to take advantage of the horizontal scrolling with my daily data (and possibly later for other time scales like hour or minute).

Hmm maybe I could add the zoom to custom values type.
The issue with hours for the scale is the data will be hugeeee from two
dates 2018 vs 2019
On Sun, Jun 16, 2019, 3:33 AM synw<EMAIL_ADDRESS>wrote:

This is not just about the axis labels. I have long series of daily data
and CUSTOM does not zoom and nicely reshape the data and labels like with
the other predefined scales. Example screenshot:
[image: 4]
https://user-images.githubusercontent.com/12034394/59561088-2b7a8300-900b-11e9-966a-cd753beb9bd6.png
The CUSTOM scale works with xAxisCustomValues and footerValueBuilder but
when I try to use fromDate and toDate I get an assertion error:
Unhandled Exception: 'package:bezier_chart/src/bezier_chart_widget.dart':
Failed assertion: line 58 pos 11
This one:
assert(
(bezierChartScale == BezierChartScale.CUSTOM &&
xAxisCustomValues != null &&
series != null) ||
bezierChartScale != BezierChartScale.CUSTOM,
"The xAxisCustomValues and series must not be null",
),
I don't get it. Did I miss something here? I would like to take advantage
of the horizontal scrolling with my daily data (and possibly later for
other time scales like hour or minute).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aeyrium/bezier-chart/issues/3?email_source=notifications&email_token=ABFL3UDHBRBK3WJOA52U6J3P2X3EDA5CNFSM4HYQIF32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXZIEWI#issuecomment-502432345,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABFL3UGDMDFF2EPS57ZCZXDP2X3EDANCNFSM4HYQIF3Q
.

Maybe you can limit the zoom level depending on the scale: and hourly scale does not need yearly level zoom. As I said what would be ideal is to have these other scale levels and not use CUSTOM because it handles automatically the x axis labels with datetimes, which I think is a great advantage for charting timeseries.

Yes. Definitely can use a daily scale in my app.
This feature will be most welcome!

The Weekly type works as a daily Chart ,  what do you mean?

What i need is a past 24 hours timescale. Is it possible?
As @synw mentioned, using the custom scale does not handle the x axis label automatically with datetimes...

I updated the package, Could you try using the dependency in this way:
bezier_chart:
    git:
      url: git://github.com/aeyrium/bezier-chart.git
      ref: master

And run : flutter packages upgrade
Check the method sample8 from main sample.
New date scale type added BezierChartScale.HOURLY
Let me knows if it works.


Add DualSense support

Thanks!

(turns out it had already been added)

More specifically, this was reverted because it was duplicated in f6166e22. The change in f6166e22 was a bit more complete, because it also added the equivalent permissions setup for a DualSense connected via Bluetooth.
Looks like everyone wants their PS5 controllers to work as soon as possible :-)


Tyne and Wear does not show on counties map

On the counties view, the Tyne and Wear region does not show because the GSS code E11000007 does not match the code E11000004 used in the ONS shape file. We have determined that the code has changed, and E11000007 is correct, so the root issue is that we do not have a shapefile that contains a boundary with the correct identifier.

confirm T&W can now be selected on the map (assuming you know where it is)


Add Azure Bicep extension

Please add back the Azure Bicep extension.
The Bicep extension was included for 14 months before being recently removed (f2caeaf0bcdf8b901f1863d24aa6bafd5b7205ba).

Can i try?

Bicep now has its own extension: https://marketplace.visualstudio.com/items?itemName=ms-azuretools.vscode-bicep
I'm not sure if the snippets are as good yet, but if they were to exist anywhere, it would be here now.


[9.x] Add toResourceCollection and toAnonymousResourceCollection to Collections

This PR aims to add two new collection methods: toResourceCollection and toAnonymousResourceCollection.
toResourceCollection allows a user to specify a ResourceCollection that the current collection can be cast into instead of having to wrap the collection in a ResourceCollection::make().
Same with toAnonymousResourceCollection -- this allows a user to specify a JsonResource that the current collection would be put into with the existing JsonResource::collection() method.
class UserResource extends JsonResource {
}

class UserResourceCollection extends ResourceCollection {
    public $collects = UserResource::class;
}

// UserController.php
// using toResourceCollection
class UserController extends Controller {
    public function oldIndex(Request $request){
        return UserResourceCollection::make(
            User::whereIn('name', ['Bennett', 'Taylor', 'et all'])->get()
        );
    }

    public function newIndex(Request $request){
        return User::whereIn('name', ['Bennett', 'Taylor', 'et all'])
            ->get()
            ->toResourceCollection(UserResourceCollection::class);
    }
}

// UserController.php
// using toAnonymousResourceCollection()
class UserController extends Controller {
    public function oldIndex(Request $request){
        return UserResource::collection(
            User::whereIn('name', ['Bennett', 'Taylor', 'et all'])->get()
        );
    }

    public function newIndex(Request $request){
        return User::whereIn('name', ['Bennett', 'Taylor', 'et all'])
            ->get()
            ->toAnonymousResourceCollection(UserResource::class);
    }
}

The benefit here is that this method is just end of the chain of the collection pipeline and doesn't need to be wrapped in the resource methods to achieve the same effect.
This is definitely just a DX improvement, and does not introduce any new functionality so it would not break any existing features related to collections and resources.
Or...
I think this method could potentially be replaced with a more generic wrapper function that would end a chain and return the collection to a closure, maybe something like:
public function wrappedIn(\Closure $callback){
    return $callback($this);
}

This would allow for these two functions to be just created in userland, such as:
class UserController extends Controller {
    public function namedResourceCollection(Request $request){
        return User::whereIn('name', ['Bennett', 'Taylor', 'et all'])
            ->get()
            ->wrappedIn(fn($collection) => new UserResourceCollection($collection))
        );
    }

    public function anonymousResourceCollection(Request $request){
        return User::whereIn('name', ['Bennett', 'Taylor', 'et all'])
            ->get()
            ->wrappedIn(fn($collection) => UserResource::collection($collection));
    }

    public function anotherWrapper(Request $request){
        return User::whereIn('name', ['Bennett', 'Taylor', 'et all'])
            ->get()
            ->wrappedIn(fn($collection) => new CustomCollection($collection));
    }
}

Note: I am not 100% on the PHPStan type assertions -- I have them passing now but I don't think they're as specific as they can get, but I've tried all the combinations I could think of to get the classes to pass around correctly, but no dice.
Also, not hard-stuck on the names toResourceCollection, toAnonymousResourceCollection, or wrappedIn.

TIL there is a pipe method that does this very thing. I just was looking through methods again to check if there was something that does this, and found the pipe method for the first time (and I've used just about every collection method at least once before -- just not ever pipe!)
I'm going to leave this open though to see if the toResourceCollection or toAnonymousResourceCollection has any interest -- feel free to close otherwise.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Create Test-Coverage.yml

Workflow that checks the test coverage of a push

Try removing src/Example.sln from the workflow, as it might be what is causing issues.


org.hibernate.HibernateException: More than one row with the given identifier was found

version：4.0
description：
In the case of sub-libraries, JPA's ManyToOne is associated with foreign key entities under multiple data sources.

Maybe you should learn or understand how to use hibernate, or submit this issue to hibernate's github repo


Bump inmanta-dev-dependencies[async,extension,pytest] from 2.108.0 to 2.109.0 (#528)

Pull request opened by the merge tool on behalf of #528

Merged in 7db2a0bb60a43200b03ec977173eb11d000b1547


Replace discordbots.org with top.gg

Proposed Changes

Replaces the old discordbots.org domain with the new top.gg domain.

please push this ^^^


How to pretrain "raw" text?

Hi! I would like to use QLora to "pretrain" a model and wanted to ask if that is possible, in the release time of qlora I've heard something about a 'raw' mode not existing right now
For example, let's say I had a big dataset in the style of 'the pile' but in another language, how can I pretrain a llama model with that without construction complete prompt response pairs?
Or is QLora only designed for full prompt - response pairs?
I am looking very forward to any help!

Hi, I've ran into the same task, is there any suggestions how to approach it?

As you point out, pretraining and finetuning are similar concepts. In fact, the way we load the Guanaco Open Assistant dataset is similar to how you would load an unlabeled dataset. Just leave the input field blank and put your unlabeled data directly in the output field in the dataset. You will need to adjust the number of tokens you accept in the source/target.

Oh, so I could for example just provide data like 'the stack' in the output only?
Would that computationally be the same as splitting a 'page' of data to input and output randomly multiple times? (so what I am asking is, is the input/output computationally irrelevant in the sense that putting unlabeled data in output is the same as mixmatching input output?)
Thank you very much for your answer :)

@artidoro Thank you very much for the clarification!
If I understood everything correctly, we should put the raw text in the "output" field in json solely. That pretty much mean no system command provided to the LLM and no context (input). This is the same as a plain causal language model training. Though we should care about 'pagification' of the data.


[Question] Can a plugin dependency version be included in a pre-compiled script

@ivorywang commented on Wed Oct 09 2019
Expected Behavior
I am using the kotlin-dsl plugin to build a pre-compiled script plugin that depends on the artifactory plugin as shown below. What I would expect is that the version of the artifactory plugin would be included in the jar that gets produced so I could use my plugin in a different project without including the artifactory plugin with a defined version at the top level.
build.gradle.kts
plugins {
    `kotlin-dsl`
}

repositories {
    maven {
        url = uri("http://artifactory.wal.hds.com/artifactory/maven/")
    }
    maven {
        url = uri("http://artifactory.wal.hds.com/artifactory/plugins-gradle-remote")
    }
}

dependencies {
    implementation("com.jfrog.artifactory:com.jfrog.artifactory.gradle.plugin:4.9.9")
    implementation("org.jfrog.buildinfo:build-info-extractor-gradle:4.9.9")
}

myPlugin.gradle.kts
plugins {
    id("com.jfrog.artifactory")
    `maven-publish`
}

val artifactoryUrl = "http://some.artifactory.url"

artifactory {
    setContextUrl(artifactoryUrl)
    publish(closureOf<PublisherConfig> {
        repository(delegateClosureOf<GroovyObject> {
            setProperty("repoKey", project.properties.get("artifactoryPublishRepository"))
            setProperty("username", project.properties.get("art-uname"))
            setProperty("password", project.properties.get("art-pw"))
        })
        defaults(closureOf<ArtifactoryTask> {
            publications("forDist")
            setPublishPom(false)
            setPublishIvy(false)
        })
    })
}

Current Behavior
Currently when I want to use my published plugin I have to do so as shown below. Without the artifactory plugin line, my-plugin complains that it doesn't know what the com.jfrog.artifactory plugin is like so. I am fairly certain that this error message is because my-plugin doesn't know which version of the artifactory plugin to use.
> Failed to apply plugin [id 'fnd-artifacts']
   > Plugin with id 'com.jfrog.artifactory' not found.

Some other project
build.gradle.kts
plugins {
    id("com.jfrog.artifactory") version "4.9.9"
    id("my-plugin") version "1.0"
}

Is it possible to do what I am trying to do? If so could an example be created to help other people down the line?
Thanks

@eskatos commented on Wed Oct 09 2019
This repository, kotlin-dsl-samples is only for extra Kotlin DSL samples. Please open Gradle Kotlin DSL issues on the gradle/gradle repository.  I'll move this issue to gradle/gradle.

It's basically a question which should go into the gradle forum I guess 😅

I have also posted this question on the forum but my account is still being reviewed before my post is allowed. Any help at all would be much appreciated!

@eskatos I found this post from last year where you detail a lot of the stuff I am trying to do https://discuss.gradle.org/t/plugins-and-apply-from-in-the-kotlin-dsl/28662/8 except I am trying to publish the script plugin to a maven repository and use it in various unrelated projects where I don't want to duplicate the script in a buildSrc directory. Is what I am trying to do possible?


ACC telemetry

Implement telemetry for ACC with what provided by memory-mapped files and UDP socket.

[ ] Read MM telemetry
[ ] Receive UDP telemetry (requires #28 or it would be a lot of work for nothing)

[ ] Ignore UDP telemetry data if older than the current session
[ ] Handle reconnections if socket can't be opened or if it closes

[ ] Use the current MM data and the latest received UDP data
[ ] Support all gaming modes

[ ] Menu
[ ] Race (SP, MP driving, MP connected)
[ ] Practice
[ ] Replay (during race or standalone)
[ ] Spectator
[ ] Between sessions

Notes:

Official MM documentation.
UDP SDK can be found on the Dedicated Server installation folder under sdk\broadcasting.
MM has no track length, the position of only 60 cars and the number of cars stops at 60. UDP has track length and all cars but no position. Some data is not anywhere like the sector length, and will have to be calculated. It's really terrible 🤦🏻♂
Crew Chief code.
Second Monitor code.


[Docs] Sticky sidebar needs to stop before footer

Needs to add a wrapper to anchor to body only.

Seems to work for me. Is there a specific page you see it on?

Looks good! Closing


Is this project dead?

Support does not answer questions, I've tried a few times on multiple days and not received a response. 😢

Almost 2 years without updates...  Anyone have good alternatives?

Hashicorp Vault
Thycotic Secret Server / DevOps Server
Cyberark Conjure
Keywhiz
Stackexchange Blackbox
lyft confidant

There are also

Passbolt
Psono
Passit

etc. find self-hosted alternatives at https://alternativeto.net/software/vault-by-hashicorp/?platform=self-hosted


Consistent parens in arrow functions

@Atinux @alexchopin @ishitatsuyuki @clarkdo I understand there may be a strong personal preference involved here, but with this PR I suggest we adopt the rule of no paren-less arrow functions.
As I was studying Nuxt today, I found a few occurrences of inconsistency usage (places where there was a single param and parenthesis, and plenty of cases where there were none).
I hope I don't come off as nitpicky, just strongly believe adopting this rule makes things more readable and promotes more consistency.
I also made minor edits to a couple of places where the logic would extend too far into a single line, and I felt they could be broken down into multiple lines for the sake of readability.

Codecov Report

Merging #3630 into dev will decrease coverage by 0.09%.
The diff coverage is 96.36%.

@@           Coverage Diff            @@
##              dev   #3630     +/-   ##
========================================
- Coverage   99.19%   99.1%   -0.1%
========================================
  Files          18      18
  Lines        1122    1120      -2
  Branches      302     302
========================================
- Hits         1113    1110      -3
- Misses          9      10      +1

Impacted Files
Coverage Δ

lib/builder/webpack/utils/perf-loader.js
100% <100%> (ø)
:arrow_up:

lib/core/renderer.js
100% <100%> (ø)
:arrow_up:

lib/builder/webpack/utils/style-loader.js
100% <100%> (ø)
:arrow_up:

lib/core/middleware/nuxt.js
100% <100%> (ø)
:arrow_up:

lib/builder/webpack/base.js
100% <100%> (ø)
:arrow_up:

lib/core/nuxt.js
97.64% <100%> (-1.2%)
:arrow_down:

lib/builder/webpack/utils/postcss.js
100% <100%> (ø)
:arrow_up:

lib/core/module.js
100% <100%> (ø)
:arrow_up:

lib/builder/builder.js
100% <100%> (ø)
:arrow_up:

lib/common/utils.js
100% <100%> (ø)
:arrow_up:

... and 6 more

Continue to review full report at Codecov.

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

Note: I'm not a Nuxt.js maintainer nor core member. I have a strong objection to that spam pinging.
Please remove it immediately from your PR description so that it will not cause endless commit mentions when merged.

@ishitatsuyuki Sorry about that. I had absolutely no idea there were people sensitive to this, or that was doing anything reprehensible at all. Once again, my apologies.

@galvez: In the end, this is a matter of opinion but I'd prefer no parens when they aren't needed (unary functions). However, a uniform style is definitely a goal that should be worked towards.

@manniL I used to prefer no parens when possible, but eventually realized the cognitive dissonance that causes. Take this example:

When quickly glancing at code, it's much easier to identify an arrow function if it consistently always has parenthesis than otherwise.
So that caused me to change my mind and understand why the ESLint setting exists. But yeah, this is really just a suggestion, with a bit of manual work done for the team, if they choose to adopt it.

@galvez There are definitely situations where you can recognize an arrow function faster with parens (directly on ( instead of argv =). On the other hand, the difference isn't as substantial (in my eyes, as I likely read code very frequently and I'm using no-parens) and it makes small functions "larger".
Nevertheless, no matter if no-parens or forced-parens, a consistent style will help everyone :relaxed:

I think it's less readable.
- a.filter(x = >x.bool).map(x => x.prop)
+ a.filter((x) = >x.bool).map((x) => x.prop)

I believe "as-needed" is a lot more preferable.
Cases like
export const func = arg => { /* code */ }

should fail not because they are without parenthesis, but because
arg has an implicit `any` type, prompting its change to
export const func = (arg: type) => { /* code */ }

Other cases when parentheses are inserted:

async
multiple arguments
zero arguments const defragment = () => {}

Hence, no-parens only stays where it should, and where even git diff isn't an argument - truly anonymous functions being passed as an argument.

@qm3ster So yeah, this adds up to the idea that's it's truly up to the core maintainer's personal preference, but I do see more value in consistency. We'll see :)
BTW: There's no problem with untyped function parameters in JavaScript. I just wanted say that out loud because there's a lot of TS mentality flowing into JS-land and I don't think that's such a good thing.

Actually I like the comment of @qm3ster about some specific cases where it's more readable. Personally I always use parentheses for consistency but I am open to other people code of style.
Is there a way to force parentheses except in the cases where it's used as "callbacks"?
Example:
a.filter(x => x.bool).map(x => x.prop)

@Atinux as it turns out, there's an config prop just for that, requireForBlockBody.
I'll update the PR. This seems like a fair compromise.

I'll agree with @Atinux and @qm3ster that this looks pretty good now :)

Great consent :+1:


add a Sasquatch sightings submission

Hah, what a dataset! Where did you find it?

Far out!
On Fri, Nov 3, 2023 at 6:19 PM Daniel Shiffman @.***>
wrote:

Merged #1276 https://github.com/CodingTrain/thecodingtrain.com/pull/1276
into main.
—
Reply to this email directly, view it on GitHub
https://github.com/CodingTrain/thecodingtrain.com/pull/1276#event-10861217801,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA447MSP7YDDZF3CDLC3353YCV3YFAVCNFSM6AAAAAA63GWUIOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQHA3DCMRRG44DAMI
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Hello Sir!
I documented the data sources here:
https://github.com/onebeartoe/software-development-tools/blob/master/data-visualization/sasquatch/src/main/resources/resources.text
Have a great day!
Roberto
On Fri, Nov 3, 2023 at 6:19 PM Daniel Shiffman @.***> wrote:

Hah, what a dataset! Where did you find it?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Scrollbar bug caused by scrollintoView()

A carousel that is using scrollIntoView() that cannot be deactived
A Optiscroll on top of that carousel

Please checkout this fiddle: http://jsfiddle.net/ty4ca48z
Comment in the last line to enable the weird scrollbar bug.
You now can enable and disable one of the following inline styles with Firebug and the bug disappears.

element.style {
    bottom: -13px;
    overflow: scroll;
    right: -13px;
}

Update: It seem like the margin on .pf-item-overview is causing this issue!
Please checkout this fiddle: http://jsfiddle.net/ty4ca48z/1/
Any idea how Optiscroll can prevent issues with such a buggy / negative scroll offset?

Apparently, scrollIntoView scrolls the main element up weirdly, even if it has overflow: hidden.
By adding this line after calling scrollIntoView you can revert the hidden scroll position back to 0:
document.getElementById('scroll').scrollTop = 0;

I would be curious to know why browser internal method feels the need of moving the container's scroll up... Moreover, it looks like it is doing it recursively: also JSFiddle page moves up :confused:
Really weird.

Haha, well I spend hours to point out what is going on here, thanks for the suggestion.

Ok, I will close this one because there is nothing Optiscroll could do to prevent it.

Thank you, the scrollTop fix worked for my project...

Nice :wink:


Debugging Python C sources with PythonView

For debugging .pyd in PTVS， when tracking in the C sources it's easy to see the Python objects' information. Is there any way to enable this feature when I just debugging the Python core sources?

Yes, though there are a few little tricks.
The first is you need to set a registry key to enable some secret options. If you create HKEY_CURRENT_USER\Software\Microsoft\PythonTools\Debugger and add a DWORD value PythonDeveloper with the value 1 (assuming you're on the latest VS 2017 - if not, let me know and I'll get you the correct key for whatever version you have).
Next, you need to attach the debugger. This is a little tricky right now (we have some work to make it easier), but the best way to do it is to start python.exe running and then Debug/Attach to it. When you do this, you'll want to customize the debug engines to include "Native" and "Python".
Once that is done, you'll get some extra right-click menus where the "Show Python view" menu normally is. The "Show Native Python Frames" option is the one you want - normally we don't let you step through pythonXY.dll, but this option will enable that.
(The future work we have to do for the second step is make our debugger an option for C++ projects. We know how to do it, just haven't written or released it yet.)
(+ @int19h to correct anything I got wrong)

Slight correction: the "Show Native Python Frames" will show native frames from python*.dll and _ctypes.pyd in the stack, so you can see the interpreter state; but it doesn't enable native stepping.
There's a separate option there called "Use Python Stepping" - which is enabled by default - which makes stepping inside those modules (regardless of whether you're viewing the exposed native frame, or the synthesized Python frame) behave semantically as a Python step, i.e. run the debuggee until the next Python line or breakpoint. Unchecking that option will disable this Python semantics, at which point all steps are handled by the native debugger.
Again, it doesn't matter if you're viewing the C source or the Python source - when this option is unchecked, and you're e.g. at a Python breakpoint (and looking at the .py file in the editor), stepping will step through the C code for the main Python interpreter loop.

create HKEY_CURRENT_USER\Software\Microsoft\PythonTools\Debugger and add a DWORD value PythonDeveloper with the value 1
I have done with these steps, but I can't find the Show Native Python Frames option anywhere, the Use Python Stepping also.

By the way, I usually use the watch window change the object pointer to the expect type pointer to see the structure. This is a troublesome way, Any good idea for seeing the variable just like libpython in GDB?

Any good idea for seeing the variable...

Yep, we're trying to get you using it right now :)
Did you catch the bit about using Debug/Attach rather than F5? Right now, F5 is going to use the native-only debugger, so you need to Debug/Attach to get native and Python together.

Yes, Attach, not Run.
Besides, I still mind where can I find Show Native Python Frames and Use Python Stepping.😅
And I found there are 3 situation using attach.
Registry I seted:

If I set the attach option to Python Code and Native Code, It will only break in main funtion, but my break point was on bltinmodule.c.

本机 代码 => Native Code, I haven't install English pack, I don't know the original words exactly 😹

If I only choose Native Code, it would break on my break point I setted, but no PythonView

If I use the auto option or only Python Code, It can only break in py source.

Those two options show up in the context menu - right-click on e.g. the Locals window, and you should see the Python item there. The options are underneath that.
Your first screenshot shows the correct behavior with the options set to default values - it does actually hit your breakpoint, but because it is inside python.dll, the call stack doesn't show that, since it hides all python.dll frames (and synthesizes Python frames out of them). Once you enable "Show Native Python Frames", you will see it as it actually is under the hood.

I found Show Python View Nodes on locals window context menu, but I still can't find Show Native Python Frames and Use Python Stepping normally.
Finally I found them in customize menu and add them to the menu temporary. Is any other way to get them genteelly? (+o+)
I checked Show Native Python Frames(显示本机 Python 框架 in screenshot) it shows up the frams in python.dll.
But even I checked Show Python View Nodes, I still can't see the python view.

And with Use Python &Stepping, I can't step into a function call using F11. If I uncheck it, it would go straight just like F10
this is a pre frame:

after F11:

The registry key is really what's supposed to make them show up... I wonder if something regressed in our command placement. If you can add them manually, I can't think of any other reason. I'll look into it.
Can you see the Python view nodes in native code outside of python.dll?
And I'm not sure what's going on with stepping there. It doesn't even look like F10, since it didn't end up on the next line. It actually looks more like F11 was handled by the Python stepping logic, which treated it as stepping into a native (as in, outside of python.dll) function call - which would be why it stopped where it is on the stack. But it shouldn't be trying to handle F11 at all here.
To clarify, does F10 work for you as expected - i.e. does it step over a single line of native code?

It can hit the break point on pyd source, but still have no python view.

To clarify, does F10 work for you as expected - i.e. does it step over a single line of native code?
No, after F10 it just go to PTVS's frame

So you don't see any observed difference between "Use Python Stepping" being checked or unchecked?

So you don't see any observed difference between "Use Python Stepping" being checked or unchecked?

Yes, they didn't work correctly as I expected.
I just try these again on VS2015(above results are base on VS2017) with PTVS 2.2.5, on VS2015, it can show the Python view, and the context menu on local window has 4 options: Show C++ View Nodes, Show Python View Nodes, Show Native Python Frames Use Python Stepping. But F10 and F11 still not work correctly.
So...it means they are new bugs on PTVS3.0(released on VS15.2).

I'm not 100% sure whether this is still an issue (I saw it recently but it ended up being unrelated to this, as far as I can tell). We should check it again as soon as possible.

I can't repro stepping issues on our most current bits.
The issue with Python View is because we're registering our .natvis file incorrectly. This is from error log with NatVis warnings enabled:
Natvis: Fatal error: Unable to open natvis file 'C:\USERS\INT19\APPDATA\LOCAL\MICROSOFT\VISUALSTUDIO\15.0_B6255AABEXP\EXTENSIONS\MICROSOFT CORPORATION\PYTHON\3.0\PythonDkm.natvis'.

The path where it's actually installed is ...\3.0\Visualizers\PythonDkm.natvis, but our .vsixmanifest doesn't include that subpath. Need to fix either the manifest or the install location.

I'll spin off a separate bug for Python View:
https://github.com/Microsoft/PTVS/issues/3018

It looks like the bug has been fixed.


add registerForRootCreateElement

Add registerForRootCreateElement to handle various cases where the hook should only run on the root node of the component (e.g. copy classNames, event handlers, etc..).

perhaps registerForRootElement ?

👍


Best way of using pgoapi-swift

Hi! Thanks for getting started with this.
I'm wondering what the best way of using this api? Currently, I'm having to copy over the classes/files into my own project and using it that way.
I'm making a Pokémon Rater, putting it up on git soon.

Hey! Glad you're finding it useful. I'll be adding it to Cocoapods soon, so that'll definitely be the easiest way. For now though, you could open the project separately, compile the framework, and then add the framework to your project. That's probably the cleanest way to do it, and it means you won't need to delete all the files once it is on Cocoapods.

https://github.com/rowellx68/pgorate
For now, I've opted to copy the files. Just currently dealing with login. I was going to implement the Google login, but the python versions of the api aren't so descriptive.

@Isapan I can't seem to built it using it as a framework. All the methods and properties that should be used from externally should have the public keyword. After I added the public keywords, I get this error:

Undefined symbols for architecture x86_64:
"type metadata accessor for ProtocolBuffers.GeneratedMessage", referenced from:
PogoSwift.ViewController.didReceiveApiResponse (pgoapi.ApiIntent, response : pgoapi.ApiResponse) -> () in ViewController.o
type metadata accessor for [ProtocolBuffers.GeneratedMessage] in ViewController.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Am I using this wrong? Could I use it without adding "public"?

@YufeiG no actually you're right. I've been meaning to update it to work properly as a framework and be installed with Cocoapods, but I've been more focused on getting getMapObjects to work. I'll get there though! For now you can do as Rowell did and just copy the files. I'll update here when it's ready.

Niantic is blocking logins from cloud hosters now. Time for something native! :)

I've restructured the project as a CocoaPod, so it should be much easier to use now! I'll get some installation instructions written up shortly.