Compile fails to find types after adding import statement

TypeScript Version:  2.1.1 / nightly (2.2.0-dev.201xxxxx)
Something I've been stuck on for way too many hours. Not sure if a bug or my bother.
Initially things compile:
Code
src/my.ts:
/// <reference path="../types/index.ts" />

const foo: Foo = "foo";

../types/index.ts:
type Foo = "foo";

However compile fails after adding import statement in ../types/index.ts:
../types/index.ts:
import * as fs from "fs";

type Foo = "foo";

Errors
src/my.ts(3,12): error TS2304: Cannot find name 'Foo'.
src/my.ts(3,12): error TS4025: Exported variable 'foo' has or is using private name 'Foo'.

My tsconfig is:
{
    "compilerOptions": {
      "target": "es2015",
      "lib": [
        "es2015"
      ],
      "module": "amd",
      "moduleResolution": "node",
      "declaration": true,
      "noUnusedLocals": false,
      "noUnusedParameters": false
    }
}

Compiling with aforementioned named files to single output file with --outFile command line option.
Expected behavior:
Should compile with import statement added
Actual behavior:
Doesn't compile

Adding an import makes it a module. I cover it here https://basarat.gitbooks.io/typescript/content/docs/project/modules.html 🌹


RSE Attributes: what they do and how to use them

related: #293

I should probably expand this to include RSE settings. I think that might take more effort than the attributes.
I'm also not sure if I found every RSE attribute that's out there in the wild. I may have missed some in my grepping.

I should probably expand this to include RSE settings. I think that might take more effort than the attributes. I'm also not sure if I found every RSE attribute that's out there in the wild. I may have missed some in my grepping.

I admit, now I'm confused at the differences between settings and attributes. The page changed in PR #403 just references members of the RSEAttr, (I think those can be anything...) which I think also includes the settings as defined by RSESettingDict
Now uh. I don't know what to do with that.


Give proper credit in the official webpage to JavaScript Mastery

What would you like to share or ask?
Dear Nishitbaria,
I have noticed that this project is straight up cloned from JavaScript Mastery youtube channel which disincentivises the original creator. While getting inspired from a project is no crime, there is a line between stealing credit from the original creator and taking inspiration.
I see that you are actively mainting the project which definitely is a great thing but the original source code is more that 90%.
I also noticed that a similar PR earlier was closed by you without taking any action. Giving appropriate credits will only make you a better programmar and a humble human being.
Do acknowledge my request.
Elvin
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

I support this request. Utilizing community projects without proper credit is unprofessional and disrespectful to the original creators. It shows a lack of integrity and appreciation for the hard work put in by others.   @Nishitbaria

Hey @ritviz344 I appreciate you can do that but i will not provide any gssoc label for this issue because you are not doing a meaningful contribution

Hey @Nishitbaria I am not a GSSOC contributer this year. May be you can add the credits yourself "Inspired from JavaScript Mastery and pushing new boundaries to the project" on the homepage. I will look in future if thats implemented by you.

Done with that


Use a mappable path for images.

I would imagine an absolut path /images/timestamp from launch/no. of image.png for the images.
Maybe talk with @skrrpet about the path and what ros parameter to use for it.
And then integrate the mapping into the docker_run.sh file
https://github.com/THI-Drone/qrcode_detection_package/blob/761def5b93ffbbb0101e5e20900ff7b4d2d05875/qrcode_detection_package/main.py#L343

fixed


:direction: "alternate" does not work

First, thanks for this npm package. It save a lot of time ;)
My lottie files runs fine, but it does not reverse after finished and it shows up this error message in the console. 🤔

Here my Code:

I was reference it from the docs.

Do I made something wrong or is it a bug? 😅

Thank you for using the library. So your issue is :direction. What you need to be using for the attribute name is either direction="alternate" or something like :direction="'alternate'". If you use the : before the attribute name, vue is expecting to see a js variable, and not a string (unless you add with a clear string format). Remove the : and it should work for you :)

@megasanjay
Oh. I didn't notice it xD
Now it works. Thanks for the help :)


Linux compatibility

Hi, this looks like a really cool project! I wanted to give it a try on arch linux, but when I install the venv_requirements.txt dependencies in a virtual environment with python 3.8, I get the following error:
ERROR: Ignored the following versions that require a different python version: 0.1.0 Requires-Python >=3.9,<4.0; 2.10.0 Requires-Python >=2.7,<3.0; 2.3.0 Requires-Python >=2.7,<3.0; 2.4.0 Requires-Python >=2.7,<3.0; 2.5.0 Requires-Python >=2.7,<3.0; 2.6.0 Requires-Python >=2.7,<3.0; 2.7.0 Requires-Python >=2.7,<3.0; 2.8.0 Requires-Python >=2.7,<3.0; 2.9.0 Requires-Python >=2.7,<3.0; 8.13.1 Requires-Python >=3.9; 8.13.2 Requires-Python >=3.9
ERROR: Could not find a version that satisfies the requirement pywin32==304 (from versions: none)
ERROR: No matching distribution found for pywin32==304

It still results in the same error. Running pip install pywin32 in the virtual environment outputs:
ERROR: Could not find a version that satisfies the requirement pywin32 (from versions: none)
ERROR: No matching distribution found for pywin32

It's because pyenv32 isn't compatible with linux

I'm afraid I can't help you. I'm not practical with Linux. Maybe try to intall pywin https://www.geeksforgeeks.org/how-to-install-pywin32-on-linux/amp/


Go Live

Push to live

Docs Build status updates of commit 8140ec7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/meeting-app-extensibility.md
:white_check_mark:Succeeded
View

msteams-platform/feed.atom
:white_check_mark:Succeeded
View

msteams-platform/tabs/tabs-link-unfurling.md
:white_check_mark:Succeeded
View

msteams-platform/tabs/what-are-tabs.md
:white_check_mark:Succeeded
View

msteams-platform/task-modules-and-cards/task-modules/invoking-task-modules.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Use of created nested types in dynamic assembly builders cause TypeLoadExceptions

When attempting to generate a dynamic type that contains a nested type, use of the type to generate fields on the parent type fails with a TypeLoadException if the type has been realized using GetTypeInfo().
This issue is not present on Mono, and the type can be successfully created. Furthermore, defining the type as a non-nested type in the dynamic module also succeeds.
A minimal reproduction project has been included which targets both the full framework and .NET Core 2.0. The aforementioned exception is thrown at line 23 when the project is ran using .NET Core 2.0.
To run the project, unzip the file and open it in your IDE of choice, then compile and run. If you are not using an IDE, execute the following commands:
unzip Scratchpad.zip
cd Scratchpad
dotnet run --project Scratchpad --framework netcoreapp2.0 // for .net core
dotnet run --project Scratchpad --framework net461 // for .net 4.6.1

Moving to https://github.com/dotnet/coreclr/issues/17772.


Allow Bulgarian numbers with either 6 or 7 digits

Bulgarian numbers can have either 6 or 7 digits after the area code.
Reference:

https://www.numberingplans.com/?page=plans&sub=phonenr&alpha_2_input=BG
https://en.wikipedia.org/wiki/Telephone_numbers_in_Bulgaria

Coverage increased (+0.9%) to 99.4% when pulling c76e2806fbde2c0a1e0eb78e3313e51f83c707a2 on SofiaMargariti:fix_bulgarian_number_validation into a1f113f5f1d8cf9d65cf9e1c4e8703da0db11eb4 on floere:master.

Coverage increased (+0.9%) to 99.4% when pulling c76e2806fbde2c0a1e0eb78e3313e51f83c707a2 on SofiaMargariti:fix_bulgarian_number_validation into a1f113f5f1d8cf9d65cf9e1c4e8703da0db11eb4 on floere:master.

cc @floere

@SofiaMargariti Apologies for the late merge – I was holidaying! 😅 Anyway, thanks so much for all your PRs, amazing! 🎉 Very much appreciated! 😊 This PR is released in 2.16.5 🚀


prefix '/' conflicts with get('*')

new Elysia()
.use(staticPlugin({prefix: '/' }))
.get('*', "hello")
.listen(3000)

When build elysia as above, all requrest goes to get('*'). So could set get() as a fallback? That is, the request first goes to match static resources, and then matches the get route if static file is not found.

maybe like this
const loadStatic = (root: string) => async (app: Elysia) => {
  readdirSync(root, { recursive: true, withFileTypes: true })
    .filter(file => file.isFile())
    .forEach(file => app.get(file.name, Bun.file(`${root}/${file.name}`)))
  return app
}

Thanks for reporting this finding!
Can't reproduce it on my end. Can you give it a try with the latest versions?

Yes. It works when I set the prefix to any path, like /p.
I can get test.txt from /p/test.txt. But if i set prefix to /, I get hello from /test,txt
elysia 1.1.26
elysia-static 1.1.1

You've to use the alwaysStatic option:
import { Elysia } from "elysia";
import { staticPlugin } from "@elysiajs/static";

new Elysia()
  .use(staticPlugin({ prefix: "/", alwaysStatic: true }))
  .get("*", "hello")
  .listen(3000);

Otherwise, the routes are resolved lazily when they are called; your wildcard route basically overrides them.

it works :)


Update Coffeelint gem

Why:

We are a few versions behind.
There is potential bug that was reported by our customer.

Is there any change that needs to be made to our default config files for this update?

No. The gem changes are mostly bug fixes.

LGTM in that case


Support for 'list' functionality in Siddhi

Description:
Siddhi supports maps but it does not support lists yet. This will be useful for processing the collection of events like the output of a counting-pattern and solves issues like https://github.com/siddhi-io/siddhi/issues/1398

+1, there are situations that I came across such requirements.

Iniated a discussion on siddhi dev mailing list https://groups.google.com/forum/#!topic/siddhi-dev/C2aF70NjFEc

This is implemented and v1.0.0 of the siddhi-execution-list was released earlier today.


improvement allow to ignore unknown image module

