PostResponse not firing when using LoadFromWebAsync()

Here is what to include in your request to make sure we implement a solution as quickly as possible.
1. Description
PostResponse does not fire when using LoadFromWebAsync(). It fires fine when changed to Load().
2. Exception
No exception. The handler just doesn't fire.
4. Any further technical details
The LoadFromWebAsync() is working fine. It's just that the PostResponse is not firing.

HAP version: 1.11.42
NET version: .net core 2.2

Hello @riiight ,
Unfortunately, both have completely different implementations.
The Load call the Get under the hood and get a HttpWebResponse
See: https://github.com/zzzprojects/html-agility-pack/blob/127fe37b86c552dae95b144ae2e4037f2d2e99d7/src/HtmlAgilityPack.Shared/HtmlWeb.cs#L1661
The LoadFromWebAsync  use a Client.GetAsyncand get aHttpResponsesMessage` (which is not compatible with the handler)
See: https://github.com/zzzprojects/html-agility-pack/blob/127fe37b86c552dae95b144ae2e4037f2d2e99d7/src/HtmlAgilityPack.Shared/HtmlWeb.cs#L2400
We can surely create another handler for the LoadFromWebAsync, will getting a HttpResponsesMessage in the handler will be fine for you?
Best Regards,
Jon

Sponsorship
Help us improve this library
Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Hey can we implement this?


A function that can map list of objects

I have been using this library and had cases where I would pull list of entities from database and map it to the DTO. I ended up creating a utility function that would map all objects of the list.
I think having a map function which accepts list can be useful.
I can open a PR if this sounds good.

You can use the standard map function if you define the destination listType like this (replace List<String> with your actual type):
Type destinationListType = new TypeToken<List<String>>() {}.getType();
Then you can use it with map:
List<String> characters = modelMapper.map(sourceList, destinationListTypeListType);
Solution taken from this documentation page:
http://modelmapper.org/user-manual/generics/


file upload broken in master

I installed OpenSlides (just now again) like in explained in https://github.com/OpenSlides/OpenSlides/blob/master/DEVELOPMENT.rst on two arch linux machines. File Upload does not work. It works on the OpenSlides Demo though.
What I am doing in detailed:
python manage.py start
in webbrowser
login with "admin" "admin"
Go Files -> New
Fill Form to upload an image

Klick save and get to see the File List

Inspection of the link results to the supposedly correct path:

Once I click the link to the image ("world") I get to back to "http://localhost:8000/" again instead of seeing the image I just uploaded in a new tab. This error exists on two of my machines but might be the result of insufficient writing privileges of a OpenSlides sub-process?
 This looks like a bug beeing introduced by the change of the backend server (from tornado to django-channels). Maybe the use of "insecure" does only work for statics but not for media. @ostcar ?

@ostcar @normanjaeckel This regression blocks us by working on issues with mediafiles (e.g. motion pdf with included images). Can one of you help to find the bug here?

Note:
The actual upload is not broken. The file is in the according (=correct) folder. Any requests to retrieve the file are failing.

I don't know, whether this regression was introduced during django channel upgrade or even earlier.
Suspicious commit may also be 8d638a908d6405dfda81db7e04d0e6b5c6656628 where routing was changed heavily.
As far as I can see is /media/ not mapped, though configured in settings as MEDIA_URL.
urlpatterns += [
    url(r'^(?P<url>.*[^/])$', RedirectView.as_view(url='/%(url)s/', permanent=True)),
    url(r'^rest/', include(router.urls)),
    url(r'^agenda/', include('openslides.agenda.urls')),
    url(r'^assignments/', include('openslides.assignments.urls')),
    url(r'^motions/', include('openslides.motions.urls')),
    url(r'^users/', include('openslides.users.urls')),

    # The urls.py of the core app has to be the last entry. It contains the
    # main entry points for OpenSlides' browser clients.
    url(r'^', include('openslides.core.urls')),

Have I overlooked some setting?

The channels commit fe64941aabd0228fc8090ca21f2f99df5204ee20 introduces the problem. With OpenSlides 2.0 the routing of uploaded mediafiles works successfully.

@normanjaeckel Any idea?

@emanuelschuetze
You're right. I just read the diff again. I think line 150 of autoupdate.py solves the mystery.
 (r'%s(.*)' % settings.MEDIA_URL, was the line, I was searching for.
This is the part of the configuration of tornado 🌪. If I understand that correctly, Tornado delivered the media files. I suppose that with the replacement of Tornado, there is no handling of the /media/ route anymore.

Yes. One may try to (re)add the media handling to the urls.py. The static file handling which was formerly also done by Tornado is now done with the insecure option in the start command and the corresponding line in the urls.py. We need a way like this for media files.
In a big production environment the admin may configure a frontend webserver like Apache to serve static and media directly so we do not need to pay much attention on performance here.

Here is described how: http://stackoverflow.com/a/28063707


<EMAIL_ADDRESS>Appended a URL fragment(#/)

Appended a URL fragment(#/) to change extension of downloaded file from php to zip.
Otherwise, a file with php extension will be downloaded and decompression will fail.

[x] I have read the Contributing Guide.

Thank you!
I opened a discussion in the Scoop discussions, to figure out why it is not working since I appended the URI fragment as well, now I seek that I forgot the  "/"

https://github.com/ScoopInstaller/Scoop/discussions/6098


Update/Pull translations from Transifex

After reading #4471 and working on #4470 I think we need to improve our flow about our own translations.
The discussion about how to automate this can be tracked under #4471. In the meantime, we should update our app translation manually and that is what this PR does.
$ tx pull --force
tx INFO: Pulling translations for resource readthedocs.readthedocs (source: readthedocs/locale/en/LC_MESSAGES/django.po)
tx WARNING:  -> zh_TW: readthedocs/locale/zh_TW/LC_MESSAGES/django.po
tx WARNING:  -> fr: readthedocs/locale/fr/LC_MESSAGES/django.po
tx WARNING:  -> ja: readthedocs/locale/ja/LC_MESSAGES/django.po
tx WARNING:  -> ru: readthedocs/locale/ru/LC_MESSAGES/django.po
tx WARNING:  -> uk: readthedocs/locale/uk/LC_MESSAGES/django.po
tx WARNING:  -> zh_CN: readthedocs/locale/zh_CN/LC_MESSAGES/django.po
tx WARNING:  -> ca: readthedocs/locale/ca/LC_MESSAGES/django.po
tx WARNING:  -> pl: readthedocs/locale/pl/LC_MESSAGES/django.po
tx WARNING:  -> ar: readthedocs/locale/ar/LC_MESSAGES/django.po
tx WARNING:  -> es: readthedocs/locale/es/LC_MESSAGES/django.po
tx WARNING:  -> vi_VN: readthedocs/locale/vi_VN/LC_MESSAGES/django.po
tx WARNING:  -> gl: readthedocs/locale/gl/LC_MESSAGES/django.po
tx WARNING:  -> pt_BR: readthedocs/locale/pt_BR/LC_MESSAGES/django.po
tx WARNING:  -> it: readthedocs/locale/it/LC_MESSAGES/django.po
tx WARNING:  -> nb: readthedocs/locale/nb/LC_MESSAGES/django.po
tx WARNING:  -> de: readthedocs/locale/de/LC_MESSAGES/django.po
tx WARNING:  -> sk: readthedocs/locale/sk/LC_MESSAGES/django.po
tx WARNING:  -> eu: readthedocs/locale/eu/LC_MESSAGES/django.po
tx INFO: Done.

$ dj makemessages --all
processing locale de
processing locale vi_VN
processing locale fr
processing locale ru
processing locale sk
processing locale pl
processing locale gl
processing locale pt_BR
processing locale uk
processing locale eu
processing locale es
processing locale ca
processing locale nb
processing locale ja
processing locale en
processing locale ar
processing locale zh_TW
processing locale it
processing locale zh_CN

I suppose that I should run this command also, to update our strings under Transifex:
$ tx push --source

Although, before doing this I want to be sure of this and I may need extra permissions that allow me to do this. @ericholscher @agjohnson, is this command correct?

@xrmx would also be useful to have your input here

@xrmx you are right.
Question: is there any problem by pulling from Transifex without pushing first? I thought that in by doing that, I'm bringing newer translations from what already exists in Transifex to our application. Of course, new/updated strings from sources are not pulled because they don't exist yet in Transifex.
I want to know if doing this pull without pushing first could break something.

@xrmx the function for docs does what I was thinking: https://github.com/rtfd/readthedocs.org/blob/master/fabfile.py#L27

pull
update .pot files
push

We may need to split as you did for the app translations, but also, we should include the new resources under the .tx/config file as I did in #4478. What do you think?

@xrmx :+1: to all you said :)

I think the optimal workflow is to push the source at each merge and pull before each release.

I'm not sure how to make this automatically in each merge, but I think that we could do this easily on each release at least.
Also, there is an issue #4471 that suggests a way to include a checking in the CI.

I'm closing this PR since I need permissions to push to Transifex first. After that, I will run pull and create a new PR.

The push can be automated in travis, and the source may be commited with the pulled translation at release time?

For the sake of it, the doc on conditional stages on Travis:
https://docs.travis-ci.com/user/conditional-builds-stages-jobs/#conditional-stages
And an example integration of django-oscar that pushes to transifex from travis:
https://searchcode.com/file/92738132/transifex.sh


Can't get out of prompt with CTRL-C in accessible mode

Describe the bug
When running a form with a selection list in accessible mode, CTRL-C is treated as an invalid input and triggers prompting again.
Expected behavior
CTRL-C should abort prompt and terminate the program in accessible mode.
Screenshots

Desktop (please complete the following information):

Linux Mint 22.04
No multiplexer used
Run locally
Occurs in Mate terminal, Alacritty

Interesting, I'm not able to reproduce.

Which terminal are you using?
Are you using a multiplexer?
Are you running this locally (i.e. not over ssh)?

I should have specified it's a list selection in accessible mode, let me update the bug report.
I'll post an example in a few minutes

Ok that's interesting. I can't share the code so I made a simple program to reproduce the error but can't either. I will dig more, might not be a huh problem after all.

I can confirm the pb was on my side, due to a signal handler gobbling the CTRL-C. Closing
Thanks for your help


fix typo: provideTaskFailreStore -> provideTaskFailureStore

@natemurthy thanks!

@gkleiman looking forward to making more contributions, hopefully more than just spellchecking  eventually :)


[Bug] __assert_fail declaration in mscclpp breaks "assert()" usage in host functions.

Hi,
There is an __assert_fail declaration at
https://github.com/microsoft/mscclpp/blame/a3cd95bd423381d4bc15a782be36da078ab1eecd/include/mscclpp/poll_device.hpp#L17
When using assert() in host functions, the above __device__ only declaration will break the compilation. Example error:
error: calling a __device__ function("__assert_fail") from a __host__ function("main") is not allowed

Meanwhile, the __assert_fail related declaration in cuda/targets/x86_64-linux/include/crt/common_functions.h is dual __host__ __device__.

Thanks for reporting. Fixed in #303.


Add a project

Adds the req project to the projects page

Could you make the image a but bigger? Also, please add the project card to the bottom of the list. Thanks! It's an amazing project by the way.


GKE Versioning Constraint v1

Resolves #268
This PR creates a GKE Cluster Versioning constraint that checks either the currentMasterVersion or currentNodeVersion field of a GKE cluster with a list of user defined versions.
This constraint provides the following features:

Check currentMasterVersion or currentNodeVersion against list of user defined versions
Allowlist/denylist mode
Exemptions

Also included in this PR:

Sample constraint
Tests

@morgante @briantkennedy Could this get a review from one of you? Thanks!

@morgante Thanks for your review! Have added more tests for glob functionality, and cleanup some of the doc descriptions- PTAL and let me know your thoughts.


[SDK] Farcaster Extension

Problem solved
Adds default and helper functions for interacting with Farcaster's core contracts. Functionality includes registering FIDs, adding signers, renting storage and more. Future PRs will add additional functionality like removing signers.
Changes made

[ ] Farcaster core contract functions generated
[ ] Common FC contract addresses added and exported
[ ] Default FC contract generation functions
[ ] Utility functions to generate signatures for various operations
[ ] getFid to retrieve an FID based on address
[ ] getNonce to retrieve an address's key gateway nonce
[ ] getRegistrationPrice and getUsdRegistrationPrice to get the current cost to register an fid
[ ] getStoragePrice and getUsdStoragePrice to get the current code to rent storage
[ ] registerFid to register a new fid for the transacting account
[ ] registerFidAndSigner to register a new fid via an app account using the bundler
[ ] createSigner to generate a new E25519 keypair to be used as an account key/signer
[ ] addSigner and addSignerFor to add signers to an existing account
[ ] rentStorage to rent additional storage

All custom functions are made available at the top level extension via extensions/farcaster. All generated functions can be accessed by directly importing their contract extension e.g. extensions/farcaster/keyGateway.
Testing

All custom read functions have unit tests added
To test write functions, I created a basic repository with example usage which can be found here

PR-Codex overview
This PR adds new features and event handling for the Farcaster extension in the thirdweb package.
Detailed summary

Added @noble/ed25519 dependency
Updated waitForReceipt import path
Introduced FarcasterContractOptions type
Exported functions related to Farcaster bundler and key gateway
Added error checking function for bigint inputs
Defined constants for Farcaster extension addresses
Generated ABIs for Farcaster contracts
Created event handling functions for Farcaster events

The following files were skipped due to too many changes: packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/events/SetValidator.ts, packages/thirdweb/test/globalSetup.ts, packages/thirdweb/src/extensions/farcaster/signers.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdGateway/events/SetStorageRegistry.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IBundler/read/idGateway.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IStorageRegistry/read/maxUnits.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/read/idCounter.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/read/idGateway.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IBundler/read/keyGateway.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdGateway/read/idRegistry.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IStorageRegistry/read/unitPrice.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/read/idRegistry.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/read/keyGateway.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyGateway/read/keyRegistry.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IStorageRegistry/read/rentedUnits.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyGateway/read/ADD_TYPEHASH.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/read/gatewayFrozen.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/read/gatewayFrozen.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IStorageRegistry/read/usdUnitPrice.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/read/maxKeysPerFid.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdGateway/read/storageRegistry.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/read/REMOVE_TYPEHASH.ts, packages/thirdweb/test/src/chains.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdGateway/read/REGISTER_TYPEHASH.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/read/TRANSFER_TYPEHASH.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IStorageRegistry/read/deprecationTimestamp.ts, packages/thirdweb/src/extensions/farcaster/contracts/getBundler.ts, packages/thirdweb/src/extensions/farcaster/contracts/getIdGateway.ts, packages/thirdweb/src/exports/extensions/farcaster/idGateway.ts, packages/thirdweb/src/extensions/farcaster/contracts/getIdRegistry.ts, packages/thirdweb/src/extensions/farcaster/contracts/getKeyGateway.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/read/CHANGE_RECOVERY_ADDRESS_TYPEHASH.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/read/TRANSFER_AND_CHANGE_RECOVERY_TYPEHASH.ts, packages/thirdweb/src/extensions/farcaster/read/getFid.test.ts, packages/thirdweb/src/extensions/farcaster/contracts/getStorageRegistry.ts, packages/thirdweb/src/exports/extensions/farcaster/storageRegistry.ts, packages/thirdweb/src/extensions/farcaster/read/getUsdRegistrationPrice.test.ts, packages/thirdweb/src/extensions/farcaster/read/getRegistrationPrice.test.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/events/AdminReset.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/read/idOf.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IStorageRegistry/read/price.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/write/remove.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyGateway/read/nonces.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IBundler/read/price.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdGateway/read/price.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/read/custodyOf.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/read/recoveryOf.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/events/Remove.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/events/Register.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/events/Transfer.ts, packages/thirdweb/src/extensions/farcaster/read/getUsdStoragePrice.test.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/events/AdminReset.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/read/keysOf.ts, packages/thirdweb/src/extensions/farcaster/read/getStoragePrice.test.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/read/keyDataOf.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/read/totalKeys.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/write/changeRecoveryAddress.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/events/Recover.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IStorageRegistry/write/rent.ts, packages/thirdweb/src/utils/bigint.test.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IStorageRegistry/write/batchRent.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/events/ChangeRecoveryAddress.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/events/Add.ts, packages/thirdweb/src/extensions/farcaster/read/getFid.ts, packages/thirdweb/src/extensions/farcaster/read/getNonce.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/read/keyAt.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/write/transfer.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdGateway/write/register.ts, packages/thirdweb/scripts/generate/abis/farcaster/IKeyRegistry.json, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/write/recover.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyGateway/write/add.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyRegistry/write/removeFor.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/read/verifyFidSignature.ts, packages/thirdweb/src/extensions/farcaster/read/getStoragePrice.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/write/transferAndChangeRecovery.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/write/changeRecoveryAddressFor.ts, packages/thirdweb/src/extensions/farcaster/read/getRegistrationPrice.ts, packages/thirdweb/src/extensions/farcaster/write/rentStorage.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdGateway/write/registerFor.ts, packages/thirdweb/src/extensions/farcaster/read/getUsdStoragePrice.ts, packages/thirdweb/src/extensions/farcaster/write/registerFid.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/write/transferFor.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/write/recoverFor.ts, packages/thirdweb/src/extensions/farcaster/read/getUsdRegistrationPrice.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IKeyGateway/write/addFor.ts, packages/thirdweb/src/exports/extensions/farcaster/keyRegistry.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IIdRegistry/write/transferAndChangeRecoveryFor.ts, packages/thirdweb/scripts/generate/abis/farcaster/IIdRegistry.json, packages/thirdweb/src/extensions/farcaster/eip712Signatures/registerSignature.ts, packages/thirdweb/src/extensions/farcaster/__generated__/IBundler/write/register.ts, packages/thirdweb/src/exports/extensions/farcaster.ts, packages/thirdweb/src/extensions/farcaster/eip712Signatures/addSignature.ts, packages/thirdweb/src/exports/extensions/farcaster/idRegistry.ts, packages/thirdweb/src/extensions/farcaster/write/addSigner.ts, pnpm-lock.yaml, packages/thirdweb/src/extensions/farcaster/write/addSignerFor.ts, packages/thirdweb/src/extensions/farcaster/eip712Signatures/keyRequestSignature.ts, packages/thirdweb/src/extensions/farcaster/write/registerFidAndSigner.ts

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@jnsdls, I added the changes we discussed this morning, let me know if you get the functions as inputs to generated functions working and I can update those as well

@gregfromstl I'll take a look shortly! thanks

oh and yes, currently working on the functions as inputs!

also looks like there's some test failures but I think that's because in your fork you don't have access to some ENV vars for CI :-/

also looks like there's some test failures but I think that's because in your fork you don't have access to some ENV vars for CI :-/

Yeah the CI tests keep going back and forth between passing and failing every commit I push, not sure what's wrong

yeah the reason these fail is because ur branch does not have access to the repo secrets that make the forking of the RPC work (you're getting ratelimited on the forks)

all tests pass when ran with proper secrets, merging! thanks @gregfromstl - we can iterate further on subsequent PRs!


Multi-node test failure on optimize_spmd_launch_twice.rg

I'm seeing a test failure on optimize_spmd_launch_twice.rg. The test fails by producing an incorrect result. The failure is non-determinstic on sapling: I appear to hit the failure on about 1 in 10 runs. However, the failure appears to be very reliable on Gitlab. It may be that oversubscribing the machine improves the odds of reproducing the bug.
Note:

Despite the name of the test that's failing, SPMD is not required to fail this test. (I have been able to reproduce both with or without SPMD. The bug appears to reproduce more frequently without SPMD: 1 in 10 vs 1 in 100.)
Gitlab is using the MPI conduit, but I have successfully reproduced with both IBV and MPI conduits on Sapling.
So far I have only been able to reproduce with ranks running on the same physical node.
I have not yet been able to reproduce with Legion Spy enabled (after over 500 runs).
I can reproduce with the following Regent optimizations disabled: leaf tasks, inner tasks, RDIR (so no conversion to/from dataflow at all), vectorization.
I can also reproduce on the olddma branch.

Sample failure in Gitlab: https://gitlab.com/StanfordLegion/legion/builds/15192176
Steps to reproduce on Sapling. Run this on the head node. The following configuration has SPMD disabled.
./install.py --debug --gasnet
i=0; while LAUNCHER='mpirun -n 2 -H localhost --bind-to none -x REALM_SYNTHETIC_CORE_MAP= -x INCLUDE_PATH -x LD_LIBRARY_PATH -x TERRA_PATH' ./regent.py ./tests/regent/run_pass/optimize_spmd_launch_twice.rg -ll:cpu 2 -fflow-spmd 0; do let i++; echo $i; done

Sample bad output. Note the bad (zero) entry on the last line. The position of this bad entry is non-determinstic and varies between runs.
x 10000 0 0
x 10010 0 0
x 10020 0 0
x 10030 0 0
x 9669 -88 -747
x 10410 122 918
x 9676 -71 -867
x 0 21 680
assertion failed: test failed

Sample good output:
x 10000 0 0
x 10010 0 0
x 10020 0 0
x 10030 0 0
x 9669 -88 -747
x 10410 122 918
x 9676 -71 -867
x 10385 21 680

Interesting assertion that I noticed while testing, couldn't get a backtrace.  Still need to investigate more.
/local/home/mebauer/legion_versioning/runtime/realm/operation.cc:425: void Realm::OperationTable::event_triggered(Realm::Event): Assertion `it != table.end()' failed.

