Implament Headers

Headers were not initialized and Dictionary did not convert to WebHeaderCollection. so I changed the type.
Addresses issue #84

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

Travis failed on pester?

Ah. I believe the PowerShell team removed Pester as a dependency recently. We'll need to update the Travis build to install Pester.

Do I need to do anything to trigger the Travis tests?

@BladeFireLight Just merged in the fix for this: https://github.com/PowerShell/Polaris/pull/86
just grab the latest master bits via merge or rebase and you should be good to go

Also, I'd really like to see at least one Pester test for this so that we can avoid this in the future. I think it would be an addition here:
Add a route that sends back a test header here:
https://github.com/PowerShell/Polaris/blob/master/test/PolarisServer.ps1
And then do an Invoke-WebRequest and check to make sure the header is in the response here:
https://github.com/PowerShell/Polaris/blob/master/test/PolarisServer.Tests.ps1

I noticed while setting this up all the tests are dependent on IE. and IE having been run once under the profile that runs the test.  Unless the tests actually required IE's rendering engine, why not use -UseBasicParsing?

You're right. We should be using -UseBasicParsing

This PR looks great! Thanks for contributing!


feat/ini light space

chore: update
chore: update

Codecov Report

Merging #110 (d0d1485) into main (4d3b5c0) will increase coverage by 3.05%.
The diff coverage is 11.11%.

@@           Coverage Diff            @@
##            main    #110      +/-   ##
========================================
+ Coverage   1.41%   4.46%   +3.05%
========================================
  Files        634     636       +2
  Lines       5160    5168       +8
  Branches    1070    1071       +1
========================================
+ Hits          73     231     +158
+ Misses      4910    4764     -146
+ Partials     177     173       -4

Impacted Files
Coverage Δ

...mponents/ModalTwitterVerify/ModalTwitterVerify.tsx
0.00% <0.00%> (ø)

apps/app/src/hooks/useProvider/index.ts
0.00% <0.00%> (ø)

apps/app/src/hooks/useProvider/useProvider.ts
8.33% <8.33%> (ø)

...useProviderCyberConnect/useProviderCyberConnect.ts
12.50% <25.00%> (+12.50%)
:arrow_up:

...s/app/src/components/FollowButton/FollowButton.tsx
1.26% <0.00%> (+1.26%)
:arrow_up:

...tRecommendations/useCyberConnectRecommendations.ts
1.61% <0.00%> (+1.61%)
:arrow_up:

...s/useCyberConnectPopular/useCyberConnectPopular.ts
1.66% <0.00%> (+1.66%)
:arrow_up:

...useCyberConnectRankings/useCyberConnectRankings.ts
1.66% <0.00%> (+1.66%)
:arrow_up:

apps/app/src/hooks/useAuth/useAuth.ts
1.88% <0.00%> (+1.88%)
:arrow_up:

apps/app/src/hooks/useNetworks/useNetworks.ts
2.50% <0.00%> (+2.50%)
:arrow_up:

... and 117 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


feat: dependency updates and using axios

Moving from sync-request to axios with async await.

for a long time (since the beginning), we were using sync-request.


RMC 124 Encrypt print files and generate manifest data based on the encrypted file

Motivation and Context

All files being sent to our third party suppliers need to be encrypted as per the spec: https://collaborate2.ons.gov.uk/confluence/display/CATD/Proposed+File+Encryption+and+Transfer+Mechanism
in summary:

suppliers will share their public key with us which we use to encrypt.
we should also encrypt the file with multiple recipients (one for the supplier and one for us), this will enable us to keep our own decryptable copy of the file
each file should be encrypted with the specific supplier and us as recipients

What has changed

added encrypt with PGPy function
new expected environment variables:

OUR_PUBLIC_KEY_PATH
OTHER_PUBLIC_KEY_PATH

CSV print files are now encrypted before sending elsewhere
tests updated, including decrypting to verify with dummy secret key

How to test?

Running the tests locally

make test

Running the scripts in k8s

generate two pgp key pairs (or use the test ones in tests/resources)
kubectl create secret generic pgp-keys --from-file=our-public-key=tests/resources/our_dummy_public.asc --from-file=other-public-key=tests/resources/supplier_dummy_public.asc
make apply-deployment
make connect-to-pod
python generate_qid_batch.py unaddressed_batch.csv  # outputs batch ID
python generate_print_files.py unaddressed_batch.csv <print file directory path> <batch ID>

Running acceptance tests

make up  # in docker-dev repo

cat > .env << EOS
export OUR_PUBLIC_KEY_PATH="tests/resources/our_dummy_public.asc"
export OTHER_PUBLIC_KEY_PATH="tests/resources/supplier_dummy_public.asc"
EOS

pipenv run behave acceptance_tests/

Links

Trello card #779

@NickGrantONS admittedly, it was lazy to pick the one key labelled "supplier" as the private key example for the test. I'll change / rename that so that it's more obvious that it should only ever be our own.
The plan for the public keys is to store them as secrets and mount them as a volume through k8s, so this checks out. The test keys purely exist in the repo for the tests not as a pattern for how the scripts are used in GCP.

Once #5 is merged, the acceptance test will need to be updated to decrypt before verifying the contents of the file/s.

@NickGrantONS the supplier private key is used in the acceptance test to validate that it can be decrypted with either. (Shouldn't be present outside of the acceptance test directory).
I'll remove the duplicated public sections.
The dummy files are duplicated but that was due to the docker ignore of the tests directory. Perhaps the unit tests can use the acceptance test key/s.


[Bug] response get stuck with Flask when mimetype="text/event-stream"

reproduce demo code:
import time
from flask import Flask, Response

app = Flask(__name__)

@app.route("/stream-demo")
def stream_demo():
    def _stream():
        while True:
            time.sleep(0.1)
            yield "data: %s\n\n" % str(time.time())

    return Response(_stream(), mimetype="text/event-stream")

if __name__ == "__main__":
    app.run(debug=True)

raw Flask run:

run with granian:
granian --interface wsgi app:app

@gi0baro Windows 11, Python 3.9.19, granian 1.4.2, Flask 3.0.3.

@CNFeffery can you please try the same with 1.4.4 and verify the issue is still there?

@gi0baro has the same status with 1.4.4, the stream response does not return data step by step smoothly.


tests for full markdown reporter container

Describe the feature or problem you’d like to solve
I realised I didn't write the tests for the container of the full markdown reporter in #157
Proposed solution
Write the tests
Additional context
None

Yup! :)


fixes spacing on Angular logo

@naomiblack this should fix the spacing on the logo in the header

Closed by 73dad5d


Unable to create a profile" err="unable to build DefaultEvictor plugin

Hi!
I'm facing an error in Descheduler pod logs:
"unable to create a profile" err="unable to build DefaultEvictor plugin: unable to initialize \"DefaultEvictor\" plugin: indexer conflict: map[metadata.ownerReferences:{}]" profile="Descheduler"

I'm installing Descheduler via Helm Chart v0.30.1 with Deployment mode.
My policy config looks like this:
deschedulerPolicy:
  profiles:
    - name: "Descheduler"
      pluginConfig:
        - name: "DefaultEvictor"
          args:
            evictLocalStoragePods: false
            evictSystemCriticalPods: false
            ignorePvcPods: false
            evictFailedBarePods: false
            nodeFit: true
            minReplicas: 2
        - name: "RemoveDuplicates"
        - name: "LowNodeUtilization"
          args:
            thresholds:
              cpu : 30
              memory: 30
              pods: 20
            targetThresholds:
              cpu : 80
              memory: 80
              pods: 50
        - name: "HighNodeUtilization"
          args:
            thresholds:
              cpu : 80
              memory: 80
              pods: 50
        - name: "RemovePodsViolatingInterPodAntiAffinity"
        - name: "RemovePodsViolatingNodeTaints"
        - name: "RemovePodsViolatingNodeAffinity"
          args:
            nodeAffinityType:
              - "requiredDuringSchedulingIgnoredDuringExecution"
        - name: "RemovePodsViolatingTopologySpreadConstraint"
          args:
            topologyBalanceNodeFit: true
            constraints:
              - "DoNotSchedule"
              - "ScheduleAnyway"
      plugins:
        balance:
          enabled:
            - "RemoveDuplicates"
            - "LowNodeUtilization"
            - "HighNodeUtilization"
            - "RemovePodsViolatingTopologySpreadConstraint"
        deschedule:
          enabled:
            - "RemovePodsViolatingInterPodAntiAffinity"
            - "RemovePodsViolatingNodeAffinity"
            - "RemovePodsViolatingNodeTaints"

I also see that when pod starts, everything works fine until CronJob finishes... Then I'm starting to get these errors...
Maybe I'm configuring something wrong? 🤔

It seems related to minReplicas. Removing it fixes the problem for me..

unfortunately I am unable to reproduce this.
My assumption is that it's happening for naked pods that do not have an owner ref. would you be able to confirm?

Thank you for reporting this @timetofaceit
/assign


Add a SetInParent equivalent so that is possible to set empty messages as fields of another message

for a message like
message ListRequest {
}

message ServerRequest {
   optional ListRequest list = 1;
}

There is no way (that I know of) to conveniently set the list field to be present. In the original protobuf you could
req = mypackage.ServerRequest()
req.list.SetInParent()

to ensure that the list would be included although optional and empty. But there seems that there is no way to do that with protobuf3. Or ?

As I know empty repeated (or optional) field doesn't have any signs in binary representation, so this call wouldn't affect serialization result

It does show up in the binary encoding:
message StartRequest {
}

message StopRequest {
}

message ControlRequest {
  optional StartRequest start = 1;
  optional StopRequest stop = 2;
}

with Python 2.7 protobuf:
import message_pb2 as pb
req = pb.ControlRequest()
req.start.SetInParent()
req.stop.SetInParent()
with open("out.bin", "wb") as f:
    f.write(req.SerializeToString())

the out.bin shows the fields 1 and 2
hexdump -C out.bin
00000000  0a 00 12 00                                       |....|
00000004

and cat out.bin | protoc --decode_raw show that both fields are present:
1: ""
2: ""

if I comment out the  req.start.SetInParent() line then I get a different binary of course:
00000000  0a 00                                             |..|
00000002

and protoc --decode_raw show that field start with tag 1 is missing:
1: ""


Strike detection doesn't work until Overdrive is activated

Windows 10 x64, 0ea558b x64
dlkfjdlkfdlkfjdlkfjdlkjfdlkjfdlkjfdlkfj
https://github.com/Encore-Developers/Encore-Raylib/assets/43602802/171fc0b4-3cb0-4743-9537-7f37261b2acb

