Firmware modification not saving (v2.2): modified firmware exactly the same (size) as original

To illustrate the problem, I've removed mostly everything except 3 effects from original G3Xn firmware and saved it. The saved file is exactly the same as original file, i.e. no change is applied.
Please see the pic below.
What could cause this?

EDIT:
I think this has to do with fact that I'm trying it on firmware version 2.2.
It seems there's an issue with it, see the link: https://github.com/Barsik-Barbosik/Zoom-Firmware-Editor/issues/7

I think the file will remain the same size just like my experience.  I recommend you use another name (to make another file) as it may not overwrite the existing file being opened by the tool itself.  Then open the newly created file to validate if the changes you intended were saved.


Unable to view bonobo home page

I have installed Bonobo git server on my windows 8 machine.
however login/home page is not displayed on http://localhost/Bonobo.Git.Server/.  Rather page displays directory structure of bonobo.

According to this https://forums.asp.net/t/1282080.aspx?Why+am+I+getting+a+directory+listing+instead+of+my+site+ it could be a setting on the server. However I think its beyond Bonobo and therefore our limited free support :)
Hope you can find help elsewhere.


Improvement: Adding the registration form in the website

This is an issue that tracks the creation of a registration form in the website instead of keeping it native on the google forms.
This depends on #38

Closing this as it's too late now.

Closing this as it's too late now.


Ad requests do not load ads after second request

Hi!
I'm experiencing an issue after two successful ad requests. On the third (and any other subsequent) request, our adTagUrl returns a valid response, but the request hangs here: [135.019s] [ima.loader.SequentialAdsLoader] Starting request from queue (full debug below). Ad events aren't triggered, so nothing loads.
Our use case is rather complicated - we have 4 unique instances of VideoJS, each with its own instance of the IMA plugin. Before each new ads request (except the first), I call setContentWithAdTag(null, this.adTagUrl, false) on each plugin instance. Our adTagUrl changes slightly on each request, based on the current page's URL.
When I test our adTagUrl in the Video Suite Inspector, I get an ad every time. I'm sure the issue is with my implementation; I'm just having trouble figuring out why.
Fortunately, this doesn't affect content playback, but I do rely on ad events to display the right UI at the right time.
Do you have any ideas as to what might be causing this? Thank you in advance!
Best,
Abi
Debug of third request:
[ 55.808s] [ima.loader.AdsLoaderWrapper] Requesting ads using new ads loader.
videojs.ima.js:242  [ 55.809s] [ima.loader.AdsLoaderWrapper] requestAds, processing external request.
videojs.ima.js:242  [ 55.809s] [ima.loader.SequentialAdsLoader] Enqueued new request.
videojs.ima.js:242  [ 55.810s] [ima.loader.SequentialAdsLoader] Starting request from queue.
videojs.ima.js:242  [ 55.810s] [ima.loader.AdsLoaderImpl] creating instance of GptRequestActivity.
videojs.ima.js:242  [ 56.352s] [ima.loader.AdSourceFetcher] Creating VastAdSource.
videojs.ima.js:242  [ 56.353s] [ima.loader.VastAdSource] hasMoreAds, this.ads.length: 1 , this.currentAdIndex: -1
videojs.ima.js:242  [ 56.353s] [ima.loader.VastAdSource] getNextAd, this.currentAdSource: null
videojs.ima.js:242  [ 56.354s] [ima.loader.VastAdSource] hasMoreAds, this.ads.length: 1 , this.currentAdIndex: -1
videojs.ima.js:242  [ 56.355s] [ima.loader.VastAdSource] processNextAd_, currentAdIndex: 0, ads.length: 1
videojs.ima.js:242  [ 56.355s] [ima.loader.VastAdSource] processInlineAd, successCallback with 1 ads
videojs.ima.js:242  [ 56.356s] [ima.loader.VastAdSource] hasMoreAds, this.ads.length: 1 , this.currentAdIndex: 0
videojs.ima.js:242  [ 56.357s] [ima.loader.AdsLoaderWrapper] dispatchAdsManagerLoadedEvent_, ads.length: 1
videojs.ima.js:242  [ 56.358s] [ima.managers.AdsManagerFactory] createAdsManagerFromAds_, adType: video
videojs.ima.js:242  [ 56.360s] [ima.loader.SequentialAdsLoader] Starting request from queue.

HI there,
It sounds like you do indeed have a rather complicated use case :) Do you have a sample page I could take a look at to replicate this on my end?
Thanks
Shawn

@shawnbuso Thank you for your quick response! We've set up a test page here: https://www.earthcam.com/clients/google/pagetest.html?cam=tsrobo1
My implementation can be found in the script here: /js/earthcam/ecnplayerhtml5/js/player_ima.js?20171212a
An associated script that may help shed some light is here: /js/earthcam/ecnplayerhtml5/js/player.js?20171212a
The JS is minified, so my implementation is somewhat difficult to debug. Do you have an email address that I could send the relevant unminified JS to?
Thank you!
Abi

Sure thing - you can use<EMAIL_ADDRESS>to send us info you don't want to share publicly. We have a frontline team that fields questions there, so put a note in your message that you're already working with me and ask them to escalate to me.
Do you have repro steps for that page? When I first loaded it I got an ad, but if I refresh the page or switch streams I don't see any additional ad requests going out.

@shawnbuso Fantastic - I sent a note to the support mailbox.
We fixed some data settings on our side to force ads to always play - if you clear the cache/cookies and refresh, there should be an ad request every time you switch streams. Please let me know if you need anything else; thank you again for being so responsive!

Sounds good - I got your email so I'll follow up there :)

Hi,
we are facing the same issue in one of our players.
The player stops at "Starting request from queue.", without any ad_error or content_resume_requested event.
We having this issue on iOS >= 10 and only with a couple of ADs and is working fine on android and desktop browser.
I can provide a testpage by mail if needed.
Thanks
Sebastian

If you could send a test page to<EMAIL_ADDRESS>that would be great, thanks :) Let them know you're working with me via the issue tracker and they'll make sure it gets to me.

Hey there,
I was taking a look at this again today, and it looks like your test page may be having issues - when I load it up the player loads forever and I get a few errors in the console:
events.event.class.min.js?20171212a:5 TypeError: this._videoJS360PanoPlayer1.panorama is not a function
    at window.earthcam.Player.initializePlayer (player.js?20171212a:318)
    at window.earthcam.Player.windowMountedHandler (player.js?20171212a:157)
    at Array.e (jquery.3.2.1.min.js?20171212a:2)
    at a.dispatchEvent (events.event.class.min.js?20171212a:4)
    at window.earthcam.VideoWindow.render (videowindow.js?20171212a:154)
    at window.earthcam.VideoWindow.init (videowindow.js?20171212a:77)
    at window.earthcam.PlayerArea.initVideoWindow (mediaplayercomponent.js?20171212a:233)
    at window.earthcam.PlayerArea.playerChangeHandler (mediaplayercomponent.js?20171212a:163)
    at Array.e (jquery.3.2.1.min.js?20171212a:2)
    at a.dispatchEvent (events.event.class.min.js?20171212a:4)

and
events.event.class.min.js?20171212a:5 TypeError: Cannot read property 'paused' of undefined
    at window.earthcam.Player.volumeChangeHandler (player.js?20171212a:1046)
    at Array.e (jquery.3.2.1.min.js?20171212a:2)
    at a.dispatchEvent (events.event.class.min.js?20171212a:4)
    at window.earthcam.VolumeSlider.dispatchControlEvent (volumeslider.js?20171212a:169)
    at window.earthcam.VolumeSlider.<anonymous> (volumeslider.js?20171212a:96)

Do you have a working page I can take a look at?
Thanks!
Shawn

It's been a while since we've heard from you, so I'm going to close this out. If you're still seeing issues feel free to respond here and we'll re-open it. Thanks!


setting a connectedClass breaks dropping

