Support features with resources, e.x. shell scripts

Sometimes a feature has some resources that would be copied to a docker image, e.x. official Golang docker image comes with go-wrapper:
https://github.com/docker-library/golang/tree/1eab0db63794152b4516dbcb70270eb9dced4cbd/1.5

I have a very raw protoype at https://github.com/PetrGlad/pazuzu-registry/tree/32-attachments
Respective UI and CLI parts are missing at the moment.
It uses filesystem for storage. Potentially this can be used in it current form with root of file storage mapped to necessary backend (S3, Docker data volume). I plan following changes:

Implement Dockerfile preprocessor that replaces COPY commands with actual source file references (COPY can use URL for fetch resources). This way CLI would process attachments without any changes. For support of "ADD" instruction see below.
Use PostgreSQL to store files. Motivation is to provide data integrity and simplify deployment/maintenance. Also this would simplify life if audit/versioning information will be added to database. This is possible with assumption that data volumes of attachments will not be out of capacities of the PostgreSQL (it seems unlikely that there will be more than several thousands of attachments).
Review REST API: discoverability, get rid of file paths as identifiers.
Support for ADD command would require changes in CLI: files have to be copied into local folder before build. This probably requires to leave special hints for CLI on how to obtain files (within Docker file on via separate instructions that are to be sent with Docker file).

There is now working server-side implementation in https://github.com/PetrGlad/pazuzu-registry/tree/32-attachments. Missing pieces are: swagger config, proper error/status reporting, file checksums. See also https://github.com/zalando/pazuzu-registry/wiki/File-attachments

To complete implementation we need UI part (add/remove files, and highlighting in code editor for present/missing files in Docker instructions) and support from CLI side (fetch files from registry before build).
You can ask me if you have any questions regarding this.

@PetrGlad regarding API, File object has name which has unique constraint, this raises next problems:

What name should be chosen by UI when uploading new files? (it should be unique and generated somehow)
How it is possible to link already uploaded file with Feature? (From UI need somehow to figure out which file should be linked with particular feature - when names are uniquely - say randomly generated - user needs to find file that he uploaded)

I think easiest solution would be to provide path, like:
https://pazuzu.io/registry/storage/featureX/fileY

@pgronkiewicz you mean provide this path as file name for attachment to be uploaded? I thought about this too (not full, url, but just feature-name/file-name), but current implementation not allows this:
java.lang.IllegalArgumentException: Nested paths are not supported.
        at org.zalando.pazuzu.feature.file.FileContentService.getRealPath(FileContentService.java:62) ~[pazuzu-registry.jar!/:na]
        at org.zalando.pazuzu.feature.file.FileContentService.putFile(FileContentService.java:46) ~[pazuzu-registry.jar!/:na]
        at org.zalando.pazuzu.feature.file.FileService.create(FileService.java:44) ~[pazuzu-registry.jar!/:na]

Some changes in the code may be needed. I don't think it should be passed though, as from the source you have access to feature name you are creating, so it should create a path using that.

Ok, so current File REST API should be rewritten, currently it looks like:

POST /api/file?name= create file
GET/DELETE /api/file/id get/delete file

So, no relations to particular feature. Probably there should be any relation to particular feature:

POST /api/feature/{feature:id}/file?name=
etc.

But then need to know featureId should be created, so first need to store Feature, get id, then associate attachments with it.

One thing we should keep in mind is that we should not expose those files until they are approved. Otherwise we risk having our repository as a storage for uknown and potentially illegal/harmful content.

Ok, then we can add endpoint for approval:

PUT /api/feature/{feature:id}/file/{file:id} { approved: true }
Calls:
GET /api/feature/{feature:id}/file can return list of all files (whether they approved or not)
GET /api/feature/{feature:id}/file/{file:id} can return all files for oauth user with unapproved-files scope and 401 otherwise

We should write some good tests about it, to make sure we don't expose unapproved resources.

Ok, I will handle this

First of all, why do you need to strictly bind a file to a particular feature? What's wrong with sharing them?
Second, you can group files with "group/filename" just make sure it is not interpreted directly as a filename on storage - for security reasons. I Implemented file storage using filesystem, but now I think it's worthwhile to just use DB blobs for that - it will simplify many things. E.g. you can have /s in file name - and for pazuzu registry they will be just part of name without any special meaning.
Regarding filenames - they can be arbitrary. If UI needs to select one it can just start with feature name with a generated suffix, however I was expecting that user enters some meaningful name when file is added.
Current API binds existing files to feature automatically when feature is saved - any references found in Dockerfile are linked (and missing are unlinked).
You can read files attached to a feature with existing API, /api/features/{feature-name}/files

And there's unclear purpose of this project. If it's intended for in-house use by other organizations then it requires one set of features, if it's intended as public service there's host of completely different requirements. I do not see where feature/files approval fits here.

We should never create tooling just for one company :)
Approvals are needed to ensure quality and security of features available through the registry. You should always trust what you are getting back, so we cannot have unchecked features and resources inside. This issue is viable both for running this software inhouse (so Security team is happy) and as an open-source registry (so people don't use it for sharing torrents, etc)

It is what I have now: https://github.com/whiter4bbit/pazuzu-registry/commit/baf1e003f63028628d9080fab69e21b9b565660a


How to compile the presentations on the server

NOTE - only assessment presentation working right now
Open the beamer/SRG/Assessment/beamer-hake-assessment.rnw file
In R, setwd() to the same directory and:
knitr::knit("beamer-hake-assessment.rnw")
Then open a terminal and go to the same directory and run:
lualatex beamer-hake-assessment.tex
Open in the document browser. Sometimes the bottom will look cut off a little, if you press F11 you'll see that it isn't.
To make faster progress (When moving legends around a little etc on one figure), In the file explorer go to the knitr-cache sub directory and open up the image of the chunk name figure  you are working on, and make it fullscreen. Leave it open. Run the knit command only after changing one small thing (or anything) and go back to the image and watch it change. No need to close and reopen it.
If you do that, it;s way faster.
Re-compile the whole presentation after finishing each slide and check it. No need to close that either.
To add the next slide, copy and paste the commented-out slides, one by one from below the \end{document} tag, paste them above it, uncomment and edit as above.
Use the first few slides in the assessment presentation as a guide for the legend font sizes and placement, etc.

Also, to clean up the directory by removing all compile files, do this:
./clean
To remove knitr cache:
rm -rf knitr-cache
To do both in one line:
./clean;rm -rf knitr-cache


cern: user login with "access_token" not working

When you try to do a call through the API and using the access_token an error is thrown,
since it tries to get resources that are not there. This is due to token expiration from CERN oAuth (works when you login through the browser, and try the access_token first minutes before expiration).
We need to create a "refresh_token" mechanism for this to get fixed or find another workaround like storing "Groups" in the extra_data field (not ideal since User might be removed from a group and it won't get synced with our instance)
Related with  #68
  File "/Users/p/.envs/cap/lib/python2.7/site-packages/invenio_oauth2server/ext.py", line 201, in verify_oauth_token_and_set_current_user
    valid, req = func(valid, req)
  File "/Users/p/.envs/cap/src/analysispreservation.cern.ch/cap/modules/oauth2server/views/server.py", line 57, in login_oauth2_user
    identity=Identity(oauth.user.id))
  File "/Users/p/.envs/cap/lib/python2.7/site-packages/blinker/base.py", line 267, in send
    for receiver in self.receivers_for(sender)]
  File "/Users/p/.envs/cap/lib/python2.7/site-packages/invenio_oauthclient/contrib/cern.py", line 357, in on_identity_changed
    account_groups(account, resource, refresh_timedelta=refresh)
  File "/Users/p/.envs/cap/lib/python2.7/site-packages/invenio_oauthclient/contrib/cern.py", line 224, in account_groups
    groups = fetch_groups(resource['Group'])
KeyError: 'Group'

Ping @egabancho

Is there any ETA on trying to merge the fix from CERN Analysis Preservation into invenio-oauthclient? ATM I just have it patched from a mere replace, but encountered some issues, it would be good to have it all upstream for discussion/compatibilities.

ping @lnielsen @zzacharo  similar issue happens for new cern oauth. Makes Personal access tokens unusable.
Details:
Once I create a new token it only works for 5 minutes: it returns 500 after that. To work again I need to logout and login in the UI to be able to user that token for 5 more minutes.
Stack:

After discussion, it is not ideal to re-fetch CERN resource on each login via access_token. It is responsibility of the final app to decide how often synchronize the local Invenio user account with the CERN remote one, and not on each HTTP request.
Closed in favor of: https://github.com/inveniosoftware/invenio-oauthclient/issues/222


I Added mySelf

Related issue
What changes does this PR do?
Added my name and GitHub profile link
Screenshots (If any)
Questions


Migration to depthcharge v0.4.8

This version of Casanovo is now based on depthcharge v0.4.8 instead of v.0.2.3.

Hi @wfondrie ,
thanks for the comments :)
are you taking care of the updates to the unit tests, so that the CI checks pass or is it better if I have a look?
I think the documentation probably needs to be updated a bit, as well as the download of the latest weights for prediction, as the old ones are not compatible anymore.
Best,
Daniela

I think the documentation probably needs to be updated a bit, as well as the download of the latest weights for prediction, as the old ones are not compatible anymore.

Yes, we'll cut a new release v5.x.x for this implementation, as these are some breaking changes. We'll have to train a new model, but with the new major version the downloading code won't get confused.

are you taking care of the updates to the unit tests, so that the CI checks pass or is it better if I have a look?

Some of the first fixes might be relatively straightforward, with some renamed modules that have to be updated in the unit tests. If you have some bandwidth to look at it, feel free to do so.

It looks like there's a bug in Spec2Pep._finish_beams where beams that have not been predicted to end aren't checked for early termination due to exceeding the precursor m/z tolerance if the tokenizer doesn't have any residues with negative mass.
Particularly, this loop
for aa in ([None] if finished_beams[i] else aa_neg_mass_idx):

that does the early termination check is never entered if the beam isn't finished and there is nothing in aa_neg_mass_idx.

I've initialized the model's tokenizer with the residues from the tiny config as a work around to get the test_beam_search_decode test to run as we look into potential fixes, assuming this actually is a bug.

Yes, from a quick check I think you're right.
In the current version, there's always at least None included for no AAs with a negative mass. This is missing in the changed version.

The tests still seem to fail on GitHub, in contrast to the latest comment message. @Lilferrit is this expected behavior?

Best I can tell looking at the GitHub actions logs, the reason the tests fail on GitHub is due to the Pylance hf_converter keyword issue in Depthcharge. The test do pass in my local environment, but I manually downgraded Pylance to v0.15.0. Should I update Casanovo's pyproject.toml to require Pylance v0.15.0? That should solve the issue with the tests not passing on GitHub.