When does this happen? Is it after starting another song? I dont seem to encounter this issue when I play, although I typically close the game afterwards, instead of playing another song

Got a little ahead of myself.
Doing a little more testing, it looks like you need to use overdrive at least once after launching the game, then restart or drop out.
And while activating overdrive does immediately fix it, it just goes away on its own after a seemingly arbitrary amount of time, depending on the song. Though it will still occur on restarts/new songs.


switch.production entity does not toggle correct - it swiches off then it returns to original state[BUG]

Describe the bug
A clear and concise description of what the bug is.
Account type

[x] Installer
[ ] DIY / DHZ
[ ] Home Owner (This integration is not (fully) functional with a home owner account!)

Envoy

[ ] Metered with CTs installed

[ ] Metered without CTs

[x] Standard

FW version: D7.xxx

Amount of micro inverters connected: x

Home Assistant

Version: 2023.x

Additional context
Relevant snippet of Home Assistant error log.

Thanks for your response ,
did the lower caching time resolve the issue?
Regards
Frans
Op wo 17 jan 2024 om 14:03 schreef Vincent Wolsink @.***

Please check #102
https://github.com/vincentwolsink/home_assistant_enphase_envoy_installer/issues/102
—
Reply to this email directly, view it on GitHub
https://github.com/vincentwolsink/home_assistant_enphase_envoy_installer/issues/104#issuecomment-1895769092,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKFSPSQMX7H5DC4OFLIP43YO7D2BAVCNFSM6AAAAABB6PUCWWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJVG43DSMBZGI
.
You are receiving this because you authored the thread.Message ID:
<vincentwolsink/home_assistant_enphase_envoy_installer/issues/104/1895769092
@github.com>

Try the version mentioned in that issue and you tell me 🙂


Use default output to generate new output path for all outputs

default output has been bin/default which doesn't make much sense. This is never good default for eclipse. I changed that to bin and used the value where bin was hard coded.

#23032

@bot-gradle test and merge

I've triggered a build for you.

Your PR is queued. See the queue page for details.

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

Pre-tested commit build failed.

I've triggered a build for you.

@donat do you know when this will be available in Buildship?


Not loading in WSL2 with Ubuntu 20.04

I've installed the deb release 0.10 in an Ubuntu 20.04 instance on Windows 11 with WSL2. In this same instance, I can run GIMP for example and the GUI will work. With Refi App, it does not fully start up. Is there anything more I can do to make this work? I have attached a screenshot. The full-screen window belongs to refi-app.

Hi, I believe you need to run the frontend.
Here is the instruction for setting up development env: https://github.com/thanhlmm/refi-app/blob/main/DEVELOPMENT.md


Messages are repeated under certain conditions

Environment

Microsoft Edge is the latest version.
Version 84.0.522.44 (official internal version) (64 bit)

BUG

When you open the web.kahla.app group, everything is normal, but when you want to view more messages, these messages will appear repeatedly. For example, click the "Click to read more" button.

Please do not submit any issue about Kahla to Nexus!


Bubble Sort.cpp

This is Bubble Sort algorithm program.

You are not allowed to add any algorithms, just provide solutions for the given problems.


Add Simplify.web

Web Frameworks / Simplify.Web - https://github.com/SimplifyNet/Simplify.Web
A lightweight and fast server-side .NET web-framework based on MVC and OWIN patterns for building HTTP based web-applications, RESTful APIs etc.

Sorry, but the community hasn't rallied behind this project to get it included. Thanks for your contribution.


Recommend major version tag instead of patch level version tag

Recommending users install a tag leads to them never updating to later (patch) versions. We should follow GitHub best practices and recommend a major version tag instead.
Please note that a v2 tag will need to be created first.
Documentation from GitHub: https://github.com/actions/toolkit/blob/master/docs/action-versioning.md

Turns out GitHub doesn't recommend using a branch but force pushing a tag, same concept though: https://github.com/actions/toolkit/blob/master/docs/action-versioning.md


makeSingleInstance clarification

Removed callback return value from the sample code for makeSingleInstance because it appears to not be used for anything.

:+1:


Typo

Coverage remained the same at 100.0% when pulling 8b7456d744cad804515e10681e75d6b13b5aace1 on LimeBlast:patch-1 into 0d1bb030bc19f9bfcd50ad046358ef7fd0a513b0 on edelpero:master.

Coverage remained the same at 100.0% when pulling 8b7456d744cad804515e10681e75d6b13b5aace1 on LimeBlast:patch-1 into 0d1bb030bc19f9bfcd50ad046358ef7fd0a513b0 on edelpero:master.

Thanks


MiguelL, 30 Dec. 2022: Added an extra 40 pbunits for the height, as b…