We have 2 arrays. You can drag items from one to the other if a condition is satisfied. The condition is checked in beforeMove; it it is not satisfied, cancelDrop will be set. Works fine.
If I set the connectedClass option on the first array, and specify this class in my css file,
the background of the table column that holds the array list, is now reformatted according
to the css. However, it's not possible to drop elements onto the 2nd array anymore.
The css class is simply:
.connectedSortable {
background:brown;
}
In my opinion, setting a css class should not change the behaviour of the page. Only javascript should change the behaviour.
what are we missing?

@maz4focus - are you still having this issue?  can you put together a jsfiddle or describe your code a bit more?


Autologin for just created users closes #4180

@Kartones , @viddo , what do you think about this approach for autologin at organization signup?
Some context:
Normal, Central signup direct authentication works because it can authenticate before actual user creation: a Central-created user is always valid. If creation doesn't work because of whatever reason it doesn't mind, and if it works final redirection to dashboard will be authenticated thanks to Central cookie.
Box-centric signup, nevertheless, creates users that can't be authenticated in advance, because of the need for email validation (enable_account_token ) and because box-central communication might fail, so I think the polling for user creation state might be a good place for a one-shot authentication. Since it's performed through https and based on user creation uuid it can't be intercepted, and time constraint, enable_account_token and dashboard_viewed_at checks make it safer.

PD: I've been thinking about a way to do this and experimenting for a while, if you think this is fine I'll add some tests.
PD2: this should cover LDAP and any other authentication mechanism transparently without any additional effort.

Retest this, please.

As long as there is no way the user creation id can be stolen (e.g. sent by email as the token etc.) so an attacker might hijack the account upon provisioning ending, I'm more than fine :+1:

Ok, @Kartones , @viddo , now formal CR after tests :)

Looks legit, but I'm afraid I don't know too much about our current login system actually works to be able to say yes or no with confidence.

:+1:


Camera->MoveToView and Camera->FadeScreen interference

To reproduce:
1-add new flowchart
2-add command Fade screen (duration 1, alpha 1 wait until finished unchecked)
3-add command move to view (set a view, assign, etc)
expected behavior:
camera will fade and move to screen simultaneously
actual behavior:
only move to view will be executed.
Remarks:
-just to make sure disable move to view. Fade screen will be working again.
-after inspection, it is due to StopAllCoroutines in CameraManager.cs line 428.
One possible fix is, not to stop all coroutines, only stop the ones that are specific to Panning, not fading
Another fix is to separate these functionalities, so they don't interfere
Finally, another possible fix is to redesign the lerp in a way to allow for mixing.
Something like this:

remove StopAllCoroutines in CameraManager.cs line 428.