Yes.

Yes. And also make an issue and link it to the one in DepthCharge to track this.

Done, all of the test on GitHub pass now, and seem to run faster than they have historically as well.

I overwrote the previous merge with dev with the latest rebase to make the diff more representative of any new functionality.


Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
No response
Which platforms are affected?
Android
Description
I am using the firebase_messaging:10.0.8 package to get the token, but on Realme 6 (Android 11) and Redmi Note 8 (Android 11) devices, it returns null with the exception [firebase_messaging/unknown] java.io.IOException: PHONE_REGISTRATION_ERROR.
Reproducing the issue
var messaging = FirebaseMessaging.instance;
String regId =await messaging.getToken();
Firebase Core version
1.16.0
Flutter Version
2.10.2
Relevant Log Output
No response
Flutter dependencies

Expand Flutter dependencies snippet

Replace this line with the contents of your `flutter pub deps -- --style=compact`.

Additional context and comments
No response

Hi @roja321 . Kindly provide a minimal sample code reproducing this issue. Also, add the steps followed to reproduce the issue.

@SelaseKay Thank you for your attension.
import 'package:firebase_messaging/firebase_messaging.dart';
String  regId =await FirebaseMessaging.instance.getToken();
here in regId i got null. for above mention devices.

Hi @roja321 , I'm unable to reproduce this issue. Upgrade your firebase_messaging plugin and try again.

Hello @roja321, it looks like your devices doesn't have the Google Play Services to use Firebase Messaging. Nothing the FlutterFire repository can fix.


Unsupported demangle type: transaction clone

http://cyberjes.us/cc1
GCC apparently uses some obscure mangling: https://gcc.gnu.org/ml/gcc-patches/2008-11/msg00200.html