Proposed changes
Add new FE config ignore_unknown_metadata_module. Default is false.
If set to true, when reading metadata image file, and there are unknown modules, these modules
will be ignored and skipped.
This is mainly used in downgrade operation, old version can be compatible with new version Image file.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

LGTM


[Feature request] Improve the type readability

The typing in PriorityQueue does the job, but it is very hard to understand the intention.
It is recommended to use declare to cope up with multiple constructors, which should achieve the same thing, and more readable.

Thank you for the suggestion.
Yes, that's exactly what I'd like to do, which is more standard. I think it's a right time to do it in case it gets too hard to refactor them.


Require patient city in patient registration

Description
Patient city is currently not a required field on the patient registration form in the "add/edit patient" flow in SimpleReport. This is resulting in states not receiving complete patient information.
Acceptance criteria

[ ] Patient city is required on the front-end; this does not include changes to the backend (we don't want to backfill data entries on the backend)

Look into how many city fields are left blank to determine if it would be worth cleaning up the database/backfilling.

@lauraykerr want to confirm what the Spanish translation should be for the error message for missing required field (city).

English went from "City is incorrectly formatted" -> "City is missing".
Previous Spanish error message was "La ciudad tiene un formato incorrecto"

@DanielSass Translation is: Falta la ciudad


manager: ignores references whose target column is a foreign key itself

Closes https://github.com/simolus3/drift/issues/3251
Rebuild all Projects

@simolus3
It seems CI is broken...
 Invalid argument(s): Failed to load dynamic library 'libsqlite3.so': libsqlite3.so: cannot open shared object file: No such file or directory

The CI should work again on develop.


User ID method handling

Hi, thanks for the great work on this project!
I am using Auth0 and have been using the now deprecated django-rest-framework-jwt library but would like to switch to simplejwt. Part of the implementation I was using used a function to alter a username from the token and authenticate the user in Django's User model.
The relevant function is:
# auth0authorization/user.py

from django.contrib.auth import authenticate

def jwt_get_username_from_payload_handler(payload):
    username = payload.get('sub').replace('|', '.')
    authenticate(remote_user=username)
    return username

and the setting in django-rest-framework-jwt is:
JWT_AUTH = {
    'JWT_PAYLOAD_GET_USERNAME_HANDLER':
        'auth0authorization.user.jwt_get_username_from_payload_handler'
   ...
}

This is necessary because Auth0 uses usernames that look like "auth0|username" and they recommend storing as "auth0.username" in the User model (not fully sure why) and also because I need to authenticate the remote user before the token user_id claim means anything.
Do you know if there is a way to use a function upon receiving the JWT token in simplejwt? Alternatively, if there is a workaround happy to implement it!

I’m planning on creating a signal handler (with lots of warnings for when to use them, but this scenario seems fine) for post authentication. I can add a pre_authenticate handler as well. @ethanjahn

The other method is to add the username on token creation. So you'd include it in the refresh and access token payload.

@Andrew-Chen-Wang To further detail this issue, I recently opened a new issue with the auth0 Django API tutorial. I've linked it back to this discussion.

Hi @asoundlife You'll have to give me until next week, possible Monday to re-visit the tutorial to see why this is necessary. For now, I'm gonna ask some questions so I may better understand this issue (I've been running out of time lately to maintain some of my repositories with college comin' up).

Does auth0 use their own library ? AFAIK, they just use Django's built-in authentication middlewear and rest-framework-jwt.
The callback to a new authentication method is for customizing login I suppose. Is it absolutely necessary to have Auth0 authenticate first? If anything, imo, you should be handling authentication first. A user on Auth0 does not mean your user necessarily exists in the first place, not to say the shabby record of Auth0 due to its flexibility for developers.
What does this mean?

This is necessary because Auth0 uses usernames that look like "auth0|username" and they recommend storing as "auth0.username" in the User model (not fully sure why)

Are we putting the username in the payload? If so, that can be done in the serializer in the "validate" method. Is the username the same as your database's username or Auth0's? Doe the payload look like "auth0|Andrew-Chen-Wang" or "auth0|BASE64_BLAHS_IN_AUTH0"?
Storing "auth0.username" in the User model? I must've passed it in the tutorial.
I do see the RemoteAuthentication middlewear. Again, adding a callback would help for additional authentication measures, I suppose. If anyone does make a callback, only make a PRE-authentication callback. If it were POST/after, then people will think they can use it for last_login, which they should NOT.
Thanks for helping me understand the issue!

Thanks @Andrew-Chen-Wang ,
To illustrate the answers to your questions, consider the scenario of having a Django backend serving an API to a frontend like Vue, using auth0 to serve as the authentication provider.
The auth0 team recommends setting up an authentication system in the following way:

Configure a new auth0 application & API with the auth0 dashboard (their website).  This gives the basic structure for managing your users and the rules behind authentication.  For our example scenario above, the dashboard serves as scaffolding until a user management tool is created for the Vue frontend, which interacts with the auth0 API via JWT.

User authenticates via a "universal" (customizable) login screen. This is hosted by auth0, which is enabled by default and mandatory if you're not on a paid subscription. The universal login screen processes the form with the auth0 backend and then redirects the user to our example scenario site, along with an (RS256) JWT authentication token.

Auth0 provides an SDK for SPAs. This SDK allows you to create a Vue component that manages the connection between the Vue frontend and provides all the necessary methods to connect and interact with the authentication pieces of the auth0 API.

One key note before moving on: For all but the "Enterprise" version of auth0, Auth0 stores your user profiles.  In the Enterprise version, you can configure your own database to manage your user profiles. However, they still recommend migrating your user database to auth0. As such, I hope this adds clarity to your question #2. Auth0 actually recommends that you shouldn't manage your own authentication, generally speaking.  This also means that in many use cases you won't store user account information on the backend Django database, though, auth0 does provide methods to do this.
In keeping with our example, for Vue to consume the backend Django API it sends an axios request with an authentication header that embeds the JWT token managed by Vue/Auth0 SDK. The token includes basic user information and user permissions.
In the Django API tutorials posted by auth0, they rely on the unmaintained drf-jwt package to manage the communication between the Vue frontend and the Django API backend. This solution still currently works, but users such as myself are looking to migrate away from it, given the status of that package.  SimpleJWT appears to be the best solution :).  See my linked issue for details regarding the auth0 implementation with drf-jwt and where I've taken things with SimpleJWT.
In auth0's recommended implementation, drf-jwt decodes the token and automagically authenticates the user to the backend server (using authenticate(remote_user=username)). The username is grabbed from the decoded JWT token and a new entry is automatically created in the user database table on the backend.  This is probably best reviewed on the Django API tutorial under the "Validate Access Tokens" section.
In this regard, I'm not sure how to answer your question #3. Is the username part of the payload? I don't believe so.  I think the user is being authenticated via authenticate(remote_user...) and not necessarily relying on any specific username from auth0.  Especially since authenticate(remote_user) will authenticate any username provided and add them to the database.  Here's an example entry of a username created via this method in the Django database:
"auth0.5f13fca1de3b59001924f993".
I hope this provides more detail.  Let me know if I can provide any further detail. I'm happy to share my configuration or edits to SimpleJWT backends.py and tokens.py that implements the token decode method required to interact with auth0.

I see. I'm quickly going through my GitHub mail atm, so you'll still have to give me a little bit of time before big reviewing. There isn't a PR for this yet, but #227 may be a solution. Take a look at it for now.
Thank you for all the info @asoundlife! Looking forward to getting y'all a solution hopefully within this week.

Awesome!  After a quick glance of #227, it looks like some of the work done there could be applicable to implementing a solution here. If that's the case, it might make sense to merge these issues once things become a bit more clear.  Thanks again @Andrew-Chen-Wang . I'll work through #227 and see if I can get the authentication side working for auth0.


Create and Establish repos for additional Equella ecosystem components

Decompose these into multiple issues as identified
These are items known so far

Equella Metadata Utility (EMU)

Equella Bulk Importer (EBI)

Basic/stripped down export and import utilities

Several simple scripts frequently used by clients (e.g. ping utility, etc).

Would like Edalex to review and add any additional items they are aware of.

Discuss this one with Jolse...

@doolse -- This one seems like a good one for you.
Please remember to scrub secrets & remove history, just as we did with the main sources.

Office and BIRT are completed - waiting for Maven ID.

Blocked by #36

I don't believe the task to "Create and Establish repos" for these components is blocked.
Are all the repos created?  (At least the immediate/pre-incubation set?)  Do they all have sources?  If so, I'd say this item is complete.  Could someone with knowledge of those facts do a quick review and close this ticket?
If there are 1 or 2 remos that we still need, let's create specific tickets for those (and close this one).

The 3 repos mentioned in this ticker have been created.


Deleting a model generates an error

The error message:

Should the entire directory of the model be deleted or only the model file itself?

Should the entire directory of the model be deleted or only the model file itself?

The whole directory as in the future a model can have several files

Is this happening when the model is being mounted or in use ? If this is happening just after the download of a model ?

Is this happening when the model is being mounted or in use ? If this is happening just after the download of a model ?

Just after the download on my laptop


chore(package): switch to @fabiosantoscode/uglify-es

Ref: https://github.com/webpack-contrib/uglifyjs-webpack-plugin/issues/262

Looks like CircleCI have problem with naming branches :confused:

there has been a bit of discussion around what to do about this particular problem.
For the moment, just leave this open and i'll and this to the list of options being considered.

Close in favor https://github.com/webpack-contrib/uglifyjs-webpack-plugin/pull/296


Issue in setting up the bot

I tried installing the bot on my repo but its not posting any comment on a PR with failing builds. I am using CircleCI for the tests. I also created a config file but it ain't helping either. Am I missing any steps?
Thanks

@JasonEtco Ping? Can anybody help me please? Thanks

It's only been two days, you don't need to ping me 😅 Can you please share some more details? What does your config file look like? Is it a private repository?

I am just trying out the bot in my local repo so that I can use it in my org. The ci-reporter.yaml file is:
# Set to false to create a new comment instead of updating the app's first one
updateComment: true