change PanInternal function in a way to add only the delta changes, instead of overriding all values:
    ```
if (camera != null)
          {
  			float deltaOrthographicSize = Mathf.Lerp (startSize, endSize, Mathf.SmoothStep (0f, 1f, t)) - camera.orthographicSize;
  			Vector3 deltaPosition = Vector3.Lerp (startPos, endPos, Mathf.SmoothStep (0f, 1f, t)) - camera.transform.position;
  			Quaternion deltaRotation = Quaternion.Inverse(camera.transform.rotation) * Quaternion.Lerp(startRot, endRot, Mathf.SmoothStep(0f, 1f, t));
  			camera.orthographicSize += deltaOrthographicSize;
  			camera.transform.position += deltaPosition;
  			camera.transform.rotation *= deltaRotation;

              SetCameraZ(camera);
          }

did not make a pull request yet, just to check if this would break anything elsewhere, as im new to the fungus project.

Hi iBicha, sorry for slow reply on this. It helps to add the bug label to bug reports so I can spot them easier.
This is some of the oldest (and tbh worst!) code in Fungus so any improvements would be appreciated. Your suggestions are all good approaches. You might also consider using the built in Lean Tween library to perform the tween.
Keep in mind an important constraint. It's really important to keep the default move behaviour and the properties for the Move To View and Fade To View commands consistent with the existing system. Basically, you can make improvements but it should still be very easy for someone to upgrade an existing project without breaking anything and for someone following an old tutorial video to use the new system without confusion.


Patch 1 - Enclosing various powershell or other commands w/code apostrophe

Module: various
Lab/Demo: various
Fixes # .
Changes proposed in this pull request:
Through various instructions documents Powershell or other commands are only indented, but not enclosed in a markdown structure denoting them as 'code' or as 'code blocks'.  It is desirable for ALHs consuming these instructs to have these code formatting items present in order that items that will be input into the VMs can be automated properly.  This is what I have modified and proposed in this pull request.
Note that in this case I used the simpler form of: (single-apostrophe enclosing command)
`example-powershell-command`

I used this in place of an actual code block (3-apostrophe enclosing commands(s)) since the vast majority of these commands were not multi-line.  If one has multi-line code blocks, one could also have chosen this as an equally valid way to fix for this issue:
```Powershell
sample-ps-command-line1
sample-ps-command-line2
```

this PR simply merged tags on PowerShell scripts to help ALH hosting.


PyJWT 2 breaks _create_token function

PyJWT has released version 2.0.0 (and newer versions), removing python2 and python < 3.6 compatibily.
This means the jwt.encode() function now returns a string. string type in python3 has no .decode() method.
This can be fixed 2 ways:

add <2 to the pyjwt in setup.py install_requires
fix the code to not run .decode() on the created jwt

Merged and distributed to pypi in 0.1.6. Thanks!


MSID: 88404 Version: 2 DOI: 2<IP_ADDRESS>4978

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=88404)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/88404/v2)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>4978)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/88404/v2/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/88404)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/88404

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/88404

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/88404/v2/88404-v2.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/88404)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

eLocations missing from reference list.


New feature: templating

It would be great to provide some templating (as the Twilio Serverless CLI provide templates / scaffolding for functions). I created an example of that on https://github.com/vernig/plugin-twilio-infra/tree/templates.
In this repo I use two flags:

template which refers to a Twilio template scaffolding stored on https://github.com/vernig/plugin-twilio-infra/tree/templates/templates
pulumi-template: this is the pulumi project template (which in the original project was an arg not a flag)

In order to try, you can use:
twilio infra:new --template taskrouter-quick-start

Currently this is only a proposed implementation and the templating needs to be expanded and more robust (e.g. including multiple files and folders)
@kaiquelupo what do you think?

That's exactly the plan for the next steps  @vernig. As we discussed on the pull request, now we need to figure out the best way to create those template repos and decide if the pulumi tool for templating is good enough for our gols or if we should create our own. I really like the idea of templating not just in the sense of copying the code but also to keep people that copy the repo updated with the new fixes or features added to the template. Therefore, we need to assess the options here.

That's exactly the plan for the next steps  @vernig. As we discussed on the pull request, now we need to figure out the best way to create those template repos and decide if the pulumi tool for templating is good enough for our gols or if we should create our own. I really like the idea of templating not just in the sense of copying the code but also to keep people that copy the repo updated with the new fixes or features added to the template. Therefore, we need to assess the options here.

About the implementation you made, I wrote my opinion in here: #3

About the implementation you made, I wrote my opinion in here: #3

Haven't had a chance to test @vernig 's addition to this repo yet, but wanted to add my comment in Slack for posterity: My main concern is around using an existing repo as a project template which I think is closer to what @kaiquelupo is referring to with "I really like the idea of templating not just in the sense of copying the code but also to keep people that copy the repo updated with the new fixes or features added to the template."
I am currently dealing with an issue in which the Flex CLI auto-removes code that it deems syntactically incorrect without any warning; thereby breaking the plugin.

Haven't had a chance to test @vernig 's addition to this repo yet, but wanted to add my comment in Slack for posterity: My main concern is around using an existing repo as a project template which I think is closer to what @kaiquelupo is referring to with "I really like the idea of templating not just in the sense of copying the code but also to keep people that copy the repo updated with the new fixes or features added to the template."
I am currently dealing with an issue in which the Flex CLI auto-removes code that it deems syntactically incorrect without any warning; thereby breaking the plugin.

Thanks @bermudezmt @kaiquelupo for the feed-back!
Just to summarise, we have to decide which of the following two flows to use:
CLI flow

User install CLI
Using CLI a user initialise a local folder and a pulumi project (infra:new)
Optionally the user can specify a template, which in this case is a stab javascript file with an example implementation for the specific Twilio product (e.g. Task Router)

GitHub template flow

User create a new repo based on template
User works on this repo and made modification / customisation for its use case
If there is any change / improvement / bug fixes, user can quickly merge the new change

I feel the two are both valid and complementary use cases.
The "GitHub template" flow is more for complex product, and (if I understood correctly @kaiquelupo ) forces the user to base its implementation on the template repo, so it may be more difficult to integrate with existing solutions.
The "CLI flow" is more similar to the  Twilio Serverless CLI approach, where we provide simple scaffolding that are useful for getting something started (i.e. https://github.com/twilio-labs/function-templates for serverless). I agree we don't have to host all templates in the same repo, but I like the fact that all templates are grouped together (e.g. we can always create a monorepo?).
I feel the latter is a low hanging fruit that can make the use of the CLI and the pulumi provider easier to adopt, whereas the second is a medium / long term project that can definitely help more powerful user, but it takes more time to make it stable and robust.
So my proposal is to start working on a bunch of small templates to put out there, while we learn and improve the provider and the plugin CLI for a more ambitious series of product templates.
What do you think?

Thanks @bermudezmt @kaiquelupo for the feed-back!
Just to summarise, we have to decide which of the following two flows to use:
CLI flow

User install CLI
Using CLI a user initialise a local folder and a pulumi project (infra:new)
Optionally the user can specify a template, which in this case is a stab javascript file with an example implementation for the specific Twilio product (e.g. Task Router)

GitHub template flow

User create a new repo based on template
User works on this repo and made modification / customisation for its use case
If there is any change / improvement / bug fixes, user can quickly merge the new change

I feel the two are both valid and complementary use cases.
The "GitHub template" flow is more for complex product, and (if I understood correctly @kaiquelupo ) forces the user to base its implementation on the template repo, so it may be more difficult to integrate with existing solutions.
The "CLI flow" is more similar to the  Twilio Serverless CLI approach, where we provide simple scaffolding that are useful for getting something started (i.e. https://github.com/twilio-labs/function-templates for serverless). I agree we don't have to host all templates in the same repo, but I like the fact that all templates are grouped together (e.g. we can always create a monorepo?).
I feel the latter is a low hanging fruit that can make the use of the CLI and the pulumi provider easier to adopt, whereas the second is a medium / long term project that can definitely help more powerful user, but it takes more time to make it stable and robust.
So my proposal is to start working on a bunch of small templates to put out there, while we learn and improve the provider and the plugin CLI for a more ambitious series of product templates.
What do you think?

Let's remember to discuss this at tomorrow's sync.


🐛 Bug Report: Unable to register entities in software catalog that have URL longer than 255 chars

📜 Description
Registering a git repo with a Url longer than 255 chars results in the follow errors:
024-09-17T16:11:27.926Z backstage error Filtered internal error with logId=2ed5833c8aa18af867ba from response insert into "locations" ("id", "target", "type") values ($1, $2, $3) - value too long for type character varying(255) type=errorHandler logId=2ed5833c8aa18af867ba length=99 name=error severity=ERROR code=22001 detail=undefined hint=undefined position=undefined internalPosition=undefined internalQuery=undefined where=undefined schema=undefined table=undefined column=undefined dataType=undefined constraint=undefined file=varchar.c line=632 routine=varchar stack=error: insert into "locations" ("id", "target", "type") values ($1, $2, $3) - value too long for type character varying(255)

stack=error: insert into "refresh_keys" ("entity_id", "key") values ($1, $2) - value too long for type character varying(255)

👍 Expected behavior
The entity should be registered in the catalog.
👎 Actual Behavior with Screenshots
It results in the errors specified above.
Screenshot:

👟 Reproduction steps
Try to add a repo with a long Url (eg: https://gitlab.com/backstage-test-1/very-long-group-name-that-exceeds-255-characters-just-to-test-the-limits-of-url-length-in-the-catalog-info-yaml-file-and-see-how-the-backstage-system-handles-it-making-it-as-long-as-possible/test-1/-/blob/main/catalog-info.yaml) as an entity into the software catalog
📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

Yep we need to be smarter about huge refresh keys. See discussion in the PR.


Add Profunctor

https://github.com/rpominov/static-land/blob/master/docs/spec.md#profunctor

@gcanti insight on how you would encode partial type application in order to follow the spec?

Module must match the Profunctor signature for some type T, support Functor algebra for all types U created by setting the first parameter of T to an arbitrary concrete type (for example type U = T<number, a>)

Other than that, would that be a start (I'm hesitating between 'promap' or 'dimap')?
export interface StaticProfunctor<F extends HKT2S> {
  lmap<A, B, C>(fa: (a: A) => B, m: HKT2<B, C>[F]): HKT2<A, C>[F]
  rmap<B, C, D>(fb: (c: C) => D, m: HKT2<B, C>[F]): HKT2<B, D>[F]
  promap<A, B, C, D>(fa: (a: A) => B, fb: (c: C) => D, m: HKT2<B, C>[F]): HKT2<A, D>[F]
}

export interface FantasyProfunctor<F extends HKT2S, B, C> {
  lmap<A, B, C>(fa: (a: A) => B): HKT2<A, C>[F]
  rmap<B, C, D>(fb: (c: C) => D): HKT2<B, D>[F]
  promap<A, D>(fa: (a: A) => B, fb: (c: C) => D): HKT2<A, D>[F]
}

Not sure what interesting instances could be implemented.. (Arrows ?)

I'm hesitating between 'promap' or 'dimap'

Yeah, purescript uses dimap https://github.com/purescript/purescript-profunctor/blob/master/src/Data/Profunctor.purs#L23, I would stick with static-land though (as done so far), so promap
import { HKT2, HKT2S } from './HKT'
import { StaticFunctor, FantasyFunctor } from './Functor'
import { identity } from './function'

export interface StaticProfunctor<F extends HKT2S> extends StaticFunctor<F> {
  promap<A, B, C, D>(ab: (a: A) => B, cd: (c: C) => D, fbc: HKT2<B, C>[F]): HKT2<A, D>[F]
}

export interface FantasyProfunctor<F extends HKT2S, B, C> extends FantasyFunctor<F, C> {
  promap<A, D>(ab: (a: A) => B, cd: (c: C) => D): HKT2<A, D>[F]
}

lmap and rmap can be derived (like lift in Functor)
export function lmap<F extends HKT2S, A, B, C>(profunctor: StaticProfunctor<F>, f: (a: A) => B, fbc: HKT2<B, C>[F]): HKT2<A, C>[F] {
  return profunctor.promap<A, B, C, C>(f, identity, fbc)
}

// this is just the functor map
export function rmap<F extends HKT2S, A, B, C>(profunctor: StaticProfunctor<F>, f: (a: B) => C, fab: HKT2<A, B>[F]): HKT2<A, C>[F] {
  return profunctor.map(f, fab)
}

insight on how you would encode partial type application in order to follow the spec?

extends StaticFunctor<F> and  extends FantasyFunctor<F, C> should be enough I guess

Also, not sure what interesting instances should be implemented

Theoretically functions -> (though we need a wrapper in TypeScript, see below), profunctor optics, other examples here https://github.com/purescript/purescript-profunctor/tree/master/src/Data/Profunctor and here https://www.schoolofhaskell.com/school/to-infinity-and-beyond/pick-of-the-week/profunctors
Functions as profunctors
declare module './HKT' {
  interface HKT<A> {
    '->': Fun<any, A>
  }
  interface HKT2<A, B> {
    '->': Fun<A, B>
  }
}

export const URI = '->'

export type URI = typeof URI

export class Fun<B, A> implements FantasyProfunctor<URI, B, A> {
  readonly _B: B
  readonly _A: A
  readonly _URI: URI
  constructor(public readonly value: (b: B) => A) {}
  run(b: B): A {
    return this.value(b)
  }
  map<C>(f: (a: A) => C): Fun<B, C> {
    return this.promap<B, C>(identity, f)
  }
  promap<C, D>(cb: (c: C) => B, ad: (a: A) => D): Fun<C, D> {
    return new Fun<C, D>(c => ad(this.run(cb(c))))
  }
}


💡[FEATURE]: About Us Page

Create About Us Page Component

all components under /components/about

About Section

Why this project started or about this Project
ss

Contributor Section

Acknowledge and thank contributors who have contributed significantly to the project. You can list their names, GitHub handles, or any other relevant details.
ss

Rescue / Adopt Section

Explain how your project contributes to rescuing or addressing a specific problem or need.  Provide clear steps or calls to action such as:

How to contribute code (link to contribution guidelines)
How to report issues or suggest improvements (link to issue tracker)
How to join discussions or community channels (link to chat platforms or forums)
ss

I would like to work on this issue

@Sanjanah8 Choose an Component and start working on it

@Sanjanah8 Still on it? Should i assign it to you?

@cb7chaitanya , Yup
Assign Me

@Sanjanah8 Still on it? Should i assign it to you?

Yes still working

any update? @Sanjanah8

Hi there, i would like to work on the Our Team section


How do I get new airports added to the available list?

How do I get new airports added to the available list?

Unfortunately that list is from the FAA itself as the airports that the API supports. I have seen other airports pop up before on the other API endpoint that lists all airports with delays, but I'm not sure that those would then be supported at the endpoint that this integration uses.

You can actually try yourself, by using the url, and adding the airport code.
For example:
https://soa.smext.faa.gov/asws/api/airport/status/SWF
Works fine.

You are right, and this is something I've wondered. On the API reference here, they list that those are the only airports that return data. But if you call the API with an unsupported airport it just returns a "SupportedAirport":false and then works anyways. I'll probably release a point update to allow unsupported airports with an info in the logger that the airport isn't officially supported and then allow it to run anyways.

Support for unsupported airports (clear as mud, I know) has been added in v0.3.0. There are details in the release notes, but essentially the component will attempt to get data for anything passed into the ID field. If it does not get any data, it returns an error and does not setup the component. Otherwise, it sets it up and returns a warning if it is not an "officially supported" airport.


Update Vat rates

Add tax calculator class
Add vat number to clients
Validate vat numbers
Update tax calculation to correctly calculate inclusive tax

Coverage decreased (-0.004%) to 25.187% when pulling 6633565f568caad4c0020e2bb877f73670003238 on pierredup:vat-rates into 2f6de4494e84acf490f1403afd21d21c3898ee4b on CSBill:master.

Coverage decreased (-0.04%) to 25.146% when pulling cfff03dc181069263eae76f58f708085e5f5d1a7 on pierredup:vat-rates into 2f6de4494e84acf490f1403afd21d21c3898ee4b on CSBill:master.

Coverage decreased (-0.9%) to 24.336% when pulling c74868cbbf9cf6a9a2054a0fff4b8c060add0e2e on pierredup:vat-rates into 2f6de4494e84acf490f1403afd21d21c3898ee4b on CSBill:master.

Coverage decreased (-0.2%) to 24.985% when pulling c74868cbbf9cf6a9a2054a0fff4b8c060add0e2e on pierredup:vat-rates into 2f6de4494e84acf490f1403afd21d21c3898ee4b on CSBill:master.

Coverage decreased (-0.2%) to 24.96% when pulling 089b5835eec823c10891e738451e8f64425a79a7 on pierredup:vat-rates into 2f6de4494e84acf490f1403afd21d21c3898ee4b on CSBill:master.

Coverage decreased (-0.2%) to 24.96% when pulling 089b5835eec823c10891e738451e8f64425a79a7 on pierredup:vat-rates into 2f6de4494e84acf490f1403afd21d21c3898ee4b on CSBill:master.

Coverage decreased (-0.5%) to 24.705% when pulling 2a3865fbba05b5dff8d85be10ab6bd8ecbcaf454 on pierredup:vat-rates into 2f6de4494e84acf490f1403afd21d21c3898ee4b on CSBill:master.


Select modules at configure time

The entire library now requires OpenIGTLInk, VTK, Qt, CTK. It would be nice to be able to turn the GUI parts on/off in cmake, possibly others modules as well.
Components that could be optional:

Devices/Logic
GUI
Examples

As far as I can tell, dependency on is only there for GUI. It should remain like this. Once we implement build options, it will be possible to create a client without CTK by simply disabling GUI.

Yes, this applies to both Qt and CTK.


README: use svg badges, for retina friendlyness

Thanks!

FYI I also checked for the pledgie badge, but it seems they dont support SVG badges atm.


Fasta allows unsound abstract override

This is the same missing error message that DDC and the analyzer has.  See https://github.com/dart-lang/sdk/issues/30568 for details.

The check for missing implementations in https://dart-review.googlesource.com/c/sdk/+/43660 fails to detect this case, where a concrete method is overridden by (via a subclass or implemented interface) an abstract method with a more general signature (i.e. adding optional parameters).

Note that in this case, noSuchMethod forwarders are not generated, so this is an error whether or not the class has a noSuchMethod implementation different from the one in Object.

Partially fixed in 40864cc6985f2ea24169c7f832bc82a6d533f48a (only checks for difference in arity, missing checks for named parameters and types). The partial fix reuses the error message for missing members, which is misleading (especially the tip). The fix for the remaining checks should add one or more error messages to use for all mismatch situations.

Fixed in https://dart-review.googlesource.com/c/sdk/+/74642, but blocked on https://github.com/dart-lang/dartdoc/pull/1755 being rolled into the SDK deps.


Native executable missing after nativeCompile task

Expected Behavior
I'm upgrading project from Micronaut 3.1.1 to 3.7.4 (Gradle plugins from 2.0.6 to 3.6.5).
When I run ./gradlew nativeRun then the native executables are compiled and the compiled task is run.
Actual Behaviour
The application executable is not generated, only a header file.
A problem was found with the configuration of task ':pierrot:nativeRun' (type 'NativeRunTask'). |
  | - In plugin 'org.graalvm.buildtools.gradle.NativeImagePlugin$Inject' type 'org.graalvm.buildtools.gradle.tasks.NativeRunTask' property 'image' specifies file '/Users/ladin/development/projects/agorapulse/pierrot/apps/pierrot/build/native/nativeCompile/pierrot' which doesn't exist. |
  |   |
  | Reason: An input file was expected to be present but it doesn't exist. |
  |   |
  | Possible solutions: |
  | 1. Make sure the file exists before the task is called. |
  | 2. Make sure that the task which produces the file is declared as an input. |
  |   |
  | Please refer to https://docs.gradle.org/7.2/userguide/validation_problems.html#input_file_does_not_exist for more details about this problem. |

See the build scan for more details
https://scans.gradle.com/s/3kx2olyuyux4o/console-log?anchor=147&page=1
Steps To Reproduce

clone pull request from Pierrot repository https://github.com/agorapulse/pierrot/pull/58
install GraalVM sdk install java 22.3.r11-grl
run ./gradlew nativeRun

Environment Information

Operation system: Darwin aarch64
Java: OpenJDK 64-Bit Server VM Zulu11.60+19-CA (build 11.0.17+8-LTS, mixed mode)

Example Application
https://github.com/agorapulse/pierrot/pull/58
Version
3.7.4

It appears that you are using a quite old version of Gradle (7.2), you may consider upgrading.
The issue is that between the 2 versions of the plugin, we upgraded the Native Build Tools plugin, which now supports building libraries (.so) in addition to applications. However, if you look at your project, you can see that it uses the java-library plugin:
https://scans.gradle.com/s/cxjb7g2grtccy/plugins?toggled=WyJqYXZhLWxpYnJhcnktMTMiXQ#java-library::pierrot-13-0
This is a mistake, since an application doesn't expose an API, it must not both apply the application and java-library plugins (you are either an application or a library, but not both at the same time).
A workaround is to configure explicitly that you don't want to build a shared library:
graalvmNative.binaries.all {
    sharedLibrary = false
}

I am not familiar with how your build is setup (with the Kordamp plugins) so you'd know better than me where to put that configuration.

Thanks! That removing java-library from the CLI project helped. I'm still getting some other errors but I guess I need to migrate to new way of compiling the binaries and stop using Docker builds.

What do you mean by "new way of compiling the binaries"? You should call nativeCompile to build a native binary, and dockerBuildNative to build a native image within a docker container.

the build is using the old approach - calling native-image directly from the command line using the generated Shadow JAR.
https://github.com/agorapulse/pierrot/blob/master/.github/workflows/graalvm-snapshot.yml#L38-L49
BTW I haven't seen anything like distNativeZip or distNativeTar tasks in the tasks list. is the distribution still as easy as grabbing the generated command file and archiving it myself or do I also need any of the header files?

Creating a custom distribution with the native binary should only be about adding:
distributions {
    register("native") {
        contents {
           from(tasks.named("nativeCompile"))
        }
    }
}

This works great, thanks. I need a last piece of puzzle - I need to label the archives based on the current operating system to be able to upload them using SDKMAN Vendor API but I can't find out how to reach org.gradle.nativeplatform.platform.OperatingSystem instance to do so.

There is no public API in Gradle to access the operating system (and it's a shame). You can use org.gradle.internal.os.OperatingSystem#current, but as the name implies, it's internal.

thanks works well!


xmake getter

Please see docs.codecov.io/docs/about-the-codecov-bash-uploader. Codecov simply uses one command to upload coverage data:
bash <(curl -s https://codecov.io/bash)

So I'm wondering if it would be better to add a oneline xmake getter. I know up to now we could use ubuntu ppa or brew to simply install xmake, but they're not newest version from head of branch. For developing xmake and new feature testing, it's meaningful to add this especially using it on CIs.
I have written get.sh & get.ps1 to do so. After merging, one could simply use this:
bash <(curl -s https://raw.githubusercontent.com/tboox/xmake/master/scripts/get.sh)

or
Invoke-Webrequest 'https://raw.githubusercontent.com/tboox/xmake/master/scripts/get.ps1' -OutFile get.ps1
. .\get.ps1

(the powershell version does not fetch the newest snapshot because I'm afraid the popularity of git on Windows)

Thanks! 👍
By the way, you can run brew install xmake --HEAD to install newest version on macOS.

Oh, seems that I need to know more about brew
I think brew install xmake --HEAD must be able on linux as well
Seems I did redundant work

Although we can use linuxbrew on linux, but your script is more convenient.  👍
Thanks!

Now I think my script is kind of naive that the get.sh expect git & build-essential to work but do not even check them. I will work on it in future.

Great! 👍

I found a simpler oneline way to run getter on Windows:
Invoke-Expression (Invoke-Webrequest 'https://raw.githubusercontent.com/tboox/xmake/dev/scripts/get.ps1' -UseBasicParsing).Content

I attempt to run this script in powershell on win10, It seems to have some problem:
PS C:\Program Files\PowerShell\6.0.0-alpha.18> Invoke-Expression (Invoke-Webrequest 'https://raw.githubusercontent.com/t
boox/xmake/dev/scripts/get.ps1' -UseBasicParsing).Content
Invoke-Webrequest : Operation timeout
At line:7 char:1
+ Invoke-Webrequest "https://github.com/tboox/xmake/releases/download/$ ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (Method: GET, Re...rShell/6.0.0
}:HttpRequestMessage) [Invoke-WebRequest], HttpRequestException
    + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeWebRequestCommand

I found Operation timeout in output
github uses amazon aws s3 to store release files which might have poor connection in China

Here is a new in-progress get.sh
I added:

auto tools checking and installing
auto path detection

#!/bin/bash

# xmake getter
# usage: bash <(curl -s <my location>) [branch]

if [ 0 -ne $(id -u) ]
then
    sudoprefix=sudo
else
    sudoprefix=
fi
test_tools()
{
    {
        git --version &&
        make --version &&
        {
            cc --version ||
            gcc --version ||
            clang --version
        }
    } >/dev/null 2>&1
}
install_tools()
{
    { apt-get --version >/dev/null 2>&1 && $sudoprefix apt-get install -y git build-essential; } ||
    { yum --version >/dev/null 2>&1 && $sudoprefix yum install -y git 'Development Tools'; } ||
    { zypper --version >/dev/null 2>&1 && $sudoprefix zypper install -n git && $sudoprefix zypper install -n -t pattern devel_C_C++; } ||
    { pacman -V >/dev/null 2>&1 && $sudoprefix pacman -S --noconfirm git base-devel; }
}
test_tools || { install_tools && test_tools; } ||
{
    rv=$?
    echo 'Dependencies Installation Fail'
    exit $rv
}
branch=
if [ x != x$1 ];then branch="-b $1";fi
git clone --depth=1 $branch https://github.com/tboox/xmake.git /tmp/$$xmake_getter || exit $?
make -C /tmp/$$xmake_getter --no-print-directory build || exit $?
IFS=':'
patharr=($PATH)
prefix=
for st in ${patharr[@]}
do
    if [[ "$st" = "$HOME"* ]]
    then
        cwd=$(pwd)
        mkdir -p "$st"
        cd "$st"/..
        mkdir -p share 2>/dev/null || continue
        prefix=$(pwd)
        cd "$cwd"
        break
    fi
done
if [ x$prefix != x ]
then
    make -C /tmp/$$xmake_getter --no-print-directory install prefix="$prefix"|| exit $?
else
    $sudoprefix make -C /tmp/$$xmake_getter --no-print-directory install || exit $?
fi
rm -rf /tmp/$$xmake_getter
xmake --version

Support these package manager to install build essential

[x] apt-get
[x] yum
[x] zypper
[x] pacman
[ ] More? Could you help me to think up? @waruqi

But some of them are untested because I could not have all these distributions :smiley:

Great 👍  @TitanSnow homebrew or macports?

I opened a repo to test getter in dockers on travis-ci

Now get.ps1 also supports getting newer version from github!
I'm shocked that powershell is cross-platform :scream: I could use powershell on my ubuntu :cool:

@TitanSnow Cool 👍


Home work 3

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Written queries to aggregate, temp table creation, retrieving specific data from the database based on requirements using SELECT statement, INSERT records etc
What did you learn from the changes you have made?
Learned various operations that can be performed in SQL - arithmetic operations, inserting values, retrieving data from tables, usage of different clauses like HAVING, GROUP BY, ORDER BY, aggregate functions etc
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist

[x] I can confirm that my changes are working as intended

You have unfortunately made a pull request to the wrong repo. You shouldn't be making a pull request to uoft-dsi/sql, but rather your own forked sql repo. Please read the instructions on your onboarding Assignment Submission Guide more carefully.


Autocomplete (not only the one Hydrogen provides) stops while the kernel is busy

Thank you for maintaining this cool package! I love to use this, but there is a thing I regret.
It is that autocomplete (not only the one Hydrogen provides) stops while Hydrogen is evaluating codes(at least in my environment, Windows 10 + Atom 1.12.7 + Hydrogen 1.5.0.) As below:

I often run codes which take pretty evaluation time (several minutes or longer) and I want to work on other parts using autocomplete at the same time.
I know we can avoid this behavior by disabling the Hydrogen's autocomplete in advance, but I think it's convenient we can use it when its kernel is idle and use completions provided by some other packages even when the kernel is busy.

We definitely shouldn't block other completion providers. I'll submit a PR later today.
@rgbkrk If the kernel is busy we generally cannot receive any completions from the jupyter kernel, is that right?

@lgeiger In general you are correct, but there are exceptions, e.g. IJavascript allows execution requests to be asynchronous.

Closed by #567

If the kernel is busy we generally cannot receive any completions from the jupyter kernel, is that right?

To expand on what @n-riesco said, the Python kernel will not send completions when it is busy. Kernels can send completion information over ZMQ while the kernel is busy, regardless of if the frontend will handle it -- jupyter notebook avoids asking for completion when the kernel is busy.

Thank you for rapid fix!


Fix #47

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://guh.guru:8080/job/guh-webinterface-pull-request/37/
Test PASSed.


Add umbrellaID in addition to ORCID ?

ORCID does not want to be an IdP. Therefore orcid useful for finding authors of applications, but not for login

This may be changing... ;-)


TypeError running README.md example via ghci

Template haskell isn't in my bag of tricks yet so this could just be something I'm overlooking but, regardless, I thought I'd report-back.
Need to keep banging out code so I don't have time to comment further right now but if I find a moment to revisit I'll update with any additional information that I come up with. A preemptive thank you for your work and efforts here!
-dituri
$ ghci
GHCi, version 7.8.3: http://www.haskell.org/ghc/  :? for help
Loading package ghc-prim ... linking ... done.
Loading package integer-gmp ... linking ... done.
Loading package base ... linking ... done.
ghci> import System.Random
ghci> import System.Random.TH
ghci> :set -XTemplateHaskell
ghci> data Color = Red | Green | Blue deriving (Enum, Bounded)
ghci> $(deriveRandom ''Color)

<interactive>:6:3:
    Couldn't match type `[Language.Haskell.TH.Syntax.Dec]'
                  with `Language.Haskell.TH.Syntax.Exp'
    Expected type: Language.Haskell.TH.Lib.ExpQ
      Actual type: Language.Haskell.TH.Syntax.Q
                     [Language.Haskell.TH.Syntax.Dec]
    In the expression: deriveRandom ''Color
    In the splice: $(deriveRandom ''Color)
ghci>

Note: here's my current module listing  for your reference:
C:\Some\User\Path\Goes\Here\Yay >ghc-pkg list | egrep -i " (random|template-haskell|base)-"
    base-<IP_ADDRESS>
    template-haskell-<IP_ADDRESS>
    random-1.1
    random-derive-<IP_ADDRESS>

I think you're doing everything correctly, but yet there is no issue here: as far as I know, you cannot run Template Haskell "splices" in ghci which results in multiple toplevel declarations (such as new instances being declared) like that: the same issue occurs if you try to use e.g. the deriveJSON function from Data.Aeson.TH. You can, however, print (or pretty-print) splices and show them which gets you the (somewhat) verbose code which is generated:
Frerichs-MBP:tmp frerich$ ghci
GHCi, version 7.8.3: http://www.haskell.org/ghc/  :? for help
Loading package ghc-prim ... linking ... done.
Loading package integer-gmp ... linking ... done.
Loading package base ... linking ... done.
λ: :set -XTemplateHaskell
λ: import System.Random
λ: import System.Random.TH
λ: import Language.Haskell.TH
λ: data Color = Red | Green | Blue deriving (Enum, Bounded)
λ: putStrLn $(deriveRandom ''Color >>= stringE . pprint)
[..]
instance System.Random.Random Ghci1.Color
    where System.Random.randomR (lo_0,
                                 hi_1) g_2 = (GHC.Enum.toEnum a_3, g'_4)
                                    where (a_3,
                                           g'_4) = System.Random.randomR (GHC.Enum.fromEnum lo_0,
                                                                          GHC.Enum.fromEnum hi_1) g_2
          System.Random.random = System.Random.randomR (GHC.Enum.minBound,
                                                        GHC.Enum.maxBound)

If you paste the code you wrote into a file and then load that into ghci, you'll see that it loads just fine and you can use the randomColors function to get an infinite stream of random colors.

@frerich thanks for following up - you're absolutely correct ... it seems ghci cannot cope with multiple top-level declarations generated from Template Haskell. Nothing to see here: issue closed and rightfully so! :-)


"Print" btn doesn't work in Main menu

System information:

Operating system (distribution) and version Win 10
DBeaver version 6.3.2

Describe the problem you're observing:
"Print" btn doesn't work for Data Editor
Steps to reproduce, if exist:
1 A table data editor is opened and active
2. Press File> Print
Actual result:
Print dialog window doesn't appear.

https://dbeaver.testquality.com defect D49. Do not remove or edit.


DR - Date picker: months getting stacked when arrow keys are clicked

Describe the bug
When making an edit (minor, legal name, or repeal), when the date picker is opened, clicking the arrow (< or >) keys causes months to become stacked on top of each other.
To Reproduce
Steps to reproduce the behavior:

Go to any site
Click on Edit and select Minor edit, Legal name, or Repeal
Open the date picker
Click the forward or back arrow keys and notice the month panels stack up

Expected behavior
Month should go forward or backward with only one calendar panel visible.
https://github.com/bcgov/parks-data-register/assets/65189530/1ce2cb70-81eb-44c8-9bf0-83db53bf8568

From #453:

Not a datepicker issue so much as a lazy loading issue. The datepicker rendering changes but the old rendering is not removed. Patched in a similar way to protected areas - detectChanges wrapped in a 0s timeout.


fix: Use netifaces instead of socket for get_local_ip

fixes #99
Currently, netifaces and requests are not in our install requirements. Should they be, now that we are importing them ourselves?

Alright, I'm back home, and it turns out I'm not getting the same error as #99. Since this fixes that, let's merge it. Since we do use netifaces and requests, they should be in our requirements, yes. Do you want to make the change?

Done.

Thank you!


Create eisti.txt

See https://eisti.fr/ for the website.
This is a french engineer school !

@hn6pot I am sorry for the delay with processing your request. Pull request merged. Thank you!


can not reproduction acc with lfw dataset

I try to reproduction acc with lfw dataset， but error occur.
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/PIL/Image.py", line 2772, in fromarray
mode, rawmode = _fromarray_typemap[typekey]
KeyError: ((1, 1, 3), '<f4')
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "deepface_test.py", line 33, in
obj = DeepFace.verify(img1, img2, model_name = 'Dlib', model = dlib_model, enforce_detection=False, distance_metric = 'euclidean')
File "/usr/local/lib/python3.6/dist-packages/deepface/DeepFace.py", line 152, in verify
, detector_backend = detector_backend)
File "/usr/local/lib/python3.6/dist-packages/deepface/commons/functions.py", line 454, in preprocess_face
img = align_face(img = img, detector_backend = detector_backend)
File "/usr/local/lib/python3.6/dist-packages/deepface/commons/functions.py", line 437, in align_face
img = alignment_procedure(img, left_eye, right_eye)
File "/usr/local/lib/python3.6/dist-packages/deepface/commons/functions.py", line 360, in alignment_procedure
img = Image.fromarray(img)
File "/usr/local/lib/python3.6/dist-packages/PIL/Image.py", line 2774, in fromarray
raise TypeError("Cannot handle this data type: %s, %s" % typekey) from e
TypeError: Cannot handle this data type: (1, 1, 3), <f4
What shoud I do？

what exactly image pairs in your experiment? could you share your whole code?

what exactly image pairs in your experiment? could you share your whole code?
you got same error when you test it for another model, e.g. Facenet?

I copy the code from here(https://github.com/serengil/tensorflow-101/blob/master/python/LFW.ipynb), and use Facenet got same error.

it seems mtcnn causes an error. You can skip the error if you set the detector backend to different to mtcnn.
DeepFace.verify(img1, img2, detector_backend = 'opencv', enforce_detection = False)
I will study why mtcnn causes this error

yes, mtcnn find facial area in a scale of 1, 1, 3 that's not true. It seems mtcnn related error. The best way is using different detector.


Feature Request: Master Toggle

Since this project has the difficult task of chasing any official changes, there will always be some doubt about whether some buggy behavior originates with this extension or with the app (which is, to date, still very buggy).
While we can just disable the extension in our browser, it would be neat to have a master switch in the extension to quickly toggle off -> reload -> check if bug keeps happening.

Fair enough, I like that idea.

Me too :-).  I came here looking for a way to toggle this VERY useful extension.

Alrighty, I've gone ahead and added this to the Roadmap: https://trello.com/c/GuKHuuTR/197-master-toggle-turn-all-features-on-and-off-with-a-single-switch-in-the-options-page
I'll go ahead and close this issue so conversation on it can happen with the rest of the feature requests. Thanks again!


Issue-9: Added shields and LICENSE

Signed-off-by: Joe Pearson<EMAIL_ADDRESS>In addition to adding the shields, I also had to add the Apache 2.0 license file.

@johnwalicki or @t-fine please review this PR.  I had to add the missing Apache 2.0 license file before the license shields would work.


Is R-FCN code included in this repository?

The README shows the performance on VOC2007 when R-FCN is adopted. But I have not found the related code in this repo.

NO, R-FCN training schedue is adopted, they use 80k/110k training in the single gpu mode, which I guess is need to converge the model on the larger dataset on 07+12.


Rduncan rebus

Associated Issue: #121
Summary of Changes

added new Rebus

Pull Request Test Coverage Report for Build 394

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 86.617%

Totals

Change from base Build 393:
0.0%

Covered Lines:
154

Relevant Lines:
175

💛  - Coveralls

I have run the test locally and they all pass. I am not super familiar with deciphering Travis errors.


Remove token types String, Comment and Regex from line before determining if there should be indentation or not

It has been noticed that the regex indentation patterns try to account for comments and strings containing braces that could mislead the regex pattern in determining whether indentation should happen. This PR aims to strip the analyzed string from its comments, strings and regexes before applying the indentation rules. This also simplifies the indentation regex patterns.

Apologies, I forgot to mark this PR as a draft. It contains unfinished work.


Migration Guide link returns 404

I was looking to use angular-schema-form in a brand new Angular 4 app and was hoping it was possible.

@thardy you would still need the Angular upgrade module but it can work and I am aware of people doing it.

Thanks for letting me know about the link :)


Add teacher emails to various places courses are displayed

Depends on:

https://github.com/hypothesis/report/issues/118

There is a strong desire to show teacher emails in the interface against courses.
We can't say how we'll do this until we have completed the spike, but we can say where we would like this to appear and how.
Questions

We must use separate arguments for region and id to make the query efficiently. How can we marry this with a single input value required to configure filters in dashboards? Split it with SQL?

Data required
user_sensitive

We need a non-materialized view for things like user names and emails
We need to start capturing this data in the first place, rather than relying on old tables

Tasks

https://github.com/hypothesis/lms/pull/4835
https://github.com/hypothesis/report/pull/153
https://github.com/hypothesis/report/pull/157

This is deployed out and done now. I think we're going to wait for feedback at a later date.


status”:500,“error”:“Internal Server Error”,“exception”:“com.netflix.zuul.exception.ZuulException”,“message”:“REJECTED_THREAD_EXECUTION”}

{"timestamp":1531026130373,"status":500,"error":"Internal Server Error","exception":"com.netflix.zuul.exception.ZuulException","message":"REJECTED_THREAD_EXECUTION"}
I am getting above exception while running stress testing on my one of the API redirecting from zull gateway.
Here is some of the more points which I want to mentioned here.
I am using netflix zull and eureka for discovery my microservices (spring boot).
Everything is working fine from my angular application, postman and swagger.
Now I am trying to do stress testing using Jmeter tool which is working fine for <=10 no. of thread usersload. But as soon as I am mentioning more than 10 no. of thread users. It is giving me above mentioned exception.
I also tried to test my API directly (without gateway -zull) and that is also working fine in case of more than 10+ no. of thread users and that is working fine too.
I tried to replicate same scenario on my local laptop and used the same jmeter. And this time rather giving the server host name I try to test with my laptop hostname (all code and services are same). I mentioned 10+ no. of thread users in jmeter and that is working fine with no above exception.
I read lot of stack-overflow posts, github links and other web search for this issue but none of solution seems to working. I tried and some other
ribbon: ReadTimeout: 60000 ConnectTimeout: 60000
hystrix: command: default: execution: isolation: strategy: THREAD thread: timeoutInMilliseconds: 60000

HI @rohitdec01
Could you please change hystrix.threadpool.default.coreSize the number, because in my mind it default value is 10.
In addition i notice you have those parameter
ribbon. ReadTimeout: 60000
ribbon.ConnectTimeout: 60000
hystrix: command: default: execution: isolation: strategy: THREAD thread: timeoutInMilliseconds: 60000

Could you please change it like this
ribbon. ReadTimeout: 60000
ribbon.ConnectTimeout: 60000
hystrix: command: default: execution: isolation: strategy: THREAD thread: timeoutInMilliseconds: 240000

In my mind Hystrix timeout should include all retry time(include retry same server and next server), because ribbon retry same server default is 0 and retry next server default is 1.
Hope it can help you
thanks

thanks for your quick reply. I already tried below properties as well but getting some other Zull exception after this. (mentioned below)
ribbon:
ReadTimeout: 60000
ConnectTimeout: 60000
hystrix:
threadpool:
default:
coreSize: 1000
maximumSize: 2000
maxQueueSize: -1
command:
default:
execution:
isolation:
timeout:
enabled: false
strategy: THREAD
thread:
timeoutInMilliseconds: 240000
{"timestamp":1531112495424,"status":500,"error":"Internal Server Error","exception":"com.netflix.zuul.exception.ZuulException","message":"SHORTCIRCUIT"}

No, still the same issue. Attached is the screenshot.

My manager told me to remove zull and do the load testing untill this issue get fixed. So, I will come back in a day or 2 on this.
I will share the entire exception.

Any update here

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.

i also got same exception

I also got the same exception

I also got the same Exception


use_slice_data in slice_json calls (#5643)

(cherry picked from commit 492ad6c49ac965c2956c46bae13aefbef6b870d8)
@john-bodley @graceguo-supercat

Codecov Report

Merging #93 into airbnb-master will not change coverage.
The diff coverage is 0%.

@@              Coverage Diff               @@
##           airbnb-master      #93   +/-   ##
==============================================
  Coverage          63.25%   63.25%
==============================================
  Files                351      351
  Lines              22034    22034
  Branches            2468     2468
==============================================
  Hits               13937    13937
  Misses              8085     8085
  Partials              12       12

Impacted Files
Coverage Δ

superset/views/core.py
73.69% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Env var interpolation in name and namespace

Hi,
since it was done in #36 for repositories: Would it be possible to extend the interpolation to name and namespaces for automated deployments?
For namespace it looks like it can be easily extended in flagsForRelease(). For name it's currently scattered between SyncReleases and DiffReleases.. but generally just wondering: Any reason why it's not (yet) possible / why it should not be possible for those two fields?

Added a PR to illustrate what I was trying to say/ask.

Could this be extended to other (or all?) fields? I'm currently using YAML anchors and references to deploy the same chart with multiple configurations, e.g.
RELEASE_COMMON: &RELEASE_COMMON
  chart: jupyterhub/jupyterhub
  version: v0.7-4104389

releases:

  - <<: *RELEASE_COMMON
    name: jupyterhub-1
    namespace: jupyterhub-1
    ...
  - <<: *RELEASE_COMMON
    name: jupyterhub-2
    namespace: jupyterhub-2
    ...

But interpolating chart and version would be more readable.

Yes. Which fields? Chart/version?

Those are the two I'm interested in, but for consistency perhaps it's easier to say which fields shouldn't be interpolated?

@manics @danielcb I rather want to start by limiting helmfile's dynamism to be necessary and sufficient for intended use-cases, so that hopefully helmfile.yml doesn't get mess.
So, would you mind sharing us your exact use-cases of wanting templates everywhere for further discussion? Thx!

Anyway, closing this as the original issue is resolved via #90 and #87. Please feel free to submit separate feature requests for another places you want templating!

Somehow I think using YAML anchors is not that bad. Just using the capabilities of the format.
But I had a similar request in #59, using a defaults section. How do you think these two compare?


Support int and uint as a valid data type for humanize functions

Related discussions at the end of https://github.com/prometheus/prometheus/issues/8303
Proposal
Support infered int in yaml templating functions such as humanize, humanize1024, humanizeDuration, etc.
Go templating treats round numbers as int in YAML. When trying to use the humanize functions, it fails because it only support float64 or string. So, a template like this will fail :
      expr: some_expression
      annotations:
        description: You are approaching the maximum value of {{ 123456789 | humanize}}
        summary: Some summary

int should be supported out of the box without having to put 123456789.0.
I'll follow up with a PR shortly.
Thanks

To add some historical context here the templates purposefully and exclusively deal with float64 as Go won't let you do 2 + 2.0 so I figured it was easiest to stick to one number type.

mmmh; I will run a few tests; we might indeed want to reconsider this. Thanks @brian-brazil  !

I'm not sure I understand the issue here, are you talking about reverting the string conversion as well? To me, this version is just more robust as inputs gets converted to floats if possible and then using a common type for the rest of the conversions.
But, like I said, I might not get the full picture of @brian-brazil's comment.

To add some historical context here the templates purposefully and exclusively deal with float64 as Go won't let you do 2 + 2.0 so I figured it was easiest to stick to one number type.

We do not really choose what we get from go templating here, we are not converting to int but from int.
I have nothing against the attached pull request.
Note that a workaround is description: You are approaching the maximum value of {{ "123456789" | humanize}}

In case somebody ends up here with google, the workaround we used using terraform is hideous but was necessary since we weren't using prom with a recent enough version to get the string conversion :
description: threshold = {{ ${format("%.2f", config.some_value)} | humanize}}.


Add link to upstream repo in quick-repo-deletion

After deleting a repo with quick-repo-deletion, only a header bar remains:

If the deleted repo was a fork, I often want to go to the upstream repo afterwards. So I suggest to add a link to that in the header bar:

Example URL
https://github.com/FloEdelmann/streetcomplete-mapstyle (doesn't really make sense, since it is deleted now), forked from https://github.com/streetcomplete/streetcomplete-mapstyle

Leaving a small note: both the deleted repository name and upstream name would look better in bold.


fixed invisible text in light mode

Fixes #61
The text for a user's projects is visible now on light mode.
Screenshot:

I also removed the js code for project.html from script.js because it was already present in script tag of project.html. There were also a few typos in the class names. "projectName" was written as "projectNam" in few profiles. I fixed that as well.

Fixes #61
The text for a user's projects is visible now on light mode.
Screenshot:
I also removed the js code for project.html from script.js because it was already present in script tag of project.html. There were also a few typos in the class names. "projectName" was written as "projectNam" in few profiles. I fixed that as well.

your branch has conflict
Resolve conflicts and follow the account


Update rulesets in templates

Point to the new rulesets

No changes needing a change description found.

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/905/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/905/


Running command with no output fails to parse

I'm trying to delete an RMS stack, but the successful response from the CLI is empty, and the runner doesn't seem to like that.
Executing Oracle Cloud Infrastructure CLI command
| [command]/usr/local/bin/oci resource-manager stack delete --force --stack-id ocid1.ormstack.oc1.***.<real_stack_id>
|
| /run/act/actions/oracle-actions-run-oci-cli-command@v1.3.1/src/main.ts:67
|       const stdout = JSON.parse(cliResult.stdout)
| ^
| SyntaxError: Unexpected end of JSON input
|     at JSON.parse (<anonymous>)
|     at runOciCliCommand (/run/act/actions/oracle-actions-run-oci-cli-command@v1.3.1/src/main.ts:67:1)
[Test/test]   ❌  Failure - Main Destroy RMS Stack
[Test/test] exitcode '1': failure
[Test/test] 🏁  Job failed

I've tried to force output by using the --wait-for-state flag, but the runner doesn't seem to like this output either.
| Executing Oracle Cloud Infrastructure CLI command
| [command]/usr/local/bin/oci resource-manager stack delete --force --wait-for-state DELETED --stack-id ocid1.ormstack.oc1.***.<real_stack_id>
| Action completed. Waiting until the resource has entered state: ('DELETED',)
|
| /run/act/actions/oracle-actions-run-oci-cli-command@v1.3.1/src/main.ts:67
|       const stdout = JSON.parse(cliResult.stdout)
| ^
| SyntaxError: Unexpected end of JSON input
|     at JSON.parse (<anonymous>)
|     at runOciCliCommand (/run/act/actions/oracle-actions-run-oci-cli-command@v1.3.1/src/main.ts:67:1)
|     at processTicksAndRejections (node:internal/process/task_queues:95:5)
[Test/test]   ❌  Failure - Main Destroy RMS Stack
[Test/test] exitcode '1': failure
[Test/test] 🏁  Job failed

The only way I've found to get the runner to not fail with this command is to set the --debug flag, which somehow the runner has no problem parsing. However, this is not an acceptable solution since it exposes some internals.
This seems similar to an old issue - https://github.com/oracle-actions/run-oci-cli-command/issues/18

Except I added a check to see if the output is JSON or not, which it seems to not be working for this command. I think the simplest option here is to provide an flag that indicates there is no expected response and to mark the action as successful based on exit code.

Can you please test with v1.3.2 and let me know if this works for you? Open a new issue if it doesn't. Thanks!

Can you please test with v1.3.2 and let me know if this works for you? Open a new issue if it doesn't. Thanks!

Tested with v.1.3.2 and the RMS stack delete step completed without issue. Confirmed stack was deleted on console.

 I'm glad it worked.


Add search query prop to Table component

@vinnybod sorry for the delay on this, week has been pretty hectic. I plan on looking it over tomorrow morning.

@xaksis I rebased on your new changes

@vinnybod thanks again! there will be a few more changes coming in. I'm just adding the fixes that don't require a major version upgrade before we decide to merge this pull request.

Alright I will wait on that then :+1:


ROX-24767: Ignore function call if the function is nil

Description

Sensor integration tests create k8s resources to test sensor's behavior. This resources are deleted after the test execution most of the times with defered calls to remove functions. If the resource creation fails the remove functions will be nil. The defered calls to these remove functions need to check whether the remove function is nil or not, otherwise the test will panic.
This PR introduces three changes:

Modifies the IgnoreError function to check whether the passed function is nil or not.
Changes the use of t.Fatal in some functions of the integration tests helper to avoid the subtest calling FailNow and therefore failing the parent tests.
The defered calls to remove functions should be done after require.NoError. If err != nil the resource was not created so it's not necessary to call the remove function. Also, if err != nil the remove function will be nil (which is handled by (1) but still we can save one call to IgnoreError).

User-facing documentation

[ ] CHANGELOG is updated OR update is not needed
[ ] documentation PR is created and is linked above OR is not needed

Testing and quality

[ ] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change

[ ] The integration tests should pass.

@vikin91 I've updated the description with your suggestions 🙂

/test gke-sensor-integration-tests
☝ Test_SensorIntermediateRuntimeEvents failed for reasons unrelated to these changes.