…uttons were being cut off by the lower border of the window. See: https://github.com/OpenSourcePFCLibraries/2021/issues/12
(I only changed one line of code (with a comment "MiguelL". The other changes must be due to a simple regenerate or someone else maybe doing any changes?).

Please don't forget to modify the ZIP file.

Thank you Bruce and happy new year!On 1 Jan 2023 21:55, Bruce Armstrong @.***> wrote:
Merged #1 into main.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Service validations

Description:
Added service validations to various components, did most of the 'easy' ones. One config validation for the shell_command component got included but those are definitely more work because they tend to require some updates to tests as well.
Related issue (if applicable): #1657
Checklist:
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Daaaang, nailed it. Looks all good 🐬


How can I get the train_set.txt and test_set.txt of lasot?

I don't find the files in the link of your provided.

It is provided here (https://drive.google.com/drive/folders/1IYnzEYbSNdj__Tx0nIX-epkJ4PBUtHB4). When you prepare the dataset, you should put them under the (LaSOT) dataset dir.
Thanks.

Thank you very much!

No prob.
Jingchao Wang @.***>于2021年12月28日 周二下午7:26写道：

Thank you very much!
—
Reply to this email directly, view it on GitHub
https://github.com/LitingLin/SwinTrack/issues/19#issuecomment-1002352414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQDPW5CJOQ7W6RLVB6WP6DTUTJPTXANCNFSM5K3XM2NA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>


FFI::NullPointerError: invalid memory read at address=0x00000000000000

When trying to decode the goforward.raw audio, anytime I run decoder.hypothesis.  I get the following error:
FFI::NullPointerError: invalid memory read at address=0x00000000000000
In pointer.rb with get_string method

Thanks.  I'd guess this is due to an interface change in the Pocketsphinx API.  Unfortunately until some up to date releases are made this is going to happen from time to time.
I'll sort it later this week unless anyone beats me to it!
@nshmyrev Are there likely to be any official stable releases in the near future?

Yes, we going to make a stable release soon, few bits left.


Borderlands

This URL is preferred: https://www.epicgames.com/store/purchase?namespace=85189f7cf7a64f86aa6aa91d81d36c08&showNavigation=true&highlightColor=0078f2&offers=a61c6602b5784d36801d0cab73e02337

Thanks


1446 instrument modeling

Accidentally starting a new branch instead of working off the branch where class Instrument is already being modeled.


Container Stat shows javascript error message for stopped container

Bug description
For stopped container, Container Stat page shows Red Prompt at top right corner.
But it is showing some javascript error message
Expected behavior
Red prompt should show "Container XXX is not running" as in v1.19.2
Steps to reproduce the issue:
Steps to reproduce the behavior:

Select a stopped container
Go to 'Containers'
Click on 'Container Stat' icon
See error Red Prompt showing some javascript error

Technical details:

Portainer version: 1.19.2-26-g719299d7 (fix(container-stat) : exclude cache from the Memory Usage chart to avoid misinterpret… (#2371))
Portainer version: 1.19.2-34-g07c1e1bc feat(container-stats): display cache in memory usage chart (#2383)
Docker version (managed by Portainer): 18.09
Platform (windows/linux): Linux
Command used to start Portainer (docker run -p 9000:9000 -v /var/run/docker.sock:/var/run/docker.sock portainer/portainer):
Browser: Firefox 63

@xAt0mZ please have a look at this one when you can, should be reproducible in develop.

In 1.19.2 we are still showing the stats & console icons in containers list for offline containers.
Fixed by PR #2389 .
Still, accessing the stats with direct URL /containers/XXX/stats displays Container XXX is not running in the notification message on portainer/portainer:[ latest | 1.19.2 ].
The JS error may have been introduced by commit 07c1e1b
and is still present on develop when accessing the stats directly with URL.


Feature: add s3 key name prefixes

Allows configuration of a key prefix for S3 uploads. This allows custom folders for uploads within your bucket, fulfilling https://github.com/mattheworiordan/capybara-screenshot/issues/194.
Example of new configuration:
Capybara::Screenshot.s3_configuration = {
  ... # other config here
  key_prefix: "some/folder/"
}

@mattheworiordan Made the requested changes and rebased to master... Let me know if there's any other issues

Excellent, thank you very much for this contribution!

This has now been released in v1.0.16


Some seed values gives predictable/same result of RAND

Setting the Seed with max smallint returns the same non-random result.
select RAND(32767)
consistently returns:
0.324119942249554
but, Select RAND() on it's own run several times works as expected.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f43f1f35-c57f-379a-7117-5ba1ec7ddb7a
Version Independent ID: a16ea319-1189-61a0-3082-dc094ca506ce
Content: RAND (Transact-SQL) - SQL Server
Content Source: docs/t-sql/functions/rand-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@OzBob  Thank you for the feedback.
This is the expected behavior and is stated in the article as such.
Is this a suggestion to change the function?

Closing the issue now, please comment below if there are any questions.


"Conv2DBackpropFilter uses a while_loop. Fix that!"

Describe the current behavior
Currently when I compute Jacobian with the option "experimental_use_pfor=True" on Conv2D layers, the GPU memory will explode. For other layers like FC (with the same amount of parameters), there is no such issue.
Describe the expected behavior
If there is any efficient alternative way of doing this (hopefully YES, as indicated by the warning message in the code "Conv2DBackpropFilter uses a while_loop. Fix that!"), please kindly update.
Code to reproduce the issue
Call tape.jacobian on a Conv2D network of a moderate size with "experimental_use_pfor=True".

@hnyu, Thanks for reporting this issue.
Please provide the complete standalone code to reproduce the reported issue. And also provide the Tensorflow version. Thanks!

@hnyu, Could you post your code snippet to analyze the issue.

Sorry for the delay because of the vacation. Below is a minimal reproducible example:

if __name__ == "__main__":
    gpus = tf.config.experimental.list_physical_devices('GPU')
    for gpu in gpus:
        tf.config.experimental.set_memory_growth(gpu, True)

    img = tf.zeros([64, 10, 10, 1])
    conv = tf.keras.layers.Conv2D(
        padding="valid",
        filters=5,
        kernel_size=4,
        strides=2)
    deconv = tf.keras.layers.Conv2DTranspose(
        padding="valid",
        filters=1,
        kernel_size=4,
        strides=2)

    with tf.GradientTape() as tape:
        pred_img = deconv(conv(img))
        cost = tf.reduce_sum(tf.square(pred_img - img))

    jacob = tape.jacobian(
        cost,
        conv.trainable_variables + deconv.trainable_variables,
        experimental_use_pfor=True,
        unconnected_gradients=tf.UnconnectedGradients.ZERO)

It seems that computing Jacobian using pfor on Conv2DTranspose will throw this message "Conv2DBackpropFilter uses a while_loop. Fix that!", while Conv2D alone will not. I also had a memory explosion problem accompanied with this message (which shouldn't happen because I can compute Jacobian on an MLP with a roughly equal amount of parameters). I suspect the memory issue is related to this warning message, but am not sure.
But as the warning message suggests, it might be better if it can be addressed.

I could replicate the issue with Tenosrflow 2.0 and 2.1-rc0.
Please see the gist here. Thanks!

It's not clear why the while_loop based fallback for handling pfor should be causing the memory increase. The warning here is because the fallback could cause slowdowns compared to a more optimized vectorization. It will be worth looking at some memory profiles to pin down the cause.
Jacobian computation is going to be expensive and memory intensive. It's worth doing some calculations on expected memory growth to see if the observed memory is inline or not.

It looks like cost is a scalar. Why not use tape.gradient here ? tape.jacobian is expected to be more memory inefficient.


Feature/ContextIdentifier for the mutate queries

Add contextIdentifier for the mutate queries in order to be able to add headers in the queries

@VladimirK-ah: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

This PR closed https://github.com/apollographql/apollo-ios/issues/2514.


Multiple Users Able to Register with the Same Username[Feature]:

I have identified a potential security vulnerability within our application's user registration system. It appears that the system allows multiple users to register with the same username, which poses significant risks to data integrity and user account security.
To address this security concern, I recommend the following actions:
Implement server-side validation to enforce uniqueness constraints on usernames during the registration process.
Enhance error handling to provide clear feedback to users when attempting to register with a username that is already in use.
Perform thorough testing to ensure that the fix is effective and does not introduce any regressions.
@digitomize please assign this issue to me under GSSOC.

@digitomize please assign this issue to me under GSSOC.

Hey @divyanshahlawat, can you create 2 users with same name and different emails? I think we already have this restriction. Please let me know the emails - to check if they have same username. (You can connect on discord for this)

hey @pranshugupta54 i have checked this issue and yes you are able to create any number of accounts with same username and different email id. I would like to implement the unique user name module which first checks for the user name in the db and then gives error about username uniqueness. Can you please assign me this issue

@amit429, you created 3 dummy accounts but all of them have different usernames. 🤔

We never allot same username to multiple accounts - if the user used same username while signup - then we just generate a random username. We should instead show an error message saying that username is not available. We can make a separate API for username check - whenever the user writes inside the input box, we'll make a call to the server to check if it's available and then show it directly.
We would required you guyz to first setup the project locally and then we'll assign it.

Separate API for username check
#823
Hey @pranshugupta54 , I think i can make a separate api for backend calling, PLEASE assign me this issue

@KoushikBaagh, we can surely make it. Make a separate issue for it - to add a username check with backend API calling but make sure that we don't have too many calls with it.

@pranshugupta54 Please Check issue number #1007
I have created a new issue.
Please assign me to that !

@pranshugupta54 I was just reading this thread and saw you talking about making minimum API calls regarding username check,
do you think anyone who is implementing this feature should also keep in mind that they are not querying the whole database to find any instance of the name that the user searched for? Or is that not a concern right now?
If that is something worth worrying about I do have some ideas to reduce that with some overhead


Add comment ignores to example

buf's allow_comment_ignores functionality does not work out of the box. It requires the bazel --experimental_proto_descriptor_sets_include_source_info to work. See: https://github.com/bufbuild/rules_buf/issues/64#issuecomment-2125324929
Many of these changes mirror https://github.com/bufbuild/rules_buf/pull/80, and I suspect adding them here as well will save a lot of folks needing to dive a layer down to figure out how to configure buf correctly.

Changes are visible to end-users: yes

Searched for relevant documentation and updated as needed: yes
Breaking change (forces users to change their own code or config): no
Suggested release notes appear below: no

Test plan

Manual testing; please provide instructions so we can reproduce: I'm guessing the behavior of the example directory is tested manually. To test, I suggest modifying the items in example/buf.yaml and the comment ignores in example/src/file.proto and verifying they behave as expected.

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

Yeah, we would like to not maintain that. Sounds good to us!


Support generation property for Cloud Storage

When..

pulling a file down
setting ACLs or other meta-data
deleting a file

Also be able to get all generations of a file

Assigning for triage and fixing-up :)


chore(logos): Adding Cure.fit and Diligent Success Story Logos

Solved with properly escaped and character codes for special characters


Bug found pdf

The DOWNLOAD button in PDF box does not work

Its working fine here. If the browser is set to preview the pdf, it won't download. Thats a browser related issue.


Add header array parameter support

Also

fix content type issue (write header should be done after other headers)
adds docs on readme

write header should be done after other headers

How so? 😸

On an unrelated note, net/textproto is a very interesting package.

I used net/textproto for the type.
About the write header being after setting other headers I found this

// Changing the header map after a call to WriteHeader (or
// Write) has no effect unless the modified headers are
// trailers.
https://golang.org/pkg/net/http/#ResponseWriter

So I guess that is not that you can't but it doesn't work out of the box (content type is not working on the original version).

Updated


Improve KnativeEventing finalization for TLS resources

For optional resources like cert-manager's Certificates and Issuers, we don't
want to fail finalization when such operators are not installed, so during
finalization, we split the resources in

optional resources (TLS resources, etc)
resources (core k8s resources)

Then, we delete resources first and after we delete optional resources
while also ignoring errors returned when such operators are not instaled.

/cherry-pick release-1.11

/cherry-pick release-1.11

Done manually https://github.com/knative/operator/pull/1591, bot is broken https://github.com/kubernetes/test-infra/issues/30937


Games always show 0

as in title and on photo, after playing 2 games, tracker still shows 0

bug is gone


firefox/chrome: support from_visit field

Might be potentially interesting for Promnesia. Although also possible that the utility is very marginal and traversing history by timestamps is good enough.

firefox: moz_historyvisits.from_visit. On mobile the field is present but seems to always be NULL
chrome: visits.from_visit

Perhaps belongs to metadata, but also means that we'd need to keep original visit ID from the sqlite database to match the visit. And even worse, when all visits from different historic exports are merged in a single stream, the ids don't make sense anymore (they are 'internal' to a specific export in general). Although this would be possible to workaround if we somehow remap the ids in browserexport itself

Ah -- yeah, somewhat similar to #16 in complexity, as that requires 'merging' the results back into a database, which would require mangling the from_visit field somehow.
It does become pretty useless across exports -- maybe it would be better to resolve the from_visit id to the URL instead? Previously had thought about making an ordered list of visits after merging; resolving the from_visit id to the URL in individual exports, and iterating backwards through merged visits till you hit that URL, connecting those two? May not always be accurate, but may be useful anyways?
So you'd have something like:
class Metadata(NamedTuple):
    title: Optional[str] = None
    description: Optional[str] = None
    preview_image: Optional[str] = None
    duration: Optional[Second] = None
	from_url: Optional[str] = None

Could also expose the id, but I think promnesia gets the same data if its the URL
Remapping the IDs in browserexport sounds like a pain... would make this less functional and require connecting lots of internal state, which is a similar reason to why I've put #16 on low prio.

Ah, the reason I was thinking of ids is because it might be interesting to trace through visits via from_visit, like a chain of visits, and that wouldn't be possible with just URL. Perhaps would make more sense to just keep a Visit reference instead (although this would mess with caching)
But tbh, it's worth just playing with the databases and checking first -- seems that from_visit is often unset (for whatever reason), and perhaps in practice these 'chains' of visits end up too short, so may be not worth bothering

from_visit is often unset

I think it may depend on the visit_type (in firefox at least). It would only be set if visit_type was maybe 1,5 or 6? in other cases the source isnt another link


Blog post introducing scheduler simulator

Hello. Kube-scheduler-simulator is approaching its first release. (under-development for almost 2 years!)
https://github.com/kubernetes-sigs/kube-scheduler-simulator
And, we'd like to publish the blog post somewhere. Recently, blog posts of Kwok published in

https://www.kubernetes.dev/blog/2023/03/01/introducing-kwok/
https://kubernetes.io/blog/2023/03/01/introducing-kwok/

So, I'm guessing the simulator can proceed the same way.
/assign
/assign @196Ikuchil
/kind feature
/language ja
As we'll write the Japanese translation as well.
We may need some additional work because there is no Japanese blog post so far.
/cc @kubernetes/sig-docs-ja-reviews

Hello, thanks for reaching out!
Feel free to ping me when you need me for Japanese contents.

Thank you for creating this.
I will review that if you create. :smiley:

/remove-lifecycle stale

/triage accepted
Thank you!!
If there is anything I can contribute, I would like to contribute to https://github.com/kubernetes-sigs/kube-scheduler-simulator as well :)


[TIR] Add sugar method Schedule.work_on

This PR introduces Schedule.work_on, which instructs
Schedule.get_block to find the correct PrimFunc to retrieve from
without having to specify func_name in every time if the PrimFunc's
name is not main.

CC: @tqchen


bump 0.9.13

Problem Statement
What is the problem you're trying to solve?
Related Issue
Fixes #...
Proposed Changes
How do you like to solve the issue and why?
Checklist

[ ] I have read the contribution guidelines
[ ] All commits are signed with git commit --signoff
[ ] My changes have reasonable test coverage
[ ] All tests pass with make test
[ ] I ensured my PR is ready for review with make reviewable

/ok-to-test-managed provider=aws sha=72e6806

/ok-to-test-managed provider=gcp sha=72e6806

/ok-to-test-managed provider=azure sha=72e6806

/ok-to-test-managed provider=aws sha=72e6806

/ok-to-test-managed provider=gcp sha=72e6806


Clone error

When cloning a bzr repo, I'm left with an unusual git tree :
$ git bzr clone lp:libzeitgeist libzeitgeist
You have not informed bzr of your Launchpad ID, and you must do this to
write to Launchpad or access private data.  See "bzr help launchpad-login".
Branched 212 revision(s).hing revisions:Finishing stream:Done 3265/32653265
bzr: ERROR: unknown command "fast-export"
ERROR:root:bzr export failed
Why do I need to provide a launchpad id just to clone a repo ?

Sorry, apparently I was missing some bzr tools.


[MRESOURCES-299] Be more accurate on using filtering element

like #80 but for master

Given it was not vetoed for few weeks - may I ask to have it merged, please?


[T3A3][T10-A2] Low Yong Cheng

Revise the Address.java and created individual attributes of the Address. Also Changed the input and output files so that no errors will occur and all the Address keyed in will match the format.

Ready for Review

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.


fix: send ordinal routes, closes #4444

Try out this version of Leather — download extension builds.

This PR fixes some introduced errors with send ordinal routing.
I got the routes fixed but sending the actual ordinal errors as you can see here:
https://github.com/leather-wallet/extension/assets/2938440/93c79fa4-4ac4-4118-9707-cf14ecfc469b

Merging already for release

Curious if this can also be fixed when changing routes with relativeTo prop? Or we have to have the / prefixed path?

I tried that and it wasn't working. I can check a cleaner solution, I just wanted to get a fix out ASAP


deoplete-clang can't deal with files open with Netrw

Problems summary
I recently tried to open a file over the scp:// protocol which is handled by Vim's "Netrw" plugin. When trying to autocomplete a word, the message
[deoplete] Traceback (most recent call last):
[deoplete]   File "/home/me/.vim/bundle/repos/github.com/Shougo/deoplete.nvim/rplugin/python3/deoplete/deoplete.py", line 100, in gather_results
[deoplete]     ctx['candidates'] = source.gather_candidates(ctx)
[deoplete]   File "/home/me/.vim/bundle/repos/github.com/zchee/deoplete-clang/rplugin/python3/deoplete/sources/deoplete_clang.py", line 151, in gather_candidates
[deoplete]     params)
[deoplete]   File "/home/me/.vim/bundle/repos/github.com/zchee/deoplete-clang/rplugin/python3/deoplete/sources/deoplete_clang.py", line 314, in get_completion
[deoplete]     tu = self.get_translation_unit(fname, args, buf)
[deoplete] Traceback (most recent call last):
[deoplete]   File "/home/me/.vim/bundle/repos/github.com/zchee/deoplete-clang/rplugin/python3/deoplete/sources/deoplete_clang.py", line 302, in get_translation_unit
[deoplete]   File "/home/me/.vim/bundle/repos/github.com/Shougo/deoplete.nvim/rplugin/python3/deoplete/deoplete.py", line 100, in gather_results
[deoplete]     tu.save(ast_file)
[deoplete]     ctx['candidates'] = source.gather_candidates(ctx)
[deoplete]   File "/home/me/.vim/bundle/repos/github.com/zchee/deoplete-clang/rplugin/python3/deoplete/clang/clang/cindex.py", line 2638, in save
[deoplete]   File "/home/me/.vim/bundle/repos/github.com/zchee/deoplete-clang/rplugin/python3/deoplete/sources/deoplete_clang.py", line 151, in gather_candidates
[deoplete]     'Error saving TranslationUnit.')
[deoplete]     params)
[deoplete] clang.cindex.TranslationUnitSaveError: Error 1: Error saving TranslationUnit.
[deoplete]   File "/home/me/.vim/bundle/repos/github.com/zchee/deoplete-clang/rplugin/python3/deoplete/sources/deoplete_clang.py", line 314, in get_completion
[deoplete] Could not get completions from: clang.  Use :messages for error details.
[deoplete]     tu = self.get_translation_unit(fname, args, buf)
[deoplete]   File "/home/me/.vim/bundle/repos/github.com/zchee/deoplete-clang/rplugin/python3/deoplete/sources/deoplete_clang.py", line 302, in get_translation_unit
[deoplete]     tu.save(ast_file)
[deoplete]   File "/home/me/.vim/bundle/repos/github.com/zchee/deoplete-clang/rplugin/python3/deoplete/clang/clang/cindex.py", line 2638, in save
[deoplete]     'Error saving TranslationUnit.')
[deoplete] clang.cindex.TranslationUnitSaveError: Error 1: Error saving TranslationUnit.
[deoplete] Could not get completions from: clang.  Use :messages for error details.

is shown.
Expected
No error message and the autocompletion popup is filled as it would be with a local file.
Environment Information

OS: Gentoo GNU/Linux 4.4.6
Neovim version: NVIM v0.2.0-44-g8089292

Provide a minimal init.vim with less than 50 lines and not plugin manager (Required!)
set runtimepath+=/tmp/deoplete.nvim/
set runtimepath+=~/tmp/deoplete-clang/
let g:deoplete#enable_at_startup = 1

The reproduce ways from neovim starting (Required!)

run nvim -u /tmp/vimrc scp://server/tmp/tmp.c
Press a (in order to go into insert mode)
press "int". Once the 't' is pressed, the error messages will be shown.

Upload the log file
Since the log file is very large I uploaded it here: https://gist.github.com/glacambre/bb959fd61acd3385c95e8aa4b5e134d9
There's nothing interesting in my .nvimlog.
I think this is a bug with deoplete-clang because other autocompletion plugins (deoplete-jedi for example) do not have this bug.

The file in the remote server is not supported.
And it is hard to test.

Alright, thanks. Do you know what the difference between deoplete-clang and deoplete-jedi is? Could the way deoplete-jedi works be implemented in deoplete-clang? (I'm willing to try to do it if you think it is)

@glacambre Thanks for issue.
But sorry, Now I don't know why the only jedi haven't problem.
However,

Alright, thanks. Do you know what the difference between deoplete-clang and deoplete-jedi is? Could the way deoplete-jedi works be implemented in deoplete-clang? (I'm willing to try to do it if you think it is)

Note that now deoplete-clang will be planned dynamically change using zchee/clang-server backend.(now development).
So, If you try to solve this problem(very thanks), only for deoplete-go is better.
Because all of the implementation will change, such as not using the libclang-python3 module, msgpack-rpc(or google/grpc) server/client architecture and deoplete-clang will client side only, or etc.


Update Rebellion sidebar elements

Bring into line with other themes.
scss code generated using http://sebastianpontow.de/css2compass

Merged, thanks!


Update SSL/TLS info for static website hosting

Fixes #118

LGTM. Thanks @chris-griffin!

@jlevy would you mind checking this out for final review/merge?

LGTM too. 👍


add locked account check to api authentication

This is part of #142 for the API side. Adds the "lockStatus" attribute to the the User class and database. The attribute is a custom type LockStatus, the value is either LockStatus.Locked or LockStatus.Unlocked.  Also adds a check of the attribute in the authentication.

Codecov Report

Merging #172 into master will decrease coverage by 0.13%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##           master     #172      +/-   ##
==========================================
- Coverage   94.25%   94.12%   -0.14%
==========================================
  Files         113      113
  Lines        4802     4814      +12
  Branches       96       98       +2
==========================================
+ Hits         4526     4531       +5
- Misses        276      283       +7

Impacted Files
Coverage Δ

.../scala/vinyldns/api/route/VinylDNSDirectives.scala
37.5% <0%> (-5.36%)
:arrow_down:

...i/repository/dynamodb/DynamoDBUserRepository.scala
96.39% <100%> (+0.1%)
:arrow_up:

...la/vinyldns/api/route/VinylDNSAuthentication.scala
84.84% <66.66%> (-8.01%)
:arrow_down:

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 4b0feb5...8b207e6. Read the comment docs.

Also wary about the coverage drop for VinylDNSDirectives.scala.

That's because I added more lines to the function. It's tested in the func tests, the new authentication_test.py, but that's not included in CodeCov. We weren't testing that explicitly previously.

Increased VinylDNSDirectives.scala from 37.50% coverage to 46.43% coverage.


Duplicate Identifier

I am using Typescript with typings, and everything for me was working fine until I decided to use async/await instead of Promises in Node.js. When I replaced the 'es5' to 'es6' inside the tsconfig.json file then I started getting 'Duplicate Identifier' errors. I have attached the screenshot of the errors as well

with target ES6, the ES6 definitions are included in your compilation. Promises are part of the ES6 spec, so the es6 library has that. you probably are using es6-promise through typings, you can remove this.


make benchmark_model for tflite build

benchmark_model doesn't build for desktop as reported at https://github.com/tensorflow/tensorflow/issues/20313

This fix worked for me
Ubuntu 16.04
Bazel 0.10.0

Unfortunately merging this will have to wait, see
https://groups.google.com/a/tensorflow.org/forum/#!topic/announce/6ojJ7ykrmZ0


fix:#111 - HTTP verbs be always uppercase

if #111 is considered a bug then this can be merged.

Why the whole String.prototype.toUpperCase thing? It doesn't prevent any errors if method doesn't exist, so I'm not sure I see the point (also you forgot call, so this wouldn't work anyway).
I would suggest something like (o.method || 'GET').toUpperCase(); which prevents the exception you were trying to avoid.

let me fix that and raise a new PR.


New Cool features for NativeUI (for now in FiveM but portable in Master)

Updated everything to Master Branch methods and classes,
Added new cool colors in Colors.cs
Added Panels to NativeUI
Added Heritage Window to NativeUI
Added new Items ( UIMenuSeparatorItem and UIMenuSliderHeritageItem)
Replaced Function.Call with actual Natives as this fasten up execution on FiveM ( https://i.imgur.com/bRh2Eim.png)
Changed some drawing methods to async (fivem seem to like it more since it's threaded this will fasten up execution and mantain milliseconds counter at a low rate)
Added new Methods (like the new UpdateDescription that makes you update items description without having to close and reopen your menu to see the effects)
Changed some Point to PointF and Size to SizeF (needed for Panels and Window precision) this is backward compatible as you can use a Point instead of a PointF but not the contrary

appveyor is configured for Master branch.. it will not work i think for fivem branch right now

Hi, I like what this PR introduced but there is a slight problem: Everything is mixed up.
Can you please create a Pull Request per feature, fix or update? So we can approve and deny individual changes.

Yup all right I'll do it asap

Also: new features should be pushed onto master and then backported to FiveM.

I'm not using master and cannot test on it.. if I can't pr directly on fivem is useless to pr directly  😅

You need to use master sadly, because the SHVDN version is more used than FiveM.

yeah i get it.. but i can't test the changes on master.. i only use FiveM.. i con only PR to FiveM branch.. it can't be ported to master directly? i mean.. even if in the same project.. FiveM branch and Master branch are 2 separate environments and could actually act on their own


Static Bundle Firebase Build Error

Link: https://github.com/commercetools/commercetools-bundles-starter/tree/master/packages/bundles-static
I ran into a couple of issue when following the directions.
Issue: Errors are thrown when compiling the firebase html. It is able to compile if I remove the headers and config option but the app doesn't function properly on the price tab.
command: yarn deploy:firebase
Error:
$ cross-env FIREBASE_TARGET=bundles-manager mc-scripts compile-html --headers=C:/GitHub/Sunday/commercetools-bundles-starter/packages/bundles-static/production-firebase.headers.json --config=C:/GitHub/Sunday/commercetools-bundles-starter/packages/bundles-static/production-firebase.env.json --use-local-assets --transformer C:/GitHub/Sunday/commercetools-bundles-starter/packages/bundles-static/config/transformer-firebase.js
C:\GitHub\Sunday\commercetools-bundles-starter\node_modules\cac\dist
index.js:404
throw new CACError(Unknown option }}${name.length > 1 ? {{-${name} : ${name}}``);
^
CACError: Unknown option --headers
at Command.checkUnknownOptions (C:\GitHub\Sunday\commercetools-bundles-starter\node_modules\cac\dist
index.js:404:17)
at CAC.runMatchedCommand (C:\GitHub\Sunday\commercetools-bundles-starter\node_modules\cac\dist
index.js:602:13)
at CAC.parse (C:\GitHub\Sunday\commercetools-bundles-starter\node_modules\cac\dist
index.js:541:12)
at Object.run (C:\GitHub\Sunday\commercetools-bundles-starter\node_modules@commercetools-frontend\mc-scripts\cli\dist
commercetools-frontend-mc-scripts-cli.cjs.dev.js:326:7)
at Object. (C:\GitHub\Sunday\commercetools-bundles-starter\node_modules@commercetools-frontend\mc-scripts\bin
cli.js:3:51)
at Module._compile (node:internal/modules/cjs/loader:1101:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
at Module.load (node:internal/modules/cjs/loader:981:32)
at Function.Module._load (node:internal/modules/cjs/loader:822:12)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Issue: This may be related to the issue above. The links for pricing and display for pricing are showing an error in the deployed application. Also the mcURL is mentioned in the documentation but this is not a configurable option.
Clicking add price has unknown for the URL.

See also, the comments from @emmenko here: https://github.com/commercetools/merchant-center-application-kit/issues/2803#issuecomment-1246840413

@praveenkumarct As a first step should we remove the headers and config options from any commands in the repo?

@praveenkumarct - thank you for taking this issue.  I'm hoping that you were able to review and determine the changes needed.  Let me know if there are any updates or if you have an estimate when you might have a PR that we are able to test.

@amilges
All the config issues are fixed for both static and dynamic Bundles starter app.
Please check the latest release: https://github.com/commercetools/commercetools-bundles-starter/releases/tag/v1.1.0
This issue shouldn't exist anymore. But we have removed the legacy deployment config from the repository. Please find an alternative for deployment examples here.


[Backport 1.2] Allow null value for params in method mappings

Allow user to input null value for parameters field for KNNMethodContext
and MethodComponentContext. Adds tests to reproduce Mapping Parsing Error when the parameters take
the value null as well as BWC tests.
Backport: #354
Signed-off-by: John Mazanec<EMAIL_ADDRESS>(cherry picked from commit b08127c2c4dcd1844072302e838f44897c8f18f5)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

BWC tests were not introduced until after 1.2 so they could not be backported back.

Codecov Report

Merging #357 (3917fd6) into 1.2 (af3c80a) will increase coverage by 0.11%.
The diff coverage is 78.57%.

@@             Coverage Diff              @@
##                1.2     #357      +/-   ##
============================================
+ Coverage     83.19%   83.31%   +0.11%
- Complexity      864      875      +11
============================================
  Files           123      123
  Lines          3779     3799      +20
  Branches        358      363       +5
============================================
+ Hits           3144     3165      +21
+ Misses          475      474       -1
  Partials        160      160

Impacted Files
Coverage Δ

...g/opensearch/knn/index/MethodComponentContext.java
91.76% <75.00%> (+1.09%)
:arrow_up:

...ava/org/opensearch/knn/index/KNNMethodContext.java
93.25% <100.00%> (+0.23%)
:arrow_up:

...n/java/org/opensearch/knn/index/KNNIndexShard.java
93.75% <0.00%> (+0.72%)
:arrow_up:

...a/org/opensearch/knn/index/codec/KNNCodecUtil.java
85.71% <0.00%> (+1.50%)
:arrow_up:

...c/main/java/org/opensearch/knn/index/KNNQuery.java
77.77% <0.00%> (+5.55%)
:arrow_up:

Continue to review full report at Codecov.

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


cwltoil issues on AWS.

I opened a t2.xlarge AWS instance and attempted to run hello_world.cwl and md5sum.cwl as per Synapse instructions here:
https://www.synapse.org/#!Synapse:syn9630940
https://www.synapse.org/#!Synapse:syn10167920
These workflows seem to work with cwltool just fine, but not cwltoil.  Each gave different errors (hello_world omitted getting a directory when initially fetching the files; md5sum looks like it allocated too few resources), which are attached.
hello_world.log.txt
md5sum.log.txt

Thank you @DailyDreaming for making a first pass at this. Why are you running them on AWS? These small examples can run locally.
MD5SUM EXAMPLE
There is a problem in dockstore-tool-md5sum.cwl. The outdirMin value is specified in MB (mebibytes actually). It is asking for 512000, so 1/2 a TB. I changed this value to 5 and it ran fine.
See the spec for more details - http://www.commonwl.org/draft-3/CommandLineTool.html#ResourceRequirement.
HELLO_WORLD EXAMPLE
Your error text file is for the results validation. Does that mean that cwltoil ran without a problem?

Toil doesn't have a --non-strict option. I removed the creator field from a few cwl files and things ran fine.

I was told to run Toil on AWS.  We have a checklist and that was one of my tasks.  Running toil locally wasn't on there.
Good catches on the cwl files, it would be nice though if cwltoil could handle these issues the way that cwltool does.  These files are the templates provided for the challenge and since this is meant to be an easy run.  I'll update and run again on AWS to see if it works.

@DailyDreaming are these running now?

Yes, this can be closed.


Deploy Problem

Hi, thanks your great work!
Based on my understanding, in the deployment of DSVT, you converted the Transformer network part of DSVT into DSVT_TrtEngine, while DSVT_Input_Layer still uses original PyTorch code. I would like to ask if DSVT_Input_Layer can also be converted into ONNX-TRT? Because there are operators such as torch.sort and torch.unique in it that are not supported by TRT, I plan to convert DSVT_Input_Layer into a single CUDA kernel when deploying the entire model. Do you have any suggestions for a faster and more convenient approach?
Wish your reply.

Good question, this is the future direction of optimization. The input layer cannot be deployed, because it is not a large matrix calculation of the network, but the consumption time is fixed. This step takes about 8ms and will not grow as the network deepens. If you want to speed up, you can cache the position embedding and write some cuda OP.


Add AutoRegistering protocol for automatic cell registration

We implemented auto-Registration of tableViewCell types. This was done such that it is an optional, non-breaking change. The features is disabled by default, although that could be changed without breaking anything.
When calling dequeueReusableCell(...), one can now specify autoRegister: true, and it will take care of ensuring that the class/nib is registered. Note that in order to get around generic constraints this required duplicating the dequeueReusableCell function for each of Reusable and NibReusable.
Hopefully this is useful to someone!

The generic constraints issue relates to the dequeueReusableCell<T: UITableViewCell> function casting T as 'Reusable', meaning that the subsequent call to register(cellType:) tries to register the class rather than the nib (in cases where it is a NibReusable).
Swift doesn't seem to like conditionally casting to a protocol, so this was the simple workaround. There might be a non-duplicating way around this that I haven't thought of though!

FYI the CI failure is due to the following linter errors:
…
Linting 'MyCustomWidget.swift' (25/25)
/Users/distiller/project/Example/ReusableDemo iOS/TableViewController.swift:19: warning: Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)
/Users/distiller/project/Example/ReusableDemo iOS/TableViewController.swift:66: warning: Line Length Violation: Line should be 120 characters or less: currently 146 characters (line_length)

@apptekstudios I took the liberty to push a commit on your fork to show my idea of mutualising the code. (Also added missing documentation on the new method)
There are still things missing in that PR imho:

You implemented this for UITableView.dequeueReusableCell, but why not for all the other stuff? Like dequeueHeaderFooter, and like for the similar methods in UICollectionViews as well?
The more I think about this, the more I wonder if it wouldn't be nicer — both for API and for usage — to use additional protocol conformance instead of adding the autoRegister: Bool parameter (that you'll have to not forget to add at call site on every place you dequeue your cells otherwise)?

So maybe it would be more powerful, flexible, and less error-prone for the end user, to declare a protocol AutoRegistering {} for example in Reusable, and make different implementations for where T: NibReusable, T: AutoRegistering vs just where T: NibReusable (the compiler always choose the most specific implementation anyway)?
That way users of the Reusable library would just have to just add AutoRegistering to their custom UITableViewCell subclasses (the same way they just add Reusable or NibReusable or NibLoadable already), making auto-registration still an opt-in feature, but without having to think about explicitly adding autoRegister: true everywhere they want to dequeue such cells. This would also have the advantage for the lib user to not have to remember which cells are auto-registered or not so to which dequeue calls they have to add autoRegister: true vs not add it.

Thanks for implementing the protocol - I agree that it's definitely a much clearer solution.
I've implemented the AutoRegistering for Headers/Footers and added an example 👍. Hopefully I'll find time to write a quick documentation update tomorrow 🙂

Ah, bummer.
(I now remember why I'm personally not a fan of doing the auto registering dance in my own projects but prefer explicit registration 😅)
Ok maybe we could indeed register every time before dequeuing (but only for collection views then), but we'd need to clearly document that and the performance trade-off in the documentation to make it crystal clear to users of Reusable then.
Another way would be to keep our own list of registered cell classes per collection view (maybe using ObjC's associated objects?) so we can test before registration. But I wonder if that's not gonna be a similar performance impact anyway…

I think associated objects is a good way to do it. I went ahead and implemented it 👍
Shouldn't have any performance impact, because all that is done is creating a single object per tableView which then tracks whether the cellType has been registered by us yet.

Disclaimer: Did not checked the code so you may be already doing this.
In a project I was doing mostly trhe same thing. For testing if the cell is registered I was trying to dequeue. If it throw -> catch, register then dequeue again

@Alex293 but doesn't it throw an ObjC NSException in those cases? Which can't be caught from Swift (*) and are different from Swift's thrown Errors?

(*) well, unless if you create an ObjC helper method which catches the NSException from ObjC then converts it to an NSError returned by reference which is interpreted as a thrown Error in Swift… some ObjC pods provide such a helper but I wouldn't want Reusable to depend on another pod just for that — nor to implement that in Reusable itself, requiring to add ObjC to my pod and a bridging header and all that just for that…

@apptekstudios What do you think about the alternate solution of catching the ObjC exception (instead of going the whole way of using associated objects)?
Something like:
// ObjCException.h
#import <Foundation/Foundation.h>
@interface ObjC : NSObject
+ (BOOL)catch:(void(^)(void))tryBlock error:(__autoreleasing NSError **)error;
@end

// ObjCException.m
#import "ObjCException.h"
@implementation ObjCException
+ (BOOL)catch:(void(^)(void))tryBlock error:(__autoreleasing NSError **)error {
    @try {
        tryBlock();
        return YES;
    }
    @catch (NSException *exception) {
        *error = [[NSError alloc] initWithDomain:exception.name code:0 userInfo:exception.userInfo];
        return NO;
    }
}
@end

Then using it like this from Swift:
let cell = try? ObjCException.catch {
  return collectionView.dequeue(…)
}

I had a go at catching the objective C exceptions, however couldn't get anything to compile properly with the objective C included in the pod.
https://github.com/apptekstudios/Reusable/tree/objcExceptions

All right then. Could you just address the nitpickings I made in my latest review? I'll look into merging that next.

Sorry for taking so long to get back to this - I've fixed those minor issues, pulled all changes from your master repo, and updated the changelog.


Templating XML or XPath from request.body

Hello! Your WireMock is very good! I use it standalone.
Now if i have XML or XPath in request body, i can not get any parts of request.body by handlebars.
Is it possible to do this now?
If not, will this be improved?
And can i use some plugins to do it now?
Thank you!

You should be able to get the whole request body, but you can't at present extract parts of it via xpath. Is that what you had in mind?

I mean, if i have xml in request body like ...Hello..., i can not get "Hello" string by {{request.body}} in response. Will this be developed in the future?

I'm not exactly sure what you mean, but I am intending to develop handlebars helpers for extracting parts of request XML documents via XPath if that helps.

It would help a lot

We had similar needs so I extended wiremock with helpers for xml via xpath and another one especially for soap and one for json via jsonPath. I am currently working in a fork of wiremock but the helpers are not there yet.
The reason is that I created a gui for wiremock which I updated to angular in this new fork. I want to add the helpers in a new branch an then create a pull request. I hope this helps.


getting Traccar up en running

i downloaded and installed (https://github.com/tananaev/traccar/releases/download/v3.7/traccar-windows-64-3.7.zip) and did the same for Java >>(http://javadl.oracle.com/webapps/download/AutoDL?BundleId=216434).
I installed traccar client on android phone and i think i configured it correct.
but it keeps saying "send failed".
if i test with port check tool it says Port open  and tracker-server log doesnt show anything after "Starting server..."
2016-10-23 21:08:08  INFO: Operating system name: Windows 10 version: 10.0 architecture: amd64
2016-10-23 21:08:08  INFO: Java runtime name: Java HotSpot(TM) 64-Bit Server VM vendor: Oracle Corporation version: 25.111-b14
2016-10-23 21:08:08  INFO: Memory limit heap: 3628mb non-heap: 0mb
2016-10-23 21:08:08  INFO: Character encoding: UTF-8 charset: UTF-8
2016-10-23 21:08:08  INFO: Version: 3.7-SNAPSHOT
2016-10-23 21:08:09  INFO: Starting server...

everything worked ok at demo server
Is there anything  more I need to install to get this working?
server address is my public IP? or the one from my PC?
standard port is 8082?

Port for Traccar Client is 5055. IP address is the address of your server.

I got it working :) i made a mistake  with IP address


1.16.4 version in curseforge?

Version Information
Minecraft version:
Forge version:
Other mods (if applicable):
Steps to reproduce:

Expected Behavior:
Actual Behavior:
Link to crash log (if applicable - use a site such as hastebin:

These versions are still in development, not ready for general use yet. Update for newer MC version will be released when it's ready.


Node.js backward compatibility

The application I am currently working on is deployed on Render server where they got only Node.js 14 on their free servers (at least for me). I am also seeking support on their side for any chance of upgrading Node but in the meantime I wonder is there any chance for Qdrant js client to have backward compatibility support for Node version previous than 18?
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module.
Expected version ">=18.0.0". Got "14.17.0"

For now I'll just tell Node to --ignore-engines.

Well this just in: actually I can't seem to use qdrant client under Node 14.17.0 because it cannot get the headers
{
    "error": "Headers is not defined",
}

it might be related to the way the client is getting the headers for authentication, like described here. I might look into the source code again to be sure when I have the time, but for now, well, back to the good old trusty handle-request-by-oneself 😂

My bad. The Render's support just contacted me that I could change the node version on the server as documented here, and this has been there from the start! Stupid me 😵
But I'll leave this thread for a while for the devs to close it if they deem this problem trivial at this moment.

Since Node.js 17, the fetch API was available using a flag, e.g.: node --experimental-fetch. Though, this library works with Node 16 too, not sure if with 14 too or how far back, haven't tested. If you must use a previous version, polyfill AbortController, AbortSignal Headers, Request, Response and global.fetch. It's something you can achieve with undici.js or node-fetch. Under the hood, Node 18 with native fetch uses undici.js (for now HTTP/1.1 until they bring us HTTP/2).


0.31.0

These is the list of changes going into 0.31.0, due to be released on Thursday. Thanks!
cc/ @kennykerr feel free to move things around in case they've not been categorized right.

Added

Derive PartialEq and Eq for scoped enums to support constant patterns by @kennykerr in https://github.com/microsoft/windows-rs/pull/1438
Make handle fields const by @kennykerr in https://github.com/microsoft/windows-rs/pull/1468
Make applicable fields const by @kennykerr in https://github.com/microsoft/windows-rs/pull/1470
Support functions that don't return by @kennykerr in https://github.com/microsoft/windows-rs/pull/1485

Changed

Trait-based implement macro by @kennykerr in https://github.com/microsoft/windows-rs/pull/1450
Test samples by @kennykerr in https://github.com/microsoft/windows-rs/pull/1456
Simpler error interop by @kennykerr in https://github.com/microsoft/windows-rs/pull/1462
Improve core/xaml app sample ergonomics by @riverar in https://github.com/microsoft/windows-rs/pull/1463
Improve enum code gen by @kennykerr in https://github.com/microsoft/windows-rs/pull/1457
Update Win32 metadata to 17.0.9 by @kennykerr in https://github.com/microsoft/windows-rs/pull/1482
Split clippy into separate build runner by @kennykerr in https://github.com/microsoft/windows-rs/pull/1487
Update target lib files by @kennykerr, @riverar in https://github.com/microsoft/windows-rs/pull/1488

Fixed

Use correct lib for internal bindings by @kennykerr in https://github.com/microsoft/windows-rs/pull/1432
Require delegates to be Send by @kennykerr in https://github.com/microsoft/windows-rs/pull/1458
Adds test to confirm fix by @kennykerr in https://github.com/microsoft/windows-rs/pull/1461
Remove dead code by @kennykerr in https://github.com/microsoft/windows-rs/pull/1467
Avoid generating invalid Debug traits by @kennykerr in https://github.com/microsoft/windows-rs/pull/1473
Reliably handle Unicode string literals by @kennykerr in https://github.com/microsoft/windows-rs/pull/1480

Other

Document how to build docs + release order by @yoshuawuyts in https://github.com/microsoft/windows-rs/pull/1430
Check windows-sys with 1.46 as MSRV by @kennykerr in https://github.com/microsoft/windows-rs/pull/1460
Improve crate naming and build validation by @kennykerr in https://github.com/microsoft/windows-rs/pull/1469
Improve tests on CI by @kennykerr in https://github.com/microsoft/windows-rs/pull/1452, https://github.com/microsoft/windows-rs/pull/1453, https://github.com/microsoft/windows-rs/pull/1455

Full Changelog: https://github.com/microsoft/windows-rs/compare/0.30.0...0.31.0

Thanks I'll give it a once over on Wed afternoon.

@Josh015 How do you want to handle additions? Few more commits in today. Happy to edit your post or let you handle it.

@riverar feel free to update it.
Also, adding this comment from Wesley (on Teams):

Just as a suggestion, it might be helpful to indicate somehow which changes are for windows and which are for windows-sys. I see one item mentions windows-sys explicitly but I'm not sure if the other changes impact that crate. Perhaps two summaries of added/changed/removed/other for each of the crates would be clearer?

Changes to windows-sys are intentionally far less substantial and it would be great to make that clear.

Here are the changes specifically affecting the windows-sys crate:

Make handle fields const by @kennykerr in https://github.com/microsoft/windows-rs/pull/1468
Make applicable fields const by @kennykerr in https://github.com/microsoft/windows-rs/pull/1470
Support functions that don't return by @kennykerr in https://github.com/microsoft/windows-rs/pull/1485
Improve enum code gen by @kennykerr in https://github.com/microsoft/windows-rs/pull/1457
Update Win32 metadata to 17.0.9 by @kennykerr in https://github.com/microsoft/windows-rs/pull/1482
Update target lib files by @kennykerr, @riverar in https://github.com/microsoft/windows-rs/pull/1488

Reminder to remove the entries I just added if the snapped commit was already decided.

We'll include this week's fixes as they include critical customer bugs.

https://github.com/microsoft/windows-rs/commit/3c3c5b92d90e3995b6b53b44b4d77d2edafd6778 includes all the fixes.


⚠ Senior School Website has degraded performance

In 74f4b75, Senior School Website (https://www.cranleigh.org) experienced degraded performance:

HTTP code: 200
Response time: 15069 ms

Resolved: Senior School Website performance has improved in ad36421.


Buttons not clear

Hey, basically, I built it and have no idea what I need to do to activate the modules, could you possibly add some documentation?

to activate module you need to press the key bind next to the name on the hud. ill try and make some docs soon

Yeah, I realized later that I didn't use the right command to compile and
when I found the right one it wouldn't work

Message ID: @.***>

Yeah there may be a few issues as im adding and removing different stuff all the time testing featues


Week 8: Refactoring And Linting

This is a request to pull changes reflecting pylint suggestions into previous work in this repository.
The work item ticket is "Week 8: Refactoring and linting" in section Week 8 on page Modules for course Deep Learning offered in Summer 2024 at the University of Virginia.
I don't remember any changes that are bug fixes and believe I can't add descriptions to individual commits.

Awesome job on the branch chaining.
Some generic feedback on the PR description:
PR descriptions will usually include a short description of a feature added, bug fix, changes and link to some work item ticket.
For example:                                                                                [link to ticket](http://acme.db/issue_1234
Added workflow automation

run linter                                                                                * run tests
use two versions of python
TESTING: [link to passing tests or a workflow run]

Kudos on your commit messages, very clear to see what you did or intended to do from the commit messages!

AH, did you add a git badge to your README?

@EfrainOlivaresUVA , I addressed your comment from 2024-08-03 03:49 EDT.

@EfrainOlivaresUVA , there is a badge in the README for this branch.


hawktrace support

it is pending a 0.5 release of rust-hawktracer, the code is complete.

It depends on https://github.com/AlexEne/rust_hawktracer/issues/19

Coverage decreased (-0.003%) to 75.611% when pulling f0f2e2d7456f457752228bb716442f7edd033c57 on rust-av:hawktrace into 6722d755244c2e3cd204f92e42a1a08845425ed3 on xiph:master.

The previous refactoring broke it, now it works as intended on macOS and Linux.

I've released a new (0.5.0) rust_hawktracer version.


Refactor mission transfer

This moves mission/geofence/rally point transfer into core which means the work that has to be done in the plugins can be simplified

Try again tomorrow, now libressl download is down:
 -- Using src='https://ftp.openbsd.org/pub/OpenBSD/LibreSSL/libressl-3.0.2.tar.gz'
CMake Error at libressl-stamp/download-libressl.cmake:159 (message):

@JonasVautherin ready for review :smile:

Thanks for merging. I didn't have the courage :smile:.

FYI @hamishwillee we're on the way with the plan we decided on a while ago :tada:

Oh perfect. Thanks for that work! In March then I will be able to work in the needed plugins

Well done!


Update index.md

Just a handful of missing punctuation marks.

Sounds good!
Thanks,
Obi Oberoi – (M.Sc.)
Cell:<PHONE_NUMBER>

On Jul 14, 2021, at 12:30 PM, Smit Patel @.***> wrote:
I will run this page through doc tool, we can just go with whatever it suggest for this commas unless we think that they are ugly and we don't want them.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I ran through the tool and tool suggest that comma shouldn't be used (along with the fact that this should not be used without noun too). Closing this PR as the change doesn't align with doc writing guidelines.

Thanks for the input!


Zoom Auth + Meeting Creation POC

Adds a POC of authenticating and refreshing the token with Zoom.

@sparkplug0025 I tested this with our app client_id and secret. Works great!

I noticed the meeting is set to record automatically, locally. Nice!
Do you know if there's any option for us to disable the waiting room so that the host doesn't have to manually let people in?

@sparkplug0025 I tested this with our app client_id and secret. Works great!

I noticed the meeting is set to record automatically, locally. Nice!
Do you know if there's any option for us to disable the waiting room so that the host doesn't have to manually let people in?

These were the options set on that meeting:

I removed the join before host and changed the waiting room to false!

yeah, just to recap, we want to make sure that the settings allow for anyone to join the room at any time. So I believe "Waiting Room" should be false and "Allow participants to join at any time" should be true (I'm referring to the UI settings above...of course, the underlying configuration flags might be worded differently.)


not show when using addInitialFiles()

HI,
I'm using  the low components to render  a   and a   , files is loaded from server by  addInitialFiles(), thumbnails render is ok, but  is no show.
My code :
<template>

    <dropzone class="dropzone column is-6 has-text-centered"
              :uploader="uploader">
      <file-input multiple
                  accept='image/*'
                  :uploader="uploader">

Drop files here or click to upload

      </file-input>
    </Dropzone>

        <thumbnail :id="file.id"
                   :fromServer="Boolean(file.uuid)"
                   :uploader="uploader" />

        <cancel-button class="button"
                       v-if="!file.uuid"
                       :id="file.id"
                       :uploader="uploader">

        </cancel-button>
        <delete-button :id="file.id"
                       :uploader="uploader">

        </delete-button>

        <progress-bar class="progress-bar"
                      :id="file.id"
                      :uploader="uploader"></progress-bar>

</template>


how to user

please  upload the demo

or an example on how to start

just noticed documentation is in the wiki: https://github.com/aleclarson/emitter-kit/wiki

the docs are a bit poor. more examples would be welcome

in the terminal:
My-MacBook-Pro:Emitter101 myname$ pod install
Updating local specs repositories
...
fatal: repository 'github.com/aleclarson/emitter-kit.git' does not exist
???

Sorry for the lack of communication, guys! I'm extremely busy these days. Plus, I'm not actively using this library. Hopefully you can figure things out on your own. Good luck!


Error: 0x80070040 on reinstalation

IMPORTANT
Please read README and Known issues before creating the issue.
Please fill out the below information:
Describe the issue
while running Arch.exe again to install again it posts this error
To Reproduce
Install Arch linux via WSL then delete Arch Linux WSL files (i did his by accident like a tard.) then attempt to reinstall ArchWSL by extracting the files and running like before.

Have you completely deleted your registration from the system?

Please unregister from the system with the following command:

wslconfig /u Arch

re-install

yup that worked, however the app install dosen't work.

the APPX installer gives the following error Either a new certificate must be installed for this app package or you need a new app package with trusted certificates. Your system administrator or the app developer can help. A certificate chain was processed but terminated in a root certificate that isn't trusted (0x800B0109)

Yup this worked perfectly, thankyou!


native strides are not performed with MC descriptors

Reported by hankin on 14 Oct 2003 17:50 UTC
Currently (??) when strides are applied to an MC data set, Ferret reads the data
at full resolution and then performs strides in memory.  This will not be
acceptable if MC descriptors are used inside of an FDS-accessed data set.
Fixing the problem should be a matter of

at the point where the special trap allows netCDF data sets to perform
strides (see regrid_lims.F), also allow MC data sets
In the TMAP library routines that handle MC data sets, pass the XYZ striding
information along to the netCDF IO layer.  If strides are on T axis, then handle
them in the MC code

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/651

Comment by<EMAIL_ADDRESS>on 21 Oct 2003 20:15 UTC
Moving from the<EMAIL_ADDRESS>user to the<EMAIL_ADDRESS>user.

Comment by<EMAIL_ADDRESS>on 6 Nov 2003 17:56 UTC
reassign to Ansley

Comment by<EMAIL_ADDRESS>on 16 May 2005 16:44 UTC
Removing Joe Mclean from the CC list after Joe's departure.


Add testing framework and tests

React Virtalized Pivot should have a front end testing framework.

Closed in #13


Refactor linter and fix bugs/ missing functionality

Refactored linter/parsetemplates/templates.go so it becomes almost possible to understand it.
Added test cases & tested the logic on the istio-csr chart.

/approve


npm package

@kimdhamilton
I'd like to directly invoke (i.e, not through REST) the operations in sign-and-verify (because I'm trying to build that pseudo-wallet I mentioned - i.e, a web page that pretends to be a wallet for testing purposes).
Specifically, I want the methods that are returned by createIssuer(config: Config) in issuer.ts:
sign
signPresentation
createAndSignPresentation
verify
verifyPresentation
requestDemoCredential
I think the new NPM package (https://www.npmjs.com/package/@digitalcredentials/sign-and-verify) is probably intended to do this, and maybe just needs a few more changes (or is the NPM package meant for something else)?
I'm thinking that the issuer.ts (and related files) could be moved to its own repository, leaving the RESTful part (the fastify stuff in index.ts) in sign-and-verify. (or vice versa, move the fastify stuff into a new repository)  This new 'dcc-issuer' (or whatever it should be called) repo becomes the NPM module, and has an index.js (that is set as 'main' in package.json) that exports one method: createIssuer(config: Config)
Then the createIssuer method could be directly used in another project (say a Wallet - either app or web based) by installing the NPM module (with npm/yarn), importing the method, and using it (passing in a config):
import createIssuer from "issuer";
const { sign, verify, createAndSignPresentation, signPresentation, verifyPresentation, requestDemoCredential } = createIssuer(getConfig());
let didAuthResponse = createAndSignPresentation(null, presentationId, holder, options)
// then send off didAuthResponse to the issuer endpoint.
The fastify code would also now install this new npm module and use it just like it had used it before, only now it is coming from the npm repository.
If this all seems right I am happy to create the new repository tomorrow (I'd leave the sign-and-verify as is so as not to break anything - basically I'd just create a new repo that can publish an npm package that exposes the createIssuer method)

@jchartrand
+1, seems like a decent idea (to separate out the fastify http API app and the core library). My vote is to name the other one - sign-and-verify-core.

Thanks @jchartrand -- that was definitely originally the intent, but I probably lost track of that.
The only reason I'm reluctant to separate out repos is that it can be a pain to keep everything up to date when APIs are new, so I wanted to see if other options would work:

@stuartf did something similar for credgen, exposing a lib while also exposing the CLI.
Orie uses lerna for multi-module management. I haven't figured that out yet, but that at least lets you keep everything in the same repo.

@kimdhamilton @dmitrizagidulin @stuartf
I totally understand not wanting a proliferation of npm packages.  I've out-clevered myself on that before.
In this case, though, the part that just doesn't seem to belong in an npm package is the fastify stuff.  I'm not sure how one would even use that part after installing the sign-and-verify npm package?
One option for making simultaneous changes during development to the web app (fastify) and the sign-and-verify-core -- without having to constantly re-publish to npm, and npm install for every change -- is to use npm link to pull in a local dev copy of the sign-and-verify-core, like described here:  https://www.deadcoderising.com/how-to-smoothly-develop-node-modules-locally-using-npm-link/
I hadn't heard of Lerna - it looks very useful, and I'm happy to help incorporate it into sign-and-verify (or elsewhere), although for the moment I think the single sign-and-verify-core might be the only npm package we have (again, I'm just not sure the fastify stuff belongs in an npm package, but happy to be proven wrong, and learn something new).

I see what you're saying with fastify. It would be nice if we could use some multi-module/monorepo solution, at least in the early days, to minimize repo proliferation. I'd definitely be interested in that, if you could help. If it starts to look too complicated, let me know and we can do multiple repos instead

Tagging @blarghmatey -- xPro's service deployment depends on the structure of this repo, so I want to make sure we don't break him

My deployment only depends on the output of the Dockerfile build, so as long as we can keep that working then I have no horse in this race.

I won't change this repo.  I'll just make a new one (called sign-and-verify-base as suggested by @dmitrizagidulin) that includes everything except fastify, and publish that new repo to npm.  Once I'm done maybe then we can look at importing the new npm package into fastify?

Strong no. I'm not sure that approach is sustainable because we'd have to copy code changes to both repos. Can we at least look into multimodule solutions?

See for example this, which keeps the code in the same repo, but allows independent publishing of modules. I have to put my foot down to code copying in multiple repos as the most abhorrent possible option :)

I didn't at all mean that we'd keep two copies of the same code (that would indeed be silly, as you point out).  I just meant that I could get everything working with this extracted core module (in a new repo), and only then consider how it could be incorporated into fastify (which would likely just be as a standard import).

My ill-expressed point was just that I won't go breaking anything in this repo while I'm extracting the npm core module.

thanks for clarifying :) After a long morning of CCG mailing list shenanigans my brain is fried. :)
When you say import "into fastify" to you mean import into this repo, which uses fastify? i.e. structurally:
This repo (sign-and-verify) would do the following, in its end state?

exposes service endpoint
calls operations on sign-and-verify-core

Does it have to be a new repo as opposed to multi-module, which could be easier to manage (unless there's some github automation that would help update this repo when core changes? maybe there are other options? ).

When you say import "into fastify" to you mean import into this repo, which uses fastify? i.e. structurally:
This repo (sign-and-verify) would do the following, in its end state?

exposes service endpoint
calls operations on sign-and-verify-core

Yes, exactly.

Does it have to be a new repo as opposed to multi-module, which could be easier to manage (unless there's some github automation that would help update this repo when core changes? maybe there are other options? ).

I thought it would be confusing to have the api server stuff (fastify) or Docker for that matter - neither of which are 'packagable' in any useful 'importable' sense (I don't think?) - in the same repo as code that is meant to be published as an importable NPM package (or packages) - even if using lerna - but I see that at least one person describes doing just that:
https://github.com/erzhtor/javascript-monorepo-with-lerna
so happy to give it a try if that's best.

Ok, I buy that. The server part is package-able in the docker sense, but you're right, it's not an npm package.
Figuring that out and maintaining it may actually be more awkward than just splitting the repos (I haven't looked into it), so feel free to create a new repo for this. You should have permission to do so (let me know if not)

@blarghmatey @kimdhamilton @dmitrizagidulin @rhysyngsun @stuartf
Very sorry all, I accidentally committed a change that was meant for a new copy of the repo (I hadn't changed the remote origin).  I've reverted the commit.

@jchartrand fixed this by adding sign-and-verify-core and updating this

@jchartrand fixed this by adding sign-and-verify-core and updating this


Dev: Web - Console error on click of Connect online with Plaid button

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Create workspace if not created
Go to settings > Workspaces >Connect bank account
Click on Connect online with Plaid button

Expected Result:
Should not show any console error
Actual Result:
Multiple console error appears
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: Dev 1.3.37-6
Reproducible in staging?: n/a
Reproducible in production?: n/a
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/bdce3177-82a1-4c16-861c-e4bcbf4db333
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688724565368739
View all open jobs on GitHub

I can't reproduce this ... I think it's Dev only? Will ask C+ to help

Oh hey @allroundexperts - you already have a proposal for this 😅 - I can't reproduce it but I'm assuming you have? Also what's the protocol for accepting your own proposal if you get assigned to an issue as C+ after creating a proposal? lol not sure I've seen that before

I can reproduce this error and I can see a lot of errors in console.
https://github.com/Expensify/App/assets/53470896/fdd027e7-e1b1-44cf-99ba-30caef4ebf71
All these are saying as required - translate, numberFormat, datetimeToRelative etc. all are stated as required but still not passed. So, I think as stated by @allroundexperts with we remove withLocalizePropTypes and use use useLocalizethen it should solve all the problems as when I was migrating BankAccountManualStep.js file I saw all these same errors but after removing withLocalize HOC and replacing it with useLocalize hooks along with removing the extra prop types as well using ..._.omit(StepPropTypes, _.keys(withLocalizePropTypes)) which are not required.

wait i'm confused here so @allroundexperts are you already working on the fix PR?

HAHAHAHA, My proposal looks good to me. That's lit 🔥🔥🔥

haha, then I read this

regression from https://github.com/Expensify/App/pull/21778. Since the mentioned PR is within the 7 days regression period, the related C+ and contributor will fix this.

so is this already handled by the PR author or we have to assign the contributor to this?

ah I see then this issue will be automatically closed when the PR is deployed

Okay so this was a regression and is being handled here (payments, etc.): https://github.com/Expensify/App/issues/16242

@greg-schroeder i think reporting bonus should be done here, right?

Ye, but the C/C+ are in the other issue

You have an offer for reporting in Upwork.

@greg-schroeder i do not have any offer for reporting


Exception: Error compiling program when running system

Dear all,
I am trying to run an MCMD simulation using BLUES. When installing using Conda, I had problems setting the Cuda platform, so I am using a manually compiled version. All TestCuda* run fine. I prepared a system using tleap and minimization (with simulations.md.minimizeEnergy(maxIterations=0)) and a short equilibration (simulations.md.step(10000)). But then:
blues = BLUESSimulation(simulations, cfg['simulation'])
blues.run()

yields
Traceback (most recent call last):
  File "../../../../../../scripts/commands.py", line 180, in <module>
    blues.run()
  File "/mnt/nfs/exa/work/lmartins/miniconda3/envs/blues/blues-master/blues/simulation.py", line 1246, in run
    self._stepNCMC(nstepsNC, moveStep)
  File "/mnt/nfs/exa/work/lmartins/miniconda3/envs/blues/blues-master/blues/simulation.py", line 1056, in _stepNCMC
    ncmc_state0 = self.getStateFromContext(self._ncmc_sim.context, self._state_keys)
  File "/mnt/nfs/exa/work/lmartins/miniconda3/envs/blues/blues-master/blues/simulation.py", line 905, in getStateFromContext
    state = context.getState(**state_keys)
  File "/nfs/exa/work/lmartins/miniconda3/envs/blues/lib/python3.6/site-packages/simtk/openmm/openmm.py", line 5064, in getState
    state = _openmm.Context_getState(self, types, enforcePeriodicBox, groups_mask)
Exception: Error compiling program: (710): error: expected a ")"

(711): error: expected a ")"

(759): error: expected an expression

(769): error: too many arguments in invocation of macro "make_TempType3"

(769): error: expected an expression

(770): error: expected an expression

(772): warning: variable "gaussianIndex" was declared but never referenced

(773): warning: variable "uniformIndex" was declared but never referenced

(761): warning: variable "forceScale" was declared but never referenced

6 errors detected in this compilation.

What do those errors mean? I am using a system with PBC and truncated octahedral box (generated using solvateOct in tleap). My ligand is indeed called LIG.
Thank you!

Sorry for the delay, missed the GitHub notification. This looks a problem with OpenMM and not with BLUES, I think, as you're getting "compilation" errors from OpenMM. But tagging @sgill2 and @bergazin ...

Hi @luancarvalhomartins,
Could you post the scripts/files you used (or a minimal example)? From the traceback you posted it looks like there's a problem with openmm, but I can take a look and see if anything else seems out of place.  Also, have you tried running normal MD simulations on the CUDA platform with openmm?


Union types with Generics causes type-checking bugs

This issue seems like it might overlap with #3803, but I'm not sure because this one isn't about inheritance. Although, I'm not sure if the type system considers union types to be a form of inheritance.
With the following code:
alias MyValue = Int32 | String | Array(Int32 | String)

def foo(bar : MyValue)
  bar.is_a? MyValue
end

foo(12) # => true
foo([12]) # => false

It returns true for the Int32 input but false for the Array(Int32) input. But if it were not that type, that line of code could not run. Through some experimentation, it seems to be the fact that it's a generic. Swapping Array with my own generic class does the same thing.
alias MyValue = Int32 | String | Thing(Int32 | String)

class Thing(T)
end

def foo(bar : MyValue)
  bar.is_a? MyValue
end

foo(12)
foo(Thing(Int32).new)

I run into this a lot and usually I get around it with arrays by using array.map(&.as(InnerType)).as(OuterType). I'm generally not too worried about copying them to satisfy the type checker because they're usually pretty small. But for the use case I'm looking at right now, these arrays can be arbitrarily large so I'd prefer to avoid that if possible.
I tried using unsafe_as, assuming that the ABI would align correctly since the value is passed in as the type it's expecting, and well … my compliments for that very accurate method name (it crashed the app with SIGILL). :joy:

An array of T is not an array of T | U. If that were the case you would be able to put a U inside the first array.
You need to do [1] of Int32 | String
I don't think this is a bug?

Maybe calling it a bug was a bit presumptuous but it is behavior I wouldn’t expect.
Your explanation makes perfect sense and I’m completely on board with it except that the value matches the method signature. I feel like if an Array(Int32) isn’t an Array(Int32 | String), then an Array(Int32) wouldn’t be a valid argument to the method. That’s the part that I feel like I’m misunderstanding the most.
I think I also assumed that matching the type signature of the method indicated that the compile-time type of bar in this example would be MyValue, but it seems to retain its original compile-time type from outside the method according to typeof. This seems like it’d facilitate duck typing, though, so I don’t think I’m quite as confused by this one, but both of these together made it a little difficult for me to grok, even as someone who’s used Crystal for 3-4 years.
On the bright side, this is only something I’ve ever run into when building libraries like protocol implementations that can take many different types of data, including collections of those types like arrays and hashes. I rarely (if ever?) need to deal with this in application code.

Ah, yeah, that makes sense. The reason is that type restrictions work differently than is_a?. I'm aware of this but I don't know how to fix it. The type system is a bit of a mess right now. I hope other core team members can formally define and change the semantics of the language

This is essentially defining how covariance and contravariance work.
So far, I think generics inheritance, covariance, and recursive aliases are the main things that are really messy in the language.

They seem like hard problems, to be sure. I never thought about types the way Crystal does them before I tried Crystal. My main statically typed languages before this were Java, C, and C++, which don't exactly have advanced type systems. 🙂 So even though they're hard problems to solve and the implementations are probably pretty wacky in some places, I love that you've built these ideas into the language because it has absolutely changed my perspective about static types. It's honestly the first time I've found a static language fun! 😃


[BUG] Markdown Inline code blocks / latex expressions not highlighted correctly

Describe the bug
Using the TS markdown_inline parser should highlight inline codeblock / math, but gruvbox.nvim currently does not.
Expected behaviour
It should highlight those.
Screenshots

hey @gmmyung do you have the expected output? can you provide the :Inspect result for this block?

https://github.com/nvim-treesitter/nvim-treesitter/issues/5397
It should work like this in theory.

Sorry, my bad. I had TS highlighting disabled in my setup. Thanks again for your awesome library!