# Use a custom string, or set to false to disable
before: false

# Use a custom string, or set to false to disable
after: false

And the circleci file is very simple and it's used to just test a single file :
# Python CircleCI 2.0 configuration file
#
# Check https://circleci.com/docs/2.0/language-python/ for more details
#
version: 2.1
jobs:
  build:
    docker:
      # specify the version you desire here
      # use `-browsers` prefix for selenium tests, e.g. `3.6.1-browsers`
      - image: circleci/python:2.7.14

      # Specify service dependencies here if necessary
      # CircleCI maintains a library of pre-built images
      # documented at https://circleci.com/docs/2.0/circleci-images/
      # - image: circleci/postgres:9.4

    working_directory: /home/circleci/hello-world

    steps:
      - checkout

      # run tests!
      # this example uses Django's built-in test-runner
      # other common Python testing frameworks include pytest and nose
      # https://pytest.org
      # https://nose.readthedocs.io
      - run:
          name: run file
          command: |
            python ex.py
      - run:
          name: run tests
          command: |
            sudo pip install coverage
            coverage run ex.py

      - run:
          name: Generate backend coverage report
          command: |
            sudo pip install codecov
            codecov
          when: on_success

      - store_artifacts:
          path: test-reports
          destination: test-reports

And no its not a private repo. Thanks

It looks like this issue: https://github.com/JasonEtco/ci-reporter/issues/36
Where because your job is named build, it's not being picked up by CI Reporter.

Closing as a duplicate of #36


Update pom.xml

upgrade snakeyaml

seems cadl-ranch gets 0.35, which is good news (had to have every dependency on 0.35 before we can upgrade our cadl dependency).
ignore cadl CI, will increase cadl dependency in next PR.


As a developer, I need to restablish the connection between the Explore tool (Dive) and the MS workspace

(not sure if this is the right repo for this issue...):
The Resource viewer from Dive is no longer connected to the MS. In V3 it was possible to open a resource from Dive in the MS resource viewer, and perform all operations. The bookmarked and annotated resource/fragments could be found back in the MS workspace.

@lilimelgar ok I guess this has to be handled for V4 :P I put it in the backlog

Opening a resource in the resources didn't work because DIVE is using https and the resource viewer popup was still using http. Mixed connections are not allowed by the browser.
This is fixed in DIVE; however it requires mediasuite.clariah.nl also to fully support https.
e.g. this url has to work to open a https resource viewer from within DIVE
https://mediasuite.clariah.nl/tool/default-item-details?id=oai%3Aopenimages.eu%3A681879&cid=open-beelden-beeldengeluid&bodyClass=noHeader
Storing full sessions to the MediaSuite is not possible right now, as the api/infrastructure that was provided by the mediasuite to store the sessions has been deprecated/removed.

@whelmich thanks for checking. Setting the media suite to HTTPS completely will be tackled after the release, since there are some problems related to the OpenConext single sign on service we use.
Since this service is ready to be replaced with something else (right after the release), it is not worth the effort to fix the problems with OpenConext.
Conclusion we'll be able to provide the HTTPS later on.
However is it possible to support both HTTP & HTTPS for DIVE? That could fix the problem in the meantime.


Fonts with contextual alternates are not correct rendered

Pay attention to the e's, for example:
AdoptOpenJDK 8: wrong rendering for font size less than 101 and other wrong rendering for font size greater than 100

AdoptOpenJDK 11-16: wrong rendering for font size less than 101 and correct rendering for font size greater than 100

Example code:
FontRederingErrorDemo.zip
Fonts with feature contextual alternates:

https://bboxtype.com/typefaces/Krikikrak
http://www.glukfonts.pl/font.php?font=Kleymissky

OS:

Linux:

lsb_release -a
LSB Version:	1.4
Distributor ID:	Arch
Description:	Arch Linux
Release:	rolling
Codename:	n/a

uname -a
Linux 5.11.15-zen1-2-zen #1 ZEN SMP PREEMPT Sat, 17 Apr 2021 00:22:34 +0000 x86_64 GNU/Linux

Windows 10

It would be great if this could be fixed for OpenJDK 8 (for all sizes).

@BuZZ-dEE Which operating system is this on?

@karianna It's on Linux.

@karianna It's on Linux.

Thanks, can you give us the full version of the kernel, Linux Distor and desk top Windowing that you're using? (GNOME, KDE etc)

@karianna I already updated the initial post with that information, but somehow it was not shown. Strange.
I'm using GNOME 40. Can you explain, why the desktop environment is relevant?
Also my colleagues, which are using Ubuntu 20.04., had this problem.

Java is WORA but it needs to make calls into native window managers for display purposes, so it's good to know which binding we're hitting.

@karianna thx for the explanation. A colleague of mine tested it on Windows 10 and the problem is there too.

@karianna you closed openjdk-support project. what does that mean for this issue?

openjdk-support project

That's just project management tracking via GitHub, we're moving to use ZenHub going forwards so the issue still stays open :-)

Hmm, it would be great if this could be fixed.

Bad bot, this still needs looking at.

Still a bad bot, this still needs looking at.

I hate you Bot. @karianna how can I submit this issue directly to the OpenJDK project?

I hate you Bot. 😉 @karianna how can I submit this issue directly to the OpenJDK project?

Annoyingly you have to be an OpenJDK author already.  I see the Corretto team have done some deeper investigation.  I'd recommend having them file the bug and I'll follow up with the link to this issue.

@karianna I wonder why I had to open the bug also in the corretto issue tracker to get the bug forwarded to openjdk? Is there a reason for this?

The OpenJDK Issue tracker is only open to folks with author status in that project and they prefer that we triage and repro upstream and update ensure it's a genuine upstream concern before we submit.

https://bugs.openjdk.org/browse/JDK-8302697


Add blurb about large changes to CONTRIBUTION.md

This copies the line about checking with maintainers before pursuing
large changes from the website to our CONTRIBUTING.md. This is a
very important point to emphasize, so it doesn't hurt to make it clear
in as many places as possible.
Signed-off-by: Erik Sipsma<EMAIL_ADDRESS>

LGTM, I think we should also write those lines in the contribution page of Dagger's website ;)

It's on the second paragraph, right at the beginning (https://docs.dagger.io/1227/contributing). Perhaps needs to be more visible.


[Improvement-14112][UI] Add a quick link to visit workflow instances of the current workflow.

Purpose of the pull request
This PR will close #14112 .

Brief change log

Verify this pull request
This change added tests and can be verified as follows:

Manually verified the change by testing locally.

The test result is as follows:

Codecov Report

Merging #14134 (7738a25) into dev (db2da82) will decrease coverage by 0.01%.
The diff coverage is n/a.

:exclamation: Current head 7738a25 differs from pull request most recent head 7938798. Consider uploading reports for the commit 7938798 to get more accurate results

@@             Coverage Diff              @@
##                dev   #14134      +/-   ##
============================================
- Coverage     38.40%   38.39%   -0.01%
+ Complexity     4457     4453       -4
============================================
  Files          1223     1223
  Lines         42543    42497      -46
  Branches       4715     4681      -34
============================================
- Hits          16337    16316      -21
+ Misses        24408    24383      -25
  Partials       1798     1798

see 35 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


:@hariks789_react-native-background-geolocation-common:compileReleaseJavaWithJavac'. > Compilation failed; see the compiler error output for details.

Your Environment
buildToolsVersion = "33.0.0"
minSdkVersion = 21
compileSdkVersion = 33
targetSdkVersion = 33

Plugin version:
Platform: iOS or Android
OS version:
Device manufacturer and model: mac M1

Running in Simulator:
React Native version:0.72.4
Plugin configuration options:
Link to your project:

Context

:@hariks789_react-native-background-geolocation-common:compileReleaseJavaWithJavac'. > Compilation failed; see the compiler error output for details. got this error..
Expected Behavior

wanted to build app
Actual Behavior
problem in build

Possible Fix

Steps to Reproduce

Context

Debug logs

Task :hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars FAILED

FAILURE: Build failed with an exception.

What went wrong:
A problem was found with the configuration of task ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars' (type 'LibraryJniLibsTask').

Gradle detected a problem with the following location: '/Users/nirmal/rioapp/node_modules/@hariks789/react-native-background-geolocation/android/lib/build/intermediates/stripped_native_libs/release/out'.
Reason: Task ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars' uses this output of task ':@hariks789_react-native-background-geolocation:stripReleaseDebugSymbols' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed.
Possible solutions:

Declare task ':@hariks789_react-native-background-geolocation:stripReleaseDebugSymbols' as an input of ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars'.
Declare an explicit dependency on ':@hariks789_react-native-background-geolocation:stripReleaseDebugSymbols' from ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars' using Task#dependsOn.
Declare an explicit dependency on ':@hariks789_react-native-background-geolocation:stripReleaseDebugSymbols' from ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars' using Task#mustRunAfter

@hariks789

Task :hariks789_react-native-background-geolocation:packageDebugResources FAILED

same

@Nirmal-cod @lahed
Sorry, I am no longer maintaining the package.
We started using the paid package for the app.
This fork from @mak12 might help you guys
https://github.com/mak12/react-native-background-geolocation

Task :hariks789_react-native-background-geolocation:packageDebugResources FAILED

I think your settings.gradle file like this.
include ':@hariks789_react-native-background-geolocation-common'
project(':@hariks789_react-native-background-geolocation-common').projectDir = new File(rootProject.projectDir, '../node_modules/@hariks789/react-native-background-geolocation/android/common')
include ':@hariks789_react-native-background-geolocation'
project(':@hariks789_react-native-background-geolocation').projectDir = new File(rootProject.projectDir, '../node_modules/@hariks789/react-native-background-geolocation/android/lib')
Remove the two lines .
include ':@hariks789_react-native-background-geolocation'
project(':@hariks789_react-native-background-geolocation').projectDir = new File(rootProject.projectDir, '../node_modules/@hariks789/react-native-background-geolocation/android/lib')
Use this line only. It is working
include ':@hariks789_react-native-background-geolocation-common'
project(':@hariks789_react-native-background-geolocation-common').projectDir = new File(rootProject.projectDir, '../node_modules/@hariks789/react-native-background-geolocation/android/common')
But we using alternate package. That package also I got same error. After remove the lines working fine.
https://www.npmjs.com/package/@drewaker/react-native-background-geolocation