I've narrowed down the cause of this bug but I still don't understand it.  It has to do with pruning operations from the set of 'current' and 'previous' epoch lists for a physical instance.  Recall that (conceptually) we keep track of two lists of operations for every field of a physical instance.  One for the current epoch of operations (operations which don't interfere with each other) and one for the previous epoch of operations.  When a new operation comes along it looks for operations that it interferes with in the current list.  If it finds no operations or doesn't interfere with all the operations then it does the same analysis on the previous list.  If the operation dominated all operations in the current list then we clear the previous list, make the current list the previous list, and add the operation to the current list.  Otherwise we can just add the operation to the current list directly.
The source of the problem seems to be that as we're running along and doing these tests, we have an optimization that first checks to see if the event for an operation in the list has already triggered. If we detect that the event for an operation has already triggered (by calling 'has_triggered') then we can eagerly prune it from the list without even testing for interference.  Somehow this eager pruning of the lists is causing the non-deterministic failures.  If I disable all the eager pruning of both lists then everything works fine.  Surprisingly even eager pruning of either list individually also causes the bug to manifest (but disabling eager pruning of the previous list has a much greater effect on reducing non-determinism for this test).  I'm completely at a loss as to why pruning at all is causing any issues as those operations have by definition already completed.  Any ideas @elliottslaughter @streichler @magnatelee @TWarszawski @alexaiken?
There are two other potential things going on here.  First, I'm trusting that the 'has_triggered' implementation is correct.  If there is a bug that is causing 'has_triggered' to prematurely signal that an event has triggered then that would explain all of the behavior.  I think this is highly unlikely given our use of 'has_triggered' in other parts of the runtime that all seem to work, but it is worth mentioning.  Second, it's possible that not pruning the lists is just creating more work for the runtime and causing a sufficient timing delay to prevent a race somewhere else from occurring.  I haven't see any evidence of this (e.g. disabling the pruning always makes the non-determinism go away without exception over ~10K tests), but it's still a possibility because I can't rule it out.

Another important observation: enabling/disabling the pruning seems to perfectly predict whether we see the non-determinism or not regardless of whether we run with optimizations on or off.  The massive difference in timing with optimizations on/off would probably perturb a race condition if there was one, so I feel like this is the actual cause of the problem.  I just want to understand why it is a problem before I go changing things.

I can kind of understand why we can't prune from the 'current' epoch list because it might influence when the current epoch list needs to become the previous epoch list.  What I can't understand is why pruning from the previous epoch list causes problems.  This is just completely baffling to me.

Ok, back to thinking it is a timing thing.  Running with Legion Spy but keeping the pruning in and 150 runs later I still have yet to see the non-deterministic behavior.  Clearly the cost of logging for Legion Spy prevents the non-deterministic behavior from occurring even with the pruning in.
Another hypothesis: one unusual thing about this example is it has a number chained cross-node copies.  I'm wondering if there is a race somewhere in the multi-node copy code that is causing the finish event to trigger before all the data is actually in place in the destination.

More evidence for a bug in Realm: the non-determinism occurs even when running with -lg:inorder to force in order execution of the Legion tasks (and all their copies) in strict program order.

Switching from index space task launches to individual task launches using -findex-launch 0 causes the non-determinism to go away regardless of whether we run with -lg:inorder or not.  It's unclear if this is due to a timing difference or a bug in the index space launch code somewhere.

With -findex-launch 0, is the mapper even sending tasks to remote nodes? This test just uses the default mapper, and I don't think the default mapper does anything fancy here.

Yes, it's still spreading the tasks around cause I can see the copies across nodes in the event graph.

  I know that it is not caused by index close operations.  I was able to confirm that the issue goes away with local mapping suggesting that the problem involves some kind of meta-data movement (if all the meta-data is on one node then we don't have any issues).  I then went and investigated the composite instances and how they are migrating the view trees.  They all seem to be doing the right thing and bringing in the right meta-data.  That leaves the next suspect which is migration of the materialized view meta data for the specific physical instances being used. I will look into that tomorrow.

@streichler @jiazhihao Can one of you guys look at these dma log files from a good and a bad run and tell me if the same number of copy requests where generated?
dma_logs.tar.gz

I've confirmed that Legion is issuing all the appropriate copies in every failed run.  That leaves two potential sources for the non-determinism: either Legion is not specifying the proper event dependences for the copies which is causing a race or there is something racing in the DMA system to cause the data not to be in the proper location/arrive late/who knows.

Also happens with structured index spaces so nothing to do with Realm bit masks.
optimize_spmd_launch_twice.txt

@elliottslaughter Can you try to reproduce the bug with the olddma branch? I just ran 500 iterations without it reproducing for me in the olddma branch.  In the master branch it usually reproduces within the first 10 tries.

@jiazhihao Using registered memory for all the instances instead of system memory significantly reduced the probability of the non-determinism happening (went from 1 in 5 to about 1 in 60), but it still did occur.  This means the race is still there (either in the DMA system or the Legion runtime), just that the timing was affected by the change in mapping decisions.

It took longer, but I repro'd on the olddma branch after 625 iterations. I don't remember it taking so long before, maybe there was other activity on the machine, or something.

This bug should be fixed with 3798b00. The moral of the story is that you can't ever trust Realm::merge_events to give you back a unique event.  In fact, it may give you back one of the events that you asked it to merge if it is the only one that hasn't triggered yet.  All of this I knew, but just didn't fully remember when writing some of the event merging code for composite instance views.  Assigning back to @elliottslaughter to confirm the fix.

I can't reproduce any more on sapling (after 600+ runs), but unfortunately I'm still seeing this failure in gitlab:
https://gitlab.com/StanfordLegion/legion/builds/15396524

There's some unrelated junk in that log output that I'll have to look at separately. For the purpose of this bug, search the log for "spmd_launch_twice".

Update: The bug with -fflow-spmd 0 appears to be gone. The crashes above are only happening with -fflow-spmd 1. They might be a different bug.

Here are the Legion Spy logs from a run that gets the wrong answer with -fflow-spmd 1.  Legion Spy validates the runtime's execution.  It does look weird because I don't see any of the compiler's normal generation of explicit copies into and out of the must-epoch launches.
bad_logs.tar.gz
dataflow_main_2.pdf
event_graph_main_2.pdf

@lightsighter The explicit copies into shards are now being done inside each shard. You can see from your event graph that each shard starts with some copies.

@magnatelee So I see the copies in at the start of the shard tasks, but I don't see the copies back out to the original regions at the end of the shard tasks.

@magnatelee @elliottslaughter Also, how are you guys guaranteeing that all the copies-in happen before any copy-out from a different shard is done?  Do you have explicit phase barriers for that?  I don't think you can trust that such an ordering will be enforced by the data dependences inside of all the shards unless you explicitly analyze that.

@lightsighter the copies out of a must-epoch launch are inserted only when the partition bloated by the spmd transformation is not completely covered by any other partitions. I guess in this test case the aliased partition that spmd choses to bloat is completely covered by the primary partition. The ordering between the must-epoch tasks and the copy-outs is guaranteed by data dependence and the copies into shards are explicitly synchronized with inlined phase barrier awaits at the beginning of shards. I believe the placement and synchronization of copies are both deterministic and correct.

I still have questions: first where are the copy outs?  I can't see any copy outs in any of the shard tasks from any of the three must epoch launches.
Second: there is no guarantee of ordering between the must epoch launches.  A shard from the second must epoch launch can start running as soon as the processor that it is mapped to allows it to start running.  This follows from  simultaneous coherence.  This means that the execution of the shards from different must epoch launches can be interleaved and the only way to enforce ordering across shards from different must epoch launches is with phase barriers.  There is no dependence enforced by the runtime.  You can see this in the event graph: the shards from all three must epoch launches are interleaved with each other.

I have what I hope is a fix in aa44670aae2e91cf795344b2974da0119c3e65c0. The hack for now is to add execution fences before and after every must epoch launch. In theory, it would be cleaner to use an inner task with exclusive coherence, but I'm concerned that approach would have a much higher risk of introducing performance bugs.

This issue appears to be fixed; I have not seen this test fail on the last several commits in Gitlab.


Add prompting for missing template parameters.

Fixes https://github.com/Azure/azure-cli/issues/1778

Codecov Report

Merging #2364 into master will decrease coverage by -0.22%.
The diff coverage is 28.15%.

@@            Coverage Diff             @@
##           master    #2364      +/-   ##
==========================================
- Coverage    72.2%   71.99%   -0.22%
==========================================
  Files         323      323
  Lines       18098    18186      +88
  Branches     2654     2680      +26
==========================================
+ Hits        13068    13093      +25
- Misses       4206     4266      +60
- Partials      824      827       +3

Impacted Files
Coverage Δ

src/azure-cli-core/azure/cli/core/prompting.py
30.12% <23.52%> (-6.05%)
:x:

...ource/azure/cli/command_modules/resource/custom.py
54.46% <32.69%> (-3.03%)
:x:

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 5f77858...cce6005. Read the comment docs.

Could you provide a print of the new -h output for the revised command, as well as an example of the output the command produces when prompting for deployment parameters?

@tjprescott here's the new help:

Command
    az group deployment create: Start a deployment.

Arguments
    --resource-group -g [Required]: Name of resource group.
    --mode                        : Incremental (only add resources to resource group) or Complete
                                    (remove extra resources from resource group).  Allowed values:
                                    Complete, Incremental.  Default: incremental.
    --name -n                     : The deployment name. Default to template file base name.
    --no-wait                     : Do not wait for the long running operation to finish.
    --parameters                  : Provide deployment parameter values, either json string, or use
                                    '@<file path>' to load from a file.
    --prompt                      : If true, prompt for parameters in the template that are missing
                                    values.
    --template-file               : A template file path in the file system.
    --template-uri                : A uri to a remote template file.

Global Arguments
    --debug                       : Increase logging verbosity to show all debug logs.
    --help -h                     : Show this help message and exit.
    --output -o                   : Output format.  Allowed values: json, jsonc, table, tsv.
                                    Default: json.
    --query                       : JMESPath query string. See http://jmespath.org/ for more
                                    information and examples.
    --verbose                     : Increase logging verbosity. Use --debug for full debug logs.

Examples
    Create a deployment from a remote template file.
        az group deployment create -g MyResourceGroup --template-uri
        https://myresource/azuredeploy.json --parameters @myparameters.json

    Create a deployment from a local template file and use parameter values in a string.
        az group deployment create -g MyResourceGroup --template-file azuredeploy.json --parameters
        "{\"location\": {\"value\": \"westus\"}}"

Comments mostly addressed, please re-check.

@tjprescott Here's the example output:
> .\az group deployment create --template-file=..\azuredeploy.json --prompt -g foo
Please provide a value for 'sshRSAPublicKey' (? for help): ?
SSH public key used for auth to all Linux machines.  Not Required.  If not set, you must provide a password key.
Please provide a value for 'sshRSAPublicKey' (? for help): fakekey
Please provide a value for 'linuxAdminUsername' (? for help): fakeuser
Please provide a value for 'clientPrivateKey' (? for help):
The base 64 client private key used to communicate with the master
Please provide a value for 'clientPrivateKey' (? for help):
Please provide a value for 'apiServerCertificate' (? for help):

@tjprescott @derekbekoe can you folks decide which behavior (default prompt vs default non-prompt) you want, and I'll implement?

@brendandburns agree with @derekbekoe. Flag isn't necessary for any reasonable use case.

Ok, I removed prompt_for_params please take another look.
--brendan


improve widget highlighting

An improved (still hacky) way of programmatically highlighting a widget.
Rather than inserting a new widget into the hierarchy, this chases down the widget's renderObject and uses renderObject.debugHandleEvent to highlight and unhighlight.
@lukechurch

LGTM, thanks!


WHERE Clause not working on enum type

Hey there, I'm running into an issue with the Electric-sql, and I've put together a minimal reproduction using Next.js to help illustrate the problem. I've included the schema for reference.
For some context, my main project is built with SvelteKit, and I initially thought the issue might be related to my Svelte implementation of the electric-sql TypeScript client. However, after creating this Next.js example, it seems the problem might be on enum columns.
Basically, any column defined as an enum in the schema is causing issues, while other column types are working just fine. I'm not entirely sure what's going on, but I wanted to provide this reproduction to see if anyone has encountered a similar problem or has any insights to share.
https://github.com/ihsanmohamad/react-electric-repro

Hey, sorry you're running into issues. Could you modify your README with instructions on how to run the reproduction? Ideally a single command can e.g. start a docker compose file and run the migration. I don't know how to run your app as is.

ok, will do tomorrow.

hi I updated the repo. I also add the README with instructions. I'm sorry I am unable to make fully docker compose as I'm not familiar with docker and nextjs. I'll try to work on docker compose again if you have issues with the steps.

We got a similar bug report on Discord yesterday. Could this be related?

that was me 😁


Matter.Bounds.create is setting min and max to infinity

Hi,
I'm using the following code to setup bounds for a viewport:
let viewportBounds = Matter.Bounds.create( { min: { x: 0, y: 0 }, max: { x: 800, y: 600 } });
When I check viewportBounds, after the above code has run, I see viewportBounds max and min are set to infinity, why is that and what is the proper way to create a bounds?

Why not add static wall bounds?
https://github.com/liabru/matter-js/blob/master/examples/airFriction.js#L41

@liabru
var bounds = Bounds.create( { min: { x: 0, y: 0 }, max: { x: s.width, y: s.height } })
World.add(this.engine.world, bounds)
console.log(bounds);

@vostokwork the API for Bounds.create is a little different from the others, check it out in the docs (check the first argument). I'd like to make it consistent eventually when the project is ready for breaking changes :)
In this case you can just go ahead with { min: { x: 0, y: 0 }, max: { x: s.width, y: s.height } } as your bounds object.
One other thing, you don't need to add it to the world.

@liabru But this is doen't work.
I use p5.js for render.
Can you give me an example?

@liabru
Like this? Doesn't work too.
Bounds.create([
    { x: 0, y: 0 },
    { x: s.width, y: 0 },
    { x: s.width, y: s.height },
    { x: 0, y: s.height },
    { x: 0, y: 0 },
])

@vostokwork no, if you just need to create some bounds just can just use the plain object:
const bounds = { min: { x: 0, y: 0 }, max: { x: s.width, y: s.height } }
The Bounds.create(vertices) is meant for automatically calculating bounds from Bodies.vertices, which you don't need here.


Support to show  keyboard keys based on locales

This fixes partially issue #110 and #65.
Keys are now showed after transforming them with win32 KeyCodeToUnicode.
If a ctrl or alt is used it defaults to the lowercase default letter => shortcut
If shift is pushed without ctrl or alt it shows directly the mapped character, as was handled before with the hardcoded list, but now it is locale specific and automatic.
However there are some improvements which could be added:

correct handling of dead keys
Keymaps parsing still uses the hardcoded values.
alt-gr shows as ctrl + alt + base key instead of special character

Based on the tests shortcuts letters have to be capital?


contrib/k8s: add service and rc definition for khealth

rc has a single rcscheduler pod as of now. we can add more containers and ports as we add more modules

Probably a good idea to include a namespace creation thing too.

LGTM


No tag for release 0.17.1

0.17.1 is released but untagged here.

seems to have been tagged at some point, thanks


Bugfixes and addition after OSCAP run

Also, there is quite a bit of merge conflicts, you need to rebase as I see you're contributing things which have already been addressed in the Master branch

Ah, at first there was only a devel branch. Do we rebase off master now?

Oops sorry, I meant devel

The sshd fixes are breaking so +1.

Another day wasted.


Add support for overriding the number of generated rows in DataTables

This PR:

Adds new configuration method .WithDataTableRowCount to the config builder interface.
Implements it in the config object.
Adds unit tests that can only pass when it is properly implemented.
Updates the DataTable and DataSet generators to implement it, making the tests pass.
Includes a corresponding update to the documented API surface in AutoBogus.xml.

Hello @nickdodd79 :-) Do you have a bit of time to check out the four additional PRs that I created in response to using the new System.Data functionality in our codebase? (This being one of them :-) )