Yup there are a few types that we don't support currently. I didn't feel like looking into what the types of these actually were.
			LogWarn("Unsupported demangle type: hidden alias\n");
			throw DemangleException();
		case 'R': //TODO reference temporaries
			LogWarn("Unsupported demangle type: reference temporary\n");
			throw DemangleException();
		case 'T': //TODO transaction clones
			LogWarn("Unsupported demangle type: transaction clone\n");
			throw DemangleException();```


Missalinices

Create a README.md file.
Create a .gitignore file.
Create a .travis.yml file.

Link to the example repo: https://github.com/nirgn975/open_pension_scraper

miscellaneous?

@uda Yeah, didn't know how to call it.
As you can see It's all sort of thing that not related to each other, just to get this repo of the ground. If you have a better name edit it (;


specific iOS issues

I'm creating a separate ticket from #1908 for issues a user experienced at the workshop using chrome on a newer ipad air.
The main issuers were:

Jobs page didn't load at all.
The blast input form (and probably others) was requiring two keypresses to enter 1 letter.

Bob reports that jobs page doesn't load on his iPhone.  I don't own any apple mobile devices, but we should be able to tether to debug.

Apple uses the WebKit Browser Engine in Safari on both MacOS and iOS. In the case of iOS though, it's the only browser engine used no matter if you use Firefox, Edge, etc on your iPhone or iPad.
For simplicity sake let's assume iOS versions are updated and the same on iPhone or iPad. The biggest change between those two devices then is resolution, so those errors are usually related to CSS styling issues, and should be straightforward to fix.
Physically, I have an iPhone X and iPad Gen 6 (2018) w/ support for the Apple Pencil. I might be able to get my hands on a few other models.
Using Xcode we can use the iOS SDK to simulate multiple other devices and iOS version but a standard iPhone size and standard iPad size should suffice for now.
Issues: (adding as we go)

[ ] Jobs page does not load
[ ] Input forms require multiple presses to input characters.
[ ] ...


Deleting shapes with connectors doesn't work with Ctrl+X shortcut

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
When I like to delete a shape with the related connector, I can select it (or a specific subset) and then right-click and select the option "Cut". Besides the option it is said that I could also use Cmd+X to do this. But when I use the classic Cmd+X shortcut, only the shape is deleted, not the connector. I know that I can use Cmd+Delete to use both objects at the same time, I just wanted to inform you that the shortcut isn't working.
To Reproduce
Steps to reproduce the behavior:

Select a shape that is connected to at least one other shape
Richt-Click and choose "Cut". Then try the same procedure with Cmd+X

Expected behavior
The shortcut Cmd+X acts the same as the mouse command.
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

I tested it in Confluence Cloud and Server at the latest version of draw.io and Confluence

Thanks for the report. This will be fixed in the next release as follows:

Alt/Cmd/Ctrl+Delete (keyboard shortcut, icon in toolbar, menu and context menu item): Delete cells with edges
Shift+Delete (keyboard shortcut, icon in toolbar, menu and context menu item): Delete labels
Cut (keyboard shortcut, icon in toolbar, menu and context menu item): Delete only selected cells

Use Select Edges in context menu to include edges in Cut.


Use new seed_stack puppet module

praekeltfoundation/seed_stack
Have to do a dependencies hack for now...

My fresh vagrant up fails because puppet/modules doesn't exist.

Aww.. will fix

:+1: -- the build failures are unrelated and I can confirm that the VM provisions correctly on my machine.


restyled news page

Description
Redesigned news page as per the theme of the website,added picture slider to make it attractive, added functionality to read more
Related Issues
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]

Closes #714

Type of PR

[x] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
https://github.com/user-attachments/assets/2beab93c-71d8-4f50-90ed-d071cd54714e
Checklist

[x] I have gone through the contributing guide
[x] I have updated my branch and synced it with project main branch before making this PR
[x] I have performed a self-review of my code
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
GSSoC'24

Please review my pull request @varshith257 @Anishkagupta04

Please review my pull request @varshith257

Kindly take a look @varshith257

@samarsajad Fix merge conflicts

@samarsajad Fix merge conflicts

Done @varshith257

@samarsajad Looks good to merge after fixing this suggestions:
Make sure all readme buttons
be center at each card
Duplicate navbars resulted. Fix it

@samarsajad Looks good to merge after fixing this suggestions:
Make sure all readme buttons be center at each card
Duplicate navbars resulted. Fix it

I did a thorough screening of my code but I couldn't find anything related to duplicate nav bar. In fact in my local system the duplicate nav bar is not showing at all so I couldn't even use the inspect tool. Please help me out with this @varshith257

See here

See here

IK it is in the deploy preview but not in my local system. Even if i try fixing it, I wouldn't know if it is fixed cause this is not visisble in my local system. If you could merge this pull request and if the duplicate nav bar appears in the main, I'd raise a new issue for that cause as if now i am not able to fix this

Also align hover me button center to card and make sure not stick with image

i will create a new pull request. things got a lil messed up @varshith257

created a new pull request #961  ptal @varshith257


Fix old commands in documentation

Fixes some old references to rake and yarn commands which have now changed.
Types of Changes

[ ] Bug fix.
[ ] New feature.
[ ] Performance improvement.
[x] Documentation.

Testing

[ ] I added new tests for my changes.
[ ] I ran all the tests locally.

Thanks!


[REQUEST]Make semirage

Can someone make semirage tab? Semirage include fov,silent,aimlock,hitgroup,autoshot and resolver. Osiris good for semirage

Code yourself a resolver and you have a "semirage tab" you want.

I tried but it won't compile


feat: Import Job for Opportunity Data from beta.sam

[x] You've included unit or integration tests for your change, where applicable.
[x] You've included inline docs for your change, where applicable.
[x] There's an open issue for your PR that you are making. Please open an issue to discuss changes or new features.

PR Title
Summary of the changes.
PR Description
Detail 1
Detail 2
Fixes #10

@sushilkumarss-sdn Please go over comments and rework into a background service. This and the contacts service are all supposed to be .NET background workers, not WebAPIs.
The project name you can use for the worker project is Medulla.Actions.SamGov and the tests in Medulla.Actions.SamGov.Tests.
There should be no modifications to the existing projects in this PR at all.

@sushilkumarss-sdn Please go over comments and rework into a background service. This and the contacts service are all supposed to be .NET background workers, not WebAPIs.
The project name you can use for the worker project is Medulla.Actions.SamGov and the tests in Medulla.Actions.SamGov.Tests.
There should be no modifications to the existing projects in this PR at all.

@AndrewBabbitt97, I have pushed the latest changes as per your comment, I reverted the changes from Medula.Frontend.Server and created 2 different projects i.e Medulla.Actions.SamGov and the tests in Medulla.Actions.SamGov.Tests.

@sushilkumarss-sdn this still isn't a .NET background worker.
This should not be a WebAPI project. Use the .NET background worker template.


[recreated] Detect read function aborting by messaged require statement

[Reposted because the repository was recreated]
Dear all:
If a require statement with a message aborts the reading function, an unexpected value may be returned as a response.
For example, if you call the following function,
function testC3( uint val ) public pure returns( uint256 ){
  require( val != 0, "aborted by testC3" );

  return( 0 );
}

The following response will be returned.

// Succeeded: testC3( 1 )
["0": 0]
// aborted: testC3( 0 )
["0":<PHONE_NUMBER>197344453575983046348115674221700746820753546331534351508065746944]

If the reading function is aborted by a require statement that has a message, it seems that the data with the following configuration will be returned.(By the way, if there is no message specified in the require statement, [0] bytes will be returned as a response, so require could not be detected.)

byte[00-31]: 08C379A000000000000000000000000000000000000000000000000000000000
byte[32-63]:<PHONE_NUMBER>000000000000000000000000000000000000000000000000000000
byte[64-67]: length of require message
byte[68-(68+length)]: require message body

Since it is helpful to know whether the reading function was aborted by require statement, I made a simple correction and sent the pull request.
please confirm.
If you call the above sample function with the modified code, empty data will be returned with the following log

// Aborted: testC3( 0 )
read function aborted by require statement: aborted by testC3
["0": 0, "_abortedByRequire": true, "_errorMessageFromRequire": "aborted by testC3"]

Also, I have confirmed the operation by checking the operation with the following contract.
pragma solidity >= 0.5.0 < 0.7.0;

contract HelloWorld {
    //-----------------------------
    // test A
    //-----------------------------
    function testA1( uint val ) public pure{
      require( val != 0 );      // no message
    }

    function testA2( uint val ) public pure{
      require( val != 0, "" );  // empty string
    }

    function testA3( uint val ) public pure{
      require( val != 0, "aboreted by testA3" );
    }

    //-----------------------------
    // test B
    //-----------------------------
    function testB1( uint val ) public pure returns( string memory ){
      require( val != 0 );      // no message

      return( "hello, world" );
    }

    function testB2( uint val ) public pure returns( string memory ){
      require( val != 0, "" );  // empty string

      return( "hello, world" );
    }

    function testB3( uint val ) public pure returns( string memory ){
      require( val != 0, "aborted by testB3" );

      return( "hello, world" );
    }

    //-----------------------------
    // test C
    //-----------------------------
    function testC1( uint val ) public pure returns( uint256 ){
      require( val != 0 );      // no message

      return( 0 );
    }

    function testC2( uint val ) public pure returns( uint256 ){
      require( val != 0, "" );  // empty string

      return( 0 );
    }

    function testC3( uint val ) public pure returns( uint256 ){
      require( val != 0, "aborted by testC3" );

      return( 0 );
    }

    //-----------------------------
    // test D
    //-----------------------------
    function testD1( uint val ) public pure returns( uint256[5] memory ){
      require( val != 0 );      // no message

      uint256[5] memory temp;
      return( temp );
    }

    function testD2( uint val ) public pure returns( uint256[5] memory ){
      require( val != 0, "" );  // empty string

      uint256[5] memory temp;
      return( temp );
    }

    function testD3( uint val ) public pure returns( uint256[5] memory ){
      require( val != 0, "aborted by testD3" );

      uint256[5] memory temp;
      return( temp );
    }

    function testD4( uint val ) public pure returns( uint256[5] memory ){
      require( val != 0, "long message from testD4, abcdefghijklmnopqrstuvwxyz, ABCDEFGHIJKLMNOPQRSTUVWXYZ,<PHONE_NUMBER>.abcdefghijklmnopqrstuvwxyz, ABCDEFGHIJKLMNOPQRSTUVWXYZ,<PHONE_NUMBER>.abcdefghijklmnopqrstuvwxyz, ABCDEFGHIJKLMNOPQRSTUVWXYZ,<PHONE_NUMBER>.abcdefghijklmnopqrstuvwxyz, ABCDEFGHIJKLMNOPQRSTUVWXYZ,<PHONE_NUMBER>.abcdefghijklmnopqrstuvwxyz, ABCDEFGHIJKLMNOPQRSTUVWXYZ,<PHONE_NUMBER>.abcdefghijklmnopqrstuvwxyz, ABCDEFGHIJKLMNOPQRSTUVWXYZ,<PHONE_NUMBER>.abcdefghijklmnopqrstuvwxyz, ABCDEFGHIJKLMNOPQRSTUVWXYZ,<PHONE_NUMBER>.abcdefghijklmnopqrstuvwxyz, ABCDEFGHIJKLMNOPQRSTUVWXYZ,<PHONE_NUMBER>.abcdefghijklmnopqrstuvwxyz, ABCDEFGHIJKLMNOPQRSTUVWXYZ,<PHONE_NUMBER>." );

      uint256[5] memory temp;
      return( temp );
    }
}

Then, thank you for the above.

Thank you very much! Great addition!
And thank for represent test cases as well! As final addition before merge, I would like to setup test of feature on rinkeby network.

Thank you for your reply.
Please check the code and operation.

@gitcoinbot  I received acceptance notification about 3 hours ago. Please I am on it .. I would update in 24 hours time.

@janus would you like to add test as @hakumai-iida described above? is so, i will create separate bounty for that. (this bount reserved for @hakumai-iida  (please, apply for this task in https://gitcoin.co/issue/matter-labs/web3swift/276/100023790?mutate_worker_action=reject&worker=surajsingla333)

@skywinder  Okay .. I would do as per your instruction

@skywinder
Sorry for the my late action.
(I was going to apply once your check was complete.)
And, i tried to apply it, but it failed with the following error.
"There is already someone working on this bounty."

No worries @hakumai-iida no it should be fine￼  please try again and feel free to contact me by email if you need any support.
￼
I Check Code is excellent.
just want to finish some automated tests before the merge￼ 👍

dear  @hakumai-iida ,
thank you for your outstanding work!
Since you already made PR, Please submit for this bounty. I would be happy to reward you!
Even more, I really appreciate the quality of your code and test coverage. So I will double the amount of this bounty upon submission.
Happy new year! 🎉

@janus
we need to cover it by tests it before the next major release
feels free to apply for updating tests. I will release bounty for this case!

@janus
we need to cover it by tests it before the next major release
feels free to apply for updating tests. I will release bounty for this case!

Thanks. Let me have the url of the bounty.

@janus there is no URL (yet). you have to create an issue and I will link bounty to it. 👍

@janus there is no URL (yet). you have to create an issue and I will link bounty to it. 👍

Okay

dear @skywinder
I am glad the pull request merged without problems.
And I submitted the work on gitcoin.
Thank you for the double bounty!


add regtest to coin.py

How do you feel about this? It is merely because I am preparing a PR for electrumsv to add regtest support.
Alternatively I can just point to BitcoinTestnet because technically it is the same.

Thanks! This is the PR https://github.com/electrumsv/electrumsv/pull/378


How to avoid double tap on button that fires drop down.

I need to know when drop down is shown or not shown to avoid double tap on button that opens drop down. Can you please provide checked like isShowingNow? Or something like this.

@kevin-hirsch hey man, we really need this feature. please, provide as soon as posible

Hello @Said13 👋
No need to rush, you're always welcomed to do a pull request 😉
But for the case you mention, you shouldn't have any problem:
calling dropDown.show() on a dropDown which is already displayed shouldn't do anything 😉
Now if you really need to now if the dropDown is displayed already, since it's a UIView you can use:

dropDown.isHidden property
dropDown.alpha = 0
DropDown.VisibleDropDown == myDropDown

@kevin-hirsch , thank you for your fast feedback. Well, it should not, but actualy willShowAction is happening even if dropDown is already displayed.

I see the problem. A workaround would be to move the code you put in the willShowAction next to where you call dropDown.show()


*UI and Backend Update:

Classic progress bars added for food item tracking.
Requirements for tracking calorie in different ways provided with necessary arrangements.
*Backend Update
Circular Progressive Bars implemented with necessary code arrangements.
All the data requirements provided without database for now.
The data and all the progress bars linked correctly.

Accepted.


Replace tab event bind with .delegate?

The tabs.js code can use a less memory intensive jQuery function, .delegate, to accomplish the same functionality in fewer LOC and with a smaller memory footprint. Instead of iterating over each ul.tabs and creating a new anonymous function each time, you could bind as follows:
$('body').delegate('ul.tabs > li > a', function(e) {
    //Get Location of tab's content
    var contentLocation = $(this).attr('href');

    //Let go if not a hashed one
    if(contentLocation.charAt(0)=="#") {

        e.preventDefault();

        //Make Tab Active
        $(this).parent().siblings().children('a').removeClass('active');
        $(this).addClass('active');

        //Show Tab Content & add active class
        $(contentLocation).show().addClass('active').siblings().hide().removeClass('active');

    }
});

The above is an example of the approach, and not actually tested. I can create a tested pull request if you'd prefer. An additional advantage of this approach is that it supports dynamic tabs; developers can create tabs that will still have the same functionality without rebinding events, or add new tabs to existing tab bars. The number of cases where that will be useful is limited, hence why the primary reason to modify the tabs is the reduced memory footprint/advantage in speed.

Scott forgot to include one of the parameters in the delegate function. It should be:
$('body').delegate('ul.tabs > li > a', 'click', function(e) {

instead of
$('body').delegate('ul.tabs > li > a', function(e) {

As to what the benefits are, I wouldn't know. :P

I actually forked and initiated a pull request shortly after Dave noted the issue, so it's been fixed for a while. If you're curious about the benefits, this is a good starting point, and you can find references all over the place.
Though, on researching that link again, there's now a disclaimer that the functionality will be encompassed in the on method... so we'll see how this evolves alongside jQuery.

Couldn't merge in cause the typo, but this is in the branch now :)

Hey Dave -
Since the commit for this is so recent, I'd recommend switching to the .on syntax instead of .delegate. The jQuery docs state that the latter is being phased out, and Skeleton now includes jQuery-1.7.1 by default. It's a quick fix -
$('body').delegate('ul.tabs > li > a', 'click', function(e) {

to
$('body').on('click', 'ul.tabs > li > a', function(e) {

I can pull-request this after a quick test, if you'd like; but the two calls are functionally identical, according to jQuery.delegate docs, and it really is very cosmetically trivial. It's only for future-proofing when .delegate is properly deprecated.

Hey man -
Go for that and get a pull request and I will merge it in :) Thanks so much!

Merged new ".on" and jQuery 1.7.1


git hub help

please can someone send me some coding help and just some coding stuuf i can start on
thanks,
w2019fgatfield


Check if crud in controller breaks MVC pattern in the client side

when modifying the model, the changes are automatically passed to the view. The execute and paint operations could be also a a  call to controller instead of in the model. Check what is best regarding MVC pattern
Another topic is that the controller is managing changes in the client and in the server. Should create read update delete used for the backend or used instead of model methods and rename the methods to something than updateRemote createRemote etc?

also check redux
In general try to diverge from existing UIs and state management paradigms


💭 How to wrap an overlay around the preview and render a square output

Question
Looking to try to do what instagram do here, providing the user with a clear view into what they are taking, whilst also maintaining quality when i use photoAspectRatio: 1 in format the preview goes terrible quality even with video resolution set to 'max'
Looking to achieve the effect shown in this photo

What I tried
using useCameraFormat
VisionCamera Version
4.5.1
Additional information

[X] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

Hi @mrousavy I just used the provided question template, doesn't seem like an bug or issue.

Hey - for stuff like this, please use the community discord. This doesn't seem like any question related to camera internals, it's more of a "how do I lay out the react view properly" thing.


Add fusion.py with GraphInfo, FusionElement, CommType,  _scan_graph, _insert_buffer

This is first part of the fusion code.
Currently the run_fusion will populate a master graphinfo object (which accounts for the lack of tail in graph.py),
and then does a _scan_graph to populate a list of FusionElements, including the parameter size of each element (from node meta data).
FusionElements represent the current communication sections of the graph that will then be fused based on the optimizing algorithm.
The detection is flexible in that it uses a section pattern match with adjustable comm type.
Assumptions are that all communication insertions are based around the 'dummy_add' function in api.py.
The file is not currently 'live' in terms of being called by api.py yet.

Thanks @lessw2020 for breaking up the PR. Looks great :)


Package not honoring sphinxcontrib namespace package structure

Should the autodoc_pydantic sphinxcontrib/__init__.py file file exist?  Or does it need to declare the namespace like sphinxcontrib.apidoc does?  According to PyPA Packaging namespace packages, it shouldn't exist.
When the __init__.py file is present and the autodoc_pydantic extension is registered (using Sphinx 5.3.0), I get an error:
Extension error:
Could not import extension sphinxcontrib.autodoc_pydantic (exception: No module named 'sphinxcontrib.autodoc_pydantic')

If I manually remove the __init__.py file then run sphinx again, it seems to run fine.
Looking at other extensions, they don't define a sphinxcontrib/__init__.py file like:

https://github.com/sphinx-contrib/autofile/tree/main/src/sphinxcontrib/
https://github.com/sphinx-contrib/confluencebuilder/tree/main/sphinxcontrib
https://github.com/sphinx-contrib/spelling/tree/main/sphinxcontrib

From what I can see, apidoc may be the only one declaring the namespace?

Also, after I set up the repository locally on my Ubuntu 22.04 and Python 3.10.12 VM with poetry (after sudo apt-get install graphviz graphviz-dev too), then ran poetry run pytest, I get the same errors in the tests:

I'll work to submit a small PR and see how things go.

Having the same issue now trying to set this up for development, wonder if this is why I am having issues getting the autodoc_pydnatic functionality to build on my project

Maybe related so posting this here:
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic/docs$ make clean && make html
Running Sphinx v5.3.0

Extension error:
Could not import extension sphinx_rtd_theme (exception: No module named 'sphinx_rtd_theme')
make: *** [Makefile:20: html] Error 2
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic/docs$ exit
exit
daquintero@dxps:~/flexcompute/autodoc_pydantic$ poetry install -E dev
Installing dependencies from lock file

Package operations: 20 installs, 0 updates, 0 removals

  • Installing distlib (0.3.7)
  • Installing filelock (3.12.2)
  • Installing platformdirs (4.0.0)
  • Installing iniconfig (2.0.0)
  • Installing mccabe (0.6.1)
  • Installing pluggy (1.2.0)
  • Installing py (1.11.0)
  • Installing pycodestyle (2.7.0)
  • Installing pyflakes (2.3.1)
  • Installing six (1.16.0)
  • Installing sphinxcontrib-jquery (4.1)
  • Installing virtualenv (20.24.7)
  • Installing coverage (7.2.7)
  • Installing flake8 (3.9.2)
  • Installing pytest (7.4.3)
  • Installing sphinx-copybutton (0.4.0)
  • Installing sphinx-rtd-theme (1.3.0)
  • Installing sphinx-tabs (3.4.4)
  • Installing sphinxcontrib-mermaid (0.7.1)
  • Installing tox (3.28.0)

Installing the current project: autodoc_pydantic (2.0.1)
daquintero@dxps:~/flexcompute/autodoc_pydantic$ poetry shell
Spawning shell within /home/daquintero/.cache/pypoetry/virtualenvs/autodoc-pydantic-QOa8Dwz7-py3.11
gpg-agent[51818]: option file '/home/daquintero/.gnupg/gpg-agent.conf': No such file or directory
daquintero@dxps:~/flexcompute/autodoc_pydantic$ . /home/daquintero/.cache/pypoetry/virtualenvs/autodoc-pydantic-QOa8Dwz7-py3.11/bin/activate
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic$ ls
changelog.rst  docs  LICENSE  poetry.lock  pyproject.toml  README.md  sphinxcontrib  tests  tox.ini
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic$ cd docs/
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic/docs$ ls
make.bat  Makefile  source
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic/docs$ make clean && make html
Running Sphinx v5.3.0

Extension error:
Could not import extension sphinxcontrib.autodoc_pydantic (exception: No module named 'sphinxcontrib.autodoc_pydantic')
make: *** [Makefile:20: html] Error 2
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic/docs$ cd
(autodoc-pydantic-py3.11) daquintero@dxps:~$
(autodoc-pydantic-py3.11) daquintero@dxps:~$ ls
Desktop  Documents  Downloads  flexcompute  java_error_in_pycharm_.hprof  legacy  Music  phd  Pictures  Public  snap  someconfig.txt  Templates  Videos
(autodoc-pydantic-py3.11) daquintero@dxps:~$ cd flexcompute/
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute$ ls
autodoc_pydantic  tidy3d  tidy3d-notebooks
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute$ cd autodoc_pydantic/
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic$ ls
changelog.rst  docs  LICENSE  poetry.lock  pyproject.toml  README.md  sphinxcontrib  tests  tox.ini
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic$ make
make: *** No targets specified and no makefile found.  Stop.
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic$ sphinx
sphinx: command not found
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic$ exit
exit
daquintero@dxps:~/flexcompute/autodoc_pydantic$ cd docs/
daquintero@dxps:~/flexcompute/autodoc_pydantic/docs$ poetry run make html
Running Sphinx v5.3.0

Extension error:
Could not import extension sphinxcontrib.autodoc_pydantic (exception: No module named 'sphinxcontrib.autodoc_pydantic')
make: *** [Makefile:20: html] Error 2
daquintero@dxps:~/flexcompute/autodoc_pydantic/docs$ code .
daquintero@dxps:~/flexcompute/autodoc_pydantic/docs$ cd ..
daquintero@dxps:~/flexcompute/autodoc_pydantic$ ls
changelog.rst  docs  LICENSE  poetry.lock  pyproject.toml  README.md  sphinxcontrib  tests  tox.ini
daquintero@dxps:~/flexcompute/autodoc_pydantic$ cat docs/make.bat
@ECHO OFF

pushd %~dp0

REM Command file for Sphinx documentation

if "%SPHINXBUILD%" == "" (
        set SPHINXBUILD=sphinx-build
)
set SOURCEDIR=source
set BUILDDIR=build

if "%1" == "" goto help

%SPHINXBUILD% >NUL 2>NUL
if errorlevel 9009 (
        echo.
        echo.The 'sphinx-build' command was not found. Make sure you have Sphinx
        echo.installed, then set the SPHINXBUILD environment variable to point
        echo.to the full path of the 'sphinx-build' executable. Alternatively you
        echo.may add the Sphinx directory to PATH.
        echo.
        echo.If you don't have Sphinx installed, grab it from
        echo.http://sphinx-doc.org/
        exit /b 1
)

%SPHINXBUILD% -M %1 %SOURCEDIR% %BUILDDIR% %SPHINXOPTS% %O%
goto end

:help
%SPHINXBUILD% -M help %SOURCEDIR% %BUILDDIR% %SPHINXOPTS% %O%

:end
popd
daquintero@dxps:~/flexcompute/autodoc_pydantic$ cat docs/Makefile
# Minimal makefile for Sphinx documentation
#

# You can set these variables from the command line, and also
# from the environment for the first two.
SPHINXOPTS    ?=
SPHINXBUILD   ?= sphinx-build
SOURCEDIR     = source
BUILDDIR      = build

# Put it first so that "make" without argument is like "make help".
help:
        @$(SPHINXBUILD) -M help "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)

.PHONY: help Makefile

# Catch-all target: route all unknown targets to Sphinx using the new
# "make mode" option.  $(O) is meant as a shortcut for $(SPHINXOPTS).
%: Makefile
        @$(SPHINXBUILD) -M $@ "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
daquintero@dxps:~/flexcompute/autodoc_pydantic$ poetry run sphinx docs/source/ docs/build
Command not found: sphinx
daquintero@dxps:~/flexcompute/autodoc_pydantic$ poetry shell
Spawning shell within /home/daquintero/.cache/pypoetry/virtualenvs/autodoc-pydantic-QOa8Dwz7-py3.11
gpg-agent[52760]: option file '/home/daquintero/.gnupg/gpg-agent.conf': No such file or directory
daquintero@dxps:~/flexcompute/autodoc_pydantic$ . /home/daquintero/.cache/pypoetry/virtualenvs/autodoc-pydantic-QOa8Dwz7-py3.11/bin/activate
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic$ pip list
Package                       Version    Editable project location
----------------------------- ---------- ---------------------------------------------
alabaster                     0.7.13
annotated-types               0.5.0
autodoc_pydantic              2.0.1      /home/daquintero/flexcompute/autodoc_pydantic
Babel                         2.13.1
certifi                       2023.11.17
charset-normalizer            3.3.2
coverage                      7.2.7
distlib                       0.3.7
docutils                      0.18.1
filelock                      3.12.2
flake8                        3.9.2
idna                          3.4
imagesize                     1.4.1
iniconfig                     2.0.0
Jinja2                        3.1.2
MarkupSafe                    2.1.3
mccabe                        0.6.1
packaging                     23.2
pip                           23.3.1
platformdirs                  4.0.0
pluggy                        1.2.0
py                            1.11.0
pycodestyle                   2.7.0
pydantic                      2.5.1
pydantic_core                 2.14.3
pydantic-settings             2.0.3
pyflakes                      2.3.1
Pygments                      2.17.2
pytest                        7.4.3
python-dotenv                 0.21.1
requests                      2.31.0
setuptools                    68.2.2
six                           1.16.0
snowballstemmer               2.2.0
Sphinx                        5.3.0
sphinx-copybutton             0.4.0
sphinx-rtd-theme              1.3.0
sphinx-tabs                   3.4.4
sphinxcontrib-applehelp       1.0.2
sphinxcontrib-devhelp         1.0.2
sphinxcontrib-htmlhelp        2.0.0
sphinxcontrib-jquery          4.1
sphinxcontrib-jsmath          1.0.1
sphinxcontrib-mermaid         0.7.1
sphinxcontrib-qthelp          1.0.3
sphinxcontrib-serializinghtml 1.1.5
tox                           3.28.0
typing_extensions             4.7.1
urllib3                       2.0.7
virtualenv                    20.24.7
(autodoc-pydantic-py3.11) daquintero@dxps:~/flexcompute/autodoc_pydantic$ exit
exit
daquintero@dxps:~/flexcompute/autodoc_pydantic$ poetry run Sphinx docs/source/ docs/build
Command not found: Sphinx
daquintero@dxps:~/flexcompute/autodoc_pydantic$ poetry run python3 -m sphinx docs/source/ docs/build
Running Sphinx v5.3.0

Extension error:
Could not import extension sphinxcontrib.autodoc_pydantic (exception: No module named 'sphinxcontrib.autodoc_pydantic')

@devmonkey22 and @daquinteroflex thanks a lot for looking into this issue and providing a PR. I agree with your observation that __init__.py should no longer be present in the sphinxcontrib namespace package. In fact, I used to have issues running tests in some environments due to failed imports, too. But I wasn't aware that this could be caused by the somewhat incorrect/inconsistent usage of namespace packages.
I will take a closer look at it. While removing the init file, we need to be sure that we don't cause unrecognized backwards-incompatible breaking changes.

@all-contributors please add @devmonkey22 for code, bug and test

@all-contributors please add @daquinteroflex for review

Before merging the related PR, it would be great if you could test the changed behavior. To do so, please install the current dev release in your doc-building-environment via pip install<EMAIL_ADDRESS>and rebuild your docs.
Please feel free to reopen if any problem remains. I will release the new version upcoming Monday.


Possible to add labels to inlets and outlets?

Hello, thanks so much for the effort on this toolkit, I'm just starting to scratch the surface and learning PD as I go, it's really helpful to study your patches!
I would like to ask if it would be possible to include inlets and outlets labels on the patches, that would be really helpful for checking quickly what sort of input the node expects without having to open the help file.

I could volunteer to do it myself but I'm sure there will be some patches I'm not gonna understand fully at a first.
Thank you!
edit: I notice now I was just unlucky to find the 'rand' example without them. Others such as tempo already have this, so it might not be too relevant.

I agree labels would assist readability of patches. I'm not aware of an elegant method for doing this in Pd, beyond checking the help patches that include inlet and outlet info., but I am happy to take advice. Adding free-floating text descriptions does not seem like a sustainable solution.


ErrorException 8 - Where to start troubleshooting?

Hi, I've installed an application that throws the following error:
Slim Application Error
The application could not run because of the following error:
Details
Type: ErrorException
Code: 8
Message: Use of undefined constant onAddSubscriber - assumed 'onAddSubscriber'
File: /var/www/clients/client2/web16/web/API/functions_subscriber.php
What's the meaning of "Use of undefined constant onAddSubscriber - assumed 'onAddSubscriber'"?
Please advice.

onAddSubscriber is not part of Slim. I would look at the file  /var/www/clients/client2/web16/web/API/functions_subscriber.php suggested by the error message.

Use of undefined constant is a PHP error: See here: https://encrypted.google.com/search?hl=en&q=Use of undefined constant

Thanks, guys! With your help I found the source of the problem.
I changed this code:
if (function_exists(onAddSubscriber)) { 	onAddSubscriber($email_list_id); }
into
if (function_exists('onAddSubscriber')) { 	onAddSubscriber($email_list_id); }
Needed to wrap the function name in single quotes.
Problem solved!


New paths feature from 5.8 not showing in VSCode

I am running Spectral 5.8.1 locally and absolutely love the new Paths feature!
I am running 0.2.5 of the VSCode extension but not seeing the paths in the problems

To reproduce, remove a description from a path and you should see the error misses the path in the VSCode extension.

Give https://github.com/stoplightio/vscode-spectral/releases/tag/v1.0.0 a try, should sort that right out!

Hi,
Given the following bad example:

I can see the path being returned in the cli

but within the problems list it remains without the path

Am running 1.0.0 of the extension

Happy to show / help reproduce if it helps

Ah of course, sorry.
Do you actually want to see the path, or do you want to be able to open the file up at the right point to work on it?
is there a use case I’m not thinking of?


Update issue/feature templates

Tagging @dgasmith for comments since he recommended we add templates for issues features.

You can find ours here. A nice example as well.

Whoops, thought this was an issue! Reading now.


Need tests

Currently, there are no test code for wolper. At least some unit tests and E2E tests are required.

#6


chore: camel cardinal_mk

Rename cardinal_mk in lemma names to cardinalMk to follow the naming convention.

The fact that #α is precisely the constructor of Cardinal applied to α is not really intuitive unless you're aware of it. I'd also prefer that we simply rename these theorems to use card.

Did you see @eric-wieser 's suggestion on Zulip to rename it to something with card in name? I suggest that we make a decision first to avoid 2 renames in a row.

Yes I saw, but this is a much bigger rename than the one at hand, which is why I would rather do this one first. I have started a Zulip thread anyway.

@eric-wieser WDYT?

Eric says that he had no intention to block, so I'm re-delegating it. @YaelDillies , sorry for a delay caused by my misunderstanding.
bors d+

bors merge

bors merge


HOSTEDCP-978: Bump openshift/api version and fixed KCM flags (k8s 1.27)

What this PR does / why we need it:
follow-up fixes after the bump of k8 to 1.27 https://github.com/openshift/api/pull/1424

/test verify

Merging this so we can get unstuck in CI
/approve
/lgtm


add delta-orthogonal initializer

Hi, thank you for your great work.
I really like haiku!
I want to use  Delta-Orthogonal initialization for my Reinforcement Learning repo, so I implemented it.
It's better to add it to the initializers_test.py, but it needs len(shape) in [3, 4, 5]. So I haven't added it to the test script yet.
I hope this initializer will be integrated into master so that I can use it just through pip.
Anyway, thank you for your work:)

@googlebot I signed it!


Enhancements to Model factory from a client engagement

Recently in a client engagement, this accelerator was used to bootstrap the model factory.
Here are some of the enhancements from the engagement that will be added back to this accelerator.

#15
#16
#17

This will be the placeholder/parent issue to keep track of these enhancements.

closing this since all the issue are addressed, and PRs are merged 🎉


I18n

@Darkmift
Following issue #138 ,
I added i18n using this library: next-intl that adapted to Next.js.
I only applied it on the about page for example.
After I added this feature, I found few bugs:

The web is RTL even when the selected language is English
The English font 'Birzia' only supports capital letters in English
After switching between hebrew and english 3 times, it doesn't switch again (you can see it in the application -> cookies)

@Darkmift Following issue #138 , I added i18n using this library: next-intl that adapted to Next.js. I only applied it on the about page for example.
After I added this feature, I found few bugs:

The web is RTL even when the selected language is English
The English font 'Birzia' only supports capital letters in English
After switching between hebrew and english 3 times, it doesn't switch again (you can see it in the application -> cookies)

RTL we need to implement logic that toggles the dir= in the layour html
Will resolve this down the line
Will resolve down the line


[fast-client] Wrong value schema id used when handling multi-get response in FC

[fast-client] Wrong value schema id used when handling multi-get response in FC
batchGetTransportRequestCompletionHandler should obtain the correct dataRecordDeserializer by using the MultiGetResponseRecordV1's per record value schema id field. The handler was using the transport response header's schema id which is inteded for the protocol version and not the write schema version for the data records. As a result, FC is running into data issues when the user record's write schema id > 1 with new fields/data.
How was this PR tested?
Existing tests and modified an unit test to ensure the correct value schema id is obtained. Also war dropped to confirm the root cause and fix.
Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Thanks @xunyin8 for figuring out this issue and the quick fix. I think the change looks good. Should we modify one integration test to have 2 schema versions and verify the data based on v2?

I think the unit test is sufficient for this specific issue, but will add one in case there are other issues with multiple value schemas.


Value is being cached

Hi,
I have a strange problem, I'm not sure if it is my code or what but whenever I try to clear the value of the inputMask it always returns the old value. I clear it by setting the property bound to it to null, it does clears the input but as soon as I enter a new value the old one comes back. I hope I make sense.

Hello,
Could you send me a code exerpt you used to achieve this ? You should reset the underlying model by setting it to either null or string.Empty...

This is how we define the control

This is the ValueChanged()

And this is how we set the model to empty strings.

Do you think changing it to string.Empty() would make a difference instead of using " "?

Hello,
Why don't you use an InputForm with an EditContext like this sample: https://github.com/raphadesa/BlazorInputMask/blob/master/BlazorInputMask/Pages/Index2.razor  ?
I haven't tested the component with your approach.

Hi,
Even if I do it that way I still get the same issue, it's like the value is being cached somewhere for few minutes. Maybe the problem is with my code.
Thanks,

Hi,
I have made few changes to you code, now everything is working. Do you mind if I push my changes?


Add ability to support custom api and console ports

What does this PR do?
Updated heat templates to accommodate customized master API ports.
How should this be manually tested?
Add the following to inventory/group_vars/all.yml:
openshift_master_api_port: 443
openshift_master_console_port: 443

And provision an Openstack cluster
Is there a relevant Issue open for this?
n/s
Who would you like to review this?
cc: @tomassedovic @bogdando @oybed

Looks good to me! The end to end CI is busted now. I'll run it after we've sorted it out and we can merge it afterwards.

@tomassedovic bump. any progress on CI?

Right, sorry!
The end to end CI's running here: https://travis-ci.org/openshift/openshift-ansible-contrib/builds/274649465
I'll merge this when it passes.


Checker not working on Waypoint: Target the same element with multiple jQuery Selectors

Challenge http://www.freecodecamp.com/challenges/waypoint-target-the-same-element-with-multiple-jquery-selectors has an issue. Please describe how to reproduce it, and include links to screenshots if possible.

Checker is not accepting #target1 even though jQuery is correct:

Wrap your classes in quotes, i.e. .addClass("animated")

@jbmartinez, thanks for the help.
when we add the quotes, the activity works for "btn-primary" even though the exercise calls for #target1 to use animated, shake, and btn-primary.

I've already created a pull request ( #1995) for that issue, @jentrification, hopefully it get merged soon. Thanks for reporting

Suggestion below is for modifying learner instructions.
Current - Now you know three ways of targeting elements: by type: $("button"), by class: $(".btn"), and by id $("#target1").
Suggested - Now you know three ways of targeting elements! Elements can be targeted by element type: $("button"), by class: $(".btn"), and by id $("#target1").
Current - Use each of these jQuery selectors to target your button element with the class btn and the id target1.
Suggested - Use each of these jQuery selectors to target your button element, the class btn, and the id target1.
Current - Use the addClass() jQuery function to give the element one new class for each selector: animated, shake, and btn-primary.
Suggested - Use the addClass() jQuery function to give the button element one new class through each selector. The classes to be used are animated, shake, and btn-primary.
The highlighted task below needs to be deleted to agree with current and suggested instructions.

Looks like this has been fixed! Closing.

THANK YOU TanaLandis! I was confused about what exactly I was supposed to do on this WayPoint according to the "Current" instructions. Your "Suggested" instructions made it much more clear.

@otherholli Glad they could help! Kudos to you for searching and finding what you needed :).

Can someone give an example how this is done. I seemed to have done the same thing, and I can't get the code to go thru.

@creator2earth There is no bug. Solution below in this post.
@jbmartinez or @benschenker or @whoevermightbeinterested in improving user-experience of Challenge #115
Still quite a confusing challenge for the combination task-checkpoints.
Description of the task leaves you without any example, which is nice to start thinking, but previous tasks got us used to copy examples.
It took me quite a while to figure out what my mistake was.(used . instead of - in btn-primary)
Checkpoint 5 red made my doubt whether I understood the challenge.
Suggestion: "Your #target1 button should show animated, shaking and blue."
Correct=
<script>
$(document).ready(function() {
$("button").addClass("animated");
$(".btn").addClass("shake")
$("#target1").addClass("btn-primary");
});
</script>

Finally passed this level after three days of research..Good job to the team..it was a very simple error..

Hi, Could you explain this problem?
My code was like this but it was not working until when i made copy of LisB2016's code. What is it? What happened here? Should i make copy paste? I left my code here. Here is no difference at all.

This challenge is really confusing. I thought they meant to add all the functionality in a single line.
But What I noticed was they tried to explained how the element inhered the class. For example the first line
$("button").addClass("animated");  Since all the button element received the class animated, it is not necessary to put the class again in the next line because it is already assigned to button.
$(document).ready(function() {
    $("button").addClass("animated");
    $(".btn").addClass("shake");
    $("#target1").addClass("btn-primary");
});

hi

i too have the same problem

<script>
  $(document).ready(function() {
  $("button").addclass("animated");
  $(".btn").addclass("shake");
  $("#target1").addclass("btn-primary");

  });
</script>

please help me

@TanaLandis got correct how tell us i can't nderstand

i figured out capitalise C in Class

@lokirajagopal
Please use the HelpJavaScript chat room for getting challenge related help.
Happy Coding.


Add Unicode support to String.to_*_case() methods

I think this is necessary because GDScript supports Unicode identifiers (#71676) and we have the editor/naming/default_signal_callback_name setting that should also support Unicode (#75814).

Add is_unicode_upper_case() and is_unicode_lower_case() functions.
Optimize is_unicode_identifier_start() and is_unicode_identifier_continue() functions by using binary search.

Thanks!


Custom Validator Duplicate Entry Error when Resubmitting the Form :)

Hi Saddam,
I am trying to create a custom validation rule. It works great when I submit the form first time. But after changing the value for form field when I resubmit the again it shows me:
 http: panic serving [::1]:57565: validator: password_confirmed is already defined in rules
goroutine 26 [running]:
net/http.(*conn).serve.func1(0xc4203b6000)
        /usr/local/go/src/net/http/server.go:1697 +0xd0
panic(0x13ecf40, 0xc420268c80)
        /usr/local/go/src/runtime/panic.go:491 +0x283
github.com/thedevsaddam/govalidator.AddCustomRule(0x1473f19, 0x12, 0xc420268c60)
        /Users/rbrahul/go/src/github.com/thedevsaddam/govalidator/rules.go:22 +0x154
main.createUser(0x1671740, 0xc42039a0e0, 0xc42026bd00)
        /Users/rbrahul/Projects/go/src/users/userHandlers.go:364 +0xca
net/http.HandlerFunc.ServeHTTP(0x148b688, 0x1671740, 0xc42039a0e0, 0xc42026bd00)
        /usr/local/go/src/net/http/server.go:1918 +0x44
github.com/gorilla/mux.(*Router).ServeHTTP(0xc42031e000, 0x1671740, 0xc42039a0e0, 0xc42026bd00)
        /Users/rbrahul/go/src/github.com/gorilla/mux/mux.go:159 +0xed
net/http.serverHandler.ServeHTTP(0xc420296a90, 0x1671740, 0xc42039a0e0, 0xc42026bb00)
        /usr/local/go/src/net/http/server.go:2619 +0xb4
net/http.(*conn).serve(0xc4203b6000, 0x1671e00, 0xc420259dc0)
        /usr/local/go/src/net/http/server.go:1801 +0x71d
created by net/http.(*Server).Serve
        /usr/local/go/src/net/http/server.go:2720 +0x288

And here is my source code:
...
govalidator.AddCustomRule("password_confirmed", func(field string, rule string, message string, value interface{}) error {
		fmt.Println("Val ", req.PostFormValue(field))
		if req.PostFormValue(field) != req.PostFormValue("password") {
			return fmt.Errorf("Field %s is miss matched with password", field)
		}
		return nil
	})
...

@rbrahul Please add the custom rules inside init function or inside sync.One so that same rule does not register twice, I think you already figured it out bhai :)


Pybind11

Basic pybind11 functionality added to a few classes in refspec and it works. There is a simple_demo Python driver which appears to produce results identical to C++. Plenty of stuff to review still but this is a good base to go forward with.

Getting closer, but following instructions it still doesn't quite work for me:
/usr/bin/python3.10: No module named pybind11
Makefile:49: warning: overriding recipe for target 'src/pfb.o'
Makefile:45: warning: ignoring old recipe for target 'src/pfb.o'
Makefile:49: warning: overriding recipe for target 'src/SpecConfig.o'
Makefile:45: warning: ignoring old recipe for target 'src/SpecConfig.o'
Makefile:49: warning: overriding recipe for target 'src/SpecOutput.o'
Makefile:45: warning: ignoring old recipe for target 'src/SpecOutput.o'
Makefile:49: warning: overriding recipe for target 'src/SignalGenerator.o'
Makefile:45: warning: ignoring old recipe for target 'src/SignalGenerator.o'
Makefile:49: warning: overriding recipe for target 'src/RefSpectrometer.o'
Makefile:45: warning: ignoring old recipe for target 'src/RefSpectrometer.o'
g++ -c -fPIC -g -std=c++17 -Iinclude -Iextern  src/refspec.cpp -o src/refspec.o
src/refspec.cpp:17:10: fatal error: pybind11/pybind11.h: No such file or directory
   17 | #include <pybind11/pybind11.h>
      |          ^~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make: *** [Makefile:50: src/refspec.o] Error 1
anze@kosovel: ~/work/lusee/refspec [16462] ❱❱ ls extern/pybind11/
CMakeLists.txt  docs  include  LICENSE  MANIFEST.in  noxfile.py  pybind11  pyproject.toml  README.rst  setup.cfg  setup.py  tests  tools


Always return the raw table through the rest api

The table property was really intended for internal use.  This renames properties raw_table to table and table to indexed_table on the csv_file model.  As a result, the table returned through the REST API for the cleanup UI will always be the same.
Fixes #54

@jbeezley thanks!


[packages][ci] Tests running both on main and master for the same hash

It seems that we're running tests both in main and master, and the Github UI is getting updated TWICE (once with the successful run, then with the results of the run in the second branch. The first run logs are lost!).
Some "proof" here:

https://cirrus-ci.com/task/6591136247578624 (master)
https://cirrus-ci.com/task/6243828481916928 (main)

I only managed to get those two links because I saw the test restart in real-time, and I had already gotten the first link to complain in discord about being throttled. THE GITHUB UI GETS COMPLETELY REWRITTEN in main, and it seems that tests begin again.

🚨 Crazy status right now!

(Note: release having completed before android-blah, CaseyHillers' commit being merged without the previous tests are 🟢)

Some speculation:
May this be the root cause of the "release" action some times running without waiting for all tests to complete? (because they completed the first time, but not the second time!), also (maybe) why we were being throttled today?

Possibly caused by this: https://github.com/flutter/flutter/issues/100557

Should we exclude the flutter/plugins:master branch from the .cirrus.yml file, somehow?

(I'd maybe P2 this thing 😱)

There's some environment variables set by Cirrus that can be used to skip/noop flutter/packages|plugins:master:
$CIRRUS_REPO_OWNER (in our case would be flutter)
$CIRRUS_BRANCH
And this: https://cirrus-ci.org/guide/writing-tasks/#conditional-task-execution (only_if?)
(This is a band-aid, ideally Cirrus shouldn't look at master at all!)

(This is a band-aid, ideally Cirrus shouldn't look at master at all!)

I believe the expected behavior is that it runs on all branches by default. Otherwise things like release branches, PRs, etc. wouldn't get tests. I just forget that when it comes to things like having two identical branches...

@stuartmorgan I've just marked the flutter/packages as waiting for tree to go green, it should land shortly. Very cool link: https://cirrus-ci.com/github/flutter/packages ! Cirrus remembers! :P

flutter/packages and flutter/plugins seem to have recovered. Their release action seems to work too. Thanks for the super quick fix @stuartmorgan!!!

Fixing it was easy once we knew what was going on; you get all the credit here for figuring out why our test results have been acting strange. And halving the number of CI resources we are spending!

Tests are still being run multiple times, but now it's a little bit more rare than before. They're correctly not running for master, but they run for main, AND then again for the tag that gets pushed by the release action (if any).


Update to 1.0.0-rc.3 protos

Update to 1.0.0-rc.3 protos
Verified with local cargo build and cargo build --examples

@gdhuper Can you please review this pr ?

@gdhuper Can you please review this pr ?

@gdhuper Can you please review this pr ?

Will do.

@gdhuper Can you please review this pr ?

Will do.

@gdhuper RFAL!

@gdhuper RFAL!

RFAL

LGTM!

RFAL

LGTM!


D3.js Support

is it support d3.js? Or any ways to use it with d3.js.

Sorry I haven't used d3.js, so I can't say how straightforward it would be.


Coverage >= 5 cannot handle Cython public or readonly members in .pxd file

Describe the bug
After upgrading to coverage >= 5,
coverage run outputs this error message for a Cython pxd file with certain conditions I describe below:
Can't add file tracer data for unmeasured file '/home/kai/tst/cy/kai.pxd'

Then coverage report fails with this error:
Couldn't parse '/home/kai/tst/cy/kai.pyx' as Python source: 'invalid syntax' at line 1

To Reproduce

What version of coverage.py are you using? The output of coverage debug sys is helpful.

-- sys -------------------------------------------------------
                        version: 5.1
                       coverage: /home/kai/tst/lib64/python3.7/site-packages/coverage/__init__.py
                         tracer: -none-
                        CTracer: available
           plugins.file_tracers: Cython.Coverage.Plugin
            plugins.configurers: -none-
      plugins.context_switchers: -none-
              configs_attempted: .coveragerc
                                 setup.cfg
                   configs_read: /home/kai/tst/setup.cfg
                    config_file: /home/kai/tst/setup.cfg
                config_contents: '[coverage:run]\n# http://blog.behnel.de/posts/coverage-analysis-for-cython-modules.html\nplugins = Cython.Coverage\n'
                      data_file: -none-
                         python: 3.7.6 (default, Feb 26 2020, 20:54:15) [GCC 7.3.1 20180712 (Red Hat 7.3.1-6)]
                       platform: Linux-4.19.76-linuxkit-x86_64-with-glibc2.2.5
                 implementation: CPython
                     executable: /home/kai/tst/bin/python3
                   def_encoding: utf-8
                    fs_encoding: utf-8
                            pid: 9474
                            cwd: /home/kai/tst
                           path: /home/kai/tst/bin
                                 /usr/lib64/python37.zip
                                 /usr/lib64/python3.7
                                 /usr/lib64/python3.7/lib-dynload
                                 /home/kai/tst/lib64/python3.7/site-packages
                                 /home/kai/tst/lib/python3.7/site-packages
                    environment: -none-
                   command_line: /home/kai/tst/bin/coverage debug sys
                sqlite3_version: 2.6.0
         sqlite3_sqlite_version: 3.7.17
             sqlite3_temp_store: 0
        sqlite3_compile_options: DISABLE_DIRSYNC
                                 ENABLE_COLUMN_METADATA
                                 ENABLE_FTS3
                                 ENABLE_RTREE
                                 ENABLE_UNLOCK_NOTIFY
                                 SECURE_DELETE
                                 TEMP_STORE=1
                                 THREADSAFE=1

What versions of what packages do you have installed? The output of pip freeze is helpful.

coverage==5.1
Cython==0.29.16

Tested with coverage 4.5.4 (OK), 5.0.4 (fail), 5.1 (fail)

What code are you running?  Give us a specific commit of a specific repo that we can check out.
cy/__init__.pxd (empty file)

cy/kai.pxd
cdef class Kai:
    cdef readonly bint x

    cpdef bint get_x(self)

cy/kai.pyx
cdef class Kai:
    def __init__(self):
        self.x = False

    cpdef bint get_x(self):
        return self.x

cy/setup.py
from Cython.Build import cythonize
from distutils.core import setup
from distutils.extension import Extension

define_macros = [('CYTHON_TRACE', '1')]

compiler_directives = {'always_allow_keywords': True, 'profile': True, 'linetrace': True}
extensions = [
    Extension('cy.kai', ['cy/kai.pyx'], define_macros=define_macros),
    ]
setup(ext_modules=cythonize(extensions, nthreads=4, annotate=True, language_level="3", compiler_directives=compiler_directives))

runner.py
from cy import kai

k = kai.Kai()
print(k.get_x())
print(k.x)

What commands did you run?

$ python3 cy/setup.py build_ext --inplace
$ coverage run runner.py
$ coverage report

Expected behavior
Coverage statistics for pyx and pxd files
Additional context
The problem manifests itself only when

coverage >= 5
public or readonly members are in .pxd file, separate from .pyx file, and
runner.py actually accesses the members directly

My understanding is readonly and public modifiers make Cython define wrapper functions for them transparently. coverage >= 5 seem to play badly with them.

Thanks, I see the error from "coverage report", but I don't see the message you show for "coverage run".  Do you have any other context I need?

I got it from the "debug sys" output you showed: you have a .coveragerc file with this:
[coverage:run]
# http://blog.behnel.de/posts/coverage-analysis-for-cython-modules.html
plugins = Cython.Coverage

Now I see the same results you do.

xref cython/cython#3515 tentative workaround is to add a dummy inline function in the affected .pxd file

The best workaround I could find for this was to edit the .coveragerc file adding the relative path of all Cython .pxd files in the list of files to be ignored by coverage. More details on https://github.com/saullocastro/composites:
[run]
plugins = Cython.Coverage
source = composites
omit =
    composites/core.pxd

I was able to bisect the commit of the issue: https://github.com/nedbat/coveragepy/commit/106828c2cc8bbce1e5fb31c6a89ea3ac025225c1#. The root cause is following: reproducer described in this issue is yielding following data in parameter d:
https://github.com/nedbat/coveragepy/blob/5ec587caf0bd670a0af34ad5b766119689b91b56/coverage/collector.py#L417
{
    '/Users/matus/dev/cython/test/coverage/runner.py': {1: None, 3: None, 4: None, 5: None},
    '/Users/matus/dev/cython/test/coverage/cy/kai.pyx': {5: None, 1: None, 16: None, 11: None, 3: None, 6: None},
    '/Users/matus/dev/cython/test/coverage/cy/kai.pxd': {}
}

It can be seen that kai.pxd file is mapped to empty dictionary. This is obviously filtered in following line: https://github.com/nedbat/coveragepy/blob/5ec587caf0bd670a0af34ad5b766119689b91b56/coverage/collector.py#L434
This is causing that kai.pxd is not inserted in database here:
https://github.com/nedbat/coveragepy/blob/5ec587caf0bd670a0af34ad5b766119689b91b56/coverage/sqldata.py#L383-L394
When in the later stage the data needs to be fetched from database, the final error is raised due missing record in DB:
https://github.com/nedbat/coveragepy/blob/5ec587caf0bd670a0af34ad5b766119689b91b56/coverage/sqldata.py#L549-L553
As a proof of concept I have created a PR which seems to fix the issue. I am not familiar with coverage codebase so I cannot tell if this is proper fix or the issue needs to be fixed somewhere else.

This is now released as part of coverage 6.4.3.

This is now released as part of coverage 7.1.0.


Initial attempt at efficient range queries across multiple Set objects.

This is not compiling yet (using rustc 1.24.0-nightly (0a3761e63 2018-01-03)):
(dblewett)Wed Jan 10, 10:03 | /home/dblewett/src/python-rust-fst/fstwrapper
rusty% cargo build
 Compiling fst-wrapper v0.3.0 (file:///home/dblewett/src/python-rust-fst/fstwrapper)
error[E0277]: the trait bound `for<'a> &fst::set::Stream<'_>: fst::Streamer<'a>` is not satisfied
 --> src/set.rs:152:36
  |