Task :hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars FAILED

FAILURE: Build failed with an exception.

What went wrong:
A problem was found with the configuration of task ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars' (type 'LibraryJniLibsTask').

Gradle detected a problem with the following location: '/Users/nirmal/rioapp/node_modules/@hariks789/react-native-background-geolocation/android/lib/build/intermediates/stripped_native_libs/release/out'.
Reason: Task ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars' uses this output of task ':@hariks789_react-native-background-geolocation:stripReleaseDebugSymbols' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed.
Possible solutions:

Declare task ':@hariks789_react-native-background-geolocation:stripReleaseDebugSymbols' as an input of ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars'.
Declare an explicit dependency on ':@hariks789_react-native-background-geolocation:stripReleaseDebugSymbols' from ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars' using Task#dependsOn.
Declare an explicit dependency on ':@hariks789_react-native-background-geolocation:stripReleaseDebugSymbols' from ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars' using Task#mustRunAfter

I think your settings.gradle file like this.
include ':@hariks789_react-native-background-geolocation-common'
project(':@hariks789_react-native-background-geolocation-common').projectDir = new File(rootProject.projectDir, '../node_modules/@hariks789/react-native-background-geolocation/android/common')
include ':@hariks789_react-native-background-geolocation'
project(':@hariks789_react-native-background-geolocation').projectDir = new File(rootProject.projectDir, '../node_modules/@hariks789/react-native-background-geolocation/android/lib')
Remove the two lines .
include ':@hariks789_react-native-background-geolocation'
project(':@hariks789_react-native-background-geolocation').projectDir = new File(rootProject.projectDir, '../node_modules/@hariks789/react-native-background-geolocation/android/lib')
Use this line only. It is working
include ':@hariks789_react-native-background-geolocation-common'
project(':@hariks789_react-native-background-geolocation-common').projectDir = new File(rootProject.projectDir, '../node_modules/@hariks789/react-native-background-geolocation/android/common')
But we using alternate package. That package also I got same error. After remove the lines working fine.
https://www.npmjs.com/package/@drewaker/react-native-background-geolocation

Task :hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars FAILED

FAILURE: Build failed with an exception.

What went wrong:
A problem was found with the configuration of task ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars' (type 'LibraryJniLibsTask').

Gradle detected a problem with the following location: '/Users/nirmal/rioapp/node_modules/@hariks789/react-native-background-geolocation/android/lib/build/intermediates/stripped_native_libs/release/out'.
Reason: Task ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars' uses this output of task ':@hariks789_react-native-background-geolocation:stripReleaseDebugSymbols' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed.
Possible solutions:

Declare task ':@hariks789_react-native-background-geolocation:stripReleaseDebugSymbols' as an input of ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars'.
Declare an explicit dependency on ':@hariks789_react-native-background-geolocation:stripReleaseDebugSymbols' from ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars' using Task#dependsOn.
Declare an explicit dependency on ':@hariks789_react-native-background-geolocation:stripReleaseDebugSymbols' from ':hariks789_react-native-background-geolocation:copyReleaseJniLibsProjectAndLocalJars' using Task#mustRunAfter

I think your settings.gradle file like this.
include ':@hariks789_react-native-background-geolocation-common' project(':@hariks789_react-native-background-geolocation-common').projectDir = new File(rootProject.projectDir, '../node_modules/@hariks789/react-native-background-geolocation/android/common') include ':@hariks789_react-native-background-geolocation' project(':@hariks789_react-native-background-geolocation').projectDir = new File(rootProject.projectDir, '../node_modules/@hariks789/react-native-background-geolocation/android/lib')
Remove the two lines .
include ':@hariks789_react-native-background-geolocation' project(':@hariks789_react-native-background-geolocation').projectDir = new File(rootProject.projectDir, '../node_modules/@hariks789/react-native-background-geolocation/android/lib')
Use this line only. It is working
include ':@hariks789_react-native-background-geolocation-common' project(':@hariks789_react-native-background-geolocation-common').projectDir = new File(rootProject.projectDir, '../node_modules/@hariks789/react-native-background-geolocation/android/common')
But we using alternate package. That package also I got same error. After remove the lines working fine.
https://www.npmjs.com/package/@drewaker/react-native-background-geolocation

Thanks
You saved my time
  include ':@mauron85_react-native-background-geolocation-common'
  project(':@mauron85_react-native-background-geolocation-common').projectDir = new File(rootProject.projectDir, '../node_modules/@mauron85/react-native-background-geolocation/android/common')

worked for me


Report REPL errors

I started off with the Either idea, and after some amount of work I realised this might get us pretty much all the way there (the only downside being it just stops when it hits the first error, whereas if we stored errors in Either Error ProofState we could accumulate them in the graph). Is that worth the more complex approach?
I could use some feedback @ttuegel

Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock

the only downside being it just stops when it hits the first error, whereas if we stored errors in Either Error ProofState we could accumulate them in the graph

If I understand correctly, this means that the user cannot continue down another branch of execution when they encounter an error on any branch, is that right? I suppose that's better than the status quo: at least the user can inspect the states leading up to the error, but part of what we had in mind here is to allow users to work around a broken branch (while we fix it, hopefully), to continue other branches of the proof.

No, what happens is we catch all exceptions (then print and discard them) as they happen during each step of execution. What this means is the error is not persisted in the graph, but rather just displayed and discarded. However, the graph is restored to the value it had before the exception occurred, so the user can either continue on another branch, or indeed reproduce the exception as many times as he wants to.


Add tilt compensation

See:
https://github.com/omegatao/M5StackGrey_MPU6886_BMM150_AHRS_sample
and
https://github.com/9PEO0xNd/M5Gray_MPU6886_BMM150

Done


auto mode switch position missing for Kumocloud zone

this issue requires testing when it's safe to put the head in auto mode


Fix for Issue #28

Added option to convert output to json format.

Please ensure that all checks pass before creating a PR.


tric init should not require parameter if already in a target folder

tric init same as #44 for tric use

already does this! :)


Fixed undefined constants and layout issues.

damn, why didnt circleCI launch, I should have fixed the integration...


add all C# 11 error codes

Adding them to this pages F1 keywords means I'll be able to track traffic and prioritize which to write about first.
Fixes #31744

ping @adegeo for review


SMTP notifications

Hello,
I'm having difficulties activating the smtp notifications.
I have tested it first on the server of a friend and it worked OK (with uwsgi).
Now, I'm trying to replicate this on an OpenShift server (with mod_wsgi) and it doesn't work anymore. The trace I get is:
   File "isso/__init__.py", line 98, in __init__
     subscribers.append(SMTP(self))
   File "isso/ext/notifications.py", line 47, in __init__
     with self:
   File "isso/ext/notifications.py", line 79, in __enter__
     self.conf.get('password').encode('ascii', errors='ignore'))
   File "/usr/lib64/python3.3/smtplib.py", line 623, in login
     "%s %s" % (AUTH_LOGIN, encode_base64(user.encode('ascii'), eol='')))
 AttributeError: 'bytes' object has no attribute 'encode'

The configuration file I'm using (the same in both cases):
# Isso configuration file
# vim: set filetype=ini

[general]

# file location to the SQLite3 database, highly recommended to change this
# location to a non-temporary location!
dbpath = comments.db

# required to dispatch multiple websites, not used otherwise.
name =

# Your website(s). If Isso is unable to connect to at least on site, you'll
# get a warning during startup and comments are most likely non-functional.
#
# You'll need at least one host/website to run Isso. This is due to security
# reasons: Isso uses CORS_ to embed comments and to restrict comments only to
# your website, you have to "whitelist" your website(s).
#
# I recommend the first value to be a non-SSL website that is used as fallback
# if Firefox users (and only those) supress their HTTP referer completely:
#
# host =
#     http://example.tld/
#     https://example.tld/
#
host = http://example.tld

# time range that allows users to edit/remove their own comments.
# It supports years, weeks, days, hours, minutes, seconds.
# 3h45m12s equals to 3 hours, 45 minutes and 12 seconds.
max-age = 15m

# Select notification backend for new comments. Currently, only SMTP is
# available.
# Select notification backend(s) for new comments, separated by comma.
# Available backends:
#
# stdout
#     Log to standard output. Default, if none selected.
# smtp
#     Send notifications via SMTP on new comments with activation (if
#     moderated) and deletion links.
notify = smtp

# Log console messages to file instead of standard out.
log-file =

[moderation]
# enable comment moderation queue. This option only affects new comments.
# Comments in modertion queue are not visible to other users until you activate
# them.
enabled = false

# remove unprocessed comments in moderation queue after given time.
purge-after = 30d

[server]
# interface to listen on. Isso supports TCP/IP and unix domain sockets: UNIX
# domain socket listen = unix:///tmp/isso.sock TCP/IP listen =
# http:///localhost:1234/
#
# When gevent is available, it is automatically used for http:// Currently,
# gevent can not handle http requests on unix domain socket (see #295 and #299
# for details).  Does not apply for uWSGI.
listen = http://localhost:8080

# reload application, when the source code has changed. Useful for development.
# Only works with the internal webserver.
reload = off

# show 10 most time consuming function in Isso after each request. Do not use
# in production.
profile = off

[smtp]
# Isso can notify you on new comments via SMTP. In the email notification, you
# also can moderate (=activate or delete) comments.

# self-explanatory, optional
username =<EMAIL_ADDRESS>
# self-explanatory (yes, plain text, create a dedicated account for
# notifications), optional.
password = someverycomplicatedpasswordwithalotof!@#$%^&*(1212345647898

# SMTP server
host = smtp.googlemail.com