Hello @nickdodd79 :-) Do you have a bit of time to check out the four additional PRs that I created in response to using the new System.Data functionality in our codebase? (This being one of them :-) )

Just another ping @nickdodd79, we are actively using these features through a private build on our internal artefacts server, and it would be nice to be able to go back to the mainline, if these changes are agreeable to you. :-)

Hey @logiclrd
Apologies for the delay. I was out of action for a bit and have been slowly getting back into things. Unfortunately a couple of other project have taken my full focus, meaning I have kept tabs on this one - plus my GitHub notifications seemed to have stopped.
Nick.

Welcome back, in that case! :-) Hope everything's okay.


Document a-star

Comment from Bret:
"Astar works on cost, the higher the cost the less likely it is to travel that path. There are no hard limits, it works on minus infinity and positive infinity. It will take a starting position and and end position, then find the path between the two with the lowest cost. This is perfect for heightmaps for example, because you can use it to find path through mountain/hills between villages. ;)"

https://github.com/Mindwerks/worldengine/commit/d36e2fde0c8505822b8b7ef2493f3d1b8311f1e9


Spike and Sail are getting different initial values of misa

Hello,
I am writing a basic test of checking read/write access of MISA reg in RISCOF. I am at M-mode, reading the initial values of misa, then updating its value, and re-reading it in the attached test file (.elf also attached). Spike and Sail was getting different initial values in RISCOF. For reference, signature files of spike and sail are also attached for the written test.
CSR MISA value for Spike ==> 0x40141104 (RV32IMC)
CSR MISA value for Sail ==> 0x4014112D (RV32IMACDF)
issues.zip