152 |     let ob = set::OpBuilder::new().add(stream);
  |                                    ^^^ the trait `for<'a> fst::Streamer<'a>` is not implemented for `&fst::set::Stream<'_>`
  |
  = help: the following implementations were found:
            <fst::set::Stream<'s, A> as fst::Streamer<'a>>

error[E0277]: the trait bound `for<'a> &fst::set::Stream<'_>: fst::Streamer<'a>` is not satisfied
 --> src/set.rs:168:8
  |
168 |     ob.push(stream);
  |        ^^^^ the trait `for<'a> fst::Streamer<'a>` is not implemented for `&fst::set::Stream<'_>`
  |
  = help: the following implementations were found:
            <fst::set::Stream<'s, A> as fst::Streamer<'a>>

error: aborting due to 2 previous errors

error: Could not compile `fst-wrapper`.

@jbaiter @BurntSushi Sorry this got delayed so long. I have this working now.

The tests pass for me (cPython 3.7, Ubuntu 18.04.1):
(fst-test)
(dblewett)Wed Sep 12, 15:46 | /home/dblewett/src
rusty% py.test python-rust-fst/tests
==================================================================================================================== test session starts ====================================================================================================================
platform linux -- Python 3.7.0b3, pytest-3.8.0, py-1.6.0, pluggy-0.7.1
rootdir: /home/dblewett/src/python-rust-fst, inifile:
collected 47 items