# SMTP port
port = 465

# use a secure connection to the server, possible values: none, starttls or
# ssl. Note, that there is no easy way for Python 2.7 and 3.3 to implement
# certification validation and thus the connection is vulnerable to
# Man-in-the-Middle attacks. You should definitely use a dedicated SMTP account
# for Isso in that case.
security = ssl

# recipient address, e.g. your email address
to =<EMAIL_ADDRESS>
# ender address, e.g. "Foo Bar"<EMAIL_ADDRESS>from = "Some name"<EMAIL_ADDRESS>
# specify a timeout in seconds for blocking operations like the
# connection attempt.
timeout = 10

[guard]
# Enable basic spam protection features, e.g. rate-limit per IP address (/24
# for IPv4, /48 for IPv6).

# enable guard, recommended in production. Not useful for debugging purposes.
enabled = true

# limit to N new comments per minute.
ratelimit = 2

# how many comments directly to the thread (prevent a simple while true; do
# curl ...; done.
direct-reply = 3

# allow commenters to reply to their own comments when they could still edit
# the comment. After the editing timeframe is gone, commenters can reply to
# their own comments anyways. Do not forget to configure the client.
reply-to-self = false

[markup]
# Customize markup and sanitized HTML. Currently, only Markdown (via Misaka) is
# supported, but new languages are relatively easy to add.

# Misaka-specific Markdown extensions, all flags starting with EXT_ can be used
# there, separated by comma.
options = strikethrough, autolink, fenced_code, no_intra_emphasis

# Additional HTML tags to allow in the generated output, comma-separated. By
# default, only a, blockquote, br, code, del, em, h1, h2, h3, h4, h5, h6, hr,
# ins, li, ol, p, pre, strong, table, tbody, td, th, thead and ul are allowed.
allowed-elements =

# Additional HTML attributes (independent from elements) to allow in the
# generated output, comma-separated. By default, only align and href are
# allowed.
allowed-attributes =

[hash]
# Customize used hash functions to hide the actual email addresses from
# commenters but still be able to generate an identicon.

# A salt is used to protect against rainbow tables. Isso does not make use of
# pepper (yet). The default value has been in use since the release of Isso and
# generates the same identicons for same addresses across installations.
salt = Eech7co8Ohloopo9Ol6baimi

# Hash algorithm to use -- either from Python's hashlib or PBKDF2 (a
# computational expensive hash function).
#
# The actual identifier for PBKDF2 is pbkdf2:1000:6:sha1, which means 1000
# iterations, 6 bytes to generate and SHA1 as pseudo-random family used for key
# strengthening. Arguments have to be in that order, but can be reduced to
# pbkdf2:4096 for example to override the iterations only.
algorithm = pbkdf2

Sorry to bother,
Thanks in advance.

A regression for Python 3 introduced with https://github.com/posativ/isso/commit/6a3a9ea114417b344caeb88c0cd1c8921b67a1ff. I'm working on it.

Fixed in 0.9.9.


Add setNodes() method

Motivation
TransientVector from persistent-ts uses an optimization for faster setting of many nodes where each new node is only created once.
Description
Apply a similar strategy to the binary tree where with multiple gindexes and nodes each BranchNode is navigated and created at most once. Depending on the distribution of the array on gindexes this approach is either x2 faster or as fast as updating individually
    ✓ setNode loop depth 40 count 8 - contiguous                          118764.8 ops/s    8.420000 us/op        -      92889 runs   1.00 s
    ✓ setNodes depth 40 count 8 - contiguous                              232450.0 ops/s    4.302000 us/op        -     158404 runs   1.01 s
    ✓ setNode loop depth 40 count 8 - spread                              120380.4 ops/s    8.307000 us/op        -      98473 runs   1.01 s
    ✓ setNodes depth 40 count 8 - spread                                  132432.8 ops/s    7.551000 us/op        -     104445 runs   1.01 s
    ✓ setNode loop depth 40 count 32 - contiguous                         32047.17 ops/s    31.20400 us/op        -      26036 runs   1.00 s
    ✓ setNodes depth 40 count 32 - contiguous                             75585.79 ops/s    13.23000 us/op        -      48814 runs   1.00 s
    ✓ setNode loop depth 40 count 32 - spread                             27498.97 ops/s    36.36500 us/op        -      22871 runs   1.00 s
    ✓ setNodes depth 40 count 32 - spread                                 36836.48 ops/s    27.14700 us/op        -      28710 runs   1.00 s
    ✓ setNode loop depth 40 count 256 - contiguous                        3738.248 ops/s    267.5050 us/op        -       3146 runs   1.00 s
    ✓ setNodes depth 40 count 256 - contiguous                            10048.43 ops/s    99.51800 us/op        -       6658 runs   1.00 s
    ✓ setNode loop depth 40 count 256 - spread                            3236.675 ops/s    308.9590 us/op        -       2763 runs   1.00 s
    ✓ setNodes depth 40 count 256 - spread                                4283.077 ops/s    233.4770 us/op        -       3546 runs   1.00 s

Closes https://github.com/ChainSafe/persistent-merkle-tree/issues/49

@wemeetagain Would you mind taking over this PR?


min number of characters?

How about only allowing success if there are atleast 5 characters in the edittext?
How would that be facilitied?
thanks!

But that would be better implemented into xml
thanks

you can do it in xml using the regexps. :)


=> ERROR [5/5] COPY dist /usr/share/nginx/html

FrontEnd dockerfile tries to copy file that does not exist

Hi. U must run from ambar folder
cd ambar/FrontEnd/
npm install
npm run compile

OK thanks for the reply, got it.

ur welcome


User Interface suggestions

Digging elfeed. Coming from feedly (via Web client and the superior (in most ways) GReader app on Android), previously "Google Reader" (sniff)).
Below are a few personal prefs you might appreciate:
I think elfeed would be better to display date as the last column, and for it to be expressed as aging, in relative time, (i.e. 2d for 2 days ago, 3h for 3 hours ago, etc).
When I read/scan feeds/titles I want know when the feed I am reading before I know the Title of the entry. For this reason, I would prefer to see the feed title before the article Title. In fact, I don't need to see it repeating (at least if sorted by title). My eyes dance right and left reading the current layout. My preferred layout, which GREADER provides, is to have the feed source appear once only, in a different face than the items for all the article from the feed. It takes a little more vertical space, but makes scanning MUCH faster.
Also, I would by default not display the unread tag
So, in summary, my ideal layout would be something like:

Feed1 info .................................................... .........  # unread items | # read items | # tagged
.. title1 ................................................................  2d
.. title2 ................................................................  3d
Feed2 info ...............................................................  # unread items | # read items | # tagged
.. title3 ................................................................  1h kittenz
.. title4 ................................................................  1d kittenz

etc
I think this layout allows improved top-down left->right scanning.
If having sections or reorganizing columns like this is infeasible , simply suppressing repeated feed names would reduce much visual clutter and allow the eyes to dance much less.
Thanks for elfeed. Much appreciated!

Thanks for sharing your ideas. However, I prefer the listing the way it
currently works. You can get some control over the listing with
elfeed-search-print-entry-function, but this only works on a line by
line basis and won't allow you to group.
Fortunately you could realistically write your own custom front-end to
Elfeed. In fact, elfeed-search has nothing special going for it other
than being launched with "M-x elfeed". It's practically a separate
package. It uses hooks to get notifications about database changes, and
you could do exactly the same thing with your own front-end. The filter
stuff from elfeed-search should be re-usable, too.

@malcook You could customize this function, which inserts separators where the date changes:
(cl-defun ap/elfeed-search-add-separators (&key (min-group-size 2))
    "Insert overlay spacers where the current date changes.
If no group has at least MIN-GROUP-SIZE items, no spacers will be
inserted. "
    ;; TODO: Use column-specific functions so that, e.g. date column could be grouped by month/year
    (let ((data-fn (lambda (offset)
                     (when-let ((entry (nth offset elfeed-search-entries)))
                       (elfeed-search-format-date (elfeed-entry-date entry)))))
          (insert-date-fn (lambda (date)
                            (ov (line-beginning-position) (line-beginning-position)
                                'before-string (propertize (concat "\n" date "\n")
                                                           'face 'elfeed-search-date-face)
                                'type 'date-separator))))
      (ov-clear)
      (save-excursion
        (goto-char (point-min))
        (cl-loop with largest-group-size = 1
                 with offset = (- 1 elfeed-search--offset) ; 1 is first line
                 with prev-data = (funcall data-fn offset)

                 initially do (funcall insert-date-fn prev-data)

                 while (not (eobp))
                 do (progn
                      (forward-line 1)
                      (incf offset))

                 for current-data = (funcall data-fn offset)
                 if (not (equal current-data prev-data))
                 do (progn
                      (funcall insert-date-fn current-data)
                      (setq prev-data current-data))
                 else do (incf largest-group-size)

                 finally do (when (< largest-group-size min-group-size)
                              (ov-clear))))))

@alphapapa I really like your setup. Could you be more explicit about how you get elfeed to look like this? How do you add your function to elfeed?