Sail will eventually initialize its MISA values based on the ISA string
embedded in the riscv-config YAML, but right now it appears to default to
D&F enabled. I'm not even sure it can be turned off
Spike probably needs to explicitly enable it in a command-line argument. It
would be nice to get them onto the same page, and work is afoot to do this
in Sail, but not spike.
On Mon, Sep 26, 2022 at 5:08 AM Danish Hussain @.***>
wrote:

Hello,
I am writing a basic test of checking read/write access of MISA reg in
RISCOF. I am at M-mode, reading the initial values of misa, then updating
its value, and re-reading it in the attached test file (.elf also
attached). Spike and Sail was getting different initial values in RISCOF.
For reference, signature files of spike and sail are also attached for the
written test.
CSR MISA value for Spike ==> 0x40141104 (RV32IMC)
CSR MISA value for Sail ==> 0x4014112D (RV32IMACDF)
issues.zip
https://github.com/riscv-non-isa/riscv-arch-test/files/9646025/issues.zip
—
Reply to this email directly, view it on GitHub
https://github.com/riscv-non-isa/riscv-arch-test/issues/287, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPXVJRTAJB7FYP7FKLSEOTWAGG3HANCNFSM6AAAAAAQVXLFO4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Documentation Links

There are a few spots in the docstring comments in arbiter-core that have a TODO: to link to external documentation. We shouldn't forget to do this after we write external docs for arbiter.

Closed by #378


CMCC - Central Maine Community College

Central Maine Community College - CMCC
https://cmconnect.cmcc.edu/ICS (matching subdomain of student email)
https://www.cmcc.edu/ (Main CMCC website)

@gnumetal Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Create new qualifying test report

As an admin
I want to create a qualifying test report
So that I can analyse scores across one or more tests

Form to allow the creation (and editing) of a qualifying test report.
The form will allow the selection of one of more qualifying tests for the current exercise. The tests should be 'complete' and should not be 'mop up' tests.
A query like the following will cover this:
.where('status', '==', 'completed')
.where('mode', '!=', 'mop-up')

Data model for qualifyingTestReport
exercise: {
  id: String,
},
qualifyingTests: [
  {
    id: String,
    title: String,
    maxScore: Number,
  },
],

This relies on digital-platform#419 so cannot be tested yet. Have moved to 'blocked' whilst we complete that work.

This is now complete.


Stencil: Add Library Error

Hi,
Describe the bug
I tried to set up a new workspace and added a stencil lib but got the following error:
Cannot find module '@angular-devkit/core'
Require stack:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>
To Reproduce
➜ npx create-nx-workspace --preset=empty

➜ npm install @nxext/stencil --save

➜ nx g @nxext/stencil:library ui-components

Used latest nx version @nrwl/workspace": "12.0.8"
I also tried with nx version 11.6.3 but got the same error:
Expected behavior
Added stencil library to workspace

Thanks for the Issue, there was a missing dependency


Support for Unicode labels

Adding support for mailbox with non-ascii name

@googlebot I signed it!

@googlebot I signed it!

Thanks!


Compile on save not working