python-rust-fst/tests/test_map.py ..................                                                                                [ 38%]
python-rust-fst/tests/test_set.py .............................                                                                                [100%]

================================================================================================================= 47 passed in 0.18 seconds =================================================================================================================```

@jbaiter, @BurntSushi  what do you think of this? I wasn't sure about the UnionSet name.

@jbaiter is there interest in including this in this package? If not the entire PR, maybe the supporting C functions? it would be cumbersome to do completely outside this package.


feat: add command Copy Styles with Clases From Template

to bootstrap  content using for all classes from

@ILScc lgtm


[WIP] run realy wal p0 test

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


[FALSE-POSITIVE]  www.glassdoor.ca glassdoor.ca

Domains or links
www.glassdoor.ca glassdoor.ca
More Information
How did you discover your web site or domain was listed here?
Could not access website
Have you requested removal from other sources?
No
Additional context
This domain is a employer review site
:exclamation:
We understand being listed on a Phishing Database like this can be frustrating and embarrassing for many web site owners. The first step is to remain calm. The second step is to rest assured one of our maintainers will address your issue as soon as possible. Please make sure you have provided as much information as possible to help speed up the process.
Send a Pull Request for faster removal
Users who understand github and creating Pull Requests can assist us with faster removals by sending a PR on the whitelist.me file

https://github.com/mitchellkrogza/Phishing.Database/blob/master/whitelist.me/whitelist.me
Please include the same above information to help speed up the whitelisting process.

@mitchellkrogza @funilrys looks like this is still blocked. Do I need to review which other of my submitted tickets were reintroduced?

Added to my backlog. It's probably a sync issue with @Ultimate-Hosts-Blacklist.

@funilrys  @mitchellkrogza
It is in hosts1 file. It is not in phishing-database-ACTIVE.txt

@funilrys @mitchellkrogza this is still being blocked


utils/update-checkout: Rework for more parallelism in updates

Reworks the update_all_repositories() and update_single_repository()
functions to benefit from more per-repo parallelism.

Branch-scheme search loop is relocated from update_repository_to_scheme()
to update_all_repositories().
Functions update_repository_to_tag() and update_repository_to_scheme()
are removed.
Per-repo work of these functions is shifted to update_single_repository().
Repeated work (repeated pushd's, etc.) has been eliminated.
As much work as possible is performed within update_single_repository()'s
outer try clause to catch more errors.
Arguments passed to update_single_repository() have been changed
appropriately.
New helper functions confirm_tag_in_repo() and get_branch_for_repo() added
to keep update_single_repository()'s length under control.
Unnecessary setting of the cross_repo flag by the --tag argument has been
removed, so repos which lack the tag are now properly rebased when updated.

In PR #7181 (which was eventually rejected in favor of a different patch), @erg suggested that I look into further parallelizing update-checkout's --tag option, since it still did some work in serial form. When I looked further, I saw the same was true for --scheme and cross-repo-PR processing, and @erg encouraged me to parallelize everything that made sense. I decided to rework repository updating, keeping all the common work in update_all_repositories() and performing all the per-repo work in update_single_repository() or subfunctions called from it. This allowed for some simplification and reduction of repeated work. I also took the opportunity to remove the setting of the cross_repo flag from --tag processing, since @erg had said it was only a workaround for rebasing bugs, and PR #7232 (now merged) is a better solution.
The number of arguments passed to update_single_repository() has increased from five to eight; this was necessary to get all the needed info into the reworked function without repeating work or resorting to making some variables (like config) global.
This is obviously a significant patch. I've been using it on my own system for about a day to perform repo updates both from the main repo and my private fork, also exercing various functions like --tag, --scheme, --reset-to-remote, --clean, and cross-repo checkout with --github-comment. I haven't seen any problems.
Along with @erg, I'd also like @gottesmm and @shahmishal to critique this, since it looks like update-checkout was largely made by you two, and since @shahmishal seems to be in charge of the CI testing infrastructure which uses it.

@swift-ci Please smoke test

@colinhowell Awesome. This works really well and the Linux test failure is obviously unrelated. Merging!

@erg Wow, thank you! Any other comments? Any feedback at all is welcome.


Remove x-powered-by header

Proposed Changes

add helmet as dependency
use helmet.hidePoweredBy() to remove the x-powered-by header

Screenshot (If Applicable)
Checklist

[x] Tested locally
[x] Ran yarn ci to test my code
[x] Did not add any unnecessary changes
[x] All my changes appear after other changes in each file
[x] Included a screenshot (if adding a new button)
[x] 🚀

Thank you!


Specification Clarity

I've been asked to implement JSONAPI on a project that I'm working on. Reading of the specification has raised a few questions for me.

What am I supposed to do with unknown parameters?
I asked around the office and the interpretation varies. If I send a request with correct page[number] params, but throw in another one foo=true, should I be 400-ing?

Pagination - Requesting Pages out of Range
Say I have 1000 records. I request page[number]=1002, page[size]=100, what should I be doing here?
Should I 400? Should I send back an empty data: []  section? If I return an empty data array, how should I format the links section.

Pagination - Requesting the First and Last Page
What should the links section look like?
If I request the first page, should I just omit the prev section from links, or should I include it, and leave it empty?

What am I supposed to do with unknown parameters? If I send a request with correct page[number] params, but throw in another one foo=true, should I be 400-ing?

interp is up to you. It could be a parameter error or not http://jsonapi.org/examples/#error-objects-invalid-query-parameters

Should I send back an empty data: [] section? If I return an empty data array, how should I format the links section.

empty data would be correct semantics, in my opinion. links should probably be the same, no? first, last and anything else that is useful

If I request the first page, should I just omit the prev section from links, or should I include it, and leave it empty?

I believe that is up to you. None of the links are required. It's just a spec for their format

:+1: to everything @bf4 said.
Re:

If I request the first page, should I just omit the prev section from links, or should I include it, and leave it empty?

I would omit the prev link altogether.


Using Dropdown only, without input

Hello,
I have a custom component that works as input but it's not a common input field. It is a React component based on react-tagsinput.
I'd like to use this component and then show a dropdown menu like react-autosuggest, probably using Redux. But I can't find a way to inject data and make the dropdown suggestions appear, only if I use the input field. Could you give me any directions?
I've tried the redux demo on codepen but i'm not sure it is injecting data from outside the component
Thanks!

I've solved it. It's easier than I thought, it's just a matter of setting  alwaysRenderSuggestions={true} and creating the proper list of suggestions . So whenever I receive a query I do this:
this.setState ({             value: storeObj.query,             suggestions: getSuggestions(storeObj.query)         });
and I don't need to use the input field


Adding methods for interconverting wavelength and energy

This PR will add two new methods to rs.utils, ev2angstroms and angstroms2ev.

Codecov Report

Merging #58 (9a96de8) into main (358d229) will decrease coverage by 0.06%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##             main      #58      +/-   ##
==========================================
- Coverage   98.78%   98.72%   -0.07%
==========================================
  Files          36       37       +1
  Lines        1482     1488       +6
==========================================
+ Hits         1464     1469       +5
- Misses         18       19       +1

Flag
Coverage Δ

unittests
98.72% <83.33%> (-0.07%)
:arrow_down:

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

Impacted Files
Coverage Δ

reciprocalspaceship/utils/units.py
80.00% <80.00%> (ø)

reciprocalspaceship/utils/__init__.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

in gemmi the same is done by multiplying/dividing by gemmi.hc.
Checking the exact value of the constant, it's the same up to 9th significant digit:
12398.41984332
12398.41973862
which I suppose is perfectly fine. I calculated the constant with Linux program units:
units -d15 'h * c / eV / angstrom'


Improvements to the Rel transpiler

Various improvements to the Rel transpiler:

Add support for the restrict operator, ensuring that a restrict following join or project does not materialize unnecessary columns in intermediate relations in the transpiled Rel query.
Improvements to the way that the schema is constructed. These changes should make it easier to ensure that the required classes are added to the schema before the query can be transpiled to Rel. These changes will make it easier to transpile queries which use match.
Improved formatting of the output Rel query for better readability.
Improved debugging and logging in the case of errors during query compilation.
Added test cases.
Property mappings are now correctly used by the transpiler to map onto Rel relations corresponding to properties, whereas previously only class mappings were used.
Fix bug in handling of groupby.
General improvements to the generated Rel, for example, improved handling of isEmpty().
Miscellaneous code cleanup and bug fixes.

This PR builds on the work of https://github.com/finos/legend-engine/pull/2622, and therefore that PR must be merged first.
What type of PR is this?
Improvement
What does this PR do / why is it needed ?
Support for more types of query by the Rel backend. This PR also improves the logging of errors when a Pure query is not supported by the Rel transpiler.
Does this PR introduce a user-facing change?
This PR is backwards compatible in the sense that existing queries will continue to be supported without any changes.
Some function names related to schema creation have changed. See test cases for examples.

This PR can be closed because it is superseded by PR#2702 which has now been merged.

This PR is stale because it has been open for 30 days with no activity. Please remove stale label or add any comment to keep this open. Otherwise this will be closed in 5 days.

This PR was closed because it has been inactive for 35 days. Please re-open if this PR is still relevant.


dealing with tests that cannot succeeded

Sometimes tests cannot be fixed quickly and you expect to fail. In such cases it's a common practice to mark them accordingly with statuses like XFail or Skip.
A Skip means that you expect your test to pass unless a certain configuration or condition prevents it to run. And XFail means that your test can run but you expect it to fail because there is an implementation problem.
It would be nice to have functionality to set certain test status in a test source code.

you can use tags and then include/exclude those tags based on the conditions in which you run the tests.
But I might not understand your request exactly...

@Tieske filtering using tags excludes test from a test report and it is not desired.
see how this functionality implemented in pytest -
https://docs.pytest.org/en/latest/how-to/skipping.html

Here's how we do stuff like that (untested):
local platform_it = function(platforms, description, ...)
    if type(platforms) ~= "table" then
        -- plain 'it' call
        return it(platforms, description, ...)
    end

    local platform = get_platform()
    local test = false
    for _, plat in ipairs(platforms or {}) do
        if plat == platform then
            test = true
            break
        end
    end
    return test and it(description, ...) or pending("[skipping on "..platform.."] "..description, ...)
end

platform_it({ windows, osx }, "a test as usual", function()
    -- test something, only on Windows and OSX, not on Linux
end)

platform_it({ osx, linux }, "another test as usual", function()
    -- test something, only on OSX and Linux, not on Windows
end)

The important aspect of an xfail test is that it still runs but it's expected to fail.
This is useful to document the expected behavior for a scenario that's know to be failing (e.g., a bug report) but hasn't been fixed yet. If something changes that fixes the test, you're alerted to it because the test passing is treated as a failure.
At that point, you can verify whether the behavior change is intended and simply switch it from "xfail" to a normal test.

We might be able to extend the reporting functionality to report on excluded tests maybe? Tags exist specifically so skip, xfail, etc can all be handled the same way anyways

I don't think reporting on excluded tests answers this question, the point of an Xfail test is that they are included but the mode is reversed. They are not excluded, they are run, but the expected output is anything but the declared expectation. That way you sound an alarm if a know-broken test starts passing and you fixed a bug you didn't realize was being affected (for the better).
I don't see way to do that with the tag system. We can include and exclude but not reverse modes.


Support for Lumen?

Any plan to support for Lumen?
I just made a few change at the LandlordServiceProvider.php, but I do not know if this is the lumen way or not.. Seems it works for me...
Here is what I've done so far:
<?php

namespace HipsterJazzbo\Landlord;

use HipsterJazzbo\Landlord\Facades\LandlordFacade;
use Illuminate\Contracts\Foundation\Application;
use Illuminate\Foundation\Application as LaravelApplication;
use Illuminate\Foundation\AliasLoader;
use Illuminate\Support\ServiceProvider;
use Laravel\Lumen\Application as LumenApplication;

class LandlordServiceProvider extends ServiceProvider
{
    /**
     * Bootstrap the application events.
     *
     * @return void
     */
    public function boot()
    {
        // setup config
        $source = realpath(__DIR__ . '/../config/landlord.php');

        if ($this->app instanceof LaravelApplication && $this->app->runningInConsole()) {
            $this->publishes([$source => config_path('landlord.php')]);
        } elseif ($this->app instanceof LumenApplication) {
            $this->app->configure('landlord');
        }
    }

    /**
     * Register the service provider.
     *
     * @return void
     */
    public function register()
    {
        $this->app->singleton(Landlord::class, function () {
            return new Landlord();
        });

        // Define alias 'Landlord'
        if ($this->app instanceof LaravelApplication && $this->app->runningInConsole()) {
            $this->app->booting(function () {
                $loader = AliasLoader::getInstance();

                $loader->alias('Landlord', LandlordFacade::class);
            });
        }
    }
}

If you open a PR with passing lumen tests, I'll merge it. Otherwise I'm afraid it might be a while before I can tackle it myself.

Try the v2.0-wip branch. There are still a few tests missing, and I haven't yet had time to document the breaking changes, but just go through the readme and you should be okay. Mostly some classes and methods have been renamed to more closely match what they're actually doing now.
Should fully support Lumen now. Feel free to open a new issue/pr is you try it and it doesn't work, but like I said, it's not quite done yet.


Can you do new demo for image picker latest version and send with multipart form data to server. Because no demo for that.

Make sure to check the demo app(s) for sample usage
Make sure to check the existing issues in this repository
If the demo apps cannot help and there is no issue for your problem, tell us about it
Please, ensure your title is less than 63 characters long and starts with a capital
letter.
Which platform(s) does your issue occur on?

iOS/Android/Both
iOS/Android versions
emulator or device. What type of device?

Please, provide the following version numbers that your issue occurs with:

CLI: (run tns --version to fetch it)
Cross-platform modules: (check the 'version' attribute in the
node_modules/tns-core-modules/package.json file in your project)
Runtime(s): (look for the "tns-android" and "tns-ios" properties in the package.json file of your project)
Plugin(s): (look for the version numbers in the package.json file of your
project and paste your dependencies and devDependencies here)

Please, tell us how to recreate the issue in as much detail as possible.
Describe the steps to reproduce it.
Is there any code involved?

provide a code example to recreate the problem
(EVEN BETTER) provide a .zip with application or refer to a repository with application where the problem is reproducible.

Hi @saihtethtet, check out this playground sample app. To use it with multi-part, find the commented-out code in home.component.ts and uncomment it. Note that you should also remove the code that does the uploadFile() method call. Hope this helps!