@timmli I use elfeed-goodies, so I replace the drawing function like this:
(advice-add #'elfeed-goodies/entry-line-draw :override #'ap/elfeed-goodies/entry-line-draw)

@alphapapa Sorry, I still don't get it. Where does ap/elfeed-goodies/entry-line-draw come from?

Oh, that's in my config:
(defun ap/elfeed-goodies/entry-line-draw (entry)
  "My version of this function.  Prints ENTRY to the buffer."

  (cl-flet ((add-faces (str &rest faces)
                       (dolist (face faces str)
                         (add-face-text-property 0 (length str)
                                                 face 'append str)))
            (tags (entry)
                  (seq-difference (--map (substring-no-properties (symbol-name it))
                                         (elfeed-entry-tags entry))
                                  '("unread" "starred"))))
    (let* ( ;; Choose color and faces first
           ;; See https://www.reddit.com/r/emacs/comments/7a976a/face_applied_to_result_of_symbolname_becomes/
           (site (pocket-reader--url-domain (elfeed-entry-link entry)))
           (hash (rainbow-identifiers--hash-function site))
           (site-face (rainbow-identifiers-cie-l*a*b*-choose-face hash))
           (title-faces (elfeed-search--faces (elfeed-entry-tags entry)))

           ;; Feed
           (feed (elfeed-entry-feed entry))
           (feed-title (when feed
                         (or (elfeed-meta feed :title) (elfeed-feed-title feed))))
           (feed-width elfeed-goodies/feed-source-column-width)
           (feed-column (elfeed-format-column feed-title feed-width :left))
           (feed-column (apply #'add-faces feed-column site-face title-faces))

           ;; Tags before title (so title can use the width of the tags column for this item)
           (tags (tags entry))
           (tags-str (s-join "," tags))
           ;; Use raw tag list to check for starred
           (starred-p (member 'starred (elfeed-entry-tags entry)))
           (tags-width (min (length tags-str)
                            elfeed-goodies/tag-column-width))
           (tag-column (elfeed-format-column tags-str tags-width :right))
           (tag-column (apply #'add-faces tag-column site-face title-faces))

           ;; Title
           (title (or (elfeed-meta entry :title) (elfeed-entry-title entry) ""))
           (title-width (- (window-width) feed-width tags-width 4))
           (title-column (elfeed-format-column (truncate-string-to-width title title-width nil nil 'ellipsis) title-width :left))
           (title-column (apply #'add-faces title-column site-face title-faces)))

      (insert feed-column " ")
      (insert (if starred-p
                  (propertize "*"
                              'face 'pocket-reader-favorite-star)
                " ")
              " ")
      (insert (propertize title-column 'kbd-help title) " ")
      (insert tag-column))))

@alphapapa Thanks! One more question: How am I supposed to include your function ap/elfeed-search-add-separators there?

Just add it to elfeed-search-update-hook, like:
(add-hook 'elfeed-search-update-hook #'ap/elfeed-search-add-separators)

Just add it to elfeed-search-update-hook, like:
(add-hook 'elfeed-search-update-hook #'ap/elfeed-search-add-separators)

hy i stumbled upon this and try to use it but its not working as shown in screenshot
only first date is shown and after that all feeed come under same date
i have this in doom emacs config
(after! elfeed-goodies

(add-hook 'elfeed-search-update-hook #'ap/elfeed-search-add-separators)
(cl-defun ap/elfeed-search-add-separators (&key (min-group-size 2))
    "Insert overlay spacers where the current date changes.
If no group has at least MIN-GROUP-SIZE items, no spacers will be
inserted. "
    ;; TODO: Use column-specific functions so that, e.g. date column could be grouped by month/year
    (cl-labels ((count-date-items (date)
                                  (cl-loop for entry in elfeed-search-entries
                                           when (equal date (elfeed-search-format-date (elfeed-entry-date entry)))
                                           count it))
                (insert-date (date &key count)
                             (ov (line-beginning-position) (line-beginning-position)
                                 'before-string (propertize (format "\n%s (%s)\n" date count)
                                                            'face 'elfeed-search-date-face)
                                 'type 'date-separator))
                (entry-date (offset)
                      (when-let ((entry (nth offset elfeed-search-entries)))
                        (elfeed-search-format-date (elfeed-entry-date entry)))))
      (ov-clear)
      (save-excursion
        (goto-char (point-min))
        (cl-loop with largest-group-size = 1
                 with offset = (- 1 elfeed-search--offset) ; 1 is first line
                 with prev-data = (entry-date offset)

                 initially do (insert-date prev-data
                                           :count (count-date-items prev-data))

                 while (not (eobp))
                 do (progn
                      (forward-line 1)
                      (incf offset))

                 for current-data = (entry-date offset)
                 if (not (equal current-data prev-data))
                 do (progn
                      (insert-date current-data
                                   :count (count-date-items current-data))
                      (setq prev-data current-data))
                 else do (incf largest-group-size)

                 finally do (when (< largest-group-size min-group-size)
                              (ov-clear))))))
)


General question

How do I approach securing the connection to the ws-tcp-relay ?
I have a web app (react) that needs to connect the ws-tcp-relay but i dont want any old person on the internet to access the websocket

A very simple way to do this at the HTTP endpoint would be to use a reverse proxy with HTTP basic auth. I know Caddy (see below) provides this functionality out of the box but I imagine you could do the same with nginx or Apache.
Another option would be to build an authentication handshake into your application protocol; in other words, allow anyone to connect but require an auth token to be sent over the websocket connection (and close the connection if the token is invalid).
Perhaps HTTP basic auth could be added to ws-tcp-relay, but I figure in most scenarios where endpoints are exposed to the public internet there is either auth built in to the protocol being relayed or it's already running behind a reverse proxy for firewalling/load-balancing/etc.
Specific Caddy docs:
https://caddyserver.com/docs/websocket
https://caddyserver.com/docs/basicauth

Thanks !


Meta description fallback value / Posted from translations dictionary

Found these 2 minor bugs when trying out the starterkit in a translated site.
Thanks for all the effort!

Thanks so much for catching these.
I've merged them into this but I'll be releasing a new version later today which comes from a different repo


pspnet training

hello, when I trained the pspnet, I found the ouput of this network has 2 tensors. So, the loss function cannot accept this output and lead to crash. Could you tell me which loss function should be used in this network ?

When training, there are two loss functions, only one in the test phase.

hello, when I trained the pspnet, I found the ouput of this network has 2 tensors. So, the loss function cannot accept this output and lead to crash. Could you tell me which loss function should be used in this network ?

When using PSPNet, it seems that you should use "multi_scale_cross_entropy" instead of "cross_entropy" in your config file.


[BUG] Xamarin Forms repeated TabNavigation causes app to slow and eventually lock

Describe the bug
When implementing Xamarin Forms Tab Navigation, where each tab is a page with its own navigation, the app will quickly start to Slow down on navigations, where the first navigation will be instant, but the 5th will take seconds.
Steps To Reproduce
Provide the steps to reproduce the behavior:
setup a tab page with each tab having its own routedviewhost
            var homeDashboardView = Locator.CurrentMutable.GetService<DashboardViewModel>();
            homeDashboardView.Router.NavigateAndReset.Execute(homeDashboardView).Subscribe();
            Locator.CurrentMutable.RegisterConstant(homeDashboardView, typeof(IScreen));
            hostDashboardView = new RoutedViewHost { Router = homeDashboardView.Router, Title = "\uf0b1 Jobs" };

            var homeMapView = Locator.CurrentMutable.GetService<MapViewModel>();
            homeMapView.Router.Navigate.Execute(homeMapView).Subscribe();
            Locator.CurrentMutable.RegisterConstant(homeMapView, typeof(IScreen));
            hostMapView = new RoutedViewHost { Router = homeMapView.Router, Title = "\uf279 Map" };

            var homeDocumentsView = Locator.CurrentMutable.GetService<DocumentsViewModel>();
            homeDocumentsView.Router.Navigate.Execute(homeDocumentsView).Subscribe();
            Locator.CurrentMutable.RegisterConstant(homeDocumentsView, typeof(IScreen));
            hostDocumentsView = new RoutedViewHost { Router = homeDocumentsView.Router, Title = "\uf0c6 Document(s)", IsVisible = false };

            NavigationPage.SetHasNavigationBar(hostDashboardView.CurrentPage, false);
            NavigationPage.SetHasNavigationBar(hostMapView.CurrentPage, false);
            NavigationPage.SetHasNavigationBar(hostDocumentsView.CurrentPage, false);

            this.Children.Add(hostDashboardView);
            this.Children.Add(hostMapView);
            this.Children.Add(hostDocumentsView);

            Locator.CurrentMutable.RegisterConstant(homeDashboardView, typeof(IScreen));

Expected behavior
you should navigate to the next page, but the tabs persist. This works, except for the fact that each navigation takes longer and longer, until the point where the app will no longer respond. Pages/VM are never GC'ed.
Environment(please complete the following information):

OS: Windows
Version 9.2.1
Device: Xamarin.UWP

@lackrin Thank you for reporting this issue.    I see that you provided some sample code.  Do you have capacity to provide a full sample? This helps verifying the fix when the issue is resolved.
I also noticed you explicitly stated that it is a Xamarin.UWP issue, are you seeing this in any other Xamarin targets?

I can try to make a sample app to get the same results.
I actually haven't tested with anything else because UWP is our primary platform
Do you have any examples of tab navigation where the tab remains the parent and only the tab content navigates?
On Mar 25, 2019 11:50 AM, Rodney Littles II<EMAIL_ADDRESS>wrote:
@lackrinhttps://github.com/lackrin Thank you for reporting this issue. I see that you provided some sample code. Do you have capacity to provide a full sample? This helps verifying the fix when the issue is resolved.
I also noticed you explicitly stated that it is a Xamarin.UWP issue, are you seeing this in any other Xamarin targets?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/reactiveui/ReactiveUI/issues/1983#issuecomment-476308866, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATXZ8ZaKpny9YtgupbvWo9v3XzXnROygks5vaQxsgaJpZM4cHeto.

https://dev.azure.com/LackrinProjects/_git/ReactiveUITabNavigation
Sample APP. Problem is that it works and does not replicate the issue. I have noticed though that the View and ViewModels for the subpage are not being disposed of. I suspect this is the root of my issue, can you please verify?

I will pull this down this evening and see if I can make sense of it.  Thank you for taking the time to create the sample.  Appreciated.

Sorry, found issue with slowdown, it was of my own doing. Had an event being added recursively

Still the Model not disposing is an issue still.


Placeholder doesn't work on .tscn scenes.

Godot version:
Built from source 7913e7
Issue description:
The placeholder option doesn't work on a instance of a .tscn scene. But an instance of this same scene saved as .scn works as a placeholder.

Actually, superseded by #7345.


Add Electron support for theming MacOS titlebar

Latest versions of Atom added the ability to theme or hide the MacOS titlebar, which apparently is due to the ability being added to latest versions of the Electron shell.

May be duplicate with #12377 ?

Dup of #12377.


Handling not root level common folder

Problem
When providing a path (e.g. src/common) instead of a folder name to the common_folder input the action fails to include the common code into the zip file.
Slack thread
https://cognitedata.slack.com/archives/C010VCGKBCM/p1669210271655089

Bumping this thread, it would be great to be able to deploy a nested common folder when working with monorepos. Considering a nested structure is currently supported for the functions themselves, I think it would be better if this works the same for the common folder as well. I currently can't find a way to work around this because of this restriction (see e.g. https://github.com/cognitedata/function-action-oidc/pull/73). Would this be solved by supporting a config for setting the working directory within the github action?

Nvm, I see now that this is being worked on in https://github.com/cognitedata/function-action-oidc/pull/75

@ddonukis @Magssch  This is now available by using v2 thanks to @petreeb !
- name: Deploy and schedule ${{ matrix.function }}
  uses: cognitedata/function-action-oidc@v2

I would treat this as beta for now, please provide feedback if anything is not working as expected!


Vue3 + TypeScript  = TS7016: Could not find a declaration file for module 'vue-yandex-maps'

При попытке в компоненте сделать
import { yandexMap } from 'vue-yandex-maps'
Получаю ошибку
TS7016: Could not find a declaration file for module 'vue-yandex-maps'. {{path_to_project}}/node_modules/vue-yandex-maps/dist/vue-yandex-maps.umd.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/vue-yandex-maps` if it exists or add a new declaration (.d.ts) file containing `declare module 'vue-yandex-maps';`

package.json
"dependencies": {
    "@vueuse/core": "^6.9.0",
    "axios": "^0.21.4",
    "vee-validate": "^4.5.4",
    "vue": "^3.0.0",
    "vue-class-component": "^8.0.0-0",
    "vue-router": "^4.0.12",
    "vue-yandex-maps": "^0.12.3",
    "vuex": "^4.0.2"
  },
"devDependencies": {
    "@typescript-eslint/eslint-plugin": "^4.18.0",
    "@typescript-eslint/parser": "^4.18.0",
    "@vue/cli-plugin-babel": "^4.5.15",
    "@vue/cli-plugin-eslint": "^4.5.15",
    "@vue/cli-plugin-typescript": "^4.5.15",
    "@vue/cli-service": "^4.5.15",
    "@vue/compiler-sfc": "^3.0.0",
    "@vue/eslint-config-typescript": "^7.0.0",
    "babel-eslint": "^10.1.0",
    "eslint": "^6.7.2",
    "eslint-plugin-vue": "^7.20.0",
    "sass": "^1.43.4",
    "sass-loader": "^10.0.3",
    "typescript": "~4.1.5"
  },

Скоро будет бета-версия плагина на TS для Vue3. Дописываю документацию

You can create file vue-yandex-maps.d.ts with
declare module 'vue-yandex-maps';

@rodial Please, can you be more specific?

@Goryyn
In src/ create file vue-yandex-maps.d.ts with one line
declare module 'vue-yandex-maps';

It's not good solution, but removes the error and I used it
After it, if you don't want to see warnings like "Unknown html tag ymap-marker" use
import {yandexMap as YandexMap, ymapMarker as YmapMarker} from "vue-yandex-maps";

Also you can try to declare export, like
declare module 'vue-yandex-maps' {
  import {Component} from "vue";
  export const yandexMap: Component
  ...
}

...but I was too lazy for it
Information about it

Fixed with 1.x/2.x


Crash on startup

Terminal: foot
OS: Gentoo
on Wayland
thread 'main' panicked at /home/rat/.cargo/registry/src/index.crates.io-6f17d22bba15001f/bk-0.6.0/src/epub.rs:60:63:
called `Result::unwrap()` on an `Err` value: UnknownEntityReference("nbsp", TextPos { row: 12, col: 32 })
stack backtrace:
   0: rust_begin_unwind
             at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/std/src/panicking.rs:645:5
   1: core::panicking::panic_fmt
             at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/core/src/panicking.rs:72:14
   2: core::result::unwrap_failed
             at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/core/src/result.rs:1649:5
   3: bk::epub::Epub::new
   4: bk::main

That's an old bug, https://github.com/aeosynth/bk/issues/13 . Your book has bad xml, can't do anything unless we change the xml library.


[Issue]: version changing and alternative try install.ps1 file miss issue

What happened?
if i change the version it was getting sucess but if i check nvm -v it was getting the older version or if i clone this git https://github.com/coreybutler/nvm-windows.git install.ps1 file w

as missing
What did you expect to happen?
the version want to change or install.ps1 file i want to get
Version
1.1.11 or newer (Default)
Which version of Windows?
No response
Which locale?
None
Which shell are you running NVM4W in?
No response
User Permissions?
Administrative Privileges, Elevated
Is Developer Mode enabled?
None
Relevant log/console output
PS C:\nvm> nvm on
nvm enabled
Now using node v20.11.1 (64-bit)
PS C:\nvm> nvm use 18
Now using node v18.19.1 (64-bit)
PS C:\nvm> nvm -v
1.1.12

Debug Output
i tried

Anything else?
no

I don't see anything wrong here. You ran nvm use 18, which configured Node 18.19.1. This is shown in the output of the debug command (Active Node.js Version: v18.19.1).
nvm -v gives you the version of NVM for Windows.
node -v gives you the version of Node.
It appears you're expecting nvm -v to show the Node version.

After I gave nvm use different version it was taking the older version 1.1
version is this not a issue
On Wed, 27 Mar, 2024, 08:16 Corey Butler, @.***> wrote:

I don't see anything wrong here. You ran nvm use 18, which configured
Node 18.19.1. This is shown in the output of the debug command (Active
Node.js Version: v18.19.1).
nvm -v gives you the version of NVM for Windows.
node -v gives you the version of Node.
It appears you're expecting nvm -v to show the Node version.
—
Reply to this email directly, view it on GitHub
https://github.com/coreybutler/nvm-windows/issues/1118#issuecomment-2021818103,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZCKV3V4GJUN5PRBCBA66ATY2IXIBAVCNFSM6AAAAABFIV6SFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRRHAYTQMJQGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm sorry, but I'm not following what the problem is here. This is the output you presented:
PS C:\nvm> nvm on
nvm enabled
Now using node v20.11.1 (64-bit)
PS C:\nvm> nvm use 18
Now using node v18.19.1 (64-bit)
PS C:\nvm> nvm -v
1.1.12

That is what I would expect for the commands you ran. However; I think you should be running these instead:
PS C:\nvm> nvm use 18
Now using node v18.19.1 (64-bit)
PS C:\nvm> node -v
v18.19.1

I don't really understand why you're running nvm -v in the first place.
We don't ship an install.ps1 file (never have). You can't run more than one version of nvm.exe on a system at a time. It looks like you upgraded NVM4W to v1.1.12. Upgrades overwrite prior versions. Maybe that's what the install.ps1 you have does... but I have no idea because I didn't create that file.

I didn't ran nvm -v at first it was in last.after I change the version it
was getting the older and the downloaded version was not changing to other
versions.
On Wed, 27 Mar, 2024, 08:47 Corey Butler, @.***> wrote:

I'm sorry, but I'm not following what the problem is here. This is the
output you presented:
PS C:\nvm> nvm on
nvm enabled
Now using node v20.11.1 (64-bit)
PS C:\nvm> nvm use 18
Now using node v18.19.1 (64-bit)
PS C:\nvm> nvm -v
1.1.12
That is what I would expect for the commands you ran. However; I think you
should be running these instead:
PS C:\nvm> nvm use 18
Now using node v18.19.1 (64-bit)
PS C:\nvm> node -v
v18.19.1
I don't really understand why you're running nvm -v in the first place.
We don't ship an install.ps1 file (never have). You can't run more than
one version of nvm.exe on a system at a time. It looks like you upgraded
NVM4W to v1.1.12. Upgrades overwrite prior versions. Maybe that's what the
install.ps1 you have does... but I have no idea because I didn't create
that file.
—
Reply to this email directly, view it on GitHub
https://github.com/coreybutler/nvm-windows/issues/1118#issuecomment-2021849273,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZCKV3QBCD2BJE4E5S25IGDY2I26FAVCNFSM6AAAAABFIV6SFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRRHA2DSMRXGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Trying to implement a permissionless home and user profile page

Scenario
I'm trying to implement a placeholder user for home and user profile page.
auth-context.tsx
// ... (other imports)
import { useRouter } from 'next/router';
...
export function AuthContextProvider({
  children
}: AuthContextProviderProps): JSX.Element {
...

  // Use useRouter from next/router to determine the current route
  const router = useRouter();

  // Placeholder user object
  const placeholderUser: User = useMemo(() => ({
    id: 'placeholder',
  }), []);

  useEffect(() => {
    // Check if the current page is the homepage or user profile homepage
    const isHomeOrUserProfile = router.pathname === '/' || router.pathname === '/[id]';

    // If it is, set the user state to the placeholder user
    if (isHomeOrUserProfile) {
      setUser(placeholderUser);
      setLoading(false);
    }
  }, [router.pathname, placeholderUser]);

  // Modify the handleUserAuth function to update the placeholder user when explicitly logging in
  const handleUserAuth = (authUser: AuthUser | null): void => {
    setLoading(true);

    if (authUser) {
      manageUser(authUser);
    } else if (!router.pathname.startsWith('/[id]')) {
      // Only set to null if not on a user profile page
      setUser(null);
      setLoading(false);
    }
  };

...
  return <AuthContext.Provider value={value}>{children}</AuthContext.Provider>;
}

Problem
It loads the placeholder successfully on homepage, yet the firebase auth throws 401.
Question
Could someone please point me to the right way get auth info out of this placeholder user for firebase?

You would need to update the firestore and storage rules to allow unauthorized/logged out users to have read permissions and then you would need to remove ProtectedLayout from the home and user page.


[ALLUXIO-1783] Fix Javadoc in SetAttributeOptionsTest.defaultsTest()

The javadoc for defaultsTest() should refer to SetAttributeOptions.

Can one of the admins verify this patch?

okay to test

@d9liang thanks for the fix ... LGTM


upload

What this Pull Request (PR) does
Please briefly describe what this PR does.
Related issues
Please reference any open issues this PR relates to in here.
If it closes an issue, type closes #[ISSUE_NUMBER].
Screenshots
Provide any screenshots you may find relevant to facilitate us understanding your PR.

Absolutely no PR description...

We're getting ready to migrate to Golang; please resubmit then if this hasn't been addressed. Thank you!


Muxado Heartbeat example with regular Stream

Hi,
I'm struggling to implement heartbeat with muxado. The example provided forces to use TypedSession which does not implement AsyncRead and AsyncWrite traits. Is there any way to get around that?

So I figured out how to get actual stream, but still I'm not sure how to implement heartbeat.
I guess this needs to be set up on both server/client since typed sessions are prefixed with type. So I implemented roughtly something like this (don't mind broken indentation):
Client:
        let typed = Typed::new(sess);
        let (mut heartbeat_sess, hbctl) = Heartbeat::start(
            typed,
            HeartbeatConfig {
                handler: Some(Arc::new(|lat| async move {
                    info!("got heartbeat {:?}", lat);
                    Result::<(), Box<dyn std::error::Error>>::Ok(())
                })),
                ..Default::default()
            },
        )
        .await?;

// later in stream handling loop:
let mut stream = heartbeat_sess.open_typed(StreamType::default()).await?;
do_stuff_with_stream(&mut *stream);

Server:
        let typed = Typed::new(sess);
        let (mut heartbeat_sess, hbctl) = Heartbeat::start(
            typed,
            HeartbeatConfig {
                handler: Some(Arc::new(|lat| async move {
                    info!("got heartbeat {:?}", lat);
                    Result::<(), Box<dyn std::error::Error>>::Ok(())
                })),
                ..Default::default()
            },
        )
        .await?;

// later in stream accepting loop
let mut stream = heartbeat_sess.accept_typed().await?;
do_something_with_the_stream(&mut *stream);

Now at first this seemed to work - server was printing 'got heartbeat' but without latency info (got None there). I checked network activity with tcpdump and dicovered there is no activity so no heartbeat is sent accross network.
I managed to kinda get it to work manually by creating a task that runs hbctl.beat().await? and sleeps. This indeed generates some traffic every time it beats, but I guess this functionality is used for manual checking  of underlying network quality.
Help much appreciated.

Ok, managed to get this to work, but I don't know why it failed. The code initializing heartbeat was calling a tokio::spawn that would .open_typed(...) on this heartbeat. This for some reason caused the internal heartbeat requester to fail to write to stream and die. When I moved stream init into same spawn where .open_typed(...) happens it started working. whew :D


Tax_funcs.py condensation of functions

Condense all the total_taxes functions into one function, with multiple methods for the various shapes of inputs/outputs needed.

Also, combine the payroll.py file with tax_funcs file.  We don't need an entire file for one payroll function. This fits better with tax_funcs.py.

Combined payroll.py with tax_funcs.py by adding the replacement_rates function as a function in tax_funcs.

Didn't mean to close it.  The lump sum functions have been combined, but still need to combine other functions.

Total tax functions have all been combined into one function with multiple methods to specify which tax.


Spec text editor signoff?

@bterlson, you're up :-) the designated reviewers have signed off in #8.
Can you review the spec text? I'd love to shoot for stage 3 at the November meeting.

Remove explicit length unless needed.
Clause 4 step 6 seems superfluous given the prose at the end of clauses 1-3.

Otherwise seems fine. There is a bit of duplication here with GetOwnPropertyKeys but given that that API returns an Array not a List I am not sure if it's worth using it :-P.

Thanks, I'll remove the length.
Regarding Clause 4 step 6 - this is present in the current spec in both Object.keys and EnumerableOwnNames - I think that if it should be removed from EnumerableOwnNames, it should be done directly to the spec and not as part of this proposal.
Regarding GetOwnPropertyKeys - unlike EnumerableOwnNames, which only returns enumerable string keys, GetOwnPropertyKeys also returns nonenumerable keys, and can return either String or Symbol keys. I could refactor EnumerableOwnProperties to call GetOwnPropertyKeys with a Type of String, but then I'd need to use an enumerability check in step 5.a - so EnumerableOwnProperties wouldn't get any simpler. Alternatively, if I implemented GetOwnPropertyKeys in terms of EnumerableOwnProperties - then EnumerableOwnProperties wouldn't be "Enumerable" anymore. In other words, I don't think this is actual duplication, nor do I think it's easily resolved, without a mega-method that allowed for specifying "enumerable or non-enumerable or both", as well as "string or symbol or both" - it'd be pretty hairy.
Given these, do you think this can be counted as officially signed off?

This has been officially signed off, and the proposal is now at stage 3!


TESTING: DO NOT MERGE

TESTING: DO NOT MERGE

Codecov Report

Merging #475 into develop will increase coverage by 74.47%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop     #475       +/-   ##
============================================
+ Coverage    19.36%   93.83%   +74.47%
============================================
  Files          175      238       +63
  Lines         2846     3615      +769
  Branches       259      282       +23
============================================
+ Hits           551     3392     +2841
+ Misses        2295      223     -2072

Impacted Files
Coverage Δ

...om/twitter/inject/thrift/ThriftMethodBuilder.scala
81.81% <0%> (ø)

...inatra/thrift/modules/ExceptionManagerModule.scala
100% <0%> (ø)

...r/inject/thrift/modules/ThriftClientIdModule.scala
100% <0%> (ø)

...ter/inject/thrift/modules/ThriftClientModule.scala
100% <0%> (ø)

...witter/inject/thrift/utils/ThriftMethodUtils.scala
100% <0%> (ø)

.../twitter/inject/thrift/ThriftClientException.scala
100% <0%> (ø)

.../main/scala/com/twitter/finatra/sample/Queue.scala
75% <0%> (ø)

...scala/com/twitter/hello/HelloWorldController.scala
100% <0%> (ø)

.../twitter/inject/exceptions/PossiblyRetryable.scala
91.3% <0%> (ø)

...ect/thrift/filters/ThriftClientFilterBuilder.scala
100% <0%> (ø)

... and 212 more

Continue to review full report at Codecov.

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


over 250 warning, error. can't be displayed.

VSCode Version: 1.3.1

over 250 warning, error. can't be displayed.
_maxDiagnosticsPerFile that are hard-coded.
extHostDiagnostics.ts#L16
want to change in user setting.
(of course understanding impact on performance)

That's by design for (1) performance reasons (we don't want to allow the extension host spamming the main thread) and (2) we don't believe a user will appreciate/being able to act on so many errors.
@owayo What is your use-case that requires to have more than 250 diagnostics per file?

Thanks for reply.
use-case
code review of too meny warnings include script from before the change.
e.g.
originally script too many warning include. (over 250)
the script end line edit. (new warning add)
too many warnings to reach the change point, can't be checked a warning or error of Changes

I still have trouble understanding why more than 250 messages are desirable. Is it that only the 251st diagnostic information is important? The model isn't append but replace when DiagnosticCollection.set is called so there is some control on the extension side.

yes, 251st diagnostic imformation is important.
It should marked-up warning is not marked-up.

And the first 250 diagnostics messages aren't important? Then just update the collection with the last n (n < 250) messages. The limit of 250 is per diagnostics collection so entirely up to you. Can you by any chance point me to some of your code where this happens?

yes. first 250 diagnostics messages are not important.
sorry, It can not be published because it is the company app.
e.g.
VSCode installed ESLint.
Lint rule
"rules": {
    "quotes": [
        "warn",
        "single"
    ]
}

JavaScript (check target)
var hoge = ""
    + ""
    + ""
    + ""
    ~~~ repeat 250 over ~~~
    + ""
    + "";

var fuga = ""; // <- want to check!

double-quote of fuga is not warning display.

So, you are not authoring the extension but just using it (like eslint)?

I'm afraid I don't understand. sorry.
English is not my first language.
I hope 251st(and over) warnings, error display. (good in any way)
If the implementation is difficult, just use use it.

I still believe that not showing thousands of errors is a good idea because the squiggle should be a call for attention. Also the editor not more than 500 squiggles for the same reasons. My general advice to get rid of those warnings - either by changing your code or lint configuration.
What we can do (tho I doubt it will help in your specific case) is to be smarter when cropping the list of errors. Instead of taking the first 250 we could sort by severity such that you get errors over warnings. Obviously that doesn't help if all diagnostics are of the same type. Last, we could also add a synthetic error that tells you how many errors have been ignored

thank you a lot.
looking forward to seeing release.

(1) performance reasons (we don't want to allow the extension host spamming the main thread)

is 252 not okay for performance?

(2) we don't believe a user will appreciate/being able to act on so many errors.

It's not just errors that this applies to it's also linting, a program still works if it has lint errors, as it is I just can't see those linting errors in 90% of my program

I hope this hard arbitrary limit will be changed to be user configurable

It also limits the number of spelling errors that the extension Spell Right can find, to 250.


Proposal to use Github and single VSK_Map or single VSDK_Avatar source repositories

Goal: Store source assets in version control and allow remixing in a social network.

Make a separate personal organization
Single VSK_Map or Single VSK_avatar per git repo
Use github
git-lfs, DVC and git-annex are banned
Regular limits of github

100 MB single files
total repository size on Github is < 5 GB

git config --global alias.squash-all '!f(){ git reset $(git commit-tree HEAD^{tree} -m "${1:-A new start}");};f' git squash-all

.tres and .tscn files over 10MB should ideally not be committed. It might be a good rule to avoid committing .res, .scn or .tres or .tscn files with binary assets.

Todo: Transfer into a proposal.

I have created https://github.com/V-Sekai-fire/.