I just uninstalled completely and reinstalled Atom, then installed Atom-Typescript and its dependencies, so I'm using latest versions. The tsconfig.json file has the compileOnSave option set to true, however the feature simply doesn't work.
If I press F6 to run "build", all I see is a red box on the bottom side that says "Emit Failed" without any more information.
It was working before (over a month ago), before this plugin required "lint".

Do you have any output in the console? You might be using a little older version of Typescript, say, 1.8? Right now it fails silently if the compiler API doesn't support the right command.

How do I know which version of Typescript is Atom using? According to the FAQ it uses the one in node_modules? It was 1.8, I updated it to the latest which is 2.2.2, but it still doesn't work.
An error is logged upon file save:

It should show up next to the selected grammar on the bottom right:

This is something different, though. If you click on the little (...), it'll show you where actually the error was thrown which would help diagnose this.

Oh, I didn't realize the dots show the rest of the error message while the arrow is for the stack trace, I'll paste the message below. As for the version, it says 2.2.2.
This is unrelated, but everytime I open any file that isn't a ts file, an error saying Missing filePath or not a Typescript file appears. Shouldn't the extension try to deal only with .ts files?
TypeError: Cannot read property 'name' of undefined
    at mergeSymbol ({...}\node_modules\typescript\lib\tsserver.js:21546:46)
    at {...}\node_modules\typescript\lib\tsserver.js:21570:21
    at Map.forEach (native)
    at mergeSymbolTable ({...}\node_modules\typescript\lib\tsserver.js:21560:20)
    at initializeTypeChecker ({...}\node_modules\typescript\lib\tsserver.js:37494:21)
    at Object.createTypeChecker ({...}\node_modules\typescript\lib\tsserver.js:21430:9)
    at Object.getTypeChecker ({...}\node_modules\typescript\lib\tsserver.js:52546:79)
    at ConfiguredProject.Project.getReferencedFiles ({...}\node_modules\typescript\lib\tsserver.js:70333:48)
    at ModuleBuilder.getReferencedFileInfos ({...}\node_modules\typescript\lib\tsserver.js:69693:56)
    at ModuleBuilder.updateFileReferences ({...}\node_modules\typescript\lib\tsserver.js:69724:42)
    at mergeSymbol ({...}\node_modules\typescript\lib\tsserver.js:21546:46)
    at {...}\node_modules\typescript\lib\tsserver.js:21570:21
    at Map.forEach (native)
    at mergeSymbolTable ({...}\node_modules\typescript\lib\tsserver.js:21560:20)
    at initializeTypeChecker ({...}\node_modules\typescript\lib\tsserver.js:37494:21)
    at Object.createTypeChecker ({...}\node_modules\typescript\lib\tsserver.js:21430:9)
    at Object.getTypeChecker ({...}\node_modules\typescript\lib\tsserver.js:52546:79)
    at ConfiguredProject.Project.getReferencedFiles ({...}\node_modules\typescript\lib\tsserver.js:70333:48)
    at ModuleBuilder.getReferencedFileInfos ({...}\node_modules\typescript\lib\tsserver.js:69693:56)
    at ModuleBuilder.updateFileReferences ({...}\node_modules\typescript\lib\tsserver.js:69724:42)
    at MessageStream.TypescriptServiceClient.onMessage (C:\Users\aster\.atom\packages\atom-typescript\dist\client\client.js:40:41)
    at emitOne (events.js:96:13)
    at MessageStream.emit (events.js:188:7)
    at readableAddChunk (_stream_readable.js:176:18)
    at MessageStream.Readable.push (_stream_readable.js:134:10)
    at MessageStream.Transform.push (_stream_transform.js:128:32)
    at MessageStream._transform (C:\Users\aster\.atom\packages\atom-typescript\dist\client\client.js:198:18)
    at MessageStream.Transform._read (_stream_transform.js:167:10)
    at MessageStream.Transform._write (_stream_transform.js:155:12)
    at doWrite (_stream_writable.js:307:12)
    at writeOrBuffer (_stream_writable.js:293:5)
    at MessageStream.Writable.write (_stream_writable.js:220:11)
    at LineStream.ondata (_stream_readable.js:555:20)
    at emitOne (events.js:96:13)
    at LineStream.emit (events.js:188:7)
    at readableAddChunk (_stream_readable.js:176:18)
    at LineStream.Readable.push (_stream_readable.js:134:10)
    at LineStream.Transform.push (_stream_transform.js:128:32)
    at LineStream._pushBuffer (C:\Users\aster\.atom\packages\atom-typescript\node_modules\byline\lib\byline.js:126:17)
    at LineStream._transform (C:\Users\aster\.atom\packages\atom-typescript\node_modules\byline\lib\byline.js:117:8)
    at LineStream.Transform._read (_stream_transform.js:167:10)
    at LineStream.Transform._write (_stream_transform.js:155:12)
    at doWrite (_stream_writable.js:307:12)
    at writeOrBuffer (_stream_writable.js:293:5)
    at LineStream.Writable.write (_stream_writable.js:220:11)
    at Socket.ondata (_stream_readable.js:555:20)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)
    at readableAddChunk (_stream_readable.js:176:18)
    at Socket.Readable.push (_stream_readable.js:134:10)
    at Pipe.onread (net.js:543:20)

That actually seems like an error in the compiler. This might be it: https://github.com/Microsoft/TypeScript/issues/15128
That Missing filePath or not a Typescript file error is completely normal though it's a little confusing so I'll get rid of it.

After looking the message I thought the error isn't related to this extension. But the linked issue shows no fixes. Is there a fix for this?

Unless you want to use a different Typescript version, you best bet is to wait for the Typescript team to fix it. There isn't anything atom-typescript could do except maybe provide a more useful error message.

Alright, thank you for your time and help.


Add access operator (->, ->>) to Agtype

Added the ->, ->> operator to agtype for approaching like Json(b).

The primary concern I have is that the cypher command itself will not use these operators as is. Cypher by default only uses Agtype, so I believe these operators and indices built on these operators will not be usable by cypher, rather only standard sql queries.


Problem using output in a pipeline

I am using the plugin to look at the redis plans
cf traverse service_offering instances_of redis
which works as expected, however when attempting to pipe the stdout into another process the data is not there
cf traverse service_offering instances_of redis > foo.txt
wc -l foo.txt
 0 foo.txt

cf traverse service_offering instances_of redis  2> foo.txt
(venv) GDS3821:cf-traverse pauldougan$ wc -l foo.txt
19793 foo.txt

looks like the output is going to stderr

Thanks, Paul. I hadn't noticed that! Resolved in release 0.7.0

Thanks, Paul. I hadn't noticed that! Resolved in release 0.7.0


deps: update karma-webpack

The current version of karma-webpack has a peer dependency
conflict with the local version of webpack. This causes
installs of this repo to fail with npm 7.
Updating karma-webpack to ^3.0.0 resolves this conflict and
all of the tests seem to still pass 🎉
There is a 4.x release of karma-webpack but I figured minimal
improvement lowered the chance of breaking stuff

Coverage remained the same at 93.711% when pulling 510e39f370f030f9a6f01ac92d200e146788f507 on MylesBorins:update-karma-webpack into 675006d4a46085122994cd3e64731d5a0a3667e7 on FormidableLabs:master.


limactl factory-reset: keep "lima-version" and "protected"

Fix #2656

But I'm wondering if limactl protect shouldn't also protect against a factory reset. I know, it is a separate issue, but would be a trivial 3-liner to add to this PR.

done


Misc thoughts and suggestions

I'm curious if rotations and reflections of the board would give the same value and policy outputs.  If they give different values would be interesting to explore if averaging them converges to the correct result faster.  I suspect human cognitive biases result in humans playing stronger for particular orientations than for others.
Another thought was predicting score for parts of the board and using those predictions to predict the overall value.  Again this seems like it might be a way to converge to the correct answer faster.  The same edge/corner relationships will occur in all four corners and four edges.
I noticed that the paper didn't seem to use CLOP for parameter tuning (not that they mention many parameters to tune).  It has given nice boosts in strength for other Go programs, might be worth looking at.
CLOP: Confident Local Optimization for Noisy Black-Box Parameter Tuning
http://www.remi-coulom.fr/CLOP/
<EMAIL_ADDRESS>

Rotations/reflections are actually used by DeepMind's AlphaGo already! They do this during training to get more examples and during evaluation like this:

we exploit symmetries at run-time by dynamically transforming each position s using the dihedral group of eight reflections and rotations, d1(s), …, d8(s). In an explicit symmetry ensemble, a mini-batch of all 8 positions is passed into the policy network or value network and computed in parallel.
(Silver et al. page 8)

Not sure what you mean by predicting scores for parts of the board - could you elaborate?
Optimizing hyperparameters parameters for a speed/accuracy tradeoff would not be a bad idea. Thanks for the link. Something to keep in mind once we get to a prototype.


"No service hubot  is running!!!  No service found!"

I'm trying to use hubot steps for jenkins file, But I keep having this message
"No service hubot  is running!!!
No service found!"
looking at the code seems like it could be missing the env variables
https://github.com/fabric8io/fabric8-jenkins-workflow-steps/blob/746d06c753f006901ce34f8ed5204107adc9b81f/src/main/java/org/jenkinsci/plugins/fabric8/support/RestClient.java#L68
https://github.com/fabric8io/fabric8-jenkins-workflow-steps/blob/746d06c753f006901ce34f8ed5204107adc9b81f/src/main/java/org/jenkinsci/plugins/fabric8/support/HubotClient.java#L71
but even this does not work:
node {
  withEnv(['HUBOT_SERVICE_HOST=<IP_ADDRESS>', 'HUBOT_SERVICE_PORT=80']) {
      sh 'env | sort'
      hubot room: 'test', message: "release started"
  }
}

and the env variables are actually printed
any ideas?

@enxebre you may try the new plugin, which does work with the example that you are trying out here.
https://github.com/jenkinsci/hubot-steps-plugin


fix: remove nullability from id fields

Identifier values across all models cannot be null, thus this PR removes nullability from all id attributes

:tada: This PR is included in version 1.2.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


UI - Add ability to access SSH keys to EKS driver

What kind of request is this (question/bug/enhancement/feature request):
Enhancement
Steps to reproduce (least amount of steps as possible):
keyPairName is the new resourceField to be used
Result:
Able to specify a previously created EC2 ssh key pair.
Other details that may be helpful:
https://github.com/rancher/rancher/pull/17868
https://github.com/rancher/kontainer-engine/pull/125
https://github.com/rancher/rancher/issues/14598
┆Attachments: image-20190429-155702.png | image-20190429-160518.png

The ssh key is for the nodes in the cluster and so I think it should go under the Nodes section not Cluster Options. Could we change this?

@davidnuzik that makes sense I'll make the change.

➤ David Nuzik commented:
Version: Master (4/29/19)
Confirmed the “SSH Key” drop down was added and then moved down to the Nodes section (I requested that change). The list enumerated correctly and I saw my key and was able to choose this. HTTP/API looks good.
HTTP/API for the cluster:
keyPairName: "David.nuzik-us-west-2"Screenshot of this:
!image-20190429-155702.png|width=614,height=180!
FYI in the API this is of type String, as expected. Also, you can start typing in the drop-down to filter through all the results which makes life a lot easier.
!image-20190429-160518.png|width=302.5,height=71!
You must pick an item from the drop-down however (which is good as this should prevent mistakes due to mistype). For example as pictured above I cannot submit with “David.nu” if I click off the field it will blank out again since I did not pick an item from the drop-down menu.
I made sure I could connect to the nodes via SSH without any issues.
NOTE: Your security group must be configured to allow inbound port 22 (SSH) – I had to manually change the default SG to allow this before I could connect as user ec2-user.


The latest update automatically turned off the 'Enable Keyboard Manager' option

Microsoft PowerToys version
0.68.1
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
Keyboard Manager
Steps to reproduce
Set a keyboard shortcut.
Update from the previous to the latest version.
✔ Expected Behavior
Keep all the settings as they were before the update process.
❌ Actual Behavior
Deactivated the 'Enable Keyboard Manager' option.
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/24339


update

wandb experiment integration with github workflow , pr comment will be generated based on wandb comparison report and model registry url

/wandb 2acwxb8n

/deploy


lint failure: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)

https://travis-ci.org/github/theupdateframework/tuf/jobs/719237881
pylint seems to have been upgraded on travis -- or pip on python3.8 now installs pylint 2.6.0 based on the test output. This seems to make linter fail because of the added raise-missing-from check:
************* Module tuf.formats
/home/travis/build/theupdateframework/tuf/tuf/formats.py:958:4: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)
/home/travis/build/theupdateframework/tuf/tuf/formats.py:964:4: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)
************* Module tuf.repository_lib
/home/travis/build/theupdateframework/tuf/tuf/repository_lib.py:544:4: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)
/home/travis/build/theupdateframework/tuf/tuf/repository_lib.py:573:4: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)
/home/travis/build/theupdateframework/tuf/tuf/repository_lib.py:613:4: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)
/home/travis/build/theupdateframework/tuf/tuf/repository_lib.py:669:4: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)
************* Module tuf.keydb
/home/travis/build/theupdateframework/tuf/tuf/keydb.py:345:4: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)
************* Module tuf.scripts.repo
/home/travis/build/theupdateframework/tuf/tuf/scripts/repo.py:458:6: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)
************* Module tuf.client.updater
/home/travis/build/theupdateframework/tuf/tuf/client/updater.py:1589:10: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)

We should consider "raise from" (although it's a bit of a pain with python2) or disable the linter check.

six.raise_from(exc_value, exc_value_from) is probably the reasonable choice: it keeps old functionality for py2 but uses 'raise from' on py3

Thanks, @jku, six is already used in the project for python 2.x compatibility so it does seem the best fit.
I opened #1116 based on your suggestion.

Kindly fixed in #1116. Thanks for the efforts @jku and @sechkova!


API examples

This generalizes the autoRIFT API example previously created for production HyP3.
GitHub view of the markdown: https://github.com/ASFHyP3/ASFHyP3/blob/sdk-examples/docs/using/api.md

Also
Updates the build and deploy workflow to build on PRs and all workflow dispatches (so we can have a pre-deployment check), and only deploy on main.

Converting back to draft because when I render these changes locally, in both FF and Chrome, our header looks like:

Nothing I've done here should have affected the header, so some theme troubleshooting needs to happen before shipping

I've backed out the SDK examples for now


Avoid needless selections and clear on transition

fix #739
Looks like this was caused by a deprecated command in SimpleTable, which selected the entire screen when it was supposed to select and inject text into a specific element. Updated to the latest browser features and it worked fine!

Ahhhh
So much better 😆


Give command when?

The bot needs some economy changes, because one its getting boring with no commands since a long time.
And secondly, you can't actually give coins to someone, then why even collect coins? Items are there but a give command which can give people money would work much better and would bring some new features into the economy commands.
So, is there going to be a give command? Maybe a command that can even gift items from one user to another, that way people can buy and discuss stuff (like if a product is not enabled, someone must have it which is why that will sell) and some more things which can possibly make economy better.
Tell me what you think about it?

You know you can make a pull request with code for a gift or give command right....

You know you can make a pull request with code for a gift or give command right....

yes but I don't have the code, so I can't right?

but @BobDotCom <--- he can!

@NightZan999 you do realize you are on the open source repo of this bot, aka you do have the code, right?

@NightZan999 you do realize you are on the open source repo of this bot, aka you do have the code, right?

yes, but in the economy.py doesn't have a give command

@NightZan999 you do realize you are on the open source repo of this bot, aka you do have the code, right?

yes, but in the economy.py doesn't have a give command

then make one 😮

@NightZan999 you do realize you are on the open source repo of this bot, aka you do have the code, right?

yes, but in the economy.py doesn't have a give command

then make one 😮

okay, hmm I'll try ig


Requesting changes in permission section

InputbufferChanges.docx
HI Team,
I have added an attachment to describe one use-case scenario specific to the permission part of this article and what steps were taken to resolve the issue. Please review and let us know if the mentioned changes can be made to improve the documentation.

@rwestMSFT
Can you review the proposed changes and update the article if needed?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@sunainaTA We do not accept Word documents as attachments. Please make your edits inline, in Markdown.
You can learn about contributing to Microsoft Docs in the contributor guide.
#please-close

@sunainaTA Just following up to say we'll open an internal issue to track this. Thank you for your feedback.


fixing the packs-sdk build

make build and make docs were not working properly.

I made eslint less strict about what happens in documentation/examples/...

I can't repro this at head either, make docs and 'make lint` run without error.

@ekoleda-codaio make bs then make build breaks on head for me.
I'll look into the broken docs...

I wonder if we would just want to disable linter for documentations/ at all.

Anybody have a preferred way forward here? Should we revert the offending typedoc-plugin-markdown? Or do they have a fix already that we can fast-forward to?

It seem we could either a) pin typedoc and the plugin to the last working version of b) turn off the option allReflectionsHaveOwnDocument, which works around the bug.

There is no fix in their github that i can see.
Also, I reverted to the old 3.10.4 dependency and regenerated the docs but that didn't fix this issue, so maybe i have a local env issue...

I've opened an issue on their repo: https://github.com/tgreyuk/typedoc-plugin-markdown/issues/253

@ekoleda-codaio nice find! I set allReflectionsHaveOwnDocument: false here now. WDYT?

I'd say maybe let's just pin to the old version until the issue is resolved.

It doesn't look great IMO (https://coda-packs-sdk.readthedocs-hosted.com/en/patrick-fix-build-docs/reference/sdk/) but it's a fine workaround until they fix the bug.


Build does not complete - floskell, ormolu, hlint failures

I got errors when building just now, too.
In Windows 10, I cloned this repo, then did stack install.
...
... (snip) ...
...
brittany          > Installing executable brittany in C:\sr\snapshots\0b20af29\bin
brittany          > Registering library for brittany-<IP_ADDRESS>..

--  While building package floskell-0.10.2 using:
      C:\sr\setup-exe-cache\x86_64-windows\Cabal-simple_Z6RU0evB_<IP_ADDRESS>_ghc-8.8.2.exe --builddir=.stack-work\dist\29cc6475 build --ghc-options " -fdiagnostics-color=always"
    Process exited with code: ExitFailure 1

--  While building package ormolu-<IP_ADDRESS> using:
      C:\sr\setup-exe-cache\x86_64-windows\Cabal-simple_Z6RU0evB_<IP_ADDRESS>_ghc-8.8.2.exe --builddir=.stack-work\dist\29cc6475 build --ghc-options " -fdiagnostics-color=always"
    Process exited with code: ExitFailure 1

--  While building package hlint-2.2.11 using:
      C:\sr\setup-exe-cache\x86_64-windows\Cabal-simple_Z6RU0evB_<IP_ADDRESS>_ghc-8.8.2.exe --builddir=.stack-work\dist\29cc6475 build --ghc-options " -fdiagnostics-color=always"
    Process exited with code: ExitFailure 1
Progress 8/14

Any ideas about what's going wrong here? I'm not an experienced Haskell developer.

It looks like the GHC version it's using is 8.8.2.
$ stack ghc -- --version
Stack has not been tested with GHC versions above 8.6, and using 8.8.2, this may fail
Stack has not been tested with Cabal versions above 2.4, but version <IP_ADDRESS> was found, this may fail
The Glorious Glasgow Haskell Compilation System, version 8.8.2

When I cloned, I did not add the submodule flag.
That is I did this: git clone https://github.com/haskell/haskell-ide-engine
Where the README says to do this: git clone https://github.com/haskell/haskell-ide-engine --recurse-submodules
Could this be part of the problem?

It looks like the current version of the .gitmodules file is empty, so maybe submodules are no longer used in the project?
https://github.com/haskell/haskell-ide-engine/blob/6a12fb05da8c2ea9647b84a4289b0e77cd3643c1/.gitmodules

Submodules are no longer needed, indeed.

I got the build to work using these commands:
# Remove the git clone directory which had the failing build.
$ rm -rf hie

# Clone it again into C:\hie
$ git clone https://github.com/haskell/haskell-ide-engine --recurse-submodules hie

$ cd hie
# Make another mistake because the build instructions say it uses shake, so I try to reverse the command.
$ stack install shake
$ stack purge # I also deleted the exe from my local bin directory
$ rm -rf .stack-work/
$ stack install

# Maybe I cancelled that previous one? Because I ran this one
$ stack ./install.hs hie

After that last command, I got a success.
...
... (snip) ...
...
haskell-ide-engine               > Registering library 'hie-test-utils' for haskell-ide-engine-1.3..
Completed 229 action(s).
Copying from C:\hie\.stack-work\install\38e963dd\bin\hie.exe to C:\Users\Alex\AppData\Roaming\local\bin\hie.exe
Copying from C:\hie\.stack-work\install\38e963dd\bin\hie-wrapper.exe to C:\Users\Alex\AppData\Roaming\local\bin\hie-wrapper.exe

Copied executables to C:\Users\Alex\AppData\Roaming\local\bin:
- hie.exe
- hie-wrapper.exe
# stack (for hie-8.6.5)
stty: 'standard input': Inappropriate ioctl for device
Build completed in 33m41s

No longer an issue! User error!
I'll submit a PR to clarify the README.

Glad to see the issue is resolved.
The key is to use stack install.hs hie, it runs the install script that uses the latest ghc version that works with windows (ghc-8.6.5). It does a git submodule update which, although it is not needed now maybe will do.
stack install uses the default stack.yaml config file, that makes it use ghc-8.8.2 and i am afraid it cant be used in windows due a compiler bug.


B-18830-Addition

Issue ticket
Summary
This is an extension of the fix for the <ConnectedFlashMessage /> component in src/pages/SignIn/SignIn.jsx. This edit adds the component back to the sign in page BUT it only renders IF office is the site page. This prevent sit from trying to render when the admin page is loaded or any other page where this component is not needed.

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against 01afa44443315c53aa0a3a13649378685298242e


fix get_txn_client check

just view code to use tiki txn client, fix it

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Connect to bees by IP instead of host names

Sometimes hostnames are not resolved.

If this change is to be adopted, I think it should be accompanied by a command line option which enables "ssh by IP address".  Because:

With EC2 supporting IPv4 and IPv6 now, forcing IPv4 would be a bad practice and a regression from the more-expected behavior of SSHing by name
This may change behavior for organizations that have instance names configured a certain way, an option would prevent a change to default behavior

The option would probably allow for selection between "SSH by" name, IPv4 address, IPv6 address.

I think that SSH by instance name is a bad idea. AWS allows to reconfigure internal VPC DNS through Route53. It is better to use IP addresses.
PS.
Too many things were changed since 2015. Let's just close this PR.


[ENH] Update dependencies in docker image

Adds pytest/codecov libraries, pyAFQ atlases

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (master@bfbc77a). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #768   +/-   ##
=========================================
  Coverage          ?   29.77%
=========================================
  Files             ?       97
  Lines             ?    14577
  Branches          ?     1885
=========================================
  Hits              ?     4341
  Misses            ?    10110
  Partials          ?      126

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


remap eslint/prefer-spread to unicorn/prefer-spread

In this PR, unicorn/prefer-spread rule was migrated to eslint:prefer-spread.
So should we update status of unicorn/prefer-spread? (It seems to be not implemented status)
Originally posted by @tbashiyy in https://github.com/oxc-project/oxc/issues/684#issuecomment-2512853953
Needs the same mapping like jest ot vitest
and then update the doc tasks too:
https://github.com/oxc-project/oxc/blob/main/tasks/lint_rules/src/oxlint-rules.cjs#L181-L198

imo, it would be easiest just to split the rules.
rules are now allowed to be the same name, which bypasses the original reason for why it was merged

imo, it would be easiest just to split the rules.

Should we do the same for all remapped rules in https://github.com/oxc-project/oxc/blob/main/crates/oxc_linter/src/utils/mod.rs ?

so my reasoning for spliting them is, iirc, eslint/prefer-spread and unicorn/prefer-spread are completly differnt rules

eslint
unicorn

Whereas the typescript ones as usually (i haven't checked this) the same as the eslint rule just with TS support
Vitest is basically the same - its 99% the same code across jest/vitest for rules such as no identical title so no need to split

This is my fault. You're correct that these are completely different rules.
Let's revert that decision and separate these two rules.


No kernal for language python found

Great work creating a more interactive environment for atom, but alas for me I cannot get it to work ...
I think this maybe something to do with setting correct kernal path, but I cannot find kernal.json file use as a reference in the settings.
When I go run a line of python code (cmd+shift+P hydrogen: run), this is the error i get (see screen shot attached):
No kernal for language python found
Check that the language for this file is set in Atom and that have a Jupyter kernal installed for it.
I have plenty of kernals installed on my machine as i typically use virtual environment and I also have the anaconda distrubution installed as well.

If your editor grammar is set to MagicPython instead of Python, hydrogen is searching for a kernel named magicpython rather than python. You can check this setting at the lower right corner of Atom.
Either change the grammar of your editor to Python or if you like to keep the syntax highlighting of MagicPython add {"MagicPython": "python"} to Language Mappings in the Hydrogen settings.

@lgeiger Are sure? I have opposite key/values:
{"babel":"Babel ES6 Javascript", "babel":"Javascript with jsx"}

@lgeiger This isn't working for me, I keep getting the No kernel for language magicpython found error.

I reproduced this issue.
If you add {"python": "magicpython"} to your language mappings everything should work fine.
This setting is usually used to map a kernel language name to atom's grammar name. Since MagicPython changes atom's grammar name this setting has to be applied the other way round.
You can also use the kernel mapping setting instead. Just add {"magicpython": "Python 2"} there.

Great, I can confirm that exchanging the two names works.

Awesome! 🎉

[ Problem solved ] " no kernel for python language found"
I am using Linux Operating System ubuntu 16:04
Atom 1.7.4
Open your beloved Terminal <3
$ sudo apt autoremove ipython
$ sudo apt install python-pip -y
$ pip install --upgrade pip
then, install ipython[notebook]
$ sudo pip install ipython[notebook]
Done


tikv: support coverage

Tested and verify the coverage can be uploaded correctly. As it's supposed to run one time per day, I don't do anything to speed up the job. It takes about 1 hour to finish.

Tested and verify the coverage can be uploaded correctly. As it's supposed to run one time per day, I don't do anything to speed up the job. It takes about 1 hour to finish.

We can observe for a period of time to see if you can accelerate it. Maybe cargo cache or not enough CPU or memory resource.


Vscode: Error encountered when viewing previous version from Git history

View file content from Git history, starts getting this error.
[crates\ra_ide\src\syntax_highlighting.rs:218] match element.parent().map(|it| it.kind()) {
    Some(LIFETIME_PARAM) | Some(LABEL) => h | HighlightModifier::Definition,
    _ => h,
} = Highlight {
    tag: Lifetime,
    modifiers: HighlightModifiers(
        0,
    ),
}

Error will persist even if the history file is closed.

I think this is just a stray dbg! which was remofed in https://github.com/rust-analyzer/rust-analyzer/commit/8f3677a94a3ff77c9d2d1784b0c8d5f777e46c3b#diff-1a46f1499436ea116d345698c150068a


False positive for MissingOverride

Error-prone 2.32.0:
public record BigQueryTransactionHandle(UUID uuid)
        implements ConnectorTransactionHandle
{
    public BigQueryTransactionHandle()
    {
        this(UUID.randomUUID());
    }

    public BigQueryTransactionHandle
    {
        requireNonNull(uuid, "uuid is null");
    }
}

And violation:
[ERROR] BigQueryTransactionHandle.java:[25,12] [MissingOverride] <init> is an explicitly declared accessor method for a record component; expected @Override
[ERROR]     (see https://errorprone.info/bugpattern/MissingOverride)
[ERROR]   Did you mean '@Override public BigQueryTransactionHandle()'?

Related to https://github.com/google/error-prone/commit/be9921779231a78f353e815e1edf41b4d9d8b6e0
cc @cushon

Hmm, somehow https://github.com/google/error-prone/commit/81ae61d2d103c5cef1bf6d29e92c2e4152a61e3c didn't automatically close this.

Thanks @cpovirk !


cmd/link: Add DW_AT_decl_line for DW_TAG_subprogram in DWARF table

Currently, the DWARF table does not contain DW_AT_decl_line in the DW_TAG_subprogram entries, i.e. the source code line number of the function definition is not listed in the DWARF entry of said function. I propose to add this information to the DWARF table.
DW_AT_decl_line of functions is needed to convert absolute line numbers into relative line numbers w.r.t. to the first line of the function from DWARF information. Such information is useful in the context of PGO.
CC @aclements

cc @cherrymui @thanm

I am fairly certain that adding new DWARF data does not require the proposal process. Removing.

@hoeppi-google is this attribute needed for subprogram corresponding to abstract functions? Or only regular DW_TAG_subprogram entries corresponding to vanilla functions?

discussed this offline with @prattmic , sounds like we need abstract function support as well.


Updating K3s and Docker versions

Updating K3S to version v1.18.4
Updating docker to version 19.03.7

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the PR!


camelCase for ids and hyphens for class names

I've noticed in the forms example that ids use camelCase, while classes use dash-names.
Since the consensus seems to be on using hyphens for both ids and class names, I was wondering if there are some Bootstrap-specific aspects that have led to this decision, and whether this is a guideline for Bootstrap projects.

I'm reasonably sure we do that so as to help avoid any confusion that those names are somehow part of Bootstrap itself.

We try to limit our IDs and make them stand out, thus the camelCase. Classes and everything else though should be dashes.


Problem with ExceptionDispatchInfo

Sometimes some of our classic test fail with the following exception:
      System.InvalidOperationException : ValueFactory attempted to access the Value property of this instance.
      Stack Trace:
           at System.Lazy`1.CreateValue()
        --- End of stack trace from previous location where exception was thrown ---
           at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
           at System.Lazy`1.get_Value()
        BenchmarkDotNet\Jobs\Job.cs(45,0): at BenchmarkDotNet.Jobs.Job.Equals(IJob other)
        BenchmarkDotNet\Jobs\JobExtensions.cs(105,0): at BenchmarkDotNet.Jobs.JobExtensions.TryGetShortInfoForPredefinedJobs(IJob job, String& shortInfo)
        BenchmarkDotNet\Jobs\JobExtensions.cs(53,0): at BenchmarkDotNet.Jobs.JobExtensions.GetShortInfo(IJob job, IJob[] allJobs)
        BenchmarkDotNet\Running\Benchmark.cs(14,0): at BenchmarkDotNet.Running.Benchmark.get_ShortInfo()
        BenchmarkDotNet\Running\BenchmarkRunner.cs(73,0): at BenchmarkDotNet.Running.BenchmarkRunner.Run(Benchmark[] benchmarks, ILogger logger, String title, IConfig config, String rootArtifactsFolderPath)
        BenchmarkDotNet\Running\BenchmarkRunner.cs(50,0): at BenchmarkDotNet.Running.BenchmarkRunner.Run(Benchmark[] benchmarks, IConfig config)
        BenchmarkDotNet.IntegrationTests.Classic\ReferencesTests.cs(51,0): at BenchmarkDotNet.IntegrationTests.Classic.ReferencesTests.CanCompileAndRun[TBenchmark]()
      System.InvalidOperationException : ValueFactory attempted to access the Value property of this instance.

List of tests with troubles:
Error: .NET 4.6 tests has failed:
    BenchmarkDotNet.IntegrationTests.Classic.AssemblyConfigTests.IsActivatedTest [FAIL]
    BenchmarkDotNet.IntegrationTests.Classic.ReferencesTests.VisualBasicIsSupported [FAIL]
    BenchmarkDotNet.IntegrationTests.Classic.ReferencesTests.BenchmarksThatReturnTypeThatRequiresDifferentRuntimeAreSupported [FAIL]
    BenchmarkDotNet.IntegrationTests.Classic.ReferencesTests.BenchmarksThatReturnTypeFromCustomPathDllAreSupported [FAIL]
    BenchmarkDotNet.IntegrationTests.Classic.ReferencesTests.BenchmarksThatUseTypeFromCustomPathDllAreSupported [FAIL]
    BenchmarkDotNet.IntegrationTests.Classic.ReferencesTests.BenchmarksThatUseTypeThatRequiresDifferentRuntimeAreSupported [FAIL]
    BenchmarkDotNet.IntegrationTests.Classic.ReferencesTests.FSharpIsSupported [FAIL]

Another crazy bug! It is really strange because I usually run all our tests few times a day and it has never happened to me.. Anyway sorry for that, the blame is on me ;)


Time Dimension, Big problem with methods 'groupby' and 'to_netcdf'

My problem is that I would like to use the easy functionality of the xarray-library in python, but I run into problems with the time dimension in case of aggregating data and in case of writing netcdf.
I am using pandas version 0.17.1 and xarray 0.9.6.
I have opened a dataset, which contains daily data over the year 2013:  datset=xr.open_dataset(filein).
The contents of the file are:
<xarray.Dataset>
Dimensions:       (bnds: 2, rlat: 228, rlon: 234, time: 365)
Coordinates:
  * rlon          (rlon) float64 -28.24 -28.02 -27.8 -27.58 -27.36 -27.14 ...
  * rlat          (rlat) float64 -23.52 -23.3 -23.08 -22.86 -22.64 -22.42 ...
  * time          (time) datetime64[ns] 2013-01-01T11:30:00 ...
Dimensions without coordinates: bnds
Data variables:
    rotated_pole  |S1 ''
    time_bnds     (time, bnds) float64 1.073e+09 1.073e+09 1.073e+09 ...

Attributes:
    CDI:                       Climate Data Interface version 1.7.0 (http://m...
    Conventions:               CF-1.4

When I use now the groupby method to compute the monthly means, the time dimension is destroyed:
datset.groupby('time.month').mean('time')
<xarray.Dataset>
Dimensions:    (bnds: 2, month: 12, rlat: 228, rlon: 234)
Coordinates:
  * rlon       (rlon) float64 -28.24 -28.02 -27.8 -27.58 -27.36 -27.14 ...
  * rlat       (rlat) float64 -23.52 -23.3 -23.08 -22.86 -22.64 -22.42 -22.2 ...
  * month      (month) int64 1 2 3 4 5 6 7 8 9 10 11 12
Dimensions without coordinates: bnds
Data variables:
    time_bnds  (month, bnds) float64 1.074e+09 1.074e+09 1.077e+09 1.077e+09 ...

Now I have instead of a time dimension a month dimension with values from 1 to 12. Is this a side effect of the 'mean' - function? As long as i do not use this mean function, the time variable is retained.
The examples given in the documentation seems to have a different behaviour. That is, the timestamps are retained and the first date of each month is used.
It seems to be impossible to reinvent my old time dimension.

Method A: I have tried to create my own time variable with endresult.assign_coords(time=pd.date_range(start='2013-01',end='2014-01',freq='M'   . That perfectly gives me a new coordinate with the correct dates. Afterwards, I have to swap the dimensions from month to time. It was only possible by changing the dimension of the coordinate 'time' to the dimension of the coordinate 'month'. However, the netcdf file contained wrong dates as output, i.e. values from 1 to 12. Thus the first time step was at 31-January 2013 and the next one day later and the next one day later and so on. If I add the attributes 'calendar' and 'units' to the time-coordinate, then the output seems to be correct but type int64 is not readable by programs like ncview.
Method B: Create the own time variable by using pandas and then converting the datetime64-dates to the usual python datetime-object. Further, the datetime-object is converted to numbers with the netcdf4.datetime.date2num method. Further, I assign this numbers to the time-coordinate and add the encoded attributes for units and calendar. However, the encoded units are not writen to the netcdf-data. So I have to add them with an external program like ncatted.

How to improve method A und B in order to have a correct time stamp in my nc-file.

As far as I know I can imagine this is the intended functionality.

The examples given in the documentation seems to have a different behaviour. That is, the timestamps are retained and the first date of each month is used.

I cannot find where this is the case, apart from when using .resample. Could you put a link to the doc page?
The issue is perhaps more with the example that you present (of only 1 year data) and expected behaviour. Normally groupby('time.month') would be applied to multiple years of data. i.e. group data by month and find the monthly averages for Jan-Dec for 30 years of data, e.g. a climatology.
And so in this case it absolutely makes sense to keep the months as 1 to 12, or something similar (perhaps 'Jan','Feb'etc). Applying a datestring of the first day of the month wouldn't make sense because which year would you choose when you have 30 years of data?
If you do want a time series of monthly means, then .resample is the function you want and it will give you the datestamps in the format that you desire.

@rpnaut - @byersiiasa is correct. It sounds like you want
datset.resample('MS', dim='time', how='mean')

The month labels (1, 2, 3, ...) come from pandas TimeGrouper and this is functionality we will want to keep around.

You are so right. I did not realize that there is the resample method, which hopefully can also be combined with the 'apply' functionality.
The documentation I mentioned was from "nicolasfauchereau.github.io/climatecode/posts/xray" (look at In[24] and In[25]. As I understand he is getting monthly data out of groupby-method and in his example the "time" survives. It seems to be that the functionality of groupby-month changed during the years, because the groupby-method in Nicolas's example did not aggregate same calendar month to one time stamp.

As I understand he is getting monthly data out of groupby-method and in his example the "time" survives. It seems to be that the functionality of groupby-month changed during the years, because the groupby-method in Nicolas's example did not aggregate same calendar month to one time stamp.

There has been no change in xarray's groupby behavior. Nicolas' example would work with today's code. When you call datset.groupby('time.month').mean('time'), you remove the time dimension by aggregating over it. If you had applied a different (non-reducing) function to the group (e.g. datset.groupby('time.month').apply(lambda x : x**2), you would preserve the time dimension.


Dual setup with gripper fails

Launching the following dual setup with grippers:
ros2 launch xarm_moveit_config dual_uf850_moveit_fake.launch.py add_gripper:=true

leads to the ros2_control_node process being terminated:
[ros2_control_node-7]   what():  Hardware name XArmGripperSystem is duplicated. Please provide a unique 'name' in the URDF.

The fix is as easy as parametrizing each gripper ros2_control tag name using the prefix, e.g. this one. I've done that locally and was able to ros2_control both arms and both grippers.
Of course, this should be extended to the rest of the grippers as well.

@mcres
Thanks for the heads up, we will fix this issue.

@mcres
Thanks for pointing this out, we will fix it.


fix: alwaysRedirect does not work with redirectOn 'all' and 'no prefix'

🔗 Linked issue

#1859
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[X] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

When the strategy is set to anything except 'no_prefix' and with alwaysRedirect enabled, redirection only happens on the root path. This change modifies the check when alwaysRedirect is set to check redirectOn strategy before choosing to redirect.
Previously, going to a page like '/about' would not redirect to '/fr/about' because it is not the root. This fix will allow us to redirect correctly so that '/about' and '/ja/about' will redirect to '/fr/about' when using 'all', and redirect '/about' to '/fr/about' when using 'no prefix'
📝 Checklist

[X] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

@kazupon I'm not sure but I think the check is stuck. I checked the linked Netlify build and it was completed

Thank you for your contribution!
I’ll check your PR soon.

That's great works!
Thanks!


Add new polyglots

Reminder to myself to add this:  https://github.com/0xsobky/HackVault/wiki/Unleashing-an-Ultimate-XSS-Polyglot

+1

It was your find, great one.

+1

Added!


Fixed issue #113

Description
No description found for endpoint /groups and method get
Fixes #113
Type of change

Documentation update

Testing steps
N/A

Codecov Report

Merging #134 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #134   +/-   ##
=======================================
  Coverage   74.27%   74.27%
=======================================
  Files           9        9
  Lines         381      381
=======================================
  Hits          283      283
  Misses         79       79
  Partials       19       19

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 c009b93...a52e6e3. Read the comment docs.


Fix description of alt-shift-/

This request fixes the description of keybind for alt-shift-/.
It is currently "open keybind window" but the actual behavior is to toggle the keybind window.
Note: this request accidentally includes unrelated change (in line 311-313) that fixes its newline code.
I could not remove this unrelated change from the commit when I made a commit with using my editor (VSCode) and code editor provided by Github website.

PRs haven’t been merged in months and Rick haven’t been answering emails either.
Right now it seems a fork is needed to keep workspacer going forwards.
I’m discussing this in a issue here: https://github.com/rickbutton/workspacer/issues/211
Feel free to let me know your thoughts.


Allow indexer to write to a file instead of stdout. [phab:D953]

Created by benjyw at 2016-07-08 18:37:47:
Some build tools may not have easy access to stdout.

schroederc closed via commit 498d7bc8341d5efd63141f60be6348ebed92c764 at 2016-07-11 20:22:38


Publish to crates.io

This means adding a few items to Cargo.toml, and creating a small README.
Also it's probably a good idea to either rename this to rust-core-graphics-sys. Alternatively one could move the externs into an ffi folder and build an RAII API around them, but that's probably a job for another crate.

https://crates.io/crates/core-graphics


Add breakpoints/media queries mixins

Feature description
Mixins and functions are good ways to wrap some code. With a breakpoint mixin, devs can reuse media queries with better syntax and control breakpoints with a single source of truth.
Code Example
This is just an example of using mixin, but there are other ways of doing so.
File: mixin.scss
$breakpoint-phone-only: 599px;

@mixin for-phone-only {
    @media (max-width: $breakpoint-phone-only) {
        @content;
    }
}

File: style.scss
@use "./../../styles/mixins.scss" as *;

.quick-questions-wrapper {
    display: flex;
    gap: 18px;
    flex-wrap: wrap;

    @include for-phone-only {
        flex-direction: column;
  }
}

References with different examples of how to use it.
https://www.section.io/engineering-education/facilitating-css-media-queries-using-scss-mixins/
https://dev.to/rembertdesigns/how-to-write-media-queries-with-sass-mixins-32e

Obrigado por abrir a issue
Verifique os nossos guidelines:

Código de conduta
Contribuição
Guia de Estilo

Eu quero!!!